Reading Gripper position with RWSStateMachineInterface

Hi,
Is it possible to read the Grippers positions with RWSStateMachineInterface using StateMachins-Addin similarly to g_GetPos() in RAPID? in order to get the grippers positions and publish them later to the /joint_states to be seen by MoveIt? thanks in advance.

@jontje I see the g_GetPos() method has been added to the EGM/RWS. (I had not made any changes to the main RAPID files. No custom module added.) I can get the g_GetPos printed on the Flex Pendant, but it is confusing how to publish them as a separate ROS topic like joint_states. Would you please suggest a solution to this problem?


minor refactoring

Hi, I've removed duplication of code where ever possible, centralised the appender implementation at CloudwatchAppender. So, CloudwatchLogsLog4j2Appender & CloudwatchLogsLogbackAppender is using CloudwatchAppender, which makes the code much easier to maintain. As in future we need to modify appender logic at one place. I've also created LogEventWrapper which wrap different LoggingEvent to a common interface. That really make it easy to integrate any new Logging api, as we just need to implement the LogEventWrapper interface.

Hi @axelfontaine , no this is not solving any current issues. just refactored the existing code.

Since this actually adds 200 lines of code to reduce what is currently very little duplication without offering any tangible advantages, we prefer not to merge this.
For future pull requests, please post an issue describing your intentions first to give us a chance to give you feedback before putting effort in the implementation.


GDScript 2.0: Named Enums cannot be set to use Flags when exported

Godot version
Godot 4.0 beta 11 x64 Windows
System information
Windows 11
Issue description
When exporting named enums in 3.5.x you could define them outside of the export declaration and then export them with flags. The enum could be defined within the file you were working on, or in an autoload file.
For example, this works in 3.5
Enum defined in an autoload file named Types:
enum ColorTypes{
HUMANSKIN,
HUMANHAIR,
RED,
BLUE,
YELLOW,
GREEN,
VIOLET,
ORANGE,
REDORANGE,
YELLOWORANGE,
YELLOWGREEN,
BLUEGREEN,
BLUEVIOLET,
REDVIOLET,
BROWN,
GREY,
BLACK,
WHITE}

extends Resource

class_name DefinedColor

export (Types.ColorTypes, FLAGS) var colorType

export (Types.ColorValue) var colorValue

export (String) var name

export (Color) var vColor

tldr for character generation I want to create predefined colors, and then flag them with what category they fall in, and whether the naturally occur as a hair or skin color. I'd also want to do this for other named enums as well.
And this works well in 3.5.x- when defining a export var using a named enum there are checkboxes available I can use to set the var up in editor where in a node or custom resource. However in 4.0 beta that no longer seems to be the case
As shown in the documents, we can do this for enums if we define them during the export
@export_flags("Fire", "Water", "Earth", "Wind") var spell_elements = 0
We should have the same functionality for Named Enums in 4.x
My attempt at implementing in Godot 4.0 beta 11 in my project, to be clear not messing with the editor's code, just playing around to see if the functionality is there:

class_name DefinedColor
#The following two ways I tried don't work and I wasn't able to find how to do it in the documentation so I believe this is missing functionality

@export_flags(Types.ColorTypes) var colorType

@export(Types.ColorTypes, FLAGS) var colorType2

@export var colorValue: Types.ColorValue

@export var name: String@export var vColor: Color

Steps to reproduce

Create a new Godot project
Create a new script extending resource or node
Create a named enum with multiple options
Attempt to create an exported variable using the previously created named enum with flags

Minimal reproduction project
N/A

Is there a workaround for this until it's added? I used to rely on this in Godot 3.x. I guess our best bet for now is probably just hardcoding :mask: the bit flags :nauseated_face: for now :vomiting_face:.

I think it should be implemented as a separate annotation:
@export_flags_enum(named_enum: Dictionary, values_as_bits: bool)

enum Enum1 {A, B, C, D} # 0, 1, 2, 3
enum Enum2 {A = 1, B = 2, C = 4, D = 8}

@export_flags_enum(Enum1, true) var var1: int
@export_flags_enum(Enum2, false) var var2: int

Or we can have an annotation for the enum declaration to make it auto-assign values using power of 2 instead of linearly? Because that would make those enums useful for scripting in general, not just in the export notation.
Either way, this is a proposal material, so it should be argued as a feature proposal.


Add teslamate docker guide

I found that the Teslamate config documentation is not detailed enough, which can lead to some confusion.
Therefore, I have added a supplementary guide for users of Teslamate Docker.

Wonderful. Thank you!


docker compose up: execution expired (Net::OpenTimeout)

Have you checked that your issue isn't already filed?

[X] I read through FAQ and searched through the past issues, none of which addressed my issue.
[X] Yes, I have checked that this issue isn't already filed.

Bug description
docker compose up: execution expired (Net::OpenTimeout)
How to reproduce the bug
just follow the INSTALL.md:

install wsl in windows 11
install Ubuntu by wsl --install -d Ubuntu
install docker & docker-compose
clone this repo & run docker compose pull, docker compose up

Error messages and logs
when I run docker compose up:
[+] Running 1/0
 ✔ Container chancodygithubio-jekyll-1  Created                                                                    0.0s
Attaching to jekyll-1
jekyll-1  | Resolving dependencies...
jekyll-1  |   Logging at level: debug
jekyll-1  |     Jekyll Version: 4.3.3
jekyll-1  | Configuration file: /srv/jekyll/_config.yml
jekyll-1  |   Logging at level: debug
jekyll-1  |     Jekyll Version: 4.3.3
jekyll-1  |          Requiring: /srv/jekyll/_plugins/cache-bust.rb
jekyll-1  |          Requiring: /srv/jekyll/_plugins/details.rb
jekyll-1  |          Requiring: /srv/jekyll/_plugins/external-posts.rb
jekyll-1  |          Requiring: /srv/jekyll/_plugins/file-exists.rb
jekyll-1  |          Requiring: /srv/jekyll/_plugins/google-scholar-citations.rb
jekyll-1  |          Requiring: /srv/jekyll/_plugins/hideCustomBibtex.rb
jekyll-1  |          Requiring: /srv/jekyll/_plugins/remove-accents.rb
jekyll-1  |          Requiring: jekyll-archives
jekyll-1  |          Requiring: jekyll-email-protect
jekyll-1  |          Requiring: jekyll-feed
jekyll-1  |          Requiring: jekyll-get-json
jekyll-1  |          Requiring: jekyll-imagemagick
jekyll-1  |          Requiring: jekyll-jupyter-notebook
jekyll-1  |          Requiring: jekyll-link-attributes
jekyll-1  |          Requiring: jekyll-minifier
jekyll-1  |          Requiring: jekyll-paginate-v2
jekyll-1  |          Requiring: jekyll/scholar
jekyll-1  |          Requiring: jekyll-sitemap
jekyll-1  |          Requiring: jekyll-toc
jekyll-1  |          Requiring: jekyll-twitter-plugin
jekyll-1  |          Requiring: jemoji
jekyll-1  |             Source: /srv/jekyll
jekyll-1  |        Destination: /srv/jekyll/_site
jekyll-1  |  Incremental build: disabled. Enable with --incremental
jekyll-1  |       Generating...
jekyll-1  |            Reading: /_layouts/about.liquid
jekyll-1  |            Reading: /_layouts/archive-category.liquid
jekyll-1  |            Reading: /_layouts/archive-tag.liquid
jekyll-1  |            Reading: /_layouts/archive-year.liquid
jekyll-1  |            Reading: /_layouts/bib.liquid
jekyll-1  |            Reading: /_layouts/cv.liquid
jekyll-1  |            Reading: /_layouts/default.liquid
jekyll-1  |            Reading: /_layouts/distill.liquid
jekyll-1  |            Reading: /_layouts/none.liquid
jekyll-1  |            Reading: /_layouts/page.liquid
jekyll-1  |            Reading: /_layouts/post.liquid
jekyll-1  |            Reading: /_layouts/profiles.liquid
jekyll-1  |        EntryFilter: excluded /Gemfile
jekyll-1  |        EntryFilter: excluded /docker-compose-slim.yml
jekyll-1  |        EntryFilter: excluded /package-lock.json
jekyll-1  |        EntryFilter: excluded /LICENSE
jekyll-1  |        EntryFilter: excluded /FAQ.md
jekyll-1  |        EntryFilter: excluded /README.md
jekyll-1  |        EntryFilter: excluded /INSTALL.md
jekyll-1  |        EntryFilter: excluded /purgecss.config.js
jekyll-1  |        EntryFilter: excluded /docker-compose.yml
jekyll-1  |        EntryFilter: excluded /CUSTOMIZE.md
jekyll-1  |        EntryFilter: excluded /lighthouse_results
jekyll-1  |        EntryFilter: excluded /.jekyll-cache
jekyll-1  |        EntryFilter: excluded /readme_preview
jekyll-1  |        EntryFilter: excluded /Dockerfile
jekyll-1  |        EntryFilter: excluded /CONTRIBUTING.md
jekyll-1  |        EntryFilter: excluded /bin
jekyll-1  |        EntryFilter: excluded /Gemfile.lock
jekyll-1  |        EntryFilter: excluded /package.json
jekyll-1  |            Reading: _posts/2015-03-15-formatting-and-links.md
jekyll-1  |            Reading: _posts/2015-05-15-images.md
jekyll-1  |            Reading: _posts/2015-07-15-code.md
jekyll-1  |            Reading: _posts/2015-10-20-disqus-comments.md
jekyll-1  |            Reading: _posts/2015-10-20-math.md
jekyll-1  |            Reading: _posts/2018-12-22-distill.md
jekyll-1  |            Reading: _posts/2020-09-28-github-metadata.md
jekyll-1  |            Reading: _posts/2020-09-28-twitter.md
jekyll-1  |            Reading: _posts/2021-07-04-diagrams.md
jekyll-1  |            Reading: _posts/2022-02-01-redirect.md
jekyll-1  |            Reading: _posts/2022-12-10-giscus-comments.md
jekyll-1  |            Reading: _posts/2023-03-20-table-of-contents.md
jekyll-1  |            Reading: _posts/2023-03-21-tables.md
jekyll-1  |            Reading: _posts/2023-04-24-videos.md
jekyll-1  |            Reading: _posts/2023-04-25-audios.md
jekyll-1  |            Reading: _posts/2023-04-25-sidebar-table-of-contents.md
jekyll-1  |            Reading: _posts/2023-05-12-custom-blockquotes.md
jekyll-1  |            Reading: _posts/2023-07-04-jupyter-notebook.md
jekyll-1  |            Reading: _posts/2023-07-12-post-bibliography.md
jekyll-1  |            Reading: _posts/2023-12-12-tikzjax.md
jekyll-1  |            Reading: _posts/2024-01-26-chartjs.md
jekyll-1  |            Reading: _posts/2024-01-26-echarts.md
jekyll-1  |            Reading: _posts/2024-01-26-geojson-map.md
jekyll-1  |            Reading: _posts/2024-01-27-advanced-images.md
jekyll-1  |            Reading: _posts/2024-01-27-code-diff.md
jekyll-1  |            Reading: _posts/2024-01-27-vega-lite.md
jekyll-1  |            Reading: assets/css/main.scss
jekyll-1  |        EntryFilter: excluded /_pages/about_einstein.md
jekyll-1  |            Reading: _pages/news.md
jekyll-1  |            Reading: _pages/profiles.md
jekyll-1  |            Reading: _pages/blog.md
jekyll-1  |            Reading: _pages/repositories.md
jekyll-1  |            Reading: _pages/about.md
jekyll-1  |            Reading: _pages/publications.md
jekyll-1  |            Reading: _pages/projects.md
jekyll-1  |            Reading: _pages/teaching.md
jekyll-1  |            Reading: _pages/404.md
jekyll-1  |            Reading: _pages/cv.md
jekyll-1  |            Reading: _pages/dropdown.md
jekyll-1  |            Reading: robots.txt
jekyll-1  |            Reading: _news/announcement_1.md
jekyll-1  |            Reading: _news/announcement_2.md
jekyll-1  |            Reading: _news/announcement_3.md
jekyll-1  |            Reading: _projects/1_project.md
jekyll-1  |            Reading: _projects/2_project.md
jekyll-1  |            Reading: _projects/3_project.md
jekyll-1  |            Reading: _projects/4_project.md
jekyll-1  |            Reading: _projects/5_project.md
jekyll-1  |            Reading: _projects/6_project.md
jekyll-1  |            Reading: _data/coauthors.yml
jekyll-1  |            Reading: _data/cv.yml
jekyll-1  |            Reading: _data/repositories.yml
jekyll-1  |            Reading: _data/venues.yml
jekyll-1  |         Generating: JekyllGetJson::GetJsonGenerator finished in 0.0001286 seconds.
jekyll-1  |         Generating: Jekyll::Scholar::DetailsGenerator finished in 2.23e-05 seconds.
jekyll-1  | "Fetching external posts from medium.com:"
jekyll-1  | /usr/lib/ruby/3.0.0/net/http.rb:987:in `initialize': execution expired (Net::OpenTimeout)
jekyll-1  |     from /usr/lib/ruby/3.0.0/net/http.rb:987:in `open'
jekyll-1  |     from /usr/lib/ruby/3.0.0/net/http.rb:987:in `block in connect'
jekyll-1  |     from /usr/lib/ruby/3.0.0/timeout.rb:107:in `timeout'
jekyll-1  |     from /usr/lib/ruby/3.0.0/net/http.rb:985:in `connect'
jekyll-1  |     from /usr/lib/ruby/3.0.0/net/http.rb:970:in `do_start'
jekyll-1  |     from /usr/lib/ruby/3.0.0/net/http.rb:959:in `start'
jekyll-1  |     from /usr/lib/ruby/3.0.0/net/http.rb:1512:in `request'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/httparty-0.21.0/lib/httparty/request.rb:156:in `perform'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/httparty-0.21.0/lib/httparty.rb:612:in `perform_request'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/httparty-0.21.0/lib/httparty.rb:526:in `get'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/httparty-0.21.0/lib/httparty.rb:645:in `get'
jekyll-1  |     from /srv/jekyll/_plugins/external-posts.rb:14:in `block in generate'
jekyll-1  |     from /srv/jekyll/_plugins/external-posts.rb:12:in `each'
jekyll-1  |     from /srv/jekyll/_plugins/external-posts.rb:12:in `generate'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/site.rb:193:in `block in generate'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/site.rb:191:in `each'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/site.rb:191:in `generate'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/site.rb:79:in `process'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/command.rb:28:in `process_site'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/commands/build.rb:65:in `build'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/commands/build.rb:36:in `process'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/command.rb:91:in `block in process_with_graceful_fail'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/command.rb:91:in `each'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/command.rb:91:in `process_with_graceful_fail'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/lib/jekyll/commands/serve.rb:86:in `block (2 levels) in init_with_program'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `block in execute'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `each'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/mercenary-0.4.0/lib/mercenary/command.rb:221:in `execute'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/mercenary-0.4.0/lib/mercenary/program.rb:44:in `go'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/mercenary-0.4.0/lib/mercenary.rb:21:in `program'
jekyll-1  |     from /var/lib/gems/3.0.0/gems/jekyll-4.3.3/exe/jekyll:15:in `<top (required)>'
jekyll-1  |     from /usr/local/bin/jekyll:25:in `load'
jekyll-1  |     from /usr/local/bin/jekyll:25:in `<main>'

What operating system are you using?
Windows
Where are you seeing the problem on?
Running locally with Docker
More info
FYI:

When I run docker compose up, it hangs a few seconds at log:

jekyll-1  | "Fetching external posts from medium.com:"

I've tried docker compose up with and without proxy, and the error logs are the same.
windows version: Winows 11 Pro, 21H2, 22000.2538
wsl version: <IP_ADDRESS>, fresh installed with Ubuntu 22.04.3 LTS

I've searched through all the issues and FAQ, but not found any similar error.
I'm new to Ruby, web, jekyll and al-folio, so apologies if there is an obvious solution.

Probably your docker is not being able to access the internet. The easiest way to solve this would be to just comment these lines in your _config.yml:
# External sources.
# If you have blog posts published on medium.com or other external sources,
# you can display them in your blog by adding a link to the RSS feed.
external_sources:
  - name:
    rss_url:

@george-gca Thanks for your quick reply!
After commenting out these lines, another error occurs. I found the issue is due to network problems. I’m currently in China, where many websites can only be accessed through a proxy. Unfortunately, Windows proxy settings don’t work for WSL by default. Configuring WSL to use the Windows proxy can be a pain in a**. I’ve tried many methods, and here’s what works for me:

Set proxy in Ubuntu, following this blog
Set windows firewall to allow wsl to access the host machine, following this answer
DON"T FORGET TO REBOOT YOUR PC!

You can check if the set-up works by wget www.google.com, a index.html file should be downloaded.
After all these, I can set up my page locally by docker compose up. But still, I have to comment those "external" lines. I can access  medium.com in wsl by wget https://medium.com/@al-folio/feed, but docker compose up still hangs at jekyll-1  | "Fetching external posts from medium.com:, which is pretty weird.
Anyway, now I can custom my page locally! Happy coding!
Yet, I have another question: why does the "about" page not have a name part like other pages, which makes it looks weird:
"about" page:

"blog" page:

Happy that it worked for you.
This was a design decision. In the about page your name is already being shown as the first text in that page and in a larger font, so it would be kind of redundant.

Got it

@george-gca Thanks for your quick reply!
After commenting out these lines, another error occurs. I found the issue is due to network problems. I’m currently in China, where many websites can only be accessed through a proxy. Unfortunately, Windows proxy settings don’t work for WSL by default. Configuring WSL to use the Windows proxy can be a pain in a**. I’ve tried many methods, and here’s what works for me:

Set proxy in Ubuntu, following this blog
Set windows firewall to allow wsl to access the host machine, following this answer
DON"T FORGET TO REBOOT YOUR PC!

You can check if the set-up works by wget www.google.com, a index.html file should be downloaded.
After all these, I can set up my page locally by docker compose up. But still, I have to comment those "external" lines. I can access medium.com in wsl by wget https://medium.com/@al-folio/feed, but docker compose up still hangs at jekyll-1 | "Fetching external posts from medium.com:, which is pretty weird.
Anyway, now I can custom my page locally! Happy coding!
Yet, I have another question: why does the "about" page not have a name part like other pages, which makes it looks weird: "about" page:  "blog" page:

Thank you! I finally managed to access my localhost page with the help of your pieces of advice (especially the second one). Speaking about that issue with Medium, I've got exactly the same behavior both on Windows 10 and 11. Actually, I've been using this tool in order to set up a VPN on Windows 10 (WSL 2), and this "built-in" solution for Windows 11. In the described approaches, we consider that one turns on a VPN on the Windows side of an OS. However, as you see, I still haven't worked it out completely and am looking for existing approaches so as to be able to automatically load one's posts made on Medium to the corresponding personal website (meaning, using the external_sources field in the _config.yml file).
Getting back to the localhost access problem, it's worth noting that in the Windows 11 case, one should try installing a web browser inside the WSL and using the WSL's browser in order to open a localhost webpage. All the solutions for Windows 11 mentioned here haven't worked for me except this one.


Override the node command when using pnpm run

Frequently there are actions made with node in scripts. When dependencies are installed with pnpm, it is recommended to run node with the --preserve-symlinks option.
The easiest solution might be to override the run command in pnpm. The pnpm version of the run command would run all the node commands in scripts with the --preserve-symlinks option

seems fragile

but the only way, i think

There will be a better solution once this gets released: https://github.com/nodejs/node/issues/8509 (Configuration option to permanently turn on --preserve-symlinks)

so it will be in next node release?

in the 7th for sure. The 6th is under question for now.

6 is LTS now, so i doubt it

Here they say they will discuss it. If they decide not to include it, we can implement the hack solution for v6

I think we don't need --preserve-symlinks anymore. As of v0.47.0, Node.js 4 should work fine without preserve-symlinks. It is used on Node 6 and newer but can be turned off via the preserve-symlinks config.


Missing Resources

Similar to: https://github.com/asticode/go-astilectron-bundler/issues/87
Building in Ubuntu:
astilectron-bundler cc && astilectron-bundler -w
$ cat bundler.json
{
  "app_name":"testApp",
  "icon_path_linux": "resources/icon.png",
  "icon_path_windows": "resources/icon.ico",
  "version_astilectron": "0.37.0",
  "version_electron": "8.2.0"
}
$ ls ./resources/app/js
app.4aa14500.js
app.4aa14500.js.map
chunk-vendors.a69e73a7.js
chunk-vendors.a69e73a7.js.map
$ ls /tmp/astibundler/bind/resources/app/js
app.4aa14500.js
app.4aa14500.js.map
chunk-vendors.a69e73a7.js
chunk-vendors.a69e73a7.js.map

From Windows:
When I download the executable and run it:
Not allowed to load local resource: file:///D:/Users/${me}/AppData/Roaming/testApp/resources/app/index.html
D:/Users/${me}/AppData/Roaming/testApp/vendor exists but resources does not.
Similar issue when I build for linux.

I've several questions :

could paste output of ls  ./resources/app to see whether index.html exists?
did you add ${me} in D:/Users/${me}/AppData/Roaming/testApp/vendor for Github issues or is it the actual path it's trying to load?
can you try running the windows executable from the terminal and paste the output here?

background.png  css  favicon.ico  icon.png  index.html  js  main.css

I added ${me}
Running from the terminal launches the app but doesn't log anything more.
Here is my logger in main.go
// Application Vars
var (
	fs    = flag.NewFlagSet(os.Args[0], flag.ContinueOnError)
	debug = false // fs.Bool("debug", false, "enables the debug mode")
	w     *astilectron.Window
)

func main() {
	// Create logger
	l := log.New(log.Writer(), log.Prefix(), log.Flags())

	// Parse flags
	fs.Parse(os.Args[1:])

       ...

Thank you for looking into this.

Are you providing the logger to the boostrap options like here ?

yes I am,
...,
		Debug:  debug,
		Logger: l,
		MenuOptions: []*astilectron.MenuItemOptions{{
...

What logs are you getting on linux when you run the binary from the terminal ?

2021/07/26 15:50:36 Running app built at 2021-07-26 15:23:35.566987934 +0000 UTC m=+7.368436612
2021/07/26 15:50:36 astikit: starting worker...
2021/07/26 15:50:36 Starting...
2021/07/26 15:50:36 Provisioning...
2021/07/26 15:50:36 Astilectron has already been provisioned to version 0.37.0, moving on...
2021/07/26 15:50:36 Electron has already been provisioned to version 8.2.0, moving on...
2021/07/26 15:50:36 Listening...
2021/07/26 15:50:36 Executing...
2021/07/26 15:50:36 Starting cmd /home/me/Development/testApp/output/linux-amd64/vendor/electron-linux-amd64/electron /home/me/Development/testApp/output/linux-amd64/vendor/astilectron/main.js <IP_ADDRESS>:45509 true
2021/07/26 15:50:36 Astilectron says: {"name":"app.event.ready","targetID":"app","displays":{"all":[{"id":168,"bounds":{"x":0,"y":0,"width":1440,"height":900},"workArea":{"x":0,"y":31,"width":1440,"height":820},"accelerometerSupport":"unknown","monochrome":false,"colorDepth":24,"colorSpace":"{primaries:INVALID, transfer:INVALID, matrix:INVALID, range:INVALID}","depthPerComponent":8,"size":{"width":1440,"height":900},"workAreaSize":{"width":1440,"height":820},"scaleFactor":1,"rotation":0,"internal":false,"touchSupport":"unknown"}],"primary":{"id":168,"bounds":{"x":0,"y":0,"width":1440,"height":900},"workArea":{"x":0,"y":31,"width":1440,"height":820},"accelerometerSupport":"unknown","monochrome":false,"colorDepth":24,"colorSpace":"{primaries:INVALID, transfer:INVALID, matrix:INVALID, range:INVALID}","depthPerComponent":8,"size":{"width":1440,"height":900},"workAreaSize":{"width":1440,"height":820},"scaleFactor":1,"rotation":0,"internal":false,"touchSupport":"unknown"}},"supported":{"notification":true}}
2021/07/26 15:50:36 Sending to Astilectron: {"name":"window.cmd.create","targetID":"1","sessionId":"2","url":"file:///home/me/Development/testApp/output/linux-amd64/resources/app/index.html","windowOptions":{"backgroundColor":"#333333","center":true,"height":1000,"icon":"/home/me/Development/testApp/output/linux-amd64/resources/icon.png","resizable":true,"title":"TestApp","width":1200}}
2021/07/26 15:50:36 Stderr says: (electron) The default value of app.allowRendererProcessReuse is deprecated, it is currently "false".  It will change to be "true" in Electron 9.  For more information please check https://github.com/electron/electron/issues/18397
2021/07/26 15:50:36 Astilectron says: {"name":"window.event.move","targetID":"1"}
2021/07/26 15:50:36 Astilectron says: {"name":"window.event.resize","targetID":"1"}
2021/07/26 15:50:36 Astilectron says: {"name":"window.event.focus","targetID":"1"}
2021/07/26 15:50:36 Astilectron says: {"name":"window.event.did.finish.load","targetID":"1"}
2021/07/26 15:50:36 Sending to Astilectron: {"name":"menu.cmd.create","targetID":"3","menu":{"id":"3","items":[{"id":"4","options":{"label":"File"},"rootId":"app","submenu":{"id":"5","items":[{"id":"6","options":{"label":"Settings"},"rootId":"app"},{"id":"7","options":{"label":"About"},"rootId":"app"},{"id":"8","options":{"role":"close"},"rootId":"app"}],"rootId":"app"}}],"rootId":"app"}}
2021/07/26 15:50:36 Astilectron says: {"name":"menu.event.created","targetID":"3"}
2021/07/26 15:50:36 astikit: worker is now waiting...
2021/07/26 15:50:38 Astilectron says: {"name":"window.event.blur","targetID":"1"}
2021/07/26 15:50:38 Sending to Astilectron: {"name":"window.cmd.web.contents.open.dev.tools","targetID":"1"}

OK so this is working in linux?

for both windows and linux the application will launch, but the app is blank and within the console I can see that it fails to load index.html
For Linux: /home/me/Development/testApp/output/linux-amd64/resources/ does not get created
For Windows:  D:/Users/me/AppData/Roaming/testApp/resources does not get created.
In Linux if I create a symbolic link:
$ ln -svf /tmp/astibundler/bind/resources ./output/linux-amd64/resources
then the application works.
It seems to me that something is going wrong when resources is bound to the executable or when the resources folder is created by the executable. I tried to figure out where/how those things happen but haven't been able to yet.

In logs, between astikit: starting worker... and Starting... there should be a step where resources are disembedded, this is really weird 🤔 Can you paste your full func main() code?

I am so sorry for taking up your time. I had commented out:  RestoreAssets: RestoreAssets, because it was failing go vet in my test script. I have added it back in and resources is building correctly. Thank you so much for your help!

No worries, glad it's working now ❤


Fix the bloom filter synopsis deserialization

:notebook_with_decorative_cover: Description

:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on docs.tenzir.com/vast, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions

The CI on Mac fails with a known issue: https://app.clubhouse.io/tenzir/story/20882/race-condition-in-node-queries-unit-test
As discussed offline, I'm merging this now and will include the final style changes in a follow-up PR.


copy icon to where it is being used

copy the icon to resource deployment extension folder that has a dependency on this icon.

Coverage remained the same at 33.842% when pulling f358dd178baba8ed3b22f11a1cfede8b2299ebf7 on alanren/move-icon into dee02796c9f3bac13d1226c75904bb859183fb91 on master.


Allow wildcard in CORS "origins"

As per example for adding CORS on ambassador module we have the following setting :->

apiVersion: v1
kind: Service
metadata:
name: ambassador
namespace: development
annotations:
getambassador.io/config: |
---
apiVersion: ambassador/v0
kind: Module
name: ambassador
ambassador_id: ambassador-development
config:
cors:
origins: http://foo.example.com,http://bar.example.com
methods: POST, GET, OPTIONS, PUT
headers: Content-Type
credentials: true
spec:
ports:
- port: 80
selector:
service: ambassador
type: LoadBalancer
But as per ambassador document wildcard for "origins" does not seem to be feasible at this moment.
We would like to have cors "origins" support wildcard feature such that we can add :->

config:
cors:
origins: .example.com,.restservice.com

So that we don't need to care about allowing all the sub-domain and also whether allowed domain url is working on http or https

This should already be possible at a very high-level by setting origin: *.
The CORS specification does not support sending back wild card origins (e.g. *.example.org) so while we might be able to coerce Envoy into sending back wildcard origins it is undefined behavior for a browser adhering to CORS understands how to handle such an origin.
The solution most often employed by other HTTP proxies is to dynamically populate the Access-Control-Allow-Origin header when the request is received with the value of the Origin header. This should already be possible if you use an Ambassador External Auth Service. Please read: https://www.getambassador.io/reference/cors#authservice-and-cross-origin-resource-sharing
Reference Material

https://www.w3.org/TR/cors/ (the complete spec)
https://www.w3.org/TR/cors/#access-control-allow-origin-response-header (the ACAO header... interestingly the W3C calls out the ACAO as usually only allowing a single value rather than a list ... probably because of browser implementations)

Yes "origin: *" but we would like to have same for sub-domain. So you mean to say that CORS validation should be added on Auth-Service level other than on ambassador?

Apparently matching origins based on a regex is an upcoming feature in envoy 1.8, so instead of using "*.example.com", it would look something like ".*\\.example\\.com", I think?
Looking at the cors_filter's documentation, it would also need to define the new property allow_origin_regex.
In the meantime, it does seem like implementing it via extauth is the best way to go about it.

The solution most often employed by other HTTP proxies is to dynamically populate the Access-Control-Allow-Origin header when the request is received with the value of the Origin header. This should already be possible if you use an Ambassador External Auth Service.

@plombardi89 how would this be possible with an external auth service ?
Access-Control-Allow-Origin is a response header and the request flows through the ext auth service but the response does not, am I mistaken ?
I can add whatever header I want for the request in the auth service but that won't help as only the upstream api and ambassador will be able to add response headers...

@mderazon Yeah....thats right, are there any other way to do it now ?

Would it be possible to allow wildcard[ Eg: https://*.foo.com] in orgin?


Introduce a custom ApolloResponse.toString()

Use case
Since ApolloResponse changed to being a standard class (I think it used to be a data class?), the toString() now is not very useful because it prints the default hashcode / instance code based toString.
Can we introduce a custom toString for ApolloResponse? While printing all of data does not make sense, I'm trying to think what might be useful -- perhaps the cache key?
Describe the solution you'd like
No response

Agreed. This is also recommended by the Kotlin API guidelines. While I wouldn't put toString() on each and every class, ApolloResponse is central enough that it deserves one.


Typo "Azure blob storage"→"Azure Blob Storage"

https://docs.microsoft.com/en-us/sql/integration-services/control-flow/azure-blob-upload-task?view=sql-server-ver15

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The change aligns with current guidance in the internal cloud style guide; merging.

The change aligns with current guidance in the internal cloud style guide; merging.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix false positive Confusing Ordering warning

Fixes #574 .
I've also noticed a vulnerabilities warning from NPM:
$ npm i

><EMAIL_ADDRESS>prepare
> npm run build

><EMAIL_ADDRESS>build
> del-cli lib && tsc -p tsconfig.lib.json

added 1031 packages, and audited 1032 packages in 40s

147 packages are looking for funding
  run `npm fund` for details

7 vulnerabilities (6 moderate, 1 high)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

I've run npm audit fix which updated some packages in package-lock.json and fixed the 1 high vulnerability, but 6 moderate still remain:
$ npm audit fix

changed 3 packages, and audited 1032 packages in 1s

147 packages are looking for funding
  run `npm fund` for details

# npm audit report

got  <11.8.5
Severity: moderate
Got allows a redirect to a UNIX socket - https://github.com/advisories/GHSA-pfrx-2q88-qq97
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/got
node_modules/package-json/node_modules/got
  npm-name  <=6.0.1
  Depends on vulnerable versions of got
  node_modules/npm-name
    np  >=2.2.0
    Depends on vulnerable versions of npm-name
    Depends on vulnerable versions of update-notifier
    node_modules/np
  package-json  <=6.5.0
  Depends on vulnerable versions of got
  node_modules/package-json
    latest-version  0.2.0 - 5.1.0
    Depends on vulnerable versions of package-json
    node_modules/latest-version
      update-notifier  0.2.0 - 5.1.0
      Depends on vulnerable versions of latest-version
      node_modules/update-notifier

6 moderate severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

Codecov Report

Merging #575 (64443da) into main (c5dfb9f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #575   +/-   ##
=======================================
  Coverage   99.07%   99.08%
=======================================
  Files          12       12
  Lines         432      435    +3
  Branches       90       91    +1
=======================================
+ Hits          428      431    +3
  Misses          4        4

Impacted Files
Coverage Δ

src/umzug.ts
98.66% <100.00%> (+0.01%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Just noticed that you've approved but did not merge. Please let me know if you need anything from me. No rush, only want to make sure I don't block.
As for the error message, changing it is just a suggestion, I don't have a strong opinion.


https://angularjs.org/ sample wire up a backend with firebase

the versions of firebase and angularfire used in the sample are:

the current versions of firebase and angularfire are:

the firebase link opens to the old console https://www.firebase.com/ that will shutdown march 15th 2017
the new website is https://firebase.google.com/
when i repro the code locally i get a white screen with a project on new firebase, when i copy and paste the sample code the add end edit buttons don't work with no error message in the developer console.

i suggest we update this sample to the new versions of firebase and angular fire to ensure the developer has a good experience with firebase, on the other hand we are wired to fix things that are broken. this can also get them hooked. observed and read that a lot.

This issue is being rejected on the basis of violation of our Code of Conduct by the contributor.
More detailed explanation was emailed to the contributor.


Tests in app-template-svelte-typescript don't run

When you install the app-template-svelte-typescript and run yarn test, the test runner starts but does not complete
Original Discussion: https://github.com/snowpackjs/snowpack/discussions/1401
/cc @ebeloded

We're in the middle of moving app templates to use web/test-runner for this exact reason! Lets tackle this issue by just upgrading the template to use wtr (see React templates for what this looks like, post merge)

Fixed in 7f1690b2


ww new line after a link produces BRs

Version

1.2.2
Test Environment

ALL
Current Behavior

Insert a link and newlines in WYSIWYG editor
switch to markdown editor
see trailing
s

Expected Behavior

should convert
 to newline

We should treat ,  tags also.


WIP schema names

Summary of the changes / Why this improves CrateDB
Checklist

[ ] Added an entry in CHANGES.txt for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Please have another rough look @matriv, @seut. This time I am keeping the schema names unquoted and quote them when displaying.
Note that UppercaseSchemaNameIntegrationTest.java is ported from my second attempt to check the functionalities, and it will be removed.

This looks good.
Just keep in mind that we need to allow at least the _ character as the first char of quoted(delimited) identifiers as it is already known to be used by some tools, see https://github.com/crate/crate/issues/12110#issuecomment-1108628045.
But could be also a follow up change.
I think as another follow up (definitely not inside this scope) we could try if we can allow any characters inside quoted(delimited) identifiers beside double quotes itself as also allowed by other databases e.g. PG https://www.postgresql.org/docs/8.1/sql-syntax.html#SQL-SYNTAX-IDENTIFIERS and somehow a loose described at the SQL standard (e.g. https://crate.io/docs/sql-99/en/latest/chapters/02.html#delimited-identifier).

Is it possible to avoid the ITest and have unit tests for this instead?
Thanks for reviewing, @matriv. I agree that for the amount of changes here, the integration test seems big but I was not sure which units to test to make sure that we can use upper case letters.

Follow up:
https://github.com/crate/crate/pull/13528#issuecomment-1406215556
https://github.com/crate/crate/pull/13528#discussion_r1091664027


Type error in the wrong location

Shared Program: https://code.world/#Pt48lqUYMt6_Yt4KBK2c3CQ
Description
The type error points to the application of simulationOf, and the second argument, step.  Actually, the mistake was made in defining bounce.  That's two layers removed in the call stack.  This confuses students.
The fix is not obvious.  It's not clear how or if the computer should have known that the inferred types for bounce and step were not the intended ones.  Adding a type annotation to step would have caught this, but students rarely add type annotations.  One could argue that when a symbol is defined and used only once, the type of the single use site should be considered the type for the symbol, and we should prefer blaming the callee rather than the caller.  But that probably will confuse people, too.
Anyway, I'm recording this in case there's something to be done.

There's really nothing reasonable to do here.  Closing.


[MRTCore] Get OSS approval for new binary Microsoft.ApplicationModel.Resources.PriTasks.dll

This requires multiple steps:

Check in the code into the INTERNAL, PRIVATE repo.
Make changes to get approved.
Check in here.

What code is this that you are referring to? From the assembly name, I presume it is MSBuild tooling for generating PRI files.

Also: Would this entail releasing the source behind processing the priconfig.xml files? Ever since I started doing Desktop Bridge apps circa build 14393, they still remain incomprehensible to me. I have never been able to get them to work right, except when following someone else's tutorial. Thanks!

This work isn't planned anymore. Closing.


optimize Tree.from_list()

It'd be nice if we could figure out a way to optimize Tree.from_list(), taking more than 1s to loa d one .dir file.
        2    0.017    0.009    2.621    1.310 __init__.py:23(load)
        2    0.000    0.000    2.604    1.302 tree.py:175(load)
        2    0.563    0.281    2.452    1.226 tree.py:152(from_list)
   202605    0.090    0.000    1.433    0.000 <attrs generated init dvc_data.hashfile.diff.Change>:1(__init__)
   202605    0.228    0.000    1.343    0.000 diff.py:36(_)
   405207    1.044    0.000    1.191    0.000 meta.py:75(from_dict)
   202604    0.060    0.000    0.816    0.000 _make.py:1718(__ne__)
   202604    0.678    0.000    0.756    0.000 <attrs generated eq dvc_data.hashfile.diff.TreeEntry>:1(__eq__)
   405210    0.492    0.000    0.621    0.000 diff.py:103(_in_cache)
   405210    0.252    0.000    0.564    0.000 diff.py:94(_get)
       42    0.001    0.000    0.444    0.011 __init__.py:1(<module>)
   405207    0.231    0.000    0.326    0.000 hash_info.py:20(from_dict)
   810421    0.209    0.000    0.299    0.000 diff.py:26(__bool__)

@skshetry Do you have a more "high-level" use case where you can show how much of an impact it has?

@dberenbaum, Tree.load() is used to .dir file from cache.
Tree.from_list is a bit generic in that it can also load from files section in .dvc files.
Tree loading creates 2 objects: Meta, HashInfo, and this above profiling is for CelebA where we build 500k of those instances. You can see above that it takes 1.5s just to load that in memory.
Of course, we are not doing anything bad here. It's just Python being slow, we have already optimized this in terms of memory and performance. I opened this issue for exploring other ways to improve this.
This affects a lot of commands. Eg: data status has to load all .dir files in memory times 2 (from Git HEAD and from the hash in .dvc file). So a lot of time is lost just loading this in memory.

I played with Cython (which required 0 code changes), and got 20-30% improvement. But it will require a lot of infrastructure changes. It has to be 5x-10x improvement to be worth it here.


Use the govuk_sidekiq gem

https://trello.com/c/JQREOuvh/424-use-sidekiq-gem
Use the govuk gem to setup and initialize sidekiq.

Can you test that signon sends an email correctly in integration please?

Sure, I flagged "Ask user to setup 2-step-verification" in my account in
integration and I received an email.
Is that enough to test that?
On 27 June 2016 at 09:59, Paul Bowsher<EMAIL_ADDRESS>wrote:

Can you test that signon sends an email correctly in integration please?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alphagov/signonotron2/pull/501#issuecomment-228678002,
or mute the thread
https://github.com/notifications/unsubscribe/AGlrBZF-S516vnwfJdSTvgnLeK8GMmNhks5qP4LkgaJpZM4I8olQ
.

👍


Blue Pill: USB serial example fails to enumerate

All right, so I've been breaking my head for a while, so maybe it's time to ask for help...
IDK if this is the correct place to ask this, but as I'm using this HAL and this repo's examples...
I'm trying to get the USB Serial example to work (examples/usb_serial.rs).
My build environment is FreeBSD 12.1 amd64, programmer is STLink V2, target board is BluePill.
I have to add a line in .cargo/config, because Rust's LLD doesn't output any binary on FreeBSD:
diff --git a/.cargo/config b/.cargo/config
index 34ba36c..03bbff1 100644
--- a/.cargo/config
+++ b/.cargo/config
@@ -2,6 +2,7 @@
 runner = 'arm-none-eabi-gdb'
 rustflags = [
   "-C", "link-arg=-Tlink.x",
+  "-C", "linker=ld.lld",
 ]

 [build]

Using this cargo config works fine if I flash the blinky example, but when I try to flash the usb_serial example, disconnect the STLink, connect an USB cable, USB enumeration fails with

On FreeBSD:

usb_alloc_device: set address 5 failed (USB_ERR_IOERROR, ignored)
usbd_setup_device_desc: getting device descriptor at addr 5 failed, USB_ERR_IOERROR
usbd_req_re_enumerate: addr=5, set address failed! (USB_ERR_IOERROR, ignored)
ugen0.5: <Unknown > at usbus0 (disconnected)
uhub_reattach_port: could not allocate new device

On Linux:

usb 2-1.1: device not accepting address 19, error -32
usb-2-1-port1: device descriptor read/64, error -32

I tried

With OpenOCD + GDB:$ openocd -f interface/stlink-v2.cfg -f target/stm32f1x.cfg &
$ cargo run --features="stm32f103 rt stm32-usbd" --example usb_serial --release

With OpenOCD only:$ cargo build --features="stm32f103 rt stm32-usbd" --example usb_serial --release
$ openocd -f interface/stlink-v2.cfg -f target/stm32f1x.cfg -c "program target/thumbv7m-none-eabi/release/examples/usb_serial verify reset exit"

With stlink:$ cargo build --features="stm32f103 rt stm32-usbd" --example usb_serial --release
$ arm-none-eabi-objcopy -O binary target/thumbv7m-none-eabi/release/examples/usb_serial usb_serial.bin
$ st-flash write usb_serial.bin 0x8000000

Nothing works...
I searched on the web, found nothing either.
I don't have a lot of experience on embedded programming. But I can't work out how to fix this.
As a side note, if I turn on the PC13 LED before the USB setup, and then turn it off just before the loop, the led blinks quickly once.
Any help would be greatly appreciated.

Great.
Absolutely great.
The problem wasn't software neither the BluePill.
Just use a good USB cable, and you're good to go...


Segmentation fault with go/delve

Hello,
I try to setup my Neovim for debugging with nvim-dap for go (with leoluz/nvim-dap-go) and python (with mfussenegger/nvim-dap-python). When starting dap-ui while debugging a go file my Neovim crashes.
The same steps are working when debugging a python file.

open a go file
toggle a Breakpoint with require'dap'.toggle_breakpoint()
start debugging with require'dap'.continue()
open dap-ui with require'dapui'.open()
Seg fault

Neovim nightly
delve 1.9.0
go1.19 darwin/arm64
MacOS 12.5 M1

Thanks for any hint!

I don't know how and why but it looks like it works now...


[UBS courier] Improvement to 'Phone number' field validation

Environment: OS: Windows 10 Pro, Browser: Google Chrome, Version 90.0.4430.212
Reproducible: Always
Build found: 27.05.2021
Actual result
Validation 'Phone number' field on 'Personal data' tab, UBS courier page is carried out by the country code and the number of characters allowed for the field only.

Expected result
Validating the 'Phone number' field is the connection of the library google-libphonenumber, which allows you to validate the phone number by country code, city, etc.
Related to:
User Story #2531

Code phone number is duplicated

Telephone number is displayed correctly with country and operator code


Linux documentation

This PR has setup documentation changes related to the usage of Linux.
I wasn't sure if it would have been better to just create a new section, or merge the Linux specific details with the Windows setup steps, so here's my first draft.

I was wondering how you actually build the release package. I thought I could have included the .so file for dolphin-memory-engine, instead or writing the long winded explanation about how to install the library.

I was wondering how you actually build the release package. I thought I could have included the .so file for dolphin-memory-engine, instead or writing the long winded explanation about how to install the library.

For this, same as you did, pip install dolphin-memory-engine. So, just for consistency, we could include that file in the release download for Linux rather than directing users to install it themselves. However, I would use Python 3.11 instead of 3.12 just because Python 3.12 isn't supported by AP yet.

I think I can run a GH Action to make a Mac and Linux build of the randomizer, so you won't need to direct users to run from source. Let me look into that and I'll get back to you.

For this, same as you did, pip install dolphin-memory-engine. So, just for consistency, we could include that file in the release download for Linux rather than directing users to install it themselves. [...]

Since linux can use the same AP world package as Windows, with the only difference being the dolphin-memory-engine library. I don't know if you wanted to create two separate packages, but I think it would be possible to just put both files in the dolphin-memory-engine folder and still only create one package, and that way the actual install step would be the same for linux and windows.
If you want to get both the windows and linux libraries, you can use a script similar similar to the following:
#!/bin/bash
mkdir tmp dolphin-memory-engine
pip install --target tmp/windows --platform win_amd64 --no-deps dolphin-memory-engine
mv tmp/windows/dolphin-memory-engine/_dolphin_memory_engine*.pyd dolphin-memory-engine/
pip install --target tmp/linux --platform manylinux2024_x86_64 --no-deps dolphin-memory-engine
mv tmp/linux/dolphin-memory-engine/_dolphin_memory_engine*.so dolphin-memory-engine/
rm -fr tmp

[...] However, I would use Python 3.11 instead of 3.12 just because Python 3.12 isn't supported by AP yet.

I think I can run a GH Action to make a Mac and Linux build of the randomizer, so you won't need to direct users to run from source. Let me look into that and I'll get back to you.

When it comes to the AP world I don't think it matters since I think Archipelago embed its own version of python which is 3.11. I'm asking people to install 3.12 since the ISO generator says it requires 3.12, so I didn't test to see if 3.11 works. But if there's a way to generate both a windows and linux build, then it would in fact simplify the install process and Linux users won't need to run the generator from source.

I've added a Linux build to the randomizer download. Could you please test that it works, and if so, update your setup guide to use that download rather than building from source?

At first glance, the build doesn't seem to work.
I'm looking into why it might not. I'll let you know what I find.

I've cleaned up the documentation following the changes we've talked about.

Could you also rebase to include the changes made in #dbdc884?

Could you also rebase to include the changes made in #dbdc884?

I just did it. It should now be up to date with your repo

Before merging we'll need to address this (https://github.com/tanjo3/tww_apworld/pull/1#issuecomment-1973891777) also, I don't know if you already included it in your pipeline, but I'd like to test it before we call done.

I wonder if it would be better to programmatically install dolphin-memory-engine? If TWW were included in the AP build, the requirements.txt would be picked up by ModuleUpdate. However, since you'd be running on a frozen build, the update doesn't run. But we could just include the code in that file separately to install requirements. Thoughts?

Sadly I don't know all that much about the inner workings of Archipelago nor python, so I don't know if I supply the insight you require.
But from what I can understand, the thing that would be "frozen" is Archipelago, right. In development (or during the production build), it actually fetches the dependencies using ModuleUpdate. Once the production build is done, then ModuleUpdate doesn't actually fetches any module. Right?
From what I can understand from the code in ModuleUpdate it might not actually work since it uses sys.executable and in since Archipelago embeds python and I don't think there's any executable anymore at this point I think. So it will probably fail. Also I'm not sure if the built project actually has the pip module in it. So for your solution to work, the embedded version of python would need to ship with pip, and either sys.executable would need to resolve to something which isn't None, or be able to call an install function directly from the shipped pip module (by importing it). Otherwise you would still need to require the user to install python. Also it would probably would still only be a temporary solution until your apworld becomes official.
Taking that into consideration, then dolphin-memory-engine probably should still be part of your requirements.txt since if your integration gets merged into the main repository and becomes official you would need it anyway.
But for now since you need to bundle your apworld on your side and ship it, I think shipping it next to your apworld file might be the best you can do. But to make it easier for you, I don't know if you have a build script to make your production build (I don't think I saw one). But you might want to consider creating one.

As a quick note. I saw the change you made to your README.md, I think you should try to direct your user to use pyenv (and pyenv-win) instead. At least on linux messing around with the system version of python using apt-get isn't all that recommended since it can breaks some system essential utilities. Also apt-get isn't portable to every linux distribution, since it would only apply to debian based linux distributions (Okay I know it's the what most people uses, but still).

Sounds good, I can make those changes to the README. And I guess I still to including a DME folder in releases for now, until TWW gets merged in one day, hopefully. Could you test this to make sure the .so I generated works for you? And yeah, I don't have a build script for this; just doing it manually.
tww_apworld_v2.2.0.zip

Would you like me to add a build script for you?

Sure, that'd be appreciated. Would this be a GitHub actions workflow? I can commit the sample YAML to the repository.

Let's move the conversation about the build script to #2.
I'll test your build once I'm off work. Then if things work as expected, we'll be able to merge this PR.

Sounds good, I can make those changes to the README. And I guess I still to including a DME folder in releases for now, until TWW gets merged in one day, hopefully. Could you test this to make sure the .so I generated works for you? And yeah, I don't have a build script for this; just doing it manually.
tww_apworld_v2.2.0.zip

Have I missed something? The archive you've attached doesn't have any .so file in it.

Oh, my bad. This should have it. tww_apworld_v2.2.0.zip

There's still only the .pyd file, no .so. If there's something we could always just close this PR with #2, since we'll need too validate that build too anyway.

Are you sure? I downloaded the zip myself to check and I see the .so file there. If you don't see it, then that's rather odd.

Okay, either I was very blind or something odd happened. But I re-downloaded it and this time it was there, and the package you made work as expected.

Alright, then I'll merge this, add the .so to the release, and any necessary changes needed for the setup guide related to #2 can go along with that PR. Thanks!


MPI Extension for Julia >= 1.9 + Distributed support (Single node only)

... so that we can implement pinthreads_mpi and pinthreads_distributed based on the actual rank / worker id information.
See #61. (Doesn't close it because in this PR we focus only on a single node.)

[x] getcpuids_distributed docstring
[ ] Distributed tests
[x] MPI tests
[ ] Add :cores option to pinthreads_hybrid
[ ] Documentation

[ ] getcpuids_distributed docs
[ ] Distributed tests
[ ] MPI tests

Superseded by #99  and #100


Upgrade twilio libs

Upgrade twilio libs
Replace Qunit w/ karma and mocha

@juancarlospaco you're almost there. Thanks.

Thanks! LGTM.
Please squash the commits, wait until the new (squashed) commit is green, then merge it.


Return calibration tip to tiprack

overview
After calibrating labware on the deck of the robot, the pipette should return the tip that was used for calibration to the tiprack from which it was picked up.
behavior
Current behavior: After calibrating labware on the deck, the robot does not return the tip and it has to be manually removed
Desired behavior: After calibrating labware on the deck, the robot should return the tip to the tiprack where it was picked up
steps to reproduce
Use the app to calibrate a deck setup including at least one tiprack and at least one non-tiprack labware. This should also cover #532

The server does not currently keep track of information about where the current tip was picked up. This is part of the plan for resources tracking


Cannot uninstall slack

$ brew cask list
google-chrome	 middleclick	  openemu	   slack
$ brew cask uninstall slack
Error: slack is not installed

$ brew cask doctor
==> OS X Release:
10.10
==> OS X Point Release:
10.10.1
==> Hardware Architecture:
intel-64
==> Ruby Version:
2.0.0-p481
==> Ruby Path:
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
==> Homebrew Version:
0.9.5
==> Homebrew Executable Path:
/usr/local/bin/brew
==> Homebrew Cellar Path:
/usr/local/Cellar
==> Homebrew Repository Path:
/usr/local
==> Homebrew Origin:
https://github.com/Homebrew/homebrew.git
==> Homebrew-cask Version:
0.52.1
==> Homebrew-cask Install Location:
/usr/local/Cellar/brew-cask/0.52.1
==> Homebrew-cask Staging Location:
/opt/homebrew-cask/Caskroom
==> Homebrew-cask Cached Downloads:
/Library/Caches/Homebrew
/Library/Caches/Homebrew/Casks
0 files
0.00 megs
==> Homebrew-cask Default Tap Path:
/usr/local/Library/Taps/caskroom/homebrew-cask
==> Homebrew-cask Alternate Cask Taps:
<NONE>
==> Homebrew-cask Default Tap Cask Count:
2224
==> Contents of $LOAD_PATH:
/usr/local/Cellar/brew-cask/0.52.1/rubylib
/Library/Ruby/Site/2.0.0
/Library/Ruby/Site/2.0.0/x86_64-darwin14
/Library/Ruby/Site/2.0.0/universal-darwin14
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/x86_64-darwin14
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/universal-darwin14
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/x86_64-darwin14
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin14
==> Contents of $RUBYLIB Environment Variable:
<NONE>
==> Contents of $RUBYOPT Environment Variable:
<NONE>
==> Contents of $RUBYPATH Environment Variable:
<NONE>
==> Contents of $RBENV_VERSION Environment Variable:
<NONE>
==> Contents of $CHRUBY_VERSION Environment Variable:
<NONE>
==> Contents of $GEM_HOME Environment Variable:
<NONE>
==> Contents of $GEM_PATH Environment Variable:
<NONE>
==> Contents of $BUNDLE_PATH Environment Variable:
<NONE>
==> Contents of $PATH Environment Variable:
PATH="/Users/vladimir/.opam/system/bin:/Applications/Postgres.app/Contents/Versions/9.3/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/Library/ENV/scm"
==> Contents of $SHELL Environment Variable:
SHELL="/bin/bash"
==> Contents of Locale Environment Variables:
LANG="en_US.UTF-8"
LC_ALL="en_US.UTF-8"
LC_CTYPE="UTF-8"
==> Running As Privileged User:
No

Hi!
Try brew cask uninstall --force slack.  The underlyng issue will be addressed very soon.


Modify es:addGroupCommand in es:addCommand for admin and superadmin

Hi dude, first of all thank you for your scripts.
I noticed that all the players being outside the group 'USER' so the 'ADMIN' and 'SUPERADMIN' could not use the command and I took the liberter to modify the function which adds the command of :
--old command /givecarkeys
TriggerEvent('es:addGroupCommand', 'givecarkeys', 'user' , function(source, args, user)
TriggerClientEvent('esx_darcoche:Dar', source)
end, function(source, args, user)
TriggerClientEvent('chat:addMessage', source, { args = { '^1SYSTEM', 'ERROR - Regardez codeina.pro pour plus d information.' } })
end)
to this so that the group 'admin' and 'superadmin' can use the command too :
--new command /givecarkeys
TriggerEvent('es:addCommand', 'givecarkeys', function(source, args, user)
TriggerClientEvent('esx_darcoche:Dar', source)
end
Here I do not know if you intend to change your rating but those who want to do it themselves have the code at least

All the other groups worked for me, on my end at least.
But yeah, thanks for your contribution, hope people know how to change them haha


Updated readme to reflect XAML Usage

Updated README.MD file to reflect usage information from https://www.nuget.org/packages/Xam.Plugin.HtmlLabel

Sorry, the README is right because starting with 3.0.0 the namespace is changed.
I'll release it today.


tracing color on circles

I'm colorblind and cannot for the life of me tell the difference between a node in 'tracing' mode vs not. I had to dig through the code to figure it out because I thought I was losing my mind after looking at your gifs in action :)
Any chance you'd be interested in making the color difference more obvious? I can throw together a pull request, but you may not want me choosing colors.
Awesome tool, though!

Hey there, thanks. :)
I'd love to help you out, but I'm afraid it'd be the (color)blind leading the (color)blind, I'm strong protan. I just borrowed the color palate from a d3 example.
The red is what I assumed that normal eyeballs would be able to see. I actually thought that some of the display code was wrong, too, since I couldn't pick out the red either. I think I wasted about twenty minutes on constant wtf'ing, trying to figure out what I'd done wrong.
I'm totally down to change the colors if you'd like to get a new palate from a non-malfunctining set of biological spectrometers. :)

@koudelka haha. well, i'll see if I can get someone to glance at this with me and give me some color feedback and I'll toss a PR your way when I get a chance.
Thanks!

sounds good :)


CP-15835: chore(github): Add CODEOWNERS file

This PR adds a CODEOWNERS file to the repo, assigning ownership to [@cloudzero/cirrus].

hm, we already have a codeowners file on the cloudzero-optimized branch, which is what we release from. I think we probably wouldn't need one on master, right?


PC - Add BlockExternalSync policy (B2BSync)

Add BlockExternalSync policy for the B2BSync feature on Windows
policy friendly name is "Prevent users from syncing libraries and folders shared from other organizations"
Also added see also link at bottom to Mac deployment document where the policy is described for Mac.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Thanks for the review Johan!


GithubCI,Controls.Sample.Gtk: add github CI and fix sample build

Description of Change
This commits remove the unsupported Fill property and adds
GithubCI to build the sample project.

the build is ok:

Successfully installed workload(s) gtk.

so i think we should take that pr.
i have some changes concerning package / workload usages in the solution, but i'll make an extra PR then i'm finished.

manifest is on nuget.org now:
https://www.nuget.org/packages/GtkSharp.NET.Sdk.Gtk.Manifest-6.0.400/
lastest version <IP_ADDRESS> too:
https://www.nuget.org/packages/GtkSharp

manifest is on nuget.org now:

Do you mean with this that we should remove the commit titled GithubCI: skip manifest update from this PR?

@lytico @jsuarezruiz We switched the package source to nuget instead of github packages.

@jsuarezruiz Any comment on this PR?


Make use of cert-manager types

What this PR does
Fixes https://github.com/giantswarm/giantswarm/issues/31130
Switches to using the actual types from Cert-Manager when getting the ClusterIssuer and Certificate CRs.
Checklist

[x] Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites TARGET_SUITES=./providers/eks/standard


Docs: type assertion example missing angle brackets ""

Page URL:
https://www.typescriptlang.org/docs/handbook/basic-types.html#type-assertions
Issue:
let strLength: number = (someValue).length;

should ne
let strLength: number = (<string>someValue).length;

Recommended Fix:
Browser:

Looks like this is a twoslash bug - #405 should fix it

Confirmed 405 - didn't fix it


Requires update to 1.16.5

Given the changelog this should just be a change to the hard requirement =]

oops, thanks for the report. Uploaded a new file to curse (same mod version) that should work,

oops, thanks for the report. Uploaded a new file to curse (same mod version) that should work,


Using a saw to open up the ribcage does not include saw sound.

Description
Parity. https://github.com/cmss13-devs/cmss13/blob/master/sound/surgery/saw.ogg
Reproduction
N/A
Screenshots
N/A
Additional context
N/A

Fixed


extensions do not seem to be registering

I am trying to add a custom tag via extensions however I have not been successful both via requiring the .js file, nor by creating my function in an object. Both are illustrated below. I don't believe I should be having to call addExtension() as I believe gulp-nunjucks-api should be doing that for me, but is that incorrect?
'use strict';

const
	path 			= require( 'path' ),
	gulp 			= require( 'gulp' ),
	nunjucksRender 	= require( 'gulp-nunjucks-api' ),
	nunjucks 		= nunjucksRender.nunjucks,

	temp_pth		= './src/templates/'
	;

gulp.task( 'compile-templates', () => {

	return gulp.src( temp_pth+'pages/index.nunjucks' )
		.pipe( nunjucksRender({
			verbose: true,
			src: [
				path.join( __dirname, '..', temp_pth, 'pages' ),
				path.join( __dirname, '..', temp_pth, 'components' )
			],
			// extensions: require( path.join( __dirname, 'template-extensions/component.js' ) )
			extensions: {
				component: function() {

					this.tags = ['component'];

					this.parse = function(parser, nodes, lexer) {
						var token = parser.nextToken();

						var args = parser.parseSignature(null, true);
						parser.advanceAfterBlockEnd(token.value);

						return new nodes.CallExtension(this, 'run', args);
					};

					this.run = function(context, name, data) {
						return nunjucks.render('./src/templates/components/' + name + '.nunjucks', data);
					};

				}
			}
		}))
		.pipe( gulp.dest( path.join( GLOBAL.config.buildPath, 'site' ) ) )
		;

});

This is the error I get when trying to compile:
[11:47:09] Handling error: Template render error: (/Projects/proj/src/templates/pages/index.nunjucks) [Line 25, Column 5]
  unknown block tag: component

events.js:141
      throw er; // Unhandled 'error' event
      ^
Template render error: (/Projects/proj/src/templates/pages/index.nunjucks) [Line 25, Column 5]
  unknown block tag: component
    at Object.exports.prettifyError (/Projects/proj/node_modules/nunjucks/src/lib.js:34:15)
    at Obj.extend.render (/Projects/proj/node_modules/nunjucks/src/environment.js:468:27)
    at /Projects/proj/node_modules/nunjucks/src/environment.js:311:35

Take a look at the last line of the example code here - http://mozilla.github.io/nunjucks/api.html#custom-tags
It shows this:
env.addExtension('RemoteExtension', new RemoteExtension());

gulp-nunjucks-api is looping over the property keys of the opts.extensions object basically calling that line above for you...but it's not doing addExtension(new opts.extensions.component()) it's just doing addExtension(opts.extensions.component).
So, in your code above - you should be setting your component field to an instance of the function, not the function...something like this:
extensions: { component: new MyComponent(); }

Or I guess you could even do something like this:
extensions: { component: new (function() { ... }) }

Basically component should be an object that has parse, tags and run functions, not a constructor function.
I will clarify that in the README sometime.

Ah ha, great thanks!


question about evaluation on EPFL dataset

Thanks for sharing this great work. I am curious about the evaluation on EPFL fountain and Herzjesu dataset. It seems that you use this script to evaluate on the DTU dataset. Do you use the same script to evaluate on EPFL dataset ? What's more, could you please provide the masks of images and gt point clouds of EPFL dataset?

Thanks for your quick and detailed reply!


strange error from together fine-tuning create --training-file "$file_id" --model "meta-llama/Meta-Llama-3.1-8B-Instruct-Reference"

getting strange error from: together fine-tuning create --training-file "$file_id" --model "meta-llama/Meta-Llama-3.1-8B-Instruct-Reference"
together fine-tuning create --training-file "$file_id" --model "meta-llama/Meta-Llama-3.1-8B-Instruct-Reference" > fine-tuning-job.json
message='The default LoRA rank for meta-llama/Meta-Llama-3.1-8B-Instruct-Reference has been changed to 64 as the max available.\nAlso, the default learning rate for LoRA fine-tuning has been changed to 0.001.'
message='The default LoRA rank for meta-llama/Meta-Llama-3.1-8B-Instruct-Reference has been changed to 64 as the max available.\nAlso, the default learning rate for LoRA fine-tuning has been changed to 0.001.'
message='Starting from together>=1.3.0, the default batch size is set to the maximum allowed value for each model.'
message='Starting from together>=1.3.0, the default batch size is set to the maximum allowed value for each model.'
Traceback (most recent call last):
File "/home/silvacarl/.local/bin/together", line 8, in
sys.exit(main())
File "/home/silvacarl/.local/lib/python3.10/site-packages/click/core.py", line 1157, in call
return self.main(*args, **kwargs)
File "/home/silvacarl/.local/lib/python3.10/site-packages/click/core.py", line 1078, in main
rv = self.invoke(ctx)
File "/home/silvacarl/.local/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/silvacarl/.local/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/silvacarl/.local/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/silvacarl/.local/lib/python3.10/site-packages/click/core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "/home/silvacarl/.local/lib/python3.10/site-packages/click/decorators.py", line 33, in new_func
return f(get_current_context(), *args, **kwargs)
File "/home/silvacarl/.local/lib/python3.10/site-packages/together/cli/api/finetune.py", line 239, in create
response = client.fine_tuning.create(
File "/home/silvacarl/.local/lib/python3.10/site-packages/together/resources/finetune.py", line 256, in create
response, _, _ = requestor.request(
File "/home/silvacarl/.local/lib/python3.10/site-packages/together/abstract/api_requestor.py", line 249, in request
resp, got_stream = self._interpret_response(result, stream)
File "/home/silvacarl/.local/lib/python3.10/site-packages/together/abstract/api_requestor.py", line 632, in _interpret_response
self._interpret_response_line(
File "/home/silvacarl/.local/lib/python3.10/site-packages/together/abstract/api_requestor.py", line 701, in interpret_response_line
raise self.handle_error_response(resp, rcode, stream_error=stream)
together.error.InvalidRequestError: Error code: 404 - {"message": "Invalid finetune param: ", "type": "invalid_request_error", "param": "", "code": ""}

the most recent.
from the API, it works.
doing the exact same thing via the CLI creates the above error.

Could you please send the version to verify?

together --version
Version 1.3.9

Hi @silvacarl2 , can you verify that $file_id is set and not empty? I am able to reproduce this error when an empty string is passed into the training file.

yup will do.  right now we have everything working correctly using the API.
by the way, your stuff and your company is awesome.

Glad to hear! Let us know how it goes.


Bump ESLint dependency to 8.x

Splitting out of #33: per https://eslint.org/docs/latest/extend/custom-processors#file-extension-named-processor, this package's file extension based processor approach is now deprecated. Fun!

Relevant: https://github.com/eslint/rfcs/pull/31 🙃


Interpolating inside content of pseudo element changes value

I am trying to use the rtlcss in the compiled CSS file. The problem is node-sass brings multiline comment on the same line to new line.clear
While I was able to place a multiline comment on the same line using interpolation which works great on another part except for the content of pseudo-elements. While compiling it adds strange block on the content.
a:after {
	position: absolute;
	content: "\f105" #{"/*rtl: newvalue */"};
	line-height: 1.2;
}

is compiled to
 a:after {
	position: absolute;
	content: "" /*rtl: newvalue */;
	line-height: 1.2;
}

NPM version: 6.1.0
Node version : 8.9.4
Node Process:
http_parser: '2.7.0',
node: '8.9.4',
v8: '6.1.534.50',
uv: '1.15.0',
zlib: '1.2.11',
ares: '1.10.1-DEV',
modules: '57',
nghttp2: '1.25.0',
openssl: '1.0.2n',
icu: '59.1',
unicode: '9.0',
cldr: '31.0.1',
tz: '2017b' }
Node Platform : darwin
Node architecture : x64
node-sass version :
node-sass       4.9.0   (Wrapper)       [JavaScript]
libsass         3.5.4   (Sass Compiler) [C/C++]
npm node-sass versions :
<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

The block is fine. It's another way of representing unicode characters.
On Wed., 30 May 2018, 11:35 am Sanjeev Shrestha<EMAIL_ADDRESS>wrote:

I am trying to use the rtlcss in the compiled CSS file. The problem is
node-sass brings multiline comment on the same line to new line.clear
While I was able to place a multiline comment on the same line using
interpolation which works great on another part except for the content of
pseudo-elements. While compiling it adds strange block on the content.
a:after {
position: absolute;
content: "\f105" #{"/*rtl: newvalue */"};
line-height: 1.2;
}
is compiled to
a:after {
position: absolute;
content: "" /*rtl: newvalue */;
line-height: 1.2;
}

NPM version: 6.1.0
Node version : 8.9.4
Node Process:
http_parser: '2.7.0',
node: '8.9.4',
v8: '6.1.534.50',
uv: '1.15.0',
zlib: '1.2.11',
ares: '1.10.1-DEV',
modules: '57',
nghttp2: '1.25.0',
openssl: '1.0.2n',
icu: '59.1',
unicode: '9.0',
cldr: '31.0.1',
tz: '2017b' }
Node Platform : darwin
Node architecture : x64
node-sass version :
node-sass 4.9.0 (Wrapper) [JavaScript]
libsass 3.5.4 (Sass Compiler) [C/C++]
npm node-sass versions :
<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2400, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWM6QkemNbA_1Zy1usecdmZdf_7Geks5t3mf9gaJpZM4US7d1
.


DaphneLib: Implement Indexing

Indexing is missing in DaphneLib which would be necessary for certain use cases
Left indexing, starting with 0, incrementing has higher prio imho

Please assign me to this issue.

Thanks, go ahead!


enable ENABLE_HARD_LINE_BREAK by default

This matches GitHub's rendering.
Fixes: https://github.com/go-gitea/gitea/issues/11155

Yeah, it will be "breaking" for users that don't specify the option in their config.

make lg-tm work


style: Fix last row partially visible on small screens.

Discussion: CZO
When we scroll down to the last row of the recent conversations table on small
screens, the row is completely visible compared to only partially before.

Screenshots and screen captures:

Self-review checklist

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

Communicate decisions, questions, and potential concerns.

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

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

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

Completed manual review and testing of the following:

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

How did you pick 160x? We might want to use calc or something to ensure it's correctly related to other things.

How did you pick 160x? We might want to use calc or something to ensure it's correctly related to other things.

The value here before the change was 120px and I think that was picked by looking at the screen too. I picked 160px because that was the value that brought everything in order. But I'll explore if we can do this using calc or something.

This is happening due to participated filter wrapping to the next line at 555px which has an height of ~30px. So, media query around that should work just right.

Changing the height from 100vh to 100% solves the problem. I think that automatically takes care of the wrapping of filters. Waiting for the response on renaming the id in CZO...

You can rename it with what you think is right, then we can review.

I was wrong @amanagr. 100% worked for small screens because the min-height of one of the parent elements was defined as 100vw which I think is probably wrong and with this 100% did not work for larger screens. Anyway, I tried to correct the parent's min-height to 100vh and working out some solution but no matter how I modify the css, I was not able to remove the use of max-height and if I define max-height I've got to subtract height of the recent_topics_filter_buttons and which can wrap and change in height and hence I've got to subtract the maximum height it could get (in case of wrapping) of recent_topics_filter_buttons from the max-height.
One possible solution was that we remove the max-height and let the flex determine the height of the element automatically and in that case, the scroll didn't work and if we define overflow: auto everything is perfect except for the element, there are two scrolls.

Now, if we go with declaring max-width everything is fine except the scroll go below the compose box.

So, maybe you can suggest a better option to me or should I go ahead and not care about the scroll going below the compose box as this is also how we are doing this currently.

I am not sure what you are trying to say but the right solution here is to correct the max-height of table_fix_head. We can use resize.js library to do this for use. We need to use 100vh - (height of filters + compose box) as max-height of table_fix_head

I don't like this direction; we have a project goal of deletion resize.js in favor of doing everything in CSS, and I don't want to add more resize.js if it's at all possible to avoid. Can you find a CSS solution?

@amanagr Can you suggest something here, I tried to find a CSS solution but was unsuccessful. Simplebar prevents a CSS solution but not allowing overflow: scroll. We can shift the resize function out of resize.js to recent_topics_ui.js and have some CSS to give some extra padding/margin bottom for resize cases but that will still cause some ambiguous behavior on resizing from small to a large window. The behavior will be just right when the screen is not being resized though.
What do you think?

As I said, a simple solution would be to reduce the max-height of table_fix_head further. Right now, it is 100vh - 100px, which works for large screens, we can switch it to  100vh - 165px for width less than md_min (768px).
The source of this bug is due to filters overflowing to the next line and thus taking more height on small screens. So, another solution would be to take #recent_topics_filter_buttons out of #recent_topics_table so that it is not a variable in calculating height for table_fix_head. This might not be worth doing since we are planning to redesign it anyway.

Ok, going with the simple solution.

@amanagr Reminder.

@lakshaykananiya thanks for the update! I advised using 100vh - 165px for width less than md_min (768px). so that we don't have a lot of blank space when the user is scrolled to the bottom.
We should probably replace these filters with a multi-select dropdown as planned. Added that to my TODO.

This was fixed as part of larger migration of scroll bar from .app to html.


Allow the ability to not include an image in the members page

We do not have an image for each member of the council, but the site mandates an image be added when adding a new member

done


[bug] Windows 11 - Bundle identifier folder not being created in configDir or appDir

Describe the bug
As per the docs the appDir should open configDir/bundleIdentifier, however it just opens User/AppData/Roaming with no sign of the bundle identifier named folder being visible.
Use case is that I want to store some config related data (folder paths/directories and names) related to my App in this easily accessible place. If this isn't it, where should it be placed? If appDir is the place, how do I bring my folder to be present there
tauri.conf.json
{
  "$schema"<EMAIL_ADDRESS>  "build": {
    "beforeBuildCommand": "yarn build",
    "beforeDevCommand": "yarn dev",
    "devPath": "http://localhost:1420",
    "distDir": "../dist",
    "withGlobalTauri": false
  },
  "package": {
    "productName": "Drafts",
    "version": "0.1.0"
  },
  "tauri": {
    "allowlist": {
      "fs": {
        "all": true,
        "scope": [
          "**"
        ]
      },
      "path": {
        "all": true
      },
      "dialog": {
        "all": true
      }
    },
    "bundle": {
      "active": true,
      "category": "Productivity",
      "copyright": "",
      "deb": {
        "depends": []
      },
      "externalBin": [],
      "icon": [
        "icons/32x32.png",
        "icons/128x128.png",
<EMAIL_ADDRESS>        "icons/icon.icns",
        "icons/icon.ico"
      ],
      "identifier": "dev.drafts.ink",
      "longDescription": "",
      "macOS": {
        "entitlements": null,
        "exceptionDomain": "",
        "frameworks": [],
        "providerShortName": null,
        "signingIdentity": null
      },
      "resources": [],
      "shortDescription": "",
      "targets": "all",
      "windows": {
        "certificateThumbprint": null,
        "digestAlgorithm": "sha256",
        "timestampUrl": ""
      }
    },
    "security": {
      "csp": null
    },
    "updater": {
      "active": false
    },
    "windows": [
      {
        "fullscreen": false,
        "height": 768,
        "resizable": true,
        "title": "Drafts",
        "width": 1280
      }
    ]
  }
}

Tried installing the .msi file from the yarn tauri build script, thinking the folder doesn't get created in dev but didn't work.
Awesome framework btw! Just need to get over this small hill :)
Reproduction
No response
Expected behavior
Should see the bundle identifier named folder in the appDir (per the docs)
Platform and versions
Environment
  › OS: Windows 10.0.22622 X64
  › Webview2: 105.0.1343.53
  › MSVC:
      - Visual Studio Build Tools 2022
  › Node.js: 16.13.1
  › npm: 6.14.9
  › pnpm: Not installed!
  › yarn: 1.22.10
  › rustup: 1.25.1
  › rustc: 1.63.0
  › cargo: 1.63.0
  › Rust toolchain: stable-x86_64-pc-windows-msvc

Packages
  › @tauri-apps/cli [NPM]: 1.1.1
  › @tauri-apps/api [NPM]: 1.1.0
  › tauri [RUST]: 1.1.1,
  › tauri-build [RUST]: 1.1.1,
  › tao [RUST]: 0.14.0,
  › wry [RUST]: 0.21.1,

App
  › build-type: bundle
  › CSP: unset
  › distDir: ../dist
  › devPath: http://localhost:1420/
  › framework: React (Next.js)
  › bundler: Webpack

App directory structure
  ├─ .git
  ├─ .next
  ├─ dist
  ├─ node_modules
  ├─ public
  ├─ src
  └─ src-tauri

Stack trace
No response
Additional context
No response

appDir only returns the path to the suggested directory, but Tauri does not create the folder for you! You need to use createDir to create it yourself.

@FabianLars Thanks for the quick response.
So the ideal flow would be,

Mount app
Check if appDir/${myAppName} folder exists
Create if it doesn't.
Store all my configs there?

Or is there a different documented approach to how best to handle this?

Also, since the docs are then misleading, mind if I open a PR for it? Since the configDir/${bundleIdentifier} doesn't actually happen?

So the ideal flow would be, ...

Yes but the exists check is optional because createDir shouldn't overwrite the folder if it exists already.

Also, since the docs are then misleading, mind if I open a PR for it? Since the configDir/${bundleIdentifier} doesn't actually happen?

What do you suggest changing it to? The docs doesn't mention that the dir will be created for you or am i missing something?

What do you suggest changing it to? The docs doesn't mention that the dir will be created for you or am i missing something?

Specifically this line indicates that the folder with the name bundleIdentifier will be resolved back. Kinda reads like I'd get the entire path with the bundle name also attached to it
Returns the path to the suggested directory for your app config files. Resolves to ${configDir}/${bundleIdentifier}, where bundleIdentifier is the value [tauri.bundle.identifier](https://tauri.app/v1/api/config/#bundleconfig.identifier) is configured in tauri.conf.json.

The reason this ticket was created is because the bundleIdentifier line caused confusion when I couldn't see it! Removing it since it adds no value (my assumption) would make it simpler to understand. Optionally add a disclaimer that you can create a folder here to store app related config using the createDir function

Kinda reads like I'd get the entire path with the bundle name also attached to it

But that's exactly what it does, it returns the path, like C:\Users\username\AppData\Roaming\com.bundle.identifier. But it's only the path, and it doesn't create the folder for you

Oh! Got it! Okay maybe last edit?
Returns the path to the suggested directory for your app config files. Resolves to ${configDir}/${bundleIdentifier}, where bundleIdentifier is the value tauri.bundle.identifier is configured in tauri.conf.json. Tauri does not create the bundleIdentifier folder for you here, but rather just returns the path if it exists. You can if you wish to, create the folder using createDir function
Does this make it better?

Hmm idk, in theory we'd need to add that warning to all other methods too then because you can never expect the paths to actually exist on the file system (at least on Linux, on Windows it's basically impossible to not have the AppData folder).

Okay! Atleast we'd have this issue present if someone fumbles like I did!
Appreciate you helping me out on a weekend! Cheers!


Feature to add alerts

Feature to add alerts

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
see 1 file with indirect coverage changes

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Event Documentation for Interior left nav

Please provide event documentation for for the Interior left nav events on carbondesignsystem.com. For example 'left-nav-beingtoggled'

Issue moved to carbon-design-system/carbon-addons-cloud-vanilla #27 via ZenHub


Planning on pushing this repo in GCP, you think it will work?

You mean Google Cloud Computing right?
I dont know much about that but if you can install packages via apt and pip you should be able to do it.

Yea Goolge Cloud Platform, will keep you update


Stale Repo?

This repo hasn't seen updates in a while. Any updates?

What type of update were you looking for? There's almost no code in this module, it just rebuilds the underlying react-redux module for Preact.

I meant a verbal update on where the current issues and PRs stand. There’s some pretty old ones.


Documentation request: how to configure to match behaviour of prettier

We use Prettier to check formatting in our CI/CD.
I use Dprint to format more quickly locally.
Could the Dprint documentation please explain how to configure it to match Prettier's behavior?
(Aside from just giving up and using the Prettier plugin for Dprint.)
For example,
Prettier prefers:
{
  key: condition ? value1 : value2,
}

...even when well under the target line length.
Dprint prefers:
{
  key: condition
    ? value1
    : value2,
}

Dprint's formatting looks great to me, but fails on CI/CD.
To try to mimic Prettier's behaviour I have tried the following, but it does not work:
{
  ...
  "binaryExpression.linePerExpression": false,
  "binaryExpression.operatorPosition": "sameLine",
  "binaryExpression.preferSingleLine": true,
  ...
}

We use Prettier to check formatting in our CI/CD.
I use Dprint to format more quickly locally.

There's no way that dprint-plugin-typescript is going to match prettier. The implementations are very different so even with config it would be very tough for me to match it.
That said, the closest to prettier for dprint-plugin-typescript is probably to use what deno uses. There's a high level "deno" config that applies the settings:
{
  "deno": true,
  "indentWidth": 4,
  "quoteProps": "asNeeded" // this will become the default in deno's config later
}

By the way, that example you gave is just because by default dprint-plugin-typescript respects explicit newlines https://dprint.dev/plugins/typescript/#explicit-newlines -- there is the "preferSingleLine"-like config, but it's kind of experimental at this point and I wouldn't recommend using it.

Oh, by the way, if you're only looking for faster formats, if you use the dprint-plugin-prettier locally it should be faster than prettier's cli (especially with incremental formatting enabled). One issue though is the version used in dprint-plugin-prettier might be different than the version used on your CI.

OK cheers. I might just continue to use Dprint for personal readability while I type and try to remember to run Prettier once manually before each commit.


"Telecine" is a Brazilian trade mark

Just for information: Telecine is a brazilian entertainement trademark from Globo's news corporation. This could start some troubles/misunderstands in future. :smile:

Ugh. I don't really expect this app to blow up and get hundreds of millions of installs or anything. Thanks for letting me know, i'm going to look the other way for now... :eyes: !


Add support for Utf8View -> numeric in can_cast_types

Describe the bug
It looks like utf8View -> numeric support is available in cast_with_options however can_cast_types does not reflect that.
can_cast_types should be gone over closely to verify it reflects what is actually supported wrt Utf8Views
To Reproduce
Done via DF's slt tests
query B
select arrow_cast(arrow_cast('1', 'Utf8View'), 'Int32') < 2;
----
true

DataFusion error: This feature is not implemented: Unsupported CAST from Utf8View to Int32
Expected behavior
This should work as expected
Additional context

take


.kill() on single process kills all solution console apps

It would be faster to read mine StackOverflow post and few comments there ;)
https://stackoverflow.com/questions/46488212/kill-only-one-console-app-when-multiple-are-started

@gregg-miskelly in case he has any insights. This isn't the best place to track this issue if there is anything to be done about it so I'll close this in a few days but I want to give folks a chance to decide if there is anything to fix and open a tracking issue for it elsewhere.

@Petermarcu I do not know if I understand correctly...
My question here is: Why in .Net Core when I will start for instance 3 apps (with Start mode) i cannot kill only one of them. Isn't it a bug ? For comparison, when exactly the same process will be made in .Net 4.6 everything works as expected (only one app is closing).
Solution from stackoverflow is only workaround because changing mode of one app to "start without debugging" blocks debugging possibility of one app.

Assuming this happens only when you are debugging - this is a known bug in the .NET Core project system in Visual Studio. I can't seem to find the bug to reference it at present. You can open or search for a problem through Visual Studio's Report a problem feature if you want to track it.

Found it. Here is the issue: https://developercommunity.visualstudio.com/content/problem/88707/debugging-multiple-dotnet-core-applications-all-te.html

@gregg-miskelly Thanks for your help Greg !

Issue moved to dotnet/project-system #2874 via ZenHub


BUG: BRAINSConstellationDetector Requires VTK and ITKVTK

BCD needs VTK and ITK build with VTK to build properly

AleXander,
BCD only needs VTK when GUI is on.   Please verify that VTK is really needed when gui is not on.
Sent from my iPhone

On Jan 18, 2016, at 9:37 AM, ajjl<EMAIL_ADDRESS>wrote:
BCD needs VTK and ITK build with VTK to build properly
You can view, comment on, or merge this pull request online at:
https://github.com/BRAINSia/BRAINSTools/pull/240
Commit Summary
BUG: BRAINSConstellationDetector Requires VTK and ITKVTK
File Changes
M SuperBuild.cmake (6)
M SuperBuild/External_ITKv4.cmake (4)
Patch Links:
https://github.com/BRAINSia/BRAINSTools/pull/240.patch
https://github.com/BRAINSia/BRAINSTools/pull/240.diff
—
Reply to this email directly or view it on GitHub.

FYI. While working on topic do-not-use-dynamic-cast-with-transform, I add to do a similar commit to allow build with default option to succeed. See https://github.com/jcfr/BRAINSTools/commit/b22769d9036398704094eabf95931330c0d60bcd

@ajjl This change was incorporated in another patch set.


Cannot run the colab notebook for version 2.1

I cannot get it run, get the following errors:
Installing requirements for Web UI
Launching Web UI with arguments: --share --force-enable-xformers
Loading config from: /content/stable-diffusion-webui/models/Stable-diffusion/v2-1_768-ema-pruned.yaml
LatentDiffusion: Running in v-prediction mode
DiffusionWrapper has 865.91 M params.
Downloading: 100% 3.94G/3.94G [00:51<00:00, 77.1MB/s]
^C

it is working now I did nothing 😋

I still have the same issue. Did you run Colab with high ram mode?

are we talking about same thing right 🤔 https://github.com/camenduru/stable-diffusion-webui-colab/blob/main/stable_diffusion_v2_1_webui_colab.ipynb is working right now

Yes, we talk about the same thing. When this one loads the model, it consumes more than 12 GB RAM, and Colab supports only 12 GB. Do you have Pro accounts?

I am testing with free account please use this command !nvidia-smi and please post here what it returns
Wed Dec 14 21:04:12 2022
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 460.32.03    Driver Version: 460.32.03    CUDA Version: 11.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  Tesla T4            Off  | 00000000:00:04.0 Off |                    0 |
| N/A   49C    P0    28W /  70W |      0MiB / 15109MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+

I mean RAM not GPU RAM. In this picture below:

I think it is broken on the Auto1111's repo. Normally they have an option for "low ram" but it doesn't work anymore.

please run this command !grep MemTotal /proc/meminfo

Interesting, I have the same size of RAM as you. Here is the output:
13297220 kB

If I run the notebook from your github, no problem. If I run the cloned version of it, it crashes. Interesting bugs right there.

is everything same with this repo

I tried ony with version 2.1. When I have time, I will try with the rest.


This project is abandoned

Swiper JS is at version: 6.5.7
and yet
vue-awesome-swiper wants to work with version: 5.2.0
Version 5.2.0 RELEASED 10/26/2019
coming up on two years ago

How complex would it be to update vue-awesome-swiper to swiper v6?
Maybe we can collaborate with the owner or take over ownership?

It works with v6.  How to use it with v6 is even documented in the README.  If you install v6 though recent versions of npm refuse to proceed.  Somebody should just fork this and and update the package.json.

Swiper <6.5.1 also contains a critical vulnerability, so updating makes more sense than ever.

@surmon-china is there anything that we can do? We love this library would be nice that you at least could make it compatible with v6 or even maybe have someone taking over. Thank you 🙂

This package is based on the famous swiper
As of late 2020 they support Vue,js out of the box! (Though you must at least use Vue.js version 3). So you should migrate to that if you can.
More info here: https://swiperjs.com/vue

Example to work nuxt 2.15.7 with swiper 6.8.4


[COMMUNITY] Update contributor list to reflect new guideline.

This PR updates the contributor list as per https://docs.tvm.ai/contribute/community.html, the list of changes are:

All the committers and are also added to the reviewer list, so when new contributors can request code reviews from committers as well when looking at the list
Add github id so it is easy to copy them and request reviews
Mark all the current PMC members
Remove code-owner tier

We have sent a committer invitation to each of current code-owners
A followup PR will be opened to welcome these new committers

cc @dmlc/tvm-team

followup PR https://github.com/dmlc/tvm/pull/2139