Revamp lima website using docsy

Fixes #1499
Creating as draft to keep track of the progress

[X] Home page
[x] Documentation - Installation
[ ] Documentation - Getting Started
[ ] Documentation - Templates
[ ] Documentation - Architecture and other informations present under docs folder
[x] Documentation - Cli reference
[ ] Contribution
[ ] FAQ

@AkihiroSuda
In recent changes i automated generation of markdown for docsy using cobra api. This way the cli-references will always be upto date.
Hope this is fine. Let me know if you some feedbacks.
As next steps will update examples & contribution guidelines.

In recent changes i automated generation of markdown for docsy using cobra api. This way the cli-references will always be upto date.

Why do this in this PR, and why in the "gen-man" command ? There was already a "gen-doc" on the way...

Why do that in this PR

Will put down a separate PR if this approach is fine.

and why in the "gen-man" command

Ah yes my bad, gen-man should not be touched for this. Maybe will add a gen-doc like you have done before. But the markdown generated looks somewhat acceptable only.

This PR is ready. I have mostly covered all sections and also fixed relative links in a dynamic fashion to avoid too many duplicates.
Feel free to put down ideas/comments on contents as well as look and feel.
We can review and merge this after https://github.com/lima-vm/lima/pull/1778 is merged

Please update website/CONTRIBUTING.md and similar files

Please update website/CONTRIBUTING.md and similar files

I have removed those files from website folder for now. I will raise a separate PR for contributing to website.

@lima-vm/maintainers
I'm merging this, but please do review this and open follow-up issues and PRs.

What is with the Title Case ?
$ ls website/content/en/docs/
Examples  faq  _index.md  Installation  Reference  Templates  Usage

Probably a Mac thing.


Make 'add all' button announce changes to screen reader

Summary of the pull request
This makes the "Add all" hyperlink button in the "machine configuration" > "end-to-end setup" page make an announcement via the screen reader when a screen reader is attached.
References and relevant issues
Closes https://dev.azure.com/microsoft/OS/_workitems/edit/45058508
Detailed description of the pull request / Additional comments
Most of the interesting bits are in PackageCatalogView.xaml.cs. We formally create an automation peer for the view, then we leverage that automation peer to push an announcement via UI Automation.

Closing this since the internal issue is now closed.


WSGI Content-Length and Transfer-Encoding

this is a WIP for #74

[x] Use chunked when Content-Length is absent
[x] Use Content-Length and try_end on happy path
[x] Close connection if Content-Length is too small
[ ] add on_writable and on_aborted to send more chunks that failed to send with try_end

PS. This will require new binaries

@ckcollab thanks to get the type, I think I just typed too fast at this moment lol


Help

Zhuowei my blocklancher pro and blocklancher just quite work or crashed and this happens 2 days ago after I got the gun mod fan u please fix this.        Thanks

Did you update MC to 0.15¿

If you did pls close ur issue and w8 for BL update!

i updated it to 0.14.4 but blocklancher every time i tap on it it will pull
up a blank screen for 5 seconds and go home its been doing this since i got
demosgun mod can u help me i realy want to play it again.
On Thu, Jun 16, 2016 at 11:01 AM, PatrikXop<EMAIL_ADDRESS>wrote:

If you did pls close ur issue and w8 for BL update!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226531279,
or mute the thread
https://github.com/notifications/unsubscribe/ATDXvBDAGD79Zvuh-gDO3S2oZV6vEGrUks5qMXM8gaJpZM4I3hKT
.

If you updated to 0.15:
It has not been updated for 0.15. It takes time to re-hook into all the functions. It was stated on Twitter that the author is going to TRY to get it ported for 0.15 by the end of the week. Please close this issue as it is wasting the author's time. Instead of porting the program he is spending his time closing issue like this which are just fodder. Porting the program is going to happen, so again, please close this so you do not waste the author's time.

i think i have 0.7 i think
On Thu, Jun 16, 2016 at 8:05 PM, Leigh The Ugly Neko Freak <
<EMAIL_ADDRESS>wrote:

If you updated to 0.15:
It has not been updated for 0.15. It takes time to re-hook into all the
functions. It was stated on Twitter that the author is going to TRY to get
it ported for 0.15 by the end of the week. Please close this issue as it is
wasting the author's time. Instead of porting the program he is spending
his time closing issue like this which are just fodder. Porting the program
is going to happen, so again, please close this so you do not waste the
author's time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226656068,
or mute the thread
https://github.com/notifications/unsubscribe/ATDXvHC4VxTQP0aoJwmpBCEK0wlTBbzVks5qMfLggaJpZM4I3hKT
.

You can check by going on the store you bought it from and looking go see
if there's an update or look at the bottom left of the screen when opening
Minecraft
On Jun 16, 2016 10:27 PM, "wyattthomas51"<EMAIL_ADDRESS>wrote:

i think i have 0.7 i think
On Thu, Jun 16, 2016 at 8:05 PM, Leigh The Ugly Neko Freak <
<EMAIL_ADDRESS>wrote:

If you updated to 0.15:
It has not been updated for 0.15. It takes time to re-hook into all the
functions. It was stated on Twitter that the author is going to TRY to
get
it ported for 0.15 by the end of the week. Please close this issue as it
is
wasting the author's time. Instead of porting the program he is spending
his time closing issue like this which are just fodder. Porting the
program
is going to happen, so again, please close this so you do not waste the
author's time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226656068
,
or mute the thread
<
https://github.com/notifications/unsubscribe/ATDXvHC4VxTQP0aoJwmpBCEK0wlTBbzVks5qMfLggaJpZM4I3hKT
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226665415,
or mute the thread
https://github.com/notifications/unsubscribe/AS_38fN2Q8GBs1ssPyWKM2E53uhAYepPks5qMgYqgaJpZM4I3hKT
.

I'm not sure where you are getting these numbers from...lol
there was no 0.14.4, there was 0.14.3.  I'm not sure what 0.7 is?  Regardless, as was pointed out, go into the play store and you will see if there are updates as well as what compatibility there is.
Please close this issue if you are using 0.15, as BL has NOT been updated for this yet.  You will end up wasting the author's time and we would much rather he spend the time coding block launcher instead of working on this.  As was copy/pasted from my other response (listed above by @LTUNF [thanks!]) the author stated he is going to TRY to get this updated by the END of this week.  Please have patience as it takes TIME to re-hook into all the functions to make Block Launcher work.

then ya 0.14.3 anyway i got it on silk on a amozon kindle fire HD.
On Fri, Jun 17, 2016 at 1:12 PM, Daniel Rozwood<EMAIL_ADDRESS>wrote:

I'm not sure where you are getting these numbers from...lol
there was no 0.14.4, there was 0.14.3. I'm not sure what 0.7 is?
Regardless, as was pointed out, go into the play store and you will see if
there are updates as well as what compatibility there is.
Please close this issue if you are using 0.15, as BL has NOT been updated
for this yet. You will end up wasting the author's time and we would much
rather he spend the time coding block launcher instead of working on this.
As was copy/pasted from my other response (listed above by @LTUNF
https://github.com/LTUNF [thanks!]) the author stated he is going to
TRY to get this updated by the END of this week. Please have patience as it
takes TIME to re-hook into all the functions to make Block Launcher work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226841779,
or mute the thread
https://github.com/notifications/unsubscribe/ATDXvPkaIgxllqXpCKoDxEMVF85UVXT5ks5qMuOngaJpZM4I3hKT
.

What version of Block Launcher?

it wont pull up so i do know.sorryit will just sow a blank screen for about
5 seconds a crash
On Fri, Jun 17, 2016 at 1:39 PM, Daniel Rozwood<EMAIL_ADDRESS>wrote:

What version of Block Launcher?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226848520,
or mute the thread
https://github.com/notifications/unsubscribe/ATDXvITf8tVqREJ26ftVQd6Map3J_LVWks5qMunVgaJpZM4I3hKT
.

Go into the play store.  Select the BlockLauncher Application.  If you do not see "Update" then you are on the most current version, which is NOT for the released 0.15 Minecraft.  Under the "READ MORE" you will see the version number of the most current submission.
You can also UNINSTALL Minecraft and open BlockLauncher.  That could tell you what version you are on.  I am not positive about that.  Everything you are describing leads me to believe that you are using a mismatched version of BlockLauncher and Minecraft.
For 0.14.1 you need BlockLauncher 1.12.5
For 0.14.2 you need BlockLauncher 1.12.7 or 1.12.8
For 0.14.3 you need BlockLauncher 1.12.8 (the version in the store)
Aside from that, there are NO other versions out there to be used with anything unless you are a beta tester.  Especially if you are on 0.15.

I just noticed you are using Amazon...I do not know where to go in there, however you should be able to use the Amazon STORE to get the version you are on.  Worst case, we can GUESS you are on 0.14.3's version if you go into the Amazon store and do NOT see an update notification for BlockLauncher.

im using SILK on a AMOZON KINDLE FIRE HD  and i just looked at my minecraft
pe and it says i got v0.15.0
On Fri, Jun 17, 2016 at 3:58 PM, Daniel Rozwood<EMAIL_ADDRESS>wrote:

I just noticed you are using Amazon...I do not know where to go in there,
however you should be able to use the Amazon STORE to get the version you
are on. Worst case, we can GUESS you are on 0.14.3's version if you go into
the Amazon store and do NOT see an update notification for BlockLauncher.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226880018,
or mute the thread
https://github.com/notifications/unsubscribe/ATDXvIs7iAweiWYJKAccS--_w1v6D1V-ks5qMwqSgaJpZM4I3hKT
.

Please close this issue since you are using 0.15, it is not compatible at this time.

how long will it take to be fixed
On Fri, Jun 17, 2016 at 4:29 PM, Daniel Rozwood<EMAIL_ADDRESS>wrote:

Please close this issue since you are using 0.15, it is not compatible at
this time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zhuowei/MCPELauncher/issues/1101#issuecomment-226886054,
or mute the thread
https://github.com/notifications/unsubscribe/ATDXvJ6BeVs_Z_cnSKUIMt7I-ZoF4uYKks5qMxGngaJpZM4I3hKT
.

Whatever time it takes @zhuowei to update it; have patience... He can't just magically update it instantaneously.


[Bug] hot reload calls deeplink again

Describe the bug
A clear and concise description of what the bug is.
final appLink = await _appLinks.getInitialAppLink();

_linkSubscription = _appLinks.uriLinkStream.listen((uri) {
  myMethod(context, url: appLink ?? uri);
});

when I did hot reload it calls deep link again!!!
Does it related to
[ ] App Links (Android)
[ ] Deep Links (Android)
[o] Universal Links (iOS)
[ ] or Custom URL schemes? (iOS)
Does the example project work?
[o] Yes
[ ] No
[ ] Irrelevant here
Did you fully read the instructions for the targeted platform before submitting this issue?
Uploaded your files to webserver, HTTPS, direct connection, scheme pattern setup, ...
[ ] Yes
[o] No
[ ] Irrelevant here

yep, indeed.
Even worse:

I start the app without deep link (no initial link)
receive deepLink via stream (no initial link)
do a hot restart
receive initial link (which is not initial, but the last one received from stream)

Need to move away initialLink & latestLink to dart code.
A thing to note here, we can't listen to hot restarts because the app is destroyed and re-created.

I digged into the code tonight and there's nothing I can do for this.
Plugins are not aware of hot reloads or restarts and I must keep initial link on native side to ensure that it will not be skipped at launch from cold state.
Notice that this is not a specific issue on iOS platform, the same happens on Android too.
Anyway, this can occur only on debug sessions.

I also have the same issue. Is there any fix?
above comment, @llfbandit mentioned a solution but it is not working anymore.

Need to move away initialLink & latestLink to dart code.
let me know if any other fixes.

can you provide a reset option to reset the URI?

I am also getting the same issue still. Is there any fix or workaround for this bug? The app never stops executing getInitialAppLink() when doing hot restarts.

Hello all,
I am having the same issue but not only with hot restarts. My issue is that I navigate through the app and then I navigate back to the statefull widget that does the call. Then the getInitialAppLinkis excecuted again and I end up navigating to the same loop.

I had this issue, the stream was emitting the same link over and over again. I came up with this workaround:
_linkSubscription = _appLinks.uriLinkStream.distinct().listen((uri) {
  myMethod(context, url: appLink ?? uri);
});

Using distinct() will not trigger the listener unless the link is a new one.


Unpublished products are still shown in offers

Issue Summary
Offers show non-public products
Steps to Reproduce

Create an Offer
Unpublish a product which is part of the offer (set product.is_public = False)
The product is still shown in Offer List and Detail

Technical details

Python version: 3.9.2
Django version: 3.2.16
Oscar version: 3.2.0

#4053 fixes this issue

#4053 merged


upgrade fuse to fuser for Mac support

Upgrade to support Mac.
fuse package returns error even osxfuse has installed for Mac.
Package osxfuse was not found in the pkg-config search path.
Perhaps you should add the directory containing `osxfuse.pc'
to the PKG_CONFIG_PATH environment variable
No package 'osxfuse' found
Package osxfuse was not found in the pkg-config search path.
Perhaps you should add the directory containing `osxfuse.pc'
to the PKG_CONFIG_PATH environment variable
No package 'osxfuse' found
', /Users/jwel/.cargo/registry/src/github.com-1ecc6299db9ec823/fuse-0.3.1/build.rs:10:76
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

However, it is works well for fuser which is fork of the fuse.

Could you rebase and push again?  I believe I fixed the CI.

This still needs rebasing and please get rid of the merge commits.  I'm not too sure what this is doing so it might make sense to send individual PRs.

#68 follows this. CI will fail if divide the PR as fuse-rs v0.3 use Timespec but fuse-rs v0.4 and fuser does not. I described the change in the commit instead. Feel free to ask me if some change is strange.


Revisit the events model

@PinpointTownes this PR was from your branch + renames, I've only done Cookies for now, but that should demonstrate the idea, I'll take a look at OpenIdConnect as well now to make sure it still is consistent there.

@HaoK,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

There's some friction with the new HandlerContext and the intermediate events like CreatingTicket

So after doing a few of the conversions, it looks to me like it might be helpful to add a interface around each of our handlers that simply types the Options appropriately so we can constrain the events and access them... basically:
    public interface IOAuthHandler : IAuthenticationHandler
    {
        OAuthOptions Options { get; }
    }

That said in 2.1, we'd like follow pattern and add IXyzHandler when we make all of our handlers use this pattern.

Ok @PinpointTownes I'm done working on this PR for this week, I'll get notifications from github so I can answer any questions/help with any issues as you discover them.
Twitter/JwtBearer/OIDC/tests/samples are the things that need to be updated next.

That said in 2.1, we'd likely follow this pattern and add IXyzHandler when we make all of our handlers use this pattern.

Just to expose the options? Sounds like a very weird pattern to me :fearful:

@HaoK so concretely, the only advantage we have when exposing the handler in the context classes is that we get an access to the logger? (we could definitely add a property in BaseContext if you think it's worth it).
Not sure this tiny advantage outweighs the very annoying cons.

Well what actually drove the handler side of things was mostly due to making the base events generic, if you are going to make them generic, they should be generic on THandler not TOptions.

I mean today we have a bunch of base context's that effectively are doing the same thing that IXyzHandler would be doing, BaseXyzContext

Well what actually drove the handler side of things was mostly due to making the base contexts generic, if you are going to make them generic, they should be generic on THandler not TOptions.

Could you please explain what's wrong with having a TOptions generic parameter?

Could you please explain what's wrong with having a TOptions generic parameter?

There's nothing intrinsically wrong with the TOptions generic, but the combination of HttpContext + Options + Scheme matches an IAuthenticationHandler exactly now.  Something just seems off still with the events...
For example we still have the gross
        protected new OAuthEvents Events
        {
            get { return (OAuthEvents)base.Events; }
            set { base.Events = value; }
        }

So lets set the HandlerContext aside for now, keep your generic Options and leave it as BaseContext
@PinpointTownes forget my changes for now, and lets just apply the less controversial naming changes first, I'll noodle around with events a bit...
BaseContext [Instead of HandlerContext]
ResultContext : BaseContext
HandleRequestContext : BaseContext
RemoteResultContext : HandleRequestContext
And I think you can try just adding a virtual AuthenticationProperties to BaseContext so you can use it for Challenge/SignOut, and then just override it to return the Properties in the Ticket

So I noodled around with the events again, and I guess we can just fix one of the things that has always bothered me a different way, even if we can't push the Handler into the base context now, we can at least clean up all of the Context constructors to take their respective handlers and use the properties (make them public if needed), i.e.
var context = new OAuthCreatingTicketContext(ticket, Context, Scheme, Options, Backchannel, tokens);
becomes:
var context = new OAuthCreatingTicketContext(this, ticket, tokens); and have the ctor extract the properties.
This way it'll be easier to expose the handlers in the future since the context code will already be factored for that.


Why version control all semantic source using gulp install?

Your recepies suggest just doing npm install semantic-ui but then all the semantic-ui source code is added my project folder
(also see https://github.com/Semantic-Org/example-github )
I do not wish to mix both sources, and there's no need for me to version control your source files, but - what I do need is to version control the site .variables and .overrides which i add.
so what I did is added everything in /semantic to .gitignore except the my config folder /site.
but then when another teammate npm install semantic-ui it will override the /site folder....
I don't understand whats the strategy on theming and version control,
thanks!!

I thought about using git submodules, but still when i want to update semantic ui it will override my /site folder with custom code.
what do you suggest?

Semantic UI includes install scripts that correctly manage updates without overriding theme changes.
There is no simple way for theming to work with theme files inside your project and definitions inside node_modules.

this should be noted in the getting started page.


Make python bot internals less complex

Part of the reason that we switched over to a custom python implementation was that we wanted to react quickly to Slack API changes that break things as the official APIs are slow to react.
I'm worried that our current implementation of the wrapper uses too much python magic that is very beginner unfriendly.
In addition, as we move towards rate limiting, we are going to need to have every API call asynchronous (since it is processed on a queue, needs to be handled like promises) so the rethink of the internals should be addressed at the same time.
I propose that:

We remove the APIMethodProxy

Instead we just have the API class which has a bunch of async methods, this means we don't need bot.run_async either.
There may be duplicated code between them, but I'd rather that than
There is no harm in using strings for the endpoints rather than hacking around with __getattr__

Remove the underscored_getters

They break type hinting in PyCharm
Not really sure what they do either/why we can't just use the properties themselves

Extract out UQCSBot stuff into more classes

It does way too much and it's really complex, we should reduce the amount of things it does

The architecture of the internals is documented in a wiki page

All design decisions of the architecture should be justified
Most decisions have been made because of limitations of the Slack API, so someone new to the project won't know about those

We actually don't need to have every API call async FYI, can just implement them using normal blocking primitives

I'd definitely like to discuss this more at some point. #282 is somewhat relevant
Realistically, I wouldn't be against the idea of removing the majority of our asyncio in the interest of beginner friendliness; which might make ideas like #286 more important (for user feedback during blocked calls)


Fix UI 1208

[ ] Border radius to 0

preview

[ ] Change text to Link address to create proposal.
[ ] Optimize Layout

preview

[ ] Display balance when active proxy vote

preview


Running an External batch (.bat) file

Hi,
I want to run an external .bat file by clicking on a UI Button in Unity 3D. Can I do this using HoloToolkit. If so, how?
Thank you

I think this is more a general Unity question.

I'm not even sure a bat file will run on HoloLens

I have followed an answer to this question on Unity Answers but I always end up getting an error reading "The type or namespace name "Process" could not be found. Are you missing a using directive or an assembly reference?". Nothing I do fixed this problem. So, I was trying to do it using HoloToolkit instead.

Take a look at this class I wrote. Does something similar, but again, I'm not even sure it will run on the device.

Windows Store applications are not allowed to start new processes.

Direct reply from Unity

Hi,
I never realized that. Thanks for the early reply.
Regards


⚠ ClamAV has degraded performance

In e42f637, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 214 ms

Resolved: ClamAV performance has improved in 54b72df.


[Feature]: Binding keys to specific devices

It would be cool if each device could have binds specifically for that device.
In the config file it would look something like this:
device:some-device {
    bind=SUPER,w,exec,notify-send "pressed w with <some-device>"
}

Another solution, would be to be able to interact with key presses in a script. maybe hyprland can write keypresses (with the device that caused it) to the event socket, or a special one specifically for key presses.

Another solution, would be to be able to interact with key presses in a script. maybe hyprland can write keypresses (with the device that caused it) to the event socket, or a special one specifically for key presses.

no, cuz keyloggers.

I am also very interested in the addition of device-specific binds.
This would allow me to forgo input-remapper completely, and make my Razer Naga side buttons actually functional by default.

@Euro20179 Upon some digging around, I think its already implemented, see https://wiki.hyprland.org/Configuring/Keywords/#per-device-input-configs

I have another use case, unless there is a solution for that:
I have two keyboards: one that only does qwerty, so I set it's keymap to colemak in software (hyprland here), and one that outputs colemak, so I configure a us or eurkey key for this device.
Now, when I try SUPER+d from one keyboard, it open wofi, but on the colemak one I end up having to use SUPER+g which ends up being a key at a different position than the D on colemak.
Is there any way to make that work?

Would also love to see this functionality natively in hyprland. If the devs aren't interested or as a stop gap the mention of tools such as input-remaper on the wiki would be greatly appreciated.

If the devs aren't interested

hah, devs, lol, I wish there were multiple of me.
It's not "not interested", it's "not a priority and I have 294 other open issues"
If you feel the wiki could use a mention, make a pr to it.
If you feel hl could use a feature, make a pr to it.


Implement forecast-hourly

(forecast) invoked by (forecast-hourly) overwrites:
(setf forecast--hourly-mode nil)
Even after fixing this, I do not get per hour weather forecast, just descriptive info:
9°C - Mostly Cloudy
Feels like 6, Light rain overnight.

------------------------<
Pressure 0.993 atm; Humidity 81.0%
Wind 5.04 m/s, from SW

Is it what forecast-hourly should look like?
regards

Whoops sorry, that's not yet implemented.  The function forecast--insert-hourly-forecast which is supposed to set up the view has to be implemented.  I'll sort it out when I have time.  My bad, I should have not committed it to master  (P.S. pull request welcome if you wish to, I'm a bit short of time nowadays, because university).

I have implemented forecast-hourly as a 24-hour dot graphic that appears above the upcoming forecasts, as part of v0.4.0.  @Mafeuser I hope it will be useful to you, sorry for the delay.


Synthesis failed for java-core

Hello! Autosynth couldn't regenerate java-core. :broken_heart:
Please investigate and fix this issue within 5 business days.  While it remains broken,
this library cannot be updated with changes to the java-core API, and the library grows
stale.
See https://github.com/googleapis/synthtool/blob/master/autosynth/TroubleShooting.md
for trouble shooting tips.
Here's the output from running synth.py:
t checkout 9f26800e9c531ba0b4f9f9bde0786f72ef14c0df
Note: switching to '9f26800e9c531ba0b4f9f9bde0786f72ef14c0df'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 9f26800 chore: release 1.95.5-SNAPSHOT (#481)
2021-07-01 01:34:17,639 autosynth [DEBUG] > Running: git checkout 87254ac89a9559864c0a245d6b150406439ce3d8
Previous HEAD position was 333fd90 chore: update precommit hook pre-commit/pre-commit-hooks to v4 (#1083)
HEAD is now at 87254ac fix: Add `shopt -s nullglob` to dependencies script (#1130)
2021-07-01 01:34:17,648 autosynth [DEBUG] > Running: git branch -f autosynth-26
2021-07-01 01:34:17,651 autosynth [DEBUG] > Running: git checkout autosynth-26
Switched to branch 'autosynth-26'
2021-07-01 01:34:17,656 autosynth [INFO] > Running synthtool
2021-07-01 01:34:17,656 autosynth [INFO] > ['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2021-07-01 01:34:17,656 autosynth [DEBUG] > log_file_path: /tmpfs/src/logs/java-core/26/sponge_log.log
2021-07-01 01:34:17,658 autosynth [DEBUG] > Running: /tmpfs/src/github/synthtool/env/bin/python3 -m synthtool --metadata synth.metadata synth.py --
2021-07-01 01:34:17,893 synthtool [DEBUG] > Executing /home/kbuilder/.cache/synthtool/java-core/synth.py.
On branch autosynth-26
nothing to commit, working tree clean
2021-07-01 01:34:18,156 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
.github/CODEOWNERS
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/blunderbuss.yml
.github/generated-files-bot.yml
.github/readme/synth.py
.github/release-please.yml
.github/snippet-bot.yml
.github/sync-repo-settings.yaml
.github/trusted-contribution.yml
.github/workflows/approve-readme.yaml
.github/workflows/auto-release.yaml
.github/workflows/ci.yaml
.github/workflows/samples.yaml
.kokoro/build.bat
.kokoro/build.sh
.kokoro/coerce_logs.sh
.kokoro/common.cfg
.kokoro/common.sh
.kokoro/continuous/common.cfg
.kokoro/continuous/java8.cfg
.kokoro/continuous/readme.cfg
.kokoro/dependencies.sh
.kokoro/nightly/common.cfg
.kokoro/nightly/integration.cfg
.kokoro/nightly/java11.cfg
.kokoro/nightly/java7.cfg
.kokoro/nightly/java8-osx.cfg
.kokoro/nightly/java8-win.cfg
.kokoro/nightly/java8.cfg
.kokoro/nightly/samples.cfg
.kokoro/populate-secrets.sh
.kokoro/presubmit/clirr.cfg
.kokoro/presubmit/common.cfg
.kokoro/presubmit/dependencies.cfg
.kokoro/presubmit/integration.cfg
.kokoro/presubmit/java11.cfg
.kokoro/presubmit/java7.cfg
.kokoro/presubmit/java8-osx.cfg
.kokoro/presubmit/java8-win.cfg
.kokoro/presubmit/java8.cfg
.kokoro/presubmit/linkage-monitor.cfg
.kokoro/presubmit/lint.cfg
.kokoro/presubmit/samples.cfg
.kokoro/readme.sh
.kokoro/release/bump_snapshot.cfg
.kokoro/release/common.cfg
.kokoro/release/common.sh
.kokoro/release/drop.cfg
.kokoro/release/drop.sh
.kokoro/release/promote.cfg
.kokoro/release/promote.sh
.kokoro/release/publish_javadoc.cfg
.kokoro/release/publish_javadoc.sh
.kokoro/release/publish_javadoc11.cfg
.kokoro/release/publish_javadoc11.sh
.kokoro/release/snapshot.cfg
.kokoro/release/snapshot.sh
.kokoro/release/stage.cfg
.kokoro/release/stage.sh
.kokoro/trampoline.sh
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
SECURITY.md
codecov.yaml
java.header
license-checks.xml
renovate.json
samples/install-without-bom/pom.xml
samples/pom.xml
samples/snapshot/pom.xml
samples/snippets/pom.xml
2021-07-01 01:34:18,345 synthtool [DEBUG] > existing pom file found (.github/sync-repo-settings.yaml) - keeping the existing
2021-07-01 01:34:21,431 synthtool [DEBUG] > Wrote metadata to synth.metadata.
2021-07-01 01:34:21,491 autosynth [INFO] > Changed files:
2021-07-01 01:34:21,491 autosynth [INFO] > M .kokoro/coerce_logs.sh
 M .kokoro/dependencies.sh
 M synth.metadata
2021-07-01 01:34:21,491 autosynth [DEBUG] > Running: git log 87254ac89a9559864c0a245d6b150406439ce3d8 -1 --no-decorate --pretty=%s
2021-07-01 01:34:21,495 autosynth [DEBUG] > Running: git log 87254ac89a9559864c0a245d6b150406439ce3d8 -1 --no-decorate --pretty=%b%n%nSource-Author: %an <%ae>%nSource-Date: %ad
2021-07-01 01:34:21,498 autosynth [DEBUG] > Running: git add -A
2021-07-01 01:34:21,503 autosynth [DEBUG] > Running: git status --porcelain
2021-07-01 01:34:21,509 autosynth [DEBUG] > Running: git commit -m fix: Add `shopt -s nullglob` to dependencies script

This prevents processing the unexpanded glob "**/.flattened-pom.xml" in repos which do not flatten any POMs.

Source-Author: dpcollins-google<EMAIL_ADDRESS>Source-Date: Sat Jun 19 18:35:58 2021 -0400
Source-Repo: googleapis/synthtool
Source-Sha: 87254ac89a9559864c0a245d6b150406439ce3d8
Source-Link: https://github.com/googleapis/synthtool/commit/87254ac89a9559864c0a245d6b150406439ce3d8
[autosynth-26 b80065f] fix: Add `shopt -s nullglob` to dependencies script
 3 files changed, 6 insertions(+), 5 deletions(-)
2021-07-01 01:34:21,516 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at b80065f fix: Add `shopt -s nullglob` to dependencies script
2021-07-01 01:34:21,522 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2021-07-01 01:34:21,527 autosynth [DEBUG] > Running: git diff --binary HEAD..autosynth-26 -- . :(exclude)synth.metadata
2021-07-01 01:34:21,531 autosynth [DEBUG] > Running: git diff --binary HEAD autosynth-26
2021-07-01 01:34:21,535 autosynth [DEBUG] > Running: git apply /tmpfs/tmp/tmpizmhicrd/autosynth-26.patch
2021-07-01 01:34:21,538 autosynth [DEBUG] > Running: git add -A
2021-07-01 01:34:21,543 autosynth [DEBUG] > Running: git status --porcelain
2021-07-01 01:34:21,549 autosynth [DEBUG] > Running: git commit -m fix: Add `shopt -s nullglob` to dependencies script

This prevents processing the unexpanded glob "**/.flattened-pom.xml" in repos which do not flatten any POMs.

Source-Author: dpcollins-google<EMAIL_ADDRESS>Source-Date: Sat Jun 19 18:35:58 2021 -0400
Source-Repo: googleapis/synthtool
Source-Sha: 87254ac89a9559864c0a245d6b150406439ce3d8
Source-Link: https://github.com/googleapis/synthtool/commit/87254ac89a9559864c0a245d6b150406439ce3d8
[autosynth 366ae01] fix: Add `shopt -s nullglob` to dependencies script
 2 files changed, 2 insertions(+), 1 deletion(-)
2021-07-01 01:34:21,556 autosynth [DEBUG] > Running: git diff --binary HEAD..autosynth-27 -- . :(exclude)synth.metadata
2021-07-01 01:34:21,560 autosynth [DEBUG] > Running: git diff HEAD..autosynth-27 -- synth.metadata
2021-07-01 01:34:21,564 autosynth [DEBUG] > Running: git diff --binary autosynth-27..autosynth-30 -- . :(exclude)synth.metadata
2021-07-01 01:34:21,567 autosynth [DEBUG] > Running: git diff autosynth-27..autosynth-30 -- synth.metadata
2021-07-01 01:34:21,571 autosynth [DEBUG] > Running: git diff --binary autosynth-30..autosynth-40 -- . :(exclude)synth.metadata
2021-07-01 01:34:21,574 autosynth [DEBUG] > Running: git diff autosynth-30..autosynth-40 -- synth.metadata
2021-07-01 01:34:21,577 autosynth [DEBUG] > Running: git push --force origin autosynth
remote:
remote: Create a pull request for 'autosynth' on GitHub by visiting:
remote:      https://github.com/googleapis/java-core/pull/new/autosynth
remote:
To https://github.com/googleapis/java-core.git
 * [new branch]      autosynth -> autosynth
2021-07-01 01:34:24,349 autosynth [DEBUG] > Running: git log -3 --pretty=%b
2021-07-01 01:34:27,510 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 356, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 191, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 336, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 92, in synthesize_loop
    if change_pusher.check_if_pr_already_exists(fork.branch):
  File "/tmpfs/src/github/synthtool/autosynth/change_pusher.py", line 117, in check_if_pr_already_exists
    prs = self._gh.list_pull_requests(repo, state="open", head=f"{owner}:{branch}")
  File "/tmpfs/src/github/synthtool/autosynth/github.py", line 61, in list_pull_requests
    return cast(List[Dict], _get_json_or_raise_exception(response))
  File "/tmpfs/src/github/synthtool/autosynth/github.py", line 478, in _get_json_or_raise_exception
    json = response.json()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/requests/models.py", line 900, in json
    return complexjson.loads(self.text, **kwargs)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


Upgrade io.takari:maven 0.7.6 -> 0.7.7

This PR contains the following updates:

Package
Update
Change

io.takari:maven
patch
0.7.6 -> 0.7.7

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Takari Maven plugin 0.7.6 -> 0.7.7

See:
- https://github.com/takari/takari-maven-plugin/blob/master/CHANGELOG.md
- https://github.com/takari/takari-maven-plugin/compare/maven-0.7.6...maven-0.7.7

Takari Maven plugin or takari-maven-plugin? @Stephan202

Suggested commit message:
Upgrade Takari Maven plugin 0.7.6 -> 0.7.7

See:
- https://github.com/takari/takari-maven-plugin/blob/master/CHANGELOG.md
- https://github.com/takari/takari-maven-plugin/compare/maven-0.7.6...maven-0.7.7


Letsencrypt cloudflare doesn't work

Every time I try to install letsencrypt I receive this error. Tried with wildcard, with global api and api token - every time receive an error.
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] file-structure.sh: executing...
[cont-init.d] file-structure.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[21:01:58] INFO: Selected DNS Provider: dns-cloudflare
[21:01:58] INFO: Use propagation seconds: 60
[21:01:59] INFO: Use CloudFlare token
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Plugins selected: Authenticator dns-cloudflare, Installer None
Obtaining a new certificate
Performing the following challenges:
dns-01 challenge for domain.name
Cleaning up challenges
An unexpected error occurred:
KeyError: 'code'
Please see the logfiles in /var/log/letsencrypt for more details.
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.
There is nothing in /var/log/letsencrypt (no any files and no directory)
Tried also http challenge - file wasn't created, so on certificate can't be installed.

This log is for http
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] file-structure.sh: executing...
[cont-init.d] file-structure.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[21:37:15] INFO: Selected http verification
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Plugins selected: Authenticator standalone, Installer None
Obtaining a new certificate
Performing the following challenges:
http-01 challenge for domain.name
Waiting for verification...
Challenge failed for domain domain.name
http-01 challenge for domain.name
Cleaning up challenges
Some challenges have failed.
IMPORTANT NOTES:

The following errors were reported by the server:
Domain: domain.name
Type:   unauthorized
Detail: Invalid response from
http://domain.name/.well-known/acme-challenge/5Ew1_pXG6cxXwv9DO8qDeDrQIgXWb1e7SUfYznFP2CQ
[<IP_ADDRESS>]: 404
To fix these errors, please make sure that your domain name was
entered correctly and the DNS A/AAAA record(s) for that domain
contain(s) the right IP address.
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.


Support reference and namespace normalization

http://json-ld.org/spec/latest/#methods-2
Basically VIE needs to understand that <http://dbpedia.org/resource/Germany> and <dbpedia:Germany> are the same entity

This is now working


brush settings are not working for draw line [bug]

Version
3.9.0
Environment
Windows 10 Chrome Version 83.0.4103.116 (Official Build) (64-bit)
Current Behavior
Brush settings are not working for the draw straight line in use API direct, however, when we switch to free drawing and back to the straight line, applied settings work fine.

To Reproduce

Go to http://nhn.github.io/tui.image-editor/latest/tutorial-example02-useApiDirect.
Select the DrawLine option from the menu.
Switch to Straight line.
Change brush settings i.e. brush color and brush width.
Settings do not work when you draw a line

Yes thanks for letting me know. I will confirm this in a short time.

@syammehar
This issue was resolved in v3.11.0. Thank you!


Automatic deployment upon successful build

Not gonna happen with how the rest of Dtek has their services setup at the moment. Builds start when a new version gets tagged which is good enough in my opinion.


RE-178 Use rpc-maas 1.0.0 tag

This is in preparation for the r14.1.1 rpco release and the 2017.8 meta
release. It is important that we use a tagged version of rpc-maas so
that the 2017.8 meta release can be accurately recreated.
Issue: RE-178

There should be an equivalent patch to master too, otherwise the same mistake may be made. It's plausible that this change may also affect how tests are run in the rpc-maas repo too.

recheck_cit_artifact
recheck_cit_deploy_swift
recheck_cit_trusty_leapfrogupgrade_swift
recheck_cit_xenial_deploy_swift

recheck_cit_trusty_deploy_swift


Corrupted collision map when background is resized/removed

Note: for support questions, please use one of the following channels.
Reddit: /r/gbstudio
Discord: Join Chat
This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

Do you want to request a feature or report a bug?
Report a bug

What is the current behavior?
If you resize or rename/remove the background image for an existing scene, the collision map becomes corrupted.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
I trimmed some of the empty space from this background PNG in Photoshop, and when I switched back to GBStudio, the scene looked like this:

(Something similar happened when I renamed a background image in Finder.)

What is the expected behavior?
The editor should either attempt to preserve the collision map, or remove all collision tiles if the background changes size or is removed.

What is the motivation / use case for changing the behavior?
It's not a huge issue, but it's annoying having to fix the collision every time.

Please tell us about your environment:

Version: 1.0.0
Platform: macOS

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)
Maybe there could be a prompt if the editor detects background changes in scenes with collision maps? (Could ask if you want to remove the scene if a background is removed, etc.)

Good point, in 1bb2e94a3b172ec216e5a662e2b7c5efa6540e26 this will now clear the collisions array if the dimensions of the background image has changed


PushNotification Event described in docs but no class

I  cannot find any classes to this. Has it been removed from code and not from documentation?
Or not implemented yet?
There actual seem to be more differences between https://docs.snowplowanalytics.com/docs/collecting-data/collecting-from-own-applications/android-tracker/android-1-4-0/  and implementation.

Thanks for the heads up.
It's an error in the documentation. The Push Notification event has been implemented for iOS but not for Android yet. It has been probably added in the Android documentation for mistake.
I'll take care to fix the issue.


split_controller: reorganize the structs and add more comments

Signed-off-by: JmPotato<EMAIL_ADDRESS>What is changed and how it works?
What's Changed:
Reorganize the structs and add more comments.

Check List
Tests

Unit test
Integration test

Release note
None.

/cc @lhy1024 @AndreMouche @nolouch

/merge

/test

/test


Expose encoding of path fragment in public API

I would like to reqwest.get(format!("https://some.site/{filename}") where filename is user-supplied, but haven't found a convenient way to escape filename with reqwest's public API.
Am I supposed to use the percent_encoding crate and copy over the character sets already defined within reqwest?
reqwest::Url::join seems to assume that the path being joined is a path, not path segment, so it doesn't escape /.
I have spent 20 minutes for something that should be readily available. Even JavaScript with its crappy standard API provides the convenient encodeURIComponent function. Also, quite surprisingly, I haven't found any StackOverflow questions or existing issues regarding this, I would say, shortcoming in reqwest. Am I missing something obvious?
It would be nice to have access to the PercentEncoding struct defined in src/async_impl/multipart.rs.

It appears that the functionality does exist on the Url type: https://docs.rs/url/latest/url/struct.PathSegmentsMut.html#method.extend
let mut url = Url::parse("https://yolo.wat/").unwrap();
url
    .path_segments_mut()
    .unwrap()
    .extend(&[
        "/usr/bin/yeet.sh"
    ]);
println!("{}", url);

This prints: https://yolo.wat/%2Fusr%2Fbin%2Fyeet.sh.

Thanks! It is less pleasant to write than something like
format!("https://yolo.wat/{}", PercentEncoding::PathSegment::percent_encode("/usr/bin/yeet.sh"));

but I guess it solves the problem. Please feel free to close this issue if you wish.


[DO NOT MERGE][Java][Datetime] Fix broken test cases for DatePeriodParser

Some tests were broken due to changes in test cases.
Also, some tests were skipped due to error in other classes that were fixed afterwards.

Do not merge this PR since it braked C# unit tests. We'll review and update it.


Update/Add [knockd]

Tool information

Tool name: knockd
Add or update? Add
Desired version: latest
Approximate size:
If this is an add request:

Brief description of tool: Port knocking is a simple method to grant remote access without leaving a port constantly open. This preserves your server from port scanning and script kiddie attacks.
URL for tool's homepage: https://help.ubuntu.com/community/PortKnocking

Area for Triage:
Containers
Question, Bug, or Feature?:
Feature
Virtual environments affected

[ ] macOS 10.15
[x] Ubuntu 16.04 LTS
[x] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Can this tool be installed during the build?
Not too much
Are you willing to submit a PR?
Yes, why not!

Tool can be installed in runtime using the following script:
sudo apt-get install knockd -y

Installation takes about 7 seconds

Thanks for your suggestion, @moiseslodeiro. Considering how quick it is to install during your run, this is not something we will add to the image at this time.


Include changelog in package descrption

Include autogenerated ChangeLog in the long descrption, so users can see
what changed when visiting https://pypi.python.org/pypi/git-stacktrace

Looks like this needs work

Include changes in long_description too show it shows up on the PyPI page?


File Upload Not Working

Hello!
I recently switched from Apollo Client over to graphql-request because of my switch to using SWR for all of our requests. As part of this switch, file uploads appear to have broken across the site.
Following the documentation on the front of the page does not appear to make things work as intended.
Here is my query:
mutation($deckId: ID!, fileData: Upload!) {
  uploadFlashcards($deckId: ID!, fileData: Upload!) {
    errors {
      key
      message
    }
    status
  }
}

And here are the variables that are being sent along (as copy/pasted from my browser console):

The issue is that the request that is being sent to the server does not indicate that it has any files or anything of the like.
Headers
POST /api HTTP/1.1
Host: localhost:4000
Connection: keep-alive
Content-Length: 295
accept: application/json
authorization: Bearer <token>
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Safari/537.36 Edg/87.0.664.66
content-type: application/json
Origin: http://localhost:3000
Sec-Fetch-Site: same-site
Sec-Fetch-Mode: cors
Sec-Fetch-Dest: empty
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9,ja;q=0.8

Payload
{
  "query":"
    mutation($deckId: ID!, fileData: Upload!) {
      uploadFlashcards($deckId: ID!, fileData: Upload!) {
        errors {
          key
          message
        }
        status
      }
    }
  ",
  "variables": {
    "deckId":"d7dbd1bf-bcfe-409a-b4da-1b2b7e0f2d02",
    "fileData":{}
  }
}

Any assistance is greatly appreciated here.

same issue here,  let me know if you solved it please !

same issue here,  let me know if you solved it please !

Update latest version fixed the issue

Update latest version fixed the issue

For me the version 3.4.0 it's not working but 3.3.0 is

@lynxtaa I saw that you had a PR #175 for Upload spec support. I am at the point where I'm trying to upload a file, via the Upload scaler, and with graphql-codegen, in the most-recent version of Apollo v3; this requires integration with graphql-upload.
As it pertains to this issue, and PR #175, do you have an example on how to use the GraphQLClient, and typescript-graphql-request to upload a file? I'm faced with the same issue as @shadyendless , where the incoming request nullifies the file data:
const sdkUploadFileVariables = {
      file: new Promise(resolve => resolve({
        filename: 'twitter.png',
        mimetype: 'image/png',
        encoding: '7bit',
        createReadStream(): ReadStream {
          return fs.createReadStream("./test_files/twitter.png");
        },
      }))}

gets messed up by GraphQLClient and produces an empty file: { } object.

For me the version 3.4.0 it's not working but 3.3.0 is

@guiquintelas This downgrade did not work for me, neither did an upgrade to v3.5.0.

@gblikas In graphql-request there is a heuristic for detecting files in variables https://github.com/prisma-labs/graphql-request/blob/a6d1365ae0fc6694c45ea404e4a44b93a8c06479/src/createRequestBody.ts#L10 it supports instances of File and Blob and duck-types NodeJS streams.
Does it work without typescript-graphql-request?
Sorry, I don't have any examples because right now I'm using my own implementation of graphql client and not using graphql-request

Does it work without typescript-graphql-request?

@lynxtaa I poked around with awesome-graphql-client, but I'd like to stick with graphql-request, since it is an SDK generator via graphql-codegen...
https://github.com/prisma-labs/graphql-request/blob/a6d1365ae0fc6694c45ea404e4a44b93a8c06479/src/createRequestBody.ts#L10
works without typescript-graphql-request, and with it; the end of
https://github.com/prisma-labs/graphql-request/blob/a6d1365ae0fc6694c45ea404e4a44b93a8c06479/src/createRequestBody.ts#L19
has a FormData object, with valid FormData.entries() - I am unsure if it is properly formatted, but when cross-referenced with Postman, both FormDatas have the same content:
[
      'operations',
      '{ "query":"mutation uploadFile($file: Upload!){uploadFile(file: $file){ id }}"}'
]
[ 'map', '{"1":["variables.file"]}' ]
[ '1', '[object Object]' ]

ℹ  FYI, Postman does send through a valid File, and Upload, via multipart/form-data.

@lynxtaa After taking your comments into consideration, and using multipart/form-data header, from what I can tell, the error seems to lay with Busboy, and graphql-upload, regardless of using GraphQLClient, or request?;
BadRequestError: Missing multipart field ‘operations’ (https://github.com/jaydenseric/graphql-multipart-request-spec).
    at Busboy.<anonymous> (/Users/username/projects/projectName/node_modules/graphql-upload/public/processRequest.js:330:11)
    at Object.onceWrapper (events.js:421:28)
    at Busboy.emit (events.js:327:22)
    at Busboy.EventEmitter.emit (domain.js:467:12)
    at Busboy.emit (/Users/username/projects/projectName/node_modules/busboy/lib/main.js:37:33)
    at /Users/username/projects/projectName/node_modules/busboy/lib/types/multipart.js:304:17
    at processTicksAndRejections (internal/process/task_queues.js:75:11)

Perhaps, @jaydenseric can shed some light on this issue? Here are the relevant packages I'm using,
// package.json

"apollo-server": "2.22.2",
"apollo-server-express": "^3.3.0",
"graphql-upload": "^12.0.0",
"graphql-request": "^3.5.0",

Although I installed apollo-server, it is not inuse, we are using apollo-server-express only. We also have, {upload: false} in our ApolloServer config, however this shouldn't matter, since v3.3.0 has removed scalar Upload support.

I also found some other github issues that seem to be related to this one (or legacy solutions, which have not proved helpful); https://github.com/jaydenseric/graphql-upload/issues/241, https://github.com/jaydenseric/graphql-upload/issues/238

@gblikas use the Chrome inspector network tab to double check the client is sending a valid GraphQL multipart request.

ℹ FYI, Postman does send through a valid File, and Upload, via multipart/form-data.

Do you mean to say the files upload and are processed by the GraphQL API correctly when you do the file upload requests via Postman? In that case you problem is not the server; the client is not sending valid requests.

Although I installed apollo-server, it is not inuse, we are using apollo-server-express only.

Uninstall it. Why bloat your node_modules with megabytes of junk and confuse us with that detail? Uninstalling it also makes sure it's not accidentally being used somewhere in your codebase.

We also have, {upload: false} in our ApolloServer config, however this shouldn't matter, since v3.3.0 has removed scalar Upload support.

Remove that config that does nothing.

ℹ FYI, Postman does send through a valid File, and Upload, via multipart/form-data.

Do you mean to say the files upload and are processed by the GraphQL API correctly when you do the file upload requests via Postman? In that case you problem is not the server; the client is not sending valid requests.

@jaydenseric Yes, this seems to be the case; this is why the issue is in graphql-request. I mention you in this issue, because https://github.com/jaydenseric/graphql-upload/issues/238 is very similar but doesn't work, as per Busboy. 🤔  Wondering about input from @lynxtaa , now.

@gblikas Could you create a repository with a minimal reproduction of this issue? I'll look into it

Also I noticed that graphql-request uses form-data package for NodeJS which is highly popular but not spec-compliant and it's usage is discouraged https://github.com/node-fetch/node-fetch/pull/1212

@gblikas Could you create a repository with a minimal reproduction of this issue? I'll look into it

@lynxtaa Will do!

@gblikas Could you create a repository with a minimal reproduction of this issue? I'll look into it

@lynxtaa Will do!

@lynxtaa sorry that it took so long! Here is a very bare-bones implementation of the error in question; it was based on the graphql-upload example code,

🔗  https://codesandbox.io/s/zen-joana-6fy4z

In order to run the sample, make sure the server is live, and then open another terminal;
# graphql-codegen to build the Request SDK, [as per the examples](https://www.graphql-code-generator.com/)
npm run generate
# run a node-based GraphQL request using the Requests SDK for file uploading, [as per the graphql-upload docs](https://github.com/jaydenseric/graphql-upload#function-graphqluploadexpress)
npm run test

⚠ I left the Content-Type header "application/json". If you want to see the upload error, please change the Content-Type to multipart/form-data, etc..

Let me know if you're able to reproduce this error.

@gblikas

You shouldn't set Content-Type header to multipart/form-data when sending multipart/form-data via fetch. Fetch by spec must set Content-Type to multipart/form-data; boundary=, followed by the multipart/form-data boundary string generated by the multipart/form-data encoding algorithm: https://fetch.spec.whatwg.org/#bodyinit-unions
Just putting a stream as a graphql variable will work

Check out https://codesandbox.io/s/unruffled-paper-rxxct?file=/test.ts

@gblikas

You shouldn't set Content-Type header when sending multipart/form-data via fetch. Fetch by spec must set Content-Type to multipart/form-data; boundary=, followed by the multipart/form-data boundary string generated by the multipart/form-data encoding algorithm: https://fetch.spec.whatwg.org/#bodyinit-unions
Just putting a stream as a graphql variable will work

Check out https://codesandbox.io/s/unruffled-paper-rxxct?file=/test.ts

@lynxtaa The sandbox worked great. I'll take a moment to integrate this in to our internal framework, and get back with a confirmation of it working.

@lynxtaa For next time, where could I have found better documentation around how to upload a file using graphql-request? Uploading a file progamatically, via graphql-<package> isn't mentioned anywhere, out of perhaps knowing what you pointed out in (1) and (2), already. Perhaps I wasn't looking in the right area?

@gblikas I'm glad it helped!

@lynxtaa For next time, where could I have found better documentation around how to upload a file using graphql-request? Uploading a file progamatically, via graphql-<package> isn't mentioned anywhere, out of perhaps knowing what you pointed out in (1) and (2), already. Perhaps I wasn't looking in the right area?

You can check out sources, they are rather minimal https://github.com/prisma-labs/graphql-request/tree/master/src
Under the hood graphql-request will use extract-files to detect streams, Blobs and Files in variables and change request body to a multipart/form-data according to a GraphQL File Upload spec

@shadyendless Did the solution @lynxtaa and I have been discussing help solve your current problem?

Hi there, maybe it's obvious to others but since I spent sometime figuring out why my generated sdk client cannot upload file I will describe what happened to me here:

I used codegen to generate a sdk.ts file.
Use  @golevelup/nestjs-graphql-request inside my nestjs application.
Use the sdk inside some controller to upload file to a graphql server and fail miserably.
Find the issue here and try everything here.
Solved by deleting the Content-Type header inserted when I declare the module (I just follow the instruction in the 2nd step package README):

GraphQLRequestModule.forRootAsync(GraphQLRequestModule, {
      imports: [],
      inject: [ConfigService],
      useFactory: async (configService: ConfigService) => ({
        // Exposes configuration options based on the graphql-request package
        endpoint: configService.get('API_URL'),
        options: {
          method: 'POST',
          headers: {
            'Content-Type': 'application/json', <=== delete this
            Authorization: 'Bearer ' + configService.get('API_TOKEN'),
          },
        },
      }),
    }),

It seems the Content-Type header is not overridden automatically by whatever handled the actual request.

https://github.com/jasonkuhrt/graphql-request/issues/500


When read PoolSubpage's variant fields, it should lock on PoolSubpage's head lock, instead of PoolArena's lock

Motivation:
When we read PoolSubpage's variant fields:doNotDestroy and numAvail, we should lock on PoolSubpage's head subpage, instead of PoolArena's lock.
Because the writing on doNotDestroy and numAvail is always under PoolSubpage's head subpage lock.
Modification:

Introduced a int headIndex field in PoolSubpage to locate the head subpage, lock on PoolSubpage's head subpage instead of PoolArena's lock.
This also help to locate head subpage when doing bytebuf release. Fixed 13625.

Result:
See above.

@laosijikaichele Thanks!

@laosijikaichele @chrisvest
this PR is also work for https://github.com/netty/netty/issues/13777?

@echooymxq I am not sure, but it might be related, in some cases, it locks on wrong head when releasing direct buffer, which will make the modification of numAvail not thread safe, please refer https://github.com/netty/netty/issues/13625.


Update lkdemo.cpp because of memory leak

In MacOSx x64 it was getting a memory leak even without initializing points:
http://code.opencv.org/issues/4250

makes sense :+1:


requests.exceptions.ConnectionError: ('Connection aborted.', ConnectionResetError(10054, 'An existing connection was forcibly closed by the remote host', None, 10054, None))

when download videos, got this error

Can you provide more information about how to produce the error and more info about the error?

If you have solved your issue please share the info about how you solved the issue.
If you are still having the issue please provide more info to help solve the issue.

@coderhh I will be waiting until next week for your reply. I you don't reply by the next week I will close this issue.


KAFKA-3177: log warning when topic/partition doesn't exist

LGTM

Since this is not the full fix for KAFKA-3177, let's either file a new issue or make sure it remains open after we merge this.

I thought we had merged this, but looks like we dropped it. @hachikuji, is it worth reviving it?

@ijuma We may as well get this in. Even if the fix may not satisfy all users, at least problems should become more apparent in the logs.

Discussed offline with @ijuma. We think this patch may not be providing much additional value at the moment and there's some risk of excess log spam. It's probably better to leave this issue open and come back to it after the dust has settled as far as the long-term handling of unknown topics/partitions. This problem should primarily affect users using manual assignment and they can currently enable DEBUG logging to get more details.

@hachikuji If this patch isn't going to be useful, maybe close the PR? And do you have thoughts on the JIRA? If we're not addressing it in 0.10.2 then I'm going to remove the version. I also suspect we can reduce the priority given that we haven't gotten to it for so long (it started with fix version <IP_ADDRESS>). @ijuma do you have thoughts since you were the one that originally bumped priority from major to critical? I haven't noticed many mailing list questions about this.

@ewencp, I'm OK with reducing the priority and removing the "fix version". The original thinking for increasing the priority is the change we made to return UnknownTopicOrPartition in case the user is not authorized to describe the topics. Given feedback so far though, it seems that it's not as bad as I thought it would be.


Very good project but some good features are missing.

I would like the following features to be added:-

Player Shooting, Throwing Items, etc. actions
Flexible UI system. Suppose I want to make a UI inventory for items.
Custom Actors as well as objects.
More Flexibility on scripting. Though the Engine is no-coding, it would be much better if there were a scripting option available.
Isometric Game support and collisions.
Waiting for 3D mode too

Very good engine. It will be a lot useful if the features are implemented. Vote who of you agree with me.

Player projectiles are available in 2.0.
Inventories and other such menus can already be done. Example: https://toxworks.itch.io/menu-system-in-gb Of course, more flexibility is always welcome, e.g. being able to use Actors as items in a GBS menu.
What would a custom actor entail? You can already customize how actors look, move, and behave, and you can customize their animations beyond the default animation types with scripts. The main missing feature is different actor sizes.
You can already Eject Engine and edit the C code directly to add additional features, and subsequent builds from within GBS will use your modified code, AFAIK.
An isometric option would be interesting to see! The main differences would be player movement, which would be at angles rather than directly vertical/horizontal, and some way to mark the heights of tiles in the collision editor. Presumably there would need to be a way to set how far an actor can jump (i.e. whether a given height difference counts as collision or not), but this could probably be controlled with scripts too.
The GB can't do 3D, 3D can only be faked with different backgrounds. If you're talking about first-person games, those can already be created quite easily. Just hide the Player and switch scenes based on the directional inputs. Example: https://jojoruteon.itch.io/3dmaze-gbstudio


Improve snapshot_download offline mode

Follow-up after https://github.com/huggingface/huggingface_hub/pull/1899 and https://github.com/huggingface/diffusers/issues/6168#issuecomment-1857675973.
At the moment if snapshot_download fails because of a connection error, because repo not found, because Hub is down, etc., we do not try to locate the cached snapshot directory. This is not consistent with 1. what happens in hf_hub_download and 2. what happens in snapshot_download itself if local_files_only/HF_HUB_OFFLINE is set. This PR fixes this by adapting the existing workflow of hf_hub_download to snapshot_download. It should help diffusers users that have an unstable connection to load their weights when already cached.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Thanks for the thorough review! I've updated the error message that was wrong. I'll wait for the CI to complete and merge it to include it in next release :)

Failing test is unrelated. Let's get it merged!


fix: an issue where client features wouldn't correctly update

Client features were being read on update, correctly calculated and then subsequently discarded. This patch makes incoming features actually get written to the memory provider

James Bond and Jason Bourne, what a combination +1

I'm glad you noticed, I several minutes looking up famous spies so I could keep our test data consistent haha


Add initializer for a URL with an optional String

🚀
I often have an object loaded from a server that has optional paths, e.g.
struct Food: Codable {
    imagePath: String?
}

With this extension users can simply call
if let imageURL = URL(food.imagePath) { ... }

instead of
if let imagePath = food.imagePath,
   let imageURL = URL(imagePath) { ... }

Alternatively I could have defined the property as imagePath: URL?, but I don't do this for decoded values because if the URL is invalid the entire decoding process fails as opposed to just returning nil, so this allows for smoother error handling.
Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 5.0.
[x] New extensions support iOS 10.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 644 tests, with 0 failures (0 unexpected) in 9.941 (10.870) seconds

:book:
tvOS: Executed 612 tests, with 0 failures (0 unexpected) in 2.499 (3.030) seconds

:book:
macOS: Executed 475 tests, with 0 failures (0 unexpected) in 0.678 (0.850) seconds

  Generated by :no_entry_sign: Danger


Please release

cdnjs releases are outdated.
Please release the jquery 3 compatible version.
Btw, this is a very nice plugin. 👍

Just ran into this myself.
It says
Uncaught TypeError: map.size is not a function
because size() was removed from jQuery 3.

Ah, so https://github.com/kemayo/maphilight/pull/70 was merged in but no new release has been made yet?

Using https://rawcdn.githack.com/kemayo/maphilight/7932449fbc49c7767fbd72a807110c632c11ee95/jquery.maphilight.js as a workaround for now

Done. 1.4.0 is released.


Merge recent changes in dev (codebase structure)

Requesting changes implemented in dev to be merged to main.

Merged suggested changes into main


Code generation query_root$ type error with execute function

The result of the code generation I got, doesn't seem to contain query$ only query_root$
const { AsyncGenerator, loadRemoteSchema } = require( "graphql-ts-client-codegen" );
const path = require( "path" );

const generator = new AsyncGenerator( {
    schemaLoader: async() => {
        return loadRemoteSchema( "https://beta.pokeapi.co/graphql/v1beta" );
    },
    targetDir: path.join( __dirname, "/pokeapi-graphql" ),
} );
generator.generate();

I thought nothing of it it and just used query_root$ and it worked in the query definition, everything was like I expected it
export const POKEMON_LIST_QUERY =
query_root$.pokemon_v2_pokemon(
{
    offset: ParameterRef.of( "offset" ),
    limit: ParameterRef.of( "limit" )
},
pokemon_v2_pokemon$
.name.weight.height
.pokemon_v2_pokemonspecy(
    pokemon_v2_pokemonspecies$
    .pokemon_v2_pokemonspeciesnames(
        {
            where: {
                pokemon_v2_language: {
                    iso3166: {
                        _eq: ""
                    }
                }
            }
        },
        pokemon_v2_pokemonspeciesname$
        .genus
    )
).pokemon_v2_pokemonsprites(
    pokemon_v2_pokemonsprites$.sprites
) );

But when I actually wanted to execute it, I got this error when trying to pass the query
Types of property 'fetchableType' are incompatible.
Type 'FetchableType<"query_root">' is not assignable to type 'FetchableType<"Query" | "Mutation">

execute( POKEMON_LIST_QUERY, {
    variables: {
        offset: 0,
        limit: 50,
        //lang: "en"
        where: {
            pokemon_v2_language: {
                iso3166: {
                    _eq: "en"
                }
            }
        }
    }
} ).then( ( value ) => {

    console.log( value.pokemon_v2_pokemon.toString() )
} )

The type information VS-Code shows me
const POKEMON_LIST_QUERY: query_rootFetcher<{
    readonly pokemon_v2_pokemon: readonly ({
        readonly name: string;
    } & {
        readonly weight?: number | undefined;
    } & {
        readonly height?: number | undefined;
    } & {
        readonly pokemon_v2_pokemonspecy?: {
            readonly pokemon_v2_pokemonspeciesnames: readonly {
                ...;
            }[];
        } | undefined;
    } & {
        ...;
    })[];
}, UnresolvedVariables<...> & ... 1 more ... & UnresolvedVariables<...>>

The full error message:
Argument of type 'query_rootFetcher<{ readonly pokemon_v2_pokemon: readonly ({ readonly name: string; } & { readonly weight?: number | undefined; } & { readonly height?: number | undefined; } & { readonly pokemon_v2_pokemonspecy?: { readonly pokemon_v2_pokemonspeciesnames: readonly { ...; }[]; } | undefined; } & { ...; })[]; }, Unres...' is not assignable to parameter of type 'Fetcher<"Query" | "Mutation", { readonly pokemon_v2_pokemon: readonly ({ readonly name: string; } & { readonly weight?: number | undefined; } & { readonly height?: number | undefined; } & { readonly pokemon_v2_pokemonspecy?: { ...; } | undefined; } & { ...; })[]; }, UnresolvedVariables<...> & ... 1 more ... & Unreso...'.
  Types of property 'fetchableType' are incompatible.
    Type 'FetchableType<"query_root">' is not assignable to type 'FetchableType<"Query" | "Mutation">'.
      Type '"query_root"' is not assignable to type '"Query" | "Mutation"'.ts(2345)

I cannot access the SDL.
Generally speaking, the type name of the query in the SDL of the server should be "Query", not "Query_root". The framework generates the "Query_root" type in the client according to the DSL.
I'll investigate whether this behavior is GraphQL-compliant. If it is, I will add two configuration arguments to the code-generator: "queryTypeName" and "mutationTypeName".
This workaround can be used now: copy the "Async.ts" in the generated directory to another directory, and modify the execute function
export async function execute<TData extends object, TVariables extends object>(
     // Change "Query" to "Query_root"(Please read the sdl carefully to get the right typeName)
     fetcher: Fetcher<"Query_root" | "Mutation", TData, TVariables>,
     ...other code should not be changed, omit...

I also can't access the SDL, it seems like their using Hasura Engine for the graphql server.
I will try out the workaround, thanks

I did the workaround bot got a similar type error.
I changed
fetcher: Fetcher<"Query_root" | "Mutation", TData, TVariables>

to
fetcher: Fetcher<"query_root" | "Mutation", TData, TVariables>,

and it worked without problems

I'm having another issue, but with the conditional where. Should I open another Issue or not?

Yes

Ok, resolved it myself. Just didn't know enough about graphql variables
Thanks for the help though


Fix serverPath conflict with index files, fix paths output into build

I started leaving some final comments on https://github.com/hapipal/lalalambda/pull/9, but they were esoteric enough that I thought it would be best to just ping-pong some code back your way!  I was experiencing one bug: index files in the root of the project were competing with the default server path due to the way getServer() fell back to require.resolve(path).  It always resolved successfully when there was no server path but there was an index file at the root.  I fixed this by applying the default serverPath of 'server' eagerly rather than trying to fall back to it at the last moment.
I also did cleanup on some related code.  Now we escape some characters that could reasonably show-up in file paths when writing them into a string literal in entrypoint().  And I try to make super sure to pass a relative file path to entrypoint, since absolute files wont make sense in the build.  As you pointed out, an absolute serverPath is already a bad idea, but I figured it couldn't hurt to tighten that up.
If you like these feel free to merge then we can go back over to https://github.com/hapipal/lalalambda/pull/9, but if you'd like to approach any of this differently I'd be open to that as well!

  I was looking at the commits for a good 5 minutes until I realized this was not the hapipal namespace and it was my own.


Fixed #30614 -- Added constraint name validation to system checks.

Ticket: #30614

Thanks @pope1ni I missed it I will add asap.

Thanks for this patch :+1: and I'm really sorry but it is backward incompatible. I don't think that we should add this restriction.
Closing per ticket.


Fabric module naming changes module names but does not change instantiation and .v file names

Hi. I defined module_names.xml file as mentioned in . And added "rename_modules --file ${OPENFPGA_RENAME_MODULE_FILE}" to OpenFPGA run script. Fabric module naming changes module names but does not change instantiation and .v file names. Shouldn't these names be the same?

@mustafaarslan0 Currently, this is expected. We only change the module name so far for grid blocks. It would be easy to change the file name accordingly. But for instance name, it may require more effort.


Meters for httpclient exceptions

Adds counters for exceptions that happen after InstrumentedHttpRequestExecutor.execute has been invoked (some errors, like connection refused, invalid URLs, etc. don't make it that far). By default, use the simple name of the exception as the name of the meter, yielding names like     org.apache.http.client.HttpClient.foobar.SocketTimeoutException.

Thanks, I'll take a look!

Thanks for the attention and feedback! I've updated according to your comments.

Thanks for the contribution!


masking to combine variations

I was going to combine this with https://github.com/invoke-ai/InvokeAI/issues/1270, which would make it more useful, but I decided that they should really be considered and discussed separately.
This is also not something that I expect to happen on a short timescale, but it might be worth keeping in mind while developing some more immediate features, like inpainting.
A feature I would like to see is an interface for combining the best parts of different images when generating variations using the same seed, or when generating multiple images with an img2img prompt. I'm imagining layering two or more of these variations (which have identical compositions but different details) and using a soft masking brush to reveal parts of one behind the other. This would be useful, for example, for combining the subject in one variation with the background from another, or for quickly repairing a variation that sprouted an extra limb, but is otherwise superior to the others.
Now, the obvious way to do this is to take both images into Photoshop/GIMP, but they're kinda overkill for this use case. My computer also isn't the happiest about having both Photoshop and InvokeAI running at the same time, because both are extremely RAM-hungry. Most of the necessary stuff to make it work is already needed for the impainting interface, and it would be really convenient to be able to do it within the invokeai webui, then be able to drop that directly back to img2img to create new variations that will conceal the seams.
Thoughts?

As variations have been removed, I'm going to close this one out.


vimPlugins.harpoon: add plenary-nvim dependency

Description of changes

Add plenary-nvim dependency for vim plugin harpoon.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

1 package built:

vimPlugins.harpoon

Result of nixpkgs-review pr 218291 run on aarch64-darwin 1

1 package built:

vimPlugins.harpoon


Remove includes of external sites

PowerDNS-Admin is bound to be used inside enterprises premises, managing very sensible data. It should not include any links to providers like fonts.googleapis.com, cdnjs.cloudflare.com or gravatar.com! All such data should be included into the sources, or if not possible (for Gravatar) enabled only after confirmation by the user.

DNS is part of core infrastructure. I can't believe this is the default, even after 4 years since the issue was opened! We have the exact same problem, we deal with real sensitive (not sensible) data. That we would give google hints about when we access our dns interface is outrageous, I'm sorry to say. This is a big confidentiality/trustworthiness problem.

OFFLINE_MODE = True removes requests to Google and Gravatar with no other disadvantages.
That being said, I dislike the wording (and the default), because it implies that contacting 3rd party services for fonts and pictures is at all desirable. HTTP/2 obsoletes the performance advantage, and traffic should be negligible for a DNS frontend. Also, images can be delivered as claims in a more data protection friendly manner when using e.g. OIDC.
My recommendation would be to:

Kill the code path that uses Google for fonts entirely
Support OIDC claims for pictures
Make Gravatar an optionally supported feature
Remove OFFLINE_MODE

If  @ngoduykhanh agrees, I would draft a PR for this. If not, I would draft one explaining offline mode in README.md.

Hi @danimo,
given #1032, I'm inclined to say: please go ahead. With @RGanor and I, there are at least two maintainers now, so we should be able to make it happen.

@danimo what you say make sense, but bare in mind that offline mode is not just for the reasons you mentioned above, some deployments are completely isolated from the internet, so relying on external service is not possible.
If any changes will be made, it still need to support such deployments.

I am aware of the offline modes intent, but with a privacy-by-default mode, the whole point is that there should no outside services left to contact in a default deployment.

Hello @danimo
I agree with you. We don't have resources to adress it right away but it should be done in a not too far future.
Regards


The reward function of 'Jumper-v0' might be incorrect

# collect pre step information
pos_1 = self.object_pos_at_time(self.get_time(), "robot")

# step
done = super().step({'robot': action})

# collect post step information
pos_2 = self.object_pos_at_time(self.get_time(), "robot")

# compute reward
com_1 = np.mean(pos_1, 1)
com_2 = np.mean(pos_2, 1)
reward = (com_2[1] - com_1[1])*10 - abs(com_2[0] - com_1[0])*5

https://github.com/EvolutionGym/evogym/blob/9a1a5e7b26702184821e6e64587220ead2ab0e21/evogym/envs/jump.py#LL36-L54C71
When jumping up, the reward is positive, but when falling, the reward is negative. Thus the cumulative reward is 0 when landing. Nothing will be learned. The optimal policy is to jump up before the end of an episode and reach the highest point exactly at the end.

Perhaps the possible explanation is that each time it will jump higher and higher.

Thanks for the comment. It seems that this reward was sufficient to learn a good jumping behavior, but perhaps is not ideal/optimal given what you have pointed out. If you find that something else works better feel free to let us know.

Thanks. It is indeed sufficient to learn a good policy, but the policy might not be stable during training. Perhaps more investigations are needed.


Creating of env list

When creating the env list i am splitting with the '=' character but if there is the same character in the value it will be also split and the new list will be wrong. need to implement a new function that splits by the first = it finds if it finds one.

Done


Created new challenge

I hope this is what you were talking about.

Can you change the method signature to pass in a string/char of "P", "M", or "T".  Then the user is more likely to use a switch statement when coding the challenge.

The examples show it is passing the whole word.  Can you line it to with the tests.
Thank you


make sure the new default config remains overridable in .config for all images

How are we supposed to provide a config for hyprland before this issue is fixed? Or is it just not possible yet?

How are we supposed to provide a config for hyprland before this issue is fixed? Or is it just not possible yet?

Not possible yet but I can get around to it soon (next few days). PRs are also welcome. It's a very straightforward fix. All we need is for the SDDM entry to invoke a script that checks for the presence of a config in .config, then falls back to the current config from /usr if there's no config there.
Also, these images are all very much in an alpha state and I just realized that isn't made clear in the readme. Added that here: https://github.com/wayblueorg/wayblue/blob/live/README.md

@rompetomp https://github.com/wayblueorg/wayblue/commit/1b3d39aabd50186effaa61ea01376da1a1a33a6a
tested it very briefly. please reopen this if you run into issues.

Thanks, I can confirm it works :+1:


Please reopen closed issue 'Unable to run any micropython in PyCharm for ESP8266 #120'

Please try to fix https://github.com/vlasovskikh/intellij-micropython/issues/120#issuecomment-872582972 it shouldn't have been closed. I started with pycharm and the micropython plugin a couple of days ago. Great stuff. Keep it up! I'm unable to solve the problem and couldn't find any information which helps. I'm not a programmer. Please see also https://forum.micropython.org/viewtopic.php?f=15&t=10771

That issue does not need to be reopened, it has been solved.
You are doing the same thing that AstroDabb was. You are running a micropython script with regular python instead of uploading it to a microcontroller.
Once you have enabled micropython support (I assume you have since you can access the REPL), you need to right click your main.py and choose Run Flash main.py.
You should familiarise yourself with the platform you are using
This is the documentation for Run Configurations in PyCharm
Keep in mind with micropython, you are not running the code directly, instead you are running an upload script to put your code on your microcontroller.

@hcet14 The problem you have (PyboardError: failed to access COM3) is not related to the original problem of issue #120. That issue was about some confusion with which run configuration to use: Python or MicroPython.
I'll update this issue so it reflects the issue you have if you don't mind.

@hcet14 I don't have a Pyboard device and a Windows machine at hand, so I cannot reproduce your issue. The MicroPython plugin actually tells the exact command it runs to flash files to your device. It's the first line in the Run tool window:
...\python.exe ...\microupload.py [parameters...]

You might experiment with running microupload.py and try to debug the issue.
To other Pyboard users: any help on reproducing and fixing the issue is appreciated.

@vlasovskikh I do not believe what @hcet14 is seeing is a bug. Take a look at the micropython.com Forum post  he made. It shows some pictures and you can see that he is running the script with pure python, not flashing it to the microcontroller. He just need to change his run configurations and I think he will be good to go.

@jdjjm In the comment by @hcet14 in #120 (July 2) they show a screenshot where they run a MicroPython run configuration: it's microupload.py with the corresponding parameters.

@vlasovskikh Woops, I missed those screenshots. Looks like this might be a bug.


The description for get token for Rest API is outdate

Environment

Documentation version: 7.0
Part of the documentation: Manual
Existing page: rest_api_v2_ex_get_token

Description of the bug or enhancement
The description for get token for Rest API is outdate.
The password need add a prefix :{noop}
Discussion in forum: https://www.cuba-platform.com/discuss/t/geting-token-for-rest-api-throw-exception/8846/7

Duplicates https://github.com/cuba-platform/documentation/issues/427


Local imports don't ever set album art

The code is commented out in ImportOperation. Did it ever work?

Ohhhhhhhh, the covers directory would be quite different, since it wouldn't belong to a server. That'd mean a refactor for relative covers to support serverless.

The code puts it in Covers/Artist/Album.ext, which seems a bit fragile in case an artist and server have the same name. Perhaps it could have a specialized subdirectory under Covers, used when server is nil?


Add support for command line parsing and execution of code examples.

This PR does three things:

Add a minimal command line parser. This is largely inspired by Mono.Options, except that since Mono.Options isn't signed, I cannot use it for the libraries for nuget releases. All the relevant code is in Flag.cs.

Move all the ExampleUtilities, ExampleBase classes into the Utils folder. This way, the examples project only has Examples, and so additional copy-paste of dependent files are not needed when running an example.

Split the code-example running logic into an ExampleRunner class and made it more robust.


Enclose event read loop in try/finally to ensure close is done

This ensures that we handle both close cases that exit the loop normally and those (like ping timeout) that result in an exception.
Fixes #51

LGTM - where is the 12-hour test that validates the fix? :-)

I thought about how to test, but I don't think the testing infrastructure lets us simulate an error from the network layer.  On my laptop I was able to simulate by simply breaking in the ping handler on the Java side and waiting until the python side timed out (in about 20s).  That's how I noticed the behavior difference between that case and what happens when you simply nuke the Java side (still closes, but no exception in that case for some reason).


Admin user gets logged in to the the last default environment that was created for other users.

Server version - V0.33.0-rc1
Set up - Rancher server with LDAP auth enabled.
Log in as any user (who does not have access to any existing environments) - say "test1".
This results in a default environment created for this user.
Log in as admin user. We get automatically get logged in to Default environment of "test1".

I think this is a result of setting the defaultProjectId userPreference failing (@hibooboo2).  So the default isn't set or is a project that isn't available, and the UI just picks one, and LDAP/admin/etc has nothing directly to do with it.

Unable to duplicate on master. Can you try and duplicate it again @sangeethah

This issue is also reproducible using local auth enabled set up.
Server version - V0.37.0-rc1
Set up - Rancher server with local auth enabled.
Log in as any user (who does not have access to any existing environments) - say "test1".
This results in a default environment created for this user.
Log out.
Log in as admin user. We get automatically get logged in to Default environment of "test1".

Using master unable to replicate given steps to reproduce.

Just reproduced on v0.47.0-rc2.

Version - master 1/29
Verified fixed


Is there a way to dissociate public IP address from NIC through SDK?

Is your feature request related to a problem? Please describe.
I am looking for a way to dissociate public IP addresses from my nic that is associated with a VM that is part of a scale set. There is ways to do it through the Azure portal and CLI (https://docs.microsoft.com/en-us/azure/virtual-network/remove-public-ip-address-vm). My question is that is there a way to do it through SDK?
Describe the solution you'd like
A solution would be clearly documented steps attached to the above-mentioned link that describes how to dissociate a Public IP from NIC.
Describe alternatives you've considered
Perhaps there is an existing solution as part of networkmgmtClient.PublicAddressOperation under create_or_update()? Is there an example anywhere to do this?
Additional context
Add any other context or screenshots about the feature request here.

Moving this to the Azure Networking team who owns this space.

Hi @wjia7 ,
Here are the steps to dissociate public IP address from NIC( You can click the link for your reference)：
Step 1, create a Network Management Client Object.
Step 2, get subnet_obj.
Step 3, configure network interface parameters.  When the value of the parameter public_ip_address  is None for disassociation. When the value of the parameter public_ip_address is 'ip_address_object ', it is associated with the specified public IP object.
Step 4, use method create_or_update to update network interface configuration.
The detailed code is shown below about dissociate public IP address from NIC:
from azure.mgmt.network import NetworkManagementClient
from azure.common.credentials import ServicePrincipalCredentials
GROUP_NAME = 'ResourceGroupName'
NIC_NAME = 'NetworkInterfaceName'
subscription_id='<Your Subscription ID>'
credentials = ServicePrincipalCredentials(
   client_id='<Your service principal App ID>',
   secret='<Your service principal password>',
   tenant='<Your tenant ID>'
)
# Step 1, create a Network Management Client Object.
network_client = NetworkManagementClient(credentials, subscription_id)
# Step 2, get subnet_obj.
subnet_obj = network_client.subnets.get(resource_group_name=GROUP_NAME, virtual_network_name='VirtualNetworkName',
                                    subnet_name='default')
# Step 3, configure network interface parameters.
params = {'location': Location,
              'ip_configurations': [
                 {
                    'name': ip_configuration_name，
                    'subnet': {'id': subnet_obj.id},
                    # None: Disassociate;
                    'public_ip_address': None,
                 }
              ]
              }
# Step 4, use method create_or_update to update network interface configuration.
network_client.network_interfaces.create_or_update(
   resource_group_name=GROUP_NAME,
   network_interface_name=NIC_NAME,
   parameters=params)

@jongio

Hi @wjia7 ,
How is this issue going? Feel free to let us know if further help is needed.

We are going to close this issue because we haven’t heard from you in a few days, but please re-open it if the issue still persists.


Moto SSM put parameter allows empty string as a value, real service does not

Hello,
I noticed that moto SSM service allows putting parameter with empty string. This is not available in boto or via AWS web UI.
Moto is running in server mode.

boto3==1.17.35
botocore==1.20.35
moto==2.0.2

Code example that is invoking parameter creation using moto:
client.ssm.put_parameter(             Name='test',             Description='string',             Value='',             Type='String',             Overwrite=True         )
It creates the parameter and returns empty string when fetched.
This is, on the other hand, output from boto parameter creation:
$ aws ssm put-parameter --name "/test" --value "" --profile saml An error occurred (ValidationException) when calling the PutParameter operation: 1 validation error detected: Value '' at 'value' failed to satisfy constraint: Member must have length greater than or equal to 1.
That makes inconsistency. Is this a feature somehow or should be fixed?
Regards

Hi @tokwiat, thanks for raising this!
Will mark it as an enhancement to implement value-validation, as we don't have anything at the moment.

Ok, cool, thanks for answer.

I've just opened a PR which I think will fix the issue! @bblommers


[docs] Add rocket icon to docs nav items

Not that important, but a rocket launch icon could look nice for the deployment section of the Toolpad docs.
https://github.com/mui/mui-toolpad/pull/1829

#36408 would probably make this change irrelevant.

I guess it wouldn't hurt to merge this anyway for now?


Add publish buttons to the dataset and workflow

Purpose:
The current hub site does not distinguish between private and public workflows. This pr aims to introduce this concept and add methods for hub users to publish workflows from texera hub site.
Description:

Added a toggle button to the workflow list item to help switching between public and private workflow
Changed the database schema to add is_public boolean attribute to the workflow table (Added 15.sql for database updates).
Implemented access checks so that only users with read or write access can change a workflow to publish

Partially fixes https://github.com/Texera/texera/issues/2741 (still need to add a clone button in the detail page).
Demo:
old share access button:

new share access button:

old workspace top bar:

share access button in workspace:

share access button in workspace after click:

@aglinxinyuan please remember to add ddl-change label for PRs that changes the sql.


31 overhaul alm

Codecov Report
Attention: Patch coverage is 99.56710% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 98.18%. Comparing base (8276a40) to head (9479ee2).

Files
Patch %
Lines

django_hierarchical_models/models/node.py
71.42%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #34      +/-   ##
==========================================
+ Coverage   96.77%   98.18%   +1.40%
==========================================
  Files          13        8       -5
  Lines        1366      495     -871
==========================================
- Hits         1322      486     -836
+ Misses         44        9      -35

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