Modify connect-src CSP for DeckGL component

Insert a description of your pull request (PR) here, and check off the boxes below when they are done.

Contributor checklist

[ ] :tada: This PR closes #ISSUE_NUMBER.
[ ] :scroll: I have broken down my PR into the following tasks:

[ ] Task 1
[ ] Task 2

[ ] :robot: I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR.
[ ] :book: I have considered adding a new entry in CHANGELOG.md, and added it if should be communicated there.

Interesting. I guess we might not want to open this up in the default CSP settings (since not all Webviz apps will be using webviz-subsurface-components). Even though a lot of web sites out there in general still don't use CSP at all, we should aim at keeping our CSP settings strict. Snippet from https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/connect-src:

    A scheme such as http: or https:. The colon is required. Unlike other values below, single quotes shouldn't be used. You can also specify data schemes (not recommended).

data: Allows data: URIs to be used as a content source. This is insecure; an attacker can also inject arbitrary data: URIs. Use this sparingly and definitely not for scripts.

Two possible ways forward:

Any other ways webviz-subsurface-components can go ahead with DeckGL without allowing data : in connect-src?
If not - we might want to investigate a way of plugin projects to set CSP settings based on their needs.

For reference: https://github.com/niklasvh/html2canvas/issues/2521


Don't write meta when we should only be generating it

This PR fixes https://github.com/statamic/eloquent-driver/issues/258#issuecomment-1997579873 and https://github.com/statamic/eloquent-driver/issues/251 in a different manner, by going to the root cause of the extra meta rows.
This was being caused by overly aggressive caching (cache vs blink) and the generateMeta method writing meta, which again was a cache optimisation that we have had to roll back.
@jstoks @modrictin @shawinigan I've run this with all the filenames you provided and it works ok for me - can you pull the changes down and verify they work in your environments?

I appreciate the follow up, however we decided to remove Statamic from our application. I won't be able to test this.

Hey @ryanmitchell ,
sorry for the late reply.
This works for me and it has resolved all the issues.
Thanks,
Tin


Made changes so that pages of project open on the same tab

🛠 Fixes Issue
Closes #1251
👨💻 Changes proposed
Made changes so that pages of project open on the same tab
✔ Check List (Check all the applicable boxes)

My code follows the code style of this project.
This PR does not contain plagiarized content.
The title of my pull request is a short description of the requested changes.

📷 Screenshots
https://github.com/SauravMukherjee44/Aec-Library-Website/assets/112184998/6af82d63-96c1-4e25-871c-c557f50de4a6

I have resolved the merge conflict. Please look into this PR.


Output "Final Email HTML" is a mess

How can we reproduce this bug?
Paste the following HTML and CSS into http://foundation.zurb.com/emails/inliner-v2.html
HTML: https://gist.github.com/bbbenji/37ac835de54423463cb601c8e15c6630
CSS: https://gist.github.com/bbbenji/7b64748f1d7c8146c1c82bec23eff0c8
What did you expect to happen?
OUTPUT: https://gist.github.com/bbbenji/949b25436d7e52740c2fb073a06e83e4
What happened instead?
OUTPUT: https://gist.github.com/bbbenji/189b22b9c74918dda40c0cd61daa4666
SCREENSHOT: https://cloud.githubusercontent.com/assets/1678118/20481850/a1e82f88-afe9-11e6-8100-7477b519cad8.png

The inliner might be choking on your custom src/href [iai:] placeholders, have you tried without them? Also, you have an empty style tag before your doctype

@hellocosmin I have removed the custom src/href placeholders as well as the empty style tag, issue still exists.
  seem to have appeared among other issues.
https://cloud.githubusercontent.com/assets/1678118/20485480/f5bde28a-affb-11e6-871a-d1b8a45f73e1.png

Looking at the link you shared (https://gist.github.com/bbbenji/37ac835de54423463cb601c8e15c6630), I see other issues with a span and anchor tag inside the doctype and html elements...
Is that code in the link above, the final output from the inliner? If yes, looks like the inliner has several issues:

  gets transformed to &nbsp; (I think this is an old bug)
href attribute src is populated with a full anchor, i.e.  becomes <a href="">

Let's wait for someone from ZURB to chime in...

@hellocosmin Regarding the span and a tags in the doctype, that was an incorrect paste on my part. I have updated the gist to reflect the correct code. However, this is not related to the issue I am posting about. Will wait for ZURB.

If those span and a tags in the doctype were not the output of the inliner, you should check your HTML, because you have many anchors like that. Check all of it, look for any unclosed tags and attributes too.

@hellocosmin the span and a tags are the output of the inliner. They do not exist in the input file. My last comment was in regards to an incorrect paste of the input HTML which I had corrected already.
No unclosed tags unless you call
 and  unclosed.

Ok, I have just realised the problem here! The issue occurs when clicking the "Copy to clipboard" button.
When manually selecting the output with the cursor and copy pasting, it works just fine, although when using the auto copy button, the code pasted is incorrect.

Glad to hear you found the issue.
I think it would help the guys if you could change the title of this issue to reflect what the actual problem is :)

https://github.com/zurb/foundation-emails/issues/635


Suggestion: AddPropRecursively (or as you like)

Hey! I searched a lot but couldn't find this type to use in my project, so I spent some hours learning more advanced typescript and came with this:
(Maybe there is a type in this lib that does exactly this, but I really couldn't find / understand the Readme enough)
type AddPropDistributive<Obj extends {}, Prop> =
  { [K in keyof Obj]: (Obj[K] extends {} ? AddPropRecursively<Obj, Prop> : Obj[K]) };
type AddPropRecursively<Obj extends {}, Prop> =
  Obj & Prop & AddPropDistributive<Obj, Prop>;

Usage:
You have any type object, let's say
type A = {
  B: number,
  C: {
    D: string
  },
  E: {
    F: {
      G: boolean
   }
}

type X = AddPropRecursively<A, {_new: number}>

X is = {
  _new: number,
  B: number,
  C: {
    _new: number,
    D: string
  },
  E: {
    _new: number,
    F: {
      _new: number,
      G: boolean
   }
}

Hey @SrBrahma! Is this utility type still valuable for you?

Haha I don't! My days of doing these crazy TS stuff are over! :laughing:
Thanks!


Ended task should not be able to start

Note that Monterey knows better :smile: !

 Previously, when you had "jspm install" and "gulp watch" (where gulp watch depends on jspm install) then it would start "gulp watch" when you would cancel "jspm install". But when you cancel jspm install then there is no chance that gulp watch will run successfully. So now when you cancel the "parent" task, it cancels all "children" (and children of children etc)

In light of https://github.com/monterey-framework/monterey/issues/250, should we display cancelled tasks?

should we display cancelled tasks?

Yes, I agree with your reasoning - as long as we can get rid of them eventially.


Ignore case in aliases dll names

The previous code was expecting the passed in name to be a lowercase substring of the dll file name. It's now expected to be the full name (not a substring) but allow the case to differ.
Fixes #1732

Oops binary files, i'll remove & rebase.

Actually let's reopen this PR, I don't know what's the policy @forki
The files are the same as the previous test so don't really take space in git, but I don't know why the files were present in the other one. To accelerate the tests ? to ensure that it's working if nuget remove the old versions ?
(I have no problem changing the PR to not have them)

they are there to keep things stable.

thanks!


Zip file installation failed due to an invalid structure

Could not install jio TV and hotstar in Kodi due to the following error:
Zip file installation failed due to an invalid structure

Issue resolved by reinstalling zip file ..


It still will not download after many tries

Please do not open more issues, I'll be closing this as a duplicate of #55.

Please do not open more issues, I'll be closing this as a duplicate of #55.

Ok sorry, I just really want this to work


 The initial filetype on instruction is redundant as the second filetype call includes the on setting.

Ah, well spotted. Thanks!


Background plugin not compatible with ImagePicker plugin

Ionic version:
[ ] 2.x
[ x ] 3.x
[ ] 4.x
I'm submitting a ...
[ x ] bug report
[ ] feature request
Current behavior:
Background plugin not compatible with ImagePicker plugin
Expected behavior:
ImagePicker  shouldn't return empty array
ImagePicker work good with out Background plugin but when I install Background plugin with or without adding it to app.module.ts ImagePicker return empty array.

You should log an issue against the background plugin repo or the image picker repo as the issue would be with the plugins not with ionic.


Hotfix: sub overflow bug

Description
This pool on Arbitrum (0x64541216bafffeec8ea535bb71fbc927831d0595000100000000000000000002) was throwing a SUB_OVERFLOW error when loading the invest page. It was triggered by exactTokensInForBPTOut being called with zero values. This doesn't usually trigger this error but for some reason in this pool it was. The fix is simply to not call this function if the form has no input values.
Fixes https://sentry.io/organizations/balancer-labs/issues/3788917839
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[ ] Test you can load the invest page for arbitrum/pool/0x64541216bafffeec8ea535bb71fbc927831d0595000100000000000000000002
[ ] Test that investments in weighted/stable pools continue to work as expected.

Visual context
n/a
Checklist:

[x] I have performed a self-review of my own code
[x] I have requested at least 2 reviews (If the PR is significant enough, use best judgement here)
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] If package-lock.json has changes, it was intentional.
[x] The base of this PR is master if hotfix, develop if not

@sssmi this issue appeared after we deployed the boosted pool work so I guess it's caused by the changes to the component structure.


Add Statigraphy / Time terminology selector

Just as users can now select a language, allow them to select the type of labelling for columns from these terminology options

Timescale
Stratigraphy
Both

Selecting Timescale sets headings to: Eon, Era, Period, Epoth, Age
Selecting Stratigraphy sets headings to: Eonothem, Erathem, System, Series, Stage
Selecting Both applies both, as per current chart:

In the future, there may be additional places that this selection alters, such as the Chart's overall title, but for now, I think it's only the headings as above.

Stratigraphic, Timescale & Both implemented


Fix iPad form sheet layout bug

https://github.com/jivesoftware/PDTSimpleCalendar/issues/47

before:

after:

Thanks @IanHoar !


[FIX] Find common stem in find_stem instead of largest common substring

Closes #471. While we should still revisit how images are validated and paths to images are stored, this at least should fix the immediate issue.

Changes proposed in this pull request:

Refactor and clean up find_stem, so that it finds the common stem (or beginning substring) instead of the largest common substring in any part of the set of strings.
Add test for find_stem.

@jdkent I don't think the proposed changes will require a deep review, but I'd appreciate it if you could take a quick look when you have some time.

does one of the tests cover where the function was previously failing?

The third and fourth ones do. In both cases the longest matching substring doesn't come from the beginning of the strings.


First API endpoint draft and required updates to the executor

For Issue: https://github.com/zio/zio-flow/issues/183
PR Status
This is parked for now, some details on the status:
The commented out unit test fails, as the result that is expected to be found when polling does not seem to be completed when it actually should.
The unit test that polls for the result of a simple flow succeeds.
I see two possible explanations for this:

It is due to some things I don't know about the testing in conjunction with ZIO.sleep, Clocks and the environment
Polling is currently broken because of the underlying implementation of how results are retrieved. A cursory look indicates that some Hubs are involved, and if we subscribe to the hub after the result is published, we’ll miss it.

Will be merged as part of https://github.com/zio/zio-flow/pull/224


Airport data parsing fails when a channel >99 is used

When there is a wifi in reach using a channel >99 the parsing the airport output with the title indexOf will result in faulty data. E.g.
                       A Wifi 58:b8:f3:85:55:0e -84  44      Y  DE WPA2(PSK/AES/AES)
                 Another Wifi 5 GHz 58:5e:12:48:35:24 -53  100,+1  Y  DE WPA2(PSK/AES/AES)

will result in
ssid: "Another Wifi 5 GHz 58"
bssid: ":5e:12:48:35:24 -5"
signal_level: "3 10"

It could be fixed using a regexp like this: https://github.com/ancasicolica/node-wifi-scanner/blob/master/lib/airport.js#L10

Hi,
Sorry for the delay. Thank for noticing this bug. I will fix it next week. I am trying to improve the stability of node-wifi and for that I need outputs from different OS. Could you please send a airport output relating to your situation ?

I am sorry, I don´t have the tested wifi around me any more. It was a 5GHz only wifi with a channel >100.


Address transcript number in alleles

SPAC22A12.08c for instance

we had some discussion about this here:
https://github.com/pombase/curation/issues/3309

duplicate of #24


add p0f as example

add p0f extract as one example, while it can only support 5 TCP options and more options would lead to BPF app too complex that failed the Linux BPF verifier.
hope it could be simplified and pass the Linux verifier.
Signed-off-by: Kevin Sun<EMAIL_ADDRESS>

I am approving this PR and initiating the build tests.
It will take few hours to finish. After that, I will revisit this PR and complete the merge process.
Thanks @kevi-sun


Room and basic color support and some minor stuff

Maybe you are interested in that.
I implemented full api support for rooms. That means:

Create/Modify/Delete rooms/lightgroups
When creating rooms with the hue app "Add default scenes" must always be unchecked in order for it to work correctly.
Get information about group list / individual group, including "Group 0".
Group actions: Change lighting for the whole group (no scenes, only on/off/dim and color for real hue lights, in the future color for custom devices)
Note that group actions iterate over all lights in that group, so if one url takes long to execute this is very noticeable.

I tried for everything to give a good response similar to the real hue bridge.
For now the rooms can only be configured through the api and apps, it's not visible/changeable through the web gui. I also removed the default room so one has to create one himself, like with every new hue bridge.
The rooms are getting saved in a "group.db" which is stored at the specified data location. Path can be specified in bridge control below the device.db path.
I tried it with the official hue app and "all 4 hue" and with manual api calls, seems to work. Of course i will test it further.
I also did some minor stuff as described in the commit messages including:

A ColorUrl additional to on/off/dim which gets executed when a color command arrives for that device
Possibility to change the "edit mode" during device edit to manual json input for the urls instead of the table thing for better copy/paste, insertion and reordering capabilities
Filter in device list with: show only devices that are visible to a given ip-address

Edit: What i've done regarding color support can be found in the commit comment

Great work! Just took a quick code review.
So, for groups, you have class types that do not seem to be types that the HUE API would give, they seem more like group names. Here is a snippet of the HUE API that I refer to:
1. Groups in Hue
The hue system has a notion of groups. Grouping of lights can be useful to execute a command for the set of lights within a group. The advantage of using group commands is that you will get a synchronous change of state of the lights within that group, and in the process, eliminate popcorn effect.

To change the state of the lights part of a group, use a PUT on the action endpoint:

/api/<username>/groups/<id>/action

For a description of all the possible actions, please visit:

http://www.developers.meethue.com/documentation/groups-api#25_set_group_state

It is possible to change the color of all lights in a group, or to recall a scene for that group.

A group can consists of different type of lights, and a light will perform the action which will fall within its capabilities, e.g. a “Dimmable Light” will not change in case the group command asks for a “xy” color change.

The hue system has the following types of groups:

LightGroup (group 0 is a special group, i.e. all lights)
Luminaire (created by bridge)
LightSource (created by bridge)
Room (with additional Class specifications)
All detailed, and latest updated, information about groups and the actions on groups can be found back in the description of the API:

http://www.developers.meethue.com/documentation/groups-api

Take into account:

maximum number of groups is 64
LightGroup 0 is a special group created by the bridge. All lights in the system are part of the LightGroup 0. This group can not be created, modified or deleted using the API.
“Luminaire” and “LightSource” are specific groups which are created automatically by the hue bridge, and can not be created, modified or deleted using the API. These groups are created as soon a multi-source luminaire is added to the hue system, e.g. hue Beyond.
As soon a “LightGroup” is empty, it will automatically be deleted from the bridge.
Rooms are specific, in that a lightpoint can not be present in mutiple “Rooms”. And a “room” is not automatically deleted when it is empty.
A scene action will only have impact on the light which are part of that group AND that are part of the lights within the scene.  A scene action for LightGroup 0 will have impact on lights part of the scene.
A light cannot have its hue, saturation, brightness, effect, ct or xy modified when it is turned off.
There are 3 methods available to set the color of the light – hue and saturation (hs), xy or color temperature (ct). If multiple methods are used then a priority is used: xy > ct > hs. All included parameters will be updated but the ‘colormode’ will be set using the priority system.
In case your application makes use of a room-based approach to control IoT devices in the home, it is best to re-use the rooms created using hue app gen 2.

Great work! Doing a code review. Deleted my comment about group types as I found it in the group api page.

 IT seems to have broken the sorting. I am making some changes so I will fix that. Other wise this looks good. Thanks!

Oh sorry, forgot that these changes where in there, should have excluded that.
I did these changes intentional for myself. Wanted it always sorted by name and without the height constraint on the table, because i liked it more if i see everything instead of that small height.
inside scrollable-table.css i removed the fixed 310px height for .scrollableContainer
in the configuration.html in the ng-repeat for the device list is a constant orderBy:'Name'
These should be the only changes in that "personal" way.
Except for maybe the column min/max widths in editeevice.html as commented in this issue. Not sure if this is good for everyone.

Removing the height property was satisfying enough for me. I want to use my entire screen and not a small portion of it and i prefer scrolling with the main scrollbar and not in multiple separate boxes (like in editdevice with fixed height for the scrolltable)
Adding the orderBy:'Name' does sort my list by name from A-Z for me, but it disables the sorting function of the scrolltable. Would be better if we could somehow access the current sorting of the table and modify it. Or if the table would always remember the last sorting, even for new tabs. (But i wasn't so keen for that to actually invest more time, so i did this simple "fix" for me)

Yeah, definitely a default sort would be great. I have a request for that to the author of that table, but it is not going anywhere. Maybe I will fork it and fix it.

So, we have an issue. There is a piece of code that is breaking people's setups in the HueMulator changeState method. It is the if statement that checks for lightIds that are greater than 10000. I believe that is not needed any more. Comments?

This has somethting to do with rooms/groups regarding Alexa.
I did implement the full room functionality, you can create rooms in the hue app, assign lights to it and execute all the room stuff which works just fine with apps.
However, i discovered that Rooms are handled via the hue skill in alexa, so it will not discover HA-Bridge Rooms on its own.
The thing with lightid > 10000 was a workaround i did in this commit to get my rooms into Alexa.
It applied the following logic: If someone requested the lights list it appends all rooms for which this is enabled as additional fake lights with light id 10000 + groupid. If one tries a GET or PUT to a light or its state with lightid > 10000 then it's assumed that a room is meant and extracts the corresponding groupid and continues with that.
This feature has to be enabled for every room by editing the group.db entry with "exposeAsLight":"echo-ip-address" (Since there is no web ui for group editing).
I assumed that lightids are assigned automatically and continuously, so that it should be impossible for real lights with such lightids to exist. Apparently this assumption was wrong?

The unfortunate thing is, before I changed the numbering, they were random and above 10000. So people are having issues with that.

Ok, didn't knew that. I based my assumption on the current tech state. So we either have to undo these code changes if this is to big of a problem or tell everyone to renumber their devices for compatibility with v5.

How about we put in a setting to use groups and if you do, you will have to renumber of ids area above 10000.

Yes this would be a possibility. Per default turned off. I would name this setting something like "Enable rooms for Alexa" or something like that, since rooms in general do function without that lightid > 10000 stuff, so no reason to turn that off. It's only necessary if you want to enable a room to be visible for Alexa.

I think this will be needed as people really have a lot of device they carried forward overtime and trying to keep this backward compatible.

@bwssytems
I saw that you started to implement the "Enable rooms for Alexa" setting.
I also saw that you applied the setting to everything group related. This is not really necessary since the problematic id > 10000 stuff is not needed for the entire rooms feature. Rooms with apps work just fine, the id > 10000 stuff is only a workaround to get the rooms displayed in Alexa without the skill.  That's the reason why i suggested the name "Enable rooms for alexa" and not just "Enable rooms".
Also, if you apply the setting to everything group related you would have to provide all lights in a default group - like it was before - in order for habridge to be usable with apps (otherwise you would get a blank screen). I might be wrong but i think i haven't seen that in your recent changes?
I would suggest to only toggle the alexa workaround related stuff with this setting (this means everything where 10000 is mentioned) and to keep the rooms feature itself intact.

That's interesting. Do you mean this?
if (path.endsWith("/")) { // it should work with or without a trailing slash
    response.redirect(path.substring(0, path.length() - 1));
}

If I remember correctly I added this because without it some apps I tried didn't work (not the official hue app though) because they called it with a trailing slash and habridge didn't  know what to do with it.

Yeah, that is it. What calls are you having issues with?


Add CF-Community repository for installing the plugin

Add -r CF-Community to the cf install-plugin command in the readme

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

@sclevine I wasn't aware of that 😄  I just checked and I'm still on version 6.25.0+787326d95.2017-02-28 which explains why I ran into difficulty.
Since this is fixed in the latest CLI version you can probably just close this PR if you want to. I was just worried that folks just getting started on CF wouldn't know how to resolve this if this was happening on the latest version!

If you'd like to modify the PR to add an extra section to the README.md for CLI versions < v6.27.0, I would definitely merge it. Otherwise I'll wait to see if other users run into the same issue, and make the change then.

@sclevine How's this? Feel free to hold off on merging or close this for now if you want to wait to see if others report this issue!

Can you change the wording to Furthermore, if..., so it's clear that the "outdated" part refers to the plugin version, not the CLI version?

@sclevine done!

Sorry, one last thing: the option is -r, not -R, and it comes after the plugin name by the docs:
cf install-plugin PLUGIN_NAME [-r REPO_NAME] [-f]

Or I can merge it and add a commit on top when my plane lands, either way.

Sure. If it’s still open on Monday I can fix it, otherwise feel free to
either merge and fix or just push a commit directly and close this!
On Fri, 13 Oct 2017 at 17:01, Stephen Levine<EMAIL_ADDRESS>wrote:

Or I can merge it and add a commit on top when my plane lands, either way.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sclevine/cflocal/pull/6#issuecomment-336494902, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AG7UzHGSzEEzJOcbjoZkxo9aRWiGbXAiks5sr4lVgaJpZM4P4T5P
.

@sclevine Totally forgot about this until now, but I updated the PR following your comments in case this is still useful, otherwise feel free to close!

Sorry for the delay, merging now!


Tell user to use PASS / instead of just the username.

In versions prior to 1.0, ZNC did not allow multiple networks per user. The string telling users to connect has never been changed after ZNC 1.0, and this fixes just that.

<user>:<pass> still works. It uses network named "default" by default, if it exists. If such network doesn't exist, it uses just some network.
<user>/<net>:<pass> has more variables to substitute == more things for user to do wrong :(

How about just saying Set password as <username>/<network>:<password> in your IRC client and reconnect and forgetting that manual /pass?

We could do both, changing it into two messages.
You need to send your password. Configure your client to send a server password.
To connect now, you can use /quote PASS <username>:<password>, or /quote PASS <username>/<network>:<password> to connect to a specific network.

it uses just some network.

Is that deterministic or....?

:+1: for @Zarthus's both messages.

@md-5 https://github.com/znc/znc/blob/600f91f128802ea6225d457b7247aba4a4119362/src/Client.cpp#L690

Definitely agree. This is something that should have been adjusted many versions ago :+1:

@Zarthus may you update the pull request?

@DarthGandalf Done, was there anything else that needed doing?


Review Material of where last project was stopped

past a a doc of SOP
type a new project for what to do next
share project on linked in

Working on updating a wiki and posting to linked in


IN PROGRESS: Add support for PowerShell Editor Services

This change enables basic integration of PowerShell Editor Services with the PowerShell plugin to provide rich editing functionality for PowerShell scripts.  This change adds code completions (IntelliSense), syntax and semantic markers, basic script execution, and some level of PowerShell-based extensibility to Sublime Text.
I'm sending this PR out before it's fully done so that we can talk about the implementation details.  The code is not "production quality", there are a lot of TODOs around and some code is commented out.  I'll clean all of that up before merging.  The most important thing is to take a look at the code design and let me know if you see any big problems.
I would love it if someone could try this out and see if it works for you.  I've included installation instructions in the README.md file.
Let me know what you think!
⚠  Issues to be fixed before merge ⚠
Here's a list of known issues that I still need to fix before this PR is merged:

[ ] IntelliSense sometimes inserts the incorrect substring.  This will be corrected soon.
[ ] There are no keybindings for the commands that come with the plugin.  We are investigating the issue.
[ ] Can't hover over syntax/semantic markers in the script and see the message.  This is a limitation in  Sublime Text's APIs and should be resolved in an upcoming Sublime update.
[ ] PowerShell process do not get closed when Sublime exits.
[ ] Clean up the code (remove or file bugs for TODOs, get rid of commented out sections)

I got the following message on startup in OSX
2016-08-28 22:06:21,578:<PHONE_NUMBER>35168: INFO: PowerShell plugin starting...
2016-08-28 22:06:21,581:<PHONE_NUMBER>35168: DEBUG: PowerShell Editor Services log path: /var/folders/h4/hwtcfdnd7_718x5hzvl12y180000gn/T/1472418381-EditorServices.log
2016-08-28 22:06:21,582:<PHONE_NUMBER>35168: DEBUG: Using bundled modules path: /Users/jason/Library/Application Support/Sublime Text 3/Packages/PowerShell/editorservices/modules
2016-08-28 22:06:21,582:<PHONE_NUMBER>35168: DEBUG: Command parameter arg: & "/Users/jason/Library/Application Support/Sublime Text 3/Packages/PowerShell/editorservices/Start-EditorServices.ps1" -EditorServicesVersion "0.7.1" -HostName "Sublime Text Host" -HostProfileId "SublimeText" -HostVersion "3.0.3114" -LogLevel "Verbose" -LogPath "/var/folders/h4/hwtcfdnd7_718x5hzvl12y180000gn/T/1472418381-EditorServices.log" -BundledModulesPath "/Users/jason/Library/Application Support/Sublime Text 3/Packages/PowerShell/editorservices/modules"
2016-08-28 22:06:22,764:<PHONE_NUMBER>35168: DEBUG: Launch response: b'needs_install\n'
Traceback (most recent call last):
  File "./python3.3/json/decoder.py", line 367, in raw_decode

But I did install the modules
✔ ~/Library/Application Support/Sublime Package 3/Packages/PowerShell/editorservices/modules [editor-services|…25]
22:41 $ tree
.
├── PSScriptAnalyzer
│   ├── Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.dll
│   ├── Microsoft.Windows.PowerShell.ScriptAnalyzer.dll
│   ├── PSScriptAnalyzer.psd1
│   ├── PSScriptAnalyzer.psm1
│   ├── PSv3
│   │   ├── Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.dll
│   │   └── Microsoft.Windows.PowerShell.ScriptAnalyzer.dll
│   ├── ScriptAnalyzer.format.ps1xml
│   ├── ScriptAnalyzer.types.ps1xml
│   ├── Settings
│   │   ├── CmdletDesign.psd1
│   │   ├── DSC.psd1
│   │   ├── PSGallery.psd1
│   │   ├── ScriptFunctions.psd1
│   │   ├── ScriptSecurity.psd1
│   │   └── ScriptingStyle.psd1
│   ├── coreclr
│   │   ├── Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.dll
│   │   └── Microsoft.Windows.PowerShell.ScriptAnalyzer.dll
│   └── en-US
│       ├── Microsoft.Windows.PowerShell.ScriptAnalyzer.dll-Help.xml
│       └── about_PSScriptAnalyzer.help.txt
└── PowerShellEditorServices
    ├── PowerShellEditorServices.psd1
    ├── PowerShellEditorServices.psm1
    └── bin
        ├── Desktop
        │   ├── Microsoft.PowerShell.EditorServices.Host.dll
        │   ├── Microsoft.PowerShell.EditorServices.Protocol.dll
        │   ├── Microsoft.PowerShell.EditorServices.dll
        │   └── Newtonsoft.Json.dll
        └── Nano
            └── Microsoft.PowerShell.EditorServices.Nano.dll

I ended up copying the Packages folder to another folder that didn't contain a space, and hacked your code to point to the new location, then it worked great!

Damn, thanks for catching that!  After having been bitten by this a couple times already I really should know better ;)

Oh, and thanks a lot for giving it a try!

Completion seems a little iffy though... is that a problem with the Sublime code or the backend server?

e.g.

Guess this has to be a sublime issue - if the completions are returned correctly, the server did it's job.

Ahhh yeah, looks like this particular case may be a bug in the logging I'm using there.  Will get that fixed :)
In general though, the completions can be a little wrong at times because of the way I have to translate the completions into buffer edits.  Sublime's API assumes that the completion will literally just be the rest of the symbol name after the cursor but PowerShell often returns completions that actually replace some of the existing text.  I'm having to do some extra work there to manipulate the buffer after the completion is committed so it's not 100% reliable yet.
Did you have to do a similar thing for OmniSharp?  I know the TypeScript package for Sublime has to do it this way, I've been learning a lot from their implementation.

I didn't work on the OmniSharp sublime plugin (apart from some guidance), so not sure. That API sounds a bit sucky  though :) I did work on emacs, vim and Atom intellisense though, and none of those had that issue - they just did the right thing.

Sublime's API is the most inflexible that I've seen, hoping that the other editors will be less challenging ;)

Just remembered that with all those editors, you need to tell the editor what the 'prefix' is - the part of the line that you are completing on / part to be replaced

Hmmm, is that something I have to know in advance before resolving completions?  Am I supposed to use the editor's tokenization of the source to pre-determine the prefix string, or can I give it the prefix after the user has selected the completion they want?

Vim prefix lookup code just uses a regex - https://github.com/OmniSharp/omnisharp-vim/blob/master/autoload/OmniSharp.vim#L17-L28

Not sure if I'm missing something, but I can't seem to get the Editor Services up and running...
I've got the module installed:
PS /Users/dmoore44/Documents> ./Setup-EditorServices.ps1
/Users/dmoore44/Documents/Setup-EditorServices.ps1 : The PowerShell package already exists at the following path: /Users/dmoore44/Library/Application Support/Sublime Package 3/Packages/PowerShell.
We do not automatically delete this path in case you need to back up your current installation.
Please move or delete this path before continuing.
  If you move Sublime Package 3/Packages/PowerShell to /Users/dmoore44/Library/Application Support/Sublime Text 3/Packages does it work?

@daviwil You're correct - changing the path to ../Sublime Text 3/Packages works!  I was wondering if that would be the solution, but had to step away for a bit.
No worries on the install script - I have to say that I'm quite impressed with the level of functionality on an "Alpha" project.

Phew, glad to hear it :)  I'll fix that script so that the next person doesn't hit the same issue.


[POC] CS - force blank line before return

See #14658

force blank line before return

why? I never understood the reasoning behind this. Makes code look weird in a lot of cases.

seems you agree with me: https://github.com/yiisoft/yii2/pull/14658#issuecomment-323025765 ? :)

Yes. Many changes from this PR are valid and could be merged, but as a hard rule it does not have any sense to me.

I'd vote to not apply it, it does not really help and makes a lot of code look weird if its forced to be applied everywhere.

I could reduce changes to "add empty line before return if it is preceded by }" - this will separate if blocks and return statements (which IMO always increase readability).

See #14682


Package the project with nix flake init --template templates#rust

Great, merging. Thanks!
Note that I don't use Nix myself (yet!) and I can't really maintain this Nix config, so in case of any issues with this you will be pinged :) I hope that's ok.

@yvan-sraka  That ubuntu build fails: https://github.com/asciinema/agg/actions/runs/4589523213/jobs/8104469344#step:4:8

@yvan-sraka That ubuntu build fails: https://github.com/asciinema/agg/actions/runs/4589523213/jobs/8104469344#step:4:8

I created a new PR that try to address the issue: https://github.com/asciinema/agg/pull/37


Compiled Statements with Object as parameter

I understand the reason why you implemented Compiled Statements instead of marshal(). But why don't provide a bind method with an Objects instead of all fields?

duplicate of #572


CI/CD: need to restore NuGet publish functionality

The current FAKE script has a lot of hard-coded defaults that point to Lutando's original repository and feeds. Need to change these to:

Point to the AfterLutz NuGet organization's credentials, which can / should be stored as a secret in the repository so it can be accessed via Github Actions
Publish a Github Release each time a tag is pushed, which is standard practice.

https://github.com/AfterLutz/Akkatecture/issues/34#issuecomment-1004847924 - done


pass certain env vars during elevation

Description

Comes from https://github.com/conda/menuinst/issues/127#issuecomment-1623155201
Checklist - did you ...

[ ] Add a file to the news directory (using the template) for the next release's release notes?

[ ] Add / update necessary tests?
[ ] Add / update outdated documentation?

pre-commit.ci autofix


[epoch] Add get epoch and fix get dex twaps

Still working on testing, but this introduces get epoch functionality into the sei-cosmwasm package

update: tested the epoch functionality

also tested dex twaps


make install gone?

It appears make install has been removed, which breaks the AUR package build.
Also make binaries is gone, and make tests apparently does nothing?
Note that AUR PKGBUILDS have the following general structure:
build() {
	cd "$srcdir/${pkgname%-git}"
	# Make all depends on tests, but we want to run them in check() instead.
	make binaries
}

check() {
	cd "$srcdir/${pkgname%-git}"
	make tests
}

package() {
	cd "$srcdir/${pkgname%-git}"
	make install PREFIX=/usr DESTDIR="$pkgdir/"
	install -Dm 644 COPYING.md "$pkgdir/usr/share/licenses/$pkgname/COPYING.md"
	# Installing docs is not yet supported in a reasonable way by upstream.
}

Since you have your own home grown build system it is kind of hard to figure out how to do this now after the recent changes.

Oh, whoops. I forgot you were using that. Will fix.

Hm, doesn't actually seem to work yet:
==> Starting package()...
INSTALL
'.obj/src+fluxengine/src+fluxengine' -> '/build/fluxengine-git/pkg/fluxengine-git//usr/bin/fluxengine'
install: cannot create regular file '/build/fluxengine-git/pkg/fluxengine-git//usr/bin/fluxengine': No such file or directory
make: *** [Makefile:59: install-bin] Error 1

Not sure if it is on my side or your side, but too late for me to look at it today.

Whoops. Forgot to add the -D to create the directory. Fixed (or will be as soon as the build completes).


Pressing shift+Enter in chat sends the text to terminal

Which API Provider are you using?
OpenRouter
Which Model are you using?
GPT 4o-mini, Sonnet 3.5, Qwen-2.5
What happened?
I pressed shift+Enter to break the line, and the text was sent and executed in the terminal, the same way it happens when I'm editing Python code. However, using a different extension - Continue.Dev - pressing shift+Enter simply breaks the line without other effects.
Steps to reproduce

Cline chat side bar open
Write text
Press shift + Enter
the text is sent and executed in the terminal window

Relevant API REQUEST output
No response
Additional context
MacOS 13.6.9, VSCode 1.94.2, Cline 2.1.1

My list of shortcuts associated with shift + Enter might be useful:

I am not able to repro this. I think it might be one of your keyboard shortcuts, can you try remove the shortcut for interactive.execute?


GLOBALS is not defined

api.tracker.globals   = GLOBALS;
Uncaught ReferenceError: GLOBALS is not defined. What should I be doing to fix this?

Using the example from  @KartikTalwar, I kept getting:
Uncaught ReferenceError: Gmail is not defined
I had to add the following into my manifest:
"permissions": ["activeTab"]
And that seemed to fix it.

Hello KartikTalwar,
it always display error when call var gmail = new GMAIL() from other js files. when i call it from the main.js file then it is working , but it is not working for my file.
I put a same code which is in main.js file , but doesn't work, it display a error like GLOBALS is not defined.
Note: I am using a angularjs

Hello @KartikTalwar
It is necessary to inject all the js file in to the load.js which you describe above ,  i can not use without that ?
becuase in my extension there is lots of js files is included.

Not sure if I am thick or what, but from KartikTalwar's example above, load.js the line:
port.postMessage({ type: "save_email_id", id: event.data.message_id});
What is port?
--Uncaught ReferenceError: port is not defined

@mmccallum looks like I missed a line in the example.
var port = chrome.runtime.connect();

https://developer.chrome.com/extensions/content_scripts#host-page-communication

Thanks.  Still not sure if I'm thick no not knowing what it should have been.  No one else noticed something missing for a year and a half?  :)


International sharing

@ncase The titles (and therefore the tooltips) of the sharing buttons were not included in the translation so far. I completely understand if this change is too minor to include it.
btw: If the <sharing> tag has a function beyond having the text somewhere parsable, I missed it. I hope it was ok to add more attributes there.

Yes, it's necessary changes


Minimal example?

Dear Jannik @jmbuhr,
The extension looks very fun, however, I can't make it work on my quarto slides. Do you mind providing a minimal qmd example to illustrate how to use it? Should I add - qrcode to either filter or extension in the YAML? (Neither works on my side, btw)
Thanks!

If https://github.com/jmbuhr/quarto-qrcode#installing and the example (index.qmd) is not working I'd be happily review your example in which it doesn't work.


chore: Run pre-commit hooks

@bryantbiggs What do you think we can do to be able to pass checks in examples?
I am personally in favor of bumping versions requirements of many modules to be 0.13 but the code actually works with 0.12 too, so it does not feel completely ok to bump it without real need. What do you think?

I've been trying to upgrade to 0.12.26 which has a lot of experimental support without making the leap to 0.13.x and its worked for most cases - however, maybe we are trying to support too much at once. I have this PR where the examples fail due to a 0.13.x feature but the module works fine on 0.12.26 https://github.com/terraform-aws-modules/terraform-aws-route53/pull/31

Terraform 0.12.26 seems to be the easiest solution to make big dependencies/users happy. Resolved in #70


After deployment, bucket returns 404 and API returns 500

Bucket

API

Seems like the lambda function the SESSION_SECRET env variable.
"errorType": "Error",
"errorMessage": "Invariant failed: SESSION_SECRET must be set"

Did you do this step in the README?

I did yes, successfully too.
Though for some reason it wasn't added to the lambda function.
Just added it manually and it works 🥳
Great work with Remix! Love it!

Glad you got it working :)

Yea, I have the same issue... Seeing arc env variables being added, but not being set in the lambda.

Did you do this step in the README?

I ran these commands locally but I'm a little confused as to how to "give your CloudFormation a SESSION_SECRET variable..."
Should the deploy.yml file be updated to have:
SESSION_SECRET: ${{ secrets.SESSION_SECRET }}
ARC_APP_SECRET: ${{ secrets.ARC_APP_SECRET }}

Under the relevant staging/production deploy? Or is this something I need to add to the stack somehow on AWS?

Environment variables created with arc env --add are persisted between deployments in the same CloudFormation "stack".
So if you set them from your local machine, and your GitHub Action deploys to that same target, your Action does not need to set them again.
Note that if you change the region or name of your project in app.arc you will create a new Stack and will need to set the environment variables again.

Ah, that makes sense, thank you very much!


Titles don't show up on 1.8

This code works perfectly fine, it just only shows up for players using 1.12+ clients
Only tested [1.8, 1.12, 1.14.4]

Just realized that travertine only supports 1.7, closing lol...

Wrong, Travertine supports 1.7.x - 1.14.x

Would more be https://github.com/PaperMC/Waterfall, would also be interesting to know if this occurs on bungee too

It appears setting fadeIn, stay and fadeOut fixed this. It might also be good to note that it seemed to break randomly.


remove lazy initialize

Fix #308

@xoofx would you help to release a new version for us to reference if the change is ok for you?


Linker Error when building arrow bundled dependencies

Describe the bug, including details regarding any error messages, version, and platform.
I was trying to do a static build of arrow in my project and ran into a linker error. Here are the exact commands I ran:
git clone https://github.com/apache/arrow.git
cd arrow/cpp
mkdir build
cd build
cmake -DARROW_PARQUET=ON -DARROW_CSV=ON -DARROW_BUILD_STATIC=ON ..
cmake --build . --target install

After running those commands, during the build process, I get the following error.
  Performing install step for 'thrift_ep'
  -- thrift_ep install command succeeded.  See also C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/cpp/build/thrift_ep-prefix/src/thrift_ep-stamp/th
  rift_ep-install-*.log
  Completed 'thrift_ep'
  Bundling C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/cpp/build/release/arrow_bundled_dependencies.lib
LINK : fatal error LNK1370: failed to copy file 'C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\cpp\build\thrift_ep-install\bin\thriftmd.lib'; error
 code 3 [C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\cpp\build\src\arrow\arrow_bundled_dependencies_merge.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(247,5): error MSB8066: Custom build for 'C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\cpp\build\CMakeFiles\155e07f2be9d7893eea39b427b71fbbb\arrow_bundled_dependencies_merge.rule;C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\cpp\src\arrow\CMakeLists.txt' exited with code 1370. [C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\cpp\build\src\arrow\arrow_bundled_dependencies_merge.vcxproj]

Any help would be greatly appreciated.
Component(s)
C++

Could you specify -DCMAKE_BUILD_TYPE=Release explicitly?
git clone https://github.com/apache/arrow.git
cd arrow/cpp
mkdir build
cd build
cmake -DCMAKE_BUILD_TYPE=Release -DARROW_PARQUET=ON -DARROW_CSV=ON -DARROW_BUILD_STATIC=ON ..
cmake --build . --config Release --target install

I ran each of those commands specifically while explicitly stating the build type and It seems to produce the same error.
  Performing install step for 'thrift_ep'
  -- thrift_ep install command succeeded.  See also C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/thrift_ep-prefix/src/thrift_ep-stamp/thr
  ift_ep-install-*.log
  Completed 'thrift_ep'
  Bundling C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/release/arrow_bundled_dependencies.lib
LINK : fatal error LNK1370: failed to copy file 'C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\thrift_ep-install\bin\thriftmd.lib'; error
code 3 [C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\src\arrow\arrow_bundled_dependencies_merge.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(247,5): error MSB8066: Custom build for 'C:\Users\gu
ppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\CMakeFiles\294c83af6aef239dd1ca2299792d917b\arrow_bundled_dependencies_merge.rule;C:\Users\guppy\CLionP
rojects\NewDataframes\libs\arrow\arrow\cpp\src\arrow\CMakeLists.txt' exited with code 1370. [C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build
\src\arrow\arrow_bundled_dependencies_merge.vcxproj]

Could you show the outpuf of dir C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\thrift_ep-install\bin\?

This is the output:
 Directory of C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\thrift_ep-install\bin

06/15/2023  05:35 PM    <DIR>          .
06/15/2023  05:35 PM    <DIR>          ..
06/15/2023  05:35 PM         4,586,458 thriftmd.lib
               1 File(s)      4,586,458 bytes
               2 Dir(s)  289,212,059,648 bytes free

Hmm... Could you add a debug message and run again?
diff --git a/cpp/cmake_modules/BuildUtils.cmake b/cpp/cmake_modules/BuildUtils.cmake
index aaf4b0bb5..3bf44c4d2 100644
--- a/cpp/cmake_modules/BuildUtils.cmake
+++ b/cpp/cmake_modules/BuildUtils.cmake
@@ -140,7 +140,7 @@ function(arrow_create_merged_static_lib output_target)
                     ${BUNDLE_COMMAND}
                     DEPENDS ${ARG_ROOT} ${ARG_TO_MERGE}
                     BYPRODUCTS ${output_lib_path}
-                    COMMENT "Bundling ${output_lib_path}"
+                    COMMENT "Bundling ${output_lib_path}: ${BUNDLE_COMMAND}"
                     VERBATIM)

   message(STATUS "Creating bundled static library target ${output_target} at ${output_lib_path}"

Here's the output from that comment:
Performing install step for 'thrift_ep'
  -- thrift_ep install command succeeded.  See also C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/thrift_ep-prefix/src/thrift_ep-stamp/thr
  ift_ep-install-*.log
  Completed 'thrift_ep'
  Bundling C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/release/arrow_bundled_dependencies.lib: C:/Program Files/Microsoft Visual Studio/
  2022/Community/VC/Tools/MSVC/14.34.31933/bin/Hostx64/x64/lib.exe;/NOLOGO;/OUT:C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/release/arro
  w_bundled_dependencies.lib;C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/thrift_ep-install/bin/thriftmd.lib
LINK : fatal error LNK1370: failed to copy file 'C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\thrift_ep-install\bin\thriftmd.lib'; error
code 3 [C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\src\arrow\arrow_bundled_dependencies_merge.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(247,5): error MSB8066: Custom build for 'C:\Users\gu
ppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\CMakeFiles\294c83af6aef239dd1ca2299792d917b\arrow_bundled_dependencies_merge.rule;C:\Users\guppy\CLionP
rojects\NewDataframes\libs\arrow\arrow\cpp\src\arrow\CMakeLists.txt' exited with code 1370. [C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build
\src\arrow\arrow_bundled_dependencies_merge.vcxproj]

Could you run C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.34.31933/bin/Hostx64/x64/lib.exe /NOLOGO /OUT:C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/release/arrow_bundled_dependencies.lib C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/thrift_ep-install/bin/thriftmd.lib?

Hopefully, I ran this correctly, but I'm uncertain because I got an error. I navigated to the folder
C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.34.31933\bin\Hostx64\x64>

and then ran the following command:
lib.exe /NOLOGO /OUT:C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/release/arrow_bundled_dependencies.lib C:/Users/guppy/CLionProjects/NewDataframes/libs/arrow/arrow/cpp/build/thrift_ep-install/bin/thriftmd.lib

and i got this error:
LINK : fatal error LNK1370: failed to copy file 'C:\Users\guppy\CLionProjects\NewDataframes\libs\arrow\arrow\cpp\build\thrift_ep-install\bin\thriftmd.lib'; error code 3

Oh... We can't get more information...
Sorry. I can't help you more without reproduced environment...
Could you debug the command line on your environment?

@kou I got the same exact same error on a fresh windows install


Fix unecessary security groups added to the ELKLaunchConfig

I haved added those security groups by mistakes. They are not necessary.
@satterly

👍


change Base64Encoder

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#61  👈
#60
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @malezjaa and the rest of your teammates on  Graphite


Add test for Mac vs Linux test count. Randomize test order on Linux. …

…IBM-Swift/Kitura#1056
Description

Add a test class that checks if every other test class has included all their tests in their allTests var
Add extensions in LinuxMain.swift to shuffle an array, and shuffle the order that the test classes are run, as well as the tests within the classes

Motivation and Context
#1056
How Has This Been Tested?
OSX: Swift 3.1
Linux: Swift 3.1 & Swift 3.0.2

Codecov Report

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

@@          Coverage Diff           @@
##           master     #68   +/-   ##
======================================
  Coverage    90.4%   90.4%
======================================
  Files           1       1
  Lines         198     198
======================================
  Hits          179     179
  Misses         19      19

Flag
Coverage Δ

#KituraSampleRouter
90.4% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix typo

Quick typo fix in jquery.big-slide.js

hahahaha, of all the places to make a typo! Thanks.


Compatibility with whiteboards

Hi,
Thanks for this great plugin! I noticed two issues when using it with whiteboards. Hope they can get fixed soon.
Info:

OS: Windows 11
Logseq ver: 0.8.18
Plugin ver: 1.15.10
Theme name: Default Light Theme

Description:
(1) Favorites icons for whiteboards are overridden by the defaultPageProps icon, which is not desirable
(Awesome Links disabled)
(Awesome Links enabled)

(2) Whiteboard title texts are truncated when Awesome Links is enabled.
 (Awesome Links disabled)
 (Awesome Links enabled)

(1) Favorites icons for whiteboards are overridden

just go to settings and delete the default page config )

Now no icon is displayed after deleting the default page config...

(2) Whiteboard title texts are truncated

fixed


Is touch /var/run/reboot-required wiser then systemctl restart k3s?

https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/cf2ef6bbac78097015cd969c71882145b534df68/modules/host/main.tf#L199
The docs at https://docs.k3s.io/installation/private-registry state:
"In order for the registry changes to take effect, you need to restart K3s on each node."
if we replace
touch /var/run/reboot-required

with
if systemctl status k3s > /dev/null; then
  systemctl restart k3s;
elif systemctl status k3s-agent > /dev/null; then
  systemctl restart k3s-agent;
fi

would this change be a good or a bad idea?

@valkenburg-prevue-ch It's unwise because we need to node to be drained properly before restarting k3s on it. So by touching reboot-required, kured does all that magic for us.

There is another possibility, maybe we can create another system-upgrade-controller plan, just for that, and activate it via labels. And it is extremely suited for working safely with a k3s/rke2 node.

Thanks for explaining, I thought so much!
I would hope that workloads continue even if the k3s service is restarting for a few seconds. I mean, k3s is the orchestrator, but hopefully is not involved in every bit of communication of the cluster? I don't know what I'm talking about, this is just my story-telling understanding of k8s.

I cannot find info about this so quickly. Surely someone with more expertise knows the way to the answer.

Actually, the system-upgrade-controller has the possibility to execute commands, but we must double-check with their team if it's safe to just restart k3s. Also, it is very important to use it to make sure it does one node at a time, starting with control planes of course.

We would basically force a k3s upgrade to the same version (if possible), and that would cause the system upgrade controller to do all the magic, including the draining and k3s restart for us.
An interesting starting point could be https://docs.k3s.io/upgrades/automated and our own current mechanism to upgrade based on checking the version channel https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/master/templates/plans.yaml.tpl

https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/cf2ef6bbac78097015cd969c71882145b534df68/modules/host/main.tf#L199
The docs at https://docs.k3s.io/installation/private-registry state: "In order for the registry changes to take effect, you need to restart K3s on each node."
if we replace
touch /var/run/reboot-required

with
if systemctl status k3s > /dev/null; then
  systemctl restart k3s;
elif systemctl status k3s-agent > /dev/null; then
  systemctl restart k3s-agent;
fi

would this change be a good or a bad idea?

https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/cf2ef6bbac78097015cd969c71882145b534df68/modules/host/main.tf#L199
The docs at https://docs.k3s.io/installation/private-registry state: "In order for the registry changes to take effect, you need to restart K3s on each node."
if we replace
touch /var/run/reboot-required

with
if systemctl status k3s > /dev/null; then
  systemctl restart k3s;
elif systemctl status k3s-agent > /dev/null; then
  systemctl restart k3s-agent;
fi

would this change be a good or a bad idea?

@valkenburg-prevue-ch, coming full circle here, this shows that your proposed method is actually safe, as is. PR welcome! 🚀

Thanks for explaining, I thought so much!
I would hope that workloads continue even if the k3s service is restarting for a few seconds. I mean, k3s is the orchestrator, but hopefully is not involved in every bit of communication of the cluster? I don't know what I'm talking about, this is just my story-telling understanding of k8s.

Thanks for explaining, I thought so much!
I would hope that workloads continue even if the k3s service is restarting for a few seconds. I mean, k3s is the orchestrator, but hopefully is not involved in every bit of communication of the cluster? I don't know what I'm talking about, this is just my story-telling understanding of k8s.

You were right! That seems to be the way it actually works... I had no idea myself lol 🎯

So the orchestra keeps on playing when the conductor goes to the bathroom to ...

Exactly lol

There is another possibility, maybe we can create another system-upgrade-controller plan, just for that, and activate it via labels. And it is extremely suited for working safely with a k3s/rke2 node without requiring a reboot.
FYI @captnCC, if we can remove the need for reboot, it would be awesome!

@mysticaltech While working on the helm transition for v2, I was already thinking about consolidating the upgrade behaviours for k3s and the system with system-upgrade-controller plans, to make it all a cohesive configuration

@captnCC I understand the idea, but for the system kured does it automatically and has a whole system to do it well, it came from them, even the interaction with kured. So best left independent.
The beauty of the system upgrade controller upgrade of k3s is that it doesn't need a reboot. But for the OS, a reboot is needed to apply the new snapshot.


App crash on using getAllowedApps() this funcation

Hello
I try to get all installed apps list so app is crash when I call this function
 List<String> allowedApps = await sstpFlutter.getAllowedApps();
  print('Allowed Apps: $allowedApps');

Help me

Hello ,
please provide more info about the exception.
also you are trying to retrieve the allowed apps to use your vpn, not all installed apps.
for get installed apps do this :
// Get installed apps
List installedApps = await sstpFlutter.getInstalledApps();
print('Installed Apps: $installedApps');

Hello , please provide more info about the exception.
also you are trying to retrieve the allowed apps to use your vpn, not all installed apps.
for get installed apps do this :
// Get installed apps List installedApps = await sstpFlutter.getInstalledApps(); print('Installed Apps: $installedApps');

Same Problem when i call await sstpFlutter.getInstalledApps();

Hello , please provide more info about the exception.
also you are trying to retrieve the allowed apps to use your vpn, not all installed apps.
for get installed apps do this :
// Get installed apps List installedApps = await sstpFlutter.getInstalledApps(); print('Installed Apps: $installedApps');

Any update sir

Hello Dear,
Thank you for your report , it's fixed now just update to last version.


depth shift

Hi,
I did the IR and and depth calibration, I am wondering though why this depth shift between the depth provided by the kinect2 and and the computed plane exists?
Do you think there is a similar depth shift with Kinect1?
regards,

If you're referring to a vertical difference between your base plane (as you may have in ROS) and that of your Kinect cloud, this could be related to noise below what your actual ground plane. If you mean your feed is flipped vertically (ground plane is aligned in the wrong direction), then a simple translation should see to it. This may look somethings like "0 0 -pi / 2 0 -pi / 2 0" though I've likely got the actual order wrong on that.

I was referring to the shift in Z between the base plane and kinect2 depth measurements, using kinect2_calibration ros package. I applied the same code to the kinect1 and I found 5cm depth shift. Which sounds really huge to me.
For the kinect1 IR calibration, I covered the IR projector with 2 post-it notes as explained in some ros tutorials, over-sampled  the ir images and equalized the histogram using @wiedemeyer code. And after calibration, I got a depth shift of 5cm. For the kinect2 using @wiedemeyer code, I have around 3cm depth shift.
cheers

The depth calibration is a very preliminary piece of code. It just tries to lower the average error by subtracting it from the depth values. A much better solution would be to create a depth error model and apply it to the the depth data, but I didn't had time to implement something like that.
I never tried it on the first Kinect, but for the first Kinect there are tools for depth calibration available, for example this: http://www.ee.oulu.fi/~dherrera/kinect/
I guess you will get the best results if you use those tools.

Thanks for the link. I think i will first try to compute the ground truth plane (that you computed in IR frame) in the color frame. I will get the depth registered frame to color frame, thanks to the depth_registration package. And then will compare the depth data with the ground truth plane (still in the color frame). And I'll see if this depth shift is still the same.


Use built-in types as objects

I propose to use built-in Python types like tuple[int, ...] for the objects of concrete categories such as Tensor, instead of wrappers like Dim. To keep the syntax of DisCoPy uniform, we might as well define tensor.Ty = tuple[int, ...].
Similarly, the Function class defined in issue https://github.com/oxford-quantum-group/discopy/issues/108 should have python.Ty = tuple[type, ...] as domain and codomain. For Matrix it's even simpler, we can just use int as domain and codomain.
This is related to issue https://github.com/oxford-quantum-group/discopy/issues/107 because we want to be able to whisker a tuple with a Tensor. It's also related to issue https://github.com/oxford-quantum-group/discopy/issues/106, indeed the code for Functor may need some refactoring so that it does not need to assume that its object codomain has a tensor method. Instead, it can use sum for the product and the init of the class with empty arguments for the unit, e.g. int() returns 0, tuple[int, ...]() returns (), etc. See here:
class Functor(cat.Functor):
    dom = cod = Category(Ty, Diagram)

    def __call__(self, other : Ty | Diagram) -> Ty | Diagram:
        if isinstance(other, Ty):
            return sum([self(obj) for obj in other.inside], self.cod.ob())
        ...

Addressed in PR https://github.com/discopy/discopy/pull/129


Allow for picking directly from component level - even body

Currently to allow for picking a user needs to set up its own plotter and do it manually. We should expose this capability directly from the component/design/body.plot() methods as well.

We will leave body-picking for another time. This will require proper face identification and tessellation. Related to #759 partially - which I need to take over again.


Update pip in docs build env

What does this PR do?
This PR continues debugging the docs build to accomodate INC. This time we upgrade pip and apt as an added test.

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


Updated zh-cn doc

Thank you very much. There are some things that need fixing before I merge this.


[ENH] - Update 1d vs 2d (rows & columns) support for array utilities

This relates to #141 - and sweeps through procedures that can work on row / column arrays (both general utilities and space-specific functionality), and works through things to allow them to support either row or column organized 2d arrays.
The general strategy is to try and infer the orientation of the data, and work from there, meaning that these functions should still work as expected (no extra arguments) on row data, but should also work if the data is column organized. This should maintain backwards compatibility, while supporting columnar data, including supporting position data organized in columns.
In practice, this means that passing in row position data should still work just as it does currently, but also that passing in column-oriented data should work, and provide the same outputs.

Codecov Report

Merging #152 (202d183) into main (fcc77ae) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #152   +/-   ##
=======================================
  Coverage   98.34%   98.34%
=======================================
  Files          94       94
  Lines        3074     3082    +8
=======================================
+ Hits         3023     3031    +8
  Misses         51       51

Impacted Files
Coverage Δ

spiketools/tests/utils/test_extract.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Codecov Report

Merging #152 (7f44c8d) into main (fcc77ae) will increase coverage by 0.07%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   98.34%   98.41%   +0.07%
==========================================
  Files          94       94
  Lines        3074     3157      +83
==========================================
+ Hits         3023     3107      +84
+ Misses         51       50       -1

Impacted Files
Coverage Δ

spiketools/utils/epoch.py
100.00% <ø> (ø)

spiketools/spatial/occupancy.py
97.87% <100.00%> (+0.02%)
:arrow_up:

spiketools/spatial/utils.py
97.22% <100.00%> (+3.88%)
:arrow_up:

spiketools/tests/spatial/test_occupancy.py
100.00% <100.00%> (ø)

spiketools/tests/spatial/test_utils.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_checks.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_epoch.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_extract.py
100.00% <100.00%> (ø)

spiketools/utils/checks.py
94.91% <100.00%> (+0.57%)
:arrow_up:

spiketools/utils/extract.py
98.83% <100.00%> (+0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Codecov Report

Merging #152 (09355b0) into main (fcc77ae) will increase coverage by 0.07%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   98.34%   98.41%   +0.07%
==========================================
  Files          94       94
  Lines        3074     3164      +90
==========================================
+ Hits         3023     3114      +91
+ Misses         51       50       -1

Impacted Files
Coverage Δ

spiketools/utils/epoch.py
100.00% <ø> (ø)

spiketools/spatial/occupancy.py
97.87% <100.00%> (+0.02%)
:arrow_up:

spiketools/spatial/utils.py
97.36% <100.00%> (+4.03%)
:arrow_up:

spiketools/tests/spatial/test_occupancy.py
100.00% <100.00%> (ø)

spiketools/tests/spatial/test_utils.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_checks.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_epoch.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_extract.py
100.00% <100.00%> (ø)

spiketools/utils/checks.py
94.91% <100.00%> (+0.57%)
:arrow_up:

spiketools/utils/extract.py
98.83% <100.00%> (+0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Codecov Report

Merging #152 (a11598c) into main (fcc77ae) will increase coverage by 0.07%.
The diff coverage is 98.46%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   98.34%   98.41%   +0.07%
==========================================
  Files          94       94
  Lines        3074     3222     +148
==========================================
+ Hits         3023     3171     +148
  Misses         51       51

Impacted Files
Coverage Δ

spiketools/utils/epoch.py
100.00% <ø> (ø)

spiketools/plts/spatial.py
91.86% <75.00%> (+0.19%)
:arrow_up:

spiketools/utils/checks.py
94.11% <93.75%> (-0.23%)
:arrow_down:

spiketools/spatial/occupancy.py
97.87% <100.00%> (+0.02%)
:arrow_up:

spiketools/spatial/place.py
96.66% <100.00%> (+0.23%)
:arrow_up:

spiketools/spatial/utils.py
97.36% <100.00%> (+4.03%)
:arrow_up:

spiketools/tests/spatial/test_occupancy.py
100.00% <100.00%> (ø)

spiketools/tests/spatial/test_utils.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_checks.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_data.py
100.00% <100.00%> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Codecov Report

Merging #152 (f861779) into main (fcc77ae) will increase coverage by 0.07%.
The diff coverage is 98.47%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   98.34%   98.41%   +0.07%
==========================================
  Files          94       94
  Lines        3074     3223     +149
==========================================
+ Hits         3023     3172     +149
  Misses         51       51

Impacted Files
Coverage Δ

spiketools/utils/epoch.py
100.00% <ø> (ø)

spiketools/plts/spatial.py
91.86% <75.00%> (+0.19%)
:arrow_up:

spiketools/utils/checks.py
94.20% <94.11%> (-0.14%)
:arrow_down:

spiketools/spatial/occupancy.py
97.87% <100.00%> (+0.02%)
:arrow_up:

spiketools/spatial/place.py
96.66% <100.00%> (+0.23%)
:arrow_up:

spiketools/spatial/utils.py
97.36% <100.00%> (+4.03%)
:arrow_up:

spiketools/tests/spatial/test_occupancy.py
100.00% <100.00%> (ø)

spiketools/tests/spatial/test_utils.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_checks.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_data.py
100.00% <100.00%> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Codecov Report

Merging #152 (fcc21b8) into main (fcc77ae) will increase coverage by 0.08%.
The diff coverage is 98.63%.

:exclamation: Current head fcc21b8 differs from pull request most recent head 76f2a5c. Consider uploading reports for the commit 76f2a5c to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   98.34%   98.42%   +0.08%
==========================================
  Files          94       94
  Lines        3074     3243     +169
==========================================
+ Hits         3023     3192     +169
  Misses         51       51

Impacted Files
Coverage Δ

spiketools/utils/epoch.py
100.00% <ø> (ø)

spiketools/plts/spatial.py
91.86% <75.00%> (+0.19%)
:arrow_up:

spiketools/utils/checks.py
94.80% <96.42%> (+0.46%)
:arrow_up:

spiketools/spatial/occupancy.py
97.87% <100.00%> (+0.02%)
:arrow_up:

spiketools/spatial/place.py
96.66% <100.00%> (+0.23%)
:arrow_up:

spiketools/spatial/utils.py
97.36% <100.00%> (+4.03%)
:arrow_up:

spiketools/tests/spatial/test_occupancy.py
100.00% <100.00%> (ø)

spiketools/tests/spatial/test_utils.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_checks.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_data.py
100.00% <100.00%> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@claire98han - thanks for checking! Were there other docstrings you noticed that could use updating? If so, please let me know which ones - we might as well fix them now!

@claire98han - thanks for checking! Were there other docstrings you noticed that could use updating? If so, please let me know which ones - we might as well fix them now!

Sorry I commented on the wrong PR, I meant for #153.

Sounds good! Thanks both for checking this one - I'll merge it in now!

Codecov Report

Merging #152 (2d82bda) into main (fcc77ae) will increase coverage by 0.12%.
The diff coverage is 99.12%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   98.34%   98.46%   +0.12%
==========================================
  Files          94       94
  Lines        3074     3251     +177
==========================================
+ Hits         3023     3201     +178
+ Misses         51       50       -1

Impacted Files
Coverage Δ

spiketools/utils/epoch.py
100.00% <ø> (ø)

spiketools/plts/spatial.py
91.86% <75.00%> (+0.19%)
:arrow_up:

spiketools/spatial/occupancy.py
97.87% <100.00%> (+0.02%)
:arrow_up:

spiketools/spatial/place.py
96.66% <100.00%> (+0.23%)
:arrow_up:

spiketools/spatial/utils.py
97.36% <100.00%> (+4.03%)
:arrow_up:

spiketools/tests/spatial/test_occupancy.py
100.00% <100.00%> (ø)

spiketools/tests/spatial/test_utils.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_checks.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_data.py
100.00% <100.00%> (ø)

spiketools/tests/utils/test_epoch.py
100.00% <100.00%> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Need PS7

Install-Module Az.Resources -RequiredVersion 1.12.1-preview -AllowPrerelease does not work with earlier versions of powershell.
Had to upgrade to Powershell version 7 to be able to execute this command. Might need to specify this in the requirements.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2a06520c-ba95-a3c7-85e4-9e9737ca1b69
Version Independent ID: fa6dbc5f-8301-6355-d899-2903e60d58f9
Content: Template deployment what-if (Preview) - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-deploy-what-if.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@averkinderen Thanks for your comment! We will review and provide an update as appropriate.

@averkinderen I have tried to reproduce the issue that you have observed but ended up successfully running the command in lower PS versioned environment. Please find below screenshot for reference. I suggest you to fetch verbose / debug information to understand the issue better.

I'm getting this.

Only way to get it working was with ps7

Interesting! Can you first run the below command to update PowerShellGet.
Install-Module PowerShellGet –Repository PSGallery –Force
and then after it's successful execution, run the below command.
Install-Module Az.Resources -RequiredVersion 1.12.1-preview -AllowPrerelease
If the above approach resolves your issue then it would be good to consider adding a note regarding the same in the referred Azure document.
However, if issue still persists and if it's test environment then remove only Az* and Azure* folders from below locations and then re-try above 2 commands.
%ProgramFiles%\WindowsPowerShell\Modules
Unfortunately, if issue still persists then I recommend you to raise issue here as it is better place to resolve Azure PowerShell issues.

Hi,
I have exact same issue. Tried your first suggestion and it did not help:

Will check the second one...

To install the preview, you must have PowerShell Core (6 or 7). I'll update the docs.

The update will be published shortly.
#please-close


Demande d'affiliation

Merci de bien vouloir ajouter notre école à la liste des écoles accréditées.

@FabriqueDWWM First stage of verification passed, official domain is fabriquenumerique.fr as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@FabriqueDWWM Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Use _id for mongodb ids

Problem
I'm migrating an existing mongodb database to use Prisma. Problem is, Prisma insists on using id field instead of _id. If I were to switch, all my existing models would break.
Suggested solution
I'm fine if Prisma internally stores id but when I do crud operations, I want to be able to pass objects with _id field

Bump

You can use @map.
model User {
  id String @id @db.ObjectID @map("_id")
}

There is also undocumented schema feature which lets you create a specific id type:
type MongoID = String @id @default(dbgenerated()) @map("_id") @db.ObjectId

model User {
  id MongoID
}

(This is undocumented as this is subject to change. We can rename to alias instead of type)
I would close this as duplicate of https://github.com/prisma/prisma/issues/6839

Not sure why this is closed. There's literally no way to use _id with an existing mongodb database. If I use @map("id"), prisma generate throws an error:
 Error validating field 'id': MongoDB model IDs must have a @map("_id") annotation, found @map("id")

Oh, thats what you mean.

, Prisma insists on using id field instead of _id

I interpreted this incorrectly. I will reopen this.

@shash7 What do you models in MongoDB look like? The error message you posted indicates you did not use what @pantharshit00 suggested (id String @id @db.ObjectID @map("_id")) but did the opposite of trying to use @map("id") - which indeed can not work if you data itself uses _id.

I did some testing and it turns out that there's no way to set _id
Basically something like this won't work:
let user = {
                _id : 'someid',
		name: '',
		metrics: {
			accountType:''
		},
	};

	const newUser = await prisma.user.create({
		data: user,
	});

@janpio if there's a way around this let me know otherwise I get it - Prisma doesn't allow setting _id field(it has to be a id field)

It will still be stored as _id in the database. The schema spec doesn't allow identifiers to start with _ so that's why you use a @map there.
What I initially though you meant that you are trying to store ids as id field in the database instead of _id in mongodb.
You will use id in the client in and it map to _id in the database. If you need to use id field in the database as well without _ then this is a valid feature request.

@janpio if there's a way around this let me know otherwise I get it - Prisma doesn't allow setting _id field(it has to be a id field)

Exactly. The Client code can not handle _id (as this is not a valid identifier in that world) so you use @map in the schema to make a id available that is persisted as _id in the database as it is required for MongoDB.

I got it. Thanks guys.
@janpio btw any way we can use a custom validator instead of the objectid validator? For instance, I create my record with the id field prefilled by another library(nanoid) and Prisma won't accept that as it is using the objectid validator(generating a objectid by myself and setting it as the id field works fine)

I don't think that is possible right now, but might be worth creating a new issue for with some additional explanation so the  team working on MongoDB support can look at it explicitly and see if we can do something about that.

Yeah exactly, that's why I closed this one.
Btw some custom apps do use custom ids for mongodb and even for postgresql.


Update README.md

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Test for CICD actions

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

Fixes #3


Add documentation for overriding get_upload_to in custom image models

Fixes #2601
Previous PR: #8228
@lb- Please review and let me know if I should make any other changes.

Thank you for giving this a go, hopefully the feedback will help you refine this.
Imagine you are a Wagtail developer, you are creating a custom image, so you go to the docs page https://docs.wagtail.org/en/latest/advanced_topics/images/custom_image_model.html#custom-image-model
In that section you see that code example of how to create the custom models. Then further down you see what you can override to change the behaviour. You will have to assume someone doing this knows a bit about Django and how Django's ImageField works. But they may not know how we (Wagtail) has changed this in different ways to support custom behaviour.
The other thing to note is that once we document a feature, it becomes part of the public API and we will do our best not to change it in the future. Which is why we do not always document every method on everything.

Ok, I will make the required changes.
Where can I find the original functions?

See
https://github.com/wagtail/wagtail/blob/8a5d228d37662e7d21ace970e2e352a8da5c1ac2/wagtail/images/models.py#L302
https://github.com/wagtail/wagtail/blob/8a5d228d37662e7d21ace970e2e352a8da5c1ac2/wagtail/images/models.py#L1266

just updated the docs. Review at your convenience.

As this is a small docs improvement I will cherry pick this into the 6.0 release.

Thank you also @Dharma-09 - I have added you as a contributor to this fix.


[Java] Accounted size went negative when locallyHeldMemory overflow

Describe the bug, including details regarding any error messages, version, and platform.
Bug description:
The following code will throw Accounted size went negative exception.
The buffer allocator a will ask its parent allocator to do allocation, however the locallyHeldMemory of parent allocator already reaches Long.MAX_VALUE when creating child actor ChildB
        RootAllocator parentAllocator  = new RootAllocator();
        BufferAllocator a = parentAllocator.newChildAllocator("ChildA", 0, Long.MAX_VALUE);
        BufferAllocator b = parentAllocator.newChildAllocator("ChildB", Long.MAX_VALUE, Long.MAX_VALUE);
        IntVector intVector = new IntVector("intVector", a);
        intVector.allocateNew(1);
        intVector.set(0, 2);
        intVector.setValueCount(1);

        IntVector intVector2 = new IntVector("intVector2", a);
        intVector2.allocateNew(1);
        intVector2.set(0, 2);
        intVector2.setValueCount(1);

        intVector.close();
        intVector2.close();
        a.close();
        b.close();

error message:
Exception in thread "main" java.lang.IllegalArgumentException: Accounted size went negative.
	at org.apache.arrow.util.Preconditions.checkArgument(Preconditions.java:136)
	at org.apache.arrow.memory.Accountant.releaseBytes(Accountant.java:214)
	at org.apache.arrow.memory.RootAllocator.releaseBytes(RootAllocator.java:29)
	at org.apache.arrow.memory.Accountant.releaseBytes(Accountant.java:221)
	at org.apache.arrow.memory.AllocationManager.release(AllocationManager.java:152)
	at org.apache.arrow.memory.BufferLedger.decrement(BufferLedger.java:157)
	at org.apache.arrow.memory.BufferLedger.release(BufferLedger.java:124)
	at org.apache.arrow.memory.BufferLedger.release(BufferLedger.java:104)
	at org.apache.arrow.vector.BaseValueVector.releaseBuffer(BaseValueVector.java:114)
	at org.apache.arrow.vector.BaseFixedWidthVector.clear(BaseFixedWidthVector.java:248)
	at org.apache.arrow.vector.BaseFixedWidthVector.close(BaseFixedWidthVector.java:238)
	at Test.main(Test.java:46)

version:
12.0.0
Component(s)
Java

Thanks! It seems likely there's an underflow or overflow here. (I'm curious why it lets you allocate from a in the first place, so I'll dig into that too; there's an "overlimit" situation but AFAIK it only applies when transferring buffers between allocators)


Added Great Learning Free Power BI Course Link

@avinash201199 I have added a Power BI Course Link on Great Learning. Please review and accept the PR.


How to extend deferred example to add Sampler?

I came to know that subpass load cannot address specific pixels which i need for various effects, what are the alternatives?
Also how to extend the deferred triangle example to render Position and other data to a sampler image?

If you want to be able to address random pixels and not just the current pixel, then you need to use multiple render passes and not just one.

Thanks. An example would work. Or atleast please tell me steps. Thanks

Drawing to an image is exactly the same as drawing to a swapchain image. There's an example here: https://github.com/vulkano-rs/vulkano-www/blob/master/examples/guide-triangle.rs
Then after you have drawn to the image, use it as a sampled image.

Great man

Btw just for the sake of self promotion, heres my WIP.
Im going to attempt Hemisphere ambient occlusion for the terrain thats why...
If anyone of you have time please contribute to my thread at gamedev..
https://www.gamedev.net/forums/topic/693892-what-it-takes-to-make-a-great-looking-terrain/

Sounds like this was resolved


[PT Run] Spec for Registry Search

Summary of the Pull Request
First spec for the Registry Search Plugin for PT Run
PR Checklist

[x] Applies to #5495
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[x] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

only the Spec (first version)

Validation Steps Performed

no code -> not needed

neat, i think we need to show some mock UX here as well for some of the permutations you have here.

I think we should have a dependency of https://github.com/microsoft/PowerToys/issues/5273 per https://github.com/microsoft/PowerToys/wiki/Roadmap
I also think this is a feature that is Action Key enabled only by default

questions i have here is how does the drop down look like, I would like to think about the results and how those are visualized to the end user.

@TobiasSekan maybe we should continue to discuss on #5495 instead of a spec format right now.  The UX interaction is a massive open question.

I'm going to close this out, lets continue the discussion on #5495


Questions about using GES2pose.py

I have a question about generating trajectory pose files. I used the 56leonard.esp file you provided, imported it to Google Earth Studio, then exported the pose file 56leonard.json, and converted it using the script file GES2pose.py you provided, but the output pose is not the same as the poses_enu.json file provided directly in the dataset.

Hello, I have encountered the same problem as you mentioned. Have you resolved this issue？

Hi, we additionally rotated the scene for 29 degree (https://en.wikipedia.org/wiki/Manhattanhenge#:~:text=In accordance with the Commissioners,the streets on that grid.) in the provided pose, but that's not necessary. You can still run the experiment using extracted poses.


Replace kernel in user agent with more detailed OS

Looks a bit like this:
docker/1.8.0-dev go/go1.4.2 git-commit/bfccd32-dirty os/Ubuntu_14.04.2_LTS arch/amd64

This is a follow-up to #14089.
Registry operators (e.g. Docker Hub) use this data to better understand their users. Exposing the kernel version a user is using could be a potential security risk.
Docker, Inc uses the data from Hub to understand what operating systems users are using so we can prioritise features, OS support, etc. I'd really like to find a way to release aggregate anonymous data so the entire community can benefit. A WIP...

+1

Exposing the kernel version a user is using could be a potential security risk.

How?
How is this more, or less dangerous, than reporting the docker engine version?
How exactly can that be exploited by an attacker?
How many kernel vulns are remotely exploitable vs. docker vulnerabilities?
The other PR was closed because this is churn, changing existing behavior without much justification, (not) addressing a (questionable) issue.
If there is a reason motivating this , let's spell it clearly.
If changing the content of this is a solution, let's first discuss what kind information we should have here.
-1

@dmp42 Agreed there is not a big security risk, assuming registries are trusted.
Having a more detailed OS gives us much better information about who is using Docker. E.g. the new boot2docker kernel is indistinguishable from Debian, so we have no idea whether people are using it or not.
If having the kernel there is not a security issue, perhaps we could just leave the kernel and add the more detailed OS in addition?

I agree with @dmp42. We should not remove the kernel version, but I also like the idea of incorporating information about the OS.
So, I vote for kernel version + os info. :smile_cat:

Cool. Sounds sensible. @stevvooe also suggested changing to "distro".
Thought about renaming this but it's not just a different thing. Sooo ---> https://github.com/docker/docker/pull/15206

I dont like the idea of tracking anything

The argument that a secure and trusted registry doesn't pose a risk is simply wrong. I can see how collecting these stats can be useful to figure out what kernels people are using and making it easier to make decisions when it comes to support, but there are problems.
The first issue is that HTTPS has been compromised before for some cipher suites and this will most likely happen again. The second issue is that crypto bugs in openssl and similar libraries can pose new risks, even if the cipher suites themselves are in no way a problem.
This means that even though the registry is trusted and wouldn't do anything nefarious with the information it receives, that doesn't mean attackers wouldn't carry out attacks on HTTPS connections to those registries. Should there be any kind of breach, that information can be collected in bulk directly from the network traffic.
We can even take this further and say that pulling signed images would be safe even in case of a hack, but sending the kernel version would pose a risk by revealing information about the systems which pull/push images.

@unclejack While I don't disagree with you outright, this implies that any kernel version is effectively a master key to any machine. Are you implying this supposition is true? Is there evidence we can point to that indicates this is a real, exploitable risk?
I'm not saying that you're wrong or that I even disagree. I'm saying that making a decision on the basis that "we all know it's true" is not the right way to make a rational decision.
Anything demonstrating this is a real risk will help move this forward. Something as simple as an attacker model demonstrating remote access through just a kernel version would be sufficient; a list of exploits; academic papers; data showing the kernel version is the key to a particular attack. When someone has the sophistication to sniff all of your HTTPS traffic, having a kernel version seems like a very small part of that attack. Let's find something that shows that assertion is false.
If we are going to make this omission, we should have a good foundation on which to base it, rather than security boogeymen.

@stevvooe Here's a scenario: a machine is running a kernel with a bug in the network stack which crashes the host upon receiving a certain type of network packet, let's call this kernel version x.y.z. This machine pulls an image from the registry and also sends the kernel version down the wire. A compromised registry (through bulk traffic collection or through compromised hosts running the registry) exposes this machine. A network packet can be sent to this host to take it down.
This is just a denial of service attack, but worse attacks are possible. Unpatched systems falling behind with kernel updates can be exposed to even worse things (remote code execution, being served malicious images based on the kernel version, etc).

@unclejack Scenario: an attacker has compromised a registry. The kernel version is no longer sent in the HTTP requests. The registry adds a layer with a crontab that sends the contents of uname -a to a remote attacker in place of the original image. (This will be less possible with registry v2 + trust/notary).
Do we have an example that leverages the kernel version without compromising the registry?

@stevvooe Compromising the transport is still a risk which doesn't require compromising the registry, compromising the network close to the client would be enough in some cases. While it may seem difficult to exploit at first, there are companies out there which run less-than-the-latest RHEL/CentOS kernel and that means they're not up to date with their security patches.
Some users are also running insecure registries and I can't say for sure that nobody is running an insecure registry over the Internet.
A previous discussion on #1988 illustrates some of the concerns around allowing image pulls in production environments and there's interest in disabling pulls/push from registries not authorized explicitly as well. I'd say this is a good argument in favor of thinking about this further and not dismissing it as quickly.
It might be OK to have staging machines provide this information, but machines running in production shouldn't provide that information. There are companies which wouldn't even allow something like this in their environments.
This really means this should be done based on an opt in process, not by default and not by opting out.
Perhaps users should be asked whether they'd like to provide some usage stats or provide settings to opt into this based on the registry.

With plans for content-addressable images that could be hosted by less-trusted registries and mirrors, I think we should not continue with an attitude of assuming the registry is trusted.
The information disclosure of the kernel version is indeed a risk which, for some users, perhaps for only a minority, will be deal-breaking.
I do think there were issues with patch #14089 that make it unusable for merging as-is, but I am +1 on the intention.
The Docker Privacy policy is very much boilerplate and designed specifically for the hosted services. There is no policy for the Engine and products in what they will collect, either by default or otherwise. I think it would be fair to have some simple mechanism for indicating reporting anonymous statistics. Ideally this feature would be designed specifically in a way to improve anonymity.

We can keep the discussion in #14089 and stick with facts there.