getting ERROR in undefined

I've created a sample repo to show the error:
https://github.com/deezahyn/webpack-htmlplugin-test/
npm install && webpack

Actually 1.4. should be backwards compatible.
The error was a missing new Error() and was fixed by #45.
We should release a 1.4.1 version and roll this fix out.

The fix for now is to use the inject option or to use the files object instead of the assets object in your template: https://github.com/ampedandwired/html-webpack-plugin/blob/master/default_index.html#L15

uh! I been struggling with this, and finally glad to see this is not caused by my code :-)
http://stackoverflow.com/questions/30423710/webpack-verbose-error-message

It is related to this! https://github.com/webpack/webpack/pull/1146

Will be fixed as soon as @ampedandwired released 1.5.

thanks for the update @jantimon

@deezahyn FYI, 1.5 is out now.


[feat] Drag and drop external web links.

Describe the problem
I'm trying to make an asset management app, and want to download images for user if they drag the link of image from their browser. But the cursor turns into a banned icon when I'm dragging the link into app window. Also, the events are not fired.
https://github.com/user-attachments/assets/dfba0b8b-5f6e-497c-b38e-c3a837bc4a19
Describe the solution you'd like
Add events like tauri://link-drop that listens such event.
Alternatives considered
No response
Additional context
No response

In the meantime, if you don't need the tauri file drop events (because you need the full file paths) you can disable enableDragDrop in the window config and use the webview/browser native drag-drop events.

@FabianLars  But it seems that the native webview events doesn't support drag and drop image links from other pages.

oh okay, ignore me then x)


User Voice Issues to Migrate

User Voice Request
Status

1. Vim-style Keybindings
vscode/issues/3600

2. PHP IntelliSense
extensions exist

3. Auto Indent, Code Format, and Beautify Source Code
vscode/issues/4039

4. Unity Integration
extensions exist

5. Minimap
vscode/issues/4865

6. Haskell
extensions exist

7. Auto close / rename tags like in VS
extensions exist

8. Add file icons in Tree View (Sidebar)
vscode/issues/211

9. ActionScript 3 Support
extensions exist

10. Haxe Support
extensions exist

11. Integrate with TFS
extensions exist

12. Wider Python Support
extensions exist

13. Open Multiple Folders at once
vscode/issues/394

14. Support the D programming language
extension candidate

15. MeteorJS Support
extensions exist

16. FTP Editing
extensions exist

17. HTML editing within .js / .ts code (e.g. in es6 template strings)
https://github.com/Microsoft/vscode/issues/130 https://github.com/Microsoft/vscode/issues/2000

18. Support Rust
extensions exist

19. Path/folder IntelliSense
https://github.com/Microsoft/vscode/issues/17331

20. Ruby on Rails - like html.erb, more ruby support
extension candidate

21. Split both vertical and horizontal
vscode/issues/5911

22. Support for Mercurial
extension candidate

23. Visual Studio Code Portable
vscode/issues/12401

24. Add ability to prettify code
extensions exist

25. Auto save temporary files
vscode/issues/101

26. Run / Preview HTML files
extensions exist

27. Enable more Git commands
extensions exist

28. Auto Imports in TypeScript
vscode/issues/2635

29. Graphical Git Merging like in VS
vscode/issues/5770

30. Scala Support
extensions exist

31. Add Elixir Support
extensions exist

32. SVN Support
extensions exist

33. Save all files with ctrl+shift+s like in VS
vscode/issues/11433

34. Better Bracket Highlighting
vscode/issues/12402

35. Support for ReactJS
extensions exist

36. Function List View
vscode/issues/5605

37. Aurelia Support and Integration
extensions exist

38. Drag 'n Drop Selections
vscode/issues/1046

39. Lucee + CFScript (CFML) Support
extensions exist

40. Integrated REST Client
extensions exist

41. Faster Start Time
ongoing priority

42. Detachable Split View
vscode/issues/367

43. Chrome OS Support
vscode/issues/1031

44. sass, css, and less Format Code
extensions exist

45. Remote Debugging
feature exists

46. Julia Language Support
extensions exist

47. Add Language Support for Groovy
extension candidate

48. Django
extension candidate

49. Add Support for Dart
extensions exist

50. Live HTML, CSS Support
vscode/issues/12080

-- Sorry for the English, I used Google Translator --
@waderyan if you can include in your list:
51. Autocomplete suggestion base on word completion. I think it's similar with #7552.

For Auto close / rename tags like in VS, extensions exist: Auto Close Tag  and Auto Rename Tag

Also Run / Preview HTML files has 2 really good extensions; I've tried both and I didn't complain, so I wouldn't spend time there either.

Cool. Thank you for the feedback.

VSCode won't start on LINUX if you are using a Linux xrdp session (Debian based), can someone reproduce this?

Hey @jtlivio thanks for your comment, however this isn't a great place. Please see these guidelines and open a new issue.

@waderyan, I realize after post, Thanks.

I request that can you include us of html snippets and css class completion in *.php file rather than changing file extensions every time like in sublime or atom

@hassansw please open an issue to describe your request. Pease see these guidelines.

says scala extension exists.

Thanks to author Iliya, but it just supports keyword highlighting and far from being used as scala extension. Supporting language like scala is a hard thing, since it has a complex type system compared to other languages. Even jetbrains has some issues on scala support. I don't now to whom we should ask for scala support, maybe we can team up.

I would really like some extension that can make code to use windows bash libraries.
like i want code to compile the python code using ubuntu libraries that are installed in windows bash and debug the code using windows bash
So python developers that are using python in ubuntu can switch to Windows 10 easily and use windows bash.
@waderyan

Closing as these have all been successfully migrated.


Return value of dispatch

@vslinko
What should redux.dispatch() return? Currently the code is
    function dispatch(action) {
      return typeof action === 'function' ?
        action(dispatch, state) :
        dispatchSync(action);
    }

but I don't remember the circumstances when the return value of the callback form can be useful.
Because the callback form may schedule additional dispatches, it seems that the calling code still doesn't have enough information. Can we just return whatever action was passed?
    function dispatch(action) {
      if (typeof action === 'function') {
        action(dispatch, state);
      } else {
        dispatchSync(action);
      }
      return action;
    }

?

My usage example
export function runTransition(url) {
  return async perform => {
    const route = routing
      .filter(({route}) => route === url)
      .shift() || notFound

    const transition = transitions[route.transition]
    const componentKey = route.component

    const {title, status, ...props} = await perform(transition())

    perform(changePage(componentKey, props))

    return {
      url, title, status, componentKey, props
    }
  }
}

export function navigateTo(url) {
  return async perform => {
    progressIndicator(async () => {
      const {title} = await perform(runTransition(url))

      document.title = title
      history.pushState({url}, title, url)
    })
  }
}

I have runTransition action creator that fetching all data. This AC called on server side.
And I have navigateTo AC that calling runTransition and touching history API. This AC called on client side.

Server side usage example:

export default async function renderPage(url) {
  const dispatcher = createDispatcher()

  const {title, status} = await dispatcher.perform(runTransition(url))

  const atom = dispatcher.getAtom()

  const pageContent = React.renderToString(
    <Application dispatcher={dispatcher} />
  )

  return {
    status,
    body: appTemplate(title, pageContent, atom)
  }
}

Thank you for the examples! Let's leave this open for now, I'll come back to it later..

Returning a promise is the uber example. In all the Flux code I've ever written, I've only ever needed the return value for that reason.

I'm not sure if the dispatch should return anything at all, the whole idea around the dispatcher is to do "fire and forget" and handle data that follows that action as app state, whenever it is received.

I'm in favor of the the base dispatch() method return nothing. For those cases where you do need it to return a promise, that is easily implemented at the middleware level (refer to https://github.com/gaearon/redux/pull/63):
function promiseMiddleware(next) {
  return action =>
    action && typeof action.then === 'function'
      ? action.then(next)
      : next(action);
}

I also think the dispatcher should just "fire and forget"

But how to implement action creators from my example if dispatch doesn't return anything?

I don't know exactly what those transition() functions are doing and why you need to dispatch them. If runTransition is not being called as an AC, and only via navigateTo and renderPage then you can inject it the dispatch (or perform) method and make it a function that returns something and calls that dispatch as well. You have full control of that on your side, so no need to have the dispatcher return anything. Same thing can apply to those transition() functions

@vslinko Couldn't you just get it from the atom? You have access to it right there a few lines down.

Middleware can always return things, but the base dispatch method would not.

I don't know exactly what those transition() functions

Transition is action creator that fetches all data for page.
Transition should be action creator because it could change application state.
Transition should return value for runTransition
Simplified example:
return function homepageTransition() {
  return async perform => {
    const apiResponse = (await homepageApiRequest())

    perform(mergeResources(collectResources(apiResponse)))

    return {
      status,
      title,
      queryParams,
      apiResponse,
    }
  }
}

runTransition is another action creator that mathes transition by url.
runTransition should be action creator because it could change application state.
runTransition should return value for navigateTo
Actial example:
export function runTransition(url) {
  return async perform => {
    const route = routing
      .map(route => {
        return {...route, match: matchRoutePattern(route.route, url)}
      })
      .filter(({match}) => !!match)
      .shift()

    if (!route) {
      return perform(runNotFoundTransition(url))
    }

    const transition = transitions[route.transition]
    const componentKey = route.component

    const {title, status, ...props} = await perform(transition(route.match || {}))

    if (status === 404) {
      return perform(runNotFoundTransition(url))
    }

    if (status === 500) {
      return perform(runErrorTransition(url, props))
    }

    perform(changePage(componentKey, props))

    return {
      url, title, status
    }
  }
}

navigateTo is action creator too.
navigateTo should be action creator because it should perform another action creator.
Simplified example:
export function navigateTo(url) {
  return async perform => {
    const {title} = await perform(runTransition(url))

    document.title = title
    history.pushState({url}, title, url)
  }
}

@vslinko Couldn't you just get it from the atom? You have access to it right there a few lines down.

Only on server side. But how to get it in navigateTo?

I think you can solve your problem if you manually call the other actionCreators by manually passing the perform function, which you already have in your root actionCreator. Then you can do:
instead of this:
const {title} = await perform(runTransition(url))

You do this:
const {title} = await runTransition(url)(perform)

And the same inside the runTransition action creator:
Instead of this:
const {title, status, ...props} = await perform(transition(route.match || {}))

You do this:
const {title, status, ...props} = await transition(route.match || {})(perform)

So conceptually you only call the dispatcher with one action creator from outside, and that action creator dispatches several actions, only the logic is spread in other functions that receive the dispatch function.
Another analysis that should be done is if you really need the transitions and the runTransition functions to actually be action creators (that is, actually called from the outside app directly), or just functions that can have the dispatch function injected.

I think you can solve your problem if you manually call the other actionCreators by manually passing the perform function, which you already have in your root actionCreator. Then you can do:

Yes, I know that.
Moreover I'm created PR that adds return to dispatcher because I don't want to pass perform and state into action creator by myself.
My original concerns described here https://github.com/gaearon/redux/issues/48#issuecomment-109578338

Yes, you're right about that, you don't want to pass state just because. Anyway, I still think you shouldn't need to use the returned values from functions. For example in the navigateTo AC, you need the title and the url to set the document.title and the historyApi. If you managed those values in your app state (i.e. handled by a Store), you can then just model those inside your React application as components at the root level like <DocumentTitle /> and <CurrentUrl />, that render nothing to the dom but modify something. Another option is to have an extra subscribe on the dispatcher and listen to state changes and directly set the title and url in that listener.
The server side part is trickier I think, since it's hard to know when there are no remaining actions to be dispatched if we cannot await on anything.

You can await a promise if you use middleware. Just because the dispatcher doesn't return anything doesn't mean your middleware can't. See my example above https://github.com/gaearon/redux/issues/61#issuecomment-110393712

Makes sense. But now I'm wondering if I didn't want to use promises but still wanted to know when the dispatch ends on the server side in order to send the request, how would I do that if i have ACs using the async function way?

@leoasis if you don't use promises and async/await you could subscribe to dispatcher and wait for store changes. Your AC could set flag completed in store. That a reason I want to return value from AC.
@acdlite I don't mind about dispatcher "fire and forget" if middleware could change that behaviour. As I see middleware already in master so I'm satisfied.

Is this solved my middleware in 0.10? Can we close?

I didn't updated to 0.10 yet, so I don't know.
I think you could close this issue. I can (re)open issue If I'll have some questions.


Add an Icon in the view menu section

Add an icon to the "View Page" in the view section in the menu of the dashbaord.html.
You can add two icons(to the left and right) but make sure to include a redirect icon because we will be redirecting the user to the "View Page" html file in a new tab.
This is a really simple task. Enjoy it.

can you assigne me this task ?

Hey, I want to contribute to this issue, Please assign me so that I can start working on it.

Hello, Im interested in working on this. Also, I have experience for contributing an icon to a different repository.
Link - https://github.com/feenotifier/apps
Please assign me this issue.

hey I added Icon in view page

that's a perfect example of what we're looking for. Nice one. I will assign
you 👍
On Sat, Oct 1, 2022, 12:17 AKASH @.***> wrote:

[image: image]
https://user-images.githubusercontent.com/67678310/193406801-80d0d823-a00a-4f02-a4cc-0300f5b4e62f.png
hey I added Icon in view page
—
Reply to this email directly, view it on GitHub
https://github.com/KwickerHub/frontend/issues/24#issuecomment-1264333351,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKWIBLBTOGLQNSGXW645Y3WBAMUTANCNFSM6AAAAAAQ2LL7HE
.
You are receiving this because you authored the thread.Message ID:
@.***>

thank you

Hey,
anything else is required?

Hey, anything else is required?

Yes the software is far from finish. Just keep checking for more issues the page.


sendUnsubscribe is not work

Hi.
I found sendUnsubscribe does not work on RabbmitMQ，
the return is true but still subscribe the topic.

Strange. If it returns true then a message acknowledging the unsubscribe has been received from the server. I'd try to understand what mqtt message the server actually gets and how it responds to it. This looks more related to the server than the client.


Currency code 'X' is not supported in 0.10

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
After updating to the newest docker image, the server is no longer reacting, only keeps outputting Currency code 'E' is not supported on the command line.
Steps to Reproduce
$ docker compose -f ~/docker-scripts/inventory/docker-compose.yml run --rm inventree-server invoke export-records --overwrite --include-permissions --delete-temp --filename /home/inventree/data/data.json
$ docker compose -f ~/docker-scripts/inventory/docker-compose.yml down
$  docker compose -f ~/docker-scripts/inventory/docker-compose.yml pull
$ docker compose -f ~/docker-scripts/inventory/docker-compose.yml run --rm inventree-server invoke update
$  docker compose -f ~/docker-scripts/inventory/docker-compose.yml up -d
$  docker compose logs --follow

At the end of invoke update, this happens:
Requirement already satisfied: webencodings==0.5.1 in /root/.local/lib/python3.9/site-packages (from -r requirements.txt (line 263)) (0.5.1)
Requirement already satisfied: xlrd==2.0.1 in /root/.local/lib/python3.9/site-packages (from -r requirements.txt (line 269)) (2.0.1)
Requirement already satisfied: xlwt==1.3.0 in /root/.local/lib/python3.9/site-packages (from -r requirements.txt (line 271)) (1.3.0)
Requirement already satisfied: zipp==3.11.0 in /root/.local/lib/python3.9/site-packages (from -r requirements.txt (line 273)) (3.11.0)
Requirement already satisfied: zopfli==0.2.2 in /root/.local/lib/python3.9/site-packages (from -r requirements.txt (line 275)) (0.2.2)
Requirement already satisfied: setuptools in /usr/local/lib/python3.9/site-packages (from django-money==2.1.1->-r requirements.txt (line 103)) (65.6.3)
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
Backing up InvenTree database...
Currency code 'E' is not supported

Expected behaviour
Start normally..
Deployment Method

[X] Docker
[ ] Bare metal

Version Information
docker image ls
REPOSITORY                        TAG             IMAGE ID       CREATED         SIZE
inventree/inventree               stable          358c662a4948   6 days ago      991MB

Relevant log output
inventree-server  | Currency code 'E' is not supported
inventree-worker  | WARNING: Translation files could not be compiled:
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-worker  | InvenTree translation coverage: 23%
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-worker  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported
inventree-server  | Currency code 'E' is not supported

Relevant parts of:
config.yaml (working on previous versions).
# Base currency code
base_currency: EUR

# List of currencies supported by default.
# Add other currencies here to allow use in InvenTree
currencies:
  - EUR
  - GBP
  - USD
  - SEK
  - NOK
  - CHF

prod-config.env:
# Other settings
INVENTREE_BASE_CURRENCY=EUR
INVENTREE_CURRENCIES=EUR,SEK,USD,NOK

Ok, I can not reproduce on a fresh stable install; can you try with a fresh instance? On what kind of host are you running - maybe a raspberry pi or similar arm architecture?

At a guess, I'd say that it is due to setting INVENTREE_CURRENCIES environment variable:
INVENTREE_CURRENCIES=EUR,SEK,USD,NOK

The error message Currency code 'E' is not supported indicates that the code is running a for loop over the string and not a list. I am not currently at my dev computer so unable to test this right away.

It seems to have nothing to do with the update, tried on a fresh install without data, same result.
However, commenting out the list INVENTREE_CURRENCIES made the error go away.
I think you're right about the loop, changing the list can change the error to Refer to 'U' instead.
No special architecture, on Ubuntu VMs, two different ones.
On 13 February 2023 23:57:57 CET, Oliver @.***> wrote:

At a guess, I'd say that it is due to setting INVENTREE_CURRENCIES environment variable:
INVENTREE_CURRENCIES=EUR,SEK,USD,NOK

The error message Currency code 'E' is not supported indicates that the code is running a for loop over the string and not a list. I am not currently at my dev computer so unable to test this right away.
--
Reply to this email directly or view it on GitHub:
https://github.com/inventree/InvenTree/issues/4338#issuecomment-1428818530
You are receiving this because you were mentioned.
Message ID: @.***>

Work-around is to comment the line out, as mentioned above. This allows server to start. The config.yaml is not changed, so doesn't seem to have any effect directly.

@miggland thanks for reporting this - has been fixed!

Thanks for the quick fix!


STPO-2713: fix syntax

Just a simple syntax fixing.
Sorry for my carelessness.

@cjestel Please help merge this at your earliest convenience.

LGTM


build-arg: allow setting additional omego args

usage: docker build -t test . \
    --build-arg OMERO_VERSION=OMERO-build
    --build-arg OMEGO_ADDITIONAL_ARGS=--ci=http://devspace/

Could possibly help with gh-4

Does this work? I thought devspace defaults to https with a self-signed cert?

It worked for me. Also have devspace doing the same regularly now.

See @kennethgillen's comment under https://github.com/openmicroscopy/omero-web-docker/pull/16#issuecomment-351410892

I have tried using the IP of the devspace created in openstack (both http and https)
It failed with the following
TASK [openmicroscopy.omero-server : omero server | install omero] **************
fatal: [localhost]: FAILED! => {"changed": true, "cmd": ["/opt/omero/omego/bin/omego", "install", "--release", "OMERO-build", "--sym", "OMERO.server", "--ice", "3.6", "--no-start", "--ignoreconfig", "-qq", "--ci=http://<IP_ADDRESS>", "--dbhost", "localhost", "--dbuser",

i tried to specify the port, no luck

Using same ci-server as for omero-web-docker:
  docker build -t omero-server-docker-test .  --build-arg OMERO_VERSION=OMERO-build --build-arg OMEGO_ADDITIONAL_ARGS=--ci=https://ci-server
Built version matches next-ci as of https://ci-server/job/OMERO-server/31/console
LS27172:omero-server-docker kenny$ docker exec -it server-test bash
bash-4.2$ /opt/omero/server/OMERO.server/bin/omero admin diagnostics | grep 5
WARNING:omero.util.UpgradeCheck:UPGRADE AVAILABLE:Please upgrade to 5.4.1. See http://downloads.openmicroscopy.org/latest/omero for the latest version.
OMERO Diagnostics (admin) 5.4.0-128-6b28ede-ice36-b31

@jburel : can you show the command you ran as well as the output of docker ps on your host? (and/or add my key)

@joshmoore sorry for the confusion I  have just realised  that in the command I ran I had a typo in the omero version
I have tested both http and https and it works

see https://github.com/openmicroscopy/omero-web-docker/pull/16#issuecomment-353102533


getExternalProfile takes a long time

Checked for duplicates
Yes - I've already checked
Is this a regression?
No - This is a new bug
Version
aerie-ui-2.11.2
Describe the bug
Feel free to put "bug" in quotes, but I don't think this is a feature request...
the getExternalProfile query takes several orders of magnitude longer than all other queries when loading a plan page. this is extra weird in the context of europa clipper because we don't use external profiles.

I saw this same result in some recent load testing:

Reproduction
open network tab, load the page, sort by time on network requests table
Logs
No response
System Info
M2 Mac, Chrome

Severity
Minor

This query likely suffers from the same join troubles we've encountered before with simulation datasets. It may be worth splitting into two queries - one to look up the plan_datasets and their corresponding dataset_id, and another to fetch things from the partitioned profile_segment table.
See this query, which includes dataset_id in the where clause for the nested profile_segment query: https://github.com/NASA-AMMOS/aerie-ui/blob/78d20f3f3766ef048b77a51209c5bb7aed071450/src/utilities/gql.ts#L1556

This issue should get moved to the UI repo, as it pertains to the UI's query for External Events


Proposal: $page->excerpt() on CollectionItem

If you guys (Matt, Keith, other maintainers) think this is a good idea, I will write it myself and make a PR.
I have this helper in my config, but I think it is a good SSG feature.
Here's a rough example of what it would look like:
In CollectionItem.php:
<?php

use Illuminate\Support\Str;

public function excerpt($limit = 250, $end = '...')
{
    $newlines = ["\r", "\n", "\r\n"];
    return Str::limit(str_replace($newlines, ' ', strip_tags($this->_content)), $limit, $end);
}

If one of the below actions seems unnecessary I can just exclude it from the code.

strips tags
removes line breaks so that it outputs content as a straight line in blade/html (this is specifically useful for outputting part of the content in meta tags)
limits characters, configurable (I looked up average meta description lengths and decided 250 is a good default)
appends an ellipsis, also configurable

Example of real-world use (SEO and social sharing):
<!-- code is specific to posts/markdown pages -->
<meta name="description" content="{{ $page->excerpt() }}">
<meta property="og:description" content="{{ $page->excerpt() }}">
<meta name="twitter:description" content="{{ $page->excerpt() }}">

It can also be useful in a blog index where posts are looped and content excerpts are shown.

Wouldn't this be the case for a helper method?

I do have this as a helper method in my config. But I think that this is a feature that fits well in a SSG.
 So more people would want to add stuff like this into their sites.
<meta name="description" content="{{ $page->excerpt() }}">
<meta property="og:description" content="{{ $page->excerpt() }}">
<meta name="twitter:description" content="{{ $page->excerpt() }}">

Having this built in helper would be helpful.

I think this would be better suited as a helper method in config.php, where it can be more easily user-configurable (preserving certain tags, for instance).
That said, it might be worthwhile adding this to the default config.php stub as a sample helper method, since it's a pretty good demonstration of how helper methods work.
If we do that, I would make the function respect word boundaries, either using preg_replace('/\s+?(\S+)?$/' ... ) or strtok(wordwrap($str, length, "...\n"), "\n")

Merge it as a sample helper in the config stub and don't cut off the string mid-word, got it.


Multi Hub support, include deviceId in message and set msg.topic

enhancements to support multiple hubs with specifying different post endpoints per config
adding the deviceId to the sendEvent message
setting msg.topic to the device name or to "hubitat-mode" for mode updates.

Ok, I will close this pull request, revert the other changes back to the original and submit a pull request with just the last two changes. I can submit another PR with the other changes once the first one is closed again.


fix: Add fallback for cygwin ps

Description
Add fallback for cygwin version of ps command. This should fix the ps: unknown option -- o error displayed currently when completing kill **<tab>.

Fixes: https://github.com/junegunn/fzf/issues/3924
Notes
I tried to test the change in zsh completion and the error is indeed gone however I noticed that it doesn't show any content in fzf. I dig a bit and it seems like all of the completions that use _fzf_complete have the same issue on my machine, this includes export **<tab>, unalias **<tab>, unset **<tab>, etc.
A cause I can think of is that mkfifo is not working the same in windows and that seems to be the mayor difference I see between the bash and zsh completions.

Thanks!


from mpi4py import MPI ImportError: DLL load failed

I operated according to your instructions, and all the packages are complete, but I don't know how to report this error! Give advice or comments please!

Did you use Windows environment? Currently the support of OpenAI for Windows is weak, everything works smoothly in macOS or directly ubuntu.

I operated according to your instructions, and all the packages are complete, but I don't know how to report this error! Give advice or comments please!

Did you use Windows environment? Currently the support of OpenAI for Windows is weak, everything works smoothly in macOS or directly ubuntu.

https://stackoverflow.com/questions/14004457/error-loading-mpi-dll-in-mpi4py
openai/large-scale-curiosity#15

I tried ubuntu on Windows but still failed, only works in macOS.
ERROR: Command errored out with exit status 1: /home/zhuhx/FinRL-Library/venv/bin/python -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-8llehrxw/mpi4py/setup.py'"'"'; file='"'"'/tmp/pip-install-8llehrxw/mpi4py/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' install --record /tmp/pip-record-72buf3c_/install-record.txt --single-version-externally-managed --compile --install-headers /home/zhuhx/FinRL-Library/venv/include/site/python3.6/mpi4py Check the logs for full command output.

The detailed instructions for installation can be found here: https://ai4finance.medium.com/finrl-for-quantitative-finance-install-and-setup-tutorial-for-beginners-1db80ad39159


model_name missed if not specify and there is "/" at the end of model_dir after model convert

Describe the bug

model_name missed if not specify and there is "/" at the end of model_dir after model convert

>>> model_dir = "/Users/xx/.local/share/instructlab/checkpoints/instructlab-granite-7b-lab-mlx-q/<<<<<"
>>> model_dir = os.path.expandvars(os.path.expanduser(model_dir))
>>> mlx_q_suffix = "-mlx-q"
>>> model_name = model_dir.split("/")[-1]
>>> model_name = model_name.replace(mlx_q_suffix, "")
>>> model_name
''

output dir is:
-trained

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots

Device Info (please complete the following information):

Hardware Specs: [e.g. Apple M2 Pro Chip, 16 GB Memory, etc.]
OS Version: [e.g. Mac OS 14.4.1, Fedora Linux 40]
Python Version: [output of python --version]
InstructLab Version: [output of ilab system info]

Additional context

2799


When adding alarm data, the notification mode selects the email by de…

…fault, and one click less

Is there any special purpose in setting it as mail by default?

A form for users to fill out, needs to simplify the user's operation as much as possible;
Therefore, it is best to have a default selected value for selection items, of course, the default value must be the value that most users will choose; in the two options of email and dingding, I think most people may choose email;
With default values,
For most users: just fill in the email, eliminating the choice of this step;
For a small number of users: the step has not changed at all, or is it the same as before, click dingding and enter dingding secret key.
Summary: This optimized change will improve the efficiency of form filling for most users; for a few users, it will keep the steps the same as before without adding any operational steps.


Add lift for other collections/optional

maybe nice to have liftCollector ?

maybe nice to have liftCollector ?

Hmm, I couldn't image it well.
Does it mean to lift Arguments1Validator<A1, X> to Arguments1Validator<Collector<WOWOWO, ?, A1>, Collecotr<WOWOWO, ?, X> ?

Ah, I meant traverseCollector you had previously proposed.

Ah, I see traverseCollector.
mmm, It's troubling.... If indexing is to be done properly, it cannot be done CONCURRENT, and the implementation may not be practical in terms of performance.

Got it. Let's include liftCollection/Set/Optional in the 0.8.0 release.

I'll try my best this weekend.
One point, about the method placement, in the current PR, I put it in both the instance method and the static method, which do you think is better?

all as static methods of ArgumentValidators as well as the existing liftList, but not providing instance methods of ValueValidator.

define them both as static methods of ArgumentValidators and as instance methods of ValueValidator.

define them only in the instance methods of ValueValidator, leaving the existing ArgumentValidators.liftList as @deprecated.

@gakuzzzz Thanks for your support.
Ideally these methods look better to be in ValueValidator. However, Arguments1Validator has split / combine method that does not exist in ValueValidator, and basically it is assumed that the user will use Arguments1Validator instead of ValueValidator. So I think lift methods exist in ArgumentsValidators (option 1).

Option 2 is also available.

I've got it all set up, so please review it @making.
Currently, I am choosing option 2.
I haven't been able to run the .sh locally, so it would be helpful if you could take a look at that as well.

Oops, I forgot to add the tests. I'll add them.

done

@gakuzzzz thanks for your work! much appreciated!
I found that there are circular references between Arguments1Validator and ArgumentsValidators. I want to avoid circular references as much as possible.
Do changes in https://github.com/making/yavi/commit/786ab09a63a602b6acad62d32832a2e8b6f6986b make sense?
I tried to keep the upper and lower bounds, but I couldn't fix the compilation error.

I'll merge this branch with https://github.com/making/yavi/pull/137/commits/865f36eb00e87bb3742d176a6159f8b2a646d5dd

Ops, sorry, I have been caught your comment just now!
I'll check https://github.com/making/yavi/commit/786ab09a63a602b6acad62d32832a2e8b6f6986b later!

mmm...
This change has made the interface less flexible.
There is no reason to keep the static methods since they are no longer as expressive as the instance methods.
If this is the case, it may be better to remove all static methods and use only instance methods...
@making

There is no reason to keep the static methods since they are no longer as expressive as the instance methods.

ArgumentsValidators#lift**** provides the conversion from ValueValidator to Arguments1Validator (upcast) while ApplicativeValidator#lift**** returns ValueValidator (downcast).
But Arguments1Validator.from(valueValidator.lift****()) should be fine. Maybe deprecate ArgumentsValidators#lift**** in the next release.

🤔

ArgumentsValidators#lift**** provides the conversion from ValueValidator to Arguments1Validator (upcast) while ApplicativeValidator#lift**** returns ValueValidator (downcast).

Isn't "from  ValueValidator to Arguments1Validator" a downcast?
since Arguments1Validator extends ValueValidator

Arguments1Validator.from(valueValidator.lift****()) should be fine. Maybe deprecate ArgumentsValidators#lift**** in the next release.

sounds good.
I am more concerned about the fact that we can no longer create a ValueValidator<Iterable<T>, List<SuperClass>> from a ValueValidator<T, SubClass>.
Similarly, we can no longer create a ValueValidator<Iterable<SubClass>, List<A>> from a ValueValidator<SuperClass, A>.

@gakuzzzz please take a look at https://github.com/making/yavi/pull/149

Isn't "from ValueValidator to Arguments1Validator" a downcast?

yeah, it's opposite 😅 (fixed)

Oh, I see #149


'istioctl version' should make "istiod" distinct from "pilot"

If users do istioctl version -o json on a system with IstioD, it appears as Pilot.
Normally users don't use -o json.  In that case the component name is suppressed in the output ... UNLESS the control plane is mismatched (for example, Istio contributors may run a private build for the ingress, and then the regular output names each version).
The "component" reported by istioctl version should be "istiod".
(I realize I am creating this item today and 1.5 is due for today.  I have marked this as 1.6 but hope to have it for 1.5.0 or 1.5.1)

Closing this because no one seems to care.


No method to rename a table in the migration

Hello, as of 0.12.1 Ecto contains no method to rename a table in the migration without going into raw SQL call.

Someone is welcome to send a pull request but otherwise you need to emit raw SQL statements indeed. I will close because it is not our priority right now. :)


Test Python 3.12.

Description

Test on Python 3.12. Also limit the number of jobs that run on limited macOS and windows runners.
Motivation and Context

Ensure that dashboard works with Python 3.12.
Previously, the actions workflow ran the full matrix of tests on all 3 operating systems. This redundant testing adds significantly to the total time needed to test a push as GitHub Actions provides only a limited number of free macOS and Windows runners.

This PR limits macos and windows to 1 job each on the newest version of python and tests the full matrix on Linux.
Checklist:

[x] I am familiar with the Contributing Guidelines.
[x] I agree with the terms of the Contributor Agreement.
[x] The changes introduced by this pull request are covered by existing or newly introduced tests.
[ ] The package documentation and framework documentation in signac-docs are up to date with these changes.
[ ] I have updated the changelog and added any related issue and pull request numbers for future reference.

Correction: Windows runners are not limited (https://docs.github.com/en/actions/learn-github-actions/usage-limits-billing-and-administration). However, Windows runners are slower than the Linux runners, so Linux is preferred for the full matrix.


Add a spring boot skeleton.

Hereen, I know you're on vacation but if you have any thoughts on your return? Here's the backend interview prompt https://github.com/getconvey/eng-interviews/tree/master/take-home-exercises/backend under the sentence_search subdirectory if you want to see what we've been asking people to build based on the template.

This point in the prompt seems a little clear to me:

If there is more than one sentence containing the word, they should be ordered by count value decreasing with ties broken by sentence text.

Broken using the sentence text in what way?
https://github.com/getconvey/eng-interviews/tree/master/take-home-exercises/backend/sentence_search

I like that the spring-boot-starter-test is in the pom so paired with this from the instructions:

You should test the code enough to know it's correct and are encouraged to include tests in your submission.

I hope it counts against the candidate if they do not submit at least some unit tests as testing is not only crucial, but design in testing elucidates some important thought processes.


installation fail

@EDkinGG
@SadatRafi11
It seems like the issue is originating from scikit-learn. As there is no setup.py file, I am assuming this wheel file uses sklearn as package name while installing via pip.
Now this should be scikit-learn.
I have forked this repo and updated the dependencies here bnltk-revamped.
Now you should be able to install this package by running
pip3 install bnltk-revamped==0.1.0 this command
TLDR: run pip3 install bnltk-revamped==0.1.0

@SadatRafi11 @EDkinGG Thanks for reaching out. The package has been unmaintained for a long time. The error you were getting is about the old version of sklearn. They changed the package name from sklearn to scikit-learn.
Now we have released bnltk v0.7.7 which should resolve your issue.


Tooltip examples with html content

It would be nice to have simple example(s) showing how to:

define an inline html content for a tooltip
use a function to define html content for a tooltip

Retargetting at v4 since v3 is in maintenance mode.

No specific plans to do this for tooltips—if you need custom HTML, I suggest using the popover instead <3.

Wait, misread that. We could add some examples for that since we already support it. My bad!

This was added with #16941—coming in Alpha 5.


[BUG] Fix printing variable issue

Describe the bug
When printing variables (specifically functions) it returns invalid values and/or errors. This includes input values as well.
To Reproduce
Steps to reproduce the behavior:

Go to the installation tab of this project's README, and follow the steps.
Create a zsq file, and copy and paste the contents in the tests directory of the file index.zsq (tests/index.zsq)
Run the program/language.
See error.

Expected behavior
The variable value is to show up, or the function value is to properly show up.
Desktop (please complete the following information):

OS: ChromeOS
Browser: Chrome
Version: 96

Found the solution (https://github.com/JBYT27/zsq/commit/c9a0c321b01c885b84116193de91bbbf2d6e85dc)


Commander pro to 6 argb fans

I've been looking through your repository and it's really something. I was wondering if you have any advice on creating a controller that takes the data sent by a channel on the Commander pro, and converts it so that you'd be able to control multiple 3 pin argb devices. This would be very helpful when using signalRGB.
Kind regards,
A fellow embedded fanatic

What do you mean by 3 pin devices? Normal addressable led stripes? Have a look at https://github.com/Legion2/CorsairLightingProtocol/blob/dev/examples%2FMultipleFans%2FMultipleFans.ino. it controls multiple led strips per channel.

I read the code and I couldn't really figure out how the avr board is supposed to recieve data from the lighting node (or in my case the commander pro). With 3 pin argb I meant the goto argb connection.

What do you mean by 3 pin devices? Normal addressable led stripes?do you know how to use the FastLED library and how to wire the components?


Add OpeningHours, Category, Location(Repo, Service, Tests)

Add repository, service and test services for OpeningHours, Category, Location entities

https://github.com/SoftServe-Social-Projects/GreenCity/tree/%2342-OpeningHours-Category-Location-repo_service_tests


Redis does not properly start (config file is not read)

Describe the bug
When running app-redis with KraftKit it hangs, and no connection is possible. This is very likely due because the command line argument (the configuration file) is not properly passed. If Redis were to read the configuration file, it would provide an ASCII art logo.
Steps to reproduce
Run app-redis using KraftKit. Sample run command:
kraft run --target redis-qemu-x86_64-initrd --initrd ./fs0 -p 6379:6379 -- /redis.conf

The output is frozen, as if the configuration file is not read:
en1: Interface is up
Powered by
o.   .o       _ _               __ _
Oo   Oo  ___ (_) | __ __  __ _ ' _) :_
oO   oO ' _ `| | |/ /  _)' _` | |_|  _)
oOo oOO| | | | |   (| | | (_) |  _) :_
 OoOoO ._, ._:_:_,\_._,  .__,_:_, \___)
                  Atlas 0.13.1~67ccbbf6
1:C 18 Aug 2015 03:52:31.020 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
1:C 18 Aug 2015 03:52:31.024 # Redis version=7.0.11, bits=64, commit=c5ee3442, modified=1, pid=1, just started
1:C 18 Aug 2015 03:52:31.029 # Configuration loaded
[    0.161082] ERR:  [libposix_process] <deprecated.c @  348> Ignore updating resource 7: cur = 10032, max = 10032
1:M 18 Aug 2015 03:52:31.066 * Increased maximum number of open files to 10032 (it was originally set to 1024).
1:M 18 Aug 2015 03:52:31.071 * monotonic clock: POSIX clock_gettime

Expected behavior
The output should be full. See the difference between running KraftKit vs qemu-system-x86_64 directly:
$ kraft run --initrd ./fs0 -p 6379:6379 .unikraft/build/redis-qemu-x86_64-initrd_qemu-x86_64 -- /redis.conf
[...]
en1: Interface is up
Powered by
o.   .o       _ _               __ _
Oo   Oo  ___ (_) | __ __  __ _ ' _) :_
oO   oO ' _ `| | |/ /  _)' _` | |_|  _)
oOo oOO| | | | |   (| | | (_) |  _) :_
 OoOoO ._, ._:_:_,\_._,  .__,_:_, \___)
                  Atlas 0.13.1~67ccbbf6
1:C 18 Aug 2015 03:52:31.020 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
1:C 18 Aug 2015 03:52:31.024 # Redis version=7.0.11, bits=64, commit=c5ee3442, modified=1, pid=1, just started
1:C 18 Aug 2015 03:52:31.029 # Configuration loaded
[    0.161082] ERR:  [libposix_process] <deprecated.c @  348> Ignore updating resource 7: cur = 10032, max = 10032
1:M 18 Aug 2015 03:52:31.066 * Increased maximum number of open files to 10032 (it was originally set to 1024).
1:M 18 Aug 2015 03:52:31.071 * monotonic clock: POSIX clock_gettime

$ cat run-qemu-x86_64-initrd.sh
#!/bin/sh

kernel_image="build/redis_qemu-x86_64"

if test $# -eq 1; then
    kernel_image="$1"
fi

cd fs0
find -depth -print | tac | bsdcpio -o --format newc > ../fs0.cpio
cd ..

sudo ip link set dev virbr0 down 2> /dev/null
sudo ip link del dev virbr0 2> /dev/null
sudo ip link add dev virbr0 type bridge
sudo ip address add <IP_ADDRESS>/24 dev virbr0
sudo ip link set dev virbr0 up

sudo qemu-system-x86_64 \
    -enable-kvm \
    -cpu host \
    -kernel "$kernel_image" -nographic \
    -netdev bridge,id=en0,br=virbr0 -device virtio-net-pci,netdev=en0 \
    -append "netdev.ipv4_addr=<IP_ADDRESS> netdev.ipv4_gw_addr=<IP_ADDRESS> netdev.ipv4_subnet_mask=<IP_ADDRESS> -- /redis.conf" \
    -initrd fs0.cpio

$ ./run-qemu-x86_64-initrd.sh .unikraft/build/redis-qemu-x86_64-initrd_qemu-x86_64
[...]
Booting from ROM..1: Set IPv4 address <IP_ADDRESS> mask <IP_ADDRESS> gw <IP_ADDRESS>
en1: Added
en1: Interface is up
Powered by
o.   .o       _ _               __ _
Oo   Oo  ___ (_) | __ __  __ _ ' _) :_
oO   oO ' _ `| | |/ /  _)' _` | |_|  _)
oOo oOO| | | | |   (| | | (_) |  _) :_
 OoOoO ._, ._:_:_,\_._,  .__,_:_, \___)
                  Atlas 0.13.1~67ccbbf6
1:C 18 Aug 2015 03:24:30.028 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
1:C 18 Aug 2015 03:24:30.033 # Redis version=7.0.11, bits=64, commit=c5ee3442, modified=1, pid=1, just started
1:C 18 Aug 2015 03:24:30.040 # Configuration loaded
[    0.168579] ERR:  [libposix_process] <deprecated.c @  348> Ignore updating resource 7: cur = 10032, max = 10032
1:M 18 Aug 2015 03:24:30.076 * Increased maximum number of open files to 10032 (it was originally set to 1024).
1:M 18 Aug 2015 03:24:30.084 * monotonic clock: POSIX clock_gettime
                _._
           _.-``__ ''-._
      _.-``    `.  `_.  ''-._           Redis 7.0.11 (c5ee3442/1) 64 bit
  .-`` .-```.  ```\/    _.,_ ''-._
 (    '      ,       .-`  | `,    )     Running in standalone mode
 |`-._`-...-` __...-.``-._|'` _.-'|     Port: 6379
 |    `-._   `._    /     _.-'    |     PID: 1
  `-._    `-._  `-./  _.-'    _.-'
 |`-._`-._    `-.__.-'    _.-'_.-'|
 |    `-._`-._        _.-'_.-'    |           https://redis.io
  `-._    `-._`-.__.-'_.-'    _.-'
 |`-._`-._    `-.__.-'    _.-'_.-'|
 |    `-._`-._        _.-'_.-'    |
  `-._    `-._`-.__.-'_.-'    _.-'
      `-._    `-.__.-'    _.-'
          `-._        _.-'
              `-.__.-'

1:M 18 Aug 2015 03:24:30.223 # WARNING: The TCP backlog setting of 511 cannot be enforced because SOMAXCONN is set to the lower value of 128.
1:M 18 Aug 2015 03:24:30.238 # Server initialized
1:M 18 Aug 2015 03:24:30.258 * Ready to accept connections

Which architectures were you using or does this bug affect?
x86_64
Which operating system were you using or does this bug affect?
linux/debian
Relevant log output
en1: Interface is up
Powered by
o.   .o       _ _               __ _
Oo   Oo  ___ (_) | __ __  __ _ ' _) :_
oO   oO ' _ `| | |/ /  _)' _` | |_|  _)
oOo oOO| | | | |   (| | | (_) |  _) :_
 OoOoO ._, ._:_:_,\_._,  .__,_:_, \___)
                  Atlas 0.13.1~67ccbbf6
1:C 18 Aug 2015 03:52:31.020 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
1:C 18 Aug 2015 03:52:31.024 # Redis version=7.0.11, bits=64, commit=c5ee3442, modified=1, pid=1, just started
1:C 18 Aug 2015 03:52:31.029 # Configuration loaded
[    0.161082] ERR:  [libposix_process] <deprecated.c @  348> Ignore updating resource 7: cur = 10032, max = 10032
1:M 18 Aug 2015 03:52:31.066 * Increased maximum number of open files to 10032 (it was originally set to 1024).
1:M 18 Aug 2015 03:52:31.071 * monotonic clock: POSIX clock_gettime

Thanks for the report.
It seems that the config does get loaded but redis doesn't spin up.
If you were to change kraft args to:
kraft run --target redis-qemu-x86_64-initrd --initrd ./fs0 -p 6379:6379 -- /redis.doesntexist
Redis will complain about not being able to find the file:
en1: Interface is up
Powered by
o.   .o       _ _               __ _
Oo   Oo  ___ (_) | __ __  __ _ ' _) :_
oO   oO ' _ `| | |/ /  _)' _` | |_|  _)
oOo oOO| | | | |   (| | | (_) |  _) :_
 OoOoO ._, ._:_:_,\_._,  .__,_:_, \___)
                   Atlas 0.13.1~513cee9
1:C 14 Sep 2015 22:56:46.005 # Fatal error, can't open config file '/redis.doesntexist': No such file or directory

I'll be diving deeper into this issue to identify the root cause.

This seems to be a networking configuration issue. The redis.conf file makes it bind to <IP_ADDRESS>
kraft run works if redis is made to listen on all interfaces via config change to:
bind <IP_ADDRESS>
port 6379
tcp-backlog 511
timeout 0
tcp-keepalive 300
protected-mode no

See:
[    0.000000] ERR:  [libukallocbbuddy] <bbuddy.c @  488> 12000: Failed to add memory region 42b0ae-42c000: Not enough space after applying page alignments
en1: Added
en1: Interface is up
Powered by
o.   .o       _ _               __ _
Oo   Oo  ___ (_) | __ __  __ _ ' _) :_
oO   oO ' _ `| | |/ /  _)' _` | |_|  _)
oOo oOO| | | | |   (| | | (_) |  _) :_
 OoOoO ._, ._:_:_,\_._,  .__,_:_, \___)
              Prometheus 0.14.0~513cee9
1:C 14 Sep 2015 23:18:08.005 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
1:C 14 Sep 2015 23:18:08.006 # Redis version=7.0.11, bits=64, commit=c5ee3442, modified=1, pid=1, just started
1:C 14 Sep 2015 23:18:08.008 # Configuration loaded
[    0.111319] ERR:  [libposix_process] <deprecated.c @  348> Ignore updating resource 7: cur = 10032, max = 10032
1:M 14 Sep 2015 23:18:08.012 * Increased maximum number of open files to 10032 (it was originally set to 1024).
1:M 14 Sep 2015 23:18:08.014 * monotonic clock: POSIX clock_gettime
                _._
           _.-``__ ''-._
      _.-``    `.  `_.  ''-._           Redis 7.0.11 (c5ee3442/1) 64 bit
  .-`` .-```.  ```\/    _.,_ ''-._
 (    '      ,       .-`  | `,    )     Running in standalone mode
 |`-._`-...-` __...-.``-._|'` _.-'|     Port: 6379
 |    `-._   `._    /     _.-'    |     PID: 1
  `-._    `-._  `-./  _.-'    _.-'
 |`-._`-._    `-.__.-'    _.-'_.-'|
 |    `-._`-._        _.-'_.-'    |           https://redis.io
  `-._    `-._`-.__.-'_.-'    _.-'
 |`-._`-._    `-.__.-'    _.-'_.-'|
 |    `-._`-._        _.-'_.-'    |
  `-._    `-._`-.__.-'_.-'    _.-'
      `-._    `-.__.-'    _.-'
          `-._        _.-'
              `-.__.-'

1:M 14 Sep 2015 23:18:08.032 # WARNING: The TCP backlog setting of 511 cannot be enforced because SOMAXCONN is set to the lower value of 128.
1:M 14 Sep 2015 23:18:08.034 # Server initialized
1:M 14 Sep 2015 23:18:08.036 * Ready to accept connections
en1: Set IPv4 address <IP_ADDRESS> mask <IP_ADDRESS> gw <IP_ADDRESS>

Considering that there were no complaints with this, and redis was recently pushed to the registry, I will assume that this works now just fine and will close


Add operator ROI pooling

ROI Pooling projects bounding box proposals from data input layer to internal layer and pools to a manually set output size. As suggested in the reply from mshadow pull request 113, I moved kernel functions of ROI pooling from mshadow to mxnet.
Note that this still remains a naive port from caffe. Any suggestions to improve this implementation is welcome.

Please fix the lint:
src/operator/roi_pooling.cu:185:  Add #include <algorithm> for min  [build/include_what_you_use] [4]
src/operator/roi_pooling.cu:286:  Add #include <vector> for vector<>  [build/include_what_you_use] [4]

please make a rebase against the most recent master

see http://mxnet.readthedocs.org/en/latest/how_to/contribute.html

@precedenceguo can you do the rebase and fix conflicts with the current master?

Thanks for pointing out the documentation on this. Will work on that in a
few hours.
On Sunday, April 24, 2016, Tianqi Chen<EMAIL_ADDRESS>wrote:

@precedenceguo https://github.com/precedenceguo can you do the rebase
and fix conflicts with the current master?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/pull/1928#issuecomment-213841537


Generation of record UUI by orchestrator

Description
First of all - this feature is still in a refinement state and needs to be aligned with stakeholders
UUID:
There should be a UUID generated by the orchestrator (separated issue will be created) to identify where the cleaned dataset was created by and where it needs to go back. UUID should not be shown to the sharing member.
Peergroup:
@nicoprow, @HeyHardy , @maximilianong , +more will be added soon.
Additional information

[x] I'm willing to contribute to this feature

Will add @Sebastian-Wurm to the peer-group.
Will setup a deep dive meeting.

This is part of the 24.08 release.


Clipboard action errors outside secure context environments

Current Behavior
According to the mdn docs you must use the Clipboard API only in secure context (https), this is not the case when running local dev server causing your javascript to crash when using the clipboard action provided by Skeleton.
Expected Behavior
Javascript won't crash when using the clipboard action.
Steps To Reproduce

Using the clipboard action
Run the local dev server outside secure context (you can do this using the --host flag)
Trigger the clipboard action to run
Error gets thrown (see ss)

Link to Reproduction / Stackblitz
No response
More Information
A easy fix for this would be to add this check inside the action near the top:
if (!window.isSecureContext) {
    console.error('Clipboard action failed: website not running in secure context, see: https://developer.mozilla.org/en-US/docs/Web/API/Clipboard');
    return;
}

This bug was discovered in a support thread on discord and might provide more context.

https://github.com/skeletonlabs/skeleton/pull/2029 should address this as well :)


Add ability to hide controls

A lot of the options currently displayed on the right sidebar could potentially be simplified and toggled/hidden in some way to give more space for the graph itself. Always displaying these options adds clutter when the graph itself is already busy and takes up too much space on smaller screens.

On smaller screens, the options won't all fit and can't be accessed:

CSS heaven! May I ask what size of screen and what resolution you are using? I do need to add small screen/low res to my test workflow and brush up on my CSS... Also looking at collapsing the whole menu without adding bulky dependencies...

It's a laptop so 1920x1080 on a 14 inch display.

I had to work hard on my CSS skills but I finally implemented a collapsible menu you can show and hide and made everything more tidy and better looking. Thanks for the great suggestion! It was indeed though to use the plugin on 2K displays. Only tested on 4K before so I had missed it.

Other than addressing the horizontal scroll bar which I mentioned, I think this issue can be closed now.

One more thing which would improve the user experience is for the plugin to remember if the controls are hidden/expanded upon restart. Right now whenever I restart Joplin the controls are always expanded regardless of what I previously set.
Alternatively, just hide the controls by default (maybe have an option under settings to change this behavior for people who don't want this).

with release 1.1.0 there is an option to remember to show the control panel on startup or not.

New option works great!


No speedup from quantization

I ran demo.py with LLaMA-2-7B model to check throughput improvement across precisions.
8-bit model is about 10% faster than FP16 baseline but all precisions from 3-bit to 8-bit showed same throughput.
Tested on A6000 and A100 GPU.
How can I resolve this problem?

The current demo may not demonstrate a speedup if the CPU is not fast enough, causing the software stack running on the CPU to become a bottleneck. In this case, you might not see any end-to-end speedup even with the GPU kernel speedup. With faster CPU cores, you might observe the speedup.
Please note that this demo is just a proof-of-concept implementation. To fully translate the kernel speedup into an end-to-end speedup, it needs to be integrated with a high-performance inference engine like TensorRT-LLM, as reported in our paper. We are working on finalizing the code for TensorRT-LLM end-to-end integration, but it may take some time to release.

Thank you for the response.

Hello. Is TensorRT-LLM integration is still going on?

Hi, I've tried reproducing the benchmark on an H100 and an A6000 and I'm also seeing no speedup on these platforms at lower precisions. Could you explain further what are the CPU processes bottlenecking the performance, and how TensorRT-LLM overcomes the issue? @ilil96

Hello. Is TensorRT-LLM integration is still going on?

It is currently delayed. I apologize for not being able to provide an exact timeline at this moment.


Partitioning versus persistent subscription

Hi,
In the docs on sink partitioning it is written that "If you don’t care much about events order in $all, you can configure Replicator to use concurrent writers, which will increase performance."
In the comments of issue 41 it is written that "The reason for replicator not to use persistent subscriptions is that: 1) No guarantee for events on $all to be processed in order, which is often the requirement; 2) Additional IO on the source database leader as it produces checkpoint events."
Does this mean that, if the order of $all events is not important, it makes to sense to consider replicator using partitioning similar to using multiple consumers of a persistent subscription?
Also, would the same count for using Kafka as this could probably cause the order of $all to differ from the source as well?
Curious on your thoughts.
Cheers,
Peter

I'd prefer if you use Discussions for questions, not issues. Thanks.

Does this mean that, if the order of $all events is not important, it makes to sense to consider replicator using partitioning similar to using multiple consumers of a persistent subscription?

Yes

Also, would the same count for using Kafka

Using Kafka for what exactly, I don't understand the question.


chore: fix GH actions

This PR separates integration tests (which require auth) from unit tests (which do not require auth). The PR adds conditional statements to the steps for running an integration test and getting auth. The conditionals are only true if a PR originates from this repo. This should allow PRs authored directly in the repo to run integration tests while PRs originating from outside the repo only trigger unit tests.

/gcbrun


Add Data.Enum properties

Alot of properties already exist in the documentation of Data.Enum.
Adding them as automated property tests shouldn't be too hard.

:+1:


Read file functionality not working in babelfish

Hi @bill-ramos-rmoswi,
trying to read xml file from my local path in babelfish, so i have write one below helper function in postgre and calling from babelfish t-sql, but got absolute path not allowed error while calling the function in babelfish t-sql, but it is working fine when I call the function from postgre sql itself, any idea about the issue or any idea to read xml file to local path?
postgre helper function:
CREATE OR REPLACE FUNCTION master_dbo.OPENXMLFile(
filepath text
) RETURNS XML
LANGUAGE plpgsql
COST 100
VOLATILE PARALLEL UNSAFE
AS $BODY$
BEGIN
RETURN pg_read_file(filepath):: xml;
END;
$BODY$;
ALTER FUNCTION master_dbo.OPENXMLFile(text) OWNER TO master_dbo;
Babelfish T-sql:
SELECT master.dbo.openxmlfile('C:\Reconfigure\sample2.xml')

Hi @JayakumarEMIS , try saving the function in the sys schema in the PG connection. Then grant permission to public.

@JayakumarEMIS I wonder if you can arrange the incoming files to be written to a subdirectory of C:\Program Files\WiltonDB Software\wiltondb3.3\data dir? If it is possible, then relative paths from data dir should work without using pg_read_server_files role (that may be undesired for security reasons).

Hi @JayakumarEMIS , try saving the function in the sys schema in the PG connection. Then grant permission to public.

Hi @bill-ramos-rmoswi, I checked your suggestions for created the function under sys schema in postgre after grant pubic access to function but still get same issue, and I attached the can you look it, if I missing anything?
Function:
CREATE OR REPLACE FUNCTION sys.OPENXMLFile(
filepath text
) RETURNS XML
LANGUAGE plpgsql
COST 100
VOLATILE PARALLEL UNSAFE
AS $BODY$
BEGIN
RETURN pg_read_file(filepath):: xml;
END;
$BODY$;
Permission access:
GRANT EXECUTE ON FUNCTION sys.OPENXMLFile(text) TO public;
Babelfish t-sql:
SELECT sys.openxmlfile(
'C:\Reconfigure\sample2.xml'
)

@JayakumarEMIS I wonder if you can arrange the incoming files to be written to a subdirectory of C:\Program Files\WiltonDB Software\wiltondb3.3\data dir? If it is possible, then relative paths from data dir should work without using pg_read_server_files role (that may be undesired for security reasons).

Hi @staticlibs, if  place the file wilton db subdirectory location C:\Program Files\WiltonDB Software\wiltondb3.3\data I can able to get read it, but my scenario that not possible to move the file from original location to wilton db directory location, why because if we place any file inside the C:Programfiles directory the window user should have the admin access.
I tried  to gave pg_read_server_files role access as well to my user(wilton), but still not working. and my question is I can able to read the xml file while calling the below postgre custom helper function, inside babelfish postgre(5432)  but not able to do it babelfish tsql(1433)
Function:
CREATE OR REPLACE FUNCTION sys.OPENXMLFile(
filepath text
) RETURNS XML
LANGUAGE plpgsql
COST 100
VOLATILE PARALLEL UNSAFE
AS
BEGIN
RETURN pg_read_file(filepath):: xml;
END;
;
Permission access:
GRANT EXECUTE ON FUNCTION sys.OPENXMLFile(text) TO public;
Babelfish Postgre: -> Working
SELECT master_dbo.openxmlfile(
'C:\Reconfigure\sample2.xml'
)

Babelfish t-sql: -> Not working
SELECT sys.openxmlfile(
'C:\Reconfigure\sample2.xml'
)

Try @staticlibs suggestion of using the data directory.

@JayakumarEMIS I wonder if you can arrange the incoming files to be written to a subdirectory of C:\Program Files\WiltonDB Software\wiltondb3.3\data dir? If it is possible, then relative paths from data dir should work without using pg_read_server_files role (that may be undesired for security reasons).

Hi @staticlibs, if place the file wilton db subdirectory location C:\Program Files\WiltonDB Software\wiltondb3.3\data I can able to get read it, but my scenario that not possible to move the file from original location to wilton db directory location, why because if we place any file inside the C:Programfiles directory the window user should have the admin access.
I tried to gave pg_read_server_files role access as well to my user(wilton), but still not working. and my question is I can able to read the xml file while calling the below postgre custom helper function, inside babelfish postgre(5432) but not able to do it babelfish tsql(1433)
Function: CREATE OR REPLACE FUNCTION sys.OPENXMLFile( filepath text ) RETURNS XML LANGUAGE plpgsql COST 100 VOLATILE PARALLEL UNSAFE AS
BEGIN RETURN pg_read_file(filepath):: xml; END; ;
Permission access: GRANT EXECUTE ON FUNCTION sys.OPENXMLFile(text) TO public;
Babelfish Postgre: -> Working SELECT master_dbo.openxmlfile( 'C:\Reconfigure\sample2.xml' )

Babelfish t-sql: -> Not working SELECT sys.openxmlfile( 'C:\Reconfigure\sample2.xml' )

@bill-ramos-rmoswi  I tried @staticlibs suggestion that was working fine, but there will be an issue in some of our client systems don't have admin window user account, so it may cause the issue our customers, do you have any idea why absolute path not working in babelfish t-sql?

@JayakumarEMIS , would it be possible for you to set WiltonDB log_directory to some other dir outside of Program Files:

And then create a writable directory underneath log_directory and give appropriate write/read permissions to such dir? This way pg_read_file will work on TDS connections with absolute paths to files under log_directory:
select pg_read_file('C:\tmp\log\foo.txt')

If you need to set the specific dir permissions (or check effective access), default WiltonDB SCM account is LocalService, its principal name is Local Service:

I will follow up on pg_read_server_files in Babelfish on TDS connections, but in general I suggest to not use it and use the setup above instead.

Hi @bill-ramos-rmoswi  and @staticlibs, I have added few modification in function to access file in pg_read_server_files in Babelfish on TDS connections, and that is working fine, using I can able to read file anywhere in local path. I think it would be helpful.
-- GRANT pg_read_server_files access to master_dbo
GRANT pg_read_server_files TO master_dbo;
-- Add secuity definer keyword during function creation
CREATE OR REPLACE FUNCTION master_dbo.openxmlfile(
filepath text)
RETURNS xml
LANGUAGE 'plpgsql'
SECURITY DEFINER
COST 100
VOLATILE PARALLEL UNSAFE
AS $BODY$
BEGIN
RETURN pg_read_file(filepath):: xml;
END;
$BODY$;
GRANT EXECUTE ON FUNCTION master_dbo.openxmlfile(text) TO PUBLIC;
GRANT EXECUTE ON FUNCTION  master_dbo.openxmlfile(text) TO master_dbo;
-- verify master_dbo have pg read file access
SELECT rolname
FROM pg_roles
WHERE rolname = 'master_dbo' AND pg_has_role('master_dbo', 'pg_read_server_files', 'member')

@JayakumarEMIS , thanks for sharing the solution!
Basically, for pg_read_server_files permission to work it needs to be granted to the following role name (that depends on the current active T-SQL DB):
select db_name() + '_' + current_user

Grant needs to be done on Postgres connection, after that pg_read_file will work on TDS connection.


feat: Warn if select rules are not used on catalog

Add method to emit a warning log message if a selected field is not present in the discovered catalog.
Patch function in unrelated test

I haven't quite figured out yet how to add a test that validates this function is emitting the log message. (I'm stuck on figuring out how to set up the test such log is emitted, but I'd gladly accept any guidance to implement it, specially if the lack of this test is considered a blocker to this PR getting accepted).
However, I ran it against an actual target (marketo), and found it to be working. Intentionally, the field industry was mis-spelt as Industry, and as can be seen from the screenshot below, the 2nd log line emits the warning Property `leads.Industry` was not found in the tap's Catalog.
This is useful as it lets the data engineer know that a specified property was found not to exist in the Catalog.

closes: https://github.com/meltano/meltano/issues/8336

@edgarrmondragon it looks like the commit checks don't like %-formatting for logs, which were your suggestions
https://results.pre-commit.ci/run/github/378997214/1705342208.pEmW_fVhQQWMEIKYNntG7A
I've reverted those to use f-strings so that the code is consistent with the pre-commit expectations. Let me know if I should try a different approach.


very need inlineQuery example commads to /expamples

most wanted! :)

There is already an inline query command example in the library, you didn't spot it? Or you think thath is not enough ?
https://github.com/akalongman/php-telegram-bot/blob/master/src/Commands/SystemCommands/InlinequeryCommand.php

Sorry, i lost eyes :)


created a models for user credentials

@CodiumAI-Agent /review

PR Reviewer Guide 🔍

⏱ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review
Data ValidationConsider adding more robust validation for fields like 'gstin', 'tradeName', 'legalName', etc., to ensure data integrity and prevent potential issues with incorrect data formats.
Schema DesignThe 'stateCode' and 'pinCode' fields are defined as Numbers. Ensure that this aligns with the expected data format and consider if any additional formatting or validation is needed.


Show Images in browse page for input type "mutlple images"

I dont think thats the way to go.
I have a few hundred entries with 10-30 images each.
This would make the table brek completely.
A carousel would be useful. Plus it would be nice for bread-browse and edit also.
Or take the first image and make it clickable to a lightbox gallery.

I agree. I don't think we should be showing multiple images on browse.  Just too congested.


Add real-time H3 index reader

Add the H3 index support to real-time segments.
Updated the RealTimeQuickStart example to utilize the H3 index. For example

@yupeng9 missing header

@yupeng9 missing header

@kishoreg Updated.

@kishoreg Updated.


docs: rename "token" to "access_token"

Update the param name according to RFC6749 section 4.2.2 (Access Token Response)
casdoor/casdoor#1793

@joezhang-sh Hi, thanks for your contribution. The semantic PR title also needed in your first commit. You can refer to our contributing guidelines to fix

It would be OK after squashed


Add DateFilter trait and moved functions

Added new trait DateFilter for implementing methods like:
all_with_error, all_before_with_error, all_after_with_error and all_between_with_error and moved/added methods like all, just_before, etc.
Function all was moved to DateFilter and returns a Result now.
Removed function between, use DateFilter::all_between instead.
Removed function after, use DateFilter::just_after instead.
Removed function before, use DateFilter::just_before instead.

This allows for a shared codebase for RRule and RRuleSet functions and improved error reporting.
More functions return Result instead of just silently returning less or no dates.
All functions like all, just_before, all_after_with_error also use a shared codebase so this should reduce potential errors and result in common behavior.
Some now functions for _with_error variant of functions where added.
Let me know if there are any remarks.

Hi @ralpha,
I think the return type from the DateFilter is not really a issue and should not block the merge of this great PR. Are there anything you would like to improve upon or do before merging? Thanks again for the amazing work.

Haha, forgot I still had open changes here. :)
Pushed some things I still had locally.
Also noticed there where some properties that are not used. They should be checked out at some point.

Unrelated: I think the "warning" in the README is not really relevant anymore. The library is fairly stable at this point. (as in: it does not crash anymore). At this point we just have to make sure the interfaces of the library are where we want them to be. But that will come over time.

Unrelated: I think the "warning" in the README is not really relevant anymore. The library is fairly stable at this point. (as in: it does not crash anymore). At this point we just have to make sure the interfaces of the library are where we want them to be. But that will come over time.

Yeah, I agree, thanks to your hard work with refactoring and improving the robustness of this library we can finally remove the warning! :tada:
Hopefully, I will be able to get some spare time soon to fix some of the remaining bugs so that we can release v1.0 within not too long :)

Hopefully, I will be able to get some spare time soon to fix some of the remaining bugs so that we can release v1.0 within not too long :)

I think v1.0 is still a bit to early. This usually means the library functions/API is stable too and will not change much.
And I don't think where there yet.
Some things I would like to see before v1.0 is #44, #40, #38 decision, ...
Although I would like to see it reach v1.0 I think we should not rush it. I'm perfectly fine using a v0.x version until we got all the kinks worked out.


Deploy and configure ORA grading MFE

User Story

See https://github.mit.edu/mitxonline/mitxonline-issues/issues/148 for the user story

Description/Context

This MFE replaces one tab in the Instructor Dashboard (similar to the grade book MFE).
The MFE is not quite ready to run in our deployments. We will need to include a commit that hasn't been merged yet: https://github.com/openedx/frontend-app-ora-grading/pull/146
Acceptance Criteria

[ ] set up the automation to build this MFE, including the commit in https://github.com/openedx/frontend-app-ora-grading/pull/146
[ ] deploy to mitxonline-qa for review
[ ] determine how to turn on the MFE

Uh @mitodl/devops can this be closed? It seems ancient history.


Minification issue

Greetings!... i've been using this great module on a project without any problems, until i minified the app. I'm using $filter('tzAlign') but this gives
Error: [$injector:unpr] Unknown provider: aProvider <- a <- tzAlignFilter

The weird thing is that i inspected the minified js and the tzAlign filter is declared. Other filters like uppercase are working fine. I guess that ultimately i could use the unminified version of the module, at least no mangled, but i'm trying to reduce the load time of the app. So, what other solution may be?

Hey @blackotruck, I'm not actively maintaining this project anymore (time constraints, and I'm actively moving away from Angular). However, I do have a recommendation.
See line 160 of angular-timezones.js that reads:
module.filter('tzAlign', function ($timezones) {
  /* ... */
})

Your minification might be rewriting variable $timezones, which uses Angular's clever function signature reading to determine which injections to perform. Either by copying this script into your project, or forking on GitHub and changing your Bower dependencies, try varying this to make explicit the dependencies for tzAlign as follows:
module.filter('tzAlign', ['$timezones', function ($timezones) {
  /* ... */
}])

That should become impervious to any mangling minifiers typically perform. And please let me know how this works out for you!

Wow! thanks, that worked great, and i new it was something like that, but i didn't checked the js file, as all other 3rd parties use the array notation i assumed you used too. Again, thanks for the quick response.

You're welcome, and I'm glad to help!


Acceptance Tests for #264 Filter Rooms by Event Criteria

User story
#264
Acceptance criteria

#
Given
When
Then

1
I am a logged in and authenticated user
I want to find rooms or resources matching my event criteria
I can filter them based on event details

Accepted


geojson api

This PR:

[x] adds a geojson POST end point that can be used by TM for getting task level ml predictions
[x] tests

@batpad thanks! going to keep the test fixtures like that for now because I think we'll replace them with some fixtures we get from looking-glass soon.


DeepDraw pseudo-3d

Would you share you deepdraw code.
Thank you

Do you have any progress？ @moaljazaery  @ZhaofanQiu

no


Updating multiple traces textsList in the live server does not update for each value

Hi me again with another bug I'm afraid
0.3.0 fixes the issue raised in #51 however there remains a very similar issue, this time it is the textsList property that is not updated correctly, however I suspect the source of the issue is different as the data going through the websocket appears correct.
Repro based on the example one
import hep.dataforge.meta.invoke
import kotlinx.coroutines.GlobalScope
import kotlinx.coroutines.delay
import kotlinx.coroutines.isActive
import kotlinx.coroutines.launch
import kscience.plotly.Plotly
import kscience.plotly.models.Bar
import kscience.plotly.plot
import kscience.plotly.server.close
import kscience.plotly.server.pushUpdates
import kscience.plotly.server.serve
import kscience.plotly.server.show
import kotlin.random.Random

fun main() {
    val initialValue = (1..10).toList()

    val traces = (0..2).associate { i ->
        val name = "Series $i"

        name to Bar {
            x.strings = initialValue.map { "Column: $it" }
            y.numbers = initialValue
            textsList = initialValue.map { "Initial value of this datapoint is: ${it}"} // <-- LINE ADDED
            this.name = name
        }
    }

    val server = Plotly.serve(port = 3872) {

        //root level plots go to default page
        page { plotly ->
            plot(renderer = plotly) {
                traces(traces.values)
                layout {
                    title = "Other dynamic plot"
                    xaxis.title = "x axis name"
                    yaxis.title = "y axis name"
                }
            }
        }

        pushUpdates(100)       // start sending updates via websocket to the front-end
    }

    server.show()

    //Start pushing updates
    GlobalScope.launch {
        delay(1000)
        while (isActive) {
            repeat(10) { columnIndex ->
                repeat(3) { seriesIndex ->
                    delay(200)
                    traces["Series $seriesIndex"]?.let {bar->
                        println("Updating ${bar.name}, Column $columnIndex")
                        //TODO replace with dynamic data API
                        val yValues = bar.y.doubles
                        yValues[columnIndex] = Random.nextInt(0,100).toDouble()
                        bar.y.doubles = yValues
                        bar.textsList = yValues.map { "Updated value of this datapoint is: ${it}"} // <-- LINE ADDED
                    }
                }
            }
        }
    }

    println("Press Enter to close server")
    readLine()

    server.close()
}

See my two  // <-- LINE ADDED comments for the diff.
When running this example you would expect the hover label to contain the text "Updated value of this datapoint is: ${it}" where it is the y value of that bar, however the value appears to be being set to the value of the first bar in that series always.

Ok this was an easier fix, raised #54 to fix this


Listen for event of deleting the collection folder on filesystem and remove the collection from the app state

This functionality appears to have already been implemented here (I was looking for old issues I might be able to tackle).
This  issue can be closed???

I tested this issue. The issue still exists.

Expected result:
Collection and requests should be removed from Bruno.
Current result:
Still able to see the folder
And while trying to create a new request  under the folder, it throws error.
Expected result:
Folder should be removed from sidebar or should have some indicator to show it  was deleted.


Replace insecure tempfile.mktemp with tempfile.mkstemp

This PR replace insecure tempfile.mktemp with tempfile.mkstemp,
as the former are insecure and deprecated:
https://docs.python.org/3/library/tempfile.html#tempfile.mktemp
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Hi @yongtang Can you please check @mihaimaruseac's comments and keep us posted ? Thank you!


Need callback for test harness to get received command requests on simulated nodes

Problem
Currently, we have emberAfPostAttributeChangeCallback to get notified everyting an attribute is changed, we also need similar calback for commands when a command request is received.
Proposed Solution
Add a callback to interaction model for get notified when a command is received.

Stale.


[pom] Add fix for jdk 11 javadoc plugin issue

Fixes #1403

Thanks for the pull request, but this is already fixed in assertj-maven-parent-pom. see: https://github.com/joel-costigliola/assertj-maven-parent-pom/commit/9b521bc33ac81b9a809de39c00599a9f928934d8


Provide sample of yaml entries

Thanks for the addon.
any chance you could post some sample entries?
notify:
  - platform: manish
    name: MaNish whatsapp notifire
    target: # WhatsApp number for notificatin (Without the + sign of the country_code)
    token: #T he token for the Whatsapp cloud API
    phone_number_id: #Phone number id from the Whatsapp cloud API
    template: # Template's name's to use
    language: # Template's language

For example
should language: be language: en_US   or  language: English (US)
It would also be helpful to see a sample message that fills in the variables declared in the template

indeed. And is the Target required or not? Because in HA you can give a target also.

any updates here?

notify:

platform: manish
name: MyName
target: abc # WhatsApp number for notificatin (Without the + sign of the country_code)
token: xyz #T he token for the Whatsapp cloud API
phone_number_id: xxx #Phone number id from the Whatsapp cloud API
template: my_template # Template's name's to use
language: en # Template's language

in the template you have to intergrate the {{1}} otherwise there will be a parameter error


⚠ Samsung has degraded performance

In 7849567, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1896 ms

Resolved: Samsung performance has improved in 9f6d6a8.


Add dilation option to ResNet

This PR adds support for replacing 2x2 strides in ResNet with dilation.
This functionality is useful for segmentation models, where high-resolution feature maps are desirable.
Instead of hard-coding the resolution of the feature map, we let the user specify explicitly which blocks they want the 2x2 stride to be replaced by a dilated convolution.
Given that there are only 3 blocks with 2x2 strides (if we don't consider the initial convolution, which also performs downsampling), the user needs to specify a 3-element list of booleans.
Suggestions for a better naming (instead of replace_stride_with_dilation) are welcome.
cc @aadcock

Codecov Report

Merging #866 into master will increase coverage by 0.09%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master     #866      +/-   ##
==========================================
+ Coverage   54.49%   54.58%   +0.09%
==========================================
  Files          36       36
  Lines        3307     3318      +11
  Branches      542      545       +3
==========================================
+ Hits         1802     1811       +9
- Misses       1372     1373       +1
- Partials      133      134       +1

Impacted Files
Coverage Δ

torchvision/models/resnet.py
83.43% <80%> (-0.12%)
: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 ccbb322...298a5df. Read the comment docs.

I'm merging this, but let me know if anyone disagrees with something or has a better idea on how to improve this part.

This change is backward compatible? For now I have been using the torch.nn.apply function to directly change the parametrization of the convolution layers, replacing padding, strides and dilation?

@labor00 yes, this change is backwards-compatible.
Old models and code still work the same as before, because we do not modify the strides / dilations if the parameters are the default (None)

The repository for the gluoncv is very well documented and have lots of tests. i believe that some of the models could be ported to pytorch without hurdle.

@usdflt what models do you have in mind? Note that we are adding segmentation and detection models to torchvision soon.

@fmassa I was thinking of models like pspnet, yolo-v*, ssd. Is there any plans to merge the maskrccn repo within the torch vision?

@usdflt the plan is to include some simplified version of faster r-cnn and mask r-cnn in torchvision


fix: offload ticker lookup from BRC-20 activity query

Remove redundant indexes from locations table
Add locations_block_height_tx_index index

:tada: This PR is included in version 2.1.2-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


test(package_info_plus): run integration tests

Description
Move disabled web specific unit tests to integration tests and start running them together with existing integration tests. This is easier than using flutter test --platform chrome. --platform does not seem to be official supported anyway.
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR.
e.g.

Fix #123
Related #456

Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I titled the PR using Conventional Commits.
[x] I did not modify the CHANGELOG.md nor the pubspec.yaml files.
[ ] All existing and new tests are passing.
[x] The analyzer (flutter analyze) does not report any problems on my PR.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[x] No, this is not a breaking change.

@vbuberen Hey :) Not sure who to ping to get a review on this PR. The motivation for this change is to be able to add the tests that were requested for https://github.com/fluttercommunity/plus_plugins/pull/1330.

You are just in time with your ping as I am doing some changes to do another release in nearest days.


zfs_2_2: 2.2.5 -> 2.2.6

This release brings official support for Kernel 6.10
Also update zfs_unstable as it would otherwise lag behind
Upstream now links zpool completion, so only do so on older zfs (since we still need to support 2.1.x)

Changelog: https://github.com/openzfs/zfs/releases/tag/zfs-2.2.6
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

looks like a lot 6.11 compat merged as well … maybe good enough to compile with 6.11 for zfs-unstable’s maximum version for the bold?

I don't think that's a good idea, maybe OpenZFS team had a reason not to mark it compatible with 6.11

Does anyone know how to lock the kernel version? zfs is slower for the latest kernel, plus mergers, etc., and cannot be updated for a long time.

Does anyone know how to lock the kernel version? zfs is slower for the latest kernel, plus mergers, etc., and cannot be updated for a long time.

Can you elaborate on slower ? For reference for others that pin to the latest zfs compatible kernel

I personally recommend pinning to a specific kernel release line, e.g. linuxPackages_6_6. Using zfs.latestCompatibleLinuxPackages can expose you to downgrades, which I consider harmful, due to misalignment between the zfs project and kernel.
For example, imagine 6.6 is the LTS and 6.9 is the latest kernel. When 6.10 is released, the kernel team will drop 6.9 as it is no longer patched upstream. If zfs is not compatible (it's not) on day one of the release, latestCompatibleLinuxPackages will now resolve to 6.6, forcing a downgrade.
In this scenario I'd pin to linuxPackages_6_9. This will force a failure during eval if a kernel you're using is out of support and allow you to make the decision at that time. If a downgrade is acceptable, make it explicit at that time.

I personally recommend pinning to a specific kernel release line, e.g. linuxPackages_6_6. Using zfs.latestCompatibleLinuxPackages can expose you to downgrades, which I consider harmful, due to misalignment between the zfs project and kernel.
For example, imagine 6.6 is the LTS and 6.9 is the latest kernel. When 6.10 is released, the kernel team will drop 6.9 as it is no longer patched upstream. If zfs is not compatible (it's not) on day one of the release, latestCompatibleLinuxPackages will now resolve to 6.6, forcing a downgrade.
In this scenario I'd pin to linuxPackages_6_9. This will force a failure during eval if a kernel you're using is out of support and allow you to make the decision at that time. If a downgrade is acceptable, for something like the kernel I'd suggest to make it explicit.

thank you, I hadn't thought on that. Will apply your suggestion.

Does anyone know how to lock the kernel version? zfs is slower for the latest kernel, plus mergers, etc., and cannot be updated for a long time.

Can you elaborate on slower ? For reference for others that pin to the latest zfs compatible kernel

https://github.com/NixOS/nixpkgs/issues/337617#issue-2488142638
在youtube有评测结果lqx内核为游戏提供最佳的性能 2.2.5-6.9.11是其zfs最新可用的内核
由于不会指定版本目前我使用boot.kernelPackages = pkgs.linuxKernel.packages.linux_xanmod
There are reviews on youtube with results of the lqx kernel providing the best performance for gaming 2.2.5-6.9.11 is the latest available kernel for its zfs
Since I can't specify the version I currently use boot.kernelPackages = pkgs.linuxKernel.packages.linux_xanmod

Can someone take care of the stable backport?

@mweinelt to 2.2.6? I think @amarshall was giving it more time but recently did https://github.com/NixOS/nixpkgs/pull/340074#event-14167162003

Whatever will provide support for 6.10 :)


Calling undefined getter on self defined class instance yields global variable

Describe the bug
When calling a non existing getter with the same name as a global variable on another global variable that holds an instance of a self defined class, the value of the first global variable is returned, instead of throwing a NoSuchMethodError.
To Reproduce

Enter the following lines in order:

>>> a=1
1
>>> class Foo {}
>>> f=Foo()
>>> f.a
1

Expected behavior
A NoSuchMethodError is thrown, just like if you were to define f='a string' instead of f=Foo() for example.
Actual Behavior
The value of the global variable a is returned even though an instance of the class Foo clearly does not have a getter called a.

You are right, feel free to PR!


fix master build break - VSO

One particular project has dependency to Allocator.h hence pulled the definitions into Allocator. Also updated the NO_EXPORT shape a bit in order to match with old and new compilers we have.

@dotnet-bot test Ubuntu ubuntu_linux_test please

@dotnet-bot test Ubuntu ubuntu_linux_debug please


typo in string 6933

Hi,
in transifex, we have a text to translate with the number 6933, context = "imagery.LSA-DOP20.description".
The original text is : "areal image of the state saxony-anhalt, 20cm resolution"
Maybe this text would be better with something like "aerial image of the state Saxony-Anhalt, 20 cm resolution".
Best regards

It seems to be for name suggestion index project


Add support for RSA and ECDSA signed tokens

Support external tokens signed with  RSA or ECDSA
(Making sure to call jwt.ParseRSAPublicKeyFromPEM() only at startup.)

I think I need some guidance on writing tests. This is my first encounter with golang. I struggled a bit to run the existing tests also. The docs/contribute/_index.en.md and docker-compose.yml are not updated after test database rename to 'prest-test'. There is no testdata/schema.sh as documented. docker-compose.yml does not work for me without editing, "condition: service_healthy" is  docker-compose version 2 thing. Finally  I got to run some tests by looking in .travis.yml.  Then I found that the tests does not clean up after them selves, so I have to nuke the postgres container between each test run.

I think I need some guidance on writing tests. This is my first encounter with golang. I struggled a bit to run the existing tests also. The docs/contribute/_index.en.md and docker-compose.yml are not updated to reflect test database rename to 'prest-test'. There is no testdata/schema.sh as documented. docker-compose.yml does not work for me without editing, "condition: service_healthy" is docker-compose version 2 thing. Finally I got to run some tests by looking in .travis.yml. Then I found that the tests does not clean up after them selves, so I have to nuke the postgres container between each test run. If I go the docker-compose.yml route, then it will mount a volume that is owned and readable only by root, so "go test ./..." errs out because it can not scan all folders.

Really that's a problem we currently have, we have an issue for about local testing #510, I'll look at it
Sorry for the bad experience in your first contribution

It should been testdata/test.sh, not testdata/schema.sh in previous comment.
It seems middlewares/config_test.go could be a place to add a test with RSA token. Should I embed the cert PEM as a string in  middlewares/config_test.go, or should I put it in a file under middlewares/testdata? Is middlewares/config_test.go supposed
to run from middlewares folder?
BTW TestMiddlewareAccessNoblockingCustomRoutes seems to be broken.
It seems to depend on middlewares/testdata/prest.toml, but loads testdata/prest.toml. Also it seems to depend on middlewares/testdata/schema.sql, but middlewares/testdata/schema.sh does not work. (should be psql -h $DB_HOST -p $DB_PORT -U $DB_USER -b $DB_NAME -f testdata/schema.sql) Fixing all that, and TestMiddlewareAccessNoblockingCustomRoutes still fails, because test_write_and_delete_access actually have both read and write access.
So I think if you want tests from contributors, you need to clean the house.

Would it be an idea if the middlewares/config_test.go talked directly to postgres to set up the database in test setup functions, and clean it in test teardown  functions.

I added a unit test for RSA, embedded the cert in the config_test.go. For some reason the codacy job has hung. Maybe because I force-pushed a update (with sign-off) while the job was still running.

I added a unit test for RSA, embedded the cert in the config_test.go. For some reason the codacy job has hung. Maybe because I force-pushed a update (with sign-off) while the job was still running.

I don't know what happened, your PR didn't show up on travis:
https://travis-ci.com/github/prest/prest/pull_requests

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