text:
Alternatives to scribe for text formatting

As per conversation on #410 to consider an alternative to Scribe for the text formatting on Sir Trev.
The reasons for asking to consider this is based around my own and other Sir Trev users' issues with the lack of documentation from Scribe (even they admit it themselves but very little has been done since this acknowledgement) as well as poor browser support (Scribe officially only supports Chrome and Firefox desktop versions).
As a replacement to consider that uses very similar methods to Scribe would be Quilljs which has strong browser support, better documentation as well as a more active development scene behind it.

This is a really good idea. We've been following quill for a while, however we weren't aware of it at the time that we decided to implement Scribe. We need to do a bit of investigation into whether it meets all our needs and how me might go about integrating it. Good call though. Maybe @callum has some opinions?

Quill is certainly a much nicer solution than Scribe. It abstracts away the complexities of patching HTML and instead handles events and builds an internal document structure, of which HTML can be generated from. A much cleaner, better-supported approach in my opinion.
I don't see any reason not to go ahead with it, but then I am least familiar with the Sir Trevor codebase. I defer @raffij to make a decision on it.

OH HAI
Agree with Callum on this. A clean document document structure is something
which is just not possible while allowing people to write HTML. No matter
how cleverly you transmute it.
Therein lies the theory behind ST separating -- from a users perspective as
well as in document structure -- block from inline elements
http://www.w3schools.com/html/html_blocks.asp.
However, we were little too literal with this in terms of UI. The theory
remains sound, I believe, however others
http://slackhq.com/post/130081834025/refreshing-posts-20 have handled it
better.
On Thu, 3 Dec 2015 at 12:13 callum notifications@github.com wrote:

Quill is certainly a much nicer solution than Scribe. It abstracts away
the complexities of patching HTML and instead handles events and builds an internal
document structure http://quilljs.com/docs/deltas/, of which HTML can
be generated from. A much cleaner, better-supported approach in my opinion.
I don't see any reason not to go ahead with it, but then I am least
familiar with the Sir Trevor codebase. I defer @raffij
https://github.com/raffij to make a decision on it.
—
Reply to this email directly or view it on GitHub
https://github.com/madebymany/sir-trevor-js/issues/413#issuecomment-161617421
.

The idea with Scribe wasn't to allow people to write HTML, just for rich text to be serialised as HTML. If Quill does this better than Scribe, super duper.

Quill works for what i've been doing with some new features. Manipulating delta arrays is much nicer than working with all the edge cases in browsers i've been working to fix. If it can read and write html in exactly the same way in all browsers then i'm happy with it as a solution.
Also if it works with mobile browsers then this is a big bonus too.
@SgtOddball thanks for starting the discussion.

Sorry, I didn’t put that well.
As far as the human is concerned, they are inputting rich text, be it
Scribe or Quill, but as soon as it hits the browser it becomes HTML, which
then has to be then transformed into something structured in a way which
can then be reliably stored, queried and transformed in a view.
Be that transformation from messy HTML to clean HTML, or from messy HTML to
JSON, or from messy HTML to a RDBMS, it always starts with messy HTML so
long as you rely on browsers to take those key strokes and impart their
opinion on what the user use meant without that opinion being made explicit.
So, for what it’s worth, I believe that content should not be stored as
HTML, temporarily in the browser, nor permanently the database; HTML it too
opinionated, as much as it pretends to be a structured language, it’s
really a presentation layer.
Quill is closer to this than Scribe, but it’s still essentially a
work-around.
I think what Medium, Slack etc. have shown is that it’s possible to stick
to this clean technology approach while providing an elegant user
experience.
So, why are we still trying to hack HTML input/output?
A
On Thu, 3 Dec 2015 at 14:00 Dan Brown notifications@github.com wrote:

The idea with Scribe wasn't to allow people to write HTML, just for
rich text to be serialised as HTML. If Quill does this better than Scribe,
super duper.
—
Reply to this email directly or view it on GitHub
https://github.com/madebymany/sir-trevor-js/issues/413#issuecomment-161647789
.

I knew what you meant really and agree.
If we do use quilljs then we can store data as deltas and set the format on the text blocks.

If we don't store Quill deltas (and I can't see that being a good plan), we'd have to choose something to serialise to. You could either use HTML or something equivalent to it. HTML is better understood, more widely supported, and easier to eyeball than some odd other format. It's totally possible to generate clean HTML: Scribe does it, it's other aspects of Scribe we're not so happy with.

@dwb as you say html is best for us, but just wanted to say it's possible now with the updates we did when scribe was added.
It's also different when you control the entry and output in a closed environment such as medium / slack.

You need to store deltas and HTML.

HTML should be enough?

Eep, I must admit I didn't think this would strike up such a storm so quickly.
Personally, so long as the HTML block/inline states are preserved in the Sir Trev data arrays then that should be ideal as I can apply any further customisation when storing the block elements. (Which reminds me, I really should look at outputting to flat pages...) as I currently do now.
@raffij With regards to setting format on the blocks, would this include inline formatting as well or would we talking the whole span/div/{insert html element of choice}? As I would have thought that would be a step back for those of us who've done some enhancements already to the format bar (underlines, font resizing and that kind of jazz) which relies on being able to apply the HTML over the select text.
(Which reminds me, need to check how Quill handles anchor tags. I like to use things like phone numbers, email addresses and the like which usually end up getting ignored)

Don't you only need to store deltas if you want to preserve undo stages? Which isn't really necessary?

I'm gonna look like a right idiot on this one, but would it be possible to clarify what you mean by delta arrays (I get arrays and how to handle them for just about anything, I'm not that slow) as I get the feeling its not what I think it is.
(Yes, I never actually went to uni/college to do programming, how did you know?)

@SgtOddball http://quilljs.com/docs/deltas/

......RTFM..... excuse me while I go cry in a server room....

In that case after actually doing some reading, I'd agree with @dwb on this one.
Once the edits have been saved/committed then the deltas aren't needed any more since the changes should be in the HTML as it appears on the page.
The only reason I can see for them to be changed would be if you wanted version/authorship type facilities so you can add who did what over the life of the block/text.
This would however cause issues if the block was removed as it should then also remove said history making it a half measure if saved or requiring versioning against the blocks as well.
Your own requirements may of course differ but thats what comes to mind after having a quick nosy.

To my understanding deltas are instructions for rendering a document. History is managed elsewhere. So the point of storing deltas would be to restore the state of a document to the editor when you're making changes to it. I believe it can also parse the HTML back into a delta, but whether that's how you're supposed to do it, I don't know.

I like this idea in theory but doesn't it present us a problem with
backwards compatibility?
On Thursday, 3 December 2015, callum notifications@github.com wrote:

To my understanding deltas are instructions for rendering a document.
History is managed elsewhere. So the point of storing deltas would be to
restore the state of a document to the editor when you're making changes to
it. I believe it can also parse the HTML back into a delta, but whether
that's how you're supposed to do it, I don't know.
—
Reply to this email directly or view it on GitHub
https://github.com/madebymany/sir-trevor-js/issues/413#issuecomment-161788198
.

--
Made by Many Limited is registered in England and Wales (No.06346338).
Registered office: Diespeker Wharf, 38 Graham Street, London, N1 8JX, UK
Made by Many Inc is registered in the State of New York (No.4427021). 65
Greene Street, 4th Floor, New York, NY 10012, USA

Something we have to consider is quill produces valid html, but not like you'd expect from an html editor.
https://github.com/quilljs/quill/issues/252
https://github.com/quilljs/quill/issues/228
This is a paragraph

This is another paragraph
with a breakline.

------

This is a paragraph

This is another paragraph

with a breakline.

That's annoying. Perhaps that matters less to us as we're using blocks for each new paragraph now?

We'll want to see what the overhead of an editor per paragraph is with this.

The default branch on quill repo is develop and includes a complete rewrite of the editor, which is unfortunately not finished yet.
The current stable version doesn't look great: https://github.com/quilljs/quill/tree/0.20.1

I've opened a can of worms with this one!? I'll have a further look over the weekend and possibly see if there's any others that fit the bill better than Quill or Scribe (though at least with Quill they seem to be doing something and are willing to support the community).
@raffij I wouldn't have thought the overhead was that much greater than scribe, it seems to push about the same about of stuff into the console et al as scribe does but again I'll see if I can't find a way of comparing them both (been itching for something new to play with ff dev edition on so this looks ideal).
As for the current stable version @lukaszsagol at least it's being developed further. Scribe just seems to get minor bug fixes and thats your lot (at least from what I've been following since starting with Sir Trev and trying to get my head around scribe as well) but I do see your point, wonder if it'd be worth forking an experimental version of Sir Trev and see just what it takes to at least get some basic blocks up and going with it instead.

I think that Quill is still a good alternative, I would just wait for them to release the next version (after the rewrite) and asses it then. I'm really glad we're having this conversation as Scribe is what it is, and we should be on the lookout for something better.
Because we decided to use HTML as a data format for text blocks we need to make sure that editor we use produces consistently good quality output across different browsers - not sure if current Quill can promise that with their current version.

@lukaszsagol Thats a fare comment, and with regards to consistent HTML at least they're covering more browsers than scribe (that said I've only had seriously weird behaviour on Android chrome (Android FF seems to be behave well) and <IE11 (.natch but I'll be damned if I'm going to have to develop for <IE any more, bloody thing has taken far too many weeks/months/years of my life trying to fix it's idiocy.)

authors: SgtOddball andrewsprinz callum dwb lukaszsagol ninjabiscuit raffij

repo: madebymany/sir-trevor-js


text:
Not working with newer version of Julia

Is there any plan to update this repo?

I know that this work was primarily part of Madeleine's thesis. I'm not sure how much time for maintenance she has available. If you are using it in your work, you might want to make a fork.

source: gharchive/issue

authors: alibeyram jpfairbanks

license_type: permissive

license_source: bigquery


text:
🛑 qBitTorrent UI is down

In 10fc4f7, qBitTorrent UI (https://qbittorrent.$BASE_URL_MADHOGS) was down:

HTTP code: 0
Response time: 0 ms

Resolved: qBitTorrent UI is back up in 4d66975.

repo: madhogs/upptime

license: MIT

license_source: github-api


text:
u-boot env in sdcard partition

Hi,
this is not an issue just more request for some help. For OTA update system I need to handle u-boot envs (board jetson nano SDcard boot). AFAIK when checked u-boot code there is no env support enabled at all (I'm using warrior branch from meta-tegra). I can add it but as I'm newcomer in Nvidia world I have no idea how to add extra partition (just raw no filesystem necessary). Any help appreciated. Thanks.

AFAIK when checked u-boot code there is no env support enabled at all (I'm using warrior branch from meta-tegra)

There are some examples in https://github.com/mendersoftware/meta-mender-community/tree/zeus-test/meta-mender-tegra/recipes-bsp/u-boot/patches for dealing with u-boot environment on Tegra platforms.  This might be a good place to start if you haven't looked at these already.  See https://github.com/mendersoftware/meta-mender-community/blob/zeus-test/meta-mender-tegra/recipes-bsp/u-boot/patches/0001-Move-jetson-nano-env-to-SDcard.patch for instance.
You may find the meta-mender-community zeus-test branch useful to see an example implementation for mender which supports u-boot environment variables on Tegra.  There are some options implemented there to locate uboot environment in spi flash or MMC on nano platforms using TEGRA_MENDER_UBOOT_ENV_IN_SPIFLASH which may be useful as well.

@dwalkes thanks a lot. Yes I got some inspiration from mender patches and have a look at them. Bit of magic for me is still to understand how can I add dummy raw partition for env storage. I saw some xml files but I'm not sure if they are generated by some tool or manually? Thanks.

@nandra sure, glad to help

how can I add dummy raw partition for env storage

In the mender case I don't add a dummy partition to the xml file.  I just use mmcblk0boot1 space located in the empty space before the secondary_gpt.  For the history of how the offset was selected you can see this post on the nvidia forum ultimately reflected in the setting of MENDER_UBOOT_ENV_STORAGE_DEVICE_OFFSET.

I saw some xml files but I'm not sure if they are generated by some tool or manually?

The xml files in meta-mender-community are manually created/updated today based on values provided in NVIDIA source for a given L4T release (see some early discussion about this here), but then sizes for images and other parts are filled in automatically to correspond with yocto build parameters by the logic in meta-tegra, see examples related to APPSIZE for instance which you can find updated here.

I just use mmcblk0boot1 space located in the empty space before the secondary_gpt.

Unfortunately, SDcards don't have the bootX partitions that eMMCs have, so creating a custom XML partition layout file is the only choice, if you want the U-Boot environment on the SDcard.
The warrior branch doesn't include any U-Boot environment support on the Nano, since there was no support for it in L4T R32.1.  IIRC, NVIDIA added support for that in R32.3.1, so the branches based off that version of L4T will have it by default.  I also added it to the R32.2.x-based U-Boot fork we use in the zeus branch... although the specific offset into the flash may be different than in R32.3.1+, since my implementation pre-dates NVIDIA's.
If using the QSPI flash is that the environment is present regardless of whether you swap SDcards. That might be a good thing or a bad thing, depending on what you need.  As @dwalkes pointed out, the meta-mender-tegra layer has some patches that switch the storage over to SDcard, and adding an extra entry in the partition layout XML file isn't too hard to do.

@madisongh @dwalkes thanks for valuable info. I stole some bits and pieces from mender integration (I'm adding swupdate support). Now u-boot env is working fine. Again thanks a lot.

authors: dwalkes madisongh nandra

repo: madisongh/meta-tegra

license: mit


text:
paste does not get reset back to nopaste after doing :AIChat

I saw that 'paste' gets set inside of the aichat buffers in order to get the formatting to work right. Unfortunately, it does not appear to get reset to 'nopaste' after leaving the aichat buffer. I demonstrate this in the following recording:
Screencast from 06-06-2024 09:57:54 PM.webm
Here you can see the paste option is set to 'nopaste' before I run :AIChat. After :AIChat, the paste option is set to 'paste' both inside the aichat buffer and in other buffers. Closing the aichat window also does not work to set paste ack to 'nopaste'.
This is a problem for me because I normally have the autoindent option set. Setting 'paste' disables autoindent, which is useful for the aichat window, but I would like it to be turned back on automatically after I'm done with the AIChat buffer.
Side note, :setlocal paste is not a "valid" vim command, since 'paste' is strictly a global option with no option to set it locally (see :h option-summary and :h paste), but this is a minor point since :setlocal paste still works to set it globally.
I am working on a solution, but wanted to put this out there in the meantime.

I am working on a solution using vim.command('put ...') instead of going into insert mode to avoid the need to set 'paste' entirely. Apparently I may be the only one experiencing this particular bug, so it may be a result of some peculiarity unique to my system, but as evidenced by #94 and #96, I am not the only one experiencing difficulty with the 'paste' option in general. My fix may resolve those issues as well if paste is no longer needed. I will likely have this ready to go over the weekend.

Yes, there's https://github.com/madox2/vim-ai/pull/94 and https://github.com/madox2/vim-ai/pull/96 to make up for it, but they weren't vetted yet; would you mind testing them?

See also the discussion in https://github.com/madox2/vim-ai/issues/106

Apparently I may be the only one experiencing this particular bug

You are not alone, that's why the PRs have been proposed

I just pushed functionality handling paste mode (https://github.com/madox2/vim-ai/commit/dac5aad10863d4ff7c00b349c59a3857f3bf8bd6, https://github.com/madox2/vim-ai/commit/9cd477a11e3044c1af59bc07a0bfc5b6094e912e, highly inspired by @Konfekt PRs). Please let me know if that resolved the issue.
@Konfekt is there anything else that we might be missing from #94 and #96 now?

@madox2 The latest commits seem to have fixed the issue. Thank you!

@madox2 No, it only differs in, I'd suppose, negligible details

authors: Konfekt madox2 misterbuckley


text:
Expressions are not being properly evaluated

Hey there! Wondering if Im doing something wrong or there is a bug here.
Crontab.Scheduler.get_next_run_date(~e[*/2 * * * *], ~N[2023-12-02 14:40:00.298713])
{:ok, ~N[2023-12-02 14:42:00]}

Crontab.Scheduler.get_next_run_date(~e[*/3 * * * *], ~N[2023-12-02 14:40:00.298713])
{:ok, ~N[2023-12-02 14:42:00]}

Crontab.Scheduler.get_next_run_date(~e[*/4 * * * *], ~N[2023-12-02 14:40:00.298713])
{:ok, ~N[2023-12-02 14:44:00]}

On the 3 minutes interval, Im getting a wrong minute number.
And more random results
 Crontab.Scheduler.get_next_run_date(~e[*/4 * * * * *], ~N[2023-12-02 14:50:01.989811])
{:ok, ~N[2023-12-02 14:52:00]}

Any clue?

Forget about it. This is why you shouldn't have to code until very late. 😓
Thanks a lot for your quick response!

repo: maennchen/crontab


text:
Windows redownload fails?

Could not evaluate: Could not find command 'test'

Presumably due to $unless_test in /fetch.pp ?

I have the same error.  it's actually this line that is failing
$unless_test = "test -s ${destination}"

so what would work in windows for the unless test? I don't have access to a windows box

exist ${destination}
cmd -D exist ${destination}
if exist ${destination} true else false
?

Fixed in #54

authors: TrentBartlem carlossg richardwadsworth

repo: maestrodev/puppet-wget


text:
[dy] Fix frontend routing

Summary
/pipelines/{uuid} was getting appended to the url every time I refreshed the page.

Tests
tested locally

cc: @johnson-mage

@dy46 ah, thx david

source: gharchive/pull-request

repo: mage-ai/mage-ai


text:
fix missing param for origURL in addOptions()

fix #26
Currently, the request URL remains unchanged even after passing through the addOptions function, even though the values for the Page & Limit attributes in ListSubscriberOptions have been filled in. To address this issue, I have made some adjustments in this pull request.

Thanks for this @alunparanggi much appreciated not sure how this was missed 👍

repo: mailerlite/mailerlite-go


text:
Add support to PHP and Ruby like multiple fields

Purpose
PHP and Ruby frameworks pass multiple parameters as param[0], param[1] for PHP and param[], param[] for Ruby.

lgtm

repo: mailgun/scroll


text:
NOISSUE - Fix default db name for storage databases

Signed-off-by: Ivan Milošević iva@blokovi.com

Codecov Report

Merging #1194 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1194   +/-   ##
=======================================
  Coverage   77.24%   77.24%
=======================================
  Files         102      102
  Lines        6817     6817
=======================================
  Hits         5266     5266
  Misses       1181     1181
  Partials      370      370

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 ea3a891...0099079. Read the comment docs.

authors: blokovi codecov-commenter

repo: mainflux/mainflux


text:
[FEATURE] Add integration to deploy models via Seldon Core

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

As this issue is open for a few weeks now, and the main contributors have not been able to prioritize this ticket, we have decided to take this feature request into the ROADMAP. This maintains transparency, while also clearing up the issue board, which can then be a place for more short-term or WIP features/bugs/enhancements etc.

repo: maiot-io/zenml


text:
fix(operator): add new groupname to csv config

related to #891

Just fix this unit test :)

authors: aslakknutsen bartoszmajsak

repo: maistra/istio-workspace


text:
MAISTRA-1897: Allow disabling IngressClass support

This adds an option to istiod that allows disabling reading
IngressClass resources, which are cluster-scoped.  When this is
disabled, processing of Ingress resources that use the new (as of
Kubernetes v1.18) ingressClassName field in their spec is not
supported.  Only resources that use the deprecated annotation will be
processed.
Cherry-pick of abb958783e for Maistra 2.1 / Istio 1.8 rebase.

/retest

authors: bison jwendell

repo: maistra/istio


text:
Edit temp_message to be the final message with the generated pictures

Instead of deleting temp_message, we should edit it instead.

Instead of deleting temp_message, we should edit it instead.

Just use await ctx.defer

I would like it to show the prompt, so I just made it edit the message instead of deleting it.


text:
🛑 Personal Homepage is down

In 9bf0b6b, Personal Homepage (https://aurelienooms.be) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Personal Homepage is back up in f9dfb00.

repo: make-github-pseudonymous-again/monitor


text:
Update database-email-update example README instructions

Small change: add instruction for giving integration access to database. An error is thrown if this step is not completed first.

 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.

repo: makenotion/notion-sdk-js


text:
Update MIP0c13-SP1.md

Offboarding is now voluntary and does not go through Governance.
This subproposal is therefore obsolete.

@LongForWisdom , @CPSTL , please review.

@LongForWisdom , @CPSTL , please review.

Agreed, looks good to me. Merging in with the updated status.

authors: CPSTL juan-fintech

repo: makerdao/mips


text:
Bug in Ackerman function calculator (not working)

Describe the bug

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

If applicable, add screenshots to help explain your problem.

Hi, I want to work on this issue I will start working on it as soon as I get assigned!! I am a part of GSSoC'21.
Kindly assign this issue to me!
Discord Username : Abhijeet Sinha(P)
Discord Tag: #4018

repo: makesmatheasy/makesmatheasy


text:
feat(): validation on boolean

Hi,
The example here doesn't seem to work on the last version:
class Foo(val bool: Boolean)
val v = Validator.builder<Foo>()
        .constraint(Foo::bool, "bool") { it.toTrue() } // this method signature isn't valid

The constraint method never has as first parameter a Function<T, Boolean> so this won't compile.
Due to that, I think the example in the doc isn't working > https://github.com/making/yavi/blob/develop/docs/FromBeanValidationToYAVI.md#asserttrue

Good catch!
ToBoolean has been completely forgotten ...

👍

0.0.19 has just been released.
It's under the process of synchronizing maven central.

authors: davinkevin making

repo: making/yavi


text:
[FEATURE] Proper text copying/pasting

Is your feature request related to a problem? Please describe.
Text is hard or nearly impossible to copy/paste right now.
Describe the solution you'd like.
A way to highlight text across multiple lines to copy. Also a pasting option.
Describe alternatives you've considered.
None.

Copying almost done

Finally done :) Phew...


text:
progress Handling, compatibility with Pace.js

Hi,
Is it possible to get progress of loaded content ?
I am trying to use pace.js with l.js but it does not work.
I have lots of assets and I need to display progress as just showing busy-spinners dont give information to users on how much is loaded and how much remaining.
Something like
ljs
.addAliases(aliases)
 .load('all-assets',function(){ /* your callback here */})
 .onProgress( function(value,total ){ /* could display progress here in percentage */})

l.js don't check any progress of the loading resources. I don't know about pace.js i'll have a look into it and come back to you as soon as possible

after more invesigation on the subject i think it's more to pace.js to be able to detect new resources insertion in the dom. Adding a progress event management in l.js will kill its purpose (be the smaller it can be without doing extra things). So for now i won't make any modification in that way and close the issue at won't fix.
Thanks for your report anyway

authors: djangosdk malko


text:
Fix #1302: File#tell: SEEK_CUR is a local constant

This fixes issue #1302.

:open_mouth: you're fast!

Makes you wonder if we can have a spec for this though.

I was thinking about doing one, but I'm not sure how to do a spec that involves dealing with a file. Create a temp file, operate on it and then delete it?

I think we already do that a couple of times, yeah.

Oh, this would be really nice. Note that this class is buffered and right now the result of tell is broken IIRC. You can always compre against Ruby to check the correct behavior :)

I have updated with some specs and fixes for seek and tell on buffered io. This also fixes #1306.

:+1:

Is this normal?

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
The build has been terminated

Most likely not your issue at least, the 32bit build seems to stall sometimes, maybe a sleep call going wrong or idk.

Awesome, thanks!!

Forgot to link this: https://github.com/manastech/crystal/commit/6cf0ebc88625132981b441695593058a2f4e4954
I'm wondering, would it make sense to move tell and seek to File? For pipes and sockets these will always fail...

@asterite All the spec for it is located at file_spec.cr because it indeed makes no sense anywhere else. I would agree with this move.
As a side note, StringIO should support seek and tell as well, so maybe the Seek enum should still remain on IO, where each specialization is free to implement seek/tell as it pleases.

authors: BlaXpirit asterite jhass lbguilherme

repo: manastech/crystal


text:
GitHub Action to lint Python

Start towards #15...  Blocked by #34  Output: https://github.com/cclauss/TrustTrees/actions

Thanks again @cclauss!

authors: KevinHock cclauss

repo: mandatoryprogrammer/TrustTrees


text:
:robot: Add kerbrute.vm

Automated PR from https://github.com/mandiant/VM-Packages/issues/523 :rocket:

Tested this locally. Package and shortcut work great.

repo: mandiant/VM-Packages


text:
use pre-commit to invoke linters

adds a pre-commit configuration so that its easy to invoke all of our linter checks w/associated configuration, as well as install the pre-commit and pre-push hooks consistently:

quickstart
do this now @mr-tz @mike-hunhoff @yelhamer @colton-gabertan (FYI)
$ pre-commit install    # now pre-commit hook installed!

or if you want to run all the linters on-demand:
$ pre-commit run --all-files

the PR also introduces some additional plugins (and fixes) for flake8:

bugbear - find likely bugs, which it did, such as use of str.lstrip("many characters here")
encodings - enforce text read from files has an explicit encoding
comprehensions - use list/tuple/set comprehensions where appropriate (and don't, when not appropriate)
logging-format - don't use format strings in logging calls
no-implicit-concat - catch potential bugs by disallowing implicit string concatenation, like "foo" "bar" == "foobar"
print - catch print statements in library code
todos - enforce TODOs are formatted like TODO(author): message and have an associated link to a tracking github issue

im really excited about some of these lints (like logging format), since we've often had to explain and manually check for these during PR reviews. the other ones enforce better hygiene (imho).
closes #1579
closes #1391
Checklist

[ ] No CHANGELOG update needed

[x] No new tests needed

[x] documentation update

just getting tests to pass and then i'll open this up for review

i have a few more linters in the works (flake8 plugins) so i'll get those hammered out before we merge.

@ajnelson-nist check out how we integrated pre-commit here (thanks to your pointer).
specifically, we use a local configuration rather than using the third-party pre-commit hooks. this let's use declare the linter dependencies in setup.py/pyproject.toml and therefore get updated by Dependabot while still getting invoked by pre-commit.
https://github.com/mandiant/capa/blob/f983307c978b82911d823ac291f4e397393f4ba2/.pre-commit-config.yaml

@williballenthin And now you've explained something I hadn't quite understood from the examples I'd drawn from.  Local configurations!  That's how they'd gotten around pinning-vs-range version freshness issues.
This was a fruitful back and forth.  Thanks for letting me know what you found!

authors: ajnelson-nist williballenthin

repo: mandiant/capa


text:
Silent refresh timing issue

Hi,
Silent refresh timeout is calculated wrong if user does a page reload. calcTimeout() returns constant timeout when ever it is called. Shouldn't it take into account current time, not stored_at?
E.g. in Scenario:

Token expiration time is 1min
timeoutFactor 0.5
calcTimeout() returns 30000 (30sec) when user does a page reload, causing that app looses valid token, depending on the timing the page reload occurs.

Following change seems to work in our use case (+ changes to calling code). Is there some reason behind to use stored_at?
private calcTimeout(storedAt: number, expiration: number): number {
    let delta = (expiration - storedAt) * this.timeoutFactor;
    return delta;
}

=>
private calcTimeout(expiration: number): number {
let delta = (expiration - Date.now()) * this.timeoutFactor;
return delta;
}

The timeout calculation is still causing problem.
If I reload the page a few seconds before the token lifetime ends I'm still logged in but on REST cals I get an 401 (UNAUTHORIZED).
So there are two options.

send logout event when token expires
change timeout calculation to refresh token in time

We were having the exact same issue in our application.
It's easy to reproduce if you refresh the site while the silent refresh timeout is running. When the page reloaded it will start a new timer but with the same timeout as before (because storedAt and expiration have not changed). This causes the token to expire before the silent refresh renews the token.
Example:
Current time: 12:00:00
Token is stored at: 12:00:00
Token expires at: 12:01:00

calcTimeout returns: 45 seconds
Silent refresh will happen at: 12:00:45

--- Wait 20 seconds. Refresh the page ---

Current time: 12:00:20
Token is stored at: 12:00:00
Token expires at: 12:01:00

calcTimeout returns: 45 seconds <-- This is the error!
Silent refresh will happen at: 12:01:05
but token already expired...

We manually overwrote the calcTimeout function with the implementation @gerkirill suggested in  https://github.com/manfredsteyer/angular-oauth2-oidc/pull/213#issuecomment-373363958 which fixes the problem

We manually overwrote the calcTimeout function with the implementation @gerkirill suggested in #213 (comment) which fixes the problem

@joostme, can you please elaborate on how you managed to overwrote calcTimeout ?

@morvans

@joostme, can you please elaborate on how you managed to overwrote calcTimeout ?

In the Angular Service we first inject the OAuthService from 'angular-oauth2-oidc' we overwrite the calcTimeout function the following way:
(<any> this.oAuthService).calcTimeout = function (storedAt: number, expiration: number): number {
    const tokenLifetime = expiration - storedAt;
    const refreshTime = storedAt + (tokenLifetime * this.timeoutFactor);
    const timeLeft = Math.max(0, refreshTime - Date.now());
    return timeLeft;
};

@joostme :+1:
Thanks !

@joostme Thanks for the tip. I didn't know about this override ability.
@manfredsteyer This will really need to be somehow fixed and released.

authors: Kotohitsu Lasseku ici-la-lune joostme morvans


text:
How to remove the warning message for v9?

Since v9 there's a warning message printed to console.
PLEASE READ THIS CAREFULLY:

Beginning with angular-oauth2-oidc version 9, the JwksValidationHandler
has been moved to an library of its own. If you need it for implementing
OAuth2/OIDC **implicit flow**, please install it using npm:

  npm i angular-oauth2-oidc-jwks --save

After that, you can import it into your application:

  import { JwksValidationHandler } from 'angular-oauth2-oidc-jwks';

Please note, that this dependency is not needed for the **code flow**,
which is nowadays the **recommented** one for single page applications.
This also results in smaller bundle sizes.

It's also present when creating a prod build.
How do you remove it?

The warning already explains it, right? Which part can we clarify for you?

Hi, yes, I do want to still use implicit flow, especially, when your own practice branch can't find a way to do a silent refresh.
Also I already installed the OIDC-JWKS package and I would want a way to stop this error appearing in a console.

How. do. you. remove. the. warning. message. from. production. builds?
idk how I can word it any differently

Ok apparently you remove the line
this.oAuthService.tokenValidationHandler = new JwksValidationHandler();
And no it's not self-explanatory.

authors: Mysq dalu jeroenheijmans


text:
Issue #257: Update codecov script

Issue #257

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@88bff65). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #258   +/-   ##
=========================================
  Coverage          ?   60.70%
=========================================
  Files             ?       37
  Lines             ?      855
  Branches          ?      216
=========================================
  Hits              ?      519
  Misses            ?      239
  Partials          ?       97

Impacted Files
Coverage Δ

src/AwssumePlugin.groovy
92.00% <0.00%> (ø)

src/BuildStage.groovy
24.13% <0.00%> (ø)

src/StageDecorations.groovy
100.00% <0.00%> (ø)

src/TerraformApplyCommand.groovy
86.84% <0.00%> (ø)

src/ConditionalApplyPlugin.groovy
66.66% <0.00%> (ø)

src/TerraformPlugin.groovy
51.42% <0.00%> (ø)

src/TerraformValidateCommand.groovy
87.50% <0.00%> (ø)

src/DestroyPlugin.groovy
75.00% <0.00%> (ø)

src/DefaultStrategy.groovy
0.00% <0.00%> (ø)

src/ParameterStoreBuildWrapperPlugin.groovy
39.13% <0.00%> (ø)

... and 27 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 88bff65...9dcdbac. Read the comment docs.

authors: codecov-commenter kmanning

repo: manheim/terraform-pipeline


text:
🛑 Mesa Freeworld is down

In 9117984, Mesa Freeworld (https://nonfree.eu) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Mesa Freeworld is back up in adae218 after 12 minutes.

repo: manjaro-contrib/upptime


text:
🛑 Mesa Freeworld is down

In b441837, Mesa Freeworld (https://nonfree.eu) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Mesa Freeworld is back up in 9ad38ba after 18 minutes.


text:
🛑 Mesa Freeworld is down

In 3121e0a, Mesa Freeworld (https://nonfree.eu) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Mesa Freeworld is back up in 9e72dd5 after 1 hour, 26 minutes.


text:
🛑 Mesa Freeworld is down

In 7fb8c31, Mesa Freeworld (https://nonfree.eu) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Mesa Freeworld is back up in 981c40d after 22 minutes.


text:
correction of authors name

corrected the authors name in the following news posts:

manjaro-arm-dec
manjaro-arm-nov
manjaro-arm-oct

Thanks!

authors: RahafEK romangg

repo: manjaro/website


text:
Sort get_model_scales_available result values

Sort get_model_scales_available result values to avoid unit test fails

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (ba01759) 99.83% compared to head (6674e67) 99.83%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #13   +/-   ##
=======================================
  Coverage   99.83%   99.83%
=======================================
  Files          14       14
  Lines        1237     1238    +1
=======================================
+ Hits         1235     1236    +1
  Misses          2        2

Files Changed
Coverage Δ

imgtools_m8/model_conf.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


text:
Add Club Page

Please do work for this task in a branch called issue-45.
You should:

[x] Make a page where admins can add a new club

Add Club page created, closing issue

authors: msumaylo sumidaca


text:
Create Three.js loader for progressive mesh format

Depends on #1276
NLNet milestone 2.4

In progress at https://github.com/takahirox/three-gltf-extensions/pull/99


text:
Prettier 3.0 support

Hi, I recently updated prettier but its seems this plugin is not compatibly with the new prettier 3.0 api.
["ERROR" - 9:14:27 AM] printer.embed has too many parameters. The API changed in Prettier v3. Please update your plugin. See https://prettier.io/docs/en/plugins.html#optional-embed
Error: printer.embed has too many parameters. The API changed in Prettier v3. Please update your plugin. See https://prettier.io/docs/en/plugins.html#optional-embed
    at printEmbeddedLanguages (file:///.../node_modules/prettier/index.mjs:17619:11)
    at printAstToDoc (file:///.../node_modules/prettier/index.mjs:17750:9)
    at async coreFormat (file:///.../node_modules/prettier/index.mjs:18061:14)
    at async formatWithCursor (file:///.../node_modules/prettier/index.mjs:18259:14)
    at async Module.format2 (file:///.../node_modules/prettier/index.mjs:21449:25)

Do you see any chance to update the plugin ?

Hi !
Thanks for the report 😃
I see that prettier 3 was out some days ago and that it did break the plugins system. I will look at it today to see if it's an easy upgrade 👍

Hi !
Thanks for the report 😃
I see that prettier 3 was out some days ago and that it did break the plugins system. I will look at it today to see if it's an easy upgrade 👍

Thnx for the quick response ! At a glance it does not look that big :)

@MarcelRoozekrans the plugin is upgraded for prettier 3 and tested on our main project.
See https://github.com/mapado/prettier-plugin-gherkin/blob/main/CHANGELOG.md#200---2023-08-31 for upgrade guide 👍

authors: MarcelRoozekrans jdeniau

repo: mapado/prettier-plugin-gherkin


text:
Is leaflet-image compatible with Ionic? undefined is not an object (evaluating 'url.match')

When using leaflet-image with the Ionic platform, I get the error:
TypeError: undefined is not an object (evaluating 'url.match')
In function layersDone, line 58
(Latest github repo version).
Any clues?
   leafletImage(map, function (err, canvas) {
      // now you have canvas
      // example thing to do with that canvas:
      var img = document.createElement('img');
      var dimensions = map.getSize();
      img.width = dimensions.x;
      img.height = dimensions.y;
      img.src = canvas.toDataURL();
      document.getElementById('images').innerHTML = '';
      document.getElementById('images').appendChild(img);
    });

I got it to work if I removed all the geoJSON layers on the map. I'm assuming it is not compatible with geoJSON layers?

Committed a workaround. I can't confirm that it'll work for your usecase without a complete testcase: if it doesn't, please try your hand at fixing the issue.

authors: metalaureate tmcw

repo: mapbox/leaflet-image


text:
nodejs dependencies deprecated.

When getting from nodejs it throws warnings about dependencies being in @mapbox namespace.
While @mapbox/leaflet-omnivore is pointing to ones without @mapbox namespace

did u solve the issue ?

I really don't remember, but I believe I ignored it.

authors: audreyspency msmolka

repo: mapbox/leaflet-omnivore


text:
Location layer crash after commit with support library update (26.1.0)

Hey guys.
After commit Sep 27 (#111) crash occurs on screen rotates:
Fatal signal 11 (SIGSEGV), code 1, fault addr 0x0 in tid 1263
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x0
 r0 00000000  r1 8b07030f  r2 00000002  r3 3fe37715
 r4 9fb99340  r5 9fb99340  r6 bee6b640  r7 bee6b630
 r8 bee6b698  r9 bee6b638  sl bee6b700  fp b4f07800
 ip 9cdf5cc4  sp bee6b628  lr b6fb1387  pc 9cb1f9f4  cpsr 600b0030
backtrace:
 #00 pc 000fd9f4  /data/app/com.reverllc.rever-1/lib/arm/libmapbox-gl.so
 #01 pc 000bd715  /data/app/com.reverllc.rever-1/lib/arm/libmapbox-gl.so
 #02 pc 000bf68b  /data/app/com.reverllc.rever-1/lib/arm/libmapbox-gl.so
 #03 pc 000a2541  /system/lib/libart.so (art_quick_generic_jni_trampoline+32)
  #04 pc 0033b8c4  /dev/ashmem/dalvik-alloc space (deleted)

I clone location layer plugin as module and noticed that crash occurs after adding support (26.1.0) dependency and after removing lifecycle (1.0.0-alpha5) dependencies. Because new support version depend on new lifecycle versions. Problem with new lifecycle versions.
Please solve the problem.

@cammace do you know if this is still an issue? if so, would be great to be included with #148.

Deps have been updated

authors: giperwlad tobrun

repo: mapbox/mapbox-plugins-android


text:
Publish to NPM

The installation instructions for this utility seem odd relative to other Node-based utilities. Typically I'd expect to see something like
npm install -g mts-data-sync

To install a global utility, rather than cloning and linking. That would give the utility the benefit of only installing on tagged releases rather than whatever is in the main branch of the repository. Is there some reason why this repository uses a different technique?

I agree that this should be an NPM package. Modified the title of this issue to reflect that intent.

I agree that this should be an NPM package. Modified the title of this issue to reflect that intent.

For the other API endpoints there is a section in the docs telling you which SDK's have been made for it. For example geocoding. However for the tilesets API docs instead of referencing the generic JavaScript driver it points here to a NodeJs only (not browser) driver that seems a little clunky given you can't just npm install  it. However the tilesets service does seem to be exposed in the generic Javascript SDK and documented here. So unless I'm missing something then most people would probably prefer to use that than this repo.

authors: james-innes jbranigan tmcw

repo: mapbox/mts-data-sync


text:
Refactored news handling

I also touched a few of the other parts (i can remove those changes if you want ^^)
Resolves #358
here are a screenshots of the changed pages:

Please make sure this looks good on mobile devices as well. Thanks!!

Good point.
I have tweaked the design slightly (adding non breaking spaces, reducing a bit of vertical spacing), but nothing major:

Sorry about that bug, that was a last minute fix that I did not test properly apparently 😰
It looks like this now (what do you think?):

I am still not seeing any spaces for some reason.

Oops.
I will look into this in a few minutes when I'm off the train (the non braking space there would be pretty ugly in terms of overflow)

@CommanderStorm I had to add a nbsp and add a - to trim the whitespace in the Go template (otherwise the formatter would break it again).

One last thing, let's remove the 'x minutes read' so it's easier to fit on one line.

unless you still want to make a change

I changed this to not be non-breaking spaces. This way, splits at all the red spaces are possible for the authors part:

Sorry, looking at the final result I think I like the current header better. Do you mind reverting it?
It's easier to skim over the authors (which is not super relevant information) when they are in the current button style.

Shure can revert it (which part of the news section? should I also keep the removed read timer) ^^
There are these alternatives

This would look like this:

I think I prefer it as it was before, including the read timer.

Have mostly reverted the changes (the changes that I kept to the page don't affect the design)

authors: CommanderStorm HarelM louwers


text:
How to give priority for a marker to be visible when two markers appear at same location?

There are two markers located close to each other and they overlap. But one of the marker is important and it needs to be shown all the times.

There is no solution for this right now. There are some solutions on JS-Side (like spider-clustering) but currently this can't be done with this plugin.

Reopening this - it's been 5 years. Is this possible now?

markerOptions.zIndex
https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/v2.6.0/class/Marker/addMarker/README.md

authors: Hirbod giladrom ramsabarish wf9a5m75

repo: mapsplugin/cordova-plugin-googlemaps


text:
Documentation

Maputnik needs better documentation. Over the course of the year there have been a few issues raised and comments in issues with people confused what Maputnik actually does. I don't think it's that clear from the project what it does and what it's aims are.
How about we create a documentation repository (maputnik/docs) and start building some good documentation.
My initial thoughts of what would be good to cover

The goal of Maputnik (an editor for the MapboxGL style spec)
Worked through use cases

Base map
Data vis

An explanation of data sources
Common issues when using Maputnik

So basically it'd be a guide to both using Maputnik and to some extent working with the MapboxGl style spec. Although if you have better ideas feel free to run with them
Would anybody be interested taking on the challenge?
I would prefer someone with prior experience, I'm going to nominate @gregorywolanski @pathmapper @keum. I realise this is quite a lot of work, so there's no pressure intended from me.

This issue appears in the answers to question “What annoys you about Maputnik?” in the survey (original spelling):

The lack of instructions about style properties.

Hard to connect local datasource.

Hard to know what to start.

No toturial from raw data to maputnik to map tile.

Missing docs. about eg. labelling.

Lack of/unclear documentation.

We do a lot of re-engeneering to get the detail bits out of a style.

Some parts of the documentation and video tutorials are outdated and not replicable.

I cant make the labeling work.

lack of documentation to configure it from a developer point of view

Just a note that we may want to document https://github.com/maputnik/editor/issues/301.

heatmap tutorial (#540)

authors: gregorywolanski orangemug pathmapper

repo: maputnik/editor


text:
No way to document parameters

New django-rest-swagger versions have deprecated YAML docstrings. There's no documentation demonstrating how to document request fields anymore.
There's some things you can do by changing the filter_backends on ViewSets, for example, but it just doesn't feel right.
I still haven't found a way to do this on function based views.

It also seems that coreapi is less descriptive than openapi so the openapi schema will be incomplete since we generate from rest_frameworks coreapi document (via https://github.com/core-api/python-openapi-codec). I need the ability to specify response codes and bodies.  IIUC the old way to get around this was with YAML docstrings but those are deprecated.

I still do not know how to document parameters.
Hope there will be an example in the tutorial app.

I write a custom schemagenerator to deal with parameter:
class SchemaGenerator(schemas.SchemaGenerator):
    def get_link(self, path, method, callback, view):
        """Custom the coreapi using the func.__doc__ .

        if __doc__ of the function exsit, use the __doc__ building the coreapi. else use the default serializer.

        __doc__ in yaml format, eg:

        desc: the desc of this api.
        ret: when success invoked, return xxxx
        err: when error occured, return xxxx
        input:
        - name: mobile
          desc: the mobile number
          type: string
          required: true
          location: form
        - name: promotion
          desc: the activity id
          type: int
          required: true
          location: form
        """
        fields = self.get_path_fields(path, method, callback, view)
        yaml_doc = None
        func = getattr(view, view.action) if getattr(view, 'action', None) else None
        if func and func.__doc__:
            try:
                yaml_doc = yaml.load(func.__doc__)
            except:
                yaml_doc = None
        if yaml_doc and 'desc' in yaml_doc:
            desc = yaml_doc.get('desc', '')
            ret = yaml_doc.get('ret', '')
            err = yaml_doc.get('err', '')
            _method_desc = desc + '
' + 'return: ' + ret + '
' + 'error: ' + err
            params = yaml_doc.get('input', [])
            for i in params:
                _name = i.get('name')
                _desc = i.get('desc')
                _required = i.get('required', True)
                _type = i.get('type', 'string')
                _location = i.get('location', 'form')
                field = coreapi.Field(
                    name=_name,
                    location=_location,
                    required=_required,
                    description=_desc,
                    type=_type
                )
                fields.append(field)
        else:
            _method_desc = func.__doc__ if func and func.__doc__ else ''
            fields += self.get_serializer_fields(path, method, callback, view)
        fields += self.get_pagination_fields(path, method, callback, view)
        fields += self.get_filter_fields(path, method, callback, view)

        if fields and any([field.location in ('form', 'body') for field in fields]):
            encoding = self.get_encoding(path, method, callback, view)
        else:
            encoding = None

        if self.url and path.startswith('/'):
            path = path[1:]

        return coreapi.Link(
            url=urlparse.urljoin(self.url, path),
            action=method.lower(),
            encoding=encoding,
            fields=fields,
            description=_method_desc
        )

@daimon99, on list routes, you can set the filter_backends property on the ViewSet to correctly generate the schema, still it feels a little hacky.

just wait rest framework 3.5. tomchristie  said: "Gothca, understand the issue now! Yes, this has been an issue in how we generate the schema for swagger UI. It's resolved as part of the upcoming 3.5 release."

https://github.com/tomchristie/django-rest-framework/issues/4241.

So no updates yet regarding this issue??

Given that the 3.5 release is out, has this ticket been handled?

Maybe mentioning @tomchristie ?

There's some level of support for this, but it's a little patchy.
You'd need to look at the implementation of get_link() to see how the fields are built up. The description attribute on fields can then be pulled out by the swagger UI renderer...

Serializer fields (request body/form) have the help_text attribute pulled into the description.
Filter fields and pagination fields (query params) have whatever get_schema_fields returns (currently won't populate any field descriptions, tho you could override this)
URL path fields have no way of setting a description.

Does CoreAPI even support custom type definition? I know you can specify anything in the field.type.
I think you would need to change rest_framework_swagger.renderers.OpenAPICodec so that the returned Swagger object would be extended with definitions:.
Those definitions could be somehow pushed to coreapi.document.Document with SchemaGenerator maybe.
The coreapi.document.Document seem to be accepting any data.
class Document(itypes.Dict):
    # ...
    self._data = {key: _to_immutable(value) for key, value in content.items()}

@slykar - I think you're mixing up different concerns in that comment. (What datatypes can be included in a Document, vs. how do we describe parameter help text and parameter types.
I'm currently working on a documentation generator to tie in with Core API, which is helping tease out some of the docs related requirements. I'd suggest we hold further discussion until we can use that as a starting point.
Aside: Also worth re-iterating that I want to see any spec changes to Core API be driven by very explicit tooling requirements. Eg. "Adding X to Core API will allow me to do Y".

@tomchristie - To give some background. What I was trying to achieve was a documentation of profile parameter on my endpoint, which is an embedded document with it's own schema. SchemaGenerator specifies the type as 'object', where it should be a type on it's own, like 'UserProfile'.
When using OpenAPI, you can document such field by setting custom type. In OpenAPI this type is defined by definitions section. This is something you can not do with CoreAPI as far as I know, even tough the CoreAPI Field.type takes arbitrary string and you could put 'UserProfile' there.
The spec mismatch makes it hard to properly document endpoint parameters, utilizing OpenAPI type definitions. This is something I require, otherwise the generated OpenAPI spec is worthless for me.
Right now I'm back to writing the OpenAPI spec file by hand.

Gotcha. And yes, we may well move towards allowing for annotation information about more complex types. Out of interest what tooling are you using to document the API - Swagger UI, or something else?

@tomchristie I'm using Swagger UI, tough, when writing spec by hand I write it using API Blueprint (IMO easier to write and read than OpenAPI), then convert it to OpenAPI spec for Swagger UI.

Hey guys any update on this issue?

Yeah upcoming version of REST framework is working towards improving this.

look forward to the new version of REST framework!
for the time being i'm using this class decorator:
https://github.com/ministryofjustice/moj-product-dashboard/blob/develop/dashboard/libs/swagger_tools.py#L5
use case:
https://github.com/ministryofjustice/moj-product-dashboard/blob/develop/dashboard/apps/dashboard/views.py#L106

@cliffxuan Interesting solution.
@tomchristie What are your plans for the next version? Would you consider extending the override methods like get_path_fields and get_filter_fields currently on the schema generator onto APIView?

@cliffxuan  https://github.com/marcgibbons/django-rest-swagger/issues/549#issuecomment-276993383
Your solution works fine for list action. what if we want to customize that at action level, i.e i want a specific query param for retrieve method or update or create method for that matter. How are we going to do that. can we modify your decorator to support at method level?

@gjain0 my decorator works only for list view. i don't have a use case for yours yet. feel free to modify it.

Has this issue been addressed in any way over the past 8 months?  It appears that currently there is no concise readable way to specify parameters for API endpoints, which effectively renders Swagger as useless.

@FilmCoder  I'm using @daimon99 solution. modified for my use case.
It's working perfect.
here is the complete code
https://gist.github.com/gjain0/f536d3988eb61e693ea306305c441bb7

It really is mind boggling that this issue remains open after all this time.

Using help_text on Serializer works great for parameters - but I'm still missing a way to add a description to the Path parameters.
Would be great if that could be implemented!

If all you care about is showing parameter types, is required or optional, and help_text, the "Model" view on the swagger page is an easy alternative, as described here: http://idratherbewriting.com/2015/12/10/ten-realizations-using-swagger-and-swagger-ui/
Pros: Can be implemented with just Serializers and isn't a hack.
Cons: Bad at displaying request data's nested objects and lists.

@bwmello , this is about documenting input parameters and (more importantly) the ability to enter parameter content within swagger. Your page suggests using markdown, but this ability was removed when the software went to coreAPI, without any viable replacement. This thread is not related to documenting the output parameters, which is what your page also describes.

Hi,
Any update on this issue ?

Bizarre that a documentation tool for an API, doesn't- wait for it- allow you to document parameters. Kinda like a car that doesn't drive, or a plane that doesn't fly. Can I have the past hour of googling in disbelief back?
Here's my workaround, param_schema.py --
from rest_framework.compat import coreapi, coreschema
from rest_framework.filters import BaseFilterBackend

class ParamSchemaFilter(BaseFilterBackend):
    def get_schema_fields(self, view):
        assert coreapi is not None, 'coreapi must be installed to use `get_schema_fields()`'
        assert coreschema is not None, 'coreschema must be installed to use `get_schema_fields()`'
        fields = super().get_schema_fields(view)
        if hasattr(view, 'get_param_fields'):
            print(view)
            fields += view.get_param_fields(view)
        return fields

And in your settings --
REST_FRAMEWORK = {
    ...
    'DEFAULT_FILTER_BACKENDS': (
        ...
        'param_schema.ParamSchemaFilter',
    ),

And in your view/viewset --
class MyAwesomeView(viewsets.GenericViewSet):

    def get_param_fields(self, view):
        fields = [
            coreapi.Field(
                name='my_field',
                required=True,
                location='query',
                schema=coreschema.String(
                    title='My awesome field',
                    description='This is my really awesome field right here, so awesome'
                )
            ),
        ]
        return fields

Seems like things are moving forward: https://github.com/encode/django-rest-framework/issues/4502

Hi all, I have recently integrated Django Rest Swagger 2 into an existing project, I faced alot of issues, and resolved them by finding solutions at different forums. Now I have documented all the required steps here: https://github.com/m-haziq/django-rest-swagger-docs  with all possible issues I could face and their solutions.

This is the way to go: https://github.com/axnsan12/drf-yasg

There are currently two decent Swagger schema generators that I could find for django-rest-framework:
django-rest-swagger
drf-openapi
Out of the two, django-rest-swagger is just a wrapper around DRF 3.7 schema generation with an added UI, and thus presents the same problems. drf-openapi is a bit more involved and implements some custom handling for response schemas, but ultimately still falls short in code generation because the responses are plain of lacking support for named schemas.

Is anyone currently working on this?

@gregorypease280  yes I am. Try reading  https://github.com/m-haziq/django-rest-swagger-docs

For people interested by making swagger work with the last version of django rest framework (3.7.7), you'll notice the tweak gave by @daimon99  in this comment https://github.com/marcgibbons/django-rest-swagger/issues/549#issuecomment-250145712 which I used is no longer working.
I've just ported the new code for making it work with django rest framework 3.7.7 and django rest swagger 2.1.1
Create a file schema_view.py:
from django.utils.six.moves.urllib import parse as urlparse
from rest_framework.schemas import AutoSchema
import yaml
import coreapi
from rest_framework_swagger.views import get_swagger_view

class CustomSchema(AutoSchema):
    def get_link(self, path, method, base_url):

        view = self.view
        method_name = getattr(view, 'action', method.lower())
        method_docstring = getattr(view, method_name, None).__doc__
        _method_desc = ''

        fields = self.get_path_fields(path, method)

        try:
            a = method_docstring.split('---')
        except:
            fields += self.get_serializer_fields(path, method)
        else:
            yaml_doc = None
            if method_docstring:
                try:
                    yaml_doc = yaml.load(a[1])
                except:
                    yaml_doc = None

            # Extract schema information from yaml

            if yaml_doc and type(yaml_doc) != str:
                _desc = yaml_doc.get('desc', '')
                _ret = yaml_doc.get('ret', '')
                _err = yaml_doc.get('err', '')
                _method_desc = _desc + '\n
' + 'return: ' + _ret + '
' + 'error: ' + _err
                params = yaml_doc.get('input', [])

                for i in params:
                    _name = i.get('name')
                    _desc = i.get('desc')
                    _required = i.get('required', False)
                    _type = i.get('type', 'string')
                    _location = i.get('location', 'form')
                    field = coreapi.Field(
                        name=_name,
                        location=_location,
                        required=_required,
                        description=_desc,
                        type=_type
                    )
                    fields.append(field)
            else:
                _method_desc = a[0]
                fields += self.get_serializer_fields(path, method)

        fields += self.get_pagination_fields(path, method)
        fields += self.get_filter_fields(path, method)

        manual_fields = self.get_manual_fields(path, method)
        fields = self.update_fields(fields, manual_fields)

        if fields and any([field.location in ('form', 'body') for field in fields]):
            encoding = self.get_encoding(path, method)
        else:
            encoding = None

        if base_url and path.startswith('/'):
            path = path[1:]

        return coreapi.Link(
            url=urlparse.urljoin(base_url, path),
            action=method.lower(),
            encoding=encoding,
            fields=fields,
            description=_method_desc
        )

schema_view = get_swagger_view(title='Product Testing API')

Your urls.py file references it like:
from .schema_view import schema_view

urlpatterns = [
    url(r'^v1/api/', include([
        url(r'^doc/', schema_view),
    ])),

No the sad part is that you have to override all your APIView or ModelViewSet with
schema = CustomSchema()

as described in the 3.7.7 release : http://www.django-rest-framework.org/api-guide/schemas/#per-view-schema-customisation
@m-haziq Hope it will help for making the migration in your README file ;)

There are also problems with adding parameters to function-based views, like this:
def register(request):
    #read parameter "code" and check it
    response_data = {'code': 'success.', 'token': 'sometoken'}
    return Response(response_data, status = status.HTTP_201_CREATED)

how to add "code" from here to swagger I can't find. Unfortunately, don't have enough time to work on it and post an answer, so maybe someone else will post the answer here.

@GuillaumeCisco Great. Thanks for sharing it, I will definitely take some time out of my routine to add this update to my documentation sometimes soon!

@Swich1987 Please read this part of doc. You can document FBVs smoothly: https://github.com/m-haziq/django-rest-swagger-docs#advance-usage

authors: FilmCoder GuillaumeCisco JorisBenschop Swich1987 ameade angvp bwmello cliffxuan daimon99 fazpu fijemax frennkie gjain0 gregorypease280 kevin-brown leonardoarroyo litchfield m-haziq marcgibbons nikparmar psychok7 slykar tomchristie

repo: marcgibbons/django-rest-swagger


text:
🛑 Diner is down

In 7322aed, Diner (https://www.dinersfx.pe/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Diner is back up in 9220941.

repo: marcoadasilvaa/health


text:
Add User property to Chat Completion request

The Chat Completion API supports a User property, as described here: https://platform.openai.com/docs/api-reference/completions/create#completions/create-user.

A unique identifier representing your end-user, which can help OpenAI to monitor and detect abuse.

This property must be added to ChatGptOptions.

What about adding it to the ChatGptParameters class? It should be useful to be able to modify the property based on some condition: for examples in the documentation it says that it would be best to create one per user.
Putting on the ChatGptParameters class will give the developer the option to populate it on every call (maybe checking the authenticated user) rather than one per application

If you need to popolate it on every call, you can use the AddChatGpt overload that provides a ServiceProvider and that is called for every request. With this method, you can change even the API key (assuming, for example, that each authenticated user has a different key).

repo: marcominerva/ChatGptNet


text:
feature request to add ":loading" to md-button

Hi,
It would be great if you could add loading indicator for buttons after submit and wait for end of submit. Something like:
<md-button :loading="isLoading">Submit</md-button

Thanks!

This is such a use case that can be developed by your own logic.

ok fair enough!

authors: dohomi marcosmoura

repo: marcosmoura/vue-material


text:
Added RecurrentTabularExplainer

This addresses issue #56, and adds a class to handle tabular data when the model is a stateless keras-style recurrent neural network. In that case, the model expects data with shape (n_samples, n_timesteps, n_features). This mostly uses the machinery of the existing LimeTabularExplainer, but reshapes data in a few places so that the user doesn't have to worry about reshaping themselves.

Can you share a notebook with an example of a model and explanations, so I can play with it a little bit as well?
Thanks : )

I'll work on this, but it will take a few days to find a suitable example dataset that isn't proprietary. A lot of the LSTM examples out there are for text data, which LIME handles differently.

This is cool. Thank you very much!

I find that there is no regression mode in this recurrent explainer, can you add that? Thanks!

And it seems that categorical_features needs to be updated too, so that it can match the columns of the reshaped data.

I trained a Recurrent Neural Network on MNIST data by transforming it to (n_samples=55000, n_timesteps=28, n_features=28) and then used RecurrentTabularExplainer to generate explanations but it doesn't work very well to explain it. Do you think there might be a problem in fitting the flattened data from RecurrentTabularExplainer on linear model because of the Curse of Dimensionality?

@kgullikson88 @marcotcr can this RNN explainer work with multiple input features, where every input feature is a sequence of equal length?

This explainer will run fine with multiple input features. Keep in mind though, that it is going to fit a linear model with n_features * sequence_length features. As @shakti365 mentions, that dimensionality can blow up pretty easily and so the explanations might not be that great.

@kgullikson88 thanks for explanation! I was able to use your RNN explainer. The prediction is quite slow and explanations does explain very well yet.
I'm getting this warning and it does not make much sense:
https://github.com/marcotcr/lime/blob/master/lime/lime_tabular.py#L290
                    Prediction probabilties do not sum to 1, and
                    thus does not constitute a probability space.
                    Check that you classifier outputs probabilities
                    (Not log probabilities, or actual class predictions

Since you say it doesn't make sense, I assume your predict_proba function does in fact give values that sum to 1? If so, then I'm not sure; maybe @marcotcr would be able to help with that.

@kgullikson88 I have been working on an implementation where we don't flatten out the data to n_features * sequence_length. Instead I am trying to mask features across all time steps very similar to how LimeImageExplainer works. I have tried this on MNIST data and the explanations make lot more sense the current implementation.

@kgullikson88 @marcotcr Would you be willing to review this approach if I submit a PR?

I would take a look, but don't have a lot of time to spend so it might take a while to review.

Same! I'm trying to graduate, so will probably neglect this repo for a couple more months

repo: marcotcr/lime


text:
Support other rename methods, e. g. git mv

I really like this approach for bulk renaming!  It would be nice to be able to use this with git mv.  Currently, I can call pipe-rename and then git add -A, but this only works if there are no unstaged changes.
What do you think about adding a --rename-command option so that I could call ls * | renamer --rename-command "git mv"?

I like it! Cool idea. I can take a crack at this in the next couple days, or feel free to submit a PR

This should be covered by ec1576a3aa6d69d922ef57e35d5145b2b9dd491f .
Ex: ls | renamer --rename-command="git mv" or ls | renamer -c="git mv"

Nice, thank you!  I think this might cause some issues with strange file
names (e. g. starting with -), but I’ll double-check that.

Ah I think you're right, the files should be passed as args probably instead, or escaped in some way

authors: marcusbuffett robinkrahl

repo: marcusbuffett/pipe-rename


text:
In WebViewJavascriptBridge, there is no way to remove a handler

In the WebViewJavascriptBridge, you should be able to remove a registered handler, in particular:
a) To avoid memory leaks
b) To unsubscribe
(I can work on this)

Sure!
I suggest registerHandler returns a value (possibly an opaque id value) which can later be used to remove the registered handler:
    id handlerId = [_bridge registerHandler:@"testObjcCallback" handler:^(id data, WVJBResponseCallback responseCallback) {}];
    ...
    [_bridge removeHandler:handlerId];

@oveddan: I would still accept a PR for this :)

Any updates on this?

Also see #216

authors: kaunteya marcuswestin oveddan

repo: marcuswestin/WebViewJavascriptBridge


text:
Client: Make lock screen

This is just to train actions to server (thus, User and UserAttributes are needed to complete this).

This will require to save encrypted pin on User.

repo: marek-hanzal/DeRivean


text:
ggml_new_tensor_impl: not enough space in the scratch memory pool

I have a real bug this time.
ggml_new_tensor_impl: not enough space in the scratch memory pool (needed 575702016, available 268435456)
Segmentation fault

This affects ggml implementation in llama.cpp so it is likely not a bug with ctransformers but an inherited bug from the ggml code. This appears on the second run, which to me means that there is a memory leak - the buffer is either not correctly sized and/or it is not cleared upon exit

Can you please provide the code or steps to reproduce this error. Also please mention the model and parameters you are using.

Thank you for a quick reply...and again it is not an actual bug. I had mistakenly set the batch size to a very high number (2048) which caused the scratch memory pool error.

authors: marella vmajor

repo: marella/ctransformers


text:
Push new run_saved_model

Since i can't build a valid whl (because of the duplicate symbols i mentioned in the original ticket), i can't generate a model featuring the new output tensor. Could you check in the new model and let me know if that pip build target actually works for you? (and if it doesn't, how you're getting your changes into Python..)
Thanks

You should now be able to build the pip package without linking issues. Again i am using my other repository: https://github.com/marhlder/parsey_mcparseface_export/tree/documents-from-tensor to export my model to a SavedModelBundle.

Builds fine - thanks :- )

authors: marhlder quaeler

repo: marhlder/syntaxnet-ops-hack


text:
Is there has a way to add custom utilities

I face with the pain-point that I can not add a custom utilities. so that I can not benefit by IntelliSense of tailwind.
Is there has a way like below to define my own class?
plugins: [
    plugin(function ({ addUtilities }) {
      addUtilities({
        '.text-base': "text-dark dark:text-white"
      })
    })
  ]

You should be able to add that directly to your tailwind.config.js. You may need to reset your babel cache after doing so https://www.nativewind.dev/guides/troubleshooting

Such syntax text-dark dark:text-white is not working fo in addUtilities. this syntax can only valid in the css file but nativewind can not suport css file😢

I believe the correct syntax is
addComponents({
    '.text-base': {
      "@apply text-dark dark:text-white": {},
    }
  });

Please note that Tailwind CSS discourages this way and recommends simply creating new components https://tailwindcss.com/docs/reusing-styles

@marklawlor wow. my bad. thanks u very much

@marklawlor It's weird that the dark style(dark:text-white) is working but the light style(text-dark)  is not working.

text-dark is not a valid Tailwind class. If this is a custom class you are using please provide a link to your repo

@marklawlor here is my repo link https://github.com/liaoliao666/v2ex
I totally understand what u mean. but I still wanna using @apply to add new utilities for composing className more easier insead of abstract into components.
and the text-dark is a typo. I did use text-black in my project, but it's still not working

even I using the standard css to add utilities but still caused same issue. like below
addUtilities({
        '.border-color': {
          'border-color': '#eff3f4',
          '@media(prefers-color-scheme: dark)': {
            'border-color': '#2f3336',
          },
        },

@liaoliao666 Does your plugin actually work with Tailwind CSS? It looks like it generates invalid CSS. Because of that, NativeWind cannot parse it.
This version should be parsable
addUtilities({
  '.border-color': {
    'border-color': '#eff3f4',
  }
  '@media(prefers-color-scheme: dark)': {
    '.border-color': {
      'border-color': '#2f3336',
    },
  }
},

@marklawlor It's regret that this is also not working for dark mode.

repo: marklawlor/nativewind


text:
User-defined patch replacement functions unusable with patch builder

You can't use a user-defined replacement constructor function through PatchBuilder because the namespace setup by PatchBuilder.library() is wrong. It is using the module path instead of the namespace. Either that or the contract described in the spec is wrong, but I'm gonna go with the impl being incorrect. It does this:
 return {'replace-library':{
    ns: '/ext/marklogic/patch/apply/'+rootname,
    at: '/ext/marklogic/patch/apply/'+module
  }};

Instead of this:
 return {'replace-library':{
    ns: 'http://marklogic/patch/apply/'+rootname,
    at: '/ext/marklogic/patch/apply/'+module
  }};

P.S. There's no unit test for patch UDFs. :)
Example to reproduce the problem follows. Currently, when you run the patch script, you get this cruelly undeserved error:
{ [Error: patch document: cannot process response with 400 status (on /patch/udfs.json)]
  message: 'patch document: cannot process response with 400 status (on /patch/udfs.json)',
  statusCode: 400,
  body:
   { errorResponse:
      { statusCode: 400,
        status: 'Bad Request',
        messageCode: 'XDMP-IMPMODNS',
        message: 'XDMP-IMPMODNS: (err:XQST0059) Import module namespace /ext/marklogic/patch/apply/my-lib does not match target namespace http://marklogic.com/patch/apply/my-lib of imported module /ext/marklogic/patch/apply/my-lib.xqy' } }
}

UDF impl, which might not work as I ran into this bug when I was trying to test it. However, you should at least invoke it without being told you're ugly and your mother dresses you funny. Copy into my-lib.xqy.
xquery version "1.0-ml";

module namespace my-lib =
  "http://marklogic.com/patch/apply/my-lib";

(: Double the value of a node :)
declare function my-lib:dbl(
  $current as node()?,
  $args as item()*
) as node()*
{
  if ($current/data() castable as xs:decimal)
  then
    let $new-value := xs:decimal($current/data()) * 2
    return number-node {$new-value}
  else
    fn:error((), "RESTAPI-EXTNERR",
      ("400", "Bad Request", "xml", <reason>Non-decimal data</reason>))
};

(: Find the minimum value in a sequence of value composed of :)
(: the current node and a set of input values. :)
declare function my-lib:min(
  $current as node()?,
  $args as item()*
) as node()*
{
  let $new-value := fn:min(($current/data(), $args))
  return number-node {$new-value}
};

Installation script to stuff it into the modules db. It expects to find the UDF impl lying about as ./my-lib.xqy.
var fs = require('fs');
var marklogic = require('marklogic');
var my = require('./my-connection.js');
var db = marklogic.createDatabaseClient(my.connInfo);

db.config.extlibs.write({
  path: '/marklogic/patch/apply/my-lib.xqy',
  permissions: [
    {'role-name': 'app-user', capabilities: ['execute']}
  ],
  contentType: 'application/xquery',
  source: fs.createReadStream('./my-lib.xqy')
}).result(function(response) {
  console.log('Installed module ' + response.path);
}, function(error) {
  console.log(JSON.stringify(error, null, 2));
});

Script that inserts a document and attempts to patch it using the installed UDFs:
var marklogic = require('marklogic');
var my = require('./my-connection.js');
var db = marklogic.createDatabaseClient(my.connInfo);

// (1) Insert the base document into the database
db.documents.write({
  uri: '/patch/udfs.json',
  contentType: 'application/json',
  content: {
    inventory: [
      {name: "orange", price: 10},
      {name: "apple", price: 15},
      {name: "pear", price: 20}
  ] }
}).result().then(function(response) {
  // (2) Patch the document
  var pb = marklogic.patchBuilder;
  return db.documents.patch(response.documents[0].uri,
    pb.library('my-lib.xqy'),
    pb.replace('/inventory[name eq "orange"]/price', pb.apply('dbl')),
    pb.replace('/inventory[name eq "pear"]/price',
               pb.apply('min', 18, 21))
  ).result();
}).then(function(response) {
  // (3) Emit the resulting document
  return db.documents.read(response.uri).result();
}).then(function(documents) {
  console.log(JSON.stringify(documents[0].content, null, 2));
}, function(error) { console.log(error); throw error; });

Closing backlog issue, the issue has regression test

authors: ayuwono kcoleman-marklogic

repo: marklogic/node-client-api


text:
Jenkins failure - Tests in probe.js

Two tests in probe.js with title "probe in transaction using param list.should find written doc in same transaction"-  throw 403 exception in Jenkins. The user - rest-writer need xdmp-invoke privilege in order to run these tests.
As a fix, rest-evaluator role (this internally has xdmp-invoke priviledge), needs to be added to the user - rest-writer

No failures seen.


text:
Please describe how to use it

Please describe how to use it

Ok, I will add it today

I added it to readme.

authors: Sluwayu markowanga

repo: markowanga/stweet


text:
Losing changes on save

Hi guys!
I need a help here cause this error may be related to redux-form or something, but I'll describe as clearly as possible.
My issue is: When I hit the "save" button, any field that CHANGED, that's not new, will have the value reset to the previous value.
Just in case I updated AOR to the latest version (1.3.2) and it still happens.
I tried to debug the restClient, but it already receives the values reset.
This issue is really new to me and I have no idea what it might be.
What more information can I offer to figure this out?

Hi Sid, and thanks for your question. As explained in the admin-on-rest contributing guide, the right place to ask a "How To" question, get usage advice, or troubleshoot your own code, is StackOverFlow.
This makes your question easy to find by the core team, and the developer community. Unlike Github, StackOverFlow has great SEO, gamification, voting, and reputation. That's why we chose it, and decided to keep GitHub issues only for bugs and feature requests.
So I'm closing this issue, and inviting you to ask your question at:
http://stackoverflow.com/questions/tagged/admin-on-rest
And once you get a response, please continue to hang out on the admin-on-rest channel in StackOverflow. That way, you can help newcomers and share your expertise!

authors: fzaninotto sidferreira

repo: marmelab/admin-on-rest


text:
Add state property for Edit, Create, Delete Buttons to use it in reactr-router

When I use Edit Button I often want to use not only property 'to' of Link component, but also I want to pass 'state' property with some data.

This is already possible in the next branch (2.0.0). The to prop is passed to React-Router Lin and can be an object.
See https://codesandbox.io/s/pp0o4x40p0. The relevant code parts are:

the CreateCommentButton component inside src/posts.js
the CommentCreate component inside src/comments.js (note how we set the defaultValue prop on SimpleForm)


text:
Panic trying to print balance

System Information

Version:  v0.2.0-alpha-modified
System: windows/amd64
Go:  go1.12.5
Source Revision:  36f7f1924ae315fe7fbd03b95cd5e08037ed514b

What did you do?
After transferring both cash and stock out of a budget, I executed the following command: baronial bal .\budget\queue\
The balance file contains several different 0.000 line items. All seem well-formed.
What did you expect to happen?
It should have printed a zero balance.
What actually happened?
A recovered panic is generated while figuring out the balance string.
Stack Trace

Total:  [redacted]
Balance:  %!v(PANIC=String method: bytes.Buffer: truncation out of range)
Children:
        espp: [redacted]
        payable: USD 0.000

The trouble here seems to be that when the balance function simplifies the multi-term balance, everything gets removed, and then it tries to truncate an empty buffer.

repo: marstr/envelopes


text:
[BUG] Untranslated strings in 3.1.1

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.1
Windows version, edition, and architecture
Windows 11
Describe your issue
There are two untranslated text strings in tab Package bundles
Steps to reproduce the issue
Open tab Package bundles and check screenshot below
UniGetUI Log
NA

Package Managers Logs
NA

Relevant information
"The current bundle..." appears to missing from Tolgee
"Add packages..." appears to be present in Tolgee but its translation is not picked up (yet)
Screenshots and videos

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

O dear that was a quick response. Did I report an issue already reported before?

Quite frankly, I think I will pass on the download offered above. I'm not sure to trust it.
I'll await the next release proper.

@marticliment Are above postings pointing to mediafire downloads trustworthy?
I noticed them too in another issue thread.

PLEASE, DO NOT DOWNLOAD THE FILES LINKED ABOVE OR BELOW
They are likely to be virus, I am not the uploader of them

Sorry for that. I will investigate your issue

The string should now be available on Tolgee

repo: marticliment/UniGetUI


text:
Added in New Ghosts, added in a way to pull out generic info !ghostInfo take 2

All ghosts have been updated to the new evidence types
Pulled the possible ghost into a single function to make it a bit more reusable
Added a new text file for PossibleGhosts for anyone that wants to overlay that part.
I think the wiki was off, contains a fix for a ghost that didn't align wit the game.

Looks good. Your last commit reverts your second commit (the uptick in script version) though. Can you fix that? Optionally also feel free to update the readme's.

Oh, you are right. My bad. I'm unused to having to manually adjust my
version, ahaha. I will do another pull request with Verizon uptick and
readme
On Mon, Aug 30, 2021, 6:05 AM Martijn Stolk @.***>
wrote:

Looks good. Your last commit reverts your second commit (the uptick in
script version) though. Can you fix that? Optionally also feel free to
update the readme's.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/martijns/PhasmophobiaSolver/pull/2#issuecomment-908213978,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADDHSBC7RBZCHE4SUWO64MLT7NJVRANCNFSM5DAMBE2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Fixed the version merge, added to the readme.

authors: GreatisNate martijns

repo: martijns/PhasmophobiaSolver


text:
Vypisování souboru

Chtělo by prohodit vypisování souborů, tak aby nevadil dlouhý název.

@martin-nohava mohl by jsi to prohodit až budeš něco commitovat?.

repo: martin-nohava/kryzbu


text:
Managing case when several row groups per file for 'remove_row_groups()'

Hi @martindurant
I would like to make following proposal to manage the critical flaw you identified.
Here is the general logic:

if file written by fastparquet and file scheme different than flat, proceed 'normally' (no check)
if file with file scheme flat, this corresponds to the case you highlight: several files written with file scheme = simple, and then 'merged' together in a single ParquetFile object. In this case, check if the row group removal will lead to removing files that contain both row group to be kept and row group to be removed
if file not written by fastparquet, same as situation 2 above (when file scheme is flat)

In situations 2 and 3, a map of all row group is created. It is a dict in which file names are the keys, and values are the list of row group each file contain.
A similar map for row groups to be removed is also created, and a comparison of row group lists is done file per file (for file to be removed.
It seems to me that this manages this critical flaw.
I have added 2 test cases for testing this:

with data written by pyarrow (through pandas)
with file scheme is flat

For each case, files contain 2 row groups.
I am testing that an error is correctly raised if removing a single group, or that the removal proceeds when removing 2 row groups in the same file.
Please, there is one thing in the code you implemented with default_remove.
There is an except IOError both in  default_remove and in remove_row_groups.
There is redundancy here. Shouldn't one be removed?
Also, there should be no more error now with new code, because file removal is done once per file, on the opposite to previous code, when file was trying to be removed as many times as there are row groups to be removed within.
Bests

Thanks @yohplala

authors: martindurant yohplala

repo: martindurant/fastparquet


text:
Get the default languages in getStaticProps

How can I get the default languages in getStaticProps with next-export-i18n like this:
export const getStaticProps: GetStaticProps = async context => {
  console.log('context: ', context)
  if (locale === 'de') {
     return {
      notFound: true
    }
  }

  return {
    props: {}
  }
}

Currently the context return undefined. My problem is that when a page has a default lang is de then it should be redirect to 404 page
Please help me to help this issue. Thank you.

Hi @NguyenChiTrung1310,
I don't exactly understand your issue here. You set the default language by yourself, it's a fixed value. If you want it to be dynamic, you can read it from the configuration file. Anyway, getStaticProps is run only once on server side and at build time and the use case for the library is to provide i18n on exported pages on the client side.
I assume you are looking for the current language instead of the default language and redirect to 404 for pages that are not available in a specific language? In this case, I would suggest to check for the query parameter on the url and redirect accordingly in the body of your page.
Let me know if you have additional questions.
Cheers


text:
Set current_bandwidth to gauge type

Bandwidth estimate for an interface may decrease or increase

Good catch, thanks!


text:
Request handlers not disposed when using transient lifetime

builder.Services.AddMediator(options => options.ServiceLifetime = ServiceLifetime.Transient);

If you implement IDisposable or IAsyncDisposable, the handler is not disposed.
Switching back to MediatR and it works correctly.

Thanks for reporting, will fix this and publish as part of 3.0 preview release. Will comment here when that is done

repo: martinothamar/Mediator


text:
Update circe-core, circe-generic, ... to 0.14.8

About this PR
📦 Updates

io.circe:circe-core
io.circe:circe-generic
io.circe:circe-parser

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

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

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.circe" }
}]

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

Superseded by #166.

repo: martinpallmann/gchat


text:
🛑 Nate Meidinger Counselling is down

In e72739b, Nate Meidinger Counselling (https://natemeidingercounselling.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nate Meidinger Counselling is back up in 1fb04ad.

repo: martypenner/upptime


text:
Preview is convoluted when page contains several bullets

The above will generate following:

I also tried your @Zwierlein previous implementation in another git, it doesn't have this problem, but the rendering is not responsive compared to this one.

Hi, just seeing this. Is it ok now?

authors: jiewangjs marzwipan

repo: marzwipan/TexFlowyTM


text:
"list elements not inline" warning

Encountered this when I'm trying to build a more compact formatter
Here is a simple way to reproduce the warning
formatter = Formatter(max_compact_list_complexity=999)
data = [
    {f'key{i}':f'value{i}' for i in range(2)},
    {f'key{i}':f'value{i}' for i in range(5)},
    {f'key{i}':f'value{i}' for i in range(2)},
    {f'key{i}':f'value{i}' for i in range(2)},
]
print(formatter.serialize(data))

with unlimited complexity it tries to format the dict as multi-line compact, and as there is a very long object that is not inline, the warning is triggered.
[
    {"key0": "value0", "key1": "value1"},
    {
        "key0": "value0",
        "key1": "value1",
        "key2": "value2",
        "key3": "value3",
        "key4": "value4"
    },
    {"key0": "value0", "key1": "value1"},
    {"key0": "value0", "key1": "value1"}
]
C:\Users\ZeroRin\Documents\Projects\compact-json\src\compact_json\formatter.py:414: RuntimeWarning: list elements not inline (please report an issue)

C:\Users\ZeroRin\Documents\Projects\compact-json\src\compact_json\formatter.py:462: RuntimeWarning: list elements not inline (please report an issue)
  self.indent(buffer, item.depth + 1)

I'd suggest allowing list tobe multiline compact even with multiline elements in it, so that list with many small element but a few large element would not take much space. Here is an example output with the same setup as above:
[
    {"key0": "value0", "key1": "value1"},
    {
        "key0": "value0",
        "key1": "value1",
        "key2": "value2",
        "key3": "value3",
        "key4": "value4"
    },
    {"key0": "value0", "key1": "value1"}, {"key0": "value0", "key1": "value1"}
]

This could be done by modifing the newline condition so that any multiline elements starts from a newline and ends with a newline:
            if (
                line_length_so_far + segment_length > self.max_inline_length
                and line_length_so_far > 0
                or child_index != 0
                and (
                    item.children[child_index].format != Format.INLINE
                    or item.children[child_index - 1].format != Format.INLINE
                )
            ):

I'm trying to make that dict in the middle also multiline-compact  so that hopefully the final version would look better than the current showcase

Also fixed in the PR

authors: ZeroRin masaccio

repo: masaccio/compact-json