Style and add JS buttons

Style the buttons, and dynamically add "Hide fun fact" button that appears and disappears along with fun fact text, and allows user to "close" that div.

I'm not certain whether styling and adding a button with JS is enough to warrant a CL like this; should I add more before requesting review, or is this an appropriate size?

In answer to your question above this is a great size for a PR! It's a quick read and all the changes here are related.

I see that the changes are approved, but I made additional commits after you both reviewed this CL; what is the appropriate action here? Should I merge without requesting additional review, wait for further review, or use my best judgment based on the changes that I made?


Installation of jupyter notebooks fails when running Config & Demo option

After selecting S - RasQberry Setup - CD - Configs & Demo, the following packages fail to install with error shown below:

jupyter notebooks
Jupyter Notebook Widgets

configure jupyter notebooks
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
/home/pi/.local/bin/rq_jupyter_conf.sh: line 6: /home/pi/.local/bin/jupyter: No such file or directory
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'notebook'

sed: can't read /home/pi/.jupyter/jupyter_notebook_config.py: No such file or directory
sed: can't read /home/pi/.jupyter/jupyter_notebook_config.py: No such file or directory
Jupyter Notebook Widgets
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
/home/pi/.local/bin/rq_jupyter_conf.sh: line 14: jupyter: command not found
/home/pi/.local/bin/rq_jupyter_conf.sh: line 15: jupyter: command not found
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.

SSH is enabled and the default password for the 'pi' user has not been changed.
This is a security risk - please login as the 'pi' user and type 'passwd' to set a new password.

Cloning into 'Fun-with-Quantum'...
remote: Enumerating objects: 390, done.
remote: Counting objects: 100% (63/63), done.
remote: Compressing objects: 100% (45/45), done.
remote: Total 390 (delta 40), reused 17 (delta 17), pack-reused 327
Receiving objects: 100% (390/390), 11.36 MiB | 2.67 MiB/s, done.
Resolving deltas: 100% (214/214), done.
Cloned Fun-with-Quantum Repository
Cloning into 'qiskit-tutorials'...
remote: Enumerating objects: 11562, done.
remote: Counting objects: 100% (11562/11562), done.
remote: Compressing objects: 100% (4600/4600), done.
remote: Total 11562 (delta 6889), reused 11492 (delta 6873), pack-reused 0
Receiving objects: 100% (11562/11562), 150.82 MiB | 2.71 MiB/s, done.
Resolving deltas: 100% (6889/6889), done.
Cloned Qiskit-tutorials Repository
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
libsdl2-2.0-0 is already the newest version (2.26.5+dfsg-1).
libsdl2-2.0-0 set to manually installed.
libsdl2-image-2.0-0 is already the newest version (2.6.3+dfsg-1).
libsdl2-image-2.0-0 set to manually installed.
libsdl2-mixer-2.0-0 is already the newest version (2.6.2+dfsg-2).
libsdl2-mixer-2.0-0 set to manually installed.
libsdl2-ttf-2.0-0 is already the newest version (2.20.1+dfsg-2).
libsdl2-ttf-2.0-0 set to manually installed.
0 upgraded, 0 newly installed, 0 to remove and 19 not upgraded.
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
Installed Kivy

Verifying whether or not the installation went through:
pi@raspberrypi:~ $ jupyter notebook --version
-bash: jupyter: command not found
pi@raspberrypi:~ $ pip3 list installed | grep jupyter
pi@raspberrypi:~ $

We do however see this at the end of the installation process nonetheless.

Suggested workaround - perhaps install using the pipx command if python3 -m venv path/to/venv. is not an option. That was what I used to install gist which also had failed in the initial installation.
pi@raspberrypi:~ $ pip3 install getgist
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

$ sudo apt install pipx

3/

pi@raspberrypi:~ $ pipx install getgist
  installed package getgist 0.2.2, installed using Python 3.11.2
  These apps are now globally available
    - getgist
    - getmy
    - lsgists
    - putgist
    - putmy
⚠  Note: '/home/pi/.local/bin' is not on your PATH environment variable. These apps will not be globally accessible until your PATH is updated. Run `pipx ensurepath` to
    automatically add it, or manually modify your PATH in your shell's config file (i.e. ~/.bashrc).
done! ✨ 🌟 ✨
pi@raspberrypi:~ $ pipx ensurepath
Success! Added /home/pi/.local/bin to the PATH environment variable.

Consider adding shell completions for pipx. Run 'pipx completions' for instructions.

You will need to open a new terminal or re-login for the PATH changes to take effect.

Otherwise pipx is ready to go! ✨ 🌟 ✨

Hi @cybersecfaith  thanks a lot for the detailed issue report and the suggested workaround !
On what exact version of Raspberry Pi OS did you try this?
Which option did you choose when initiatlizing the SD card with Raspberry Pi Imager?

Hi @JanLahmann - that might explain it. I used the default recommended OS version for my pi on imager which was the 64bit. I'll reimage and use the 32bit. Shouldn't be a problem.

Specific version that I am running is bookworm.
pi@raspberrypi:~/RasQberry/config $ uname -a
Linux raspberrypi 6.6.31+rpt-rpi-v8 #1 SMP PREEMPT Debian 1:6.6.31-1+rpt1 (2024-05-29) aarch64 GNU/Linux
pi@raspberrypi:~/RasQberry/config $ lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 12 (bookworm)
Release:	12
Codename:	bookworm
pi@raspberrypi:~/RasQberry/config $ cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 12 (bookworm)"
NAME="Debian GNU/Linux"
VERSION_ID="12"
VERSION="12 (bookworm)"
VERSION_CODENAME=bookworm
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"


New release

Is there ever going to be a new released put out? Currently we have to point to latest commit in master to have RTL support for instance.

#135 similar question


desktop.files.json kept modifying folder properties (modified date/time)

Description
desktop.files.json kept modifying folder properties (modified date/time). Adding that json file might be important, but it messes up sort options of folders according to new/old
Steps To Reproduce
No response
Expected behavior
it should not affect folder properties, and should be treated as a hidden system file
Files Version
Version: <IP_ADDRESS>
Windows Version
OS Version: 10.0.19044.1348
Relevant Assets
No response

desktop.files.json does not get created anymore in latest (insider) version.

hope to see that change reflected soon in the stable version. thanks.

hope to see that change reflected soon in the stable version. thanks.

Agree. This bug is bad enough that I can't use Files as my main file manager because of this. Hope we can get a stable release soon! Anyway, a big thank you to the dev team for this awesome app!

In the meantime you can give the preview version a try: https://files.community/download/preview. It installs alongside (not replacing) the Store version.

In the meantime you can give the preview version a try: https://files.community/download/preview. It installs alongside (not replacing) the Store version. Store is unfortunately blocking new Files versions atm, so it may take some time for a new Store version.

Thanks a lot @gave92 , I installed the preview version. Looks good so far! Hope the store issue gets resolved soon. Again, thanks for your work on this amazing app!

In the meantime you can give the preview version a try: https://files.community/download/preview. It installs alongside (not replacing) the Store version. Store is unfortunately blocking new Files versions atm, so it may take some time for a new Store version.

I'm not able to install the preview version through website. I found some info about this problem:
https://docs.microsoft.com/en-us/windows/msix/app-installer/installing-windows10-apps-web
https://techcommunity.microsoft.com/t5/msix-deployment/the-ms-appinstaller-protocol-has-been-disabled/m-p/3038361
System: win11  21H2  22000.376

In the meantime you can give the preview version a try: https://files.community/download/preview. It installs alongside (not replacing) the Store version. Store is unfortunately blocking new Files versions atm, so it may take some time for a new Store version.

I'm not able to install the preview version through website. I found some info about this problem:
https://docs.microsoft.com/en-us/windows/msix/app-installer/installing-windows10-apps-web
https://techcommunity.microsoft.com/t5/msix-deployment/the-ms-appinstaller-protocol-has-been-disabled/m-p/3038361
System: win11  21H2  22000.376


fix: EnvironmentName should be case-sensitive

EnvironmentName should be case-sensitive. Some file systems are case-insensitive.
The batch app may fail to read a appsettings.Development.json.
in Windows
C:\> dotnet run
info: MicroBatchFramework.BatchEngine[0]
      GlobalValue: GLOBAL VALUE!!!!, EnvValue: ENV VALUE!!!!

in Linux
$ dotnet run
info: MicroBatchFramework.BatchEngine[0]
      GlobalValue: GLOBAL VALUE!!!!, EnvValue: ENV VALUE!!!!(PRODUCTION)

This code is used in ReadMe, I'll merge and fix ReadMe too.
cc: @guitarrapc

awesome, thanks for point out.


rewrite unit tests

... since the schema's all different, and much of the (python) codebase has been simplified or rewritten, this is probably necessary.
@ejhumphrey Any objection to switching from unittest.TestCase objects (old and busted) over to nosetest functions (new hotness)?  This will make tests easier to write, and play nicely with #25 .

Sketching out a plan for test design:

[ ] JObject

[ ] serialization, deserialization
[ ] equivalence testing
[ ] dict-like methods
[ ] search

[ ] Sandbox

[ ] just test for construction; all else inherits from JObject

[ ] JamsFrame

[ ] construction, conversion
[ ] add_observation
[ ] interval_values conversion
[ ] serialization (sparse and dense)

[ ] Annotation

[ ] construction
[ ] append

[ ] Curator

[ ] construction

[ ] AnnotationMetadata

[ ] construction

[ ] FileMetadata

[ ] construction

[ ] AnnotationArray

[ ] construction
[ ] search
[ ] serialization

[ ] JAMS

[ ] construction
[ ] load
[ ] save
[ ] add
[ ] validate

[ ] pyjams functions

[ ] load
[ ] append
[ ] load_lab

[ ] Utilities

[ ] timedelta_to_float
[ ] serialize_obj (should be implicit from JamsFrame tests)
[ ] read_lab (do we still need this?)
[ ] load_textlist
[ ] expand_filepaths
[ ] smkdirs
[ ] filebase
[ ] find_with_extension
[ ] match_query (semi-redundant with JObject.search test)
[ ] query_pop (also semi-redundant with JObject.search test)

Nosetests +1
This test list is.... well, I have no words, I'm so impressed.
On Thu, Feb 26, 2015 at 10:27 AM, Brian McFee<EMAIL_ADDRESS>wrote:

Sketching out a plan for test design:

JObject

serialization, deserialization
equivalence testing
dict-like methods
search

-  Sandbox
  -  just test for construction; all else inherits from JObject
-  JamsFrame
  -  construction, conversion
  -  add_observation
  -  interval_values conversion
  -  serialization (sparse and dense)
-  Annotation
  -  construction
  -  append
-  Curator
  -  construction
-  AnnotationMetadata
  -  construction
-  FileMetadata
  -  construction
-  AnnotationArray
  -  construction
  -  search
  -  serialization
-  JAMS
  -  construction
  -  load
  -  save
  -  add
  -  validate
-  pyjams functions
  -  load
  -  append
  -  load_lab
-  Utilities
  -  timedelta_to_float
  -  serialize_obj (should be implicit from JamsFrame tests)
  -  read_lab (do we still need this?)
  -  load_textlist
  -  expand_filepaths
  -  smkdirs
  -  filebase
  -  find_with_extension
  -  match_query (semi-redundant with JObject.search test)
  -  query_pop (also semi-redundant with JObject.search test)

—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-76197370.

This test list is.... well, I have no words, I'm so impressed.

save being impressed for after we've implemented it!

sounds great; any preferred examples to share of this in practice? or any
ole google search will do?
On Thu, Feb 26, 2015 at 10:32 AM, Brian McFee<EMAIL_ADDRESS>wrote:

This test list is.... well, I have no words, I'm so impressed.
save being impressed for after we've implemented it!
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-76198316.

any preferred examples to share of this in practice?

nose is super easy to use, but librosa has tons of (not so clean) examples

In the process of implementing tests, I've been doing a little refactoring and simplification.
Specifically:

All pyjamsutil functions not relating to file IO have been moved into pyjams
namespace (now ns) functions no longer depend on pyjams or util.  Namespace schema are now loaded in the top-level module after all submodules have been imported.
This means that the submodule dependency chain now looks like:

ns < pyjams < util

As such, I've moved the lab->annotation loader from pyjams into util, where it makes more sense.
So, the question for all of you is:

do we still need the previous read_lab?  It seems more brittle than the pandas-based parser in import_lab.  (Once again, never write csv parsers yourself.)
what about load_textlist?

We do need a function to convert labs to jams. As long as it works, I don't care whether we use the current read_lab or not (sorry, is that what you're asking?).

Yes; to clarify, we currently have two:

read_lab reads csv directly, and returns a flat array.  This function exists in the initial jams release.
import_lab uses pandas csv loader, requires a target namespace, and constructs an Annotation object.  Optionally, it can append directly into an existing JAMS.  I added this function to streamline the process and work around some of the inevitable quirks of csv parsing.

I'm advocating dropping read_lab, but if there's a compelling reason to include our own csv/tsv->array parser, I'm open to keeping it.

I'd rather not use a custom implementation if another one already exists
in, e.g., pandas.
On Sat, Mar 7, 2015 at 6:15 PM, Brian McFee<EMAIL_ADDRESS>wrote:

Yes; to clarify, we currently have two:

read_lab reads csv directly, and returns a flat array. This function
exists in the initial jams release.
import_lab uses pandas csv loader, requires a target namespace, and
constructs an Annotation object. Optionally, it can append directly into an
existing JAMS. I added this function to streamline the process and work
around some of the inevitable quirks of csv parsing.

I'm advocating dropping read_lab, but if there's a compelling reason to
include our own csv/tsv->array parser, I'm open to keeping it.
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-77715778.

pandas +1

Coming back to this thread now.
What are people's thoughts on dropping load_textlist as well?  It's a one-liner that gets used twice in the (old) import scripts.  It sucks an entire text file into an array of strings, in memory.
I'd like to drop it, since it's A) simple enough to reimplement as needed (with increased clarity in the resulting code), and B) kind of a bad pattern, since it could crash out of memory if executed on a very large file.  I think, in all cases, we can replace it with calls to pandas csv/table parsers with no loss of functionality.

++
On Wed, May 13, 2015 at 2:40 PM, Brian McFee<EMAIL_ADDRESS>wrote:

Coming back to this thread now.
What are people's thoughts on dropping load_textlist as well? It's a
one-liner that gets used twice in the (old) import scripts. It sucks an
entire text file into an array of strings, in memory.
I'd like to drop it, since it's A) simple enough to reimplement as needed
(with increased clarity in the resulting code), and B) kind of a bad
pattern, since it could crash out of memory if executed on a very large
file. I think, in all cases, we can replace it with calls to pandas
csv/table parsers with no loss of functionality.
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-101771823.

++ too
On Wed, May 13, 2015 at 2:52 PM, Eric J. Humphrey<EMAIL_ADDRESS>wrote:

++
On Wed, May 13, 2015 at 2:40 PM, Brian McFee<EMAIL_ADDRESS>wrote:

Coming back to this thread now.
What are people's thoughts on dropping load_textlist as well? It's a
one-liner that gets used twice in the (old) import scripts. It sucks an
entire text file into an array of strings, in memory.
I'd like to drop it, since it's A) simple enough to reimplement as needed
(with increased clarity in the resulting code), and B) kind of a bad
pattern, since it could crash out of memory if executed on a very large
file. I think, in all cases, we can replace it with calls to pandas
csv/table parsers with no loss of functionality.
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-101771823.

—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-101775569.

:+1:

Do we really need the 'jams.append()` function?  It seems to do too much: load, append, save..  I would prefer to factor out the load/save (already done) and just have a single function that appends one jam to another.
(Again, this is in the interest of simplifying the API and unit tests.)

+1

I would also support this. However, I would like to sit down with you
because I'm lost about the state of things with JAMS, and I would really
like to contribute.
On Mon, May 18, 2015 at 4:27 PM, Justin Salamon<EMAIL_ADDRESS>wrote:

+1
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-103199801.

Ok folks:
I've started pushing up some namespace validation tests.  They're super easy to write, but there are a lot of them, so help would be appreciated.
Existing examples are to be found in jams/tests/namespace_tests.py.
Some basic guidelines:

Include both pass and fail cases
If your namespace includes string values, be sure to try with both str and unicode
If your namespace has object or array values, be sure to try with missing fields or wrong-length arrays
If your namespace defines a confidence schema, test it

And, because I'm such a benevolent dictator, I made homework problems for everyone!

Assignments

@urinieto - segment_*, pattern_jku
@rabitt - pitch_class, chord_roman,  key_mode
@justinsalamon - pitch_* (except class)
@ejhumphrey - chord_harte, tag_*
@bmcfee - beat_*, lyrics, tempo, mood_thayer, onset

In the course of writing namespace tests, I uncovered a fun bug in the serialization logic.
JObject.json was (intentionally) skipping over fields whose values evaluated to False, which includes empty strings and zeros.  When you serialize the object, this omits several fields from the resulting dictionary structure, including things like FileMetaData.release and Curator.email.  Then, when you validate the entire object, since these fields are required, it barfs.
I've fixed it so that even empty fields are returned in the structure serialization.  This inflates the object on disk somewhat, but improves safety since we can guarantee that all fields are defined (if not populated).

Nice.
On Thu, May 28, 2015 at 11:16 AM, Brian McFee<EMAIL_ADDRESS>wrote:

In the course of writing namespace tests, I uncovered a fun bug in the
serialization logic.
JObject.json was (intentionally) skipping over fields whose values
evaluated to False, which includes empty strings and zeros. When you
serialize the object, this omits several fields from the resulting
dictionary structure, including things like FileMetaData.release and
Curator.email. Then, when you validate the entire object, since these
fields are required, it barfs.
I've fixed it so that even empty fields are returned in the structure
serialization. This inflates the object on disk somewhat, but improves
safety since we can guarantee that all fields are defined (if not
populated).
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-106397189.

I just realized that the JAMS don't validate now if the duration of the
file doesn't exist in the file_metadata field. What if we don't really have
this info? (e.g., it is missing in some of the SALAMI songs listed in the
metadata.csv file).
On Thu, May 28, 2015 at 11:38 AM, Oriol Nieto<EMAIL_ADDRESS>wrote:

Nice.
On Thu, May 28, 2015 at 11:16 AM, Brian McFee<EMAIL_ADDRESS>wrote:

In the course of writing namespace tests, I uncovered a fun bug in the
serialization logic.
JObject.json was (intentionally) skipping over fields whose values
evaluated to False, which includes empty strings and zeros. When you
serialize the object, this omits several fields from the resulting
dictionary structure, including things like FileMetaData.release and
Curator.email. Then, when you validate the entire object, since these
fields are required, it barfs.
I've fixed it so that even empty fields are returned in the structure
serialization. This inflates the object on disk somewhat, but improves
safety since we can guarantee that all fields are defined (if not
populated).
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-106397189.

What if we don't really have
this info? (e.g., it is missing in some of the SALAMI songs listed in the
metadata.csv file).

You should be able to set it to np.nan in this case.  But don't all of the salami annotations have end-of-track event markers?

Unfortunately, these end-of-track event markers are not always correct
(e.g., see track number 10).
On Thu, May 28, 2015 at 7:33 PM, Brian McFee<EMAIL_ADDRESS>wrote:

What if we don't really have
this info? (e.g., it is missing in some of the SALAMI songs listed in the
metadata.csv file).
You should be able to set it to np.nan in this case. But don't all of the
salami annotations have end-of-track event markers?
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-106628317.

Unfortunately, these end-of-track event markers are not always correct
(e.g., see track number 10).

I looked at track 10 and don't entirely understand the problem?  Yes, many of the annotations give different end time, but they're pretty close.  Why not just take the max of candidate end times as an estimate?

Oh yes, you're right. I misread the annotations. Ok, so I will go ahead and
add the max estimate to the duration field in the metadata in the SALAMI /
Isophonics parser.
On Fri, May 29, 2015 at 9:21 AM, Brian McFee<EMAIL_ADDRESS>wrote:

Unfortunately, these end-of-track event markers are not always correct
(e.g., see track number 10).
I looked at track 10 and don't entirely understand the problem? Yes, many
of the annotations give different end time, but they're pretty close. Why
not just take the max of candidate end times as an estimate?
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#issuecomment-106801035.

Do we have any jams files that validate against the latest schema kicking around anywhere? Or a quick way to randomly generate some? Sheesh, it's been a while...

Do we have any jams files that validate against the latest schema kicking around anywhere? Or a quick way to randomly generate some?

The SMC examples should validate.  There's also one copied over into test/fixtures that I use in the generic unit tests.
If you want to generate random ones, check this for an example of how.  For the namespace tests, you only need to make Annotation objects, not full JAMSes.

Trying to write a test for pitch_hz. Neither of the following two validate (I wasn't sure how to build a dense annotation since pitch_* seems to be the only case?):
def test_ns_pitch_vz_valid():

    ann = Annotation(namespace='pitch_hz')

    for time in np.arange(5.0):
        ann.append(time=time, duration=0.0, value=100.0, confidence=None)

    ann.validate()

And:
def test_ns_pitch_vz_valid():

    ann = Annotation(namespace='pitch_hz')

    seq_len = 5
    time = np.arange(seq_len)
    duration = np.zeros(seq_len)
    hz = np.random.uniform(-22050., 22050., seq_len)
    confidence = np.array([None]*seq_len)
    ann.append(time=time, duration=duration, value=hz, confidence=confidence)

    ann.validate()

Any ideas?

Super weird; I'm getting the same thing.
If I turn off strict validation, I get the following warning:
In [18]: ann.validate(strict=False)
/home/bmcfee/git/jams/jams/pyjams.py:775: UserWarning: 'duration' is not of type u'object'

Will investigate later...

Not sure if related, but when I create a pitch_hz annotation, the duration field gets converted into days. Could it be that duration is converted into some time-object that is not allowed by a dense annotation?

Not sure if related, but when I create a pitch_hz annotation, the duration field gets converted into days. Could it be that duration is converted into some time-object that is not allowed by a dense annotation?

No, that's just how timedelta fields get repr'd when the value is 0 and there's no other values to set an appropriate scale (ie seconds).  The objects do serialize correctly.
You're right that it has to do with dense-vs-sparse serialization though:
In [6]: ann.data.dense = False

In [7]: ann.validate()
Out[7]: True

I think this is entirely down to the validate method, and not a problem with the serialization itself.  I'll dig into it though.

I think this is entirely down to the validate method, and not a problem with the serialization itself. I'll dig into it though.

aha!  Yep. We are implicitly assuming a SparseObject serialization that can bet iterated over record-wise in this logic.  This is easily fixable.

17d510f should fix this error via some list comprehension wizardry.

pitch_hz allows negative values due to the "x<=0 implies unvoiced" convention. Do we want to keep this convention for pitch_midi? I'm not sure how an actual midi synth would handle negative note values, and it definitely doesn't use "0==silence". On the other hand, if we disallow these cases, then conversion between pitch_hz and pitch_midi becomes non-trivial.

Do we want to keep this convention for pitch_midi?

I would say no, since negative midi numbers are technically possible.  For midi-type pitch annotations, I would expect silence to be indicated by the lack of annotation.

but how would the lack of annotation be represented in a dense jams?

but how would the lack of annotation be represented in a dense jams?

Dense events still have duration, so I don't see a problem here.

Dense events still have duration, so I don't see a problem here.

In the case of melody (pitch_hz) since the events happen on a uniform time grid the duration values are not expected to carry any meaningful information (they can all be 0 for example) - so you can't really use the duration to encode silence for dense annotations (at least not the way they are set up right now)

[documenting offline conversation with @justinsalamon]

There is no difference between dense and sparse types in JAMS; the distinction only exists when objects are serialized.
pitch_hz-type annotations are often used as samples of a (persumably) continuous F0 contour.  Assumptions of continuity are implicit, and the sample times typically correspond to centers of windows, rather than the leading edge of an interval over which the annotation applies.  As such, the duration field has no meaning, and should be set to 0.
I whined a whole bunch about implicit assumptions in annotation formats, and advocated for not imposing any semantics on pitch_hz annotations.  The semantics should be encoded in the annotation_rules field.
since pitch_midi is for notating discrete events, rather than sampling contours (as in pitch_hz, usually, i guess), it does not necessarily make sense to convert between the two.

@rabitt can you link the site you mentioned for generating positive/negative examples from a regexp?

@bmcfee
http://fent.github.io/randexp.js/

I took the liberty of finishing the pattern_jku tests...
and we've got full tests!  Closing this one out.

Thanks titan!
On Mon, Aug 17, 2015 at 10:37 AM, Brian McFee<EMAIL_ADDRESS>wrote:

Closed #26 https://github.com/marl/jams/issues/26.
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/26#event-384300778.


Remove Paas Postgres

Trello card URL
https://trello.com/c/qRuJfuKh/272-decommission-paas-environments)
Changes in this PR:
Removes the Postgres Instance from PaaS

 Decommission PaaS environments


Extra blank lines appear when I rename symbols

Type: Bug
As the title suggests.
Man, what can I say?
This question has been bothering me for long.
VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:20:17.278Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Sorry for the trouble caused by not providing enough information.

Sorry for the trouble caused by not providing enough information. I am a programming amateur，so I don't know the rules. Excuse me.
VS Code version: Code 1.87.2
OS version: Windows_NT x64 10.0.22631
Extensions Installed: Python(by Microsoft), Python Debugger(by Microsoft)
Problem description
Suppose I edit the following Python3 code:
a = 123
d = 321
c = "abc"

If I choose the variable b and perform the Rename Symbol operation, it will look like this:
a = 123

d = 321

c = "abc"

As the title suggests, extra blank lines appear inexplicably. I guess it's a problem with the Python Debugger extension.


Code mismatch between github and NPM registry

Found something odd while debugging; The latest version cached (0.14.0) on on the NPM registry doesn't match the code in the Github repo here.
Compare https://registry.npmjs.org/backbone-forms/-/backbone-forms-0.14.0.tgz and the file distribution/backbone-forms.js in particular, it doesn't contain the submit events for example as declared in https://github.com/powmedia/backbone-forms/blob/r0.14.0/distribution/backbone-forms.js#L32-L36

Similar to #508

Published v0.14.1 tag to npm so it means bower / npmjs are in sync with the versioned releases.


Use the new vertx generator for f-m-p

We can simpify the pom.xml to use the f-m-p plugin to build the JAR and whatnot
https://maven.fabric8.io/#generator-vertx

@kameshsampath maybe you want to take a look and do a PR to update this sample to use your new plugin

@davsclaus we just need to make the first release to Maven central ;-) (which we will do today)

will check this out later today @davsclaus and do the required update

@davsclaus i have updated the project with v-m-p, but enroute found an https://github.com/fabric8io/vertx-maven-plugin/issues/15, will send a PR once  we do the next release of the v-m-p

@davsclaus and @rhuss  - fixed this via https://github.com/fabric8-quickstarts/vertx-web/pull/57 , please review and merge. Tested locally via k8s and using plugin both works

Thanks just ran it locally and it works, however not sure why SVC shows the port as 0 and not the actual external port
[INFO] F8:[SVC] vertx-web: http://<IP_ADDRESS>:0
I had to use gofabric8 service vertx-web to get the external url so I can hit it from my web browser. The other quickstarts report correct port number than 0.

Ah I see the same port 0 in the cdi-cxf quickstart
[INFO] F8:[SVC] cdi-cxf: http://<IP_ADDRESS>:0

@rhuss @jstrachan wonder if you guys have seen this with the new f-m-p that the service url listed dont include the correct external port number
The port can be retrieved from gofabric8
davsclaus:/Users/davsclaus/$ gofabric8 service cdi-cxf --url
http://<IP_ADDRESS>:31144

@davsclaus will check that tomorrow, thanks for reporting ;-)

sorry, for the delay, its not forgotten. will tackle that hopefully today.


Add support to login ghcr.io in gitpod

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7714e9c). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #112   +/-   ##
========================================
  Coverage          ?   5.34%
========================================
  Files             ?      74
  Lines             ?   21072
  Branches          ?       0
========================================
  Hits              ?    1126
  Misses            ?   19873
  Partials          ?      73

Flag
Coverage Δ

unittests
5.34% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

/lgtm


Some libs may return a function from render()

If so, we might think about checking the return value from render() and calling it if it's a function. If not and you use something like https://github.com/thejameskyle/incremental-dom-react-helper, you have to do:
// Note the extra "()"
return (<span />)();

Closed as it was fixed in #777.


Date Changing issue: how to perform scenario analysis with example datasets

Dear Lisphilar,
Under the section:-
Predict the number of cases: with actions from 15th day
There is a delay between the time point of starting actions and that of appearing the effect. Because  I  is the main variable, the length of delay can be estimated as sum of latent period and waiting time for confirmation. This value [day] was calculated in "SIR-F with exposed/waiting cases" section.

max(Infected)
argmax(Infected)
Infected on 17Aug2020
Fatal on 17Aug2020

232807
2020-03-23
104
57457

74113
2020-04-14
4197
37796

160879
2020-03-26
83
5919

221598
2020-03-23
22
50508

You have taken date 22 Jan as 15th day...,but I want to take 10th April as the 15th day...,so how to make changes in these scenarios...please help if possible...
With regards,
Rakesh

Dear Rakesh,
Sorry for my late reply.
No, the following codes (In[87] of the Kaggle Notebook) takes 22Jan2020 as 15th day.
Please confirm the steps commented here.
# Create province named "Lockdown" in Example country
simulator = cs.ODESimulator(country="Example", province="Lockdown")
# Add 15 days withOUT lockdown
simulator.add(
    model=cs.SIRF,
    step_n=15,
    population=eg_population,
    param_dict=param_dict,
    y0_dict={"Susceptible": 999_000, "Infected": 1000, "Recovered": 0, "Fatal": 0}
)
# Add int(latent_wating_day) days (=13 days) withOUT lockdown
simulator.add(
    model=cs.SIRF,
    step_n=int(latent_waiting_day),
    population=eg_population,
    param_dict=param_dict
)
# Add 180 days with lockdown
simulator.add(
    model=cs.SIRF,
    step_n=180,
    population=eg_population,
    param_dict=lock_param_dict
)
# Run simulation
simulator.run()
# Set 22Jan2020 as 0th day and plotting
pred_df_lock = simulator.dim(tau=eg_tau, start_date="22Jan2020").set_index("Date")
cs.line_plot(
    pred_df_lock[["Infected", "Fatal", "Recovered"]],
    "SIR-F model: Lockdown", h=eg_population, y_integer=True
)

This may be confusing because we are familier with Scenario class and JHUData now. However, we have ExampleData class (> version 2.5.0), a child class of JHUData, which produces example records.
Currentry, we have example/sirf_model.py and API reference, but I will create a .ipynb file to explain the usage of ExampleData class.

Dear @lisphilar ,
Thankyou...,it worked....

Dear Rakesh,
Usage of ExampleData was documented in https://lisphilar.github.io/covid19-sir/usage_theoretical.html
Please kindly create issues when you have any questions.


Fix changelog for 4.0.1

PR #72 landed in 4.0.1, not 4.0.0.

As suggested, maybe we could add an # Unreleased section at the top so people who just add to the top paragraph wouldn't be wrong?

Yeah. Added. I'll update the release instructions too at some point.


fix: Set sender when sending feedback

Signed-off-by: JacobSMoller<EMAIL_ADDRESS>Summary of Changes
Sets the sender to current user, same style as its done for notifications.
Tests
Don't see any use in testing if the flask functionality works.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes, including screenshots of any UI changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes all tests documented in the developer guide

In my mind it makes the most sense if setting the current user email is set at this point, instead of having to be handled in individual clients.
The individual clients can then handle email_type: feedback and set a recipient that makes sense for individual use cases.

In my mind it makes the most sense if setting the current user email is set at this point, instead of having to be handled in individual clients.
The individual clients can then handle email_type: feedback and set a recipient that makes sense for individual use cases.

Maybe it doesn't make sense.
I will think some more on how to handle this.

Maybe it doesn't make sense.
I will think some more on how to handle this.


Added warning for adding extra modules.

Fixes #2129
This code is still at primary stage.
Still a few things left:

[x] Add advanced usage warning on select game screen
[x] Add warning for adding extra module on select modules screen
[x] Change color of warning message

Also, Fixes #2077

[x] Complete the fix

[x] fix all the checkStyle warnings

Can one of the admins please verify this patch?

I couldn't find any example where the color of label is changed in a .ui file. Also, for making the color variable, I need to access the label from within Java.
How do I do both of these? Please help.

I first searched for "use-content-width" which was a unique string used in an UI field. Then I found out that SerializedName is the annotation used to map json field names to field in java classes. Then I searched for other SerializedName annotation usages and found out that there is one called "text-color" in UIStyleElement. Then I searched for usages of text-color in files and found hits in *.skin files.
So it seems to be possible to give labels colours via skins. I hope that helps.

@flo  Thanks a lot. I'll try that.

ok to test
Thanks for working on this, @genuinelucifer :-)

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/461/
Hooray Jenkins reported success with all tests good!

@Cervator I was planning to explore the codebase step by step. First some file via #2201 . Then via this. And after this, I plan to try #2077, if I learn hwo java files are linked to ui files while doing this one.

@genuinelucifer sounds like a plan :)

Finally successful in changing the color. Probably not a good idea. But, couldn't find another method.
Thanks @flo for the help.
.
Also, Only adding variable color left now. Will try it. But, unable to find a way to change the color in java code. Any help?

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/466/
Hooray Jenkins reported success with all tests good!

Eh the color changing thing is pretty non-essential at this point, really :-)
If you look at the create world -> modules screen the module name labels will change color as you select and enable/disable them. Maybe that would be a good example?
Or we could just go without. The severity of extra enabled modules is very subjective/inconsistent. You could be adding three modules with nothing other than Core selected and probably be alright, or add just one one top of a full TTA template and break everything :-)
I'm happy to just test and merge this as-is unless you'd really like to try taking it even further.

Oh great. I'll squash the commits into one then.
I have been trying but there seems no dynamic way of changing. The modules
colors are a set of pre specified colors. Which are determined by their
state. I was trying to make it dynamic from yellow to orange to red with
all the shades in between :P
I'll move on to the next issue then for providing a box to search modules
and display only those.
.
Or is there any other issue (more important) you would like me to work on?!
:)

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/468/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/469/
Hooray Jenkins reported success with all tests good!

@Cervator Sorry for so many rebases. I have been just trying to minimise the number of changes. From my side, it's complete now. Please review and merge if it's ok.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/470/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/471/
Hooray Jenkins reported success with all tests good!

@Cervator Please don't merge. I'm fixing #2077 also in this.

Currently the list is updated only when a new character is typed. The list is NOT updated when a character is deleted by pressing backspace. My only guess is that the CursorUpdateListener is not called in such case. I'm looking into it.
Also, I used the CamelCaseMatcher as suggested by @msteiger in the issue thread. The only problem I found is the matcher insist on macthing from the start.
Ex: Sample is NOT matched to Builder Sample Gameplay it is matched to only those names which start with the word Sample. I guess we should match in between the strings too.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/472/
Hooray Jenkins reported success with all tests good!

That's a good use case for another topic branch btw :-) Then we could merge the first fix and the other later.
Being that the CamelCaseMatcher was previously used for auto-completion that may explain why it only considers starting from the beginning of the string. Maybe an option / variant could be added that filters instead, probably shouldn't modify the current behavior.
Unfamiliar with the CusorUpdateListener, maybe we hadn't previously needed the erase part. Let me know what you figure out and when to merge :-)

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/474/
Hooray Jenkins reported success with all tests good!

@Cervator From my side this PR is complete. It is working as required. Also, I minimised the changes and squashed commits into 2, one for each issue fixed.
Please review and merge if it's acceptable.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/475/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/476/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/477/
Hooray Jenkins reported success with all tests good!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/478/
Hooray Jenkins reported success with all tests good!

Finally all done and checked passed. :dart:
.
@Cervator Please provide me some other issue to work on. I guess I have got enough experience with the codebase to work on more dificult issues now.

Great :-) Will review and merge after I get off work. Can look for something more then as well.
In the meantime feel free to check out this category of issues and see if anything sticks out: https://github.com/MovingBlocks/Terasology/labels/Contributor-friendly

I think #2218 would be a thing that would bring terasology a huge step towards a more user friendly UI.
I think it should be something of your skill level now.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/483/
Hooray Jenkins reported success with all tests good!

@msteiger Changes done. Please review.
.
@flo Thanks. I'll start working on it.
.
Since, this now has merge conflicts I'll rebase after some time.

@msteiger Changes done. Please review.
.
@flo Thanks. I'll start working on it.
.
Since, this now has merge conflicts I'll rebase after some time.

Closing because of merge conflicts.


[6.0] Laravel stable release

Laravel doesn't have an issue section, so I am making one here.
Where is the stable release for laravel, the version that goes with 6.0 of the framework?
Yes I know I can get it from master, But---
This https://github.com/laravel/laravel/releases used to be maintained, and is where I would go to get the latest stable release.
What happened?  If laravel 6.0 framework is out, shouldn't there be a stable release of Laravel as well for that version?
Are you guys just a little behind and have to catch up, or is the stable release section going away?
Last stable is:
v5.8.16
@driesvints driesvints released this on May 7

While the releases are nice, the important bits are the tags https://github.com/laravel/laravel/tags, and the 6.0.x tags are there, so you can download them i guess.

The best place to look is the upgrading guide in the docs, and to just do a diff via the GitHub branch comparison. That'll show you the exact differences between the version of the laravel/laravel app you started with, and the current default.

Has tags replaced the release page.  Usually by now there is an update on release page?
And how did you find the tag page, I goto https://github.com/laravel/laravel and clik releases.
Why would there be a release page and a tag page.
I know the release page is supposed to be stable.  But is the tag page stable?
Thanks

There will probably be releases too. GitHub's UI isn't very hepful. Tags are effectively releases, just with no extra data attached to them in the UI. Tags function as "releases" in the natural language sense of the word.

@GrahamCampbell
From this page https://github.com/laravel/laravel how do you find https://github.com/laravel/laravel
https://github.com/laravel/laravel/tags page

There's 2 tabs

Also you can see the newer tags link right below too.

Page in my browser has php  framework  laravel
Not Realeases   Tags
Why am I missing that?

You just need to go to the releases page, then you can see the "Releases" and "Tags" tabs/subnav.

https://i.imgur.com/XPIdJhs.jpg

There's a releases link there, 96 releases, click on it.

Found it, so those tags are only laravel, not framework?

I'll do the releases today. Last week was a little bit crazy.

Updated the releases: https://github.com/laravel/laravel/releases


PaymentInfo does not exist and could not be loaded

According to the following documentation Order , I try to call method:
$paymentInfo = $payment->getPaymentInfo();

I suppose that $paymentInfo it is instance of PaymentInfo , however I get a weird error:

Warning: class_implements(): Class \Commercetools\Core\Model\Cart\PaymentInfo does not exist and could not be loaded in vendor/commercetools/php-sdk/src/Model/Common/AbstractJsonDeserializeObject.php on line 62

Could you please advise what I'm doing wrong?
Maybe in the commit src/Model/Order/Order.php instead of:
class Order extends Resource
{
	public function fieldDefinitions()
	{
		return [
			'paymentInfo' => [static::TYPE => '\Commercetools\Core\Model\Cart\PaymentInfo']
		];
	}
}

there should be something like the following:
class Order extends Resource
{
	public function fieldDefinitions()
	{
		return [
			'paymentInfo' => [static::TYPE => '\Commercetools\Core\Model\Payment\PaymentInfo']
		];
	}
}

Could you please tell what is your idea on this?

You are right. Just hotfixed it and tagged it as RC6.
Thanks! :+1:


No work to Symfony 2.7

i am use symfony new myproject and download the Bundle
config.xml
liip_theme:
themes: ['theme1', 'theme2', 'theme3']
active_theme: 'theme2'
path_patterns:
app_resource:
- %%app_path%%/themes/%%current_theme%%/%%current_device%%/%%template%%
- %%app_path%%/themes/fallback_theme/%%current_device%%/%%template%%
- %%app_path%%/views/%%current_device%%/%%template%%
routing.xml
liip_theme:
resource<EMAIL_ADDRESS>prefix: /theme
Appkernel.php add line:
new Liip\ThemeBundle\LiipThemeBundle(),
no work.
=========
when disable the code after work

I seem to remember that we had similar issues in the past already ..
/cc @dbu

sorry lukas, i don't remember having this problem.
are you running in app_dev.php or with the prod environment? if in prod, did you clear the prod cache with app/console cache:clear --env=prod? or are you switching themes inside the same request?


Look into using smooth approximations for min, max, heaviside

Functions with discontinuous derivatives can cause problems for adaptive time-stepping algorithms: https://discourse.julialang.org/t/handling-instability-when-solving-ode-problems/9019/5
Need to look into whether this is actually a problem for the solvers in pybamm. I suspect it might be given that linear interpolants have been giving similar issues.
If so, we could add a setting where pybamm automatically provides the smooth form instead of the exact form. e.g. by setting pybamm.settings.smooth_approximation_steepness, which is k in the link above, and setting k to zero could give the exact form.

Using smooth approximations definitely helps in a toy model:
import pybamm
def smooth_heaviside(left, right, k):
    return (1 + pybamm.tanh(k * (right - left))) / 2

def rhs_exact(x):
    return (x <= 1) * x + (x > 1) * (2 * x - 1)

def rhs_smooth(x, k):
    return smooth_heaviside(x, 1, k) * x + smooth_heaviside(1, x, k) * (2 * x - 1)
    # return (v2 - v1) * pybamm.tanh(k * (x - switch)) / 2 + (v1 + v2) / 2

x = pybamm.Variable("x")
y = pybamm.Variable("y")

model_exact = pybamm.BaseModel(name="exact")
model_exact.rhs = {x: rhs_exact(x)}
model_exact.initial_conditions = {x: 0.5}
model_exact.variables = {"x": x, "rhs": rhs_exact(x)}

model_smooth = pybamm.BaseModel(name="smooth")
k = pybamm.InputParameter("k")
model_smooth.rhs = {x: rhs_smooth(x, k)}
model_smooth.initial_conditions = {x: 0.5}
model_smooth.variables = {"x": x, "rhs": rhs_smooth(x, k)}

pybamm.set_logging_level("INFO")
# solver = pybamm.ScikitsDaeSolver()
solver = pybamm.CasadiSolver(extra_options_setup={"print_stats": True})
print("Exact-------------------------")
sols = [solver.solve(model_exact, [0, 2])]
for k in [1, 10, 50]:
    print(f"Smooth, k={k}-------------------------")
    # solver = pybamm.ScikitsDaeSolver()
    solver = pybamm.CasadiSolver(extra_options_setup={"print_stats": True})
    sol = solver.solve(model_smooth, [0, 2], inputs={"k": k})
    sols.append(sol)

pybamm.dynamic_plot(sols, ["x", "rhs"])

Results:
Exact: 4ms
Smooth (independent of k): 1ms
Solutions line up well for k=10,100 (k=1 is too small)
Not super obvious from the stats printed by casadi where the improvement comes from though


Button is ignoring the flex style on Row

react-native, react and native-base version:
"react-native": "0.52.0"
"react": "16.2.0"
"native-base": "^2.3.10"
Actual behaviour and steps to reproduce it
Hi, i put this simple code:
<Content contentContainerStyle={{flex: 1}} style={{padding: 10}}>
          <Grid>
            <Col>
              <Row style={{justifyContent: 'center', alignItems: 'center'}}>
                <Text style={{textAlign: 'center'}}>Here is the TurnActive component</Text>
                </Row>
              <Row style={{justifyContent: 'center', alignItems: 'center'}}>
                  <Button info onPress={this.closeSession}>
                    <Text>CERRAR SESIÓN</Text>
                  </Button>
              </Row>
            </Col>
          </Grid>
        </Content>

And the result is: The 'Here is the TurnActive component' is using the style provided by the Row component using flex (for example: if i put alignItems: 'flex-end' then the text is correctly aligned). But with the button this not work. The button is not adapting to the style provided in the Row component. if i put alignItems: 'flex-end' or 'center' or another one, the button ignores this.
Screenshot of emulator/device

Expected behaviour
The button must to adapt their design by the flex config provided
Can you help me?

@aebmknrl it is due to the default style provided to button. Just override the style of button and set alignSelf to null
<Button info style={{ alignSelf: null }}>
    <Text>CERRAR SESIÓN</Text>
</Button>


Correct handling of RHSonly nodes, and nodeFunctionID assignment

It seems RHSonly nodes should not be assigned nodeFunctionIDs, but still should be handled correctly by getNodeNames(), e.g.

This now works.


IOS app not opening from crop image

My iOS configurations working fine when sharing image from the gallery. but when  trying to share image from edit image page it self application not opens.  However application icon is showing there. Does anyone face this issue? Please help

Facing the same issue. Have you been able to fix please?

@Abu-muhab Still not. Any chance from your side?

@Ruhasi @Abu-muhab I having same issue... no news yet?

@bbig979 No. No one attending

@Ruhasi I discovered that the following works, but with some downside : https://pub.dev/packages/share_handler

Image share is not consistent. It succeeds in some occasion and fails in some occasion ( not sure if this is just a simulator issue )
Sharing while app is closed is not working

Me too facing same issue when I take a screenshot in IOS it opens a crop image view and if i try to share it doesn't open the app.

As anybody found an alternative to this, I'm presently facing the same issue.

Found a solution to this @YashSavsani @Ruhasi @Abu-muhab . You can check this out https://github.com/dretoba91/receive_sharing_intent/blob/master/example/ios/Sharing Extension/ShareViewController.swift

@dretoba91 Have you tried adding function to support sharing contacts from phonebook? i.e. sharing VCards in iOS or VCF in android?

@dretoba91 I tried your code, but when I try to share an image into my app, the share sheet just closes, even if I didn't attempt to crop the image. I'm not sure this code is working properly as I have checked my appGroupId and hostAppBundleIdentifier to be sure its not my error.

@dretoba91 solution code is not working. May I know everything working fine for you?

Everything is working fine for me.
On Sun, Dec 3, 2023 at 7:50 PM Ruhasi @.***> wrote:

@dretoba91 https://github.com/dretoba91 solution code is not working.
May I know everything working fine for you?
—
Reply to this email directly, view it on GitHub
https://github.com/KasemJaffer/receive_sharing_intent/issues/240#issuecomment-1837566266,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APN6TPQI3CC426H7TSCJZLDYHTCVVAVCNFSM6AAAAAAUOKATT6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZXGU3DMMRWGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


chore: License Compliance

Ensure License Compliance Using Apache SkyWalking Eyes
Description:
We need to ensure that our project adheres to proper license compliance. To achieve this, the following tasks should be completed:

Use Apache SkyWalking Eyes to ensure that all source files contain the correct license headers.
Verify that all dependencies have appropriate license permissions.
Configure CI to automatically check for and fix any license compliance issues.

Thank you very much! Will add this to the latest push. Much appreciated.


grace period during which lnmux waits for a new subscriber

close #94

Good point. I was in a hurry, so I didn't take the time. I will create tests for this


meson: use nix*Log functions

They are cleaner than echoCmd.
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

I think whatever meson does it should match stdenv, which is still using echoCmd for its equivalents of what we're doing here.

I would prefer to do this bottom-up, since changing stdenv is arguably more sensitive.


Fire OnTrack before reading first RTP

Prior to this, we would wait for a single RTP packet to figure out the codec which is not to spec.
@Sean-Der, as far as I can tell based on what chrome does and what the spec says, we're suppose to fire off on track, even if the codec may not be set yet. What this means is you need to read at least one RTP packet on your own within OnTrack before getting that info. This may be a breaking change though.

@Sean-Der it seems likely users would be relying on Codec and PayloadType being accessible when OnTrack is called. With this change, you'd need to read first before calling those. I could put the peek code into those two methods instead which would preserve the old behavior. What do you think?

Good change to make the ontrack behavior consistent with the browser! It is useful when client want to publish a muted track. Agree it is a breaking change that make Codec() and PayloadType() would block until the first packet arrives.

So if we make it blocking in v3 until the first packet on those two methods, I don't think it's breaking so long as we handle the other cases where the values are already set outside of the peeking process. For v4 I think this can go in as is. What do you think?

Yep that's fair. I'm looking at something else I've written for a client as well and it's too subtle to not cause problems. Let's just do this for v4

@cnderrauber tagged you for review for v4 inclusion only

Looks good for normal track. Another case is the simulcast track still relies on rtp packet to fire the  event, we need consistent behavior for all tracks to avoid confusing developers.

Looks good for normal track. Another case is the simulcast track still relies on rtp packet to fire the event, we need consistent behavior for all tracks to avoid confusing developers.

The simulcast code is still intercepting packets so I think we're fine there https://github.com/pion/webrtc/blob/master/peerconnection.go#L1608-L1626

Oh, I see. You're saying it should also fire off the track as well. Let me take a look.

It seems like the probing and what's populated into the track details before on track for simulcast is pretty tightly coupled right now. It's going to take me a bit to wrap my head around the right way to fire off the events as quickly as possible while avoiding invalid state. A naive first stab of setting up the on tracks all in the same place resulted in panics due to not having the right receiver available for RTCP.

This is no longer a priority for the client I'm doing this work for, so I'll close this for now. Anyone is free to pick this work up though!


Timepicker and datepicker in v1?

Versions

Material-UI: v1

https://github.com/callemall/material-ui/issues/4787 and https://github.com/callemall/material-ui/issues/4796

alright thanks, I'll try porting/redesigning when I need it. I am just stuck between choosing 0.18.6 or v1 alpha 👍

@state0fmind You can use both


Release

Make a release for PKGBUILD url
:package:

And make a setup.py for pip => QP2 !
:pig:

I guess we chould use the jaimergp fork
https://github.com/jaimergp/ebsel

Yes!
:fire_engine:


extract features

Hi！How to use I3D to extract features from other datasets? If n represents the frames of a video, How can I extract features of size n*1024 with rgb_videos as the input?

@chenjiakun Hello, did you work on this further and reached a solution?

@Tortoise17 how to make this code to extract features from other datasets rather than kinetic?


Change tsconfig target to ES6

Before: Our TypeScript configuration targeted "esnext", which means the output JavaScript could use any of the latest JavaScript features.  The rational was: any developer project will use a bundler, which will transpile the JavaScript into whatever version they want to ship.
Unfortunately, the default configuration for create-react-app does not support this philosophy :(
After: Our TypeScript configuration targets "es6", a much older and more conservative target.
For example, we use "optional chaining":
let name = patient?.name?.[0]?.given?.[0];

That feature was added in ES2020.  To be used by create-react-app, it needs to be transpiled into ES6:
let name = patient && patient.name && patient.name.length && patient.name[0].given && patient.name[0].given[0];

create-react-app throws errors if a dependency uses ES2020 features.

Don't know why code coverage would change 🤔


Marketplace fields not showing in accounts screen when VUE_APP_MARKETPLACE_INTEGRATION is set to true in cloud run vars

Describe the bug
Marketplace fields not showing in accounts screen when VUE_APP_MARKETPLACE_INTEGRATION is set to true in cloud run vars
To Reproduce
Steps to reproduce the behavior:

Set the VUE_APP_MARKETPLACE_INTEGRATION variable to "true" in cloud run.

Expected behavior
Marketplace related fields should show in the accounts table UI

The entrypoint.sh script replaces the config.json placeholders on start up. It's currently performing the replace and for cloud run it's replacing the value with
"true"

rather than
true


fix(@angular-devkit/build-angular): switch to license-checker-webpack-plugin

Fixes #16193, #14095.
This replaces the existing license-webpack-plugin with the new license-checker-webpack-plugin. The former would output multiple 3rdpartylicenses.txt files which would trigger warnings in Webpack. The later does not emit this warning and still appears to log all third party licenses in a reasonable fashion.
The output format is changed, see https://www.diffchecker.com/puCPV7LQ for a more complete before-and-after comparison of a simple example Angular app (with jQuery and Bootstrap installed).
It is worth discussing if we want to go down this route or if we should stick with license-webpack-plugin and attempt to fix it rather than changing the entire dependency.

Out of curiosity can you check if it fixes also https://github.com/angular/angular-cli/issues/14095

@alan-agius4, I tried out on their repro case and I do see open-iconic in the output. I see it as:
--------------------------------------------------------------------------------
open-iconic v1.1.1 - Iconic
git+https://github.com/iconic/open-iconic.git
--------------------------------------------------------------------------------

(MIT License OR SIL OFL 1.1)

The user said they expected to see that repo's FONT-LICENSE and ICON-LICENSE. I don't think those particular files are picked up, as this seems to come from the package.json. I think that would be expected as those two *-LICENSE names are non-standard. If so, we can probably close that issue as well with this PR.

Thanks @dgp1130 for checking the ionic issue. Will update the issue as suggested.

Discussed this PR, and we'd rather not change a dependency like this so close to the v9 release.
Decision is to instead suppress the warnings for v9, and then for v9.x we can re-evaluate if we want to try and fix the existing dependency or change to this one.

Now is probably a good time to make this happen, I'll revisit and update this PR.

Rebased the PR on master and double checked that it still appears to be working as expected. I also added a second commit reverting the log suppression code that is no longer necessary.
I believe this can merge to patch as I don't think we make guarantees about the format of the 3rdpartylicenses.txt file. Let me know if this should be considered a breaking change and merged only to master for v10.

The revert commit message should be reworded to
revert: "fix(@angular-devkit/build-angular): suppress duplicate 3rdpartylicenses.txt warning

I think since we are changing the dependency and the output, it's safer to target only master.

Fixed the commit message, rebased to master again, and updated the labels to target master only.

I also rebased onto master.


Working again in March 2024

These changes fix the flaws below making it work on 03/04/2024

Fixed  - https://github.com/JanLoebel/face_recognition/issues/12
Fixed  - https://github.com/JanLoebel/face_recognition/issues/11

Thank you for your work!


BLUI Does not work On IOS

Hello, getname, i use blui plugin, can work in windows ,when i package to ios ,it does not work. why?

and Can i get All files under the plugins/BLUI/ThirdParty/cef/Mac/Shipping ?

iOS likely has a pretty complicated setup for plugins with dependencies. I only prebuild and maintain the windows platform, but you can get CEF library builds here: http://opensource.spotify.com/cefbuilds/index.html
As you can see neither iOS nor Android are built by default, you'd need to compile from source (https://bitbucket.org/chromiumembedded/cef/src)

Hello, sorry for bumping so old issue, just want to share info. I was thinking to add BLUI into my project, but as it uses CEF, on iOS it will not work, because Apple allows using only their UIWebView(Safari). Chrome on iOS is just "Safari with another UI":) Anyway, thanks for your work!

Recommended to use https://docs.unrealengine.com/5.1/en-US/API/Runtime/WebBrowser/ for iOS/Android platform support


Made Toggle Buttons on customize addon synchronous

Signed-off-by: hs05june<EMAIL_ADDRESS>Notes for Reviewers
This PR fixes #6924
https://user-images.githubusercontent.com/103871275/213940810-fe053a2b-4af3-456a-873d-887f68c56e95.mp4
Signed commits

[x] Yes, I signed my commits.

Hey @leecalcote Please review the PR.


File and class names should reflect actual service name: Analytics, not Analytic

The full name of the service whose API we're covering is IBM Analytics Engine powered by Apache Spark, or Analytics Engine for short. All artifacts in our repo, from variable, function/method, and class names to file names, should reflect that (unless abbreviated to AE or ae).
Specifically,

Rename the file analytic_engine_client.py to analytics_engine_client.py
Rename the class AnalyticEngineClient to  AnalyticsEngineClient
Resolve any breakage where these names are referenced

I have fixed this and the code is now in the master branch.
class renamed from AnalyticEngineClient to AnalyticsEngineClient
file renamed from analytic_engine_client.py to analytics_engine_client.py
Also, I have made changes in the test scripts where we import the module.


Support for CO Detector

Addresses #33

Damn you're fast! Thanks @nikonratm !!!
I'll update the documentation and push it out.


Green tests in phantom 2.

They pass in the browser, but phantom 2 cannot handle syncronous ajax requests (we use synchronous requests to load json fixtures in our tests).
I guess we have to wait for phantom 2.2 which supposedly solves this issue

Or you change to chrome see travis.yml for the configuration.

Turns out, it had nothing to do with Phantom. Chrome failed locally as well.


[Fabric] Shim RCTTextInput

Please select one of the following

[x] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
Fabric on macOS implementation:
Shim RCTTextInput to work w/ Fabric
closes #1547
Changelog
[macOS][Added] - Shim RCTTextInput for Fabric
Test Plan
[x] Build RNTester-macOS w/ Fabric - doesn’t run yet, but no RCTTextInput errors

[x] Build RNTester - iOS w/ Fabric

[x] Build RNTester-macOS w/ Paper - should work

[x] Build RNTester - iOS w/ Paper -

Note: This might change a bit with 0.71 because 0.71 introduced this commit (facebook#33653) that conflicts with some of the macOS specific changes we made to TextInput (#1423). So this PR might be worth holding off till after 0.71.

Once we have Fabric building, we plan to revisit TextInput in depth & use the latest Fabric changes.


Load function modules into a reserved namespace package

Function modules are now loaded into the special __azure__ namespace
package to prevent any possible name collisions with non-function
modules.  Additionally, this makes it possible to avoid polluting
sys.modules too much.
The loaded module structure is as follows:
azure.
func1.
func1mod
func2.
func1mod
...
Closes: #26

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


feat: added systemd and files module

What's currently done:

added a systemd module
added a files module
Containerfile adjustments

What needs to be done:

Documentation
Debugging/testing

Everything is done, ready to merge after review

Overall looks good. More testing will be done when this is merged into the super-pr. These comments are things I could fix, but unfortunately can't today. If you want to quicken the merge, you can implement the suggestions yourself. Or, if you object, object.

Everything should be resolved, thanks!


Option to have a shortcode stripped version

There should be option to have text without any shortcode in it. Wordpress has function for that.
http://codex.wordpress.org/Function_Reference/strip_shortcodes

I assume this would be for things like excerpts of content when the shortcode content is not needed. I'll add it to the next release.

I was working on it today and I'm almost done. Buuuuuuuuuuuut, I just came out with that problem:
How do we recognize the content that should be completely stripped (like a photo gallery) and content that should stay (like the "[bold]This is bold text and is an important part of the sentence[/bold] and this part does not make sense without the bold part") ?
I have a solution: by default, everything is stripped. If something should not be (like bold text or images) we could have the possibility to provide a "stripped" template that could be rendered (with a naming convention). That way, people can chose to strip the entire shortcode, keep only the content or one attribute to render or render everything the same way.
Another solution would be to add some config options to chose the default behavior and some exceptions. It can be used with the "stripped templates" and become very powerful together.

I would expect this feature to simply strip the shortcode tags out and nothing more. So Some [bold]bold text[/bold]. would become Some bold text.. Essentially anything inside square brackets is removed including the square brackets with no regard for start or end tags.


More docs on how to get started

I've tried out VS Code and Atom a few times. It's early days yet but I'm wondering if a bit more documentation would help me know what to expect and what to do.  For example::

How do I open a project? Is opening the directory containing the project correct? Or the solution?  I'm just guessing right now by opening either the folder containing the project or solution somewhat randomly.

Can I edit F# standalone scripts and get strong editing support?  Should I just open the script or do I need to open a project containing the script?

Should I expect autocomplete to work in VS Code and what keybindings do I use?  (It didn't seem to work for me and I couldn't spot autocomplete commands in the menus)

How to send code to F# Interactive and should it work?  IN VS Code I tried Alt-Enter on a single "open" line and got this:

In each case I'm either not quite sure how to do something, or if it's something I haven't installed yet, or a bug.   It would help greatly if the docs showed animated gifs of each thing that should/ahouldn't work and listed the default keybindings.
Thanks!
Don

I agree that documentation is something we need to work on. And it's definitely on my todo list. For last few weeks I was rather busy with developing plugins for VS Code, but now (as we already have similar set of features for Atom and VS Code) I'm going to focus more on documentation, some getting started tutorials etc.
And to answer those questions quickly:

How do I open a project? Is opening the directory containing the project correct? Or the solution? I'm just guessing right now by opening either the folder containing the project or solution somewhat randomly.

In most cases opening solution folder should work, when user opens F# file plugin looks for fsproj file in same folder F# file is in or in ancestors folder. If fsproj file was not found, parsing it can be forced by opening fsproj file in editor.

Can I edit F# standalone scripts and get strong editing support? Should I just open the script or do I need to open a project containing the script?

Editing standalone F# script file should work. That's actually how i do most of my demos during talks.

Should I expect autocomplete to work in VS Code and what keybindings do I use? (It didn't seem to work for me and I couldn't spot autocomplete commands in the menus)

Yes it definitely should work (and it should be invoked on every key press like in VS). If there is any problem with that, please create specific issue on ionide-vscode-fsharp repo.

How to send code to F# Interactive and should it work? IN VS Code I tried Alt-Enter on a single "open" line and got this

Fsi needs to be in PATH to get interactive working.

I'm also lost with this. I get VS Code, install Ionide+Ionide-Fake, then do build and get a error:
No handler found for the command: 'fake.buildDefault'. Ensure there is an activation event defined, if you are an extension.
Then try with the .ionide file and set:
[Fake]
linuxPrefix = "mono"
command = "build.cmd"
build = "Program.fs"
And nothing.
Is necesary to detail how setup this so it build.

I'm also lost with this the current documentation states:

Since version 1.1.0, ionide-fake allows the user to override the default conventions used to find and run FAKE builds. To do so You need to create an .ionide file in the root folder of Your project opened by Atom. The configuration file uses the TOML language.
Here is the default configuration values used if the .ionide file doesn't exist or some entry is missing:
[Fake]
linuxPrefix = "mono"
command = "build.cmd"
build = "build.fsx"
Linux Prefix - command used as prefix on Linux / Mac - usually sh or mono
Command - command executed as build taking build name as parameter - usually build.cmd, build.sh, build.ps1
Build - FAKE build script, which is parsed to obtain list of possible builds - usually build.fsx, fake.fsx

I'm confused whether that default configuration applies to a Non Windows/Linux platform, what if I am using Windows Platform? The linuxPrefix doesn't seem to make sense as to why it would use Mono or why would it even be there for default? If I override it, do I remove linuxPrefx or do I add something else?
Trying out Ionide with Atom right now and I gave up and just went back to VS as I have been spending too much time figuring out just to scaffold and build a simple Suave project.


Change queueingStrategy default to BestEffortFIFO

/lgtm
/hold

/hold cancel

Here's one more thing to do.
Listed here
https://github.com/kubernetes-sigs/kueue/pull/135#discussion_r840729523_

@denkensk are you planning on sending a fix?

@ahg-g  I will send the PR tomorrow(It's about 11pm in my city).
When will we cut down the release?

Tomorrow sounds great! Thanks :)

When will we cut down the release?

We don't have deadlines at this moment, but I think we should try and cut one asap so that people who want to try it don't need to clone the repo and build from head.

I couldn't agree with you more. We need to have a version as soon as possible. And then it will roll up like a snowball.
from 0.0.1 --> 0.1.0 --> 0.2.0


Whoa (alpha0.5 history rewrite)

Fixes #99

Ok this actually removes some files we need. Going to do a safer rewrite now

Alright, the files removed were as follows:
Deleted files
-------------

	Filename      Git id
	---------------------------------------------------------
	WormData.n3 | b1e582d7 (10.2 MB), cb5032c2 (10.3 MB), ...
	__db.002    | 34d9888d (3.2 MB), b85bbeb9 (3.2 MB), ...
	__db.003    | 72b68568 (62.5 MB), 997b9d1a (58.0 MB), ...
	c^o^s^p^    | 343fcf4c (29.5 MB), f7aade06 (12.7 MB), ...
	c^p^o^s^    | 0d250b42 (29.7 MB), 373460ba (15.3 MB), ...
	c^s^p^o^    | 17e87401 (13.2 MB), a0818aa1 (13.2 MB), ...
	export.trig | 7a5ac377 (10.1 MB)
	export.xml  | fb3cd31e (39.0 MB)
	i2k         | ee6bd6b2 (6.5 MB), 8baf5f39 (2.9 MB), ...
	k2i         | f7e8687b (5.2 MB), 88f64d6d (10.1 MB), ...
	out.n3      | 495fe302 (12.6 MB)
	out.n4      | a4571104 (1.5 MB)

This doesn't interfere with the current version of PyOpenWorm alpha0.5

Awesome that we've done this.  Looking at the changelog though (84 files changed) -- how do I assure myself that this doesn't change anything when github seems to suggest it changes a lot?! :)  I mean I can always revert it, but any way to reassure myself before I merge?

Right. The reason this is happening is because all of the commits and blobs now have different SHA-1 tags which are unknown to github. If you look at my fork it says "648 commits ahead, 647 commits behind openworm: alpha0.5" (one more because I've made one bugfix commit since).

@slarson
Just to close the loop on verifying the history rewrite did not change file contents at the current commit, I cloned both the pre- and post-rewrite versions locally and did a gnu diff:
diff -brq \
  --exclude=".git" \
  --exclude=".pyc" \
  PyOpenWorm_post/  PyOpenWorm_pre/

The result only showed binary data files (and WormData.n3, which is generated from those files) as differing:
Files PyOpenWorm/OpenWormData/WormData.n3 and PyOpenWorm_ow/OpenWormData/WormData.n3 differ
Files PyOpenWorm/OpenWormData/scripts/worm.db and PyOpenWorm_ow/OpenWormData/scripts/worm.db differ
Files PyOpenWorm/OpenWormData/scripts/worm.db.index and PyOpenWorm_ow/OpenWormData/scripts/worm.db.index differ
Files PyOpenWorm/OpenWormData/scripts/worm.db.lock and PyOpenWorm_ow/OpenWormData/scripts/worm.db.lock differ
Files PyOpenWorm/OpenWormData/scripts/worm.db.tmp and PyOpenWorm_ow/OpenWormData/scripts/worm.db.tmp differ
Files PyOpenWorm/dist/PyOpenWorm-0.5.0a0-py2.7.egg and PyOpenWorm_ow/dist/PyOpenWorm-0.5.0a0-py2.7.egg differ

Combined with passing tests, it should be it is safe to say that the contents of the repos is functionally identical at this point.


[Bug] BD can't open Finder on Mac

Describe the Bug
When you try to open the plugins folder on Mac, it says "The application Finder can't be opened"
To Reproduce
use BD on a mac, and try to open the plugins folder
Expected Behavior
Open the BD plugins folder in finder
Screenshots/Video
Video Here: https://user-images.githubusercontent.com/77711834/125179158-ec4a9c00-e1a8-11eb-9de0-0333e4b90c05.mov
Discord Version
Stable

Heres the file path until its fixed ~/library/application support/betterdiscord/

This is known, but currently works on Discord Canary.

Reopening as this bug is still technically valid on Stable

I don't even have a better discord folder for some reason, any suggestions?

This has the same root issue as #889

hello I have precisely that same problem, which tells me that I cannot open the finder to be able to enter new plugins, and I searched for the path manually and I did not even get the betterdiscord folder.
If anyone knows how to solve this error I would appreciate it!

hello I have precisely that same problem, which tells me that I cannot open the finder to be able to enter new plugins, and I searched for the path manually and I did not even get the betterdiscord folder.
If anyone knows how to solve this error I would appreciate it!

You need to change some details on the path that @doggybootsy provided:
~/Library/Application\ Support/BetterDiscord
that way you can open the folder on Finder using the open command.

I'm also experiencing this issue.

@Macintosh-Fan in the future please just give a 👍 to the original issue if you are experiencing the same issue unless you have further insight/information to provide.

@Macintosh-Fan in the future please just give a 👍 to the original issue if you are experiencing the same issue unless you have further insight/information to provide.

There was no photo evidence before, so that's why I sent it.

There's literally a video linked in the original issue

hello I have precisely that same problem, which tells me that I cannot open the finder to be able to enter new plugins, and I searched for the path manually and I did not even get the betterdiscord folder.
If anyone knows how to solve this error I would appreciate it!

You need to change some details on the path that @doggybootsy provided:
~/Library/Application\ Support/BetterDiscord
that way you can open the folder on Finder using the open command.

hello, how do i make the button to open in the correct location?

Same here


scroll mouse zoom

Hi, I'm very new to d3, so long i've just studied a little this example I need
http://mbostock.github.io/d3/talk/20111116/pack-hierarchy.html
I'm wondering how to accomplish 2 feature:

zoom action on mouse scoll, not on node click.
keyboard event for multiple node selection (in the example, pressing ctrl key and clicking on nodes "scale" and "display" I want to show some infos related to both nodes in a div)

this question should be asked on stackoverflow, tagged with d3.js. the issues on github are for bug reports and feature requests.


example programs currently unrun

There is a program examples/C/format.c which is currently not being run. However, it can be. I will fix and put up in a PR.
This is also true for quick_large_files.c, quick_small_files.c, sfc_pres_temp_more.c, simple.c.

This was fixed by a recent PR.


Clarification in Algorithm 1

Hi - wonderful results. Could you please give more details of what r_{t'}(x) is in Algorithm 1. If it is the denoised version of x_{t'} , which diffusion model and which method (DDIM etc.) is used to generate r_{t'}(x)?

Yes, it is the denoised version of x_{t'}, and we use SD 2.1 with DDIM.


Show Puzzle: Not in Progress

Show Puzzle: Not in Progress

[ ] If no puzzle is in progress, display message saying there is no puzzle in progress

[ ] If no game is in progress, display message saying there is no puzzle in progress

Primary: Tess
Secondary: Devon
Size: S
Actual Size: (e.g. t-shirt size after completion)
Actual Time: (in minutes or hours -- after completion)

Assigned devon as secondary to this since hes working on show puzzle anyways


Incorrect exercise: [courseId=grammar-intro_eng][lessonNumber=5][exerciseId=1770]

Description
If possible, please provide a brief description of what is wrong with this exercise.
I do not understand why the extra .i in the beginning makes me wrong.
Diagnostics information (do not remove)

Exercise identifier: [courseId=grammar-intro_eng][lessonNumber=5][exerciseId=1770]
Exercise payload: {"shouldDisplayHint":false,"exercise":{"canonicalAnswer":"xu lo prenu ku ciska","sentences":[{"text":"Do people write?","lojbanic":false}],"title":"Translate this sentence","type":"typing"}}
Attempted solution: {"text":".i xu lo prenu ku ciska "}
Attempted solution was classified as correct: no
Validation payload: {"canonicalAnswer":"xu lo prenu ku ciska","correct":false}

My bad, duplicate of #161 and #166.


[report]Command "vcpkg x-history [package]" cannot execute.

Describe the bug
The command "vcpkg x-history [package]" cannot execute.
Environment

OS: [Windows 10/ ubuntu 16]
Compiler: VS 2017, gcc/g++ 8.4

To Reproduce
Steps to reproduce the behavior:

./vcpkg x-history [any package]
See error

Expected behavior
Shows the full history of CONTROL version of a port, including the vcpkg commit hash, CONTROL version, and date of vcpkg commit.
Failure logs
Windows10: run "./vcpkg x-history vtk"
"Failed to run command: "D:\vcpkg-master\downloads\tools\git-2.26.2-1-windows\mingw32\bin\git.exe" --git-dir="D:\vcpkg-master/.git" log --format="%H %cd" --date=short --left-only -- ports/vtk/."
Ubuntu: run "./vcpkg x-history qt5"
"Failed to run command: "/usr/bin/git" --git-dir="/home/super/Downloads/vcpkg-master/.git" log --format="%H %cd" --date=short --left-only -- ports/qt5/."
Additional context

The git program runs OK.
I have tried to use administrator rights. The command cannot be executed.

Hi @artzers, thanks for reporting this issue!
Did you use powershell? I can't repro this issue on Windows 10 or Ubuntu now.
Can you run D:\vcpkg-master\downloads\tools\git-2.26.2-1-windows\mingw32\bin\git.exe" --git-dir="D:\vcpkg-master.git" log --format="%H %cd" --date=short --left-only -- ports/vtk/. manually and provide me the results?
Thanks.

@JackBoosY  First I use CMD to run the command:  D:\vcpkg-master\downloads\tools\git-2.26.2-1-windows\mingw32\bin\git.exe --git-dir="D:\vcpkg-master\.git" log --format="%H %cd" --date=short --left-only -- ports/qt5/
The result is : fatal: not a git repository: 'D:\vcpkg-master\.git'
Then I use powershell to run the command. The result is : fatal: not a git repository: 'D:\vcpkg-master\.git'

@artzers Can you try to re-clone a new vcpkg and retry again?

@JackBoosY  I have downloaded the vcpkg zip from github, decompressed and built again. The same error still occured. The default git source is PortableGit-2.26.2-32-bit.7z.exe.
D:\vcpkg>vcpkg x-history vtk
A suitable version of git was not found (required v2.26.2). Downloading portable git v2.26.2...
Extracting git...
Failed to run command: "D:\vcpkg\downloads\tools\git-2.26.2-1-windows\mingw32\bin\git.exe" --git-dir="D:\vcpkg.git" log --format="%H %cd" --date=short --left-only -- ports/vtk/.
D:\vcpkg>"D:\vcpkg\downloads\tools\git-2.26.2-1-windows\mingw32\bin\git.exe" --git-dir="D:\vcpkg.git" log --format="%H %cd" --date=short --left-only -- ports/vtk/
fatal: not a git repository: 'D:\vcpkg.git'

@artzers You need to clone vcpkg using command git clone https://github.com/microsoft/vcpkg.git instead of download zip package.

@JackBoosY   Thanks 👍 . I git clone vcpkg, build and run x-history successfully.
Commands "vcpkg.exe x-history vtk" and "vcpkg.exe x-history qt5-base" run correctly.
version          date    vcpkg commit
9.0-4    2020-07-02    2ad4074aea2415e962153544ca942942e7d52323
9.0-3    2020-06-23    03766d0834c1ca8dea25ffafbe4fe1c60786a412
.........
However, I met error when running command "vcpkg.exe x-history qt5"
Failed to run command: "D:\vcpkg-test\vcpkg\downloads\tools\git-2.26.2-1-windows\mingw32\bin\git.exe" --git-dir="D:\vcpkg-test\vcpkg.git" show 890a81bebc3642454e3833c789a35ba9a7989c9d:ports/qt5/CONTROL
I donot think it is a bug, but may confuse many beginners.