Add guard to _TASK_SCOPE, so it can be configured externally

Need access to internals when deriving from OO style task, so I'm defining _TASK_SCOPE as protected, but it would be redefined in TaskSchedulerDeclarations.h.
Adding a guard would be great.

Would you mind sharing the use case? Why do you need access to private methods/variables and for what?
Maybe they should not private for a good reason.

I’m writing a few modules deriving from Tasks.
I need to iterate though the modules to init them, send signals, etc, and having access to the linked list pointers would avoid having to reimplement it, just because of accessibility modifiers.
Sent from Mail for Windows 10
From: Anatoli Arkhipenko
Sent: Monday, August 5, 2019 10:46 PM
To: arkhipenko/TaskScheduler
Cc: Asti; Author
Subject: Re: [arkhipenko/TaskScheduler] Add guard to _TASK_SCOPE, so it can beconfigured externally (#76)
Would you mind sharing the use case? Why do you need access to private methods/variables and for what?
Maybe they should not private for a good reason.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I’m writing a few modules deriving from Tasks. I need to iterate though the modules to init them, send signals, etc, and having access to the linked list pointers would avoid having to reimplement it, just because of accessibility modifiers.

Is it not possible to init using OnEnable and let the scheduler deal with the sequence?

I need to call some methods on all registered modules, so I literally need the sequence.
Sent from Mail for Windows 10
From: Anatoli Arkhipenko
Sent: Monday, August 5, 2019 11:57 PM
To: arkhipenko/TaskScheduler
Cc: Asti; Author
Subject: Re: [arkhipenko/TaskScheduler] Add guard to _TASK_SCOPE, so it can beconfigured externally (#76)
I’m writing a few modules deriving from Tasks. I need to iterate though the modules to init them, send signals, etc, and having access to the linked list pointers would avoid having to reimplement it, just because of accessibility modifiers.
Is it not possible to init using OnEnable and let the scheduler deal with the sequence?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm having the same use case. I'm using the TaskScheduler to manage the derived tasks and would need to access the list.
A cleaner change would be to add functions to get the next/previous task instead of misusing _TASK_SCOPE and _TASK_DEBUG. This would open a read-only interface to the queue.

I've use the protected visibility to essentially implement those getters.
The base class should implement them, but it was a much bigger change.
@arkhipenko Can you tell us of a way to take this forward?

@deviousasti Interesting that you mention protected visibility. The _TASK_SCOPE can only be set to public or private in src/TaskSchedulerDeclarations.h:
#ifdef _TASK_DEBUG
    #define _TASK_SCOPE  public
#else
    #define _TASK_SCOPE  private
#endif

@moritz89 Hence the pull request. :)

I guess a simple getter method for previous and next tasks will be ok?
Let me get to that sometime soon.

Hi @arkhipenko: I think the getter would be perfect, if there also is a getter on the scheduler to be able to get the first task (otherwise you still can't iterate over all of them).
Would imo only be insufficient on multi-task-systems (since task order might be changed while iterating). Thanks for the response btw!

Should be able to push something into testing this weekend

Please test v3.1.4 from the testing branch.
#define _TASK_EXPOSE_CHAIN

Task::Task&  getPreviousTask()
Task::Task&  getNextTask()

and
Scheduler::Task&  getFirstTask()
Scheduler::Task&  getLastTask()

Will check.

Why is it Task& instead of Task*?
If there is no next Task, you'll be de-referencing a null pointer.

Good point. Didn't think about that.  Just followed a pattern of Scheduler's currentTask method, which, come to think now should be a pointer as well.
Will fix.

Please re-test v3.1.4 from the testing branch. Just updated it
#define _TASK_EXPOSE_CHAIN

Task::Task*  getPreviousTask()
Task::Task*  getNextTask()

and
Scheduler::Task*  getFirstTask()
Scheduler::Task*  getLastTask()

Scheduler::Task&  currentTask()  // <== DEPRICATED (still supported)  use
Scheduler::Task*  getCurrentTask()  // instead

Will check.

This looks fine.
I my version, I had getFirstTask as begin and getLastTask as end .
This was following the std::begin pattern.
Do you feel like it's more meaningful to keep getFirstTask and getLastTask?

Unfortunately, I was also using iPreviousMillis to calculate how long it's been since the task last ran, which I could still access using the protected scope.

Hmm. I have a method that allows you to know time until next run already. Never thought it would be interesting to know how long since last run. I can add that as well.

This looks fine.
I my version, I had getFirstTask as begin and getLastTask as end .
This was following the iterator pattern (std::begin).
Do you feel like it's more meaningful to keep getFirstTask and getLastTask?

Yes. I think begin and end is way too generic and usually are associated with a process (like serial comms), not an object (like the processing chain is).

Unfortunately, I was also using iPreviousMillis to calculate how long it's been since the task last ran, which I could still access using the protected scope.

iPreviousMillis is not when the task has started. It's when it was supposed to be started. It could be quite misleading if you are in the catch-up situation actually.

Yes. I think begin and end is way too generic and usually are associated with a process (like serial comms), not an object (like the processing chain is).

std::begin is actually exactly what you're doing with first and last
https://en.cppreference.com/w/cpp/iterator/begin

I assumed that based on the comment:
volatile unsigned long    iPreviousMillis;       // previous invocation time (millis).  Next invocation = iPreviousMillis + iInterval

Is there a correct way of calculating the time elapsed since a task last ran?

I assumed that based on the comment:
volatile unsigned long    iPreviousMillis;       // previous invocation time (millis).  Next invocation = iPreviousMillis + iInterval

Is there a correct way of calculating the time elapsed since a task last ran?

Currently, the scheduler does not capture the actual time of a task's callback start. Each task can do that by saving millis() value as the first line of entry. I never had a use case for this functionality, as everyone was more interested in the future time of task invocations, not the history of them.

Yes. I think begin and end is way too generic and usually are associated with a process (like serial comms), not an object (like the processing chain is).

std::begin is actually exactly what you're doing with first and last
https://en.cppreference.com/w/cpp/iterator/begin

Interesting, but I am not following the std:: style anywhere else, so why do it in an isolated case?
(Not saying I should not follow it, but it just never happened).

Yes. I think begin and end is way too generic and usually are associated with a process (like serial comms), not an object (like the processing chain is).

std::begin is actually exactly what you're doing with first and last
https://en.cppreference.com/w/cpp/iterator/begin

Interesting, but I am not following the std:: style anywhere else, so why do it in an isolated case?
(Not saying I should not follow it, but it just never happened).

The proper way would be to create a real iterator class, then use begin and end and next and previous. I was not planning on doing it simply because of lack of a use case and library size.
If you'd like to contribute, perhaps we can have an iterator under the _TASK_EXPOSE_CHAIN option.

I think your current implementation is good.
New branch works great.
Thank you for all your hard work!


Provide quick fix for defining an undefined variable as /global/

@egamma I close this one. IMO such a fix should now go into eslint auto fix infrastructure which is now surfaced in the eslint extension.

@dbaeumer agreed to close this one.


Small changes

replace operator[] with at() for bounds check
extract some code into a function to avoid duplication
use TAB as separator throughout in the periodic table data for consistency

@ptosco I got rid of the .at() usage (which I generally agree with) since we know that the valence vectors are not empty


Allow passing table hints to the db via Ecto

Environment

Elixir version (elixir -v): 1.5.1
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): MySQL 5.6.38
Ecto version (mix deps): 2.1.6
Database adapter and version (mix deps): mariaex 0.8.1
Operating system: OSX 10.11.6

Current behavior
Previously discussed on SO here: https://stackoverflow.com/questions/48394148/how-to-give-an-index-hint-to-mysql-through-ecto
Please bear in mind I'm an Elixir/Ecto noob.
I need to pass an index hint to MySQL in an Ecto query and it seems this is not currently supported.
The syntax for MySQL is defined here. It seems from a brief googling that it's not standard SQL (Postgres doesn't support it at all) and each DB adapter would need to handle it differently.
For example, if the following query needs manual optimization:
SELECT *
FROM posts AS p
  JOIN comments AS c ON c.post_id = p.id

I might want the following query produced:
SELECT *
FROM posts AS p
  JOIN comments AS c USE INDEX(comments_index) ON c.post_id = p.id

I have tried a number of hacks, like:
from p in Post
  join: c in fragment("comments AS c USE INDEX(comments_index) ON c.post_id = p.id")

# produces the SQL

SELECT *
FROM posts AS p
  JOIN (comments AS c USE INDEX(comments_index) ON c.post_id = p.id) AS f1

# which is obviously invalid SQL syntax

and also
from p in Post
  join: c in "comments AS c USE INDEX(comments_index) ON c.post_id = p.id"

# produces the SQL

SELECT *
FROM posts AS p
  JOIN `comments AS c USE INDEX(comments_index) ON c.post_id = p.id`

# which is results in a 'table name too long' error

Expected behavior
Ideally there'd be a feature added so I could do:
from p in Post
  join: c in Comment
  use: "comments_index"

# produces the SQL

SELECT *
FROM posts AS p
  JOIN comments AS c USE INDEX(comments_index) ON c.post_id = p.id

if nothing else it would be nice if I could do something like:
from p in Post
  join: c in fragment("comments AS c USE INDEX(comments_index) ON c.post_id = p.id", raw: true)

# produces the SQL

SELECT *
FROM posts AS p
  JOIN comments AS c USE INDEX(comments_index) ON c.post_id = p.id

We have changed how Ecto.Query.join/5 accepts options, which means that we can add a :use_index key to JoinExpr and it should be an atom or a list of atoms. It will only work for MySQL, other databases should raise.

I have changed the title of the issue to be about table hints. Both SQLServer and MySQL supports this and they support a variety of commands. For example, in MySQL:
SELECT * FROM t1 USE INDEX FOR JOIN (i1) FORCE INDEX FOR JOIN (i2);

Or in SQL Server:
FROM t WITH (TABLOCK, INDEX(myindex))

My suggestion is to introduce a new option called :hints that is either a literal string or a literal list of strings. Translating both examples above:
from t, hints: "USE INDEX FOR JOIN (i1) FORCE INDEX FOR JOIN (i2)"
from t, hints: ["TABLOCK", "INDEX(myindex)"]

:hints will be supported in from and join. Thoughts?

That looks good to me 👍

Same here.

Hey 👋
I maintain our own fork of clickhouse_ecto for connecting to ClickHouse. Trying to make their SAMPLE hint work with ecto, here are the docs for it but it's pretty straightforward. I need to compile a query like this:
SELECT .. FROM events SAMPLE 10000 ...

The hints feature works OK for this, I can access the hints in the ecto adapter and compile it as necessary. However, I ran into a limitation with trying to make the sampling threshold configurable for the app:
from(e in "events", hints: ["SAMPLE #{sample_threshold()}"])

** (Ecto.Query.CompileError) `hints` must be a compile time string or list of strings, got: `["SAMPLE #{sampling_factor()}"]`

We want different sampling levels based on available server resources and different data characteristics. It should be configurable for each instance. I'm curious as to why the hints field is limited to compile-time strings? Defense against SQL injection? Ideas on how this SAMPLE use-case could be supported?
Thanks

Hi @ukutaht! Yes, the issue is to avoid SQL injection. If you can interpolate anything in there, someone may pass external data, which can break things.
Maybe in this case what we can do is to allow hints to be a tuple and then it becomes responsibility of the adapter to encode that tuple properly. The built-in SQL adapters should raise if they see something that is not a binary. At compile-time, we will enforce them to be binaries or tuples. Tuples are not checked on their contents.
WDYT?

Thanks for the quick reply! By a tuple do you mean something like this?
from(e in "events", hints: [{"SAMPLE", sample_threshold()])

It would work great for me. It's a simple enough change that I can probably PR it as well some time next week.

Yes! In this case I would even do: from(e in "events", hints: [sample: sample_threshold()]). :)


Fix info_bar.hide() and some little cast problems

Fix some compiler warnings and add the info_bar.hide() function,

@zzeroo: Thanks for the PR! I update rustc version for appveyor and then I restart it. Sorry for the delay. :-/
@gkoz: we forgot to update appveyor version here. I open a PR.

If this code has casts problems it might be stale and need syncing with the tip of examples/pending.

:+1:

Thanks!


dependencies for PDF extraction: pyvips, click, Pillow

If you use PDFs instead of images, the script uses pyvips to extract the images from pdf.
To install pyvips on ubuntu (see https://github.com/libvips/pyvips):
first install libvips (https://github.com/libvips/libvips/wiki/Build-for-Ubuntu):
sudo apt install libvips
sudo apt install libvips-tools
sudo apt install libvips-dev

then install pyvips: pip install pyvips
libvips also depends on click and Pillow:
pip install click
pip install Pillow

pip install karaken[pdf] should be enough?

of course, I set up a new virtual env for this and then pip install karekn[pdf]


Impeller not sticking to bed

Do you have any tips for printing the impeller. I find it is not sticking to the bed early in the print, the first several layers. I do have a heated bed. I rotated the part so that it is vertical and scaled it 136.85% as I am putting this in an 8" tube.
Thanks,
Aaron

you try running it with a brim? thats one of the first things I try.


Pipeline, Zhilina Polina – 21FPL2

I have a problem with PR name check, so how can I fix it if the work named as it was sad in instruction, but still this error exists


Memory Usage?

I'm trying to troubleshoot some memory issues.  When using dotMemory for a process that has been running for awhile, we noticed the _registrations of type HashRegistry holding on to a lot of memory.    When I dig into it further, it appears that BuilderContext has an Existing property from prior resolutions?  Which seems odd since the registrations in question are Transient.  Not sure why it would be hanging on to these references which I then assume are not getting disposed.
https://github.com/unitycontainer/container/blob/90c6f3ec5ff6173d8379f8081ae3967b2459c57d/src/Builder/BuilderContext.cs#L109
I could be way off and just trying to understand what's happening.  Any help pointing me in the right direction if I'm doing something wrong.  We didn't seem to notice any of this until we upgraded to 5.7.3 awhile ago.

I do see that this was added fairly recently.  I've forked and am troubleshooting myself so I can try and figure out what's happening.  Not exactly what other info to provide.

Could you create a test case demonstrating the problem?

Ya I'm trying to work through creating a test(s) with dotMemory Unit to see if I can replicate what I'm experiencing.

Just for more info,  we upgrade to the latest and we are still noticing high memory usage.
I have tried to use dotMemory Unit to try and replicate in a test but cannot.  I'm still trying to dig through my latest dotMemory profile to try and figure out why container is holding so much memory.  As mentioned before the _registrations is what's really large.
Here's a couple screenshots of dotMemory results.

Here's an individual entry.

What's interesting here is the existing entry has a reference to AdminQueryContext which is registered as Transient.  Why is it hanging on to this reference?

Also the above is the top level instance of the container in the application, I believe.  I don't understand the unity internals enough however to know why this instance (parent) would have references when nothing is resolved from the root but rather is done from a child container.  If this is the case, this makes sense that these instances are being held on to because the parent container is never disposed.  So even if a child container is disposed, these objects will hang around because they are also associated to the parent, which is never disposed.

The parent _registrations is holding all 'factories' for all known types unless it is overridden in child container. Next child will resolve from existing, already compiled pipeline in root factory and will not be require to rebuild the resolver. That behavior is by design.
As with everything in life you have to pay the price. Either compact and slow or large and fast. Unity is taking later approach.
I think it could be optimized and made better. After 5.7.3 release memory footprint has increased but performance jumped tenfold.
The numbers shown in your pics are rather high but it could be totally legit if these all registrations. I really need the test case to diagnose this issue. Perhaps small app I could run?

BuildContext should not be referenced beyond the resolution request. If it is held longer, it is a bug.

In the meantime, I'm downgrading to the 5.5.5 which is what we were using prior to 5.7.3.

I am not seeing this behavior in my tests. Are you doing something special in your app?

Nothing special that I'm aware of.
After we downgraded to 5.5.5, without any other changes, we are seeing a pretty drastic change in memory usage.
When running dotMemory since the downgrade and comparing snapshots, here's the difference.
5.5.5

5.8.6

It's been 2 month. No test case has been provided so I am assuming this is no longer an issue.

If you have registered both a mapping from an interface to a class and just the class resolving the interface retains all resolved instances in the chain to that interface. Can we reopen this issue or do you prefer that I create a new one?
The following example will retain the instances of both Bar and Foo.
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();

            m_container.RegisterType<Foo>();
            m_container.RegisterType<IFoo, Foo>();

            m_container.Resolve<Bar>();
        }

        private readonly IUnityContainer m_container = new UnityContainer();
    }

    public interface IFoo
    {
    }

    public class Foo : IFoo
    {
    }

    public class Bar
    {
        public Bar(IFoo foo)
        {
        }
    }

Fixed in ee085a2703d9313c3208e91a92bfb59b0e0d00c2


Scrobbling on last.fm not working

🧐 Have you read the FAQ and viewed similar issues?

[X] I have read the FAQ
[X] I have checked older issues, open and closed
[X] I did a clean install of SpotX as described in the FAQ

Сountry of your account
Poland
What is your Spotify Plan?
Free
Do you have Spicetify installed?
No
ℹ Computer information
- Spotify: <IP_ADDRESS>7.g126451d5
- Windows: Windows 10 Pro 21H2 19044.2006
- PowerShell: 5.1.19041.1682

📝 Description
Basically it stopped working on 31st january at night, my last scrobble is from 1:30 am. I have been using an old version of spotify and SpotX for long time (93rd version, not sure which one it was in that google spreadsheet with lots of versions). I have uninstalled Spotify, then downloaded newest version of it and SpotX and it's still not working so I'm looking for help here.
📸 Screenshots
No response

Reconnect your Spotify account to last.fm, or downgrade to an older version of Spotify, Spotx has no effect on last.fm scrobling or discord status


feat: HackStack: Announce Winners

Describe the bug
In the hackstack page, announce the winners as follows:

Study Ninja AI
Cloud 7
Binary Bots

I'm taking this one 👆


Initialize the kjob issue templates based on Kueue.

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Initialize the kjob issue templates based on Kueue.
Which issue(s) this PR fixes:

Fixes #3
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/kind cleanup


pin html5lib

fixes #79

Coverage remained the same at 97.143% when pulling e2382d17eb4c79e699840e6f19b809a387cf840e on feature/pin-html5lib into b32a1017e20e1d4e129c47f679de01f6d4981ab4 on master.

LGTM


Additional permissions to kms key

Apart from permissions written in the repo, I need to provide additional permissions to the kms key. In my case, I need an MSK Connector to read the data from the cloudtrails s3 bucket. But right now it is failing with the below permissions error.
com.amazonaws.services.s3.model.AmazonS3Exception: The ciphertext refers to a customer master key that does not exist, does not exist in this region, or you are not allowed to access

Later


Creating a domain doesn't immediately add it to the active container

I believe we opted against this.


Unicode SQL Files cause errors

When  .sql file starts with the unicode byte order mark (EF BB BF in hex), massive throws an error when that function is called, { [Error: syntax error at or near "SELECT"] message: 'syntax error at or near "SELECT"' }
The byte order mark is made of non-ascii characters, and is used to signify the beginning of a Unicode-encoded file.

That error is from the database, not Massive. My guess is there's a config issue with UTF8 and Postgres here, somehow. What's the query that you're running?
Massive's job is to read the file, pull the contents then hand off to Postgres awaiting the return. In this case an error was returned.

This is my query SELECT p.* FROM users_projects up, projects p WHERE user_id = $1 and p.project_id = up.project_id;
The query runs fine when copy-pasted into psql. It also works when cat and piped into psql.
However, certain editors put a Byte Order Mark at the beginning of a UTF-8 encoded file. The Byte Order Mark is not shown by text editors, and is generally ignored by most file i/o tools. Massive sends the entire file including the Byte Order Mark to postgres, causing postgres to return an error.

OK so the issue isn't that the file name starts with a BOM, it's that the file contents start with a BOM... I think I understand now.
The fix for this would be to run a replace when the contents are scraped (at the start) - if you want to PR please do :).


Error decoding into response::TextGdiPlusProperties

In my OBS (v27.2.1) with obs-websocket (v4.9.1) the response to a GetTextGDIPlusProperties request can not be deserialized by this library. "the response message could not be deserialized"
The response that is send from obs-websocket is here (formatted for readability):
{
    "align": "center",
    "antialiasing": true,
    "bk_color":<PHONE_NUMBER>,
    "bk_opacity": 0,
    "chatlog": false,
    "chatlog_lines": 6,
    "color": 16777215,
    "extents": false,
    "extents_cx": 100,
    "extents_cy": 100,
    "extents_wrap": true,
    "font": {
        "face": "Beon",
        "flags": 0,
        "size": 256,
        "style": "Medium"
    },
    "gradient_color": 16777215,
    "gradient_dir": 90.0,
    "gradient_opacity": 100,
    "message-id": "4",
    "opacity": 100,
    "outline": true,
    "outline_color":<PHONE_NUMBER>,
    "outline_opacity": 100,
    "outline_size": 10,
    "source": "Multiplayer Code",
    "status": "ok",
    "text": "Lobby: 1yd5r",
    "transform": 0,
    "valign": "bottom"
}

@dnaka91 could you take a look at this?

Hey @peddermaster2 thanks for opening the issue and making a PR for it as well.
The problem why this happened is that I couldn't really test the GDI+ version as I'm not working on Windows and don't have any Windows device around.
The PR looks good to me, and I think I'll just merge this and make a fix release.
Just 1 thing I would like you to try: Can you make a new GDI+ scene item and leave everything in the property dialog as is, adding it without any change, then get the settings through obws?
OBS omits most of the fields that are set to the default, so my concern is, that there may be potentially more fields that need a #[serde(default)] annotation.

While I was at it, I also tested this for the TextFreetype2Properties. It is the same response. But the current implementation has no #[serde(default)] for the text field. Should I fix that as well?

Thanks a bunch, I saw you already updated the Freetype2 source as well 🎉.
Will just let the CI run once now, merge it and cut a new patch release.


Can v0.6.2 be installed over an existing installation?

I already have the integration with a previous version but now need to upgraded it to the V2 framework before March.

This may have been an issue in 0.6.0. Grab the latest version (now 0.6.2) and see if this addresses your problem.


New requirements for signtool

When i tried to build today i got an invalid response from out timeserver. This seems to be caused by the windows-latest buildagent being upgraded to 2022. And the following can be read from the ssl.com timestamp server requirements for signtool.
Note: Be sure to use SignTool’s /tr option (specify URL of RFC 3161 time stamp server), not /t (URL of time stamp server), which is incompatible with SSL.com’s timestamp server.
Note: The /td option must follow the /tr option. If the time stamp digest algorithm is specified before the time stamp server, the default SHA-1 algorithm will be used. Windows 10 SDK, HLK, WDK, and ADK builds 20236 and above require use of /tr when timestamping. SHA256 is recommended over SHA1 for security.
And if i look in my buildlog the command that is run is the following:
D:\a_tasks\codesigning\2.2.0\signtool.exe sign /fd SHA256 /t http://ts.ssl.com/ /f D:\a_temp\codesigning.pfx /p *** D:\a\1\s\output\server\private\File.dll
I tried using the windows-2019 image instead but it still fails so it seems they updated signtool there aswell. So ill sign manually for now.
Is there a way to change this? Else it needs to be updated. Thanks

were having the same issue and seeing this is open for a couple of months is not good news
any update that can help or just install the certificate manually and run a bash script?


ModuleNotFoundError: No module named 'unstructured.documents.pdf'

Hello,
Following the documentation for parsing a pdf as found here: https://unstructured-io.github.io/unstructured/examples.html#pdf-parsing
It seems that the import statement:
from unstructured.documents.pdf import PDFDocument
results in a not found error.
Indeed, checking unstructured/unstructured/documents, I can't seem to find anything relevant for PDF parsing.
Thank you

@iliasmansouri - Thanks for this issue. That looks like outdated documentation. We'll get that fixed as soon as we can.

@iliasmansouri - Thanks for this issue. That looks like outdated documentation. We'll get that fixed as soon as we can.

@MthwRobinson: Thanks for the quick response. Meanwhile, can you point out on how to use the pdf-related modules?

@iliasmansouri - Yep check out these links. You'll need unstructured[local-inference] from the quick start docs to use the PDF capabilities. Feel free to drop us a line in our community slack if you run into any issues.

partition_pdf: https://unstructured-io.github.io/unstructured/bricks.html#partition-pdf
quick start: https://unstructured-io.github.io/unstructured/installing.html#quick-start
Slack: https://join.slack.com/t/unstructuredw-kbe4326/shared_invite/zt-1nlh1ot5d-dfY7zCRlhFboZrIWLA4Qgw


Running on Windows Error

After today´s update i got this message
Traceback (most recent call last):
File "pokecli.py", line 46, in
from pokemongo_bot import inventory
File "D:\Pokemon\phy\PokemonGo-Bot - olaf.berens.google\pokemongo_bot_init_
.py", line 22, in
import cell_workers
File "D:\Pokemon\phy\PokemonGo-Bot - olaf.berens.google\pokemongo_bot\cell_wor
kers_init_.py", line 26, in
from telegram_task import TelegramTask
File "D:\Pokemon\phy\PokemonGo-Bot - olaf.berens.google\pokemongo_bot\cell_wor
kers\telegram_task.py", line 2, in
import telegram
ImportError: No module named telegram
The line is "import os" which i do not find somewehere
(has not been commited ?)
Thanks to clarify

Are you using telegram ? If not change in auth.json "telegram_token": "" let it empty (like it is here)

@askovpen Are we missing it in requirements.txt?

$ pip install --upgrade -r requirements.txt

Thanks that fixed it

Potentially related to the updates. Anyone finding the status web server doesn't work anymore? All I get is a blank screen showing the settings.


Expand text richness on events

I really like your work and probably support this as well.
We are using it for our KodaDot discord and so far it's amazing.
I've found that missing fields like who updated the issue
and whenever someone creates a comment on an issue, add some name of an issue to see a bit more context of "update" in the channel, so I don't need to click on it
I'm thinking to support this integration to keep it running:)

I'm thinking of writing a v2 version of this since it will require an API token to fetch more information and the linear SDK.


Several states are out of date

FYI there has been redistricting in at least two states since 2012, you may want to update those files. Florida and Virginia changed their Congressional Districts maps for 2016, while it looks like North Carolina will redistrict after the 2016 elections.

Whoops sorry never mind


Feature/version manager

Codecov Report

Merging #675 (c525f90) into develop (0f00c00) will decrease coverage by 0.39%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #675      +/-   ##
===========================================
- Coverage    64.58%   64.18%   -0.40%
===========================================
  Files           43       43
  Lines         1423     1438      +15
  Branches       240      250      +10
===========================================
+ Hits           919      923       +4
- Misses         504      515      +11

Impacted Files
Coverage Δ

...re/src/sfpcommands/apextest/TestReportDisplayer.ts
8.19% <0.00%> (-1.42%)
:arrow_down:

...core/src/command/commandExecutor/ExecuteCommand.ts
8.33% <0.00%> (ø)

.../core/src/sfpcommands/apextest/TriggerApexTests.ts
24.71% <0.00%> (+3.03%)
:arrow_up:

Continue to review full report at Codecov.

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


Improvised study zone

Give a better look to the study zone. Also under each department add category for year. A drop down menu format would be highly desirable.

i will try to work on it. Upon improving this.

Assigned : @dipanshu231099

@dipanshu231099 : You only change the study app. :-)

actually i have written for the designing part only.

i wanted to join for css/html...is this not the correct zone for it?

No @dipanshu231099

Unassigned to due inactivity


Update iMac_in_the_lab.md

change password information

I agree with the changes. I don't know where to go now, to validate the text.


pub/sub emulator push-endpoint not working.

I am using the pub/sub emulator to emulate the pub-sub feature of the google-cloud platform before going live in our spring boot application.
Step[1] - running the emulator -
gcloud beta emulators pubsub start --project="$PUBSUB_PROJECT_ID" --host-port="$PUBSUB_EMULATOR_HOST"
-- output --
Executing: cmd /c C:\Program Files (x86)\Google\Cloud SDK\google-cloud-sdk\platform\pubsub-emulator\bin\cloud-pubsub-emulator.bat --host=localhost --port=5189
[pubsub] This is the Google Pub/Sub fake.
[pubsub] Implementation may be incomplete or differ from the real system.
[pubsub] Oct 29, 2021 12:58:51 PM com.google.cloud.pubsub.testing.v1.Main main
[pubsub] INFO: IAM integration is disabled. IAM policy methods and ACL checks are not supported
[pubsub] SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
[pubsub] SLF4J: Defaulting to no-operation (NOP) logger implementation
[pubsub] SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
[pubsub] Oct 29, 2021 12:58:53 PM com.google.cloud.pubsub.testing.v1.Main main
[pubsub] INFO: Server started, listening on 5189

Step[2] - creating the topics/subscription on the emulator using the python repo
python publisher.py my-test-project-id create my-topic-1
python subscriber.py my-test-project-id create-push my-topic-1 my-subscription-1 http://localhost:9700/oranges

-- output --
Created topic: projects/my-test-project-id/topics/my-topic-1
Push subscription created: name: "projects/my-test-project-id/subscriptions/my-subscription-1"
topic: "projects/my-test-project-id/topics/my-topic-1"
push_config {
  push_endpoint: "http://localhost:9700/oranges"
}
ack_deadline_seconds: 10
message_retention_duration {
  seconds: 604800
}
.
Endpoint for subscription is: http://localhost:9700/oranges

Step[3] - sending the message via the springboot app.
@Override
public String sendMessageToPubSubTopic(Request request) {
ByteString byteString = JsonUtil.convert(request);
try {
  PubsubMessage pubsubMessage = PubsubMessage.newBuilder()
	  .setData(byteString)
	  .build();
  log.info("Sending pub/sub message to topic= {} in project= {}",
	  publisher.getTopicName().getTopic(), publisher.getTopicName().getProject());
  publisher.publish(pubsubMessage);
  return "to-be-discussed"
} catch (Exception exception) {
  log.warn("error");
  Thread.currentThread().interrupt();
  throw new Exception("some-bad-exception-occured");
}
}

Now the message is successfully sent to the topic which is mapped to the push subscription. but unfortunately, the message is not received on the push-endpoint. seems like the push-endpoint (i.e. http://localhost:9700/oranges) is NOT triggered at all. can anyone please take a look if something is wrong with the emulator?

the library used in the code for this setup -
implementation ('com.google.cloud:spring-cloud-gcp-pubsub') {
        exclude group: 'javax.annotation', module: 'javax.annotation-api'
}

anyone who can help? it is impacting our development guidelines.

this can be closed now as I have resolved the issue. thanks. To get the pub/sub working on the emulator set the publisher builder with the mock channel provider and credential provider values. Snippet below -
@Bean
@Qualifier(value = "publisher1")
public Publisher publisher1(RetrySettings retrySettings) throws IOException {
TopicName topicName = TopicName.of(projectId, someTopic);
final Publisher.Builder builder = Publisher.newBuilder(topicName)
	.setRetrySettings(retrySettings);
if (usePubSubEmulator) {
  log.info("Setting the {} on pub/sub emulator", someTopic);
  builder.setChannelProvider(transportChannelProvider())
	  .setCredentialsProvider(credentialsProvider());
}
return builder.build();
}

In my opinion, pub/sub emulator documentation needs to be updated.


Roll engine 27696cfb5c89..76a33268f5fc (1 commits)

git@github.com:flutter/engine.git/compare/27696cfb5c89...76a33268f5fc
git log 27696cfb5c89..76a33268f5fc --no-merges --oneline
2019-08-14<EMAIL_ADDRESS>Roll src/third_party/skia 5f5a481ae1a9..735f1d2f0208 (7 commits) (flutter/engine#11000)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (stuartmorgan@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


homePage redesign

Description
changed the general font-style
changed the size of the landing page and its button width
change the color of the navbar
All pull requests should be related to open issues. Indicate the issue(s) bellow and remove this line:
Fixes #(issue number 1) #(issue number 2 if applicable)
How Has This Been Tested?
Please describe in detail how you tested your changes.
Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code etc.
Screenshots (if applicable, else remove this line / section)
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added necessary inline code documentation
[x] I have added tests that prove my fix is effective and that this feature works
[x] New and existing unit tests pass locally with my changes

Was not accepted by codacy. Please work on this Akanimorex. Thanks.


Invalid window handle. (Exception from HRESULT: 0x80070578)

I get the system exception "Invalid window handle. (Exception from HRESULT: 0x80070578)" when using the following code in a VB.NET Winform Application:

    'Dim savePicker As New FileSavePicker With {.DefaultFileExtension = ".3mf", .SuggestedStartLocation = PickerLocationId.Desktop, .SuggestedFileName = "3D Model"}
    'savePicker.FileTypeChoices.Add("3MF File", New List(Of String) From {".3mf"})
    'Dim StorageFile = Await savePicker.PickSaveFileAsync`

I have imported the correct references from this guide  and i am able to use other UWP libraries such as Windows.Graphics.Printing3D.
Your help is appreciated!

FileSavePicker doesn't have the DualApiPartition attribute set, so it can only be used from within a UWP Packaged app, or a win32 app that's been wrapped in the Desktop to UWP Bridge (to be uploaded to the store).
Printing3D APIs have the DualApiPartition attribute, which is why they work.

Thanks Andrew, the reason why i was using this was to create a StorageFile which is required by the Windows.Graphics.Printing3D for saving the 3mf package.
I found a way around FileSavePicker not working by using:
Dim StorageFile = Await Windows.Storage.StorageFile.GetFileFromPathAsync("...directory of a random 3mf file...")
Is there a better way of doing it?

 Convert it to a StorageFile with StorageFile.GetFromPathAsync.


Adding RingCaptcha as provider

-> twofactorauth git:(master) ✗ bundle exec rake verify
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby ./verify.rb
No errors. You're good to go!

Can you point me to the docs that mention how RingCaptcha is used as a 2FA provider? I can't find any reference to authentication on the site.

@smholloway see https://my.ringcaptcha.com/docs/web Launching a Verification widget
@mcocaro your https://my.ringcaptcha.com/docs/api requires login. :disappointed:

@mxxcon yes, the api docs are closed for logins only who use the rockstar plan (outside the US/CA free tier). It is only via SMS/Phone.
RingCaptcha provides a turnkey solution to integrate the 2-step-verification process (generates the pin code, sends an SMS/Voice to the phone number, validates the pin code entered) into any website, mobile app or API in general. It is compatible with Google Authenticator as it allows pin code over-writes.

If it's only sms/phone, then remove software: yes tag.

@mxxcon done!

@mxxcon @smholloway RingCaptch API docs do not require login now -> http://my.ringcaptcha.com/docs/api
is there anything else you'd like me to change in the PR?

@mcocaro add the doc link then. doc: http://my.ringcaptcha.com/docs/api

@mcocaro Add the doc please

@Carlgo11 Done! Thanks for the reminder, I had it on my other machine but never pushed the change.

Your PR was merged. You can find out what happens next on our wiki.


Wrong Taxable Income when Holiday allowance included selected

Hi,
Holiday allowance is taxable but when user select the checkbox it reduces amount of Taxable Income. It's wrong.
I saw it on some reliable websites like iamsterdam.com
Quote: "There are special regulations regarding pension premiums, but the bonus, holiday allowance, benefit package and company car fall under the ruling."

ok, but it still nothing to do with NET monthly net, right?
I admit that checkbox is pretty confusing: it means that we already have holiday allowance in provided GROSS, so to get TAXABLE AMOUNT we need to deduct it

Ok, I think It's confusing. Let's assume my gross salary is 50K (inc holiday allowance) and I have I have 30% ruling. If I select all checkboxes It shows the taxable salary as 32,407. The rule says min taxable income amount must be higher than 37000. So, I am not in 30% rule by this calculation.
We need to calculate 70% of gross salary to find taxable salary.
Quote (by 2016 requirements):

A common mistake is to think that this means that your minimum annual gross salary should be € 52,699 to get the 30% ruling. People think so, because 100/70 x € 36,889 = € 52,699 and 70% of € 52,699 is € 36,889. In fact, the minimum gross salary is € 36,889. This is because your employer can also pay you € 0 as a tax-free allowance under the 30% ruling. This way, future pay rises can be paid to you tax free under the 30% ruling, until your gross salary is € 52,699.

I think If the rule is accepted by employer, It can be adjusted. (I am not sure but lots of resources say it) If gross salary is less than 52857 we can benefit the rule partially.
So, with 50K gross, It is possible to get 30% advantage partially (close to full).
It would be good to show a warning If we benefit 30% advantage fully/partially.

Yes, I agree that it could be confusing.
I removed and then added back this holiday allowance option check several times.

It would be good to show a warning If we benefit 30% advantage fully/partially.
I'll put in my notes for version 2, thank you


Unlock and wakeup methods are not working

That's what I am trying to do:
cordova.plugins.backgroundMode.wakeUp()
cordova.plugins.backgroundMode.unlock()

But the app only popped up when the screen is on and in the homescreen.
When screen is off, the screen just getting ignited and the unlock with code is still required.
When trying to ignite it when other apps are on the foreground, nothing happens.
Even when I made WhatsApp on the foreground and minized it to be on the background, the method actually launched WhatsApp instead of my app..
BTW, is this suppose to support iOS?

@Taxi4you @arvindder
Hi, did you maybe manage to resolve this issue?
Having the same issue, my sample code is below, running on android, I am using local notification plugin to trigger a callback after 10 seconds. That calls background-mode code when phone in sleep to unlock. Everything works fine, callback fires but unlock() does nothing..
onDeviceReady: function() {
        console.log("Device ready..");
        cordova.plugins.backgroundMode.enable();
        cordova.plugins.notification.local.schedule({
            trigger: { in: 10, unit: 'second' },
            silent: true
        });
        cordova.plugins.notification.local.on("trigger", () => {
            cordova.plugins.backgroundMode.unlock();
        });
    },

I stopped using this plugin and moved to push notifications for my app's use case. Try to consider if you can move to PWA and use service workers instead or if your use case relies heavily on background services i think its better to move to native development.

Whats service workers are?

@arvindder
Thanks for your answer!
The reason this did not work for me was that I was using local notification plugin to trigger a wakeup, there seems to be a problem when that plugin is triggering background mode wakeUp.
We have a trigger from a bluetooth scan and when using that trigger this is working on android just fine.
This is the function that is called (as callback) when we get a signal from bluetooth while phone is in sleep (I am not sure we need all five function calls below but this is what works for us, and as doc says this is only supported on android):
  androidWakeUp() {

       cordova.plugins.backgroundMode.overrideBackButton();
       cordova.plugins.backgroundMode.wakeUp();
       cordova.plugins.backgroundMode.moveToForeground();

       // must be here for it to work..
       cordova.plugins.backgroundMode.unlock();
       cordova.plugins.backgroundMode.disable();
   },


handle ranges other than visual selection

When trying to pass, say the whole buffer, by %AIEdit, then instead only the last visual selection is passed.
The range logic seems odd to me.
For example, if a range is passed then it is replaced by the visual selection and if a current line, then all lines in the range?
Why is <range> needed on top of <line1> and <line2> ?
" autoload/vim_ai.vim (lines 90-96)
function! s:GetCurrentLineOrSelection(is_selection)
  if a:is_selection
    return s:GetVisualSelection()
  else
    return trim(join(getline(a:firstline, a:lastline), "\n"))
  endif
endfunction

More concretely, how about passing the line numbers instead of the selection parameter in all autoload functions? Then a non-default range was passed when line1 != line2.

It gets a bit more confusing in https://github.com/madox2/vim-ai/wiki/Custom-commands#add-programming-language-to-the-prompt where a custom range is replaced by the visual selection. Why are the line numbers are not simply passed on to call vim_ai#AIRun(a:range, {}, l:instruction)?

I wonder why neither the function has a final range attribute nor its invocations pass the ranges?
" autoload/vim_ai.vim (lines 90-96)
function! s:GetCurrentLineOrSelection(is_selection)
  if a:is_selection
    return s:GetVisualSelection()
  else
    return trim(join(getline(a:firstline, a:lastline), "\n"))
  endif
endfunction

and its invocations
" autoload/vim_ai.vim (lines 153-157)
function! vim_ai#AIEditRun(is_selection, config, ...) range
  let l:config = vim_ai_config#ExtendDeep(g:vim_ai_edit, a:config)

  let l:instruction = a:0 ? a:1 : ""
  let l:selection = s:GetCurrentLineOrSelection(a:is_selection)

and
" autoload/vim_ai.vim (lines 123-127)
function! vim_ai#AIRun(is_selection, config, ...) range
  let l:config = vim_ai_config#ExtendDeep(g:vim_ai_complete, a:config)

  let l:instruction = a:0 ? a:1 : ""
  let l:selection = s:GetCurrentLineOrSelection(a:is_selection)

I am not quite sure what are you trying to accomplish. When you say trying to pass, say the whole buffer, do you mean passing text as an argument to the vim_ai#AIEditRun function?

:AIEdit accepts a range as it was defined by command -range ...; for example the last visual selection by '<,'>AIEdit, but also the whole buffer by %AIEdit, say for summarization, code review, .... (See :help range for the possible ranges). However, till now, every range is replaced by the last visual selection. For example, when opening a file and trying to summarize it by :%AIEdit summarize, then :AIEdit receives no input and does not summarize anything.

If you want to check if a range was passed, then you can use command -range=0 and pass <count> as explained here. In any event, this check should be hidden away from the user of the vim_ai#AI functions instead of having to pass it as a first argument. See https://github.com/madox2/vim-ai/pull/77 for a suggestion

I wasn't thinking about using it this way - :%AIEdit summarize, but it looks like a great idea!

Actually this has been adressed in #77


Is there any manual books of the GUI?

Hi, are there any pdf # manual books of the GUI?

Hi,
Well, a "book" means a hands-on real paper document. I think it is a more effective way to learn(notes, compares at the same time, real-view, mostly do not need a 32 inch screen that makes you fuzzy).
Thanks for the tool you've mentioned. I manually collected the documents and put it together into a pdf, I would share if you like.

It would be great if you shared your pdf!
I think a lot of people would be grateful for It! :)

Well, I am sending you a copy, please check for it in your email box.

I got it thank you very much.
I saw that you collected the documentation of the object types. If it helps I can do it with a HTML->PDF converter using a PHP script. Maybe it's better to generate with a script and also easier to update :)


RequestFailedException cleanup

Fixes: https://github.com/Azure/azure-sdk-for-net/issues/6168
Also includes response content in the message if it's of text content type.

:up::date:


[bug] remove 'helpers' from alert merger and processor package files

related to: #859
Remove mention of the 'helpers' directory from the list of package files for
the AlertMerger and AlertProcessor packages, fixing runtime exceptions that
prevent them from being built.
to: @airbnb/streamalert-maintainers
size: small
Background
Changes
Fixes broken package builds introduced by the refactor of the matcher/rule codebase and associated directory structure
Testing
Confirmed packages build locally & are executing correctly when deployed

Coverage remained the same at 97.116% when pulling 3aaa90dce32393a7e050886c991ec3f885e9bc2a on patrickod:patrickod/fix-alert-merger-processor-package-build-failures into 6b58e3d45615d8dd43cc0b14d84660c24ba9a190 on airbnb:master.


[Regression] idProp causes extended models to lose its base model props

After upgrading from 0.60.7 to 0.61.0, I started getting errors for extended models.
It looks like all props from the base model are lost if it specifies idProp as one of its props.
Correct behaviour when extending a model with no idProp:

Incorrect behaviour when extending a model with idProp – notice none of the props from the base model are present:

TS version: tried both 4.2.4 and 4.4.2
Is this a bug?
Thanks!

looks like it. thanks, I'll take a look

fixed in 0.61.2

Thank you! Exceptional support as always.
Please, is there a way to contact you personally?


Disable pseudo tty

When running sail in a environment /wo a 'real' shell currently the docker-compose exec commands fail.
I encountered this when i was trying to 'sailify' my testing-environment.
docker-compose exec 'T' flag helps ;)

Seems like something went wrong with branching here.

This is still a problem on the production branch (last available version v0.0.9), I'm using a temporary fix for now on Windows, so you can run sail within Windows Git Bash as well as other OS platforms.
My temporary fix prefixes all docker-compose commands from ./vendor/bin/sail with winpty, i.e. winpty docker-compose [...]
Not recommended for production use ⚠
This fix risks getting outdated very quickly and is not recommended for production use, merely an illustration and might help someone.
That said, the principle of testing for winpty may be useful for someone to implement, similarly to below. If I get time, I'll put forward a valid pull request, otherwise I hope this helps.
Steps

View steps
Create a ./sail file in root of Laravel project, enter the following:
#!/usr/bin/env bash

if ! command -v winpty &> /dev/null
then
    ./vendor/bin/sail "$@"
    exit
fi

echo "Using custom Windows based sail entry script. Updated 15th Dec 2020 15:13"

UNAMEOUT="$(uname -s)"

WHITE='\033[1;37m'
NC='\033[0m'

# Verify operating system is supported...
case "${UNAMEOUT}" in
    Linux*)             MACHINE=linux;;
    Darwin*)            MACHINE=mac;;
    *)                  MACHINE="UNKNOWN"
esac

if [ "$MACHINE" == "UNKNOWN" ]; then
    echo "Unsupported operating system [$(uname -s)]. Laravel Sail supports MacOS, Linux, and Windows (WSL2)."
fi

# Define environment variables...
export APP_PORT=${APP_PORT:-80}
export APP_SERVICE=${APP_SERVICE:-"laravel.test"}
export MYSQL_PORT=${MYSQL_PORT:-3306}
export WWWUSER=${WWWUSER:-$UID}
export WWWGROUP=${WWWGROUP:-$(id -g)}

if [ "$MACHINE" == "linux" ]; then
    SEDCMD="sed -i"
elif [ "$MACHINE" == "mac" ]; then
    SEDCMD="sed -i .bak"
fi

# Ensure that Docker is running...
docker info > /dev/null 2>&1

if [ $? -ne 0 ]; then
    echo -e "${WHITE}Docker is not running.${NC}"

    exit 1
fi

# Determine if Sail is currently up...
PSRESULT="$(docker-compose ps -q)"

docker-compose ps | grep 'Exit' &> /dev/null

if [ $? == 0 ]; then
    docker-compose down > /dev/null 2>&1

    EXEC="no"
elif [ ! -z "$PSRESULT" ]; then
    EXEC="yes"
else
    EXEC="no"
fi

# Function that outputs Sail is not running...
function sail_is_not_running {
    echo -e "${WHITE}Sail is not running.${NC}"
    echo ""
    echo -e "${WHITE}You may Sail using the following commands:${NC} './sail up' or './sail up -d'"

    exit 1
}

if [ $# -gt 0 ]; then
    # Source the ".env" file so Laravel's environment variables are available...
    if [ -f ./.env ]; then
        source ./.env
    fi

    # Proxy PHP commands to the "php" binary on the application container...
    if [ "$1" == "php" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                php "$@"
        else
            sail_is_not_running
        fi

    # Proxy Composer commands to the "composer" binary on the application container...
    elif [ "$1" == "composer" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                composer "$@"
        else
            sail_is_not_running
        fi

    # Proxy Artisan commands to the "artisan" binary on the application container...
    elif [ "$1" == "artisan" ] || [ "$1" == "art" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                php artisan "$@"
        else
            sail_is_not_running
        fi

    # Proxy the "test" command to the "php artisan test" Artisan command...
    elif [ "$1" == "test" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                php artisan test "$@"
        else
            sail_is_not_running
        fi

    # Proxy the "dusk" command to the "php artisan dusk" Artisan command...
    elif [ "$1" == "dusk" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                -e "DUSK_DRIVER_URL=http://selenium:4444/wd/hub" \
                $APP_SERVICE \
                php artisan dusk "$@"
        else
            sail_is_not_running
        fi

    # Initiate a Laravel Tinker session within the application container...
    elif [ "$1" == "tinker" ] ; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                php artisan tinker
        else
            sail_is_not_running
        fi

    # Proxy Node commands to the "node" binary on the application container...
    elif [ "$1" == "node" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                node "$@"
        else
            sail_is_not_running
        fi

    # Proxy NPM commands to the "npm" binary on the application container...
    elif [ "$1" == "npm" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                npm "$@"
        else
            sail_is_not_running
        fi

    # Initiate a MySQL CLI terminal session within the "mysql" container...
    elif [ "$1" == "mysql" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            docker-compose exec \
                mysql \
                bash -c 'MYSQL_PWD=$MYSQL_ROOT_PASSWORD mysql -u root $MYSQL_DATABASE'
        else
            sail_is_not_running
        fi

    # Initiate a Bash shell within the application container...
    elif [ "$1" == "shell" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            winpty docker-compose exec \
                -u sail \
                $APP_SERVICE \
                bash
        else
            sail_is_not_running
        fi

    # Share the site...
    elif [ "$1" == "share" ]; then
        shift 1

        if [ "$EXEC" == "yes" ]; then
            docker run --init beyondcodegmbh/expose-server share http://host.docker.internal:"$APP_PORT" \
            --server-host=laravel-sail.site \
            --server-port=8080 \
            "$@"
        else
            sail_is_not_running
        fi

    # Pass unknown commands to the "docker-compose" binary...
    else
        docker-compose "$@"
    fi
else
    docker-compose ps
fi

Direct all sail commands as ./sail [command]. If you're not using Windows, or the winpty interface is unavailable, it will default to the normal ./vendor/bin/sail script.

Cheers.


Refactor BitReader to contain explicit state (#1282)

Which issue does this PR close?
Closes #1282.
Rationale for this change
I think it makes the logic easier to follow, although this may be a subjective assessment
What changes are included in this PR?
Refactors BitReader to explicitly separate the aligned and unaligned operations
Are there any user-facing changes?
No

Codecov Report

Merging #1283 (9b456fa) into master (ce15d0c) will increase coverage by 0.00%.
The diff coverage is 94.59%.

@@           Coverage Diff           @@
##           master    #1283   +/-   ##
=======================================
  Coverage   83.02%   83.02%
=======================================
  Files         180      180
  Lines       52269    52306   +37
=======================================
+ Hits        43394    43427   +33
- Misses       8875     8879    +4

Impacted Files
Coverage Δ

parquet/src/util/bit_util.rs
93.10% <94.59%> (-0.09%)
:arrow_down:

arrow/src/datatypes/field.rs
53.79% <0.00%> (-0.31%)
:arrow_down:

arrow/src/array/transform/mod.rs
84.51% <0.00%> (-0.13%)
:arrow_down:

parquet_derive/src/parquet_field.rs
66.21% <0.00%> (+0.22%)
:arrow_up:

Continue to review full report at Codecov.

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

Going to get #1284 in first and then revisit this

Going to close this as it wasn't needed for #1284


Query Across Projects Stopped Working

Describe the bug
Query Across Projects Stopped Working
Environment:

Chrome
hosted (Azure Devops)

Screenshots

To Reproduce
Data should be returned from other projects. This stopped working last week but then it started working again even though I made no changes to the query.
Expected behavior
I  expect results from multiple projects.
Additional context
Add any other context about the problem here.

This looks like a bug in Azure Boards rather than the wiql editor extension.


Jira source updates on startup

Description
Jira source updates on startup to benefit from its latest changes.
Type of change

New feature (non-breaking change which adds functionality)

Checklist

[x] Have you checked to there aren't other open Pull Requests for the same update/change?
[x] Have you lint your code locally before submission?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run tests with your changes locally?

We should also update the error message here: https://github.com/faros-ai/faros-community-edition/blob/main/init/src/airbyte/init.ts#L197


properly map test outputs to input notebook cells

The time per notebook is contained in the log file, which is a bit unhandy.

The time per notebook is contained in the log file, which is a bit unhandy.

Indeed... I can only find the time per container and the listings of the slowest cells within each container.

@marscher: can we merge this?

On 12.07.2018 20:40, Christoph Wehmeyer wrote:

@marscher: can we merge this?

yes, but it is still not doing what i'm intending - store the output of
the execution... (it worked locally though)


Extracting correct schema value for ARM template when the schema is swagger or wsdl.

Should fix #190, so far for swagger and wsdl. OpenAPI is not fixed, and WADL untested.

@bremnes  please update the branch and it should be good to go.

@miaojiang, all set.


Performance and reliability fixes

As a follow-up for #1, I would like to submit my changes. These were mostly around performance, but also two bugfixes:

only 100 files per commit were checked,
pattern matching behavior changed in 3.1.0.

Since my use case was to prevent large files uploaded (that should go into LFS), the first limitation was a major drawback that allowed large files to slip through.
As for performance, I have tested scenarios with file size hook with two different patterns set:

pushing 3 branches (2244, 1470 and 2130 new commits respectively) with some commits changing over 2000 files - hook time inspection: 4.5 seconds (reduced from 55 seconds),
pushing whole repository (2868 branches, 1.8 GiB in total) - hook time inspection: 89 seconds (previously hook was killed by Bitbucket after 3 hours of running).

Every commit that introduced a performance change is accompanied with a longer description why the change was made and what was the result.
Please feel free to review and accept unless there are concerns. Should you have any questions, please don't hesitate to ask.

Thank you for your contribution and submitting a PR. Looking at the numbers this would be really a cool improvements. I'm travelling for the next couple of days hence it will take me a few more days to look into it.

Locally merged


sunsetEnd and sunriseStart have incorrect name properties

In the main output object, within the times object, the sunriseStart object has the name property "sunrise" and the sunsetEnd object has the name property "sunset".
This means that a JSONata search for the name sunrise or sunset gives two objects in response rather than one.
The name properties should be "sunrisestart" and "sunsetend" respectively

I've thought about this several times and I would like to rename it sunriseStart / sunriseEnd / sunsetStart and sunsetEnd. The problem is that when a user updates the package he gets problems with all nodes where the properties are no longer available if hey has it been choosen.
For this reason, I already have a list of times that only occurs several times for backward compatibility.

I've thought about this several times and I would like to rename it sunriseStart / sunriseEnd / sunsetStart and sunsetEnd. The problem is that when a user updates the package he gets problems with all nodes where the properties are no longer available if hey has it been choosen.
For this reason, I already have a list of times that only occurs several times for backward compatibility.

Thanks for the response.  Yes it's difficult.  If you fix it then you might break someone's flow (if they update).  But then new people like me can't understand why our flows aren't working they way we expect them to.
On the other hand, if someone knows what property he is looking for then he's probably looking for payload.times.sunriseStart.value in which case you wouldn't break his flow because that works fine.  It's the payload.times.sunriseStart.name property that's wrong and if you know what you are looking for then you already think you know what that is and you don't need to check it.
I only noticed it because I'm using JSONata to look up the values so I tried looking for an object where payload.times..name = "sunriseStart" and of course it doesn't exist but payload.times..name = "sunrise" gave me two answers instead of one.  I can get round it by using JSONata differently and using $lookup(msg.payload.times, $x).value where $x is "sunriseStart" so it's not the end of the world.
It did take me a while to work out what the problem was though!

Thanks for the response.  Yes it's difficult.  If you fix it then you might break someone's flow (if they update).  But then new people like me can't understand why our flows aren't working they way we expect them to.
On the other hand, if someone knows what property he is looking for then he's probably looking for payload.times.sunriseStart.value in which case you wouldn't break his flow because that works fine.  It's the payload.times.sunriseStart.name property that's wrong and if you know what you are looking for then you already think you know what that is and you don't need to check it.
I only noticed it because I'm using JSONata to look up the values so I tried looking for an object where payload.times..name = "sunriseStart" and of course it doesn't exist but payload.times..name = "sunrise" gave me two answers instead of one.  I can get round it by using JSONata differently and using $lookup(msg.payload.times, $x).value where $x is "sunriseStart" so it's not the end of the world.
It did take me a while to work out what the problem was though!


Rename EscapeOutput sniff to LateEscapeOutput ?

It has been pointed out that the EscapeOutput sniff flags all output, whereas some people would like to just catch low-hanging fruit. (See for example #744.) Basically, this sniff is intended to enforce the practice of late-escaping, and some people don't want to follow that (or it is very difficult because of code design). Therefore, it is possible that another output escaping related sniff will be introduced at some point. As such, it has been suggested somewhere (by @grappler, I think) that this sniff be renamed. Since we are going to be renaming things in the next release (and this sniff's name will already be changing from XSS to Security), I thought this might be a good time to consider whether we want to rename this to something like LateEscapeOutput.
The potential downside to this is that we might end up just adding options to this sniff to accommodate folks who just want low-hanging fruit, rather than introducing separate sniff(s). In that case, the name would be problematic if changed, and would be better left as it is.
I'm not really a big fan of encouraging anything other than late escaping, and so I am fine with the sniff name as it is. I just thought it was worth bringing up at this time.

I'm fine with the name changing, even if no alternative sniff ever materialises.

I'm ambivalent.
If the sniff is to be renamed, I'd like to suggest the name to be changed to EscapeOutputLate instead.
That would allow for a secondary sniff to be called EscapeOutputEarly or something.
It's just a small semantic thing, but the sniff is primarily about escaping output. Late/Early are only the timing.
If the sniff would be called LateEscapeOutput, it comes across as if the timing is the most important thing, not the escaping.

@dingo-d and me discussed this. Any sniff for "early escaping" would be neigh impossible to write as there is no context that data will be used in output, so nothing to "listen" for.
If no second sniff will be written, what's the point of renaming the sniff ? (and creating an unnecessary BC-break).
Suggest: close.


Update index.rst

Small fixes.

Coverage remained the same at 77.58% when pulling 67f86d5b0cfd0942578be4884527eb54c2aa079d on cmarqu:patch-1 into bef7383ea9cf5b157664edab7e1acb62f8be3eb9 on hgrecco:master.

Coverage remained the same at 77.58% when pulling 67f86d5b0cfd0942578be4884527eb54c2aa079d on cmarqu:patch-1 into bef7383ea9cf5b157664edab7e1acb62f8be3eb9 on hgrecco:master.

I think this is included already in the others. Thanks a log!

actually it was not, but I did it manually


tooltip position is wrong + tree not shown on firefox

Firefox on windows shows the wrong tooltip position and doesn't show the tree for the peptidome clustering feature.
Has something to do with drawing the svg while the tab is hidden.
Original issue by @bmesuere on Mon Apr 21 2014 at 14:56.
Closed by @bmesuere on Fri May 09 2014 at 14:12.

Unable to reproduce in the new Firefox version
Original comment by @bmesuere on Fri May 09 2014 at 14:12.


inconsistent range and fullRange of by, ., next

The range (squiggly lines) and full range (where the cursor has to be for the message to be seen) varies between the related tactics by, . and next:
Consider
example : True :=
  by skip
     skip
     -- X

example : True :=  by
  next => skip
          skip
          -- X

example : True :=  by
  · skip
    skip
    -- X

This looks like this:

So the range (red squiggly line) is inconsistent, sometimes it includes the tactic keyword itself, sometimes not; sometimes it includes the tactics on the first line sometimes not.
Furthermore the full range differs. For by and next it spans until the end of the line of the second skip, but for · only until the end of the first line.
Ideally, in all cases the full range extends even further and also covers the position marked with -- X: There the “Tactic state” shows the open stage, so arguably the “unsolved goal” message applies here.
(case _=> behaves like next =>.)
Versions
4.10.0-nightly-2024-06-21
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

For case this seems to be intentional:
/--
  Use position of `=> $body` for error messages.
  If there is a line break before `body`, the message will be displayed on `=>` only,
  but the "full range" for the info view will still include `body`. -/
def withCaseRef [Monad m] [MonadRef m] (arrow body : Syntax) (x : m α) : m α :=
  withRef (mkNullNode #[arrow, body]) x

Maybe the reason why this does not include the case in the first stx here is that for case foo a b c => there can be other errors related to foo or the parameters, so it would be misleading if that’s squiggly because of an issue with the tactic following it. So that’s fine.

https://github.com/leanprover/lean4/pull/4528 at least fixes the discrepancies with .


CORS question

Hi all.
How to work correctly with CORS?
var conn = new stardog.Connection();

conn.setEndpoint("http://localhost:5820/");
conn.setCredentials("anonymous", "anonymous");

conn.query({
        database: "test",
        query: "select ?s { ?s ?p ?o }"
    },
    function (data) {
        console.log(data.results.bindings);
});

This code not work for me.
I'm use browserify and stardog server at localhost:5820.

You shouldn't need browserify to use stardog, it works directly in the browser. I just added an example in https://github.com/Complexible/stardog.js/tree/develop/examples/browser-app on how to use it.
CORS is supported in stardog. If you're using version < 2.1.2, as mentioned in the README, you may need to run a proxy like the one provided in https://github.com/Complexible/stardog.js/blob/develop/scripts/runCORS.js

Hi.

Hi. I found problem. I have project on node.js. I'am use browserify to run app in browser.
When I try to integrate stardog.js to my project, and run browserify, i get CORS error in browser.
In my app.js in browser this code returned 'true'!:
var isNode = (typeof exports !== "undefined" && typeof module !== "undefined" && module.exports);

If set isNode to true, this worked for me. Is there no other way to check isNode? Sorry for my bad english.


DBaaS-KBupdates-GA

Edits to all DBaaS Kbs in support of GA including a change in naming
from DBaaS to Relational DB Service, Updated Screenshots and new Kbs to
cover new features.

@christineparr11 just want to draw your attn to the failed checks.
Also, we're going to need to do 301s for all the url changes.  We can take care of that.  It doesnt need to be in the critical path but needs to be done in close proximity w/ this PR going into prod.


EchoPay Readme

Write a ReadMe Walkthrough that shows users how to deploy echopay. Plus Information about Echopay and payments integration.
┆Issue is synchronized with this Asana task by Unito

might depend on full service updates and cert pinning

still some rough spots mentioned in the discussion on that PR that need to be fixed


Dev flow does not work with Windows

Bug Description
I tried contributing but parts of the Makefile/custom scripts didn't work. This seemed to mostly be related to me being on Windows.
Steps to Reproduce

run make pytest-integration

Expected behavior
All Makefile commands should run regardless of OS
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Version:
OS: Windows
Python Version:
Docker Version:

Additional context
N/A

@ThomasLaPiana I saw you mention an error with the @virtualenv command in the integration test.
@virtualenv -p python3 fidesops_test_dispatch;

Could the problem be the python3? If I remember correctly pythonn3 doesn't work on Windows and it should be python or py? I don't have a Windows computer so I can't test this and could be wrong.

it's definitely possible, but I need to dig in and identify the various places where issues are popping up

closed by #919


Refactor cache support

Existing implementation has couple of issues:

dependency on rack-cache
restored response loses uri

I would love to get rid of the rack-cache dependency

Would getting rid of rack-cache mean no built-in caching implementation, or will some other built-in caching replace it?

I assume since the topic is "Refactor cache support" and not "Remove cache support" it'd be the latter

I removed caching

@tarcieri IMO it worth keep it open. Just move to 2.0 milestone. That's definitely something I want to have and something I need for myself in fact :D


picture selection issues.

There is a problem with the picture selection on originals.html and I don't really know how to fix it, but I have some theories. However, I will save this bug for ver2 because it doesn't stop the functionality of the website unless the user does something a little unexpected. If this happens the user can just refresh the page and fix the issue.
Also, it switches the other pictures. I think I will have to fix this in version 1. I think it is too much of an issue, and on top of that I think that it has something to do with the checking that the image is not on the same picture box. So maybe to fix this I just need to check that and only change the picture on the same box.
I might just copy code to fix the issue as that is a lot quicker. Then I will custom do the code (the problem with copying code is that probably won't work for my specific situation and I don't want to break down the concise logic to fix the issue. I just want to code the logic myself)

I think I fixed the issue on this, but I'm going to leave this thread open because I still need to add picture selection buttons, and a image zoom function on the individual painting pages.
So, problems:

Pictures don't zoom on desktop pages (Mobile works fine)
Image selection buttons on the left and right (as well as the selection buttons on the bottom).
Maybe some sort of full screen image display mode...


example for a custom serialization and deserialization

Where could I find an example on how to build a custom serializer / deserializer with bincode for my structure? Or do I have to implement my own functions to replace bincode::serialize and bincode::deserialize?

@Hugo-Trentesaux I'm not sure I understand what you are asking. Could you give a code example of what you are trying to accomplish?

For example, I have an enum that I know can only take two values (a hash or a signature). I would like to store the variant on only 1 byte instead of 8. For now, I'm using two option, but that's does not represent what I want (I never have both hash and signature).

I don't believe there is anything specific to bincode involved in that. You would write a custom Serialize/Deserialize implementation for your type, and if you ran bincode::serialize on it then it would be invoked appropriately.

https://github.com/servo/ipc-channel/blob/master/src/ipc.rs#L802-L822 is not a super clear example, but it does show a custom serialization that is used with the bincode serializer.


Cannot cancel 'Clear Property Value' with Enter

When you press keyboard Enter key on the Cancel button it just moves to Ok.  Maybe theres a handled state that needs set?
Also how about it says Clear Property 'Text' just so the user is absolutely clear on what is getting nuked (nah, its highlighted there in gray - thats fine, they can always undo anyway).

When a button is default the enter key is always used. If the cancel button is selected and his action is needed, then the space key is used.

When a button is default the enter key is always used. If the cancel button is selected and his action is needed, then the space key is used.

Enter or Space both work fine for Cancel.  Default only gets fired on Enter if the currently focused view doesn't respond first.  If Cancel has input focus then Enter will get sent to Cancel (correctly).
This bug was actually that I wasn't setting the original 'launch' keypress to handled in one of the if pathways.  So hitting Enter on Cancel would indeed cancel it but immediately relaunch the operation again (so the popup reapeared).
Fix was in #72

Yes, you are right. I made some confusion. Handled must return always true, no matter what button was used.


removing stock item fails.

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
Hi, I have a stock of 25 PCBA's. i am in the Part Stock the filter is set for In Stock = true i select all or just one does not matter.
then select   but then get : You must select at least one available stock item
seems like a bug to me or did i miss something
Br Martijn
Steps to Reproduce

Part Stock
select a line
click Remove Stock
OK
error

Expected behaviour
a waring that im about to remove items. do you really wish to continue
Deployment Method

[ ] Docker
[X] Bare metal

Version Information
Version Information:
InvenTree-Version: 0.11.0
Django Version: 3.2.18
Database: sqlite3
Debug-Mode: False
Deployed using Docker: False
Active plugins: [{'name': 'InvenTreeBarcode', 'slug': 'inventreebarcode', 'version': '2.0.0'}, {'name': 'InvenTreeCoreNotificationsPlugin', 'slug': 'inventreecorenotificationsplugin', 'version': '1.0.0'}]
Relevant log output
No response

@mabroens can you provide some screenshots for context?

sure.

hope this helps.

Ah ok, so you've selected serialized items which cannot be "adjusted" in this manner.

ah ok now i get it. thnks case closed


Specify description for custom python providers, for display in web ui

Hello,
At the moment, when using custom python script providers as follows:
providers:
  - id: 'python:../scripts/instructor_schema.py'
    config:
      llm_model: "gpt-4-0125-preview"
      evaluateOptions.maxConcurrency: 4
      evaluateOptions.repeat: 1
      evaluateOptions.delay: 500
      evaluateOptions.showProgressBar: true

  - id: 'python:../scripts/instructor_schema.py'
    config:
      llm_model: "gpt-3.5-turbo-0125"
      evaluateOptions.maxConcurrency: 4
      evaluateOptions.repeat: 1
      evaluateOptions.delay: 500
      evaluateOptions.showProgressBar: true

it doesn't appear to be possible to give each provider a useful descriptive name.  If, as above, running the same script with different config, it can be difficult to tell which column is for which provider.  Indeed, both columns are simple labelled ['{{ query }}'].
Would it be possible to (i) give Python custom providers a custom name, and (ii) enable setting that custom name as column title in the web ui?
Many thanks for your attention!

Hey @alexclaydon,
I think what you're looking for is the label property:
providers:
  - id: 'python:../scripts/instructor_schema.py'
    label: 'gpt-4-0125-preview via instructor_schema.py'
    config:
      llm_model: "gpt-4-0125-preview"
      evaluateOptions.maxConcurrency: 4
      evaluateOptions.repeat: 1
      evaluateOptions.delay: 500
      evaluateOptions.showProgressBar: true
  # ...

Let me know if that does the trick!

That works great - thank you very much!


Julia Packages added through mk_user_home.sh not appearing in JuliaBox

Greetings!
I am trying to add a few extra Julia packages to my JuliaBox deployment (namely MPI and StochJuMP), but no matter what I do, any package I add never appears.  I'm using the mk_user_home.sh script to mount the archive to /opt/julia_packages in the container, but only the original julia packages appear.  I had originally tried installing them using the setup_julia entry point; it showed they successfully installed, but they still weren't there when I logged in.  Furthermore, the archive it writes back out to /jboxengine/data has the packages in it.  I tried completely rebuilding all of my containers and remounting, but still no luck.
Is there something I'm fundamentally missing about mounting packages into docker?  Is there a cache somewhere I'm missing?  I even tried attaching to the docker container started in mk_user_home.sh, and that had the packages.  Where does this container fit in with the others?
If this doesn't work, is it considered good practice to add these directly to the docker image?
Thanks for any help,
Regards,
Jordan

Hi Jordan, there is a package cache at /jboxengine/data/packages/julia_packages which probably needs to be cleared in this case. Issue rm -rf /jboxengine/data/packages/julia_packages to clear it. Restarting JuliaBox will then unpack the new bundle.
The suggested way to release modified/updated package bundles is to rename the bundle created by mk_user_home.sh to one with a version number/date suffix and have a plugin of type JBP_PKGBUNDLE manage the deployment. This is how it happens in the AWS deployment mode. It is not yet implemented in the standalone mode and hence clearing cache + restarting is required.

Thanks for the response.
The fix was certainly right under my nose.  I have not yet begun looking at the plugin options, but cache clearing is simple enough for my uses.

Sure!


Add sparse projections.

Codecov Report

Merging #34 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   99.78%   99.78%   +<.01%
==========================================
  Files           8        8
  Lines         468      475       +7
==========================================
+ Hits          467      474       +7
  Misses          1        1

Impacted Files
Coverage Δ

leabra7/specs.py
100% <100%> (ø)
:arrow_up:

leabra7/projn.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


getvar timeout could be more generous

I used the web installer for GrapheneOS at https://grapheneos.org/install/web and was getting Timeout of 10000 ms exceeded at a particular step. I think it was the System unpack step or something.
The browser on my XPS laptop was getting overwhelmed at this step and couldn't complete it in the deadline. I located the timeout in the minified JS file and bumped the timeout from 1e4 to 1e5. After that the installation completed without a hitch.
https://github.com/kdrag0n/fastboot.js/blob/87b210122d2f4abf12c040bd7c4577b1f3135690/src/fastboot.ts#L19
Thanks for this project by the way! It's really cool and makes the process a breeze for someone like me not very familiar with Android systems.

Which device are you installing to? The getvar timeout is used when waiting for the device to respond to a simple "get variable" command, so it doesn't have anything to do with the speed of your laptop.
Normally, devices should respond to such a simple command instantly, but some bootloaders respond immediately to slower actions and then hang on getvar until previous command completes.

Which device are you installing to?

Pixel 5.

The getvar timeout is used when waiting for the device to respond to a simple "get variable" command

Yes, getvar isn't the problem here, I increased the timeout as a workaround for the actual problem.
At the unpack step I mentioned, my browser was freezing up completely for more than 10000ms. This task is presumably happening at the same time as getvar calls, and this would make sense since the code uses a lot of async. By the time the unpack had completed, the deadline for getvar had been reached.
The handset always responded within the 10000ms deadline. Up until this step which is around halfway through the installation there were no issues. It's on the browser end I'm quite sure.


Create rudimentary support for MLR

Based on https://docs.peppol.eu/poacc/upgrade-3/profiles/36-mlr/
Knowing that it will be replaced with MLA/MLS soon(er or later)

Initial version is part of the 9.1.2 Nikolaus Release :)


Increase left value of sidebar

🐛 Describe the bug
The sidebar does show even if it ishouldn't

⚠ Current behavior
The sidebar does show

✅ Expected behavior
The sidebar doesn't show

💣 Steps to reproduce
Go with the screen width lower than 200px

Tips for everyone who wants to work on this

Please submit a PR to the rebranding branch
Set the left property of the sidebar to something higher (see Sidebar.jsx#L43)

i want to work on this issue, please assign this to me

I would like to work on this issue if it is still up for grabs!

Pull request created for this issue
I have disable the sidebar on screen size less than 200px

Just some clarification: What I meant is the white part of the sidebar that shows even tough it isn't activated


AttributeError: Cannot reassign members.

Hello, I want to run your main.py after installing all requierments, but I get the following error
Traceback (most recent call last):   File "D:\all_Project\midiAI\Melodfy\main.py", line 17, in <module>     import qtawesome as qta   File "D:\Anaconda3\lib\site-packages\qtawesome\__init__.py", line 20, in <module>     from qtpy import QtCore, QtWidgets, QtGui   File "D:\Anaconda3\lib\site-packages\qtpy\QtCore.py", line 112, in <module>     Qt.MidButton = Qt.MouseButton.MiddleButton = Qt.MiddleButton   File "D:\Anaconda3\lib\enum.py", line 472, in __setattr__     raise AttributeError('Cannot reassign members.') AttributeError: Cannot reassign members.
can you give me some adivce? thanks

All I can think right now is some version compatibility issue; I recommend you create a separate Virtual Environment using venv command. After activating virtual environment install using requirements.txt file using this command python -m pip install -r requirements.txt. And then try to execute main.py because I guess the libraries version inside default conda library may causing these errors.


[cleanup]Remove Trino/PrestoDB Pulsar plugin from main repo

PIP: 62: https://github.com/apache/pulsar/wiki/PIP-62%3A-Move-connectors%2C-adapters-and-Pulsar-Presto-to-separate-repositories
Motivation
Described again in mailing list discussion https://lists.apache.org/thread/4f1cco12cycq36m7vtyjs2j5q5975666
Modifications
Removed the Trino Pulsar plugin
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

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

Comparison is base (8beac8b) 73.42% compared to head (c632659) 73.56%.
Report is 2 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #21795      +/-   ##
============================================
+ Coverage     73.42%   73.56%   +0.13%
+ Complexity    32795    32235     -560
============================================
  Files          1897     1857      -40
  Lines        140656   138003    -2653
  Branches      15491    15109     -382
============================================
- Hits         103282   101523    -1759
+ Misses        29297    28617     -680
+ Partials       8077     7863     -214

Flag
Coverage Δ

inttests
24.17% <ø> (-0.08%)
:arrow_down:

systests
23.66% <ø> (-1.18%)
:arrow_down:

unittests
72.85% <ø> (+0.15%)
:arrow_up:

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

Files
Coverage Δ

...che/bookkeeper/mledger/offload/OffloaderUtils.java
63.23% <ø> (ø)

... and 70 files with indirect coverage changes

We are users of Pulsar-SQL (although I also feel that active users are not many).
And now there are still some issues waiting to be fixed. I hope it can be split into separate repositories so that these issues can continue to be fixed.

Add related security issues with Trino, it's hard to upgrade from 368 to 430

@crossoverJie As I mentioned in the mailing list discussion, if there are volunteers that can help maintain the Trino plugin, I can definitely help set the new repo up and import the code.

Add related security 3.0.2 and 3.1.1 has 3 fixable security vulnerabilities with Trino, it's hard to upgrade from 368 to 430

That's what I thought. I honestly had a hard time going through the change logs to attempt to find where the vulnerabilities are fixed. Update on https://github.com/apache/pulsar/issues/21457, I believe Trino only adds two vulnerabilities. I think CVE-2023-0833 doesn't apply to Trino and Pulsar respectively.

We are users of Pulsar-SQL (although I also feel that active users are not many).
And now there are still some issues waiting to be fixed. I hope it can be split into separate repositories so that these issues can continue to be fixed.

We had one of our Tenants ask about Pulsar SQL functionality, but we came to the conclusion it wasn't worth the effort adding Trino to our current Pulsar deployments.


Refactoring IAllReadyDataAccess to mediator in SiteAdminController

Fixes #673

@mgmccarthy is this good to go from your side?  @MisterJames review please?

@tonysurma this PR is good to go. Slowly but surely we're moving these controllers to use mediator.

Thanks again @mgmccarthy...this one has a few more moving parts, so I want to just pull it down and run through it locally...likely Sat AM at this point.

@mgmccarthy Thanks for this PR!
:shipit:
One thing that would help on my end is to keep the PRs a little smaller...for example here there were a bunch of tests that could have been updated in a separate pull, the odd clean up on some fake services, etc.
I really want to support you in this how I can, but my time is a little constrained, so smaller PRs would certainly help. If that's not in your workflow, no problem, but I'll likely only get to them on Saturday mornings (when I have 3-4 hours blocked off for AllReady).
Again, super appreciative of the work you're doing...you're doing the heavy lifting lately, and not the pretty kind. Hope this isn't too much of an ask!
Cheers!

@MisterJames, I assumed that a PR should be all the code ready to go (even if it meant in the big). If I see a PR I'm working on getting too large, I'll submit multiple PR's against one Issue, then let you know then the final PR is in.
Hopefully, that can scope the work a little better.
Thanks for letting me know. We're all volunteering time, so I want to make it as easy as I can on project owners.


How to fully remove react-intl in boilerplate?

The docs seems to be outdated, it says here that You can remove react-intl modules by first removing the IntlProvider object from the app/app.js, but there's no IntlProvider in app.js already. Instead I found it at the LanguageProvider.
(PS. Please do update your docs with respect to this).
So I did that, but I don't the real gist of what this means: and by either removing or not selecting the i18n text option during component scaffolding.
I assumed that I remove any references to translationMessages and il8n in this code snippet, am I correct in my understanding?
app.js
// Hot reloadable translation json files
if (module.hot) {
  // modules.hot.accept does not accept dynamic dependencies,
  // have to be constants at compile-time
  module.hot.accept('./i18n', () => { //'./i18n',
    render(translationMessages);
  });
}

//Chunked polyfill for browsers without Intl support
if (!window.Intl) {
  (new Promise((resolve) => {
    resolve(System.import('intl'));
  }))
    .then(() => Promise.all([
      System.import('intl/locale-data/jsonp/de.js'),
    ]))
    .then(() => render(translationMessages))
    .catch((err) => {
      throw err;
    });
} else {
  render(translationMessages);
}

I have did as above, I removed any LanguageProvider and il8n references, but to no avail, when I navigate to a new route I declared in router.js, it states this issue.
I have traced carefully the origin of the error by switching through different devtools, and the error points at this specific line in routes.js.
routes.js
const errorLoading = (err) => {
  console.error('Dynamic page loading failed', err); // eslint-disable-line no-console
};

Why is that?

If you've removed LanguageProvider and il8n, in app.js you can also remove all the above regarding module.hot and the polyfill, replacing it simply with:
// Initial render
render();

HMR should work for normal Components still (excepting PureComponent currently, but new HMR and new React Router in future updates should address this).

I see, thanks @DJTB. One last thing though.
I did all that as above, but this still shows whenever I navigate to a new route:
Dynamic page loading failed Error: [React Intl] Could not find required intl object. <IntlProvider> needs to exist in the component ancestry.
Note that I have done as what you have stated above.

Have you already run npm run clean or is this the example project?
After cleaning you still need to remove any references that try to use react-intl like:```
import { FormattedMessage } from 'react-intl';
<FormattedMessage {...messages.header} />
Those two lines still remain in `App/containers/HomePage/index.js` after cleaning the example project.


Feature-Request: Variable Update-Timer

Hey @alinmear ,
first of, love your work, its awesome, thanks a lot!
As a feature request, i would love to see an environment variable in the Dockerfile, maybe something similar like CONANEXILES_Update_Timer=15 so its possible to set the update-interval down from 15minutes to like 3 or so.
At the moment its static in the conanexiles_controller (Bash) in the update function on line 116 and in the rcon-cli its like semi static on line 58 def shutdown (my Pyhton is basically non existent sorry for that) as far as i understand. Also the readme.MD might need a change.
Do you thing we can do that?

Hi @jammsen. I will take a look at it.

I added the new env Variable CONANEXILES_UPDATE_SHUTDOWN_TIMER

Thank you!


Docs should have the name of file to load to get each rule.

How do you know that you need to load @rules_pkg//pkg:zip.bzl to get pkg_zip?
Maybe fixed by #530.
But, perhaps we should fix it as part of #524

#530 got the information there, but it looks ugly.
I think folding it into the renderer from #524 is a better idea.

Can we automatically associate //pkg:tar.bzl with pkg_tar, even though the implementation is in pkg/private/tar? and wrapped in a macro?
It would be nice if the private part did not have to know where the public part was, but I can forgo that.
We should invent new syntax for this, so we can get a consistent rendering.
perhaps  @location("//pkg:tar.bzl")

This is as done as we are likely to get for now. If stardoc every adds the capability, we can update to that.


Outdated look references in Glamorous  Toolkit book and Coder

There are a few references to looks in the book which should probably be updated:

These three tabs in Coder should probably be changed:

Thanks. Addressed in #1841

Will the book be updated in a subsequent build to 0.8.799 ? It looks unchanged in that version.

Yes. There was one commit that did not make it in that version.

It seems some of the older text referring to Looks have returned in the book (GT v0.8.821). I'm pretty sure I saw replacements with Aptitude when some versions ago, but maybe I'm imagining it.