[WIP] fix(replica): persist last IO offset and size

This commit fixes/improves following:

WO replica was going for full rebuild in case if there is only one missing IO
by persisting the offset and size of the last IO so that we can sync the last
I/O during initializing the replica.

Create autogenerated snapshots only when volume is in RW mode, since it
doesnot make any sense to create a new snapshot even when volume is in RO,
this was also making the resync of last missing IO bit complex, since new
snapshots were created due to which the need of persisting file name was
arising and again this name was getting changed after generating new snapshot.

sync head file as well when volume is in RO mode

Fixes recurring ReadOnly logs

Design Link: https://docs.google.com/document/d/1DzjIB2Fu_EvNZ4VpeWCUjaa37MEWZztbFXpaw9yYjUQ/edit
Signed-off-by: Utkarsh Mani Tripathi<EMAIL_ADDRESS>

Closing this PR due to following reasons:

Upgrade related code needs to be added in data path
It is not guaranteed that client will send the same IO if failed to write.
Design constraints like syncing of head files if volume is in RO mode.


Issue 275 - Crash in testSendData.js

PR is added to the queue for testing as 1. task. (19fea12)

Test (Success) 50648605 build is completed (19fea12)
See https://github.com/ThaliTester/TestResults/tree/5064860519fea12_Issue_275_-_Crash_in_testSendData_js_tobybrad/ for the logs

Test<PHONE_NUMBER>fea12(19fea12) has failed
See https://github.com/ThaliTester/TestResults/tree/5064860519fea12_Issue_275_-_Crash_in_testSendData_js_tobybrad/ for the fail logs

No longer an issue on ios9.


Area doors should always flash

We would like the doors between areas to always be open and to always flash.

This was completed prior to releasing canary


Create an offline installation of Visual Studio 2017 (Community)

Download the Visual Studio Community 2017 installer
Go to the installer Folder file where you download it. Usually it is C:\Users<YourAccountName>\Downloads
Go to the top of the Explorer and type in cmd
Type mkdir vs2017
Type vs_Community.exe --lang en-en --layout f:\vs2017 --all

5 lines of instructions

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

ID: 228076d1-94bf-c502-c018-08f7c0faf377
Version Independent ID: c64a5cd5-6236-4192-2bfc-923aa20c94da
Content: Create an Offline Installation of Visual Studio - Visual Studio
Content Source: docs/install/create-an-offline-installation-of-visual-studio.md
Service: unspecified
GitHub Login: @TerryGLee
Microsoft Alias: tglee

Type vs_Community.exe --lang en-us --layout f:\vs2017 --all
I used the same command last year but forgot the command. I have to look it up again and copy the command from stackoverflow. The person typed in en-en instead of en-us. lol

@gremlinsmasher Thanks for your contribution. FYI: You can also find further examples on the following page: https://docs.microsoft.com/visualstudio/install/install-vs-inconsistent-quality-network.
Cheers,
~ Terry

You are welcome. Thank you for posting.

Can I edit what components I can add?

how to create an installation when i already downloaded it not using cmd

Can we download only some part of it not MSDN in Visual Studio 2017 (Community) because it full size around 30gb

Can we download only some part of it not MSDN in Visual Studio 2017 (Community) because it full size around 30gb

Yes you can, instead of --layout f:\vs2017 --all you can do something like this:
vs_community.exe --layout c:\vs2017 --add Microsoft.VisualStudio.Workload.NativeDesktop --includeRecommended --lang en-US
To discover the actually names of what you need you can run a normal setup of VS with everything you need and then export a .vsconfig file from there. Open the file and you'll see the names of everything you installed. Here is mine:
{
	"version": "1.0",
	"components": [
		"Microsoft.VisualStudio.Workload.NativeDesktop",
		"microsoft.visualstudio.component.debugger.justintime",
		"microsoft.visualstudio.component.vc.diagnostictools",
		"microsoft.visualstudio.component.vc.cmake.project",
		"microsoft.visualstudio.component.vc.testadapterforboosttest",
		"microsoft.visualstudio.component.vc.testadapterforgoogletest",
		"microsoft.component.vc.runtime.ucrtsdk",
		"microsoft.visualstudio.component.windows81sdk",
		"microsoft.visualstudio.component.vc.atlmfc",
		"microsoft.visualstudio.component.vc.cli.support",
		"microsoft.visualstudio.component.vc.140",
		"component.linux.cmake"
	]
}

Just use the --add flag to add what you need.

where to find the offline folder after creating offline installer for visual studio 2017 ?

Thank you for the commands, it was helpful. However there is a small error that I was able to fix. The fifth line the command "Type vs_Community.exe --lang en-en --layout f:\vs2017 --all" --lang en-en is wrong it is supposed to be "--lang en-us" or "--lang en-uk" and so on.

Yes, it was a typo. Thanks
On Tue, Apr 16, 2019, 19:03 JohnDoit2<EMAIL_ADDRESS>wrote:

Thank you for the commands, it was helpful. However there is a small error
that I was able to fix. The fifth line the command "Type vs_Community.exe
--lang en-en --layout f:\vs2017 --all" --lang en-en is wrong it is supposed
to be "--lang en-us" or "--lang en-uk" and so on.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/visualstudio-docs/issues/741#issuecomment-483886966,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK2eGRtvOTNkpr_0OJUPQ-LBYhGQDH_9ks5vhmTlgaJpZM4S5-cF
.

hi there m new here its my first time on github platform...
well i wanna know i want to install vs2017 offline. Issue is my internet is so slow and i want to install vs2017 in my computer lab in which there are more than 50 computers so my question is.. is this article solve my problem or not ???

look here i am following the same 5 instructions... !!
after this i just have to copy this vs2017 folder to where i want to install and run the setup file is that right ????

Download visual studio 2017 offline installer from our site
Open downloaded folder and do the following steps
1.Right click and extract the zip file
2. Open the extracted folder and open the folder named certificates
3. Right-click and install all three files and click back
4. Scroll down to the last and double-click on vs setup.exe file and install.
Enjoy!!!!

Comment above with link to the nosware site was edited to replace the link away from the product downloads back to our official downloads at Microsoft.


remove the need to edit sqlnet.ora from wallet for python, nodejs, and Go ms

this should not actually be necessary and we can instead mount the files in the container at the expected locations at runtime rather than modify the WALLET_LOCATION eg.

I don't know how feasible this would be, but ideally our microservices should support all the following alternatives and combinations without code change:

all Oracle DB types (ATP-D, ATP-S, DBCS, ExaCS, ExaCC, On Prem)
all deployment options (shared schema, shared DB, multiple PDB)
all authentication methods (password, password wallet)
Encryption

Ideally, it would be nice to establish a simple model for how they can be deployed without too much variance in the deployment yaml and ideally with some standardization across coding languages.  An attempt was made to do this with nodejs, python and helidon-se.
If the sqlnet.ora edit can be avoid that would be great.

@pasimoes This is the one I mentioned would be good to knock out as part of " Inventory-go is failing on container startup #134 "
You will see in utils/db-setup.sh that the sqlnet.ora file of the wallet is being modified in order to change the WALLET_LOCATION .
This should be removed and, as stated in the issue described above we can instead mount the files in the container at the expected locations at runtime.
You will see that I have done this in the relevant deployment yamls (but it is commented out), eg...
- name: TNS_ADMIN
value: "/usr/lib/oracle/19.9/client64/lib/network/admin"
[...]
volumeMounts:
- name: creds
mountPath: /usr/lib/oracle/19.9/client64/lib/network/admin
This worked fine for all microservices except inventory-go where I got the "TNS: Could not resolve" exception you see in #134 which indicates the tnsnames.ora can't be found (presumably due to incorrect mount or some such).
Since I couldn't get inventory-go to work I commented out the mods mentioned above in the other deployment yamls.
This is why I was saying it might make sense for you to work on these two issues at the same time and by making/resolving this sqlnet.ora change you might also resolve the inventory-go issue you are hitting as well.


react-fela: change connect(mapStylesToProps) to be connect(rules)

hey :cat:
this is a proposal for a BREAKING CHANGE to react-fela.
the current connect takes a single argument mapStylesToProps which is a function of shape (props) => (renderer) => styles, where styles is an object of classNames mapped by style name.
i propose we change connect to take a single argument rules which is an object of rules mapped by style name.
for example, the current way:
const mapStylesToProps = (props) => (renderer) => ({
  container: renderer.renderRule(({ color }) => ({
    display: 'flex',
    flexDirection: 'column',
    alignItems: 'center',
    backgroundColor: color
  }, { color: props.color })),
  button: renderer.renderRule(() => ({
    marginTop: '20px'
  })
})

const connectedComponent = connect(mapStylesToProps)(Component)

the proposed new way:
const rules = {
  container: ({ color }) => ({
    display: 'flex',
    flexDirection: 'column',
    alignItems: 'center',
    backgroundColor: color
  }),
  button: () => ({
    marginTop: '20px'
  })
})

const connectedComponent = connect(rules)(Component)

why?

matches better with createComponent api, which takes a single rule.
removes renderer.renderRule boilerplate (i'm curious, does anyone have a use case where they need to call something other than renderer.renderRule?)
makes it nice and easy to have style files with fela rules alongside your react components, so you can mix and match rules as you wish. (in the current way, you have rules then you have to make another function mapStylesToProps which is separate)

what do people think about this idea?
if people think this is a good idea, i'm happy to update the respective documentation and change the code to pass the linter.
thanks for listening. :heart:
/cc @iainkirkpatrick

i'm curious, does anyone have a use case where they need to call something other than renderer.renderRule?

ah, now i see the examples/react/components/Header.js uses renderer.renderKeyframe in connect(mapStylesToProps). i wonder if we could instead support this use case by adding a second argument to connect which is given the renderer and can return an object of additional properties to add to the rule calls, or something like that. but also brings up the question how you'd do this if you prefer the createComponent api, since i guess i saw the difference being only in taste not functionality.

Hey @ahdinosaur,
First of all thanks for your PR! I would like to check the API in detail later and decide if I can find any downside actually.
Would also love to hear some other opinions @johanneslumpe @tajo @tiagojsalmeida
For now I can also point you to two other resources:
E.g. there's this one that actually "does" most of the things you're proposing: https://github.com/dustin-H/fela-styles-connector and it can be used together with fela-stylesheet perfectly.
And about the second comment regarding the renderer.renderKeyframe. That's basically the reason why this API was built that way - to allow any renderer. calls without a strict API. But nowaydays, we can actually solve this with an yet undocumented feature of rules. You can access the renderer through a second parameter:
const rule = ({ fontSize }, renderer) => ({
  fontSize,
  color: 'red',
  animationName: renderer.renderKeyframe(...)
})

Also there'll soon be a plugin that allows to inline keyframes and fonts: https://github.com/rofrischmann/fela/pull/238 which will be released with the next update.
So for now: I really like how it simplifies the API and am more than happy to improve the connect API soon :)

Ok, I am convinced this makes it much simpler! Great one =)

yay thanks @rofrischmann :man_dancing: :sparkling_heart: :cherry_blossom:

It does look pretty good ;)

Much simpler, thanks @ahdinosaur! :sparkles: This will be even more amazing together with fela-plugin-embedded
Here's the connect implementation for the people wanting to try out the api with the current<EMAIL_ADDRESS>import { connect as felaConnect } from 'react-fela'

export default connect

function connect (rules) {
  const mapStylesToProps = (props) => (renderer) => {
    return Object.keys(rules).reduce((acc, key) => ({
      ...acc,
      [key]: renderer.renderRule(rules[key], props)
    }), {})
  }

  return felaConnect(mapStylesToProps)
}


[UI/UX] Storymap footer style

Description
Please update styles of the footer:

Please find mockups here. There are two versions: one for a default (light) mode and a second one for the dark mode.
Figma link
Screenshots
No response
Mockup URL/file
No response

Also, I noticed the background of the footer hasn't been updated yet. In Figma, it's set to gray-200 without transparency


[C#] Clicking 'Save and Run' can produce runtime errors

Note -- the underlying issue may be in the Script repo but since I see this while interacting with the buttons in the portal I thought I'd start here

Create a new Queue Trigger C# function
Add a Azure Storage Table output binding (select all defaults)
Use this function:

public static void Run(string myQueueItem, ICollector<Item> outputTable, TraceWriter log)
{
}

public class Item
{
    public string PartitionKey { get; set; }
    public string RowKey { get; set; }
    public string Name { get; set; }
}

If I make a tiny change (like add a space), then click 'Save and Run', I get:
Exception while executing function: Functions.QueueTriggerCSharp1. mscorlib: Object of type 'Microsoft.Azure.WebJobs.Host.Tables.PocoEntityWriter``1[Submission#0+Item]' cannot be converted to type 'Microsoft.Azure.WebJobs.ICollector``1[Submission#0+Item]'.
Clicking Run after seeing the error works. And if I explicitly click Save and then Run after a change, it does not occur.

My guess is that it's a race condition between saving the script and running it. We wait for 200 msec between save and running the function. Probably that's not enough time to compile.

//cc @mathewc
Here is how I think we can make this be more deterministic. If the host can expose another admin api on the function for querying information about the status of the function (loading, compiling, ready) then we can wait properly for how long the function needs before running.
//cc @davidebbo
In fact since the host exposes an authenticated admin api, we can move all the functions related stuff from kudu and put them in the host. this way the code won't diverge and would require less ceremony about adding a new field or small feature like this. kudu will be just used to retrieve the admin secret.
  Then once they run it locally, the portal realize it and switches to talking to localhost. You can toggle between Azure and local machine with a simple click in the same UX experience. You can very easily test new features of the host with the portal at the same time. You can even "Bring your host" if you're running on your own VM. then the host can offer a command line interface as well for creating, running and managing functions locally that in similar to the http interface.

That's a very interesting idea! Let's explore that.
The one big downside of not having Kudu API is that we lose the ARM API which bridges over to SCM site. But potentially we can have that API only for ARM purpose, but not have portal use it.

True. also Vfs, though kudu could just be a proxy for the main site for these calls over ARM assuming kudu has access to the admin secret.
For vfs, can't see a way around adding them to the host if we want the scenario I mentioned above to work

Close this issue as there is no work item for portal. We still need some API from the host for streaming structured data about things like compilation, etc.


HASTAC Commons plugin refresh interval error

This was reported by Chris McGuinness from HASTAC for HASTAC Commons:
"I'm writing in reference to https://hastac.hcommons.org.  The front page uses the plugin WP RSS Aggregator to display recent posts. However, it is not showing any posts from after July (for example, https://educational-cultural-institutions.hastac.hcommons.org/2023/09/20/welcome-hastac-scholars/). When I checked the settings, the refresh interval was set to 2 hours. However the last refresh mentioned was 2 months prior and also next scheduled in 2 months. I changed the interval to 15 minutes and the scheduled refresh started to count up. Is there a setting I do not understand? Or is the plugin malfunctioning?"

In a previous issue (which I can't find :-/) we had a daily problem with very slow page loads across the site that was traced to scheduled tasks with wp-cron. Wp-cron was configured to run once-per day rather than on each page load to address this issue. However, this seems to have been working incorrectly, or perhaps failing at some point before completion, and so the job that updated the RSS feeds was not firing.
The cron job has been updated and wp-cron has been run manually, which corrected the HASTAC issue.
If further cron issues are discovered I will create a separate issue.


lightyear: init at 9420f9e

Motivation for this change
Depends on/tested with #19002. Lightyear is one of the major libraries of Idris so far!
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[x] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@rasendubi I think they're automatically generated in some way by default.nix? I can definitely build this with nix-build.

@siddhanathan from nixpkgs manual:

All versions of a package must be included in all-packages.nix to make sure that they evaluate correctly.

@groxxda I understand that that's how it normally works. However, I think someone may have automated this for idrisPackages. For instance, notice that the Idris package wl-pprint is not mentioned in all-packages.nix, yet remains installable.

Okay so this is installable as iridisPackage.lightyear? Then it's fine :+1:

@groxxda Exactly :smile: I just rebased to include the fix for #18998, so it will hopefully build in Travis now.

nix-shell -p idrisPackages.lightyear: ✓

Thanks, @Mic92!


gpg encryption not working (also error messages are not logged)

Hey, I just came across this project and as I was setting it up I was unable to get encryption working. I saw that the changes added were quite recent so I thought I could create an issue just in case. I have read both the code and the documentation (and I am not getting any error messages). It works without encryption enabled. I am using gpg in this case.
	resurrect.set_encryption({
		enable = true,
		method = "opt/homebrew/bin/gpg", -- Also tried with just 'gpg'
		public_key = [[
-----BEGIN PGP PUBLIC KEY BLOCK-----
mDMEZsKHLxYJKwYBBAHaRw8BAQdAYY8xYh6O8GEUO8wiAqKEqpReSGUoyriXO51m
I didnt bother adding the whole public key but you get the point
=207v
-----END PGP PUBLIC KEY BLOCK-----
    ]],
	})

Forgot to mention, I also tested deleting al the plugin files after enabling encryption, but it did not help either

Hey, Thanks for creating the issue!
I don't have a Mac, but maybe we can try and figure out what is going on anyways.
The command that is being run is this:
cmd = string.format("%s --batch --yes --encrypt --recipient %s --output %s", pub.encryption.method, pub.encryption.public_key, file_path:gsub(" ", "\\ "))

i.e. the following:
gpg --batch --yes --encrypt --recipient <recipient_email_or_keyid> --output <output_file>
You can try and change the public_key to be the key_id or email associated with the key.
You can list the keys with gpg --list-keys
This should produce an output like so:
$ gpg --list-keys

pub   rsa3072 2021-08-19 [SC] [expires: 2023-08-19]
      6D5A7D00E1A9DB75 (This is your key ID)
uid           [ultimate] Alice Example<EMAIL_ADDRESS>(this is your email)
sub   rsa3072 2021-08-19 [E] [expires: 2023-08-19]

This should probably be stated clearly in the README to avoid confusion

Also did you check the debug overlay for missing errors, or where did you expect them to be? There should be errors, so if they are not in the debug overlay, then that is a bug.

Thanks so much for the good answer, I am a little new to lua, wezterm and nvim in general so I do not know about the debug overlay. It worked when I used full path for gpg in the method (like with all utilities in wezterm I have noticed) and using the key id. I guss the Key id is public information so I dont need to be scared to push it to my dotfiles repo?
Unrelated: Thanks for the amazing plugins. One thing about workspaces I find a little annoying is that they manage all windows (since I am using a tiling window manager) so each workspaces can pop up multiple windows instead of being tied one. Is this solveable? I dont think so, since it may be a limitation in wezterm itself. And when I load a workspace using ressurect it opens the loaded setup in a new window instead of overtaking or closing the original one where i ressurected it from, is this the expected behaviour?

Thanks so much for the good answer, I am a little new to lua, wezterm and nvim in general so I do not know about the debug overlay.

The debug overlay is very nice for testing or finding errors in wezterm, plugins or your config. The default keybinding is CTRL+SHIFT+l.

It worked when I used full path for gpg in the method (like with all utilities in wezterm I have noticed) and using the key id.

You probably need the full path because of this

I guss the Key id is public information so I dont need to be scared to push it to my dotfiles repo?

From my knowlegde, you should be able to share your key id with anyone, i.e. push it to your dotfiles.

Unrelated: Thanks for the amazing plugins.

Thanks for your nice comment! 😄

One thing about workspaces I find a little annoying is that they manage all windows (since I am using a tiling window manager) so each workspaces can pop up multiple windows instead of being tied one. Is this solveable? I dont think so, since it may be a limitation in wezterm itself.

From my understanding of wezterm (which might be wrong) a workspace is a collection of windows with tabs, panes ect. So each workspace will manage its own windows, which cannot be shared. I dont think you can have more than one workspace active at a time, so it might not solvable. However if your workspace only contains one window, then when you switch to that workspace it should only open that one window.
One suggestion is that you keep your workspaces to one window and manage that only.
Another would be to drop workspaces and use this plugin to load/save window states (i.e. all the tabs and panes for each window, instead of saving/loading workspaces which also contain windows).
I dont personally use a tiling window manager and I usually only have one wezterm window open at a time so my usecase might differ a lot from yours. But I can try and help you configure a workflow which works for you if you can try and explain what you are looking for. As this workflow might also be useful for other users aswell 😄

And when I load a workspace using ressurect it opens the loaded setup in a new window instead of overtaking or closing the original one where i ressurected it from, is this the expected behaviour?

If you are using the fuzzy finder keybinding from the setup-example then it will create new windows when choosing an element as to not close important panes from the "old" window. This is expected for the setup-example, but can be changed as it is just an example of a usecase.

It worked when I used full path for gpg in the method (like with all utilities in wezterm I have noticed) and using the key id.

I had the same problem with Homebrew-installed apps on MacOS. On Mac, if you launch Wezterm by clicking WezTerm.app from your Applications folder, it will only have access to a default PATH that includes usr/bin and usr/sbin. Homebrew installs apps in a different location, so their binaries aren't found. As @MLFlexer pointed out, this is a MacOS issue: Wezterm | F.A.Q.
The solution is to either 1) setup your Wezterm config like in the Wezterm FAQ so it has full access to your PATH, 2) launch Wezterm from another terminal (or even launch wezterm from the WezTerm.app instance) and it will have access to your declared PATH variables or, 3) as you mentioned, provide the full path to the gpg binary within the method.

Thank you for the great help! Regarding workspaces and windows, I find I most often use a single window. Sometimes I do have secondary windows on other desktops, but I can live with them being removed when I switch workspace.
I was thinking (if I have time of course) to take out the statusline from my config and try to create a wezterm plugin of it that is similar to lualine for neovim, but I have never made a plugin before. Do you think that would be a nice addition?

I am closing this issue since its resolved :)

Thank you for the great help! Regarding workspaces and windows, I find I most often use a single window. Sometimes I do have secondary windows on other desktops, but I can live with them being removed when I switch workspace.
I was thinking (if I have time of course) to take out the statusline from my config and try to create a wezterm plugin of it that is similar to lualine for neovim, but I have never made a plugin before. Do you think that would be a nice addition?

I am closing this issue since its resolved :)

Making a plugin is not much different from making an extention of your own wezterm configuration. I don't see why it wouldn't be a nice addition, especially if you could make it nice and extensible such that other plugins can export "widgets" which could be included in the tabbar.
Feel free to use the plugins I have made as inspiration for how to develop a plugin yourself 😄 these two are quite a bit simpler than resurrect.wezterm if you need inspiration:
modal.wezterm
smart_workspace_switcher.wezterm


Change error messages and error types

Changes

Improves error assertions and error equal checks
Add basic golint linter configuration and enable only err113 by default
Upgrade to golang 1.13 (Major break)
Regenerate example and framework code

@seseloos ?

@donutloop I didn't had the time to review the pull request over the last two weeks but I think I can check it in the next days.

@seseloos (っ °Д °;)っ


Make meaningless [pscustomobject] casts report an error

Summary of the new feature/enhancement
As a user, I want to be notified when a cast doesn't work as intended:
# OK - a statement-terminating error occurs if the operand cannot be cast to [int]
PS> [int] 'Not a number.'
InvalidArgument: Cannot convert value "Not a number." to type "System.Int32". Error: "Input string was not in a correct format."

However, this doesn't currently work as intended with a [pscustomobject] cast:
# A meaningless cast:
# !! Seemingly quietly ignored - in reality, the operand is simply wrapped in [psobject](!)
PS> [pscustomobject] 42
42

Unexpectedly, the above is equivalent to [psobject] 42 and wraps the operand  in a large largely invisible [psobject] instance (except it isn't always invisible - see #5579).
While the utility of such a cast with [psobject] is questionably in itself, with [pscustomobject] it is downright puzzling (and even though both type accelerators refer to System.Management.Automation.PSObject, PowerShell can and does treat them differently with a hashtable operand (see below)).
The only meaningful use of a [pscustomobject] cast is (currently) the syntactic sugar supported with a single (ordered) [hashtable] operand, to convert a hash table to a custom object; e.g., [pscustomobject] @{ one = 1; two = 2 } (Sadly,  it doesn't work with  a generic Dictionary`2 instance, though #13727 may change that; it also doesn't currently work as an array cast - see #13838).
All other uses should report a statement-terminating error.
Note: While this is technically a breaking change, existing instances of such casts amount to already broken code.

Unexpectedly, the above is equivalent to [psobject] 42 and wraps the operand in a large largely invisible [psobject] instance (except it isn't always invisible - see #5579).

Oh, I guess it is fundamental PowerShell behavior. It seems there was very old PR for this.


Readme copyedits

Copyedits and some formatting tweaks to the README.

This change is

@sandersk, is this obsolete?

Yes, let's drop this. I'll just do a follow-up PR with any changes I still have.


[AppService] az webapp config set: Add new parameter --runtime to allow users to update their stack by using single parameter

Related command
az webapp config set
Description
Add new parameter --runtime for az webapp config set to allow users to update their stack by using single parameter
Testing Guide

az webapp config set -g MyResourceGroup -n MyApp --runtime "java:17:TOMCAT:10.0"
Update the stack of MyApp to Java 17 and Tomcat 10
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

AppService

@jvano


add exclamation mark to the title

Hey! What do you think of my proposal of adding exclamation mark to the title?


realTime not working catching events yet

Type: BUG
Version: Latest GIT	1.0.0-rc.8.5
Hello! I'm having this same problem as https://github.com/mean-expert-official/fireloop.io/issues/68
I see in the console as if the events were launched for upsert and create, but nothing arrives to the browser.
When I do a remove then I receive the change event, also when I first subscribe I receive the change event.
  constructor(private rt: RealTime) {
    this.rt.onReady().subscribe((status: string) => {
      console.log('rt ' + status);
      this.transactionRef = this.rt.FireLoop.ref<Transaction>(Transaction);
      this.transactions = this.transactionRef.on('change');
      this.transactions.subscribe((txs) => {
        console.log('Change TXS', txs)
      });
    })
  }

@mean-expert/loopback-component-realtime: FireLoop starting create: Transaction: {"data":{},"parent":null}
@mean-expert/loopback-component-realtime: FireLoop value broadcasting
@mean-expert/loopback-component-realtime: FireLoop child_added broadcasting
@mean-expert/loopback-component-realtime: FireLoop change broadcasting
@mean-expert/loopback-component-realtime: FireLoop stats broadcasting

https://www.dropbox.com/s/z03fr8fdia5wnn0/Screenshot 2017-05-28 03.13.14.png?dl=0
https://www.dropbox.com/s/aiqkee4b6cnhmcc/Screenshot 2017-05-28 03.12.24.png?dl=0

Hi @medisoft, it sounds to me that you are not configuring your Transaction model to use the FireLoop mixin as stated in the README https://github.com/mean-expert-official/loopback-component-realtime#enable-model-to-use-fireloop-api
{
  "mixins": {
    "FireLoop": true
  }
}

I hope this helps you fix your problem

@jonathan-casarrubias, i'm not sure where to place this, is it in server/model-config.json? thanks

@davexs2020 A mixing should be registered within any model you want to implement that functionality. In your case is common/models/transaction.json file, something like:
{
  "name": "Transaction",
  "plural": "transactions",
  "properties": {},
   "mixins": {
    "FireLoop": true
  }
}

I hipe this helps you progress on this..
Cheers
Jon

it's working with
{   "mixins": {     "FireLoop": true   } }
i think you should have generate that piece of code in fireloop generated code or in your tutorial (http://mean.expert/2016/11/10/angular-2-real-time-statistic-charts/), i mean, i almost give up from trying fireloop, becuse that official tutorial not working for me.
thanks.

@zuhrulumam I apologize you had a hard time with this configuration, it's a recent implementation and missed to update all the docs and generators.
I'm just publishing an update to generate the model config including by default the FireLoop: true flag.
Regards
Jon

I am sorry to reopen this. But this was a general problem for me too. Wasted several hours on this. I too put the mixins in model-config.json. Please be more specific on your documentations, beginners get stuck a lot because of it. Thanks a lot.

@appunnicer13 this has been specified in the readme almost a month ago
https://github.com/mean-expert-official/loopback-component-realtime#enable-model-to-use-fireloop-api
But also the fireloop cli already generates with the flag enabled
https://github.com/mean-expert-official/fireloop.io/blob/master/generator-fireloop/generators/model/index.js#L50
I'm sorry @appunnicer13 but you are in a totally different position than @zuhrulumam he didn't have a reference for this nor the cli was generating the configuration. In your case you could use the cli or simply fully read the readme for that information

{
"mixins": {
"FireLoop": true
}
}
I am sorry, I read but I could only see this. May be it's not clear
because I am new to this. It wasn't exactly specifying which file I should
add this. Totally noob, sorry if I am saying anything wrong.
On 21 Jul 2017 7:38 p.m., "Jonathan Casarrubias"<EMAIL_ADDRESS>wrote:
@appunnicer13 https://github.com/appunnicer13 this has been specified in
the readme almost a month ago
https://github.com/mean-expert-official/loopback-component-realtime#enable-
model-to-use-fireloop-api
But also the fireloop cli already generates with the flag enabled
https://github.com/mean-expert-official/fireloop.io/blob/master/generator-
fireloop/generators/model/index.js#L50
I'm sorry @appunnicer13 https://github.com/appunnicer13 but you are in a
totally different position than @zuhrulumam https://github.com/zuhrulumam
he didn't have a reference for this nor the cli was generating the
configuration. In your case you could use the cli or simply fully read the
readme for that information
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mean-expert-official/loopback-component-realtime/issues/38#issuecomment-317010981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOYLC3OQlzH0kFaIXb8UW_NlZE9vREjzks5sQLDtgaJpZM4NolZN
.

@appunnicer13 based on your last comment, I think I can improve the readme with a link for people to understand what exactly a mixin is.
It does state at the beginning of the paragraph

From version 1.0.0-rc.8 you need to specify in which models you want to enable FireLoop events to be sent.

In there I state that in whatever model you want to enable the real time, you need to enable fireloop by setting up the mixin.
So I believe the missing part is to explain what exactly is a mixin, in that way you would be able to figure out that a mixin is installed within the model.json files, for those models that you want to enable the mixin.
Based on this issue I will improve the documentation in order to stop assuming that people will know what a mixin is and be more specific in those therms.
Thanks for giving back your feeback.
Regards!
Jon

Thanks for supporting I have created a demo app myself please check it out
if you have time, Separated both front end and backend
https://github.com/appunnicer13/sample-notemaker
https://github.com/appunnicer13/sample-notemaker-frontend
here is a demo link http://sampleapp1.appunni.xyz/
Regards
Appunni M
https://www.linkedin.com/in/appunni-m-b40056a0/
https://github.com/appunnicer13
On Fri, Jul 21, 2017 at 9:32 PM, Jonathan Casarrubias <
<EMAIL_ADDRESS>wrote:

@appunnicer13 https://github.com/appunnicer13 based on your last
comment, I think I can improve the readme with a link for people to
understand what exactly a mixin is.
It does state at the beginning of the paragraph
From version 1.0.0-rc.8 you need to specify in which models you want to
enable FireLoop events to be sent.
In there I state that in whatever model you want to enable the real time,
you need to enable fireloop by setting up the mixin.
So I believe the missing part is to explain what exactly is a mixin, in
that way you would be able to figure out that a mixin is installed within
the model.json files, for those models that you want to enable the mixin.
Based on this issue I will improve the documentation in order to stop
assuming that people will know what a mixin is and be more specific in
those therms.
Thanks for giving back your feeback.
Regards!
Jon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mean-expert-official/loopback-component-realtime/issues/38#issuecomment-317041129,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOYLC5Vs86r6_f7XXpkk1wg3xbx8RFquks5sQMt4gaJpZM4NolZN
.

@jonathan-casarrubias Hi, I'm getting the same error. any update on this issue?
I appreciate if there is an alternative way.
Thanks

@jonathan-casarrubias Sorry man.
My bad, It's working now. :)


Width (height?) not updating on page resizing

This issue also exists when directive compiles before its parent directives have set widths

If we could give it the jquery selector for the parent container we could watch the parent container for changes to height and recalculate.  This would changing the height on the fly which is something I am trying to hack together now.

I've created PR that fixes this issue.

I don't think I have resolved the issue with the directive compiling before it's parent.  I believe I have run into the effects of that issue.  I have a resolution in place (ng-if) which prevents the directive from rendering until the necessary data in the parent controller is present.


vis_AH_layout function attribute value mismatch

When using the vis_AH_layout function I get the following error:
Error in i_set_vertex_attr(x, attr(value, "name"), index = value, value = attr(value,  :
  Length of new attribute value must be 1 or 149, the number of target vertices, not 148

This occurs for both my own data and the USAH input in the package. In both instances the attribute value is one less than the target. For my dataset, the edgelist and vInfo are both 148 and all values match, so there is no discrepancies between them. My dataset also works with some of the other functions in the package.
Here is what I am working with at the moment:
dh = read_adjMat("myspreadsheet.xlsx")

dv = dh %>% select(level, levelName_full, levelName, Node)

de = dh %>% adjMat_to_edgelist(vInfo = dv); de

visKey =
  tribble(
    ~layer, ~min, ~max, ~addR, ~r,
    1,250,290,0,1,
    2,250,290,0,2,
    3,260,280,0,3,
    4,260,280,0,4,
    5,250,290,0,5)

visKey

plot_layout =
  vis_AH_layout(edgelist = de,
                vInfo = dv,
                minSpacing = 0,
                maxSpacing = 1000,
                key = visKey)

plot_layout

Any assistance is appreciate! My R knowledge is fairly basic and I feel like I have exhausted my troubleshooting abilities.
Edit: I am able to get the function to work with a simple one node per level abstraction matrix. Still unclear why it is not working for larger models.

The issue seems to be due to the dummy vertex added when there is more than one functional purposes (level 1 node) in the AM. I was able to produce the plot outputs when I removed all functional purposes, bar one, from my AM.
There is a separate issue with the call to igraph::graph.data.frame in edgelist_to_igraph function which requires updating to igraph::graph_from_data_frame.


[SPARK-35723] set k8s pod container request, limit memory separately.

Currently spark driver and exec pod request and limit memory can only be same, not able to set separately when cluster get diff memory limit and request quota. then not able to fully use cluster memory resource.
E.g. cluster get total 50G memory request, 200G memory limit, but every spark pod limit memory are same as request memory, then in this cluster total memory spark can use is depends on the smaller one: 50G

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

cc  @dongjoon-hyun @yaooqinn @Yikun, can anyone help review?

The cpu limits are set by spark.kubernetes.{driver,executor}.limit.cores. The cpu is set by spark.{driver,executor}.cores. The memory request and limit are set by summing the values of spark.{driver,executor}.memory and spark.{driver,executor}.memoryOverhead. Other resource limits are set by spark.{driver,executor}.resources.{resourceName}.* configs.

Referring to the doc, we can actually set driver pod memory alone

The cpu limits are set by spark.kubernetes.{driver,executor}.limit.cores. The cpu is set by spark.{driver,executor}.cores. The memory request and limit are set by summing the values of spark.{driver,executor}.memory and spark.{driver,executor}.memoryOverhead. Other resource limits are set by spark.{driver,executor}.resources.{resourceName}.* configs.

Referring to the doc, we can actually set driver pod memory alone

Current config is setting the pod request.memory and limit.memory in the same value from by summing the values of spark.{driver,executor}.memory and spark.{driver,executor}.memoryOverhead.
But request.memory and limit.memory are different kind of params from k8s, always keep same value may not a good practice. In most case the request memory quota we can get is always smaller than limit.memory from infrastructure team. If spark pods request.memory can only same as limit.memory, then total memory we can use is based on the smaller one.
requests 定义了对应的容器所需要的最小资源量。
limits 定义了对应容器最大可以消耗的资源上限。

There was a discussion about this on the spark dev mailing list earlier, hope it helps you.
spark executor pod has same memory value for request and limit

@zwangsheng do we have any timeline to add this feature？

spark executor pod has same memory value for request and limit

You can found this in the mail discussion:

There is a very good reason for this. It is recommended using k8s that you set
memory request and limit to the same value, set a cpu request, but not a cpu
limit. More info here https://home.robusta.dev/blog/kubernetes-memory-limit


tslint.json cleanup

Context
I was getting a version mismatch warning from VSCode that complained about the version of TSLint in VSCode and the version of TSLint in PATH were different. Adding tslint as a devDependency fixes this problem.
Once that issue was resolved, I found that many of the options in tslint.json were either deprecated or slightly different than the format defined in the tsling.json schema (VSCode uses this to verify the format of the TSLint JSON config).
Notable changes

Pinned TSLint devDependency to version 4.4.2
Enabled interface-name TSLint config. Instead of ignoring interfaces names, it will now verify that all interfaces names are not prefixed with I. My reasoning here is that it's better to enforce a single style in order to keep contributions from multiple people consistent (myself included!)
Enabled no-consecutive-blank-lines. Enabling this did not result in any new warnings and resolved a warning for an invalid value in tslint.json. We can disable/remove this one if you don't like it.
Removed a number of deprecated TSLint options that are now handled by the Typescript compiler (tsc): label-undefined, no-duplicate-key, no-unreachable, and use-strict.

I have no objection about interface-name.
Looks good for me.
I got this message. Would I drop no-unused-variable similarly?
no-unused-variable is deprecated. Use the tsc compiler options --noUnusedParameters and --noUnusedLocals instead.

@misogi Correct! I must have missed that one. :)


Bug: Can't install with pyndantic >2.0 (despite documentation saying otherwise)

Expected Behaviour
I was expecting to be able to install the Python parser extension of the powertools despite needing pydantic 2.0 after reading the docs here.
Current Behaviour
I am blocked from installing due to conflicting dependencies. The source of the issue is pyproject.toml.
If this is unintentional behavior, can you update the pyproject.toml requirements, otherwise you should correct the documentation. I may make a PR.
Code snippet
ERROR: Cannot install -r requirements-runtime.txt (line 4), aws-lambda-powertools[all]==2.31.0 and pydantic>=2.0.3 because these package versions have conflicting dependencies.

The conflict is caused by:
    The user requested pydantic>=2.0.3
    geojson-pydantic 1.0.1 depends on pydantic~=2.0
    aws-lambda-powertools[all] 2.31.0 depends on pydantic<2.0.0 and >=1.8.2; extra == "parser" or extra == "all"

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

Possible Solution
No response
Steps to Reproduce
pydantic>=2.0.3
aws_lambda_powertools[all]==2.31.0
boto3==1.26.19
geojson-pydantic==1.0.1
Powertools for AWS Lambda (Python) version
latest
AWS Lambda function runtime
3.10
Packaging format used
PyPi
Debugging logs
No response

Hey @xaviernogueira! Thanks for opening this issue! The reason you can't install Pydantic 2+ is because you are using aws_lambda_powertools[all] and Pydantic v1 is the default version in Powertools.
Pls update your requirements.txt to use aws_lambda_powertools[aws-xray-sdk, fastjsonschema] and then you can bring Pydanticv2.
Pls let me know if this works.
Thanks

I'm closing this issue and feel free to reopen it if necessary @xaviernogueira!
Thanks

@leandrodamascena
So I tried your solution and got the warning:
WARNING: aws-lambda-powertools 2.31.0 does not provide the extra 'aws-xray-sdk'
WARNING: aws-lambda-powertools 2.31.0 does not provide the extra 'fastjsonschema'

I can't tell if those extras are installed, but it did allow me to resolve the environment!
That said you guys should consider supporting pydantic 2.0, I may look into making a PR.

Hello @xaviernogueira! My apologies here, I wrote the wrong command! You can try: pip install "aws_lambda_powertools[validation,tracer]"
We support Pydantic v2, but we do not make Pydantic v2 our default version because we can break customers who already use Powertools in their apps, and have not yet migrated their code from Pydantic v1 to v2.
We will drop support for Pydanticv1 and make Pydanticv2 the default version in Powertools V3 (we don't have a release estimate yet).
Thanks

@leandrodamascena Thanks for the quick response! I'll let you know how it goes.

Worked!

Thanks for letting me know @xaviernogueira! Feel free to ping us or open issues whenever you need!


TransitionIdentifiers.jl --> TransitionIndicators.jl

Needed for running Pkg.develop("https://github.com/JuliaDynamics/TransitionIndicators.jl")

@JanJereczek remember: from now on you don't use your fork. You branch directly in this repo.


Pp 1923/ePDQ capture support

WHAT
A brief description of the pull request:

New EpdqCaptureResponse to map ePDQ response payload for capture operation, only handling status code 91 as CAPTURE SUBMITTED and disregarding any other status as CAPTURE ERROR. We also map ePDQ PAYID to our transactionId
A charge should never be able to be cancelled when in AUTHORISATION_SUBMITTED, since ePDQ doesn't support cancelling of transactions in the process of being authorised (status 50 or 51)
Fixed EpdqOrderRequestBuilder to produce the correct request payload taking transactionId and merchantCode into account. Mapped ePDQ PAYID to transactionId and PSPID to merchantCode.
Implemented the capture operation in EpdqPaymentProvider, relying on EpdqOrderRequestBuilder to generate capture request.

There were no errors, go have a cup of coffee...

There were no errors, go have a cup of coffee...


Missing main.js

I'm a newbie so please forgive if this is obvious.  I've tried various installations (via BRAT, unzipp etc..) and each time I receive the error "missing main.js". I would love to use this plugin. Suggestions?

Same issue here when trying to install using BRAT.

@drnola Manual installation works. Just download the zip, uncompress / extract it and then move the uncompressed folder to your Vault/.obsidian/plugins folder. Restart Obsidian and enable the plugin in the community plugins list.


How to setup?

I want to run your sample code, but it shows some errors. I guess it is something wrong with the setup. But you don't provide like setup.py. So how do we setup this package?

if you're getting the same errors I got (ModuleNotFoundError), then:
It's a relative/absolute import issue. If we want to contain all of the source files to the hunga_bunga folder, an easy fix is to edit the imports in regression.py, classification.py and _init.py___ to reference the folder they are in.
e.g. in _init.py___ change:
from regression import HungaBungaRegressor from classification import HungaBungaClassifier
to
from hunga_bunga.regression import HungaBungaRegressor from hunga_bunga.classification import HungaBungaClassifier
and do the equivalent for the other source files where it throws import errors.
I'm sure there's a more elegant solution, but I couldn't find one quickly, and this quick fix works :)


Add option to check for bit rot during backup creation

There is currently no way of knowing if the original files deteriorate.
restic backup skips files based on mod time. Bit rot will not be detected at all, but the backup will contain the original file. If the owner modifies the file without discovering that the file is degraded, or restic backup --force is run, restic will happily backup the new file. At the time when the owner discovers that the file is degraded, it will be very hard to find and recover the correct bits. Worst case scenario, the file is lost due to backup pruning.
restic backup --force hashes all files, regardless of mod time. Bit rot will result in a new hash, and restic will interpret this as a normal modification and backup the file. The owner will be none the wiser, and will similarly have problems when the bit rot is detected.
I suggest implementing a third option, similar to restic backup --force that checks mod time to determine if the file has been changed intentionally. If the file is modified without a modified mod time it would log or err out.
(This could also be extended to verify other meta data, such as checking that the modified date is not earlier than the one recorded in the previous backup.)
This might be out of the scope of a backup program, but if we are already hashing the files, and recording the previous hashes, most of the work is already done.

You're completely right about that @oysols. And I don't think it is out of scope of restic, to check full integrity of the data it saved.
I only think this should be implemented via check command, check --full or similar. But, a good idea, definitely. 👍

Great idea, implementation will need to be discussed. Thanks for reporting this!

+1

+1

+1

Intermediate functionality which would support this could be a dump of file hashes which users can just feed to sha256sum -c; or even just a textual retrieval of hash chunks so users like myself can write our own utilities to verify against the chunks. Use case is verifying file system files after a crash or a detected drive failure and data restoration on backups which are larger than half the size of the underlying storage and/or over a network where network hashing would be unnecessarily slow.
Really really love restic. With even just a pathway to a sum-based way to verify client drive contents it would essentially be feature-complete for someone like myself.
Much obliged for all the work you all have done. Wonderful software.

There's a secondary utility of a more user-accessible interface to the hash tree structure on a file-and-directory basis might also be for generating existence proofs for timestamping in something like OpenTimestamp. A restic backup having extra utility for proving the historical existence of the backed-up contents would be pretty spectacular. :-)

Found this thread while searching for ideas on how to do bitrot detection without needing to be an enterprise sysadmin with an expensive NAS.
I concur, if hashing is already being done then most of the work is already done :)

Bitrot detection by the diff command requires both sides to be a backup, and the right side a --force'd backup.
It would be great if diff would support the right side to be the local filesystem, diffing directly without wasting backup storage and time for a full backup just to check for bitrot. Especially when backup storage charges for minimum retention of used blocks.


When launching the app around 10 times from the editor, there is approximately 1 constant error _create_offer: Failed to gather local ICE candidates

Godot version
v3.5.2.stable.official [170ba337a]
Plugin version
all v1.0.0 (rc,beta with libdatachannel)
System information
Windows 10, 64bit
Issue description
When launching the app around 10 times from the editor, there is approximately 1 constant error _create_offer: Failed to gather local ICE candidates <C++ source code>src/WebRTCLibPeerConnection.cpp:209 @ _create_offer().
I mean, in order to catch this error, you need to restart the application 10 times from the Godot editor, and not repeat the code itself 10 times. And as soon as an error occurs, all subsequent repetitions of the code also cause an error.
I also want to note that this error appears only in version v1.0.0 with libdatachannel and is not present in version 0.5 - WebRTC 4472.
Steps to reproduce
To catch this error, you need to restart the application about 10 times from the Godot editor with this code:
	for _i in 5:
		var wc1:WebRTCPeerConnection = WebRTCPeerConnection.new()
		var err1
		err1 = wc1.initialize()
		if err1 == OK:
			var channel:WebRTCDataChannel = wc1.create_data_channel("play",{"id": 1, "negotiated": true})
			if channel != null:
				err1 = wc1.create_offer()
				if err1 != OK:
					print_debug("Periodic Error!!!")

Minimal reproduction project
No response

I'm unable to reproduce using the latest plugin version 1.0.4, and Godot 3.5.3/4.2.1 .
Can you confirm if this is still an issue?

I'm unable to reproduce using the latest plugin version 1.0.4, and Godot 3.5.3/4.2.1 .
Can you confirm if this is still an issue?

I checked with version 1.0.4 on Godot 3.5.3 and indeed I did not find this bug. It looks like this issue can be closed.

I unfortunately discovered that this error still pops up with variable randomness (from about 1 to 10,000 or more attempts) and to explicitly detect it I used this code:
func _ready():
	var errMax:int = 10
	for _i in 10000:
		var wc1:WebRTCPeerConnection = WebRTCPeerConnection.new()
		var err1
		err1 = wc1.initialize()
		if err1 == OK:
			var channel:WebRTCDataChannel = wc1.create_data_channel("play",{"id": 1, "negotiated": true})
			if channel != null:
				err1 = wc1.create_offer()
				if err1 != OK:
					print_debug("Periodic Error!!!:_i:",_i)
					errMax -= 1
					if errMax <= 0:
						break
		wc1 = null


Feature/network implementation

Added agent transfer between server and client

Please do not make PR's to main, make them to release/0.2

Looks good to me


Refactor css

This seems to have increased the size of CSS files with no good advantage. I am withholding this PR.


⚠ Roblox has degraded performance

In 83ed0dc, Roblox (https://www.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 3027 ms

Resolved: Roblox performance has improved in 08cf16b.


Fixed several bugs.

Hi,
Thanks for the help ;-)
I've handpicked from this PR some changes (see 783a43e955f87c4a17df3daf225497afca8759d8, 9389dd2f34da9b6e2ced02cc354d222d3009f7d6)


Add a Divider block

Divider block for the Issue #4

Hi @praem90
Thanks for taking this up.
Please add screenshots, and update the README.md file

Attaching the screenshot using

Thanks for helping out @praem90


license: use same license as crossplane

Just adding the same license used on Crossplane. Excited to see how this project evolves! :)

Oh haha, just saw this! I was thinking about the same thing today : ). Thanks @hasheddan ! This means I don't need to have this in my changeset anymore

I'm going to try the rebase and merge strategy to see how well that works

I'm going to try the rebase and merge strategy to see how well that works

For future reference: it didn't like it, because the base branch requires signed commits, and GitHub can't automatically sign rebased commits. I clicked "retry" and it used a merge strategy, which is also fine


Invisible menu bar

On Ubuntu 24.04 when run either as DEB or AppImage the menubar is strangely invisible.
I could not find any way to modify this in setting.

PS: Thanks for an awesome project like this.

Strange! Definitely should not look like this, but I haven't been able to test on Ubuntu that much yet. All these different menu systems across OS's is tricky to manage, I'll have a look!

I tested on Fedora and it still does the problem. Its probably related to GNOME title and menu bar.

:rocket: This issue is resolved in version 1.28.1 of JET Pilot!


请添加对its站点的支持

<!--
1、在发起请求前，请先确认是否在已支持的架构内；支持

- 站点名称：its
- 站点地址：https://shadowthein.net/
- 站点描述：总合

- 资源类型：总合
- 开放注册：否
- 是否连坐：是
- 站点规则：DO NOT:
publish the full name, URL or access details of the tracker, whether directly or implicitly. Always refer to "iTS" instead.
trade an account or an invite to iTS. Giving away an invite to a stranger is the same as trading it.
give away your account.
buy or sell accounts or invites to any filesharing site, whether directly or indirectly (e.g.: for seedbox access).
create multiple accounts or share accounts with others. Your account is personal and is not to be shared with others.
cheat or attempt to cheat on the tracker.
endanger or attempt to endanger the security of the tracker in any way.
publish a review of the site without staff approval.
upload iTS exclusive releases to other file sharing sites without clearing it with the iTS uploader.
recruit iTS members to other sites through iTS resources or systematically invite iTS users to other sites without staff approval.
tag an upload as personal or internal if you are not the original ripper/releaser.
use offensive usernames or titles, or misleading ones such as "Admin", "Staff" etc.
be a repeat offender.
Depending on the severity of the action, some of these may even get you banned on other sites as well!

Please follow the issue template


nvidia-bug-report.sh requested for further information regarding why the script failed.

Hello, the README says that the script is currently not working right now, so I would like if you can run nvidia-bug-report.sh to generate a report of all the messages and errors put out by vgpu manager and vgpud. This will help in figuring out what went wrong. Thank you.

Here is the requested file: nvidia-bug-report.log.gz
The interesting section is this:
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: notice: vmiop_env_log: vmiop-env: guest_max_gpfn:0x0
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: notice: vmiop_env_log: (0x0): Received start call from nvidia-vgpu-vfio module: mdev uuid 38512783-4893-47f7-9179-b0594167e86b GPU PCI id 00:01:00.0 config params vgpu_type_id=50
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: notice: vmiop_env_log: (0x0): pluginconfig: vgpu_type_id=50
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: notice: vmiop_env_log: Successfully updated env symbols!
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: NVOS status 0x56
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: Assertion Failed at 0xf69873bf:293
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: 11 frames returned by backtrace
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: /lib/x86_64-linux-gnu/libnvidia-vgpu.so(_nv005021vgpu+0x18) [0x7ff3f69cc3c8]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: /lib/x86_64-linux-gnu/libnvidia-vgpu.so(+0xa3e3b) [0x7ff3f6982e3b]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: /lib/x86_64-linux-gnu/libnvidia-vgpu.so(+0xa83bf) [0x7ff3f69873bf]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: /lib/x86_64-linux-gnu/libnvidia-vgpu.so(+0xa98c7) [0x7ff3f69888c7]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: vgpu() [0x413e72]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: vgpu() [0x4140e9]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: vgpu() [0x40e9d7]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: vgpu() [0x40c2c9]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: vgpu() [0x40bc7c]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xeb) [0x7ff3f6e7109b]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: vgpu() [0x4033ba]
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: (0x0): init_device_instance failed for inst 0 with error 1 (error setting vGPU configuration information from RM)
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: (0x0): Initialization: init_device_instance failed error 1
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_log: display_init failed for inst: 0
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_env_log: (0x0): vmiope_process_configuration: plugin registration error
Feb 13 15:20:08 Debian-dom0 nvidia-vgpu-mgr[1562]: error: vmiop_env_log: (0x0): vmiope_process_configuration failed with 0x1f
Feb 13 15:20:08 Debian-dom0 kernel: [nvidia-vgpu-vfio] 38512783-4893-47f7-9179-b0594167e86b: start failed. status: 0x1

According to nvstatuscodes.h the status code 0x56 is NV_STATUS_CODE(NV_ERR_NOT_SUPPORTED, 0x00000056, "Call not supported"). As far as I can tell this is returned by the code inside the nv-kernel.o_binary file and I have not been able to figure out why.

error 1 (error setting vGPU configuration information from RM)
As far as I know, this is a pretty typical error when you’re using the wrong graphics card. Because Nvidia only officially supports Red Hat Linux, I used that for these tests and ram the Red Hat hypervisor on top of a KVM hypervisor. The level 1 hypervisor spoofed the PCI ID for the Red Hat to be able to detect a “Tesla P4,” and what happens is that instead of getting loads of errors, I usually get the same error 1. So far, I haven’t looked at your whole bug report, but I will compare it to some of my previous testing and try to dig up a few details. Right now I have a feeling that it has to do with ECC memory, a features that has to be disabled for vGPU to work in any way. However on GeForce, you can’t turn on or off ECC so there needs to be some additional modifications. I’ll let you know when more information surfaces.

I am closing this since the bug report has been provided and the new README explains what causes the failure.


add check go vet in lint target.

Describe the PR
add go vet check.

with.
https://github.com/swaggo/swag/pull/336


Gausum and blazing copper missing

I have no idea if this is intentional or not, but except those two I didn't notice anything else out of order

you didn't tell me what version you were on
if you're on 1.19.2 it's intentional, I rewrote the mod for that version of minecraft.


Game crashes if character has noteskin

Describe your bug here.
The game crashes if a character has a special noteskin.
Command Prompt/Terminal logs (if existing)
Image is in the replies.

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

i just forgot the xml lmao


ES Modules broken on npm@7

Environment
Knex version: 0.21.17
Database + version: N/A
OS: N/A
Bug

Explain what kind of behaviour you are getting and how you think it should do

When setting type: 'module' in package.json knex fails to use ESM to load the migration files on npm >= 7.0.0. This works on lower versions. The reason being that npm has removed the environment variable which knex uses to determine if it should use ESM or CommonJS (https://github.com/knex/knex/blob/b20a31fe5d0256234e1542c1ec4cbe2082f9970e/lib/migrations/util/import-file.js#L2). See this NPM RFC for more details https://github.com/npm/rfcs/blob/a044b355af60310b05ad1995fdc1cf672243a4e0/implemented/0021-reduce-lifecycle-script-environment.md. If the file is named knexfile.mjs it works as expected since the filename check triggers the usage of ESM.

Error message

Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /project-root/config/knexfile.js
require() of ES modules is not supported.
require() of /project-root/node_modules/knex/lib/util/import-file.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename knexfile.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /project-root/package.json.

    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1080:13)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Module.require (internal/modules/cjs/loader.js:952:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at importFile (/project-root/node_modules/knex/lib/util/import-file.js:13:7)
    at openKnexfile (/project-root/node_modules/knex/bin/cli.js:26:22)
    at initKnex (/project-root/node_modules/knex/bin/cli.js:47:13)
    at Command.<anonymous> (/project-root/node_modules/knex/bin/cli.js:170:32)
    at Command.listener [as _actionHandler] (/project-root/node_modules/commander/index.js:426:31)

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to MySQL or PostgreSQL, then single file example which initializes
needed data and demonstrates the problem.

All that's needed is a project with type: 'module' in the package.json and a knexfile ending in .js.
npm's recommended migration route is to use the variable npm_package_json which points to the package.json, load the JSON and then read values from there.

@richardsimko Uh-oh, thank you for pointing out the issue! Would you consider sending a PR with a fix?

Sure, I can probably put something together. Just in case anyone else stumbles on this before a fix is released, a workaround is to change the knexfile to end with .mjs.

@kibertoad I opened #4296

Released in 0.95.0-next2

Released in 0.21.18


GH-933: Fix NPE in PublisherCallbackChannelImpl

Fixes https://github.com/spring-projects/spring-amqp/issues/933
cherry-pick to 2.1.x

Gary, does Travis report say anything to you:
org.springframework.amqp.rabbit.core.RabbitTemplateIntegrationTests > testReceiveInExternalTransactionAutoAck FAILED
    java.lang.AssertionError at RabbitTemplateIntegrationTests.java:600

?

I don't see how your change affects that test, so I don't mind to merge this.
Just let me know when!

I'll take a look at it; it built cleanly for me but we might as well fix it.

... and cherry-picked to 2.1.x.
So, I have missed to squash for master again 😢


Enable Travis builds

This repo contains a .travis.yml but https://travis-ci.org/laumann/compiletest-rs is currently showing no builds. I believe @laumann will need to flip a switch in https://travis-ci.org/account/repositories.

I've enabled Travis. I remember I was a little concerned will the permissions that Travis wanted. I don't remember AppVeyor wanting that much. But we should be able to set it up now.
I had a question in #97 that I never got answered: Could the AppVeyor builds not be extended to run the tests in test-project, instead of creating a Travis build?

Travis has always had a better user experience than AppVeyor in my experience. I stick to Travis as primary CI, adding AppVeyor only if I need to build on Windows. Travis recently announced Windows support so I may be dropping AppVeyor altogether.


Are you upgrading to v11?

As the title says. Any news?

I didn't test all functions of the module - but by changing the module.json to be v11 compatible these features at least work fine with no console errors:

Auto rolling NPC initiative
Starting battle music
Playing fanfare
Returning to original playlist

I'll hold off making a manifest PR until more testing can be done, I just had a vested interest in those features (XP etc is handled elsewhere in pf2e, and I use milestone advancement in 5e). I can run one up if needs be though, but it's only changing a couple of lines.

sorry about the delay, I am planning on updating to v11 and im glad to hear most of the features just work. Ill go through and test some of the other stuff and will hopefully have it updated soon

finished testing and updating for version 11, let me know if you find any other bugs.

Hello,
Just tested that parts I use -
Prompt for music or sounds when combat has started 👍
Token Markers to show whose turn it is 👍
Remove NPC when defeated 👍
Music goes off when combat is over 👍
Perfect Thank you


Set MipMap bias when using upscaling

Unfortunately there seems to be absolutely no gain in MSFS for this change.


[For Discussion] Move dynamic serialization options to Azure.Core.Serialization namespace?

This PR asks the following question: what if we generalized the serialization options used by DynamicData and made them available, e.g. to anonymous types passed to RequestContent.Create()?
This is an initial proposal for what that might look like, and does the following:

Renames DynamicCaseMapping.PascalToCamel to PropertyNameHandling.ToCamelCase
Renames DynamicDateTimeHandling to DateTimeHandling, and adds a None value at 0
Moves both of these to the Azure.Core.Serialization namespace
Adds an overload of RequestContent.Create(object) that takes these enums as parameters
Makes DynamicDataOptions internal since these are no longer specific to DynamicData

API change check
API changes are not detected in this pull request.

Closing in favor of work tracked by https://github.com/Azure/azure-sdk-for-net/issues/36642,
likely either https://github.com/Azure/azure-sdk-for-net/pull/36719 or https://github.com/Azure/azure-sdk-for-net/pull/36790.


Open source media player element

Proposal: Open source media player element

Summary
Currently MediaPlayerElement has so many bugs and shortcomings, it's terriable,

It will be open sourced with WinUI 3. But I have never heard about its bugs. Maybe you are talking about MediaPlayer? I would hope to see it open sourced too, but it isn't part of WinUI.

I'm using MediaPlayerElement on uwp. Currently it causes whole screen black when playing some videos, can't keep playback rate when changed video source, can't get closed caption's exact font size, transport control get massed after adding cunstom icons, etc.
I have seen so many voices on user voice forum.

But it's a big step after MediaPlayerElement open sourced.


?escaped_fragment= returns blank white page

Doing http://stickyblood.co.uk/?escaped_fragment= returns a blanks white page but if I do http://myprerender.com/http://stickyblood.co.uk/ it works fine.
I'm using prerender-node middleware i.e
app.use(require('prerender-node').set('prerenderServiceUrl', 'http://myprerender.com'));

When you go to
http://stickyblood.co.uk/?_escaped_fragment_=
are you seeing any requests on the console output of your prerender server?
What status code are you seeing on that blank page response?

Think I've worked it out, must be due to my nginx config as this is the output at=info method=GET path="/http://<IP_ADDRESS>:10070/?_escaped_fragment_="

Ah, ok. Were you able to fix that? You can past host in prerender-node to get ride of that port when testing if it's problematic. You can also fix that upstream if needed for testing.
app.use(require('prerender-node').set('host', '<IP_ADDRESS>').set('prerenderServiceUrl', 'http://myprerender.com'));

Hello
can you guys help me on this
i am also getting blank page when access url like this to check if prerender working or not
http://dkfashions.missio.io/?_escaped_fragment_=
please help
Thanks,
Ekam


Stripes only visible on first column

Hi,
I would like to see the rows of a jstree-grid an alternating colors, so I use the following jstree option:
$.jstree.defaults.core.themes.stripes = true;
But this works only for the first column, the remaining columns have no background.

I suspect we don't pick up the stripes from the theme. Open a jsfiddle showing it?

Sure :-)
https://jsfiddle.net/gv3ho38w/

Yeah, now I see it.
Nothing in the code currently supports picking up on the stripes from the jstree themes param. More than happy to take a PR on it and look at it, but unless someone brings one it, it won't get done. This project has no sponsorship. Going to close it, but very happy to take a PR. Interested?


yarn: init at 0.18.1

Motivation for this change
Currently yarn is packaged as npm package, but i don't want to use npm, no really. So i packaged it as standalone package, from builds that yarn already provides. This allows to install with any npm residuals, amen.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks. tested and works.


Add random permutation to the indices.

Creating this pull request for Issue #43 . I have added random permutation to the fit function and predict function so that the models cannot remember the first item being the target.
I have created a new directory for this pull request because I have changed too many things in the original directory, and it might contain sensitive contents. However, I have tested the diff using the following two models and the results seem to be fine. In my original directory, the code works fine with KDA, SASRec, ComiRec and many other models. Please let me know if you observe otherwise.
python main.py --model_name TiSASRec --emb_size 64 --num_layers 1 --num_heads 1 --lr 1e-4 --l2 1e-6 --history_max 20 --dataset 'Grocery_and_Gourmet_Food'

python main.py --model_name TiMiRec --emb_size 64 --lr 1e-4 --l2 1e-6 --history_max 20 --K 6 --add_pos 1 --add_trm 1 --stage pretrain --dataset 'Grocery_and_Gourmet_Food'

I have added some comments to make the code easier to understand.

Thanks for your contribution! I have tested and it seems to work well. Although the final results will be a little different from previous ones (cause the random process has been changed), the differences are generally negligible.
One question is whether it is fine to only add random permutation to the fit function. It seems to be enough to prevent the model to remember the position.

Hi I guess adding random permutation to the fit function should also be enough. Could you check the updated pull request? Thanks.


KOGITO-5536 Changed version names for clarity in pipelines

https://issues.redhat.com/browse/KOGITO-5536
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Your code is properly formatted according to this configuration
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to retest this PR or trigger a specific build:

Run all builds
Please add comment: Jenkins retest this

Run (or rerun) specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] tests

Quarkus LTS checks
Please add comment: Jenkins run LTS

Run (or rerun) LTS specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] LTS

Native checks
Please add comment: Jenkins run native

Run (or rerun) native specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] native

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

Jenkins retest this

will be implemented as part of https://issues.redhat.com/browse/KOGITO-6273


once/docs: mention that first result is returned later

Anyone understand why this build is failing?  Travis seems to want to download the code from ramda/ramda rather than raine/ramda but I don't know why.

It just looks like Travis tried running a build upon a new branch being pushed which was subsequently removed.
Judging by the event log (IDs<PHONE_NUMBER> &<PHONE_NUMBER>), I'm guessing @raine accidentally pushed this branch to ramda/ramda instead of his own fork.

:deciduous_tree:

Judging by the event log (IDs<PHONE_NUMBER> &<PHONE_NUMBER>), I'm guessing @raine accidentally pushed this branch to ramda/ramda instead of his own fork.

Yep, pushed to wrong remote by accident.


updated v1.2 documentation

Coverage remained the same at 66.042% when pulling 2847a1b656b7b37ac7419364330afd76a29e885c on tristyxxnana:documentationv1.2 into db5aa739ec8aef0ad75722459acd63917f441f97 on CS2113-AY1819S1-T16-1:master.


Fixes #55: Correct the order of the columns returned by pgmq_read() and pgmq_pop().

The result set columns of pgmq_send() and pgmq_pop() is currently inconsistent with the order of columns in the underlying queue table (See Issue #55). This PR makes the order consistent.
Sample output for pgmq_send():
testdb=# select * from pgmq_send('my_queue', '{"foo": "bar1"}');
 pgmq_send
-----------
         1
(1 row)

testdb=# select * from pgmq_send('my_queue', '{"foo": "bar2"}');
 pgmq_send
-----------
         2
(1 row)

testdb=# select * from pgmq_read('my_queue', 30, 2);
 msg_id | read_ct |          enqueued_at          |              vt               |     message
--------+---------+-------------------------------+-------------------------------+-----------------
      1 |       1 | 2023-08-15 22:59:54.335146-06 | 2023-08-15 23:00:47.513264-06 | {"foo": "bar1"}
      2 |       1 | 2023-08-15 23:00:04.972114-06 | 2023-08-15 23:00:47.513264-06 | {"foo": "bar2"}
(2 rows)

testdb=# select * from pgmq_my_queue;
 msg_id | read_ct |          enqueued_at          |              vt               |     message
--------+---------+-------------------------------+-------------------------------+-----------------
      1 |       1 | 2023-08-15 22:59:54.335146-06 | 2023-08-15 23:00:47.513264-06 | {"foo": "bar1"}
      2 |       1 | 2023-08-15 23:00:04.972114-06 | 2023-08-15 23:00:47.513264-06 | {"foo": "bar2"}
(2 rows)

Sample output for pgmq_pop():
testdb=# select * from pgmq_send('my_queue', '{"foo": "bar3"}');
 pgmq_send
-----------
         3
(1 row)

testdb=# select * from pgmq_pop('my_queue');
 msg_id | read_ct |          enqueued_at          |              vt               |     message
--------+---------+-------------------------------+-------------------------------+-----------------
      3 |       0 | 2023-08-15 23:00:37.947369-06 | 2023-08-15 23:00:37.947369-06 | {"foo": "bar3"}
(1 row)

I verified that the following tests still pass:
> cargo test
    Finished test [unoptimized + debuginfo] target(s) in 0.10s
     Running unittests src/lib.rs (target/debug/deps/pgmq-c90b88e7628f9490)

running 7 tests
    Building extension with features pg_test pg15
     Running command "/home/rzavalet/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/bin/cargo" "build" "--features" "pg_test pg15" "--no-default-features" "--message-format=json-render-diagnostics"
  Installing extension
     Copying control file to /opt/postgres/share/extension/pgmq.control
     Copying shared library to /opt/postgres/lib/pgmq.so
     Copying extension schema upgrade file to /opt/postgres/share/extension/pgmq--0.7.4--0.7.5.sql
     Copying extension schema upgrade file to /opt/postgres/share/extension/pgmq--0.11.1--0.11.2.sql
     Copying extension schema upgrade file to /opt/postgres/share/extension/pgmq--0.7.3--0.7.4.sql
     Copying extension schema upgrade file to /opt/postgres/share/extension/pgmq--0.7.5--0.8.0.sql
     Copying extension schema upgrade file to /opt/postgres/share/extension/pgmq--0.8.0--0.8.1.sql
    Finished installing pgmq
test tests::pg_test_validate_same_type ... ok
test partition::tests::pg_test_map_partition_col ... ok
test tests::pg_test_internal ... ok
test tests::pg_test_default ... ok
test tests::pg_test_creat_non_partitioned ... ok
test tests::pg_test_archive ... ok
test tests::pg_test_partitioned ... ok

test result: ok. 7 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 3.39s

stopping postgres (pid=273762)
     Running tests/integration_tests.rs (target/debug/deps/integration_tests-2cedb8858eeabbb1)

running 1 test
test test_lifecycle ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.63s

Let me know if I should test something else.

All the tests passed in CI too. Thank you for helping!

closes https://github.com/tembo-io/pgmq/issues/55


Vector * Vector does not work

with scala 2.11.12, breeze 0.12 the following astonishingly basic operation does not work
val b = breeze.linalg.DenseVector(1.0,2.0,3.0)
b * b

but throws the following error
diverging implicit expansion for type breeze.linalg.operators.OpMulMatrix.Impl2[breeze.linalg.DenseVector[Double],breeze.linalg.DenseVector[Double],That]
starting with method collapseUred in object UFunc
       b * b

If you need a dot-product, you should rather use
a.t * a

or
dot(a, a)

Your version doesn't work because DenseVector stands for a column-vector and a product of two column vectors is not defined.

thank you very much for the fast response!!
I was experimenting a bit in the meanwhile and found that ```b :* b`` does give a element-wise product in breeze 0.12


Assert thrown because containerView too high.

Hi,
When having the container view with a height of 378, an assert is thrown. Any reason for this?

I also have one item in the container view.

Having the same problem too.

same problem

directly set assert value like,
assert(true, "contanerView.height too high")

@KumaresanSubramaniam  that is not Proper solution

the assertion throws when containerView's height is not 2 times the height of the foreground view.
So i change the heights and it works..
hope it helps


Example code for textDetection is incorrect

Following is the example code under the textDetection method:
var image = {
  source: {imageUri: 'gs://path/to/image.jpg'}
};
vision.textDetection(image).then(response => {
  // doThingsWith(response);
  console.log(response); // I added this portion; not part of the example.
}).catch(err => {
  console.error(err);
});

Running the above example, throws the following error trace:
Error: No image present.
    at _coerceRequest (/Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/@google-cloud/vision/src/helpers.js:69:21)
    at ImageAnnotatorClient.<anonymous> (/Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/@google-cloud/vision/src/helpers.js:224:12)
    at ImageAnnotatorClient.wrapper [as annotateImage] (/Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/@google-cloud/common/src/util.js:749:29)
    at ImageAnnotatorClient.<anonymous> (/Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/@google-cloud/vision/src/helpers.js:141:17)
    at /Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/@google-cloud/common/src/util.js:780:22
    at new Promise (<anonymous>)
    at ImageAnnotatorClient.wrapper [as textDetection] (/Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/@google-cloud/common/src/util.js:764:12)
    at /Users/Bensooraj/Desktop/NodeJS_Play/nisome/routes/index.js:40:10
    at Layer.handle [as handle_request] (/Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/express/lib/router/layer.js:95:5)
    at next (/Users/Bensooraj/Desktop/NodeJS_Play/nisome/node_modules/express/lib/router/route.js:137:13)

The above error came in from here:
https://github.com/googleapis/nodejs-vision/blob/8e042dc15dd82ab41f549e001368202b072c3d81/src/helpers.js#L68-L69
where the request object is expected to nest an image object within as mentioned in the documentation:

This is what worked for me:
// Imports the Google Cloud client library
const vision = require('@google-cloud/vision');
// Creates a client
const client = new vision.ImageAnnotatorClient();
// Uploaded image, available in base64Image format,
// is converted to a Buffer
let imageBuffer = Buffer.from(req.body.base64Image, 'base64');

// Call the Cloud Vision client, to detect text
client
    .textDetection({
      image: { content: imageBuffer }
    }).then(response => {
      console.log(response);
    }).catch(err => {
      console.error(err);
    });

I believe the example code in the document should change to:
const request = {
  image: {source: {imageUri: 'gs://path/to/image.jpg'}},
};
// OR, if you are using an image buffer
// const request = {
//   image: { content: imageBuffer }
// };

vision.textDetection(request).then(response => {
  // doThingsWith(response);
  console.log(response); // I added this portion; not part of the example.
}).catch(err => {
  console.error(err);
});

Or,
let image = {
  source: {imageUri: 'gs://path/to/image.jpg'}
};
// OR, if you are using an image buffer
// let image = {
//   content: imageBuffer
// };
vision.textDetection({ image }).then(response => {
  // doThingsWith(response);
}).catch(err => {
  console.error(err);
});

Environment details

OS: System Version: macOS 10.12.6 (16G29) | Kernel Version: Darwin 16.7.0
Node.js version: v9.2.0
npm version: 6.0.0
@google-cloud/vision version: 0.19.0

Yes, you're right. It's a bug in the sample code.

@alexander-fenster if the generated sample code is bad, what do we do?

Discussed it with @kinwa91 offline. This file, src/helpers.js, is a manually written wrapper, it's not auto-generated. So the error should be fixed in this file, no other changes are required.
As a rule of thumb, the generated code always lives inside src/v*. Within src/, only src/index.js can be auto-generated (or can be manually written, it's quite easy to tell by looking at the file and its edit history).


Center: Allow user provided setView options

In the bounds directive, $scope.bounds.options is used as the options for map.fitBounds. This is nice as the user can specify animation options and the like. Center should use a similar way to specify options to map.setView (leaflet docs).

Why not just use
leafletData.getMap().then(function(map){
  map.setView(whatever);
});

instead?

This issue was moved to angular-ui/ui-leaflet#104

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


The solution definitely is not production read (Is it a bug or feature?)

Restrictions for CSV are not handled at all. For E.g. if you have a user without phone_number_verified attribute - you will get UserNotFoundException during attaching the user to the group.
Request rate management is very uneffective. CognitoTps - does not work for user export (export-users.js) it just ignored and "works" only when an exception appeared. Also, the user cycle is performed without any tsp checks. For user "import" or better say "attach users to groups" tps logic is implemented manually. Have not yet checked it but javascript has a library for automatic request rate managing.
Solution uses too many resources: DynamoDb tables, SQS. You can just run lambda from the secondary region, and store all data in s3 (users, groups, user-group relations). With 30TPS hard limit for the user pool and a single thread for the user loop - this is not a problem. Why do you use DynamoDb? Just because it has a replication feature? What is the profit? The documentation said that is needed when the same logic uses this in different regions. In your application logic is different (import and export)
Solution also has a very complex deployment: lambda functions are built in a loop (about 20 minutes), the build is a platform depend (due to bash scripts), and manual actions are required. A lot of custom resources with state machines (why???). Also, the solution deploys StackSet with the same resources in 2 regions with if conditions and skips some resources in the secondary region. (+20 minutes to deployment time) It is better to just use one stack for the main (export) region and a second for the import - just deploy in the needed region. Also, the solution uses Cloudformation Custom resource for constants holding - why do not use SSM here?
No mention in the documentation that this is a one-time import!!! This I got from AWS support team.

to be continied

Logs. No separation for debug, info, warn, etc. No log level managing. For every user I have Adding user to group and User added to group in logs. So one user +2 log statements. (And this is without logs for user to group attaching). 500k user pool -> 1000k-3000k logs statements. (Where I store my user - in the user pool or cloudwatch?). For errors I got 2022-11-23T00:13:58.989Z	985180b0-e037-4f0a-9649-ce11c30948ea	ERROR	UserNotFoundException: User does not exist. Yes, just message without group name and user name. With 2 doubled stack traces in logs again.

Thank you for the thorough feedback and very sorry for the late response. You've made some good points here for us to consider. The feedback has been added to our roadmap so I will close this for now.


feat(expression router) move setting protocols back to kongstate override for expression based routes

What this PR does / why we need it:

As suggested by gateway team, it is better to configure the protocols of Kong routes in protocols field than filling them directly into the expression. This PR moves the filling protocol back to the kongstate override phase of generated Kong routes, as with traditional router.
Which issue this PR fixes:

fixes #4313
Special notes for your reviewer:

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Allowed myself to rebase onto main and regenerate golden files (needed after https://github.com/Kong/kubernetes-ingress-controller/pull/4426).


Changed a couple of things based off feedback

Travis recorded his sons Jackson and Noah playing through Skillmap. They found things that were easy fixes, so I've made the changes and I'll squash when tests pass.

A formatted block name doesn't match the block
Removed the word "shorthand"
Remove the term sprite editor and call it image editor everywhere

I also removed autoexpand from the intro tutorial. If students won't scroll, it's unlikely they'll click that button...and since this is a tutorial on how to use the blocks, it's not as easy to discover on their own like it would be when making projects.


Extends SelfEquals to cover Objects.deepEquals, Arrays.equals, Arrays.deepEquals

I saw an error at least once in the wild with deepEquals. I haven't tried running against all Google code.

(I did an experiment with extending SelfEquals to cover this. The problem is the need to extend the existing false-positive heuristics to cover more patterns. It doesn't seem too hard, but my experiment suggested that we wouldn't catch many real problems as a result of the enhancement.)


Support globing .gql files

PR for #75  I introduced quite a couple of new dependencies which may not be ideal. Anyhow Its just an early attempt let me know what you think and I will make the required changes.

Coverage increased (+0.06%) to 96.497% when pulling 154b5d5d604220b36cf944eb43a197dd06dc4a48 on epicallan:master into 5378c5a15b6bda1e22de18abaf002389875476e1 on avantcredit:master.


Move storage/mkvs/urkel to just storage/mkvs

The MKVS implementation has been changed from the initial "Urkel tree" structure and it is no longer an actual "Urkel tree" so having "urkel" in its name is just confusing.
A similar rename should happen both in Go (under go/) and Rust (under runtime/src/).

what is it now?

A merklized patricia trie variant, just not exactly urkel tree.


bug: ListView Items Change Can Not be Set

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
ListView Items Change Can Not be Set
Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
No response
Anything else?
No response

Please give a minimized example（github or zip） and reproduce stps.


Dark mode

Add dark mode to the UI and option to switch between light and dark
Example image below

shifted to discussions: #1333


Defining the abbreviation

Defining the abbreviation, so readers know what "PEP" stands for.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks for contributing @StevePeterson12!


Add CGP102 txs json

This adds the transactions for CGP102 to replace Di Wu with Redstone as oracle providers: https://github.com/celo-org/governance/blob/main/CGPs/cgp-0102.md
Below are the addresses from Di Wu:

Rate
Address
Source

CELO/USD
0xF8ffaB8cCB81BcD0DF7f9dF97Ccf8bEF782ebEeB
https://celo.stake.id/#/proposal/69

CELO/EUR
0x3F2691d5B4839c226f4Bd096DC744914cAD5737e
https://celo.stake.id/#/proposal/69

CELO/BRL
0x4eb4571a475Bea99b6A735B89aae664b82c964a3
https://celo.stake.id/#/proposal/69

USDC/USD
0x504C282C98fe9E7627DCE2c255E3BfF2eeB6e73e
https://celo.stake.id/#/proposal/86

USDC/EUR
0xBc2F9D199491786A4c68850B6438b8B5eE985735
https://celo.stake.id/#/proposal/119

USDC/BRL
0xBcBDd157e2d9737049eb3a8aFbC58b725523d002
https://celo.stake.id/#/proposal/119

EUROC/EUR
0xBD136a625299A0ac5Ca7Ce9220aCA6e08a624e37
https://forum.celo.org/t/proposal-mento-upgrade-mu03-enable-more-trading-pairs-and-continue-phasing-out-v1/6462#h-9-whitelist-diwu-as-an-oracle-provider-for-euroceur-11

which will be replaced by Redstone adapter contract deployed on mainnet https://celoscan.io/address/0x6490a3FFAD86CA14FF84Be380D5639Fb8fBD311B

RedStone adapter contract address is valid ✅


Json serialization on RegisteredFunction.Invoke includes members with null value

If I'm not mistaken, I think that serializer includes members having a null value, so if I have a type
class Tubo {
  public string Text { get; set; }
}

and I serialize new Tubo(), I'd get
{ Text: null }
instead of
{}
This is a problem when you don't want to pass properties around.

But then, if any library/component (Javascript/TypeScript) rely on the property existence if doing Interop, removing the property because it is null will return a undefined property, no ?
I think JsonUtil mimick C# consistency about objects. Maybe an overload on Serialize would be a solution for this kind of scenario you are having, totally valid when calling a web api or something like that.

Yes, it's a tricky one. But the usual behavior in web world is to ignore the null properties if I'm not mistaken. Having an option somewhere would be the best way to solve it.

We're trying to keep JsonUtil as simple as we can, so having opinionated rules about stripping out properties is out of scope. I'd recommend using a different JSON library if you want to control that sort of thing. Hope that's OK.

@SteveSandersonMS Is there a way to substitute the json serialization easily? Possibly per method call.

Actually, JsonUtil juste use SimpleJson library.
You may try to call SimpleJson directly from your code, and seeing if it fulfill your requirement in any of his option. Or use Newtonsoft Json.Net if you are able (but maybe overkill).
But SimpleJson seem to be present, as seeing from the JsonUtil source code.

Could we introduce an interface IJsonUtil and start utilizing that via dependency injection rather than as a static class?  That way you could swap out the implementation from SimpleJsonUtil to anything else (your own JsonNetUtil for instance).

@zbecknell Yep.

Any design we come up with has to work correctly when composing together multiple third-party packages. So we can't have JsonUtil vary its behavior based on your DI config, because then third-party packages that assume certain serialization conventions when making their own JS interop calls would break if you changed it.
Proposal: we could add an extra overload to RegisteredFunction.Invoke as follows:
public static TRes Invoke<TRes>(string identifier, string argsArrayJson)

This will allow you to do your own JSON serialization using whatever library or conventions you want and have it passed through JS interop in the same way. To make usage simpler, you can easily write your own static utility method like:
public static TRes MyAppInvoke<TRes>(string identifier, params object[] args)
{
    var argsJson = DoMyCustomJsonSerialization(args);
    return RegisteredFunction.Invoke<TRes>(identifier, argsJson);
}

Would that cover your requirements?

@SteveSandersonMS You are right on DI - that wouldn't be good.
Yes, it has to be per method.
Another option would be:
public static TRes Invoke<TRes>(IJsonUtil converter, string identifier, params object[] args)
where one would pass its own implemention.
Perhaps both options could be added.