Adds search bars

Adds new search bars to replace search dialogs. The search bars can be triggered using the existing buttons or the keyboard shortcuts. To close an open search, re-enter the shortcut that triggers the search bar to appear.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great!
Is there a shortcut key to jump to "next/prev" item?

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Ah great, seems to work! But I found a bug :(
Here I do a search on '2568' and I get the first item found. So far so good:

And then I press do the Ctrl+Shift+N thing, and it jumpts to the 7th item found:

In some earlier tries I got it to jump to the 3rd item found. In any case, I couldn't get it to jump to the 2nd item. Weird!

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Ah great, seems to work! But I found a bug :( Here I do a search on '2568' and I get the first item found. So far so good:

And then I press do the Ctrl+Shift+N thing, and it jumpts to the 7th item found:

In some earlier tries I got it to jump to the 3rd item found. In any case, I couldn't get it to jump to the 2nd item. Weird!

Hmm that is strange. I will look into that. Has that ever happened before? Just a quick glance at the code and the next/prev item logic is handled in dive_tree_view.cpp and command_buffer_view.cpp which have been untouched.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Ah great, seems to work! But I found a bug :( Here I do a search on '2568' and I get the first item found. So far so good:

And then I press do the Ctrl+Shift+N thing, and it jumpts to the 7th item found:

In some earlier tries I got it to jump to the 3rd item found. In any case, I couldn't get it to jump to the 2nd item. Weird!

Hmm that is strange. I will look into that. Has that ever happened before? Just a quick glance at the code and the next/prev item logic is handled in dive_tree_view.cpp and command_buffer_view.cpp which have been untouched.

Good point, maybe this is a pre-existing bug. I've only ever used the search functionality in the PM4 packet view (and I used it a lot). And it always worked there before. Maybe this bug is specific to the event view. If you can revert back to before your changes and repro the same bug, then it's not something related to your changes and shouldn't affect this PR. However, if the bug didn't exist before and exists now, then you know it's a regression. Can you give it a try? It'll be good to know that the bug isn't limited to just Dive on my machine for some reason.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Ah great, seems to work! But I found a bug :( Here I do a search on '2568' and I get the first item found. So far so good:

And then I press do the Ctrl+Shift+N thing, and it jumpts to the 7th item found:

In some earlier tries I got it to jump to the 3rd item found. In any case, I couldn't get it to jump to the 2nd item. Weird!

Hmm that is strange. I will look into that. Has that ever happened before? Just a quick glance at the code and the next/prev item logic is handled in dive_tree_view.cpp and command_buffer_view.cpp which have been untouched.

Good point, maybe this is a pre-existing bug. I've only ever used the search functionality in the PM4 packet view (and I used it a lot). And it always worked there before. Maybe this bug is specific to the event view. If you can revert back to before your changes and repro the same bug, then it's not something related to your changes and shouldn't affect this PR. However, if the bug didn't exist before and exists now, then you know it's a regression. Can you give it a try? It'll be good to know that the bug isn't limited to just Dive on my machine for some reason.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Feels great! It's definitely much easier to use than before. But I have a couple of feature requests:

After a search, could there be a shortcut key to jump to next/prev searched item? If I'm going to use the keyboard shortcut to get the new search dialog, would be great if I didn't have to resort to the mouse again just to get the "next" item
If I decide to cancel a search, could there be a button I can press or a shortcut like ESC (or both) to make the search dialog disappear?

Ah I see if I press Ctrl-F/Ctrl-Shft-F, it makes the dialog disappear. Nice. But would be nice to have a UI element to cancel it too, for those people who doesn't use shortcut keys.

Ok, were you thinking a cancel button next to the input field or just making the icon to the left a button that when clicked closes the search bar?

I mean like an X cancel button like you'd see in normal find dialogs. Like this one from Chrome:

Ahh ok, added the cancel buttons:

Looks good, the cancel button works great! Is there a shortcut key to jump to "next/prev" item?

Yes, it is Ctrl+Shift+N for the next item and Ctrl+Shift+P for the previous item.

Ah great, seems to work! But I found a bug :( Here I do a search on '2568' and I get the first item found. So far so good:

And then I press do the Ctrl+Shift+N thing, and it jumpts to the 7th item found:

In some earlier tries I got it to jump to the 3rd item found. In any case, I couldn't get it to jump to the 2nd item. Weird!

Hmm that is strange. I will look into that. Has that ever happened before? Just a quick glance at the code and the next/prev item logic is handled in dive_tree_view.cpp and command_buffer_view.cpp which have been untouched.

Good point, maybe this is a pre-existing bug. I've only ever used the search functionality in the PM4 packet view (and I used it a lot). And it always worked there before. Maybe this bug is specific to the event view. If you can revert back to before your changes and repro the same bug, then it's not something related to your changes and shouldn't affect this PR. However, if the bug didn't exist before and exists now, then you know it's a regression. Can you give it a try? It'll be good to know that the bug isn't limited to just Dive on my machine for some reason.

Turns out it was an issue with the connections for the search next/prev being duplicated every time the search bar was hidden/unhidden. To remedy that I make sure to disconnect the signals and slots when hiding the search bar and only connect when the search bar is shown


Unsupported symbols sometimes render nothing (instead of the expected question mark)

Most of the time milsymbol returns an upside-down question mark for symbols it does not support. However, I found at least two separate cases where nothing is rendered.
First case: For e.g. W*A*PH----***** (presumably valid symbol for weather / pressure systems / high pressure centre, but milsymbol does not support this, and I don't really expect it to), the question mark is neither filled nor stroked (and thus nothing is visible). Relevant code is in  https://github.com/spatialillusions/milsymbol/blob/master/src/symbolfunctions/icon.js#L237:
    iconColor = this.colors.iconColor[this.metadata.affiliation];
    var undefinedIcon = [
      {
        type: "path",
        stroke: false,
        fill: iconColor,

In case this.metadata.affiliation is "undefined", iconColor will be undefined. Proposed fix: default to black or perhaps this.colors.iconColor["Unknown"]so at least something gets drawn.
The second case occurs with e.g. totally invalid SIDC -------------- (and many other similar strings). The relevant code is at https://github.com/spatialillusions/milsymbol/blob/master/src/symbolfunctions/icon.js#L348
      if (icons.hasOwnProperty(genericSIDC)) {
        drawArray2.push(icons[genericSIDC]);
      } else {
        if (
          !(
            this.options.sidc.substr(4, 6) == "------" ||
            this.options.sidc.substr(4, 6) == ""
          )
        ) {
          drawArray2.push(undefinedIcon);
          this.validIcon = false;
          //console.info("Invalid icon code in SIDC: " + this.options.sidc);
        }
      }

If the latter condition is false, nothing gets pushed to drawArray2 (and thus nothing is drawn), and validIcon remains true. Proposed fix: Check later if both drawArray1 and drawArray2 are empty, and if they are, draw the question mark and set validIcon to false.

Faced with the same problem while using APP6 sidc symbol for weather
sidc -<PHONE_NUMBER>1101000000

None of the weather symbols are supported in milsymbol, you can see in the documentation what appendixes that are supported.

On 3 Feb 2021, at 14:58, Yevhenii<EMAIL_ADDRESS>wrote:

Faced with the same problem while using APP6 sidc symbol for weather
sidc -<PHONE_NUMBER>1101000000
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


settings: Improve handling of "message content in email notifications" setting.

This PR addresses the issue where the "Allow message content in message notification emails" setting in user settings was not accurately reflecting the organization-level configuration. When the setting is disabled at the organization level, the corresponding checkbox should appear both disabled and greyed out in the user settings, preventing users from enabling it. Previously, users could still check the box, although the emails did not contain message content.
Changes:

Disable and Grey Out Setting:

If the "message content in message notification emails" is disabled at the organization level, the user setting now appears both disabled and greyed out, accurately reflecting the organization policy.
Added a tooltip when hovering over the disabled checkbox: "Including message content in message notification emails is not allowed in this organization."

Refactoring in settings_config:

The object related to this setting in settings_config has been renamed to better reflect its purpose. This change has been made in a separate commit for clarity and easier review.

Additional Notes:

This PR builds upon the work in an existing open PR #27543, which has been inactive since March. I have proceeded with the original work, resolved the comments raised in that PR, and made a new PR to complete the fix. The previous PR had addressed most of the issue.
Implemented changes based on the discussion in this comment, following the suggestion to have separate commits for renaming the object in settings_config and for the functionality to disable the setting.

Fixes: #27262.

This updated description provides a comprehensive overview of the PR, including the previous work, new changes, and resolution details.

Screenshots and screen captures:

Before

After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thank you, @sahil839, for the corrections. I have updated the commits.
Thanks!

Maybe we can also show "not-allowed" cursor while hovering over the disabled setting. @alya thoughts?

Yes, let's change the cursor, thanks!

Hello @sahil839,
I'm currently seeing the 'not-allowed' cursor when trying to check or uncheck the disabled setting. This behavior is consistent with all the previous disabled settings to maintain uniformity. Are you suggesting that I should apply it to the entire line where the cursor changes to a pointer?
I've attached a screenshot and a video demonstrating the pointer and cursor behavior below. Could you please clarify exactly what you would like?
Thank you!

Screenshot

Screenshot
https://github.com/user-attachments/assets/b1a45a9e-3cf1-4da1-b211-679259447406

Ok, I thought we also have the "not-allowed" cursor on label since we allow changing the setting by clicking on the label. Seems fine for now then and we can change later for all the settings if needed.

Thank you @sahil839.

Merged, thanks @shuklamaneesh23 and @sahil839!

Also, the personal setting is not live-update after changing the organization setting, and it requires a reload. But this can be done as a follow-up.

Hello @sahil839, I want to work on a follow-up PR for this, but I'm having trouble figuring out how to tackle it. It would be very helpful if you could provide a reference to a file or anything related where similar functionality is implemented.
Thank you!😊

You would need to add code such that on receiving the event for change in organization setting, a function is called which enables or disables the personal setting based on the organization setting value. Updates based on events are handled in server_events_dispatch.js.
New feature tutorial also describes these changes.

Thank you so much @sahil839 , will file a PR and ping you ASAP.


Update to new rails validators

Travis CI won't pass until #299 is merged

:tada: Thanks for catching these- we just upgraded to Rails 5 last week, so there is still quite a bit of old API usage floating around.
Will merge once this re-started build passes.

Hmmm. Looks like Travis isn't actually creating the database tables for this run, causing some PG::UndefinedTable errors.

I'll merge master into this pr and that should fix it

It could be more graceful - but in the meantime, how about we add pulling that file in the travis script?

like this:
  - cp config/database.travis.yml config/database.yml
  - ./install_frontend.sh
  - bundle exec rake db:create db:migrate```

@Br3nda Cool! Looks good.


Docs: How to develop and deploy a model to Euclid

The docs show access to a web console but there's no indication on whether development should be done there or if it's code based.

The repository for the 'OLAP Web Console' is here:
https://github.com/EuclidOLAP/olapweb
currently this is a simple program, which is only used to display the query result returned from EuclidOLAP.


Reconnect DB after closing.

Description
If this is a feature request, explain why it should be added. Specific use-cases are best.
For bug reports, please provide as much relevant info as possible.
When Connection  closed, it unable to open again.
Steps to Reproduce the Problem
Please describe in painful detail what you did (so others can play along with you) to get to this point. This includes things like the exact command(s) you used, or the curl command you used, that sort of thing.
	users := []User{}
	for i := 0; i < 2; i++ {
		conn, err := pop.Connect("development")
		if err != nil {
			fmt.Println(err)
		}
		err = conn.All(&users)
		if err != nil {
			fmt.Println(err)
			os.Exit(1)
		}
		for _, u := range users {
			fmt.Println(u)
		}
		err = conn.Close()
		// conn.Store = nil // make Store to nil will continue the loop
		if err != nil {
			fmt.Println(err)
		}
	}

Expected Behavior
connection will recconnect after closing
Actual Behavior
unable to fetch records: sql: database is closed

Info
Should set Store = nil  in Connection's Close method.

Indeed, it could be an issue in a complex application that need to control the connection state. will fix.


Feature/019 delete location history

Ready to Review!

@mentos when you get a moment, can you please review? If all is good, feel free to merge and delete this branch.

@kristinpeterson looks good 👍


Error: Cannot find module 'index'

Hi, does any of the nodejs images have version 3.0 of the AWS Lambda NodeJS Runtime Interface Client installed, which supports ESModules?
I'm getting a Runtime.ImportModuleError using the Images in a local docker container with my Dockerfile, using public.ecr.aws/lambda/nodejs:20-preview as well as public.ecr.aws/lambda/nodejs:18.
Or do you have another idea why I get this error when sending a simple test request to http://localhost:9000/2015-03-31/functions/function/invocations?
$ docker run -p 9000:8080 -it docker-handler-html-to-pdf-lambda index.handler
07 Nov 2023 09:44:45,408 [INFO] (rapid) exec '/var/runtime/bootstrap' (cwd=/function, handler=)
07 Nov 2023 09:44:50,473 [INFO] (rapid) extensionsDisabledByLayer(/opt/disable-extensions-jwigqn8j) -> stat /opt/disable-extensions-jwigqn8j: no such file or directory
START RequestId: 47197920-38e9-4c66-b8f7-780eda4200db Version: $LATEST
07 Nov 2023 09:44:50,473 [INFO] (rapid) Configuring and starting Operator Domain
07 Nov 2023 09:44:50,473 [INFO] (rapid) Starting runtime domain
07 Nov 2023 09:44:50,473 [WARNING] (rapid) Cannot list external agents error=open /opt/extensions: no such file or directory
07 Nov 2023 09:44:50,473 [INFO] (rapid) Starting runtime without AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, AWS_SESSION_TOKEN , Expected?: false
2023-11-07T09:44:50.542Z        undefined       ERROR   Uncaught Exception      {"errorType":"Runtime.ImportModuleError","errorMessage":"Error: Cannot find module 'index'\nRequire stack:\n- /var/runtime/index.mjs","stack":["Runtime.ImportModuleError: Error: Cannot find module 'index'","Require stack:","- /var/runtime/index.mjs","    at _loadUserApp (file:///var/runtime/index.mjs:1087:17)","    at async UserFunction.js.module.exports.load (file:///var/runtime/index.mjs:1119:21)","    at async start (file:///var/runtime/index.mjs:1282:23)","    at async file:///var/runtime/index.mjs:1288:1"]}
07 Nov 2023 09:44:50,555 [WARNING] (rapid) First fatal error stored in appctx: Runtime.ExitError
07 Nov 2023 09:44:50,555 [WARNING] (rapid) Process runtime-1 exited: exit status 129
07 Nov 2023 09:44:50,555 [ERROR] (rapid) Init failed InvokeID= error=Runtime exited with error: exit status 129
07 Nov 2023 09:44:50,555 [INFO] (rapid) Starting runtime domain
07 Nov 2023 09:44:50,555 [WARNING] (rapid) Cannot list external agents error=open /opt/extensions: no such file or directory
END RequestId: 4a08940e-4662-4700-a4b5-6f1c85498500
REPORT RequestId: 4a08940e-4662-4700-a4b5-6f1c85498500  Init Duration: 0.14 ms  Duration: 82.04 ms      Billed Duration: 83 ms  Memory Size: 3008 MB    Max Memory Used: 3008 MB
^C07 Nov 2023 09:47:54,840 [INFO] (rapid) Received signal signal=interrupt
07 Nov 2023 09:47:54,840 [INFO] (rapid) Shutting down...
07 Nov 2023 09:47:54,841 [WARNING] (rapid) Reset initiated: SandboxTerminated
07 Nov 2023 09:47:54,841 [INFO] (rapid) Stopping runtime domain
07 Nov 2023 09:47:54,841 [INFO] (rapid) Waiting for runtime domain processes termination
07 Nov 2023 09:47:54,841 [INFO] (rapid) Stopping operator domain
07 Nov 2023 09:47:54,841 [INFO] (rapid) Starting runtime domain

Same here bro

@SinBirb any luck?

In my case I had ImportModuleError on newest Docker. Reverted Docker engine to v20 and it works.

In my case I was missing a Lambda-defined environment variable. ${LAMBDA_TASK_ROOT}
COPY index.js ${LAMBDA_TASK_ROOT}

Here is the official documentation.
https://docs.aws.amazon.com/lambda/latest/dg/nodejs-image.html


Sign in with Google temporarily disabled for this app

Solution title:
One sentence summary: Sign in with Google temporarily disabled for this app
Description (1 - 2 paragraphs):
When I click Mail Merge / Send Emails, and select an account, I get:
Sign in with Google temporarily disabled for this app
This app has not been verified yet by Google in order to use Google Sign In.
Design Details:
Please provide a high level design overview with a motivation or problem statement to help us vet your idea.
Once a repo owner has approved this issue, you may use our template directory to make a pull
request for you solution. You may either submit the full implementation to this repo or if you
already have the implementation in another repo, simply link it in the README.

I'm experiencing the same problem. Anything we can do, or is this script defunct for now?

Having the same issue. Is this fixable?

Same issue here please help! It's a google scrip that's not supported by google.../:


[do not merge] check adding -E flag in build script

For troubleshooting release script / Kokoro environment compatibility

FYI @lqiu96 - let's see if the kokoro-ubuntu check gets stuck on the same issue as the release script?

FYI @lqiu96 - let's see if the kokoro-ubuntu check gets stuck on the same issue as the release script?

Thanks @emmileaf. I think kokoro-ubuntu runs the this job: https://github.com/GoogleCloudPlatform/appengine-plugins-core/blob/master/kokoro/continuous.sh, right? The PR I just submitted passed that CI job so I think it's probably the additional -E flags in the release script causing the issue.

Not entirely sure why it was added in the first place or why it was only added to only the release job. I'll do a bit of digging before I just remove this flag.

Ok, nvm. I misunderstood the PR. This still passes even with the -E flag added into the continuous script. That's odd...

This still passes even with the -E flag added into the continuous script. That's odd...

Yeah this is just a sanity check to see if the issue can be reproduced on the continuous script, but seems not...could it have been a one-time/transient thing?

This still passes even with the -E flag added into the continuous script. That's odd...

Yeah this is just a sanity check to see if the issue can be reproduced on the continuous script, but seems not...could it have been a one-time/transient thing?

Tried re-running the job and it's the same issue. Stuck waiting for user input unfortunately.


[SUGGESTION] Can the editor stay open after closing the last file?

It should just start a new buffer.

I found a solution


Naya (Nav Bar Fixed)

Fixes #(issue number)

#134 fixed
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Deploy your changes to github page or netlify and mention deployment link:
Add the deployment link here.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] I have mentioned related issue number in PR description.
[ ] I have added screenshots and related demo links of my changes..
[ ] The title of my pull request is a short description of the requested changes.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

first merge my previous request then this one please
@QAZIMAAZARSHAD

https://apni-dukaan-brown.vercel.app/


Add group by each syntax

When using group by with a field that is itself a list, the table uses the entire list as the grouping key.
This is sometimes desirable, but sometimes not what is intuitively expected.
Unfortunately there is no current way to change this behavior.
Expanding the syntax to support both modes would alleviate this.
Suggestion:

group by X functions as it normally does, grouping by the entire list as a single key
group by each X would split the results so that each item in the list becomes a separate grouping key, and rows are duplicated into each individual key for display

Example
Given files:
[File A]
fieldName: foo

and
[File B]
fieldName:
- foo
- bar

Original query syntax: ... group by fieldName
Results in:
- foo          File A

- foo          File B
- bar

New syntax: ... group by each fieldName
Results in:
- foo           File A
                File B

- bar           File B

Looks like this may already be possible using this syntax combo:
table ...
...
flatten fieldName
group by fieldName
sort fieldName asc

The above syntax recommendation may or may not be useful, but it would be a nice little bit of syntactic sugar to remove the need to manually use flatten + group by.
Also side note, the sort doesn't seem to match the sort order used in the file browser.
I use unicode prefixes in certain notes.
In the file browser when sort is set to File name (A to Z) the following order is observed:

files prefixed with ✧ sort first
files prefixed with ✨ sort next
files beginning with [A-Za-z] sort next

When adding sort by asc to the above query the following order is observed in the dataview output:

files beginning with [A-Za-z] sort first
files prefixed with ✧ sort next
files prefixed with ✨ sort next

I've been looking at typical use cases and I've noticed that people's intuitive understanding of GROUP BY is as a visual operator (i.e., it groups fields under some header), rather than as the SQL-esque operator that I envisioned it as. I'm considering re-naming the operator to something else, during which I can also potentially add your suggested shorthand.
I use the JS-default locale-aware string sorting algorithm for strings, which probably doesn't do something useful with unicode characters (esp. emoji). I could swap to the Obsidian algorithm if they expose it in their API.

Yeah I used to work with SQL and I still default to the visual thinking when reasoning through these.
It may be that it shouldn't be renamed since it does have a SQL-like function to it, so those familiar with it would feel more comfortable working with it. Instead perhaps a new feature should be created that uses group by behind the scenes to build the visual grouping?
My first thought of an alternate name is collect but that is also the name of a potentially useful functional programming style method since you mentioned considering lambdas in the other issue. Same with map as in map to X colliding with potential use of map() later on.
Maybe gather into X? arrange by X? ¯\_(́ ◡◝ )_/¯
What about simply group visually by foo ? Then group by foo retains its current meaning so it can work both ways?
That seems elegant but may not be in practice.

Yeah I used to work with SQL and I still default to the visual thinking when reasoning through these.
It may be that it shouldn't be renamed since it does have a SQL-like function to it, so those familiar with it would feel more comfortable working with it. Instead perhaps a new feature should be created that uses group by behind the scenes to build the visual grouping?
My first thought of an alternate name is collect but that is also the name of a potentially useful functional programming style method since you mentioned considering lambdas in the other issue. Same with map as in map to X colliding with potential use of map() later on.
Maybe gather into X? arrange by X? ¯_(́ ◡◝ )_/¯
What about simply group visually by foo ? Then group by foo retains its current meaning so it can work both ways?
That seems elegant but may not be in practice.

I ran into the same problem of assuming it would work same as SQL.
I think it is very useful for cases where you want to summarize with sum for example, how much you have spend in each restaurant.

However it doesn't group the unique restaurant links

Gather into sounds good IMO.
Or be explicit about it:
GROUP BY DISTINCT X

@JoseEchave That looks like a separate bug - GROUP BY does group by unique keys, implying the link grouping is not working. If you group on GROUP BY "" + Restaurant AS Restaurant, does it group?

GROUP BY "" + Restaurant AS Restaurant

Hi, it groups all into one, but still don't show the cost sum

@JoseEchave Would you mind creating a bug report about this specific issue? I'm guessing it is likely due to your data format, though it could also be a genuine bug.

@JoseEchave Would you mind creating a bug report about this specific issue? I'm guessing it is likely due to your data format, though it could also be a genuine bug.
Done, opened a bug related to that issue.
#435
Thanks

Having groupBy each feature would be very beneficial if there are many parameters that the files can be grouped by.
If I have 12 (A,B,C....) parameters as 'file type'.
title: File 1
file type: [A, C]
and another note with
title: File 2
file type: [C, D]
Now when we map and groupBy 'file type'
We will get
Group (A+C)
File 1
Group (C+D)
File 2
which creates many permutations of these groups and the mapped information is impractical to use.
Instead the desired output is:
Group A
File 1
Group C
File 1
File 2
Group D
File 2
This output will make stumbling upon File 2 even if I chose to filter only for either file type 'C' or 'D'. This will also decrease the number of tables in the output, making the data more practical to use.


Deploy the project on maven central

Would you consider deploying the project on maven central ? It would be easier for our organisation if this was available on a central repository instead of just on our company nexus.
Thank's !


In some chapters it returns null

In some chapters of the anime "ACCA: 13-ku Kansatsu-ka" the api returns null, although in jkanime page the links exist.
PD: I have tried with "https://jkanime.chrismichael.now.sh/api/v1/video/acca-13-ku-kansatsu-ka/1/" link, and  I have tried with some more chapters of this anime.

Hello @Ruthder ,
Unfortunately right now I am not going to be able to continue to maintain the library as I will start working this week and time will not allow me to do the maintenance.
But yes, I checked and there are some anime that is not showing the data ..
I recommend that as soon as possible you give a fork to the repo and locally maintain it, since I will stop maintaining it as soon as I start working this week.
If you have any questions about the code and how to begin to understand it, you can let me know, and I will help you. So that you can continue to maintain it without problem.
I hope you understand!


Update LICENSE

with my sig - I'm seated in Row 44 ... guy with the Hawaiian shirt!

I'm  seated in Row 25 now

I'm seated in Row 44 ... guy with the Hawaiian shirt!

we're cutting you off, Larry! go home - you're drunk


Some merge-items in filter.csv don't work

Swedish filter.csv ;;Cicke- ;icke-;0;merge stripped off prefix again
Spanish filter.csv:  ;;Cno- ;no-;0;to rewrite adjectives with no- back together
Dutch filter.csv: ;;Cniet- ;niet-;0;to attach prefix back to the concept
However, there is no problem with this item:
Portuguese: ;;Cnão- ;não-;0;to rewrite adjectives with 'não-' back together
All these prefixes are negation prefixes.
Possibly relevant note: "icke-", "no-" and "niet-" get Literal labels during processing, "não-" only gets labels that are defined in labels.csv.

The filter works on lexreps, not merged Concepts.
Solution: add 'Join'-rules in rules.csv to reunite the parts before the filter is applied.


Conflict with reactotron-react-native library

Hi! First of all, awesome work on this library, it is amazing.
Well, I open this issue using the "Other" template because I really don't know if the problem that I found has some workaround.
As you can see in the image below, at this point in code, the library checks if the interceptor is enabled or not, and if it is already enabled, the code just returns:

So after debugging, I just check that the reactotron-react-native library was enabling that by default, just by importing the library, and that was making the Logger skip the setup and because of that do not log any request.
What I did was remove the Reactotron and put it again manually just when I want to debug other things in my application.
So, I was wondering if there's any way to work around this conflict between these two libraries and make them work together.
Thanks!

Hi, thanks for using the library and opening the issue. I've had a look through the react-native code and from what I can tell it looks like the way the XHRInterceptor is implemented you can only have one interceptor set at the same time.
If I removed the isInterceptorEnabled check, it would cause react-native-network-logger to override reactotron-react-native (or the other way round depending on which gets loaded first), which I'm not sure is what you would want either.
I'm not really sure what can be done about this - open to any suggestions.

Hi @alexbrazier, thanks for the answer. Well, I will think about it and see if I found another solution for that. So I will close this issue and if I found some workaround I let you know, ok?
Thanks again!

Hello, thank you for this very useful library
Did you find any way to be able to use it with reactotron please?

Hello, thank you for this very useful library
Did you find any way to be able to use it with reactotron please?

I think it would require a code change. Only thing I can think of is adding a flag to enableNetworkLogger to force it to be enabled, then you could manually call it from inside your app. If you did it would disable network logging on reactotron though.

I think it would require a code change. Only thing I can think of is adding a flag to enableNetworkLogger to force it to be enabled, then you could manually call it from inside your app. If you did it would disable network logging on reactotron though.

This issue should now be fixed. You'll need to start the logger with the forceEnable: true option. This will override any other network interceptors that are running.
startNetworkLogging({ forceEnable: true });


Mode based compression

Since we concluded today that mode isn't working any better than average, I guess I should rather skip merging this one ?


Add wire representation for the RPL protocol.

I'm splitting #627 into smaller PRs.
This PR adds the wire representation of the RPL protocol.

bors r+


Achievements - Are we only processing level.sessions achievements?

Don't show achievements for level.sessions:
https://github.com/codecombat/codecombat/blob/master/app/views/core/RootView.coffee#L39
Only POST achievements for level.sessions:
https://github.com/codecombat/codecombat/blob/master/server/achievements/earned_achievement_handler.coffee#L47

Awarding these achievements (which happen via middleware) were disabled during HoC because of memory leak issues: a9265bce42a070fe77be50a92c3f4083eeafc8e8
Re-enabling started here: 9272ceb23ec839647a9806ce77ba2c016a84984a


Clean repository after setup + hma + true range

hey could you push this to https://github.com/beatzxbt/mm-toolbox/tree/optimizations rather than main?
i have some changes to make to your code (types stuff) still and ill package this into a new version :)


Fixing issue caused by stale intent extra

Fix for issue
Presence  of an already used external intent extra causes Branch to
return +non_branch_link in init session response
Fix added to mark the intent as used after reading the link_click_id. This will prevent reading an already used intent extra.
@aaustin @EvangelosG @jdee

Test Steps
To reproduce on Android:

Run the TestBed app on a device.
Click the "Share Link" button in TestBed and mail yourself the link.
Open the link from the GMail app, launching TestBed. This will show automatic deep link routing.
Go back to the Main activity by clicking  back from the auto deep link activity
Put the TestBed app in the background.
Put the TestBed app in the foreground again, without opening a link.
Examine the logcat output. You will see something like:
09-27 14:49:56.978 30352-30534/io.branch.branchandroiddemo I/BranchSDK: returned {"session_id":"441705382440755472","identity_id":"441704212867452041","link":"https://bnctestbed.test-app.link?%24identity_id=441704212867452041","data":"{"+non_branch_link":"https://bnctestbed.test-app.link/O4OemuOaNG\",\"+clicked_branch_link\":false,\"+is_first_session\":false}","device_fingerprint_id":"441704212838089199"}
Note the +non_branch_link field. Also note that this is a Branch link, and that no link was opened. The caller should receive
{ "+clicked_branch_link": false, "+is_first_session": false }

Thanks for the quick patch 👍 !

Jimmy - can you test this Branch and see if it resolves the issue?

:+1: This looks good when tested with RN.

Ignore this, just adding the link for JIRA tracking: https://branch.atlassian.net/browse/INTENG-3788


Feature

I want a Feature Please add Pin in Notification Feature

If you don't mean to pin the upcoming reminder in notifications, you can set up a reminder to be notified after a minute, and it will stay in your notifications until you remove it yourself.


pkg_resources will fail when packaging removes LegacyVersion

See https://github.com/pypa/pip/issues/9540 for background here.
Basically, pip vendors pkg_resources, but uses it with pip's own vendored copy of packaging, not with the one setuptools vendors. In the latest release of packaging, legacy versions are deprecated and trigger a DeprecationWarning. Python's test suite runs pip with warnings converted to errors, and this resulted in a failure, which I traced back to the following code in pkg_resources:
    def _by_version(name):
        """
        Parse each component of the filename
        """
        name, ext = os.path.splitext(name)
        parts = itertools.chain(name.split('-'), [ext])
        return [packaging.version.parse(part) for part in parts]

(This is an internal function of _by_version_descending). The code is the same in the latest version of setuptools.
Note that this code parses every part of the filename, not checking whether the parse works and so relying on the fact that packaging will currently parse any old garbage, returning a LegacyVersion. This is what triggers the issue here.
Obviously, setuptools won't hit an issue here until it upgrades its vendored copy of packaging to one that finally removes LegacyVersion (or one that deprecates, if the user is running with warnings treated as errors). So it's not an immediate issue here (even though it is for pip).
The following code fixes this problem, but it's not exactly attractive, and I'm not sure whether there's any implication to sometimes returning a string and sometimes a Version. Also, I haven't checked anywhere else to see if this same pattern is repeated anywhere else in setuptools. Maybe it's useful, nevertheless.
    def _by_version(name):
        """
        Parse each component of the filename
        """
        name, ext = os.path.splitext(name)
        parts = itertools.chain(name.split('-'), [ext])
        def safe_parse(s):
            try:
                return packaging.version.parse(s)
            except (packaging.version.InvalidVersion, DeprecationWarning):
                return s
        return [safe_parse(part) for part in parts]

See also #2466, #2497, and #2478. To be sure, Setuptools is uninterested in maintaining support for non-standard versions.

Sorry I didn't spot the issues you linked. Apologies for that. This can be closed as a duplicate.


[Task]: Extract Java Beam symbols for Playground

What needs to happen?
Parse symbols so they can be suggested by the editor autocompletion.
See https://github.com/apache/beam/issues/23304 as a model.
Issue Priority
Priority: 3
Issue Component
Component: beam-playground

.take-issue

.take-issue


[FLINK-14720] Mark ExecutionVertex#deployToSlot with @VisibleForTesting annotation

What is the purpose of the change
This pull request marks ExecutionVertex#deployToSlot with @VisibleForTesting annotation
Brief change log

Mark ExecutionVertex#deployToSlot with @VisibleForTesting annotation

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @GJL


How to disable the plugin in another world?

How to disable the plugin in another world?
I could not find a config that could disable the plugin in another world

anyone?

https://github.com/bloodmc/GriefDefender/wiki/Claim-Management#how-to-disable-claiminggd-functionality


bump pipeline for build verifier fix

for https://github.com/openshift/jenkins-plugin/issues/153

[merge]


RFC: Documentation refactor

We've struggled coming to grips with Goose.
Partly because we're unfamiliar with it's code base.
Partly because we're unfamiliar with the (Locust) model it uses.
Partly because documentation is scattered across different blogs posts that refer to different Goose versions.
Partly because the documentation is structured as single file.
Looking at the possible documentation it seems the time may have come to restructure the documentation?
If this is agreed we'd be happy to contribute, but we don't want to sink time into anything that is then declined and we have to maintain as a fork.
Proposals (scope):

[ ] Restructure (not rewrite) the documentation.  Includes gathering blog content. Proposal: markdown
[ ] Documentation generated as static site. Proposal: mdbook
[ ] Static site hosting. Proposal: GitHub pages (migrate to https://goose.rs once that is setup)
[ ] Deploy static site using GitHub workflow. Proposal: actions-mdbook
[ ] Locate book source files in the repository. Proposal: book/src

My intent/goal is to turn the README and other documentation blogs into an online book, as are common for many Rust projects. In the mid-term I'm aiming for two books, one with that's the definitive reference, and one with useful recipes/snippets, structured as a cookbook. The books will still be built from the code (necessary to keep it in sync with the code), but likely will be hosted at an upcoming https://goose.rs/ website.

Okay. Is there tooling you had in mind? Or a reference site you'd like to emulate that we can replicate?

I believe the Rust book is using mdBook:

https://rust-lang.github.io/mdBook/
https://docs.rs/mdbook/0.4.10/mdbook/index.html
https://github.com/rust-lang/mdBook

Does that proposed setup look acceptable?

Yes. This is the direction I was planning to go when I had time for this. If you can review how a major Rust project (ie the language itself) is integrating this with their project, that would be ideal. I can try and make time to explore this next week as well, if that helps.

If you can review how a major Rust project (ie the language itself) is integrating this with their project, that would be ideal.

Yup, we've looked there and what we've outlined seems to be what rustaceans do/use

I can try and make time to explore this next week as well, if that helps.

We'll have a crack and see how we go.  We'll keep this issue open of issues that arise, e.g. book outline/skeleton.


added logout function to handleAuthUi.js

👷 Deploy Preview for cg-eleventy-blog processing.

Name
Link

🔨 Latest commit
13544a938a1c37d3d17e3113cd3aae2cfee16341

🔍 Latest deploy log
https://app.netlify.com/sites/cg-eleventy-blog/deploys/65cc1a91da3d7a0008938cd5


Getting this to work in VS 2015

I have tried cloning this into VS 2015, but there is an issue with the database version.  I tried changing the v11.0 reference to MSSQLLocalDB but that did not fix it.  Any suggestions?

Zero information in the Readme file is indicative of a somewhat cavalier attitude to people trying to work their way through this topic. It would be to everybody's benefit to add a couple of paragraphs explaining how to set up the solution/project so that it works on the machines of those trying to follow the course.
That would improve he ratings as well.

There is no database file to upgrade

Which project are you trying to run? I'll try to repro/get a fix

Adam
Thanks for responding - I'm in the UK, very unexpectedly we are having a General Election (the whole shebang) next month. For reasons I don't understand, I am the coordinator for one of the parties. So you will understand when I say I don't remember which project, but I will be getting back to this, once the madness is over and I've recovered.
I am genuinely grateful.

Thanks for getting back to us. I see that this project is now 2+ years old, so I am sorry for bothering you about it.
In Christopher’s Demo:  MVCMusicStore, I see that Entity framework is in the resource list, but there is nothing in the App_Data folder. Is there supposed to be a database file? SQLserver or SQL Express or something? Or is the exercise to rebuild a SQL database from the Model Classes? There isn’t a readme with it.
I will go back through the videos and figure it out. I just saw that other people were wondering the same thing, so I just added myself to the list.
Thanks for your time,
Tom
Tom Karasch
(678)596-7224
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.tomtekdev.com/ www.tomtekdev.com
From: Adam Tuliper<EMAIL_ADDRESS>Sent: Tuesday, May 9, 2017 1:45 PM
To: MicrosoftLearning/EntityFramework<EMAIL_ADDRESS>Cc: Tom Karasch<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [MicrosoftLearning/EntityFramework] Getting this to work in VS 2015 (#3)
Which project are you trying to run? I'll try to repro/get a fix
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftLearning/EntityFramework/issues/3#issuecomment-300246203 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQID0ioOFx3N0wxMVzsBMYPtdrUD9Zqbks5r4KYXgaJpZM4IrnEs .  https://github.com/notifications/beacon/AQID0o2DU2RWyq6fYUAlem8oEKHM-ELHks5r4KYXgaJpZM4IrnEs.gif

The tech should still work 2 years later though so its no worries :)
The issue here is likely that the connection string refers to v11. To
update it try this first in the web.config connectionString to use a later
version of localdb. The syntax now we just use (localdb)\mssqllocaldb
(which is also the same way we can open up all localdbs in Sql Management
Studio by connecting to that name)
change (LocalDB)\v11.0 to (LocalDB)\MSSQLLocalDB
<add name="DefaultConnection" connectionString="Data
Source=(LocalDb)\v11.0;AttachDbFilename=|DataDirectory|\aspnet-
MusicStore-20150128102003.mdf;Initial
Catalog=aspnet-MusicStore-20150128102003;Integrated
Security=True"
providerName="System.Data.SqlClient" />
to:
<add name="DefaultConnection" connectionString="Data
Source=(LocalDB)\MSSQLLocalDB;AttachDbFilename=|DataDirectory|\aspnet-
MusicStore-20150128102003.mdf;Initial
Catalog=aspnet-MusicStore-20150128102003;Integrated
Security=True"
providerName="System.Data.SqlClient" />
Let me know if that works.
On Tue, May 9, 2017 at 11:59 AM, Tom Karasch<EMAIL_ADDRESS>wrote:

Thanks for getting back to us. I see that this project is now 2+ years
old, so I am sorry for bothering you about it.
In Christopher’s Demo: MVCMusicStore, I see that Entity framework is in
the resource list, but there is nothing in the App_Data folder. Is there
supposed to be a database file? SQLserver or SQL Express or something? Or
is the exercise to rebuild a SQL database from the Model Classes? There
isn’t a readme with it.
I will go back through the videos and figure it out. I just saw that other
people were wondering the same thing, so I just added myself to the list.
Thanks for your time,
Tom
Tom Karasch
(678)596-7224 <(678)%20596-7224>
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.tomtekdev.com/ www.tomtekdev.com
From: Adam Tuliper<EMAIL_ADDRESS>Sent: Tuesday, May 9, 2017 1:45 PM
To: MicrosoftLearning/EntityFramework<EMAIL_ADDRESS>Cc: Tom Karasch<EMAIL_ADDRESS>Comment <
<EMAIL_ADDRESS>Subject: Re: [MicrosoftLearning/EntityFramework] Getting this to work in
VS 2015 (#3)
Which project are you trying to run? I'll try to repro/get a fix
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub <https://github.com/
MicrosoftLearning/EntityFramework/issues/3#issuecomment-300246203> , or
mute the thread <https://github.com/notifications/unsubscribe-auth/
AQID0ioOFx3N0wxMVzsBMYPtdrUD9Zqbks5r4KYXgaJpZM4IrnEs> . <
https://github.com/notifications/beacon/AQID0o2DU2RWyq6fYUAlem8oEKHM-
ELHks5r4KYXgaJpZM4IrnEs.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftLearning/EntityFramework/issues/3#issuecomment-300268215,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABqPDTk76BkWtrnJSNZDjxhWeshXiWWJks5r4LeTgaJpZM4IrnEs
.


static lightning address

Do you plan to add a static lightning address to your wallet?
This should now be possible, and this is very important for youtubers and other promoters to receive donations. Payment processors like coingate already accept LN so it's time for that.
Thank you.

Why not use ln-url? This is exactly what it was made for, and Phoenix already supports it.

Phoenix supports LNURL-withdraw, user is looking for LNURL-pay.

What I meant is that it shouldn't be Phoenix' role to generate such invoices on-the-fly and be always online. If a user wants to receive donations, it seems to me that this is best addressed by having the user run a node (because Phoenix can't be always online to receive donations, you don't know when those donations will arrive - even with a static lightning address). So you'd need a node that generates invoices on the fly with lnurl, and donators can pay those with Phoenix.

Right, I thought @KarolStrzelec meant: being able to pay those donations invoices.

Do you think that there is no chance for a static LN address? Maybe phoenix should block funds in the wallet until the recipient logs in? Until someone logs in, the transfer could be canceled. Do you think that a large list of waiting transfers is a big problem?

Maybe phoenix should block funds in the wallet until the recipient logs in? Until someone logs in, the transfer could be canceled. Do you think that a large list of waiting transfers is a big problem?

I think the best we can offer is a notification to a Phoenix user if offline, and holding an incoming payment for a max of 30 seconds. Otherwise this would be hurting the network too much (while a payment is blocked, liquidity from all routing nodes in the payment route is blocked) and considered as spam.
I don't believe this would work for donations that can arrive at any time of the day if user action is necessary (clicking on a notification). I really believe that for your use-case of accepting donations, you need something that runs 24/7, hence some kind of lightning node (it may be a light node, not a full routing node) not just a wallet. This is one of the main distinctions between Bitcoin and Lightning: in Lightning the recipient needs to be online.

There are two issues:

be able to generate at anytime a LN invoice from a static link (LNURL withdraw)
be able to receive at anytime a LN payment

The at anytime part is critical and makes this feature seemingly impossible: waking the app can be done, but you also have to start the node, and the node is locked behind the PIN. That's why you need user interaction. Still that would be a great feature so we'll have to think about it.

There are two issues:
1. be able to generate **at anytime** a LN invoice from a static link (LNURL withdraw)

2. be able to receive **at anytime** a LN payment

The at anytime part is critical and makes this feature seemingly impossible: waking the app can be done, but you also have to start the node, and the node is locked behind the PIN. That's why you need user interaction. Still that would be a great feature so we'll have to think about it.

Currently, if I want someone to send me a tip on LN I send him my link to tippin.me, if I collect 10,000 sat or more, I transfer it to my wallet. (a lot of people do that) Its not big deal. I think that such a function in phoenix can be very profitable for you and if someone does not take money for 1 year, you will send them on-chain to his address.

What I meant is that it shouldn't be Phoenix' role to generate such invoices on-the-fly and be always online. If a user wants to receive donations, it seems to me that this is best addressed by having the user run a node (because Phoenix can't be always online to receive donations, you don't know when those donations will arrive - even with a static lightning address). So you'd need a node that generates invoices on the fly with lnurl, and donators can pay those with Phoenix.

I think this change with Phoenix version with Google FCM notification. For now is possible to receive lightning payments without need any action from user and this is great. I love this function. Zebedee Wallet implemented static lightning with LNURL-Pay as you can see here: https://medium.com/zebedee-engineering/announcing-zbd-gg-bc0c682d9bda
Will be great to have this on Phoenix too and make sense now with Google FCM that open Phoenix automatically to receive the payment.
Obs. Even is my phone be offline, wallets could have something like this: https://github.com/ACINQ/phoenix/issues/117

waking the app can be done, but you also have to start the node, and the node is locked behind the PIN

Can't you simply not require a PIN in situations where someone is trying to pay you? Why require the PIN in that case? For situations where people don't want to be able to passively receive, there could be a settings option for that, right? I don't quite follow the barrier to waking up the app and receiving on the fly.

this is the protocol reference:
https://github.com/fiatjaf/lnurl-rfc/blob/master/lnurl-pay.md
pretty easy to spin up some servers that answer the lnurl-pay requests with wallet responses.   only downside is that those serves can route payments anywhere, so an attacker can steal future funds.
ideally it should be run on a youtuber's own hardware/node.
here's a DIY howto for anyone in this thread that wants to do it themselves:
https://github.com/andrerfneves/lightning-address/blob/master/DIY.md

The gen3 update that includes splicing and active channel management would lend itself to ad-hoc payments, as you'd expect from a lightning address payment scheme.
The lack of a Lightning address results in having to recommend a custodial wallet as well as a non-custodial wallet. This complicates matters for end users, who may (sadly) just choose to use the custodial wallet.
Having a system that can receive payments from a mobile device only when the device is on or has an internet connection presents privacy issues. A potential attacker could discover information about a user's whereabouts by checking if the user can receive a payment or performing timing analysis on the handshake (possibly at the HTLC level).
I suggest that a lightning address is a very important feature, but it would need to be implemented asynchronously for privacy and to be always-on. In other words, payments would need to be custodially cached until the user is able to receive them. A user an opt out of such a system by simply not publishing their lightning address.

static noninteractive lightning addresses must involve a server.  But it can involve in non-Trusted server or a public server of some kind to hold on to the partially signed transactions until someone comes on to take custody of them.  without covenant technology a person will have to come online fairly regularly or else risk losing funds.

Since the cost in May went to 300 sat/Vb, placing straight on-chain could cause big user losses.
We can get very close to perfect by using an opt-in third-party payment caching service. This could even be a node a user runs themselves. The API for this process would be simple and a minor extension of existing APIs.
From the UX perspective, this could be as easy as entering the Lightning address into the Phoenix wallet. The user can even enter multiple Lightning addresses.
User enters an LN address ->
Phoenix sends a registration API call to that LN address including an API URL to fetch an invoice to pay that Phoenix user->
Caching server carries out it's own validation.
If the API endpoint accepts the registration, Phoenix client may send a message to the LNURL endpoint whenever the user is ready to receive payment, or if the phoenix user nudges. For security, the caching server may limit payments to be forwarded only to a specific list of node pubkeys to limit risk of DNS poisoning attacks.
Phoenix remains completely non-custodial. User has an always-on lightning address experience.

I'd like to use Phoenix, but I'm hesitating because it lacks a persistent address to receive payments. A lightning address would be a great addition, if the technical can be worked out.


Change defaultURL behavior to consider empty path

This resolves #16

Codecov Report

Merging #17 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #17   +/-   ##
=======================================
  Coverage   66.66%   66.66%
=======================================
  Files           5        5
  Lines          84       84
=======================================
  Hits           56       56
  Misses         28       28

Impacted Files
Coverage Δ

Sources/MoyaSugar/SugarTargetType.swift
71.42% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Everything looks good to me 👍! I'd be glad if this can be merged

@lowip, this PR has been shipped with 0.4.1 release and will be available in CocoaPods soon :tada:


Validate 6.3 file against official guidance to double-check for changes

Official docs are here: http://www-01.ibm.com/support/knowledgecenter/SSHS8R_6.3.0/com.ibm.worklight.dev.doc/devref/r_integrating_with_source_contro.html

Not going to be done. Closing.


Image metadata doesn't match up with saved filenames

I am trying to build a tool to view a saved blog (such as in the event it is no longer available).
However the meta data in  images.txt and texts.txt seems to be insufficient / doesn't match up with the files that TumblThree actually downloads.
The first problem is that the array seems to be empty:
	"photos":	[

	],

So I instead tried to rewrite the tumblr image URLs that are in the post_html field, however this unfortunately doesn't reliably work since the links don't always match the filesize suffix that was downloaded.
Futhermore the problem is repeated for texts.txt - I am not sure why but some image posts are being classed as text posts, where the body contains various images on Tumblr, which whilst also automatically downloaded, there is no way to match them up.
In short I was expecting the photos array to be populated with the name of the file as saved by TumblThree, otherwise it is very difficult to actually make use of the archive!

Hello,
your goal sound interesting.
Currently we don't save additionally the "original" links as found in the posts, only the eventually converted ones used for downloading. Maybe that should be changed in a future version.
We know about the fact that some posts are classified as text posts. It's not an error in the app. They are delivered like that and can show html content. So some tools create for any reasons text/html posts containing images. Although we cannot get everything right with such posts, at least maybe the found image links could be handled differently.
We'll have a look at it.

A short remark on the photos array and how we get back the data. For the API crawler this list is only filled if the post type is photo and the post contains a photo set (multiple images posted at once). For single-photo posts it's always empty and only the photo-url-x entries are filled. For the SVC crawler the photos array is always filled.
Anyway, the downloaded files should be listed in the metadata now.


Target stun condition

Is it possible to check if a target is stunned? Something like target.state.stun, like it was in PE.
I tryed to make this:
{'215661', 'target.state.stun&player.holypower>=5&target.range<8'},
This is a retri pali's Justicar's Vengeance. Does not work. Works only if i check for Hummer of Justice debuff.

try target.state(stun)

Nope, does not work.

Seems it work on enUS clients only, is it possible?

Does NeP check for english debuff names for "player.state(stun)"? If it so, where i can find those debuff names to edit it?

@Orodriana https://github.com/MrTheSoulz/NerdPack/tree/master/locale, find the file that contains your language and translate from EnUS


Handling pyplot pathological cases

plt.plot()  fails with a
TraitError: Missing scale for data attribute x.
plt.plot(y=some_y) yields the same error, although we do handle plt.plot(some_y)
Intuitively, shouldn't plt.plot() create a Lines with empty x and y, and shouldn't plt.plot(y=some_y) be the same as plt.plot(some_y) ?

[2021-01-21] Triage: closing this as bqplot.pyplot does not accept keyword arguments by design.

[2021-01-21] Triage: closing this as bqplot.pyplot does not accept keyword arguments by design.


Update readme to clarify how to use in packages

This took me a while to figure out, added it so it might help someone in the future.

Thanks for the PR


Strange crowd annotations

Hi,
I am experiencing some issues on images with crowd annotations. Sometimes polygons seem to be connected that should not be connected. Below I have created a minimum working example. It shows the code to display the incorrect annotations and I also show the corresponding image. Whereas the round shapes are correct, the entire V shape in the top right should not be there. Is that a known issue?
imgId =<PHONE_NUMBER>63;
cocoApi = CocoApi('instances_train2014.json');
annIds = cocoApi.getAnnIds('imgIds', imgId, 'iscrowd', []);
anns = cocoApi.loadAnns(annIds);
curSegs = anns(end).segmentation;
M = double(MaskApi.decode(curSegs));
imagesc(M)

That seems to be annotated by an adversarial worker and didn't get filtered out at the quality check stage.

Haha. Okay, good to hear that it wasn't a mistake on my side :+1:


More Hardware Devices

Is there a way to access these devices:

bluetooth
usb
speakers
headphones
printers

Is there a way to access these devices:

As Linus Tordvalds said, "Do you pine for the nice days of minix-1.1, when men were men and wrote their own device drivers?" ;-)
By the way, from your list, speakers and headphones are the (almost?) the same from a driver/computer perspective, and probably the most simple ones to develop because on their most simple behaviour it's just a matter of read and write on some specific ports, specially for the internal speaker/buzzer (it was a college practice for me, but almost no computer have it these days).
The other ones are probably too much complex to write a driver from scratch, specially the USB protocol. Printers were easy to program when they used a parallel port (the same thing: just poke some port address), but today all printers are USB and no computer have a parallel port these days. Maybe you could have some luck with a serial port and program its UART...
Said that, there are bindings for Node.js for libusb library, but if you are into the mood of write a pure Javascript USB library (at least on a basic level, that's it, support a TTY-to-USB adapter so it can be connected to an Arduino board) then go for it :-)


hud: add draw time debug logging when enabled

In addition to displaying in the hud, this will now also print the draw times with DEBUG level. This should make it easier to share performance numbers when needed.

Are you aware of the csv file export in nodegl ?
This seems redundant but I don't know what your use case is...

That is news to me 😅 But I was using the HUD with app rendering, where app actions were relevant to different render timing. I think there is still a use in having draw logging intermingled with other logging contexts


Hello. Does the program now have a continuous dialogue with chatgpt?

Hello. Does the program now have a continuous dialogue with chatgpt?

I don't get your question.
You are asking if this application remembers previous questions like ChatGPT?

Yes. Is there any implementation of this function? Brother.

No.
OpenAI api doesn't (at least for now) support conversation history(passing conversation thread id).
To achieve that you need to send previous conversation text to OpenAI api which will cost you more.
https://github.com/ioanmo226/chatgpt-web-application/blob/2727646dc194c3298ed85e6795743f3a7847309a/index.js#L38


columns_comments in s3.to_csv will not work as expected

Describe the bug
The comment here https://github.com/awslabs/aws-data-wrangler/blob/master/awswrangler/s3.py#L684
said that dataset=False will not be able to use with these arguments: database, table, description, parameters, and columns_comments. But columns_comments is not a if else statement in this part.
To Reproduce
When a user called  to_csv with dataset=False and columns_comment={"foo": "bar"}, this method can run normally without any exception.

Hey @bryanyang0528 thank you one more time!
P.S.
I've just added black and isort checks in the statical check that runs on Github Actions (commit above).
So, make sure that you run this commands before any code:
cd testing/
./validations.sh

@igorborgest  No problem, thank you!

Changes released on version 1.4.0
Thanks!


Jenkins 2 support

As Jenkins 2 is due this month, I'm wondering if the Atlassian theme will be compatible or not? Has anyone tested it yet?

Hi,
Thanks for the question! I have tested it and it is compatible.

Sweet, thanks a lot!


Revisiting the package dependencies for the next major release

Currently the stable version depends on:

the Intl extension
the Json extension
PSR3
PSR16

while the PHP extension are still needed we should revisit the dependencies over the PSR interfaces and see if we can remove them or move them to suggests.

In https://github.com/jeremykendall/php-domain-parser/blob/feature/v6
the PSR3 and PSR6 dependencies are still present but not required as much

This issue is depending on how we resolve #273


save the environment if dependencies don't change

Hi, I'm wondering if you can consider saving the environment if the project dependencies don't change.
Package dependencies are supposed to be rather stable, so building the same environment for so many times is wasting time & computing resourse.
It's easy to detect whether the dependencies change or not, simply by checking the status of setup.py or readthedocs.yml.
Thanks for your wonderful website!

The pip dependencies are cached, so the next time they aren't downloaded from internet, but reused from the cache. Also, to save disk space I have understood that rtd clean these resources regularly, so if you don't build your docs in a while, there isn't a cache and the dependencies are downloaded from internet.

so it's a tradeoff between computing resource and storage resource, and rtd think the storage resource is more valuable, right?

I think is right to do it, I mean if you don't build your docs in some time there isn't a good reason to keep all that space in the servers, but if you build your docs every 5 minutes or so, the cache is used.

Hope my explanation makes sense, we actually are searching for ways of reducing the disk space of the builders.


Add login command

Mask and unmask password.
Refract login command method into smaller and more modularised methods.
Add test cases to modularised method.

Pull Request Test Coverage Report for Build 228

25 of 42 (59.52%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 86.404%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/logic/commands/LoginCommand.java
18
20
90.0%

src/main/java/seedu/address/ui/CommandBox.java
7
22
31.82%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/commons/util/FileUtil.java
2
95.24%

Totals

Change from base Build 223:
-0.4%

Covered Lines:
1843

Relevant Lines:
2133

💛  - Coveralls


added daccess check to ws handler

Although no users have been denied access from the site with a daccess restriction, I feel this will be useful for when that day comes.

*cough* pinkiepie *cough*


Raise an error when something wrong occurs

Hi ! Here is some missing feature I identified :
Expected Behaviour
When loading any data in redshift using pandas_redshift, it would be great if an error is raised so that the python script stops when something wrong occurs.
Actual Behaviour
With the current version, any fail during the import will log a warning message but will not terminate.
Steps to reproduce (If possible)
I see at least two examples:

loading data with the right structure, with append=True in a non existing table
loading data where we mispecified the column type

Hi Alexis,
I have released v1.1.1 with a fix for these issues, please run:
pip install pandas-redshift --upgrade