Option to add metadata to the thumbnail.

Hi, me again. I have another idea, that I hope can improve the plugin.
The option to add metadata to the code block will help a lot when searching for videos e.g. sometimes I remember the name of a video (or the name of the channel), but not the name of the note where I placed the thumbnail. Currently we can't search for neither the name of the channel or of the video of thumbnails.
Solution: And an option to add the name of the video and of the channel to the code block as metadata. That way the thumbnail stays the same (visually), but we now have the option to search for the thumbnail information.
The following picture shows how we can search for text inside the code block of the thumbnail, but currently we can only search for the link itself, which isn't that helpful.

Hope you like the idea.

That's a great point, I hadn't thought about searchability.
Funnily enough, there's a feature that does this already. If you turn on Save Thumbnail Info in the settings, it adds all that info into the code block. I was just thinking about avoiding the API call, though, not how that would also make it more searchable. Maybe I'll make it enabled by default because of that.

I just gave it a try, and this is just what I was looking for.
Glad you liked the idea :)


Guidance switcher missing save confirmation message

When you update guidance choices, it's not clear that these have saved (although they have). We should have a message bar saying something like "Guidance selections updated" or "Guidance choices saved"

Works fine. Closing out


Unit tests don't pass on CI

I forked orchard repo and connected it with AppVeyor CI to provide autobuilds and tests for my commits, but I encountered some problems with tests.
The first issue is that test for Scripting.DLR Module depend from IronRuby on master branch:
Error during evaluation of '5 and nil': Failed to load language 'IronRuby': Could not load file or assembly 'IronRuby.Libraries, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7f709c5b713576e1' or one of its dependencies. The system cannot find the file specified.

Build with issue
Maybe they should be with "Explicit" attribute or check if exists IronRuby and then run or mark as "Inconclusive".
The second one appears when I want to build branch 1.x and looks like below:
nunit-console.exe "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Core.Tests.dll" "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Framework.Tests.dll" "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Tests.Modules.dll" "C:\projects\orchard\src\Orchard.Web\Modules\Orchard.Projections\Tests\bin\Orchard.Projections.Tests.dll" "C:\projects\orchard\src\Orchard.Web\Modules\Orchard.Tokens\Tests\bin\Orchard.Tokens.Tests.dll" "C:\projects\orchard\src\Orchard.Web.Tests\bin\Release\Orchard.Web.Tests.dll" "C:\projects\orchard\src\Tools\Orchard.Tests\bin\Release\Orchard.Tests.dll"
NUnit-Console version <IP_ADDRESS>83
Copyright (C) 2002-2012 Charlie Poole.
Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.
Copyright (C) 2000-2002 Philip Craig.
All Rights Reserved.

Runtime Environment -
   OS Version: Microsoft Windows NT 6.2.9200.0
  CLR Version: 2.0.50727.8009 ( Net 3.5 )

ProcessModel: Default    DomainUsage: Multiple
Execution Runtime: net-3.5
Unhandled Exception:
System.IO.FileLoadException: Could not load file or assembly 'Autofac, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=17863af14b0044da' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
File name: 'Autofac, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=17863af14b0044da'

Server stack trace:
   at System.Reflection.RuntimeAssembly.GetExportedTypes(RuntimeAssembly assembly, ObjectHandleOnStack retTypes)
   at System.Reflection.RuntimeAssembly.GetExportedTypes()
   at NUnit.Core.CoreExtensions.InstallAdhocExtensions(Assembly assembly)
   at NUnit.Core.Builders.TestAssemblyBuilder.Load(String path)
   at NUnit.Core.Builders.TestAssemblyBuilder.Build(String assemblyName, Boolean autoSuites)
   at NUnit.Core.Builders.TestAssemblyBuilder.Build(String assemblyName, String testName, Boolean autoSuites)
   at NUnit.Core.TestSuiteBuilder.BuildSingleAssembly(TestPackage package)
   at NUnit.Core.TestSuiteBuilder.Build(TestPackage package)
   at NUnit.Core.SimpleTestRunner.Load(TestPackage package)
   at NUnit.Core.ProxyTestRunner.Load(TestPackage package)
   at NUnit.Core.ProxyTestRunner.Load(TestPackage package)
   at NUnit.Core.RemoteTestRunner.Load(TestPackage package)
   at System.Runtime.Remoting.Messaging.StackBuilderSink._PrivateProcessMessage(IntPtr md, Object[] args, Object server, Object[]& outArgs)
   at System.Runtime.Remoting.Messaging.StackBuilderSink.SyncProcessMessage(IMessage msg)

Exception rethrown at [0]:
   at System.Runtime.Remoting.Proxies.RealProxy.HandleReturnMessage(IMessage reqMsg, IMessage retMsg)
   at System.Runtime.Remoting.Proxies.RealProxy.PrivateInvoke(MessageData& msgData, Int32 type)
   at NUnit.Core.TestRunner.Load(TestPackage package)
   at NUnit.Util.TestDomain.Load(TestPackage package)
   at NUnit.Util.AggregatingTestRunner.Load(TestPackage package)
   at NUnit.Core.ProxyTestRunner.Load(TestPackage package)
   at NUnit.Util.RemoteTestAgent.AgentRunner.Load(TestPackage package)
   at System.Runtime.Remoting.Messaging.StackBuilderSink._PrivateProcessMessage(IntPtr md, Object[] args, Object server, Object[]& outArgs)
   at System.Runtime.Remoting.Messaging.StackBuilderSink.SyncProcessMessage(IMessage msg)

Exception rethrown at [1]:
   at System.Runtime.Remoting.Proxies.RealProxy.HandleReturnMessage(IMessage reqMsg, IMessage retMsg)
   at System.Runtime.Remoting.Proxies.RealProxy.PrivateInvoke(MessageData& msgData, Int32 type)
   at NUnit.Core.TestRunner.Load(TestPackage package)
   at NUnit.Core.ProxyTestRunner.Load(TestPackage package)
   at NUnit.Util.ProcessRunner.Load(TestPackage package)
   at NUnit.ConsoleRunner.ConsoleUi.Execute(ConsoleOptions options)
   at NUnit.ConsoleRunner.Runner.Main(String[] args)

WRN: Assembly binding logging is turned OFF.
To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
Note: There is some performance penalty associated with assembly bind failure logging.
To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].

Command exited with code -100
Build with issue

For the 1.x build, you should be targeting .NET 4.5.1

@thekaveman You are right but problem is even deeper. AppVeyor has installed .NET 4.5.1 but when I exact clr .NET 4.5 for nunit it shows me:
nunit-console.exe "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Core.Tests.dll" "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Framework.Tests.dll" "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Tests.Modules.dll" "C:\projects\orchard\src\Orchard.Web\Modules\Orchard.Projections\Tests\bin\Orchard.Projections.Tests.dll" "C:\projects\orchard\src\Orchard.Web\Modules\Orchard.Tokens\Tests\bin\Orchard.Tokens.Tests.dll" "C:\projects\orchard\src\Orchard.Web.Tests\bin\Release\Orchard.Web.Tests.dll" "C:\projects\orchard\src\Tools\Orchard.Tests\bin\Release\Orchard.Tests.dll" /framework:net-4.5
NUnit-Console version <IP_ADDRESS>83
Copyright (C) 2002-2012 Charlie Poole.
Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.
Copyright (C) 2000-2002 Philip Craig.
All Rights Reserved.

Runtime Environment -
   OS Version: Microsoft Windows NT 6.2.9200.0
  CLR Version: 2.0.50727.8009 ( Net 3.5 )

ProcessModel: Default    DomainUsage: Multiple
Execution Runtime: net-4.5
Here I found topic about it.
It means that in order to resolve this issue nUnit console should be upgraded to v3.0.
So let's do it task #5136 :+1:
I split this issue because Scripting.DLR tests belong to another case

@thekaveman I used NUnit from Orchard libs and I have the same issue.
The CLR Version: 4.0.30319.34209 ( Net 4.0.30319.34209 ) - means .NET 4.5.2
C:\projects\orchard\lib\nunit\nunit-console.exe "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Core.Tests.dll" "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Framework.Tests.dll" "C:\projects\orchard\src\Orchard.Core.Tests\bin\Release\Orchard.Tests.Modules.dll" "C:\projects\orchard\src\Orchard.Web\Modules\Orchard.Projections\Tests\bin\Orchard.Projections.Tests.dll" "C:\projects\orchard\src\Orchard.Web\Modules\Orchard.Tokens\Tests\bin\Orchard.Tokens.Tests.dll" "C:\projects\orchard\src\Orchard.Web.Tests\bin\Release\Orchard.Web.Tests.dll" "C:\projects\orchard\src\Tools\Orchard.Tests\bin\Release\Orchard.Tests.dll" /framework:net-4.0
NUnit version <IP_ADDRESS>22
Copyright (C) 2002-2009 Charlie Poole.
Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.
Copyright (C) 2000-2002 Philip Craig.
All Rights Reserved.

Runtime Environment -
   OS Version: Microsoft Windows NT 6.2.9200.0
  CLR Version: 4.0.30319.34209 ( Net 4.0.30319.34209 )

ProcessModel: Default    DomainUsage: Multiple
Execution Runtime: net-4.0
Unhandled Exception:
System.IO.FileLoadException: Could not load file or assembly 'Autofac, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=17863af14b0044da' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
File name: 'Autofac, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=17863af14b0044da'

Having the same problem. @Ermesx did you solve this eventually?


Unify http request

unify http request to standard HttpServletRequest, not private interface

Coverage decreased (-0.6%) to 84.992% when pulling 10e7028cb38d7e8273246585e2a411a897db3ea6 on wujimin:unify-http-request into e6c04f0a84ec4adf83554e4ae1c3ff385c718896 on ServiceComb:master.

Coverage decreased (-0.4%) to 85.23% when pulling<PHONE_NUMBER>9cf3142962e3e581d85cd4dafd13a9 on wujimin:unify-http-request into e6c04f0a84ec4adf83554e4ae1c3ff385c718896 on ServiceComb:master.

@wujimin  Could you take a look the code coverage report? It looks like there are some lines are not covered.

yes, i added so many test case in this PR, but coverage decreased......
i'm handing this problem from this afternoon.

Coverage decreased (-0.4%) to 85.223% when pulling 142c71843bbba371bf4b8769adb3ca9116d414ab on wujimin:unify-http-request into 528b0186703093769842734433ffb3ca715f8e91 on ServiceComb:master.

Coverage decreased (-0.4%) to 85.245% when pulling df647ac73b672b319ab5b64620a097a5921c5cb2 on wujimin:unify-http-request into 528b0186703093769842734433ffb3ca715f8e91 on ServiceComb:master.

Coverage decreased (-0.3%) to 85.327% when pulling 984027825ec96ba352fbb8784493a379c6d52139 on wujimin:unify-http-request into 528b0186703093769842734433ffb3ca715f8e91 on ServiceComb:master.

simpler some implements, and delete many useless class in  new  implements, maybe  this  cause the  coverage decreased
from  coverage detail, we can see all coverages for  this  PR file increased


Internal server error.

Its almost a month so lets try this.
(First ticket created on 26-10) and ruled out the EMMC card (but in a Kingston canvas HS EMMC) with a clone of the original image. but still got troubles.
How are we going to proceed at this point, i just want a working unit.

Email<EMAIL_ADDRESS>


Not working for me

Thanks for the nice tool and the blog post. It's unfortunately not working on my computer. Not sure what you would require as debugging info?
I followed the install guides. No errors there. After restart / upgrade the line auth       sufficient     pam_tid.so in  /etc/pam.d/sudo was missing and it didn't asked for touchID.

macOS 12.1 Beta (21C5039b)
m1 Apple Silicon

Random other ideas

Link blog post from readme
Uninstall guides :)

hi frank - thank you for reporting the issue, i have updated README accordingly. could you provide:

output of ls -l /Library/LaunchDaemons/me.birkhoff.persistent_pam_tid.plist after system upgrade
output of ls -l /usr/local/bin/pam-tid-installer after system upgrade

and check these two log files /tmp/pam-tid-installer.stdout.log /tmp/pam-tid-installer.stderr.log as well

Will do with the next update. I am under the impression that this also happens after restart / reboot (not system update).

I have some trouble re-producing the issue currently. It seems to survive restart, which was not the case before, as far as I recall. I plan to test with next macOS software update.

After system upgrade macOS 12.1 (21C51):
ls -l /Library/LaunchDaemons/me.birkhoff.persistent_pam_tid.plist
-rw-r--r--  1 root  wheel  672 Dec  7 20:09 /Library/LaunchDaemons/me.birkhoff.persistent_pam_tid.plist

ls -l /usr/local/bin/pam-tid-installer
-rwxr-xr-x  1 root  admin  282 Dec  7 20:09 /usr/local/bin/pam-tid-installer

/tmp/pam-tid-installer.stdout.log seems to empty.
/tmp/pam-tid-installer.stderr.log has this:
sed: /etc/pam.d/sudo: Operation not permitted
sed: /etc/pam.d/sudo: Operation not permitted

My best guess is that macOS Monterey has introduced new changes that breaks the standard way of modifying /etc/pam.d/sudo, or there's a change to LaunchDaemon. I'll need to upgrade my Macbook to actually test out what;s going on, and I'll give you an update on this issue thereafter.

So apparently the kernel is preventing sed from modifying /etc/pam.d/sudo. I don't get how it get sandboxed (I'm not a specialist at macOS though):
System Policy: sed(10825) deny(1) file-write-create /private/etc/pam.d/.!10825!sudo

Thanks for the updates. That correlates with my observations. sudo mode removed after simple restart again.
On the other side, a fingerprint is pretty easy to fork and probably not as secure as a secure password. But having a secure sudo password is also a hustle when to be typed often. Hm.

Yes, after some investigation it seems Apple has restricted access to /etc/pam.d/sudo in TCC on macOS Monterey, I have currently no idea how this could be fixed :(

Wanted to use this tool and saw this issue, is this only for sed?
I have an ansible playbook to setup my dev machine and in there I add the pam_tid.so to /etc/pam.d/sudo if missing, and it's working (until now I tested it only in a VM...)
I have M1 Max if interested...


[TravisCI] Update checksum for cuDNN download

The checksum changed when v5.1.10 was uploaded.
https://gitlab.com/nvidia/cuda/commit/ff2d7c34fe833a3530f57f17b1f897bae482030d

This wasn't detected because there's not a set -e at the top of .travis/install.sh. So the Travis builds haven't been building with cuDNN for ~2 weeks.

cudnn-8.0-linux-x64-v5.1.tgz: FAILED
sha256sum: WARNING: 1 computed checksum did NOT match
https://travis-ci.org/caffe2/caffe2/jobs/200415842

Thanks Luke - this seems to be also contained in your other PR, so I'll close this :)


Delete unnecessary brackets

summary
Unnecessary parentheses were removed.
corresponding section
Before change
*self = ExampleStateMachine::End(EndState));

After change
*self = ExampleStateMachine::End(EndState);

LGTM 👍

Thanks!


Exception: Timed out waiting for pid file

OS: Windows 7 Professional 64bit
Java: 1.7.0_60 JDK
Python: 2.7.8 June 30. 2014.
PIP: used script from https://pip.pypa.io/en/latest/installing.html
Ant: 1.9.4
Issue description:
Installed ccm with pip install ccm. Received an error:
$ ccm create test -v 2.1.0 -n 3 -s
Current cluster is now: test
Traceback (most recent call last):
File "c:/Python27/Scripts/ccm", line 72, in
cmd.run()
File "c:\Python27\lib\site-packages\ccmlib\cmds\cluster_cmds.py", line 137, in
run
if cluster.start(verbose=self.options.debug, wait_for_binary_proto=self.opti
ons.binary_protocol, jvm_args=self.options.jvm_args, profile_options=profile_opt
ions) is None:
File "c:\Python27\lib\site-packages\ccmlib\cluster.py", line 260, in start
p = node.start(update_pid=False, jvm_args=jvm_args, profile_options=profile_
options)
File "c:\Python27\lib\site-packages\ccmlib\node.py", line 459, in start
self.__clean_win_pid()
File "c:\Python27\lib\site-packages\ccmlib\node.py", line 1135, in __clean_win
_pid
raise Exception('Timed out waiting for pid file.')
Exception: Timed out waiting for pid file.
Removed ccm with pip uninstall ccm and pulled from git. Tried running on development branch:
$ python ccm create test -v 2.1.0 -n 3 -s
Current cluster is now: test
Traceback (most recent call last):
File "ccm", line 72, in
cmd.run()
File "c:\development\ccm\ccmlib\cmds\cluster_cmds.py", line 164, in run
if cluster.start(verbose=self.options.debug, wait_for_binary_proto=self.opti
ons.binary_protocol, jvm_args=self.options.jvm_args, profile_options=profile_opt
ions) is None:
File "c:\development\ccm\ccmlib\cluster.py", line 249, in start
p = node.start(update_pid=False, jvm_args=jvm_args, profile_options=profile_
options)
File "c:\development\ccm\ccmlib\node.py", line 491, in start
self.__clean_win_pid()
File "c:\development\ccm\ccmlib\node.py", line 1172, in __clean_win_pid
raise Exception('Timed out waiting for pid file.')
Exception: Timed out waiting for pid file.

Same issue, and set the execution policy to unrestricted not work for me.

I will start work on this bug on Monday. After fixing, it should work whether or not execution policy is set to unrestricted. @LuminusDev, what Cassandra version did you run into this problem with?

I tried with 2.0.11, 2.1.0 and 2.1.2.
I installed DataStax Community earlier, and Cassandra worked fine with it (I don't know if it can be useful to know that to understand the bug).
OS : Windows 7 Professional 64bit

Okay the fact that it didn't work with 2.0.11 is very surprising. Setting execution policy would in fact not help with 2.0.11. Unfortunately, the way the code logic works on Windows right now is that "Timed out waiting for pid file" could mean any one of three things:

Cassandra started fine via the restricted method, and did not attempt to write the pid file
Cassandra started fine via the unrestricted method, but could not write the pid file
Cassandra failed to start.

After the fix for this issue, it should be much more obvious what is going wrong. 9/10 times, it is a permission issue or an inability to bind the C* ports.

Ok, I will dig in this way to find something I did wrong maybe.

Are you trying to start more than one node via ccm? The fact that DSC worked indicates that at least the normal ports for the first node should work.

Can this be solved by requiring elevated privileges when executing this code on windows ? I know its a horrible paint to support windows and I've been there.

It should not require elevated privileges, though they do help with some of the more common problems. Currently it does require unrestricted powershell execution. I have seen non-admin users have problems shutting C* down.

I have the same error with 1 node or 3 nodes. I try to execute Powershell in administrator too. I think I will work on Linux to use ccm, it must be the best solution for me in my situation.

That would probably be best for the time being.

In cluster.conf I have this lines for install_dir :
install_dir: !!binary |
  QzpcVXNlcnNcS+lr6VwuY2NtXHJlcG9zaXRvcnlcMi4xLjI=

Is it normal ?

Definitely not. That is supposed to point to a directory.

Ok, so I discover my problem. I have an "é" in my path, and it's impossible currently (I try with a space too, it's impossible).
I changed the location of .ccm and it works now. So you're right, it is 3. Cassandra failed to start.

Okay. With the change I have made today, I can now guarantee that if ccm is being run from the develop branch, if the time out occurs AND no pid is present if you check manually, then C* just failed to start.

This should make it into a release soon. It will assist in debugging issues as we will no longer be conflating powershell policy problems.

Cassandra version 3.0.0 causing same issue for me :
File "ccm.py", line 112, in
cmd.run()
File "F:\WorkStation\Git\ccm\ccmlib\cmds\cluster_cmds.py", line 511, in run
if self.cluster.start(no_wait=self.options.no_wait,
File "F:\WorkStation\Git\ccm\ccmlib\cluster.py", line 524, in start
p = node.start(update_pid=False, jvm_args=jvm_args, jvm_version=jvm_version,
File "F:\WorkStation\Git\ccm\ccmlib\node.py", line 878, in start
self.__clean_win_pid()
File "F:\WorkStation\Git\ccm\ccmlib\node.py", line 2006, in __clean_win_pid
raise Exception('Timed out waiting for pid file.')
Exception: Timed out waiting for pid file.


Exposed loadHTML method

In my use case I've already downloaded the HTML files and needed to parse them locally.
So I exposed a function loadHTML that takes the html as string and then internally converts it into cheerio object.
Here is usage example:
const html = fs.readFileSync('./page.html', 'utf-8');
scrape.loadHTML(html).then(function(metadata){
	console.log(JSON.stringify(metadata, null, 4));
});

Fixed the unit tests.
Added fallback if not preset <html lang=""> to try and get <html xml:lang="">

Thank you for your contribution @kolarski ! Since your addition enables loading the HTML from files and strings, it would be good to actually export two functions: loadFromFile() and loadFromString(). That way, in your case (and similar ones) you could load the file and convert it in one go. Could you add/rename the functions?

May be we should turn off testing of 0.10 and 0.12 ?

Fixed in #64. Please rebase this PR on top of master.

Thanks for the suggestion. I've added the two methods and two unit tests along with them.

Updated the PR with promisifyed fs and options object to pass to readFile.
Tested for these example usages:
meta.loadFromFile('page.html').then(..)
meta.loadFromFile('page.html', {encoding: 'utf-8'}).then(..)
meta.loadFromFile('page.html', 'utf-8').then(..)
meta.loadFromFile('page.html', {encoding: 'utf-8'}, (err, results) => {..})
meta.loadFromFile('page.html', (err, results) => {..})


Ensure proper memory clean-up for secret in libsecp256k1 dependency.

Two other upstream crypto libraries are using zeroize to clear memory used for storing secrets (see https://docs.rs/ed25519-dalek/1.0.1/src/ed25519_dalek/secret.rs.html#43 and https://docs.rs/schnorrkel/0.9.1/src/schnorrkel/keys.rs.html#680), but libsecp256k1 apparently does not. This will normally not be a severe issue in substrate as this algorithm is not used much, but it might cause all secrets to memory leak in the Signer, which depends on sp-core and uses all 3 crypto algorithms to maximize future compatibility.
As discussed with @kirushik and @burdges in PM, so tagging them here as well.

@kirushik we might want to wrap this crypto object in zeroize here depending on whether it would be wrapped upstream; not as good though, I understand.
We could forbid use of that lib in Signer temporarily as temporary workaround; not sure if it is worth it though - we'll lose this crypto type then obviously.

@Slesarew since doing a better job at maintaining our libsecp fork is something we'd want to do anyway, looping Thomas in here — the linked https://github.com/paritytech/libsecp256k1/issues/93 issue might be a good starting point for the new libsecp co-maintainer.
Please just use the library "as is" in Signer — and make a note to check that this issue is fixed before moving Signer anywhere near a public beta.

and make a note to check that this issue is fixed before moving Signer anywhere near a public beta.

Oh ok, so I'm moving this deadline from private to public beta, that's a relief

since doing a better job at maintaining our libsecp fork is something we'd want to do anyway

FWIW: https://github.com/paritytech/substrate/issues/8089 we will switch away from our own fork in the future.


Move the database for dynamic FL to be located on the Node

Description
Currently, GridWorkers have their own database, but we should move this up to the Node to be centrally located.
Why?
Simply put - scaling concerns. It's difficult and often unnecessary to scale a bunch of DB's separately. Plus, we're moving to an API-like access-restricted model of permissions for data in PyGrid. Because of that, we should have one database connected to the Node. The workers will be given work and data from the Node.
Additional Context
https://github.com/OpenMined/Roadmap/blob/master/web_and_mobile_team/projects/common/pygrid_reorganization.md

Sidenote: For anyone interested, I think we will make an abstraction in Syft so that some features in Syft could be aware of such a database attached to a "Domain" object. Not much writing on this yet but when there is it'll be linked off of this Epic: https://github.com/OpenMined/PySyft/issues/3695


Create Design Guidelines [Hacktoberfest]

fix #2290 User interface Design guideline, It will give you the complete guidelines of your project idea, you just give some project ideas and it will show you the complete guidelines.
📝 Description
It will give you the complete guidelines for your project idea.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

Hello, we found that he is duplicated with 2300, so we have closed the PR


#202-Unused alias on tests files

Closes #202
Description

Delete unused alias from test files
Fix doc modules

Hey, @MrJuanGaviriaK 🤝
Could you please rebase over the v0.8 branch in order to release a new patch version (v0.8.2)?


dynamically adding a new data series to the line chart, giving problems

dynamically adding a new data series to the real line chart by calling setData(newData) seems giving problems

Not sure why this would be happening. Anyone free to look into it?

@1730wang Can you be more specific? What error is it giving, how is the problem manifesting itself?


Add Typst lexer

Typst lexer, better than nothing. I don't know what I'm doing too much, just copy paste from other lexers. But at least now I can differentiate between: content, typst functions and comments. Maybe someone that know more can start from here or get motivated to do a better lexer.

Thanks for sharing! Perhaps a link to a resource would be helpful for anyone trying to complete this.

I did this for vis, but they send me here.
Typst is a typesetting system, like LaTeX, but 200x smaller, has meaningful errors, is fast, markup similar to markdown, very active community, is supported by PanDoc and Github; in Rust. Source. Documentation.
In AUR there is:

typstfmt-bin: formatter
typst-lsp-bin: language server
tree-sitter-typst-git
prettypst: formatter. (Not in AUR).

Editor support:

VS Code/Codium
Sublime
Helix
Obsidian
VIM/NeoVIM

Awesome Typst.
I don't know if this helps.

It does help. Thanks for the info!


$(AndroidPackVersionSuffix)=preview.1; main is 35.99.x

We branched for .NET 9 GA from b90c5590 into release/9.0.1xx; the main branch is now .NET 10 Preview 1.
Using 35.99.x for now, until we know that API 36 is a stable API.

There is a FastDeployEnvironmentFiles(False) test failure I'm going to investigate in a different PR.
The changes here wouldn't cause it.


Fixes 404 error when authenticating with v2 reg

After applying this fix I can successfully authenticate with github registry
[vagrant@localhost nerdctl]$ nerdctl login -u <USERNAME> ghcr.io
Enter Password:
Login Succeeded
[vagrant@localhost nerdctl]$

[vagrant@localhost ~]$ nerdctl pull ghcr.io/<IMAGE>
ghcr.io/<IMAGE>:                            resolved       |++++++++++++++++++++++++++++++++++++++|
manifest-sha256:9d7715dd1e7a20a03c1fda6b625ab73aa8e1a9cacddad885b79876bd2a8ebce8: done           |++++++++++++++++++++++++++++++++++++++|
config-sha256:6f7f6bfd79125762caacefdb0e83ece7e101c85279d31fbc36f61d643f878a42:   done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:abd092694f3b3635a1642c6fed15a81f02bbcd44045fc5dfb138eda3531899bf:    done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:b53a42fddd847a0d1d550565601c9299fecae84e94ffcb8c9a9e5e98231e0ee0:    done           |++++++++++++++++++++++++++++++++++++++|
elapsed: 3.8 s                                                                    total:  7.8 Mi (2.1 MiB/s)
[vagrant@localhost ~]$

This maybe a slightly naive approach so welcome feedback on a better place for this - Not sure who to ping from containerd
Closes #715

Thanks, I'm closing this, as we now have https://github.com/containerd/containerd/pull/6470
https://github.com/containerd/nerdctl/pull/724#discussion_r790505829

Reopened per discussion in https://github.com/containerd/containerd/pull/6470 .
Sorry for going back ans force.


t3c Integration Tests: could not get status for service 'trafficserver'

I'm submitting a ...

bug report

Traffic Control components affected ...

TC Cache Config (integration tests)

Current behavior:

Currently, ort_test container in the the t3c integration tests use a systemctl shim that returns exit code 0 if the service provided is trafficserver (without printing anything or starting/stopping trafficserver).
If the t3c integration tests are made to use the version of this shim that CDN in a Box uses, which passes the command given to the trafficserver initscript and returns the exit code as follows:

systemctl command
trafficserver initscript command

systemctl start trafficserver
trafficserver start

systemctl stop trafficserver
trafficserver stop

systemctl status trafficserver
trafficserver status

systemctl enable trafficserver
echo "Created symlink /etc/systemd/system/sockets.target.wants/$(basename "$service_file") → ${service_file}."

then the integration tests will fail when trying to check the status of trafficserver:
[...]
DEBUG: torequest.go:513: 2021-06-01T22:41:39.43950936Z: Setting change applied for 'set_dscp_37.config'
    t3c-apply-unset-update_test.go:109: TestT3cTOUpdates t3cUpdateUnsetFlag stdout '''set TSHome from the RPM config file  list '/opt/trafficserver'
        TSHome: /opt/trafficserver, TSConfigDir: /opt/trafficserver/etc/trafficserver
        [...]
        INFO: cmd.go:292: 2021-06-01T22:41:39.439528155Z: t3c-check-reload calling with mode 'syncds' pluginPackagesInstalled '[]' changedConfigFiles '[/opt/trafficserver/etc/trafficserver/hdr_rw_first_ds-top.config/hdr_rw_first_ds-top.config /opt/trafficserver/etc/trafficserver/dscp/set_dscp_37.config/set_dscp_37.config]'
        INFO: torequest.go:1084: 2021-06-01T22:41:39.446012351Z: t3c-check-reload returned 'reload'
        INFO: torequest.go:632: 2021-06-01T22:41:39.446032445Z: IsPackageInstalled 'trafficserver' found 'trafficserver-8.1.1-44.c4e4170.el7.x86_64' in cache, returning true
        ERROR: t3c-apply.go:185: 2021-06-01T22:41:39.466826917Z: failed to start services: getting trafficserver service status: could not get status for service 'trafficserver'

        '''
INFO: t3c-request.go:96: 2021-06-01T22:41:39.473264823Z: configuration initialized
INFO: client.go:52: 2021-06-01T22:41:39.473296799Z: URL: 'https://to_server:443' User: 'admin' Pass len: '6'
INFO: client.go:55: 2021-06-01T22:41:39.473309366Z: TO URL string: 'https://to_server:443'
INFO: client.go:56: 2021-06-01T22:41:39.473321673Z: TO URL: 'https://to_server:443'
INFO: client.go:67: 2021-06-01T22:41:39.595702247Z: toreqnew.New Logged into in to Traffic Ops '<IP_ADDRESS>:443'
INFO: getdata.go:79: 2021-06-01T22:41:39.596255835Z: Getting data 'update-status'
INFO: torequtil.go:60: 2021-06-01T22:41:39.601517275Z: GetRetry server_update_status_atlanta-edge-03 retries 0 took 5ms
    t3c-apply-unset-update_test.go:127: expected UpdatePending false after syncds run
    types.go:58: ---- DeleteTestTypes ----
User: traffic_ops, Password: len 6, Hostname: db, DB: traffic_ops
------------- End of TestT3cUnsetsUpdateFlag tests ---------------
--- FAIL: TestT3cUnsetsUpdateFlag (89.84s)
FAIL
exit status 1
FAIL	github.com/apache/trafficcontrol/cache-config/testing/ort-tests	90.387s
ERROR: ORT tests failure

Expected behavior:

The t3c integration tests should successfully start Apache Traffic Server.
Minimal reproduction of the problem with instructions:

cp infrastructure/cdn-in-a-box/cache/systemctl.sh cache-config/testing/docker/ort_test/systemctl.sh

Place the RPMs and build the Docker images for the t3c integration tests

Run the t3c integration tests

@jrushford is working on getting the t3c integration tests to start ATS for testing


Status : Offline

Hi,
I installed moped so as to use the mopidy-youtube extension. However the page stated "Status Offline" in the upper right corner.
What did I do wrong ?
Thanks

Status Offline usually indicates that a JavaScript error occurred. Could you open the console in your browser and check for errors?

I misconfigured the settings, everything works now
You can close


Keep trailing spaces on comments

Hello,
I've noticed that trailing spaces on comments are stripped in some cases. Some examples that keep them:
// A line comment
var a = 2

/*
A multiline with no star
*/
var a = 2

But this one does not:
/*
* A multiline with star
*/
var a = 2

(Tested on web)
For my use case, this is bad because js-docs comments are processed as Markdown, and two trailing spaces is the markdown way to print a new line in the final rendering.
This seems like a bug, since some formats do keep the trailing space, but I would like some confirmation on that.
Happy Holidays,
Gui

Interesting.  Shouldn't be hard to fix.

https://github.com/beautify-web/js-beautify/blob/master/js/lib/beautify.js#L1157
If we change that trim to ltrim, I think that would do it.


Host now can't write or delete files on external volume used

I've compiled the SMB version for a Raspberry Pi and it seems working well (after also editing the avahi service to hide the bogus home folder visible).
One problem I'm experiencing is:
I made sure to mount the external volume (ext4) in the same directory through fstab, after making the mounting folder with correct ownership (pi), the Mac can copy and delete files without problems into the share, inside the container the /timemachine folder got the right permissions for the username I chose and the root group.
The problem is that the host computer cannot write or delete files inside the original volume, I noticed that the folder owner now results as: systemd-timesync and if I change it to pi through chown the Mac cannot write into the folder anymore.
How is this possible? Why the container changes the ownership to this strange user?
Thanks

after testing, I noticed that when I add a user with docker exec timemachine add-account
it then change the owenership of the volume filesystem in system-timesync

Solved... my bad... it was as easy as specifying uid and gid matching host user while creating user in docker:
docker exec timemachine add-account -i 1000 -g 1000


Add install directory prefix

Add a feature to customize the target directory that the project is installed in. Currently it's hardcoded to /opt.

This should now be handled in 0.2.0.


Add scrollspy as example for how this lib can be used

I'm using this library to implement scrollspying, so I thought it might be good to call it out in the README. Perhaps surprisingly, this library is by far the best lib I've found to implement Scrollspies (even better than React scrollspy-specific libraries, imo), which is why I think it's a good idea to mention it in the README.
NBD if you don't think it's a good example, though :)

This is great, we use it ourselves for scrollspying.


Make diagram of logos for technologies we're using

We probably want a small image in the project plan that displays the logo of C++, Qt, CMake, any testing tools...

This is more relevant for final report at this point.


Linked website is not working

The Website linked in the repo description (http://testdrive.microsoftedge.com/)
is not working properly here, it does a bunch or redirects:
First to https://dev.modern.ie/testdrive/demos/ and from there to https://dev.windows.com/en-us/microsoft-edge/testdrive/demos/ and from there to:
Location: /redirect?url=..%2F

So I end up at https://dev.windows.com/redirect?url=..%2F which is an error page.

The following: http://dev.modern.ie/testdrive/
does work fine, so maybe that one should be used instead.

Fixed. Thanks!


Create new_form_with_errors, edit_form_with_errors partial

Related comment

https://github.com/murugan-r/photogram_test_staging_cci_2/pull/3#pullrequestreview-151601635

Done.


feat: support serving index.html in middleware mode

Description
I want to integrate vite (as a koa middleware) to my koa app for just development.
When middlewareMode enabled, vite will not handle the html request. I should serve them like this: Setting Up the Dev Server
I am not working on SSR, and I do not like to serve the html content by myself. Instead, the internal html middleware is enough for me.
It should be able to use indexHtmlMiddleware and spa fallback even when middlewareMode enabled.
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[x] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Looks like server.middlewareMode is not documented yet in https://vitejs.dev/config/#server-options. You may also want to include that in this PR or we can do it later. We may also need to think about how to add a test for 'html' mode to avoid future regressions.

document added


Class Prefix support?

seems not support class prefix,
with this line option (.google.protobuf.objectivec_file_options).class_prefix = "LL";
protoc produces error  Option "(.google.protobuf.objectivec_file_options)" unknown.

just use original protobuf from Google, is as always fresh https://github.com/google/protobuf


Does X forwarding support every kind of GUI apps on Linux?

Hello,
This is probably not an issue but a question. I heard that sometimes X forwarding will not work for some apps native. I wonder if we can just configure X forwarding properly so that every app with GUI on Linux, such as Slack, Sublime, and Evince, could be run without any problems. Thank you.
Best,
Lei

Hi Lei,
As far as I know, X forwarding should be completely functional for all applications in an X environment -- If I have a gap in knowledge here, please let me know and I will research a bit further.
Where it won't work is in a situation where you aren't running X Server on your host -- like if you are running Wayland or whatever Microsoft Windows runs.  This is because the X forward capability ends with the container -- the host has no way of interpreting and handling the signal it is receiving.  Although there are ways to work around this in both of the aforementioned compositors.
If you come across an application that won't work with X Forwarding, please let me know.  I'd be very interested to figure out why.
For what it's worth, I worked for several months with all of my GUI happening on one machine via X Forwarding and all of the compute happening on the originating machine.  I never came across a single application that was unusable.  Although it did get confusing when trying to figure out which machine a file saved to :laughing:
Thanks,
Justin


use Netz-AG design template

It seems Netz-AG wants us to use the common house webdesign. I don’t have access to the design and therefore can’t say how much effort it would be to switch.

Some links at the welcome page has to be fixed.

WaschZoo to refer to welcome page if not logged in?
rules, documents and satzung

You’re right, that page should be probably completely remade. It’s based on a copy of the old system… Feel free to change it as you like. Rules etc. could be added as static files. The best way would be probably to use some CMS, but that might be overkill for now.


Proper way to handle removed files

~/.local/share/Trash
~/.Trash
/home/.Trash

Also, is prefix unix time needed?

35cb3a059b6c6f206d9019624016e681903dca98


Having Problems using poltergeist and cucumber/rails when using sequel in rails

I was reading poltergeist documentation that there are problems across threads and sharing transactions across threads. I'm migrating a project from AR to sequel.
When I had AR, there was never an issue with running javascript feature tests.
I wonder it's because of this: https://github.com/cucumber/cucumber-rails/blob/master/lib/cucumber/rails/hooks/active_record.rb#L1-L8
How could I reproduce/patch this for sequel?

@TiagoCardoso1983 so do you actually have a problem now with sequel.
And when you are talking about sequel what exactly do you mean?

humm, it's a combination of dependencies, as this is only failing in the javascript tests (I'm using poltergeist as driver). Apparently, server runs in a background thread, the pools are created by thread and that causes issues depending on the strategy one uses to clean the database (using database_cleaner).
I saw there was one "hack" for AR (the shared connection thing) and I'm not sure if this is the reason why it worked with AR, and it doesn't work anymore with sequel, which might require a similar hack.

Is this still an issue? If not, do you mind if we close this? Thanks!

Not really, mostly solved it by avoiding cucumber altogether. Problem solved


CakePHP 4 compatible - switch to Middleware

No tests yet - wanted to get your thoughts

@Ali1 my concern here is that we need to initialize the Table Registry as part of the middleware processing. Any thoughts on how we could avoid this?

Can't think of any other way...

How were Tables already initialised when Dispatch filters were used? Was the order Middleware->A package that initializes Tables->Dispatcher?

Also another thought, I think authentication middleware does eventually initialize Tables here https://github.com/cakephp/authentication/blob/master/src/Identifier/Resolver/OrmResolver.php so it's not necessarily inappropriate.

middleware was actually messing up paginate. I found a different way

are you going to merge?

Hello?


Make an 'OMNI' style input instead of traditional inputs

This could use NLP to guess what kind of input the user wants to create.

Not sure that this will be a good ux pattern.
Closing it for now and will revisit if nessecary


Enable pipelined write for RocksDB

Scope & Purpose
Enable pipelined write for RocksDB

Change default value of startup option --rocksdb.enable-pipelined-write
from false to true.

This is a test to see if setting the options improves write throughput in parallel setups. As all devel versions also undergo continous performance tests, we can also check if setting the option has adverse effects for single-threaded performance tests.

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[x] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Related Information

[x] Docs PR: https://github.com/arangodb/docs/pull/808
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.
[x] The behavior in this PR was manually tested

Pipelined writes are enabled by default now anyway, so this PR is obsolete.


Empty SVG

Hi
When I add a polyline to the map, it appends a svg in the .leaflet-overlay-pane. That is ok.
But when I remove the polyline, the svg keeps on the dom but empty.
I'm having performance problems with lot of data but just after adding/removing a polyline. I think there is an event on that svg and I'm getting a 3-4 seconds of lag when I try to pan.
Should that svg keeps on the dom? Or It should be like it's initial state, without the svg?
thankyou!

Can you set up a minimal JSFiddle test case reproducing your performance problem?

Not at the moment, but I will later.
Anyway... forgetting performance.. shouldn't that svg be deleted? There's no sense having that svg translating 3d and triggering events... or there is?
thanks for the reply

It shouldn't. It adds no overhead when there are no layers.

I'm closing this, as a request for more information was made more than one month ago but no further information has been provided.


Fix null Datetime Xml reading for Adjustment StartDate and UpdatedAt

When calling subscription.Preview() I got the below exception due to an empty date field on Adjustment (called by Invoice Preview). The reason is that StartDate and UpdatedAt are trying to read directly as DateTime without checking if it is possible, like EndDate and CreatedAt does already. Just updated StartDate and UpdatedAt to the same pattern.
'subscription.Preview()' threw an exception of type 'System.FormatException'
Data: {System.Collections.ListDictionaryInternal}
HResult: -2146233033
HelpLink: null
InnerException: null
Message: "The string '' is not a valid AllXsd value."
Source: "System.Xml"
StackTrace: "   at System.Xml.Schema.XsdDateTime..ctor(String text, XsdDateTimeFlags kinds)\r\n   at System.Xml.XmlConvert.ToDateTime(String s, XmlDateTimeSerializationMode dateTimeOption)\r\n   at System.Xml.XmlReader.ReadElementContentAsDateTime()\r\n   at Recurly.Adjustment.ReadXml(XmlTextReader reader) in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\Adjustment.cs:line 218\r\n   at Recurly.AdjustmentList.ReadXml(XmlTextReader reader) in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\List\AdjustmentList.cs:line 39\r\n   at Recurly.Invoice.ReadXml(XmlTextReader reader) in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\Invoice.cs:line 350\r\n   at Recurly.Subscription.ReadXml(XmlTextReader reader) in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\Subscription.cs:line 621\r\n   at Recurly.Client.ReadWebResponse(HttpWebResponse response, ReadXmlDelegate readXmlDelegate, ReadXmlListDelegate r
eadXmlListDelegate, ReadResponseDelegate responseDelegate) in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\Client.cs:line 382\r\n   at Recurly.Client.PerformRequest(HttpRequestMethod method, String urlPath, WriteXmlDelegate writeXmlDelegate, ReadXmlDelegate readXmlDelegate, ReadXmlListDelegate readXmlListDelegate, ReadResponseDelegate reseponseDelegate) in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\Client.cs:line 170\r\n   at Recurly.Subscription.Preview(ChangeTimeframe timeframe) in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\Subscription.cs:line 407\r\n   at Recurly.Subscription.Preview() in C:\Users\ben\Documents\GitHubVisualStudio\recurly-client-net\Library\Subscription.cs:line 418"
TargetSite: {Void .ctor(System.String, System.Xml.Schema.XsdDateTimeFlags)}

@thiagodpaz Which version of the library are you using? I find it unusual that the error seems to contain file paths on my local machine.

:+1:

@bhelx I'm using 1.4.5 from nuget, but I am not sure if I was referencing the nuget package or git repo when I saved the exception text.

@thiagodpaz Thanks for contributing!


Support for expression keymaps?

Hey @folke, thanks for the great plugin!
Is there support for <expr> mappings in which-key.nvim? I can't figure out how bind the example mappings from nvim-compe using this plugin. E.g. the readme has the following proposed bindigs:
inoremap <silent><expr> <C-Space> compe#complete()
inoremap <silent><expr> <CR>      compe#confirm('<CR>')

How would I bind those with which-key.nvim using lua?
I tried like this, but that does seem to work, I suspect I'm missing the <expr> part.
wk.register({
		['<C-Space>'] = { '<C-o>:call compe#complete()<CR>', 'Trigger completion'},
		['<CR>'] = { "<C-o>:call compe#confirm('<CR>')<CR>", 'Confirm completion' },
}, {mode = 'i'})

I also tried:
wk.register({
		['<C-Space>'] = { '<C-o>:call compe#complete()<CR>', 'Trigger completion', expr=true},
		['<CR>'] = { "<C-o>:call compe#confirm('<CR>')<CR>", 'Confirm completion', expr=true },
}, {mode = 'i'})

But that results in an error. Would be great if you could help me out with an example, couldn't find any in the docs.

Apparently we didn't support expr yet. I just added it.
Something like this should work:
wk.register({
		['<C-Space>'] = { 'compe#complete()', 'Trigger completion', expr=true},
		['<CR>'] = { "compe#confirm('<CR>')", 'Confirm completion', expr=true },
}, {mode = 'i'})

I'm closing this. Do let me know if the above didnt resolve this.

Just wanted to confirm, the above commit solved the issue. Thanks!


Incorporating previous binning attempts

Hello,
I was wondering if it was possible to include previous binning results into BASALT at the refinement stage? I have multiple previous outputs from other binners not included in BASALT and would like to combine these with the results of BASALT's autobinner stage.
Thanks
Huw

Yes. BASALT has a way to feed previous binning bins into it. And then you could the refinement module.  To do this, you need to manually use the script "Data_fedding_02282021.py" to feed the data to BASALT. And then, you may use the S5_outlier_remover_DL script to carry out the refinement process. Could you please send me an email to<EMAIL_ADDRESS>I will prepare a detailed instruction for doing this. I think I could make an automatic pipeline to accomplish this goal.

Sorry for the late response :(

Hi - I am also interested  in doing this. I have two sets of MAGs produced by BASALT that I want to compare to a previous binning dataset. (I could not run all assemblies together due to memory issues ...)
Camilla


Templates support for .NET MAUI 9 RC2

Templates support for .NET MAUI 9 RC2

This feature is now part of the below releases.
CLI NuGet package - VijayAnand.MauiTemplates - v5.14.0
.NET MAUI Templates Pack - v<IP_ADDRESS>


Add option to set the format code of chart data tables

Since it's possible and looks nicer to set a number format (in my case: thousands separators) for the value axis labels, it seems prudent to allow setting the same format code for values in the data table as well.
fixes #489

rebased

Thanks @hanzi !


ModernHttpClient crashes on Android when used in SignalR

Using ModernHttpClient and SignalR together gives great benefits related to performance etc and also having the connection reconnecting when switching from wifi to 3g/4g.
To use ModernHttpClient together with SignalR my solution was to subclass the DefaultHttpClient from SignalR and use that instance as input to the Start method.
public class ModernHttpClientSignalR : DefaultHttpClient
{
        protected override HttpMessageHandler CreateHandler()
        {
            return new NativeMessageHandler();
        }
}

await _connection.Start(new ModernHttpClientSignalR());

This works nicely on iOS.
On Android the ModernHttpClient does not like it when SignalR is posting data without any ContentType and I had to fix line 83 in OkHttpNetworkHandler.cs.
From:
var contentType = String.Join (" ", request.Content.Headers.GetValues ("Content-Type"));

To:
var contentType = string.Empty;
if (request.Content.Headers.ContentType != null)
{
    contentType = String.Join(" ", request.Content.Headers.GetValues("Content-Type"));
}

It would be great if this fix could be part of the original code.

Can you send a pull request to this effect?

Hi Paul,
I am not sure what you mean? Do you want a pull request as an example when the error occurs or do you want a pull request with the fix? I am a little new to the git way of working:-)

I have added pull request #141.

Fixed in #141


Get base object for Bluesky replies

As discussed in https://github.com/snarfed/bridgy/issues/1580#issuecomment-1925314697

This looks fine to me, happy to merge it once there's a test or two! Have you tested it locally though? Does it work?

Yep, working fine locally - will add some tests : )

@snarfed this sufficient? I couldn't really think what to add in terms of edge cases etc

Perfect, thank you! Merging and deploying now...


[a11y] Calendar: Use presentation & application roles to have more control over screen readers

Problem:
Huge inconsistencies across screen reader/browser combinations in terms of what information is announced and in what order (e.g., a main functionality we want is for screen readers to read the column header, the weekday, first so users can quickly navigate across rows).
I'll comment on the changes made throughout, which give us much more control over how the calendar is presented (though they sacrifice built-in table semantics which was the hesitation for doing this).
Overall the desired experience (which has now been achieved across most screen reader/browser combinations):

User enters into table, "[month] [year] table" is announced to them (not working on Legacy Edge + NVDA and VoiceOver + iOS Safari)
as user arrow key navigates around, on each day they here (before anything else) "[weekday] [date] [selection state] [month] [year]" and maybe something about them being on a button.
Page up/down will announce month/year when month changes (does not work on Firefox + JAWS)
User does not have to swipe through the entire header row before getting into the dates (was a problem on iOS)

Taking another look at the button vs. div change so this can be ignored for now


How to properly use build and dev

I am trying to setup a project with Vue on the frontend and Flask-rest on the backend. This is my index.js file:
var path = require('path')

module.exports = {
  build: {
    env: require('./prod.env'),
    index: path.resolve(__dirname, '../dist/index.html'),
    assetsRoot: path.resolve(__dirname, '../dist'),
    assetsSubDirectory: 'static',
    assetsPublicPath: '/',
    productionSourceMap: true,
    productionGzip: false,
    productionGzipExtensions: ['js', 'css'],
    bundleAnalyzerReport: process.env.npm_config_report
  },
  dev: {
    env: require('./dev.env'),
    port: 8080,
    autoOpenBrowser: true,
    assetsSubDirectory: 'static',
    assetsPublicPath: '/',
    proxyTable: {
      '/api': {
        target: 'http://localhost:5000',
        changeOrigin: true,
        pathRewrite: {
          '^/api': ''
        }
      }
    },
    cssSourceMap: false
  }
}

When I run the dev server everything works, and I am able to use the axios package to call the Flask server which runs on localhost:5000. However, what do I do once I need to deploy the app? How do I build the app so that the built files will call the correct backend?

You might have more success with your question on forum.vuejs.org


begin progress on #49 by notifying the user of redirection to Freesound

This closes #49 should we choose to merge it when appropriate.
This first commit I made at the time of this writing just lays the foundation. If we want to style the confirmation dialog, we definitely want to make changes before merging this into the main branch of this repository.

This is what the dialog box looks like at the time of this writing.

Current appearance of redirect notification window:

Goal appearance, from Figma, originally linked in #18:

Right now, the background is "hardcoded" to always be black, meaning that it won't change when the theme changes. We should make it so that it automatically changes with the theme. In addition, we should style the buttons to match the aforementioned Figma.
Other changes that I would like to make include allowing the user to press the Esc key to automatically close the dialog box this pull request seeks to add, or allow the user to click anyplace not on the dialog box to close the dialog box, as right now, the user would not be able to do that.

This is a good start 😃 eventually we'd want to use react-modal, which handles all the modal related work for you. Pretty nice!

Documentation for react-modal, in case someone on this page needs it in the future!

Current appearance:

Goal appearance, from Figma, originally linked in #18:

Improvements over last commit:
We can now exit the redirect window by pressing Esc or clicking anywhere else on the page besides the Modal! In addition, the buttons' styling matches that of the Login button, which I think is a plus.
Changes left to make from last commit:
I still haven't figured out how to make the Modal automatically use whatever background color freesound-player is using. The Modal would need to automatically use the user's theme, which is what the rest of https://freesound-player.netlify.app/ does.
I hadn't realized, but the last commit has an issue with the Waveform component where the Waveform component overlays on top of the Modal window.

I also want the Send me there! option to have emphasis it does in the Figma.
And, the buttons' spacing needs to match that in the Figma.

As a suggestion, you can probably fix that by using the className prop and passing bg-default as one of the classNames

Current appearance:

As of right now, the functionality is fine. Pressing the Esc key or clicking anywhere not on the pop-up dialog exits the dialog immediately.
The only issue remaining, I think, before this pull request is ready for merging is the interaction with the <Waveform> component. It somehow "seeps through" onto the dialog window.

Improvements over last commit:
Thanks to a discussion with @amilajack, Waveforms are no longer visible through the react-modal dialog window that opens up when users request to navigate to another site.
Things that I tried but haven't yet figured out how to do:
"Pin" close button to upper-righthand corner of react-modal dialog window. Right now it shows up in the upper-righthand corner of the entire viewport with right: 0 top: 0.
Only have the border show up when the night theme is active as specified in the Figma. I'm not sure how to do this, even without Tailwind. @akshara-sun might know.
To add a z-index to the dialog window to stop Waveforms beneath them from being visible, I had to use the overlayClassName prop, which overrides the default styles for the Modal component. That caused the transparency effect on the background to go away, and I wasn't able to recreate that effect using Tailwind.
Another thing to mention is that adding a border to the react-modal dialog window appears to squish the contents of the window together, so that the window description moves across two separate lines, instead of one, like in the Figma. This is probably a smaller issue that we can table until we fix the issues above.

Put together a quick example. Lmk if this helps:
https://codesandbox.io/s/peaceful-frost-g9umm?file=/src/App.js:441-485

Nice! The overlay background needs the bg color. See example above.

I'm not sure how to do background: "rgba(0,0,0, 0.75)" from react-modal's default styles in Tailwind. Once we figure out how to add that effect to the entire overlay except the content, and have the border only show up when the night theme is active, I think this pull request will be ready to merge. The current modal window is slightly smaller both horizontally and vertically than the one in the Figma, but it's the closest I could get with Tailwind's building blocks.
Improvements with prior commit: Probably the most notable change is that the "close" button is on the upper-righthand corner of the modal window content, rather than the entire page. I used className="relative float-right" to accomplish this using Tailwind. More visual improvements are that the buttons have more internal padding so they more closely match the Figma and I used rounded-lg on both of the buttons in the modal window's content to have that much more rounded, aesthetic appearance as seen in the Figma. And, I removed a line break (<br />), instead changing the internal padding of the modal window content to more accurately reflect the spacing in the Figma.

Adding that dynamically with tailwind will be difficult. We're not going to get much value from tailwind in that specific use case. We get value from tailwind by composing classes together to create high level styles. This is a very specific case where we need to add only one style so its fine to just add the single css property like in the example.

This is what the current appearance of the redirect notice modal window is, as of the latest commit before commenting this message, 6d9f5eff358a9a3bf1471e3a1f7f863d19f28892.

Another minor UI detail: in the design the modal is positioned a bit closer to the center of the page. We should update the positioning a bit to reflect this.

This looks great so far @hamirmahal!
Just a quick note: it looks like the overall size of the modal is a bit bigger than the the Figma. @amilajack is that something we could us tailwind to fix? Or would this need CSS Styles?

When clicking the close button, the default CSS shows the focus outline. Same case for the modal body.

This is the default css behavior. We should override this default behavior with CSS.

To clarify, we don't want any focus outlines?

@hamirmahal I realized I was pretty vague, my bad. We want the modal to be focused but we don't want to use the default browser styling for a focused element. A short read on how we can fix this: https://css-tricks.com/almanac/properties/o/outline/
.my-modal-body {
  outline: var(--secondary-color);
}

.my-modal-close-button {
  outline: none;
}

@hamirmahal I realized I was pretty vague, my bad. We want the modal to be focused but we don't want to use the default browser styling for a focused element. A short read on how we can fix this: https://css-tricks.com/almanac/properties/o/outline/
.my-modal-body {
  outline: none;
}

.my-modal-close-button {
  outline: none;
}

While it doesn't appear to be the same "under the hood", Tailwind's .outline-none class does appear to do the same thing visually. Should we just use Tailwind's class to cut down on our CSS?

@hamirmahal I realized I was pretty vague, my bad. We want the modal to be focused but we don't want to use the default browser styling for a focused element. A short read on how we can fix this: https://css-tricks.com/almanac/properties/o/outline/
.my-modal-body {
  outline: none;
}

.my-modal-close-button {
  outline: none;
}

No worries! I read through the link; thanks for suggesting it. While it doesn't appear to be the same "under the hood", Tailwind's .outline-none class does appear to do the same thing visually.
outline: 2px solid transparent;
outline-offset: 2px;

Should we just use Tailwind's class to cut down on our CSS?

@hamirmahal good find! I actually had no idea that class existed. +1 for using that class

When clicking the close button, the default CSS shows the focus outline. Same case for the modal body.

This is the default css behavior. We should override this default behavior with CSS.

Strangely enough, the outline around the close button doesn't appear to appear in Safari, but I was able to confirm that it shows up in Chrome after downloading and installing it. I was able to remove the outline from the modal window using the Tailwind class, but I couldn't remove the outline from the close button with traditional CSS or Tailwind. I've tried styling just the button, as well as just the ion-icon.

Adding the btn tailwind class should fix that

Okay. While reading up on this I've come across several references saying to add in our own focus styling for accessibility if we remove the defaults. I was thinking we can have var(--secondary-color) as the outline for focused elements.

Yea that makes sense. I think we should add it for the modal but not for the button because there's no design for the focused state for the close button yet. I'll implement the design as a followup to this PR

Old appearance, before the latest commit at the time of this writing, with modal window higher up on the page and buttons that had an outline when in focus:

Current netlify preview appearance after the latest commit at the time of this writing:

While the buttons currently do not have an outline, the modal window still does, as requested, and we should still add our own outline styling for buttons in focus for keyboard users and for accessibility.
The modal window is also lower now, too, as @amilajack requested, and I think it looks more aesthetic and more closely matches the Figma for this project.
The modal window should now have the appropriate colors for both themes, and the recent change to have a border even while the day theme is active means that we no longer need the changes made to tailwind.config.js made in an earlier commit in this pull request, so I removed them with the latest commit at the time of this writing.

Another minor UI detail: in the design the modal is positioned a bit closer to the center of the page. We should update the positioning a bit to reflect this.

Done! Changing my-24 to my-48 noticeably moved the modal window closer to the center of the page.
There's a screenshot below of how it looks now.

When clicking the close button, the default CSS shows the focus outline. Same case for the modal body.

This is the default css behavior. We should override this default behavior with CSS.

Also done! Removed the focus outline for the modal window,

Yea that makes sense. I think we should add it for the modal but not for the button because there's no design for the focused state for the close button yet. I'll implement the design as a followup to this PR

and we can add the focus outline for the buttons once we have a design for them.

When clicking the close button, the default CSS shows the focus outline. Same case for the modal body.

This is the default css behavior. We should override this default behavior with CSS.

Also done! Removed the focus outline for the modal window,

Yea that makes sense. I think we should add it for the modal but not for the button because there's no design for the focused state for the close button yet. I'll implement the design as a followup to this PR

and we can add the focus outline for the buttons once we have a design for them.
This is how it looks now.

The buttons currently do not have an outline when they are in focus, even when using a keyboard to navigate between them.

Nice! Good stuff 🥳

Thank you! I'll try to work on the next pull request I open more quickly.


K6 can't launch jobs when using LinkerD

When launching jobs in a namespace that has LinkerD enabled, the starter job is unable to contact the downstream pods to trigger them.  If I run the jobs in a workspace without LinkerD, everything works as expected.
starter pod logs
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to <IP_ADDRESS> port 6565: Connection refused
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to <IP_ADDRESS> port 6565: Connection refused
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to <IP_ADDRESS> port 6565: Connection refused
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to <IP_ADDRESS> port 6565: Connection refused

Starter pod configuration
https://gist.github.com/MarkSRobinson/702ae2c4ebb8f2a02509c72c30c80531

Hi @MarkSRobinson! Thanks for opening this. Happy to see that you're trying to run this with LinkerD!
I'm wondering, if @KnechtionsCoding has some thoughts on this, as he added support for Istio on the operator.

The first thing I'd look at is, is the linkerd sidecar being injected on jobs (I know that was a rather recent addition to support).
Secondly, linkerd, like istio, probably doesn't support IP based routing. The latest version, being released shortly, will instead use a unique service on each pod to allow for dns based routing. So that's coming and probably required.
Third, which isn't a problem till it starts, but will be the next problem you run into the proxy won't quit unless you expressly tell it too meaning the job will never finish. See: https://tech.ingrid.com/on-linkerd-and-kubernetes-jobs/ for information on how to fix that.

https://github.com/linkerd/linkerd-await appears to be an option we could look at supporting. @dgzlopes I wonder if we should have a conversation around adding a serviceMesh to the crd. And then instead of scuttle enable, or linkerd enable, a user passes in serviceMesh: istio or serviceMesh: linkerd and we generate the command for them. We could build different containers as well for each service mesh to support. So there would be a runner, controller, starter, and then for each one add a - to it

Sounds interesting. Let's have a conversation on our next meeting :muscle:

Is there ant fix found for this ?
this link does not work BTW - https://tech.ingrid.com/on-linkerd-and-kubernetes-jobs/

Is there ant fix found for this ?

@arpitrege123  check out https://github.com/grafana/k6-operator/issues/195


Fix typo in docstring

Funny typo, clearly a search-and-replace issue. :)

Heh I always like it when these turn up


Purpose of this package?

I'm currently developing the go-debug atom package which is a debugger interface for go (using delve).
Is this going to be a general purpose package to create such debuggers (standard interface and api, like in visual code)? If so what's the purpose of the "old" package atom-community/debugger then?
And is there any way one could help building such a package?

I'm not 100% sure, but I believe this is meant to co-exist with captain-hook.
/cc @joefitzgerald

@lloiser firstly, thank you for working on that package - this is awesome and precisely why I split out the logic for go-config from go-plus into a standalone package!!! When you feel like go-debug is mature, I would like to bundle it into go-plus, if you're open (go-plus is basically a package installer now).
Second, as @thomasjo describes, yes, debug is intended to work in conjunction with captain-hook. It will allow for a provider-based system to work in a similar way to nuclide's debugging packages. In essence, the language-specific providers should not need to worry about UI, or determination of which package should be used to debug what file or folder.
You should join us on Slack:

Great to hear this :grin: It seems to work fine although I'm pretty sure that there is still room for improvement here and there or bugs which I didn't found until now. Test it yourself and if it looks fine to you feel free to add it to go-plus.
So this will replace my implementation sooner or later - great, less code to maintain then :smile:

You should create an animated gif and add it to the readme. Once we have that we should post it to reddit.

Any roadmap for this package? Seems like nothing happened in the last months 😕

I'm currently reworking my go-debug package to separate the view and the actual debugger into separate projects/packages. Are you interested in moving my code over to here?

@llosier, yes!

Can there at least be some expanded README description until the illustration is ready? I've just stumbled across this, and I still can't tell what it is supposed to be (debugger for a certain language? debugger for many languages? base plugin to be used by developers of languages to implement their actual debugger?...) so having a better description than the current super vague one-liner would be hugely appreciated.

Can there at least be some expanded README description until the illustration is ready? I've just stumbled across this, and I still can't tell what it is supposed to be (debugger for a certain language? debugger for many languages? base plugin to be used by developers of languages to implement their actual debugger?...) so having a better description than the current super vague one-liner would be hugely appreciated.


i2c_param_config() fails with invalid i2c_clk_src unless we log during callback (IDFGH-4436)

Environment

Development Kit:      none
Module or chip used:  ESP32
IDF version : v4.3-dev-2136-gb0150615d
Build System:         idf.py
Compiler version : xtensa-esp32-elf-gcc (crosstool-NG esp-2020r3) 8.4.0
Operating System:     Windows
(Windows only) environment type: ESP Command Prompt
Using an IDE: CLion
Power Supply:         Battery

Problem Description
Attempting to initialise SSD1306 OLED using the u8g2 library and i2c_param_config() is reporting that the i2c_clk source is invalid, ie i2c_get_clk_src(i2c_conf) returns I2C_SCLK_MAX. But if I add a single line of log to my callback then i2c_get_clk_src(i2c_conf) returns 2C_SCLK_APB.
NB app was working fine under esp-idf-v3.3, recompiling under 4.3 causes the failure at runtime.
Expected Behavior
Logging should not change the i2c_clk_src.
Actual Behavior
i2c_param_config() fails because the i2c_clk source is invalid when using LOGd (which is not output), but is i2c_clk_src is valid when using LOGI (which is output).
`
Steps to reproduce

idf.py -p  flash monitor
Note abort

Code to reproduce this issue
https://gist.github.com/william-ferguson-au/f54ea710914d5b304a24b8e9c51a1f36
Debug Logs
E (486) i2c: i2c_param_config(645): i2c clock choice is invalid, please check flag and frequency
ESP_ERROR_CHECK failed: esp_err_t 0x102 (ESP_ERR_INVALID_ARG) at 0x40084488
0x40084488: _esp_error_check_failed at E:/Dev/Espressif/esp-idf/components/esp_common/src/esp_err.c:41

file: "../main/main.cpp" line 102
func: uint8_t u8g2_esp32_i2c_byte_cb(u8x8_t*, uint8_t, uint8_t, void*)
expression: i2c_param_config(I2C_MASTER_NUM, &conf)

abort() was called at PC 0x4008448b on core 0
0x4008448b: _esp_error_check_failed at E:/Dev/Espressif/esp-idf/components/esp_common/src/esp_err.c:42

Backtrace:0x400860c3:0x3ffb20f0 0x400868c5:0x3ffb2110 0x4008bc92:0x3ffb2130 0x4008448b:0x3ffb21a0 0x400d59b4:0x3ffb21c0 0x400d5fcd:0x3ffb2200 0x400d5dea:0x3ffb2220 0x400d61b1:0x3ffb2240 0x400e7fdd:0x3ffb2260 0x400d5ca4:0x3ffb2280
0x400d5cd5:0x3ffb2300 0x400e8c6c:0x3ffb2320 0x40088b1d:0x3ffb2340
0x400860c3: panic_abort at E:/Dev/Espressif/esp-idf/components/esp_system/panic.c:367
0x400868c5: esp_system_abort at E:/Dev/Espressif/esp-idf/components/esp_system/system_api.c:108
0x4008bc92: abort at E:/Dev/Espressif/esp-idf/components/newlib/abort.c:46
0x4008448b: _esp_error_check_failed at E:/Dev/Espressif/esp-idf/components/esp_common/src/esp_err.c:42
0x400d59b4: u8g2_esp32_i2c_byte_cb(u8x8_struct*, unsigned char, unsigned char, void*) at e:\source\wylas\wt_trinity\build/../main/main.cpp:102 (discriminator 2)
0x400d5fcd: u8x8_cad_ssd13xx_fast_i2c at e:\source\wylas\wt_trinity\build/../components/u8g2/csrc/u8x8_cad.c:546
0x400d5dea: u8x8_d_helper_display_init at e:\source\wylas\wt_trinity\build/../components/u8g2/csrc/u8x8_display.c:68
0x400d61b1: u8x8_d_ssd1306_128x64_noname at e:\source\wylas\wt_trinity\build/../components/u8g2/csrc/u8x8_d_ssd1306_128x64_noname.c:321
0x400e7fdd: u8x8_InitDisplay at e:\source\wylas\wt_trinity\build/../components/u8g2/csrc/u8x8_display.c:100
0x400d5ca4: init_display() at e:\source\wylas\wt_trinity\build/../main/main.cpp:255
0x400d5cd5: app_main at e:\source\wylas\wt_trinity\build/../main/main.cpp:261
0x400e8c6c: main_task at E:/Dev/Espressif/esp-idf/components/freertos/port/port_common.c:133 (discriminator 2)
0x40088b1d: vPortTaskWrapper at E:/Dev/Espressif/esp-idf/components/freertos/port/xtensa/port.c:168

Other items if possible
NB Swapping the ESP_LOGD to ESP_LOGI on line#145 causes the code to work.
I can't see how enabling log at that point would have that side effect but I would love to know why.

Ignore. Looks like this has been fixed with 31afe8109f4ccc1b60c76c77d9c33c683d9c5356

OK, after we are getting closer to the real underlying bug which appears to be that not logging causes the i2c_clk source to be set incorrectly and logging somehow forces it to be correct.

You are not zero initializing i2c_config_t

@negativekelvin True but I tried both
i2c_config_t conf = {};

i2c_config_t conf;
memset(&conf, 0, sizeof(conf));

And it still fails when logging is omitted.

@william-ferguson-au thanks for your issue.
i2c_config_t conf;
            conf.mode = I2C_MODE_MASTER;
            ESP_LOGI(TAG, "sda_io_num %d", u8g2_esp32_hal.sda);
            conf.sda_io_num = u8g2_esp32_hal.sda;
            conf.sda_pullup_en = GPIO_PULLUP_ENABLE;
            ESP_LOGI(TAG, "scl_io_num %d", u8g2_esp32_hal.scl);
            conf.scl_io_num = u8g2_esp32_hal.scl;
            conf.scl_pullup_en = GPIO_PULLUP_ENABLE;
            ESP_LOGI(TAG, "clk_speed %d", I2C_MASTER_FREQ_HZ);
            conf.master.clk_speed = I2C_MASTER_FREQ_HZ;
            ESP_LOGI(TAG, "i2c_param_config %d", conf.mode);
            ESP_ERROR_CHECK(i2c_param_config(I2C_MASTER_NUM, &conf));
            ESP_LOGI(TAG, "i2c_driver_install %d", I2C_MASTER_NUM);
            ESP_ERROR_CHECK(i2c_driver_install(I2C_MASTER_NUM, conf.mode, I2C_MASTER_RX_BUF_DISABLE, I2C_MASTER_TX_BUF_DISABLE, 0));

You ignored clk_flags in i2c_config_t, you'd better set it zero(0) on esp32, the related documentation is in progress.
BTW, for initializing C structure, we strongly recommand to use a more formal way, like：
struct_name_t xxx = {
    .content = value;
    ....
};

@mythbuster5 Thanks. I tried explicitly setting
conf.clk_flags = 0;
But with the same outcome.
And it still doesn't explain why logging or not logging varies the behaviour.

Closing. This went away when I included the u8g2 CPP sources. Suspect that something is not allocating or checking for memory correctly in either u8g2 or i2c but I'm not going to follow any further.


Update landing page

This PR updates index.md to focus only on the huggingface_hub library and removes mentions of the widgets and community API inference. This page provides:

A brief overview of what the Hub is and how the huggingface_hub is related to the Hub and what it can do. I think there may be some potential for confusion from users over the naming of the Hub (as a product) and huggingface_hub (as a library), so please let me know if I've made this distinction clear enough here!
A lot of guardrails that will help users down the right path and find what they are looking for. This ensures a user can choose their topic and then find a resource to help them accomplish their task. Once we've merged the other PRs, I can add the links in :)

I didn't mention integrating third-party libraries yet since we don't really have anything concrete yet. We can update the landing page once we have something (next on my to-do list!).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


LoadError in Chargeback -> Assignments

After selecting Selected Cloud/Infrastructure Provider in Chargeback -> Assignments -> Compute, I'm getting:
LoadError (No such file to load -- manageiq/providers/azure/cloud_manager/orchestration_stack_option_converter):
  activesupport (4.2.4) lib/active_support/dependencies.rb:457:in `load'
  activesupport (4.2.4) lib/active_support/dependencies.rb:457:in `block in load_file'
  activesupport (4.2.4) lib/active_support/dependencies.rb:647:in `new_constants_in'
  activesupport (4.2.4) lib/active_support/dependencies.rb:456:in `load_file'
  activesupport (4.2.4) lib/active_support/dependencies.rb:354:in `require_or_load'
  activesupport (4.2.4) lib/active_support/dependencies.rb:317:in `depend_on'
  activesupport (4.2.4) lib/active_support/dependencies.rb:233:in `require_dependency'
  lib/extensions/require_nested.rb:8:in `require_nested'
  app/models/manageiq/providers/azure/cloud_manager.rb:9:in `<class:CloudManager>'
  app/models/manageiq/providers/azure/cloud_manager.rb:1:in `<top (required)>'
  ...

Pinging @bzwei on this one.

fixed by https://github.com/ManageIQ/manageiq/pull/4981


Error when installing

Trying to install new instance of Huginn with Ubuntu 14.04
when i get to here
sudo -u huginn -H bundle exec rake db:seed RAILS_ENV=production SEED_USERNAME=admin SEED_PASSWORD=yourpassword
get error
user<EMAIL_ADDRESS>already exist. not seeding
If I move on and do all the steps. The website gives Bad Request error.

If you pull from master, I believe @dsander recently fixed this.

Bad request probably means that unicorn did not start. @signallhill Did you check the  logs?

I did the install again, making sure I follow all the instruction. I can see Huginn login page, but
sudo -u huginn -H bundle exec rake db:seed RAILS_ENV=production SEED_USERNAME=admin SEED_PASSWORD=yourpassword
Said user<EMAIL_ADDRESS>already exist. not seeding.
I cannot login to the admin page. I am guessing this command would pull the master according to the instruction.
sudo -u huginn -H git clone https://github.com/cantino/huginn.git -b master huginn

Did you already seed the database before without providing the changed SEED_PASSWORD? We changed the seed script to not change existing users, if<EMAIL_ADDRESS>already exists. Try logging in with "admin" and "password".

I just followed the instruction. Copy Paste.
Yes user: admin password: password worked.


chore(main): release 1.4.32

:robot: I have created a release beep boop
1.4.32 (2024-11-14)
Miscellaneous Chores

deps-dev: bump php-http/mock-client from 1.6.0 to 1.6.1 (#212) (6fa84b3)
deps-dev: bump phpstan/phpstan from 1.12.7 to 2.0.1 (#218) (2d0fff0)
deps: bump the symfony group across 1 directory with 2 updates (#220) (d629efc)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/mattermost-php/releases/tag/v1.4.32 :sunflower:


Have I performed  proper steps to convert PyTorch traces?

Hello.
I could run astra-sim as steps in Readme, i.e. use chakra ETconverter to convert ASTRA-sim 1.0 text input files into Chakra traces and run the simulation process with AnalyticalAstra. However, an error occurred when I tried to converte PyTorch traces to Chakra traces.
The error is
ERROR [09/23/2023 05:35:41 PM] 'nodes'
My command is
python3 -m et_converter.et_converter     --input_type PyTorch     --input_filename ../../../trace.json     --output_filename myout   --num_npus 64     --num_dims 1     --num_passes 1 --default_simulated_run_time 100
And I utilized the following code snippet to generate PyTorch trace.
with profile(activities=[ProfilerActivity.CPU, ProfilerActivity.CUDA]) as prof:
    model(inputs)
prof.export_chrome_trace("trace.json")

I wonder if there any error in my process,  and appreciate any suggestion.

Thank you for reaching out and providing detailed information about the issue you encountered.
I'd like to inform you that the et_converter for PyTorch traces is currently under active development. It's not fully mature yet, but we are aware of its limitations. We are working diligently on it and aim to release a more stable version within the next 2-3 weeks.
Regarding the method you've used to generate the PyTorch trace, it seems there is a discrepancy. Instead of using the chrome profiler, you should be utilizing the ExecutionGraphObserver within PyTorch. Additionally, you should be collecting kineto traces. In addition, you should use trace_link.py that will assist in merging execution traces with kineto traces. The execution traces provide information about the operators and their dependencies, while the kineto traces give details about the duration of the operators.
Rest assured, we'll release these tools along with detailed instructions in the upcoming 2-3 weeks.
We appreciate your patience and understanding. In the meantime, if you have any further questions or face any other issues, please don't hesitate to let us know.

Great thanks for your information，  and eagerly await the release in two or three weeks.Thanks,Yuan------------------ 原始邮件 ------------------发件人：Taekyung Heo @."时　间：2023/09/25 11:28:36 周一收件人：astra-simastra-sim @."抄送人：bluegrey23 @.", Author @."主　题：Re: [astra-sim/astra-sim] Have I performed  proper steps to convert PyTorch traces? (Issue #114)Thank you for reaching out and providing detailed information about the issue you encountered.I'd like to inform you that the et_converter for PyTorch traces is currently under active development. It's not fully mature yet, but we are aware of its limitations. We are working diligently on it and aim to release a more stable version within the next 2-3 weeks.Regarding the method you've used to generate the PyTorch trace, it seems there is a discrepancy. Instead of using the chrome profiler, you should be utilizing the ExecutionGraphObserver within PyTorch. Additionally, you should be collecting kineto traces. In addition, you should use trace_link.py that will assist in merging execution traces with kineto traces. The execution traces provide information about the operators and their dependencies, while the kineto traces give details about the duration of the operators.Rest assured, we'll release these tools along with detailed instructions in the upcoming 2-3 weeks.We appreciate your patience and understanding. In the meantime, if you have any further questions or face any other issues, please don't hesitate to let us know.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Saving state and displaying several selected atoms in molecule

I continue to transfer ngl functionality to react. I can’t figure out how to show selected atoms on a 3D model.
I found this function, but problem with displaying of  selected atom is stayed.
    stage.signals.clicked.add((pickingProxy) => {
      if (pickingProxy && (pickingProxy.atom || pickingProxy.bond)) {
        const atom = pickingProxy.atom || pickingProxy.closestBondAtom;
        updateSelectedResnos(atom.resno); // Update selected residues
      }
    });

How can i modifying my current code?
import { useEffect, useRef } from "react";
import * as NGL from "ngl";
// eslint-disable-next-line react/prop-types
const MoleculeViewer = ({ pdbId = "4hhb" }) => {
  const ref = useRef();
  const representation = useRef(null);

  useEffect(() => {
    const stage = new NGL.Stage(ref.current);
    stage
      .loadFile(`rcsb://${pdbId}`)
      .then(function (o) {
        console.log("Loaded object:", o); // For debugging
        const rep = o.addRepresentation("cartoon", {
          defaultRepresentation: true,
        });
        representation.current = rep;
        o.autoView();
      })
      .catch((error) => {
        console.error("Error loading the molecule:", error);
      });

    return () => {
      if (stage) {
        stage.viewer.wrapper.remove(); // div wrapper generated by NGL. Hides the subsequent stages if not removed
        stage.viewer.renderer.forceContextLoss();
        stage.dispose();
      }
    };
  }, [pdbId]);

  const handleChangeColor = () => {
    representation.current?.setParameters({ color: "green" });
  };

  return (

      <button onClick={handleChangeColor}>Change Color to Green</button>

  );
};

export default MoleculeViewer;

The event listener should be attached to the stage at the place where the stage is declared.
For the rest of the code that you need to add it depends what you want to do. For example, you can bind it to a react state variable (like adding to an array of all the selected atoms) and have a useEffect depend on this list of selected atoms to update a representation selection.
This is not trivial, because there are design issues related to React and functionality from NGL.
This takes a long time to explain the details and I'm afraid I cannot do more while also maintaining the NGL library.


Need to adapt row subscript style to google-cloud-ruby's recent modification.

google-cloud-ruby API has been changed in following issue: Switch returned data to use symbols
So embulk-input-bigquery's returns empty results from google-cloud-bigquery v0.24.0.
https://github.com/narita-takeru/embulk-input-bigquery/blob/master/lib/embulk/input/bigquery.rb#L62

Solved:  https://github.com/narita-takeru/embulk-input-bigquery/pull/7


Link to explorer should be provided for tx via Transak

Actual scenario :
User is able to purchase STX (outside USA) via Transak both by credit card and also bank transfer
However user receives order status without a link to the explorer of the tx id.
This can be cumbersome for users as they would want to view a link which will aid them direct access to explorer via the order status dialogue box
Suggestion
either the user's mainnet add should be made as a link to explorer or the tx id.Depending on the preferences from PM's input

Is this a change on Transak's UI?

I'll close this issue since it's out of our scope


Gorm string type turned into an integer when using SQLITE3

GORM Playground Link

GORM PLAYGROUND LINK

GO PLAYGROUND LINK

⚠  Go playground  has CGO Disabled, hence you cannot run code importing the SQLITE Driver.
I made a really easy to read and documented Minimum Reproducible Example so that you don't waste more than 2 min setting up a local test environment on your machine.
Description
Contextual information:

Go Version: 1.20
GORM Version: gorm.io/gorm v1.25.2-0.20230530020048-26663ab9bf55
DB Driver:  gorm.io/driver/sqlite v1.5.2

This seems to be a follow up on this old issue as i get the exact same behavior.
I have a User struct and a UserPermission struct:
type User struct {
	UserID string `gorm:"column:user_id;primaryKey" json:"user_id"`
}

type UserPermission struct {
	ID     uint   `gorm:"primaryKey;autoIncrement" json:"-"`
	UserID string `gorm:"type:text;column:user_id;not null;index:idx_tenant_tech;constraint:OnDelete:CASCADE" json:"user_id"`
	User   User   `gorm:"foreignKey:UserID;constraint:OnDelete:CASCADE"`
	Role   Role   `gorm:"column:role;not null" json:"role"`
}

If i migrate the User struct only all is well and good, user_id is of type text:
CREATE TABLE users (user_id text,PRIMARY KEY (user_id))
BUT ! if i start migrating the UserPermission struct alongside it's User counterpart then of all a sudden the User table has it's user_id type set to integer:
CREATE TABLE users (user_id integer,PRIMARY KEY (user_id),CONSTRAINT fk_user_permissions_user FOREIGN KEY (user_id) REFERENCES user_permissions(id) ON DELETE CASCADE).
I also tried "hardcoding" the type by explicitly setting the type:text or even type:varchar(255). tag on the field but to no avail.

Will reopen with the proper GORM Playground. i read it as Go Playground .

I created the pull request, this issue is now re-opened

For foreign key fields, it should use the same data type as the referred field, so you can change the referred field's type.


Hook Commands Don't Have All Envs

I tried something that also needed HOME.  Should we enable all the host env vars to be passed to the hook commands?

yeah I think we should totally do that.
many env vars like LC_*, TERM, EDITOR ... can come in handy to hooks.

we need to remove this:
pathEnv := fmt.Sprintf("PATH=%s", os.Getenv("PATH"))

and just construct the array like:
hookEnvs := []string{a, b, c, os.Environ()...)


[transport:ws] Invalid domain (credentials)？why?

Client paused
free.player.1.8.4.js:39 [wsp] The associated resource or media provider object (such as a MediaStream) has been found to be unsuitable.
_log @ free.player.1.8.4.js:39
<IP_ADDRESS>/:1 Uncaught (in promise) DOMException
free.player.1.8.4.js:39 Client paused
free.player.1.8.4.js:39 Client started
free.player.1.8.4.js:39 [mse] Media source opened: open
free.player.1.8.4.js:39 [info] ------------- Info ---------------------
free.player.1.8.4.js:39 [info] License expires at :  Invalid expiration time
free.player.1.8.4.js:39 [info] Requested domain   :  <IP_ADDRESS>
free.player.1.8.4.js:39 [info] -----------------------------------------
free.player.1.8.4.js:39 [transport:ws] [ctrl] close. code: 4000 Invalid Domain (credentials)
_log @ free.player.1.8.4.js:39
free.player.1.8.4.js:39 [transport:ws] Invalid domain (credentials)
_log @ free.player.1.8.4.js:39
free.player.1.8.4.js:39 [wsp] Invalid Domain (credentials)
_log @ free.player.1.8.4.js:39
free.player.1.8.4.js:sourcemap:39 Client paused
free.player.1.8.4.js:sourcemap:39 [wsp] The associated resource or media provider object (such as a MediaStream) has been found to be unsuitable.

I have actived my lisence and placed lisence at server，but i can play rtsp video only localhost.

how to close acl?

hi,I have the same problem,how to solve it?

When binding the address, fill in the address to access the application service, not the proxy server address. For example, my <IP_ADDRESS> has the application service deployed on it, and my <IP_ADDRESS> has the video proxy service deployed on it. Our application needs to access the RTSP stream address, so the address to be filled in when binding should be <IP_ADDRESS>, not <IP_ADDRESS>. The conclusion is that it restricts an activation code to bind only one domain name, and all services under this domain name can use this plug-in.

when the license is being activated on the PC where the proxy is running need to write 1 ip or 1 domain name the PC where the html page with player will be requested from by user.


Match multi value claims

Claims are currently parsed as string. Is it possible to support matching multi value claims?

hi @avthart .. admittedly I wanted this myself, but it made the parsing more complicated and as yet haven't got around to revisiting

@gambol99 I've opened a PR https://github.com/gambol99/keycloak-proxy/pull/364 to fix this

@stefan-improbable Thx!


Influence of [SEP] token

Hi, I noticed that in the paper you didn't add [SEP] token at the tail of query/document. Did you observe any performance difference with/without it?  I also wonder the effect of [Q] and [D] token.

We do in fact keep the [SEP] tokens! Sorry if this wasn't clear in the text.
That said, with very large training sets, the model can easily adjust to these minor differences in principle. [Q] and [D] serve to differentiate queries and documents for the model. But they're definitely not essential if you have a large training set where queries and documents look very different already. The model can learn to guess correctly.

Thank you!