Worker schedules

Another feature that would be nice to have in Procrastinate is to allow workers to process jobs only in specific time slots or on specific days (e.g., only on the weekend).
Here's some background on why this feature is quite important for us: We transfer large radiological image data in Procrastinate jobs for research purposes in parallel. As we don't want to stress the clinical infrastructure, we want to run some workers only at night or on the weekends.
I thought about maybe a command-line option like --schedule with a Cron-like argument, e.g. procrastinate --app=dotted.path.to.app worker --schedule "* 1-5 * * *" to process jobs at night only between 1 and 5 o'clock (croniter can be used to check that with the match method).
One thing to keep in mind in such a scenario is how to handle periodic tasks by such a scheduled worker (especially when those have other schedules than the worker itself).
It's nothing that I would be working on right away, but in the longer term (and also after #933 is done).

I wonder if this should be part of procrastinate itself. I could see multiple ways of achieving this with what's avaliable already, I'll try to take the time to expand on this later, but if it appears this is already doable without too much hacking, it's worth discussing whether we want to add it or not.
This is not a "no", it's really just a "I need to be convinced we can't easily do it yet"

Sure, no hurry. I also thought about handling this in the user space with task scheduling. But it's not guaranteed that the task is picked up in some specific time slot. So, there must also be a custom retry strategy. This alone sounds more complicated to me. I also dislike that this way, a job (in a worst-case scenario) may be picked up several times just to be delayed. Then there is still the scenario of how to ensure that, for example, only three jobs of that task are processed at daytime and eight at nighttime. I don't see a solution for that right now (even with locking). And on the other side it's not that hard to implement on the worker side (especially as we already have croniter as a dependency).

This is blind code (not on my phone this time, but still didn't execute it)
from __future__ import annotations

import asyncio
import dataclasses
import datetime

import procrastinate

@dataclasses.dataclass
class ScheduleItem:
    start_time: datetime.time
    queues: list[str]

def get_queues_at_time(
    schedule: list[ScheduleItem], dt: datetime.datetime
) -> list[str]:
    time = dt.time()
    for item in schedule:
        if item.start_time <= time:
            return item.queues
    return []

def get_next_schedule_item(
    schedule: list[ScheduleItem], dt: datetime.datetime
) -> tuple[datetime.datetime, list[str]]:
    time = dt.time()
    for item in schedule:
        if item.start_time > time:
            return datetime.datetime.combine(dt.date(), item.start_time), item.queues

    item = schedule[0]
    return datetime.datetime.combine(
        dt.date() + datetime.timedelta(days=1), item.start_time
    ), item.queues

async def run_worker_with_schedule(
    app: procrastinate.App,
    schedule: list[ScheduleItem],
):
    current_queues = get_queues_at_time(schedule=schedule, dt=datetime.datetime.now())
    while True:
        next_run, next_queues = get_next_schedule_item(
            schedule=schedule, dt=datetime.datetime.now()
        )
        await asyncio.wait_for(
            app.run_worker_async(queues=current_queues),
            timeout=(next_run - datetime.datetime.now()).total_seconds(),
        )
        current_queues = next_queues

if __name__ == "__main__":
    import logging

    logging.basicConfig(level=logging.DEBUG)

    app = procrastinate.App()
    app.add_queue("default")
    app.add_queue("low")

    schedule = [
        ScheduleItem(start_time=datetime.time(8, 0), queues=["default"]),
        ScheduleItem(start_time=datetime.time(12, 0), queues=["low"]),
        ScheduleItem(start_time=datetime.time(16, 0), queues=["default"]),
    ]

    asyncio.run(run_worker_with_schedule(app=app, schedule=schedule))

Another possibility would be to have the specific jobs you want use queues listened on by dedicated workers with autoscaling rules that switch those workers on and off at specific time. No worker, no task.
https://docs.aws.amazon.com/autoscaling/ec2/userguide/ec2-auto-scaling-scheduled-scaling.html for AWS for example, I guess you'll find similar concepts everywhere.

Scheduled scaling can be problematic in my situation, as I don't want a long-running task to be terminated. I just want to ensure that a task is only started in a specific time slot (and then it should run as long as it takes). I guess there seems to be a similar problem with the above code (unfortunately, I don't fully get the scheduling there 😞), as there is some explicit timeout, too.
What I am looking for is to tell a worker an explicit date/time range when it should fetch the next job and outside this date/time range should just wait. But maybe this is too application specific 😪.

No, the timeout in the code above doesn't actually stop the worker. After the timeout, we tell the worker to stop, and it will stop gracefully (don't accept new task, wait until remaining tasks end, stop). As long as we don't cancel again while it's cancelling, I'm pretty sure it won't stop a running task.
Regarding the code note being clear: sorry about that. I'll explicit it here.
I don't know how familiar you are with asyncio, so I'll detail things here. Sorry if this too much of an asyncio 101, please don't take it as an assumption that I know better 😅. It did take me a long time to form a correct mental model of the things here, and I'm still learning a lot. Everything I'm saying should match what you can find in the docs.
Asyncio tasks
So you can create a coroutine with async def, and await it with await, this part is usually the one that is the most visible and the most well understood. But if we want to achieve the promise of asyncio, namely concurrency, we're missing an important primitive: the ability to create asyncio tasks. asyncio tasks are a way to declare you want a coroutine to run in the event loop but you don't want the calling code to be stuck on the same line while waiting for this coroutine to end (like what would happen if you awaited the coroutine).
This means you might create multiple tasks that will do things concurrently, and any time any of the tasks or your main code would await something, you'd give back control to the event loop that would run a part of one of the tasks (or the running code).
Note that you may await a task: in this case, it mean you really want to wait until the underlying coroutine finishes.
asyncio provides synchronization primitives such as asyncio.gather() when you need to wait for a bunch of tasks to end.
Timeouts and cancellation
One very useful thing you can do with coroutines and tasks is asyncio.wait_for: this lets you wait for a coroutine, but if it lasts more than a certain time, it's cancelled.
Cancelling a coroutine means injecting a CancelledError: the next time it awakes from await, instead of executing the following statement, it will raise.
The idea of cancellation is that it's supposed to be something you're you're able to handle gracefully. Having an exception be raised from anywhere in your code would be hard to handle: you'd need to try/except every single await statement, and it would be very tedious, or you'd need to put a single try/except at the top of your coroutine, but by the time you've raised up that point, you've completely lost your context, so it defeats the purpose of gracefully doing anything.
Fortunately, there's a mechanism for handling this: shielding. By calling asyncio.shield() on a coroutine, you're wrapping your coroutine in another coroutine that will receive CancelledError when cancelled but not pass it below. This way, the code that calls shield may handle the CancelledError by signalling to the underlying coroutine (e.g. using synchronisation mechanisms such as Events) that it needs to stop ASAP
Procrastinate
That's all nice, what does it have to do with anything?
It turns out that 1/ the worker has a builtin way of signalling it's expected to stop gracefully asap: worker.stop. Also, app.run_worker_async() has code in place so that cancelling it once calls .stop() and then waits for the worker to stop.
Those 2 elements means that if you call asyncio.wait_for(app.run_worker_async, timeout), you'll run the worker for timeout, then request it to stop, and wait for it to actually stop, you'll have a graceful stop after the given timeout (and no actually timeout on the stopping time).
The script
Last element of the puzzle: what does the rest of the script do?

Lets you define what queues should be listened to at what hour of the day
When the script start, will figure out what queues should currently be listened to
Then figures out how long before the next time we change the queues
Run the worker for that long (thus the timeout)
Wait for remaining tasks to end (I edited the script, I was missing that "await")
Then figure out how long we should be running the worker again, with what queues, and so on.

I'm pretty sure there should be a smarter way to express the schedule, but the important part of the script is the idea of running the worker for a given time, stopping it gracefully, and rerunning with a different list of queues. that part is achieved just a few lines, and doesn't scream hackish to me.

Oh, that makes so much more sense now. Thanks so much. You are the best 🙂. Especially, I wasn't aware of shielding. Of course, the scheduling could also be used with date/time ranges and without any extra queues. Also, this example nicely demonstrates the strengths of asyncio. I will close the issue as I am convinced now that this is the more flexible option. I will post my implementation here sometime when it's done.


NTR: glycine/ proton antiporter activity

Please provide as much information as you can:

Suggested term label:
glycine/ proton antiporter activity

Definition (free text)
Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: glycine(out) + n H(+)(in) = glycine(in) + n H(+)(out).

Reference, in format PMID:#######
PubMed=26912364 and PubMed=16701208

Gene product name and ID to be annotated to this term

Parent term(s)
antiporter activity GO:0015297

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

Any other information
Evidence:PubMed=26912364, see Fig. S11.
Evidence:PubMed=16701208, see Fig. 5.
The transport is equally dependent on the chemical and electrical components of the proton gradient

+[Term]
+id: GO:0140799
+name: glycine:proton antiporter activity
+namespace: molecular_function
+def: "Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: glycine(out) + n H(+)(in) = glycine(in) + n H(+)(out)."


Purpose of emmet.syntaxProfiles is unclear

VSCode Version: Code - Insiders 1.14.0-insider (15b579d13dbf6817bb493feff6f36cec87577929, 2017-06-27T05:11:33.154Z)
OS Version: Linux x64 4.10.0-24-generic
Extensions:

Extension
Author (truncated)
Version

EditorConfig
Edi
0.9.3

lorem-ipsum
Tyr
1.0.0

sort-lines
Tyr
1.3.0

vscode-svgviewer
css
1.4.2

tslint
eg2
0.15.0

git-project-manager
fel
1.3.2

md-navigate
jri
0.0.1

vscode-scss
mrm
0.6.2

seti-icons
qin
0.1.3

vim
vsc
0.9.0

(1 theme extensions excluded)

#29500
I set this but it didn't allow me to expand within a .ts file.
"emmet.syntaxProfiles": {
  "typescript": "html"
},

Did you try the Emmet: Expand Abbreviation command or you are expecting the auto completion?

@ramya-rao-a I tried auto completion, if that's not supported maybe there should be a note in the setting description?

@ramya-rao-a it worked via command.

I am still in 2 minds on this.
There is one camp of users who find the suggestions noisy in a non pure markup/css file
Others who find it very useful and dont mind the noise.

@ramya-rao-a I can't imagine actually doing this and finding it useful. There should be a note in the description though to make it clear how to use it for people like me who don't really use the command.

Using syntaxProfiles for mapping a language to one of the emmet supported ones was introduced back in September. See https://code.visualstudio.com/updates/v1_5#_settings-to-enabledisable-emmet-for-languages
syntaxProfiles was already being used for a whole different purpose. Which was to define the output profile as defnied here: https://docs.emmet.io/customization/syntax-profiles/
This was re-used in September to be used for enabling emmet in other languages.
Back in February, I fixed a few issues around this. But the real fix is to separate the output profile settings from the language mapping settings.

Here is my proposal:
In the new emmet model, we will use a new setting emmet.includeLanguages to enable emmet abbreviations in languages that are not supported by default.
emmet.syntaxProfiles will be used only for setting the output profile as defined https://docs.emmet.io/customization/syntax-profiles/ and not to map languages to emmet supported modes
Pros:

Intuitive to new users
No confusion to emmet users coming from other editors as syntaxProfiles means exactly what was documented in the emmet website https://docs.emmet.io/customization/syntax-profiles/
Naming is self-explanatory and is a good opposite to emmet.excludeLanguages
This will only come into affect when emmet.useNewEmmet is true

Cons

Existing VS Code emmet users who have been using emmet.syntaxProfiles to map languages will have to make a change.
There are a few docs and tweets out there that we need to update

cc @egamma @mrmlnc

I found it now emmet.syntaxProfiles & emmet.useNewEmmet: true can not coexist (
For example: I set vue to syntaxProfiles, and then set useNewEmmet, in the vue file emmet can not be used.

@ifzm Are you referring to the Emmet: Expand Abbreviation command or the emmet abbreviations showing up in the suggestion list? The former should work as long as you have the mapping "vue-html": "html". The suggestions will work once they are implemented by language specific extensions. You can follow #29114

Thanks for your answer ).
Yes，i think I may not be clear, the actual problem is this "emmet.syntaxProfiles" not working with "emmet.useNewEmmet" (emmet 2.0)

I like your proposal!

The new settings emmet.includeLanguages is a nice problem.

The new setting emmet.includeLanguages is a nice proposal.

I had a day to think about this and am going ahead with it.
emmet.includeLanguages will be used in the new emmet model to map languages unsupported by emmet to one of the languages that is supported by emmet.
emmet.sytnaxProfiles will continued to be used for the same purpose, but only in the old emmet model
yay!

nice :+1:

@ramya-rao-a is this deployed yet?

In the Insiders, yes.
For the stable build, wait for the release of VS Code 1.14
In both cases ensure "emmet.useNewEmmet" is ser to true

My user settings contained the following:
    "emmet.useNewEmmet": true,
    "emmet.syntaxProfiles": {"handlebars": "html"},
    "emmet.includeLanguages": {"handlebars": "html"},

Emmet wasn't available in my handlebars files :(
Only fix was "emmet.showExpandedAbbreviation": "always"

@Elyx0 That is covered in #28545 Please follow that issue

If anyone else wound up here trying to get html snippets to work in *.blade.php files, the fix is to add them to blade.json instead of html.json from the User Snippets menu.


"Database Mail Profile" check looking for non-existent property?

Line 56 of Agents.Tests.ps1 is looking for a property that does not exist on my servers. Tested against SQL 2012, 2014, and 2016.
(Connect-DbaInstance -SqlInstance $psitem).JobServer.AlertSystem.DatabaseMailProfile is referencing the "DatabaseMailProfile" property of the SMO Microsoft.SqlServer.Management.Smo.Agent.AlertSystem, but that doesn't appear to exist:

Thanks toot!


Fixed the broken link to Gitlab sample

Hi @deepu-james
Thank you for this PR. Please switch to using https://github.com/ShiftLeftSecurity/sast-scan which is the replacement for AppThreat. This repo will be archived soon.


Prevent adding System Content Type "0x" to ListInstance

because it will generate an exception "The site content type has already been added to this list." in ObjectListInstance.ExtractContentTypes.
You can reproduce this by

Adding a Promoted Links list to a team.
Get Template from it
Apply the template to a brand new team site.

Hi @larry-lau, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@larry-lau, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

thx Larry, looks good. We'll test this one closely to avoid issues with list provisioning, but seems like a valid fix. Will be processed in upcoming days.

Hi, We tested this on our issues with Promoted Links lists causing null reference exception and it solved our problem. It appears the System ContentType is not written to the template and now our site is created correctly again.

Hi Larry,
Thanks for your submission. We will include the suggested changes, but we cannot merge this PR because there are quite a lot conflicts. You will be credited, but I will close this PR and redo your changes. Thanks again.


Functions as middleware

In gitlab by @treeder on May 22, 2017, 16:48
This may just be examples/docs, but would be interesting to use Functions in Middleware. For instance, an auth check could call an auth function.

closing for #399 which has a better description


slider for dice number selection

Description
Slider allows admin to select the number of dice they wish to use in the game they are creating.
Examples:
If the user selects 2 dice the first 2 characters of the private key will be selected as the hidden characters
If the user selects 4 dice the first 4 characters of the private key will be selected as the hidden characters
and so on...

Additional Information

[X] I have read the contributing docs (if this is your first contribution)
[X] This is not a duplicate of any existing pull request

Related Issues

N/A

caiocosta.eth

This looks neat to me, requested @Avelous to take the last look before merging.

@ZakGriffith looks good to me. Please go ahead, thank you!!


Add .devcontainer for Visual Studio Code and Github Codespaces

This is a pretty minimal configuration to support devcontainers. It integrates nicely with the existing docker image and hopefully will help new learners to quickstart with the examples.
Fix #94
PTAL @ageron (and thanks a lot for writing the book!)

Just did a minor update to include GPU in the definition (that's why the forced push) afterwards.


Update javafx-base, javafx-graphics, ... to 18.0.1

Updates

org.openjfx:javafx-base
org.openjfx:javafx-graphics
org.openjfx:javafx-media

from 17.0.2 to 18.0.1.
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!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.openjfx" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.openjfx" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #98.


FEATURE REQUEST:  Print running config to stdout or a temp file

I am looking for a language agnostic way to obtain the currently running config of a client. When using the new -P functionality it will compile the configuration that you pass into it. There should be a way to obtain the current configuration hash stored in the settings file.
There are a few ways to do this:

When the client starts up dump it to /tmp/sensu_settings in machine readable json (perference)
cmdline flag that will reach in and pull it out

This will require a minor change to https://github.com/sensu/sensu-settings

@mattyjones I am much more inclined to write the compiled/running configuration to a temp file :+1:

👍

The 0.25 release is going to hit today with fewer features than originally anticipated & primarily only some internal improvements. Moving this issue to 0.26.

@mattyjones Could you check if PR #1352 is acceptable for your use case? you'd have to do this:
curl http://<IP_ADDRESS>:3030/settings > current-config.json

And you would get the client configuration as json

I have renamed this issue to "Print running config to temp file", as a temporary file containing the running Sensu configuration can also be utilized by Sensu plugins (instead of using SENSU_LOADED_FILES).

@mattyjones https://github.com/sensu/sensu/pull/1488 provides /settings with &redacted=false


SwarmUI Node Pack Installing Into Standalone Comfy

Your question
I was wondering if the SwarmUI node pack is able to be installed in a separate standalone comfy,
Primarily the swarm input nodes so that I can work on making workflows specifically to be ran in the swarm ui.
I searched manager for them but couldn't find them.
Logs
No response
Other
No response

They're not in manager but you can easily add your swarm path to any comfy install via the extra model paths file, check your (SwarmUI)/Data/comfy-auto-model.yaml for an autogenerated valid one that links your swarm models and swarm's nodes.


Update test to match api change

Hello Matt,
Apparently the parameter format changed on user-approval-uri, but the tests didn't reflect this.
I noticed this while working on a fork using Aleph/Lamina instead of clj-http in case you might be interested.

Thanks!


High-performance “points inside a set of polygons” logical test

This pull request adds the functionality of high-performance “points inside a set of polygons” logical test: given a set of points and a set of polygons, the test returns a boolean list indicating whether each of the given points is inside or outside the set of polygons. Note that clipper.cpp currently contains a PointInPolygon() function. The new inside() function makes use of PointInPolygon to deal with very large datasets to achieve optimal performance.
inside() allows for logical grouping of the given points to enable short-circuit logical test (if the logic is true for one member of a group, it is true for the entire group). The algorithm has been used on large datasets containing more than 800,000 points and 2,000 polygons with execution time on the order of 10 seconds.
Within __init__.py, there are two modifications:

A new method inside()
A minor modification to the Cell.copy() method: deep copy of elements and labels (required for copying of Cell objects with hierarchical structure)

Within clipper.cpp, there are three modifications:

A new function inside()
A new helper function BoundingBox()
Related documentation

No other files are affected.

I would like to provide additional info with regard to this pull request. Our use of @okianus ’s fork with the proposed c-extension has sped up our ability to assemble large chips by a factor of 250 (from about a day to minutes). Also, in many applications, the use of groupsize and groupstate options provides a much faster alternative to the polygon overlap test using gdspy.boolean. Of note here is that while gdspy.boolean provides mathematically rigorous check for the overlapping shapes, the inside() facility is less general as it would not detect cases when overlapping shapes do not contain any points inside each other. This limitation, however, has not been found to limit our ability to use it.
To the best of my understanding, the inside() function is the only remaining difference between the @okianus  and @heitzmann versions of gdspy. As such, this pull request presents us with a fantastic opportunity to merge the two versions together back under @heitzmann maintenance.

Yes, this is to confirm that currently the only difference between the fork and the main branch of gdspy is the inside() method (earlier modifications to the Cell.copy() method involving layers to include / exclude have been reverted). If this pull request is integrated, there will be no need to maintain a separate fork,  eliminating branch divergence.

Sounds good, I'll have to take a look at the code because of the python 2/3 differences, but I'll get to it asap.

I've included the changes in the master branch, but I'd like to propose a change in the function call. I understand that the reason for having groupsize is to short-circuit the test within each group, right? But that forces all groups to have the same size. What if we changed the call to accept points as an array of coordinates or groups (each group being an array of coordinates)? In that case each group could have any number of points. The result would be a boolean list with length equal to the number of groups.
The signature could be:
inside(points, polygons, short_circuit='any', precision=0.001)
with short_circuit meaning:

'any': returns True if any point in the group is inside one of the polygons
'all': returns True if all points in the group are inside one of the polygons

In the case with no groups (just a list of points), short_circuit is irrelevant and the return list contains the test result for each point.
What do you think?

Almost forgot: the modifications in Cell.copy were included with the addition of a keyword to maintain backwards compatibility.

@heitzmann, thank you for integrating the changes! Your proposed (and implemented!) changes should work perfectly for my use cases, and indeed will provide more flexibility for varying groupsize. I will test the latest commit tomorrow and provide some feedback.

@heitzmann, I have tested your latest commit with our use cases, and all work perfectly. I looked at both any and all options, but did not experiment with varying group sizes.
In case you're interested, here's an example of one of the bigger test cases:

1.4 million points, logically grouped into groups of 4

points are compared against ~2500 polygons with a total of ~125,000 vertices
using grouped data (i.e. with short-circuiting) vs a flat list of points (i.e. no short-circuiting) for the same data set improves the performance by more than 40%.

Thank you for your quick action on this!

Good to hear everything worked. Those numbers are way beyond what we usually consider in academia! Very impressive...
I'm closing this PR without merging, since the changes are already there.


MySQL Support?

Hi, I would like to know if there's any support for MySQL?
Thanks!

There is no support for MySQL.
I'm working on a experimental driver but due to NoSQL nature of how cockpit handle data, I'm sceptical it will work out. Event if it will, I there is high possibility there will be performance issues compared to SQLite or MongoDb drivers.

Any updates for using mysql with cockpit please?

Is there any update on the DB solution?
I'm wondering what is possible for a DB solution since i now don't use a DB i guess. @piotr-cz

Last time I checked the show stopper was lack of adapter for PDO::sqliteCreateFunction for non - NoSQL drivers such as MySQL (see this comment).
Now it's being used only in lib/MongoLite/Database.php).
I guess it could be emulated by creating a query that uses CREATE PROCEDURE or CREATE FUNCTION.
This is where I've got with my research.

I'm working on MySQL driver for cockpit.
So far it seems to work fine, but it uses JSON functions so MySQL v5.7.9+ is required and some collection filters such as $func and $fuzzy probably won't be available.
In development environment with small collections it's ~5x slower than SQLite driver
repo: https://github.com/piotr-cz/cockpit/tree/feature/mysql-driver

I've decided to transform feature into SQL Driver addon for Cockpit.
Performance was slower due to large dev database.


feat: compatibility with symfony 6 and php 8

Removed conflict with Symfony 6 and PHP 8.
I had to up the PHP version to 7.1 because of PHPUnit forcing void as return type on setUp and tearDown.
Also fixed PHPUnit tests failing using symfony/filesystem:2.7 (lowest).

It would be amazing! Looking forward to using with Laravel 9 & PHP 8.1.

Yes :)

Any update on this pull request being merged? I need it to be compatible with symfony/finder v6.

How can I help to get this PR merged ?

@yosymfony 🙏

+1

@yosymfony Still waiting your move

Is there any other compatible alternative?

you can install composer req andersundsehr/resource-watcher:dev-master as a replacement for `yosymfony/resource-watcher``. Until this is resolved.

Any solution for this problem?

Fork ;)

For my part I use the fork of @lucassabreu:
"repositories": [
    {
      "type": "vcs",
      "url": "https://github.com/lucassabreu/resource-watcher"
    }
  ],

Wow, this is still not resolved? Cmonnnnn....

At this point, I would like to add it's time to support the v7 packages and PHP 8.3. Has this project gone stale?

i am not using this anymore.

i am not using this anymore.

Did you switch to an alternative? I'm running into this issue because I'm updating dependencies for spaties/phpunit-watcher which has this package as a dep. I forked your branch and updated the v7 dep for symfony/finder.


Update patch releases on 0.17

Description
Update on release branch

@abhichou4
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.


Fix import path

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


Cannot parse foot, mile, etc..

I'm trying to integrate dimensional-parsing in a project to substitute my ad-hoc and fragile parsing code but I'm unable to parse some units I need. I've tried:
> parseOnly (ucumUnit (allUcumUnits ++ [demoteUnit' foot ]) <* endOfInput) "ft_i"
Left "Failed reading: empty"

It seems this is because the (atom,unit) map is built using the result of I.name . I.interchangeName which, for foot, yields:
I.name  (I.interchangeName foot )
"[ft_i]"

Instead of ft_i. However, parsing [ft_i] fails too. Same thing happens to mile, yard, etc..

It's because the parse is based on allUCUMUnits which right now only appears to include the SI ones.
If you go through adding demoteUnit' foot (etc) lines to https://github.com/dmcclean/dimensional-parsing/blob/master/src/Numeric/Units/Dimensional/Parsing/UCUM.hs#L43-L80 it should parse the non SI ones as well.
Sorry, this whole thing isn't ready for use yet at all, still have a ways to go.

Oh, wait, no it isn't just that, sorry, spoke too soon.

Now I get:
*Numeric.Units.Dimensional.Parsing.UCUM> parseOnly (ucumUnit (allUcumUnits) <* endOfInput) "[ft_i]"
Right (Exactly 1 % 1,Just 1 ft =def= Exactly 381 % 1250 of the SI base unit)

I will work on cleaning up this entire package later. It's still on my list but I am pretty swamped this month at work.

Missed a language extension because I had been testing in GHCi with that set. Now it works.

It works now. Thanks!

No problem!
What features are you looking for in the longer term? I am going to make this library actually good, I just can't work on it for a couple of weeks. Maybe at the Compose unconference in February.


Tag behavior

This pull request is intended to extend the TagsAttribute such that it enables the runner to find migrations when the list of tags provided to the runner contains more tags than have been assigned to the migration.

@fubar-coder can you review and pull this one in please

LGTM


EVNHCM không thêm được vào Enegy

Không thêm được bác ơi

Home assistant nó update liên tục nên code ko theo kịp 😋
Vào 17:08, Th 2, 20 thg 12, 2021 dlcvietnam @.***> đã
viết:

[image: image]
https://user-images.githubusercontent.com/96235876/146750142-9a626ff5-60b0-4007-b8d1-f6b52353aa9c.png
Không thêm được bác ơi
—
Reply to this email directly, view it on GitHub
https://github.com/txq92/custom_components/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANORSQCWAIAVXMUPSDAW3UDUR36AVANCNFSM5KNJNW7Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[BUG] Content scripts not hot-reloading with freshly initialized project using --with-content-scripts-ui

What happened?
Hot reloading does not work when changing content-script code after initializing a new project using the --with-content-scripts-ui flag. Tried all of the solutions recommended by Pallas:
If you have initialized a fresh project with --with-content-scripts-ui and it is not reloading, there could be a few reasons why this is happening.

Firstly, make sure that you have installed all the necessary dependencies by running pnpm install or npm install in the root directory of your project.

If the issue persists, it could be due to a caching issue. Try clearing your browser cache and reloading the extension. You can also try disabling the cache in your browser's developer tools while testing your extension.

Another possible reason could be that the --no-reload flag was used when running the plasmo dev command. This flag disables the automatic reloading of the extension when changes are made. If this is the case, you can remove the flag and run plasmo dev again to enable automatic reloading.

FWIW using WSL2 and the terminal is spitting out logs as if it reloaded:

Version
Latest
What OS are you seeing the problem on?
Windows, Other
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
(OPTIONAL) Contact Details
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Thanks for filing the issue! - @mpcen if you check the console on the webpage where CS is injected, do you see any log? I wonder if the WS server is being sandboxed behind WSL.
Side note: HMR is only available for page-runtime such as popups and options page (due to react script injection is under the mercy of the underlying page's CSP). CS/CSUI only have live-reload, and if the tab is inactive it will show a button to manual reload.
BTW, how did you find out about Plasmo? Would love to hear more about what you're building :)


Created Internship Page with Search, Filter, and Exploration Functionality

Related Issue
I have created a new Internship Page with added features of search and filter functionality using chips. This pull request includes the necessary code changes to implement these features.
Closes: #[#457]
Description of Changes

Created a new Internship Page component that displays a list of internships.
Implemented search functionality to search for internships based on keywords.
Added filter functionality using chips to narrow down internship results based on specific categories or preferences.
Developed reusable components like Gallery Card to display individual internship cards and Search Bar for the search functionality.

Details:

Designed and implemented the Internship Page layout using reusable components for improved maintainability and extensibility.
Integrated search functionality using the Search Bar component, allowing users to enter keywords and find internships matching their search queries.
Implemented filter options using chips within the Gallery Card component, enabling users to select specific categories or preferences to refine their internship results.
Developed the necessary logic and backend integration to fetch and display the filtered internship results based on the selected search and filter options.

Technical Changes:

Created reusable components like Gallery Card, which accepts props for displaying individual internship information.
Implemented the Search Bar component to handle search functionality and update the internship list accordingly.
Updated the Internship Page component to render a list of Gallery Card components, passing the necessary props for each internship.

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Thank you for considering this pull request. I look forward to your feedback and merging these changes into the main repository.

@anmode,please preivew and merge my pull request

two things which i noticed in preview:

Remove sign up button idk why it is coming
Please use new nav bar in internship page
Same changes in your hackathon page

after these fix.. I review PR

after these fix.. I review PR, good work btw appricating. @Nishitbaria I am making a project. I need in frontend part. If you wish to collaborate. Please let me know. we can have meet

hey @anmode  i update your changes :) ✔

Cool, Congrats.
You did not respond to my message :)
so you may think what you will get in return:

Experience
goodies or share from my side. if you up for this lemme know. ( language is Angular and typescript)

Cool, Congrats.
You did not respond to my message :)
so you may think what you will get in return:

Experience
goodies or share from my side. if you up for this lemme know. ( language is Angular and typescript)

I don't have any experience in agluer and Typescript if your project in MERN STACK I CAN DO THAT BUT ISME SE MUJE KUCHA BHI NAI AATA HE  :(
IF YOUR WORK IN REACT OR NEXTJS I CAN DO IT BUT ITS IN AGULER


Getting an error when using  %matplotlib notebook magic

On Ubuntu 20.04 with the latest version of this application, when I try to use %matplotlib notebook, I get an error as shown in the screenshot below:

Here is the code shown in the screenshot:
%matplotlib notebook
import numpy as np

import matplotlib.pyplot as plt
x = np.linspace(0, 5, 500)
y = np.cos(x)
plt.scatter(x=x, y=y)

@krassowski
Thank you for the reply.
Sorry, I thought this has something wrong with Electron implementation or something like that.  Maybe opening this issue here will help future people who will experience the similar issue with this application.
[Meta]: I never heard of JupyterLab before, all I knew is Jupyter notebooks, and thanks to some random newsletter on my email informing me about this application.
Thank you :)

No worries, I'm glad you are enjoying JupyterLab.
I will close this issue as resolved now.


refactor: construct StructArray w/ FieldRef

Which issue does this PR close?
-
Rationale for this change
DataType uses Fields/FieldRef internally. Accepting Field just to wrap it into an Arc is unnecessary expensive, esp. when the Field was cloned from an FieldRef (happens in some non-test code).
I've decided to NOT allow the construction from Field anymore because in prod code this is most likely a performance bug.
What changes are included in this PR?
Change From impls for StructArray to use FieldRef instead of Field.
Are there any user-facing changes?
Breaking: Constructor changes.

FYI - https://github.com/apache/arrow-rs/pull/4064

FYI - #4064

I would take that one as well, I don't really care.

This PR appears to need a cargo fmt to fix the CI

I think  https://github.com/apache/arrow-rs/pull/4064  just needs some tests -- shall we combine efforts?

I'm therefore left wondering if it is better to just encourage people to use the constructor approach, instead of refining what is a somewhat unfortunate API? Appreciate other thoughts on this...

If we are going for this I recommend we deprecate the unfortunate API

And we could always do both 🤔

I can also push #4064 over the finish line later this week and close this PR. :)

How about we propose removing them in a follow on PR


[BUG]: Milvus 2.3.2 is breaking milvus_vector_db_service tests

Version
fea-sherlock
Which installation method(s) does this occur on?
Source
Describe the bug.
milvus==2.3.2 is breaking milvus_vector_db_service tests. Where as it works with 2.3.1
    @pytest.mark.milvus
    def test_delete_by_keys(milvus_service: MilvusVectorDBService,
                            idx_part_collection_config: dict,
                            milvus_data: list[dict]):
        collection_name = "test_delete_by_keys_collection"

        # Make sure to drop any existing collection from previous runs.
        milvus_service.drop(collection_name)

        # Create a collection.
        milvus_service.create(collection_name, **idx_part_collection_config)

        # Insert data into the collection.
        milvus_service.insert(collection_name, milvus_data)

        # Delete data by keys from the collection.
        keys_to_delete = [5, 6]
        response = milvus_service.delete_by_keys(collection_name, keys_to_delete)
>       assert response == keys_to_delete
E       assert [] == [5, 6]
E         Right contains 2 more items, first extra item: 5
E         Use -v to get more diff

Minimum reproducible example
No response
Relevant log output
Click here to see error details
[Paste the error here, it will be hidden by default]

Full env printout
Click here to see environment details
[Paste the results of print_env.sh here, it will be hidden by default]

Other/Misc.
No response
Code of Conduct

[X] I agree to follow Morpheus' Code of Conduct
[X] I have searched the open bugs and have found no duplicates for this bug report

Closed by #1348


Base URL with custom protocol is prefixed with slash

Describe the bug
After setting custom protocol, built dist files have a leading slash in path.
export default defineConfig({
  base: 'app://',
});

(my use-case for custom protocol is to load app in Electron).
Reproduction
https://stackblitz.com/edit/vitejs-vite-bu8uph?file=dist/index.html
System Info
Binaries:
    Node: 16.13.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.11 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.1.0 - C:\Program Files\nodejs\npm.CMD

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

IIUC, for electron, you can use a relative base. Setting base: '' or base: './' has the same effect (and in v3 this has been reviewed and edge cases have been resolved). See for example the config in vite-electron-builder here:
https://github.com/cawa-93/vite-electron-builder/blob/bc7dee6f7fc4ae09d041b3c5d182e0bb15fc8799/packages/renderer/vite.config.js#L22

IIUC, for electron, you can use a relative base.

I tried that and while it works in dev mode, it doesn't in prod.
The app is built with assumption of availability of absolute path because it simplifies development. I don't see a reason why I should make non-trivial changes in code that provides 0 benefits.
I think this can be fixed simply by making this RE protocol-independent.
https://github.com/vitejs/vite/blob/c0c5e1a26d2099acd65a062fcfc790b85dc94db5/packages/vite/src/node/utils.ts#L237

I think we could add app to that regex. I don't see myself a problem with that. PR welcome and we can check what other maintainers think. Do you see other protocols that should be supported? I don't think we should allow any string here. Maybe we could start only with app or a short list if you see some other common one first.

Turns out, I don't need to change the base path at all.
So yeah, although I no longer affected by this, it's still a bug.

I think we should close this though, I prefer we get a real use case to justify changing the isExternal logic. Let's wait til then.


Scope whitelist

Coverage decreased (-7.08%) to 92.925% when pulling 835f5386352da9c79fa4588eb027e6e073c0d6db on scope_whitelist into 8544c9b1ef02ca419754e0cec491d083324ac3e8 on master.

Coverage decreased (-7.08%) to 92.925% when pulling 835f5386352da9c79fa4588eb027e6e073c0d6db on scope_whitelist into 8544c9b1ef02ca419754e0cec491d083324ac3e8 on master.

Ok, tests are in. I'll need help with documentation and cleanup of obsolete code.


chore(*): update spin domain/urls

Signed-off-by: Vaughn Dice<EMAIL_ADDRESS>Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[ ] The title, template, and date` are all set
[ ] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
[ ] Has passed bart check
[ ] Has run npm run build-index
[ ] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[ ] Headings are using Title Case
[ ] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[ ] Have tested with npm run test and resolved all errors

Re: build failure, see also https://github.com/fermyon/developer/pull/295


Probably bug: Select() query always returns unexpected results!

Hi, I am a beginner of sqlite-net. It works fine except when I use select to get a subquery of a table. It seems the return value of select() subquery is wrong. Is this a bug, or just by design?
Here is the code:
public sealed class MyClass
{
    public int Col1 { get; set; }
    public int Col2 { get; set; }
}

        var DbPath = Path.Combine(ApplicationData.Current.LocalFolder.Path, "My.db");
        using (var Db = new SQLiteConnection(DbPath))
        {
            Db.DropTable<MyClass>();
            Db.CreateTable<MyClass>();
            Db.Insert(new MyClass() { Col1 = 1, Col2 = 1 });
            Db.Insert(new MyClass() { Col1 = 1, Col2 = 2 });
            Db.Insert(new MyClass() { Col1 = 2, Col2 = 1 });
            Db.Insert(new MyClass() { Col1 = 2, Col2 = 2 });
            Db.Insert(new MyClass() { Col1 = 2, Col2 = 3 });

            var query = Db.Table<MyClass>().Select(x => x.Col2);

            foreach (var item in query)
            {
                Debug.WriteLine(item); //always return 0!
            }
        }

This was a bug in the linq implementation that has been fixed. Sorry for the trouble!


[P1] Document Autoplay Policy API

There's no spec yet. See https://github.com/WICG/autoplay/issues/1
Bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1506289
Trello: https://trello.com/c/4emMB6Y8/146-autoplay-api-fx-66
Acceptance criteria

[ ] New APIs' pages are added
[ ] Indexes and overview pages are updated to include the new members
[ ] At least one usable example provided
[ ] BCD updated
[ ] Submitted to engineers for review

@a2sheppy this was slated for Fx66, but in the end I seem to recall that you didn't do most of the work on the trello card. Can you remind me what happened for this one?

Yeah, it wasn't done for 66 in the end. One bit of it was, and that bit is
documented, but the rest slipped to 67 at soonest.
--
Eric Shepherd
Senior Technical Writer
Mozilla Developer Network
https://developer.mozilla.org/
Blog: http://www bitstampede.com/
Twitter: https://twitter.com/sheppy
On April 12, 2019 at 8:34:29 AM, Chris Mills<EMAIL_ADDRESS>wrote:

@a2sheppy https://github.com/a2sheppy this was slated for Fx66, but in
the end I seem to recall that you didn't do most of the work on the trello
card. Can you remind me what happened for this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mdn/sprints/issues/1118#issuecomment-482557648, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABkL3x_8C15DjN_ynL2w_n-jWHx89N9Uks5vgH1VgaJpZM4bX3Mj
.

OK, thanks. I've put this into the Firefox 67 epic, and we'll reevaluate it when we get to the next sprint planning.

This is still not done and is not likely to ship in 67. I will keep an eye on it.

There are still decisions to be made before this API gets implemented further than we've already documented it. This will not ship in 67.

Taking it out of the Fx67 epic and the Jessye Norman milestone.


neofetch don't report correct distro

Description
Recetly I Installed emerge (gentoo's package manager) on my laptop running artix linux (an arch fork without soystemd). And now neofetch report gentoo as my distro.
Neofetch version
$ neofetch --version
Neofetch 7.1.0

Screenshot
NOTE: emerge is removed

Config file
https://gist.github.com/DorisCarson/df1aded08f76c9ddd09cd84a5c81bf03
Verbose log
https://gist.github.com/DorisCarson/70909dbce2d479e21e4b54b55acfcb45

Your /usr/lib/os-release file contains gentoo information. Not a neofetch bug.


Issue 241 generate also png for dot output

issue #241

Hi @shireenf-ibm , I tried this locally by running make test-update, but the png files were not regenerated.. when running which dot from my shell, I do get its path, and am able to run dot directly from the shell..

I am not sure that fully understood the question.
for me the first time I ran make test-update I really saw that the files were regenerated (in the commit we see that the files were deleted and new ones replaced them),
after that only if I delete a file it will be generated by running the command;
If it exists, I will not see a change as it is the same content (same happens for all formats))

Hi @shireenf-ibm , I tried this locally by running make test-update, but the png files were not regenerated.. when running which dot from my shell, I do get its path, and am able to run dot directly from the shell..

I am not sure that fully understood the question. for me the first time I ran make test-update I really saw that the files were regenerated (in the commit we see that the files were deleted and new ones replaced them), after that only if I delete a file it will be generated by running the command; If it exists, I will not see a change as it is the same content (same happens for all formats))

I thought that the png files could not be compared by git, and should have been marked as changed when re-generated, even though had the same content. Now I see they're indeed also compared and identified as identical, so it seems to work ok on my env :)  thanks!


Return value from dialog

Previous usage of Get.dialog allowed closing of the dialog using Get.back with an optional result. With the redesigned Get.dialog implementation Get.back will no longer close the dialog or return a value. Instead to close a dialog Get.closeOverlay has to be used.
This PR introduces the ability to return a value when closing a dialog with Get.closeOverlay and introduces an alias method Get.closeDialog.
Solves #3122
Pre-launch Checklist

[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
[x] All existing and new tests are passing.

Thanks for that!
My only concern is with the alias, it closes bottomsheets too.
But adding the result on pop looks nice!

Good catch.
I will add a line to limit it to dialogs only.

When will we merge this request

@jonataslaw
Any idea how soon this PR can be merged?


Make guided content fullscreen on mobile

This PR increases the size of the guided content on mobile to cover the entire screen. This is intended to address #85. I've also adjusted the sizing of the elements in the guided content to give the map as much space as is appropriate.

This looks good


proposal: cmd/go: add go version -json

Background
Issue #21207 describes an effectively similar problem, but was closed without actually resulting in a structured version output.
Problem
When using CI Tools and pipelines, it is often required to know the major/minor version of Go.
Not least of which for passing into go commands such as go mod tidy which has a -compat argument, which requires just the major.minor version number.
Current Work-Arounds
As best as I can tell, everyone seems to be using some kind of regex against the output of go version.
Proposal
An argument for go version which will output a structured format of some kind, or perhaps just the version elements requested.

CC @bcmills @matloob

You're somewhat there already via go env -json. For example:
$ go1.18.5 version
go version go1.18.5 linux/amd64
$ go1.18.5 env -json GOVERSION GOOS GOARCH
{
	"GOARCH": "amd64",
	"GOOS": "linux",
	"GOVERSION": "go1.18.5"
}

or with tip:
$ go version
go version devel go1.20-1eeb257b88 Tue Sep 20 02:58:09 2022 +0000 linux/amd64
$ go env -json GOVERSION GOOS GOARCH
{
	"GOARCH": "amd64",
	"GOOS": "linux",
	"GOVERSION": "devel go1.20-1eeb257b88 Tue Sep 20 02:58:09 2022 +0000"
}

@mvdan Thanks for pointing out the -json args and options, however GOVERSION still requires parsing out the actual version numbers from the returned text.  Perhaps more options could be added for the major, minor and point releases (or use whatever terms that Go normally uses)?
For example, to take your two examples, it would be good to have these values output independently.
$ go1.18.5 env -json GOVERSION GOOS GOARCH GOMAJORVERSION GOMINORVERSION GOPOINTRELEASE
{
	"GOARCH": "amd64",
	"GOOS": "linux",
	"GOVERSION": "go1.18.5",
        "GOMAJORVERSION": 1,
        "GOMINORVERSION": 18,
        "GOPOINTRELEASE": 5
}

and
$ go env -json GOVERSION GOOS GOARCH GOMAJORVERSION GOMINORVERSION GOPOINTRELEASE
{
	"GOARCH": "amd64",
	"GOOS": "linux",
	"GOVERSION": "devel go1.20-1eeb257b88 Tue Sep 20 02:58:09 2022 +0000",
        "GOMAJORVERSION": 1,
        "GOMINORVERSION": 20,
        "GOPOINTRELEASE": 0
}

For actual releases, it seems to me like you can extract the semver-like numbers with a fairly simple regular expression. I've done that in a couple of places and it doesn't seem particularly difficult.
For "devel" versions, note that they don't need to follow a specific format. A build can end up with any version string placed in the VERSION file when running make.bash. It's true that builds by default end up with a format that includes a commit hash and a timestamp, but - again those feel rather easy to extract via a regular expression or string handling.

Not least of which for passing into go commands such as go mod tidy which has a -compat argument, which requires just the major.minor version number.

Note that if this is your use case, you likely don't want to obey go env GOVERSION if it is not a proper stable release. The current master, even though it's 1.20-somecommit, is very different from what the final Go 1.20 will be like.
It sounds to me like you only want to use GOVERSION when it's a stable version - in which case you only need to trim the bugfix release if there is one.

When using CI Tools and pipelines, it is often required to know the major/minor version of Go.

I'm not sure I understand when this would happen at all, certainly not "often".

Not least of which for passing into go commands such as go mod tidy which has a -compat argument, which requires just the major.minor version number.

The -compat argument predates the work on forward compatibility (respecting the go version in the go.mod file) and basically should not be used anymore.
Are there other contexts where this information is needed?

Talked to @bcmills and @matloob, and we think probably if this information is anywhere it would be in go list -f context.something, but it's unclear why it's necessary so we should establish a use case first.

With the compatibility work that has landed since this issue was filed in 2022, it seems like this may not be necessary anymore.

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

No change in consensus, so declined.
— rsc for the proposal review group


Add settings for the Alex linter

The executable for the Alex linter is currently hard-coded as 'alex',
which is an issue given the fact that it conflicts with the Haskell
lexer generator, whose executable is also called 'alex', has been around
a dozen years before the linter, and is packaged in the official
repositories of the major Linux distributions.
This commit adds options to use a local executable for the alex linter
(which is a node package), and an option to set a custom executable.
As side changes:

The pattern in the alex handler is made more readable by turnig it
into a very-magic regex.
Alex handles plain text, markdown, and HTML. Specific flags for HTML
and markdown are provided when instantiating the linters for the
respective filetypes, while before those formats were treated as plain
text.

Resolves #1633

Cheers! :beers:


Image resize

I'm using the approach below to load images, with a custom image rule but I can't find a way to get the image to not have empty margins at the top and bottom while displaying in full width.
Any advice?
Here's a link to the code below running in Snack: https://snack.expo.io/ByM5M1rIM

import React, { Component } from 'react';
import {ScrollView, StyleSheet, Image } from 'react-native';
import { Constants } from 'expo';
import Markdown, { getUniqueID } from 'react-native-markdown-renderer'; // 3.0.2

const copy = `# h1 Heading 8-)

Before...

---

![](assets/1_introduction.png)

---

And after...

![](assets/expo.symbol.white.png)

`;

const images = {
  'assets/expo.symbol.white.png': require("./assets/expo.symbol.white.png"),
  'assets/1_introduction.png': require("./assets/1_introduction.png")
}

const rules = images => ({
  image: (node, children, parent, styles) =>
      <Image
        key={getUniqueID()}
        resizeMode="contain"
        style={StyleSheet.flatten([
          // {
          //   flex: 1
          // },
          styles.image
        ])}
        source={images[node.attributes.src]}
      />
});

import "@expo/vector-icons"; // 6.2.2

export default class App extends Component {
  render() {
    return (
      <ScrollView style={styles.container}>
        <Markdown rules={rules(images)}>{copy}</Markdown>
      </ScrollView>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    borderColor: 'black',
    // alignItems: 'center',
    // justifyContent: 'center',
    paddingTop: Constants.statusBarHeight,
    backgroundColor: '#ecf0f1',
  }
});

Going to take a look

Updated your stack, https://snack.expo.io/HkrED7vIM hope this works

Thanks for the help! What I am trying to achieve is to have the margins smaller rather than the image being larger.
With your fix the image is cropped whereas I’m trying to displaying full width but with not too and bottom margins.
Is that possible?

This is what I see with your update:

Note that it's possible to achieve the same effect simply by changing the resize props to
resizeMode="cover"

What I would like to see is instead:

@mientjan If you have a bit of time to look again at my question, it would be much appreciated!

@mientjan ?

@mientjan please reopen. This is still not solved.


demangler: Add API functions for classifying symbols.

To be used by lldb.
Also add a -classify option in swift-demangler to test those new API functions.

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge


vitis_quantize.VitisQuantizer.dump_model

In the vai_q_tensorflow version the dump function had the option dump_float, which was very handy.
The vai_q_tensorflow2 equivalent (vitis_quantize.VitisQuantizer.dump_model) has not such an option as it seems.
Is this true or am I missing something?

Hi @Nunigan
In VAI1.3, there is no dump_float function for vai_q_tensorflow2. We will add this feature in VAI1.4. Thank you.


Dying while falling doesn't reset velocity

Velocity does not get reset when you are falling and die?

possibly fixed in https://github.com/CryptechTest/cryptech/commit/27f54776728a6ddff6db2eb89b8a1c7d3948b7ac

is this fixed?

seems fixed..


LodeRunner: Implement the ability to Stop Test Run if requested

Type of PR

[x] Documentation changes
[x] Code changes
[ ] Test changes
[ ] CI-CD changes
[ ] GitHub Template changes

PR Checklist

[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[x] I ran lint checks locally prior to submission.
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Purpose of PR

Does this introduce a breaking change

[ ] YES
[x] NO

Validation

[x] Unit tests updated and ran successfully
[x] Update documentation or issue referenced above

Issues Closed or Referenced

Closes https://github.com/retaildevcrews/wcnp/issues/947
Closes https://github.com/retaildevcrews/wcnp/issues/948

This was a test PR to troubleshooting the github issue.


fix(query/log/tail): do not show output on scan failures

Address https://github.com/leptonai/gpud/issues/32.

lgtm


Nominate ShellyDCMS as a voting member

I would like to nominate @ShellyDCMS to become a MapLibre Voting Member.
Motivation
Participation in Maputnik project and others: https://github.com/search?q=org%3Amaplibre+author%3AShellyDCMS&type=pullrequests
Checklist

[x] The nominee contributed in a non-trivial way or donated funds to the MapLibre Organization.
[x] This PR updates VOTING_MEMBERS.md with the GitHub handle and current employer (when applicable) of the nominee.
[ ] The nominee has approved the pull request to indicate they want to be a Voting Member of MapLibre.
[ ] The nominee has shared their full name and contact e-mail with this form.
[ ] At the voting deadline[^1], more Voting Members have approved this pull request than disapproved this pull request.

More details on the process of nominating Voting Members can be found here.
[^1]: Thursday, Aug 22nd, 2024 at 17:00 CEST

@ShellyDCMS, congratulations on being nominated to become a MapLibre Voting Member. As Voting Member you can elect the Governing Board, more on the election see here.
To accept the nomination please do the following two steps until Thursday, Aug 22nd, 2024:

Approve this Pull Request
Share your email and contact details with us with this online form


Refactored build system

Tested with: https://github.com/taichi-dev/taichi/pull/6544
Refactor change:

Encapsulating irrelevant cmake logic (setting up environment, third-party libraries, copy dylibs) into cmake functions
framework and each AOT demo compiles in their own CMakeLists.txt and managed by "add_subdirectory()"
AOT files generation moved from cmake stage to build stage
Merged 1_hello_world_with_cpu_interop and 1_hello_world_with_cuda_interop

LGTM, thanks!


Make iso workspace base dir configurable

Currently, when building an iso, you have to create the filesystem which creates for you a workspace directory which is always in the system temp dir.
Allowing this to be configurable somehow could solve two issues:

I don't want to write to /tmp

Maybe I have more storage elsewhere or something faster
In my case specifically I want all of these writes to go to a volume rather than the container filesystem I'm running on

I don't want to double write all my files

My use case is: extract an iso to dir x, make changes in x, recreate an iso from all the files rooted at x
If I could set the workspace to x when creating the filesystem I would save on all the calls I'm making to recreate the whole directory tree in the workspace

As for how to solve this ... it would be tough to do both without making a breaking API change to iso9660.Create so I understand if we don't want to do that.
An alternative option is to create a convenience function specifically in the iso9660 package that could create and finalize an iso in one call given the workspace dir (pre-populated with the iso content), the existing parameters to disk.Create (or some subset) and the finalize options.
Effectively I would contribute what I have here altered for general use.
@deitch what do you think? Is this something that you think would be useful or would you rather add a parameter somewhere for the workspace base dir?

I like where you are going with this. Two distinct but related things:

tell iso9660 where to make its scratch space, default to system temp dir
tell iso9660 not to create a workspace, just build from an existing one

For the first, it is straightforward. I have no issues with extending iso9660.Create to accept an option that says to use an existing workspace directory.
Now that I think about it, if you did that, it would solve the second as well, since all of the actions on an iso9660.FileSystem just work on the files in the working directory, if it is there. It is possible that there are parts that are cached/in-memory, but it was built not to do that.
I see only one issue, easily resolvable.
The lower-level iso9660.Create usually is not used directly, but rather created at a higher-level. Specifically, it is called by disk.CreateFilesystem, which, fortunately, does take a disk.FilesystemSpec struct.
The easiest path forward to me appears to be:

Add an option to iso9660.Create to take a workspace and used it, if present.
Add a test that uses the above and make sure it accepts files already there and works with them normally
Add a property to disk.FilesystemSpec to indicate what workspace path to use. I wouldn't name it anything iso9660-specific, as other read-only filesystems have the same property
Pass the workspace from disk.FilesystemSpec to iso9660.Create

It looks quite straightforward actually. I look forward to a PR on this.


Samsung J7 and J9 the keyboardType: TextInputType.numberWithOptions(decimal: true, signed: true), doesn't show decimal separator.

On Samsung J7 and J9 the keyboardType: TextInputType.numberWithOptions(decimal: true, signed: true), doesn't show decimal separator. Also can you please reopen the issues.
Flutter doctor -v

[✓] Flutter (Channel stable, v1.9.1+hotfix.5, on Mac OS X 10.15.1 19B88, locale en-CY)
    • Flutter version 1.9.1+hotfix.5 at /Users/niyazitoros/flutter
    • Framework revision 1aedbb1835 (2 weeks ago), 2019-10-17 08:37:27 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/niyazitoros/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.1, Build version 11A1027
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 40.2.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] IntelliJ IDEA Community Edition (version 2019.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 40.2.4
    • Dart plugin version 192.7402

[✓] Connected device (1 available)
    • SM G935F • ce12160c0d86152e01 • android-arm64 • Android 8.0.0 (API 26)

Hi @NTMS2017
Are you still experiencing this issue with latest stable on these devices?
If yes, can you please provide your flutter doctor -v?
Tested on Galaxy M30

Code Sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: TextField(
          keyboardType:
              TextInputType.numberWithOptions(decimal: true, signed: true),
          decoration: InputDecoration(border: OutlineInputBorder()),
        ),
      ),
      floatingActionButton: FloatingActionButton(
        child: Icon(Icons.add),
        onPressed: () {},
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-2.0.pre, on Mac OS X 10.15.5 19F101, locale
    en-GB)
    • Flutter version 1.20.0-2.0.pre at /Users/tahatesser/Code/flutter_dev
    • Framework revision 15a28159bc (9 days ago), 2020-06-23 04:52:58 -0700
    • Engine revision 91a63d6a44
    • Dart version 2.9.0 (build 2.9.0-19.0.dev 7e72c9ae7e)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.46.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.1

[✓] Connected device (5 available)
    • SM M305F      • 32003c30dc19668f          • android-arm64  • Android 10
      (API 29)
    • Taha’s iPhone • 00008020-001059882212002E • ios            • iOS 13.5.1
    • macOS desktop • macos                     • darwin-x64     • Mac OS X
      10.15.5 19F101
    • Web Server    • web-server                • web-javascript • Flutter Tools
    • Chrome        • chrome                    • web-javascript • Google Chrome
      83.0.4103.116

• No issues found!

Yes I am and I did include the result from "flutter doctor -v". You need to be on a locale where the decimal seperator is , not .

Without additional information we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


fix: Adjust market view centering

Task name here
Fixes #21
Types of change

[x ] Enhancement

Comments (optional)

@karlavasquez8 please fix lint :)


Translation for models

As we discussed with @patrys, it will be useful to add translation feature to saleor models (mostly products, probably some CMS in future).
A few assumptions:
Create a model that adds translation to a Product instance and is connected to a product with FK.

Use Prefetch objects
Adding new language can't modify the DB schema
API like product.translated.name - how do we handle multiple languages there?
Use Django Prefetch objects
~Only products with complete translation will be available in translated version (if not fields are translated, product will be displayed only in default language)~ Only complete translation can be added to a product

Only products with complete translation will be available in translated version (if not fields are translated, product will be displayed only in default language)

So language setting could change displayed products? What about products in cart, order, etc?

My idea was quite different: only allow complete translations to be saved to the database.
We don't want to hide products depending on the language, we want to display some of them untranslated.

@patrys Currently active language, so this should be a function that will be take at least a language (or whole request) as a parameter

In search, there are several approaches for handling multiple languages. For example we can index data as nested structure like
title: {
  "en": "Some english title",
  "de": "The same title in German"}

@artursmet I imagine product.translated being an object that overrides its __getattr__. The currently active language can be obtained at any given time with a call to django.utils.translation.get_language().

For UI I want to keep the current forms for default language and a dedicated translation tab for translation work. Editing a translation should display the untranslated content next to each field.
Things to consider: provide a dedicated localization interface that allows you to select a language and see which products require translation. In future we could bring in features such as CSV import/export and external translating APIs.

@maciekjablonski Please take a look on above specification, it should be enough to start development for this. Please reach me out if something is not clear enough.

Another requirement: have the translation proxy return a correct language for use in HTML (so we can correctly tag mixed content with lang="..." (this is important for accessibility and for proper support of English-within-an-RTL-language).
Please note that HTML language codes are not the same as gettext locale names: http://www.ietf.org/rfc/bcp/bcp47.txt

I look forward to updates on this issue, if anybody could update the status as PR is pending since few weeks now.

What about using something like django-modeltranslation
? We use it for another Django project, and it works fine. The only drawback is that a Django update might break the app, as it relies on internal API. But when it did, there was an update available within a couple of weeks. As Saleor uses it’s own Admin interface, it would be some work to integrate it there.

@MacLake we explicitly don't want any "magic" that modifies the database schema.

Hello, is there any update on this? Do I understand it correctly that as of now, there is no way to provide translations for product names, etc.?

We have a PR that adds model translations but only at the backend level without any dashboard UI. Recently we also had other priorities and that's why the PR is still waiting. I'll talk to the team and we'll decide whether to merge only the backend changes and if we have time to work on the UI in the nearest future.


Web - Group chat - "Invalid email" error message appears when creating a group DM with Concierge

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Log in to any account
Click Fab > New group
Select any random email and concierge as members
Click "Create Group"

Expected Result:
Group is created successfully
Actual Result:
Invalid email error message appears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?:  Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/5d9ffdd2-867a-442c-aaaa-b2b6728c2be6
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

https://github.com/Expensify/Web-Expensify/pull/37753 appears to be the root cause for this as well. We're passing empty string as the login for concierge because the client doesn't have it:

FWIW, I'm not sure you're supposed to be able to make a group chat with concierge

Looks like we DO want to be able to create group chats w/ concierge, but not some other expensify emails: https://github.com/Expensify/App/issues/8586

as of right now now I'm unable to reproduce in main / main / main

Here's some logs from when I reproduced in main / staging

So ya technically this is fixed for starting group chats w/ Concierge - now we're chatting about starting group chats w/ other people who you don't have their login

This is fixed by https://github.com/Expensify/App/pull/21159

removing blocker as this is fixed for group chats with concierge. if there is a new bug about group dms for different cases then lets spin up a new issue.

I just tested a few different scenarios of creating group chats on staging and they all seemed to work.

Let's close this out then?

Agreed, no payment due here so we can close 👍


GitHub-1286 fix late init dbinstance

Description of your changes
Exclude deletion protection if dbinstance is a cluster member
Fixes #1286
I have:

[/] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

make test results in package github.com/crossplane/provider-aws/pkg/clients/mock: build constraints exclude all Go files in /Users/josephweigl/git/github/crossplane/provider-aws/pkg/clients/mock
make reviewable results in
18:54:58 [ .. ] Generating sfn controllers and CRDs
Error: open /Users/josephweigl/.cache/ack-generate/src/aws-sdk-go/models/apis/servicecatalog-appregistry/2020-06-24/api-2.json: too many open files

How has this code been tested
Build the provider and deployed it in our test environment.
This changes has fixed the deletionProtection issue which cases the dbinstance to be synced false

@eloo-abi are you working with Mac ? You can try ulimit -n 10240

@haarchri
thanks for the hint.. with the ulimit at least reviewable is working.
test is still failing:
package github.com/crossplane/provider-aws/pkg/clients/mock: build constraints exclude all Go files in /Users/josephweigl/git/github/crossplane/provider-aws/pkg/clients/mock


Feat: Added SetDatafileAccessToken method in OptimizelyFactory

Summary

Adds “OptimizelyFactory#setDatafileAccessToken(string) method

Test plan

Added unit tests and other unit tests should pass and all FSC scenarios should pass

Pull Request Test Coverage Report for Build 1443

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 89.484%

Totals

Change from base Build 1437:
0.01%

Covered Lines:
3957

Relevant Lines:
4422

💛  - Coveralls


[SPARK-44059] Add analyzer support of named arguments for built-in functions

What changes were proposed in this pull request?
Add analyzer support for named function arguments.
Why are the changes needed?
Part of the project needed for general named function argument support.
Does this PR introduce any user-facing change?
We added support for named arguments for the CountMinSketchAgg and Mask SQL functions.
How was this patch tested?
A new suite was added for this test called NamedArgumentFunctionSuite.

Closed as I realized the analyzer part wasn't affected by the merge conflict. Please go to #1 .


Documentation Update

Please add a link to the documentation on where to obtain the keys (consumer/consume).

This library is not stand-alone assumes you have some knowledge about Daraja.
Please get started here:
https://developer.safaricom.co.ke/
https://developer.safaricom.co.ke/test_credentials


Change bot/bot account's profile picture?

This is just an example bot really, meant to inspire and encourage others to take it and make new things with it. Not to be a completely feature complete bot.
You are looking for this documentation: https://discord.js.org/#/docs/main/master/class/ClientUser?scrollTo=setAvatar


e2fsprogs: apply patch unconditionally

Commit 49d0a5afd mistakenly inverted when to apply the patch.
Maybe it's not needed anymore, as pkgsMusl.e2fsprogs succeeded for me
even without it, but it looks harmless and better not have it inversed.
This way we also don't cause a mass rebuild :-)
Things done

Built on platform(s); no rebuild on glibc platforms

[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

OfBorg checked the darwin platforms, but as we did apply that patch in there until recently, it's not much of new information.


Update example and fix link in readme(retrofit_generator)

I decided to change the example to the new standards. I also looked at pub points and in order to increase it, I need to fix the link in the readme in retrofit_generator

Tests fall on - 'element2' is deprecated and shouldn't be used. Should I leave it like this if it was added to the analyzer 5.2.0

@Carapacik  Please update to the latest and remove the deprecated

Then you will have to upgrade the sdk version, because analyzer ^5 is not supported by dart 2.17

Is it worth updating analyzer to 5.2 and dart to 2.18 with it?

Dart sdk updated to 2.18

I don't understand what the error is...

@trevorwang plz check this

Thanks for your excellent work. Let's bump the generator version to 6.0.0 since the minimum SDK requirement is a breaking change

is it worth raising the sdk version for retrofit then? To raise his version accordingly?

I think it's better to upgrade the package version to 5.1, not 6.0.
What's the result? @trevorwang

There's no change for retrofit.  Let it stay the same.
Change is breaking. Please highlight it in README

Ok, only retrofit_generator and 6.0.0

pub will update generator to 5.1 automatically.   6.0 will stop this happening

retrofit_generator 6.0.0 and his dart >=2.18 <3.0.0


Search bar within ontology does not search in ontology

Search bar within ontology does not search in ontology:
When I am in MAXO ontology, there is a search bar that says 'search MAXO'. However, when entering a term to query maxo, it searches OLS wide and not within MAXO.

I can't reproduce this

I also noticed this: the index in solr response correctly but click on the "search" button and it will reveal results from outside the ontology


Rename plan_entry_reosolver.go to plan_entry_resolver.go

Fix a typo in filename

Summary
FIx a filename
Use Cases

Checklist

[*] I have viewed, signed, and submitted the Contributor License Agreement.
[*] I have added an integration test, if necessary.

Hi, @ybykov-a9s! Thanks for submitting this change. Once you've signed the CLA this should be good to merge.


Can't set discontinuous channel

Is this a bug or a feature request?
Bug
What did you expect?
set channel 0,1,2,3,4,5,6,20 in lora-app-server, and gateway will change to this channel automatically.
What happened?
Last channel: 20 will set as radio 0, log in packet-forward like below:

135639 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: radio 0 enabled (type SX1255), center frequency 470700000, RSSI offset -176.000000, tx enabled 1, tx_notch_freq 0
135640 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: radio 1 enabled (type SX1255), center frequency 471700000, RSSI offset -176.000000, tx enabled 0, tx_notch_freq 0
135641 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 0>  radio 0, IF -400000 Hz, 125 kHz bw, SF 7 to 12
135642 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 1>  radio 0, IF -200000 Hz, 125 kHz bw, SF 7 to 12
135643 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 2>  radio 0, IF 0 Hz, 125 kHz bw, SF 7 to 12
135644 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 3>  radio 0, IF 200000 Hz, 125 kHz bw, SF 7 to 12
135645 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 4>  radio 0, IF 400000 Hz, 125 kHz bw, SF 7 to 12
135646 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 5>  radio 1, IF -400000 Hz, 125 kHz bw, SF 7 to 12
135647 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 6>  radio 1, IF -200000 Hz, 125 kHz bw, SF 7 to 12
135648 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: INFO: Lora multi-SF channel 7>  radio 0, IF 1600000 Hz, 125 kHz bw, SF 7 to 12
135649 Sep  7 02:19:48 rhf2s001 lora_pkt_fwd[16561]: ERROR: invalid configuration for Lora multi-SF channel 7

What version are your using?
git harsh: 66ddba1fba60747087293e7b2c9e00d40e7fd87d
How can your issue be reproduced?
try to set discontinuous channel in "channel configuration, like:0,1,2,3,4,5,6,20
Could you share your log output?
this is my global_conf.json:

{"SX1301_conf":{"chan_FSK":{"bandwidth":0,"datarate":0,"enable":false,"if":0,"radio":0},"chan_Lora_std":{"bandwidth":0,"enable":false,"if":0,"radio":0,"spread_factor":0},"chan_multiSF_0":{"enable":true,"if":-400000,"radio":0},"chan_multiSF_1":{"enable":true,"if":-200000,"radio":0},"chan_multiSF_2":{"enable":true,"if":0,"radio":0},"chan_multiSF_3":{"enable":true,"if":200000,"radio":0},"chan_multiSF_4":{"enable":true,"if":400000,"radio":0},"chan_multiSF_5":{"enable":true,"if":-400000,"radio":1},"chan_multiSF_6":{"enable":true,"if":-200000,"radio":1},"chan_multiSF_7":{"enable":true,"if":1600000,"radio":0},"clksrc":1,"lorawan_public":true,"radio_0":{"enable":true,"freq":470700000,"rssi_offset":-176,"tx_enable":true,"tx_freq_max":510000000,"tx_freq_min":470000000,"type":"SX1255"},"radio_1":{"enable":true,"freq":471700000,"rssi_offset":-176,"tx_enable":false,"type":"SX1255"},"tx_lut_0":{"dig_gain":0,"mix_gain":8,"pa_gain":0,"rf_power":-1},"tx_lut_1":{"dig_gain":0,"mix_gain":10,"pa_gain":0,"rf_power":2},"tx_lut_10":{"dig_gain":0,"mix_gain":9,"pa_gain":2,"rf_power":19},"tx_lut_11":{"dig_gain":0,"mix_gain":9,"pa_gain":0,"rf_power":20},"tx_lut_12":{"dig_gain":0,"mix_gain":11,"pa_gain":2,"rf_power":21},"tx_lut_13":{"dig_gain":0,"mix_gain":13,"pa_gain":2,"rf_power":22},"tx_lut_14":{"dig_gain":0,"mix_gain":9,"pa_gain":3,"rf_power":23},"tx_lut_15":{"dig_gain":0,"mix_gain":11,"pa_gain":3,"rf_power":24},"tx_lut_2":{"dig_gain":0,"mix_gain":12,"pa_gain":0,"rf_power":4},"tx_lut_3":{"dig_gain":0,"mix_gain":8,"pa_gain":1,"rf_power":9},"tx_lut_4":{"dig_gain":0,"mix_gain":10,"pa_gain":1,"rf_power":11},"tx_lut_5":{"dig_gain":0,"mix_gain":10,"pa_gain":1,"rf_power":12},"tx_lut_6":{"dig_gain":0,"mix_gain":12,"pa_gain":1,"rf_power":13},"tx_lut_7":{"dig_gain":0,"mix_gain":13,"pa_gain":1,"rf_power":14},"tx_lut_8":{"dig_gain":0,"mix_gain":15,"pa_gain":1,"rf_power":15},"tx_lut_9":{"dig_gain":0,"mix_gain":8,"pa_gain":2,"rf_power":17}},"gateway_conf":{"forward_crc_disabled":false,"forward_crc_error":false,"forward_crc_valid":true,"gateway_ID":"0000b827eb41e673","gps_tty_path":"/dev/tty","keepalive_interval":10,"push_timeout_ms":100,"serv_port_down":1700,"serv_port_up":1700,"server_address":"<IP_ADDRESS>","stat_interval":30}}

Did you confirm that your channel-plan fits within the bandwidth of your gateway radios? For 125KHz channels the bandwidth of each radio is 0.925MHz.
I did not do the math, but I don't think you'll be able to fit your channel-plan on both radios (hence the error).

The LoRa Channel Manager feature has been merged into the LoRa Gateway Bridge. The LoRa Channel Manager component is now deprecated.


How to specify outside resources for tests?

How do we specify where to find outside resources like jQuery for tests?
Backstory:
Parsley is a fairly popular jQuery plugin for web validations. We'd love to port Parsley to ES6. We're trying this project to do so. @guillaumepotier doesn't have much time to fiddle with the setup, and I'm quite ignorant about tooling (and not that eager to change that fact, tbh).
I think it would be a good goal for this project that such a porting require very little changes to the boilerplate install, do you agree? So I thought it might be a good idea to report difficulties we're having.
We have a branch for that. The imports work fine for the Parsley code, but I don't know how to import jQuery in tests. Hopefully it's simple. It is, of course, completely out of the question to bundle jQuery with Parsley's dist/ files...
Any help would be greatly appreciated.

@marcandre , requiring in 3rd party libs in the tests should be the same as requiring them in the library itself. Usually I'll do...
// test.js

import MyLib from '../../path/to/my-lib';
import $ from 'jquery';

// ...tests here

Is this not working for you?

@marcandre , I left a comment over on that PR. It was typed pretty quickly, so do let me know if it's too vague to be of use. I'd love to help ya get that PR landed!

@marcandre , is this good to close?

Yes! Thanks so much for your help :smile:

Rad. No problem - I'm happy to help! Hopefully we can get Parsley in the green soon :)


Error trying to update python client on Ubuntu 17.04

bblfsh/pyuast.c: At top level:
bblfsh/pyuast.c:179:27: error: storage size of ‘module_def’ isn’t known
static struct PyModuleDef module_def = {

It broke because I was upgrading bblfsh with Python2, it works with pip3


Generate posts without defining routes in nuxt.config.js

Is your feature request related to a problem? Please describe.
Lessen the use of the routes field in nuxt.config.js to reduce the size of the codebase
Describe the solution you'd like
refactoring asyncData method in pages/news-and-views/_slug/.index.vue
Describe alternatives you've considered

changed axios commands
axios.all([])

Additional context
In regards to PR #104 I realized when running npm run generate the pages created by pages/_slug.vue/index.vue do not appear in the cli result message but you are able to reach them after deploying so this is not a problem but it is worth mentioning.
ℹ Generating pages                                                                                18:52:57
✔ Generated /                                                                                18:52:58
✔ Generated /news-and-views/the-importance-of-peripheral-vision                                                                                18:53:02
✔ Generated /news-and-views                                                                                18:53:02
✔ Generated /news-and-views/page/1                                                                                18:53:02
✔ Generated /news-and-views/the-walrus-talks-inclusion                                                                                18:53:02
✔ Generated /news-and-views/jutta-treviranus-evaluates-the-sidewalk-labs-digital-innovation-appendix                                                                                18:53:02
✔ Generated /news-and-views/project-we-count-evaluates-torontos-quayside-master-innovation-and-development-plan                                                                                18:53:02

	asyncData (context) {
		return axios.get(`${Config.wpDomain}${Config.apiBase}posts`).then((response) => {
			const res = response.data.filter(x => x.slug === context.params.slug)[0]
			let picture = "https://wecount.inclusivedesign.ca/wp-content/uploads/2019/10/We-Count-logos_colour-and-bw-01.png"
			let altTag = ""
			if (res._links["wp:featuredmedia"] !== undefined) {
				const api = res._links["wp:featuredmedia"][0].href
				const pic = axios.get(api)
				picture = pic.source_url
				altTag = pic.data.alt_text
			}
			return {
				title: res.title.rendered,
				content: res.content.rendered,
				picture,
				tags: res.pure_taxonomies.tags.map(({ name }) => name)

			}
		})
	}

This was the best I was able to do.
Able to get all property information from API except url for post image.
This is because the information is not in the post API and requires an additional API call using the "wp:featuredmedia" in the first API call to retrieve. The second API call being dependant on the first is the root of the problem.

It's decided to generate posts using static page rendering for better performance/loading time, so defining routes in nuxt.config.js is still required. This issue can be closed.


Updated eccv-20-2 with links

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
ba4769c58d92690f57f05d2f21296c3cd7d5fbb3

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/643282ac8f7c3b0008ca80fb


Set stretch mode to "canvas_items"

In this PR I have enabled stretch mode in the project settings to allow for different window sizes without (or barely) affecting the game or viewport (I think). This PR is still being tested, so don't merge yet until I am sure it works as it should.

I made a mistake with the latest commit. How? Apparently with aspect to  "expand", it changes the viewport so it can fit inside the window. If the width or height of the window is very small then it will cause an issue shown below:

I will now revert the commit and forget that I ever set aspect to "expand".


Add process name to stop ctrl event string

Issue: While debugging multiple processes it gets very confusing which process hit the breakpoint or got an access violation.. etc.
For instance we usually debug server as well as client side of game together and they might share some part of the codebase and it gets very confusing, the only way to tell right now is to scroll the Scheduler view and look for the highlighted thread.
This PR brings the process name in the statusbar with Thread Id and stop explaination.
Before:

After:

Manually merged in 35b97d9c206119efed945c803e0d5d034f3f34e4.


"failed to run channelserver" error seen in rancher logs

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):
On master-head - commit id: e20f472d4
following error is seen in the rancher logs:
2020/07/28 18:19:50 [INFO] kontainerdriver amazonelasticcontainerservice stopped
2020/07/28 18:19:50 [INFO] dynamic schema for kontainerdriver amazonelasticcontainerservice updating
Incorrect Usage. flag provided but not defined: -path-prefix

NAME:
   Channel Server - A new cli application

USAGE:
   channelserver [global options] command [command options] [arguments...]

VERSION:
   v0.0.0-dev (HEAD)

COMMANDS:
   help, h  Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --url value                     (default: "channels.yaml") [$URL]
   --config-key value               [$SUBKEY]
   --refresh-interval value        (default: "15m") [$REFRESH_INTERVAL]
   --listen-address value          (default: "<IP_ADDRESS>:8080") [$LISTEN_ADDRESS]
   --channel-server-version value   [$CHANNEL_SERVER_VERSION]
   --help, -h                      show help
   --version, -v                   print the version
time="2020-07-28T18:20:13Z" level=fatal msg="flag provided but not defined: -path-prefix"
2020/07/28 18:20:13 [INFO] failed to run channelserver: exit status 1
Incorrect Usage. flag provided but not defined: -path-prefix

NAME:
   Channel Server - A new cli application

USAGE:
   channelserver [global options] command [command options] [arguments...]

VERSION:
   v0.0.0-dev (HEAD)

COMMANDS:
   help, h  Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --url value                     (default: "channels.yaml") [$URL]
   --config-key value               [$SUBKEY]
   --refresh-interval value        (default: "15m") [$REFRESH_INTERVAL]
   --listen-address value          (default: "<IP_ADDRESS>:8080") [$LISTEN_ADDRESS]
   --channel-server-version value   [$CHANNEL_SERVER_VERSION]
   --help, -h                      show help
   --version, -v                   print the version
time="2020-07-28T18:21:08Z" level=fatal msg="flag provided but not defined: -path-prefix"
2020/07/28 18:21:08 [INFO] failed to run channelserver: exit status 1

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - commit id: e20f472d4
Installation option (single install/HA): single

Validated on master-3374-head

This issue was also blocking rke2 clusters from being able to be upgraded. Now this is allowing upgrades:

I do not see the aforementioned log issue anywhere in the rancher server logs.