build loadable ext

Address #148
Build example:
cargo build --example hello-ext --features="vtab bundled"
There is one issue when load in duckdb cli:
./build/debug/duckdb -unsigned

D load "/path/to/libhello_ext.dylib";
/path/to/duckdb/src/catalog/catalog_entry/table_function_catalog_entry.cpp:8:79: runtime error: member access within address 0x7ff7bbe317c8 which does not point to an object of type 'duckdb::CreateFunctionInfo'
0x7ff7bbe317c8: note: object is of type 'duckdb::CreateTableFunctionInfo'
 00 00 00 00  58 63 8e 29 01 00 00 00  19 96 04 00 10 61 00 00  00 00 e3 bb f7 7f 00 00  20 17 e3 bb
              ^~~~~~~~~~~~~~~~~~~~~~~
              vptr for 'duckdb::CreateTableFunctionInfo'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /path/to/duckdb/src/catalog/catalog_entry/table_function_catalog_entry.cpp:8:79 in

Seems related to https://stackoverflow.com/a/57304113/1203241 , but haven't figure out the solution.
From https://github.com/Mause/duckdb-deltatable-extension/blob/main/CMakeLists.txt , seems by using build_loadable_extension may work, but not sure if we have a "cleaner" way to achieve this.
@Mause Any suggestion?

If I download the cli from duckdb release page, it works


Unable to run example_scenes.py

Hello guys, I installed all necessary packages and programs in order to run manimlib. In addition, my python version is 3.7.3. Unfortunately when I try to run example_scenes.py it keeps showing this error.
Traceback (most recent call last):
File "D:/Orgil/Installs/manim-master/example_scenes.py", line 3, in
from manimlib.imports import *
File "D:\Orgil\Installs\manim-master\manimlib_init_.py", line 4, in
import manimlib.extract_scene
File "D:\Orgil\Installs\manim-master\manimlib\extract_scene.py", line 9, in
from manimlib.scene.scene import Scene
File "D:\Orgil\Installs\manim-master\manimlib\scene\scene.py", line 12, in
from manimlib.camera.camera import Camera
File "D:\Orgil\Installs\manim-master\manimlib\camera\camera.py", line 9, in
import cairo
File "D:\Orgil\Python\lib\site-packages\cairo_init_.py", line 1, in
from ._cairo import *  # noqa: F401,F403
ImportError: DLL load failed: The specified module could not be found.

You may need to install pycairo separately using a wheel.

Thanks for responding. I fixed my problem by updating my pycairo which is 1.19.1.


Heightmap collision effect is not accurate

Describe the bug
When you drive of the supposed road, you will be either have a slight bump and seems realistic or you car will be going instead of down on the lower height of the road it will go upwards and roll your vehicle over.
To Reproduce
Steps to reproduce the behavior:

try speedrun strategy
Ride the vechile
Go with some speed off the road.

Expected behavior
The car shouldn't roll over but instead go onto the lower side of the road.
Screenshots
See the last 2 attempts to reproduce this problem:
https://user-images.githubusercontent.com/25481501/121784041-24a68c80-cba1-11eb-8bbb-43cfb52e2644.mp4
Desktop (please complete the following information):

OS: Linux
Browser Chrome
Version 90

@drcmda - Pining you as this is most likely got caused with the heightmap Commit to make it more accurate.

the problem is with the original track, the road is as high as the car itself. and lowering it wont work because the dirt track isn't even. a height difference that's too low will also not be reflected in the height map. but something needs to be done for sure.

at least with the new one you cant cheat. :-P

This is what you call effective anti-cheat ;)

Can we add extra friction off the track instead?

not so easy with just a single heightmap i think

The more problem is that how the car is behaving on these heightmaps.
As I can't even normally ride over the train track and instead have to use the ramp. The behavior of this flippings & Rolling etc. Is not really in sync from what should happen.

The train track issue:
https://user-images.githubusercontent.com/25481501/121807457-bf57a780-cc43-11eb-8bde-77c183e8cfde.mp4

that would be expected i think, these are train tracks after all :-D


Timeout issue when start website

I'm getting timeout issue when try npm start
...
[ERROR] Error: Timed out waiting for API structure content from /docs/latest/api/structures/keyboard-event.md
[ERROR] Error: Timed out waiting for API structure content from /docs/latest/api/structures/resolved-host.md
...
How can I fix this issue?

Are you running it on Windows by chance?

Yes, I'm running on windows 11

It is likely an issue with Windows file paths in some of the recent code changes.
Unfortunately Windows is not really a supported platform for this repo. We don't test it on CI or develop locally on it, so it's not a priority to ensure it runs there.
PRs welcome to improve Windows support in the codebase, but I have a feeling this error is just one of several that you'd run into running this repo on Windows.

Closing this now as not planned since Windows isn't really supported.

@KostadinGorgiev - I tried on Win 10 and Win 11 using WSL + Node 20 + Yarn and it worked like a breeze.
Hope it helps anyone using Windows OS and wants to run this code.


Updated copyright/license date to 2015 for new year.

Updated copyright/license date to 2015 for new year.

Thanks for the contribution, but PRs (even these) have to go through the -dev branch. 1.9-dev is the current one :)
I'll just commit this myself later since we have three times the same PR currently in :D


TS9999 ":}" or ":{" expected

Hi, I am new to React-tslint, but I have a problem with tslint, that I can't resolve.
Link to image
Look at the code below:

const Span = styled.span``

const Label = styled.label``

const Checkbox = styled.input`
  display: none;

  & + ${Label} ${Span} {
    display: inline-block;

    width: 18px;
    height: 18px;

    cursor: pointer;
    vertical-align: middle;

    border: 1.5px solid #909294;
    border-radius: 3px;
    background-color: transparent;
  }

  & + ${Label}:hover ${Span} {
    border-color: #4ed8da;
  }

  &:checked + ${Label} ${Span} {
    border-color: #4ed8da;
    background-color: #4ed8da;
    background-image: url('https://image.flaticon.com/icons/png/128/447/447147.png');
    background-repeat: no-repeat;
    background-position: center;
    background-size: 14px;
  }
`

Tslint throws an error:
TS9999:} expected
or, in some places
TS9999:{ expected
Is it problem with tslint or with styled-components?
How to resolve it?
Thanks in advance.
My config:
"typescript": "^2.8.0-dev.20180302",
"tslint": "^5.9.1",
"tslint-react": "^3.5.1",
"tslint-config-prettier": "^1.10.0",
"tslint-config-standard": "^7.0.0",
"tslint-plugin-prettier": "^1.3.0",

This is an error from TypeScript, not from tslint-react. Your code seems fine to me though at a glance, it's possible your editor just got out of sync with your code


fix(api): allow errors in initial home z

We saw an issue on the flex where if the estop was asserted during startup, the axis controllers would swallow move messages instead of sending errors - and if those move messages were homing commands with very long timeouts, it could delay server startup indefinitely.
After https://github.com/Opentrons/ot3-firmware/pull/599 we will instead get an error message when we send a command while estop is enabled. This is better because we can now detect the problem, but an error occuring at this point in initialization would prevent the robot server from starting up, which is also bad. So let's swallow exceptions from motion here so that the robot server doesn't stop working.
Test Plan

Put this and https://github.com/Opentrons/ot3-firmware/pull/599 on a robot
[ ] stop the server; engage estop; start the server; and check that the error is logged but the server starts and nothing is homed

looking good


[templates] PR and issue templates, and move contributing

to: @mime-frame
cc: @airbnb/streamalert-maintainers
size: small
Changes
Add an issue/pr template to this repo.  Also move the CONTRIBUTING.rst file into the .github directory per Github suggestions.

Coverage remained the same at 94.745% when pulling 028e0a4517de2f8455a8d0b9a9baad33d9aeb68e on jacknaglieri-pr-templates into b8e787ba5a0ef92bc3cf2d8206e0214635bab582 on master.


background screen flashes on reload

What version of million are you using?
2.5.13
Are you using an SSR adapter? If so, which one?
Vercel
What package manager are you using?
pnpm
What operating system are you using?
Windows
What browser are you using?
Brave
Describe the Bug
When I open or reload the official million site there is a quick flash of a white background before switching to the dark mode background.
What's the expected result?
I don't expect to see a white background flash before the page switches to a dark background whenever I open the site or reload the page.
Link to Minimal Reproducible Example
https://million.dev/
Participation

[X] I am willing to submit a pull request for this issue.

Created a PR for this issue
https://github.com/aidenybai/million/pull/497

Created a PR for this issue
https://github.com/aidenybai/million/pull/530

Fixed by #530


Permission error in Quick Start (using Codebase) section of README.md in ./createdb.sh

What happened?
When executing that shell file there are many commands which take files from current directory namely updatepg.sql and explorerpg.sql.
Commands are executed as postgres user like sudo -u postgres psql ... -f updatepg.sql. I don't know if it is only me but most probably postgres user will not have access to default /home/user directory and files under that. There is a command given for that chmod -R 775 db/ but probably they missed that permission has no meaning because just parent of db/ wouldn't allow postgres user to access anything. More explanation here.
What did you expect to happen?
So rather shell scripts should consist of commands like psql -U postgres ... -f updatepg.sql this allow files to get accessed by psql and commands is executed as postgres user in database.
How can we reproduce it (as minimally and precisely as possible)?
Simply follow the README.md and make sure to check you already haven't allowed other users to access your home directory.
Anything else we need to know?
No response
OS version

# On Linux:
$ cat /etc/os-release
# paste output here
NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://bugs.archlinux.org/"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo
$ uname -a
# paste output here
Linux amit-dell 6.4.12-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 24 Aug 2023 00:38:14 +0000 x86_64 GNU/Linux
# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Doesn't it sounds like some security issue with psql?

Hi @Amit0617
Thanks for the issue. Will look into it.


Support for scala 2.11

Hi
I just had a look at old issue and there it said you had a release for 2.11. But looking here I can only find max 2.10 releases:
https://repo1.maven.org/maven2/org/xerial/sbt/
Would you consider releasing for 2.11?

Do you want to use Scala 2.11 with sbt-0.13.x? Unfortunately, sbt-0.13.x still needs Scala 2.10.x.
But sbt-sonatype can be used for publishing Scala 2.11 projects with sbt-0.13.x.


render() must be called with a dict, not a Context

render() must be called with a dict, not a Context

What would it take to merge this little PR?


w03h03 bad gradientMagnitude given in testing_build_seams_seamsArray

@felixhauptmann you are the first person I can tag, I can't tag yannic
In SeamCarvingTest.java, starting on line 163, the following is given as gradientMagnitude for the test.
new int[]{
        Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE,
        1, 1, 1, 1, 1, 1,
        69, 420, 42, 42, 42, 1337,
        100, 10, 666, 20, 3, 10,
        Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE
 }

why does this not have Integer.MAX_VALUE on the right and left sides as well?
This causes both testing_build_seams_seamWeights tests to fail for me.
I believe it should look like this:
new int[]{
        Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE,
        Integer.MAX_VALUE, 1, 1, 1, 1, Integer.MAX_VALUE,
        Integer.MAX_VALUE, 420, 42, 42, 42, Integer.MAX_VALUE,
        Integer.MAX_VALUE, 10, 666, 20, 3, Integer.MAX_VALUE,
        Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE, Integer.MAX_VALUE
 }

Have I misunderstood something or shall I adjust the test?

You're right. In practice there should be Integer.MAX_VALUE at all outer edges.
But unless the expected values are wrong or your implementation depends on the outer edges this shouldn't fail.
We could keep this test for strict testing (and annotate it accordingly) and therefore add another test with Integer.MAX_VALUE edges so implementations like yours can be tested as well.
Do you pass all the other tests?

Yes, I passed all the other tests except for the two that did this. As a byproduct of trying to deal with widths below 3, my code now passes these tests too. In a way, the test still helped me :)

Can we close this issue?

sure


Capturing and redirecting stdout/stderr for %%cython-magic

Fixes #3751 as discussed here
Some compilers (MSVC) and linkers write errors to stdout (and not stderr), so we need to catch both and if an error caught, redirect both to sys.stderr.
After fix, for this code:
%%cython
cdef extern from *:
    """
    xxxx a;
    """
    int a;
def doit():
    return a

Shows error:
.../_cython_magic_0c7dc5ea1c624caa7d5b6b78d977eedea31eb934.c:622:5: error: unknown type name ‘xxxx’
     xxxx a;

on Linux and
_cython_magic_18af9232b2d51cf1c8df01fd5af261c66fd7b162.c
...\_cython_magic_18af9232b2d51cf1c8df01fd5af261c66fd7b162.c(610): error C2061: syntax error: identifier 'a'
...\_cython_magic_18af9232b2d51cf1c8df01fd5af261c66fd7b162.c(610): error C2059: syntax error: ';'
...\_cython_magic_18af9232b2d51cf1c8df01fd5af261c66fd7b162.c(1073): error C2065: 'a': undeclared identifier

on Windows (seen in IPython-notebook).

@scoder

I think your example would make a suitable test case for TestIpythonMagic.py.

Done.

@scoder Now it has saner behavior: is read as bytes, and decoded later on - if not successful, the byte object will be printed, as last resort.
I've also added output for successful builds (https://github.com/cython/cython/pull/3872/commits/525e9d8ff7024be158f3ae0402b6ab16a6036ed2#diff-0dcc82cc046ed9866164d504b1e97da9f39f741711977d2837c8e6c4fab259c0R380).
E.g. now
%%cython  -+
def doit():
    pass

will print to stdout(!) on Linux:
Content of stderr:
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++

that is pretty noisy, but on the other hand important warnings would be otherwise lost, as they no longer shown in terminal.

@scoder

Both tests fail in Py3.5/3.6 in C++ mode (see travis). Can't say why. Could also be a Python bug since they work in newer versions. Still worth looking into, I'd say.

It looks as if Py3.5/3.6 are the only ones for which IPython is tested - it doesn't seem to be tested for Py3.7 and higher. C-backends works but not CPP, but I think it is probably just a coincidence.  When I look at failures:
======================================================================
FAIL: test_cython_compile_error_shown (Cython.Build.Tests.TestIpythonMagic.TestIPythonMagic)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/cython/cython/Cython/Build/Tests/TestIpythonMagic.py", line 161, in test_cython_compile_error_shown
    self.assertEqual("error", captured.stderr)
AssertionError: 'error' != 'warning: /home/travis/.cache/ipython/cyth[91 chars]on\n'
- error
+ warning: /home/travis/.cache/ipython/cython/_cython_magic_2dfdf1eaf3a415cd0ba76434dcfb685b35d1c019.pyx:5:9: useless trailing semicolon

======================================================================
FAIL: test_cython_link_error_shown (Cython.Build.Tests.TestIpythonMagic.TestIPythonMagic)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/cython/cython/Cython/Build/Tests/TestIpythonMagic.py", line 167, in test_cython_link_error_shown
    self.assertEqual("error", captured.stderr)
AssertionError: 'error' != ''
- error
+

it looks as if captured_fd doesn't capture anything.  My best guess would be buffering isses, maybe when redirected to a file, the output of pipe isn't flushed to stderr and captured_fd doesn't enforce it?
Locally, I run tests with
python runtests.py -3 --examples-dir=Cython/Build/Tests -vv TestIpython  &>out.txt

and cannot reproduce the problem.

and cannot reproduce the problem.

Sorry for the noise, I forgot to pull your changes. "it looks as if captured_fd doesn't capture anything. "  is solved now, but still don't know what is the original issue.

Hm, now the error is:
ERROR: test_cython_compile_error_shown (Cython.Build.Tests.TestIpythonMagic.TestIPythonMagic)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/cython/cython/Cython/Build/Tests/TestIpythonMagic.py", line 160, in test_cython_compile_error_shown
    ip.run_cell_magic('cython', '-3', compile_error_code)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 2371, in run_cell_magic
    result = fn(*args, **kwargs)
  File "<decorator-gen-864>", line 2, in cython
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/IPython/core/magic.py", line 187, in <lambda>
    call = lambda f, *a, **k: f(*a, **k)
  File "/home/travis/build/cython/cython/Cython/Build/IpythonMagic.py", line 391, in cython
    extension, lib_dir, pgo_step_name='use' if args.pgo else None, quiet=args.quiet)
  File "/home/travis/build/cython/cython/Cython/Build/IpythonMagic.py", line 507, in _build_extension
    build_extension.run()
  File "/opt/python/3.6.7/lib/python3.6/distutils/command/build_ext.py", line 339, in run
    self.build_extensions()
  File "/opt/python/3.6.7/lib/python3.6/distutils/command/build_ext.py", line 448, in build_extensions
    self._build_extensions_serial()
  File "/opt/python/3.6.7/lib/python3.6/distutils/command/build_ext.py", line 473, in _build_extensions_serial
    self.build_extension(ext)
  File "/opt/python/3.6.7/lib/python3.6/distutils/command/build_ext.py", line 533, in build_extension
    depends=ext.depends)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/ccompiler.py", line 92, in <lambda>
    m = lambda self, *args, **kw: func(self, *args, **kw)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/ccompiler.py", line 367, in CCompiler_compile
    single_compile(o)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/ccompiler.py", line 327, in single_compile
    self._compile(obj, src, ext, cc_args, extra_postargs, pp_opts)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/ccompiler.py", line 92, in <lambda>
    m = lambda self, *args, **kw: func(self, *args, **kw)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/unixccompiler.py", line 58, in UnixCCompiler__compile
    extra_postargs, display = display)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/ccompiler.py", line 92, in <lambda>
    m = lambda self, *args, **kw: func(self, *args, **kw)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/ccompiler.py", line 169, in CCompiler_spawn
    forward_bytes_to_stdout(o)
  File "/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/numpy/distutils/exec_command.py", line 102, in forward_bytes_to_stdout
    sys.stdout.write(val.decode(sys.stdout.encoding))
TypeError: decode() argument 1 must be str, not None

which is probably due to capture_fd being called with encoding=None now.
Not sure why numpy's version of distutils is called here.

@scoder
The build still fails, but it seems to be due to unrelated problems.
The issue was, that somehow for Py3.5/3.6+cpp numpy.distutils.exec_command.py was used, which captures the C-level output and prints it to sys.stdout (instead of sys.stderr as we would have done). From that some problems arise:

the error message is printed to sys.stdout instead of sys.stderr, thus the original checks fails.
a version of numpy.distutils.exec_command.py uses encoding from sys.stdout.encoding which is None (because it was replaced through io.StringIO by IPython.utils.io.capture_output) for printing and crashes.

Furthermore, replacing sys.stdout is not that staight forward due to Py2 - Py3 differences (bytes<->unicode), it is also possible that IPython2 replaces the original sys.stdout as well. I found nothing better than to roll out a capture_output-version with io.TextIOWrapper, which should work for Python2.7 and Python3.3+.

@scoder  It works now and tests are green. Do you think something sould be improved here?

Thanks! Let's get this in to see how other users react.


Fix bug with import IDL by absolute path in Windows.

Simple and correct solution for #234.

This solution works. Can we merge this?


Avoid use of LD_LIBRARY_PATH in the development shell

The setting of the LD_LIBRARY_PATH in the development shell is a bit of a hack. With better use of Nix python packaging we imagine we can avoid doing that. The PR It looks like https://github.com/tweag/chainsail/pull/310 has more discussion.

The need for LD_LIBRARY_PATH was eliminated via https://github.com/tweag/chainsail/pull/447.


Nuxt 3 + Auth Module throws 500 Error when configured with https://localhost:3000 and Self Signed Certificate

Hi,
Thanks for your fantastic work !
I am trying to set up an https devServer with this nuxt-auth Auth Module + Nuxt 3, because some OAuth2 providers (like Instagram which I would like) require that you work with https://localhost:3000, even on development mode.
After a fresh install (Nuxt 3 + Auth Module + next-auth + the two pem files), I am receiving the following error when everything is set, when I try to yarn dev the project.
I successfully created two pem files (with mkcert) with a self signed certificate (mkcert CA), and configured the nuxt.config.ts lines as stated in the official nuxt 3 site as described in this page  :
  devServer: {
    https: {
      key: './myserver+3-key.pem',
      cert: './myserver+3.pem',
    }
  }

SSL/TLS Certificates are working correctly
I am sure they do, because, when I add ssr: false on my nuxt.config.ts file, I can correctly render the site with https://localhost:3000
Environment
Nuxi 3.6.3                                                                                15:36:46

Operating System: Linux (Manjaro Linux)
Node Version:     v20.3.0
Nuxt Version:     3.6.3
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      devtools, modules, auth, devServer
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction of the Error
On project directory, type yarn dev and click on > Local:    https://localhost:3000/
Here is a repo to play with : https://github.com/rod2ik/https-auth-nuxt
Describe the bug
Both on the Terminal/IDE (I am using VSCode) and in my Browser (Firefox and Chrome)  I receive the following error when I launch the site via yarn dev
[nuxt] [request error] [unhandled] [500] fetch failed (https://localhost:3000/nuxt_vite_node/manifest)

What is weird is that it works fine (the page https://localhost:3000 is correctly rendered) if I add ssr: false in nuxt.config.ts configurations (uncomnent line 4 of nuxt.config.ts file to test it)

Browser Logs
[nuxt] [request error] [unhandled] [500] fetch failed (https://localhost:3000/__nuxt_vite_node__/manifest)  15:34:26
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  at async $fetchRaw2 (./node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:215:14)
  at async $fetch2 (./node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:239:15)
  at async ./.nuxt/dev/index.mjs:871:20
  at async ./.nuxt/dev/index.mjs:959:64
  at async ./.nuxt/dev/index.mjs:554:22
  at async Object.handler (./node_modules/h3/dist/index.mjs:1284:19)                                        15:34:22
  at async Server.toNodeHandle (./node_modules/h3/dist/index.mjs:1359:7)
[nuxt] [request error] [unhandled] [500] fetch failed (https://localhost:3000/__nuxt_vite_node__/manifest)  15:34:26
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  at async $fetchRaw2 (./node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:215:14)
  at async $fetch2 (./node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:239:15)
  at async ./.nuxt/dev/index.mjs:871:20
  at async ./.nuxt/dev/index.mjs:959:64
  at async ./.nuxt/dev/index.mjs:554:22
  at async Object.handler (./node_modules/h3/dist/index.mjs:1284:19)
  at async toNodeHandle (./node_modules/h3/dist/index.mjs:1359:7)
  at async Object.ufetch [as localFetch] (./node_modules/unenv/runtime/fetch/index.mjs:9:17)
  at async Object.errorhandler [as onError] (./.nuxt/dev/index.mjs:700:30)

Additional Context : h3 Logs
Additional Context : in the async toNodeHandle()  function, on line 1359 of the h3 module, the h3 event is  :
H3Event {                                                                                15:48:29
  __is_event__: true,
  _handled: false,
  context: {},
  node: {
    req: IncomingMessage {
      _events: [Object: null prototype] {},
      _eventsCount: 0,
      _maxListeners: undefined,
      readableEncoding: null,
      readableEnded: true,
      readableFlowing: false,
      readableHighWaterMark: 0,
      readableLength: 0,
      readableObjectMode: false,
      readableAborted: false,
      readableDidRead: false,
      closed: false,
      errored: null,
      readable: false,
      destroyed: false,
      aborted: false,
      httpVersion: '1.1',
      httpVersionMajor: 1,
      httpVersionMinor: 1,
      complete: true,
      headers: [Object],
      trailers: {},
      method: 'GET',
      url: '/__nuxt_error?url=/&statusCode=500&statusMessage&message=fetch+failed+(https://localhost:3000/__nuxt_vite_node__/manifest)&stack=%3Cpre%3E%3Cspan+class=%22stack+internal%22%3Eat+process.processTicksAndRejections+(node:internal/process/task_queues:95:5)%3C/span%3E%0A%3Cspan+class=%22stack+internal%22%3Eat+async+$fetchRaw2+(./node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:215:14)%3C/span%3E%0A%3Cspan+class=%22stack+internal%22%3Eat+async+$fetch2+(./node_modules/ofetch/dist/shared/ofetch.d438bb6f.mjs:239:15)%3C/span%3E%0A%3Cspan+class=%22stack%22%3Eat+async+./.nuxt/dev/index.mjs:1597:20%3C/span%3E%0A%3Cspan+class=%22stack%22%3Eat+async+./.nuxt/dev/index.mjs:1685:64%3C/span%3E%0A%3Cspan+class=%22stack%22%3Eat+async+./.nuxt/dev/index.mjs:661:22%3C/span%3E%0A%3Cspan+class=%22stack+internal%22%3Eat+async+Object.handler+(./node_modules/h3/dist/index.mjs:1284:19)%3C/span%3E%0A%3Cspan+class=%22stack+internal%22%3Eat+async+Server.toNodeHandle+(./node_modules/h3/dist/index.mjs:1360:7)%3C/span%3E%3C/pre%3E',
      statusCode: 200,
      statusMessage: '',
      connection: [Socket],
      socket: [Socket],
      body: null,
      __unenv__: undefined,
      [Symbol(kCapture)]: false
    },
    res: ServerResponse {
      _events: [Object: null prototype] {},
      _eventsCount: 0,
      _maxListeners: undefined,
      writable: true,
      writableEnded: false,
      writableFinished: false,
      writableHighWaterMark: 0,
      writableLength: 0,
      writableObjectMode: false,
      writableCorked: 0,
      closed: false,
      errored: null,
      writableNeedDrain: false,
      destroyed: false,
      _encoding: 'utf-8',
      statusCode: 200,
      statusMessage: '',
      upgrading: false,
      chunkedEncoding: false,
      shouldKeepAlive: false,
      useChunkedEncodingByDefault: false,
      sendDate: false,
      finished: false,
      headersSent: false,
      strictContentLength: false,
      connection: null,
      socket: null,
      _headers: {},
      req: [IncomingMessage],
      [Symbol(kCapture)]: false
    }
  }
}

Same problem.
The getUrl method of requrl does not get from the request that it is https.
https://github.com/sidebase/nuxt-auth/blob/45d7e349ee0bb8791afdba69c7c35f39c9fe5a75/src/runtime/composables/authjs/useAuth.ts#L163
https://github.com/sidebase/nuxt-auth/blob/45d7e349ee0bb8791afdba69c7c35f39c9fe5a75/src/runtime/utils/url.ts#L7

Actually, after testing, I think this is more like a Nuxt 3 + https devServer (https://localhost:3000) problem.
I am opening an issue an the nuxt/nuxt repository.

any plan to fix soon? or workarond? thanks

Bump... 🙏

Thanks to this comment https://github.com/sidebase/nuxt-auth/issues/374#issuecomment-1559411256
I fixed the issue for myself - the auth.origin has to be auth.baseURL instead in your nuxt config 😑 The docs are still not updated since May (they have been updated in 0.6 beta)

Hello everyone!
I assume this issue has been resolved. I will once again link the correct docs for Version 0.6 and above here: https://sidebase.io/nuxt-auth/v0.6/configuration/nuxt-config


KeyGenerator from string

While explaining the metadata / mosaic transactions, I found challenging to follow the examples if we have to refer to the key as random UInt64 value that the dev has to store its meaning in a separate database.
To permit the dev to use NEM as a fully decentralized service without having to store data somewhere else and enhance the usability in the SDK, we could provide with a KeyGenerator that takes a string as an input and returns a deterministic UInt64 object.
You can see how this is possible currently using the NamespaceIdGenerator (https://github.com/nemtech/nem2-docs/blob/master/source/resources/examples/typescript/restriction/RestrictingMosaicsTransfersMosaicGlobalRestriction.ts#L42).
In the docs, we can explain more about the maximum value the string can have to be translated to a valid UInt64 value, as well as if can contain special chars or not.

See https://github.com/nemtech/nem2-sdk-typescript-javascript/pull/288

@contractormario Thanks!


AZURE_* environment variables

Please consider supporting the AZURE_* environment variables like the azure CLI, especially:

AZURE_STORAGE_ACCOUNT
AZURE_STORAGE_KEY

For testing it would also be useful to be able to configure AZURE_STORAGE_ENDPOINT (not currently supported by the CLI, as far as I can tell).

Cpplite is different from Azure-CLI, which itself is an executable. But cpplite is a library. I'm not sure whether it's a good idea to handle these environment variables in a library. Maybe these account name/key and endpoint information should loaded from environment variables by the executable and passed to this library. Anyway, we will evaluate this feature request.

Maybe these account name/key and endpoint information should loaded from environment variables by the executable and passed to this library.

Yes, that is what we are doing for now. Just to note that the AWS S3 C++ SDK does fall back to environment variables, which is useful for several situations. Thanks for evaluating this feature!

@ihnorton Thanks for your feedback, we'll add this feature to our backlog, and maybe support it in future release.


on_conflict_do_update and on_conflict_do_nothing returns None it typing, but actually returns 'Insert'

Describe the bug
Using on_conflict_do_update method with  returning and PyCharm  highlight return with "Cannot find reference 'returning' in 'None'"
Expected behavior
Don't see any highlight.
To Reproduce
Install packages, and try to use
insert(self.table).on_conflict_do_update(
      index_elements=(self.table.c.id,),
      set_={
          'count': self.table.c.count + insert_query.excluded.count,
      },
  ).returning(self.table.c.id)

Error
"Cannot find reference 'returning' in 'None'"

Proposition
Change:
def on_conflict_do_update(
        self,
        constraint: Optional[Any] = ...,
        index_elements: Optional[Any] = ...,
        index_where: Optional[Any] = ...,
        set_: Optional[Any] = ...,
        where: Optional[Any] = ...,
    ) -> 'Insert': ...
    def on_conflict_do_nothing(
        self,
        constraint: Optional[Any] = ...,
        index_elements: Optional[Any] = ...,
        index_where: Optional[Any] = ...,
    ) -> 'Insert': ...

Versions.

OS: Ubuntu 20.4
IDE: PyCharm Professional 2022.1
Python: 3.8
SQLAlchemy: 1.4.32
mypy: 0.782
SQLAlchemy2-stubs: 0.0.2a21

Have a nice day!

Hi, thanks for reporting.
Current effort is focused on typing the library for v2, but if can provide a pr we can merge and release

Can you give me contribution access? I cannot push and make PR

The usual way if to fork a repository, make changes there then create a PR

released as v0.0.2a22


[Doc modernization][H1 and H2 fix]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/overview.md
93
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Glance

Since the latest beta update the order of Glance doesnt work. I use a custom note (a greeting) and I place it below musicplayer... I allways want the current som showing up. After closing settings the order falls back and my custom note shows up even when using Spotify. If I turn of custom notes, it works.
To Reproduce
Steps to reproduce the behavior:

Go to settings
Click on Glace
Scroll down to the different options and set the order. Musicplayer on top
Closing settings and nothing changes... in settings its back as "standard setting" (the order)

Smartphone (please complete the following information):

Device: Galaxy S10 Plus
OS Version: Android 10.1
App Version 2.0.15 (108)

Aditional... the permissions are correct


Unable to authenticate Desktop App through gitlab OAUTH2 Login since 9.1.x

Summary
Since upgrade to Server 9.1.0 (and 9.1.1) it does no longer work to log in trough gitlab authentication (oauth2) to the desktop app.
After downgrade to 9.0.0 it works again.
Steps to reproduce

Self-hosted Mattermost Server behind nginx reverse proxy on /subpath
Open Windows Desktop app (5.4.0, 5.5.0 or 5.5.1 - makes no difference)
Ensure to be logged out of Mattermost
Klick "gitlab" on mattermost login page and notice difference in behaviour:

-> In 9.1.x The gitlab login page opens in an external browser window,
-> In 9.0.0 The gitlab login page opens inside the desktop app

After logging in to gitl gitlab, offers the possibility to continue in the browser (wich works) or in the desktop app trough the mattermost: pseudo-protocol (mattermost://subdomain.domain.tld/chat/login/desktop?client_token=xxxx&server_token=yyyy)
Klick Desktop App button

Expected behavior
Mattermost desktop app should be logged in and display the chat-channel
Observed behavior (that appears unintentional)
The "Channels" tab is being shown with white background and a gray hyperlink to subdomain.domain.tld/subpath, after reload with Ctrl+R the mattermost login page with username/password and the "gitlab" button (same as in Step 4) is being displayed.
Possible fixes
Workaround: Downgraded Server to 9.0.0
I could not find any dubious requests in the Mattermost Developer Tools or nginx reverse-proxy except some 401 errors like the request for
https://subdomain.domain.tld/subpath/api/v4/cloud/subscription/self-serve-status

@wech71 Can you share which OS you are on? If it's Linux, can you share the distribution as well?

Of course.
It's a linux ubuntu 20.04.6 (focal) server.

@wech71 Yes I meant the Desktop OS, all good :)
So what's supposed to happen is after you install the Desktop App it should register a protocol handler that accepts mattermost:// links.
So your users should see a prompt that has them open the Desktop App from their browser. Is that not happening?

Yes, the protocol handler seems fine. When i click the mattermost:// link after logging in in the browser, the mattermost app reacts to the click, but I'm not logged in as expected and only see a white background (with a slight shadow of the mattermost-logo on it) and containing the gray text "subdomain.domain.tld/chat"  ("chat" being the subpath).
When I press Ctrl+R the mattermost login page (with the "gitlab" buttton) appears again.

Yes, the protocol handler seems fine. When i click the mattermost:// link after logging in in the browser, the mattermost app reacts to the click, but I'm not logged in as expected and only see a white background (with a slight shadow of the mattermost-logo on it) and containing the gray text "subdomain.domain.tld/chat" ("chat" being the subpath).
When I press Ctrl+R the mattermost login page (with the "gitlab" buttton) appears again.

Are you able to provide logs from the Dev Tools? You can go to View > Developer Tools for Current Server and something should show up there. Looks like something is happening when the navigation happens.

 Did you mean /login/desktop with the client and server token request params?
Both of them have "preserve log" and "disable cache" activated.
After clicking "Authorize" in gitlab in the browser, I can see the following requests:
@Browser:
https://subdomain.domain.com/code/oauth/authorize
-> sets no cookies
https://subdomain.domain.com/chat/signup/gitlab/complete?code=15fxxxxxxxxxxxxxxxxxxxxxxxxxxxxx6d&state=eyJxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx9
set-cookie "MMOAUTH=; Path=/chat; Max-Age=0; HttpOnly"
set-cookie "MMAUTHTOKEN=ktxxxxxxxxxxxxxxx; Path=/chat; Expires=Fri, 01 Dec 2023 06:34:39 GMT; Max-Age=2592000; HttpOnly; Secure"
set-cookie "MMUSERID=upxxxxxxxxxxxxxxxxxxxxxxxxxxxxx; Path=/chat; Expires=Fri, 01 Dec 2023 06:34:39 GMT; Max-Age=2592000; Secure"
set-cookie "MMCSRF=wn7xxxxxxxxxxxxxx; Path=/chat; Expires=Fri, 01 Dec 2023 06:34:39 GMT; Max-Age=2592000; Secure"
https://subdomain.domain.com/chat/login/desktop?client_token=a6cxxxxxxxxxxxxxxxxxxxxxxxxxxxxx07b54f&server_token=mgxxxxxxxxxxxxxxxxxxxxxxxxxxxxx15ih
-> sets no cookies
I set a breakpoint to forwardToDesktopApp  and it  sets window.location to
mattermost://subdomain.domain.com/chat/login/desktop?client_token=a6cxxxxxxxxxxxxxxxxxxxxxxxxxxxxx07b54f&server_token=mgxxxxxxxxxxxxxxxxxxxxxxxxxxxxx15ih'
This then triggers the desktop app where the following requests happen:
@mattermost-desktop-app
https://subdomain.domain.com/chat/api/v4/teams?page=0&per_page=200&include_total_count=false&exclude_policy_constrained=false
401 -> sets no cookies
https://subdomain.domain.com/chat/api/v4/cloud/subscription/self-serve-status
401 -> sets no cookies
All three request use the following cookies in the requests:
rl_anonymous_id   %22d00xxxxxx-xxxx-xxxx-xxxx-9axxxxxxf6ae%22
rl_user_id	  %22%22
preferred_language de
known_sign_in	  WHdsxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxbmc9PQ%3D%3D--409xxxxxxxxxxxxxxxxxxxxxdb
As far as I understand, the mattermost: protocol runs another instance of the mattermost.exe with the specified url, but as this would be a different instance of the mattermost.exe, so I was thinking that the cookies are being set in the wrong instance. So I tried to set window.location in the running mattermost desktop app developers console to the mattermost:-Link, but this also does not produce any requests at all in the network tab, but setting it to https://subdomain.domain.com/chat/ instead does, so it works.
In fact setting it to anything 'https://subdomain.domain.com/chat/login[?/].*' does not trigger any reaction.
All test where made with Mattermost Desktop App 5.4.0 on Windows Terminal Server 2016

Thanks for the detail :)
So in the second instance case, we should be seeing something in the logs when it opens, see the function that does the second instance check here:
export function handleAppSecondInstance(event: Event, argv: string[]) {
    log.debug('handleAppSecondInstance', argv);

    // Protocol handler for win32
    // argv: An array of the second instance’s (command line / deep linked) arguments
    MainWindow.show();
    const deeplinkingURL = getDeeplinkingURL(argv);
    if (deeplinkingURL) {
        openDeepLink(deeplinkingURL);
    }
}

This is what should be handling the deep linking.  Have you tried using v5.5.1 Desktop App?

How can I debug the second app (the one started through mattermost:-Protocol)? - it immediately exits after processing the mattermost:// link, and I have no time to open the developer tools an this second instance. And running mattermost.exe without parameter also just activates the first (already running) instance and exists immediately.
I also just tried it wit the v5.5.1 app - unfortunately the problem persists.

@wech71 So it seems like it's working as intended as the second instance is popping up and doing something.
This is more than likely something on your server-side at this point, so I'm going to send this back over to the server repo to be investigated there.


irmin-pack: no mmap syscall

Since the mapping file is read-only, we can simulate the mmap in userland by lazily reading chunks on demand (and avoid the file descriptor leak from mmaped bigarrays..) There are a bunch of optimizations that we could try here, but I'm curious how the benchmarks are impacted by this change before going too deep into this rabbit hole :)
(also added the expected mapping_size in the control file as a consistency check to detect a partially written mapping, after a gc or a snapshot)

I still need to take a close look at the code, but as discussed earlier, I ran some preliminary benchmarks locally based on the benchmark I did last week. Early results are very promising! :heart_eyes_cat: Below is a graph of no mmap (this PR) vs mmap (main, essentially). The title says "commit load times" but this is really "sample commits and load their entire trees"; it doesn't take this long to just "load a commit" :smile:.
Based on these results, I definitely think it is worth doing a full replay trace benchmark (and potentially looking at the other optimization ideas you have!).

Results of a benchmark run using a Tezos Context replay: https://github.com/tarides/irmin-tezos-benchmarking/blob/main/benchmarks/2023-04-no-mmap/stats.txt

Thanks a lot for your benchmarks and feedbacks! <3
@adatario > If you have some time, I think there's a small glitch with the memory usage graph as it's a bit flat during GC.. I'm guessing it's because the GC is running in a child process and doesn't get measured by smem (?)

@adatario > If you have some time, I think there's a small glitch with the memory usage graph as it's a bit flat during GC.. I'm guessing it's because the GC is running in a child process and doesn't get measured by smem (?)

Here's a fixed graph:

@adatario @art-w what do you think explains the higher memory during GC (but not otherwise)?

@art-w here is an updated bench with the "no bigarray" version. For now, lets stick with bigarray. :sunglasses:

Thanks a lot @adatario !!

what do you think explains the higher memory during GC (but not otherwise)?

Yeah sorry I should have mentioned that I expected the memory usage to grow slightly during GC following PR #2215 (the current PR is unrelated). Previously an array was used to represent the Live ranges of the mapping file during GC, and this old PR switched to a list (same quantity of elements, but about 2/3 larger in memory used). I'll push a tighter representation of the Live ranges soon :)

No need, I managed to run it! (but thanks!)
It's not yet as good as before hm:


Solve deprecation warning

/home/daniel/workspace/envs/bmw/lib/python3.5/site-packages/sphinx/application.py:445: RemovedInSphinx17Warning: app.status_iterator() is now deprecated. Use sphinx.util.status_iterator() instead.
RemovedInSphinx17Warning)
/home/daniel/workspace/envs/bmw/lib/python3.5/site-packages/sphinx/application.py:426: RemovedInSphinx17Warning: app._display_chunk() is now deprecated. Use sphinx.util.display_chunk() instead.
RemovedInSphinx17Warning)

Usage of deprecated status_iterator got solved.
But following warning is coming from plantuml diagram:
RemovedInSphinx17Warning: sphinx.util.compat.Directive is deprecated and will be removed in Sphinx 1.7


PSM Connection Fails to Audit Windows Session When Onboarded Automatically via Lambda

After reviewing the code, it is not apparent anywhere that $env:COMPUTERNAME associated with the Windows EC2 instance is being captured.  This will cause issues when attempting to RDP using the WinServerLocal platform as LogonTo should be given the AWS EC2 instance's HOSTNAME in order to properly copy over the audit agent needed for Windows Session Event capture.
Including the capture of the Windows internal hostname and populating in EPV with the account object being onboarded will prevent a lot of hair-pulling from customers utilizing this Lambda functionality.

Hi @infamousjoeg ,
An internal ticket has been created for this issue (CLI-576).
With best regards,
Ziv


Remove optics from the sbt-common classpath

What is this PR trying to achieve?
Per the commit messages:

Move SierraRecord into the sierra_adapter common lib
This model is internal to the Sierra adapter pipeline -- by the time it reaches the VHS, it's been coerced into a BibRecord or an ItemRecord. It doesn't need to be available to the transformer or other applications.

Move optics out of the sbt-common classpath
Now that SierraRecord is out of sbt-common, we don't need optics everywhere.  Since it brings in a dependency on Scalaz and cats, there's likely a small benefit to getting it out of some of our applications.

Will this make a massive difference to build times? Unlikely. But it continues to reduce the footprint of sbt-common.
Who is this change for?
🚢 🤖

Okay, maybe circe depends on scalaz and cats in other ways. Still good tho!


I want show page for has_many association images as thumbnail.

ruby 2.5.1p57
rails <IP_ADDRESS>
rails_admin 1.4.3
carrierwave 1.3.1
I already found similar gem https://github.com/colavitam/rails_admin_grid. but that gem outdated and not maintained now. even not working anyway. but actually that gem support for list page but i want show page has_many images same way like that.

i also found rails_admin config way like this but not working into the partial page.
models

class PhotoAlbum < ActiveRecord::Base
  has_many :attachments

  def create_associated_image(image)
    attachments.create(image: image)
  end
end

class Attachment < ActiveRecord::Base
  mount_uploader :image, AttachmentUploader
  belongs_to :photo_album
end

rails_admin config
  config.model PhotoAlbum do
    show do
      field :attachments do
        render do
          bindings[:view].render partial: '_image_preview', locals: {object: self}
        end
      end
    end
  end

app/views/rails_admin/main/_image_preview.html.erb

what exactly code insert here for image show thumbnail?

rails_admin loading do not check that path.
can you find any problem?

I found solution now inject to here. but That is band-aid patch. I need more refactoring code.
module RailsAdmin
  module Config
    module Fields
      class Association < RailsAdmin::Config::Fields::Base
        def self.inherited(klass)
          super(klass)
        end

        # Reader for the association information hash
        def association
          @properties
        end

        register_instance_option :pretty_value do
          v = bindings[:view]
          [value].flatten.select(&:present?).collect do |associated|
            amc = polymorphic? ? RailsAdmin.config(associated) : associated_model_config # perf optimization for non-polymorphic associations
            am = amc.abstract_model
            wording = associated.send(amc.object_label_method)
            # monkey patch here only for check model name...
            if am.model_name == 'Attachment'
              v.link_to(v.image_tag(associated.image.url, class: 'img-thumbnail', size: '200x200'), v.url_for(action: show_action.action_name, model_name: am.to_param, id: associated.id), class: 'pjax', target: '_blank')
            else
              ERB::Util.html_escape(wording)
            end
          end.to_sentence.html_safe
        end
...


Option to have a curved line

I would love to have the option to have a curved chart like https://github.com/pichukov/LightChart

Would be great! I am not good enough with mathematics do to that 🤦🏻♂
Maybe anyone can help?

@amankumar303 Sure, it would be great. Thank you!


Add browsers support

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[ ] description of changes is added in CHANGELOG.md
[ ] update .d.ts typings

Landed in https://github.com/metarhia/metautil/commit/f84ce4bf8bd669097fe29efd37aa5161e1a183dc and https://github.com/metarhia/metautil/commit/c50053c1f69947dd8ecfdc32e87881bce309ae8d


Add SELF reference

As discussed in https://github.com/opentracing/specification/issues/81

Coverage increased (+0.6%) to 79.152% when pulling 5da56c899eb5934d2a16a28cd53274d35632b581 on felixbarny:self-reference into cdc9601709dee7332d5615369901f253645e4ee1 on opentracing:v0.31.0.

I think this change revealed an issue with the SELF reference - one cannot create a proper parent reference unless the parent ID is propagated on the wire, which many systems do not do, and it's not considered as part of https://github.com/TraceContext/tracecontext-spec

Could you explaint that a bit more? If the parent id is not propagated how would you then create a parent reference? How is this any different when the tracer impl creates the SpanContext?
For me, it sounds like TraceContext does include the parentId: https://github.com/TraceContext/tracecontext-spec/blob/master/trace_context/HTTP_HEADER_FORMAT.md#span-id

I can explain using Jaeger spans as example. When Jaeger span is created, we always record in it trace ID, (new) span ID, and the ID of the parent span.  Let's give some values: traceID=15, spanID=5, parentID=1. When we serialize that span into an RPC request, we always include traceID and spanID (Jaeger also happens to include parentID, but that's legacy behavior we can ignore for this discussion). So the server that receives the request gets traceID=15, spanID=5. The server creates a new span as ChildOf, with a new random span ID (let's say 7):

context extracted from the wire: traceID=15, spanID=5
new span's context: traceID=15, spanID=7, parentID=5

Note the parentID=5 part, it establishes the link to the span from the caller service. However, if we serialize that context, we'd only get traceID=15, spanID=7, so if you try to use that as a SELF reference, you're missing the ID of the parent span (5), and the linkage is broken.
Again, in Jaeger today it so happens that SELF will work because we actually send parentID over the wire too, but not if we were to use the TraceContext spec, which only sends two IDs, traceID and spanID.
It also won't work with the MockTracer in your current PR - you added parentID to the context, but not to the serialization format, so if you marshal and unmarshal the context and create a SELF span, it will be missing the parent.

so if you marshal and unmarshal the context and create a SELF span, it will be missing the parent

That's not the quite use case for the SELF reference but rather ETL. For example if you are given a JSON document of a span which is in a specific format and you want to create a Java representation from it. So serializing aka. Tracer#inject the span context does not matter for this use case, only deserializing aka. Tracer#extract.
But indeed, when using a TraceContext io.opentracing.propagation.Format, you can't construct the trace context even if you have the parentID as associated extractor does not read the parentID. But even when using TraceContext as the main propagation format you could still extract the SpanContext using the B3 format, for example. Another alternative is to implement a custom Format and extractor which is specific to the format you want to read and the tracer you want to support.

I am not sure why it didn't occur to me earlier, but I think there is a simple solution here. For tracers like MockTracer that do not store parent span id in the SpanContext (and by extension for any generic instrumentation using SELF reference), one just needs to construct another instance of SpanContext and pass it as  CHILD_OF.
For example, assume we want to create SELF span with traceID=1, spanID=2, parentID=1 (using Python for shorter syntax):
tracer.start_span(
    "operation name",
    references=[
      opentracing.self(createVendorContext(traceID=1, spanID=2)),
      opentracing.child_of(createVendorContext(traceID=1, spanID=1)),
    ],
)

I have incorporated @yurishkuro's suggestions and rebased against master.


OCTO mode for single region

Hello,
Again, thanks for this wonderful work.
I made modifications asked in my previous PR, refactoring octo configuration, and adding decription in Readme.
I also added a new boolean (described in Readme) to expose metrics to Prometheus using annotations.
Sincerely yours

Thanks for all !


Add exclude tags functionality

Add exclude tags functionality
Changes

Added excludedTags array to settings
Updated settings tab UI to allow comma-separated input of excluded tags
Added new filter function to exclude files with specified tags

Testing

Tested with multiple excluded tags
Verified compatibility with existing tag filter

Screenshots

Looks good. Thanks for improvement!


Add John Walicki to MAINTAINERS.md for Docs WG and all Docs repos

@dabooz @t-fine @naphelps @bencourliss per the expedited process we've discussed on TSC calls, I'd like to formally nominate @johnwalicki to be promoted from Contributor to Maintainer in the Documentation Worrking Group repositories.
Maintainers are described as "Contributors who have commit access to repositories, may fill leadership roles in Working Groups, and have demonstrated positive contributions over time to further the goals of the project."
John has demonstrated both the knowledge and ability to handle the technical aspects of reviewing commits, managing tasks within the Working Group, and has shown leadership abilities in taking large changes from concept through to completion/merge.
To cement this, I'll submit a PR adding his name to the MAINTAINERS.md file in this repo, and tag you four as reviewers.  You can then give feedback as needed in the PR review process.

Thanks!
With gratitude and readiness to serve.
-- John


Changed try/except in local control calls to catch all

Since the local control API isn't officially documented there is know way to know what might happen when trying to send requests to the hub. I ran into another issue today where requests were not being sent because the connection was being refused. This didn't happen for long, but does appear to happen every now and then. This change will catch all errors during the local control calls and if anything fails, forward that request online.

Coverage remained the same at 94.818% when pulling c902fb5c9160ab8e32d3f29f918f235d464bca56 on w1ll1am23:master into 57517571fb456a167db42a854cf46b450b279407 on python-wink:master.


MRT tends to deteriorate the performance while fine tuning a pre-trained Transformer.

I suppose this phenomenon reasonable since I have not found records with MRT claiming better performance than Transformers' with MLE. Do you have any empirical evidence corresponding to this? Otherwise I would double check my implementation. XD

I have seen a report that MRT outperforms MLE on Transformer. However, I cannot find that paper now.
In my attempt to applying MRT on Transformer. The BLEU score slightly increase first (+0.5 BLEU score) and then decreases significantly. I am not sure my implementation is correct either. MRT is very sensitive to learning rate. Tuning the learning rate for MRT is highly recommended.
To conclude, the usefulness of MRT on Transformer is still in doubt. There are reports that MRT is useful on Transformer, but we cannot get the same result with our implementation.

Thank you.


Fix segfault that occurs when mappedOps does not contain op during unpatch

close pr

delete pr

I assume you wanted to close this PR here.


Utilize .type

At the moment, the types for the results are hard coded. This may results in issues thus I am proposing to use the .type from youtubei.js to check for types first.

Implemented!


additions to template

add docker-compose changes to https://github.com/UCSB-PSTAT/devcontainer-template

Hey @392781 I think you might have opened this issue on the wrong repo. :)
If not, can we at least add what packages/changes are needed for the comm160ds image?

Project interface bugged out on me earlier this week... I wouldn't be surprised if I opened more issues in unrelated repos :^(


Add ChatClient.Builder auto-configuraiton support

add ChatClinetCustomizer support.
add enable/disable property - enabled by default.
add ITs.

squashed and merged at 0571aac44c2a99be439dac371c022c706bfe52dc


chore(ci): get masked license for each integration test

Codecov Report
Base: 8.30% // Head: 8.30% // No change to project coverage :thumbsup:

Coverage data is based on head (9866afc) compared to base (96caec0).
 Consider uploading reports for the commit 9c936b9 to get more accurate results

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #439   +/-   ##
=====================================
  Coverage   8.30%   8.30%
=====================================
  Files          5       5
  Lines        855     855
=====================================
  Hits          71      71
  Misses       779     779
  Partials       5       5

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix for #69

Fixed peer dependency of vuex to allow any version of vuex >= 2.0.0

Thanks @jrast! Could you make this ^2.0.0 || ^3.0.0 instead though? That's somewhat more future proof in case of a non backwards compatible 4.0.0 release.


Connection dropping

Hello!  For some reason, connection  has started dropping a couple of times throughout the drive.  Was previously pretty solid, so not sure what changed.  I'm using  Android 12 and all battery restrictions are off, for the apps and the phone.  Not sure if this has anything to do with it, but noticed the message about being paused on the Aaidrive screen:

While AAIdrive is running in the background, it checks that it hasn't been paused or been killed unexpectedly. In the Background Pause case, it runs a timer every 4 seconds and will raise that alert if the timer hasn't fired for 10 seconds.
When the app connection breaks, please check the car's Apps checkbox to confirm if the Apps tunnel is still connected. In my experience, the connection usually breaks if the MyBMW/Mini app is shut down or paused, and the car will disconnect after 10 seconds if its keepalive messages are not handled.
Some phone brands are very aggressive about pausing and shutting down programs in the background. Please check for brand-specific battery optimization settings, if the standard Android settings are already configured.

I'm not sure if it has any relation to your expierence or at with BMW and AAID, but what I noticed of few weeks during a road trip, that the sound spontanously (about every 20 minutes) stopped even though all connections were still active and on iDrive Display music was also playing just no sound via Bluetooth. Now, prior I enabled the setting of TomTom AmiGo Navi to auto-start when a bluetooth connection is established using the app for navigation this time looks nicer and shows all speedtraps, speed restrictions and so on. One solution was reconnecting USB and Bluetooth. We had the same problem in the drive home, I couldn't figure it out but when I at the gas station came up to disable the auto-start of the Navi app upon Bluetooth connection and never had the Problem on the drive. Last week on another trip I used the same app without auto-start, no problems with sound muting.
My guess could be, if you have any App enabled to auto-start upon Bluetooth connection could cause the sound muting. Though in my case connection itself was never lost, there's just no sound.


Closing notebook during startup can cause infinite loop setting preferred kernel

I opened vs code with several notebooks open, and started closing them while the extension was still activating.
The logs show that there is some code stuck in a loop. Not much of a usability impact, but really blows up the logs
verbose 15:53:55.651: Jupyter Extension: Notebook document c:\src\test\3\otherlanguages\julia.ipynb got closed
extensionHostProcess.js:144
verbose 15:53:55.703: Jupyter Extension: Preferred Remote kernel for c:\src\test\3\otherlanguages\julia.ipynb is undefined
extensionHostProcess.js:144
verbose 15:53:55.743: Jupyter Extension: Notebook document c:\src\test\3\otherlanguages\julia.ipynb got closed
extensionHostProcess.js:144
verbose 15:53:55.818: Jupyter Extension: Preferred Remote kernel for c:\src\test\3\otherlanguages\julia.ipynb is undefined
extensionHostProcess.js:144
verbose 15:53:55.865: Jupyter Extension: Notebook document c:\src\test\3\otherlanguages\julia.ipynb got closed
extensionHostProcess.js:144
verbose 15:53:55.946: Jupyter Extension: Preferred Remote kernel for c:\src\test\3\otherlanguages\julia.ipynb is undefined
extensionHostProcess.js:144
verbose 15:53:55.973: Jupyter Extension: Notebook document c:\src\test\3\otherlanguages\julia.ipynb got closed
extensionHostProcess.js:144
verbose 15:53:56.065: Jupyter Extension: Preferred Remote kernel for c:\src\test\3\otherlanguages\julia.ipynb is undefined
extensionHostProcess.js:144
verbose 15:53:56.272: Jupyter Extension: Notebook document c:\src\test\3\otherlanguages\julia.ipynb got closed
extensionHostProcess.js:144
verbose 15:53:56.417: Jupyter Extension: Preferred Remote kernel for c:\src\test\3\otherlanguages\julia.ipynb is undefined
extensionHostProcess.js:144
verbose 15:53:56.476: Jupyter Extension: Notebook document c:\src\test\3\otherlanguages\julia.ipynb got closed

This looks like a bug in VS Code,
This is the only place we log when a notebook is closed
        workspace.onDidCloseNotebookDocument(
            (e) => {
                traceVerbose(`Notebook document ${getDisplayPath(e.uri)} got closed`);
                this.disposeOldKernel(e.uri);
            },
            this,
            disposables
        );

Given this event can only be fired when a notebook is closed, it looks like VS Code code into a state where the event was fired more than once and leading to this crazy loop.
@rebornix any idea what could cause this?
I'm assuming its a flaky issue.
@amunger Please can you try to replicate this again, I cannot replicate this.

I only hit it once, haven't seen it since

Closing this for now, as there's really nothing in jupyter extension that we can to do address this.
Must have been a flaky vscode issue.


Read Only Property

We need to add property of read only for some text boxes.

TextFields and Comboboxes on the Profile are disabled.


[5.5] Make Collection::dd() & Collection::dump() show the same result

If you do
collect([1, 3])->dump(5);

echo "------------------------------------------\n";

collect([1, 3])->dd(5);

You'll see dd() and dump() shows a completely different result:
5
Illuminate\Support\Collection {#783
  #items: array:2 [
    0 => 1
    1 => 3
  ]
}
------------------------------------------
array:2 [
  0 => 1
  1 => 3
]

This PR fix this so they both behave the same way:
5
Illuminate\Support\Collection {#783
  #items: array:2 [
    0 => 1
    1 => 3
  ]
}
------------------------------------------
5
Illuminate\Support\Collection {#783
  #items: array:2 [
    0 => 1
    1 => 3
  ]
}

This is possibly a breaking change? Maybe best to target 5.6

dd() is the last frontier, since there's nothing after it, it cannot break our apps...

I think it's a great PR.
Technically @GrahamCampbell could be correct though.
If someone was running an API and returning the output of a HTTP call via dd($var) - this would change the output of the API result?
I know no one should be doing that - but I've seen this type of silly stuff in people's questions/answers on StackoverFlow, reddit etc - so it is occurring out there.
The question is, because they are not using it in the intended manner, do we hold off on a PR for people doing it the "wrong" way... I dont mind either way - just putting it out there.


cmd/go: get -u pulls in unreferenced v2 version

As popular Go modules tend to spread, I think this problem is widespread (people having problems doing go get -u on their Go projects) (see https://github.com/spf13/cobra/issues/805 https://github.com/gohugoio/hugo/issues/5954).
 Is it that after running go mod graph again, you get a blackfriday@v2 ?
If that is so, I am unable to reproduce it. For me, it's always that one line of 1.5.2. I am on 1.12.4 though.

What is the exact issue ? Is it that after running go mod graph again, you get a blackfriday@v2 ?

Yes.  I added a newline to my initial comment (seems GitHub hid the important "go get -u" step).
I just tried the above again, and now I get another (but related error), indicating that there is some cache thing going on:
# github.com/cpuguy83/go-md2man/md2man
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:11:16: undefined: blackfriday.EXTENSION_NO_INTRA_EMPHASIS
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:12:16: undefined: blackfriday.EXTENSION_TABLES
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:13:16: undefined: blackfriday.EXTENSION_FENCED_CODE
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:14:16: undefined: blackfriday.EXTENSION_AUTOLINK
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:15:16: undefined: blackfriday.EXTENSION_SPACE_HEADERS
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:16:16: undefined: blackfriday.EXTENSION_FOOTNOTES
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:17:16: undefined: blackfriday.EXTENSION_TITLEBLOCK
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/md2man.go:19:29: too many arguments to conversion to blackfriday.Markdown: blackfriday.Markdown(doc, renderer, extensions)
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/roff.go:19:9: cannot use &roffRenderer literal (type *roffRenderer) as type blackfriday.Renderer in return argument:
	*roffRenderer does not implement blackfriday.Renderer (missing RenderFooter method)
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/roff.go:102:11: undefined: blackfriday.LIST_TYPE_ORDERED
../../../go/pkg/mod/github.com/cpuguy83/go-md2man@v1.0.10/md2man/roff.go:102:11: too many errors

@jayconrod @bcmills

go get -u intentionally upgrades the versions of dependencies.
blackfriday v2.0.0 predates the introduction of modules (as evidenced by its lack of go.mod file), so go get -u considers it to be a more recent version of the import path github.com/russross/blackfriday rather than github.com/russross/blackfriday/v2.
One of the invariants of Go modules is that each import path indicates one unique version of the package. So github.com/russross/blackfriday must resolve to one specific version, and which version is intended by which importer is not obvious from the source code alone: for example, we have no way of knowing whether you're running go get -u in order to write new code that depends on the v2.0.0 API.
Semantic import paths in module mode are specifically intended to address this problem, but unfortunately they don't help with dependencies that made breaking changes before the introduction of modules.


Milvus vector db Integration

integrated Milvus vector db into private GPT

Changes are all updated, thanks for the patient and time to review!

All conflicts and minor format issues solved. Thanks for the patient and time to review, will follow up the future changes!

LGTM, i think it is  ready for merging :)


Training on custom data, only the first object in the list is ever trained

I have created my own dataloader and I'm successfully training VoteNet on my data. Things appear to work, but only one object ever gets actually trained in, and it's always the first one in the list. I'm trying with two or three objects, and even when I switch their order in the list, it still only ever gets the first one. Here's a batch going through training so you can see the values output, and things look okay. These are just in the first epoch, so that's why it looks so bad:
 ---- batch: 010 ----
mean box_loss: 0.561006
mean center_loss: 0.207614
mean heading_cls_loss: 2.056063
mean heading_reg_loss: 0.030713
mean loss: 17.246320
mean neg_ratio: 0.713672
mean obj_acc: 0.860593
mean objectness_loss: 0.144644
mean pos_ratio: 0.059570
mean sem_cls_loss: 0.683496
mean size_cls_loss: 0.900661
mean size_reg_loss: 0.027006
mean vote_loss: 1.022955

As it trains, the numbers appear to do what they should. And then, when I get to an eval phase, when it calculates average precisions, this is what I see:
Eval batch: 10
eval mean box_loss: 0.463892
eval mean center_loss: 0.242243
eval mean heading_cls_loss: 0.987535
eval mean heading_reg_loss: 0.024332
eval mean loss: 16.180108
eval mean neg_ratio: 0.658729
eval mean obj_acc: 0.869922
eval mean objectness_loss: 0.150041
eval mean pos_ratio: 0.075721
eval mean sem_cls_loss: 0.108614
eval mean size_cls_loss: 0.700695
eval mean size_reg_loss: 0.028494
eval mean vote_loss: 1.068237
D:\votenet\utils\eval_det.py:154: RuntimeWarning: invalid value encountered in true_divide
  rec = tp / float(npos)
D:\votenet\utils\eval_det.py:154: RuntimeWarning: invalid value encountered in true_divide
  rec = tp / float(npos)
0 0.024822130858915763
1 nan
2 nan
eval object1 Average Precision: 0.024822
eval object2 Average Precision: nan
eval object3 Average Precision: nan
eval mAP: nan
eval object1 Recall: 0.600000
eval object2 Recall: nan
eval object3 Recall: nan
eval AR: nan

In my custom data class, if I switch object1 with object2 in the list at the top (DEFAULT_TYPE_WHITELIST = ['object2', 'object1', 'object3]) and in the other files (object_utils.py and model_utils_object.py) then object2 works fine and object1 and object3 gets the nan average precision. It always seems to only work with just the object in the first position of the list. I have double-checked my annotations and the data itself, and it appears like the correct data is going to the network training. That is, the data returned by my getitem() function does contain all the object labels and data. Has anyone else come across something like this? What can I do to narrow down what could be causing this? I'm glad to post more data if it would help.

I ended up figuring this out on my own. In case others come across this kind of problem: the issue was in my dataset's __getitem__(self, idx) method, where I was not returning consistent size classes for the objects. Now it is training on all my objects, though the AP at the end is terrible, so I still have work to go. I think it has something to do with the size and angle classes, which I am not really using but the data still getting trained with them (pretty much just either 0 or 1 for those). I think I need to figure out how to take that stuff out of training so it'll just use bounding boxes and semantic classes.

I ended up figuring this out on my own. In case others come across this kind of problem: the issue was in my dataset's __getitem__(self, idx) method, where I was not returning consistent size classes for the objects. Now it is training on all my objects, though the AP at the end is terrible, so I still have work to go. I think it has something to do with the size and angle classes, which I am not really using but the data still getting trained with them (pretty much just either 0 or 1 for those). I think I need to figure out how to take that stuff out of training so it'll just use bounding boxes and semantic classes.

For me, this is because the self.num_class, self.num_size_cluster, self.type2class aren't correct.

hello @rXYZkit
I am using two classes, so how I should set up self.num_class, self.num_size_cluster, self.type2class and also self.num_heading_bin. I would be thankful for any help.


Type annotation error with Python 3.8

With the recent release of version 0.8.0 of mkdocs-alias, we started having errors when building our documentation using Python 3.8 (See these CI logs).
It seems that the package's code started using list for annotating lists of objects instead of using List from the typing package and that is not supported in Python 3.8.
There are 2 possible solutions:

Add from __future__ import annotations at the top of the plugin module.
Use List from the typing package instead of list until Python 3.8 is no longer supported.

Hi Anes, thanks for the bug fix! I've merged it, and it will go out in version 0.8.1. I agree with your comment on the PR. I will implement CI checks for all the Python versions that MkDocs still supports.
Thanks again!


add postgres persistence v2

This PR adds postgres persistence. This PR provides an independent postgres persistence module leaving the existing mysql persistence module unmodified and introduces no breaking changes. (eg. mysql properties remain the same) This PR supersedes 1327 which introduced a sql-persisence module that contained code common to both mysql-persistence and postgres-persistence.

Added license to all java files

@mactaggart Thank you for making the changes. Please resolve conflicts and we should be good to merge.

@apanicker-nflx I resolved conflicts and squashed commits.

Hi, I encountered the following error:
FlywayException: Found non-empty schema "cndr_schema" without metadata table! Use baseline() or set baselineOnMigrate to true to initialize the metadata table.
I put flyway.baselineOnMigrate=true in my properties which I include when running the server but it seems it doesn't work cos I still get the same error.

@checkingchecks I'll look into this

@checkingchecks I'll look into this

hi, any updates? so I initially created the same schema and tables via script to set it all up before running the server but when I ran the server it still prompts the flyway migration exception even though it has the similar tables and indexes before running.

@checkingchecks I'll look into this

hi, any updates? so I initially created the same schema and tables via script to set it all up before running the server but when I ran the server it still prompts the flyway migration exception even though it has the similar tables and indexes before running.

@checkingchecks While flyaway has an option for baselineOnMigrate, conductor does not. I created a branch that adds this configuration option to conductor. To test this, create a new build from my branch:
git clone -b baseline-on-migrate https://github.com/mactaggart/conductor.git
cd conductor
./gradlew build -x test

In your conductor config, set:
baseline.on.migrate=true
If this works in your environment, I can submit a PR. Hope this helps.

@checkingchecks While flyaway has an option for baselineOnMigrate, conductor does not. I created a branch that adds this configuration option to conductor. To test this, create a new build from my branch:
git clone -b baseline-on-migrate https://github.com/mactaggart/conductor.git
cd conductor
./gradlew build -x test

In your conductor config, set:
baseline.on.migrate=true
If this works in your environment, I can submit a PR. Hope this helps.

thanks. i'm trying it out with conductor server 2.24.5 but i think the changes are not reflected when applying baseline.on.migrate=true. Is there a part of the config that i haven't done?

do you also have flyaway.enabled=true?

Hi, I encountered the following error:
FlywayException: Found non-empty schema "conductor_schema" without metadata table! Use baseline() or set baselineOnMigrate to true to initialize the metadata table.
I put flyway.baselineOnMigrate=true in my properties which I include when running the server but it seems it doesn't work cos I still get the same error.

Hi @checkingchecks, I had this issue when the Postgres schema I was using was already pre-populated with tables. To work around the the FlywayException issue,

Configured Conductor an empty schema, "new_conductor_schema".
Start Conductor - this used flyway to create the objects.
(optional) migrated the data from the old schema into the new one.

The succeeding runs of Conductor no longer had FlywayExceptions.


Nested grouping

It would be nice if we could support nested grouping. (Or, put differently, allow colours to vary/repeat across units.) This would mostly be useful for line plots where we want to avoid joining the end of one line with the start of another. The idea is similar to how ggplot2 allows you to specific aes(col = var1, group = var2) separately.
Here is an illustration using the following dataset. The setting is a difference-in-differences research design with staggered treatment. So we have treatment cohorts (first_treat) superimposed on individual units (id).

First, points. (Fine.)

plot2(y ~ time | first_treat, dat)

Second, lines. (Not fine, because we have lines rejoining across units in the same cohort.)

plot2(y ~ time | first_treat, dat, type = "l")

Of course, we could group (colour) by the individual IDs. This stops the rejoining, but means that we lose the colouring by treatment group (which is the interesting thing from a causal inference perspective).
plot2(y ~ time | id, dat, type = "l", legend = FALSE)

I don't have a solution right now, but it probably requires a new argument like bycol. On the formula side, we could potentially represent this via a / nesting interaction. So the call would become plot2(y ~ time | first_treat / id, dat, type = "l"), i.e. units are nested within first treatment cohorts.

Ran into this again recently and am now thinking simpler solution is just to support passing a variable to col. It should be pretty simple to grab the corresponding colour breaks and pass them to our group-split data. Manual proof of concept:
library(tinyplot)

set.seed(123456L)

# 60 time periods, 30 individuals, and 5 waves of treatment
tmax = 60
imax = 30
nlvls = 5

dat =
  expand.grid(time = 1:tmax, id = 1:imax) |>
  within({

    cohort      = NA
    effect      = NA
    first_treat = NA

    for (chrt in 1:imax) {
      cohort = ifelse(id==chrt, sample.int(nlvls, 1), cohort)
    }

    for (lvls in 1:nlvls) {
      effect      = ifelse(cohort==lvls, sample(2:10, 1), effect)
      first_treat = ifelse(cohort==lvls, sample(1:(tmax+20), 1), first_treat)
    }

    first_treat = ifelse(first_treat>tmax, Inf, first_treat)
    treat       = time>=first_treat
    rel_time    = time - first_treat
    y           = id + time + ifelse(treat, effect*rel_time, 0) + rnorm(imax*tmax)

    rm(chrt, lvls, cohort, effect)
  })

cols = with(dat, tapply(factor(first_treat), id, FUN = `[[`, 1))  # grab group colours
cols
#>  1  2  3  4  5  6  7  8  9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26
#>  1  3  3  5  4  1  3  2  2  4  2  1  3  4  5  3  4  2  1  2  3  4  3  2  3  5
#> 27 28 29 30
#>  5  3  1  3

plt(y ~ time | id, dat, type = "l", col = palette()[cols], legend = FALSE)
#> Warning in tinyplot.default(x = x, y = y, by = by, facet = facet, facet.args = facet.args, :
#> Continuous legends not supported for this plot type. Reverting to discrete legend.

Created on 2024-08-30 with reprex v2.1.1
TBD on how to handle legends, as well as NSE vs formula arguments.


[RHCLOUD-17965] Add Oracle Cloud Infraestructure as a new source

Adds Oracle Cloud Infraestructure as a source, by supporting the Oracle Cloud IDs (OCIDs)
Links
[RHCLOUD-17965]

/retest


Add ResponseContentError for raised UPS error

Errors from UPS were getting thrown as runtime errors which are harder to catch. This change mirrors how errors are thrown in the FedEx carrier.

Thanks for the fix @jgillman!


Complete wireframes for boxes

Full mockup of the site: http://imgur.com/a/g1cio
Regarding the user account preferences, the editable info is listed on #13.


[zh-cn]Fixed a few words in Update images.md

fixed a word, optimized a link text.

This localized page was still in sync with the en version.
The revision makes sense and it clean.
/lgtm
/approve


Feature requrest : Mark Clustering

I find the x5-gmaps now does have all the functions i need except the mark clustering (https://developers.google.com/maps/documentation/javascript/marker-clustering), as the markers number is getting bigger and unmanageable soon. thanks.

I agree this is a good feature

This will be the next change after the upgrade of Typescript and Vue 3 support.

I've broken my docs build and am waiting for some help to fix them, but I have made this component.
Still needs to be tested and refined, but the basic function is there.
Check it out by selecting Marker Cluster from the dropdown here: Demo
And have a look at what the docs will eventually say about them here: Cluster docs


refactor(avro): use native dart avro serializer

removed avro js and tagparser.js scripts
moved other js scripts to js folder

Dependencies downgrade was probably because someone updated it using a different flutter/dart version


Review Cards Implementation

Goal: Check for the deployed result of the Cards implementation against the design
Cards Design implementation: #678
Cards implementation details: #709
Deployed: http://sap.github.io/techne/www/components/cards.html - stand June 23th.
Results
Commons cards: https://github.com/SAP/techne/issues/678#issuecomment-305724490

Label on the read only card: color don't match the design

Design
Implementation

Card showcasing a different layout is missing

When reducing the card width to 370px. The labels dont's wrap correctly

Design
Implementation

Cards With Media

Media element is not square

Design
Implementation

Missing Card with image on top
Missing Example where the image is smaller than the place holder element

For the implementation please use these Hybris stock images to test the implementation

Square: https://techne.yaas.io/images/product-thumbnail.png
Portrait: https://techne.yaas.io/images/product-thumbnail-tall.png
Landscape: https://techne.yaas.io/images/product-thumbnail-wide.png

The applicable changes have been implemented ... see #776
Overall changes
After looking at the current card implementation and at the "alternate" use case for Cards, I decided to make the button option required just as the current cards require cardRow cardRowClickable to enable the effects. So now the static/readonly card becomes the base component.
Common card

Label has been adjusted to match the design. A colored label here, however, would be allowed. Note, there is nothing coded in the Cards for to indicate status, a dev would need to use the appropriate label to indicate status. I'm just simulating the colors with helper classes. This will be changed once we complete labels
The "alternate" card layout is now included. I'm viewing this as a custom usage of cards. As long as you include the __content element you get the padding but you can bring your own custom HTML to the body of the card. The content shown looks close to a "readonly" form field, once those are complete this markup example can change from the simulated code.
Leo and I have discussed the labels for the Cards and decided against using a formal divider. Devs will be able to use a pipe | between the statuses. It is hard to enforce presentational dividers (see comments). The __status element is just a container so labels inside will be independent.  If this is a very common use case, we can revisit it with the Card groups when we see responsive behavior.

Cards with media

Only the icon card was part of this implementation task — https://sapse.invisionapp.com/share/DTBXJCSJW#/screens/239183942
Updated with Hybris stock image.

@xak
Changes are not beed merged or deployed
The stand https://github.com/SAP/techne/commit/9c8adda0c9e56d79c7115b118a5f908bc91065c9 doesn't content the changes yet.
Let me know when they are merged and deployed to make the review.

The PR is #776 and ready for review. I don't know what the deploy commit you reference means for this issue.

Deployed means that we are using the v2.0 as develop branch, and the changes are reviewed when they are deployed on
http://sap.github.io/techne/

I included details here so the code reviewer has some context for what changed. I have moved this issue back to "In Progress" and removed the "review" label.

Sorry if this was discussed already -

Do we not wanna show multiple cards?
There is no responsive behaviour on the cards (or the left nav which is a separate issue)
Are we doing away with list view? I recall this discussion about using a list vs. card list, but just wanted to confirm.

@saadmhybris
The whole design process for the cards went down at
https://github.com/SAP/techne/issues/678
There we decided to removed the cards lists and adopt the Product Thumbnail into the cards.

The card groups is at #730. We don't have exact specs for the layout yet but we can at least build out the 2 and 3 cols. Leo and I are working through the breakpoint and layout specs now. Those will be put into issues soon.

ok, thanks for the info.

http://sap.github.io/techne/www/components/cards.html

@xak
Points still missing from https://github.com/SAP/techne/issues/754#issue-238057596
Cards With Media

Media element is not square

Design
Implementation

Missing Card with image on top

Missing Example where the image is smaller than the place holder element

Please see #724. The card types in Invision and present in the PSD provided to me are all implemented. I have not seen final designs for 1 and 3, and 2 (the vertical card) has been implemented in the branch feature/724 which is now assigned for review.
Please open a new issue and connect with Leo to provide the PSD and update Invision with the addt'l card types and we can implement those.
Thanks.

@xak
All information have to be visible on github. That's why we are using it. To have one single point of up to date information.
See that invision is not always up to date and have broken links, so please use the information available on github. That's always up to date.
Is simply not possible to miss.
The cards design was, as stated before, already posted to github at https://github.com/SAP/techne/issues/678#issuecomment-305724490 - Jun 2nd, 2017
and you commented on the same thread at https://github.com/SAP/techne/issues/678#issuecomment-306950136 - Jun 8th, 2017.
This still belong to the same issue that is been reviewed. No need to open any new issue. All information was available at the right time.
@LeoT7508
We agreed to have the latest files always, right?
On this folder on Box:
https://app.box.com/folder/29116880878

Leo and I have been working to maintain a single repo for all the designs on Invision. We are still ironing it out so I do apologize for broken links but I believe those have been fixed now.
When designs are complete, a link to the final PSD and updated screens on Invision (or Zeplin maybe down the line) will signal the handoff to development. I will expect those deliverables so the dev team can squarely focus on the implementation.
This does not preclude you from posting the screens and maintaining the documentation you desire here on Github for the sake of transparency.
Thanks.

@joseegm @xak
I think a lot the confusion comes from requests and issues the Smart Edit team has, if we are going to be using their feedback and input then we need to make sure they follow up with us promptly, not a week later about details they feel need to be prioritized. As of now we have a lot of small details floating around that aren't being finalized, so its hard to keep an "updated" design when there is still back and forth going on. I think we need to do a better job of managing how we work together, how Techne impacts and cascades down to other products and not how other products affect Techne.

@LeoT7508 Agreed, we've discussed what I need for the implementation. I've synced the Box folder locally and always use the files in that folder. Photoshop syncs with Invision so there shouldn't be much overhead there. If any of that is tedious for you I'm sure we can find a compromise. It's just most important to me that the design be "close to final" before implementation begins.
I have recommendations on how to improve the UX process but in the spirit of moving forward, I'll let you and Jose come to agreement on what you need and consider a successful handoff from UX. Please keep me updated so we can all be aware of the expectations.
Thanks.

@xak
Im 100% on board with using Invision and syncing things that way, all good on that end.
I just synced files again to make sure everything is fresh for Cards, Input Fields, and colors to make sure things are updated.
Thanks Zack.

@xak
Okay, I'll start including the task "update PSD on Box" from now on, in every Visual Design issue.
Design issues get the label "design done" after the review period. At that point they are ready to for implementation. List available at
https://github.com/SAP/techne/issues?q=is%3Aissue+is%3Aclosed+label%3A"design+done"
That was agreed months ago. And communicated in a meeting. The same process has been followed since then and hasn't changed.
When all is completed all the info will be posted on the wiki, the same we did with the inputs fields. https://github.com/SAP/techne/wiki/Form-Inputs
Keep in mind the cards was the first component, so the information and process was already improved on the inputs -the second- already.
@LeoT7508
There is a standard period to respond to issues, 48 working hours, two full working days. That was clearly communicated also, to everybody collaborating actively in this project, including Jeanne and Irla.
If no feedback has been provided by that period, issues can move on, with the current information available on the issue.
That will stop any possible bottle neck.
Thx

@joseegm
48 working hours, two full working days for feedback. This is noted and I will stick to this.
Thank you.

@joseegm - is there still work to be done here? just making sure all loose ends are taken care of before working on #775