OS 2.0 doesn't start on Fedora 24/Python 3.5 (module 'html5lib.treebuilders' has no attribute '_base')

I tried the installation instructions, creating a virtual environment and installing Openslides with pip. Trying to start openslides I get the following:
(.virtualenv) mwenzel@mwenzel-desktop:/home/mwenzel/src/OpenSlides
$ openslides start
Traceback (most recent call last):
  File "/home/mwenzel/src/OpenSlides/.virtualenv/bin/openslides", line 11, in <module>
    sys.exit(main())
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/openslides/__main__.py", line 41, in main
    known_args.callback(known_args)
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/openslides/__main__.py", line 176, in start
    execute_from_command_line(['manage.py', 'migrate'])
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/django/core/management/__init__.py", line 327, in execute
    django.setup()
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/django/__init__.py", line 18, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/django/apps/registry.py", line 115, in populate
    app_config.ready()
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/openslides/motions/apps.py", line 15, in ready
    from . import projector  # noqa
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/openslides/motions/projector.py", line 6, in <module>
    from .views import CategoryViewSet, MotionViewSet, WorkflowViewSet
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/openslides/motions/views.py", line 28, in <module>
    from .pdf import motion_poll_to_pdf, motion_to_pdf, motions_to_pdf
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/openslides/motions/pdf.py", line 6, in <module>
    from bs4 import BeautifulSoup
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/bs4/__init__.py", line 30, in <module>
    from .builder import builder_registry, ParserRejectedMarkup
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/bs4/builder/__init__.py", line 314, in <module>
    from . import _html5lib
  File "/home/mwenzel/src/OpenSlides/.virtualenv/lib64/python3.5/site-packages/bs4/builder/_html5lib.py", line 70, in <module>
    class TreeBuilderForHtml5lib(html5lib.treebuilders._base.TreeBuilder):
AttributeError: module 'html5lib.treebuilders' has no attribute '_base'

http://stackoverflow.com/questions/38447738/beautifulsoup-html5lib-module-object-has-no-attribute-base
this is possible the issue. The following restriction possibly solves the issue: "html5lib<=0.9999999"

Right, for install OpenSlides 2.0 you have to install a specifc (old) version of html5lib which does not break our installation. This command works for a fresh install process:
pip install openslides html5lib==0.9999999

You closed this pretty quickly. Is there anything we can do to fix it, instead of having this (pretty bad) workaround?
Is there a way to do it within the requirements-file if there are known bad versions of html5lib?

@maweki Sorry. This issue is already fixed in current master (which is going for 2.1 release). For the 2.0 release we can not fix it and we does not want to make an extra 2.0.1 release (because 2.1-beta1 is coming in the next weeks). So we prefer to use this new install instruction for 2.0 on linux/macOS with this new install comand. Hope this helps.

This issue is already fixed in current master

This is what I wanted to hear. Thank you :)


TextField is not scrollable on android. It selects text

Software Versions

Python:3.8
OS:android
Kivy:master
Kivy installation method:buildozer

Describe the bug
whenever i type in series of character that exceed the height of a textfield and i want to scroll through it on my android phone, instead of scrolling, it selects the text

The default behavior for a text input in Kivy doesn't include scrolling. You could try to implement a solution using a ScrollView and text input box together. Here's a discussion on StackExchange that I found. I haven't tested rhe code, but in theory the approach makes sense to me. https://stackoverflow.com/questions/14117317/kivy-scrollview-implementation-in-textinput

ScrollView solution doesn't work properly, please find the details here: https://github.com/kivy/kivy/issues/7320

ScrollView solution doesn't work properly, please find the details here: #7320
Hmm. I see. Does this happen both when the font size is small and when large relative to the overall text field size? Does it happen if you overwrite the default on_double_tap method to pass?

Neither are a solution, but the might help narrow down one. I'm hoping they will help answer if it 1) behaves properly when the initial tap is on white space, which might be more feasible at different text sizes or 2) if this is a bug with double-tap detection on Android.

Fixed via #7610


Unable to launch "Automation Browser"

I don't see an option to use the "Automation Browser" when I open the Web Recorder. Is there some configuration item I am missing?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 360793f4-ad0e-2f85-1dcf-30b2f811bbe9
Version Independent ID: 703a9c74-c568-3a72-c441-ca259caa7ba2
Content: Using browsers and managing extensions - Power Automate
Content Source: articles/desktop-flows/using-browsers.md
Service: flow
GitHub Login: @georgiostrantzas
Microsoft Alias: getrantz

I found the solution. The Automation Browser is not an option when you select the Web Recorder. You have to choose Internet Explorer, and then edit the launch mode

I found the solution. The Automation Browser is not an option when you select the Web Recorder. You have to choose Internet Explorer, and then edit the launch mode


Updated to 2.7.3

Updated to the latest 2.7.3 Chart.js version

Apologies on the neglect of this.
0.1.7 is now published on RubyGems: https://rubygems.org/gems/chart-js-rails/versions/0.1.7 with Chartjs v2.8.0
Closing in favor of #30


V13 Icon broken due to rxjs dependency conflict

Reproduction link
https://github.com/ant-design/ant-design-icons/blob/af69440ddb02b1a417716d26a40904eebee6d70e/packages/icons-angular/src/package.json#L17
Steps to reproduce
Environment:
Angular CLI: 13.1.2
Node: 16.13.1
Package Manager: npm 8.1.2
OS: win32 x64

Angular: 13.1.1
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1301.2
@angular-devkit/build-angular   13.1.2
@angular-devkit/core            13.1.2
@angular-devkit/schematics      13.1.2
@angular/cli                    13.1.2
@schematics/angular             13.1.2
rxjs                            7.5.1
typescript                      4.5.4

Upgrade Angular v12 to v13 (also upgrade rxjs to v7)
Upgrade ng-zorro-antd to v13.0.0
Remove package-lock.json
Remove node_modules
Remove .angular/cache
Re-npm install the project with some WARNINGs below

npm WARN deprecated<EMAIL_ADDRESS>flatten is deprecated in favor of utility frameworks such as lodash.
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.

> xxxxxxx@xxx prepare
> husky install

husky - Git hooks installed

added 1313 packages, and audited 1314 packages in 2m

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

found 0 vulnerabilities

Then modules not found

import { IconDefinition } from '@ant-design/icons-angular';
import { DeleteOutline } from '@ant-design/icons-angular/icons';

Dive into dependencies, can not found @ant-design/icons-angular under node_modules

Install @ant-design/icons-angular manually: npm install @ant-design/icons-angular, with error below

npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/rxjs
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>|| ^7.4.0" from<EMAIL_ADDRESS>npm ERR!   node_modules/@angular/common
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@ant-design/icons-angular
npm ERR!       @ant-design/icons-angular@"*" from the root project
npm ERR!     1 more (@angular/platform-browser)
npm ERR!   1 more (@angular/core)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@ant-design/icons-angular
npm ERR!   @ant-design/icons-angular@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry

Dive into @ant-design/icons-angular source code, found a outdated rxjs dependency here
What is expected?
Bump rxjs dependency in @ant-design/icons-angular to v7
What is actually happening?
Wrong rxjs peer dependency. Icon not work.

Environment
Info

ng-zorro-antd
13.0.0

Browser
Edge 96

Bump rxjs dependency in @ant-design/icons-angular package.
https://github.com/ant-design/ant-design-icons/tree/master/packages/icons-angular

rxjs                            7.5.1

Rxjs uses 7.4 0 version

@umaim fixed by https://github.com/ant-design/ant-design-icons/pull/514 version >= 13.0.2


Error: while running on npm run develop

Getting errors while executing 'npm run develop'

Error generating due to the const path:

Go to freeCodeCamp/client/plugins/fcc-create-nav-data/create-navigation-node.js
Scroll down to Line No 26 and you can see const [,path] and this comma is generating an error as it can not get the path

OS: Ubuntu 18.04 LTS
Browser chrome
Version 70.0.3538.67 (Official Build) (64-bit)

As you remove the comma from the const[path] variable the 'npm run develop' command works fine.

We have resolved this specific issue with local setup. Please re-clone and make a fresh installation. A few other issues in the setup are still being worked upon. We will post an update in Contributors Chat room when the local is 100% ready.
Thanks


Port Forward is getting stopped very frequently when running python script to get data from remote to local

Elasticsearch version:
"number" : "6.2.3",
"build_hash" : "e605ecd",
"build_date" : "2018-07-18T19:19:00.787857Z",
"build_snapshot" : false,
"lucene_version" : "7.2.1",
"minimum_wire_compatibility_version" : "5.6.0",
"minimum_index_compatibility_version" : "5.0.0"
JVM version (java -version):
openjdk version "1.8.0_171"
OpenJDK Runtime Environment (build 1.8.0_171-8u171-b11-1~bpo8+1-b11)
OpenJDK 64-Bit Server VM (build 25.171-b11, mixed mode)
OS version (uname -a if on a Unix-like system):
Linux elassandra-0 4.15.0-1023-azure #24~16.04.1-Ubuntu SMP Wed Aug 29 12:54:36 UTC 2018 x86_64 GNU/Linux
Description of the problem including expected versus actual behavior:
Tried on All elassandra instances. E0924 12:32:34.310290  244680 portforward.go:303] error copying from remote stream to local connection: readfrom tcp4 <IP_ADDRESS>:9042-><IP_ADDRESS>:57857: write tcp4 <IP_ADDRESS>:9042-><IP_ADDRESS>:57857: wsasend: An established connection was aborted by the software in your host machine.   E0924 12:31:01.288127 250812 portforward.go:316] error copying from local connection to remote stream: read tcp4 <IP_ADDRESS>:9042-><IP_ADDRESS>:57844: wsarecv: An existing connection was forcibly closed by the remote host. E0924 12:31:01.771293 250812 portforward.go:303] error copying from remote stream to local connection: readfrom tcp4 <IP_ADDRESS>:9042-><IP_ADDRESS>:57844: write tcp4 <IP_ADDRESS>:9042-><IP_ADDRESS>:57844: wsasend: An existing connection was forcibly closed by the remote host.
Steps to reproduce:
Please include a minimal but complete recreation of the problem, including
(e.g.) index creation, mappings, settings, query etc.  The easier you make for
us to reproduce it, the more likely that somebody will take the time to look at it.

Provide logs (if relevant):

This is probably related to your kubernetes environment ?


Arithmetic overflow

Hi,
I get the following when I try to run the example in the readme. I'm using crystal 0.32.1 on Ubuntu Server 16.04 LTS.
Unhandled exception: Arithmetic overflow (OverflowError)
from lib/icmp/src/icmp/packet.cr:0:16 in 'calculate_checksum'
from lib/icmp/src/icmp/request.cr:29:19 in 'initialize'
from lib/icmp/src/icmp/request.cr:14:5 in 'new'
from lib/icmp/src/icmp.cr:54:9 in '__crystal_main'
from /usr/share/crystal/src/crystal/main.cr:97:5 in 'main_user_code'
from /usr/share/crystal/src/crystal/main.cr:86:7 in 'main'
from /usr/share/crystal/src/crystal/main.cr:106:3 in 'main'
from __libc_start_main
from _start
from ???
Thanks,
-G

Thanks @packetq


.signInRedirect vs .signInPopup

I have followed the sample to implement oidc-client from IdentityServer samples and here it has an illustration of the .signInPopup going to popup.html.  If I instead wish to do a .signInRedirect to IdSvr3 to login and then come back to the same page that called it how is that done? I simply don't want a new window but reuse the existing one?  In the case of popup in the sample it goes to popup.html and the new Oidc.UserManager().signinPopupCallback();resolves.  How would I do this without the popup to complete the call and retrieve the User?  Basically what needs to be different from that sample for a scenario with no popup window?  Thanks in advance for your help.

If I instead wish to do a .signInRedirect to IdSvr3 to login and then come back to the same page that called it how is that done?

Then in that page you'd have to detect the difference between an initial load, and the OIDC callback. This is usually done by detecting a hash fragment. But to be honest, having the callback be the same page as the rest of your app logic is discouraged. A standalone callback page that only does the callback processing is recommended. Once processing is successful, then redirect the user back into your app.
I don't follow the rest of your questions. The .signinRedirectCallback() returns a promise with the user passed as the param, which is the same as the .getUser() method. Perhaps this sample is better to look at: https://github.com/IdentityServer/IdentityServer4.Samples/tree/dev/Clients/src/JsOidc/wwwroot

Thanks Brock.  That sample is more clear to me and I had my manager code in the index.html like the prior sample rather than app.js which seems more logical. One followup question as I have this working with your suggestion up above until the callback attempts to happen.  How do you figure out this kind of error?

enable logging in identityserver

Instrumentation and a verbose log is a beautiful thing once found.  Much appreciated.


Difference between attributes and model.attributes on saving/updating/creating

I want to know the difference in the different events like saving, updating, creating
this.on('saving', function(model , attrs, options) {
  // this.attributes === attrs?
  // model.attributes === attrs?
}.bind(this));

What's wrong with uncommenting those lines, removing the ? and placing those comparisons inside console.log statements?

@ricardograca Nothing is wrong, I want to know if those are equals, for example if do somechange, example remove a property from model.attributes will affect the rest or remove something from attrs affect the other

Again, what does the console.log tell you? Also, I think you shouldn't be modifying the attributes object directly, but use the .set() and .get() instead.

I misunderstood your point at first. let me check

Don't bind your functions. It's slow. saving is the umbrella event. creating OR updating depends on what's actually going on. Calling model.save always means the model is saving. Creating vs. updating just means an insert query or an update. They're mutually exclusive.

thanks @bendrucker, I understand that part, but somethimes I need saving event for example for validations before save (updating or creating)

Ah, so the title versus your original post got me confused. Use get/set. Don't treat attributes as mutable.


TypeScript issues with the Yjs dependency

I’ve submitted a PR to Nitro, and their build is failing because it’s seem to check the types of all dependencies (which we skip I think) and the build fails because of Yjs:
Error:  Error building /home/runner/work/nitro/nitro: SyntaxError: Unexpected token (1:14) in /home/runner/work/nitro/nitro/node_modules/.pnpm/@scalar+api-reference@1.20.7_postcss@8.4.38_typescript@5.4.3_unified@11.0.4_vue@3.4.21_yjs<EMAIL_ADDRESS>Error:  Unexpected token (1:14) in /home/runner/work/nitro/nitro/node_modules/.pnpm/@scalar+api-reference@1.20.7_postcss@8.4.38_typescript@5.4.3_unified@11.0.4_vue@3.4.21_yjs<EMAIL_ADDRESS>  at pp$4.raise (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:21621:13)
  at pp$9.unexpected (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:18829:8)
  at pp$9.expect (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:18823:26)
  at pp$8.parseImportSpecifiers (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:19955:12)
  at Parser.parseImport (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:26126:32)
  at pp$8.parseStatement (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:19005:49)
  at pp$8.parseTopLevel (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:18886:21)
  at Parser.parse (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:18658:15)
  at Function.parse (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:18708:35)
  at Graph.contextParse (node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:25754:38)

Error:  Unexpected token (1:14) in /home/runner/work/nitro/nitro/node_modules/.pnpm/@scalar+api-reference@1.20.7_postcss@8.4.38_typescript@5.4.3_unified@11.0.4_vue@3.4.21_yjs<EMAIL_ADDRESS> ELIFECYCLE  Command failed with exit code 1.
Error: Process completed with exit code 1.

I can probably work around this for the PR, but our packages shouldn’t fail the TS check in other environments.
Can we do something to fix that? Or get rid of the Yjs dependencies? 🤔 And we should probably go back to type checking our dependencies again.

@hanspagel if we want to get rid of the yjs dependency we need to remove codemirror yjs.next out of this repo and pass it in from org (which we should do)
wdyt? I actually have a draft PR open for this :)

I think we did this already and intentionally moved it to the OSS repo! I think we’d all prefer to not have yjs shipped with the OSS package, but @geoffgscott probably knows whether we can work around this.

yeah sadly y-codemirror.next includes yjs, so we need to add that as a peer dep :)


[1.12] Blocks & items changing into other Items

Not sure really how to describe this bug but when ever I craft certain machines or blocks they turn into thermal foundation materials once I exit my inventory.
An example is the Dense Obsidian turns into a silver ingot...
All item changes: https://pastebin.com/2Zei3ZUm
MC: 1.12
Forge: <IP_ADDRESS>3
DeepResonance: 1.12-1.4.9
ThermalFoundation: 1.12-<IP_ADDRESS>-universal

That doesn't sound like a bug in Deep Resonance but more in some other mod that influences crafting

pulled the items right out of the creative menu. test crafted the obsidian in a minecraft crafting table.

Can you describe better what exactly you are doing?

Sorry for the late response, went out. I am working on a modpack and was trying to make a harder crafting recipe in another mod (using crafttweaker) and was going to use blocks from your mod. But when I pulled them out of the creative menu they would convert into other items from thermal foundation or basic minecraft.
The steps to reproduce this are:
Step 1: open D.R. creative tab.
Step 2: place one of the items in the list I had provided in your hot bar.
Step 3: close inventory.
after that The item would convert into another item.

Well not sure what to say but this is not something that can be caused by Deep Resonance. Check what other mods are in the pack. Something must be messing with this somehow

So updated my pack to 1.12.1 and the issue is no more. Not sure what mod was causing the issues. But it is fixed in 1.12.1


Goto callback not marked as optional

Using the bot.pathfinder.goto() function, the callback option does not have a default value and will throw an error if the function is used without providing a callback.

This issue was resolved a few PRs ago.

This issue was resolved a few PRs ago.


turbiniactl does not show help when only config is specified

When you run turbiniactl with no arguments it will show the help as expected, but when you run it with -c to specify a config file to run with, it gives a non-intuitive error.  We should fix this to show the help message in that case.
2021-03-11 07:12:42 [INFO] Turbinia version: unknown
2021-03-11 07:12:43 [WARNING] Command None not implemented.
2021-03-11 07:12:43 [INFO] Done.

Hello! I am new to open source, and I would like to work on this

@sosa-e Sorry, I didn't see this older comment until now.  I'm actually going to close this out because most of the features of this tool have been moved to turbinia-client and users will not be using this interactively (it's only used for start/stop of components).   There are some other issues with the good first issue label if you're still interested in contributing.  Thanks!


Add section on creating a pipeline

As @sj26 pointed out in https://github.com/buildkite/docs/issues/410, we don't have any docs on adding a pipeline. Having a section about it and a screenshot of the setup page will help people get going, and will hopefully help people evaluating us get a better idea of what's involved in setup. I got a bit distracted updating this doc while I was there 🤷🏻♀
This PR:

Adds a new section to the Defining Steps doc that describes creating up a new pipeline
Renames what used to be 'getting started' to 'adding steps'
Splits up 'adding steps' into 'visual pipeline editor' and 'pipeline.yml file' (happy to change these if y'all have any better ideas for headings)
Reworks the old pipeline.yml section a bit in an attempt to clarify it and give more direct instructions and information

Here's some screenshots of the new bits:

🌟🌟🌟

🌟🌟🌟


Feature: Ability to refer to individual headers and paragraphs

Hi, thanks for the great work!
It would be great to be able to link to individual headers and paragraphs inside wikilinks.
For instance, say I have doc1.md with the following contents:
# Header 1

Paragraph 1 text ^id1

Parahraph 2 text ^id2

Header 2

Inside doc2.md it would be great to use (perhaps) the following syntax:
[[doc1.md#Header 2]]

[[doc1.md#^id1]] or [[doc1.md#^Paragraph 1]]

That is, for links to individual paragraphs, we could either have a unique ID associated with each paragraph, or the ID could be the unique text prefix of that paragraph (perhaps easier to read, but there might be paragraphs with very long identical prefixes).
This feature would be very helpful in many scenarios. Here is one. Say you are taking meeting notes in individual files and a TODO pops up that you write down during the meeting. In your daily notes, it would be great to log the fact that you worked on that TODO item without having to copy it over. This way, you always keep the context in which a TODO item came to mind.,

seems like a duplicate of #19


HELP

Hello @mobizt ,
Need Help for my problem,
How do I keep the database saved even if I do a reset on my esp32 board.
Thanks

No, because no internet access.


fix: clear cache before call

#189

timeout
tokio::time::timeout(Duration::from_millis(100), context.read_coils(addr, len))

Maybe we can recv a expired data, after cancel a call, because of slave use more time to processing and causing timeout.
The expired data will be stored in the read buffer of transport self.framed.get_mut(), so we should clear it.
transmission error
Maybe we can recv fault data， because of transmission error.
The fault data will be stored in the read buffer of framed self.framed.read_buffer_mut(), so we should clear it.

Maybe we can use tokio::io::AsyncReadExt::read_buf to clear self.framed.get_mut(), but the length of buf is difficult to determine.

Maybe we can use tokio::io::AsyncReadExt::read_buf to clear self.framed.get_mut(), but the length of buf is difficult to determine.

We don't need to change the code if it is not possible. Adding a TODO comment that it would be desirable to read all available bytes at once is also a valid choice.

timeout
tokio::time::timeout(Duration::from_millis(100), context.read_coils(addr, len))

Maybe we can recv a expired data, after cancel a call, because of slave use more time to processing and causing timeout. The expired data will be stored in the read buffer of transport self.framed.get_mut(), so we should clear it.
transmission error
Maybe we can recv fault data， because of transmission error. The fault data will be stored in the read buffer of framed self.framed.read_buffer_mut(), so we should clear it.

Please extract the code into a function with a concise name and add a comment with the details.
Questions asked during a review do not need to be answered in the PR. I am just a generic placeholder for any future developer who stumbles over this code. Ideally, the code should answer all these questions, or even better, is written in a way that these questions don't even come up in the reader's head.

We also need a concise sentence for the CHANGELOG that summarizes the fix with its consequences and implications for users of this library. I wonder why no one complained about these serious issues earlier.

#198 has just been merged, which clears the buffer before each send in a simpler way.
Please try again with the latest tokio-modbus revision. Closing this PR - reopen if problems persist.

use std::{net::SocketAddr, pin::Pin, time::Duration};

use futures::future;
use tokio::net::TcpListener;

use tokio_modbus::{
    prelude::*,
    server::tcp::{accept_tcp_connection, Server},
};

struct ExampleService {}

impl tokio_modbus::server::Service for ExampleService {
    type Request = Request;
    type Response = Response;
    type Error = std::io::Error;
    type Future =
        Pin<Box<dyn future::Future<Output = Result<Self::Response, Self::Error>> + Send + Sync>>;

    fn call(&self, req: Self::Request) -> Self::Future {
        match req {
            Request::ReadHoldingRegisters(_addr, cnt) => {
                let data = vec![0; cnt as usize];
                Box::pin(future::ready(Ok(Response::ReadHoldingRegisters(data))))
            }
            Request::WriteMultipleRegisters(addr, values) => Box::pin(async move {
                tokio::time::sleep(Duration::from_millis(500)).await;
                Ok(Response::WriteMultipleRegisters(addr, values.len() as u16))
            }),
            Request::WriteSingleRegister(addr, value) => Box::pin(future::ready(Ok(
                Response::WriteSingleRegister(addr, value),
            ))),
            _ => {
                println!("SERVER: Exception::IllegalFunction - Unimplemented function code in request: {req:?}");
                // TODO: We want to return a Modbus Exception response `IllegalFunction`. https://github.com/slowtec/tokio-modbus/issues/165
                Box::pin(future::ready(Err(std::io::Error::new(
                    std::io::ErrorKind::AddrNotAvailable,
                    "Unimplemented function code in request".to_string(),
                ))))
            }
        }
    }
}

impl ExampleService {
    fn new() -> Self {
        Self {}
    }
}

#[tokio::main]
async fn main() -> Result<(), Box<dyn std::error::Error>> {
    let socket_addr = "<IP_ADDRESS>:5502".parse().unwrap();

    tokio::select! {
        _ = server_context(socket_addr) => unreachable!(),
        _ = client_context(socket_addr) => println!("Exiting"),
    }

    Ok(())
}

async fn server_context(socket_addr: SocketAddr) -> anyhow::Result<()> {
    println!("Starting up server on {socket_addr}");
    let listener = TcpListener::bind(socket_addr).await?;
    let server = Server::new(listener);
    let new_service = |_socket_addr| Ok(Some(ExampleService::new()));
    let on_connected = |stream, socket_addr| async move {
        accept_tcp_connection(stream, socket_addr, new_service)
    };
    let on_process_error = |err| {
        eprintln!("{err}");
    };
    server.serve(&on_connected, on_process_error).await?;
    Ok(())
}

async fn client_context(socket_addr: SocketAddr) {
    let task = async {
        // Give the server some time for starting up
        tokio::time::sleep(Duration::from_secs(1)).await;

        println!("CLIENT: Connecting client...");
        let mut ctx = tcp::connect(socket_addr).await.unwrap();

        let read_reg = tokio::time::timeout(
            Duration::from_millis(200),
            ctx.read_holding_registers(0x01, 2),
        )
        .await;
        println!("{:?}", read_reg); // Should success

        let write_mul = tokio::time::timeout(
            Duration::from_millis(200),
            ctx.write_multiple_registers(0x01, &[0]),
        )
        .await;
        println!("{:?}", write_mul); // Should timeout
        tokio::time::sleep(Duration::from_millis(500)).await; // wait finish

        let write_sig = tokio::time::timeout(
            Duration::from_millis(200),
            ctx.write_single_register(0x01, 0),
        )
        .await;
        println!("{:?}", write_sig); // Should success

        println!("CLIENT: Done.")
    };
    tokio::join!(task, tokio::time::sleep(Duration::from_secs(5)));
}

#198 not solve timeout.
You can test this example. write_sig will return error, if not clear self.framed.get_mut()


Correct rootId parameter for onClick in Vx viewer

right now on click is broken because of this.

[pr-server]
View this change on PR Server

[pr-server]
View this change on PR Server


templates/packer: increase net.core.(w/r)mem_max

In conjunction with osbuild/osbuild#1833 this should fix building very large manifests.

[x] https://github.com/osbuild/osbuild/pull/1833

With https://github.com/osbuild/osbuild/pull/1836 merge this should no longer be needed because we decided to use FDs for large data transfers. If this this PR still wants to be used we would tweak the {r,w}mem_default instead.


FIX Ambiguous column SQL error

I noticed that I couldn't use Max('LastEdited') with certain Datalists. Seems to be an issue if you use a filter that joins other tables, and results in an error such as:

"Column 'LastEdited' in field list is ambiguous"

Have included a test which fails prior to this patch.
In limited testing this seems to fix the problem and not cause side effects. To be honest though I don't understand very well what I've done here, so would appreciate the fine-toothed comb treatment from someone who understand SQL and the ORM better and would know if this is likely to cause regressions.

Change your test to a numeric column (e.g 'ID'). avg() isn't defined for dates in postgres.

Thanks Damian, have updated

There is no such thing as an average date with a developer is there? :P

great fix

Regression: This causes an error when sorting on many_many_extraFields

We need to take care of the case where table_for_object_field returns DataObject. It's a currently undesirable feature of this method.

Regression: This causes an error when sorting on many_many_extraFields

@dhensby can you give example code to reproduce? It seems to work fine for me, for example $Coupons.sort('SortOrder').Max('LastEdited') works in template where SortOrder is defined on many_many_extraFields

@jonom if you look at @tractorcow's commit above, I assume those tests would fail currently

Fix at https://github.com/silverstripe/silverstripe-framework/pull/6307

Thanks for fixing @tractorcow sorry for the regression... bug fixing is a game of whack-a-mole!


speechbot

user story:  As a slack user, I want to be able to type !speech lake and get the claim from spee.ch for the name 'lake' (https://spee.ch/lake), so that I don't have to type out the full url.
A bot to provide interaction with spee.ch from slack.  The ideal bot will be great for lbry, but also for other slack communities who can install it and benefit from the fun/helpful tools it provides.
@filipnyquist is working on this.

basic functionality includes something like !speech example that returns the claim for 'example' from spee.ch.
additional functionality will be added as separate issues.  Potential actions include: the ability for speechbot to publish slack uploads to spee.ch automatically, and to generate memes based on spee.ch assets.

This would be great, but should be handled as a separate repo.  Closing issue.


Add UI for applications

This was a mess to do.
There is the bug of application list not updating after adding new applications.

Also did a few significant refactoring

Bug has been fixed.

:(


[WIP] Make ImageFiltering.imfilter pipeline differentiable

This PR aims to make the ImageFiltering.imfilter function differentiable.

[x] Add adjoints
[ ] Get the gradient of the kernel
[ ] Put tests

The current issue I am facing causes the gradient of the kernel to be nothing. There seems to be a place where I have incorrectly put a NoTangent(), or a @nograd. Infact upon some further tracing I found out it does go inside the rule of __imfilter_inbounds!, but never calls the pullback inside it.
julia> img = testimage("cameraman");

julia> gradf(x, k, args...) = Zygote.gradient((a, b)->imfilter(a, b, args...) |> sum |> (z->z.val), x, k)
gradf (generic function with 1 method)

julia> gradf(img, t*t')
(Tangent{Gray{Float64}, NamedTuple{(:val,), Tuple{Float64}}}[Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,) … Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,); Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,) … Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,); … ; Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,) … Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,); Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,) … Tangent{Gray{Float64}}(val = 1.0,) Tangent{Gray{Float64}}(val = 1.0,)], nothing)

The img gradient is incorrect too, and I shall get it rectified when I write tests and figure out the kernel thing.

Codecov Report

Merging #21 (cf6fbf8) into main (890e927) will decrease coverage by 32.67%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main      #21       +/-   ##
===========================================
- Coverage   88.23%   55.55%   -32.68%
===========================================
  Files           3        4        +1
  Lines         102      162       +60
===========================================
  Hits           90       90
- Misses         12       72       +60

Impacted Files
Coverage Δ

src/filters/adjoints.jl
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Question: Does the plugin handle larger files by "chunking" them on write and "stitching" them back together on read?

In looking at the Benchmarks each different device has a different "max" file size it can handle before it errors, my question pertains to how/if this plugin handles files with sizes that exceed what a specific device can handle.
Does it "chunk" them into smaller more manageable writes and "stitch" them back together on read?
Does it just throw an error if the file size is too large?

That's a great idea, we should do that. However, it should stitch the parts together on write, rather than read.

I was able to wrote huge file (>5GB) using capacitor-file-chunk package. They implemented the append feature over this repo:

This plugin builds on the original concept and code of the Capacitor Blob Writer plugin by providing the ability to read and write files in chunks, offering improved performance compared to the Capacitor Filesystem plugin. It enables direct communication and exchange of binary data with a localhost web server, eliminating the need for base64 conversion.


Create ingress examples

Hi, thanks for the helm chart!
I am having trouble getting the ingress right.
Ingress specification
When I try to set the ingress in the current specification as
  - host: rsw.example.org
    paths:
      - path: /

I see
Error: failed to create resource: Ingress.extensions "rstudio-rstudio-workbench" is invalid: [spec.rules[0].http.paths[0].path: Invalid value: "map[path:/]": must be an absolute path, spec.rules[0].http.paths[0].backend: Required value: port name or number is required]

If I don't specify paths I get an error. Maybe there could be a default for this option?
  - host: rsw.example.org
    paths: []

Edit: In the template
https://github.com/rstudio/helm/blob/8b2deeecf9ba5c7229786712c02745a3c1213d53/charts/rstudio-workbench/templates/ingress.yaml#L29-L31
I found that the rules category comes before host: - so wouldn't the specification need to look as follows?
ingress:
  enabled: true
  rules:
    - host: subdomain.example.org
      paths:
        - path: /

Though this one errors with
	│ Error: failed to create resource: Ingress.extensions "rstudio-rstudio-workbench" is invalid: spec: Invalid value: []networking.IngressRule(nil): either `backend` or `rules` must be specified

Guidance how to set the ingress properly would be highly appreciated!
pathType Option
Since k8s v1.18 it is common to also provide a pathType option in the ingress.
Would it be possible to adapt the template to support this?
See the gitea helm chart as an example.
So in the end I would like to write the following
ingress:
  enabled: true
  annotations: {}
  hosts:
    - host: rsw.example.com
      paths:
        - path: /
          pathType: Prefix

After many hours of trial and error I found the following specification which works
ingress:
  enabled: true
  hosts:
    - host: subdomain.example.org
      paths: [/]

Reference
This only works if service.port mapping is added in values.yml because it is referenced in
https://github.com/rstudio/helm/blob/7caafdf5159f6c4e662bc0d5d740f1e588500b4a/charts/rstudio-workbench/templates/ingress.yaml#L3
and currently not defined in the values.yml template.
service:
  port: 8787

When missing it causes the following error
	│ Error: cannot patch "rsw" with kind Ingress: Ingress.extensions "rsw" is invalid: spec.rules[0].http.paths[0].backend: Required value: port name or number is required

After the latest ingress changes in #117, my previous ingress definition is failing.
We have several ingresses defined in our cluster and all besides the rstudio charts use the same notation, which is as follows:
ingress:
  hosts:
  - host: <subdomain.domain.topleveldomain>
    paths:
  - path: /
    pathType: Prefix

If I use this one for the RSW chart, I get
│ Error: cannot patch "rsw" with kind Ingress: Ingress.extensions "rsw" is invalid: spec.rules[0].http.paths[0].path: Invalid value: "map[path:/]": must be an absolute path

Comparing the ingress templates of several charts which use the notation from above, I see the following differences
RStudio charts use
https://github.com/rstudio/helm/blob/4685fd5504e1361e94f4f1d4fef9d9f41f42accb/charts/rstudio-workbench/templates/ingress.yaml#L32-L37
whereas the other charts we have deployed use
http:
  paths:
  {{- range .paths }}
  - path: {{ .path }}
  {{- if and .pathType (eq $apiVersion "networking.k8s.io/v1") }}
  pathType: {{ .pathType }}

The differences in the RStudio charts that I see are

the use of path: {{ . }}  instead of path: {{ .path }} which might result in the "invalid value" error WRT to the mapping error shown above
the hardcoded pathType (which should not be an issue for the reported problem)

I've forked the chart and deployed RSW using the path: {{ .path }} notation.
This deployment then worked by using (as above)
  - path: /
    pathType: Prefix

Would it make sense to make this change in the all rstudio templates? Or are there any reasons why path: {{ . }} would even be preferable?
I don't have much of an expertise on ingress definitions 😃 but would be happy to use one (canonical) definitions across all charts.

Ahh yes, sorry for the trouble! Thanks for raising! This is almost certainly a regression that we missed in PR review - I'll take a look and get a fix in! 😄

I think the reason that this was an issue is because we were passing the value directly, so this would have worked too in values:
paths:
  - /

https://github.com/rstudio/helm/blob/4685fd5504e1361e94f4f1d4fef9d9f41f42accb/examples/connect/high-availability-traefik/values.yaml#L65
It may be that we are using an older convention. Do you have a link to a reference chart that uses the pattern you talk about? Unfortunately this may end up being a breaking change, so we may need to be a bit more cautious about merging / seeing if we can make it backwards compatible, but I will have a branch up shortly for discussion 😄

Thanks!
We've been using
    - host: <domain>
      paths: [/]

before which stopped working. After that I've tried a bunch of different notations which all failed (differently).
Then I went on inspecting the template and comparing it to other charts.
Here's an example of the Gitea chart which showcases what I was referring to.
Yet I can confirm that
paths:
  - /

works! So using this for the moment (until there's been the change made to the ingress as in #134)

Unfortunately this may end up being a breaking change, so we may need to be a bit more cautious about merging / seeing if we can make it backwards compatible, but I will have a branch up shortly for discussion

Sure, completely understandable.


fix(ssr): fix hydration mismatch warning about mutiple continuous tex…

…t vnodes (#7285)
fix #7285
The following Comp will be rendered as a "
foo
" on the ssr side.
const Comp = defineComponent(() => h('div', ['fo', 'o']))

The "
foo
" will be rendered as a div element with a single child text node in the browser, but the above Comp will be rendered by Vue as a div element with two child vnodes, so will cause a mismatch warning during hydration.

@zhangzhonghe Thanks for providing a fix for this problem. Do you know when it will be merged?

when it merge?

This fixes the problem for the initial hydration, but it throws an error if the text is subsequently updated:

Example

I think most real use cases will need support for updates.


Compare versions tags

Test cases

[x] File missing
[x] Contents mismatch
[x] Release won't run without compare passing
[ ] Matching version passes

File missing:
 ./.github/scripts/compare-release-version.sh
  shell: /usr/bin/bash -e {0}
The VERSION file could not be found. Please create a VERSION file in the version/ directory. The contents of version should match the tag without the v prefix.
Error: Process completed with exit code 1.

Release won't run without compare versions passing
This is the default behaviour.

Matching version passes:
Run ./.github/scripts/compare-release-version.sh
The VERSION file matches the tag. Proceeding with the release of 1.6.0-alpha2.


refactor(FOROME-955): get rid of styled components

https://quantori.atlassian.net/browse/FOROME-955

Num_samples works the same on QA.


Videos in m3u8 format do not play correctly in IOS

I'm trying to play m3u8 videos, when I tried with android I didn't have any problem but in IOS something curious happens. When I enter the app for the first time and select any video, it played but when I select another one, the react-native-video component "loads" the video, but the screen stays black and does not send any error, it is as if something is blocking it. I already specified in the source type property that it is m3u8, pass the cookies that are requested and remove the cache prop, if someone has passed something similar, it would be helpful, thanks.

I'm experiencing the same problem


git hash check failed (hence, can't update scoop)

λ scoop install git
installing git (2.6.1.windows.1)
downloading https://github.com/git-for-windows/git/releases/download/v2.6.1.windows.1/PortableGit-2.6.1-64-bit.7z.exe#/dl.7z...done
checking hash...hash check failed for https://github.com/git-for-windows/git/releases/download/v2.6.1.windows.1/PortableGit-2.6.1-64-bit.7z.exe#/dl.7z. expected: 4e1d75f04d080cba8e798806f0c01f80a5c421445ab9d10c517e5f6dae702543, actual: 5118b86924fd586caa1f31ebda2d71ed4f7c18ba6c0df4a71f06b90d625837e8!

This is not relevant anymore. Could be closed.

Thanks @Cyianor, closing.


[feat] The transaction adapter can inject dynamic db instance

scene：I have a scenario where one database corresponds to one tenant, and i want to use transaction decorator
ClsModule useFactory

ClsModule.forFeatureAsync({
    global: true,
    provide: TENANT_CONNECTION,
    useFactory: async () => {
        return db({ host: '', port: '', /* ... */ });
    },
});

Can implement the scenario I need 👍

but, @nestjs-cls/transactional cannot support similar way, only support single ORM instance

ClsModule.forRoot({
      global: true,
      middleware: { mount: false },
      plugins: [
        new ClsPluginTransactional({
          adapter: new TransactionalAdapterPrisma({
            prismaInjectionToken: PrismaService, // PrismaModule is global
          }),
          enableTransactionProxy: true,
        }),
      ],
    }),

expect this 🌟

ClsModule.forRoot({
      global: true,
      middleware: { mount: false },
      plugins: [
        new ClsPluginTransactional({
          adapter: new TransactionalAdapterPrisma({
            injectInstance: async () => {
              return new PrismaClient({ host: /* ... */ })
            }
          }),
          enableTransactionProxy: true,
        }),
      ],
    }),

i dont know, it is best way

The transactional plugin can't easily handle request-scoped connection, BUT it does support Proxy providers.
If you can provide the instance as a Proxy-provider then it should work.
If you show me how you construct the per-tennant connection currently, I can show you how to translate it into a proxy provider.

I know I can use proxy provicer like this

@InjectableProxy()
export class DB {
  conn: PrismaClient | null;

    constructor(
        @Inject(CLS_REQ) request: Request
    ) {
        const uid = request.header.uid;
        if (!uid) this.conn = null;
        this.conn = new PrismaClient({ url: `mysql://janedoe:mypassword@localhost:3306/mydb-${uid}`  })
    }
}

ClsModule.forFeatureAsync({
    useClass: DB,
});

but I don't know how DB instances work with the @Transactional() decorator

@Injectable()
class UserService {
    constructor(private readonly prismaDB: DB) {}

    @Transactional() // ❌
    async runTransaction() {
        // ... tx
    }
}

Oh, I see. The TransactionAdapterPrisma needs something that acts like the PrismaClient.
Using Proxy providers, you have two options:

Using a Factory Proxy provider that returns the PrismaClient proxy, like so:

ClsModule.forFeatureAsync({
  provide: PrismaClient,
  inject: [CLS_REQ],
  useFactory: (request: Request) => {
    const uid = request.header.uid;
    if (!uid) throw new NoUidInHeaderError()
    return new PrismaClient({ url: `mysql://janedoe:mypassword@localhost:3306/mydb-${uid}`  })
  }
});

Using a Class Proxy provider that extends PrismaClient and overrides the constructor:

@InjectableProxy()
class PrismaService extends PrismaClient {
   constructor(
        @Inject(CLS_REQ) request: Request
    ) {
        const uid = request.header.uid;
        if (!uid) throw new NoUidInHeaderError()
        super({ url: `mysql://janedoe:mypassword@localhost:3306/mydb-${uid}`  })
    }
}

// ...
ClsModule.forFeature(PrismaService)

Now you can use PrismaClient like a regular provider, but it will refer to the request-scoped instance via the proxy and use it in the adapter like so:
adapter: new TransactionalAdapterPrisma({
  prismaInjectionToken: PrismaClient // or PrismaService,
})


[medAbstractProcess] add mesh and volume error messages

This PR allows to standardize a bit our data error messages.
description() is defined in the specific process and is going to be the name of the process.
(Thx @fcollot for the translation of the error messages ^^)
:m:

I like this idea of gathering these redundant messages.
However, from my understanding, the pb here is that we cannot use these methods in a process launched in another thread (quite common case). So I think we should fix this pb before using this PR.

I think I found a solution: https://github.com/LoicCadour/medInria-public/commit/60bc5d26682ad496c08114ca90e80b491cf9a804

Yep that's what i did on https://github.com/Inria-Asclepios/music/pull/231/files
I'm on it, thank you Loic :)

Perfect @LoicCadour i added your commit to this PR ! :)

These functions shouldn't be in medAbstractProcess. They cause a message display in case of a wrong input. This is GUI related stuff and should be in a GUI class like medToolBox.

(ok we discussed this on chat but for the record (certification rules) :) : I suggest that we define an enum with the different errors a process could have, and return the corresponding one in case of error (since that's the whole point of having DTK_SUCCESS = 0, to use the other values for error codes).

Ok I just looked at medRunnableProcess and found this:
void medRunnableProcess::internalRun()
{
    if (d->process) {
        if (d->process->update() == 0)
            emit success (this);
        else
            emit failure (this);
    }
}

This means that the idea of using values different than 0 for errors will work, since medInria tests for DTK_SUCCESS and not DTK_FAILURE. The problem though is that this also means that in the case of running a process this way (instead of directly calling update), we can't retrieve the error code.
Maybe we should be using signals instead. How about we keep the previous idea, but also add a signal processError(int type)  that we could use when it's not possible to get the error code ?
So to clarify: when the process detects an error, it returns the corresponding error code AND emits the error signal with the code.

or modify the failure signal of medRunnableProcess/medJobItem to include the error type as parameter?

I try these ideas and i come back! Thank you all :)

I've updated a new version of this PR. An enum of the error types has been added to medAbstractProcess and the corresponding functions in medToolBox. I tested on a toolbox in musicPlugins and it's working. Tell me if this is better for you :)

Done, thank you @LoicCadour :)

Why are we using the runnableProcess to emit the signal, while the medAbstractProcess could do it?
If we don't encapsulate the process inside a runnableProcess, it might useful, no?
Maybe I am missing something.

well, the system is already there in runnableProcess to emit signals of failure/success. Either we follow this system, or we add a new system for these particular signals. It's maybe better to fit runnableProcess?

Ok, after a long discussion with @LoicCadour and @fcollot this is the last version of this PR. I renamed the signal to "failure(int)" and followed your previous suggestions.
Concerning the toolboxes, if you want to display these kind of specific errors, you just have to add connect (runProcess, SIGNAL (failure   (int)),         this, SLOT   (handleDisplayError(int))); to the medRunnableProcess process. If you do not use medRunnableProcess, you can directly call the medToolBox functions.
:)

Looks good to me, nice one!

Plop, so, the user/dev can call: displayMessageError("blablablabla"); or handleDisplayError(medAbstractProcess::DIMENSION_4D); or do connect (runProcess, SIGNAL (failure   (int)),         this, SLOT   (handleDisplayError(int)));
Tell me if it's ok for you, because i'll need to change some PR :)

thanks


Cache in storage/framework/cache incompatible with Envoyer

Hey Jeff,
Been getting Mix rolled into a project this week and made my first deploy with Envoyer to a dev server. I got a message regarding being unable to find the Mix.json file when using {{ mix(...) }}. The issue occurs on Envoyer because the storage directory is symlinked on deploy.
Is this something you are coordinating with Taylor on handling or should the Mix.json file find a new home? I ended up overriding the cache path to be the root directory and everything works fine now. Just thought I would raise the issue.

I haven't yet talked with Taylor about the default cache path for Laravel projects. We will in time for 5.4. So, yeah, until then, I'd use setCachePath() to set your own.


sourcemap files are not loaded on workbench.aspx

Category

[x] Bug

Describe the bug
I am working on SPFX solution contains a web part as SPA for child components. I did not make any configuration changes but suddenly I found that after gulp serve. when I accessed workbench.html. I can find the webpack folder contains my source map files for debugging.

and I can see on terminal a request for the webpart_file.js.map.

When I tried to access /_layouts/15/workbench.aspx and add the same web part . I find that sourcemap files and webpack folder is not generated in DevTools.

and only manifests.js and /manifests.js.map are requested in the terminal

no request for the dist/webpart_files.js
this is my package.json file
"dependencies": {
"3.0.0": "^3.0.0",
"@fluentui/react": "^7.108.0",
"@material-ui/core": "^4.9.4",
"@material-ui/icons": "^4.9.1",
"@microsoft/sp-core-library": "1.10.0",
"@microsoft/sp-lodash-subset": "1.10.0",
"@microsoft/sp-office-ui-fabric-core": "1.10.0",
"@microsoft/sp-property-pane": "1.10.0",
"@microsoft/sp-webpart-base": "1.10.0",
"@pnp/spfx-controls-react": "1.15.0",
"@react-pdf/renderer": "^1.6.11",
"@tinymce/tinymce-react": "^3.8.1",
"@types/es6-promise": "0.0.33",
"@types/google-maps-react": "^2.0.5",
"@types/jspdf": "^1.3.3",
"@types/lodash": "^4.14.149",
"@types/moment-timezone": "^0.5.13",
"@types/pdfmake": "^0.1.15",
"@types/react": "16.8.8",
"@types/react-dom": "16.8.3",
"@types/webpack-env": "1.13.1",
"@uifabric/fluent-theme": "^7.1.9",
"axios": "^0.19.2",
"chart.js": "^2.9.3",
"csstype": "^2.6.7",
"file-saver": "^2.0.2",
"google-maps-react": "^2.0.2",
"html2canvas": "^1.0.0-rc.7",
"increase-memory-limit": "^1.0.7",
"jspdf": "^2.1.1",
"moment": "^2.25.3",
"moment-timezone": "^0.5.28",
"msal": "^1.3.0",
"nuka-carousel": "^4.6.6",
"office-ui-fabric-react": "^7.98.1",
"pdf-lib": "^1.11.2",
"pdfmake": "^0.1.68",
"pdfmake-unicode": "0.0.1",
"query-string": "^6.13.1",
"react": "16.8.5",
"react-aad-msal": "^2.3.4",
"react-chartjs-2": "^2.9.0",
"react-dom": "16.8.5",
"react-dropzone-component": "^3.2.0",
"react-geocode": "^0.2.1",
"react-google-autocomplete": "^1.1.3",
"react-html-parser": "^2.0.2",
"react-moment": "^0.9.7",
"react-router-dom": "^5.1.2",
"react-to-pdf": "0.0.13",
"react-toastify": "^6.1.0",
"react-visibility-sensor": "5.1.1",
"sp-pnp-js": "^3.0.10",
"spfx-fast-serve": "^1.10.13",
"strip-ansi": "^6.0.0",
"toasted-notes": "^3.2.0",
"uuid": "^8.3.1"
},
"resolutions": {
"@types/react": "16.8.8"
},
"devDependencies": {
<EMAIL_ADDRESS>"0.7.16",
<EMAIL_ADDRESS>"^0.3.5",
"@microsoft/sp-build-web": "1.10.0",
"@microsoft/sp-module-interfaces": "1.10.0",
"@microsoft/sp-tslint-rules": "1.10.0",
"@microsoft/sp-webpart-workbench": "1.10.0",
"@types/enzyme": "^3.9.0",
"@types/jest": "22.2.3",
"ajv": "~5.2.2",
"cross-env": "7.0.2",
"css-loader": "3.4.2",
"css-modules-typescript-loader": "4.0.0",
"del": "5.1.0",
"enzyme": "^3.8.0",
"enzyme-adapter-react-16": "^1.15.5",
"file-loader": "^6.0.0",
"fork-ts-checker-webpack-plugin": "4.1.0",
"gulp": "~3.9.1",
"handlebars": "4.5.0",
"html-to-pdfmake": "^2.0.10",
"identity-obj-proxy": "^3.0.0",
"jest": "22.4.3",
"jest-junit": "5.2.0",
"mock-local-storage": "^1.1.15",
"node-sass": "4.13.1",
"react-test-renderer": "^16.14.0",
"sass-loader": "8.0.2",
"style-loader": "1.1.3",
"ts-jest": "22.4.5",
"ts-loader": "6.2.1",
"webpack": "4.42.0",
"webpack-cli": "3.3.11",
"webpack-dev-server": "3.10.3"
}
It was working till yesterday but suddenly start doing this behavior.
I did not update VScode ore chrome.
Regards,

I got the problem that webpart version deployed on Sharepoint different than what in package-solution.js and package.js .
I fixed the issue and everything is working fine


uses int instead of Number for image and tile sizes

This makes getImageWidth() etc. more handy. but breaks compatibility. also Math.floor() and Math.ceil() was replaced by integer arithmetic.

I'm not sure about some of these changes.  The response types are objects to attempt handling unset fields (while writing, invalid tiffs, or not required).  The methods returning the type Number (as opposed to Integer) are also SHORT or LONG in the TIFF 6.0 spec.  A LONG is a 32 bit unsigned so the positive half of the Java long.  They are set up to try to handle multiple tiff scenarios and int values are retrieved when assumptions are made for certain logic.  Compatibility changes would also require some time to validate on applications using this library.

I realized that Number can contain a 32 bit unsigned LONG. That's why the cast to int is done as late as possible the methods where a java int should be enough, namely getImageWith(), getImageHeight(), getTileWidth(), getTileHeight(), getRowsPerStrip(). imagewidth/height must be there and if FieldTagType.RowsPerStrip is absent to indicate a tiled image, then tilewidth/height must also be there. Maybe there should be a method to check if absolutely necessary fields are present. If you read a tiff without width/height then you can't do anything with it anyway, so why not add something like isValid() that can be checked before accessing it?

You've actually moved the cast earlier rather than later.  The field retrieval methods need to support the spec declared return types.  Generic spec support to support all possible use cases, not just the most common ones for convenience sake.  Validation methods providing information for making decisions would be fine.  Appreciate the pull requests.

Is there a use case where any of getImageWith(), getImageHeight(), getTileWidth(), getTileHeight(), getRowsPerStrip() can be greater than 2^31-1? I think the probability is extremely low and if there is one you could use getNumberEntryValue(FieldTagType.ImageWidth) because it's now public ;)  Therefore getImageWidth() etc. are only convenience wrappers for 99.9% of all use cases. But it's your code, if you don't like this pull request we should drop it (maybe keep the int arithmetic instead of Math.ceil())


Read-only file system error on mac

Hey, the app doesn't seem to work on macs (intel-based). I got the following error when circling a text.
gustavo@Gustavos-MacBook-Pro ~ % /var/folders/2k/0r_v629x3311ndhjl8z6w3_c0000gn/T/AppTranslocation/7E4897D4-9A2E-4DB4-B758-3C5CAC8A30BF/d/game2text.app/Contents/MacOS/game2text ; exit;
Error: cannot record audio with selected device [Errno -9981] Input overflowed
Traceback (most recent call last):
  File "src/gevent/greenlet.py", line 906, in gevent._gevent_cgreenlet.Greenlet.run
  File "eel/__init__.py", line 259, in _process_message
  File "game2text.py", line 32, in recognize_image
  File "ocr.py", line 19, in detect_and_log
  File "ocr.py", line 35, in image_to_text
  File "util.py", line 57, in base64_to_image
  File "util.py", line 63, in base64_to_image_path
OSError: [Errno 30] Read-only file system: '/private/var/folders/2k/0r_v629x3311ndhjl8z6w3_c0000gn/T/AppTranslocation/7E4897D4-9A2E-4DB4-B758-3C5CAC8A30BF/d/game2text.app/Contents/MacOS/logs/images/temp.png'
2022-09-07T13:48:05Z <Greenlet at 0x7fec218d2050: _process_message({'call': 31.43319692611729, 'name': 'recognize_ima, <geventwebsocket.websocket.WebSocket object at 0x7)> failed with OSError

You need to move it to the applications folder.

Ooh I can't believe I missed something that obvious! Thanks a lot


Make necessary changes to update to material-ui v5 RC

The just-released material-ui RC-0 includes the new package names that material-ui is transitioning to -- material-ui to mui. This PR updates the packages to that new version and makes the necessary renames in the code.

Shout out to @Mitch528 for publishing this branch to NPM until this PR is merged https://www.npmjs.com/package/@mitch528/mdi-material-ui

I've noticed that icons imported via mdi-material-ui using a color prop doesn't get the correct theme settings for MUI5.  Does fixing the imports in this PR get this correct, or do you think there's more to do there?

@leMaik any change to release this for the compatibility of material-ui v5

This also fixes #43.

@olee Sorry, this went below my radar. :sweat:
@Philipp91 Thank you for jumping in! :tada: I'll release a new version shortly.

@ShotSkydiver Thanks a lot for updating this :pray:

Awesome, thank you, my icon colors are back to normal now!


could you release the trained model

Hi, @DylanWusee ,
I've tried this package and the training was converged at epoch 12 (accuracy = 0.888710) and then evaluated it, the resulting total accuracy is 0.894246, which is lower than the reported one (0.925)
Would you release the trained model for better comparison?
THX!

hi,I have the same problem.The accuracy at epoch 101 is 0.913306. This is the best result that I can get.  Please share the pre-trained model.

Hi,
I also tried to train the model and only get 91.6% for the ModelNet40 classification task. Could you please release the pre-trained model?

Hi,
Thank you for your interest in our work. The original implementation is in Tensorflow. It is possible that the pytorch version can give slightly different results. I will try to release a pre-trained model ASAP.
Thank you very much.
Best,
Dylan

Hi Dylan,
Thank you for your response. I also notice that the input point cloud of this code are 2048 points instead of 1024 points. I print the shape of the points variable in both training code and the testing code. Do I misunderstand the code?
Regards,
Jinyang

嗨迪伦，
感谢您的答复。我还注意到该代码的输入点云为2048点而不是1024点。我在训练代码和测试代码中都打印了点变量的形状。我会误解代码吗？
此致
金阳

Excuse me, How to modify the number of input points?

I got 91.6% for the ModelNet40.......

The pretrained model is relased.
The previous code has a bug in the inverse density scale, which might be the reason that the model does not reach the accuracy in the paper.
However, I have updated the code. And, the new code should be able to achieve 92+%.
Beside, I also upload a pretrained model. You can use it as a reference.


Adds info about setting a custom project path via env var

Summary

Use Cases

Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.

Combining with #458 in a single PR.


Prune runtime/net_policies.go

Many of the tests covered by net_policies.go are already implemented in cilium-cli.
L7 HTTP method tests are ported in this CLI PR: https://github.com/cilium/cilium-cli/pull/1069
Some of the more complex test cases were not removed, will discuss how to port these and get back to them soon.

/test-runtime

/test-runtime

/test-runtime


Current Location Pin Icon

I am currently developing an application that uses MapsUi and would like to know if it is possible to change the icon that represents the current location of the user. If so, how is this done.

No, in the moment there are only two given icons (one for still, one for moving).
Do you need such a functionality?

Currently I am working on a project that uses OpenStreetMap. The project uses a service that provides me with coordinates that use to dynamically define some pins on the map. I know that it is possible to be constantly acquiring the current coordinates of the user and my idea is to set a pin with a custom image in the current location and to update whenever there is movement.
Is this the better approach, or you have another thought about it ?

The MapView Current-Location is additional functionality offered by the MapView (on top of the MapControl). It is always possible to add such functionality yourself on top of the MapControl.
Then again it also interesting for us to hear what kind of functionality users want.

I could add the functionality, that you could add your own icon for the MyLocationLayer, which is a normal Mapsui Layer with additionaly functionality. Should I?

@charlenni Do you mean you would need a modified version of the Mapsui Layer?

At this time it is not very urgent to add this functionality. I'll test it the way I explained and see if it works, but I think it's an interesting little map customization. I'll be aware if you add this functionality

@pauldendulk As you can see, MyLocationLayer is a normal MemoryLayer with added functionality. The only thing it uses from Forms is the animation. If you set a new location, it moves smooth to this new location (as asked for in #615). It is, that the map doesn't "jump" araound.

ah, I see. Whether you should change this is up to you. It is certainly no necessity. There are probably other things you could better spent your time on. Also, perhaps @Afonsoxvp could send a PR to fix this. The code seems easy enough.
Perhaps a better way to improve this is to use a ThemeStyle. The ThemeStyle gets a method with the feature as argument. The feature should contain all possibly relevant information (like isMoving). The method can then decide whichever SymbolStyle to return based on the position information.
If you create a setter for the ThemeStyle (or IStyle) users can attach their own symbology and there will be little else to make configurable.


Previewing Lip Tracker Freezes Parameters

Keeping this short because I don't know what else to say about this. When you go to the SRanipal tab in the Quick Menu, and you preview the Lip Tracker camera, it freezes the parameter values (as seen visually on the avatar and from the Debug panel).
For example:

The LeftEyeOpen and RightEyeOpen get stuck at 1
The TongueX and TongueY got stuck at NaN
CombinedEyeWiden was stuck at 0.05

It seems to happen to me only when I preview the Lip Tracker camera, but a simple restart fixes it. The log and console do not output any error about this at all, so I won't upload a log file unless requested. Any thing else needed and I'll be happy to get back to you!

I'll be removing the lip camera functionality in the next update for this very reason. The runtime really does not like me streaming live video from it. However, I may replace the image in the quickmenu for a live selfie view of your current avatar or simply a mirror to allow people to check if their face is tracking properly

I did some testing to see if it actually see if this was on the SRanipal's end. I set up a basic Unity 2019.4.16f1 project, imported the ViveSR SDK, and added the Lip Framework Prefab.
I then setup 2 scripts: LipCameraViewer.cs and LipDataViewer.cs
I tried to set them up as close as you did, and LipData is v2. The two classes should be obvious, but I'll briefly explain what they do. LipCameraViewer will render the latest image from the tracker to a RawImage in a canvas. LipDataViewer will get the LipData and LipShapes/LipWeights and then output the TongueRoll property to a UI Text inside the same Canvas.
From my testing, I can see that running an update frame to get camera output while getting data does NOT effect anything on the SRanipal end at all. The data values seemed to still work and the camera output didn't lag or anything like that.

Issue is, the mod has another step of invoking the functions on the il2cpp domain. This causes some instability and minor lag which I feel is probably to blame. My code essentially does the same as you've done in yours so I'm not quite sure what's doing it at this point

Issue is, the mod has another step of invoking the functions on the il2cpp domain.

Changed the Scripting Backend to IL2CPP and set the Api Compatibility Level to .Net 4.x. The same results occur. TongueRoll seems to work properly and the images are still rendering properly, There seems to be no issues. If I need to, I can write a quick mod to link into VRChat and do the same thing for testing, but it wouldn't explain why it works fine in the Editor and not in-game.
A guess could be because you're modding it in with MelonLoader rather than building from unity (which I don't expect you to be able to do haha), but I don't see why that would make a difference.

I'm not quite sure what's doing it at this point

Here's my guess.
If you look at line 14 in LipTrackingMenuPage.cs ( url ), you can see it's setting the LipEnabled value to whatever b is. I think b is false which is causing all the parameters to stop working. This could explain why the parameters default to some value rather than freezing at what they last were. I don't know what event TrackingEnabled is, but it's odd how it wouldn't set the value back to true, maybe the true event never hits it because the class "ends?" If you can, I'd try registering the event somewhere else in a static class (like MainMod.cs) since the class LipTrackingMenuPage is non-static.


Tool.Handle type is any instead of BasePart

The part inside a tool called Handle is always going to be a BasePart, but the LSP gives it the any designation.

It is not guaranteed that a Tool object has a BasePart (or any instance) inside it called "Handle". You can create a Tool object and parent a Script (or any other object) to it called "Handle". In that case, the type BasePart would be inaccurate. You can also have no descendants in the Tool, so Tool.Handle would be nil.


Clean build previous artifacts after successful build

Closes #49
This is a recreation of my previous PR #61. I had to recreate this because I mess up (to my knowledge, irreversibly) on the last (master) branch. I apologize for any inconvenience I caused.
Checklist

[x] Updated CHANGELOG.md describing pertinent changes.
[ ] Updated README.md with pertinent info (may not always apply).

Superseded by #108

Superseded by #108


B1500: Add get_cmd to various nplc settings

Setting proper timeout requires precise knowledge of averaging/integration time. This PR adds get_cmd to various NPLC settings.
This PR is needed in #2001
@astafan8 @jenshnielsen

Had a quick chat with Sarat. This PR will break the ways they use use_nplc_for_high_speed_adc and other modes. For setting the modes they do not give any value to the parameter. They just use b1500.use_nplc_for_high_speed_adc(). And the system automatically takes default value of the coefficient. With this PR this will not be possible because b1500.use_nplc_for_high_speed_adc() will try to get the value.
SO ABANDONING THIS PR. Sorry for eating up your time @jenshnielsen.
For  #2001 there is easier solution which I will implement in that PR


Separate Mvcc Reader

What have you changed? (mandatory)
This PR separates the mvcc::MvccReader into multiples:

[x] mvcc::PointGetter: Provides mvcc point get operation.
[x] mvcc::ForwardSeeker: Provides mvcc forward seek operation.
[ ] mvcc::ReverseSeeker: Provides mvcc reverse seek operation.
[x] mvcc::CFReader: Other mvcc operations not listed above are managed by this struct.

Since not every piece of MvccReader is separated, MvccReader still exists for now. However the separated part has been either removed from MvccReader, or became private.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
No need.
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

@MyonKeminta I believe that test_scan_detail does not fail accidentally. It is related to the change of the scan algorithm. PTAL.

Got it.

@MyonKeminta Don't worry, there should be no logic errors. I guess it is the statistics different, for example, we may "seek" less now.

@ngaut Nice, but I cannot find it in header files. Are there references?

/run-integration-test

/run-integration-tests

/run-integration-ddl-test

/run-integration-ddl-test

/run-integration-ddl-test

/run-integration-tests

Hi, @jl2005 we have already extracted the part which greatly improves performance from this PR into master. The rest contributions in this PR are mostly better code arrangements, which is trivial and is not our high priority currently. You may expect it to be merged in late 2018.


Enable keepalive in rollbar-gem

details in https://app.clubhouse.io/rollbar/story/8493/enable-keepalive-in-rollbar-gem

Hi there, I’m closing out all issues opened before 2018 that haven’t had any activity on them since the start of this year. If this is still an issue for you, please comment here and we can reopen this. Thanks!


SSH Tunneling not works if ssh has cloudflare based authentication

Description
When I use ssh host which has cloudflare based authentication ssh tunnel is failing. Can you fix this?
DBeaver Version
23.2.4
Operating System
ubuntu 22.04
Database and driver
Postgres and version 13
Steps to reproduce
Enable cloudflare based ssh tunneling on remote db.
try to connect the database with ssh tunneling option.
Additional context
No response

Hello @SuriyaJin

ssh tunnel is failing.

Could you please provide the log files? You can find instructions here: https://dbeaver.com/docs/wiki/Log-files/. Thank you.

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.


[Feature Request]: add the color theme to the app. which will increase the UI apperance.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
To increase the UI apperance and show the app purpose using a color theme.
Use Case
A color theme makes the app easier to use by quickly showing ambulance availability and status. It helps users make faster decisions and understand the situation at a glance, while also improving the app's look and feel.
Benefits
It improves user experience
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

color theme like what? can you give example?

By using gradients in the color theme to show ambulance status and urgency levels. like green means available, and the darker the green, the more ready the ambulance is. Yellow shows the status, with light yellow meaning the ambulance is on the way and darker yellow meaning it’s getting ready. Red shows urgency, where darker red means it’s very urgent. These gradients make it easier for users to quickly see the ambulance availability and urgency.

Thank you

I CAN FIX THIS IF YOU CAN ASSINGED IT TO ME . THANK YOU

I want to do it. Sorry

I CAN FIX THIS IF YOU CAN ASSINGED IT TO ME . THANK YOU

assigned already

okay

can you give level 2 I improve the UI with decent colors and also styles the borders


Update/danfojs version

This PR updates Danfojs version to the latest Typescript release

First of all: Amazing work transitioning to TypeScript! The PR is trying to merge into the dev branch. Is that intentional? The dev branch seem to be quite far behind the main branch.

First of all: Amazing work transitioning to TypeScript! The PR is trying to merge into the dev branch. Is that intentional? The dev branch seem to be quite far behind the main branch.

Thanks! Oh I didn't realize. Is the main branch the latest? I can update that.

Yes main should be the most up-to-date branch.

WON'T MERGE. I'll rebase it and push a new PR


Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/build.js'.

Good afternoon,
This is my first post on GiT so please bear with me if I haven't provided you with enough information to answer my query.
I ran in to the following error:
Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/build.js'.
I have attached an image below showing exactly what happened:

Thanks in advance!

Hi @Fnatical, before opening an issue you always read a projects CONTRIBUTING.md. The GitHub UI tells you do so when creating a new issue. Without following that document we cannot help you.

Hi,
I'm seeing the same issue
-> lsb_release -a && sh node-sass.sh
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 7.9 (wheezy)
Release:	7.9
Codename:	wheezy
3.8.3
v5.10.0
{ http_parser: '2.6.2',
  node: '5.10.0',
  v8: '<IP_ADDRESS>',
  uv: '1.8.0',
  zlib: '1.2.8',
  ares: '1.10.1-DEV',
  icu: '56.1',
  modules: '47',
  openssl: '1.0.2g' }
linux
x64
npm WARN deprecated<EMAIL_ADDRESS>this package has been reintegrated into npm and is now out of date with respect to npm
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>graceful-fs version 3 and before will fail on newer node releases. Please update to<EMAIL_ADDRESS>as soon as possible.

><EMAIL_ADDRESS>install /root/node_modules/node-sass
> node scripts/install.js

module.js:341
    throw err;
    ^

Error: Cannot find module '/root/node_modules/node-sass/scripts/install.js'
    at Function.Module._resolveFilename (module.js:339:15)
    at Function.Module._load (module.js:290:25)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:146:18)
    at node.js:404:3

node-sass.sh: 15: node-sass.sh: ./node_modules/.bin/node-sass: not found
/root/node_modules/node-sass/lib/extensions.js:158
    throw new Error([
    ^

Error: The `libsass` binding was not found in /root/node_modules/node-sass/vendor/linux-x64-47/binding.node
This usually happens because your node version has changed.
Run `npm rebuild node-sass` to build the binding for your current node version.
    at Object.sass.getBinaryPath (/root/node_modules/node-sass/lib/extensions.js:158:11)
    at Object.<anonymous> (/root/node_modules/node-sass/lib/index.js:16:36)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at [eval]:1:13
    at Object.exports.runInThisContext (vm.js:54:17)
/root/node_modules/node-sass/lib/extensions.js:158
    throw new Error([
    ^

Error: The `libsass` binding was not found in /root/node_modules/node-sass/vendor/linux-x64-47/binding.node
This usually happens because your node version has changed.
Run `npm rebuild node-sass` to build the binding for your current node version.
    at Object.sass.getBinaryPath (/root/node_modules/node-sass/lib/extensions.js:158:11)
    at Object.<anonymous> (/root/node_modules/node-sass/lib/index.js:16:36)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at [eval]:1:13
    at Object.exports.runInThisContext (vm.js:54:17)
node-sass.sh: 22: cd: can't cd to node_modules/node-sass/vendor
/root/node_modules/node-sass/lib/extensions.js:158
    throw new Error([
    ^

Error: The `libsass` binding was not found in /root/node_modules/node-sass/vendor/linux-x64-47/binding.node
This usually happens because your node version has changed.
Run `npm rebuild node-sass` to build the binding for your current node version.
    at Object.sass.getBinaryPath (/root/node_modules/node-sass/lib/extensions.js:158:11)
    at Object.<anonymous> (/root/node_modules/node-sass/lib/index.js:16:36)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at [eval]:1:13
    at Object.exports.runInThisContext (vm.js:54:17)

@jippi you issue is covered by our documentation.
In the future please read the CONTRIBUTING.md before opening an issue.

@xzyfer I've read it, i don't see it covered - maybe point me in the right direction instead of just closing the issue?

@jippi I closed the issue for two reasons:

the original issue did not, and since not, supplied the required information to debug
your issue is not related to the original issue (as seen from the different error messages)

I did not supply a direct link to encourage you to (re-)read the relevant documentation.
Your issue covered in the TROUBLESHOOTING.md which is linked to from the contributing documentation.


Fix stop automation doc

What is being addressed

There're some issues stopping VMs in Azure Automation so updating the doc with a workaround.
Remove instruction to build the mlflow image as it's done internally by the bundle.

/test-force-approve


Stacking two embedding layers

This is a crosspost from https://stackoverflow.com/questions/46032700/concatenate-embedding-layers. Would appreciate it if you could answer it there. The question is repeated below:
I'm trying to create a model which has words as inputs. Most of those words are in the glove word vector set (~50000). However, some of the frequent words are not (~1000). The question is, how do I concatenate the following two embedding layers to create one giant Embedding lookup table?
trained_em = Embedding(len_glove_words, 50,
                       weights=np.array([word2glove[w] for w in words_in_glove]),
                       trainable=False)
untrained_em = Embedding(len(word2num)-len_glove_words, 50)

As far as I understand these are simply two lookup tables with same number of dimensions. So I'm hoping that there is a way to stack these two lookup tables.

@sachinruk you could create a function that breaks the input containing words from both the bag, feed them independently to the two embedding layers and then concatenate them back into a single Keras tensor. You could write such a function using Keras Backend (mainly using keras.backend.gather and keras.backend.concatenate). This way, it will still support back propagation of losses for the untrained embedding layer.

@samre12 is there any chance you could provide an example? Sorry not the strongest with Keras just yet.

@samre12 figured it out. Thanks for the hints! Answer is here: https://stackoverflow.com/a/46149153/2530674


Not able to build

Describe the bug
A clear and concise description of what the bug is.
Hi so everything works i was able to create the site and everyting, but when i try to build the site i get this errors:
Start building sites …
hugo v0.101.0+extended linux/amd64 BuildDate=unknown
WARN 2022/07/06 18:21:33 found no layout file for "HTML" for kind "home": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2022/07/06 18:21:33 found no layout file for "HTML" for kind "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2022/07/06 18:21:33 found no layout file for "HTML" for kind "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2022/07/06 18:21:33 found no layout file for "HTML" for kind "section": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2022/07/06 18:21:33 found no layout file for "HTML" for kind "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2022/07/06 18:21:33 found no layout file for "HTML" for kind "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
i have v0.101 extended, so i dont know what the issue is.
Could you help?sazs

I also have those warnings, but they are just warnings the command hugo server -D still works


Chain keeps stalling

3rd time in less than 2 days the chain stalls and rejects blocks.
What are you going to do to fix this? What ever you're doing to stuff a block in, is not a solution.

I am sorry, there was an issue with the hamsi code, we fixed it via soft-fork in src/primitives.block.cpp


Adding Pytest to the project

The following changes were made:

Edited bootstrap.h to automatically install pyTest when bringing up a new VM

Pipefile & Pipefile.lock are updated automatically after installing Pytest in the enviroment
(using pipenv install --dev pytest)

Created pytest.yml to run our tests automatically on PRs

Created pytest.ini to configure Django and pyTest settings

Resolves #61

After many attempts and research - i could not find a way to successfully run pyTest in my environment.
from what i see, pyTest is running on the PR, but it displays an error similar to the one i am receiving.
I'm using the command pipenv run python -m pytest -v:

If i delete the file "pyTest.ini" (where DJANGO_SETTINGS_MODULE is referenced), and then run pipenv run python -m pytest -v,
I get no errors - but obviously pyTest cannot detect the test files and nothing happens:

Help will be appreciated.

@LeonRabinovich  what you decribe looks a bit like what I experienced when I tried running this with new models but old migration files and existing DB by accident.
Try:

carefully delete DB (or not carefully destroy VM and then delete DB from root folder)
delete all migration files that are not up to date
introduce the up-to-date migration files

@LeonRabinovich what you describe looks a bit like what I experienced when I tried running this with new models but old migration files and existing DB by accident.
Try:
1. carefully delete DB (or not carefully destroy VM and then delete DB from root folder)

2. delete all migration files that are not up to date

3. introduce the up-to-date migration files

I'll try this, thanks

@LeonRabinovich  Also, try to leave in tests.py the most trivial test
I used this:

If this doesn't work the issue is probably with your models.py file

@LeonRabinovich Also, try to leave in tests.py the most trivial test
I used this:

If this doesn't work the issue is probably with your models.py file

Thanks, it did work :)
I had to re-migrate everything, and also it seems only installed pyTest and not Django-pyTest.
now eveyrthing works, and the tests passed.


Feature: Diagnostics view

Is your feature request related to a problem? Please describe.
Creating a diagnostic view in the webui and desktop app
Describe the solution you'd like
Additional metrics desired can be found in https://github.com/ipfs/ipfs-webui/issues/1220, but this issue covers the diagnostics view we wish to have in the webui and ipfs-desktop.
This view would not be focused on metrics, instead, view #1220 for that. This view would be focused on process/tooling on top of those metrics and/or other data.
Describe alternatives you've considered

https://pl-diagnose.on.fleek.co/
https://ipfs-check.on.fleek.co/

Additional context
Ideas for supported functionality:

[ ] network connection view to support users determining why or whether they're experiencing network connectivity issues.

see https://github.com/ipfs/ipfs-desktop/search?q=context+deadline+exceeded&type=issues

[ ] We could use IPFS Desktop to get some diagnostic data about the gateways with user's permission. That would allow us to get some info on why are is the gateway so slow at some times and really fast on others. (https://github.com/ipfs/ipfs-desktop/issues/1140)
[ ] command line interface to ipfs inside ipfs-desktop - ipfs/ipfs-webui#1931
[ ] Health Check - See https://github.com/ipfs/ipfs-desktop/issues/2181

[ ] check if there are public providers for a CID
[ ] Are those providers dialable from my node
[ ] Trace route for DHT

Looking for community and IPFS Implementers' feedback on this issue.
Here are some questions to help get the ideas flowing.
What kind of diagnostics should we focus on

How do you currently diagnose issues with your IPFS node/CID/etc?

Could this process be improved with a targeted solution in IPFS desktop and the webui?
What is the most difficult part of this process?
What problems have you run into that you have been unable to solve yourself, and how could a diagnostics view help to prevent a similar situation in the future?

What/Why/How

Do we need a separate diagnostics & metrics view? Would a single diagnostics view that allows us to view the metrics we need be sufficient?
What problems do you currently have that a specific diagnostics view in the webui and IPFS desktop could help you solve?
Do you agree/disagree with the Ideas for supported functionality list above?
Would you use the diagnostics view instead of your current tool of choice if the metrics were available in the IPFS desktop and webui?

@lidel @BigLep what do you think about prompting current IPFS desktop & webui issues to respond on this and the metrics (#1220) view issue?
@lidel do we have any current methods to prompt users for one-time feedback? i.e. surveys, etc..?

Thanks for the great insight! I think adding pl-diagnose into the webui is a great idea. If pl-diagnose's components were abstracted, we could both benefit from the work already done on pl-diagnose.
@laurentsenta are you still actively working on that project? How hard do you think it would be to pull out some components from pl-diagnose so we could re-use them in ipfs-webui?
@lidel @laurentsenta would it make sense to fully consume pl-diagnose (the https://pl-diagnose.on.fleek.co/#/diagnose/access-content page) as a screen in the webui and ipfs-desktop? If there are still plans to expand pl-diagnose, maybe it doesn't make sense, but hosting pl-diagnose within the webui, desktop app, and at ipns://webui.ipfs.io/#/diagnose seems like a great idea to me.
I feel like it would be a natural progression for the webui and desktop diagnostics screen to expand beyond simply troubleshooting content accessibility, but I would love to hear what the future for pl-diagnose is.

Cc @aschmahmann since would ideally like to see convergence around one main IPFS diagnostics tool in EngRes.
I believe https://github.com/ipfs-shipyard/ipfs-check has been the one to get the most investment.

Agreed, from a practical perspective I'd rather only have to maintain one tool.
Happy to take PRs to ipfs-check. Will also allow some feedback from the other folks who have contributed to ipfs-check

@laurentsenta it would be great if we could simply import pl-diagnose as a full-page view. This is probably the quickest and easiest option.
Moving forward, having that view be separated into different imported components would allow us to control the flow better as our diagnostic needs change .
It sounds like you've already got four react components, so releasing those as a library to npm should probably be our first step.

re: user research around webui/desktop diagnostics view
I'm still hoping to get more survey responses as well as source additional participants for addt'l user testing but will deliver some research findings next week, fyi

Cc @aschmahmann since would ideally like to see convergence around one main IPFS diagnostics tool in EngRes. I believe https://github.com/ipfs-shipyard/ipfs-check has been the one to get the most investment.

Agreed, from a practical perspective I'd rather only have to maintain one tool.
Happy to take PRs to ipfs-check. Will also allow some feedback from the other folks who have contributed to ipfs-check

@BigLep @tinytb @lidel
I agree that maintaining a single tool is ideal. The problem is where that tool exists in the stack and what diagnostics tooling is needed specifically within IPFS desktop and webui because they may not make sense elsewhere:

CLI/REPL inside desktop/webui to assist with learning/debugging/other
Network troubleshooting of the kubo daemon controlled by ipfs desktop
Diagnostics tooling for troubleshooting of MFS files added via IPFS Desktop
Other issues we're having trouble gathering (maybe they don't exist?)

To get our "one tool" we need to consolidate ipfs-check, pl-diagnose, and the needs of webui+desktop. If ipfs-check is the best place for that, great. I don't see that being our(webui & ipfs in general) path to success, but I'm open to being convinced.
If we need to separate the discussion of general ipfs diagnostics tooling, and this issue -- which I intended to be specific to the tooling that would be useful in IPFS Desktop -- then let's do that. If we've reached the end of the discussion because there aren't any diagnostics tooling that make the most sense inside of IPFS desktop and webui, then I'm good with closing this issue and pushing on consolidation of diagnostics tooling inside of ipfs-check.

Related: @juliaxbow survey and user research:

The users we heard back from seem to want diagnostics inside desktop/webui. So are the following next steps valid?

Consolidate diagnostics in ipfs-check
Render ipfs-check inside webui&desktop

Or should we skip the second item altogether?

Consolidate diagnostics in ipfs-check

Yes! I think we all agree on this one.

Render ipfs-check inside webui&desktop

I think it would be useful to have something inside the webui. The challenge is that ipfs-check relies on its own backend so if you render it do you use the centralised backend? Because technically if you have access to the webui, you probably have access to Kubo's RPC API which you can use to debug too.

FYI to all following this thread that this "diagnostics view" may well turn more into an effort of building some consolidated diagnostics tooling rather than a specific "diagnostics view" within webui. I feel like both efforts have merit, but from discussions during roadmap planning that seems like where we're going
p.s. discovered a tool that I hadn't seen before, https://dag.ipfs.tech/, which has source at https://github.com/ipfs-shipyard/ipfs-dag-builder-vis, that would make sense in either effort

@lidel We could revive this issue for integrating check.ipfs.network into ipfs-webui view


Scheherazade ES has JP text when english is selected

???

Turned out to be a glitch in the data loader, will resolve prior to release.


[iOS/XCode] The Dart VM service protocol port for debugging does not match the attempted connection port from the flutter run CLI.

Steps to reproduce
Primarily,

On MacOS XCode 15.3, create a sample Flutter app.
Run flutter run with a physical iOS device connected/attached.
Observe the issue: debugging is not possible.

Alternatively, use the following VSC launch.json configuration:
        {
            "name": "Launch Flutter",
            "type": "dart",
            "request": "launch",
            "flutterMode": "debug",
            "cwd": "${workspaceFolder}",
        },

Expected results
The Dart VM service protocol for debugging should listen to the same port as the connecting pair.
For example: http://<IP_ADDRESS>:49241/-hJkog_kS0Y=/
Actual results
The Dart VM service protocol port for debugging and the attempted connection port are different, resulting in an error and an inability to debug iOS devices.
For example:
From XCode:
flutter: The Dart VM service is listening on http://<IP_ADDRESS>:49241/-hJkog_kS0Y=/
and from the CLI:
Error connecting to the service protocol: failed to connect to http://<IP_ADDRESS>:49426/-hJkog_kS0Y=/
Port 49241 and port 49426 do not match.
Code sample
Code sample
Any Flutter application will do.

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
flutter run

Flutter Doctor output
Doctor output
alden@aldens-mac-mini hylist % flutter doctor -v
[✓] Flutter (Channel stable, 3.19.5, on macOS 14.4.1 23E224 darwin-arm64, locale en-US)
    • Flutter version 3.19.5 on channel stable at /Users/alden/development/flutter
    • Upstream repository https://github.com/flutter/flutter
    • Framework revision 300451adae (2 weeks ago), 2024-03-27 21:54:07 -0500
    • Engine revision e76c956498
    • Dart version 3.3.3
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/alden/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.88.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.86.0

[✓] Connected device (3 available)
    • Alden’s iPad (mobile) • 00008103-000949C90A88A01E • ios            • iOS 17.4.1 21E236
    • macOS (desktop)       • macos                     • darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)          • chrome                    • web-javascript • Google Chrome 123.0.6312.122

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @kNoAPP
It doesn't seem to be clear about the reproduction steps. I'm not sure which step should be done on XCode and which step on VSCode. Could you please share more details? If there is a recorded video, it would be helpful.

Fixed in v3.19.6.


Split country and timezone inputs to fix deprecated way of passing priority countries to country input.

fixes https://github.com/heartcombo/simple_form/issues/1781

thanks for doing the pr @nashby

@nashby thanks for doing this, can you @carlosantoniodasilva please approve and merge it or write how it can be improved? Would be happy to have it, we receive to many deprecation warnings. Thanks in advance.

@carlosantoniodasilva thanks for review! Totally makes sense to keep PriorityInput so I just put it back and just used separate calls to render country and priority inputs. Does it make sense now?

I'd love to get this deprecation warning out of my CI logs if someone has time to merge this! 🙏

@carlosantoniodasilva sorry, merged it without your approval but I think it's good to finally do it so we can fix this annoying warnings.
@jaredmoody @leifcr @leifcr it's merged. Feel free to test it with master branch and report any issue!