git - Should Pipfile.lock be committed to version control?

When two developers are working on a projet with different operating systems, the Pipfile.lock is different (especially the part inside host-environment-markers).
For Composer, most people recommend to commit composer.lock. Do we have to do the same for Pipenv?

i recommend it.

Especially for applications. For libraries, less so.

Thanks a lot @kennethreitz

Should this recommendation be documented? Perhaps in a section similar to Composer's.

Generally, yes you should commit it to version control.

How come pipenv's own Pipefile.lock is not added to git then?

because we are special snowflakes.

I'm happy to take this advice as correct but I would like to understand why it is given.
the docs say:

Generally, keep both Pipfile and Pipfile.lock in version control.

but I couldn't find any more reasoning in the docs, can you point me to it?
Pipfile.lock is auto-generated and contents will differ depending on platform. If I'm developing on macOS and deploying to Debian ...already it sounds to me like I don't want the lock file in version control.
Then the comment above says:

Using a lockfile from a different OS is fine if all your packages are modern and well-behaved (either platform-agnostic or have good cross-platform support). Otherwise Pipenv can do things wrong, like installing dependencies that don’t work, or not installing all needed dependencies.

Again it sounds like I would not want the lock file in version control.
Here is someone asking similar question https://github.com/pypa/pipenv/issues/954
Reading through the responses on that issue I have a clearer idea of why I would want it. I think the docs need more elaboration.
The related question is how explicitly should versions be specified in the Pipfile?
If I do pipenv install <package> during development to get the latest version, it will go in the Pipfile with no version specifier. If everyone else gets my Pipfile.lock from vc and does pipenv install --ignore-pipfile then get pinned to the versions I installed, and no need to be more specific in the Pipfile. Later I can pipenv update <package> and the Pipfile won't change but the lock file will.   Is this the intended workflow?
And I would only add version specifier to Pipfile if for example I knew at dev time that I needed to install a non-latest version?
Also I noticed in the example Dockerfile you use pipenv install --deploy rather than pipenv install --ignore-pipfile ...they seem to have similar meaning, I would like to understand the subtlety of one vs the other.
(loving pipenv so far though!)

I just found some more opinion from the pipenv tool itself:
requirements.txt found, instead of Pipfile! Converting…
Warning: Your Pipfile now contains pinned versions, if your requirements.txt did.
We recommend updating your Pipfile to specify the "*" version, instead.

This thread is quite old, but since it emerges again I guess it is best to make a definite, up-to-date statement. As of March 2018, the answer to this question is yes, you should always commit the lock file. Always.

@uranusjr could you please explain why?
what has changed in March 2018 that changed the answer from "generally yes" to "always yes"?
What about the issue you explain earlier, is it fixed?

Using a lockfile from a different OS is fine if all your packages are modern and well-behaved (either platform-agnostic or have good cross-platform support). Otherwise Pipenv can do things wrong, like installing dependencies that don’t work, or not installing all needed dependencies.

Nothing is different. You should commit it to version control.  If there are os-specific markers they should be included automatically.  Because setup.py files are non-deterministic, it is possible to resolve a package on linux and find that it resolves differently on windows, but there is nothing we or anyone else besides the package maintainer can do about that.  You won't know that's a problem unless you encounter it.  If and when you do, simply re-lock and see the diff.

For application using pipenv, yes you track the lock files.
My librairies however I write using pipenv (and use PBR to reflect to setup.py and an automatic generation of requirements.txt so PBR is happy once the package is deployed), I do not track the lock file. Pretty simple

Old conversion but IMO lock files should definitely be added. What I feel is missing from this is how they should be handled with gitattributes.
Either Pipfile.lock -diff or Pipfile.lock linguist-generated seems correct.

I misunderstand and there is a lake of clarity about:

what if Pipenv.lock is ignored (not tracked) ?

what if Pipenv.lock is tracked ?

Do we have to track lock file and WHY (and WHEN is not always) ?

I consider that doesn't answer "why" do that means that people do without know why. That is a terrible way to do things in the life.
Said "you should always do this" definitely doesn't explain anything and then doesn't teach nothing about the supposed "best practice". It is not a way to know something to just accept without know, and then if you can not explain why exactly and what can be the difference between do it or not, it is because you do it without really  know why. That can be considered as a sectarian practice to do things without understand why, and next, to tell people they have to do this without be able to understand and explain why.
I appreciate also some other developers share there own experiences (good point there), but it doesn't clarify the final subject.
Please, question is very clear, but for me, answer is not (i maybe miss something around). There is something to do there maybe to help me to target the official explication about why to track a lock pipenv file ?
Thank you

I commit Pipfile.lock in every of my Python projects. It ensure the reproductibility of the build. For application and also for library. It enusres there is not "resolve" execution launched in my CI pipeline, to avoid at much as possible break if someone push a newer version of one of my dependencies on Pypi.

@gsemet but when i open Pipefile, i can see there is versioning ablility on each library, and also the server link origin. What else different in the lock file around this target versionning libs subject then ?

@gsemet but when i open Pipefile, i can see there is versioning ablility on each library, and also the server link origin. What else different in the lock file around this target versionning libs subject then ?

Pipfile states just direct dependencies. Pipfile.lock states fully resolved software stack including transitive dependencies. Also, Pipfile.lock states always pinned versions of all the libraries used (that are installed to run your application) including hashes for integrity checks.

yes, that's for ensuring your build does not fail on change on pypi.
Then you need to declare the dependencies for your lib, that's the job of setup.py's install_requires, and for that I have my tool: https://pypi.org/project/pipenv-to-requirements/

@fridex and @gsemet
thank you for give time to explain
This make me understand better the idea about why to track Pipenv.lock file.

This has been 4 years long. From the security perspective there is to me a trade-off that I would like to understand if it makes sense for you as well:

Commit the Pipfile.lock makes sure that a specific hash is used for the package that will be installed, that is a good security functionality
But not committing the Pipfile.lock and using inclusive or exclusive version comparisons can make the package be always upgraded to latest version that will potentially have security fixes.

Which ones do you consider better?
Here is my reasoning on option 2):
For example, in one project I have in the Pipfile:
Django = "<4.0,>=3.2"
But on the Pipfile.lock it was locked to 3.2.4 hash.
But there is a security vulnerability fixed in 3.2.5 Django version.
So due to committing the Pipfile.lock I was using a package version with a known CVE vulnerability.
If I do not version control the Pipfile.lock my Django version would be bumped to 3.2.7 automatically on the next CI/CD build of my latest docker image.
I came from the basic requirements.txt world and have a preference for option 2), could you please say what I may be missing here in terms of security for the recommendation to commit it?

From a security point of view you should always take what is in the lock file, never try to update it silently. Remember project might not follow semver correctly. So what if 3.2.7 introduces itself a huge new bug…
so, update only on purpose with pipenv update. Maybe you want something like « pipenv update —bugfix-only », but again, you expect people to follow semver which is sadly not always the case

Totally understand, but I am trusting that my unit tests and system tests will catch a potential bug as I version control the build container with the 3.2.7 package and test the release before going into production.
I understand that if the network between my CI/CD server and pypi is compromised as I am not verifying the hash I could potentially install a insecure package.
I am just worry that not updating the versions and let it go to production with a known CVE due to the lock file and trusting developers will handle the pipenv update could be a potential security risk.
It seems the only way to go is to commit the pipfile.lock and avoid to create a release with known CVEs by using security scan of potential vulnerabilities in the pipenv.lock packages via another application. This could be a potential improvement in pipenv to inform that are packages in the pipfile.lock with known vulnerabilities.


Creates duplicate columns when changing values (GQL)

Description
When using GQL and updating the values on a column field it creates a duplicate item in GQL in columns array. It does not create or show up in CMS just in the GQL query columns result.
Steps to reproduce

Create a value.
Update value.
Duplicate appears.

Additional info

Plugin version: dev-craft-4 as 4.0.1
Craft version: 4.0.4
Multi-site: Yes

Additional context

Can you please provide the mutation query you're using?

We do not use GQL to update/create the table. This happens when we are grabbing the table info from the CMS with GQL.

Ah okay, you just state "when changing values" and in your reproduction steps you state "create a value, update a value" - what value? A column in the field? A row? Or updating an other field in the entry that isn't a Table Maker field?
I can't seem to replicate this.

It happens when you for example update the width on the table columns. Write in 30%. Save and load the page. Then go back and change it to 100%. When I do this it creates a duplicate of the column.

Thanks for confirming. I've tested that and making all sorts of changes including adding/removing new columns, changing names, widths, alignments, but everything appear to be behaving. Can you provide a screenshot of the response in the GraphiQL explorer?

Hmm, shouldn't the result be the identical? It's all the same querying mechanism. It wouldn't have anything to do with local caching on the client-side or anything (if that's something in your project)?
Can you send through a screenshot from your front-end, showing the raw payload data as a network request from the GQL endpoint to illustrate? Just to see what sort of duplicate response looks like.

Screenshot-1

Screenshot-2

Here you can see the first query. It shows and array of 6 items. This is correct. On the second query after changing the width on the first column, there are now 7 items, and the column is created again at the end of the array.
There are still only 6 columns in the CMS.

Certainly seems like some form of cache or something getting mismatched with that changes data.  This doesn't seem to be a GQL query format I'm familiar with (Apollo), which I've just tried out without being able to replicate your results. Strange.


Fix array parser

[1, 2, 3] and [1, 2, 3,] are equivalent.
[,] will throw SyntaxError.

Looks good!


Multiple Deployments Opened and Then Eventual Failure

When terraform apply is executed - using an Akash SDL with a single service and count of 1 - multiple deployments are launched and eventually the execution fails with an insufficient funds message.
Essentially the provider is creating a deployment, when this deployment attempt fails another deployment is created, and this occurs repeatedly until the associated wallet is out of necessary funds.  The individual deployment creation attempts are successful - as many deployments exists following execution - but appear to fail in provider selection/manifest send and then move onto a new deployment attempt.
The TF and SDL files used are the examples provided in this doc:
https://docs.akash.network/other-resources/experimental/akash-deployments-via-terraform/akash-sdl-hello-world-example
Attaching debug from terraform apply from an example invoke.
terraformLogsv1.txt

To add to your issue, the return message when executing the deployment creation is:
Error: error unmarshalling result: unknown field "mempoolError" in types.ResponseCheckTx

Seems to be an issue related to the response from the Akash client.


molecule lint is gone.

https://github.com/geerlingguy/ansible-for-devops/blob/master/molecule/molecule/default/molecule.yml#L6
see:
https://github.com/ansible-community/molecule/pull/3802

Indeed. I'll have to get things updated :/


Is it possible to select a printer's media tray?

I am trying to send a print job to a certain tray.
I have checked the printer attributes that the trays are available:
media-source-supported: tray-1
media-source-supported: tray-2
media-source-supported: tray-3
media-source-supported: tray-4
media-source-supported: tray-5

And have found the Media parameter in the NewJobAttributes class, I have tried setting this to 'tray-4' and '"media-source": "tray-4"' with no succes.
In the AdditionalJobAttributes I've also tried setting the Tag 'Keyword' to name=media-source and value=tray-4 but this does not seem to work either.
var attributes = new NewJobAttributes
{

    JobName = documentPath,
    AdditionalJobAttributes = new IppAttribute[]
        {
            new IppAttribute(Tag.Keyword, "media-source", "tray-4")
        },
    Media = " \"media-source\": \"tray-4\" " // also tried just "tray-4"

};
var request = new PrintJobRequest
{
    PrinterUri = printerUri,
    Document = stream,
    RequestingUserName = Username,
    NewJobAttributes = attributes
};

using var client = new SharpIppClient();
var response = await client.PrintJobAsync(request);

I have found this issue about monochrome printing: https://github.com/Zelenov/SharpIpp/issues/23
Which describes the same problem, I also tried to print in monochrome but also with no success. But unlike the other issue I cannot seem to get the attributes printed in the PageLog.

It's a bit tricky since media-source is a member of media-col. To get this working I had to comment out this line, since it removes duplicate attributes by name, but we need multiple attributes with an empty name:
https://github.com/danielklecha/SharpIppNext/blob/master/SharpIpp/Protocol/Extensions/ListExtensions.cs#L18
AdditionalJobAttributes = [
    new IppAttribute(Tag.BegCollection, "media-col", ""),
    new IppAttribute(Tag.MemberAttrName, "", "media-source"),
    new IppAttribute(Tag.Keyword, "", "tray-4"),
    new IppAttribute(Tag.EndCollection, "", "")
]

@Purrloin4 I fixed the Populate method in version 1.0.12 so it shouldn't remove attributes with the same name if the name is empty. Please try @xqrzd solution.
I modified the code to correctly support BagCollection.
I added media-col-default to printer attributes.
I need to analyze documentation and add media-source everywhere where needed.
Unfortunately, ipptool -t -V 2.0 ipp://<IP_ADDRESS> ipp-2.0.test doesn't check these attributes in job requests.

@danielklecha @xqrzd Many thanks, I'll try it today!
This would also allow me to choose the printer color in the same way?
One thing not super clear to me is where I can find what member attribute is linked to what collection attribute. But I might be searching in the wrong documentation of the ipp standard.

@Purrloin4 This page contains a list of IPP attributes with links to documentation.
https://www.iana.org/assignments/ipp-registrations/ipp-registrations.xhtml
For example: if you search for media-col then you can open PWG5100.7 and section 5.3.1 contains a list of members.
print-color-mode is not a member of media-col. I'm not sure why example from issue doesn't work.
I need to figure out how to test it with some external tool.
I think that BegCollection and EndCollection should contain NoValue but probably empty string will generate similar result.
AdditionalJobAttributes = [
    new IppAttribute(Tag.BegCollection, "media-col", NoValue.Instance),
    new IppAttribute(Tag.MemberAttrName, "", "media-source"),
    new IppAttribute(Tag.Keyword, "", "tray-4"),
    new IppAttribute(Tag.EndCollection, "", NoValue.Instance)
]

@danielklecha @xqrzd
I've been trying to get the trays and type to work but without succes.
Could you attach your code that you used to get it working? I tried this:
testcodeAttributes.txt
If i check the error logs of CUPS with Debug2 the only thing about attributes that are present are the document attributes, nut sure where i can see more details about the jobs.

Job Template Attributes are described in RFC2911 section 4.2.
Document Template Attributes are described in PWG5100.5 section 6.3 and it's an extension for IPP 1.1.
media-col is part of document-attributes-tag section, not job-attributes-tag section.
print-color-mode is part of Job Template attribute (PWG5100.13 section 6.2.3) and should work...
Library do not have document section and it require some modifications to support it.

@danielklecha I found out that for some reason the attributes only work when using the NewJobAttributes, and not the AdditionalJobAttributes property directly (which I was doing). So, if I do it like this it works!
var attributes = new NewJobAttributes
{
    //Media = "iso_a3_297x420mm",
    Media = "iso_a4_210x297mm",
    AdditionalJobAttributes = [
        /*new IppAttribute(Tag.BegCollection, "media-col", NoValue.Instance),
        new IppAttribute(Tag.MemberAttrName, "", "media-source"),
        new IppAttribute(Tag.Keyword, "", "tray-2"), // auto, tray-1, tray-2, ...
        new IppAttribute(Tag.EndCollection, "", NoValue.Instance),*/

        new IppAttribute(Tag.BegCollection, "media-col", ""),
        new IppAttribute(Tag.MemberAttrName, "", "media-type"),
        new IppAttribute(Tag.Keyword, "", "stationery"), // auto, stationery, envelope, stationery-letterhead, ...
        new IppAttribute(Tag.EndCollection, "", ""),

        new IppAttribute(Tag.Keyword, "print-color-mode", "monochrome") // color, monochrome
    ]
};

var request = new PrintJobRequest
{
    PrinterUri = printerUri,
    Document = stream,
    RequestingUserName = Username,
    DocumentAttributes = new DocumentAttributes
    {
        DocumentName = documentPath,
        DocumentFormat = "application/pdf",
        Compression = Compression.None,
        DocumentNaturalLanguage = "en"
    },
    NewJobAttributes = attributes
};

This might be a bug in the library or was I not using it in the intended way? Either way my issue is solved so thanks!


#1: Add Travis CI config w/ Coveralls

Includes coveralls integration to report code coverage.

Changes Unknown when pulling 128e5e1dc1a0635d1c5d56394d276fca72a6b40a on 1-travis into ** on master**.


Improve error detection & handling in json parsing

Current parsing is optimistic and completely ignores unknown elements, and wrong types (like array instead of object). The parser should check that there are no unknown elements and all provided elements are of the correct type. Informative errors should be printed or returned.

If there are "unknown elements", we should not fail. But we should be informative about them.
Currently, in our developmental stage, we sometimes add new items to corral.json by hand that aren't yet supported in the code. They are there as we play around with ideas that will be eventually be incorporated into corral but are currently being used from some other codebase (like the library-documentation-action).


Routing Blog post

the text shows that you are super deep in the topic 😅
I think it would be helpful to take a step back and think of a reader, who isnt that deep into the topic.

explain a few things that seem obivious a little more
I think the structure could be rearanged a little to guide the reader a bit better (or maybe not even rearanged, I think you actually have that structure already. Just some intro sentences to the sections, so the reader knows why its relevant):

Why did we choose GraphHopper (and not sth else?)
How does GraphHopper work? (you insert OSM data - always via pbf? - then you can edit the profiles and with a running instance you can use the API

maybe there is a link to a "tutorial to set up your first GraphHopper instance" that we can refer to?

How we use the GraphHopper custom profile

Preprocessing (why is it necessesary outside of Graphhopper)
Combining present bike profile (e.g. that already includes road that are allowed to ride on), steepness of the road, custom CargoBikeIndex
open issues


[READY] [infra - 532] Term-apply DynamoDB backend

Description
Fixes: infra #532
Changes data storage for term-apply from csv files stored in S3 to a dedicated DynamoDB table that can easily interface with future applicant review software.
The expected schema for each item is as follows:

applied_date: unix time - number - Sort DDB Key
email<EMAIL_ADDRESS>- string - Primary/Partition DDB Key
name: Candy Date - string
github: candydate100 - string - Secondary Global Index
role_applied: sr. software engineer - string
role_override: string - in the case were their role is different otherwise null
resume_review: bool - Resume passed or fail the review
resume_review_date: number - resume reviewed date
interviews: Map
1: {date: number, pass: bool, notes: binary}
2: {date: number, pass: bool, notes: binary}
ignore_workflow: bool - ignore from automation workflow
offer_given: bool
offer_date: number
offer_accepted:  bool
offer_accepted_date: number
rejected: bool
rejected_date: number
rejected_msg_override: binary - Message custom to applicant

However term-apply only interfaces with a subset of attributes:

applied_date: unix time - number - Sort DDB Key
email<EMAIL_ADDRESS>- string - Primary/Partition DDB Key
github: candydate100 - string - Secondary Global Index
name: Candy Date - string
role_applied: sr. software engineer - string
offer_given: bool
rejected: bool

term-apply users have the ability to modify their email after submitting an application. If this happens, the existing record will be deleted and recreated with updated fields using a transaction to ensure atomic changes. Other item edits are done in place.
This system supports repeated applications by checking if a user's previous application has been closed (either offer_given or rejected is set to true). If a user has a closed application, a new one is created. Users can only modify their most recent open application.
Tests

Submitted application as a new user in empty table and populated table

new record successfully created in DynamoDB with correct fields in both cases

Updated role for open application

Role successfully updated

Updated name for open application

Name successfully updated

Email updated for open application

delete and recreate transaction completed successfully

Email updated for open application with other attributes populated

delete and recreate transaction completed successfully, no fields lost

Role and Name updated for open with other attributes populated

Role and Name updated successfully with no other attributes changed

Tests ran with multiple open applications

No side effects observed in other table items

Application closed by setting rejected to true, then submitting a repeat application

User was able to reapply and a new record was created in DynamoDB

Tests ran with repeated application

Only the most recent open application was modified, as expected

@kylerisse let's get this thing rolling!


Update alpine image to Alpine Linux 3.15

This moves from an 8-month-old base to a 4-month-old base.
Fixes #197
Note that Alpine Linux 3.14 introduced use of the faccessat2 syscall, which caused some incompatibilities with Docker versions before Docker 20.10.0 when running particular programs such as ruby or make. More details in the changelog.

3.16 is now released: https://alpinelinux.org/posts/Alpine-3.16.0-released.html

Error relocating /tini: __fprintf_chk: symbol not found

Looks like there's going to be some further work for Alpine 3.16, I'll revert this PR 🥲 thanks CI

Closing in favour of #262


State Delete does not Delete Terraform State Secret

A terraform destroy will not actually delete the state file and the k8s backend does the same thing leaving terrastate secrets around. We should probably consider if a states CR gets removed that we run a full destroy and thusly you probably want the terrastate object removed.

This is difficult to implement due to the source of the terrastate


👋 will this package support your connectomics with 🦓CEBRA approach?

Hey @timonmerk et al, firstly thanks SO MUCH for using CEBRA in your awesome paper, https://www.ncbi.nlm.nih.gov/pmc/articles/PMC10543023/pdf/nihpp-rs3212709v1.pdf. I am currently working on teaching material, and I was wondering if you have a demo notebook, or such, for how you used CEBRA in your paper?
" In brief, voxel-wise correlations between decoding performance and whole-brain connectivity maps seeded from channel MNI coordinates were calculated to identify an optimal connectomic template fingerprint for movement decoding (so called connectomic decoding network map) across all subjects (Fig. 2i). This allows for an optimized a priori channel selection in realtime, by identifying the individual recording channel that has most network overlap with the optimal template. Finally, we have transformed neural features from the selected channel into a lower
dimensional embedding. For this, a five-layer convolutional neural network with a temporal filter length
of 1 s was trained using the InfoNCE (Noise-Contrastive Estimation) contrastive loss function. The
resulting embeddings showed exceptionally high consistency across subjects as investigated with linear identifiability"
i.e., j and k here:

Sorry if I missed it in the code, but a quick serach doesn't have cebra in this code base! Not a worry if  you are waiting for formal publication, etc -- totally get it -- just figured I'd ask 🥰. Thanks again for using CEBRA -- truly one of my fav. uses so far 🦓🚀

Many thanks @MMathisLab! It was really great to use CEBRA and we could also show that it improved the leave-one-patient and leave-one-cohort out performances significantly by quite a big factor over the other tested methods in the revision!
We are however still in the revision process and currently work a lot to make the repo more useable. I think we will also add an option to add CEBRA as a dev dependency. But still keep the feature-estimation only methods lightweight without having the need to install e.g. PyTorch.
But I definitely have an example notebook on my ToDo list  that shows the predictions using CEBRA without patient-individual training. Luckily the Pittsburgh cohort data was all published openly so I could add a reproducible example within that cohort. Is there a specific deadline for the teaching? Then I could prioritise this notebook and will let you know once it's upstream.

Sorry for my delay! No major rush :) I showed them your cool preprint last week, so would be also in future years :D

Ok, finally coming back to the example @MMathisLab. I added a CEBRA decoding example to the documentation and examples folder where the connectomic across-patient decoding is presented. It took me quite some time since hosting pure .ipynb in sphinx-gallery isn't so straightforward as I expected!


GH-40209: [C++][CMake] Use "RapidJSON" CMake target for RapidJSON

Rationale for this change
Because upstream uses "RapidJSON" for CMake target name.
What changes are included in this PR?
Rename "rapidjson::rapidjson" to "RapidJSON".
FindRapidJSONAlt.cmake provides "RapidJSON" CMake target instead of just providing RAPIDJSON_INCLUDE_DIR.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

GitHub Issue: #40209

@github-actions crossbow submit -g cpp -g r -g linux -g python -g r

+1


Add tooltip to redirect badge

In the event of a user having the extension without previous knowledge, the redirect badge may cause confusion, hence a tool-tip indicating where this is coming from can be helpful.

[ ] Mouse hover for redirect badge

Added an icon badge instead.


[TASK]: improve project search

Describe the task
Model.description and Project.overview are base64 encoded because they include images. They are HTML or Markdown content.
The goal is to decode those fields, extract just the text from it, and embedded that text only for project search purposes.

Remember yourself to do the Dataset.description.


Remove "deprecated" for jshint

I found the latest commit of this repo was about a month ago, so I guess that should be considerd as still maintaining?

There is a longer discussion here about the reasons.
I think deprecated is not the correct term here. As you point out, the tool is still receiving updates. I'm rather looking for a term to express "not recommended for new projects by the community". Maybe we could use discussion and link to the discussion issue? Opinions?

Wow, I didn't realize that there was a long discussion about deprecated.
I agree that deprecated may not be a proper term in this case, maybe discussion would be a relatively proper term.
In my opinion, maybe we can modify the definition of deprecated? maybe use this term to indicate that a repo meets the following criteria

Repo was not maintained for longer than half a year
Repo was archived
etc

And discussion may be used to indicate that a repo is not recommended by the community, and we can link to the discussion issue.

Good points.
Would you mind adding a discussion field (similar to to the source field) to both the Rust rendering code and the tools.yml? Similar to this commit: https://github.com/analysis-tools-dev/static-analysis/commit/2df2790dd26306de0141c5890ba38677e5ee3dfb (Please ignore the changes in tools.yml. I went on a rampage there and fixed some inaccuracies.)
The deprecated field can stay, but I'd add the discussion links in the next step where needed.
Thanks for your ongoing support @AristoChen. Much appreciated!

I added discussion field, if there is anything that I did wrong or anthing else you want me to do, please let me know :)

Thanks!
We should make it clear that there was a debate from the rendered README.md.
I wonder if we can have an :information_source: (ℹ) button next to the entry that links to the discussion.
Would you be able to add that?

Added, does that meet you expectation?

Perfect! Thanks so much @AristoChen. You rock! 🎸


models does not affect the public app

When having models=() and registering the models itself on public_admin, it works. All registered models are shown (https://github.com/DadosAbertosDeFeira/maria-quiteria/pull/114/).
Is it a bug or a feature? :sweat_smile:
public_app = PublicApp("datasets", models=())
public_admin = MariaQuiteriaPublicAdminSite(public_apps=public_app)
models_and_admins = [
    (CityCouncilAgenda, CityCouncilAgendaAdmin),
    (CityCouncilAttendanceList, CityCouncilAttendanceListAdmin),
    (CityCouncilExpense, CityCouncilExpenseAdmin),
    (CityCouncilMinute, CityCouncilMinuteAdmin),
    (Gazette, GazetteAdmin),
    (CityHallBid, CityHallBidAdmin),
]

for model, admin in models_and_admins:
    public_admin.register(model, admin)

Definitively a bug :bug:!
Also, a reflection of the alpha status of this package. As far as I remember, the models declared in the PublicApp are mostly used to:

hack Django’s permission system and block access to non-declared models (not necessarily related to views at this point)
highjack and remove from the AdminSite the URLs related to these models

I have to dedicate some time to replicate this bug and understand a few things before thinking about a possible solution, but I think something to explore is to have our own register method that does this verification.


Error 429 when getting tweets w/ full_archive_search

Hi, I found your R library a few days ago and I'm trying to get which can be considered a big amount of tweets trough the academic V2 API but I noticed a problem which I'd like to expose here.
I know this still is an experimental library so I have this into account.
I'm trying to get tweets for the 2017-09-01 to 2017-10-12 period, I tried both with the full_archive_search with the minimun parameters and with the function for extracting tweets by day and then saving them in a csv file but I'm keep getting the same Error 429.
`2017-09-20T00:00:01Z to 2017-09-21T00:00:01Z has been colleted!
Something went wrong!
Error in full_archive_search(token = Bearer_Token, search_query = query,  :
Error: 429`
I had also this kind of message
`2017-09-19T00:00:01Z to 2017-09-20T00:00:01Z has been colleted!
Something went wrong!
Error in full_archive_search(token = Bearer_Token, search_query = query,  :
Error: 429
In addition: Warning message:
In max(data$created_at) : no non-missing arguments to max; returning -Inf`
Looking for a possible answer in several forums I noticed that in Python some libraries include a "wait on rate limit" function. I wonder if it is possible to include a function like this in the code (I am not myself a programmer and I apologize for only 'suggesting' insted of giving a proper solution for this issue) in order to elude the Error: 429.
Thanks for doing a great work. Greetings!

Hi
Thanks for using this Package.
I am aware of this particular problem. I have seen it a few times as well whenever I try and collect bigger chunks of data.
Unfortunately it is not the rate limit as this is looked up at every call to the API and if it hits zero (for the next call) it will pause until the limit resets again.

PS: Try Dividing the Data collection into smaller time-chunks and loop (for / while / apply) through it with a tryCatch(), which will be able to handle the error. This would maybe help you figure out at what date the Error occurs. If it occurs every single Time then something differently is not right but probably not with the package...
All the best, Maël

@MaelKubli :
Thanks for your answer, I will try with a loop, I was trying with day periodos and a limit for 20K tweets and, as far as I go, it's doing great, maybe the issue is due some kind of problem with the amount of tweets (?), idk, but thanks again, I'm looking forward for any future improvements in the package, it is realy great (above all, the posibility to save all possible columns in a csv!)
Thanks a lot and keep it up!


Memoize item headers to speed up export

The code was re-calculating item headers for each distribution by looping over each distribution (so O(n^2)). I memoized the results of this so this becomes O(n). The results come back in about 4 seconds instead of 4 minutes.

I somehow missed that you had requested me.  Will look RSN.


Remove the ErrorSlot type

Motivation
We're seeing  "failed servers must set their error slot" panics when Zebra is overloaded.
This panic is caused by the design of the Client connection code. Currently, the Client tasks share an ErrorSlot mutex, which must be updated before they exit.
This ticket removes the ErrorSlot type, after we stop using it:

#4733
#4734

Designs
Remove the ErrorSlot type, and any methods that use it.
Here are some cleanups we might be able to do:

delete Connection's State::Failed and fail_with
delete the ErrorSlot

Some of this cleanup might happen in earlier tickets.
Related Work
This is part of:

#3263

We're making network issues a lower priority for now.


adding feedback section

Run it with "localhost:3000/feedback"

I am not able to run it

I am not able to run it
Is it showing any error? Try running it with localhost:3000/feedback in your browser.

There's no api used in my code. So why is it showing like this? @yatulearn  please assist!

There's no api used in my code. So why is it showing like this? @yatulearn please assist!

ignore that

there are no updates in the pr after I requested you to make, kindly check it once


Clean up player animations

Current dependencies on/for this PR:

main

PR #60   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 31, 2:39 AM: @kadhirvelm started a stack merge that includes this pull request via Graphite.


enable/disable stroke-linecap and stroke-linejoin

Hey, fantastic job on the icons and website. I have one idea for an enhacement. It would be really cool if you could provide checkboxes to enable/disable stroke-linecap and stroke-linejoin

Thanks for the feedback!
We want to toe the line between being highly tweakable, yet still having a cohesive style to the icons — and variable caps/joins is a thought we toyed with and ultimately decided against. Some icons are constructed with border radius, or organic curves that don't translate well when the overall "soft" style of the rounded caps and joins is removed. That being said, the stroke data is all there if you need to customize a "sharp" style for your own needs.
Closing this for now, but willing to revisit if it seems like a very common need.

Thanks for the feedback!
We want to toe the line between being highly tweakable, yet still having a cohesive style to the icons — and variable caps/joins is a thought we toyed with and ultimately decided against. Some icons are constructed with border radius, or organic curves that don't translate well when the overall "soft" style of the rounded caps and joins is removed. That being said, the stroke data is all there if you need to customize a "sharp" style for your own needs.
Closing this for now, but willing to revisit if it seems like a very common need.

Thanks Austin! Def want to keep this conversation going. We have this in the backlog to investigate further. I think it would be a handy feature to have, but there's a lot of testing we'd want to do to find a solution that works across the board. We'd want to consider implications to readability, spacing between strokes, pixel-snapping — and how to deal with variable corner radiuses as @rektdeckard mentioned.

Thanks Austin! Def want to keep this conversation going. We have this in the backlog to investigate further. I think it would be a handy feature to have, but there's a lot of testing we'd want to do to find a solution that works across the board. We'd want to consider implications to readability, spacing between strokes, pixel-snapping — and how to deal with variable corner radiuses as @rektdeckard mentioned.

For sure. On one hand, Im sure you want to make sure your hard work is not wasted by icons coming out bad. On the other hand, if people can preview the icons while they make their edits, then it might not matter since they can see how their choices change things.
Anyway, it's a great project regardless. Thank you so much for the work you put into it.
Since it's in the backlog already, do you want to close this issue?

For sure. On one hand, Im sure you want to make sure your hard work is not wasted by icons coming out bad. On the other hand, if people can preview the icons while they make their edits, then it might not matter since they can see how their choices change things.
Anyway, it's a great project regardless. Thank you so much for the work you put into it.
Since it's in the backlog already, do you want to close this issue?


Request: Register as a Bower.io package

Have you considered registering your package with Bower.io?

Why is it called "floatlabel.js", not "floatlabels.js"? (it seemed to me that "floatlabels.js" would be a bit more relevant name because repo and file is named like that)


Should we bring back needs_line_scan()?

We used to use this function needs_line_scan.  Is this sort of optimization worthwhile?
=head2 $res->needs_line_scan( \%opts )

Tells if the file needs a line-by-line scan.  This is a big
optimization because if you can tell from the outset that the pattern
is not found in the file at all, then there's no need to do the
line-by-line iteration.

Slurp up an entire file up to 100K, see if there are any matches
in it, and if so, let us know so we can iterate over it directly.
If it's bigger than 100K or the match is inverted, we have to do
the line-by-line, too.

=cut

sub needs_line_scan {
    my $self  = shift;
    my $opt   = shift;

    return 1 if $opt->{v};

    my $size = -s $self->{fh};
    if ( $size == 0 ) {
        return 0;
    }
    elsif ( $size > 100_000 ) {
        return 1;
    }

    my $buffer;
    my $rc = sysread( $self->{fh}, $buffer, $size );
    if ( !defined($rc) && $App::Ack::report_bad_filenames ) {
        App::Ack::warn( "$self->{filename}: $!" );
        return 1;
    }
    return 0 unless $rc && ( $rc == $size );

    return $buffer =~ /$opt->{regex}/m;
}

Did we ever use this?

@hoelzro says:  git log -p -S needs_line_scan $(git ls-files | grep -v garage/) - we never did

Doesn't look like it:
[git:dev] [12:10:56] rob@eridanus ~/projects/ack2 $ git log -S needs_line_scan $(git ls-files | grep -v garage/)
commit a596d9c
Author: Andy Lester<EMAIL_ADDRESS>Date:   Sat Feb 12 11:13:55 2011 -0600

    starting on the iterators

commit 291316d
Author: Andy Lester<EMAIL_ADDRESS>Date:   Wed Feb 2 21:05:33 2011 -0600

    added Resource.pm

ack1 did:
https://github.com/petdance/ack/blob/f423ddeb7f32a01687a880a3e2d8723b24c9cd15/Ack.pm#L1449-L1462

We have to be careful of \A and \Z because then they'll be comparing against the entire buffer, not a line.

We will need to add test files that are >100K for test coverage.

I started a needs_line_scan branch in ack3.  If this is a win, I think we want to backport it to ack2 and release ack 2.20.


[Brackets auto-update Windows/Mac] Content of status bar gets overlapped when the brackets window is resized to a smaller size.

Description
[Brackets auto-update Windows/Mac] Content of status bar gets overlapped when the brackets window is resized to a smaller size.
Steps to Reproduce

Launch brackets 1.13.
Resize the brackets window to a smaller size.
Click on Update Notification Button.
Click on Get it Now.
Check status bar.
Content of status bar gets overlapped when the brackets window is resized to a smaller size.

Expected behavior: The content shown in status bar should not get overlapped
Actual behavior: Content of status bar gets overlapped when the brackets window is resized to a smaller size.
Versions
Windows 10 64 Bit
Mac 10.13
Release 1.13 build 1.13.0-17665

https://github.com/adobe/brackets/pull/14295

Working fine now.
Verified in latest build - Release 1.13 prerelease build 1.13.0-17689.


Mac OS Open File Dialog Shift

When the Mac open dialog is first shown, it appears to be a sheet attached to the window title bar. But then it immediately jumps to a centered modal dialog! The shift is jarring and I doubt it’s what is intended.

@bobrocke Thanks for reporting this.


RegexManager: Dump both "highlight" and "highlight-article" rules

Fixes https://github.com/newsboat/newsboat/issues/635.

Coverage remained the same at 57.952% when pulling 0a81358f4e30f2737a38e08add9ee13fd53b66b1 on dennisschagt:bugfix/issue-635 into fdef6fc5558aaf0baa3b8c185b335ca577535a33 on newsboat:master.


Error on line 895 - "arch: command not found"

Preparing directories to store Sonic Robo Blast 2's source code, assets and libraries. Please wait...
/usr/local/bin/srb2bld: line 895: arch: command not found
/usr/local/bin/srb2bld: line 895: arch: command not found
/usr/local/bin/srb2bld: line 895: arch: command not found

This bug is due to arch not being part of coreutils by default on Arch Linux, would recommend using uname -m instead
https://github.com/google/fscrypt/issues/92#issuecomment-365712796

Fixed. Commit f9118b7 have arch command running only within Docker container for now.


WIP: Vtol transition failsafe RTL

Fixes: https://github.com/PX4/Firmware/issues/3663
This publishes the failsafe state after a failed vtol front transition.
Commander handles the failsafe and sets RTL if in mission mode.

This looks good! I think you could add the timeout to the other types as well.

@AndreasAntener I have updated the tailsitter to include the timeout. i had to change the transition abort handling for this. I tested in SITL and it looks good.
The same changes should be required for tiltrotor but i have no way of testing this. Can i assume it will work identical to tailsitter?

It should work more or less the same for all types. So I think we'll be good. If it doesn't look clear enough we could also organize a quick bench test with the Firefly6.

@AndreasAntener it's not, it just looks weird in the diff
It now reads:
        case FW_MODE:
	case TRANSITION_FRONT_P1:
		_vtol_schedule.flight_mode 	= TRANSITION_BACK;
		_vtol_schedule.transition_start = hrt_absolute_time();
		break;

Before it read:
           case FW_MODE:
		_vtol_schedule.flight_mode 	= TRANSITION_BACK;
		_vtol_schedule.transition_start = hrt_absolute_time();
		break;

	case TRANSITION_FRONT_P1:
		// failsafe into multicopter mode
		_vtol_schedule.flight_mode = MC_MODE;
		break;

When in TRANSITION_FRONT_P1 mode and canceling the flight it also needs to backt transition.

There are two different cases:

regular back transition from FW_MODE
instantaneous back transition from TRANSITION_FRONT_P1

Your changes removes the later.

Yes, because this is required.
if https://github.com/PX4/Firmware/blob/master/src/modules/vtol_att_control/tailsitter.cpp#L335-L359 is not executed the vehicle will not level back to MC mode and crash.
I have SITL tested this, you can try by reverting that bit and sitl test by setting transition airspeed to 20m/s. the vehicle will crash.

This is why I asked. If that's the case it points to a problem in the failsafe backtransition we need to check before this (and not just changing the logic so it works). Let's just take the timeout handling for the standard vtol in then.

I'll do a test and pull the standard vtol timeout in today, then we have a look about the other types.

When transition has started the _v_att_sp->pitch_body is changed.
This always needs to be changed back when failsafing to MC mode.
I believe that canceling a transition during P1 manually will now also fail

rebased

I'll push a new version of this without the tailsitter update as soon as I can compile again...

@sanderux Closing this. Will push the other branch to master if checks pass. After that, could you open a new PR with your last commit? So we can also look for the problem with the failsafe back transition on the tailsitter.


Build pipeline

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: carlosrodrguti sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[FEAT]: Creating new configurations from the Simulations screen

Although users can create model configurations on the Model screen, it would be more efficient if they can also do this from the Simulation screen.
Here are some designs to describe that flow:

designs have changed.


No service type 'SPARK' available for cluster with version 'CDH 6.3.0'

Hi,
I am able to install HDFS, YARN and Zookeeper. But facing issue while installing spark.
when I run
python deploycloudera.py -ihivepass -fCloudera@123 -nCloudera@123 -rCloudera@123
Traceback (most recent call last):
File "deploycloudera.py", line 1079, in
main()
File "deploycloudera.py", line 1028, in main
spark_service = deploy_spark(CLUSTER, SPARK_SERVICE_NAME, SPARK_SERVICE_CONFIG, SPARK_MASTER_HOST, SPARK_MASTER_CONFIG, SPARK_WORKER_HOSTS, SPARK_WORKER_CONFIG, SPARK_GW_HOSTS, SPARK_GW_CONFIG)
File "deploycloudera.py", line 689, in deploy_spark
spark_service = cluster.create_service(spark_service_name, "SPARK")
File "/usr/lib/python2.7/site-packages/cm_api/endpoints/clusters.py", line 161, in create_service
service_type, self.name)
File "/usr/lib/python2.7/site-packages/cm_api/endpoints/services.py", line 44, in create_service
ApiService, True, data=[apiservice])[0]
File "/usr/lib/python2.7/site-packages/cm_api/endpoints/types.py", line 137, in call
ret = method(path, data=data, params=params)
File "/usr/lib/python2.7/site-packages/cm_api/resource.py", line 148, in post
self._make_headers(contenttype))
File "/usr/lib/python2.7/site-packages/cm_api/resource.py", line 73, in invoke
headers=headers)
File "/usr/lib/python2.7/site-packages/cm_api/http_client.py", line 193, in execute
raise self._exc_class(ex)
cm_api.api_client.ApiException: No service type 'SPARK' available for cluster with version 'CDH 6.3.0'. (error 400).
What is the service name for spark in CDH 6.3.0?

SPARK2 or SPARK2_ON_YARN

with SPARK2 I get the same exception as  No service type 'SPARK2' available for cluster with version 'CDH 6.3.0'. (error 400)
When I use SPARK2_ON_YARN, I get below error.
cm_api.api_client.ApiException: Unknown configuration attribute 'hdfs_service' for service (type: 'SPARK2_ON_YARN', name: 'SPARK2_ON_YARN'). (error 400)
Please find below relevant code snippet:
`### Spark ###
SPARK_SERVICE_NAME = "SPARK2_ON_YARN"
SPARK_SERVICE_CONFIG = {
'hdfs_service': HDFS_SERVICE_NAME,
}
SPARK_MASTER_HOST = SPARK_HOST
SPARK_MASTER_CONFIG = {
'master_max_heapsize': 67108864,
}
SPARK_WORKER_HOSTS = list(CLUSTER_HOSTS)
SPARK_WORKER_CONFIG = {
'executor_total_max_heapsize': 67108864,
'worker_max_heapsize': 67108864,
}
SPARK_GW_HOSTS = list(CLUSTER_HOSTS)
SPARK_GW_CONFIG = { }
Deploys spark - master, workers, gateways
def deploy_spark(cluster, spark_service_name, spark_service_config, spark_master_host, spark_master_config, spark_worker_hosts, spark_worker_config, spark_gw_hosts, spark_gw_config):
spark_service = cluster.create_service(spark_service_name, "SPARK2_ON_YARN")
spark_service.update_config(spark_service_config)
sm = spark_service.get_role_config_group("{0}-SPARK_MASTER-BASE".format(spark_service_name))
sm.update_config(spark_master_config)
spark_service.create_role("{0}-sm".format(spark_service_name), "SPARK_MASTER", spark_master_host)
sw = spark_service.get_role_config_group("{0}-SPARK_WORKER-BASE".format(spark_service_name))
sw.update_config(spark_worker_config)
worker = 0
for host in spark_worker_hosts:
worker += 1
spark_service.create_role("{0}-sw-".format(spark_service_name) + str(worker), "SPARK_WORKER", host)
gw = spark_service.get_role_config_group("{0}-GATEWAY-BASE".format(spark_service_name))
gw.update_config(spark_gw_config)
gateway = 0
for host in spark_gw_hosts:
gateway += 1
spark_service.create_role("{0}-gw-".format(spark_service_name) + str(gateway), "GATEWAY", host)
#TODO - CreateSparkUserDirCommand, SparkUploadJarServiceCommand???
return spark_service`
Please let me know if any other places to change. Thanks :)

I am also getting same kind of issue..below. I am trying to install on CDH 5.14.4
Traceback (most recent call last):
File "cm-api-deploy.py", line 1001, in
deploy.gen_service_types_and_objects()
File "cm-api-deploy.py", line 339, in gen_service_types_and_objects
self.service_types_and_names[svc_type], svc_type)  # Service name & type
File "/usr/lib/python2.6/site-packages/cm_api/endpoints/clusters.py", line 161, in create_service
service_type, self.name)
File "/usr/lib/python2.6/site-packages/cm_api/endpoints/services.py", line 44, in create_service
ApiService, True, data=[apiservice])[0]
File "/usr/lib/python2.6/site-packages/cm_api/endpoints/types.py", line 137, in call
ret = method(path, data=data, params=params)
File "/usr/lib/python2.6/site-packages/cm_api/resource.py", line 148, in post
self._make_headers(contenttype))
File "/usr/lib/python2.6/site-packages/cm_api/resource.py", line 73, in invoke
headers=headers)
File "/usr/lib/python2.6/site-packages/cm_api/http_client.py", line 193, in execute
raise self._exc_class(ex)
cm_api.api_client.ApiException: No service type 'SPARK2_ON_YARN' available for cluster with version 'CDH 5.0.0'. (error 400)

I have installed another parcel to get install spark 2 service. here is the example parcel: SPARK2-2.2.0.cloudera2-1.cdh5.12.0.p0.232957-el6.parcel


sync main with develop

Codecov Report

Merging #54 (26c5be9) into main (33bd30e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #54   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           21        21
  Lines          248       248
  Branches        11        11
=========================================
  Hits           248       248

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 33bd30e...26c5be9. Read the comment docs.


Integration Test - Release - 2022/03/11 11:40:32

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#please-close

#unassign PRMergerTest1

#unassign NonAllowedUser


Update Message doc comments

I noticed that the doc comments on the Message struct are in the wrong order, here the order has been corrected.
Also, looking at the fcm docs, I think data values are only supposed to have string values, while serde_json::Value would allow for any json values, maybe this should be updated to a HashMap<String, String>?
For now I've updated the data comment to state these limitations.

Thank you for noticing! And you're also right about the string values, this was not implemented correctly.


Revert "解决Druid多classloader  Mbean注册卸载过程中的问题 "

Reverts alibaba/druid#3164

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

why revert     #3164 merge

动态的MBeanName相关的工具类无法获得MBean，回归测试也跑不过

是的，对应的测试类应该也需要做对应的修改，我再优化下单元测试吧

gg


redraw!

I have autoread set and use some autocmd's triggered by BufEnter,BufNew,BufRead,WinEnter,FileChangedShell,ShellCmdPost,FocusGained events to handle some menu sensitivity for a plugin that I'm working on.  After executing a version control command using asynccommand#run, a file may autoread from the permissions change, and the idea is to trigger a function.  I generally use the quickfix handler to display the results of such commands (something like file is now open for edit).  However, the menu is not redrawn after the command (not exactly sure if the event isn't firing or it fires and the menu needs an explicit redraw).  It does work if In asynccommand#tab_restore(env), I change "redraw" to "redraw!".
I'm not sure if this is something that you would want to include in the plugin or if it is only due to my own very specific usage.
Thanks,
Chad Moore
EDIT 12/15/2011: BTW, I saw a TODO comment about notification of a completed command (when there's no output returned).  This is a highly anticipated feature in my book.  Thanks!

I also have a drawing issue: after invoking any async command, the vim viewport becomes blank (completely or partially).

when the command completes and if the quickfix window opens, then the screen is redrawn partially (many artifacts remain)
the effect of changing "redraw" to "redraw!" in tab_restore, as suggested above:

screen redraws almost fully but still only when command completes and quickfix is shown
almost: because there is still a stray "0" showing up and persisting in the cursor position, that is fixed by a manual ":redraw!" but not ":redraw"

a workaround for the issue is to add a "redraw!" immediately after any async command ("redraw" is not a workaround)

System: I tested with a fresh .vimrc and only asynccommad plugin installed
Solaris
VIM - Vi IMproved 7.2
.vimrc content:  (but the same issue when: set term=dtterm or xterm)
Example command to break the screen: ":AsyncCommand echo test" (and of course same for my complicated make command)
Maybe just adding a "redraw!" somewhere right after firing the async command in the plugin would be a solution? or maybe there's  better fix that would fix the root cause of the view breaking and needing a redraw (any idea why that happens?). Also, the change to "redraw!" in tab_restore does seem to be an improvement (even if not the fix). Thank you for looking into this.

Regarding the TODO mentioned above:
In asynchandler#qf(), I have added a few lines to indicate success if there's no stdout from an operation.  I'm currently using something like this:
if len(getqflist()) == 0
" if the output is empty, indicate success
caddexpr 'The requested operation completed without error.'
endif
" in case the command modifies the file or its permissions, re-check the modified time
checktime
I'm not sure if you'd be interested in adding a checktime there like this but I found it handy.  As for a more generic way to add this to the plugin, perhaps there can just be a user-definable post-process function called from the standard handlers.

I've been making a lot of tweaks lately and adding redraw! might not be such a great idea.  It does add quite a flicker, especially when the quickfix window remains hidden (I'm working on getting flymake-like behavior out of this script).


fix misspell "destroy" in sandbox.go

This change is

@fate-grand-order Thanks for the fix. LGTM


Question: How to close buffer without killing it?

I am used to Emacs where I can close a buffer but not kill it (meaning it is still in memory). This is useful for example when I'm running a REPL in a buffer. I would like to close/hide the buffer and quickly open it back up when I need to. How can this be done?

Thanks for reaching out!
Because we treat our issues list as the Atom team's backlog, we close issues that are questions since they don't represent a task needing to be completed. For most questions about Atom there are a lot of options, check out:

The Atom Flight Manual
The Atom FAQ on Discuss
Discuss, the official Atom and Electron message board
Join the Atom Slack team

On Discuss and in the Atom Slack team, there are a bunch of helpful community members that should be willing to point you in the right direction.


CORGI-497 check if component version is empty before removing substring

@RedHatProductSecurity/corgi-devs @JimFuller-RedHat please review.

apart from failing CI - LGTM ;)


How to pass data by flutterEngineCache when native goto flutter? also how to flutter receive date?

How to pass data by flutterEngineCache when native goto flutter? also how to flutter receive date?
this is my flutter doctor
[✓] Flutter (Channel stable, 1.22.3, on Mac OS X 10.15.6 19G2021, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
[!] Android Studio (version 4.1)
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
(In fact, Flutter plugin and Dart plugin are running good. its just a AS display error)
[✓] VS Code (version 1.51.1)
[✓] Connected device (1 available)
here is java code that native to flutter
flutterEngine = new FlutterEngine(this);
flutterEngine.getNavigationChannel();
flutterEngine.getDartExecutor().executeDartEntrypoint(
DartExecutor.DartEntrypoint.createDefault()
);
FlutterEngineCache.getInstance().put("flutter_engine", flutterEngine);
startActivity(
FlutterActivity.withCachedEngine("flutter_engine").build(this)
);
here is flutter code that receive native route
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Flutter Demo',
theme: ThemeData(
primarySwatch: Colors.blue,
),
onGenerateRoute: (RouteSettings settings) {
String routeName = settings.name;
print("Open page: $routeName");
switch (routeName) {
case "RoutePage1":
return MaterialPageRoute(builder: (context) {
return RoutePage1();
});
case "RoutePage2":
return MaterialPageRoute(builder: (context) {
return RoutePage2();
});
default:
return MaterialPageRoute(builder: (BuildContext context) {
return Scaffold(
body: Center(
child: Text("Page not found"),
),
);
});
}
},
home: MyHomePage(title: 'Flutter Demo Home Page'),
);
}
}

Hi @FieeA
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Re-add content assertions to flaky tests

What does this pull request do?
Re-add content assertions to flaky tests
Related to

#376
#378

What is the intent behind these changes?
Honestly, this is grasping at straws. There's insufficient data about what causes these .expectStatus().is2xxSuccessful calls to sometimes (quite often) fail with
> POST http://localhost:39473/sent-referral/24121550-20ff-421b-b39f-72a1abca1c77/assign

{"status":500,"error":"unexpected error","message":"Could not commit JPA transaction; ⏎
  nested exception is javax.persistence.RollbackException: Error while committing the transaction"}

Adding these might just add delay, hiding the issue behind artificial lag.
However, it's getting very painful on main being blocked and needing several retries, so... let's try?

Never mind: it clearly doesn't work


feat(examples): add new coagents-travel example

This adds a new example in the form of a travel agent. Its operations are pretty simple but do good job of introducing users to what can be accomplished with agentic experiences. The most interersting part of this is combining the Agent with the ability to use the google places API to get concrete locations to visit.
Alongside this is all of the features of CoAgents that would expect.

Human in the loop
Intermediate State Rendering
Shared State
Front-end functions

I've included some documentation in critical areas relevant to CoAgents, mostly in the ui/lib/hooks directory.

Checklist

[X] I have read the Contribution Guide
[X] If the PR changes or adds functionality, I have updated the relevant documentation

Go-ahead to merge from @ataibarkai


feat: component card

Codecov Report

Merging #4 (fec53fb) into master (00c93dd) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master      #4    +/-   ##
=======================================
  Coverage    0.00%   0.00%
=======================================
  Files           9      64    +55
  Lines          70     279   +209
  Branches        4      15    +11
=======================================
- Misses         70     278   +208
- Partials        0       1     +1

Impacted Files
Coverage Δ

src/component-panel/ComponentCard.tsx
0.00% <0.00%> (ø)

src/component-panel/ComponentTree.tsx
0.00% <0.00%> (ø)

src/component-panel/index.tsx
0.00% <0.00%> (ø)

src/demos/affix.tsx
0.00% <0.00%> (ø)

src/demos/alert.tsx
0.00% <0.00%> (ø)

src/demos/anchor.tsx
0.00% <0.00%> (ø)

src/demos/auto-complete.tsx
0.00% <0.00%> (ø)

src/demos/avatar.tsx
0.00% <0.00%> (ø)

src/demos/back-top.tsx
0.00% <0.00%> (ø)

src/demos/badge.tsx
0.00% <0.00%> (ø)

... and 48 more

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 00c93dd...fec53fb. Read the comment docs.


Reload content pop

Hi!
Is there any way to load content when the pop is open?
Regards!

Same question

Im wondering the same...

I've added the new API method 'updateContent' , now I'm doing some tests, please wait .

It will be great yes. To update the content while popup is opened! This will be very helpful. Thanks!

Yes please roll it out! currently am using jquery to replace the popover contents on: onShow Event, which is creating weird positioning problems.

@sandywalker : When are you rolling out the 'updateContent' feature??

Sorry for delayed so long.
I've added the global method WebuiPopovers.updateContent(selector,newContent)  , now you can update the popover content when the pop is open.
The demo code is in demo/test-issue195.html.

updateContent works fine!! Thanks!
But with the latest update to "webui-popover": "1.2.14" , popover is closing on hover even when closable set to true. In earlier version it was fine.

Great. Now will replace the bootstrap popover with this one again. Thanks!

It seems this feature not working with async type.
Any way to achieve this? I have ajax popover and want to update content with other ajax request or even inline content will also work as I will do ajax call myself and will try to replace content but it seems not working here.

@sbthemes I'm implementing a new global method 'updateContentAsync' and doing some test now, it will be released soon after tests.

Ok Thanks :)


BPM greater than 200 without touching the sensor

Hi guys,
I have purchased original pulse sensor from sparkfun and uploaded the code as it is on my Arduino Uno board. But the sensor gives me erroneous readings eg. BPM > 200 without even touching the sensor. On touching the reading is fine 50% of the time but erroneous values are frustrating. I have been pulling my hair for past week or so due to this issue. Any help will be really appreciated. Please respond ASAP.

It sounds like you need some basic debugging.
Run this code and see how the Pulse Signal looks like in Arduino's built-in serial Monitor.


feat(server): support using a service role client

This PR adds support for using the service role key provided by supabase in the server in order to bypass RLS.
Please let me know if I need to change something.
Types of changes

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

Description

added a new environment variable SUPABASE_SERVICE_ROLE_KEY and its corresponding module option serviceRoleKey
added new function serverSupabaseServiceRoleClient which uses the service role key (also did a small refactoring to serverSupabaseClient
resolves #30
Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes (if not applicable, please state why)

Thanks for the work @amrnn90 ! I have decided to make my own PR for this feature because the doc has totally changed since you made it: we are now using Docus theme (public release coming soon) powered by nuxt-content v2). Also I wanted to make it in a slightly different way, you can check the code here:  #43.


[FEATURE] Folder/Collection Last Used

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.
Add tracking for the last time a folder was used
Describe the solution you'd like
No response
Anything else?
No response

Added in 1.4.1 later today


docs: remove snowflake, add row supported DBs

Remove Snowflake as supported source. Once we upgrade Ibis, Snowflake support can be revisited
Add Oracle and MSSQL to list of row validation supported DBs

/gcbrun

Is there any update in regarding snowflake related support? When this will be back or is there any alternative tool to validate for snowflake?


ci: label script thinks #3076 is a German translation

Just filing to look at later.

Looks like it was this change, which sort of makes sense?

Oh!  Haha, thank you.  I swear I looked through this like twice and didn't see that.


Favicon

Add a favicon.
Depends on https://github.com/StateSmith/statesmith.github.io/pull/1 and https://github.com/StateSmith/StateSmith/pull/301
(still having trouble figuring out how to stack PRs)
Closes #305

all those commits were merged previously in the 267-simulator-->main merge, not sure why they're still showing up in this PR. Will just recreate this PR.


feat(doc): Nominate SIG Chairs and Technical Leads

According to pull request :https://github.com/cubefs/cubefs/pull/3430, sigs need chairs and tecnical leads to garantee the evething runs normally

+1

+1

+1

+1


Video details page refactor

Description
Refactor video details page by breaking it into multiple smaller components.
Fix some warnings and errors with the Table component.
Optimize fetching to only fetch if video id does not match url param id.
Optimize rendering and video element GET by only continuing when video id in store matches url param id.
Shows a LoadingView when loading/fetching data from store.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Change Status

[ ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[x] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[x] Manual user testing of each component and flow

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

 Refactor Video Details Page

 Add feedback submit action to Video Details Page


realtime node's memory keep increasing

From my understanding, after each segmentGranularity(e.g.1 hour)+windowPeriod ,
the realtime node will hand off the merged segment to Historical Server , so that it's memory usage will not too high, but I found that the usage keep on increasing all the time.
and the JVM option -xmx:20G seems has no effect....
  PID USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND
32338 lucas.w  16   0 38.1g  27g 8.7g S 21.2 21.7 126:44.04 java

I am closing this. Please reopen if this is still an issue.


Hotspot Home Page

Issue

Link: https://github.com/NebraLtd/maker-starter-app/issues/54
Summary: Changing the look and feel of the home to include list of hotspots owned by linked wallet address.

How

Use linked wallet address to call helium api for hotspots owned by address.
Persist hotspot list with asyncstoroage and manage state with reduxjs
List hotstops on homepage.

Screenshots

References

Checklist

[ ] Tests added
[ ] Cleaned up commit history (rebase!)
[ ] Documentation added
[ ] Thought about variable and method names

Did it work well on android?


Reliable CosmosDB client connection configuration for Azure functions consumption plan

Query/Question
We are building an analytics application for spiky workloads on Azure functions consumption plan. We use CosmosDB to store the state of each job, but when we use the default cosmos client configuration we see 95th percentile latencies of several seconds and a large number of "connection closed" exceptions from Cosmos DB. It appears that the default connection configuration is intended to be used by large always-on instances and we have been unable to find any documentation covering configuration for the consumption plan use case. In lack of documentation we have experimented with the settings and found a configuration that produces significantly better throughput and gives much fewer exceptions than with the defaults. Our query/question is for recommendations on an optimal client connection configuration for use with Azure functions on a consumption plan. We would also like feedback on the details of our performance tuning findings given below.
Further details: The workload on our system is a scheduled spike of jobs every hour, which we want to handle as quickly as possible. This implies lots of short lived consumption plan instances (100+) are spinning up at the same time and we found that the default cosmosDB client settings are simply not suitable for this scenario. We have done load testing experiments with the Java CosmosDB SDK 4.2.0 with different client configurations and followed best practices for performance optimization (eg. client is cached). Based on these experiments we have drawn the following conclusions:
• Direct connection is the way to go, since it can be tuned to deliver much better performance than gateway. Connection tuning is required to make it reliable, however. If we simply use the default configuration for direct connection we are flooded with thousands of the “connection closed” exception seen below.
• Limiting maxConnectionsPerEndpoint to 1 reduces the number of DB exceptions very significantly and does not appear to have any negative impact on performance
• Setting a low value for maxRequestsPerConnection lowers the latency of synchronous writes very significantly (At least an order of magnitude for the 95th percentile compared to default). If it goes too low, however, we start to see an increase in the number of DB exceptions. The optimal parameter appears to be a trade off between performance and the number of DB exceptions thrown. We have found 10 be a good compromise.
• Performance varies significantly with different versions of the SDK. In our experiments 4.3.1 provides less than half the query throughput of 4.2.0 using the connection configuration seen below.
Highest throughput configuration:
DirectConnectionConfig directConnectionConfig = DirectConnectionConfig.getDefaultConfig();
directConnectionConfig.setMaxConnectionsPerEndpoint(1);
directConnectionConfig.setMaxRequestsPerConnection(10);

return new CosmosClientBuilder()
        .endpoint(environment.endpoint)
        .key(environment.primaryKey)
        .consistencyLevel(ConsistencyLevel.STRONG)
        .directMode(directConnectionConfig)
        .buildClient();

The connection closed exception we see thousands of with the default direct connection configuration:
Exception while executing function: Functions.JobPersister Result: Failure
Exception: IllegalStateException: RntbdServiceEndpoint({"id":3,"isClosed":true,"concurrentRequests":0,"remoteAddress":"cdb-ms-prod-westeurope1-fd20.documents.azure.com:14318","channelPool":{"remoteAddress":"cdb-ms-prod-westeurope1-fd20.documents.azure.com:14318","isClosed":false,"configuration":{"maxChannels":130,"maxRequestsPerChannel":30,"idleConnectionTimeout":0,"readDelayLimit":65000000000,"writeDelayLimit":10000000000},"state":{"channelsAcquired":0,"channelsAvailable":0,"requestQueueLength":0}}}) is closed
Stack: java.lang.reflect.InvocationTargetException
at sun.reflect.GeneratedMethodAccessor7.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at com.microsoft.azure.functions.worker.broker.JavaMethodInvokeInfo.invoke(JavaMethodInvokeInfo.java:22)
at com.microsoft.azure.functions.worker.broker.JavaMethodExecutorImpl.execute(JavaMethodExecutorImpl.java:54)

Why is this not a Bug or a feature Request?
Although we do see exceptions, the cause of the exceptions seems to be related to configuration.
Setup (please complete the following information if applicable):
Java Azure functions 3.0 on consumption plan
com.azure:azure-cosmos:4.2.0
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

Thanks for filing this github issue @casper-79.  An expert from the Cosmos SDK team will follow up shortly.
/cc @kushagraThapar

@kushagraThapa Any update on this issue?

@casper-79 - I will look into this issue soon, will provide an update shortly, thanks.

@casper-79 - Thanks for the detailed explanation of your issue, here are my recommendations:

Using DIRECT connection mode is the right way to go, since GATEWAY has its limitations (as it is a shared resources).
directConnectionConfig.setMaxRequestsPerConnection(10); = 10 sounds good, as it is also the default value.
directConnectionConfig.setMaxConnectionsPerEndpoint(1); = 1, this is limiting the number of connections per endpoint to only 1 which will affect your application in case of very high load, or if the endpoint goes down for any normal reasons (like upgrades / updates / bad node / unhealthy node, etc.). So setting this value to 1 will have some side effects in case of these corner cases. I would recommend having at least few connections per endpoint so when you hit these corner cases, your application is able to recover from unavailable endpoints.

But if maxConnectionsPerEndpoint = 1 is working well for you in production, then stick with it for now, and you can update it if you see any issues.

Thanks @kushagraThapar
The thing is that it is not exactly working well in production. The configuration tweak is mildening the symptoms, but we are still seeing a boat load of exceptions that come with no usable explanation of cause. I had hoped you would be able to explain why the connection is constantly closed and why the change from default configuration is mildening the symptoms? We are not in production yet, but we need to get rid of this exception soon so we can go live ...

@casper-79  Let me explain the root cause of the exception you are seeing.
The connections get dropped because the idleEndpointTime reaches and it close the connections. Also, we noticed that in the earlier v4 version of SDK, there were some issues with connections hanging which has been fixed in SDK version 4.5.0. Please upgrade to the latest version.
At this point I can suggest two options for you to take it forward:

If you have fixed number of collections, then increase the idleEndpointTimeout to a bigger value - like 5 hours or 12 hours (default value is just 1 hour), which will ensure that connections are not dropped because of sparse workload that you have, and it will also ensure that you will not pay additional cost of creating and dropping connections every now and then. You will not see connections getting closed once you increase this value.
If you have many collections (thousands), then you need to quickly close the connections and re-create them. In this case, you should rely on default settings, because they should only close the connections when the idleEndpointTimeout reaches (by default 1 hour), and you actually want to close the connections at that point, to not acquire big resources from your machine.

Thanks @kushagraThapar
We had previously experimented with 4.3.1, which turned out to provide even more exceptions than with 4.2.0. However, I am happy to confirm that my preliminary testing confirms that this is indeed resolved by 4.5.0. So far the exceptions appear to have disappeared completely and our 99th percentile request times have dropped by more than a factor of 10. I am happy so far, but will keep the issue open until we have had time to do a more substantial load test ...

@casper-79 Thanks for the update, I am glad you are no longer seeing any issues, let me know once you have the full results.

@kushagraThapar
I can confirm that everything works as it should with 4.5.0. Case closed.


react-native-vector-icons not showing android in reactNative70

"react": "18.1.0",
"react-native": "0.70.0",
"react-native-vector-icons": "^9.2.0"
Hello
After installing the react-native-vector-icons  package and running the project, it doesn't show the icon and shows it as an unknown symbol, Now, I can no longer use the link command to link it in the new version of reactNative
now me npx react-native-asset  wrote to link the icons,But I'm wrong

Thank you for helping me

hi @emel95 have you found out how to fix it?

hi @glushkina1
yeah , I used this link and the problem was solved
https://github.com/oblador/react-native-vector-icons


[Subtask]: Validate the compatibility of Amoro with Hive 4.0 and provide support for Hive 4.0

Search before asking

[X] I have searched in the issues and found no similar issues.

Description
As the Hive-4.0.0-beta-1 is already available and soon there will be a GA. Amoro should validate the compatibility with it and support Hive 4.0 as soon as the GA is published.
Parent issue
#2564
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

https://github.com/apache/hive/releases/tag/rel%2Frelease-4.0.0


SITE_ID should not be a required setting.

Django no longer includes SITE_ID in the default settings.py and no longer adds django.contrib.sites to INSTALLED_APPS.
If it is actually required by fluent-blogs, adding django.contrib.sites to INSTALLED_APPS and defining SITE_ID should be mentioned in README.rst.
A quick search of the source reveals that this might also impact fluent-contents and fluent-pages. But for fluent-blogs it causes any management command to crash with a traceback.
Traceback (most recent call last):
  File "./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/core/management/__init__.py", line 399, in execute_from_command_line
    utility.execute()
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/core/management/__init__.py", line 392, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/core/management/base.py", line 242, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/core/management/base.py", line 285, in execute
    output = self.handle(*args, **options)
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/core/management/base.py", line 415, in handle
    return self.handle_noargs(**options)
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/core/management/commands/shell.py", line 70, in handle_noargs
    get_models()
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/db/models/loading.py", line 232, in get_models
    self._populate()
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/db/models/loading.py", line 75, in _populate
    self.load_app(app_name, True)
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/db/models/loading.py", line 99, in load_app
    models = import_module('%s.models' % app_name)
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/utils/importlib.py", line 40, in import_module
    __import__(name)
  File "/Users/tailee/Projects/django-fluent-blogs/fluent_blogs/models/__init__.py", line 1, in <module>
    from .db import Entry, Entry_Translation, get_entry_model, get_category_model
  File "/Users/tailee/Projects/django-fluent-blogs/fluent_blogs/models/db.py", line 5, in <module>
    from fluent_blogs.base_models import CommentsEntryMixin, AbstractTranslatableEntry, AbstractTranslatedFieldsEntry
  File "/Users/tailee/Projects/django-fluent-blogs/fluent_blogs/base_models.py", line 9, in <module>
    from fluent_blogs.models.managers import EntryManager, TranslatableEntryManager
  File "/Users/tailee/Projects/django-fluent-blogs/fluent_blogs/models/managers.py", line 8, in <module>
    from fluent_blogs import appsettings
  File "/Users/tailee/Projects/django-fluent-blogs/fluent_blogs/appsettings.py", line 65, in <module>
    if len(FLUENT_BLOGS_LANGUAGES.get(settings.SITE_ID, ())) > 1:
  File "/Users/tailee/.virtualenvs/fluent-demo/lib/python2.7/site-packages/django/conf/__init__.py", line 55, in __getattr__
    return getattr(self._wrapped, name)
AttributeError: 'Settings' object has no attribute 'SITE_ID'

That crash is an easy fix with getattr(settings, 'SITE_ID, None), but I see that SITE_ID is also used elsewhere in the fluents-blog source.

The appsettings code is fixed in d158ec0f618a1cd96f5d5f70b01061115cdf8a33, which can become part of v1.0.1. By defaut, the code now filters in SITE_ID to support multisite support. You can disable that with FLUENT_BLOGS_FILTER_SITE_ID = False in the settings. If you still have issues, a pull request is more then welcome!


python310Packages.aardwolf: 0.2.1 -> 0.2.7

Changelog : https://github.com/skelsec/aardwolf/releases/tag/0.2.7
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 222085 run on x86_64-linux 1

3 packages built:

crackmapexec
python310Packages.aardwolf
python311Packages.aardwolf

@ofborg eval


PyPI doesn't accept a file and suggests to use the same prefix the file has

Describe the bug
PyPI doesn't accept a file and suggests to use the same prefix the file has.
Expected behavior
The file is uploaded.
To Reproduce
twine upload h2o_pysparkling_3_2-<IP_ADDRESS>-1.tar.gz -u ********* -p *********
HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/
Start filename for 'h2o-pysparkling-3-2' with 'h2o_pysparkling_3_2'.

My Platform
Ubuntu 16.04 docker image in CI environment, twine was installed with conda
also Ubuntu 22.04 with twine version 4.0.2 (importlib-metadata: 6.6.0, keyring: 23.13.1, pkginfo: 1.9.6, requests: 2.28.1, requests-toolbelt: 1.0.0, urllib3: 1.26.15)
Additional context
Project: https://pypi.org/project/h2o-pysparkling-3.2/
Previously it didn't work with h2o_pysparkling_3.2-<IP_ADDRESS>-1.tar.gz filename, I expected the error to go away after changing, but the error stayed the same.
I tried looking into the code to understand the issue myself, but I couldn't find a source for "normalize_pep426_name" function
and sometimes, after many retries uploading the file works! with h2o_pysparkling_3.2-<IP_ADDRESS>-1.tar.gz style name

I'm a little confused by this:

and sometimes, after many retries uploading the file works! with h2o_pysparkling_3.2-<IP_ADDRESS>-1.tar.gz style name

I think the correct filename should be h2o_pysparkling_3_2-<IP_ADDRESS>-1.tar.gz and that h2o_pysparkling_3.2-<IP_ADDRESS>-1.tar.gz should fail, but seems like that's the opposite of what you're describing here?
I also don't see a successful upload with any of these versions, only ones with .post1 in the version.

@di thanks for the response!
it looks a bit like some servers accept, and some servers don't accept, I don't know the architecture of PyPI, but is that possible you have different versions deployed and some load balancer sometimes directs to server which accepts those names, and sometimes to another, which doesn't?
either way what I described in the first post looks like a bug to me, as the error message suggests exactly the prefix the file name has

it looks a bit like some servers accept, and some servers don't accept, I don't know the architecture of PyPI, but is that possible you have different versions deployed and some load balancer sometimes directs to server which accepts those names, and sometimes to another, which doesn't?

No, I don't think this is possible.

either way what I described in the first post looks like a bug to me, as the error message suggests exactly the prefix the file name has

I agree, but without the original filename this will be hard to diagnose. Are you still able to reproduce this?

ok, it turned out that my colleague had different setup tools version than me and CI
but the original issue is still valid, I can reproduce it easily I think

Thanks, if you're able to reproduce and can provide us with the file that reproduces, that would be very helpful!

I have the same bug on Windows 11.  It's delaying the next release of leo.
Here is a slightly edited copy of twine issue #1069 that was referred here.
Environment
Windows 11
python 3.12.0
python -m twine --version
twine version 5.0.0 (importlib-metadata: 7.0.2, keyring: 24.3.1, pkginfo: 1.10.0, requests: 2.31.0, requests-toolbelt:
1.0.0, urllib3: 2.2.1)

Which package repository are you using?
https://test.pypi.org/project/leo/
Please describe the issue that you are experiencing
The dist directory contains two files, both of which start with “leo”:
leo-6.7.8-9-py3-none-any.whl
leo-6.7.8-9.tar.gz

The suffix -9 is to ensure a unique file name.
The wheel uploads correctly, but twine complains about the .tar.gz file:
python -m twine upload -r testpypi dist/* --verbose
INFO     Using configuration from C:\Users\Dev\.pypirc
Uploading distributions to https://test.pypi.org/legacy/
INFO     dist\leo-6.7.8-9-py3-none-any.whl (14.4 MB)
INFO     dist\leo-6.7.8-9.tar.gz (12.5 MB)
INFO     password set from config file
INFO     username: __token__
INFO     password: <hidden>
Uploading leo-6.7.8-9-py3-none-any.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 15.1/15.1 MB • 00:05 • 2.8 MB/s
INFO     Response from https://test.pypi.org/legacy/:
         200 OK
INFO

           <title>200 OK</title>

200 OK

Uploading leo-6.7.8-9.tar.gz
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 13.1/13.1 MB • 00:04 • 2.9 MB/s
INFO     Response from https://test.pypi.org/legacy/:
         400 Start filename for 'leo' with 'leo'.
INFO

           <title>400 Start filename for 'leo' with 'leo'.</title>

400 Start filename for 'leo' with 'leo'.

           The server could not comply with the request since it is either malformed or otherwise incorrect.

         Start filename for 'leo' with 'leo'.

ERROR    HTTPError: 400 Bad Request from https://test.pypi.org/legacy/
         Start filename for 'leo' with 'leo'.

Does anyone have any advice?
Please include the PKG-INFO file contents from the artifact you're attempting to upload
Metadata-Version: 1.0
Name: leo
Version: 6.7.8
Summary: An IDE, PIM and Outliner. See http://leo-editor.github.io/leo-editor/preface.html.
Description: See users own descriptions of Leo at leo-editor.github.io/leo-editor/testimonials.html.
Home-page: http://leo-editor.github.io/leo-editor/
Author: Edward K. Ream
Author-email<EMAIL_ADDRESS>License: MIT
Keywords: Outliner, IDE, PIM, DAG, Scripting
Platform: Windows, Linux, MacOS
Classifier: Development Status :: 6 - Mature
Classifier: Operating System :: MacOS
Classifier: Operating System :: Microsoft :: Windows
Classifier: Operating System :: POSIX :: Linux
Classifier: Programming Language :: Python
Classifier: Topic :: Software Development
Classifier: Topic :: Text Processing
Download-URL: https://github.com/leo-editor/leo-editor/releases

A redacted version of your .pypirc file
[distutils]
  index-servers =
    pypi
    testpypi
    leo

[pypi]
  username = __token__
  password = pypi-AgEIc...

[testpypi]
  username = __token__
  password = pypi-AgENdG...

@edreamleo That's a separate issue, due to you using a version epoch, and would technically be a duplicate of https://github.com/pypi/warehouse/issues/12245. You might want to use a post-release instead.

@di Thanks for your quick response. I'm still pretty confused.
Has anything re filenames changed recently? Do release notes for PyPi itself exist?
I also wonder why the wheel file uploads correctly with a -nnn suffix but the .tar.gz file does not.

@edreamleo Since this error message is pretty confusing, I've proposed a temporary fix for this in #15644 which will allow PyPI to permit your original filename.
Eventually though, your builder should support PEP 625, and produce a filename with a normalized version instead:
>>> from packaging.version import Version
>>> Version("6.7.8-9")
<Version('6.7.8.post9')>

Since it looks like you're using setuptools, https://github.com/pypa/setuptools/issues/3593 is the relevant issue.

@di Thanks for the work you do! It can't be easy trying to maintain compatibility while moving forward ;-)


[1321] Make update command return the number of updated rows.

Description

Resolves #1321.
How was this patch tested?

Modified existing tests.
Does this PR introduce any user-facing changes?

Yes. The update command returns the following output:

Thanks @scottsand-db!


SSAO fades out when close to screen edge

Hi there...
awesome SSAO for orthographic cams you got there!
First of all, thanks for sharing!
We run in a little problem with it, maybe we do something wrong or you have an idea how to fix it easy...
When our cam moves away from our walls, at some point near the screen edge the ssao fades out. so all corners light up, wich is very visable and make this effect nearly unusabel for us...
https://drive.google.com/open?id=1bTtlemQEY6EHYlsu4E_7mhHamVM2MuLQ
look at the top corner of this video...
We are using Unity 2018.4+
Orthographic cam:

Any Ideas? Did we miss something?

That's a general problem about SSAO; Every SSAO technique has the same issue. You can work around the problem by rendering to a bigger render texture and cropping it to hide the edges, it's not a cost effective solution though.
Please understand that this kind of problem is pretty common in screen space image effects (SSAO, SSR, etc.). If it's not acceptable, I'd recommend looking for other non-screen space techniques.

Oh wow that was quick...
Dont get me wrong, your system is really awesome and i might sound harder and more rude as i wanted to xD
Yeah i thought that this might be a generall problam of screenspace based calculations.
Is there maybe a way to give your ssao more viewfield over the actual screenspace or has this posteffect only the data rendered from the cam?
I think i will talk to my programming dude if we could try your other solution too. Our game is just a little vertical slice and we dont need to fear performance issues in the moment. game runs smooth.
If you intereseted have a look in the game i could send you a builded version to test :-)
Thanks again for sharing your work and the fast response to my tiket !
Gesendet: Montag, 24. Juni 2019 um 16:35 Uhr
Von: "Keijiro Takahashi"<EMAIL_ADDRESS>An: keijiro/MiniEngineAO<EMAIL_ADDRESS>Cc: LordHelmchen12<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [keijiro/MiniEngineAO] SSAO fades out when close to screen edge (#6)
That's a general problem about SSAO; Every SSAO technique has the same issue. You can work around the problem by rendering to a bigger render texture and cropping it to hide the edges, it's not a cost effective solution though.
Please understand that this kind of problem is pretty common in screen space image effects (SSAO, SSR, etc.). If it's not acceptable, I'd recommend looking for other non-screen space techniques.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

By the way, I noticed that you're using Obscurance in the screenshot you provided. This repository is MiniEngineAO -- not Obscurance.
Please clarify if you're actually using MiniEngineAO.

Oh i think i am in the wrong repo :-(
Sorry my bad, yes we are using obscurance! Feel Free to remove or move this dialoge over.

I'm closing this issue now. Please feel free to for further problems.


How to specify include_dir with cythonize?

Hello,
I am trying to use Cython to accelerate some slow, math-heavy code in a large existing project. Since the project doesn't use setup.py for dependencies, I would like to use the cythonize command line tool to build the cython binaries. This works well, but I am unable to figure out how to specify an include directory, to e.g. include Numpy header files.
Does anyone have guidance? E.g. to fix this issue:
$ cythonize -a -i **/*.pyx -3

Compiling math-heavy-routines.pyx because it changed.
[1/1] Cythonizing math-heavy-routines.pyx
running build_ext
math-heavy-routines.c:611:10: fatal error: 'numpy/arrayobject.h' file not found
#include "numpy/arrayobject.h"
         ^~~~~~~~~~~~~~~~~~~~~
1 error generated.
error: command 'gcc' failed with exit status 1```

Use CFLAGS="-Inumpy_dir" cythonize ... as distutils honors some common environment variables. If you need to link some shared objects you can explicitly provide LDFLAGS as well.

Note that the bug and feature tracker is not the right place to ask support questions. Use the cython-users mailing list for that.


Add custom cursors

Something that is clearly needed for Runebender is the ability to set custom cursor images, so that we can have a pen cursor when we're in the pen tool, etc.

I think this is done by #1183, right?

My hero 🥰