Test for Windows and Python 3.8

I'm seeing the same build issues on Windows 10 with Python 3.8 as your CI tests. It might be related to the BOOST library. I was able to build with Python 3.7 after installing/building boost and setting the %BOOST_ROOT% env variable. Unfortunately the same did not work for Python 3.8.
Python 3.8.2 (tags/v3.8.2:7b3ab59, Feb 25 2020, 23:03:10) [MSC v.1916 64 bit (AMD64)]
Type 'copyright', 'credits' or 'license' for more information
IPython 7.13.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import turbodbc
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-6cf0b4450009> in <module>
----> 1 import turbodbc

d:\venv\attd_py38\lib\site-packages\turbodbc\__init__.py in <module>
      2
      3 from .api_constants import apilevel, threadsafety, paramstyle
----> 4 from .connect import connect
      5 from .constructors import Date, Time, Timestamp
      6 from .exceptions import Error, InterfaceError, DatabaseError, ParameterError

d:\venv\attd_py38\lib\site-packages\turbodbc\connect.py in <module>
      4 import six
      5
----> 6 from turbodbc_intern import connect as intern_connect
      7
      8 from .exceptions import translate_exceptions, ParameterError

ImportError: DLL load failed while importing turbodbc_intern: A dynamic link library (DLL) initialization routine failed.

I also saw this in my build log not sure if this matters;
D:\VENV\PY38\lib\site-packages\wheel\pep425tags.py:80: RuntimeWarning: Config variable 'Py_DEBUG' is unset, Python ABI tag may be incorrect
  if get_flag('Py_DEBUG',

For what it's worth, I too am having difficulty getting Python 3.8 to play nice with turbodbc on Windows 10.  Using the VS2019 compiler with Boost 1.72, turbodbc appears to pip install successfully, but then I get the same ImportError error when I import turbodbc:
Python 3.8.2 (tags/v3.8.2:7b3ab59, Feb 25 2020, 23:03:10) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import turbodbc
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\keith\Downloads\pyvenvs\py38\lib\site-packages\turbodbc\__init__.py", line 4, in <module>
    from .connect import connect
  File "C:\Users\keith\Downloads\pyvenvs\py38\lib\site-packages\turbodbc\connect.py", line 6, in <module>
    from turbodbc_intern import connect as intern_connect
ImportError: DLL load failed while importing turbodbc_intern: A dynamic link library (DLL) initialization routine failed.
>>>

The only significant difference I can think of between Python 3.7 and 3.8 is that DLLs are no longer searched for in the PATH (see here for more info), but I'm not sure if that's even a factor here.
Then again, perhaps a newer of pybind11 is needed.  There have been a couple of apparently minor fixes for Python 3.8 recently (see here), but maybe those fixes are relevant in this case.  Difficult to see how they would be specific to Windows though.

Hey everyone. I'm running into this same problem. I installed Boost, set the BOOST_ROOT, was able to successfully pip install turbodbc in my venv, but when I try to import turbodbc or "from turbodbc import connect" I receive the same error (below):

(.venv) C:\path\to\repository>pip install -U turbodbc
Collecting turbodbc
Using cached turbodbc-4.0.0.tar.gz (77 kB)
Requirement already satisfied, skipping upgrade: pybind11>=2.2.0 in c:\path\to.venv\lib\site-packages (from turbodbc) (2.5.0)
Using legacy setup.py install for turbodbc, since package 'wheel' is not installed.
Installing collected packages: turbodbc
Running setup.py install for turbodbc ... done
Successfully installed turbodbc-4.0.0`

(.venv) C:\path\to\repository>python
Python 3.8.0 (tags/v3.8.0:fa919fd, Oct 14 2019, 19:37:50) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import turbodbc
Traceback (most recent call last):
File "", line 1, in
File "C:\path\to.venv\lib\site-packages\turbodbc_init_.py", line 2, in
from .connect import connect
File "C:\path\to.venv\lib\site-packages\turbodbc\connect.py", line 1, in
from turbodbc_intern import connect as intern_connect
ImportError: DLL load failed while importing turbodbc_intern: A dynamic link library (DLL) initialization routine failed.

I am using Python 3.8.0. Anyone have a workaround / fix?

I am using Python 3.8.0. Anyone have a workaround / fix?

No, otherwise this PR wouldn't still be open.

I'm pretty certain now this issue is due to the change in behavior for Python 3.8.  On Windows, there are environment variables for each user and "system" environment variables for everybody.  Before Python 3.8, both system and user PATHs were examined to find DLLs for extension modules.  Now in Python 3.8, only the system PATH is examined.  However, if you have Python installed only for you (as opposed to for everybody on your PC), then the Python library paths are in your user environment variables, not the system environment variables.
Python itself has a DLL that is located in the top-level installation directory:
C:\Program Files\PythonXX\pythonXX.dll                                <-- everybody
C:\Users\<user>\AppData\Local\Programs\Python\PythonXX\pythonXX.dll   <-- user only

Hence, if an extension module needs the Python DLL, in Python 3.8, it will not be found in user-only Python installations.  This appears to be what's happening on my PC.  For some reason, turbodbc needs pythonXX.dll, yet after pip install turbdobc, the compiled pyd file in Lib\site-packages\ does not have access to a working copy:

(I used Dependencies to get this)
In Python 3.7, the pyd file has the Python DLL path found in my user environment variables:

(Yes, that's an Amazon Python DLL, which is odd, but kind of a separate issue).
Unfortunately, it doesn't seem to be possible to install a local turbodbc repo on my PC with python setup.py install, so it's difficult to debug this and come up with a solution.  It looks like a call to os.add_dll_directory() may have to be added somewhere, probably in setup.py, or perhaps the make files will have to be updated.  For example, the following call should include the Python installation directory, even within venvs:
os.add_dll_directory(sys.base_prefix)  # note, not sys.prefix

Thanks @keitherskine for looking into this so detailed. We should not be linking at all with python38.dll as far as my understanding goes (you should only link against it if you need the interpreter which we don't). I can look at fixing the build accordingly on Monday.

The linkage against python38.dll is not actually the problem. There must be something specific about turbodbc_intern.cp38-win_amd64.pyd as importing turnodbc_numpy_support works for me.

Hi everyone, just checking, is there any update on this issue? Would love to use with python 3.8.

@xhochy If I want to add tests for #316 as request by @fjetter testing the module on Windows and Python >= 3.8, should I just merge this pull request into mine? It looks to me like the CI infrastructure has changed since this pull request, but I am unfamiliar with it.


Feature/0266 new vehicle data gear status

Implements #3216
Summary
Current PR contains an update for MOBILE_API.xml and README.md introducing new vehicle data type gearStatus

@atiwari9 please review this PR

PR Approved.
Validated with atf scripts commit: b4e023d
Logs:Test_Report_Gear_Status_b4e023d_2.zip

@theresalech PR is ready for Livio review. Thank you!

@LitvinenkoIra, thanks!  Moving forward, is it possible for Luxoft to reference the issues for the specific repos in the PR description?  For example, this PR resolves issue #217.

@theresalech Yes, I will check all of the PR's and will link them.

@ShobhitAd All comments are processed, please check the latest updates


iVentoy start failed: DHCP port is occupied by another program now!

How do I solve it?
Set to External

@brkr1 could you provide some context? Such as docker ps -a, whether the container is running, etc.


allow specification of whether or not to update existing documents in destination

We had a case in which we wanted to point the consumer to a new index for writing, and then copy the old messages into the new index.
Allowing specification of the update property allows us to fill-in-the-gaps, without worrying about overwriting new data with old data.

Good idea!
Please add tests which confirm the behavior you've added here (and the old behavior as well).

You have also broken the existing test suite: https://travis-ci.org/taskrabbit/elasticsearch-dump/builds/78822756

I can't get the tests to run on my computer. Using it from the terminal appears to work, I can't imagine how I broke the tests, I shouldn't have changed existing behavior. Are you sure my patch is to blame?

Hmm, rebased it against your master branch, and it failed again.

Okay, tests failures were a false negative; they don't inherit the default values from bin/elasticdump. Tests should be passing again.

@evantahler - the tests are fixed, and the new test is added.
I wrote the new test using standard callbacks, and it read absolutely terrible. So I brought in the ES6 Promise polyfill for dev mode. I hope you don't mind.

one note, then looks good!
Promises are OK in tests... I guess :D

That's a good point, Evan. No problem.

ES6 Promises work pleasantly well in mocha.
I don't know about you kids and your "readable code is for sissies" manifesto... :P

Can you rebase on master please?

Closing due to lack of response.


184 custom events

Fixes #184

@mtabacman  aparently WebInteractionInterpreterTest >> configureScriptWith: interpreter has lost its coverage with the latest changes. Can you take a look?

@mtabacman  aparently WebInteractionInterpreterTest >> configureScriptWith: interpreter has lost its coverage with the latest changes. Can you take a look?


How to hide fullscreen button?

I noticed on the screenshots in the readme there is no fullscreen button.
How can I replicate this as I have a fullscreen button.
Is it a config setting?

@LarveyOfficial i have full screen button too on my Spotify
But I installed it using spicetify marketplace
I haven't included that on screenshot because its not a part of pure theme
You can install full screen option via marketplace and even delete it via marketplace
Its not a part of theme

@LarveyOfficial Did my explanation clear?
Let me know
Then close the issue
If my answer is awkward don't hesitate to reach me out

Ah I understand, I actually want to use a custom fullscreen button I found on the marketplace, how can I remove the default spotify one?

@LarveyOfficial Ill tell you if i figure out a way

@LarveyOfficial Did you find a way to hide the button

I have not

Not nice

I was able to figure out how to hide it,
You need to first install the CSS Injector Extension from the Spicetify Marketplace
Then you need to make a new Snipper which includes the follow CSS code
button[title="Full screen"]{ display:none; }

If you are done plz close the issue


/v3/marketing/contacts/import CSV body field

Issue Summary
There is no documentation for how to upload a CSV file to the /v3/marketing/contacts/imports endpoint.
Steps to Reproduce
await request({
  method: "PUT",
  url: "/v3/marketing/contacts/imports",
  body: {
    list_ids: ["example-list-id"],
    file_type: "csv",
  },
})

The response is 200, but I don't know how to upload a CSV file via this API.
Technical details:

sendgrid-nodejs version: 7.4.3
node version: 16.13.0

Hi @Enijar! Please file a ticket with SendGrid customer support in order to properly debug this issue.


[20.03] dmidecode version bump to 3.2.8

(cherry picked from commit 0cd61da5f265b3e7e40baff86fad7e9d9cc3bc06)

Motivation for this change
PR#97614 was missing dmidecode's version bump.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Converted to draft until #98171 is ready.


Flexbug8 - flex-basis doesn't support calc()

https://github.com/philipwalton/flexbugs/blob/master/README.md#8-flex-basis-doesnt-support-calc

Do you want to have support of calc for 8.2 with variables? And if yes, how can it be done?
Another option here, as I can see, is to flex-basis's value to height property.

Yeah. That'll be great.

Excuse me, we're going to set flex-basis value to height property and flex-basis: auto?

And another question.
In order to set 'flex-size' property, we need to know flex-direction at first. Can't imagine a way to do this right now, because it's connected with the markup — we need to know flex-container style, and it seems to be impossible to do on css-only level.
What am I missing or overcomplicating?

Yeah. You're totally correct. We have this problem in bug 7 as well. We can't know if we need to set width or height without knowing flex direction. For now, just use width or height, and when we have the options workflow in place, we can thing on how to solve this.

Oh, it looks like possibly-breaking-markup solution. Just warning maybe?

Maybe.. But here's the thing: do we want to warn the user even though the user is doing everything correctly? I rather we only fix what we can indeed fix and call it a day.

There's two things. First: setting height (or width) when we don't know flex-direction breaks markup, I think we shouldn't do this. And second thing, user decides to look for postcss warnings explicitly using one of two plugins, one puts errors to console and second — on web page. Even if we throw these warnings, only ones who want use them will ever see them, so it looks like a good idea. Another trouble with warnings is that there are bugs we can't even detect with css-only.

Well. This makes more sense. But the problem remains. If the user has an OK css declaration, we'll still emit warnings, right?
Here are some use cases:
css has the 8.2bug, we emit a warning
.FlexContainer {
  display: flex;
 }
.FlexItem {
  flex: 0 0 calc(100%/3);
}

css is OK, but we'll still emit a warning, since we can't know if the parent is column or row, so we can't know if we should check for width or height
.FlexContainer {
  display: flex;
  flex-direction: column;
 }
.FlexItem {
  flex: 0 0 auto;
  height: calc(100%/3);
}

css is OK, but we'll still emit a warning, since we can't know if the parent is column or row, so we can't know if we should check for width or height
.FlexContainer {
  display: flex;
  flex-direction: row;
 }
.FlexItem {
  flex: 0 0 auto;
  width: calc(100%/3);
}

DOes that make sense?

The exact code you've added looks like no-problem. But having flex-basis set instead of width/height looks like possible bug-catching style, and we can assert to postcss on that. What do you think?

Yeah. That makes sense. Can you do that? I'll do #13

Yep, I'm in.

Maybe using browserslist you might detect which browser you have to fix (helper used by autoprefixer, cssnext, pixrem...). That's my 2 cents, I am really new to flexbox and so the bugs :)

@MoOx yeah, but we have to consider the default option as well. One fix can't create another bug

This can't be fixed :|


FileNotFoundError: [Errno 2] No such file or directory: 'Query.bed'

Hello, got this error running de MCscan pipeline. I uploaded both .fasta and .gff3 for query and subject. (Pnotatum is the query name)
I installed with conda for Linux. Follows the traceback on stdout. The tool looks really nice, would much appreciate using it, tahnk you in advance

Traceback (most recent call last):
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/compara/catalog.py", line 955, in
main()
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/compara/catalog.py", line 78, in main
p.dispatch(globals())
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/apps/base.py", line 110, in dispatch
globalsaction
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/compara/catalog.py", line 703, in ortholog
blastfilter_main(dargs)
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/compara/blastfilter.py", line 322, in main
blastfilter_main(blastfile, p, opts)
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/compara/blastfilter.py", line 38, in blastfilter_main
qbed, sbed, qorder, sorder, is_self = check_beds(blast_file, p, opts)
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/compara/synteny.py", line 514, in check_beds
qbed = Bed(opts.qbed, sorted=sorted)
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/formats/bed.py", line 144, in init
for line in must_open(filename):
File "/home/juan/anaconda3/envs/shinysyn/lib/python3.9/site-packages/jcvi/formats/base.py", line 433, in must_open
fp = open(filename, mode)
FileNotFoundError: [Errno 2] No such file or directory: 'Pnotatum.bed'

Hi
Thanks for using ShinySyn, the issue may caused by the format of your GFF3. Can you share your input fasta and gff3?
Bests,
Zhixia


Opts line magic now issues keyword warnings in same style as cell magic

Small PR building off #1375 - the opts line magic and cell magic now issue invalid keyword warnings in the same style.

@philippjfr  Please merge if you are happy with it and the tests pass (no reason they shouldn't!)

Looks good, merging.


Use Web Test Runner for running browser tests

Issue #, if available:

Internal JS-4722
Refs: https://github.com/trivikr/browser-testing-web-test-runner

Description of changes:
Use Web Test Runner for running browser tests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing as Web Test Runner integration is not straightforward.
We can revisit it, once Angular community adds Jasmine support to Web Test Runner
The tests fail with:
src/index.browser.spec.ts:

 🚧 Browser logs on Chromium:
      SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'Sha256'
 🚧 Browser logs on Firefox:
      SyntaxError: ambiguous indirect export: Sha256
 🚧 Browser logs on Webkit:
      SyntaxError: Importing binding name 'Sha256' is not found.


BalSelect Input value resets when options list is changed

BalSelect Input value resets when options list is changed
When list of options changes, bal-select input is set back to empty.
Steps to reproduce the issue

Have a list of options that change depending on the bal-select input.
Type something that changes the list.
Observe bal-select value reverting to empty.

Additional information
Example: https://github.com/baloise/design-system-examples/tree/bug/issue_1011

Hi @ManojlovicMilos did you use the prop remote ?

https://stackblitz.com/edit/angular-wcnqtw?file=src%2Fapp%2Fexample.component.ts&file=src%2Fapp%2Fexample.component.html&showSidebar=0

Stackblitz has a package version that works when we include it in our project. Guess that was the point of the Stackblitz. So yeah, we can confirm that it works :D


Static harvests prohibitively slow with spark .wholeTextFiles

Running a static harvest over 200k+ records reveals that this approach for reading files does not scale well:
# read directory of static files
static_rdd = spark.sparkContext.wholeTextFiles(
	'file://%s' % kwargs['static_payload'],
	minPartitions=settings.SPARK_REPARTITION
)

Some cursory research shows that Spark does not handle a large amount of small files well, at least not without some tuning.
Investigate alternatives.

In addition to reworking static files, it might be worth simultaneously thinking about other ways of harvesting/importing data, as that might have bearing.
For example, what about a more open ended, programmable endpoint?  the ability to connect to a server or script that can operate in any fashion, but returns data in a way that Spark is able to efficiently handle?  We know that OAI-PMH is a popular source, but Combine might not be the place to attempt to handle the myriad of formats, structures, and protocols.

This is currently being addressed with rewrite of static harvesting, using Spark-XML.  Closing.


.Net: Bug: Exception in Azure OpenAI GPT-4 ResponseFormat

Describe the bug
In Azure OpenAI, I created a deployment using the GPT-4 model based on version 2024-08-06, with a URI ending in api-version=2024-08-01-preview. When using the AddAzureOpenAIChatCompletion API with ResponseFormat in the kernel object, it threw an exception stating that the model does not support ResponseFormat.
To Reproduce
Steps to reproduce the behavior:
Version : Microsoft.SemanticKernel 1.28.0
Expected behavior
Screenshots
Platform

OS: Windows
IDE: Visual Studio
Language: C#
Source: NuGet package version  1.28.0

Additional context
Workaround: I used an HTTP client with the URI (including api-version=2024-08-01-preview) and applied it in the kernel object. Now, the AddAzureOpenAIChatCompletion API works as expected.

When using the AddAzureOpenAIChatCompletion for a dedicated version you need to specify as an extra parameter:
apiVersion with the version you want in this case 2024-08-01-preview.
builder.AddAzureOpenAIChatCompletion(apiVersion: "2024-08-01-preview", ...);


Celebrate all contributors 🎉

Implement a variant of the All Contributors spec, tweaked to support .adoc.

@Shadowfiend I'd like to roll out something similar on keep-network/tbtc, keep-network/keep-core, and keep-network/website. Thought I'd give it a shot here first for review / thoughts


Error when building from source in a Ubuntu machine

Hi, my computer is ubuntu 18.04 and I want to build from source, though the binary version works fine.
As shown in INSTALL, I should run
sh scripts/build_ubuntu_18.04.sh
However, when I run this command, it tells me that external/symdiff does not appear to contain CMakeLists.txt.
If I just use cmake .,
it also fails.
So how should I build from source? Or shall I edit the .travis.yml first? If it is, how?
Thanks very much!

Prerequisite packages can be installed using scripts/install_ubuntu_depend.sh
The .travis.yml builds devsim on a docker centos 7 image.  However, you can use the lines in the file to build a version that should work on most modern linux distributions.  An advantage of the travis build, is that it uses the Intel MKL, as well as their linear solver, MKL Pardiso.
The external/symdiff directory is for a submodule.  To check it out:
git submodule init
git submodule update

You will also need to call this script (build_ubuntu_18.04.sh) from a working Miniconda/Anaconda environment.  Please see scripts/install_miniconda_linux.sh
Once you have activated the environment, the build script should work.

If you would like to start out by running the simulator, there are binary releases here:
https://github.com/devsim/devsim/releases/latest
Please note that there has been a recent change in the Intel MKL library name, so you may have to modify your Anaconda environment, as mentioned in Issue #68

Thanks. I got it. I run
git submodule init
git submodule update
and get the files needed.
By the way, when calling the script in an Anaconda environment, assume I have used
source ${HOME}/anaconda/bin/activate python3_devsim_build
shall I also modify build_ubuntu_18.04.sh since this script seems using python in the root directory which is not Anaconda python?

Let me repeat the steps to install from source.
Step 1: Install prerequisite packages listed in  scripts/install_ubuntu_depend.sh, e.g. run this script.
Step 2: Use
git submodule init
git submodule update
to get the file for symdiff.
Step 3: Use scripts/install_miniconda_linux.sh to activate an Anaconda environment.
Step 4: In this environment, run
sh scripts/build_ubuntu_18.04.sh
Step 5: Run install.py to install in python.
Is that right?

Sorry to bother you again. When I am building files to devsim/linux_x86_65_release, I got error in building CXX object devsim/src/MathEval/GaussFermi
which says
error: call to non-constexpr function ‘boost::multiprecision::number<Backend, (boost::multiprecision::expression_template_option)0> boost::multiprecision::operator*(boost::multiprecision::number<Backend, (boost::multiprecision::expression_template_option)0>&&, boost::multiprecision::number<Backend, (boost::multiprecision::expression_template_option)0>&&) [with B = boost::multiprecision::backends::float128_backend]’      static constexpr T sqrt2_pi = boost::math::constants::root_two<T>() * boost::math::constants::one_div_root_pi<T>();                         ^~~~~~~~
How could I solve this?
Thanks a lot for your kindly replys!

It looks like you do not need to activate an Anaconda environment for this build.
The compiler issue is due to the Ubuntu 18 boost installation being too old to compile GaussFermi.cc, without changes.
As a workaround, please change the setting in linux_x86_64_release/CMakeCache.txt to:
DEVSIM_EXTENDED_PRECISION:BOOL=OFF

When the build is complete:
cd dist
bash package_linux.sh devsim_ubuntu

and that should create a working distribution as:
devsim_ubuntu.tgz

You should then be able to go into the devsim_ubuntu directory and type:
export PYTHONPATH=$PWD/lib

And then from a python3 prompt try:
import devsim

If this works, you can try the scripts in the examples and testing directories.
The install.py script should work, if you want to use from a Anaconda python environment.

Thanks very much for your kindly reply. I will try this approach.

Thanks a lot for your patient instructions!
I have tried this approach and it works. I built it in an Anaconda environment and it seems I do not need to run install.py again.
By the way, will this building option influence the behavior of the procedure if I only care about double/single precision?

I am glad you were able to get it to work.  Double precision is the default, so extended precision is not used unless enabled in the simulation script.

Thanks a lot for your kindly reply! I got it.


Engineering for extendable datasets

I've made some changes to try and support a hypothetical future where users can filter, search, and combine multiple datasets. This basically means that for data set level info, I have added in identifiers for them in images, and made it so the "agadata" object is actually a set of "agdataset" objects. Naming should probably be reviewed to make sure it makes sense, because I know this can get confusing.

I wonder if this can be closed?

Yes, this is a very retro issue; we are past this now :)


feat: Add GLiNERLinkExtractorComponent

GLiNERLinkExtractor doc: https://python.langchain.com/api_reference/community/graph_vectorstores/langchain_community.graph_vectorstores.extractors.gliner_link_extractor.GLiNERLinkExtractor.html

Hey @cbornet
This requires the model to be loaded constantly in memory. I think we are better off closing this for our own sanity. What do you think?


Broken regex when https in the middle of string

If there is http in the middle of the input string, which would be the contents of the clipboard, the link does not get created since the clipboard doesn't have an actual link in the first place, but an error notification is also not shown.

After further testing, this does not seem to be an issue, but I am also sure that this was an issue previously and the code pertaining to this issue hasn't changed.
Needs to be looked into further.

After some more observation, this does not seem to be an issue.


Overwrites my conf changes

I've added:
usePrimarySelectionAsInput = True

To my ~/.config/greenclip.cfg but this gets ignored by greenclip and the line is removed from the .cfg by greenclip. (I've got my conf in git so I can see the diff).
Versions:
greenclip v3.1
Xorg 7.7+19
i3 <IP_ADDRESS>-2-g1c535b18

My setup related to greenclip: https://gitlab.com/skybert/my-little-friends/commit/c22efe8f76156a43ac64d560a732add48eb720bb

You have twice usePrimarySelectionAsInput in your config file, one with True and one with False.
Can you try removing one of it ?

Ouch, me stupid :man_facepalming: I started greenclip, got a default conf file, read the documentation and added the new setting without seeing it was already there.
As a future improvement, I would suggest that greenclip shouldn't replace the user's conf file like this without notice, it's confusing IMHO.
Thanks for your quick response and remedy, @erebe.

Yep it is a poor design choice, I don't even remember why i made it that way. I think because i wanted to migrate config from version to version auto magically


Unable to create new Area dimension value in a Evaluate Company

Please include the following with each issue:
1. Describe the bug
I get the error "There are no allowed dimension values selected." When i try to create a new dimension value with the name AM in a new Evaluate Company.
There are no allowed dimension values selected.
AL call stack:
DimensionManagement(CodeUnit 408).CheckIfNoAllowedValuesSelected line 4 - Base Application by Microsoft
"Default Dimension"(Table 352).GetFullAllowedValuesFilter line 11 - Base Application by Microsoft
"Default Dimension"(Table 352).GetAllowedValuesFilter line 4 - Base Application by Microsoft
"Default Dimension"(Table 352).UpdateDefaultDimensionAllowedValuesFilter line 5 - Base Application by Microsoft
DimensionManagement(CodeUnit 408).AddDefaultDimensionAllowedDimensionValue line 10 - Base Application by Microsoft
"Dimension Value"(Table 349).OnInsert(Trigger) line 8 - Base Application by Microsoft´
2. To Reproduce
I create a new company with the "Evaluation - Sample Data" setting.

I then go into the Dimensions page and navigate to Area Code and go to Dimension Values.
I then try and create a new Dimension Value called AM
3. Expected behavior
Being able to create a new Dimension value.
4. Actual behavior
I get the error "There are no allowed dimension values selected." When i try to create a new dimension value with the name AM.
5. Versions:

AL Language:
-9.0.615906
Business Central:

20 CU1

Hi, this issue is out of scope of for this repository. You should do one of the following:

Open a support request to CSS through the PartnerSource portal
Open a bug in Collaborate
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

As a reminder, this repository only handles issues to do with the compiler or VSCode development experience for AL.


Create 0000-01-02-katana7436.md

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request


No git indicator if file is modified and renamed

No indicator is shown if file is both renamed and modified. Apparently because OMZ git lib doesn't recognize RM git status and we're depending on it for git indicators.
Screenshot

@sfarzy seems like OMZ-related issue. Have you reported it to their issues?

Not yet. Looking at the state of response to issues and PRs in OMZ I don't think there is going to be any difference. However, I'll try to send patch to OMZ.
I was just giving a heads up.

This issue was fixed by #147.


[BACKLOG-38183] Make Avro metadata discovery step compilable

:white_check_mark: Build finished in 6m 39s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/avro-format/core

:ok_hand: All tests passed!
Tests run: 42, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 4m 58s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/avro-format/core

:ok_hand: All tests passed!
Tests run: 42, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Student finance forms: Replace part time students 2015/16 option with 2017/18

Trello card
Description
This PR contains updates for the following:

End application for 2015 to 2016 session for all  part-time students.
Add support for 2017 to 2018 session for all  part-time students.

Factcheck
Preview link
GOVUK
Expected changes

All part-time students can no longer apply for 2015 to 2016.
All part-time students can obtain information about applying for 2017 to 2018.

Before

After

This commit https://github.com/alphagov/smart-answers/pull/3010/commits/f4cf0140ba4356e46282863a177e829d53a8df6c LGTM 👍

This commit cf6f19fa05d43537be961459632ba9e78c602b52 LGTM 👍


polyadenylation and cleavage

It is impossible to annotate sensibly. There are has  39 process terms to choose from!
I don't even know where to begin...
Should be 3 or 4 processes:

cotranscriptional 3’ processing of RNA polymerase II  transcripts
(i.e nuclear polyadenylation, I think this is always coupled to termination- will check)

cytoplasmic polyadenylation

mitochondrial/bacterial

This is the left-hand frame

This is the right-hand frame

There are at least 2 independent branches, there is no relationship at all between these 2 terms. The split does not seem to follow the biology in any way.

Also  polyadenylation is  a type of 3' end processing. But this is not observed in BP.

Suggestions

terms should represent the processes and not the activities. For example  "nuclear cleavage and polyadenylation"
is also referred to as:

"cotranscriptional 3’ processing of RNA polymerase II (Pol2) transcripts"
( the cleavage, and the polyadenylation are single steps in this process).
We need to obsolete all of the single steps and do a lot of consolidation.

https://github.com/geneontology/go-ontology/issues/22708
https://github.com/geneontology/go-ontology/issues/22718
https://github.com/geneontology/go-ontology/issues/22229

we also have other requests for terms which refer to the same process but described in a different way
https://github.com/geneontology/go-ontology/issues/22339

review again after the 3 tickets above are closed

I don't need this ticket.


DBAAS-69: watch DBaaSProvider inventory and connection CRs

Description

Watch the provider ConfigMaps changes and re-deploy DBaaS operator
Jira: https://issues.redhat.com/browse/DBAAS-69
Verification Steps

Type of change

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

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA or in issue number have been completed
[ ] Unit tests added that prove the fix is effective or the feature works
[ ] Documentation added for the feature
[ ] all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer

On hold pending some discussion here as we may have to change where we find those CM's: https://docs.google.com/document/d/16zmkLMFVj6l3_2P3c1Dlyu6C65Iwd5ZnXRKK60xxrLA/edit?disco=AAAANQtWYpI

Content of comment thread in previous link has been moved into its own section for discussion (outcome still pending):
https://docs.google.com/document/d/16zmkLMFVj6l3_2P3c1Dlyu6C65Iwd5ZnXRKK60xxrLA/edit#heading=h.z264ael09a0f

Removing hold, but affected by the DBaaSProvider refactor.

After some investigation, I found controller supports watching events from new sources after it is started.
I have updated the DBaaSProvider controller to add new watches.

lgtm, IMO definitely feels more like it fits the controller paradigm this way, too. If you're good to drop the hold tag, I'll merge us up. Thanks for your efforts on this, I noticed what time this last change came in! :)


I am not sure what the problem is!

I type in source bin/activate and it says source is not an internal or external command!! :(
Other Information
OS: Windows 10
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)

Probably you don't have virtualenv. its not very important

Windows 10 doesn't recognize the source command so it won't work on Windows.
What I've had to do the couple times I have ran it on windows is go into the lib/scripts or scripts/lib folder(can't remember which it is) and just type in 'activate' to run it

There isn't a lib folder, what you have to do:
Open a console window from within your PokemonGo root folder.
Type: Scripts\activate.bat


Material UI and Next

Remove children required when use component
<Button component={(props) => <Link route='index'><a {...props}>Link</a></Link>} variant="outlined" color="primary" />
Warning: Failed prop type: The prop children is marked as required in Button, but its value is undefined.
Add props for hidden
props: {   MuiHidden: {     implementation: 'css'   } }
Add own variant
MuiButton: {   MyVariant: {     '@expand': 'outlined',     border: consts.border,   }, }
Fix error with hot-reloading
Warning: Can't call setState (or forceUpdate) on an unmounted component...

I don't understand this issue. But don't miss our Link component written on top of Next for the documentation.

Issue 1. I know about "Link component" thanks. But please remove required children when this.props.component not empty. Its more powerful. Easy example.
You have
<Button>{this.props.children ? this.props.children : console.log('err') }</Button>
Need
<Button>{(this.props.children || this.props.component) ? this.props.children : console.log('err') }</Button>
Issue 2. In createMuiTheme() have 2 features overrides and props. Please add props for  component.  When use ssr need always set implementation props
<Hidden only='sm' implementation="css">content</Hidden>
Issue 3. In createMuiTheme() we can change component and component varian

MuiButton: { 'outlined': { border: consts.border } }
What about own variant? You recommend use global class?
Issue 4. All time see error in console its really problem https://github.com/mui-org/material-ui/issues/11766
Of course i can override all components but its not try way


Update README.md to resolve #270

Added language about the Apache License

Current coverage is 24.96% (diff: 100%)

Merging #318 into master will not change coverage

@@             master       #318   diff @@
==========================================
  Files            47         47
  Lines          2828       2828
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            706        706
  Misses         2052       2052
  Partials         70         70

Powered by Codecov. Last update d74ea93...354802a


Docs/free trial append

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/getkong.org/blob/master/CONTRIBUTING.md
Summary
Ran into couple free trial users having difficulty w/ overly specific install from Bintray info.
Full changelog

Edited docs to account for different path for free trial users
Removed troublesome HTTPie install guidance
Edited docs to use cURL commands to test Kong install

Checklist:

[x] Commit message & atomicity checked
[x] Documentation
[x] Spellchecked my updates
[x] Ready to be merged

Formatting is appearing incorrectly locally

Phew! Thank you @catfishbluefish that looked like a**. Relying on GitHub preview not enough. Ran local build and saw exactly what you pointed out and fixed. Much obliged when merged if worthy now.


Deleted a duplicated callout from Non-Fungible Token tutorial

Description
Deleted a duplicated callout from Non-Fungible Token tutorial.
Sorry for small units of PRs (#91, #92 and this). I'm reading your tutorial in order and enjoying it.

@ryokomy No need to apologize, this is very helpful! Thank you :)


Add parameter_name argument to generate_ansatz_from_circuit

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 95.231%

Files with Coverage Reduction
New Missed Lines
%

qiskit_addon_aqc_tensor/simulation/explicit_gradient.py
1
93.68%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.2%

Covered Lines:
599

Relevant Lines:
629

💛  - Coveralls


The jobs may break in VIC appliance

Affect master branch.
The permission has changed.  But in VIC appliance the job_logs is under /storage/data so prepare may not work in that case, will double check and provide a more consistent solution for both standalone and vic.

Fixed in #4869


fix System.lineSeparator instead System.lineSeparatorâ€‹

System.lineSeparator was committed with wrong character. It is visible on eclipse for windows.

Thanks for contributing :)

You're welcome


Core dump after huge hang on relatively small cast

When attempting to convert a relatively small cast demo.cast.txt, I got this:
Stack trace
 svg-term --in demo.cast --out demo.svg --width=104 --height=82

<--- Last few GCs --->

[55408:0x55a83c1e3250]   996781 ms: Mark-sweep 1560.5 (2027.7) -> 1560.4 (1965.7) MB, 1398.6 / 0.3 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1540 ms) last resort GC in old space requested
[55408:0x55a83c1e3250]   998192 ms: Mark-sweep 1560.4 (1965.7) -> 1560.4 (1943.2) MB, 1410.8 / 0.0 ms  last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0xf3856698fe1 <JSObject>
    0: builtin exit frame: freeze(this=0xf3856698229 <JSFunction Object (sfi = 0xf3856698191)>,0x359fe6bffd31 <Object map = 0x124b4d504989>)

    1: createElement [/usr/lib/node_modules/svg-term-cli/node_modules/react/cjs/react.development.js:~727] [pc=0x3de9c3698345](this=0x194534d27c41 <Object map = 0x83fe79620d9>,type=0x30f593638731 <String[3]: use>,config=0x1773c4e13bc9 <Object map = 0x124b4d...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [svg-term]
 2: 0x55a83b478011 [svg-term]
 3: v8::Utils::ReportOOMFailure(char const*, bool) [svg-term]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [svg-term]
 5: v8::internal::Factory::NewMap(v8::internal::InstanceType, int, v8::internal::ElementsKind) [svg-term]
 6: v8::internal::Map::RawCopy(v8::internal::Handle<v8::internal::Map>, int) [svg-term]
 7: v8::internal::Map::CopyDropDescriptors(v8::internal::Handle<v8::internal::Map>) [svg-term]
 8: v8::internal::Map::CopyReplaceDescriptors(v8::internal::Handle<v8::internal::Map>, v8::internal::Handle<v8::internal::DescriptorArray>, v8::internal::Handle<v8::internal::LayoutDescriptor>, v8::internal::TransitionFlag, v8::internal::MaybeHandle<v8::internal::Name>, char const*, v8::internal::SimpleTransitionFlag) [svg-term]
 9: v8::internal::Map::Copy(v8::internal::Handle<v8::internal::Map>, char const*) [svg-term]
10: v8::Maybe<bool> v8::internal::JSObject::PreventExtensionsWithTransition<(v8::internal::PropertyAttributes)5>(v8::internal::Handle<v8::internal::JSObject>, v8::internal::Object::ShouldThrow) [svg-term]
11: v8::internal::JSReceiver::SetIntegrityLevel(v8::internal::Handle<v8::internal::JSReceiver>, v8::internal::PropertyAttributes, v8::internal::Object::ShouldThrow) [svg-term]
12: 0x55a83ae2e02f [svg-term]
13: 0x3de9c34196dd
Aborted (core dumped)

Not sure what other diagnostic information I should provide, lmk.
I think I noticed someone else discussing node's memory limits, & that they can be increased, but I assume their default is a reasonable value, & my small cast really shouldn't be hitting that. My PC itself has 64GB.
The following works (taken from readme)
svg-term --cast=113643 --out parrot.svg

I find that it works reliably if I always convert using --cast with asciinema's UUID, so for now, it's okay to just upload them and use that

I just ran into the same thing. Have 32 GB of RAM, but it seems like it's complaining about being out of heap memory with the error:
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

Same here:
 <--- Last few GCs --->

[3566:0x55b8e2ce9260]   686477 ms: Mark-sweep 1837.5 (2382.2) -> 1837.4 (2329.2) MB, 1164.4 / 0.0 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1165 ms) last resort GC in old space requested
[3566:0x55b8e2ce9260]   687668 ms: Mark-sweep 1837.4 (2329.2) -> 1837.4 (2309.7) MB, 1191.2 / 0.1 ms  last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x2da95dc18fe1 <JSObject>
    1: new constructor [/usr/local/lib/node_modules/svg-term-cli/node_modules/load-asciicast/index.js:~901] [pc=0x17132b1f0ffd](this=0x104482b2f959 <JSObject>,a=0x2da95dc02211 <null>,b=0x2c29369ce439 <String[13]: new-line-mode>,c=0x2c29369ce439 <String[13]: new-line-mode>,d=1467504785)
    4: cljs$core$IAssociative$_assoc$arity$3 [/usr/local/lib/node_modules/svg-term-cli/node_modules/load-asciic...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [svg-term]
 2: 0x55b8e147e011 [svg-term]
 3: v8::Utils::ReportOOMFailure(char const*, bool) [svg-term]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [svg-term]
 5: v8::internal::Factory::NewCode(v8::internal::CodeDesc const&, unsigned int, v8::internal::Handle<v8::internal::Object>, bool, int) [svg-term]
 6: v8::internal::CodeGenerator::MakeCodeEpilogue(v8::internal::TurboAssembler*, v8::internal::EhFrameWriter*, v8::internal::CompilationInfo*, v8::internal::Handle<v8::internal::Object>) [svg-term]
 7: v8::internal::compiler::CodeGenerator::FinalizeCode() [svg-term]
 8: v8::internal::compiler::PipelineImpl::FinalizeCode() [svg-term]
 9: v8::internal::compiler::PipelineCompilationJob::FinalizeJobImpl() [svg-term]
10: v8::internal::CompilationJob::FinalizeJob() [svg-term]
11: v8::internal::Compiler::FinalizeCompilationJob(v8::internal::CompilationJob*) [svg-term]
12: v8::internal::OptimizingCompileDispatcher::InstallOptimizedFunctions() [svg-term]
13: v8::internal::StackGuard::HandleInterrupts() [svg-term]
14: v8::internal::Runtime_StackGuard(int, v8::internal::Object**, v8::internal::Isolate*) [svg-term]
15: 0x17132b0840bd
[1]    3566 abort (core dumped)  svg-term --cast 375764 --out controlled.svg --width=80 --height=25

I noticed I was able to work around this by using OSX instead of Linux on the exact same asciinema cast. I tried changing the heap size in Linux but still ran into the same problem.


Chef-server Nginx does not implement HSTS

Chef HTTPS server is not enforcing HTTP Strict Transport Security (HSTS).
It represents an unnecessary risk to the overall security posture.
Concretely, there is no ['nginx']['sts_max_age'] variable to set the “Strict-Transport-Security” header
( add_header Strict-Transport-Security "max-age=" XXX ) for nginx on /etc/opscode/chef-server.rb
References
HSTS on Wikipedia: https://en.wikipedia.org/wiki/HTTP_Strict_Transport_Security
HSTS for Nginx: https://www.nginx.com/blog/http-strict-transport-security-hsts-and-nginx/
Nginx cookbook mentions it: https://supermarket.chef.io/cookbooks/nginx

Work around:
Add the header in nginx addons:
cat /opt/var/opscode/nginx/etc/addon.d/10_external.conf
add_header Strict-Transport-Security "max-age=31536000" always;

Restart Nginx:
/opt/opscode/embedded/bin/sv restart nginx
Verify that you get HSTS:
curl  https://<IP_ADDRESS> -k -v | grep Strict


when tx is not signed

Describe the feature you would like
it would be nice to have a feature that handles assigning unsigned(when, r,s,v = 0) tx to tx-circuits and etc.
Additional context
No response

implemented.


BUGFIX: Version 2.x of elasticsearch does neither need nor accept index_name in fields mapping

Using an annotation like this:
    /**
     * @var string
     * @ORM\Column(nullable=true)
     * @ElasticSearch\Indexable
     * @ElasticSearch\Mapping(fields={@ElasticSearch\Mapping(index_name="raw", type="string", analyzer="case_insensitive_sort")})
     */
    protected $title;

with version 2.x of elasticsearch throws the error:
{
  "error": {
    "root_cause": [
      {
        "type": "mapper_parsing_exception",
        "reason": "Mapping definition for [fields] has unsupported parameters: [index_name: raw]"
      }
    ],
    "type": "mapper_parsing_exception ",
    "reason": "Mapping definition for [fields] has unsupported parameters: [index_name: raw]"},
    "status": 400
  }

I'm fine if we merge this one, and maybe we can use this Settings to switch the driver of the ContentRepositoryAdapater package.
@johannessteu Did you agree that the conservative 1.x defaults are fine ?

Based on the documentation https://github.com/elastic/elasticsearch/issues/6677 index_name is deprecated in favor of copy_to, so maybe we can trigger an exception when the driver is switched to 2.x and we detect index_name, like this we can help user to edit the annotation and use copy_to ? What do you think ?
In the current solution we silently ignore something that should not work.

For the ES v5 adjustments I decided to transparently map some things that have been changed compared to v2. Maybe in this case we could do the same, and for 2.x automatically transform index_name to copy_to, while logging a warning about it?

Ah, but then again… I really like the driver pattern approach!

What is the status here?
I Am currently trying to update a super old Neos-Project. Besides the Neos related stuff we updated  Elasticsearch from 2.x to 5.x.
There are two Problems now: Still the unwanted index_name (we hat a patch for that even back then)
Second: it is not possible to use a different type (e.g. keyword which is the replacement for not analyzed strings as i understand ist) in a multifield as it is always overwritten by the type if the parent field. So there is currently no way to use multifields at all.

AFAIK these issues have all been fixed and index_name is no longer supported, so closing.


Allow to wait for all tablets to be propagated to TabletStatsCache

This change adds method TopologyWatcher.WaitForInitialTopology() that will wait
until the initial topology data is read and propagated to TabletRecorder through
AddTablet() method. This also adds HealthCheck.WaitForInitialStatsUpdates()
method which will wait until information about all tablets added via AddTablet()
method is propagated to the listener through StatsUpdate() method.
This will allow resharding and schema swap code to be sure that after they
created the topology watchers all tablets has been propagated to
TabletStatsCache (or other health check listener).
The schema swap code is modified here to use these new methods. This code is not
tested (because schema swap code is not fully ready yet), but serves as an
example of how these new methods can be used.

LGTM


Localization: German

With the basic forecast implemented and running additional languages need to be available. The first language after the initial english version will be german.

Staring point: https://github.com/ruby-i18n/i18n

Done with #30.


Create a EIP-4824 "template repo" for organizations to create a 4824-ready daoURI set of data directly on GitHub

Problem
See https://github.com/metagov/eip4824 as the example.
A clear and concise description of what the problem is. Tell us why we should be working on this. Ex. I'm always frustrated when [...]
Difficulty

Select one of the following and delete the other. Add any nuance below. How much time will we need to spend on this and how does that constrain the solution?

*Tiny: This is a project that one programmer or designr can do in a day.
Small: This is a project that a team of one designer and one or two programmers can build in one or two weeks.
Big: This project takes an entire team working for a full development cycle (six weeks).
Solution

A clear and concise description of what you want to happen. Drawings and doodles are very encouraged.

References
Provide a list or overview of references and planning docs helpful in understanding the problem or solution.
Additional context
Add any other context or screenshots about the feature request here.

To do: merge Rashmi's branch into main, move the "Collections" folder from that repo (which should just be for Metagov) into a "github.com/metagov/daostar/registrations/Arbitrum DAO/daoURI.json" folder in the metagov/daostar repo.

Done

Hooray! :clap:
On Wed, Jul 31, 2024 at 9:38 AM Rashmi V Abbigeri @.***>
wrote:

Done
—
Reply to this email directly, view it on GitHub
https://github.com/metagov/daostar/issues/245#issuecomment-2260548901,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHA5PGDT5GJSNSIWLKMTL3ZPDSEVAVCNFSM6AAAAABLMQXJJKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRQGU2DQOJQGE
.
You are receiving this because you were assigned.Message ID:
@.***>

--
joshuatan.com/research


fix: integration tests

Should this be converted into a draft @petertonysmith94 until all the CI tests pass?


Chart renders incorrectly when using StackedAreaChart with brush and zoom

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior
I constructed this demo using a few examples from the stories, and I have the following issues:

When changing the range, the chart lines are being rendered incorrectly.

The same exact chart looks different between the brush and the large chart. Is that because of the different heights?

Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/react-reaviz

Use the chart brush to shorten the range
Scroll to the right.

(The codesandbox link is not working)
Environment

Libs:
- react version: 17.0.1
- reaviz version: 9.3.14

Browser:
- [x] Chrome (desktop) version 86.0.4240.198
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: v12.18.3
- Platform:  Mac

Oh - that is fun. Not sure where to start on this one but thanks for the details!

Thanks for the quick response!
I tried disabling the animation for the Series and Line, the drawing is faster but the bug still happens.

So I have found out that the issue starts here:
https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/Line.tsx#L102
I tried two different approaches that fix this issue:

Changing the condition in https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/Line.tsx#L102:
if (
  this.ghostPathRef.current &&
  (prevProps.data !== this.props.data ||
    prevProps.width !== this.props.width ||
    prevProps.xScale !== this.props.xScale ||
    prevProps.yScale !== this.props.yScale)
) {}

Spreading the data array in https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/AreaSeries.tsx#L182 and https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/AreaSeries.tsx#L197:
-data={data}
+data={[...data]}

I'm no React expert but I think that if one of the solutions is fine, it should be applied in a lot of places in the different charts.
And if you think one of them is applicable, I can open a PR. :)

So I have found out that the issue starts here:
https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/Line.tsx#L102
I tried different approaches that fix this issue:

Changing the condition in https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/Line.tsx#L102:
if (
  this.ghostPathRef.current &&
  (prevProps.data !== this.props.data ||
    prevProps.width !== this.props.width ||
    prevProps.xScale !== this.props.xScale ||
    prevProps.yScale !== this.props.yScale)
) {}

Spreading the data array in https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/AreaSeries.tsx#L182 and https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaSeries/AreaSeries.tsx#L197:
-data={data}
+data={[...data]}

Using the states in my component instead of the imported data and spreading the data array.

I'm no React expert but I think that if one of the solutions directly in the reaviz code is fine, it should be applied in a lot of places in the different charts, and I have no problem opening a PR.

Ya - can you open up a PR - happy to accept that.

I went with the first solution.
Let me know if you want me to create a story for this scenrio.

Ya - add a story in the brush stories plz :D


Class mdanter/phpecc update

Some classes puts the adapter from the last to the first

This doesn't seem to affect master. I have set the version at 0.2.0 for this library, I'll update it soon though.

@FabioCarpi https://github.com/Bit-Wasp/bitcoin-lib-php/pull/59 will bring the library up to date! I'll keep an eye out for 0.3.0 when it comes out

thanks. im anxious

Hey @FabioCarpi I saw your hangouts invite, but for some reason I can't get your email from that. I actually use another email for hangouts, sorry for the confusion :/ Send me an email at the last one


There is a visible problem!

Hello there,
This seems to be a problem for the demo, this task has been opened for you to learn.
Screenshot : http://prntscr.com/h42hrp

Hey, it is a bug of ionic old version, https://github.com/ionic-team/ionic/issues/11454. And i think it fixed it already on the latest one. But i didn't update it yet. :)

Since @sdey0081 is so busy, i will assist him for a while. {!}

You are so fast, I admire your interest. ;) 👍


[FR] PUI - Add option for plugins to add header actions

Please verify that this feature request has NOT been suggested before.

[x] I checked and didn't find a similar feature request

Problem statement
Plugins do not have a way to add quick action to the header action of items. Therefore users need to click at least twice to issue a plugin action once they are on the page for the item.
Suggested solution
Add a way for plugins to expose 0 or more custom actions for (specific) items

They should at least be able to set:

Text
Icon
Tooltip text

To enable some transparency I think all plugin action should have a standard information from which plugin the action is emitted added to its tooltip
Describe alternatives you've considered
Adding a custom panel
Examples of other systems
No response
Do you want to develop this?

[x] I want to develop this.

@wolflu05 any input on this?

Yes, that's exactly what I was planning to add with the custom UI feature system. I have added some more detail on how I imagined the system in https://github.com/inventree/InvenTree/pull/8278#discussion_r1862652208

@matmair this would be a great addition :)


cve-2020-26160 vulnerability in dgrijalva/jwt-go

There is a CVE (cve-2020-26160) detected  in the library dgrijalva/jwt-go used by one of required library that you use (present in the go.sum file).
warning from github:
CVE-2020-26160
high severity
Vulnerable versions: <= 3.2.0
Patched version: No fix

jwt-go allows attackers to bypass intended access restrictions in situations with []string{} for m["aud"] (which is allowed by the specification). Because the type assertion fails, "" is the value of aud. This is a security problem if the JWT token is presented to a service that lacks its own audience check. There is no patch available and users of jwt-go are advised to migrate to golang-jwt at version 3.2.1

I ran go mod why github.com/dgrijalva/jwt-go and it looks like this is coming from Azure:
%~> go mod why github.com/dgrijalva/jwt-go
# github.com/dgrijalva/jwt-go
github.com/gruntwork-io/terratest/modules/azure
github.com/Azure/go-autorest/autorest
github.com/Azure/go-autorest/autorest/adal
github.com/dgrijalva/jwt-go

@HadwaAbdelhalem do you think we can bump the Azure dependencies in terratest to latest version?

Hi @yorinasub17  investigating it now

Just reopening this since that whole project is archived now : https://github.com/dgrijalva/jwt-go/issues/462


Track if it was right to counsel if the person is an attorney

This should get added in to the interview as a question (when the user says that they are an attorney), and then tracked in the place we call store_variables_snapshot()


[AMQP Python] async performance analysis

write async test scripts
check the sync performance analysis issue

sending
receiving

results:

async recv perf regressed
detailed analysis: https://gist.github.com/yunhaoling/2e8f27710c0351f2d7ab9badf9abbf9d


fix: update workflow to create MbedtlsSizeOf.kt file

Problem:
Workflow for updating mbedtls is missing MbedtlsSizeOf.kt
Why?
The compileMbedtlst.sh script to create the file is never executed in the instance where the PR is created, since the downloaded artifacts only contain the binaries.
Solution:
Run the compile script to create the MbedtlsSizeOf.kt file and then remove the binaries
Create a new artifact inside compileMbedtlst.sh and pass the file from there

Looks good, still lets test it


Trap convenience class

Proposed changes
With this update users will be able to give a trap object without having to define a 2 species (immobile and implicit) and a reaction, instead just having a F.Trap() object instead and passing it to the new attribute of F.HydrogenTransporModel, traps. Traps needs to be given as a list like so:
my_trap = F.Trap(
    name="test",
    species=my_mobile_species,
    k_0=1,
    E_k=1,
    p_0=1,
    E_p=1,
    n=1,
    volume=my_vol,
)
my_model.traps = [my_trap]

This convenience class should make some simpler hydrogen trapping cases easier for users
Types of changes
What types of changes does your code introduce to FESTIM?

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code refactoring
[ ] Documentation Update (if none of the other choices apply)
[ ] New tests

Checklist

[ ] Black formatted
[ ] Unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@RemDelaporteMathurin for some reason when the final test I've added in test/test_h_transport_problem breaks everything else, but when its commented the rest of the tests pass, not sure whats going on here?

@jhdark I figured out the issue. It is related to the fact that the default value for reactions in HTransportProblem is [].
When adding a trap object (like in the test), on this line, a reaction is added to self.reactions. However, since self.reactions is a list and lists in python are class instances, the (unwanted) result is that a reaction is added to the default value of self.reactions.
Consider this where we first create a model with a dummy reaction, then another empty model, and the reaction is carried over.
import festim as F

model1 = F.HydrogenTransportProblem()
model1.reactions.append("test_reaction_as_str")

model2 = F.HydrogenTransportProblem()
print(model2.reactions)  # IT SHOULD BE EMPTY!!

@gabriele-ferrero @SamueleMeschini would you mind having a look at this, at least from the user point of view? Is this implementation fine with you? The idea is to later have even more convenience by having one default mobile species created if nothing is given by the users.

@gabriele-ferrero @SamueleMeschini would you mind having a look at this, at least from the user point of view? Is this implementation fine with you? The idea is to later have even more convenience by having one default mobile species created if nothing is given by the users.

It seems ok, the notes do a great job explaining the equivalence of the convinience class. A default mobile species would be even better because I think most of the users will do simple H transport.

Yeah a default mobile species will be needed, its just to have a default mobile species, this would require refactoring of a lot of other logic in hydrogen_transport_problem, so think we'll have to save that for another PR

It seems ok to me, and I think it is a great convenience class to have in festim. I will have a further look when it will pass the checks

when it will pass the checks

Yes this should be fixed in #661


feat: add CI config, better support older/new configs

What this PR does: This PR adds CI configuration to box intended to be used with stencil-circleci. I also took the time to make the storage type set Config as an interface{}. This should allow us to better handle older/newer versions of the library. The MinVersion setting already forces updating it, but older versions could downgrade and thus change the storage. Now we persist it as an interface and cast it back to yaml to allow us to parse it in whatever format is needed.

Pull Request Test Coverage Report for Build 7220c4db-53a2-4844-a1b5-09f0aa0be860

0 of 34 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 54.491%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/box/fetch.go
0
34
0.0%

Totals

Change from base Build 419dce7b-d4d6-407e-a290-fba80ab026bb:
-0.2%

Covered Lines:
2081

Relevant Lines:
3819

💛  - Coveralls


SERVICES: Added lab service and exception handler with unit tests

Paired with Geetika to work on this PR.

@geetika-jain && @chaehonglee rename pr with the right category
FOUNDATIONS: Retrieve All Labs


Link to static file✨

Hi there,
it's GREAT to see this plugin being picked up close to it's original concept.
What I would love is another planner mode:
Static File would just link to a static file without generating any dailies or relying on this to be done by external plugin.
Why do I want this?
I worked with dailies but found that it cluttered my workspace quite a lot. Also, I had difficulties finding stuff on the occasions that I wanted to go back to some note without rememberingthea specific date.
What I use is something different now:

I have various notes where I put tasks depending on area.
I created a document sitting in sidebar that displays part of these notes depending on due or started date as well as tags.
for displaying them, I use callouts that form a kind of time blocking system.

I always wished for a plugin to do exactly that. Day planner is the only one coming close.
Is there any chance to make this plugin link to a static file?
I would be VERY grateful indeed!

Try command mode. I happen to have daily notes, but I don't use this plugin in daily notes mode.. I use command mode, and bind to any note that has a Day Planner heading in it


python311Packages.qcodes-loop: remove patch

Description of changes
Fix build (https://hydra.nixos.org/build/239412543). Failed after https://github.com/NixOS/nixpkgs/pull/251878

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/)
23.11 Release Notes (or backporting 23.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.

Result of nixpkgs-review pr 264308 run on x86_64-linux 1

4 packages built:

python310Packages.qcodes-loop
python310Packages.qcodes-loop.dist
python311Packages.qcodes-loop
python311Packages.qcodes-loop.dist


Crate names unable to have digits

Describe the problem you are trying to solve
Today, we had a project for uni, we could use any programming language to solve the problem and I wanted to use Rust as the language to solve the problem, unfortunately, in the specifications, we were told we needed the compiled binary that needed to be named "101Pong". So I tried creating a crate with that name, and cargo didn't let me do it.
Describe the solution you'd like
I'd like to see an option that can let one force the name of the crate even tho it is not authorized to have a crate with digits in front of the name, (I will not upload that crate to anywhere except a private GitHub repository), this would be great so that person doesn't have to use an external tool (like make/Makefile) to invoke the compilation process and then rename the binary from an authorized name to a non-authorized name
Notes

You can change the name of the binary in Cargo.toml by setting the name field:
[[bin]]
name = "101Pong"

The associated filename would default to src/bin/101Pong.rs or you can explicitly set the filename like path = "src/main.rs".
I think it is unlikely we would change it to allow package names to start with numbers due to the problems that could cause, but I think it would be good to add this tip to the cargo new error message.
You can learn more about Cargo targets and default filenames at https://doc.rust-lang.org/cargo/reference/cargo-targets.html.

Okay, thanks for the tip, I wanted to be able to name my directory and crate the same name the exercise was but I guess I won't be able to

@ehuss is this supposed to be an open issue? or is the OP request not meant to be possible at all?

@ehuss is this supposed to be an open issue? or is the OP request not meant to be possible at all?

@joao-conde This issue is open to track updating the error message.  I have posted #9098 to close this.

@joao-conde This issue is open to track updating the error message.  I have posted #9098 to close this.


Do not fail on wrong source map. Fixes #9

Sorry for the lack of a more timely response to this!
I'm more than a little hesitant to merge in a change that makes browserify-hmr silently ignore and replace sourcemaps in some situations. Logging the fact the issue happened would be a bit better, though I'd rather identify the other transforms that create bad or incompatible sourcemaps and try to fix their problem.

No problem man, Sadly not all transforms play nice with source maps so for those cases I would still suggest to consider some kind of workaround. Maybe an option or something. Logging it is definitively better than failing silently. For now this workaround is good enough for my use case.


[SPARK-43659][CONNECT][PS][TESTS] Enable OpsOnDiffFramesEnabledSlowParityTests.test_series_eq

What changes were proposed in this pull request?
This PR proposes to enable OpsOnDiffFramesEnabledSlowParityTests.test_series_eq for Spark Connect.
The root cause is fixed from https://github.com/apache/spark/pull/41514
Why are the changes needed?
To improve the test coverage for pandas API on Spark with Spark Connect.
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?

Reusing the existing UT.

merged to master


c++: add header guards

Add header guards to osdp.hpp to prevent multiple inclusions of the same header file. osdp.h already has this.

LGTM. Thanks.


Felix config field for range of route tables to use

Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Docs here: https://github.com/projectcalico/calico/pull/3477


AttributeError: 'Int64Index' object has no attribute 'to_period'

Which version are you running? The lastest version is on Github. Pip is for major releases.
0.3.14b0

Do you have TA Lib also installed in your environment?
yes

Describe the bug
/pandas_ta/overlap/vwap.py:23
     21 # Calculate Result
     22 wp = typical_price * volume
---> 23 vwap  = wp.groupby(wp.index.to_period(anchor)).cumsum()
     24 vwap /= volume.groupby(volume.index.to_period(anchor)).cumsum()
     26 # Offset

AttributeError: 'Int64Index' object has no attribute 'to_period'

To Reproduce
vwap = df.ta.vwap()

## data looks like
ticker | instrument_name | instrument_type | expiry_date | strike_price | datetime | date | time | open | high | low | close | volume | open interest | vix_close
-- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | --
BANKNIFTY-I.NFO | BANKNIFTY | FUT | 2019-01-31 | 0.0 | 2019-01-02 09:15:00 | 2019-01-02 | 09:15:00 | 27410.3 | 27430.0 | 27350.5 | 27360.0 | 45120 | 1600160 | 15.5625
BANKNIFTY-I.NFO | BANKNIFTY | FUT | 2019-01-31 | 0.0 | 2019-01-02 09:16:00 | 2019-01-02 | 09:16:00 | 27358.95 | 27377.4 | 27349.0 | 27377.1 | 26800 | 1600320 | 15.7225
BANKNIFTY-I.NFO | BANKNIFTY | FUT | 2019-01-31 | 0.0 | 2019-01-02 09:17:00 | 2019-01-02 | 09:17:00 | 27379.1 | 27382.7 | 27366.05 | 27377.2 | 21180 | 1600320 | 15.72
BANKNIFTY-I.NFO | BANKNIFTY | FUT | 2019-01-31 | 0.0 | 2019-01-02 09:18:00 | 2019-01-02 | 09:18:00 | 27377.15 | 27397.6 | 27376.0 | 27397.6 | 18320 | 1600320 | 15.72
BANKNIFTY-I.NFO | BANKNIFTY | FUT | 2019-01-31 | 0.0 | 2019-01-02 09:19:00 | 2019-01-02 | 09:19:00 | 27397.35 | 27430.8 | 27395.05 | 27425.0 | 28040 | 1614560 | 15.645

## data schema
ticker                     object
instrument_name            object
instrument_type            object
expiry_date                object
strike_price               object
datetime           datetime64[ns]
date                       object
time                       object
open                      float64
high                      float64
low                       float64
close                     float64
volume                      int64
open interest              object
vix_close                  object

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Thanks for using Pandas TA!

Hello @rava-dosa,
The Pandas DataFrame can not find the DatetimeIndex that is required as stated on the README Quick Start and at the bottom of the Overlap Category.
VWAP is the only indicator that requires the DataFrame Index to be a DatetimeIndex in order to work correctly. Please set the index appropriately for your situation.
Kind Regards,
KJ

Hi @twopirllc
Thanks for your support and pinpointing the problem. It is solved, closing the issue

@rava-dosa,
Great! No worries.


Duplicate symbol error and DR2387

So I'm not sure if this is a Clang issue or a Hana one, but since https://github.com/llvm/llvm-project/commit/bb750689d51d2109b9898b53a4b69d8183ab1f26, but there are duplicate symbol errors if Hana is included in more than one translation unit. This can be shown with Hana's test.issues.github_75 (see below).
That commit to LLVM was to address DR2387 for the C++ standard. Unfortunately, I could not find a copy of the DR (I get a 404 from http://wg21.link/cwg2387, so I think it will be in the next mailing), but it looks as if it applies the same rules regarding internal linkage of const-qualified variables to const-qualified variable templates.
Because of this, the declaration of constexpr auto curry_or_call<0> in functional/curry.hpp: https://github.com/boostorg/hana/blob/9ffeb079fed195849d384fa7ed2f8dbcf9c04c88/include/boost/hana/functional/curry.hpp#L119 looks like it now has external linkage, causing the duplicate symbol error.  Adding inline (and ignoring -Wc++17-extensions) gets it to work--as does static FWIW...
Here is the compiler output from https://github.com/llvm/llvm-project/commit/bb750689d51d2109b9898b53a4b69d8183ab1f26:
$ cmake .. -DCMAKE_CXX_COMPILER=/home/colin/.local/lib64/llvm-installs/bb750689d51/bin/clang++ -DBOOST_ROOT=/home/colin/extra-storage/development/boost-installs/master -G Ninja

$ ninja test.issues.github_75 -v
[1/3] /home/colin/.local/lib64/llvm-installs/bb750689d51/bin/clang++  -DBOOST_HANA_CONFIG_ENABLE_STRING_UDL -I../include -fdiagnostics-color -ftemplate-backtrace-limit=0 -pedantic -Wall -Wextra -Wno-self-assign-overloaded -Wno-unused-local-typedefs -Wwrite-strings -Wno-gnu-string-literal-operator-template -std=c++14 -MD -MT test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o -MF test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o.d -o test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o -c ../test/issues/github_75/tu2.cpp
[2/3] /home/colin/.local/lib64/llvm-installs/bb750689d51/bin/clang++  -DBOOST_HANA_CONFIG_ENABLE_STRING_UDL -I../include -fdiagnostics-color -ftemplate-backtrace-limit=0 -pedantic -Wall -Wextra -Wno-self-assign-overloaded -Wno-unused-local-typedefs -Wwrite-strings -Wno-gnu-string-literal-operator-template -std=c++14 -MD -MT test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o -MF test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o.d -o test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o -c ../test/issues/github_75/tu1.cpp
[3/3] : && /home/colin/.local/lib64/llvm-installs/bb750689d51/bin/clang++     test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o  -o test/test.issues.github_75   && :
FAILED: test/test.issues.github_75
: && /home/colin/.local/lib64/llvm-installs/bb750689d51/bin/clang++     test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o  -o test/test.issues.github_75   && :
test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o:(.rodata+0x0): multiple definition of `boost::hana::curry_detail::curry_or_call<0ul>'
test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o:(.rodata+0x0): first defined here
clang-9: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

And from the previous commit (https://github.com/llvm/llvm-project/commit/0d923af49270ef4beb5d479dac5bda80fc0082ef):
cmake .. -DCMAKE_CXX_COMPILER=/home/colin/.local/lib64/llvm-installs/0d923af4927/bin/clang++ -DBOOST_ROOT=/home/colin/extra-storage/development/boost-installs/master -G Ninja

$ ninja test.issues.github_75 -v
[1/3] /home/colin/.local/lib64/llvm-installs/0d923af4927/bin/clang++  -DBOOST_HANA_CONFIG_ENABLE_STRING_UDL -I../include -fdiagnostics-color -ftemplate-backtrace-limit=0 -pedantic -Wall -Wextra -Wno-self-assign-overloaded -Wno-unused-local-typedefs -Wwrite-strings -Wno-gnu-string-literal-operator-template -std=c++14 -MD -MT test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o -MF test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o.d -o test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o -c ../test/issues/github_75/tu1.cpp
[2/3] /home/colin/.local/lib64/llvm-installs/0d923af4927/bin/clang++  -DBOOST_HANA_CONFIG_ENABLE_STRING_UDL -I../include -fdiagnostics-color -ftemplate-backtrace-limit=0 -pedantic -Wall -Wextra -Wno-self-assign-overloaded -Wno-unused-local-typedefs -Wwrite-strings -Wno-gnu-string-literal-operator-template -std=c++14 -MD -MT test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o -MF test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o.d -o test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o -c ../test/issues/github_75/tu2.cpp
[3/3] : && /home/colin/.local/lib64/llvm-installs/0d923af4927/bin/clang++     test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu1.cpp.o test/CMakeFiles/test.issues.github_75.dir/issues/github_75/tu2.cpp.o  -o test/test.issues.github_75   && :

I just hit the same issue... :-(

And adding C++17 inlinesolved my problem too.

Perhaps we could work around this by default constructing an instance of apply_t.
constexpr auto curry_or_call<0> = decltype(apply){};

I am using ToT Clang/LLVM.
It is not clear to me which work-around can work with all the compilers from https://github.com/boostorg/hana/wiki/General-notes-on-compiler-support
@ldionne what do you think?

Fix incoming, sorry for the delay!

@ldionne Great! Thank you.

Is there an easy work around for those of use for whom changing hana version or compiler version is not an option?

@catskul
If you don't use that instance of curry_or_call it should not be a problem. Look at the fix: https://github.com/boostorg/hana/commit/7218a046c731b2548b5ae3db55fb711ccda8fbc4

I didn't use curry directly but apparently it was used indirectly and still had the same effect.
In my case I was only using hana::for_each and hana::tuple, but including hana.hpp
Now that I think of it I'm guessing the problem is more likely use of hana.hpp perhaps if include the minimal header set instead of the catchall header, I can get around it.


vim.api.nvim_set_current_win doesn't work

After using vim.api.nvim_set_current_win, it seems that the cursor is partially still at the original window.
What I mean is:

the rendered cursor itself is still at the other window
when using jk, then the CursorLine is actually moving in the new window.

Thanks!


fix: might miss content clicks inside PopperContent

While using elk, I found the following button inside the hover card has chances failing to response to mouse clicks.
I made a video screenshot below:
https://github.com/Akryum/floating-vue/assets/3401641/046f4a26-2ab6-4b8f-8113-0adbecbe27a2
And discovered remove the style ponter-event: none can ensure that the click takes effect.
Since the visibility: hidden already made sure that the element won't response to click event, I guess pointer-event: none is no longer needed here.
I'm not sure what is the root cause here, maybe it's because the style pointer-event: none took effect before the browser can take care of the click event.

This does not seem to be the right approach for me. I think instead, on the Elk side, it should have not made the popup disappear upon clicking.

Yeah, seems a bit tricky and didn't solve it clear enough.
BTW, just found the bug is hard to reproduce on https://elk.zone/home, but relatively easy on pnpm run dev.
Anyway, close for the moment.


Project.json schema @ schemastore.org: Update reference "kpm bundle" to "dnu bundle"

http://json.schemastore.org/project
Line #79: "description": "List of files to exclude from publish output (kpm bundle).",

http://json.schemastore.org/project-1.0.0-beta4 has the latest changes

Ah! ... then it's just the link to the schema at:
https://github.com/aspnet/Home/wiki/Project.json-file
... that should be updated.

Will do. We kept them separate for now just so we don't break older consumers.

Gosh ... lots of significant deltas between the two files. I know keeping that Wiki link to the schema up-to-date might be a problem for next few months, but I wonder if it isn't a larger problem having newbies (like me) looking at such an old schema when we're jumping in for the first time. I can see that messing folks up.
The Wiki link is under the "Schema" heading at: https://github.com/aspnet/Home/wiki/Project.json-file
Actually, I'll leave this open and let you decide if you want to update the link or not and close this at your discretion. Thx, Luke

That all depends, if you're using beta3 then it makes sense. The dnx changes aren't even done yet so the question is, what should the wiki represent? Bleeding edge or released? Or should it just point to both schemas.

Good point ... and there is no way to provide a directory of all stored schema files with schemastore.org? ... then people could look and know (or guess) by filename which one they should be looking at (and it would also save your team having to keep that one link updated)

Is it easier to read the schema or the wiki?

Yeah, easier to read the wiki for sure. However, that schema link being the first thing on that page ... and my being newbie ... I clicked to open that schema file and started reading it carefully. That's really not a good idea given the deltas to the latest version for a newbie about to jump in at CTP7.
I guess what you're implying is that the Wiki (or parts of it) might be 'last CTP beta' ... beta3 right now ... and not 'bleeding edge (dev branch) beta' or 'trying to learn for the next CTPx beta' ... beta4 right now.
Maybe the way to go is just to add a note below that link that points to the schemastore JSON page:
Visit the JSON Schema Store and open the latest project-*.json file shown for the latest (dev branch) project.json file schema.

That gets bleeding edge folks to the right one and saves your team ever having to think about where that link goes as the dev branch rolls forward. Things are probably going to get out-of-sync again after CTP7 when beta5 changes to that schema occur, and having such a note there would solve this.


Ref not accessible

when using @responsive annotation, the component ref does not have methods accessible anymore

I removed support for @Responsive in the latest version.


clasp login --status is still not available?

hey guys
I found new command option status regarding of clasp login on README.
but latelest npm version downloaded doesn't exist such a option.
so I digged closed PR , but da one had merged.
hmmm, let me know how 2 use dis option or da option launch schedule,🙏

@nasat75 Can you try installing again? I just published 2.3.0.

@grant Thank u so much !! it's successful downloaded !!


Vehicle restrictions: Distinguish intercity from intracity trains

Darkangel says:
"Vehicle restrictions: Include intercity rail as well as cargo and normal passenger trains. I'm sick of seeing massive amounts of orange outer limits trains driving on the same tracks as cargo or pasasenegr. Even if i stop it from cargo lines i cant stop it clogging my normal passenger lines!"

Isn't this handled by the "Allow outside connections" checkbox on passenger train station info boxes?

Yes, but it would still be need to have more control over which rails these trains use. Especially when there is a lot of train traffic.

Assuming icons will be required, some suggestions to ponder:
Internal:

circular stops, alt
map grid, alt
tile grid
C:SL Districts info view icon - likely confusing

External:

single stop
hub and spokes
alt hub and spokes
map grid arrow, globe
Earth
C:SL Outside Connections info view icon - while it's the "official" icon, it makes me think of industrial stuff for some reason

I tried to think about possible extension of the feature to ships (cargo and passenger), planes and cars. The icons above mostly depict "stops" which won't be applicable to cars/trucks, but I assume cars/trucks are also the least likely to have internal/external toggles implemented.

Hrm, yes, I see what you mean.
There doesn't seem to be a need for restricting local trains, as that can be handled with their routes (for passenger trains at least).
So, specifically, it's the ability to block outside trains from using specific track segments (or segments from a junction)?
And the pathfinder would need some way to know "find a path for this train" relates to an external train...
How would user be alerted to fact that their track restrictions have made a station inaccessible to outside trains in case where they want outside trains at that station?
Would there be merit to considering an "outside train route" tool - allow the user to draw specific routes from a given external connection to stations they want trains from that connection to stop at?


Feature/45-implement-office-plan-adding-and-editing

Added: IOfficeService, OfficeService, IOfficeRepository, OfficeRepository, OfficeController with their methods
Modifed: Office, StartupSetupBL, StartupSetupDL

Resolve Merge conflicts and Finish task according to assignment details


Properly set depth function for NativeEngine

The shader used for the WebXR hands requires a depth prepass, and when displaying the model in native I noticed nothing was rendering to the screen. Upon further inspection, the BabylonNative engine under-the-hood defaults to a depth mode of LESS, when BabylonJS assumes the default to be LEQUAL, but there was no actual call to _native.setDepthTest that was setting the underlying value in the native side.
https://github.com/BabylonJS/Babylon.js/blob/d120c8e483e3e92f01deba34880552be27bfaed8/src/Engines/thinEngine.ts#L1140-L1143
NativeEngine initializes the depth test value to LEQUAL, but the value never actually gets set with a call to _native.setDepthTest.
https://github.com/BabylonJS/Babylon.js/blob/d120c8e483e3e92f01deba34880552be27bfaed8/src/Engines/nativeEngine.ts#L765
https://github.com/BabylonJS/Babylon.js/blob/3a9e1322b453881b59407a511fe4705fe2cd59b9/src/XR/features/WebXRHandTracking.ts#L446-L449

A screenshot from the BabylonNative playground with the fix applied. Previously, nothing was getting rendered to the screen because the color pass was getting discarded due to the depth values being equal.

Note that in ThinEngine there is:
public get depthCullingState(): DepthCullingState {
    return this._depthCullingState;
}

It means a user can set the depth function (and other states like zOffset, cull, cullFace, etc) directly without using the setDepthFunction, setZOffset, ... helper functions of the engine.
So, instead of overriding all those helper functions, for WebGPU I have created a WebGPUDepthCullingState (that extends DepthCullingState) to make sure all those state changes end up in the WebGPU implementation. Note that there's the same problem with the stencil, there's a specific WebGPUStencilStateComposer that extends StencilStateComposer.


Do something / nothing when max retries run out.

I'm doing a broad except to catch everything and then not raising it. To make retrying work, I had to catch and raise the IOError specifically. But if the max retries run out, I don't wan't it to raise and fail. Is there a way to do this?

Can we add a default return  value when reach max  retries ?

Hey there @ubehera and @PegasusWang -- I had need of a similar thing and ended up writing #78
If you're still around and have thoughts, it'd be pretty nifty if you wanted to chime in on that PR.

Just in case someone stumbles across this (like I did),  I made a change to tenacity (after seeing rholder/retrying#65) that covers this.
from tenacity import retry, stop_after_attempt

Do nothing (return None):
def do_nothing(eh):
    return

@retry(stop=stop_after_attempt(3), retry_error_callback=do_nothing)
def is_ok_to_fail():
    raise Exception('aw dang')

or, to return a default value (using lambda here because why not)
@retry(stop=stop_after_attempt(3), retry_error_callback=lambda _: True)
def return_true_on_exception():
    raise Exception('aw shucks')


doc: update the example to use lazylock instead of lazy_static

LazyLock was introduced in the latest version of the Rust compiler,
1.80. It’s similar enough to lazy_static that folks still using it will
figure out how to.

coverage: 94.03% (-1.2%) from 95.224%
when pulling 166b0125ffeaba26c881aed5c95344e75650bc0b on rust-180
into 59a8d2a2573317404753f2a94d96c0ea9599627d on master.

As expected, this fails against older rust versions. We could skip one of the examples there, but I’m afraid we will forget to test new examples as we add them. Let’s think more about this.


[DEV-10268] Token update tests

JIRA task: https://huntflow.atlassian.net/browse/DEV-10268
The token update mechanism is covered by tests

You don't see the test_token file.json because it is created in the fixture tests.fixtures.tokens.token_file_storage for the duration of the test and is deleted after the test is completed. It was done this way for the following reasons:

the test must independently create a testing environment for itself
the test should work correctly
after completion, the test should delete all its artifacts. In other words, running tests should not make changes to the project, including leaving new files after its work.

In fact, everything works exactly as specified in the task. The test_token.json file is created. When requests are made via the /token/refresh route, the contents of the file are updated. This behavior is checked in the tests.test_huntflow_token_proxy.test_update_token__ok test.
As for working with real Storage and Proxy classes. Tests really work with them, fixtures return objects of these classes. At a minimum, this approach allows you not to write template code to create test objects


GetPluginFactory not exported

Hello!  I've had no trouble compiling your repo and using the example VSTs.  However, when trying to make my own, using SimpleExample as a guide, I'm unable to get my compiled VST to load.  If I try to load it in VSTHost, nothing happens.  I tried the VST3 SDK validator and received this error:
.\validator.exe MyVst.dll
* Loading module...

        MyVst.dll

Invalid Module!
The dll does not export the required 'GetPluginFactory' function

My project does contain a reference to AudioPlugSharp.dll.  I've also tried simply adding AudioPlugSharp to the solution and then adding a project reference.  Same result.
I followed your instructions on the README... my output directory does contain the bridge, runtimeconfig, deps.json, and Ijwhost.dll.
Any other place I should be looking for how to resolve this?  I'm guessing something is wrong with my build config, but I'm at a loss for where to check.  Thanks!

The dll that actually acts as the plugin from the perspective of the host is the bridge dll, so if you are loading the plugin from a file directly in a host, you should be loading "MyVstBridge.vst3".

That solves it!  Thank you!

Great!