Need an example for when data changes or file now appears in a watched directory and that triggers a job on the network against that data

"data event driven execution"
https://www.youtube.com/live/AO8oJUijINU?feature=share&t=1629
https://www.youtube.com/live/AO8oJUijINU?feature=share&t=2038

Might watching for data also include data published as the result of a job?  Perhaps we could trigger more work based on things like the tag attached to the job (not-a-real-thing-atm), or a specific user's running a job, or a specific docker image publishing data, or a mix of the above?  I'm sure we could achieve something similar with airflow but it feels a bit more dynamic this way.


Web UI does not start

Web UI is not starting. When clicking at Congatudo option in sidebar menu nothing is displayed.
Valetudo map card work fine, showing the map and I'm able to control the robot: clean, return to dock, locate, etc.
Robot model: Conga 3690
Logs at init:
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
[20:22:23] INFO: Using configuration from '/config/valetudo.json'...
[2023-11-08T19:22:49.613Z] [INFO] Loading configuration file: /config/valetudo.json
[2023-11-08T19:22:52.499Z] [INFO] Set Logfile to /tmp/valetudo.log
[2023-11-08T19:22:52.635Z] [INFO] Starting Valetudo 2023.05
[2023-11-08T19:22:52.647Z] [INFO] Commit ID: 093d853701a06a4d07aa494c7b84650cb9ea2f3e
[2023-11-08T19:22:52.651Z] [INFO] Configuration file: /config/valetudo.json
[2023-11-08T19:22:52.659Z] [INFO] Logfile: /tmp/valetudo.log
[2023-11-08T19:22:52.662Z] [INFO] Robot: Cecotec Conga (CecotecCongaRobot)
[2023-11-08T19:22:52.663Z] [INFO] JS Runtime Version: v18.12.1-Valetudo
[2023-11-08T19:22:52.665Z] [INFO] Arch: arm64
[2023-11-08T19:22:52.684Z] [INFO] Max Heap Size: 64 MiB
[2023-11-08T19:22:52.688Z] [INFO] Node Flags: --expose-gc --max-heap-size=64
[2023-11-08T19:22:52.747Z] [INFO] Autogenerated System ID: LawfulInsecureChicken
[2023-11-08T19:23:13.029Z] [INFO] Webserver running on port 8080
[2023-11-08T19:23:14.429Z] [INFO] Added new robot with id '12545'
[2023-11-08T19:23:15.616Z] [INFO] Connected successfully to MQTT broker
[2023-11-08T19:23:17.918Z] [INFO] MQTT configured

When stopping this is the log. There is an ERR_SERVER_NOT_RUNNING. Tried to set a different port in Configuration tab for the web UI, but it still says 8080 at startup
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped
[2023-11-08T19:22:07.413Z] [INFO] Valetudo shutdown in progress...
[2023-11-08T19:22:07.562Z] [INFO] Valetudo shutdown in progress...
[2023-11-08T19:22:07.565Z] [INFO] Valetudo shutdown in progress...
[2023-11-08T19:22:07.688Z] [INFO] Disconnecting from the MQTT Broker...
[2023-11-08T19:22:07.708Z] [INFO] Disconnecting from the MQTT Broker...
[2023-11-08T19:22:07.720Z] [INFO] Disconnecting from the MQTT Broker...
[2023-11-08T19:22:07.835Z] [INFO] Successfully disconnected from the MQTT Broker
[2023-11-08T19:22:07.836Z] [INFO] Successfully disconnected from the MQTT Broker
[2023-11-08T19:22:07.837Z] [INFO] Successfully disconnected from the MQTT Broker
[2023-11-08T19:22:07.902Z] [ERROR] Error occured:  Error  -  Server is not running.
[2023-11-08T19:22:07.937Z] [ERROR] Error [ERR_SERVER_NOT_RUNNING]: Server is not running.
    at new NodeError (node:internal/errors:393:5)
    at Server.close (node:net:1827:12)
    at Object.onceWrapper (node:events:627:28)
    at Server.emit (node:events:525:35)
    at emitCloseNT (node:net:1880:8)
    at process.processTicksAndRejections (node:internal/process/task_queues:81:21)
[2023-11-08T19:22:07.942Z] [ERROR] Stacktrace that lead to the process exiting with code 1: Error
    at process.<anonymous> (/snapshot/app/backend/index.js:58:93)
    at process.emit (node:events:513:28)
    at process.exit (node:internal/process/per_thread:190:15)
    at process.shutdown (/snapshot/app/backend/index.js:33:17)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

some hours later the issue resolved itself. Web UI starts fine.


Logfmt parser for parsing out additional fields from record msg field

I implemented a parser by extending the k8Json parser so that we could parse out additional fields  from the msg field of each record. This assumes that the msg is in logfmt format(about logfmt: https://brandur.org/logfmt). The fields we want to parse out need to be specified in the Fields map in the schema of collect.json.  See below for an example.
    "Schemas": [
      {
        "PathMatcher": "/*(?:.+/)*(?P<file>.+\\..+)",
        "DataFormat": "text",
        "DateFormats": [
          "DD/MMM/YYYY:HH:mm:ss ZZZZ"
        ],
        "Meta": {
          "Tags": {
            "file": "{file}"
          }
          "Fields":{
         "field1": "field1"
         "field2": "field2"
         }
        }
      }

Here is a sample use-case.
record:
{"log":"type=hello level=debug","stream":"stderr","time":"2018-02-06T10:22:49.201168034Z"}

By specifying type, level in Fields, we could have type and level as additional fields and use in LQL by calling {vars:type} or {vars:level}
parsed result:
msg="type=hello level=debug", stream=stderr, time=2018-02-06T10:22:49.201168034Z, type=hello, level=debug

If msg contains a correctly formatted time field, then the parsed out field would replace the value in the time field of the msg.
record
{"log":"time=2018-01-01T10:22:49.71369883Z","stream":"stderr","time":"2018-02-06T10:22:49.201168034Z"}

parsed result
log="time=2018-01-01T10:22:49.71369883Z",stream=stderr, time=2018-01-01T10:22:49.71369883Z

I included tests to test the expected behavior.
One implementation detail to note is that after passing in the Field map to the parser, I take the keys in the map and save it in an array and delete all the keys in the map to avoid printing out duplicated fields in LQL. I would like to adjust the code if there is a better way of dealing with this.

The parser has been tested in the cmdline using LQL. They are sent as designed.


feat: add Dell BOSS disk support

Lets show the individual disks instead of the array
racadm storage get pdisks
Disk.Direct.0-0:AHCI.Slot.1-1
Disk.Direct.1-1:AHCI.Slot.1-1

$ racadm storage get pdisks:Disk.Direct.0-0:AHCI.Slot.1-1
    Disk.Direct.0-0:AHCI.Slot.1-1
       Status                           = Ok
       DeviceDescription                = Disk 0 on AHCI Controller in slot 1
       RollupStatus                     = Ok
       Name                             = SSD 0
       State                            = Online
       OperationState                   = Not Applicable
       PowerStatus                      = On
       Size                             = 223.571 GB
       FailurePredicted                 = NO
       RemainingRatedWriteEndurance     = 100 %
       SecurityStatus                   = Not Capable
       BusProtocol                      = SATA
       MediaType                        = SSD
       UsedRaidDiskSpace                = 223.509 GB
       AvailableRaidDiskSpace           = 0.001 GB
       Hotspare                         = NO
       Manufacturer                     = INTEL
       ProductId                        = SSDSCKKB240G8R
       Revision                         = XC31DL6P
       SerialNumber                     = PHYH951604K8240J
       PartNumber                       = WTD0CM51IPIH0T1140CS0A

Hi! Is this still needed?


test/cli: add basic amtool cli tests

Working on some acceptance tests for amtool

On the first look, it seems ok. I'm pondering whether it would make sense to have these tests live with the other acceptance tests in test/with_api_v2 as some parts of the code are already common.

Originally, I was thinking that these tests would verify backwards compatibility in amtool while making changes like the recent update from v1 to v2.  So for future updates, e.g. if there was ever an api v3, ideally the amtool tests would stay the same and would provide an automated compatibility check.  Although I'm not sure how practical this would be.

any more thoughts on this?

Ping @simonpasquier @pgier.

Rebased onto latest master, fixed a merge conflict, and made some minor simplifications.
@simonpasquier can you take a look?


asmCrypto PRNG not seeded

Using asmCrypto inside a browser web app, I get the following message:

asmCrypto PRNG not seeded; your security relies on your system PRNG. If this is not acceptable, use asmCrypto.random.seed().

I've tried some different solutions but I can't get what is the expected right way to proceed in this situation. My app is completely client side JS, there is no backend. Is there any example available that shows how to initialise the PRNG on a browser?
My targets are the latest versions of Chrome, Firefox and Edge.

The warning is indicating that PRNG hasn't been seeded and is using a default implementation. If you simply wish to silence the warning you can by setting.
asmCrypto.random.skipSystemRNGWarning = true

If this is a security critical use case, you should seed by passing some random data to
asmCrypto.random.seed()

However, make sure to heed the warnings in the src as setting a poorly generated or constant value undermines everything thereafter. Buyer beware.
Some good links and discussion in #27 and #65

Thank you for your reply; I already read all the links and source code you pointed out. Let me understand if I got it right: since I use asmcrypto.js inside a web browser client app, I can skip the warning and use the "default" PRNG implementation or I can provide some random seed from a remote server. I don't see any other way to provide random data as seed on the client; I could use some mouse movements as seed but it could create problems when using touch devices.
From your experience, is the browsers default implementation of PRNG so weak that an alternative must be provided if using the random data generation for PBKDF2 salt and AES IV generation?

The warning is indicating that PRNG hasn't been seeded and is using a default implementation.

Quick question: Doesn't asmCrypto PRNG not seeded mean the same as: seeded with a constant value? and thus this applies: **DO NOT SEED WITH CONSTANT VALUE! YOU'LL GET NO RANDOMNESS FROM CONSTANT!**
Thanks.

@xftroxgpx it means "seeded with a small amount of randomness" (take a look to https://github.com/vibornoff/asmcrypto.js/blob/master/src/random/random.js#L32-L84)

Right-on. Much appreciated!


Fix Run BLAST links

Changes the HTML method for BLAST queries.
Uses Post instead of Put. Put no longer works; NCBI botches the QUERY param when the method is Put.
Delivers #2303

coverage: 93.52%. remained the same
when pulling 14e1307b409d1f1dbd60868be7c6eb297e3fc9da on 2303-fix-blast-links
into 71a6e4aad94677ad5286a6d8b78fb874cb66559b on main.

Deployed. deploy-2024-08-12-15-40


Migrate vanilla extract astro plugin to vite plugin

Migrate from the deprecated Astro Plugin to the Vite Plugin

Blocked by: https://github.com/vanilla-extract-css/vanilla-extract/issues/1336


[Kernel] Fused MoE Config for Mixtral 8x22

This PR adds fused MoE configs for Mixtral 8x22 on A100-80G & H100 with TP4 & 8 - Roughly 10% latency speedup for high batch sizes.
Attaching mean and median results from running benchmark_latency.py on A100-80G as a sanity check
TP4
bs 8
Avg latency: 5.787593524421876 seconds
50% percentile latency: 5.785140061285347 seconds

bs 16
Avg latency: 6.900521802188208 seconds
50% percentile latency: 6.910593919456005 seconds

bs 32
Avg latency: 7.437153746870657 seconds
50% percentile latency: 7.425246893661097 seconds

TP4 with config
bs 8
Avg latency: 5.4268179705676935 seconds
50% percentile latency: 5.4153253806289285 seconds

bs 16
Avg latency: 6.107941208345195 seconds
50% percentile latency: 6.10570535203442 seconds

bs 32
Avg latency: 6.893939550562451 seconds
50% percentile latency: 6.877172949025407 seconds

TP8
bs 8
Avg latency: 3.83942656442523 seconds
50% percentile latency: 3.8410149838309735 seconds

bs 16
Avg latency: 4.428924084718649 seconds
50% percentile latency: 4.421724954620004 seconds

bs 32
Avg latency: 4.767531445249915 seconds
50% percentile latency: 4.752460908377543 seconds

TP8 with config
bs 8
Avg latency: 3.7083894722008455 seconds
50% percentile latency: 3.7043474409729242 seconds

bs 16
Avg latency: 4.047313968433688 seconds
50% percentile latency: 4.0400408857967705 seconds

bs 32
Avg latency: 4.390694409639885 seconds
50% percentile latency: 4.380223051644862 seconds

@ywang96 I'm not an expert could you explain how what would be the process to adapt this config to NVIDIA_A100-SXM4-40GB ? Thank you !


Enemy Makes You Go Through Ground When It's Momentum Is Super High

No more problem

broski opening this up again

now closing it as a joke


Does not support users who type in Japanese

bug
The Japanese language has both kanji and hiragana, and the enter key is used to convert hiragana into kanji.
In the current implementation, handleSend is executed when the conversion is confirmed and starts sending messages.
solution
When the conversion key is pressed, the message is not sent.

Merged!


Why using only NormalizeLandmark in PoseSolver?

In kalidokit, PoseSolver is using WorldLandmark and NormalizedLandmark both.
https://github.com/yeemachine/kalidokit/blob/c720d483df867996be96e2c3218e963da9190ecf/src/PoseSolver/index.ts#L15-L26
https://github.com/yeemachine/kalidokit/blob/c720d483df867996be96e2c3218e963da9190ecf/docs/script.js#L170-L173
Is there any reason only using NormalizedLandmark in Moetion here?

Actually there isn't. When I read the code, I thought they were supposed to be the same kinds of coordinates, especially with this documentation comment:
    /**
     * Combines arm, hips, and leg calcs into one method
     * @param {Array} lm3d : array of 3D pose vectors from tfjs or mediapipe
     * @param {Array} lm2d : array of 2D pose vectors from tfjs or mediapipe
     * @param {String} runtime: set as either "tfjs" or "mediapipe"
     * @param {IPoseSolveOptions} options: options object
     */

Guess that's something to fix!


feat: Add booking/list-activities

Description
Adds "List Activities" profile for booking, and a map for Bokun.io
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I haven't repeated the code. (DRY)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

There are a few critical issues, but before addressing that, I'd like to discuss naming. Since "booking" is overloaded with hospitality and services industrie, maybe we could use some other scope? For example tour (from Tour Operator Software) or travel?

I'd definitely prefer tour over travel - I'd expect travel to include stuff like bus/plane/train tickets, connections, etc. - things explicitly connected with the act of travelling (for both leisure and business purposes). tour doesn't have that, but IMHO feels too restrictive - tours are just one category from the offerings available on e.g. Bokun, the rest not being very tour-like (and by the way, generally falling under the term "activities" - which also isn't very suitable as a scope name), meaning tours may be a bit misleading.
Really, I feel like booking is the most appropriate and accurate scope name, as booking a trip/tour/activity - or managing said booking - is somewhat of a "final destination" of any application using these profiles.


Document the API

Feature Request
It would be great to include API documentation. The Usage docs are great as a walkthrough, but when you want to go fast ("what is the name for the Field parameter for a constant value?"), API docs provide a faster answer.
Another use case: I want to inspect the __fields__ on a model, and had to read the code to get the attribute name I was looking for.
API docs would be such a great addition to this amazing library! Super useful!

I disagree, documenting everything would be extremely difficult. In the end, the only real source of truth is the code itself. One of the reasons we're all writing python not c is that it's easily readable.
I'd be happy to accept a PR to add info on __fields__ to the model documentation. And many more PRs to extend the documentation in other ways. But I don't think "API docs" as a separate section either auto generated, or hand written are a good idea.

I'm not asking about documenting absolutely everything :) but Field is a good candidate.
The API documentation is usually auto-generated (meaning no inconsistency possible between the code and the doc) and makes it possible for advanced users to go straight to the answers, whereas guides provide a more narrated way of discovering the library.
Here are some good examples of API reference from some major packages:

https://reactjs.org/docs/react-api.html
https://flask.palletsprojects.com/en/1.1.x/api/
https://docs.djangoproject.com/en/3.0/ref/
https://www.attrs.org/en/stable/api.html
https://requests.readthedocs.io/en/master/api/
https://marshmallow.readthedocs.io/en/stable/api_reference.html
https://jinja.palletsprojects.com/en/2.11.x/api/
...

Here are some good arguments in favor of having an API reference:

It provides immediate access to a comprehensive interface definition, instead of having to follow a narrated guide that only focuses on certain elements.
It lets the reader discover all the functionalities easily from a single place.

See this fairly long discussion on the matter.
I'm strongly opposed to multiple different types of documentation, I think there should be one and only one section of documentation on a feature. That's impossible to achieve entirely, but adding a whole new "api" section to the docs makes it impossible.

The API documentation is usually auto-generated

Is that really true? It's not for python itself, or aiohttp, I wonder if it is for many of the projects you list above. As mentioned on the issue above, it generally doesn't work that well.
There's also no widely used way to auto-generate docs in markdown - maybe that's a strong hint that it's not a populate approach anymore.
Even if they were, the value in docs is in the examples and descriptions. That still has to be written even if it's in docstrings, not md files.

More generally I know the docs could be better, however I don't much enjoy writing docs myself and I get very little help on it from others.
I maintain pydantic for fun and I don't want to spend a significant amount of my free time writing docs, apparently neither does anyone else.
I setup patreon to see if people would who use pydantic would contribute financially, and thereby pay me to do the boring donkey work like docs. It pays me $13/mo. I charge £100/hr for development, so that would pay me for 6'30" a month to improve the docs (at the current exchange rate).
There's also the problem of upkeep once the docs are written. I find it hard enough to get people to document the features they add or modify in pull requests. That would only get harder if they had to extend the API docs as well as modify the main documentation.

In short, I think best to submit a PR to add docs for specific things instead of adding another massive section to the docs.

Hey Samuel, thanks for taking the time to answer.
I understand your position. I would still submit that it goes against what most packages are doing, but that would be a somewhat fallacious argument from authority.
I think there are two discussions.
Whether or not pydantic should have an API reference.
I strongly think it should be the case. You don't. In the end, if I don't manage to convince you that it's a useful thing, there's no point in moving further on this ticket and you can close it because you're the repo owner.
Re. your arguments:

Some of those projects don't auto-generate the docs.

That's true, but some do autogenerate:

https://github.com/pallets/flask/blob/master/docs/api.rst
https://github.com/psf/requests/blob/master/docs/api.rst
https://github.com/marshmallow-code/marshmallow/blob/dev/docs/api_reference.rst

There's no widely used way to auto-generate in Markdown.

That's true, but not really a reason you should not have an API reference.
Using Markdown or not for documentation is a pretty heated topic, with lots of good arguments on both sides.
AFAIK Sphinx does support Markdown for documentation. Never used it though. autodoc does use the rst format though.

If you agree there should an API reference, how to build it
I'd be happy to help! But it's true that the API absolutely needs to be generated, otherwise, it will be a nightmare to maintain them. Another way would be to just include the main parts of the code in the docs (e.g. the Field constructor).
Anyway, in conclusion, just wanted to say that an open source contributor and maintainer myself, I empathize with your feelings re. the time it represents.

Thanks for your input, I get where you're coming from.
Let's see what others think, if lots of people support this I'll think about it more.
Requirements:

follows the same formatting and structure as the following docs - I don't want a mkdocs site with a sphinx site bolted on somehow. It needs to work with the current menu and search.
easy way of linking from the current docs to the API docs, and the other way
docs auto-generated from signatures (including type hints) and docstrings
easy way to decide what's included in the API docs - it'll be no use if every single function and attribute is shown.

There's currently no way to render rst documents within mkdocs, see https://github.com/mkdocs/mkdocs/issues/1817.
So the options are:

find a way to render rst in mkdocs while keeping the formatting unchanged - sounds like hell
build a new library that parses the AST of a python file and generates a md file
improve or coerse an existing py -> md library. I tried a few of the libraries currently in existence a few months ago when building rtoml, but none of them worked for me.
include the entire pydantic source code in the docs, nicely highlighted and linkable with links back to the normal docs marked-up. We could then in future extend that to "fold" function definitions, mark private functions as private, go to definition etc. etc.

If we do have to do this, my preference would be option 4, but even this is a big piece of work that would require a library of its own and ~10 hours of work before we could start using it.

Re: Autogenerating documentation from markdown docstrings, https://github.com/pawamoy/mkdocstrings has sprung up as a mkdocs plugin/extension to do just that, and @pawamoy is currently working on implementing a filter option for more detailed control.
It currently:

follows the same formatting and structure as regular mkdocs
allows for easy way of linking from the current docs to the API docs, and the other way
auto-generates docs from signatures (including type hints) and docstrings (the entire point of it)
gives an easy way to decide what's included in the API docs, as far as top-level items go, e.g.# Markdown header
::: pydantic.fields.Field

Funny, seeing pydantic and fastapi docs is what made me want to use mkdocs, and then build mkdocstrings :smile:

[x] follows the same formatting and structure as the following docs - I don't want a mkdocs site with a sphinx site bolted on somehow. It needs to work with the current menu and search.
mkdocstrings a plugin for mkdocs so it's completely integrated with it. It supports the Material theme.
[x] easy way of linking from the current docs to the API docs, and the other way.
I added a cross-reference ability. As long as your headings are unique (their ids always are but sometimes are suffixed with _1, etc.), you can link back and forth using the classic markdown syntax for references: [title][heading_id] or directly [heading_id][]. Since the generated docs use a heading for every Python object, you can link to them easily, from your markdown pages or from your docstrings. Cross-references will still work even if you move the objects/heading to other pages.
[x] docs auto-generated from signatures (including type hints) and docstrings. Check. Docstrings must be written in Markdown. mkdocstrings can also parse the Google-style docstring format to generate tables for parameters, etc.
[ ] easy way to decide what's included in the API docs - it'll be no use if every single function and attribute is shown.
As @StephenBrown2 said, I'm currently working on it :slightly_smiling_face:

I would love to see Field documented. It really confused me that there is no documentation of possible Field args in the docs.

There is documentation on field arguments.

O_o
Sorry I missed it.

I tend to use sphinx for my doc.  Mostly prose, but I like generated api doc to exist so I can hyperlink to it.  That being said, there is no objects.inv provided for pydantic.  That means I need to manually ignore each class or function (using https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-nitpick_ignore) to suppress warnings in the building of my documentation, since my code contains reference to pydantic.
I think that providing objects.inv would be very nice!

FYI there's MyST for Sphinx+Markdown+RST directives now: https://myst-parser.readthedocs.io/.

API docs. Gotta have.
The  Usage docs tell you some of the things you can do. They don't tell you everything. And they don't tell you what you can't do.
I'm looking at validators, shared validators, and so bump in to allow_reuse=True. What does that do? What would be the effect of not passing it?
And, validator seems to be a decorator. Except when it's not.
All of these things would be answered by an API doc.
If "documenting everything would be extremely difficult", then maybe understanding how to use it is also "extremely difficult". Reverse-engineering the code isn't a good way to find out how to use it. And does not allow us to determine the changes we need to make when upgrading.

Great resource on the topic: https://documentation.divio.com/reference/

I'm not sure what's expected here. If the goal is to have more information on __fields__, I feel like the solution would be to expose a public method with dedicated method fields(My model) like for dataclass. But never explain the internal of pydantic.
Same if some doc is missing on validators PRs are always welcome to explain more how things work.
But I feel like the doc of pydantic is already great and if some things are missing we can just add stuff there.

I would also love to have API documentation with objects.inv that will simplify cross-linking from other documentation (sphinx based).

I have plans for this.
It it won't be using sphinx, but rather mkdocstrings which I've used very successfully recently in dirty-equals and watchfiles.

Mkdocstrings can do the same thing I think.

Indeed, it's documented here: https://mkdocstrings.github.io/usage/#cross-references-to-other-projects-inventories :)

But this is the documentation on how to add references. Did mkdocs generate objects.inv file, and if yes, then where could we find it (as it is required to reference from another project to pydantic)?

Interesting question, @pawamoy would there be a way for others to reference pydantics docs from their docs?

That's probably not very clear in the docs, but yes:

Reciprocally, mkdocstrings also allows to generate an inventory file in the Sphinx format. It will be enabled by default if the Python handler is used, and generated as objects.inv in the final site directory.

It means objects.inv would be served at https://pydantic-docs.helpmanual.io/objects.inv. I'll improve the paragraph above in the docs.

I second the need for API reference. This is important. It does not mean that every feature needs very detailed docs. At the same time, one of the first things I see is the Field object and no documentation whatsoever on it.  This is not right.

It does: https://pydantic-docs.helpmanual.io/usage/schema/#field-customization
API documentation will be done in V2, this is already stated in the blog about V2.

Thanks!


Stilynt - linter plugin

v1.2.0 landed with what was needed to successfully created a linter, WIP at plugins/linter The bulk of the effort will be in scrapping css syntax data and converting that to regex's or create them by hand to validate known css properties.

A WIP example of how this will work at https://rawgit.com/thysultan/stylis.js/master/plugins/stilynt/index.html
Exploring how to automate the collection of all the syntax rules into a regex key-value map i.e from https://developer.mozilla.org/en-US/docs/Web/CSS/Reference or https://github.com/mdn/data


Using .predict_text() method with an Iterable of text

Hello the awesome team behind this project!
I've been using this repo for some NER tasks recently, and it is fantastic and super easy to use! However, I would like to suggest that you should .predict_text() method able to run inference on multiple text instances rather than one instance at a time like now. It will be much more convenient and may allow for faster computation

HI @hieuthehungry
Thank you so much! And thank you for your feedback.
Sounds like a good idea.... I encourage you to make a Pull Request, if you want to speed up the process.
Best
Lars


Open SSH in terminal

Is this a similar or duplicate feature request?

[ ] I don't know. I will go check it.
[x ] No.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I already use remote path : (and it is no problem on Remote Explorer )
but when I use "Open SSH in terminal" , remote path is not call
i got home directory
not remote path that i set on sftp.json
Describe the solution you'd like
A clear and concise description of what you want to happen.
when i log in by ssh, remote path is call too
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Does this project help you?

[x] Yes. SFTP IS AWESOME!
[x ] Yes. I want to support you.

Support in v1.9.4.


Update-Lab doesn't work if EnvironmentPrefix was used

After creating lab with "EnvironmentPrefix" set in VMConfigurationData.psd1 command
 update-lab  tries update VM based on NodeName without EnvironmentPrefix and  returns:
Get-VM : Hyper-V was unable to find a virtual machine with name "DC1".
At C:\Program Files\WindowsPowerShell\Modules\PSAutoLab\4.9.0\functions\public.ps1:1230 char:23
+                 $vm = Get-VM -name $node.Nodename
+                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (DC1:String) [Get-VM], VirtualizationException
    + FullyQualifiedErrorId : InvalidParameter,Microsoft.HyperV.PowerShell.Commands.GetVM

AutoLab         : C:\_mo\VMs\AutoLab
PSVersion       : 5.1.17763.1007
PSEdition       : Desktop
OS              : Microsoft Windows 10 Pro
FreeSpaceGB     : 69,59
MemoryGB        : 16
PctFreeMemory   : 61,79
Processor       : Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz
IsElevated      : True
RemotingEnabled : True
HyperV          : 10.0.17763.1
PSAutolab       : {4.9.0, 4.8.0, 4.3.0}
Lability        : {0.19.1, 0.18.0, 0.10.1}
Pester          : 4.9.0
PowerShellGet   : <IP_ADDRESS>

Even though that comment is in the psd1 file, I don't think it has ever been used or tested. I'll have to look into this.

I know that if you use a prefix, the validation tests will never work And I'm trying to figure out how I could make that possible.

Update-Lab isn't expecting a prefix. Not that I couldn't make it work. In all the years I've been working with this module, this is the first time anyone has used the prefix option. I have to decide if that feature can be allowed. The biggest issue is the validation tests.
Why did you decide to use the environment prefix?

I have made what should be the necessary updates. Update to v4.10.0 of the module. Be sure to run Refresh-Host to update configuration files. You still have to manually edit VmConfigurationData.psd1 to enable the environment prefix.

Why did you decide to use the environment prefix?

I'm using it for distinguishing laboratories based on this same configuration. I just create copy of configuration and without environment prfix I have to change name of each node. With environment prefix VM name differs it is easier to manage VM in Hyper-V.

Are you trying to have 2 copies of the same configuration running at once?

Generally I run one at time but I've got many configured,
I think that running many is also possible but it requires more modifications of config files (names, IP addresses), correct me if I'm wrong.

The module was designed with the assumption users were running a single lab at a time for a given Pluralsight course. You can set up as many configurations at once as you want. Using the prefix would be a good way to keep the VMs straight and that should now work. You have to be aware of VMs in configurations that might use the same IP address. That's where the Get-LabSummary command comes in handy.  But otherwise, you are correct in that you would need to edit the configuration data. I'd suggest copying the entire folder with a new name and edit the copy.
And now with v4.10.0 commands will now work with an environment prefix. Is there anything else with this particular issue that needs to be addressed?

no thank you


chore(quantic): Replace sfdx-cli with @salesforce/cli

SFINT-5114
sfdx-cli is in maintenance only mode, all the updates are now done to the @salesforce/cli
Pretty much all the commands of sfdx still work as-is with @salesforce/cli
But still we want to update all those commands to the new sf commands format

Actually, I checked again why GHSA-67hx-6x53-jw92 was reported again, and indeed, @salesforce/cli was still using (and requiring) a vulnerable version of @babel/traverse, until yesterday or so.
I bumped SFDX again in bf2de78 to fix that.

Does this update fix the babel/travers vuln?
I updated first to that version of sf yes but they had a bug on the .8 version, they since released the .10 that you used.
So if the .10 works, then all good to update to that yes.


Issue with multiple values for MX (and possibly other TXT) record type for bare domain resulting in mixed types in map??

Description
Unless I'm doing something wrong, multiple values for MX (and possibly other TXT-style) record types are not supported and throw a Terraform error due to mixed map types in the local.recordsets value... I hope I'm missing something particularly obvious...
Reproduction
Code Snippet to Reproduce
module "server_com_records" {
  source  = "terraform-aws-modules/route53/aws//modules/records"
  version = "~> 2.0"

  zone_name = keys(module.root_zones.route53_zone_zone_id)[0]

  records = [
    {
      name = "" # Apex A Record (Works)
      type = "A"
      ttl  = 3600
      records = [
        "<IP_ADDRESS>",
        "<IP_ADDRESS>"
      ]
    },
    {
      name = "" # Apex MX Record (Sometimes works with 1 records)
      type = "MX"
      ttl  = 300
      records = [
        "5 mx1.mailserver.com"
      ]
    },
    {
      name = "" # Apex MX Record (Doesn't Work with >= 2 records)
      type = "MX"
      ttl  = 300
      records = [
        "5 mx1.mailserver.com",
        "5 mx2.mailserver.com"
      ]
    },
  ]

  depends_on = [module.root_zones]
}

Expected behavior
MX record is created with 2 records
Actual behavior
│ Error: Inconsistent conditional result types
│
│   on .terraform/modules/server_com_records/modules/records/main.tf line 15, in resource "aws_route53_record" "this":
│   15:   for_each = var.create && (var.zone_id != null || var.zone_name != null) ? local.recordsets : tomap({})
│     ├────────────────
│     │ local.recordsets is object with (2|3) attributes
│
│ The true result value has the wrong type: element types must all match for conversion to map.

Terminal Output Screenshot(s)
✗ terraform --version
Terraform v0.15.4
on darwin_amd64
+ provider registry.terraform.io/hashicorp/aws v3.42.0

Additional context

It sometimes doesn't work even with 1 record when multiple bare records are present...
I manually added output to print local.recordsets in the .terraform/modules/server_com_records/modules/records/outputs.tf and printed it in terraform console:
> module.server_com_records
{
  "route53_record_fqdn" = {
    " A" = "server.com"
  }
  "route53_record_name" = {
    " A" = "server.com"
  }
  "route53_recordsets" = {
    " A" = {
      "name" = ""
      "records" = [
        "<IP_ADDRESS>",
        "<IP_ADDRESS>",
      ]
      "ttl" = 3600
      "type" = "A"
    }
    " MX" = {
      "name" = ""
      "records" = [
        "5 mx1.mailserver.com",
        "5 mx2.mailserver.com",
      ]
      "ttl" = 300
      "type" = "MX"
    }
  }
}

Just as a heads up, I downgraded to Terraform 0.14.11 and seems to be AOK -- I'll run more testing today, but it very well could be a possible bug in 0.15.x...

It can be is a bug in 0.15.
I saw similar strange behavior with this line in step-functions module earlier today when using 0.15 but not with 0.14

Yeah, I ran this in many variations this past week and it seems the issue is present in 0.15 .... will stick to 0.14 for now.
Sorry for the bother!

No problems. Thanks for the confirmation!

I noticed that the module works when replacing tomap({}) with {} in [line 15( https://github.com/terraform-aws-modules/terraform-aws-route53/blob/736dcb71c44729d2b62ade80747e8de9abf369f8/modules/records/main.tf#L15) of the records module.
I don't know if that could be a valid workaround until the bug is fixed in terraform (btw, did you create an issue? I couldn't find one)?


Access-rulebase facts over Ansible module ?

hello guys,
Thanks for your work and modules.
Is there a way to access to access-rulebase over an existing Ansible module ?
Indeed, module to obtain access-rulebase facts semms to existing ?
With actual modules, how to compare existing ruleset with a new one we want to add ?
Thanks :)

Hi, Thanks :)
For showing the rulebase we have the module you mentioned "cp_mgmt_access_rule_facts.py".
And for changing access rules we have the module "cp_mgmt_access_rule.py"
And for comparing you can use the first module I mentioned, or just change the rulebase with the second module I mentioned, and see in the output of the playbook if the rulebase was changed.
Thanks , Or


ioredis: connect instrumentation does not report correct duration and does not report failure

What version of OpenTelemetry are you using?
v0.13.0
What version of Node are you using?
12
What did you do?
If possible, provide a recipe for reproducing the error.
While there is no redis server on localhost, I run the following code:
const Redis = require("ioredis");
const redis = new Redis();

What did you expect to see?
Expected to see spans of connect with an error indication (as the connect failed), and expected the duration to contain the full length of the operation.
What did you see instead?
Spans are reported with CanonicalCode.OK (StatusCode.UNSET in master), and the duration only tracks the connect function call, and not including the time until the callback is called.

current version of ioredis plugin is 0.11 it is not compatible with core version > 0.12 - the same as any other plugins. Contrib repo needs to be released first, there is an opened PR for new release I created last week which needs  approves first.

current version of ioredis plugin is 0.11 it is not compatible with core version > 0.12 - the same as any other plugins. Contrib repo needs to be released first, there is an opened PR for new release I created last week which needs approves first.

This bug is not related to your changes.
Look at ioredis plugin code here:
    try {
      const client = original.apply(this, arguments); // original here is `redis.connect` and returns a promise.
      endSpan(span, null);
      return client;
    } catch (error) {
      endSpan(span, error);
      throw error;
    }

If the original callback or promise returns with error, nothing is setting the span to error in the plugin.


Update AWS AMI

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[ ] 3.0
[ ] 3.1
[x] 3.2
[ ] self-compiled devel branch

Mode:

[ ] Cluster
[x] Single-Server

Storage-Engine:

[x] mmfiles
[ ] rocksdb

On this operating system:

[ ] DCOS on

[x] AWS
[ ] Azure
[ ] own infrastructure

[ ] Linux

[ ] Debian .deb
[ ] Ubuntu .deb
[ ] SUSE   .rpm
[ ] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[ ] MacOS, version:

this is an AQL-related issue:
[x] I'm using graph features
I'm issuing AQL via:

[x] web interface with this browser:     running on this OS:
[ ] arangosh
[ ] this Driver:

I've run db._explain("<my aql query>") and it didn't shed more light on this.
The AQL query in question is:
The issue can be reproduced using this dataset:
Please provide a way to create the dataset to run the above query on; either by a gist with an arangodump, or `db.collection.save({my: "values"}) statements. If it can be reproduced with one of the ArangoDB example datasets, it's a plus.
Foxx
this is a web interface-related issue:
I'm using the web interface with this browser:     running on this OS:

[ ] authentication is enabled?
[ ] using the cluster?
[ ] _system database?

These are the steps to reproduce:

open the browser on http://<IP_ADDRESS>:8529
log in as ...
use database [ ] _system [ ] other:
click to ...
...

The following problem occurs: [Screenshot?]
The AWS AMI is running version 3.2 which is not the latest patch
this is an installation-related issue:
My team had previously reported a bug that was patched in v3.2.7. I would like to update my database using the AWS AMI but hasn't been updated since summer.

Hello,
we are trying to update the AWS AMI as soon as possible, currently the update functionality of AWS for AMIs is broken though.

No worries, manual update was fine. Thanks, guys


Can a Formula just link bin and not lib etc?

I am trying to write a package for Aspera (ascp).  it comes as a Unix shell installer with a uuencoded tar.gz embedded in it.  I can get it to install in prefix but it's file structure has lib, etc and so on.
The binaries are all launced via shell scripts which set up up paths to point to the correct bundled .so dynamic libraries (which i don't really want anyway as they are for the GUI bit).
So I want the bin to symlink in, but not the rest.  So it's a "partial keg".
Any suggestions?
Plan B: I could probably delete all the X11 gumpf and just keep the ascp binary.

I would also like this feature, but it's not likely, I think. The Homebrew solution is to install to a subdirectory like libexec, and then create a symlink to the launching executable in bin.


code for the scores [ HD95, Jaccard Index ]?

kindly mention where did you extract the dice score, jaccard index, hd95 score and all.

@WuJunde kindly help me with this

hi, I provide an example evaluation code of dice and iou(jaccard) in script/segmentation_env.py. hd95 implementation can be easily found online if you need.


Action decorator wrong implementation (but working)

I have a doubt with Action decorator, here is the source code:
export function Action(options: ActionOptions = {}) {
  return function(target, key: string, descriptor: PropertyDescriptor) {
    if (!options.skipHandler) {
      options.handler = descriptor.value;
    } else {
      delete options.skipHandler;
    }

    (target.actions || (target.actions = {}))[key] = options
      ? {
          ...options
        }
      : options.skipHandler
      ? ""
      : descriptor.value;
  };
}

first if:

if options.skipHandler exists, it will be deleted, fair enough

target.actions assignation:

if options exists, then { ...options } ELSE check options.skipHandler here is the error, because if options.skipHandler exist is deleted in previous if and you are checking the "else" condition of "if (options)" then, the else condition is that "options" is not defined, then it is not possible that "options.skipHandler" could be defined.

This is giving me a typescript error:
TS2339: Property 'skipHandler' does not exist on type 'never'.
There are some other errors with "noImplicitAny: true" in tsconfig.json that avoid build in projects that are using moleculer-decorators with that config in tsconfig.json, maybe could be useful to add this compilerOptions:
    "noImplicitAny": true,
    "noImplicitReturns": true,
    "noImplicitThis": false,
    "noStrictGenericChecks": true,
    "noUnusedLocals": true,
    "noUnusedParameters": false,
    "resolveJsonModule": true,
    "strict": true,
    "strictNullChecks": true,

Could you supply an example that reproduces your type error?

use the configurarion provided for "tsconfig.json"

the error is in ternary like I said in the first message, you check "if" options exist, the in else (where "options" does not exists) you call "options.skipHandler"
This is the error. In "else" you could not use "options" because it does not exist


Close auto hide mode when clicked the collapse button

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

1 of 4 (25.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 73.649%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyqtribbon/ribbonbar.py
1
4
25.0%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.1%

Covered Lines:
981

Relevant Lines:
1332

💛  - Coveralls


Destroy PDF Object

Is there a way to destroy the PDF object when it is no longer required?

I guess I could loop through and remove all pages when the PDF is no longer needed and just set its reference to null so that it takes up less memory before gc.
Any thoughts?

@mattsputnikdigital did you come up with any solution for this problem?

Thanks @MatthewPringle.
It'd be good if the library exposed a .destroy() method.

Not sure I understand what you're looking for here.  Feel free to fill out a proposal with more details and we can go from there.


Difference between sokol_app events KEY_DOWN and CHAR

Hi,
I'm pretty new to Sokol and I've been trying to learn it with the Odin bindings. My issue is with how input is checked. So there are events for it but as far as what I noticed, even in the https://floooh.github.io/sokol-html5/events-sapp.html example, there is no difference between KEY_DOWN and CHAR. They both behave like text inputs rather than an input similar to the ones in games. Is there another way to do it?
Thanks,
Emir

Thank you for the response. Following the pacman.c example, It worked smoothly. What I meant by behave like text inputs is a delay after the first key tick If you are holding down the key. Like: a__aaaaaaaaa.
Too bad I have to keep the state of every key I wanted but It is what it is.
Other than that, are there possible SDL/Sokol usage examples out there, If you are aware? I found a lot of examples for OpenGL but the main reason I'm using Sokol is DXD11 and Metal support.

is a delay after the first key tick If you are holding down the key. Like: a__aaaaaaaaa.

Ah right, yeah that might be a bit confusing for "raw" key input, but that's how the underlying operating system events behave. There's a bool key_repeat field in the event struct which you can use to detect and ignore such followup events if desired:
https://github.com/floooh/sokol/blob/17b399b8eeb6d1db4fe0e78fbea3b1a4184470be/sokol_app.h#L1413
Closing this ticket if that's ok :)

PS:

Other than that, are there possible SDL/Sokol usage examples out there,

Not that I know of unfortunately. The sokol-samples have examples for GLFW+GL (https://github.com/floooh/sokol-samples/tree/master/glfw) plus a very simple example for GLFW+Metal (https://github.com/floooh/sokol-samples/blob/master/glfw/metal-glfw.m).
I haven't attempted to use SDL with sokol_gfx.h so far though.


Added solar thermal collective to ovm-csv NOT TESTED

Solar thermal collective heat was missing in the OverMorgen datadump csv.
Maybe ask @Charlottevm to test it.

It works in combination with: https://github.com/quintel/etsource/pull/2479

@antw Do you have time to shortly check this before the deploy? We do not understand why Semaphoreci is giving a failing check..

It's fixed now.

Great @antw! Was this something we could fix next time?

Was this something we could fix next time?

Some of ETModel's tests fetch data from ETEngine. This can be problematic as it places extra load on ETEngine and, more importantly, means that the tests aren't always reproducible: different data might be used each time the test is executed.
We use a library called VCR which saves the results of these requests to YAML files, with the data in these files use during future test runs, instead of making new requests to ETEngine. If VCR detects that you're requesting different data than was used to generate the YAML files (as was the case in this request: an extra query was added) then it will raise an error letting you know which file is to blame:
================================================================================
An HTTP request has been made that VCR does not know how to handle:
  PUT https://beta-engine.energytransitionmodel.com/api/v3/scenarios/648695
  Body: [truncated ...]

VCR is currently using the following cassette:
  - etmodel/spec/cassettes/SavedScenarioReportsController/GET_show/ovm_csv/has_a_report_template.yml
    - :record => :once
    - :match_requests_on => [:uri, :method, :body]

[truncated ...]
================================================================================

The key line in the message is:
VCR is currently using the following cassette:
  - etmodel/spec/cassettes/SavedScenarioReportsController/GET_show/ovm_csv/has_a_report_template.yml

This provides you with the path to the responsible YAML file. Delete this file, then re-run the tests locally (bundle exec rspec) to generate fresh versions, then commit the changes.
The tests for this PR had three failures, each one due to a different YAML file. I deleted all three, ran the tests, and committed the new YAML files.

Great Anthony! Thanks for the clear explanation :) Good to know


inquire now strips ANSI escape codes making colorised text inside prompts impossible

A recent commit (possibly https://github.com/mikaelmello/inquire/commit/8e515d1f046561e48a2359d028d5b520cb5b6476#diff-546b6385118f60f64674170f786acf59f0ccce53d5d6ad4400409fc8363cfce1R74) has made it so that ANSI escape codes are now stripped. This makes it impossible to have colorised text inside prompt messages, e.g. if you use Confirm with a string that contains color text the color won't show up. This is a regression as this was possible with older versions (at least with 0.5.0).

Thanks a lot. This shouldn't have happened :)
I've added a test to properly catch this scenario and ensure it doesn't happen again


Refs #27873 -- Don't assume ALLOWED_HOSTS is a list.

We previously let ALLOWED_HOSTS to be a tuple but 17e6616 (re. #26666)
regressed on this.
Signed-off-by: Chris Lamb<EMAIL_ADDRESS>

A test should also be added.

@timgraham Indeed, but any pointers on how to properly test this? I can't re-run setup_test_environment (where it currently raises an exception) by design, so simply calling @override_settings with ALLOWED_HOSTS=('value-in-tuple',) wouldn't actually test anything.

Perhaps you can use mock to patch _TestState.

@timgraham Tests added.


Does it work on Android 11?

Does it work on Android 11?

Yes, it works for Android 11. It supports scoped storage for all OS versions.


SoftEther VPN server RAM overload

Prerequisites
Can you reproduce?
yes
Are you running the latest version of SoftEtherVPN?
Yes
SoftEther version:
softether-vpnserver_4.27-9668
softether-vpnserver_4.29-9680
Component:
Server
Operating system:
Ubuntu 14.04 LTS
Ubuntu 16.04 LTS
Architecture:
64 bit
Processor:
Intel(R) Xeon(R) CPU  X5675  @ 3.07GHz
Intel(R) Xeon(R) CPU E3-1270 V2 @ 3.50GHz
Intel(R) Xeon(R) CPU E3-1230 V2 @ 3.30GHz
Intel(R) Xeon(R) CPU E31230 @ 3.20GHz
etc.
Description
I have multiple softether vpnservers running on vps/dedicated servers on ubuntu 16.04 and 14.04. Authentification is with 2 radius servers, SecureNAT is disabled. Clients are connecting to openvpn and sstp servers. No special config is done to kernel. They all have the same config and they all have the same problem: RAM is filled nearly full after a couple of days after starting vpnserver and I have to restart it manually.
I have tested it on another machine with 3  or more clients connected, RAM goes to, let's say 1 GB, I disconnect all clients from the server. After all the clients are disconnected the amount of memory occupied by vpnserver it is also 1 GB.
Expected behavior:
Memory should free up as soon as users are disconnecting
Actual behavior:
Memory does not free up after users are disconnecting
Steps to reproduce

Run softether vpnserver with attached config
Connect some clients to it, let them connect to OpenVPN and SSTP protocols  and use it for a couple days
The RAM will fill up and after all users will be disconnected the amount of occupied RAM will be the same.

vpn_server.config(1).txt

Please paste the result of
# ps auxf

0 users connected at the moment
psauxf.txt

How many VPN concurrent VPN sessions are connected on this VPN Server at the maximum moment?

Please also paste the result of
ps auxH

"How many VPN concurrent VPN sessions are connected on this VPN Server at the maximum moment?" around 13 users
psauxH.txt

Very interesting.
In normal systems (including Ubuntu Linux) the vpnserver process consumes only < 50MB RSS memory even when processing 10 ~ 30 VPN sessions. For example I am running the vpnserver in the production environment for 30 VPN sessions, but the RSS memory usage is only 47Mbytes as following.
root     19155  2.0 11.1 2224196 47364 ?       S<l  Feb28 3226:54  \_ /root/vpnserver/vpnserver/vpnserver execsvc

Your results of ps auxH seems normal except the size of RSS.

Same problem here. It is happening with the newest softether installation apparently.
Only rebooting vpnserver process  every X would mitigate this problem, with obvious disease for vpn clients (and customers).
How to solve ?

Because of impossibility to stop "server_log", we set up attributes as immutable to server_log folder  (chattr -i server_log)
Because of this, the log was saved in RAM. After unsetting this parameter all unsaved log were copied to server_log directory and the RAM gradually was cleared.
After removing the immutable parameter, no more RAM overload was observed.

@pirpiliter Could you specify how exactly you went to solving this memory issue? Where do I remove this parameter?

@pirpiliter Could you specify how exactly you went to solving this memory issue? Where do I remove this parameter?

Read again comment above + https://wiki.archlinux.org/index.php/File_permissions_and_attributes#File_attributes


build not idempotent: #speakers room alias already taken

There was an error processing your command: Error whilst creating #speakers:fosdem.org: {"errcode":"M_ROOM_IN_USE","error":"Room alias already taken"}. Tried to create the room because failed to resolve it: {"errcode":"M_UNKNOWN","error":"Failed to fetch alias"} when using build command.
Should ensure this is actually idempotent — I must have missed these special rooms in the last crack at doing this.

This is actually a non-fatal error, it will continue with the building.
It's already trying to be idempotent, but it's been configured to look for real fosdem.org as the serverpart of the alias, so it tries anyway.


updating readme file with new feature

Related issue

What changes does this PR do?
updating readme file with new feature, where developer stuck into the readme file when go to the particular alphabet and scroll up for long time to move at the top. So, I added a key (top) into the readme file where developer click and move at the top of the readme file.
Screenshots (If any)
Questions

@Panquesito7 please review :)

Thanks @Pradumnasaraf


Add .syncanoignore file support for hosting

Codecov Report

Merging #270 into master will increase coverage by 5.72%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master     #270      +/-   ##
==========================================
+ Coverage   31.51%   37.23%   +5.72%
==========================================
  Files          84       99      +15
  Lines        2421     4106    +1685
  Branches        6        6
==========================================
+ Hits          763     1529     +766
- Misses       1658     2577     +919


Finding Pivot element in sorted rotated array.cpp

I have added code for finding pivot element in sorted rotated array. Please merge my code.

Thanks for your contribution!

PR has been merged under Hacktoberfest!

Happy contribution!


Pleas hlep

Hi,
please use your wan ip (external ip) and not your local ip (<IP_ADDRESS>).

I guess this issues as been resolved.


Best Time to Buy and Sell Stock #ISSUE462

Done the code with C++ for the given problem statement in issue#462

issue closed


verify routing keys of incoming data session requests

there is currently no paper trail for rejected DataSessionTransfer Requests.  Going to look at adding that to this pr


Can't publish without node_modules present, but node_modules causes empty objects returned from require

So I've updated the peer deps required by the package and everything worked great locally. Upon trying to publish the new version though meteor complained that the postcss and postcss-load-config modules were missing. I installed them and published successfully. Upon installing it in the app that I tested it in locally, it threw an error that load was not a function. Upon investication it seems that that node_modules folder gets published with the package and somehow interferes with the package when installed.
If I symlink the package to the local app again and just remove the node_modules folder, the error is gone and everything works again. @juliancwirko has this been an issue in the past or is this something that's changed in Meteor since the previous version was published?

Hi, I stopped work on this package a couple of years ago nor have I worked with Meteor for several years. I'm afraid that I won't be able to help with that. I don't remember such problems, but I remember that there were difficulties related to Meteor architecture in the course of subsequent version updates or lack of proper changes in Meteor.

@juliancwirko Thanks for your time, and the quick reply! I guess I'll have to file and issue in the Meteor repo and see anyone can provide any insights.


Adding RTA subs for Tournaments

@adityatMS,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Update sbt-scalafmt to 2.3.4

Updates org.scalameta:sbt-scalafmt from 2.3.2 to 2.3.4.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "sbt-scalafmt" } ]

labels: sbt-plugin-update, semver-patch

Superseded by 0e9514b3b92f6815f0a7d3a178a9cf07475077ef


read_xml/read_html not in rdocumentation?

@Ironholds commented on Sun Jul 02 2017
https://www.rdocumentation.org/packages/xml2/versions/1.1.1 - I can't find either listed. @richierocks any ideas?

The version from 1.0.0 is there, but not 1.1.1.
https://www.rdocumentation.org/packages/xml2/topics/read_xml

This is fixed ! Thanks for raising !


Custom port forwards.

In my environment I have a private cloud where I can spin up new VMs with no public network access nor access to the company git repos.  With the changes in this PR I can run a squid proxy on my workstation and forward that out to each VM for http proxy. I also forward out a remote port for our git server.
These changes work pretty well for me, but I'm curious what you all think.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

Not sure what the CLA thing is about, I signed it long ago: https://supermarket.chef.io/users/causton81

I'm :+1: except for 1 issues - this needs documentation.  Can you update https://github.com/chef/chef-provisioning/blob/master/README.md#machine-options with examples and explanations of all the available keys/values that can be provided to remote_forwards?

This all sounds good to me. I will try to get something in late next week.

This probably needs some log cleanup, but I left it this way to illustrate why I chose this implementation. At least the fog driver creates multiple ssh transports for the same host and never disconnects them explicitly. This causes problems if each new transport tries to forwards ports. The worst problem is a transport that has not yet been garbage collected and its IO loop is not being processed; this looks fine on the remote (the ports are listening) but if you try to use the ports they just hang (IO loop not being processed). Sometimes the port forwards will fail immediately when net-ssh tries to create them; my guess is this is something to do with TIME_WAIT in TCP.
Since there are no guarantees on how ssh transports are used by drivers, the best option seems to be to forward the ports just before executing a command and then immediately canceling the forwards after the command completes.

I forgot to mention that there is a race condition in net-ssh that I am working to fix upstream here: https://github.com/net-ssh/net-ssh/pull/280
May want to hold off on merging this until then.

Have you tested what happens if the user specifies a remote_forward that conflicts with something setup automatically by the make_url_available_to_remote method?  The only time I know we use that is when the user runs a provisioning recipe with chef-client -z and we automatically forward the local chef-zero server to the remote instance.  I'm pretty sure the default port is 8889.
Can you test that and see what happens?  If it is a nasty error we will need to add some checking.
Otherwise I'll wait on your feedback from https://github.com/net-ssh/net-ssh/pull/280

I'll give it a test. net-ssh upstream has moved to 3.x but chef is still pinned to 2.x.  Would chef bump to 3.x or will we need to get my PR backported to net-ssh 2.10?

@causton81 Good question - @lamont-granquist @danielsdeleo @jkeiser do you know any reason we couldn't upgrade to net-ssh 3.x when https://github.com/net-ssh/net-ssh/pull/280 is merged?
The Changelog lists a breaking change for exec!  I can't find any uses of that in the Chef codebase.  Major version change because of dropping of ruby 1.9.

I have some notes on testing net-ssh 3.0.1 with the channel-close patch here: https://gist.github.com/pburkholder/05455d33d4a9f83a54b6

@pburkholder, very cool using tc to demonstrate the issue.

only issue is that probably everyone pins net-ssh with '~> 2.x' so you'll have to make sure to bump all the consumers at the same time or you'll get into gem conflicts.  this repo and chef/chef at the very least will need synchronized bumps.  i'd also advise using '>= 2.0, < 4.0' instead of '~> 3.x' everywhere that you can in order to favor compatibility, unless you really need to drop support for 2.x

:+1: on what Lamont said, though we won't get the benefits of upgrading until everything that gets bundled together upgrades. So that might be an argument for using ~> 3.x and being aggressive about fixing issues we find.

   And I suspect its probably better to pin it in one place (probably here) and then let chef/chef float on both, since that'll get you the pinning you desire while also minimizing the amount of tooling that instantly breaks.

FYI https://github.com/net-ssh/net-ssh/tree/2-9-release

@causton81 Thanks!  It looks like the fix got backported to the 2.9 release family - we should be able to take 2.9.4

Yeah, looking forward to 2.9.4 getting released. I guess the log statements need to be cleaned up if you are okay with the rest of the logic?

@causton81 We have a build of the ChefDK with only net-ssh 3.0.2 in it.  Would you mind installing with the instructions here and see if that fixes your race condition?

Sure, I'll give it a try tomorrow hopefully :D
Chris Auston
cel:<PHONE_NUMBER>
<EMAIL_ADDRESS>On Tue, Jan 19, 2016 at 2:55 PM, Tyler Ball<EMAIL_ADDRESS>wrote:

@causton81 https://github.com/causton81 We have a build of the ChefDK
with only net-ssh 3.0.2 in it. Would you mind installing with the
instructions here https://github.com/chef/chef-dk#pre-release-candidates
and see if that fixes your race condition?
—
Reply to this email directly or view it on GitHub
https://github.com/chef/chef-provisioning/pull/445#issuecomment-172983940
.

Does the current version of ChefDK include this PR? It seems not:
ArgumentError: machine_batch[batch 1] (oss-cluster::default line 321) had an error: ArgumentError: invalid option(s): remote_forwards
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/net-ssh-3.0.2/lib/net/ssh.rb:197:in `start'

For me the race condition was most obvious using the remote port forwards setup by this PR.

@tyler-ball I installed the chef-dk build you linked and updated chef-provisioning to this PR. It worked very well for me.


Could not establish a valid connection

After entering the question in the dialog box, the following error always appears：
Error: Error communicating with OpenAI: HTTPSConnectionPool(host='api.devchat.ai', port=443): Max retries exceeded with url: /v1/chat/completions (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1129)')))

Can you also copy the log output of devchat after vscode is started? Also, do you copy the log output of devchat after clicking the send message button?
If you click the regenerate button and refresh multiple times, will it be successful?

已经解决


[Flowbroker] change node: size of the input field is smaller than the field size

Test OK.


Additional telemetry data for various tutorial ticks

tutorial.start will have { activity: ["tutorial"|"recipe"|"example"], path: "/tutorials/chase-the-pizza", editor: ["blocksprj"|"tsprj"|"pyprj"] }
tutorial.shared has the script id under "tutorial"
tutorial.faileddecompile (not modified in this PR) has the tutorial name/path under "tutorialId"

@abchatra do we want the "tutorial id" dimension to have the same key in all of our ticks? not sure if it matters

It is easier to query something like following if they are under the same cusotmdimension such as tutorialID.  You can do it otherwise too. But will have to add bunch or or clauses
where  customdimensions.tutorialid == "/tutorials/chase-the-pizza"
summarize count() by name

if the tick has already existed for a while as "tutorialid", is it better change it to "tutorial" now to unify, or leave it?

Change it. We are the only one who query for this and we can change our queries.


Concatenating 'null' to a string with '+' results in null

When reporting a bug please provide the following information to help reproduce the bug:
Version of OpenRefine used (Google Refine 2.6, OpenRefine2.8, an other distribution?):
OpenRefine 2.8
Steps followed to create the issue:
In an expression editor type the grel:
"string" + null
Current Results:
Output is
null
Expected Results:
All other types (dates, numbers, booleans) are converted to strings in the case of use with the '+' sign:
"string" + 1 -> string1 (or string1.0)
"string" + true -> stringtrue
"string" + [date 2018-01-01T00:00:00Z] -> stringMon Jan 01 00:00:00 EST 2018

In each case this is equivalent to converting the type to a string using the 'toString()' function.  That is:
"string" + 1 -> "string" + toString(1) -> string1
on this basis the correct result for "string" + null is the same as "string" + toString(null) which is stringnull i.e. the null converts to the string "null"

This behaviour is controlled in /main/src/com/google/refine/grel/ast/OperatorCallExpr.java in the evaluate method

Updated to correct the expected behaviour as I had previously mis-represented the outcome of toString(null)

Note that @thadguidry describes this behaviour as correct in #820

@ostephens Thad describes it as The Most Preferred Behavior By The Majority Our Users :)

OK - closing this then as it is preferred behaviour

I find it hard to believe that the majority of users think that the described behavior is "most preferred."
The most logical results, to my mind, are:

"string" + null == "string"
"string" + null => error

What advantage do "The Majority of Our Users" find to "string" + null == null?

@tfmorris Tom, sorry, I should have been more clear.
Context is king, so let's agree there are 2 contexts here being discussed previously that we didn't perhaps make more clear:

+ as an arithmetic Addition operator
+ as a unary operator (allowed currently in our code for concatenation of a few Types)

I am restating the following to be more clear, in regards to using the unary operator for String concatenation needs:

It's useful for users to have a "smart default" for frictionless String appending or concatenation of column values that might contain null.
It's useful for some more technical users to have that "smart default" be allowed to modify more easily within a GREL statement where any null found in an argument can be replaced by their choosing.  I don't currently think we have this available to users, but happy to know if it can be done currently in version 3.4.

It's useful for users to have a "smart default" for frictionless String appending or concatenation of column values that might contain null.

My reasoning when I raised this issue initially is that this "smart default" should be that
"string" + null
and
"string" + null.toString()
would give the same result. I still believe this would be the best outcome, and still believe that the best "smart default" behaviour is to convert null to an empty string in both scenarios. I haven't gathered a body of evidence for this so it's based on my own experience and questions I've seen on the mailing list over time.

It's useful for some more technical users to have that "smart default" be allowed to modify more easily within a GREL statement where any null found in an argument can be replaced by their choosing.
I feel that we support this through various methods that more technical users can happily use including coalesce() and if in combination with isNull

Thad - to be extra clear, what do you believe should be the outcome of
"string" + null

@ostephens The "smart default" should be:
"string" + null == "string"
David and I had this conversation years ago.
I just always felt we also needed extra null handling controls separate from "" empty string, but it didn't get added into GREL at the time.  Which we offer more granular control now, thanks to all of you.


Add autoGamma, autoLevel and gaussianBlur to MagickImage.

gaussianBlur(radius: number, channels: Channels): void; was omitted as there is no way to distinguish it from gaussianBlur(radius: number, sigma: number): void;

And you also got some linting issues. Please update the individual commits or create separate PR's.

Oops! I accidentally had ESLint disabled in my IDE. All good now!


Not able to upload video for story or DM

I have taken this video into my local as test.mp4 https://www.youtube.com/watch?v=EsceiAe1B6w
i encoded video to mp4 [h264, AAC] from onlinevideo Converter https://video.online-convert.com/convert-to-mp4
const path = './test.mp4';
const file = await Bluebird.fromCallback<Buffer>(cb => readFile(path, cb));
async function uploadStoryWithMedia(file: Buffer) {
return await ig.publish.story({
    file,
    video: {
        video: Buffer,
        coverImage :Buffer
    }
    });
 }
 await uploadStoryWithMedia(file);

getting error like this :
(node:65599) UnhandledPromiseRejectionWarning: TypeError: unsupported file type:
undefined
(file: undefined)
at lookup (/Users/ram/Documents/workspace/jbm/node/instagram-private-
api/node_modules/image-size/lib/index.js:34:9)
at module.exports (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/node_modules/image-size/lib/index.js:96:12)
at PublishService. (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:319:46)
at step (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:57:23)
at Object.next (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:38:53)
at /Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:32:71
at new Promise ()
at __awaiter (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:28:12)
at PublishService.uploadAndConfigureStoryPhoto (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:313:16)
at uploadAndConfigure (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:390:41)
at PublishService. (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:481:46)
at step (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:57:23)
at Object.next (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:38:53)
at /Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:32:71
at new Promise ()
at __awaiter (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:28:12)
at PublishService.story (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/src/services/publish.service.js:375:16)
at /Users/ram/Documents/workspace/jbm/node/instagram-private-api/examples/story-upload.example.js:279:57
at step (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/examples/story-upload.example.js:44:23)
at Object.next (/Users/ram/Documents/workspace/jbm/node/instagram-private-api/examples/story-upload.example.js:25:53)
at /Users/ram/Documents/workspace/jbm/node/instagram-private-api/examples/story-upload.example.js:19:71
at new Promise ()
(node:65599) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:65599) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Can u pls help me

You are passing a video file to the photo-file and two constructors as video input.
As for DM's, the video-upload isn't stable enough for long videos as theres no transcode delay (don't worry, I'll add it).
Your code should probably look like this:
// typescript, remove types for js
const video: Buffer = await readFile(VIDEO_FILE); // *.mp4
const cover: Buffer = await readFile(COVER_FILE); // *.jpg

// uploading to the story
const resStory = await ig.publish.story({
  video,
  coverImage: coverImage,
});

// sending as DM-Story - not stable yet
const thread = ig.entity.directThread(threadId | userIds);
const resDm = await thread.broadcastVideo({
  video: VIDEO_FILE,
});

Thanks it is working fine , for now what can be the longest video that i can Send in DM and Story

In the story you can only send 15s clips as it's allowed by instagram and there's no library to split videos in the browser (if theres any lmk). For dms I haven't tested it; currently due to no transcode delay you con only upload short (depends on video bitrate/size, internet speed, host speed Andrew others) videos. With the transcode delay you can upload probably up to one minute like in the timeline.


AWS path assumptions too strict

The AWS cli can't be assumed to live in /usr/local/bin, e.g. if installed with pip3 --user.
I think using which aws would be more general.
PR incoming.

Fixed. I think. I will reopen if lingering issues.


Translations update from Hosted Weblate

Translations update from Hosted Weblate for Openscape/iOS app.
Current translation status:

This is a stale PR that is inexplicably deleting lots of strings across many translations.


Question on clamp

Hi, I have a question on clap weights
https://github.com/ShichenLiu/CondenseNet/blob/master/layers.py#L125
weight = weight.sum(0).clamp(min=1e-6).sqrt()

I don't understand the clamp function here. I tried to train condensenet-86 on cifar10 . with and without clamp functions
with clamp:  error rate = 95.06
without clamp: error rate = 94.96
Thanks in advance

Hi,
The clamp function here is to prevent numerical issues. Since the derivative of x^p is p*x^(p-1), when p < 1, it is essential to prevent x^(p-1) to be too big which introduces numerical unstable issues.

@ShichenLiu Oh! I got it, thanks a lot


SI: Fix Roles and Permissions

Describe the bug
Roles/Permissions – This feature does not appear to work. The module does not save the changes.
Software Versions

DNN: 09.06.02 (Evoq Content)
Module: 01.00.01

To Reproduce
Steps to reproduce the behavior:

Open Roles/Permissions
Make a change
Click Edit
Nothing happens

Expected behavior
The changes are saved.
Actual Behavior
Changes are not saved.
Screenshots

Additional context
The permissions grid was supposed to be read-only in this release.  Adding the ability to edit this feature is a potential enhancement for a later release.

Nothing been implement for saving permissions.
Require development on API & Client App Services for saving permissions.

@erw13n Please re-read the issue.  I think you missed the part after the screenshot.

Commit been made for it to disable "Edit" button.

We need a pull request instead. ☹

Not only should the edit button be disabled, but so should the checkboxes.  Everything should be read-only for permissions.

PR #36 only for disable Edit button


Fix to wire of wire

This fixes the wire and graphql representations of parties transactions and the conversions between them

Heads up: I fixed the JS side of things here https://github.com/MinaProtocol/mina/pull/11057/commits/499e97a88aa1834dc63c7b4564ea39ba2b681e51
I also think that, now that Party.Graphql_repr.t / Party.Body.Graphql_repr.t are the proper graphql / JS representations of this type, we should remove the original derivers on Party, Party.Body, to avoid confusion and ambiguity about how to serialize those types. I went ahead and made this change here: https://github.com/MinaProtocol/mina/pull/11057/commits/668499f4cbaa6b2f91fada9c1b52fc50c22ae95d -- feel free to revert that last commit if you didn't want this

I fixed the replayer build here. There's a remaining issue with the linters because of the use of deriving quickcheck. When #11068 is merged, and develop merged into this branch, that should fix the issue.


Updated Kubevious

Updated Kubevious manifest to include the latest released version, logo, install script and post-install description.

Thank you for wanting to submit a Pull Request to the Civo Kubernetes Marketplace repository!
If your pull request is to submit a new application to the marketplace, please answer the following questions:

[ ] Have you followed the guidelines in our Contributing document in setting up the application?
[ ] Have you checked that the application your submission is proposing to add does not yet exist on the Marketplace, or as a pull request to be processed?
[ ] Have you tested your application on the Civo managed Kubernetes service? Please include a screenshot.
[ ] Can you confirm that the software you are submitting licensed for use in a publicly-available Kubernetes marketplace?

If your pull request concerns an existing Marketplace application, please make sure you have:

[x] Notified the Maintainer of the application in this pull request so that they are aware of your proposal.
[x] Outlined the changes you are proposing, and the reasons these are required (e.g. stability, compatibility with new versions of Kubernetes implementations, etc).
[x] Tested that the application works with the proposed updates applied (including a screenshot)

Thank you for the PR @rubenhak , Merging it now, let's collaborate on a blog and potential webinar after that.


Fixing docker-compose file to handle correct linking for logging

Since Postgres is part of the compose file I feel these should be uncommented. Also, there is an error that a lot of people are getting when bringing this up with the Celery executor.
This is what people would see in the logs for a run:
*** Log file isn't local.
***** Fetching here: http://:8793/log/tutl1/print_date1/2017-04-03T09:27:24**
*** Failed to fetch log file from worker.
*** Reading remote logs...
*** Unsupported remote log location.
The webserver is not able to fetch from the worker since they are not linked and thus there is no DNS available for the worker. This PR fixes this issue.

The postgres creds are in the entrypoint. So I don't think you need them in the compose file

Hi It's not fixing the issue and like @JackStat said, default credentials are set in entrypoint.sh


⚠ Samsung has degraded performance

In 42ed18c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2457 ms

Resolved: Samsung performance has improved in 054008f.


race condition in RequestHandler?

I believe there to be a possible race condition in RequestHandler which could cause lockage during GC of the object. Specificically, if the RequestHandler object is cleaned up before the final task is marked as done here, then the thread will block here. A possible manifestation of this issue is sometimes occurring on creation of a client.

Broker is created here and sets up a request shortly thereafter.
That broker soon goes out of scope, so the only reference
to its handler is the weakref in the request thread.
The race condition occurs when a ReferenceError is thrown before or when the task is marked as done, meaning that the
join call here blocks forever.

The issue is difficult to reproduce, but I've seen it occur a number of times in our testing. I'm not really sure how one would go about fixing it, but if anyone can come up with a suggestion, I would be happy to implement and test.

Thanks for reporting this. Just out of curiosity: do you see both the logging messages about the ReferenceError and about entering stop() appear in your logs?
I coded up an alternative version of the thread worker while I was working on #258, which didn't have any weakref in it, but I decided against it because it changed more code. I'll go find it and rebase it - if you could test if that works better for you that would be great.

Just out of curiosity: do you see both the logging messages about the ReferenceError and about entering stop() appear in your logs?

Yes. Including the logs below with comments. (I added extra logging which showed the bug more explicitly)
MainThread:pykafka.cluster 2015-10-20 14:35:39,587 after broker; broker=<pykafka.broker.Broker at 0x7fd0190c15f0 (host=localhost, port=6667, id=-1)>

Broker is created here.
Thread-1:pykafka.handlers 2015-10-20 14:35:39,589 start worker
MainThread:pykafka.cluster 2015-10-20 14:35:39,595 after broker.request_metadata; response=<pykafka.protocol.MetadataResponse object at 0x7fd0190ce110>

Request is made, worker is started, future is responded to and response received.
MainThread:pykafka.handlers 2015-10-20 14:35:39,600 __del__; self=<pykafka.handlers.RequestHandler object at 0x7fd0190beb90>

Broker goes out of scope, GC begins to clean up the handler.
MainThread:pykafka.handlers 2015-10-20 14:35:39,602 before flush; _requests=<Queue.Queue instance at 0x7fd0190c1758>
Thread-1:pykafka.handlers 2015-10-20 14:35:39,601 ReferenceError in handler - dead weakref
Traceback (most recent call last):
  File "/workspace/pykafka/pykafka/handlers.py", line 158, in worker
    self._requests.task_done()
ReferenceError: weakly-referenced object no longer exists
Thread-1:pykafka.handlers 2015-10-20 14:35:39,619 RequestHandler worker: exiting cleanly

The stop() method tries to join the requests, but just before this can occur, the thread encounters the ReferenceError, and cannot mark the task as done, so the join blocks indefinitely.

I'll go find it and rebase it - if you could test if that works better for you that would be great.

Would be happy to.

@blooshirt thanks very much for the additional info, it helped me a lot to better understand the issue. I think #310 should address the problem, but like you said, it's not easy to reproduce and in fact I've not been able to come up with a suitable test to add to the test suite.

@blooshirt oops #310 still contains some sillyness, will push a fix.

@blooshirt fixed that now, sorry for the noise.

@yungchin that seems to have fixed the issue - thanks. I can reliably reproduce the race condition if i add a short sleep before this line. Doing the same with the code in #310 doesn't cause the issue.

@yungchin as an aside, am I right in saying that this issue could be a possible cause of #260?

@blooshirt thanks for testing this right away!!
I believe this race condition was newly introduced with #258, and #260 was around before #258 got merged. The problem with #260 is that so far it has seemed impossible to reproduce :(


KeyError in model name lookup

When retrieving a model, I'm getting a KeyError:
Retrieving model...
0it [00:00, ?it/s]
Traceback (most recent call last):
  File "/Users/gneubig/work/prompt2model/cli_demo.py", line 435, in <module>
    main()
  File "/Users/gneubig/work/prompt2model/cli_demo.py", line 168, in main
    top_model_name = retriever.retrieve(prompt_spec)
  File "/Users/gneubig/work/prompt2model/prompt2model/model_retriever/description_based_retriever.py", line 277, in retrieve
    model_info = model_name_to_model_info[model_name]
KeyError: 'GuiSales404/e10_lr0.0001'

Sorry, this was a data error on my side. Closing.


iOS embedding should not assume resource context presence means app is in foreground

The image decoding logic on iOS currently checks for a resource context to determine whether the app is in the foreground, but this is not really safe - the resource context survives being backgrounded, and must do so to avoid dropping live references in Dart land to images/fonts.
https://github.com/flutter/engine/blob/master/lib/ui/painting/multi_frame_codec.cc#L118 should be updated to make sure the work either finishes before going to background or can't happen when in the background. This can lead to crashes when decoding an image and going to the background, which is more likely to happen with multiframe images.
/cc @chinmaygarde @amirh @ponycui

Same issue, any progress now?

@gaaclarke has been working on this more recently.  There's a PR with some discussion at https://github.com/flutter/engine/pull/13454

This is fixed by https://github.com/flutter/engine/pull/13908 and landed in the framework as of 5fb790e


[Question] Permission denied error on slurm based systems

I tried to run deepspeed on a slurm based node (compute canada). But it always throws a "Permission denied" error whenever "deepspeed" command (in bash) is used. However, I can successfully run the "python -m deepspeed.env_report" command and get the correct report. Could you please help me find out why this error occurs? Thanks a lot.

I guess you need to use chmod. At least for me, that solved the problem.

Hi @PythaGorilla, are you able to launch processes with python -m torch.distributed.launch from https://pytorch.org/docs/stable/distributed.html

yes, i can run it without problem.

I guess you need to use chmod. At least for me, that solved the problem.
Thanks, @yyy-Apple. Could you elaborate on what file or folder to chmod?

You should make path/to/bin/deepseed executable

@PythaGorilla You should make path/to/bin/deepspeed executable.

@yyy-Apple  Thanks, it worked!


Update bottom emailx banner stylings

Updated screenshot with appropriate changes.

 Make Email X placements for bottom NL banner


is there monitor bug in mxnet code?

I just test on simple network as below,using monitor,
def gen_caffe_net():
input=mx.symbol.Variable('data')
ip1=mx.symbol.FullyConnected(data=input, name="ip1", num_hidden=200)
relu1=mx.symbol.Activation(data=ip1, act_type="relu", name="relu1")
ip2=mx.symbol.FullyConnected(data=relu1, name="ip2", num_hidden=100)
relu2=mx.symbol.Activation(data=ip2, act_type="relu", name="relu2")
ip3=mx.symbol.FullyConnected(data=relu2, name="ip3", num_hidden=16)
relu3=mx.symbol.Activation(data=ip3, act_type="relu", name="relu3")
ip5=mx.symbol.FullyConnected(data=relu3, name="ip5", num_hidden=2)
net=mx.symbol.SoftmaxOutput(data=ip5, name="softmax")
print(net.debug_str())
return net
And from monitor ,the output logs as blow:
2017-11-16 18:18:44,431 Node[0] Batch:       1 ip3_output                     [[-0.02331318 -0.01718092 -0.029297    0.01212043  0.          0.03418904
-0.06801873  0.         -0.0403567   0.08026087  0.01712877  0.05154562
0.         -0.03878605  0.04980191  0.        ]]
2017-11-16 18:18:44,431 Node[0] Batch:       1 relu3_output                   [[ 0.11219609  0.          0.10197746  0.09196122  0.15120639  0.24690789
0.          0.          0.          0.10708041  0.04835813  0.
0.15631352  0.3170726   0.          0.31656569]]
relu3_output is not expected. Why?

@liumilan i don't completely understand the question. Can you please expand on what is the problem you are trying to solve, and what is not working as expected.

Closing due to inactivity, please reopen if issue still persist.


Stuck at "Enabling gpedit.msc...Group Policy for Home Users"

Describe the bug
While running the recommended set of tweaks for "Laptop", I can see in the Power Shell terminal window, that the flow has stucked while running the step "Enabling gpedit.msc...Group Policy for Home Users"
Laptop is ASUS VivoBook S400C running Windows 10 Home 21H1 build 19043.1889 with all the windows updates applied.
To Reproduce
Steps to reproduce the behavior:

Open powershell as administrator
Run "iwr -useb https://christitus.com/win | iex" command in it
In a GUI window switch to "Tweaks" tab, click on "Laptop" button and then click on "Run tweaks" button
After a while in a powershell console the message "Enabling gpedit.msc...Group Policy for Home Users" appears without progressing any further and the GUI window is not responding. Waited for about an hour and it's still hanging

In the Task Manager I can see that "Windows Modules Installer Worker", "Antimalware Service Executable" and some windows update process are actively spinning on CPU
Expected behavior
There should be some kind of timeout detection, if some action failed to be performed, so it should be skipped while proceeding other queued actions

Closing it ourself. It just took insane long 3 hours or so


feat(upgrade): support for Angular9

Add support for Angular 9

replaced custom scaffolding with scaffolding provided by @angular/cli 9.1.1
used 'ng generate project' to create angular-crumbs library
updated version number to 9 to match latest @angular version
files under "src" folder have not been changed.

To build the project:
ng build
Publish from \dist\angular-crumbs

Hi there, any plans to update this library to Angular 11/12?
Thanks!

Hi! @emilol, thanks for make this library, and please integrate this PR, thanks


How to send an image in notifications

Hi,
How we can send an image in notifications for this package.
Please share the solution if you have.

I added this and added all forks - https://gitlab.com/akumria/findforks
searching through the repo / branches (can't do in github) - yields no relevant results.


Crashing on macOS 10.14 Mojave

Prerequisites

[x ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Saving a new file crashes Atom. At the moment I am only able to work by copy and pasting a existing script and renaming it.
Steps to Reproduce

Ctrl + N to create new file
Ctrl + S to save new file

Expected behavior:

Ctrl + N to create new file
Ctrl + S to save new file
(Window asking what to name new file)

Actual behavior:

(crashes)

Reproduces how often:
100 %
Versions
MacOS Majave
10.14.6
Atom    : 1.41.0
Electron: 4.2.7
Chrome  : 69.0.3497.128
Node    : 10.11.0

Did you try reproducing this in safe mode?

@darangi, yes I did follow the prerequisite.

Update: it seems to be an internal systems network issue accessing my account at my place of work. The IT department here was able to fix the issue. Closing issue.


Updated play2.6 branch to 2.6.0 (final) + migrated and fixed example branch

All references to M5 or RC2 versions are now to the release version of Play 2.6.0
Pulled and merged the newest version of the master
Pulled the windows test fix  #166
Migrated the example project using the 2.6.x migration guide (https://www.playframework.com/documentation/2.6.x/Migration26)
Fixed all the unit tests
Should fix #162

Coverage remained the same at 89.31% when pulling 8344c1d86566277c3e945fe97d9b43a8aa16391c on wwbakker:play2.6 into 880174282a5c0bd753554c7b490f83240bda29c0 on iheartradio:play2.6.

Coverage remained the same at 89.31% when pulling a153c6a0734410daa4da4860be603c1c62807bb7 on wwbakker:play2.6 into 880174282a5c0bd753554c7b490f83240bda29c0 on iheartradio:play2.6.

0.6.1-PLAY2.6 released.

Thank you! @kailuowang


MMWormhole

MMWormhole - Message passing between iOS apps and extensions.

@mutualmobile on what category do you think we can add?

This one is quite specific. Let's move it to Utility.

Fixed by f84d2a6


[$1000] Update IOU Preview Card Styles

Note: This issue is part of the greater Implement Our New Branding in NewDot project.
What is it:
Small design updates to the IOU request line and preview card to match our new brand ingredients. Note that we’ll save any major IOU report view changes for N7.

Does it require a mini design doc:
No, minor style changes
Design finished:
Yes
cc @GabiHExpensify
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0117247f0ce11c22f0
Upwork Job ID:<PHONE_NUMBER>748538368

@grgia What do next steps look like from here?

I don't think this one should need a Design doc, it should just be a pretty straightforward pull request. If we use our existing color variables, then we'll get the new branding ingredients added here for free when that project completes.

@shawnborton  could you link the figma here?

Yup! Here ya go.

@grgia in terms of implementation, I left a comment over in @dangrous's PR for workspace editor illustrations about making a new entry into our theme file for the cardBG color. I think we can hopefully share that style between these two items.

I started this but stopped to implement core branding changes. Planning on coming back to this one once I get the colors/logo PRs completed today/tomorrow!

Update, I started working on this again on Friday- I'll get a WIP PR up sometime today to track progress

Do we have a mockup that includes the IOU on the desktop/dark theme?

Chatted in Figma but here they are again for posterity:

Closing as completed!

Reopening for PR Review Payment. @puneetlath could you help with that?

Yep. @parasharrajat sent you a hiring offer.

Paid.


Trying to convert code block with python syntax highlighting

I am try to convert the following code (sample function, replacing back-ticks with apostrophes )
Markdown
''' python
def sample(args):
    return args[0]
'''

( Other docs here )

to Rst
... code-block:: python
    def sample(args):
        return args[0]

(Other docs here)

But the rst looks like this
rst
```python
    def sample(args):
        return args[0]

... code-block:: python
    (Other docs here)

Do you know what is going on? I looked into your code and saw that there is the function block_code that helps convert a markdown code block to a rst code block, and I see that m2r2 supports shell or bash syntax highlighting, but for some reason it's not working with python. Do you know what is going on?

Actually I found something funny that happens
So with
Markdown
''' python
def sample(args):
    return args[0]
'''

( Other docs here )

If I had some text above the code block like below
Markdown
(some text here)
''' python
def sample(args):
    return args[0]
'''

Then I get the error I showed
rst
(some text here)
```python
def sample(args):
     return args[0]

... code-block:: python
    (Other docs here)

But if I add a space:
Markdown
(some text here)

''' python
def sample(args):
    return args[0]
'''

Then I get the correct output
rst
(some text here)

... code-block:: python
    def sample(args):
        return args[0]

    (Other docs here)

So I guess m2r2 does support python syntax highlighting however there is this bug with the space. Is this something that should be fixed?

I'll be looking into this next

@elinxie I jush pushed some tests:

https://github.com/CrossNox/m2r2/commit/4932b8c6e875af89a8089dbf07f885158967dc7e
https://github.com/CrossNox/m2r2/commit/bc65f0b0a642ab460957dd904daffa2b6c170941

Or you can see the full diff here
Those tests are passing right now, so I'm not sure I have quite understood your problem... would you be able to provide a failing test?