Favorite Workspaces disappearing in Hyprland

Describe the bug
On startup all my favorite workspaces appear even when I am not at that workspace but once I open any program at any one of my favorite workspace and close it and change the workspace it disappears from  my bar.
Also I have some issues with system tray. Icons are not visible most of the times but sometimes are visible all of the sudden.
To Reproduce
Steps to reproduce the behavior:
https://github.com/JakeStanger/ironbar/assets/110732140/4c5fc841-d12d-4253-9d2c-f6517c059b8f
Expected behavior
My favorite worskpaces should be visible all the time as I understand it.
System information:

Distro: Arch Linux
Compositor: Hyprland
Ironbar version: 0.15.0-pre

Configuration

Share your bar configuration and stylesheet as applicable:

config.corn
let {
	$workspaces = {
		type = "workspaces"
		favorites = [ "1" "2" "3" "4" "5" ]
		name_map = {
			1 = "一"
      			2 = "二"
      			3 = "三"
      			4 = "四"
      			5 = "五"
      			6 = "六"
      			7 = "七"
      			8 = "八"
      			9 = "九"
			10 = "十"
		}
	}

	$focused = {
		type = "focused"
    		show_icon = true
    		show_title = true
    		icon_size = 16
    		truncate.mode = "end"
    		truncate.max_length = 50
	}

	$separator = {
    		type = "label"
    		label = "󰇘"
    		class = "separator"
  	}

	$clock = {
    		type = "clock"
    		format = "%a, %H:%M"
  	}

	$info = {
		type = "sys_info"
		interval.cpu = 3
		interval.networks = 3
		format = [
			"  {cpu_percent}%"
			"󰤨  {net_up:wlp1s0}   {net_down:wlp1s0} "
		]
	}

	$upower = {
		type = "upower"
		format = "{percentage}%"
	}

	$tray = {
		type = "tray"
	}

	$power_button = {
		type = "button"
		lable = ""
		name = "power-button"
		on_click = "!pkill rofi || /home/ayush/.config/rofi/scripts/PowerMenu-blurred"
	}

	$power_menu = {
		type = "custom"
		class = "power-menu"
		bar = [ $power_button ]
	}
}

in {
	position = "top"
	height = 18
	margin.top = 5
	margin.bottom = 0
	margin.left = 5
	margin.right = 5
	icon_theme = "Papirus-Dark"
	start = [ $workspaces ]
	center = [ $focused ]
	end = [ $tray $info $upower $clock ]
}

style.css
@import "mocha.css";
*{
	all: unset;
	font-family: "Noto Sans, Symbols Nerd Font";
	font-size: 16px;
	font-weight: bold;
	border-radius: 8px;
}

.background {
    background-color: @base;
}

/*.container, .container * {
    background-color: transparent;
}*/

/*button {
    color: @red;
    box-shadow: transparent;
}*/

#bar {
    border: 2px solid @green;
}

.popup {
    border: 2px solid @sky;
}
/* -- sys_info -- */

.sysinfo {
    color: @text;
}

.sysinfo .item {
    padding-right: 15px;
}

/* -- tray -- */

.tray {
    padding-right: 15px;
}

.workspaces {
    padding-left: 15px;
}

.workspaces .item{
    padding-right: 15px;
    color: @pink;
}

.workspaces .item.focused {
    color: @sky;
}

.upower {
    padding-right: 15px;
    color: @text;
}

.focused {
    color: @white;
}

.clock {
    color: @text;
    padding-right: 15px;
}

.popup-clock .calendar-clock {
    color: @yellow;
    font-size: 32px;
}

.popup-clock .calendar {
    color: @pink;
    border: none;
    padding: 10px;
}

.separator {
    padding-left: 10px;
    color: @white;
}

#power-button {
    padding-left: 15px;
    color: @yellow;
}

This is probably a regression with https://github.com/JakeStanger/ironbar/blob/18e8244580ccffc3eee086becb62800fc6854c61/src/modules/workspaces.rs#L346-L350

Hey, thanks for reporting and apologies for the delay in getting back to you on this one. I've just merged a fix in.
The issue was that favourite workspaces that are already open on bar startup were not correctly tracked as favourites.


feat: include influxdb exporter role

Include the Influxdb exporter role.
We rely on such exporter because we have applications that push metrics directly to Influxdb at runtime, using this exporter allows us to have those metrics in Prometheus. We have created this role getting inspired by the current Node Exporter role already present in this collection, thanks.

Can you please rebase this against main to pick up the changes in #395

@gardar it is not allowed to create homes in / https://github.com/prometheus-community/ansible/blob/4670e4ad23a081c75fe101ca68fc1f132d9b390e/roles/_common/tasks/install.yml#L40. If a role goes for the default, it fails.
I do not understand why would be needed to set the home to the config dir if anyway create_home is set to false:
https://github.com/prometheus-community/ansible/blob/4670e4ad23a081c75fe101ca68fc1f132d9b390e/roles/_common/tasks/install.yml#L41

@gardar it is not allowed to create homes in /
https://github.com/prometheus-community/ansible/blob/4670e4ad23a081c75fe101ca68fc1f132d9b390e/roles/_common/tasks/install.yml#L40
. If a role goes for the default, it fails.
I do not understand why would be needed to set the home to the config dir if anyway create_home is set to false:
https://github.com/prometheus-community/ansible/blob/4670e4ad23a081c75fe101ca68fc1f132d9b390e/roles/_common/tasks/install.yml#L41

Yeah that doesn't sound right, shouldn't it be using the config dir? "{{ influxdb_exporter_config_dir }}"

@SuperQ something is wrong with Molecule Ubuntu container

The CI issues should be fixed now #440


Replace tutorial with readme file for packaged application

Overview

See: https://jira.bf.local/browse/ECR-XYZ
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

🤔 Created https://jira.bf.local/browse/ECR-3968 for problems with license file generation.


Update Batch keyvault creation docs for user sub accounts

Key Vaults used with Batch user subscription accounts must currently have "Vault access policy" enabled. Added this requirement to the instructions for creating a new Key Vault.

Thank you, Dave. Request you to please update the ms.date as well so that we have the latest timestamp on the document?

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


support connection parameter.

I wanted to add connection as a parameter when redirecting to authorize_url, so I implemented it.
What do you think?
https://auth0.com/docs/api/authentication#social

Coverage remained the same at 0.0% when pulling 8a508fb656c8542d1b8cde8e13af78d09a4070dd on yagince:master into 0fbbf9019d344279a8c1fc0a682977d9c7004255 on sntran:master.

Coverage remained the same at 0.0% when pulling 8a508fb656c8542d1b8cde8e13af78d09a4070dd on yagince:master into 0fbbf9019d344279a8c1fc0a682977d9c7004255 on sntran:master.

Looks fine to me.


Migrating to TypeScript?

Great work!
Any interest in migrating to TypeScript? Pros and Cons?
I took a first step stab at it here. https://github.com/chauey/openapi-codegen
With Visual Studio Code, to run

ctrl ` (to open terminal)
npm install
ctrl-shift-b -> "tsc: build"
f5 (to debug with default params in .vscode/launch.json)

I wonder how best to keep things in sync if no migration to TypeScript but I want to continue to use TypeScript.
Next step I think would be to add interface for the models and remove any "any" types.
Thoughts?

Pros? You tell me :)
Cons:

I don't know typescript, though I am interested in it.
What if it goes the way of coffeescript?
I don't like build steps, they take away one of the best things about javascript

That said, I'm always looking for feedback and collaborators.

@MikeRalphson
You must (for a definition of it) try Typescript, it really improves the dev experience.
Also, (biased & personal) I tend to discard non Typescript projects as I find using Typescript is a good indicator of verifiable quality (not saying non Typescript projects are all of bad quality, here).

Improves in what way(s)?

@chauey apart from reformatting some ifs, and a sprinkling of anys, I can see very few differences between the .js and .ts source here. Is this really enough to get much of the benefit(s) of using Typescript? Plus a command-line build step (npm script) would also be required.

Update: I've added // @ts-check to the Javascript source.
Pros:

It appears to have found an actual bug.

Cons:

All the other warnings/errors look like noise to me.

@sledorze doesn't look like it's going to be "tons" based on what tsc has found so far. But as you say, probably more false positives and less real insight than if the codebase had been written in Typescript from the start, or was properly converted.

Main things I can think of:

static type checking, class based objects
autocomplete/intellisense
ideally will produce less-bugs/better-quality

May enjoy more this way after trying it.
Can cast things to "any" if don't want the typing.
It does seem to be a lot more popular than coffeescript based on google trends.
I've used things like generics and interfaces also, and looking into union types at the moment.
I video I found about TypeScript
https://www.youtube.com/watch?v=rAy_3SIqT-E
Main website: https://www.typescriptlang.org/index.html
After using Typescript, it's hard to go back, but I don't use non-Typescript much so I can't readily list all the pros and cons. Hope you have time to give it a try, you may like it.

It does seem to be a lot more popular than coffeescript based on google trends.

I meant, coffeescript was all the rage at one time, and was going to be the future of typed javascript, but now it's practically dead.
I may try Typescript on some new project to try it out, but at the moment, nothing above is compelling me to move this project to it.

Was playing with some code, and wondering if "default" and "nullable" are valid properties of "parameter" here. And also if "form" is a valid "param.in" value. These are the type of potential bugs it may help find I believe.

I pushed some updates to https://github.com/chauey/openapi-codegen/tree/typescript-conversion, experimenting.
converted the OAS 3.0 schema to typescript interfaces, created classes for it, added the typing for an operation parameter, then casted or checked "instanceof" of the variables/properties.
For this code to run correctly, I'm guessing the JSON/YAML OpenApi doc would be need to be parsed into the correct objects if using "instanceof" codes. More testing and learning needed...
I have never used union types and instanceof before... very cool

@chauey are you by any chance still interested?

@MikeRalphson interested, yes, but I do not know when I would be able to get around to it with current workload. Can't commit, so please proceed without me, maybe someone(s) else can take a shot at it ;)

Closing due to inactivity - but please feel free to reopen the issue if necessary.

Closing due to inactivity - but please feel free to reopen the issue if necessary.


update to version 0.6.0

Restructured engine classes/renamed search engines, split converter(s) into converter(s) and misc_engines)
Restructured UController functions. Unified functions for all engines of one engine class (Converter, Search_Engines, Validation_Engines, etc) are now available.
Documentation overhaul
Smaller fixes and improvements (please check git log)

Still working on the documentation, but the renaming of the search engines should be done now. Would be great if someone can have a look at it.


CASSANDRA-17687 Remove --frames option when generating javadoc

Signed-off-by: tison<EMAIL_ADDRESS>This resolves CASSANDRA-17687.
Shall I submit a patch in this way?

Closed as no response. @smiklosovic I'll appreciate it if you can ping me when Casandra supports JDK17.

Restore - it seems there is a requirement for this change.

Superseded by https://github.com/apache/cassandra/pull/2539.


Env set :default_env does not work

Gemfile.lock:
GEM
  remote: https://rubygems.org/
  specs:
    airbrussh (1.1.2)
      sshkit (>= 1.6.1, != 1.7.0)
    capistrano (3.7.2)
      airbrussh (>= 1.0.0)
      capistrano-harrow
      i18n
      rake (>= 10.0.0)
      sshkit (>= 1.9.0)
    capistrano-composer (0.0.6)
      capistrano (>= 3.0.0.pre)
    capistrano-file-permissions (1.0.0)
      capistrano (~> 3.0)
    capistrano-harrow (0.5.3)
    capistrano-symfony (1.0.0.rc2)
      capistrano (~> 3.1)
      capistrano-composer (~> 0.0.3)
      capistrano-file-permissions (~> 1.0)
    i18n (0.8.1)
    net-scp (1.2.1)
      net-ssh (>= 2.6.5)
    net-ssh (4.1.0)
    rake (12.0.0)
    sshkit (1.12.0)
      net-scp (>= 1.1.2)
      net-ssh (>= 2.8.0)

PLATFORMS
  ruby

DEPENDENCIES
  capistrano (~> 3.4)
  capistrano-symfony (~> 1.0.0.rc1)

BUNDLED WITH
   1.13.5

set :default_env, { "PATH" => "/opt/php/php-7.1.3/bin:$PATH" }
before 'deploy:started', 'deploy:php_info'

namespace :deploy do

  desc "Checking php version"
  task :php_info do
    on roles(:all) do
      execute "php -v"
    end
  end

end

Printed:
01 php -v
01 PHP 5.5.38 (cli) (built: Dec 10 2016 19:31:59)
Expected
01 php -v
01 PHP 7.1.3 (cli) (built: Mar 18 2017 19:23:29) ( NTS )
echo $PATH
01 echo $PATH
01 /usr/local/bin:/bin:/usr/bin
Expected:
/opt/php/php-7.1.3/bin:/usr/local/bin:/bin:/usr/bin

:default_env (as well as many other options) will only be applied if you are passing a symbol to execute. In other words:
# This won't use :default_env
execute "php -v"

# This will use :default_env
execute :php, "-v"

This is covered in the Basic Usage section of the SSHKit README:

Note: In SSHKit, the first parameter of the execute / test / capture methods has a special significance. If the first parameter isn't a Symbol, SSHKit assumes that you want to execute the raw command and the as / within / with methods, SSHKit.config.umask and the comand map have no effect.

Although I sympathize – this is one of the more esoteric aspects of Capistrano. If you have any suggestions on how this can be better documented, let us know or feel free to open a PR!

Thanks!


Feature v6.2.1

[x] think if we replace OperationResultbase to -> OperationResult without Base! as abstract (this feature allow to save same concept and add more extensions later)

[x] move WithStatusCode extension to Base,

[x] Append with OperationResult<>,

[x] Add ctr/method to revice all props as 'Create Instance' (more useing when you have un-know operation-prop take value after plh of condig  ).
ex:
  Status status;
  string message = String.Empty();
  :
  Int statuscode...
  :

  if(--cond--){ ..//change status }
  else if( .... cond ---.... }....
  :

  _Operation.Set(status,message,statuscode....); //auto know exactly operationResult

[x] back to implicity (success) 😉 but for limited types (IList<>,ICollection<>,IEnumerable<>,INumber(int,double,...) .Net7.0) not supported (Tuple,Object, dynamic, any not basic) under see (string)
OperationResult<List<Foo>> Get()
{  return new ();   };

[ ] Global static Isbody, Global static checkin object to serialize , sme to add xtensions for oepration.

[ ] HttpResponseMessage to OperationResult (support full options).

[ ] Stop return null/by default value as Json like as ("", [], {})

Change all namespaces as braking-change to enable  OperationResultbase to -> OperationResult

back to implicity (success) 😉 but for limited types (IList<>,ICollection<>,IEnumerable<>,INumber(int,double,...) .Net7.0) not supported (Tuple,Object, dynamic, any not basic) under see (string)
See: 07fbe6fd07293074fe9303da0beee6b19b467f84
Unable to make implicitly  without corrupted  whole logical casting when work with operation as object or string

--pull-request / merge as sub

not work default/empty to handle null in json, this leave for SerializerSettings

@MhozaifaA to complete  isBody Serialize

++ Note: no test need for `SerializerSettings`

d5375db40c4be5713b5e16d8bcc1718607c5d848
!! re-off auto explicit/implicit


GUI error

Root level exception in OnGUI(): System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
at System.ThrowHelper.ThrowArgumentOutOfRangeException (System.ExceptionArgument argument, System.ExceptionResource resource) [0x00029] in <567df3e0919241ba98db88bec4c6696f>:0
at System.ThrowHelper.ThrowArgumentOutOfRangeException () [0x00000] in <567df3e0919241ba98db88bec4c6696f>:0
at RimWorld.ColonistBar.ColonistBarOnGUI () [0x00262] in :0
at RimWorld.MapInterface.MapInterfaceOnGUI_BeforeMainTabs () [0x0003c] in :0
at RimWorld.UIRoot_Play.UIRootOnGUI () [0x0001f] in :0
at (wrapper dynamic-method) Verse.Root.Verse.Root.OnGUI_Patch1(Verse.Root)
Verse.Log:Error(String, Boolean)
Verse.Root:Verse.Root.OnGUI_Patch1(Root)

Closing bug. Many bugs have been fixed since this was reported. Please reopen in new bug report using the new templates if this issue persists.


Show snap lines across pools and subprocesses

Currently no snap lines are shown when I try to align elements from different pools unless I attach a message flow to them. It would be useful to have snap lines by default.

That would mean having thousands of snap lines and points across huge diagrams. Any idea how to circumvent / optimize this?
I.e. try to edit this diagram here and imagine all elements snapping.

Good point. Maybe only show snap lines to elements within the current view?

I would actually expect not to have too many snap lines in reality, as you typically snap across pools (resulting in some kind of "grid"). But it is a possibility. Only take the ones in the current view sounds like a pragmatic solution :-)

+1
See also #652 and #653

The latest version of the app supports grid snapping.
That should get you covered, without cross-participant snap lines I guess.


Spam Bot

It spams whatever u type in the spam.txt file in WhatsApp, Instagram etc, uses pyautogui library

Thank you for your contribution

Its is showing excluded project in my Hacktoberfest 2021 profile


In firefox, creating a shape on top of another confuses shape creation

This is presumably because the mousemove event is captured by the shape below. More investigation needed.

Actually, it doesn't just happen in firefox. Chrome has that problem too.

Fixed with 5b74abd0be0afa8dff49ed00f6a3327c3b8d7492


no player visible after docker installation

Hello Guys, just installed litepssed in docker. Homepage is working, but I dont get a player running.
I see in chrome console CORS errros from
Access to script at 'https://kit.fontawesome.com/99754263de.js' from origin 'http://rm-nt3:8888' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
Is there any source I could load the file too?

Im going to work on removing the font awesome dependency since I stopped paying for it

Im going to work on removing the font awesome dependency since I stopped paying for it

FeatherIcons is a good alternative

Hello Guys, just installed litepssed in docker. Homepage is working, but I dont get a player running.
I see in chrome console CORS errros from Access to script at 'https://kit.fontawesome.com/99754263de.js' from origin 'http://rm-nt3:8888' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
Is there any source I could load the file too?

I am also experiencing the same issue.

Could someone who was experiencing the issue check out this patch and see if it fixes your issue.

I was unable to reproduce my original issue where the player wasn't playing anything even though I pressed play. Seems everything works now.
@4563453 maybe you can ping me on the Project Lightspeed discord and I can go thru setup with you and see whats up? I'm @Froz on the server

@StarWhiz might want to edit your message, you just mentioned a random person with @ froz


The feature "maximize any pane to a new window with" is already a built-in part of tmux

The default keybinding is prefix + !. So it is not really a new feature as long as we are not talking about old versions of tmux that don't support it.
Just wanted to point that out

Hello 👋
Thanks for pointing that out!
Please let me quote README.md

The "maximize any pane to a new window with <prefix> +" feature is different
from builtin resize-pane -Z as it allows you to further split a maximized
pane. It's also more flexible by allowing you to maximize a pane to a new
window, then change window, then go back and the pane is still in maximized
state in its own window. You can then minimize a pane by using <prefix> +
either from the source window or the maximized window.


Suggestion to allow for conditional ingestor outputs

Logstash allows us to wrap an output (or input) in a condition block. Could I propose adding a condition: key to the logstash_parser.outputs array? For example:
instance_groups:
- name: ingestor
  jobs:
  - name: ingestor_syslog
    properties:
      logstash_parser:
        outputs:
        - plugin: foo
          options:
            key: "value"
          condition:  '[syslog_program] == "gorouter"'

and this would produce output config like:
outputs {
  if [syslog_program] == "gorouter" {
    foo {
      key => "value"
    }
  }
}

Happy to create a PR for this. I just want to get some feedback if this is an appropriate way to make the change. An alternative would be to create a output_overrides parameter to allow complete customisability, similar to filter_overrides.

It could also be extended to else blocks with a minor change:
outputs:
- plugin: foo
  options:
    key: "value"
  condition:  'if [syslog_program] == "gorouter"'
- plugin: bar
  options:
    key: "value"
  condition:  'else'

which would produce:
outputs {
  if [syslog_program] == "gorouter" {
    foo {
      key => "value"
    }
  }
  else {
    bar {
      key => "value"
    }
  }
}


Operator does not have permissions to delete configmaps

Summary
The operator lacks the necessary permissions to delete ConfigMaps, particularly when attempting to remove TemporalCluster.spec.dynamicConfig.
Description
Upon attempting to delete ConfigMaps in the context of managing TemporalCluster.spec.dynamicConfig, an error is encountered indicating insufficient permissions. This inhibits the operator's functionality and disrupts the intended system operations.
Steps to Reproduce

Add a dynamicConfig to cluster spec
Remove TemporalCluster.spec.dynamicConfig using the operator.
Observe the error in TemporalCluster reconciliation indicating permission denial.

Expected Behavior
The operator should have the necessary permissions to delete ConfigMaps related to TemporalCluster.spec.dynamicConfig without encountering permission-related errors.
Actual Behavior
The operator fails to delete ConfigMaps and prevents cluster reconciliation.
Environment

Operator Version: v0.18.0

Additional Information
TemporalCluster reconciliation error:
can't delete resource: configmaps "active-dynamicconfig" is forbidden: User "system:serviceaccount:temporal-system:temporal-operator-controller-manager" cannot delete resource "configmaps" in API group "" in the namespace "temporal"

Hi @AdamSendible ! Thanks for reporting this issue, this will be fixed in #739


Can't pip install on ubuntu 16.04

It thinks I'm on windows.
Running Ubuntu 16.04 LTS
pip3 v 18.0, for python 3.5
Error log below:
wingated@coban:~$ sudo -H pip3 install holodeck
Collecting holodeck
  Downloading https://files.pythonhosted.org/packages/28/b4/471bf45fe6f723f4753d50379f6b402b319a8c142aafb8f0700f33c78054/holodeck-0.0.21.tar.gz
  Ignoring pywin32: markers 'platform_system == "Windows"' don't match your environment
Collecting posix_ipc>=1.0.0; platform_system == "Linux" (from holodeck)
  Downloading https://files.pythonhosted.org/packages/c9/3e/54217da71aa26b488295d878df4d3132093253b4ae5798ac66fcb6921ef0/posix_ipc-1.0.4.tar.gz (78kB)
    100% |████████████████████████████████| 81kB 2.8MB/s
Building wheels for collected packages: holodeck, posix-ipc
  Running setup.py bdist_wheel for holodeck ... done
  Stored in directory: /root/.cache/pip/wheels/c1/27/8b/2dbcb4c5343f796f32eb48f5e07d6b4612b7462d05cfb5b4fa
  Running setup.py bdist_wheel for posix-ipc ... done
  Stored in directory: /root/.cache/pip/wheels/5a/5c/17/d0e2d421abaf3b4097bf9db11108380d734195c6d15c24269d
Successfully built holodeck posix-ipc
**holodeck 0.0.21 requires pywin32>=1.0, which is not installed.**
Installing collected packages: posix-ipc, holodeck
Successfully installed holodeck-0.0.21 posix-ipc-1.0.4

Update: It actually successfully installed (see last line) but logged an error message for requiring pywin32. This error message shouldn't show up.

It appears to be a pip bug.  I'm not sure which versions are affected, but pip installing on stock 16.04 doesn't work; after upgrading pip to the latest version (18.0 as of this writing), it installs but with an error message.


Add batch_scoring_deployment_aware.py wrapper for offline install on Windows

The pull request that added deployment awareness (#132) didn't contain a wrapper script for Windows offline install. Let's fix it and release a new minor version.
Fixes #1

Coverage increased (+0.05%) to 83.197% when pulling 59090db1085db7372d3ae1fe493389b1532d568e on deployment-aware-dist into 7b7ad81b51ec4d84b147826cf4da17b17d5f6146 on master.

I doubt that we need to bump the version of the package since it doesn't change anything in the behaviour, but it's clear release logic fix.
Also, I'm not sure if you want to do it in the another PR or this one, but Linux assets also misses deployment_aware executables.

I doubt that we need to bump the version of the package since it doesn't change anything in the behaviour, but it's clear release logic fix.

I prefer not to overwrite already released packages - it's hard to tell which version is installed in that case.

Also, I'm not sure if you want to do it in the another PR or this one, but Linux assets also misses deployment_aware executables.

I think this PR contains all the required changes. I should have mentioned this in the description. I'll double check after releasing.


Adding any content  with title Sitemap messes up it's edit and view actions

Describe the bug
If you add a Portal Type with Title Sitemap you cannot edit or view it after creation, I suspect there is some router issues since we have a sitemap url  mapped on the root of the site.
To Reproduce
Steps to reproduce the behavior:

Go to https://volto.kitconcept.com/sitemap-bug/sitemap logged in and try to view or edit the added Link
If this content will disappear then just add a Folder and inside it any Portal Type with  title Sitemap.
After you create it you can no longer edit or view it's content.

Expected behavior
I expect to be able to edit any added Portal Type even when it's id is sitemap.
Software (please complete the following information):

Volto Version [11.0.0]

Additional context
The Portal Type can contain sitemap id as long as there as other characters as well.

https://github.com/plone/volto/blob/master/src/routes.js#L64
notice no "exact"

My need was to port an older Plone site to Volto and I have a dropdown menu that has the following structure:
about -> sitemap
I cannot create any content type with id sitemap because Volto doesn't allow it.
I think that sitemap should be exact to the root of the site and allow other content with sitemap id.


TransferUtilityUploadDirectoryRequest not returning filename

I am using TransferUtilityUploadDirectoryRequest and getting the file at UploadDirectoryProgressEvents  using e.CurrentFile and this returns the file normally. However, when I set UploadFilesConcurrently = true and set ConcurrentServiceRequests = 5, e.CurrentFile returns NULL but it returns the other properties on e such as e.TotalBytes

That is by design in concurrent mode. Since there isn't a single file that is currently being uploaded we can't say what the current file is.

So there is no way to find out which file completed successfully in
concurrent mode?
In my case, I am tying to update a dynamodb table when a single file
uploads successfully.

Not currently. We'll add to our backlog to take a look at more evening information during concurrent mode but for now you will have to turn off concurrent mode if you want this information.
An alternative if it fits with your application's design is you could use S3's notification feature to call a Lambda function that you could use to update DynamoDB.

No Worries, using the Lambda function won't work in this case so I will turn off concurrent mode


vncviewer: init

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[ ] Fits CONTRIBUTING.md.

to be taken over by @angustrau

Thanks for the assist! This PR is superseded by #131443.


Point QBImagePicker to specific commit of flypaper fork

Fixes #118

Merged, but can't get the demo project to actually run on iOS. Any chance you could look at that? Seems odd that this would break it though.

Thanks! And yes I will look at it tomorrow

Alright I took a look but it seems to be an issue only with the NS Core demo. The angular demo works fine.  I am admittedly not as familiar with debugging a NS Core app so what I did was:

Reverted the pod change and tried to re-run the demo app. It failed with the same error.
Tried updating all of the NS dependencies of the demo app along with the webpack config. This had no effect, same error.
I noticed that the CI pipeline has been failing for some time and it appears to have the same error as well Error: This value is not a native object..
Updated the demo app to reference v3.0.6 of nativescript-camera-plus and that also errored with the same error
Same with v3.0.5, did not work
Updated the demo app to reference v3.0.4 and that worked. So it seems that something between 3.0.4 and 3.0.5 broke the NS Core demo

I was hoping to be able to fix it but I hope this at least is helpful for you to debug some more

yea that gives me some idea to poke around on it with. Thanks for the detailed info 👍 once I can get it working we'll get this published with the forked pod.


We need to display a trail that shows the travelled path of the drone. This can be a thin perforated line drawn between every position update. This will help users identify the path that drone has taken so far in the video.

We need to display a trail that shows the travelled path of the drone. This can be a thin perforated line drawn between every position update. This will help users identify the path that drone has taken so far in the video.
Below is the existing codebase . Edit the following code to get the desired result.But this has to be done using open layers only. Help me out.
import React, { useState, useEffect, useRef } from 'react';
import PropTypes from 'prop-types';
import { hasEmptyValue, ToggleSwitch, Text } from 'lib';
import '../../stylesheets/components/Video/VideoMap.scss';
import { dronetelemetry, location } from '../../assets/icons';
import Map from '../common/Map';
import FullScreenControl from '../common/Map/FullscreenControl';
import ZoomControls from '../common/Map/ZoomControls';
import BasemapControl from '../common/Map/BasemapControl';
import MarkersList from '../common/Map/MarkersList';
import coordinatesToPointFeature from '../../utils/map/coordinatesToPointFeature';
const VideoMap = ({ site, telemetry, isPaused }) => {
const [marker, setMarker] = useState(null);
const [isAutofocus, toggleAutofocus] = useState(false);
const ref = useRef(0);
useEffect(() => {
    let interval;
    if (hasEmptyValue(telemetry)) {
        if (!hasEmptyValue(site)) {
            setMarker({
                siteType: site.siteType,
                name: site.name,
                uuid: site.uuid,
                geojson: !hasEmptyValue(site.demarcationGeojson)
                    ? site.demarcationGeojson
                    : coordinatesToPointFeature(site.locCoordinates),
            });
        }
    } else if (!isPaused) {
        interval = setInterval(() => {
            const geojson = coordinatesToPointFeature(telemetry[ref.current]);
            setMarker({
                siteType: site.siteType,
                name: site.name,
                uuid: site.uuid,
                geojson,
            });
            // should be removed once we get an actual file
            ref.current = ref.current === telemetry.length - 1 ? 0 : ref.current + 1;
        }, 300);
    }
    return () => clearInterval(interval);
}, [telemetry, site, isPaused]);

if (!hasEmptyValue(site) && !hasEmptyValue(marker)) {
    return (

            <Map>
                <FullScreenControl />

                    <ToggleSwitch
                        name="setAutofocus"
                        checked={isAutofocus}
                        onChange={() => toggleAutofocus(!isAutofocus)}
                    />
                    <Text text="Enable/Disable Refocus on Map" />

                <ZoomControls />
                <BasemapControl />
                <MarkersList
                    markers={[marker]}
                    textKey="siteType"
                    iconUrl={!hasEmptyValue(telemetry) ? dronetelemetry : location}
                    hoverIconUrl={!hasEmptyValue(telemetry) ? dronetelemetry : location}
                    onClick={() => { }}
                    autofocus={isAutofocus}
                />
            </Map>

    );
}
return null;

};
VideoMap.propTypes = {
site: PropTypes.shape({
uuid: PropTypes.string,
name: PropTypes.string,
siteType: PropTypes.string,
demarcationGeojson: PropTypes.object,
locCoordinates: PropTypes.shape({
lat: PropTypes.number,
lng: PropTypes.number,
}),
}).isRequired,
telemetry: PropTypes.arrayOf(PropTypes.shape({
lat: PropTypes.string,
lng: PropTypes.string,
})),
isPaused: PropTypes.bool,
};
VideoMap.defaultProps = {
telemetry: [],
isPaused: true,
};
export default VideoMap;

Append the coordinates to a LineString and style that with a line dash stroke.  This is tracking the ISS but the same logic could be used to follow any moving object https://codesandbox.io/s/simple-forked-fhf067?file=/main.js

@pranjalbansal787 The components you mentioned are not part of this library, but of your application. Please talk to your coworkers that created these components to get help with them. If you're having an issue with OpenLayers, please describe it in detail and provide a minimal example to help reproduce it. Otherwise, please close this issue.

do you want something like this

i built it with Openlayer itself. if you find good i'll share the code.

do you want trail like this !!
will be hapy to share the code build completely with openlayers.

here is my solution to it.
i have live vehicle tracking where we get GPS feeds from mobile application every time there is update in movement of the vehicle and it draws a trace from where it travelled to. so what we do is we establish a Socket connection and that socket connection returns us all the last updated points from the gps table and returns a trail in array format and on UI we plot that into a line string. here the real play is from backend which keeps track of array and sort it by time.
in you code i am not sure what you are doing exactly but here i present the way I do it.
   let trailFeatures: any = new GeoJSON().readFeatures(res, { dataProjection: 'EPSG:4326', featureProjection: 'EPSG:3857' })
   trailFeatures = trailFeatures.filter(async (item: any) => {
                    let hasGeometry = item.getGeometry().getCoordinates()
                    let geometryType = item.getGeometry().getType()

                      if (hasGeometry.length && geometryType == 'LineString') {
                        item.getGeometry().forEachSegment(async function (start, end, idx) {
                          let firstpt = item.getGeometry().getFirstCoordinate()
                          styles = (feature: any) => {
                            let driverCd = feature.get('driverCd')
                            let style = [
                              new Style({
                                geometry: new Point(firstpt),
                                image: new Icon({
                                  src: `./truck-top.svg`,
                                  anchor: [0.5, 0.5],
                                  rotateWithView: true,
                                  rotation: feature.get('direction'),
                                  scale: .25,
                                }),
                              }),
                              new Style({
                                stroke: new Stroke({
                                  color: '#00AFB9',
                                  width: 4,
                                }),
                              })
                            ]
                            return style
                          }
                        })
                        item.setStyle(styles)
                        return item
                      }
                      else {
                        styles = (feature: any) => {
                          let style = [
                            new Style({
                              image: new Icon({
                                src: `${./truck-top.svg`,
                                anchor: [0.5, 0.5],
                                scale: .15,
                              }),
                            }),
                            new Style({
                              stroke: new Stroke({
                                color: '#ffcc33',
                                width: 2,
                              }),
                            })
                          ]
                          return style
                        }
                        item.setStyle(styles)
                      }
                  })

                  ```
once this is done we plot trailFeatures to map layer that we designated for plotting this trace.
we remove previous draw and draw a new one. which happens instantaneously and it does not makes us feel
that animation is not real we can also update the same layer with last updated value by getting the difference
 and adding only new ones using appendCoordinate() method in line string.!
Also we do this for more than 1 vehicles we tried for at most 10 vehicles in a single run.


Feature Request: Add toggle to switch between light vs dark mode

I'm definitely not the only person. I'm using dark mode still I like to see web pages including this json friendly formatted page in white..
Is it possible to have a dropdown select in Extension options page? like this:

System default
Light
Dark

Me too,
wish there were multiple options.

Me too, I wish it would happen.

Coming soon

This is now done in v0.7.1, currently awaiting approval in the Chrome Web Store (usually takes 1-2 days). Or you can install from source to try it now.
Here is the new Options UI:

Please let me know if it works!

Work perfectly fine on my end! Many thanks!


Support collection visibility setup & enhance error display

@knopki how to correctly use MetisCollectionDTO.visibility: MetisCollectionVisibility ?

NB I use it like this:
VISIBILITY = {
    'private': 'private',
    'shared': 'shared',
    'community': 'community',
}

metis_api.v0.collections.create(
    title=sample_designation,
    type_id=METIS_TYPE_ID,
    data_source_ids=ids,
    visibility=VISIBILITY['community'],
)

NB I use it like this:
VISIBILITY = {
...
}

metis_api.v0.collections.create(
...
    visibility=VISIBILITY['community'],
)

It's enough to use visibility="community" without dict/enum/constant. Type checker will help you:

Type checker will help you:

It's limited only with the popular IDEs, supporting that. What if used from, say we, notepad or Python notebook?

It's limited only with the popular IDEs, supporting that. What if used from, say we, Notepad or Python notebook?

I don't like this argument. If you just ignore most of the Python language..... you're in for a rough ride. But if don't use type checks you CAN use enum/const for setting of this string.


Matplotlib warning (possible future error)

window.py#L29-L30
ax.set_xticks([], []) and ax.set_yticks([], [])  have been deprecated since Matplotlib 3.2. For now, it is only a warning message, but in the future it may become a full error.
Perhaps you can replace them with the following:
ax.xaxis.set_ticks_position('none')
ax.yaxis.set_ticks_position('none')
_ = ax.set_xticklabels([])
_ = ax.set_yticklabels([])

Would you be kind enough to test the change and submit a PR?


Add support for IP_RECVERR

This pull request adds support for setting the IP_RECVERR (and corresponding IPV6_RECVERR) sockopt, and then decoding the resulting control messages from recvmsg().  It is a Linux-specific API.
This PR extends #1161 by:

Making the address associated with a IpvXRecvErr control message optional (recvmsg documentation claims it is not provided for some kinds of error), per @WGH-.
Adding basic tests for both IPv4 and IPv6 (blat a UDP packet at a hopefully-unoccupied port on localhost and observe ICMP port unreachable).
Adding a Changelog entry.

I added some trivial doc comments on the ControlMessageOwned variants, but I'm not sure exactly what documentation you had in mind earlier on #1161.

  What we've discovered is that libc::ECONNREFUSED is wrong on Linux-MIPS.  (It's 111 on x86, but 146 in the MIPS ABI.)
failures:

---- sys::test_socket::linux_errqueue::test_recverr_v4 stdout ----
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `111`,
 right: `146`', test/sys/test_socket.rs:1931:9

The failing ARM tests ("Linux arm gnueabi," "Linux armv7 gnueabihf") are hitting some totally bogus Address Family 25951, along with "Linux mipsel."  But other QEMU targets pass (including 64-bit MIPS and BE 32-bit MIPS).  I guess it's somewhat plausible that some QEMU targets are more broken than others?

Is it qemu-system or qemu-user? If it's the latter, I'm not particularly surprised.

Is it qemu-system or qemu-user? If it's the latter, I'm not particularly surprised.

My guess is user, although I'm not sure how to tell from the cirrus config.
So, I think both QEMU emulation bugs relate to do_sendrecvmsg_locked() (QEMU user): https://github.com/qemu/qemu/blob/master/linux-user/syscall.c#L3282-L3289
Both problems appears to be bugs in host_to_target_cmsg():

sa_family for sockaddrs associated with IP_RECVERR control messages is being butchered on some platforms (looks like maybe 32- / 64-bit issue).
ee_errno is not being correctly translated from host to target on platforms where sa_family was fine.

The sa_family one is tricky -- we do appear to do host_to_target_sockaddr translation.  Perhaps that isn't fully functional?  Alternatively, if the control message does not contain an offender address, or if it does not exactly match the size we expect, we just skip straight to 'unimplemented' and do a memcpy instead of translating.  Different alignment requirements between 32/64-bit might cause that, although I'm not seeing it immediately.
ee_errno bug is pretty obvious here (and same for ipv6).

Oof, that's a complicated test.  And the QEMU bug doesn't suprise me; there are plenty more where it came from.  This PR LGTM.  Just squash your commits and I'll merge it.

Thanks!  Squashed.


not linked with lm

i try to compile example from readme
duktape.c:(.text+0xb84): undefined reference to `ceil'
/home/kostya/projects/1112/libs/duktape/.build/lib/libduktape.a(libduktape.o): In function `duk__atan':
duktape.c:(.text+0xb89): undefined reference to `atan'
/home/kostya/projects/1112/libs/duktape/.build/lib/libduktape.a(libduktape.o): In function `duk__asin':
duktape.c:(.text+0xb8e): undefined reference to `asin'
/home/kostya/projects/1112/libs/duktape/.build/lib/libduktape.a(libduktape.o): In function `duk__acos':
duktape.c:(.text+0xb93): undefined reference to `acos'
/home/kostya/projects/1112/libs/duktape/.build/lib/libduktape.a(libduktape.o): In function `duk__round_fixed':
duktape.c:(.text+0x19bd): undefined reference to `floor'
/home/kostya/projects/1112/libs/duktape/.build/lib/libduktape.a(libduktape.o): In function `duk__fmin_fixed':
duktape.c:(.text+0x1a32): undefined reference to `fmin'
/home/kostya/projects/1112/libs/duktape/.build/lib/libduktape.a(libduktape.o): In function `duk__fmax_fixed':
duktape.c:(.text+0x1aa4): undefined reference to `fmax'
collect2: error: ld returned 1 exit status
Error: execution of command failed with code: 1: `cc -o 1 "/home/kostya/projects/1112/.crystal/home/kostya/projects/1112/1.cr/main.o"   -L/home/kostya/projects/1112/libs/duktape/.build/lib -L/home/kostya/projects/1112/libs/duktape/.build/include -lduktape -levent -lrt -lpcl -lpcre -lgc -lpthread -lunwind`

if i add -lm after -lduktape all is compiled

Thanks for the feedkback!
You're right, all of the Duktape documentation says to link with the standard math library, but clang on my system would throw a warning about unused compiler arguments.
Please let me know if pull request #2 fixes the issue.

#2 not fixed
crystal failed to link
`cc -o 1 "./.crystal/home/kostya/projects/1113/1.cr/main.o"   -L/home/kostya/projects/1113/libs/duktape/.build/lib -L/home/kostya/projects/1113/libs/duktape/.build/include -lduktape -levent -lrt -lpcl -lpcre -lgc -lpthread -lunwind`

if add -lm
`cc -o 1 "./.crystal/home/kostya/projects/1113/1.cr/main.o"   -L/home/kostya/projects/1113/libs/duktape/.build/lib -L/home/kostya/projects/1113/libs/duktape/.build/include -lduktape -levent -lrt -lpcl -lpcre -lgc -lpthread -lunwind -lm`

all is linked

@kostya  - I think I understand.
I've updated PR #2 to link against the standard math library in Crystal's ldflags. I believe this should solve your problem.
Can you verify that your issue is now resolved with PR #2?
Thanks!

fixed now


Not able to install

C:\Users\MSD\Desktop\project\lotteryApp>cordova plugin add https://github.com/edsonhoraciojunior/verify-automatic-date-time-zone
(node:11000) UnhandledPromiseRejectionWarning: CordovaError: Failed to fetch plugin https://github.com/edsonhoraciojunior/verify-automatic-date-time-zone via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: cmd: Command failed with exit code 1 Error output:
npm ERR! code ENOLOCAL
npm ERR! Could not install from "..\plug" as it does not contain a package.json file.
npm ERR!      C:\Users\MSD\AppData\Roaming\npm-cache_logs\2019-01-27T09_15_51_560Z-debug.log
at C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\cordova-lib\src\plugman\fetch.js:173:37
at _rejected (C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\q\q.js:864:24)
at C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\q\q.js:890:30
at Promise.when (C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\q\q.js:1142:31)
at Promise.promise.promiseDispatch (C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\q\q.js:808:41)
at C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\q\q.js:624:44
at runSingle (C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\q\q.js:137:13)
at flush (C:\Users\MSD\AppData\Roaming\npm\node_modules\cordova\node_modules\q\q.js:125:13)
at _combinedTickCallback (internal/process/next_tick.js:131:7)
at process._tickCallback (internal/process/next_tick.js:180:9)
(node:11000) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async
function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:11000) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Make sure you’re on the root directory of your app and it contains a package.json file.
Also, in cordova plugin add use the package name not URL, like:
cordova plugin add verify-automatic-date-timezone

The best option would be to create a function and try calling it after the app has initialized completely
not sure why it is undefined after the platform initialized
I have used this in this way in Ionic 1/Angular JS
$rootScope.checkTimeSetting = function(){
                window.VerifyAutomaticDateTimeZone.isAutomaticChecked(function(isIt){
                    if (isIt != 'true') {
                        Global.showAlert(GlobalConstants.errorAlertHeader, "The Automatic Time and Timezone settings are not enabled on your phone.For proper functioning of the app please enable these settings and restart the app",function(){
                            window.VerifyAutomaticDateTimeZone.openTimeSettingPage(function(isIt){
                                navigator.app.exitApp();
                            });
                        });
                    }
                });
            }


PGP private key double prompt bug

The issue here is that when you import PGP secret key via keybase import --push-secret, then when trying to decrypt, you end up with two SKBs - one local LKSec and one remote Triplesec. The order in which they are parsed during UnlockKeyById seems to be deterministic and the Triplesec one comes first.
So this is what happens when PGP decrypt is invoked through keybase after service is first ran (so passphrases are not cached yet):

run keybase decrypt, get to GetKeysById flow
find two SKBs, one from the server and one from local keyring
start working on first SKB which contains the key, it happens to be Triplesec
get prompted for passphrase because tsec/pps is not cached in passphrase stream yet
get to SKB#UnlockSecretKey, store cached tsec in passphrase cache, but only tsec
go to the second SKB, it's LKSec, try to get cached pps
oops, we don't have cached pps (despite asking for passphrase in 4), prompt for passphrase again.
Go to SKB#UnlockSecretKey, cache both tsec and pps after calling UnverifiedPassphraseStream

A proof of concept in this PR is to make SKB.UnlockSecretKey always use UnverifiedPassphraseStream (even for triplesec) and store both pps and tsec in cache, even when SKB is Triplesec and PPS isn't needed. I don't know whether this has any side effects or not though. And UnlockSecretKey is probably not the right place to do this.

non-invasive fix would be to first process LKSec keys and later Triplesec


enable low_cpu_mem_usage when loading model

we should enable low_cpu_mem_usage to avoid loading 8 copies of the model into CPU.  This should boost the memory usage as well as loading time.
@Ssukriti

from_pretrained (.., low_cpu_mem_usage=True)

is this working already?

@lchu-ibm low_cpu_mem_usage is already defaulted to True when launching with our accelerate defaults.
As you can see here when is_fsdp_enabled() = True , which it will be when that flag is set, then low_cpu_mem_usage is set to True
https://github.com/huggingface/transformers/blob/3f06f95ebe617b192251ef756518690f5bc7ff76/src/transformers/modeling_utils.py#L3442-L3443


updating a9lh question

Following the guide "https://github.com/Plailect/Guide/wiki/Updating-arm9loaderhax" it says to delete the a9lh folder is that a mistake?

No it is not, it is no longer needed after updating your a9lh payload.

On Jun 18, 2016, at 6:15 PM, Guardian9978<EMAIL_ADDRESS>wrote:
Following the guide "https://github.com/Plailect/Guide/wiki/Updating-arm9loaderhax" it says to delete the a9lh folder is that a mistake?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank You.


test: add api and doc http test

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
add http's api and doc IT.
Checklist

[ ]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
https://github.com/GreptimeTeam/greptimedb/issues/983

iirc we already have a DummyInstance, could we reuse that?

Not sure if integration-test is a better place for this kind of URL test. cc @sunng87

@waynexia Since these test only covers http server I think it's ok to keep it here


[esp32] led.read() is always 0.

Need to check gpio_read function when gpio is set to OUTPUT

gpio_read() function returns 0 always when the GPIO set to OUTPUT.

Issue is fixed.


Intel XDK Cordova 6.2.0 Android Build Fail

When adding this plugin via Intel XDK and when using Cordova 6.2.0, the build fails.
Also fails with 5.4.1, is this an Intel issue?

Log: https://gyazo.com/aa95a6af4f49a2a015bcef9124515d44

https://software.intel.com/en-us/forums/intel-xdk/topic/600642
This is from 2015

sir,
intel xdk not supporting gradle build. So could you please give some suggestion for this.


feat(tutorial): complete step 3

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

hi @alisonjoseph whenever you have time could you please review this PR, also I updated the step 5 PR regarding the feedback you put there


I can see clearly now

Just general updates for in-game Windows. (Both reinforced and non)

added construction recipes + low wall recipe
updated broken icons
made them snapgrip by default like walls

I was gonna bundle this pr with a windows update to the station but the mapping is so dumb i'll just pr it when i'm ready.

comment test


db.Scan method question

Your Question

I found in this commit the implementation of db.Scan has changed, now(v1.20.1) it use row processor to execute instead of query processor which breaks previous behaviors of db.Scan method. I try to add a callback after row, it likes db.Callback().Row().After("gorm:row").Register("decrypt_data", decryptData)  but since the dest is explicitly specified after call db.ScanRows method in db.Scan, how can I do some extra work after query(such as decrypt sensitive data)?
The document you expected this should be explained

Expected answer

Hello,
Yes, the Scan methods should not call Hooks methods, so I made the change, it doesn't make sense to have another methods to works exactly like Find, any reason that don't use Find?

Hello,
Yes, the Scan methods should not call Hooks methods, so I made the change, it doesn't make sense to have another methods to works exactly like Find, any reason that don't use Find?

No, just want to get a confirmation. I will replace Scan method in my legacy code with Find method. Thank you for your prompt reply.


Remove meaningless PersistentTypesImportStrategy.SkipOrCreate

As for PersistentTypesImportStrategy.SkipOrCreate, I do not see any use for that.
If my excel column got a value that reference an object, why would i want it so skip the assignation if the reference object exist...

Installer lab build 18.1.603.06 includes commit Remove meaningless PersistentTypesImportStrategy.SkipOrCreate #221 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS

Installer lab build 18.1.603.21 includes commit Remove meaningless PersistentTypesImportStrategy.SkipOrCreate #221 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Use kubectl apply in custom service example deploy

https://github.com/kubernetes/kubernetes/issues/29542 has been fixed,
use apply instead of create to create third party resources in custom service
monitoring example.

lgtm :+1: thanks!


accept client start time of 0

It's useful for users to be able to query opentsdb from unix time 0. This PR allows for this.

@kgryte

Unfortunately tests will fail here until the version of the validate-time repo is bumped.


Remove expected error code as it only thrown by local driver

Fixes: https://github.com/microsoft/FluidFramework/issues/9446
The error code 0x202 is only thrown by local driver. We want this test to expect container close for all drivers except the odsp driver. So, this is required.

does this pass on other servers? i thought you would need to explicitly call container.close at the end of each test, so the close event is always there

does this pass on other servers? i thought you would need to explicitly call container.close at the end of each test, so the close event is always there

This is failing on r11s too. Shouldn't this:
if (provider.driver.type !== "odsp") {
// this flow is currently only supported on ODSP, the others should explicitly reject on attach
return assert.rejects(attachP,
(err) => /(0x202)|(0x204)/.test(err.message) /* "create empty file not supported" */);
}
throw the error and make the test to pass. So we should only close container in case of odsp driver, right?

Did you run the tests locally against ODSP driver to make sure they pass?

Yes they are passing

does this pass on other servers? i thought you would need to explicitly call container.close at the end of each test, so the close event is always there

This is failing on r11s too. Shouldn't this: if (provider.driver.type !== "odsp") { // this flow is currently only supported on ODSP, the others should explicitly reject on attach return assert.rejects(attachP, (err) => /(0x202)|(0x204)/.test(err.message) /* "create empty file not supported" */); } throw the error and make the test to pass. So we should only close container in case of odsp driver, right? Although in case of other driver it will not reach to the end as we returning already.

exactly. the container is already closed and we return, so no need check by the time we close at the end

Closing this PR and will cherrypick this in here: https://github.com/microsoft/FluidFramework/pull/10041


Don't unwrap header metadata

Fix for a comment in #1092.
r? @fitzgen

:pushpin: Commit 7f0aa6c has been approved by emilio

@bors-servo r+

:hourglass: Testing commit 7f0aa6c6da1b62848eeccb40baadbe0ef3c11c1b with merge c54e64c5377d299d67b6a9f2d735ba49740a7ae6...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing c54e64c5377d299d67b6a9f2d735ba49740a7ae6 to master...

Thanks @seemyvest !


JavaScript Track: ES6 Support

Re: https://twitter.com/exercism_io/status/889983074959851520
Does the Exercism JavaScript track fully support the use of ES6 syntax, or is it only the ES6 track that does that?

From a technical point of view, it depends on the Node.js version you're using to run the code.
From the track's philosophy point of view, the JavaScript track is intended to practice ES5 features, that can be run in all modern and not-so-modern browsers (mobile, <IE11,...) while ECMAScript track is meant to practice recent features of the ECMAScript standard, that are not compatible with all browsers (eventually they will).
As it is technically possible, if you run Node.js v6.11.1 (current stable version) you can write your solutions using some ES6 features (destructuring, optional parameters, string templates, const and let). But, if you want to practice them, I'd recommend you to practice with ECMAScript track exercises.
I'm wondering about the why of your question, so sorry if the next question sounds rude, it's not my intention: do you miss some exercise in the ECMAScript track? If so, we can help adding it to the track, or we can help you in that task.

Not rude at all. I just found myself using some ES6 syntax as I was going though the earlier JS exercises without thinking about it. I was just curious as to how far that would work, given I knew there was a separate ECMAScript track.
I'm using Node.js v6.11.0 on Windows 10, and v7.2.0 on OSX

@rchavarria that one is a good question. It's still strange for me that we have two tracks for the same language - if I can put this way. ES6 is JavaScript, and vice-versa. I can understand the desire for a ES track while it was not really fully supported by browsers, but today it's not really a problem anymore.
Why do we keep two distinguished tracks for something that's really the same thing?

I'm asking that question every now and then, but I'm never convinced to join the two tracks together.
One feature that makes ES6 more different to ES5 are modules (import and export keywords). And it's not clear at all the ES6 modules will be supported by Node.js (Stack Overflow to the rescue).
I could argue that ES2016 and beyond will come with key features, and we could benefit from using the transpiler to use them before they're supported by Node.js. But I'm not sure there will be a key feature so interesting at the beginning of working with the language that justifies that.
Anyway, something I'd like to keep is that the javascript track doesn't use any transpiler. It would be just the language (and a testing tool). I don't like it to polute that track. Unfortunately, nowadays, ES6 modules is the feature that prevents us from merging the tracks :(

Good point, @rchavarria.
I understand that adding the transpiler to the experience of someone that's trying to jump on JavaScript and just want to learn the basics of the language might not seem to be the ideal and it's something we should work on. However, I don't see the point of keeping two tracks for the same language because Node.js does not support ES6 modules. They are part of the specification and will be supported at some point. It doesn't seem like a good reason to avoid merging them.
We also have the advantage of having the practicality of npm. A single npm i can install everything a practitioner would need, including the transpiler. Maybe we should consider it when deciding about it. The JavaScript experience on Exercism could leverage from this merge, since one single track would be easier to maintain than two ones, with distinguished issue trackers, test suites, setups, etc.

Closing this one, since the question has been answered and additional discussions should happen on dedicated issues.


Support for specifying column SRID

Hi,
is It possible to specify the column SRID at model creation?  (v2.2.0-preview1)
I'm using the code first approach and the POSTGIS columns created (automatically) have SRID set to 0.
I think this could be done using Migrations, but I was wonderring about annotations or Fluent API.
By the way, congrats for the awesome work!

I don't think that we do currently, but it looks like Sqlite has support for this in 2.2.
We could probably do something similar too.
/cc @roji @YohDeadfall

Makes sense.

I'm actually going to take an initial pass at this. I'll open a WIP PR in a little while.

Doing this cleanly for 2.2 isn't going to be possible because of https://github.com/aspnet/EntityFrameworkCore/issues/14000. While it's possible to hack something up (i.e. pick up the model annotation in the MigrationsSqlGenerator), it should be possible to simply specify the exact store type as described in the EF Core docs. Give this a try, if this doesn't work please post back here.
Note: I'll still work on completing #719 for automatically setting the column's spatial type correctly (e.g. GEOMETRY (POINT) based on the NTS CLR type.

Note consider dimensionality restrictions.

Similar to #738, on reexamination I'm not sure this really makes sense. Users can explicitly specify the column's store type, including both a type constraint (point vs. linestring) and an SRID. We should consider closing this...

At this point, I think closing makes sense. If anything changes between now and 3.0, we can always reopen and reconsider at that time.


Rename shell extension Verb Id

Workaround Windows issue causing shell extension to not appear in classic context menus on certain systems.
As seen in https://github.com/MediaArea/MediaInfo/issues/994#issuecomment-2541700900 and https://github.com/MediaArea/MediaInfo/issues/994#issuecomment-2541738716.
I have no idea what letter works in our case so let us just try "C" since it worked for NanaZip.
Thanks to https://github.com/M2Team/NanaZip/pull/205.

Snapshot @cjee21 @sevastopolcat

@JeromeMartinez Still works fine on my primary PC (Windows 11 24H2 26100.2605, AMD64, 10 apps with packaged context menu shell extension installed).

Yes, it works! The MediaInfo item is in the context menu of media files and folders in Explorer and Total Commander.
Windows 11 26100.2605

Yes, it works! The MediaInfo item is in the context menu of media files and folders in Explorer and Total Commander.

So it was not working with "M" as start of the string but "C" is working, that is crazy... But if it is the way, we do that...

So it was not working with "M" as start of the string but "C" is working, that is crazy... But if it is the way, we do that...

Crazy indeed and the fact that it only happens on certain systems.
At least it is not as bad as some Android bugs that cause files to disappear overnight due to special chars in folder names or download of same filename overwriting the previous file silently even after previous has been moved out of downloads folder.

Snapshot @cjee21 @sevastopolcat

This snapshot is not resolving this issue on Directory Opus, which does not have the option to use the newer Windows 11-style menu. I also don't know if they use the old-style menu as-is or if its some sort of semi-custom menu.

Directory Opus, which does not have the option to use the newer Windows 11-style menu.

If so, how the Opus developer can have it on his machine, see his own comment.
You can read the all thread from the linked comment, it is more complicated than that (and more or less random).


Use EncodeUri() if part of the URL path or EncodeUriComponent() if used as a URL parameter

https://github.com/alexminza/trackjs-agent-gtm-template/blob/399aa552e319cd8514f3f47d7fd9dac8ca87d740/template.tpl#L208
When reviewing your template we've noticed that it is using parameters to build a url. It's a best practice to use EncodeUri() if part of the URL path or EncodeUriComponent() if used as a URL parameter.

https://github.com/alexminza/trackjs-agent-gtm-template/blob/399aa552e319cd8514f3f47d7fd9dac8ca87d740/template.tpl#L209
Only scriptDomain  is used as the URI host - for the Domain Forwarding feature in TrackJS.


SeqTriad2::ChangePattern crashes Rack 0.6

I have a reasonable chance to crash Rack 0.6 on MacOS X when I switch around the patterns in the SeqTriad2 (either by mouse click or by trigger) with the latest Mac build posted here:
Thread 4 Crashed:
0   plugin.dylib                  	0x0000000102b03f31 Seq_Triad2::ChangePattern(int, int, bool) + 337
1   plugin.dylib                  	0x0000000102b072dc Seq_Triad2::step() + 796
2   com.vcvrack.rack              	0x000000010005978e rack::engineRun() + 398
3   com.vcvrack.rack              	0x00000001000592ad void* std::__1::__thread_proxy<std::__1::tuple<void (*)()> >(void*) + 93
4   libsystem_pthread.dylib       	0x00007fff8a61199d _pthread_body + 131
5   libsystem_pthread.dylib       	0x00007fff8a61191a _pthread_start + 168
6   libsystem_pthread.dylib       	0x00007fff8a60f351 thread_start + 13

I had a bug when randomizing patterns, did you randomize?  Anyway that is fixed now.

well, I was manually switching patterns and overdid it a bit it seemed,
could be same issue, not stressed it that much since ;)
On 26 April 2018 at 03:05, mschack<EMAIL_ADDRESS>wrote:

I had a bug when randomizing patterns, did you randomize? Anyway that is
fixed now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mschack/VCV-Rack-Plugins/issues/64#issuecomment-384480367,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL5WAGgN2mtS_cE9D3bD0xs2UvkJe5Nhks5tsR1ogaJpZM4TOBDr
.

Still happening here with version 6_22_2018 in Rack 0.6.1.


Handle to close a modal programmatically

Detailed description
Describe in detail the issue you're having. Is this a feature request (new component, new icon), a bug, or a general issue?
feature request or could be a bug

Is this issue related to a specific component?

yes, modals

What did you expect to happen? What happened instead? What would you like to see changed?

A way to programmatically close the modal so that I could close it using some other control or based on some other condition.
I am not sure if handleClose prop is supposed to help in this. If it is indeed supposed to do it, probably an example of how to do it could help.

What browser are you working in?

chrome

What version of the Carbon Design System are you using?

carbon-components: 8.1.8
carbon-components-react: 4.1.2

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

Working on IBM App Launch service on bluemix

Hi IIUC the code @chethankumar the open/closed state can be changed by the open prop. In case if you are concerned about our vanilla repo, too - You can call modal.hide().

@asudoh The open prop does close and open the modal, but if it is set to false, the click of the trigger button won't open the modal again.
So, currently its as if you can either do it by the open prop or by the click of the trigger button. But, I don't seem to figure out to make it work with both.

@chethankumar Would you mind sharing your code that demonstrates the problem you described, e.g. on CodePen? Thanks!

Hi there @chethankumar!
Do you have some concrete examples for the situations where you would need:

A way to programmatically close the modal so that I could close it using some other control or based on some other condition.

In addition, any API suggestions for what you would anticipate would be super helpful to make sure we're on the same page 😄

So, this is what I was trying to implement.
I have a modal wrapper which itself renders a trigger button. Clicking on this, I can open the modal.
And the close button on the top right corner will close it.
All good till here.
 I am not using the default 2 buttons provided by the wrapper as I need more of them.
Now, onClick on these buttons, I would like to close the modal.
I tried to use the open prop of the modal, to close the modal on click of these buttons.
But, when I do that, the next time I click on the trigger button, the modal doesn't open since the open prop is set to false.
It would be lot helpful if the trigger button and the close button overrides the open prop.

Anybody know how to remove the "x" icon from the modal ?

@robinchoii for UX reasons, we recommend not hiding the x from the modal. For more info: http://www.carbondesignsystem.com/components/modal/usage

Hi - Got a better handle on this this issue as I was looking at another stuffs - As far as I see:

<Modal> lets you fully control the open/closed via prop, and let you handle close button by youself (via onRequestClose prop) - In other words, <Modal> itself does not close itself when X button is clicked
<ModalWrapper> is an example controlling <Modal> by maintaining open/closed state

That said, if you need more than a simple trigger button/modal combination you may want to something similar to <ModalWrapper> by yourself.
Closing this issue for now - Feel free to re-open if anybody has any further thoughts!


API doc: Style edit. Fix formatting, typos, usage.

@stbof FYI it looks like this has a merge conflict right now.

Hey so it looks like this causes a lint error (visible in the Travis output):
mlflow/projects/submitted_run.py:39:101: E501 line too long (102 > 100 characters)
ERROR: InvocationError: '/home/travis/build/databricks/mlflow/lint.sh'

You should fix this. You can run lint.sh locally to try it too, or just wait for Travis and see its output.

Fixed. Sorry for the error and thank you for your patience while I learn this process.

Codecov Report

Merging #191 into master will decrease coverage by 0.06%.
The diff coverage is 50%.

@@            Coverage Diff            @@
##           master    #191      +/-   ##
=========================================
- Coverage   51.16%   51.1%   -0.07%
=========================================
  Files          93      94       +1
  Lines        4792    4798       +6
=========================================
  Hits         2452    2452
- Misses       2340    2346       +6

Impacted Files
Coverage Δ

mlflow/models/__init__.py
100% <ø> (ø)
:arrow_up:

mlflow/utils/cli_args.py
100% <ø> (ø)
:arrow_up:

mlflow/projects/databricks.py
45.31% <ø> (ø)
:arrow_up:

mlflow/cli.py
79.51% <ø> (ø)
:arrow_up:

mlflow/azureml/cli.py
95% <ø> (ø)
:arrow_up:

mlflow/tensorflow.py
97.05% <ø> (ø)
:arrow_up:

mlflow/projects/submitted_run.py
62.22% <ø> (ø)
:arrow_up:

mlflow/sagemaker/__init__.py
17.15% <ø> (ø)
:arrow_up:

mlflow/sklearn.py
78.68% <ø> (ø)
:arrow_up:

mlflow/sagemaker/container/__init__.py
0% <0%> (ø)
:arrow_up:

... and 10 more

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 9eb4e77...613fe37. Read the comment docs.

@mateiz is this good to go?

Yup, I'll merge it.


refactor(behavior_velocity_traffic_light_module): prefix package and namespace with autoware

Description

This PR adds the autoware_ prefix to the package and puts headers in the autoware namespace.
Part of:

https://github.com/autowarefoundation/autoware/issues/4569

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@rej55 I've fixed the conflicts. Could you have a look at the changes when you have a moment? Thanks.

@satoshi-ota @shmpwk @soblin @tkimura4 could please either of you have a look at this PR when you have a moment? Thanks :slightly_smiling_face:

@shmpwk thanks! :slightly_smiling_face:


MIPS sw instruction tooltip does not indicate any writes occur

The tooltip for sw instructions in MIPS (and possibly other architectures?) says two reads occur, and does not list any writes. The LLIR appears correct, so it's apparently just a UI nit.
Example:

Currently true for all architectures, not a bug per-se, just a feature that has never existed.


Improve repo maintenance

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

ℹ Additional Information

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Typescript error accessing this.$router.options

Version
3.0.1
Reproduction link
https://github.com/peterssonanton/vue-example
Steps to reproduce
npm run build
Gives this error:
ERROR  Failed to compile with 2 errors                                                                                08:25:37
error  in /Users/antonpetersson/my-project/src/views/Home.vue
(25,30): Property 'options' does not exist on type 'VueRouter'.
error  in /Users/antonpetersson/my-project/src/views/Home.vue
(26,25): Property 'options' does not exist on type 'VueRouter'.
What is expected?
The options property exists on VueRouter, if running 'npm run serve' the app starts (with typescript errors still) and is working.
What is actually happening?
Typescript is giving a compilation error

The example repo is created with the vue-cli: https://github.com/vuejs/vue-cli with only a few lines added code in Home.vue to reproduce the error

AFAIK, $router.options is a private api so we don't include it in typings.

Maybe I misunderstood then, but is there no way to get the routes from the router object? ($router.options.routes)

You can put them into a variable and export that variable around instead.

Sorry, my mistake. Thanks for the help

Sorry, my mistake. Thanks for the help

You can put them into a variable and export that variable around instead.

Emm, what if I am a plugin maker? I meen I am providing components, and I cannot hope user to pass their options to me?
Besides I also notice that lots of plugin are using this private api, such as antd and vuepress.

My solution was to make a separate file for the page routes like this;
export default [{
  path: 'users/:user?',
  name: 'Users'
  component: () => import(/* webpackChunkName: "main" */ '../views/Pages/Users.vue')
}, {
  path: 'organizations/:organization?',
  name: 'Organizations',
  component: () => import(/* webpackChunkName: "main" */ '../views/Pages/Organizations.vue')
}]

Then when I need to access the pages I do this;
import pageRoutes from '@/router/pages'

export default class Navbar extends Vue {
  get pages () {
    return pageRoutes
  }
}

Works for me
import VueRouter, {
  RouterOptions,
} from 'vue-router';

class VueRouterExtended extends VueRouter {
  options?: RouterOptions
}

Works for me
import VueRouter, {
  RouterOptions,
} from 'vue-router';

class VueRouterExtended extends VueRouter {
  options?: RouterOptions
}

I don't get it. Does accessing options.routes work for you by creating a new subclass? How will options prop on VueRouter suddenly become public because you extend from it? options prop is just undefined with no value.


Merge 0.12

might be a cleaner way to do this, but 0.12 is fully released so I think this should be good to go as well. I fixed the small merge conflicts

@dermotduffy not sure what the error is here (or more specifically, why it shows here but not on the frigate 0.12 branch), do you have any ideas?

@NickM-27 Hmm. No idea, sorry. Types look compatible to me. Does this pass locally for you?

@NickM-27 Hmm. No idea, sorry. Types look compatible to me. Does this pass locally for you?

mypy works succssfully for me locally

Is it possible to re-run the failed job or push an empty commit to run the whole pipeline again?

same issue

Update your local mypy (pip install --upgrade mypy) and I bet you'll have the same issue (I do).

Likely related: https://github.com/python/mypy/pull/14526
Suggest for now pegging mypy with mypy==1.1.1 in requirements_dev.txt.

@dermotduffy looks like that did it

Thanks @NickM-27 !


Remove reload_kedro_mlflow line magic

Description
The reload_kedro_mlflow line magic is used to expose a mlflow_client in jupyter notebooks automatically.
Context

To my knowledge, this is not used by anyone. People are likely not aware that it exists. Please comment if you do.
Whatever I do, kedro will remove this entrypoint in 0.18.3, so it will stop working.

To avoid code debt, this will be removed in future versions, and will not be considered a breaking change.
Possible Implementation
Remove:

the entire extras folder
the setup.py entrypoint
and all according tests

Note for anyone who wants to access the mlflow client in a notebook:  loading the kedro extension will automatically creates a context, which setups mlflow. The _mlflow_client is a private attribute you can access directly:
%load_ext kedro.ipython
context.mlflow.server._mlflow_client


galax

Contributor: [galaxy]
SSH Public key: [
git clone https://github.com/zircuit-labs/ceremony.git
cd ceremony
ssh-keygen -t ed25519 -b 256 -f ./id_ceremony -N "" -C ""
cat ./id_ceremony.pub]
Attestation: [any form of attestation for contribution (e.g., signature) - Optional]

Contributor: [galaxy]
SSH Public key: [
git clone https://github.com/zircuit-labs/ceremony.git
cd ceremony
ssh-keygen -t ed25519 -b 256 -f ./id_ceremony -N "" -C ""
cat ./id_ceremony.pub]
Attestation: [any form of attestation for contribution (e.g., signature) - Optional]

When notification to start ?

I am ready

i have started


Grocery UI template for flutter

Description
Added grocery UI template, Has 2 screens

Dashboard or landing screen
Item Details screen

Demo

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Manual testing

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@SmartAree Could you please review the above PR? This PR is for adding a grocerry UI.

But why are you adding this here?? you have to do it in the original repository

My bad, I should have seen this.

@vishalkevin11  its ok :>


Type definitions are wrong for REST helper

What are you trying to achieve?
Use the REST helper in a typescript project
What do you get instead?
Typescript compile errors because the provided type definitions for the REST helper do not match the actual method signatures.
The _executeRequest method and all of the send* methods are async and return the response object, however the type definition marks them as returning void. This means that in my code using these methods if I actually try to use the response object, typescript throws an error: Type 'void' is not assignable to type 'AxiosResponse<any>'
In addition, the arguments for each method are all marked as being required, even though most of them are optional, which means I have to pass undefined for each argument to prevent typescript compiler errors.

Example showing both of the above issues

import { AxiosResponse } from 'axios';
const res: AxiosResponse = await this.sendPostRequest('', {});

helpers/steps_file.ts:46:13 - error TS2322: Type 'void' is not assignable to type 'AxiosResponse<any>'.

46       const res: AxiosResponse = await this.sendPostRequest('', {});
               ~~~

helpers/steps_file.ts:46:45 - error TS2554: Expected 3 arguments, but got 2.

46       const res: AxiosResponse = await this.sendPostRequest('', {});
                                               ~~~~~~~~~~~~~~~~~~~~~~~

Details

CodeceptJS version: 2.6.10
NodeJS Version: 14.11.0
Operating System: Windows 10

@DavertMik @PeterNgTr Is this something that could be looked at? If you point me to the right direction I would be open to make a PR!

Hi @vdhpieter contribution is always welcome 🙏

@PeterNgTr Ok, I took a quick look around the library and couldn't find where the typings are defined, are they generated? If so based on what?

@vdhpieter i guess the types are generated by jsdocs

@PeterNgTr I made a PR but for some reason my JSDoc is not correct although every resource I check does it in the exact same way... Could you maybe have a look?

@vdhpieter did you run npm run docs to generate the docs? I guess you miss that part.

@PeterNgTr I tried that but because detox won't install it's not working :( Secondly I also get this error:
ERROR: Unable to parse a tag's type expression for source file /Users/pieter/Repos/CodeceptJS/docs/build/REST.js in line 74 with tag title "returns" and text "{import('axios').AxiosResponse}": Invalid type expression "import('axios').AxiosResponse": Expected "!", "=", "?", "[]", "|", or end of input but "(" found.

So I managed to install the packages and I think I was trying something impossible. using the typing from Axios in the JSDoc. So for now I just made it return a Promise<any> which is not perfect but already better than just void


chore: ockam_vault_core builds again with no-std feature

Since the stabilisation of the const-generics-mvp, the heapless crate removed the heapless::consts module with their 0.7.0 version.
Some changes should be done here in order to take this into account since the Cargo manifest contains said version.
The serde feature for heapless seems to be needed when enabling the no-std feature.
An implementation of From<&str> for heapless:String<U64> seems to violate the orphan rule (please correct me if I am wrong) and also it is in conflict with the
impl<'a, N> From<&'a str> for heapless::String<N>

from the heapless crate.
Proposed Changes

Change the generic parameters for the heapless crate structs
Add the serdefeature to the heapless dependency
Remove the From<&str> implementation for KeyId

Hey @devalain, changes look great to me. @jared-s @mrinalwadhwa what is our policy for rust version support? Do we always target latest? If so, we're good merging this PR

Latest stable is okay for now.  At some point we'll want to consider an MSRV, but for now we can ignore that constraint.
@SanjoDeundiak could you push this to a branch to see if CI succeeds ... the rustc version in the builder image is 1.51.0

@mrinalwadhwa after rebase and lock files update it succeeded. It's here

Merged as https://github.com/ockam-network/ockam/pull/1600 - Thank you!

@devalain Congratulations and thank you for your first Ockam contribution 🎉


make this a standalone ./bin tool

I want to be able to use this as follows:
npm install -g source-map-resolve
source-map-resolve file.js

thanks

Hi!
I won’t add a CLI in this repo, because I don’t need it myself and I don’t want to maintain it.
Maybe this one is good? https://www.npmjs.com/package/source-map-cli?activeTab=dependencies
If you end up making a CLI yourself, feel free to post here so I can link to it!


[Draft] Add Get/Set Attribute methods for sync routing panel

What does this Pull Request accomplish?
Add the following methods for sync routing panel:

GetAttributeViInt32
SetAttributeViInt32
GetAttributeViString
SetAttributeViString

Why should this Pull Request be merged?
The attribute methods are used by sync routing panel with linuxRT target
What testing has been done?
[ RUN      ] NiSyncDriverApiTest.AttributeSet_GetAttributeViInt32_ReturnsValue
[       OK ] NiSyncDriverApiTest.AttributeSet_GetAttributeViInt32_ReturnsValue (7 ms)
[ RUN      ] NiSyncDriverApiTest.AttributeSet_GetAttributeViString_ReturnsValue
[       OK ] NiSyncDriverApiTest.AttributeSet_GetAttributeViString_ReturnsValue (7 ms)

I'm going to be out of office the rest of the work so don't wait for me.  I did a quick cursory glance and this looks OK to me so I'll go ahead and approve.

I like the changes. I would like to see this once PR #169 completes.

merged in changes fro PR #169


How to get an items list of a combobox ?

I have following xpath for expanded combobox element from UI Inspector
"/Pane[@ClassName="#32769"][@Name="Рабочий стол 1"]/Window[@ClassName="QComboBoxPrivateContainer"][@Name="mmwin"]/List[@ClassName="QComboBoxListView"]/ListItem[@Name="element name"]"
ListItem @Name attribute can have any text value
How to programmatically get "/ListItem" @Name attribute ? Is it possible to get all "QComboBoxListView" elements as a list?

unfortunately the combo box are not part of the same application tree for windows form, it opens up as a separate window, you need to create a desktop seesion and swith to that window and get the list, is your application developed in windows form.

unfortunately the combo box are not part of the same application tree for windows form, it opens up as a separate window, you need to create a desktop seesion and swith to that window and get the list, is your application developed in windows form.

I use Qt framework to build the gui. How to get the list from new desktop session?

I use Qt framework to build the gui. How to get the list from new desktop session?


Installer not working

Describe the bug
double click on installer to open, cmd opens for a second and closes right after and installer gets deleted
To Reproduce
open installer
Expected behavior
cmd should open and i should be able to choose where to install trainer

and installer gets deleted

We do not do this. Are you sure you downloaded the installer from this exact site ?
Try to run a command prompt first (cmd), the run the installer from it so we can see if we can spot messages/errors.
Thanks


(draft) port the recorder into vuejs

This is a draft PR, meaning it's ready for review, but incomplete.

[x] port the choice screen.
[ ] port the start screen.
[ ] implement the recorder javascript.
[ ] port the video screen.

??? @LoganTann

Closing this PR
Terminating the port to VUEJS


Paged Attention NVCC compile issue

Describe the bug
When running cargo run with latest code, I'm getting this issue with the paged attention code:
  = note: /usr/bin/ld: /home/ec2-user/mistral.rs/mistralrs-paged-attn/libpagedattention.a(pagedattention-b5087c446beb6afd.o): relocation R_X86_64_32 against `.bss' can not be used when making a shared object; recompile with -fPIC
          /usr/bin/ld: /home/ec2-user/mistral.rs/mistralrs-paged-attn/libpagedattention.a(reshape_and_cache_kernel-160d2c98493955ed.o): relocation R_X86_64_32 against `.bss' can not be used when making a shared object; recompile with -fPIC

I tried updating the build.rs under mistral.rs/mistralrs-paged-attn with the NVCC flag code and passing fPIC flag, but still having an issue.
OS info:
NAME="Amazon Linux"
VERSION="2"
ID="amzn"
ID_LIKE="centos rhel fedora"
VERSION_ID="2"
PRETTY_NAME="Amazon Linux 2"
ANSI_COLOR="0;33"
CPE_NAME="cpe:2.3:o:amazon:amazon_linux:2"
HOME_URL="https://amazonlinux.com/"
SUPPORT_END="2025-06-30"
I'll continue troubleshooting, but I know that this has happened multiple times.
Latest commit
5503887

I got it to work. Needed to add the target_family to the CUDA_NVCC_FLAGS. Here is the PR: https://github.com/EricLBuehler/mistral.rs/pull/604

Ah great! I just merged #604.

Thanks! I did a clean clone and build. It worked without issue.
I have one concern:
2024-07-22T19:18:18.930302Z  INFO mistralrs_core::paged_attention: Allocating **17325 MB** for PagedAttention KV cache
2024-07-22T19:18:18.930315Z  INFO mistralrs_core::paged_attention: Using PagedAttention with block size 32 and 2772 GPU blocks: available context length is 88704 tokens

That paged attention allocates a lot of memory. I know you mentioned before that splitting the KV cache may not be possible, but have you heard of RingAttention (https://arxiv.org/abs/2310.01889)? I'm just not sure if it's possible without having a ring topology (like they mention in the paper).
I've been trying to work on it, but it's taking me a lot of time. Let me know your thoughts and I'll try to devote more time to it. Also, I'd probably need some guidance on how to add it to the project.

That paged attention allocates a lot of memory.

We provide 2 ways to control this: by setting the paged attention memory usage percentage (defaulting to 90%), or by setting an exact count in MBs. Perhaps you could try that?

I know you mentioned before that splitting the KV cache may not be possible, but have you heard of RingAttention (https://arxiv.org/abs/2310.01889)? I'm just not sure if it's possible without having a ring topology (like they mention in the paper).

That seems super interesting! I'll take a look at it.

I've been trying to work on it, but it's taking me a lot of time. Let me know your thoughts and I'll try to devote more time to it. Also, I'd probably need some guidance on how to add it to the project.

I think this would be an amazing addition - It would be a great strategy to have for users with multiple GPUs! To make it easier adding RingAttention to this project, I would recommend that you add it to one model at first, create the PR, and then we can work on abstracting it out.
Please feel free to reach out to me directly if you would like if you have any questions, either here (if you create a tracking issue), or via DM from the Discord.

Great! Let me add a new issue to track it and I'll close this one.


KeyError exception when trying to access a streamfield in a hook

Issue Summary
I have a page model SimplePage with a content streamfield.
Everything works fine with that page model in usual cases except when I try to access it from within a hook.
There it will raise a KeyError exception.
If I fetch the page from DB again it will work.
Note that doing a refresh_from_db will not fix the issue, only a plain objects.get will.
The above description shown in action in my debugger:
(Pdb++) page
<SimplePage: test>
(Pdb++) page.content
*** KeyError: 'content'
Traceback (most recent call last):
  File "/home/myproject/.local/share/virtualenvs/app-4PlAip0Q/lib/python3.7/site-packages/wagtail/core/fields.py", line 34, in __get__
    return obj.__dict__[self.field.name]
(Pdb++) page._meta.model.objects.get(pk=page.pk).content
[<wagtail.core.blocks.stream_block.StreamValue.StreamChild object at 0x7fba2f43fb10>]

Steps to Reproduce

Create a Page model with a simple streamfield
Setup a hook and try to access that streamfield

@hooks.register("register_page_listing_buttons")
def page_custom_listing_buttons(page, page_perms, is_parent=False):
    page.my_stream_field

It should raise.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: (no)

Technical details

Python version: 3.7.4
Django version: 2.2.4
Wagtail version:2.6.1
Browser version: not relevant

Thanks for the report @johnraz!
Looks like this is caused by the use of specific(defer=True) when fetching the page queryset. The same thing can be reproduced on the command line, from a fresh Wagtail install with a body StreamField added to the HomePage model:
./manage.py shell
Python 3.5.2 (default, Nov 12 2018, 13:43:14)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> from wagtail.core.models import Page
>>> Page.objects.filter(depth=2).specific()[0].body
[]
>>> Page.objects.filter(depth=2).specific(defer=True)[0].body
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/home/vagrant/.virtualenvsv/tableblock/lib/python3.5/site-packages/wagtail/core/fields.py", line 34, in __get__
    return obj.__dict__[self.field.name]
KeyError: 'body'

Also worth noting that the error doesn't occur if HomePage.body is a plain TextField. My guess is that obj.__dict__['some_field'] isn't valid for deferred objects, and needs to be wrapped in a method call that catches that case and fetches from the database where necessary.


template: add link to query merged github PRs

Quick way to find the recent PRs with a lot of activity. I always find myself searching for this query in my browser history, so figured adding it to the template would be helpful.

Pretty useful, thanks :+1:


Updated Erlang OTel dependencies to 1.3.0 per #924

Changes
Updated Erlang OTel dependencies to 1.3.0 per #924
Merge Requirements
For new features contributions please make sure you have completed the following
essential items:
* [ ] CHANGELOG.md updated to document new feature additions
* [ ] Appropriate documentation updates in the [docs][]
* [ ] Appropriate Helm chart updates in the [helm-charts][]

This isn't building on my m2 mac
Also would you mind re-opening from a personal fork @alexh97? We can't force update protected branches / org repos
Thanks for the contribution!

@alexh97 as we haven't heard back from you, we are closing this PR.
Please feel free to open a new one if you would still like to contribute.


Enhance serialization / deserialization

This ticket involves adding first-class support for EJSON to Quasar, and improving the ability of connectors to offer higher-performance, content-type-specific serialization/deserialization.

[ ] Add support for text/ejson for input and output on all REST APIs.
[ ] Allow connectors to return and accept a content type (such as JSON or EJSON) as a byte stream so they don't have to go through unnecessary intermediate layers.

Probably depends on #1814, #1813.


chore(deps): bump octokit and @octokit/graphql to fix seed-github-enterprise-server-instance seed script

Closing as this creates TypeScript issues that I can't be bothered to fix!


Investigate and update jblas matrix library for java

The fact that jblas cannot support boolean arrays causes all sorts of problems. We need to use a library that can deal with matrices/arrays in a better way.
These have to be worked around in particular in the meta examples at the moment, see e.g. #3550
Requires to change the typemaps....

I can ask Mikio Braun, if he'd intend to update JBLAS supporting boolean arrays…

That is an idea.
Though I don't Like the fact that jblas is turning everything into a double..... there must be an ndarray implementation for java no?

we do support org.ujmp as well, so that should be enough....


README needs updated "Translation" -> "TranslationServiceClient"

There are different exports in the latest version on npm:
node -i
> require('@google-cloud/translate');
{ v3beta1:
   { TranslationServiceClient: [Function: TranslationServiceClient] },
  v2: { Translate: [Function: Translate] },
  v3:
   { TranslationServiceClient: [Function: TranslationServiceClient] },
  TranslationServiceClient: [Function: TranslationServiceClient] }

@JustinBeckwith Any feedback on https://github.com/niftylettuce/mandarin btw?  Would love to hear your thoughts.

This looks super cool!  Sorry for the major bump here recently.  v3 is looking pretty powerful though.