Knife Vault Refresh Not Running on Server 2012R2

We've got some powershell workflows that are involved with our deployment code. They remote to a server and run some commands. One of the commands we need to run is the vault refresh (after a new node is added with the correct role) - it seems the "knife vault refresh Vault Credential -m client" hangs forever while processing.
Running the same command on an 8.1 machine - is fine.
So it seems something with the OS does not allow it to run. The chef/powershell versions are the same on both machines. We are running chef enterprise version 11.12.8, using vault 2.4.

Using the same channel to the 2012r2 server, do these commands work?
knife node list
knife vault show Vault Credential -M client
knife search node <SOLR query for your vault>

I tried this on a fresh 2012r2 VM and it seemed to work fine:
PS C:\Users\Administrator> $env:EDITOR = 'notepad'
PS C:\Users\Administrator> knife vault create foo bar
PS C:\Users\Administrator> knife vault update -S '*:*' foo bar
PS C:\Users\Administrator> knife vault refresh foo bar
PS C:\Users\Administrator> knife vault show foo bar
foo: bar
id:  bar
PS C:\Users\Administrator> knife search node '*:*'
1 items found

Node Name:   foo.nordstrom.net
Environment: foo-test
FQDN:
IP:
Run List:    recipe[foo]
Roles:
Recipes:
Platform:
Tags:

My gut feeling is that hanging forever sounds like an issue with network connectivity to the server.
I know we have had a Windows user report that knife node search for a complex query against an org with 3k nodes in it took a very long time (>3h).  Are you trying to refresh against an org of that size or with a particularly hairy SOLR query?

I won't completely rule out network problems (as I've learned not to completely rule anything out) but its unlikley. We don't have issues between my desktop to the chef node and both the server and my desktop exist on the same switch. For what that's worth.
The "knife search node ':' "  query actually exhibits the same behavior in not returning anything. Our organization has about 300 nodes, so I don't think we're hitting the same limits there.
I'd be willing to consider the chef server as a bottleneck - except the fact that it works on my desktop at the same time - seems to indicate that is not the case.
What hypervisor is your fresh VM on - maybe I'm hitting some issues there.

I was using an EC2 with their "standard" (i.e. Amazon prepared) 2012r2 image.   Set it up on a t2.medium instance type and those commands were over a VPN/VPC back to an on-prem Enterprise Chef 11.2.6 box.
When I say network connectivity, I'm was thinking more about window sizes and such.  Just as an example, PuTTY has a workaround for a problem with the Windows stack that causes git clones over SSH to hang after they have pulled everything but before they resolve deltas:
http://www.chiark.greenend.org.uk/~sgtatham/putty/wishlist/winadj-success.html
The workaround fiddles with the TCP window size to stop this:
https://jira.atlassian.com/browse/STASH-3824
Obviously we're talking about different protocols, but that's what came to mind when you said that the same search hangs.
In terms of chef-vault, if the search hangs (and the chef API doesn't throw an exception), then vault operations that depend on search will hang as well.  We could put a timeout block around it, but I'm not sure what a safe default would be - there may be people with workflows whose long searches do successfully finish, so I'd be hesitant to possibly break those outside of a major version bump.
I assume you're hitting your Chef server over https?  It would be interesting to see a TCP trace to see if there are reasons the search is hanging.  Obviously unencrypted would be better, if you can hit the server on http.

So after this message on Sunday I went to try your fixes and noticed the knife search from Thursday/Friday had finished. I kicked off another knife search - this time with the measure-command flag (to get times). Results are below:
PS C:\Users\XXXX> measure-command {knife search ':' -i}
Days              : 1
Hours             : 21
Minutes           : 44
Seconds           : 59
Milliseconds      : 785
Ticks             :<PHONE_NUMBER>492
TotalDays         : 1.90624751214352
TotalHours        : 45.7499402914444
TotalMinutes      : 2744.99641748667
TotalSeconds      :<PHONE_NUMBER>492
TotalMilliseconds : 164699785.0492
So it seems eventually it works? I ran a wireshark while this was going on - there was no communication for quite some time until it came through. Any thoughts on this?

The internal person who reported the slow search didn't let it run to completion, so I had been working under the impression that it would hang forever.
On the trace, did you see a burst of results after performing the search and then nothing until the close?   Is there any payload in the final packets before the connection closes, or does it look like a socket-level close instead of an application-level close?
I'd open a issue with Chef (via the corp support if you're an enterprise customer, otherwise Github against the chef-server project) about the search issue, as that seems to be at the heart of it.
There is a possible change we can make which is to handle the results using a block.  Right now we do the entire search and then take the results and iterate over them like this:
query.search(:node, search)[0].each do |node|

But even going back to Chef 10.34.6, you can pass a block to search and have it hand the rows off to you as they come in (sort of - it still seems to page 1000 at a time, but we could change that at the cost of more round-trips to the server).
Let me throw together a quick spike with that change and have you test it.

Great thanks - I will open up an enterprise ticket as well. Will let you know what happens regardless.

I just pushed a branch spike_search_stream_results.  This handles the search results as a block:
query.search(:node, search, 'X_CHEF_id_CHEF_X asc', 0, windowsize) do |node|

instead.  The window size defaults to 1000 as before but can be changed by setting the environment variable CHEF_SEARCH_WINDOW_SIZE to something else:
> CHEF_SEARCH_WINDOW_SIZE=5 knife vault refresh foo bar
chef-vault client search returned node foo
chef-vault client search returned node bar

I also put in a debug puts statement to show the nodes as they come back, which might help to see if the results are bursty or not.
Can you see how that works with the default window size, and if the results hang try bringing it down to lower values to see if it makes a difference?

Thanks - heading out to lunch but will try this afternoon.

I pulled the code and compiled it (using rake) - manuall installed the gem on the server (gem install) - however knife vault is no longer recognized.

Someone else had this problem, and I think it came down to permissions.  Make sure (especially if you're using 'chef gem install') that you install and run as the same user.
There is nothing in the diff that would cause the entire plugin to stop being recognized, so I'm sure it will come down to something along these lines.

I've confirmed I'm running as the right user both the vaul command that the 'gem install'. I'm out of the office today I'll take a look later this evening.

Perhaps roll back to the rubygems version and just manually make the changes shown in the diff to the working copy then?
There definitely seems to be something going wrong when people install gems they've built themselves, but I've never had it happen to me so I can't provide much guidance beyond guesses.  I'd prefer to not conflate that problem with the 'searches never complete' issue.

Any feedback on this?  Ultimately this is a 'knife search on Window' issue, but if the change provides improvement I'm willing to merge it for the next release.

I haven't had a chance to move on this. Hopefully by the end of the week. We just did our renewal with Chef - so I'll be opening a support ticket with them per your advice. I'm fine if you want to close this as not a vault issue - and I will comment here with an update.

I'll mark this for closure when I'm doing the prep for 2.6.0; hopefully you can try the patch by then and provide feedback as to whether we should add the search changes to that release.

So finally have some resolution here. For whatever reason chef-client 18.6 now works. Its possible it was some combination of windows updates and chef client or what. But everything now works.

Thanks for letting us know


Fix table height and graph/table automatic title generation in curriculum content

We were only enabling the ToolApiInterface in editable documents, but for title and height determinations it needs to be extended to read-only document contexts as well. Therefore, we move responsibility for the ToolApiInterface to the CanvasComponent and use React context to distribute it to clients rather than prop drilling.
This PR also implements automatic sizing of curriculum rows to the expected size of tables. (This approach can be extended to other tiles, but only tables support it for now.) The manual height set in the content can still override, however, e.g. for very long tables where the user is expected to scroll. Thus, to enable the automatic sizing I removed all of the explicit heights from the curriculum tables in Moving Straight Ahead, as they all seemed to warrant automatic sizing.
This PR also enables column width resizing in tables in curriculum content, which were originally disabled as part of supporting the readOnly flag, but column resizing can still be useful in read-only tables.
Finally, this PR also fixes a bug in table title cell sizing in read-only content.
These changes are more extensive than I'd like given how close we are to release, but most of the changes are mechanical, i.e. pull the ToolApiInterface from context rather than from props, so the impact is less than it appears at first blush.

Kirk this looks quite fine to me. I will be touching a lot of content tonight so I can do things like insert titles for all tables and graphs and adjust axis scaling as needed but really - super good given the invasiveness of the change. Thanks.

Thx for the prompt reviews!


Google Analytics Help?

Greetings!  I was finally able to get this fun software installed and I'm totally digging it.  However, I'm having some issues with the Google Analytics integration.
When I click on "Connect With Google" it prompts for an ID and password as one would expect but then it tells me that the redirect_uri is invalid.
I'm sure I'm doing something wrong in the configuration of the key so if anyone has a step by step for how to make it work that'd be grand.
Thanks!

Hello @sizzly,
I suppose you already created an app in Google Console, but you have to enable the redirect url, what should be something like this:
http://example.com/security/authentication/client/oauth/login?hauth.done=Google
Can you please try this?

Ugh.  I haven't had a chance to go back to this but I will!  And thanks tons for your quick response.


Nuxt components inside Storyblok Rich-Text editor

I saw your post here: nuxt static, really helpful.
I had a look at nuxt3 and storyblok just wondering if you got components in richtext editor to resolve in nuxt3.
Suggestions below don't seem to work
https://www.storyblok.com/tp/nuxt-components-inside-rich-text

Yeh exactly that.
There is a related issue in storyblok/nuxt repo:
https://github.com/storyblok/storyblok-nuxt/issues/172#issuecomment-1209010052
Would be great if that worked because I'm trying to migrate a wordpress site that has wp shortcodes embedded in the content.
I want to try and transform those into storyblok components.

Hm okay interesting. Since i am using useStoryblokApi().richTextResolver.render(props.blok.text)) and not https://github.com/m4rvr/storyblok-rich-text-renderer (since it cant handle ssr with pretender correctly) or similar to render I would have to do it myself.
Here you can see that it completely removes the blok node:

I didn't have much time to look at this today, but I have a work in progress version:

<template>

      <component
        :is="$resolveStoryBlokComponent(node.content.blok)"
        v-if="node.type === 'blok'"
        :blok="node.content.blok.content"
      />

</template>

What I do:
I loop through all the nodes in the richtext. For each node I add it to an array representing the finished content. For the nodes that is a blok, I add it to the array with the type "blok" and for the rest (that I assume is proper richtext nodes that the resolver can handle) I add to the array with type "HTML" and I also resolve the richText right away.
This means that I get an array of each "node" that I want to render in the correct order.
The finished code does this:

Let me know if any of this was confusing. I will most likely refactor this when I have more time and add it to my finished code in production.

I made a blogpost about this: https://sebbejohansson.com/blog/storyblok-richtext-in-line-block-support-for-nuxt3-with-ssr-and-prerender


Fix Symlink check when target is a symlink

My initial reaction was that this is wrong - symlinks should be resolved recursively.
But after giving it more chought I realized that in case symlink target is in turn a symlik itself, then it should be either covered by the symlink check, if it is part of package under test, or we shouldn't check that it is not dangling.
Therefore I think that the code can be corrected this way.


Duplicate control when add new control after delete section and then set it again

I try your vue form builder and it's totally awesome. Thankyou. But I have found an issue when I try to set form from default template. The issue is when I delete section and then set it back from the template and then I try to add new control the Vue form builder seem create multiple/duplicate control. It would be great to fix it. Thankyou.

Hello @harrieanto ,
Thanks for the compliment.
For the problem that you approached, it would be really helpful if you can provide me:

The Form Configuration JSON
More detail about how to reproduce the problem (Screenshot/A short clip would be really helpful).
(Optional) Create a simple live demo of your application.

Thanks

Yes. Thanks for your fast response. I'll back here and provide what's needed as soon as possible.

@sethsandaru  I was facing similar kind of issue, so I have prepared a clip to showcase the issue https://www.loom.com/share/87b888ed8afb41759e105d73506ac467

@abhimanusharma exactly like that

@sethsandaru I think the label should be changed to bug.

@sethsandaru @abhimanusharma Hi all, Already have solusiton for the issue?

@harrieanto , It is still under progress, hope we will get it on new year.


feat: add isThemeEditorDrawerOpen prop

add the isThemeEditorDrawerOpen prop to ThemeEditor for more customizability
verified changes in storybook.

@it-nalon could you take a look at these changes when you have a chance?

Hi, sorry for the delay!
I checked the code and it seems a good idea to control the open/close state outside the component.
At this point, I think we can simply make the <ThemeEditor /> component completed controlled.
It would be awesome if you can edit the PR so it accepts all the: UseDisclosureProps props.
All these props could then be passed to the: useDisclosure() hook from Chakra UI.
The could could look something like this:

...

export type ThemeEditorProps = {
  children: ThemeEditorChild[] | ThemeEditorChild
} & UseDisclosureProps

export const ThemeEditor: FC<ThemeEditorProps> = ({ children, ...disclosureProps }) => {
  const { isOpen, onOpen, onClose } = useDisclosure(disclosureProps)

 ...

}

With this changes you should be able to control the ThemeEditor state passing just the isOpen props:
const [isOpen, setIsOpen] = useState(true)

<ThemeEditor isOpen={isOpen} />

thanks for being open to this contribution @it-nalon! I incorporated your suggestions in my most recent commits, please let me know if there's anything else
I also added a story just to demo:
https://user-images.githubusercontent.com/15132025/154892685-ef1931ae-2b51-4703-83e2-67252ce0147e.mov

Hi @panbenson, I've merged the PR and it will be released on the 0.2.0 version (because you pointed the PR to the main branch), the 0.2.0 does not have a release date yet.
I think that this PR could be deployed as version: 0.1.4 before the 0.2.x version.
If you make a PR that points to the support/0.1.x branch I will deploy a new 0.1.x version as soon as possible.


Can't setup

Log:
-----> Ruby app detected
-----> Compiling Ruby/Rack
-----> Using Ruby version: ruby-2.2.3
-----> Installing dependencies using bundler 1.13.7
       Running: bundle install --without development:test --path vendor/bundle --binstubs vendor/bundle/bin -j4 --deployment
       Your Ruby version is 2.2.3, but your Gemfile specified 2.2.5
       Bundler Output: Your Ruby version is 2.2.3, but your Gemfile specified 2.2.5
 !
 !     Failed to install gems via Bundler.
 !     Detected a mismatch between your Ruby version installed and
 !     Ruby version specified in Gemfile or Gemfile.lock:
 !     https://devcenter.heroku.com/articles/ruby-versions#your-ruby-version-is-x-but-your-gemfile-specified-y
 !
 !     Push rejected, failed to compile Ruby app.
 !     Push failed

It seems like you'll need to update your Ruby version to 2.2.5 :)

@BasThomas I'm using the Heroku button

In that case, I have no idea. Sorry!


A more general and accurate implementation of MetricsAggregator for Spark

A more general and accurate implementation of MetricsAggregator for Spark which is compatible with Spark on YARN as well as executor dynamic allocation mode.
What's new:

Memory: Use actual executor memory request considering memoryOverhead instead of only spark.executor.memory.
Time: Use running time of all existed executor, not duration(sum of task executing time) of executors only including of those which are alive while ending of application.

I test this feature for three real Spark applications on our YARN cluster and compare Aggregate Memory Resource Allocation(Resource used in dr-elephant) metric of applications in YARN(showed when spark application finished in AM page), dr-original and and dr-optimized Below is result of one application for example:

YARN: 410620516 MB-seconds
dr-original:<PHONE_NUMBER>.4 MB-seconds
dr-optimized: 405405696 MB-seconds

Consider YARN showed as the standard value,

deviation of  dr-original：(410620516-1394513510.4)/410620516=239.61%
deviation of dr-optimized：(410620516-405405696)/410620516=1.27%

Average deviation of this feature is 1.88%, as  average deviation of  dr-original is 280.86%.
This is our first PR, so there has also some TODO work about code style, code reasonable, and etc, so please help us review and feel free to give us advise.
Future work we plan:

Add cpu-time resource usage for CPU are also can be scheduled and CPU resource even is bottleneck resource in our company.
Catch the real physical memory resource usage and optimize the Resource wasted metric.

Thanks!

@ericsahit, thanks for the PR. I'll take a look at it and add my comments.

@akshayrai , OK. I have some login problem with gitter, Anyone please feel free to give me any suggestions.

Hi @ericsahit @shankar37, is this PR still relevant or has it been superseded by one of the other Spark PRs?

/**
Controls the Compare Feature
*/
public static Result compare() {
DynamicForm form = Form.form().bindFromRequest(request());
String partialFlowExecId1 = form.get(COMPARE_FLOW_ID1);
partialFlowExecId1 = (partialFlowExecId1 != null) ? partialFlowExecId1.trim() : null;
String partialFlowExecId2 = form.get(COMPARE_FLOW_ID2);
partialFlowExecId2 = (partialFlowExecId2 != null) ? partialFlowExecId2.trim() : null;
List<AppResult> results1 = null;
List<AppResult> results2 = null;
if (partialFlowExecId1 != null && !partialFlowExecId1.isEmpty() && partialFlowExecId2 != null && !partialFlowExecId2.isEmpty()) {
  IdUrlPair flowExecIdPair1 = bestSchedulerInfoMatchGivenPartialId(partialFlowExecId1, AppResult.TABLE.FLOW_EXEC_ID);
  IdUrlPair flowExecIdPair2 = bestSchedulerInfoMatchGivenPartialId(partialFlowExecId2, AppResult.TABLE.FLOW_EXEC_ID);
  results1 = AppResult.find
      .select(AppResult.getSearchFields() + "," + AppResult.TABLE.JOB_DEF_ID + "," + AppResult.TABLE.JOB_DEF_URL
          + "," + AppResult.TABLE.FLOW_EXEC_ID + "," + AppResult.TABLE.FLOW_EXEC_URL)
      .where().eq(AppResult.TABLE.FLOW_EXEC_ID, flowExecIdPair1.getId()).setMaxRows(100)
      .fetch(AppResult.TABLE.APP_HEURISTIC_RESULTS, AppHeuristicResult.getSearchFields())
      .findList();
  results2 = AppResult.find
      .select(
          AppResult.getSearchFields() + "," + AppResult.TABLE.JOB_DEF_ID + "," + AppResult.TABLE.JOB_DEF_URL + ","
              + AppResult.TABLE.FLOW_EXEC_ID + "," + AppResult.TABLE.FLOW_EXEC_URL)
      .where().eq(AppResult.TABLE.FLOW_EXEC_ID, flowExecIdPair2.getId()).setMaxRows(100)
      .fetch(AppResult.TABLE.APP_HEURISTIC_RESULTS, AppHeuristicResult.getSearchFields())
      .findList();
}
return ok(comparePage.render(compareResults.render(compareFlows(results1, results2))));

}
how do conf  azkaban scheduler.xml @ @ericsahit


Prepare to support the next DAP version

Change DapVersion::DraftLatest to DapVersion::Draft09 and use DapVersion::Latest to track the next draft we'll target (currently draft 10, which is not yet published). So far there are no substantive changes in the next draft, but changes will be required to enable Mastic in its "heavy hitters" mode of operation.

Rebased.

After some self-review, I realized I forgot to add the latest version to enable the latest version in async_test_versions!. Once I did, I found several match statements where I forgot to add DapVersion::Latest to the necessary arm.

After some self-review, I realized I forgot to add the latest version to enable the latest version in async_test_versions!. Once I did, I found several match statements where I forgot to add DapVersion::Latest to the necessary arm.

why did these not get caught by CI?

After some self-review, I realized I forgot to add the latest version to enable the latest version in async_test_versions!. Once I did, I found several match statements where I forgot to add DapVersion::Latest to the necessary arm.

why did these not get caught by CI?

None of the tests were enabled for the latest version, which means the tests didn't exercise the DapVersion::Latest variant.


FYI: dotplot-style visualizations in Python for nucmer output

hello, I just wanted to let you know about some stuff I've been doing to visualize nucmer output - stacked dotplots and other stuff. Code is here: https://github.com/ctb/2020-stacked-dot-plots
thanks for all your work on mummer, it's fantastic!

I am much appreciated that you let me know and for your excellent work.
Best,
Fuyou
On Sun, Nov 22, 2020 at 1:14 PM C. Titus Brown<EMAIL_ADDRESS>wrote:

hello, I just wanted to let you know about some stuff I've been doing to
visualize nucmer output - stacked dotplots and other stuff. Code is here:
https://github.com/ctb/2020-stacked-dot-plots
thanks for all your work on mummer, it's fantastic!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mummer4/mummer/issues/144, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF3JCKE26XUMLCD55PFOW43SRFPLHANCNFSM4T6VJ57A
.

--
Fuyou Fu, Ph.D.
Department of Botany and Plant Pathology
Purdue University
USA

Thank you for your work, looks very nice! May I add your code to a future release of mummer?

of course! It's under BSD and while I can't sign the copyright over to you (it's UC :( please feel free to (re)distribute as you find useful!
I've been refactoring it somewhat over in the charcoal project and while the core plotting code hasn't yet been cleaned up, I've split out the code for reading it in more nicely, and a few other things. Suggest taking a look there in the future - I'll update this issue as I clean up the code more.  here's the current link: alignplot.py but I will merge it into the main branch soon.


disable left menu

how i can disable left menu in one controller !?

I created a pull-request just for that.
#117

i fix it by edit 1 line in UIViewController.swift file
func setNavigationBarItem() {
//self.addLeftBarButtonWithImage(UIImage(named: "ic_notifications_black_24dp")!)
self.addRightBarButtonWithImage(UIImage(named: "ic_menu_black_24dp")!)
self.slideMenuController()?.removeLeftGestures()
self.slideMenuController()?.removeRightGestures()
self.slideMenuController()?.addLeftGestures()
self.slideMenuController()?.addRightGestures()
}

@ahmedsafadii yours is a better solution

@hlandao  if you want to load the left or right menu in custom controller i can help u with that
;)

Why this can't be done?
I wanted to enable the right menu just when the user is logged in.
In that case, I think a solution should be provided in the README since this is a trivial case. Or this PR accepted.

@pbassut did it work for you ?

It did.
I'm not sure if removing and adding gesturerecognisers is a hackish way though.
Let's wait for the author.
And thank you for your help, @ahmedsafadii .

@pbassut , I need to implement the same use case. I'm not clear how you did that. How did you disable the right side menu and enable it again ?

@leizard just call self.slideMenuController()?.removeLeftGestures()

@leizard  as @pbassut  said
in the file
UIViewController.swift
comment this line
//self.addLeftBarButtonWithImage(UIImage(named: "ic_notifications_black_24dp")!)

Thanh you @pbassut, @ahmedsafadii for your support. So, call removeLeftGestures() to disable right menu ? It seem not work to me, this is my slide menu set up method put in AppDelegate.swift
`
func setUpSideMenu() {
let screenWidth = UIScreen.mainScreen().bounds.size.width
SlideMenuOptions.hideStatusBar = false
SlideMenuOptions.leftViewWidth = (80screenWidth)/100
SlideMenuOptions.rightViewWidth = (80screenWidth)/100
let storyboard = UIStoryboard(name: "Main", bundle: nil)
let mainViewController = storyboard.instantiateViewControllerWithIdentifier("mainNavigationViewController")
let leftViewController = storyboard.instantiateViewControllerWithIdentifier("LeftMenuController") as! LeftMenuController
let rightViewController = storyboard.instantiateViewControllerWithIdentifier("RightMenuController") as! RightMenuController
let slideMenuController = SlideMenuController(mainViewController: mainViewController, leftMenuViewController: leftViewController, rightMenuViewController: rightViewController)
    slideMenuController.removeLeftGestures()

    self.window?.rootViewController = slideMenuController
    self.window?.makeKeyAndVisible()
}

`
Everything works fine instead  slideMenuController.removeLeftGestures() not disable anything. Please tell me where I'm wrong, thank you very much.

Ok I did it. just call self.slideMenuController()?.removeRightGestures() in each view controller which have right slide menu. I think it will be awesome if we can enable/disable slide menu globally :D.
Thank you guys very much for your information.


Enable tests on Windows

This PR enables tests in the build script for Windows which allows the tests to run as part of the CI process.
This required increasing the test timeout from the current 5 seconds because the time between calling CreateCoreWebView2EnvironmentWithOptions and invocation of ICoreWebView2CreateCoreWebView2ControllerCompletedHandler can be more than 5 seconds when running on GitHub Actions. I have seen it take just over 25 seconds.
I would like to know why it takes so long but I have not yet been able to figure it out. I am unable to reproduce it locally. Maybe it is simply due to lack of resources on GitHub Actions but it would be nice to confirm it.
In any case I think that slow tests are better than no tests.

The following comment seems to support that the function simply does a lot of work:

that function is taking a while because that's where the runtime processes are actually getting created and hooked up, which is the bulk of starting a WebView2 control. We are always looking at ways to improve launch performance, but unfortunately there's not a ton you can do from your code. If your app starts up earlier than the WebView2, you could consider creating a hidden WebView2 in the background, so that's it's ready when you need to make it visible.

Originally posted by @champnic in https://github.com/MicrosoftEdge/WebView2Feedback/issues/1536#issuecomment-883911033

This is a bit of a silly example but I was sort of able to reproduce the long loading time with WebView2 locally in an environment with low-ish (4 GB) RAM capacity that was being saturated while doing a screen capture (saving to RAM) while starting the example app. Still it only took a few seconds for WebView2 to load.

Could we perhaps move this from the build script to a 'test' script?

@dandeto I think that'd be a great idea.

I think the confusion comes from the fact that the build scripts are essentially test scripts. They were meant for CI builds and tests as far as I understand.
I still think documentation on how to build without the build script (with an explanation of its method) would be ideal.

Instead of creating more platform-specific scripts another option could be to provide an option to disable tests while keeping them enabled by default.

Documentation sounds good. We seem to need a lot more of that haha! I could write some more in the docs repo and if we want to put that into the readme we can do that as well.
Copy on the commandline args - that's better.

At first glance, I like the idea of having them optionally enabled. That way it only builds by default (as that's the name), and it can optionally run the built files as tests. Maybe something like ./build.{bat, sh} --run-tests. Though it's a change in default behavior.

I've started writing a parser for batch that will support specifying options on the command line. For example, --run-tests would set the option to true and --run-tests=false or --run-tests false would explicitly set it to false. The solution works pretty well but I'm trying to iron out an issue with variable pollution.
As for the shell script, I've written a parser before and I think I can controbute most of that work. It supports the same style of options except --run-tests false. It also supports positional arguments which I am not sure will be needed.

@SteffenL Is a full command parser necessary for a simple flag? We shouldn't bog down the build scripts with unrelated functions if we don't have to.

A full parser is not necessary for just simple boolean switches. Anyway, here is my current overengineered version of the script.
>script\build.bat --help
This is the build script for the webview library.

Usage:
    program --help
    program [--clean] [--build-examples] [--build-tests] [--run-tests]
            [--target-arch=<all|x86|x64>]

Options:
    --clean             Clean the build directory.
    --build-examples    Build examples.
    --build-tests       Build tests.
    --run-tests         Run tests (implies --build-tests).
    --target-arch=ARCH  Build for the target CPU architecture.

By default this script builds the library for the detected CPU architecture.

@SteffenL I like the idea. It's the beginning of a build system for webview. Considering if we want to have this for Linux and macOS as well, we'll have to duplicate the code.
This, along with the confusion surrounding building on Windows, convinces me that it's time to consider using a real build system.


Test code

SonarQube analysis reported 2 issues

 2 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 2 issues

 2 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 2 issues

 2 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 2 issues

 2 minor

Watch the comments in this conversation to review them.

SonarQube analysis reported 2 issues

 2 minor

Watch the comments in this conversation to review them.


WASM binary size optimization

We can remove the zip library and create a simple zip walker ready to scan the central directory of the zip binary. Once we encounter a compressed entry, we can deflate it using libdeflater, a compact and performant library. This approach should reduce the size by at least ~100KB

Marker support is now available in ThorVG, so we can utilize it to eliminate our custom markers implementation, saving a few kilobytes


iloc issue when iterating through all the columns

Describe the problem
iloc throws a TypeError the first time I iterate through all the columns but works every time after
Source code / logs
import modin.pandas as pd

data = {
    "col3": [0, 1, 2, 3],
    "col4": [4, 5, 6, 7],
    "col5": [8, 9, 10, 11],
    "col1": [12, 13, 14, 15],
    "col2": [0, 0, 0, 0],
}

modin_df = pd.DataFrame(data)

Then the first time I run the following code block, I get a TypeError: 'int' object is not iterable, but if I rerun the code block, everything works
for i in range(len(modin_df.columns)):
    modin_df.iloc[:, i]

Closing this. Feel free to reopen if the discussion should continue or if issue was not resolved.


Add option to set default route or mock app.vue

I want to setup test environment without rendering my index page, please add config option for setting globally default route, for example '/test' instead of '/' or option to mock app.vue component.
So far I figured out workaround, in nuxt.config.ts I am doing mocking of index page manually with hooks
hooks: {
    'pages:extend'(pages: NuxtPage[]) {
      if (String(process.env?.TEST) === 'true') {
        const indexPage = pages.find((page) => page.name === 'index')
        const testPage = pages.find((page) => page.name === 'test')
        if (indexPage != null && testPage != null) {
          indexPage.file = testPage.file
        }
      }
    },
  },

Note this hook is not working when placed in vitest.config.ts overrides because function is not serializable.

Thanks for the workaround, it helped me to come up with this one :
// nuxt.config.ts

hooks: {
    'app:resolve': (app) => {
      if (String(process.env?.TEST) === 'true') {
        app.mainComponent = // path to replacement of app.vue
      }
    },
  },

I need to replace 'app.vue' entirely, not switch index page as your hook does, i thought this could match the 'or mock app.vue' part of the issue's title.

@sky-code Does using the environmentOverrides config object in vitest.config.ts meet your needs?
import { defineVitestConfig } from 'nuxt-vitest/config';

export default defineVitestConfig({
  test: {
    environmentOptions: {
      nuxtRuntimeConfig: {
        app: {
          baseURL: '/test',
        },
      },
    },
  },
});

@ShallowRed You solved a headache i've been having for days.
In our app.vue we had some logic that kept causing Error: [GET] "/xxxx": 404 Cannot find any path matching /xxxx.
And your workaround seemed to have done the trick. Thanks :D


i forget my vm password

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

There is no password on the VM - I'm confused?
Closing for now until proper replication steps are provided


Tesing accuracy is not in line with the paper?

Dear authors, thanks for your kindly sharing of dataset and code. Your work is very impressive and helpful. But when I use your dataset for experiments, I found that the accuracy results on testing set with attention U-Net cannot reach your anounced level such as 93.98% MIoU in your paper. Is there something else I missed? Could you share your training details related with attention U-Net, or your pre-trained model with this network? Looking forward your reply.

亲爱的作者，感谢您友好地分享数据集和代码。您的工作令人印象深刻且很有帮助。但是当我使用你的数据集进行实验时，我发现在使用注意力 U-Net 的测试集上的准确率结果无法达到你在论文中公布的 93.98% MIoU 的水平。还有什么我错过的吗？您能否分享您与注意力 U-Net 相关的训练细节，或者您使用该网络预训练的模型？期待你的回复。

These are parts of my results, FYI.

@Junjue-Wang Thanks for your information. Can you share your training and testing implementation?

This is my  training log
1654484791.7563822.log.

@Junjue-Wang I saw your log results, I suppose you trained with the raw data, which means you did not resize the raw data from 40003000 to 713713 as the paper did. Is that right?

I resize the raw image into 672 x 896.

To keep the same H x W ratio as the raw image.

I resized to 736x736, then used a FCN with 28M parameter and trained 100 epochs, the time is only 13hours. I only used one single 11G GPU. Your training time seems so long.

Anyway, the issued accuracy (93.98%) seems not practical based on our experiments. Acturally, I still expect the author can solve our puzzles. Thanks for replying.

Yes, now I have preprocessed these images offline. And only 5 hours for one experiment.

Respected YJ-He and Junjue-Wang, I hope you are doing good, am a student of bachelors in Computer Engineering and am doing research on post-Disaster damage assessment. Am totally new to segmentation models. I shall be very thankful if you can send me your codes because am unable to run the code given by the rescuenet authors. kindly assist me.
Thanks and regards
Haroon Asad
email<EMAIL_ADDRESS>


Feature request: keyboard navigation (← and →) to browse between pages

This feature is added. #10


Test that terminate causes EOFError

Helping to diagnose https://github.com/jupyter/terminado/pull/142

Okay, this caught a bug where we were using SIGKILL, but it doesn't yet explain what I'm seeing in terminado.  Will leave this open while I'm still debugging.

Okay, I think what happened is https://github.com/spyder-ide/pywinpty/commit/f105bcc4b15dfe3c89d2f047d9373c14ecc7ed3c removed the ability to detect EOF passively using a select listening on PtyProcess.fd.   When we manually read as I've done in the tests, we get the EOF condition.  I need to make a test that uses a select loop that fails, and then fix the behavior.

I think ConPTY doesn't like reading after EOF.  Next thing I'll try is disabling that read only when using the ConPTY backend, since it might result in lost data.

@andfoy I think pytest 7 made the tests in master start failing, I kicked the most recent job and it is also timing out with ConPTY and test_intr().  I'm trying to debug now.

Thanks @blink1073 for tracking this, what happens if pytest is ran with -s? Stream redirection is difficult to track on Windows

Well, it isn't pytest...

Maybe getting access into any CI worker via RDP may give us more information: https://github.com/spyder-ide/pywinpty/blob/b499f20711879bee8b0edcf00919fc78f09e51bc/.github/workflows/windows_build.yml#L55
You need an NGROK account to update this token https://github.com/spyder-ide/pywinpty/blob/b499f20711879bee8b0edcf00919fc78f09e51bc/.github/workflows/windows_build.yml#L64

I don't have the bandwidth to debug ConPTY, my only aim is to get terminado working with WinPTY again.

You could skip the tests for conpty entirely and see if it passes

Yep, I did that and all tests pass with WinPTY.

You could set PYWINPTY_BACKEND=1

My vote would be to skip the specific stalling ConPTY tests, cut a release that fixes terminado, and leave an open issue for full ConPTY support.

Okay, ready for review.  Once this is merged I'll kick the dependabot PRs, and open an issue tracking the ConPTY regression.

@blink1073, could you squash your testing commits and leave the ones that actually implemented the changes necessary for terminado? Thanks!

could you squash your testing commits and leave the ones that actually implemented the changes necessary for terminado

Done!

Thanks @blink1073, I'll draft a new release once I merge the dependency PRs

Done!

Thanks!
And also thank you for your hard work on solving this problem!


Several API changes

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] All static analysis checks passed.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

This PR demonstrates the following changes.
AccessManager changes:

Add RevokeAccessToken method
Add GetAccessTokens method
Add TokenId and Username to CreateAccessToken request and response

Artifactory SecurityService changes:

Add description field to CreateTokenParams

I have read the CLA Document and I hereby sign the CLA

Hi @eyalbe4 , thanks for the review!
Unfortunately, I don't have much time and I'm not sure I'll be able to fix all the issues you mentioned, but still I'll answer your questions.

@gshaibi,
The most important piece for getting this PR merged is the missing tests. If you aren't able to continue working on this currently, let's close this PR for now and if possible in the future, resume the work.


ODLM support concurrently reconciling multiple resources

issue: https://github.ibm.com/IBMPrivateCloud/roadmap/issues/60041
ODLM should be capable of concurrently reconciling two or more OperandRequests to unblock the Keycloak uninstallation process
Verify

Creates an OperandRequest requesting keycloak-operator

ODLM reconciles OperandRequest to create a new OperandRequest edb-keycloak-request to request edb-keycloak

ODLM reconcile OperandRequest edb-keycloak-request to create edb-keycloak operator and its operands.

Fail to delete the keycloak-operator even after deleting keycloak-operator from the OperandRequest and get error msg:
failed to reconcile Operators for OperandRequest keycloak/common-service: the following errors occurred:
- the following errors occurred:
- failed to delete k8s resource -- Kind: OperandRequest, NamespacedName: keycloak/edb-keycloak-request: timed out waiting for the condition
- timeout for cleaning up subscription keycloak-operator

apply the test image: quay.io/yuchen_shen/odlm:reconcile_multiresources

edb-keycloak and keycloak-operator are both deteled
I0828 21:41:50.661952       1 reconcile_operator.go:465] Deleting the ClusterServiceVersion, Namespace: keycloak, Name: cloud-native-postgresql.v1.18.5
I0828 21:41:50.690907       1 reconcile_operator.go:484] Subscription keycloak/edb-keycloak is deleted
I0828 21:41:51.692470       1 reconcile_operator.go:465] Deleting the ClusterServiceVersion, Namespace: keycloak, Name: keycloak-operator.v22.0.1
I0828 21:41:51.715218       1 reconcile_operator.go:484] Subscription keycloak/keycloak-operator is deleted

Reference

Learning Concurrent Reconciling: https://openkruise.io/blog/learning-concurrent-reconciling/

/retest

@YCShen1010
Could we apply the same mechanism to other controllers including OperandConfig, OperandRegistry and OperandBindInfo?


Include Cython source in package

Hello,
First, thanks for your work! We're using this for the new Chinese models we're releasing for spaCy v2.3.
This PR fixes a bug in your setup file that affects your current source release. The Cython files weren't being packaged, which means installation without a wheel is currently failing. This produces confusing results on Python 3.8, where no wheel installer is available.
Python packaging is very fiddly! Especially with Cython source files.

Hello,
Thanks for your support! I've fixed it in 0.0.25 version. Now it can be installed with source code.


REST endpoint Vs SignalR

I have an Angular Mobile App that uses a .NET Core REST endpoint which is hosted in Azure. This app is a mobile application for a user group. I will have a real-time functionality in the future where one of the person in the group modifies the data in the MS SQL database, the other people in the group should be notified and then update their mobile apps. But how often the data is changed is very minimum, so push feature to the mobile app is a luxury feature.
Is SignalR appropriate for this app by considering:

The push feature is used rarely.
Keep the resource usage of the Azure less compared to the REST endpoint.
Get hundreds of images from Azure Blob storage on request from the Angular client.
Get data for hundreds of people when the app starts.
Retrieve user data from the MS SQL on request.

Is SignalR useful compared to Angular HTTP or XMLHttpRequest or Fetch or JSONP?

Can you afford keeping the connection open on the client all the time?

@moozzyk I don't know whether that is expensive or in other words resource intensive. I want to keep the cloud resource usage to minimum.

I am saying that with XmlHttpRequest etc. you control when you start a new request while SignalR will try to keep the connection always open creating requests if necessary even though there is no data to send/receive (e.g. with long polling transport there will be a new request every two minutes if the server has no data to send).


MissingMethodException thrown on Django solution load

Since some time now, when I open my Django solution in VS, an error message pops up pointing to the ActivityLog.xml file which contains several instances of the following exception being thrown:

System.MissingMethodException: Methode nicht gefunden: "Microsoft.VisualStudio.Text.Editor.ITextView Microsoft.Html.Editor.Document.HtmlEditorDocument.get_PrimaryView()".
bei Microsoft.PythonTools.Django.TemplateParsing.TemplateClassifier.GetClassificationSpans(SnapshotSpan span)
bei Microsoft.VisualStudio.Text.Classification.Implementation.ClassifierTagger.d__5.MoveNext()
bei Microsoft.VisualStudio.Text.Tagging.Implementation.TagAggregator`1.d__38.MoveNext()

Any idea what is causing this and how to fix it? PTVS seems to run fine but I keep getting this error when loading a solution. I am using VS Community 2015 with PTVS 2.2.30718.00. Edit: I have recently installed ASP.NET and Web Tools 2015 (Beta8). Could it be caused by this?

It's possible that the Web Tools update has removed that property, which would be a fairly significant issue. I'll check next week and see what they've done.

We need to rewrite our template handling for Django to deal with the changed API of Web Tools. I'm working on it, but it's a fairly painful thing to have to do and it isn't clear when we'll get it out as an update or what form it will take.

I see, that's bad news. In the meantime, until a proper fix is put in place, it would be nice to at least get rid of the error message that pops up during each solution load and let it just fail silently.

So the error message will pop up once per Visual Studio session, when you open a document with the Django editor. It's only popping up on solution load because the document is being opened.
If you right-click the file and choose "Open with", you can select the normal HTML editor. I'm not currently sure what it will do with templated sections without our overrides (it used to do nothing, but they've added much more support for non-ASP.NET templates), but it can't be much worse than having the error. Once you've opened the document once it should reopen with the same editor next time you open the solution.

I've just uploaded a dev build that fixes this issue. Please give it a try and let me know if it is fixed: https://github.com/Microsoft/PTVS/releases/tag/dev-2016-01-19

Did a quick test and syntax highlighting for templates seems to work again and no error message pops up anymore. Great. Will let you know if I come across any issues with this build.

Awesome, thanks.


normalized_xcorr_fast undefined

Hi!  @proto17
I try follow manual https://github.com/proto17/dji_droneid/wiki/DJI-Mavic-3-DroneID-Analysis
But i cant find normalized_xcorr_fast.
May be you forget commit this file?

That file is found in https://github.com/proto17/dji_droneid/tree/main/matlab/updated_scripts.  Add that directory to your MATLAB/Octave path

Sorry, my mistake


Trx file generated is not beautified when opened in editor

@abhishkk Please validate on top of https://github.com/Microsoft/vstest/pull/1521

@abhishkk , please provide details on expectations, also as mentioned by Shiva, can you validate whether this has been fixed as per your expectations?

@abhishkk Assigning to you for No description provided.

Resolved. PR: https://github.com/Microsoft/vstest/pull/1709


tiller installation should not break the silence

This fixes an issue where the tiller installation was not respecting the HELM_TILLER_SILENT variable and I've also made the indentation consistent across the file.

thanks for the PR, can you also bump plugin version, thanks

Done, thanks.


Rename publisher/advertiser to supply/demand

Rename publisher/advertiser to supply/demand,
or, in the case of commitments, we can rename publisher/advertiser to beneficiery/giver
the rationale is that the interaction is not necessarily directly between publishers and advertisres

irrelevant, as the new v4 does not even contain such terms


Upgrade Gradle Wrapper to 4.x

The current Gradle wrapper for the project is 3.5, which is incompatible with JDK 10. Building the project means that the user has to switch to an older version of Java.
I tried to upgrade the wrapper myself, but trying 4.0 or later gave me the following error:
FAILURE: Build failed with an exception.

* Where:
Build file '/Users/Will/OpenSource/jdeps-gradle-plugin/build.gradle' line: 61

* What went wrong:
A problem occurred evaluating root project 'jdeps-gradle-plugin'.
> java.lang.NullPointerException (no error message)

Line 61 contains this code: compile files(org.gradle.internal.jvm.Jvm.current().toolsJar). It looks like the internal implementation has changed.
I'm not entirely sure what it's doing, but regardless, it doesn't work on Gradle 4.x.

Fixed with https://github.com/aalmiray/jdeps-gradle-plugin/commit/04a4b8578d0e32ec5ed6899681494eee7cd885cb


feat(storage): add partial write and read interfaces for state table

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
use value_indeices to partial encode/decode a Row, these interfaces will be used by agg_table and degree_table.
This PR only add a unit test for validation, next PRs will apply this change on agg_table and degree_table.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)

Please add context info about this PR.
Is it part of RFC: KV-like interface for Row-based Encoding ?

Please add context info about this PR. Either by creating a tracking issue or add more description.
Is it part of RFC: KV-like interface for Row-based Encoding ?

Updated, part of https://github.com/risingwavelabs/risingwave/issues/5101


Send full 8 byte of SDO message or full 4 byte of data

In comparison to PDO's CiA301 standard expects 8 byte of the SDO message.
Some devices would abort communication, if not full length of message was sent.
Thanks again for doing this repo

Could you please refer to any documentation which states that this behaviour is required? Thanks.
(I couldn't find any related documentation after a quick Internet search.)

https://www.canopensolutions.com/english/about_canopen/SDO-services.shtml
All SDO services are mapped to CAN data frames with an 8-byte length.
Official CiA301 standards are not free to download, but I'd have a snipped I could PM you.
I guess mostly it works fine without the full 8 bytes, but in some cases libraries check for it.

Please check out the changes I've made in the new branch refactoring. Since I don't have any CANopen devices at hand, I'm unable to fully test it – but the changes should actually fix your issue.


Move the test folder

Move the test folder inside the eclingo folder and correct the necessary paths

I've moved the test folder inside the eclingo folder and fixed the path for the functions test_yale_g94, test_eligible_g94 and test_prog_g94 by going up one parent directory on the app_path, since that was failing once moving the tests folder.
https://github.com/jorgefandinno/eclingo/blob/bc798dd9071315d06c6e5a492f868ce57630f115/eclingo/tests/test_app.py#L105-L106
All the tests are passed now, but I don't know if this fix that I just did was because how I have setup my local working directory, so it'll be good if you can clone this branch and test it too maybe.
My cwd is ../lute_eclingo/eclingo/eclingo.....
Where lute_eclingo is just a folder I created on my Desktop where I cloned the repo and it starts on the first eclingo


feat: rename interaction to questions

Is your feature request related to a problem? Please describe.
The meaning of questions is clearer

options?


zeek intel feed not work

i am  Installed Zeek : 4.2.1-0  add intel feed but my zeek intel .log now show   i am using ubuntu 20.4
i am install zeek below way:
1  apt update
2  apt-get install cmake make gcc g++ flex bison libpcap-dev libssl-dev python3 python3-dev swig zlib1g-dev
3  echo 'deb http://download.opensuse.org/repositories/security:/zeek/xUbuntu_20.04/ /' | sudo tee   	 /etc/apt/sources.list.d/security:zeek.list
4  curl -fsSL https://download.opensuse.org/repositories/security:zeek/xUbuntu_20.04/Release.key | gpg --dearmor | sudo tee /etc/apt/trusted.gpg.d/security_zeek.gpg > /dev/null
5  apt install curl
6  curl -fsSL https://download.opensuse.org/repositories/security:zeek/xUbuntu_20.04/Release.key | gpg --dearmor | sudo tee /etc/apt/trusted.gpg.d/security_zeek.gpg > /dev/null
7  apt update
8  apt install zeek
9  apt-cache policy zeek
10  export PATH=/opt/zeek/bin:$PATH
11  set PATH=/opt/zeek/bin:$PATH
12  zeekctl deploy
#my local.zeek
@load policy/tuning/json-logs.zeek

@load Zeek-Intelligence-Feeds
@load integration/collective-intel
@load frameworks/intel/seen
@load frameworks/intel/do_notice
redef Intel::read_files += {
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/abuse-ch-ipblocklist.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/abuse-ch-malware.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/abuse-ch-urlhaus.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/abuse-ch-threatfox-ip.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/alienvault.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/Amnesty_NSO_Domains.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/binarydefense.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/compromised-ips.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/cps_cobaltstrike_domain.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/cps_cobaltstrike_ip.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/cps-collected-iocs.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/Cyber_Threat_Coalition_Domain_Blacklist.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/dom-bl.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/filetransferportals.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/illuminate.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/log4j_ip.intel",
"/opt/zeek/test.zeek",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/openphish.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/predict_intel.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/rutgers.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/sans.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/stalkerware.intel",
"/opt/zeek/share/zeek/site/Zeek-Intelligence-Feeds/tor-exit.intel",
};

If you are installing on Ubuntu, why are you pulling packages from OpenSUSE?

No user response after 9 days. Please open a new ticket and I will respond as quickly as possible.


add isjlalouviere-ac

@NextSap Please provide us with:

the school official website URL
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@NextSap You might miss my previous comment. Please react.

Hello.

My school's website : http://www.isjlalouviere.be/
Here you can see timetables who proofs that my school delivers IT courses http://www.isjlalouviere.be/up/article/6a01459529e4c7f21c49dca7e796b6b6.pdf
For your last request I'm a bit confused. I don't know how to proof that.

@NextSap Thank you for clarification!
Simple request at Google, like this, usually shows that the email domain is mentioned at the official website:

"isjlalouviere-ac.be" +site:www.isjlalouviere.be

However, in this case isjlalouviere-ac.be email domain is not mentioned at the official website.
WHOIS database in Belgium does not show that the domain belongs to the school: https://www.dnsbelgium.be/en/whois/info/isjlalouviere-ac.be/details
Please attach a copy of an instruction which you were given by the school, which tells you to use this email address. You can also comment here with a link that instructs the students to use their addresses @isjlalouviere-ac.be.

Proof.pdf
Here are the instructions my school gave me.
I vonluntary hide some personal informations like my last name (orange), my license code (blue) or my password (gray).
isjlalouviere-ac.be -> ac because the school uses the plaform "Accès-Cible".
Thanks in advance.

@NextSap Thank you for the clarification. Pull request merged.


Write a Kotlin program to find surface area of a cone

Description
Write a Kotlin program to find surface area of a cone
Surface area of cone = π × Radius × (Radius + Slant Height)
Radius       : 2
Slant Height : 3

Surface area : 50.26548245743669

How to contribute

Save the solution in program/FindSurfaceAreaOfACone.kt file
Add FindSurfaceAreaOfACone.kt file in find-surface-area-of-a-cone folder

!assign


Rename “git-pull-request” to “git-create-pull-request”

@charlierudolph @allewun @ricmatsui
Fixes #413 and #507

:ship: looks like you got them all

I disagree with this and want to deal with this collision in a different way.
git pull-request and git-repo are fundamentally different from the rest of the commands as they integrate with external services and are not just a series of git commands.
I believe we should move git pull-request and git repo into their own project where we introduce our own new command, call it gr (git remote) for demonstration purposes. We then have  gr pull-request and gr repo and a user can do the same thing with hub and alias them to git if they want.

I am open to having this conversation, but let's do this separately. This is a bigger architectural refactor, with a lot of pros and cons to it. So, let's do a 2-step approach here: first get this change in to fix the immediate issue that blocks users, harms our reputation as being incompatible with git-extras and hub, and delays a bigger roll-out of GT. Step 1 then gives us the time to talk about how we handle peripheral commands in general, and whether we do a plugin system or separate projects.
Sounds good?

I have added #512 to track the idea of extracting these commands. Let's continue the conversation around extracting there.

I think this should be named new-pull-request as it opens the form for creating a new pull request instead of actually creating a pull request.

The main reason I am against this is that we are basically submitting to another tool. I believe our pull-request command is named properly, it just so happens that hub has a command with the same name. (something I brought up in #413). Thus we have a competing command with hub. I don't think competing is a bad thing, but we focus our energy on where we want to compete. git pull-request and git repo are nice commands but they have a different market from the rest of the commands.

Good conversation here.
I like the change to new-pull-request.
In terms of naming, we have a command that creates PRs, and one that ships them, so I think distinguishing the two through more specific naming is appropriate. What does git pull-request actually mean? Creating one? Deleting one? Merging one? Commenting on one? Changing one?
In terms of competition, we don't live in a vacuum, and there are other tools around us that we need to be aware of. We have our own focus (providing a generic high-level layer above Git to make it more usable in a generic, vendor-independent and workflow-independent way). There are some overlaps to other tools who provide vendor-specific or workflow-specific Git enhancements. That's okay, and will always be that way.
If we avoid competition, then we also submit to other tools. Overall I think we should focus on what's best for GT, independent of what others do, while trying to be good citizens in the ecosystem (i.e. don't be incompatible).

pull requests are something entirely different from shipping feature branches. Pull requests don't exist in git. They exist on GitHub and Bitbucket and other services. git ship would still be useful even if the idea of a pull request changes.
I don't believe the argument that pull-request is no generic holds much merit. Its simple and does what you expect. Using that same logic, I think hack and extract are too generic. I don't know what hacking means in term of git, and extracting what?

I have updated this PR to new-pull-request. I think no matter whether we extract this command later or not, this is a good intermediate step.

Leaning on OO principles, where nouns describe objects, and verbs describe actions performed on those objects, I meant that commands (which are actions), should be verbs. Many Git commands are verbs ("checkout"), and the ones that are nouns ("branch") usually need a subcommand to clarify what exactly you want to do with the object described by the noun ("git branch -d" etc). So a noun is only used to cluster a number of related actions, but it is not a stand-alone command. Similar to REST, running a noun command by itself shows all instances of that object type ("git branch" shows all branches). All of our GT commands are therefore also verbs, except "town", which also needs subcommands to qualify actions.
What does this mean for this command here (creating PRs)? If we provided a comprehensive solution that allows to do several things with PRs, the right way of naming it would be git pull-request, with subcommands like (create, approve, comment-on, delete), and with the command showing all PRs if run without a subcommand. Since we only do one thing here, showing a form to create a new PR, the command name should be a verb. "new" isn't really a verb, but close enough here. ;)

My main qualm with create is that we aren't actually creating a pull request. We are opening a browser to a form for a new pull request (thats why I suggested that). open-pull-request-form is probably the most accurate may to use a verb in my mind (though not the most fun to type out)

I agree with your reservations against create. If we lean on the Rails nomenclature for REST as a (admittedly very thin) straw here, new shows a UI for creating a resource, and create actually creates the resource. So I can live with new here. Its not perfect, but good enough for me. Also for you?

After talking in person, we agreed to respect hub and git-extras as existing state-of-the-art, and keep the name as git new-pull-request. So this PR should be ready to merge now. Creating the new command-line application separate from Git is tracked in #515.

Need to realphabetize things in a bunch of places (because of the rename of create-pull-request to new-pull-request). After that's resolved this is good to :ship:


[Feature] Ability to retrieve content of non-media columns in a table without downloading entire artifact and associated media

Is your feature request related to a problem? Please describe.
We use tables to log validation results when a training run is finished. A row in this table will typically include the path to the input image, a score, and a W&B Image object.
To access any data in this table at a later stage through, for example through my_table.get_column['path'], the entire artifact gets downloaded, including W&B image objects.
Describe the solution you'd like
Ability to do get_column on a non-media column and retrieve its content without having to download the entire artifact.
Describe alternatives you've considered
A workaround is to download the table json and load it, but this defeats the purpose of having tables in the first place.

Hi @nicjac, I completely agree. We're tracking a ticket for this internally but I'll add this request to the ticket and make sure to write back here when we implement this feature.

We are closing this issue due to inactivity. If you feel that it merits additional review, please provide a reproduction and add a comment explaining why. Thank you.


travis: add PostgreSQL to test matrix

stems from #3079 roadmap issue

Invenio-DB and most of new Invenio 3 packages are automatically tested on SQLite, PostgreSQL, and MySQL.


Fix incorrect case of 'sprig' import

What this PR does / why we need it:
The canonical import path for sprig is github.com/Masterminds/sprig. Using different cases can lead to issues when trying to import KUDO packages in other projects.
This PR also updates sprig to v2.22.0.

ugh this issue
yeah, ok. :+1:


Fix for pulumi/pulumi-component-provider-py-boilerplate#45

I was trying to understand writing pulumi provider / components, and I noticed the current main branch doesn't work out of the box. It appears to be related to #40 and #45.
In order to get it working for me, I decided to make the following updates:

Port the changes for pulumi package gen-sdk over from pulumi/pulumi-pulumiservice#240 (fixes #45 issue.)
Update the pulumi version from v3.30.0 ->v3.116.1 (current latest release, and also fixes #40.)
Update the pulumi-aws provider from v5.3.0 -> v6.37.0 (current latest release)
Delete/remove pulumi-gen-xyz directory and related files.

After the changes, I could successfully build/deploy the example in the README.md.

For me to get a working run of make build_python_sdk I needed to do a little more work. Otherwise this looks great! Thanks for putting in the time!


Fixing tests in BeginnerTutorial

Tests are not passing the way they are written in the tutorial, so I exported delay() to be able to check deeply against it in tests (deepEqual). Everything passes now.

Thanks!


No BSD downloads here

BSD link removed from download page.
This should solves #868
Tested OK on the following platforms:
| Edge | Chrome | Firefox | Opera | Vivaldi
-- | -- | -- | -- | -- | --
Windows 7 | - | OK | OK | OK | OK
Windows 10 | long links glitch | OK | OK | OK | OK
Ubuntu 18.04 | - | OK | OK | OK | OK
CentOS 7 | - | - | OK | OK | OK

Anyway, it will not be hard to add any new kind of downloads if we have to.


Can you guys bump requests to requests==2.32.0?

Dependabot says 2.31.0 has some vulnerability and resend isn't letting us bump the requests version: https://github.com/sweepai/sweep/actions/runs/9217012429/job/25358304944.

@kevinlu1248 fyi: the latest release points to request >= 2.31


When newly proposed idea is "unavailable"...cannot go to existing idea

alpha.democracy.earth/vote/should-france-exit-eu
link should take me to existing proposal

This doesn't work this way anymore, closing .


Status of go-sup?

Hello,
coming from Erlang, I naturally looked for Go equivalents of Erlang supervisors, so I found also this package.
I would like to know its status: are you using it or aware of somebody using it? According to pkg.go.dev, it seems not imported by any (public) project. Would you suggest another approach/package today?
Thanks

I have some code which uses it and has worked well enough I've never rebuilt it.
I don't know of anything better.  Not for lack of wanting it.
I've also kicked around other variations of this code periodically in spare time over the years, trying to get a more ergonomic API / better debugging under all cases (including in response to SIGQUIT) / trying to make it more foolproof.  That's been... easier said than done.
I'll be happy to push my latest to a branch if someone else wants to run with it.
I'm not sure I'm too likely to keep working on this topic, otherwise.  I'm writing considerably less golang now than I did a few years ago.  The type system and the error handling has just be too recurrently painful to me, in general.  This topic of supervisors, more specifically, seems extremely difficult to solve ergonomically without the use of macros (you really need a select in a lot of places to be fully correct, and those are a huge amount of syntax-heavy boilerplate to throw at a user... and you if you want to do it as a library, you can do it with reflection, kindasorta, but now your syntax is getting even farther from the norm, and generating tons of garbage objects, and and and... it's just... oof), or some other language features that we simply don't have.
it's very difficult to have nice things in golang :sweat_smile:

I've just pushed my last work to the https://github.com/warpfork/go-sup/tree/v6-dev branch, in case it's of interest.
It was a pretty total rewrite, and perhaps not entirely finished, but I think improved on a lot of things.  I'd probably pick up from there if trying to get further on this.
It looks like the "pingpong_noguards" test completely works and demonstrates how to use it.
There's also something called "pingpong_guarded" which was... trying to go the way of having those reflect-based features I mentioned, aiming to save some boilerplate and be more resilient against bad constructions.  It's... unfinished.
The "promise" feature looks really nice!   :))  It looks complete, and stands alone nicely.  It supports generics.  It's quite efficient.  It looks like it's well tested and also plays nicely with the race detector.  I think I'd use this!  And the documentation seems very complete about how to avoid problematic compositions (e.g. what hooks need to avoid blocking, etc).
There's probably a lot of other rough edges, though.  I've fixed a few todos already in the last few minutes just dusting it off, so no doubt there's more.

Thanks a lot for the update and the rationale within. I'll come back with my feedback once I have played a bit with this package. I plan at least to propose an example for a newcomer to get started or to get a feeling of how to approach the API.
Sometimes Go frustrates me too, but I stick to it because it frustrates me less than other languages or because I find that it still has the best trade-offs wrt complexity at scale...

Sounds good :)  Honestly, if you come back holding some strong opinions about the ergonomics and naming you'd want, but otherwise interested with the approach, I'd probably be nerd-snipe'able into helping get it where you want it to be.  (Looks like half the things tiring me out last time I put that branch down were just API naming decisions, heh...)


Code action: Merge match branches

I've often wished for this after using "fill match branches":
match x {
    Foo => (),<|>
    Bar => (),
    Baz => (),
}

->
match x {
    Foo |
    Bar => (),<|>
    Baz => (),
}

I think it should only apply if the branch expressions are the same, and only merge two branches (you can repeat it to merge more then). The reverse would of course also be useful.

Note that we currently don't have a good API to check if two arms are equivalent, but I guess for starters "compare lengths, then compare text" would work

only merge two branches
Actually, I guess if you select a range including multiple branches it could merge all of them...

I'd like to try this issue.


touchmove Event in 45 degree angle，swiper stopped halfway

maybe a bug;
When I scroll the window in 45 degree angle in the  Swiper Elements,it stopped at the half way ,not slide to the prev slide or the next slide . In Android and IOS.

I do not know why,but when I remove the  CSS swiper-container overflow-x:visible ,it work properly;
the code before and after is below,Does anyone know why?
.swiper-container{
	padding-right: 50px;
	/*overflow-x:visible;*/
	width:100%;
	-webkit-box-sizing:border-box;
	box-sizing: border-box;
	box-shadow: none;
}

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Get DCP number from agenda description

This is so it may be used to generate a link to the DCP on the web page.

@alexlyp @karamble is going to finish up the web part after this


Bump version to 0.6

Yes, exactly. Once the new version is registered, CompatHelper won't try to add compat with 0.5 anymore.


Allow full control of advertised listeners

This PR enables configuring advertised listeners without creating a service for external access. This makes it possible for helm chart deployments to work with a variety of external access methods, such as LoadBalancer service.
Fixes #258, fixes #256
Changes

made external.type optional. If undefined, then advertised listeners will be configured without creating a service
made listeners.*.external optional
removed $externalListener.nodePorts, added $externalListener.advertisedPorts:
a. when external.type is NodePort, the first element of the array is the port opened on each node
b. when external.type is undefined, each element of the array is the port to be opened externally for accessing the pod where array index == statefulset replica index
c. when advertisedPorts is undefined, listeners.*.external.*.port then listeners.*.port is used
removed container start command listener flags, add listeners other than RPC to statefulset initContainer
removed unused templates
switched external.addresses from string to array
removed listeners.*.external.type and assume external.type is used across all externally enabled listeners
fix schema_registry_api in configmap.yaml (was schema_registry)
update test-schema-registry-status to use same short hostname URL as other tests (rather than FQDN)
fixed issue after NODE_ID was renamed to POD_ORDINAL
external ports take default Redpanda values, internal ports take default Redpanda values + 1000, advertised are within

listener
internal
external
advertised

kafka
10092
9092
31092

http
9082
8082
30082

I haven't yet gone through the steps for adding a LoadBalancer (or other) service that allows making use of the advertised listeners. That isn't part of this PR, but it will be a very likely use case for some people, so I'd like to document those steps at some point.
Also I'll need to make a few minor changes to the Kubernetes quickstart docs related to this PR's changes.

Steps to make Redpanda's external Kafka listener available via a load balancer (by default port 9092 on each node):

Create the file lb-redpanda-0.yml with the following content:

apiVersion: v1
kind: Service
metadata:
  name: lb-redpanda-0
  namespace: redpanda
spec:
  type: LoadBalancer
  ports:
    - targetPort: 9092
      port: 9092
      nodePort: 31092
  selector:
    statefulset.kubernetes.io/pod-name: redpanda-0
  externalTrafficPolicy: Local

kubectl apply -f lb-redpanda-0.yaml
(For minikube only) minikube tunnel
Get the external IP with kubectl get service/lb-redpanda-0 -n redpanda
(From local rpk) rpk cluster info --broker <external_ip>:31092

A similar load balancer would need to be created for each broker.

Moving to draft because I'm seeing to issues:

The advertised host for each broker is the same
Updates to advertised listeners are rejected due to finding a duplicate advertised endpoint. This may be an issue more with the helm upgrade process than this PR, but I need to look more into this.

The two issues mentioned earlier are resolved.

This was related to a partial renaming of a variable
This seems to only be an issue in minikube. I tested upgrading the helm chart multiple times in EKS without any issues. I still want to find out why minikube behaves differently, but at this point I'm sure it is an issue with minikube and not this PR.

Could we introduce external.type=LoadBalancer? That way, the Helm chart can control those Services like we do for NodePort and users don't need to worry about manually creating one for each worker node.

Renumbering all the ports will break existing users. Please either find an alternative, or provide backward compatibility.

Hi there,
I'm waiting for this PR to be merged as well, I have the need to expose my Redpanda instance using a LoadBalancer (Private GKE).
Is there an ETA?

Hi there,
I'm waiting for this PR to be merged as well, I have the need to expose my Redpanda instance using a LoadBalancer (Private GKE).
Is there an ETA?

Hey @anggutie-dev ! No ETA yet, but this PR is in active review so shouldn't be much longer. I would expect it to be done within a week tops (unless something major takes priority in the meantime).

Regarding the port changes in this PR:
Existing
Redpanda has listeners for various services, and most of these services have an internal and external variant. At the same time, we could have multiple external listeners. So for example, the kafka and http APIs by default have the following listeners/ports:

listener
docs
internal
external
advertised

admin
9644
9644
N/A
31644

kafka
9092
9093
9094
31092

http
8082
8082
8083
30082

schema
8081
8081
8080
30081

The above state is confusing, as the Kafka API is not available anywhere via port 9092. Also, there isn't any similarity or formula for how ports are handled across the different listeners (neither within kafka listeners nor between kafka and http listeners). The listener ports are incremented by 1 for admin and kafka, but decremented by 1 for the schema registry. The reason is because incrementing by 1 is impossible when schema and http services are at 8081 and 8082 respectively. Another issue is the schema registry has an external port defined, but in reality it uses the same port across internal and external requests (same as admin API). Advertised ports must be within the NodePort range, so we are adding 22K to the API's documented port.
Future
The port changes in this PR apply the same formula across all listeners:

listener
docs
internal
external
advertised

admin
9644
9644
N/A
31644

kafka
9092
10092
9092
31092

http
8082
9082
8082
30082

schema
8081
8081
N/A
30081

The external ports match our documentation and the default behavior of the Kafka ecosystem. Most people will want to connect to a Redpanda cluster via external clients, so internal ports are modified instead (see below)
The internal port is +1K to the documented port. This is applied across each listener when needed. This modification is needed only when the API has an external port (http and kafka APIs).
Advertised ports is +22K to the documented port to bump it up into the NodePort range.
The schema registry external port is removed, since all schema clients connect to the same port. This was improperly included in the listener definition in an attempt to keep all listener definitions as similar as possible.


CGameAllocator implementations (WIP)

This pull request is now managed with CodeReviewHub. Refresh.

Currently missing functionality:

[ ] FixupAllocPtrs
[ ] FindFreeBlock
[ ] FindFreeBlockFromTopOfHeap (started, but incomplete)
[ ] CSmallAllocPool
[ ] CMediumAllocPool
[ ] Free
[ ] Alloc (started, but currently only grabs from TopOfHeap)

[ ] FixupAllocPtrs
[ ] FindFreeBlock
[ ] FindFreeBlockFromTopOfHeap (started, but incomplete)
[ ] CSmallAllocPool
[ ] CMediumAllocPool
[ ] Free
[ ] Alloc (started, but currently only grabs from TopOfHeap)

[ ] FixupAllocPtrs
[ ] FindFreeBlock
[ ] FindFreeBlockFromTopOfHeap (started, but incomplete)
[ ] CSmallAllocPool
[ ] CMediumAllocPool
[ ] Free
[ ] Alloc (started, but currently only grabs from TopOfHeap)


move ecr repos to opg-org-infra

Purpose
ECR Repositories are currently created later than they are required on the pipeline. ECR Repositories can live in Org Infra with the rest of the repositories code.
Fixes UML-1372
Approach
Remove code that creates ECR for lambda functions
Replace resource code with data sources
Run terraform state rm to stop managing the resources from this state file.
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the Use a Lasting Power of Attorney service
Checklist

[ ] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes

From what I understand looks good to me :)


Add the possibility for prefabs to have planks as floor

This is the strategy used by the mineshafts, when they are generating outside of stone. We need similar behavior for prefabs, so that mineshafts can be generated by a prefab generator

This is not quite possible to do with a merge strategy (as was the original intent), because that would require that planks weren't used anywhere else in the prefabs. However, they are - not only for the floor, but also for the "supports".
Closing as "won't implement".


Can't print to any device, win64 windows 7 with Supermium 126.0.6478.254 R4

As stated - couldn't get far enough to check for ongoing stability issues with win64 windows 7 sp1 after ver 119, which continues to be fine... Thanks for your ongoing work!

I confirm the inability of the latest version 126.0.6478.254 to print in Windows 7/SP1 64-bit. It can't print, neither to a hardware printer nor to a virtual printer like the CutePDF utility.  However, "Save as PDF" works. Could it be a spooler access issue of 254? It should be noted that twhen attempting to print, he printer icon appears in the tray.  Hovering the mouse cursor over it displays "0 document(s) pending for 'username' ", and after a while the icon disappears.

also can confirm, but with another detail
like martrant noted, there notification in the tray. But Adobe' virtual printer also showing small popup with "printing" progress and for "printing", initiated by Supermium, it stay at 0
Acrobat XI popup (seems its progress bugged after "printing" some word docs)

Also i try XPS printer - no luck.
In both cases printing queue is empty

Try 126.0.6478.256 R5. It has been fixed on my end.

Same on my end - no errors, able to "print" with XPS and Adobe virtual printers


Adding Alpaquita Linux support

Enhancement Type
A completely new feature
Describe the enhancement
Alpaquita Linux is said to be faster than compared to alpine linux, so it might be worth adding support to it.
https://bell-sw.com/alpaquita-linux/

Their site says its free to use and they already provide docker support :yum:

Thanks for pointing that out.
It looks like a lot of the tuning and benefits come from the kernel layer, in which case the container's use of their base image may not help much since containers run in the host's kernel.
So, if anything the onus is more on you to run Alpaquita on your bare metal or VM host and then run the Minecraft docker container on that to gain the most benefit.
With that said, I don't mind trying to add a build variant to use a base Java image from theirs, but my expectations for tangible benefits are low. For one thing I have previously found Bellsoft's Liberica Java images to be quite bloated, but they seem to have addressed that with this one.


Should @context be string array?

The docs show @context as string, but downloaded device templates have it a string array

The modeling language is built on JSON-LD where each value (or object of a predicate), barring other constraints, can be a single value or multiple values. In JSON-LD this is represented by a single value or an array (set) of values, so @context can be either a single value or an array of values if there is more than one context.


Move static methods to Tensor (non generic class) where we can

We should do this where possible to get generic type inference.

Fixed with https://github.com/dotnet/corefxlab/pull/1806


Can't connect to PostgreSQL database

Hi,
I can't to connect to my PostgreSQL database (running pg-amqp-bridge with PostgREST).
This is the error I get:
thread '<unnamed>' panicked at 'Could not connect to PostgreSQL: Io(Error { repr: Custom(Custom { kind: InvalidInput, error: StringError("invalid message length") }) })', /checkout/src/libcore/result.rs:859
Here is the backtrace:
   0: std::sys::imp::backtrace::tracing::imp::unwind_backtrace
             at ./checkout/src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::_print
             at ./checkout/src/libstd/sys_common/backtrace.rs:71
   2: std::panicking::default_hook::{{closure}}
             at ./checkout/src/libstd/sys_common/backtrace.rs:60
             at ./checkout/src/libstd/panicking.rs:355
   3: std::panicking::default_hook
             at ./checkout/src/libstd/panicking.rs:371
   4: std::panicking::rust_panic_with_hook
             at ./checkout/src/libstd/panicking.rs:549
   5: std::panicking::begin_panic
             at ./checkout/src/libstd/panicking.rs:511
   6: std::panicking::begin_panic_fmt
             at ./checkout/src/libstd/panicking.rs:495
   7: rust_begin_unwind
             at ./checkout/src/libstd/panicking.rs:471
   8: core::panicking::panic_fmt
             at ./checkout/src/libcore/panicking.rs:69
   9: core::result::unwrap_failed
  10: pg_amqp_bridge::spawn_listener_publisher::{{closure}}
  11: std::panicking::try::do_call
  12: __rust_maybe_catch_panic
             at ./checkout/src/libpanic_unwind/lib.rs:98
  13: <F as alloc::boxed::FnBox<A>>::call_box
  14: std::sys::imp::thread::Thread::new::thread_start
             at ./checkout/src/liballoc/boxed.rs:650
             at ./checkout/src/libstd/sys_common/thread.rs:21
             at ./checkout/src/libstd/sys/unix/thread.rs:84
  15: start_thread
  16: clone

Could someone please point me in the right direction?

Which version of PostgreSQL are you using? OS?
Also check your postgres connection uri(postgres://user:pass@host:port) if it contains reserved characters you might need to percent encode them.

Hi,
I'm using PostgreSQL 9.6 on Ubuntu 14.04 LTS.
I initially did have a problem where Postgres on Azure automatically appends @my-server-name to each username created on Postgres when trying to authenticate, and got a different error message for that (Invalid '@' character) but after URL-encoding the @ character, this is the only error I get now. There doesn't seem to be anything else that needs to be encoded.  There are -s in the username, would that also need to be URL-encoded?

Correction: Postgres is version 9.6, but since it's hosted as an Azure service, I'm not sure what OS it runs on. My docker master is running Ubuntu 14.04 and I'm running pg-amqp-bridge as a docker instance.

The - chars should be safe, when the connection fails do you get retrying? Which pg-amqp-bridge version are you using? if it's latest(0.0.6) there should be a message like:
Attempting to connect to PostgreSQL..
<Error here>
Retrying the PostgreSQL connection in 2 seconds..
<Error here>
Retrying the PostgreSQL connection in 4 seconds..
......

After upgrading to latest / 0.0.6 from Docker Hub, I now get the following:
Attempting to connect to PostgreSQL..
Error(Io(Error { repr: Custom(Custom { kind: InvalidInput, error: StringError("invalid message length") }) }))
Retrying the PostgreSQL connection in 1 seconds..
Error(Io(Error { repr: Custom(Custom { kind: InvalidInput, error: StringError("invalid message length") }) }))
Retrying the PostgreSQL connection in 2 seconds..
Error(Io(Error { repr: Custom(Custom { kind: InvalidInput, error: StringError("invalid message length") }) }))
Retrying the PostgreSQL connection in 4 seconds..
Error(Io(Error { repr: Custom(Custom { kind: InvalidInput, error: StringError("invalid message length") }) }))
Retrying the PostgreSQL connection in 8 seconds..
Error(Io(Error { repr: Custom(Custom { kind: InvalidInput, error: StringError("invalid message length") }) }))
Retrying the PostgreSQL connection in 16 seconds..

I think I've narrowed down this issue. My PostgreSQL server requires connection over SSL. What should I specify in POSTGRESQL_URI to be able to connect over SSL?

Right now there's no SSL connection support, see this line https://github.com/subzerocloud/pg-amqp-bridge/blob/master/src/main.rs#L52 that has TlsMode::None, that would have to be changed to SslMode:Require as described in this issue https://github.com/sfackler/r2d2-postgres/issues/7.

Also you could try disabling SSL on your Azure potsgres instance with https://docs.microsoft.com/en-us/azure/postgresql/concepts-ssl-connection-security#using-the-azure-portal and see if the connection now works to see if that's indeed the problem.

It was pretty strange. At first I received the invalid message length error, but after restarting the server I started to received an SSL-related error (not the one you mentioned above, but similar). I think it was Azure's custom SSL-must-be-on error message.
Everything started working when I turned off SSL on the Azure instance. It's not ideal though, but I'm limiting access to the server from a specific IP range so it's OK for now. Would you be able to add SSL support to your feature-list?

I see there it's been discussed on that project since 2015 but nothing has come of it.  Thanks for all your help. I'm going to close this issue now.


user-guide.md: add a few examples and remove mentions about futures

Add a few examples to the user guide docs and remove the mentions about futures so that it is a bit closer to what's actually in finchx.
Also add a link to giter8 template to the index.

Codecov Report

Merging #1268 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1268      +/-   ##
==========================================
- Coverage   81.16%   81.07%   -0.10%
==========================================
  Files          55       55
  Lines        1009     1009
  Branches       42       42
==========================================
- Hits          819      818       -1
- Misses        190      191       +1

Impacted Files
Coverage Δ

core/src/main/scala/io/finch/Endpoint.scala
72.22% <0.00%> (-0.56%)
:arrow_down:

Continue to review full report at Codecov.

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


What about SEO?

Google and probably other search engines are looking not just at the content, but at the layout as well to figure out where the content is places.
How does this affect SEO? Have anyone tried to see how Google sees the site in this tool for example?
https://developers.google.com/speed/pagespeed/insights/
The tool shows a screenshot of your site after generation.

That is a great question that I'm not sure I have an expert answer for. When I look up pages built with EQCSS in SEO tools, like the PageSpeed Insights tool, they show up rendered (with JS-powered styles applied) which makes me think for those algorithms that take measurements it would happen after EQCSS had done its thing. Here's an example of an SEO tool showing a page with EQCSS applied: https://developers.google.com/speed/pagespeed/insights/?url=http%3A%2F%2Felementqueries.com%2Fdemos%2Fcontent-blocks.html
My non-expert guess would be that it would affect web crawlers in a similar way to how adding a JS-powered carousel to a site would affect crawling. In both of those cases, you're putting the content you want visible in your HTML (which could look weird unstyled) and relying on JavaScript to alter the visual display of that markup.
Google can crawl and index JavaScript-powered Single Page Applications and web apps, so I think although EQCSS is an 'unknown' tool at this point for most site analysis tools, I don't believe it has a detrimental effect on how crawlers are able to access and index your content.
Also, EQCSS demos and elementqueries.com have been doing just fine in Google's index, so anecdotally using EQCSS seems to be okay :)


0.2.0 update

New update including:

Connection timeout
Widget cache
tests
setup project python 3 and 2
some refactoring and optimizations

Codecov Report

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

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage        ?   94.80%
=======================================
  Files           ?       23
  Lines           ?     1077
  Branches        ?        0
=======================================
  Hits            ?     1021
  Misses          ?       56
  Partials        ?        0

Continue to review full report at Codecov.

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


Change look of "Support persons" button

Consider changing how the "Support persons" button under Values looks like to emphasize the difference Values

My suggestion would be to change the 'mode' of the Button component to 'contained'.
https://callstack.github.io/react-native-paper/button.html


package not working in production environment

here is my code :
public function send_fcm($device){
    $optionBuilder = new OptionsBuilder();
    $optionBuilder->setTimeToLive(60*20);

    $notificationBuilder = new PayloadNotificationBuilder('Delist Request');
    // $notificationBuilder->setBody($debtor->Fullnames.' alias '.$debtor->nickname_company.' wants to delist. Tap here to open app')
    $notificationBuilder->setBody(' Tap here to open app')
                        ->setIcon('icon')
                        ->setChannelId('noty_channel')
                        ->setSound('default');

    $dataBuilder = new PayloadDataBuilder();
    $dataBuilder->addData(['title' => 'Delist Request',

                            'body' => 'kindly delist me',

'largeIcon'=>'icon',
'playSound'=>true,
'soundName'=>'default',
// 'debt_code'=>$debtor->code,
// 'name'=>$debtor->Fullnames
]);
$option = $optionBuilder->build();
$notification = $notificationBuilder->build();
$data = $dataBuilder->build();
    $token = $device;

    $downstreamResponse = FCM::sendTo($token, $option, $notification, $data);

     echo $downstreamResponse->numberSuccess();
     echo $downstreamResponse->numberFailure();

}

found out that there was a mismatch between the sender id in config file and in my android app.Thanks


Updating the code documentation on server-main

Some code smells that I would like to fix, since there are good patches of code that would be nice to have comments there explaining what the code does. The file of focus is the server-main file in the src directory. The parts of code that I would like to update documentation on is the the whole 'shouldSpawnCli' conditional and the prompt function, since it would be nice to have consistent commenting on seemingly complex functions on upcoming contributors like me.

If possible, I would like to be an assignee if this issue is available for fixing.

Thanks for the help but I think in this case the code speaks for itself and the comments are not necessary,