Connect to OAP instead of UI in E2E tests

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

One question, do we miss UI test? If the request doesn't bypass UI, how to test it works?

Please rename the PR title.

1.7 and 1.16 are unstable this time. It is strange. Another PR works.


Failed log attempts due to session not in progress

Godot version and OS
4.3
Issue description
Upon starting any game, always 12 warnings are pushed telling you that log attempts failed due to session not having been started yet.
Expected behavior
No response
Steps to reproduce

This is most likely caused by other autoloads/scripts that were initialized before the plugin, calling Log.entry(). A new warn_failed_start setting was added to disable this warning only.

Tentatively fixed by adding the warn_failed_start.


Rscript does not return non-zero exit code when it fails

Bug description
repo2docker uses Rscript to install R dependencies and rely on it exiting with a status code !=0 if installing a package fails. This doesn't seem to happen (any more). As a result the failure to install a dependency of a package we install in the CI goes unnoticed.
Expected behaviour

When installing dependencies fails the build process should exit with an error.
Actual behaviour

The build process completes successfully but no dependencies get installed.
How to reproduce

Go to tests/ in the repo2docker repository
Run pytest -s -x -v r/simple
Wait for the build to complete
Run docker images to find the name of the (latest) image that you just build for r/simple
Run docker run -it --rm r2d-2fusers-2fthead-2fgit-2fjupyterhub-2frepo2docker-2ftests-2fr-2fsimple1594276430 /bin/bash (replace the image name with your local one)
Run Rscript install.R
Immediately after the previous command run echo $?. This should return a value different from 0 but doesn't

Your personal set up

OS: OSX
Docker version: 19.3.08
repo2docker version 8bbced7ded5a21b581f1f3846ffc9f87944ba799

Debugging this further:
thead@c8d346a6ebaf:~$ Rscript -e "sdfsdf"
Error: object 'sdfsdf' not found
Execution halted
thead@c8d346a6ebaf:~$ echo $?
1

does the right thing. So the problem isn't that we set an option that influences error handling.

Is this helpful? https://stackoverflow.com/questions/26244530/how-do-i-make-install-packages-return-an-error-if-an-r-package-cannot-be-install


Differences with Symfony's PropertyAccess?

Hi, I'm curious to learn if there are differences with Symfony's PropertyAccess component?

Ah. I was not aware of the PropertyAccess component :)
They seem nearly identical. The only difference I can see is that object-path uses the the familiar -> operator for property access instead of .
Perhaps Leo should use the PropertyAccess component. I'm generally a fan of Symfony components, so I will definitely investigate.
Thanks!

OK thanks for clarifying, I wasn't sure. By the way I'm looking into Peridot and it looks really great! Congrats

Thank you for the kind words!


add kaalatrie site request

The  webring icon is placed on the always persistent sidebar(left) with few other social icons such as mastodon,  github, rss.
I hope you accept my request to be placed in the webring.
Best, Kaalatrie.

The website doesn't have it's own domain.

Your website must count at least 10 content pages and include an about page, blog posts are not counted as content pages. You must have your own domain name, we do not accept github.io subdomains.


Conditional routes

How would you go about having a conditional default route? For example, if a user is logged out, show the login screen, otherwise, skip the login and show the app home screen.

I do this in an app I'm building:
// auto requires files based on name
router(require,
  route('app', '/', { dir: '' },
    route('loading', { default: true }),
    route('splash'),
    route('tasks',
      route('create', '/tasks/create'),
      route('view', '/tasks/:id')
    ),
    route('auth', '/',
      route('cb', '/'),
      route('signin')
    )
  )
);

And in my root App.jsx
class App extends React.Component {
  componentDidMount() {
    if (!loggedIn) this.router().transitionTo('/auth/signin')
    else this.router().transitionTo('/tasks')

  }

  render() {
    return this.props.child();
  }
}

Make sure in this example in your Tasks.jsx file you need to have it be a React.Page to handle sub-routes like so (this should be cleaned up in the future):
import { React } from 'reapp-kit';

const props = [
  'tasksActive',
  'tasksCompleted',
  'tasksLastMessage',
  'user',
  'app'
];

export default store.cursor(props, class extends React.Page {
  render() {
    return (
      <ViewList
        {...this.routedViewListProps()}>

        <View></View>

        {this.childRouteHandler()}
      </ViewList>
    );
  }
});

Notice this.routedViewListProps(), this.childRouteHandler(); and React.Page usage. This is handled automatically for you with your root route, but for sub-routes you'll need to set it up like this.

Ah, perfect, worked like a charm, thank you. I was fairly close, had missed a couple of details though.
Would be nice to have this a little more documented. For instance, is there any way to control the direction from which views can transition in? I.e. it would be nice to be able to transition a view in from the bottom a la Ionic Modal or Touchstone view transitions

You can actually do this (I have it working today). I'd like to write a tutorial, but you can by using a plain ViewList, and then import the behavior for NestedViewListBehavior and DrawerViewListBehavior. Have your initial state be Nested, and spread it to the viewlist, then update it to Drawer when you need and it will just work. Taken from mine:
import {
  React,
  ViewList,
  View,
  Modal,
  Icon,
  Button,
  DrawerViewListBehavior,
  NestedViewListBehavior,
  store } from 'reapp-kit';

class Tasks extends React.Page {
  constructor(props) {
    super(props);
    this.state = {
      behavior: NestedViewListBehavior
    };
  }

  componentWillMount() {
    if (this.router().getCurrentPathname().match(/create/))
      this.setState({ behavior: DrawerViewListBehavior });
  }

  handleAdd() {
    this.action.disableKeyboardScroll();
    this.setState({ behavior: DrawerViewListBehavior }, () => this.goTo('/tasks/create'));
  }

  handleView(id) {
    this.setState({ behavior: NestedViewListBehavior }, () => this.goTo(`/tasks/${id}`));
  }

  // bugfix ios/web
  goTo(path) {
    setTimeout(() => this.router().transitionTo(path), 10)
  }

  render() {
    return (
      <ViewList
        {...this.state.behavior}
        {...this.routedViewListProps()}>

        <View>
        </View>

        {this.childRouteHandler({ id })}
      </ViewList>
    );
  }
}

If you get this all working and are feeling in a charitable mood, would love some docs help, even if it was just in the form of a super quick article on getting this working! :smile:

Ah, very slick, got it to work. Only small thing I've noticed is that the sub-view ends up 44px from the top (see the matrix3d in attached screenshot), not sure if this is expected behaviour?

Also, I was wondering, from this point, is it possible or would it be possible to also have nested views within that drawer? I've attempted to get it to work but to no avail.
As for a tutorial, I could give a stab at it within the next few weeks. I'm in the process of transitioning a large Ionic application to Reapp due to poor Ionic/Angular performance, so I'm most definitely going to be documenting the process and writing an article about it once the transition is complete and would happily help out with any help you'd want on docs.

Yea I had it because its meant to mimic the way drawers slide up but not all the way. You'd have to copy the behavior and change the animation (load in a new animation) which requires modifying the theme. Not too hard but also not well documented.
You could do nested within the drawer, you'd just need to keep nesting ViewLists. Just keep playing with it.
I want to work on next a pushable navigation system much like native and I think Ionic has it as well. Would make this stuff easier to use, but its a pretty big size project.

Got it, created a new animations.js file under /theme and called it in /app/theme/index.js under theme({animations: [iOS.animations, require('./animations)]}) and it worked.
Only thing is I had to call AnimationHelpers file with the direct path to the file (/node_modules/...), not sure if there's a better way to import that file (i.e., through an export of that file in reapp-ui?)
As for nesting within the drawer, I've tried setting up the Drawer view as follows but it doesn't work, and no error is thrown:
class extends React.Component {
  render() {
    return (
      <ViewList>

Hello, from the sub route!

      </ViewList>
    );
  }
}

I've also tried setting up as a React.Page as stated above but also to no avail. Here is the kind of setup I'd ideally like to have
route('home', '/home',
	route('drawer',
		route('sub1'),
		route('sub2'),
		route('sub3',
			route('sub3-1'),
			route('sub3-2')
		)
	)
)

Nice! Glad that worked without much pain, happy to see the system is working in general.
As for animation helpers you are right, I should export that. For now you should be safe.
And yea, the kitchen sink example is about right.
Rather than ViewList I'd use NestedViewList since it has the styles/behavior. You can't use a ViewList without props, and Nested gives you that view. Sorry I wasn't explicit on that earlier.

Ah wonderful, didn't realize there was a distinction between ViewList and NestedViewList. Works like a charm :)

Only bug i've noticed is if you use the Swipe from the edge to go back feature and pull down at the same time as you pull back, it tries to close the drawer at the same time as it tries to go back to the previous nested sub-view.
Here's an example:

You've got to disable the parent view list once you nest in. See your previous link:
https://github.com/reapp/kitchen-sink/blob/master/app/components/kitchen/ViewLists.jsx
the this.props.disableParentViewList, which then calls this:
https://github.com/reapp/kitchen-sink/blob/23e3d4dad2c0406469b840d9c0066ad1c4648de1/app/components/Kitchen.jsx#L150
You can call that on mount or like I do after it goes ahead one view. It should be pretty flexible because you could also pass in different touchable areas to your parent view list.


illegal hardware instruction (core dumped) when running rift init

When running rift init on Ubuntu 22 (Python 3.10), I am getting the following error:
➜  rift rift init
[1]    26906 illegal hardware instruction (core dumped)  rift init

Could you specify what environment is supposed to run on?

Me too! Is there a specific required OS or Python version?

Thanks for reporting the bug! Could you kindly share the result of cat /proc/cpuinfo? flags for one core are sufficient.

If you're on windows try getting HwInfo software and share an screenshot of the features part.

If you're on windows try getting HwInfo software and share an screenshot of the features part.

I'm on macOS and Linux. @LaDoger maybe you can try this on Windows.

Thanks for reporting the bug! Could you kindly share the result of cat /proc/cpuinfo? flags for one core are sufficient.

➜  ~ cat /proc/cpuinfo
processor	: 0
vendor_id	: AuthenticAMD
cpu family	: 23
model		: 49
model name	: DO-Regular
stepping	: 0
microcode	: 0x1000065
cpu MHz		: 1996.247
cache size	: 512 KB
physical id	: 0
siblings	: 1
core id		: 0
cpu cores	: 1
apicid		: 0
initial apicid	: 0
fpu		: yes
fpu_exception	: yes
cpuid level	: 13
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm rep_good nopl cpuid extd_apicid tsc_known_freq pni pclmulqdq ssse3 fma cx16 sse4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm svm cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw topoext perfctr_core ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 xsaves clzero xsaveerptr wbnoinvd arat npt nrip_save umip rdpid
bugs		: sysret_ss_attrs null_seg spectre_v1 spectre_v2 spec_store_bypass retbleed
bogomips	: 3992.49
TLB size	: 1024 4K pages
clflush size	: 64
cache_alignment	: 64
address sizes	: 40 bits physical, 48 bits virtual
power management:

I can't even get to this point on Windows! I get:
ImportError: Wasmer is not available on this system

On Linux I get illegal hardware instruction (core dumped) too.

@AminRezaei0x443 I really need this Rift lang to work! FunC is killing me!😩😩😩

I can't even get to this point on Windows! I get:
ImportError: Wasmer is not available on this system

Right, I am getting this on macOS too (see #14).

@merc1er Could you please run and share the output of this code?
from rift.runtime.riftlib import RiftLibSetup
print(RiftLibSetup.determine_lib())
print(RiftLibSetup.get_env_info())

@merc1er Could you please run and share the output of this code?
from rift.runtime.riftlib import RiftLibSetup
print(RiftLibSetup.determine_lib())
print(RiftLibSetup.get_env_info())

In [1]: from rift.runtime.riftlib import RiftLibSetup
   ...: print(RiftLibSetup.determine_lib())
https://github.com/AminRezaei0x443/ton/releases/download/v0.1.0/rift-lib-ubuntu-22.04.so

In [2]: print(RiftLibSetup.get_env_info())
---------------------------------------------------------------------------
FileNotFoundError                         Traceback (most recent call last)
Cell In[2], line 1
----> 1 print(RiftLibSetup.get_env_info())

File /usr/local/lib/python3.10/dist-packages/rift/runtime/riftlib.py:90, in RiftLibSetup.get_env_info(cls)
     87 @classmethod
     88 def get_env_info(cls):
     89     return {
---> 90         "libs": cls.get_shared_libs(),
     91         "glibc": cls.get_glibc_version(),
     92     }

File /usr/local/lib/python3.10/dist-packages/rift/runtime/riftlib.py:69, in RiftLibSetup.get_shared_libs(cls)
     67 @classmethod
     68 def get_shared_libs(cls):
---> 69     ok, res = cls._call_get("ldconfig -p")
     70     if not ok:
     71         raise RuntimeError("Error fetching shared libraries!")

File /usr/local/lib/python3.10/dist-packages/rift/runtime/riftlib.py:57, in RiftLibSetup._call_get(cls, command)
     55 @classmethod
     56 def _call_get(cls, command):
---> 57     p = subprocess.Popen(
     58         command,
     59         stdout=subprocess.PIPE,
     60         stderr=subprocess.PIPE,
     61     )
     62     out, err = p.communicate()
     63     if p.returncode == 0:

File /usr/lib/python3.10/subprocess.py:969, in Popen.__init__(self, args, bufsize, executable, stdin, stdout, stderr, preexec_fn, close_fds, shell, cwd, env, universal_newlines, startupinfo, creationflags, restore_signals, start_new_session, pass_fds, user, group, extra_groups, encoding, errors, text, umask, pipesize)
    965         if self.text_mode:
    966             self.stderr = io.TextIOWrapper(self.stderr,
    967                     encoding=encoding, errors=errors)
--> 969     self._execute_child(args, executable, preexec_fn, close_fds,
    970                         pass_fds, cwd, env,
    971                         startupinfo, creationflags, shell,
    972                         p2cread, p2cwrite,
    973                         c2pread, c2pwrite,
    974                         errread, errwrite,
    975                         restore_signals,
    976                         gid, gids, uid, umask,
    977                         start_new_session)
    978 except:
    979     # Cleanup if the child failed starting.
    980     for f in filter(None, (self.stdin, self.stdout, self.stderr)):

File /usr/lib/python3.10/subprocess.py:1845, in Popen._execute_child(self, args, executable, preexec_fn, close_fds, pass_fds, cwd, env, startupinfo, creationflags, shell, p2cread, p2cwrite, c2pread, c2pwrite, errread, errwrite, restore_signals, gid, gids, uid, umask, start_new_session)
   1843     if errno_num != 0:
   1844         err_msg = os.strerror(errno_num)
-> 1845     raise child_exception_type(errno_num, err_msg, err_filename)
   1846 raise child_exception_type(err_msg)

FileNotFoundError: [Errno 2] No such file or directory: 'ldconfig -p'

It's weird. Try to run this command so the core dump includes pid in it.
sudo sysctl -w kernel.core_uses_pid=1

System should dump the core under a file named core.<pid> in the directory you're executing. Now, use gdb so we can find out which instruction is causing this problem.
gdb program core.<pid>

Change the layout to asm with this command inside gdb so you can find the instruction code:
layout asm

Share the result so I can assist further. If you have issues with gdb you can share the core dump with me so I can take a look (Telegram: https://t.me/Amin0x443)

No file was created by the command:
➜  ~ sudo sysctl -w kernel.core_uses_pid=1
kernel.core_uses_pid = 1

By the way, I am using a fresh install of Ubuntu 22 on a Digital Ocean server. If that's easier, you can either spin up your own or I can give you access to mine (it's fresh, almost nothing installed) if give me an SSH public key.

Yes, that command just does the configuration. You should run rift init to cause a core dump. But no need, I have access to digital ocean servers so I'll check that myself.

Ah ok. I am unable to run rift init though.
➜  rift sudo sysctl -w kernel.core_uses_pid=1
kernel.core_uses_pid = 1
➜  rift rift init
[1]    37332 illegal hardware instruction (core dumped)  rift init

Didn't it create a file in the directory you were working?

Didn't it create a file in the directory you were working?

No:
<img width="551" alt="Screenshot 2023-02-12 src="https://user-images.githubusercontent.com/21960643/218319666-f308c671-3805-4e63-aebc-d6de8ba2fc25.png">

Run this:
cat /proc/sys/kernel/core_pattern
It should show the core dumps pattern.

➜  ~ cat /proc/sys/kernel/core_pattern
|/usr/share/apport/apport -p%p -s%s -c%c -d%d -P%P -u%u -g%g -- %E

You can find the core at /var/lib/apport/coredump/ but never mind, I figured it out, it's caused by an AVX-512 instruction. Working on the fix!

This issue is now fixed in the current release. Run pip install -U rift-framework to upgrade. Let me know about the result.

This issue is now fixed in the current release. Run pip install -U rift-framework to upgrade. Let me know about the result.

Great! Thanks! I can confirm upgrading rift fixes the illegal hardware issue.
I can probably close this issue, however when compiling the contract with rift build I am getting a KeyError:
➜  project git:(main) rift build contracts/bare_template.py
Building contracts/bare_template.py from bare-template project ...
Traceback (most recent call last):
  File "/usr/local/bin/rift", line 8, in <module>
    sys.exit(entry())
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/rift/cli/commands/build.py", line 55, in build
    config.contracts[target],
KeyError: 'contracts/bare_template.py'

Great, so this issue is resolved. The other issue you mentioned is not an error. You should provide a target name, not a target path. Use:
rift build bare_template

And it will work. You can use all target to build all contracts.

Closing the issue as It's resolved, Thanks for your help. I will also try to output better message for the new error you shared.


docs(gpu): Add workaround for nvidia GPU unavailable

Docs:

Update docs to add workaround for Nvidia GPU becoming unavailable after a period of time idle.
Minor: Markdown formatting fixes.

I see people logging issues and asking for help on Discord for this quite often, this workaround has had good success in fixing the issue for many folk.
e.g. https://github.com/ollama/ollama/issues/4604

Thanks @sammcj. Do you know if this might be from the nvidia_uvm kernel module being unloaded? I was wondering if that's the case

I'm not sure for everyone else, but for me at least - nvidia_uvm was still loaded when the issue occurred (or at least on the times I checked) with no related notices in dmesg.


#7 Support for Plugins

Object wrappers and support functions for working with Unreal plugins.

Fixes #7


Fix renovate

Fix for https://github.com/wayfair/brickkit-android/issues/262

@kenyee @sergenes @rnorvielwf Please review. I am able to run build and passing all unit test.

looks like renovate.json is in the wrong path:
FATAL: Custom config file specified in RENOVATE_CONFIG_FILE must exist
"configFile": "/home/runner/work/brickkit-android/brickkit-android/renovate.json"

* What went wrong:
Execution failed for task ':app:kaptDebugKotlin'.
> A failure occurred while executing org.jetbrains.kotlin.gradle.internal.KaptExecution

   > java.lang.reflect.InvocationTargetException (no error message)

Weird error...might have been caused by the targetSDK change?  Only thing that uses kapt is databinding.

Nice..JDK11 was the issue indeed...must have been been a requirement w/ the new sdk target

@kenyee  UI Tests are failing on CI. working fine in local any clue?

@kenyee UI Tests are failing on CI. working fine in local any clue?

at '/Users/runner/Library/Android/sdk/platform-tools-2'
  Installing system images.
  /bin/sh -c \sdkmanager --install 'system-images;android-33;default;x86' --channel=0 > /dev/null
  Warning: Observed package id 'platform-tools' in inconsistent location '/Users/runner/Library/Android/sdk/platform-tools-2' (Expected '/Users/runner/Library/Android/sdk/platform-tools')
  Warning: Already observed package id 'platform-tools' in '/Users/runner/Library/Android/sdk/platform-tools'. Skipping duplicate at '/Users/runner/Library/Android/sdk/platform-tools-2'
  Warning: Observed package id 'platform-tools' in inconsistent location '/Users/runner/Library/Android/sdk/platform-tools-2' (Expected '/Users/runner/Library/Android/sdk/platform-tools')
  Warning: Already observed package id 'platform-tools' in '/Users/runner/Library/Android/sdk/platform-tools'. Skipping duplicate at '/Users/runner/Library/Android/sdk/platform-tools-2'
  Warning: Failed to find package 'system-images;android-33;default;x86'

Looks like that system image for the emulator doesn't exist...

@kenyee UI Tests are failing on CI. working fine in local any clue?

at '/Users/runner/Library/Android/sdk/platform-tools-2'
  Installing system images.
  /bin/sh -c \sdkmanager --install 'system-images;android-33;default;x86' --channel=0 > /dev/null
  Warning: Observed package id 'platform-tools' in inconsistent location '/Users/runner/Library/Android/sdk/platform-tools-2' (Expected '/Users/runner/Library/Android/sdk/platform-tools')
  Warning: Already observed package id 'platform-tools' in '/Users/runner/Library/Android/sdk/platform-tools'. Skipping duplicate at '/Users/runner/Library/Android/sdk/platform-tools-2'
  Warning: Observed package id 'platform-tools' in inconsistent location '/Users/runner/Library/Android/sdk/platform-tools-2' (Expected '/Users/runner/Library/Android/sdk/platform-tools')
  Warning: Already observed package id 'platform-tools' in '/Users/runner/Library/Android/sdk/platform-tools'. Skipping duplicate at '/Users/runner/Library/Android/sdk/platform-tools-2'
  Warning: Failed to find package 'system-images;android-33;default;x86'

Looks like that system image for the emulator doesn't exist...

this is fixed. I have switched back to 29 again. Error is related to Mockito.
com.wayfair.brickkit.BrickDataManagerTest > testAddFirstVisible_nullAdapter[test(AVD) - 10] FAILED
	org.mockito.exceptions.base.MockitoException:
	Mockito cannot mock this class: class androidx.recyclerview.widget.RecyclerView.

this is fixed. I have switched back to 29 again. Error is related to Mockito.

Mockito is for unit tests, not UI tests 🤔

this is fixed. I have switched back to 29 again. Error is related to Mockito.

Mockito is for unit tests, not UI tests 🤔
yes but those are instrumented all are in andoridTest

Mockito is for unit tests, not UI tests 🤔
yes but those are instrumented all are in andoridTest

Wow...never noticed those.  Well...good luck fixing 😹
Guessing a library bump is what is causing the failures.

@kenyee @sergenes @rnorvielwf  After reverting all JDK/library version changes tests are failing on CI. Locally working fine. This has nothing to do with renovate changes.  we can create new issue for these failing test & merge this PR.

@kenyee @sergenes @rnorvielwf After reverting all JDK/library version changes tests are failing on CI. Locally working fine. This has nothing to do with renovate changes. we can create new issue for these failing test & merge this PR.

sorry..can't merge PR if it breaks the pipeline...

Overriding PR Merge block due to existing failures.
Failure issue will be tracked/resolved via #264


feat(galaxy): add variables to the second server, see #3317

This PR adds variables to the second server of our OpenAPI example, great do debug #3317.

Funny: The updated file failed the validation, because of the OpenAPI JSON Schema, which doesn’t allow variables in URLs. I’ve tested with an example from the specification and it still failed. So I’ve had to modify the JSON Schema.
Official example
servers:
- url: https://{username}.gigantic-server.com:{port}/{basePath}
  description: The production API server
  variables:
    username:
      # note! no enum here means it is an open value
      default: demo
      description: this value is assigned by the service provider, in this example `gigantic-server.com`
    port:
      enum:
        - '8443'
        - '443'
      default: '8443'
    basePath:
      # open meaning there is the opportunity to use special base paths as assigned by the provider, default is `v2`
      default: v2

Source: https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.1.0.md#server-variable-object

Nope, this doesn’t fix the issue(s)! I thought we should do this in a separate PR.


Input Variables

Upon booting a TC1, the first time a project is opened, none of the Input Name variables are detected by Companion. If the TC project is closed and re-opened, the variables populate.

Which variables are you reference specifically? The SourceName ones?

The Input Short Name's used to work. And it actually appears those are no longer functioning at all. index.js line 506 appears to pull it. I can check what version of TC that system is running. Maybe they broke something with an update.

Can you provide the exact variable text from the buttons you're having issues with? is it
$(tricaster:pgm_source) and $(tricaster:pvw_source)

In my button text, I have "$(tricaster:input2)" to show the Button Text from TC. I have found this variable works in Companion 2.4.2. But not in 3.0.0.


Minor change to documentation/comments about pageToken

I misread the description of the pageToken (and correspondingly nextPageToken) in the following way: the pageToken is all that is required to get the next page of results. This is a rather frequent practice in the web to store the state on server and send to client only token to iterate through result set. So I presumed the meaning. I discovered my error only after I browsed thoroughly through the reference server code.
In my opinion small change of wording in pageToken description may protect other implementor from repeating my mistake. I suggest changing the description from
The continuation token, which is used to page through large result sets.
Provide this value in a subsequent request to return the next page of
results. This field will be empty if there aren't any additional results.

to
The continuation token, which is used to page through large result sets.
Provide this value along with the same values in other fields in a subsequent
request to return the next page of results. This field will be empty if there
aren't any additional results.

This is a great point @shadowtramp. Should the protocol mandate that all fields be provided as well as the pageToken? My vote is yes, as this makes our lives on the server-side a bit easier. Interesting to see what other implementers think: @dglazer, @sarahhunt, what do you do for pageTokens? Is the reference server breaking the protocol, or is the protocol under specified?

I understood the protocol to mean the mandatory fields were always
mandatory. We now have a very simple token and rely on the mandatory
fields being supplied each time to perform the lookup.
On 17/09/2015 16:45, Jerome Kelleher wrote:

This is a great point @shadowtramp https://github.com/shadowtramp.
Should the protocol mandate that all fields be provided /as well as/
the pageToken? My vote is yes, as this makes our lives on the
server-side a bit easier. Interesting to see what other implementers
think: @dglazer https://github.com/dglazer, @sarahhunt
https://github.com/sarahhunt, what do you do for pageTokens? Is the
reference server breaking the protocol, or is the protocol under
specified?
—
Reply to this email directly or view it on GitHub
https://github.com/ga4gh/schemas/issues/414#issuecomment-141127876.

Google's API requires that subsequent paginated requests have the same parameters as the original (with the exception of pageToken). We also reject requests which have a pageToken which is not consistent with the other parameters.
The protocol is probably underspecified currently, I would +1 the proposed change.

Thanks @shadowtramp, I added this ticket to our documentation
improvement work.

I also included returning of exception on the conflict detected between data supplied and pageToken. I see it as a must. At least from the security perspective. While it isn't, strictly speaking, a way to abuse server to divulge more information than allowed, but absence of the check for data vs. token consistency may lead to security bugs.
So I'm completely pro the Google approach, mentioned by @calbach

@shadowtramp --- agreed, the approach @calbach outlined should be made definitive.


Discuss latest changes with Angeline

Recommendations

Weighted average default.

Send her the JOSS paper draft.

She has seen the JOSS paper draft. There is no change in the sample size in the code so this can be reviewed by the commitee.


Provide ability to pass custom entr args to restart wrapper.

This PR provides the ability for the user to supply custom arguments to entr when needed.
This is needed for some of our python development workflows that rely on pip install -e ., and only need the live reload to copy the new app files into the container, then restart the main entrypoint. The problem we encounter is that if the application crashes, entr exits, tilt loses its active container and will rebuild everything from scratch, which takes far too long. If I pass in just -r to entr, when the app under development crashes, entr keeps running, leaving the container up which lets us continue our live reload loop instead of having to rebuild everything.

if you rebase on the code i just merged, you should be able to make this change Tiltfile-only!

Thank you for the support @nicks. Tests pass and changes are ready for review once more.


EvalAI taking too long

Hi, first of all thank you for developing such a wonderful benchmark for zero-shot learning assessment.
I have submit some predictions to EvalAI, and it has been running for more than 1 hour. Is the evaluation system working properly?
Thanks.

The eval server is up and running now. Your submission is evaluated, and please let us know if you experiment issues with future submissions. Thanks.

Thank you for solving the issue so quickly.

Hi @haotian-liu , I have another question.  Are the test accuracy reported at the end of the log files same as the value obtained from EvalAI? Thank you.

They should be the same.

Thanks for the clarification.


Update to the latest version

2 failed checks are in style, since we disabled pdf read functions.
Tested within argus, it runs smoothly as usual.


vscode.TerminalRenderer.onDidChangeMaximumDimension doesn't fire if renderer window height increases

Issue Type: Bug
details
VS Code version: Code 1.33.1 (51b0b28134d51361cf996d2f0a1c698247aeabd8, 2019-04-11T08:27:09.080Z)
OS version: Windows_NT ia32 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.92GB (22.10GB free)

Process Argv
--folder-uri file:///c%3A/src/cascade/vscode/extension

Screen Reader
no

VM
0%

Repro steps:

In VSCode extension, create a terminal render with vscode.window.createTerminalRenderer(name);
Subscribe to onDidChangeMaximumDimensions event on the renderer.
In the event handler, set renderer.dimensions to the new maximum dimensions after some time the event has fired (like in 100 ms).
Try to resize the terminal pane in VSCode UI.

Expected:
onDidChangeMaximumDimensions event fires whenever the terminal pane is resized, regardless of whether it is increasing its height, or decreasing.
Actual:
onDidChangeMaximumDimensions event fires only when the terminal pane height is decreased. It does not fire when the height is increased. It does fire for both increasing and decreasing of the panel width though.

To verifier:

Clone, build and run https://github.com/Microsoft/vscode-extension-samples/tree/master/terminal-sample
Run the "Create terminal renderer" command and focus the new terminal renderer in the panel
Open devtools console
Resize the terminal vertically, you should see logs saying the dimensions changed
Run the "Set dimensions" command and put something small
Resize the terminal vertically, you should still see logs when resizing beyond what was set in step 5


iOS: Expose fontScale to JavaScript

A related Android PR is #11008.
Font scale was exposed through:

The getContentSizeMultiplier method
The didUpdateContentSizeMultiplier event

These are now deprecated. The reason is that there was already an API that exposed font scale. However, it was Android only. We now expose font scale through that API on iOS as well. Specifically:

Font scale is now available as PixelRatio.getFontScale().
The change event on the Dimensions object now fires when font scale changes.

This change also adds support for Dimensions.get('screen') on iOS. Previously, only Dimensions.get('window') was available on iOS. The motivation is that, according to this comment, we'd like to deprecate window dimensions in favor of screen dimensions in the future.
Test plan (required)
Verified that PixelRatio.getFontScale() and the change event work properly in a test app.
Adam Comella
Microsoft Corp.

Thanks for the great summary. Looks great to me. Merging.


History shouldn't extrapolate values when HA wasn't running.

Home Assistant release (hass --version):
0.23.1
Python release (python3 --version):
Python 3.5.1+
Component/platform:
History graphs on web interface
Description of problem:
At times when home assistant isn't running, e.g. due to power failure, it still fills in states and values in the History section of the web interface, presumably using the last known state or value.  This can result in blatantly incorrect information in the History view.  This happened to me even with a clean shutdown and startup, when HA should be aware that it was not running for that period.
Expected:
Time regions when HA was not running should be blank in the History, or something like dashed lines or 'unknown' could fill in the areas that it currently extrapolates.

We only know that when we start, we were not closed correctly. However we do not know for how long we have not been running.

Can this be closed?

Closing this issue as it is old. You can re-open it if you are able to reproduce it running the latest version.


User can schedule task even if scheduler is disabled

User can access Add new Task form even if scheduler is disabled.
To Reproduce
Steps to reproduce the behavior:

Deploy Accelerator without meter support.'
Click on subscription -> manage usage
Click schedule task

Expected behavior
Schedule task button should be hidden in case of scheduler disabled.

This should now be fixed in #626


Nested types for TabView and TableView

Fixes #1239

@tznind can you see why we have a merge conflict here?

Looks like conflict comes from:
b29240f362b29b05e8f5e32a8b347433cf35a1c1
The Code Coverage pull request has a change to TableView.  Just looks like moving it's location int he file though nothing more:
https://github.com/migueldeicaza/gui.cs/pull/1235/files#diff-cabc06df9590a035dadf256277de6facda529d77c24898509033704c9f32c43bL146
Moving the class resulted in 2 copies of it, I have resolved conflict by deleting the extra copy


Create provisional UI

The temporary UI has been created. The first pages can be accessed via different tabs. A leaflet map was added and the first customized buttons were added.


Alignment of average voice data

Hi everyone.
Does anyone have a suggestion if it is preferable to run forced (state)  alignment on the individual speakers of an avg model separately (given e.g. 1000-4000 sentences per speaker) or align them all at once (probably with speaker-dependent normalisation)?
I'm currently doing a few quick experiments on that question but probably someone has some experiences with it?
Thanks

Seems alignment with speaker normalization works well.


fix node_modules install cache by moving node_modules path addition to build phase

As first discussed in Discord, this node_modules path addition in the install phase is invalidating the docker cache every single time for us in CI, causing npm modules to always be installed even when optimizing only_include_files.
Here you can see two different build runs where RUN printf '\nPATH=/app/node_modules/.bin:$PATH' >> /root/.profile invalidates the cache.
1:04:26 PM: #6 [stage-0  2/16] WORKDIR /app/
1:04:26 PM: #6 sha256:4b5e767673d5546e1b805d90dc48abd7b28b9f26e46273af3af21220c4df0b85
1:04:26 PM: #6 CACHED
1:04:26 PM:
1:04:26 PM: #9 [stage-0  4/16] RUN nix-env -if .nixpacks/nixpkgs-293a28df6d7ff3dec1e61e37cc4ee6e6c0fb0847.nix && nix-collect-garbage -d
1:04:26 PM: #9 sha256:e48ef8aadb2a73954263ab6faf873f16c8501cf662a4147318b1736222ba099d
1:04:26 PM: #9 pulling sha256:b65bcf19d1445822c0d6f15ea82c9ed82ac1d903cfd6c1284ba7b2409a092845
1:04:26 PM: #9 pulling sha256:eacfd968fa4e8b968bd5881543c7a84ca7350e3638e9ff1550e219639c68823f
1:04:26 PM: #9 pulling sha256:88eeafe67020271b9d01ee5450f7ddac5d7485a4f1a4f0b3083cf14d0c0e3bb6
1:04:26 PM: #9 pulling sha256:b65bcf19d1445822c0d6f15ea82c9ed82ac1d903cfd6c1284ba7b2409a092845 1.0s done
1:04:26 PM: #9 pulling sha256:012ca49434089586bd23bb8e180e7a4d81bb5957d69b8a538c811820ea7c360d
1:04:28 PM: #9 pulling sha256:eacfd968fa4e8b968bd5881543c7a84ca7350e3638e9ff1550e219639c68823f 1.0s done
1:04:28 PM: #9 pulling sha256:88eeafe67020271b9d01ee5450f7ddac5d7485a4f1a4f0b3083cf14d0c0e3bb6 1.1s done
1:04:28 PM: #9 pulling sha256:012ca49434089586bd23bb8e180e7a4d81bb5957d69b8a538c811820ea7c360d 0.1s done
1:04:28 PM: #9 pulling sha256:8ed289fc5fdae9a2f05c803eb73a1239e6baa29eaaf49416ee06f6b761706359 0.1s done
1:04:28 PM: #9 pulling sha256:6e53634a717d9bc3590c2532cf9f6454db4e46db16b71431d27d92dc69931a10
1:04:30 PM: #9 pulling sha256:6e53634a717d9bc3590c2532cf9f6454db4e46db16b71431d27d92dc69931a10 3.7s done
1:04:36 PM: #9 CACHED
1:04:36 PM:
1:04:36 PM: #10 [stage-0  5/16] RUN printf '\nPATH=/app/node_modules/.bin:$PATH' >> /root/.profile
1:04:36 PM: #10 sha256:c765a3a9df262b682adb82c261b51d1702c1afd4b71bffec13a65b41cd16b55f
1:04:38 PM: #10 DONE 3.1s
1:04:38 PM:
1:04:38 PM: #11 [stage-0  6/16] COPY .npmrc /app/.npmrc
1:04:38 PM: #11 sha256:f3d28b6d933fdd404bf7c2d6d94b6a91c0bb0a2af85410d536f23b66c1fe297d
1:04:38 PM: #11 DONE 0.1s

---

12:55:47 PM: #6 [stage-0  2/16] WORKDIR /app/
12:55:47 PM: #6 sha256:4b5e767673d5546e1b805d90dc48abd7b28b9f26e46273af3af21220c4df0b85
12:55:47 PM: #6 CACHED
12:55:47 PM:
12:55:47 PM: #8 [stage-0  3/16] COPY .nixpacks/nixpkgs-293a28df6d7ff3dec1e61e37cc4ee6e6c0fb0847.nix .nixpacks/nixpkgs-293a28df6d7ff3dec1e61e37cc4ee6e6c0fb0847.nix
12:55:47 PM: #8 sha256:96642b880335fd870dd59479723e60ae30ffa3216a103a807848de2c8dd8c3af
12:55:47 PM: #8 CACHED
12:55:47 PM:
12:55:47 PM: #9 [stage-0  4/16] RUN nix-env -if .nixpacks/nixpkgs-293a28df6d7ff3dec1e61e37cc4ee6e6c0fb0847.nix && nix-collect-garbage -d
12:55:47 PM: #9 sha256:e46612b80c0168bc4c2f0d3c2ef872f614dc13541401c7cc238d5b448bc7a7a7
12:55:47 PM: #9 CACHED
12:55:47 PM:
12:55:47 PM: #10 [stage-0  5/16] RUN printf '\nPATH=/app/node_modules/.bin:$PATH' >> /root/.profile
12:55:47 PM: #10 sha256:7837f6161528cfa2afe9dfa3189061392f9e28bfc2013dcf2b5b8200eb4e2f76
12:55:47 PM: #10 DONE 0.8s
12:55:47 PM:
12:55:47 PM: #11 [stage-0  6/16] COPY .npmrc /app/.npmrc
12:55:47 PM: #11 sha256:d409936d66d70833358ca72dcd4c4efd0798a204f2eb5055c53a06c881a1a2b7
12:55:47 PM: #11 DONE 0.1s

Since RUN commands are not supported to break caching, it seems maybe it's a bug in docker?
This PR works around it by moving this path addition from install to build phase, because node_modules in path is not needed until build in (almost?) all cases.

closing this in favor of a different fix here: https://github.com/railwayapp/nixpacks/pull/842


s3ext: refactor Makefiles

Put Makefile in each subdirectory, instead of using one Makefile at top
directory to do all the things, and extract the common parts.
Signed-off-by: Adam Lee<EMAIL_ADDRESS>Signed-off-by: Haozhou Wang<EMAIL_ADDRESS>

Do we need Makefile under src directory?  Could not see it in this patch.
Others LGTM

@yydzero .so module's Makefile is still at the root directory because I think it is the main compiling object, better to move it to src/ ?

LGTM. Look pretty then before, pls merge it.

utils is better, or could just put it under src directly.
On Fri, Jun 17, 2016 at 7:06 AM, Kuien Liu<EMAIL_ADDRESS>wrote:

LGTM. Look pretty then before, pls merge it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/adam8157/gpdb/pull/10#issuecomment-226639151, or mute
the thread
https://github.com/notifications/unsubscribe/AAfRk6dnNYSkoX7t5a4KQ0jAG0-t955Rks5qMdcJgaJpZM4I3G_f
.

--
Best Regards,
Yandong


Pick a font for text

Right now the app is using the default Android font for most text views. We should make sure we use a font we chose on purpose.

The font I'm using now is Nunito Sans.
Done with 2d12a3d046cce2a91464fb19cb67a7d1dbbf356d.


Provide an Apply button for diagram component properties

Describe what problem your feature request solves
any changes to the properties of a component do not seem to be applied immediately, only seemingly applied on change of focus
Describe the solution you'd like
Provide an Apply button for diagram component properties
Additional context
User feedback "For objects, the Properties window needs an Apply button or something. I have to click outside the window and lose focus to see the changes applied"

Increasing the priority of this because this issue makes the diagramming difficult to use


Refactor out unused var in presto-hive

#4188
This PR is a preparation before enable UnusedVariable error prone check.

Thanks!


Update image for text kpi inference

This PR

Updates farm, torch, torchvision, mmcv-full to work for text kpi inference
Adds text_kpi_infer.py from corporate data pipeline to define TextKPIInfer class

/approve


Add tests for getConfig

Description

Add unit tests around getConfig, this includes refactoring to be able to test better.
Changes/Tasks

[x] Add tests around getConfig

Motivation and Context

This is the beginning of epic #439 which is to add more testing around react-static.
Screenshots (if appropriate):

Types of changes

[x] Refactoring/add tests (refactoring or adding test which isn't a fix or add a feature)
[ ] Bug fix (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)

Checklist:

[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[x] My changes have tests around them

@tannerlinsley unsure about deployment being broken. Have written some test for getConfig and refactored some, could you have a look, cheers. Thanks.

This looks fantastic. Ignore the deploy preview. Gitdocs is broken again 😔. I'm good to merge if you are!

should be fine to merge if you've checked behaviour is the same 👍.

Needs to be fixed up

What's the status on this @jasonlafferty ?

@tannerlinsley, they got out of date. had not time. Will try update them this week and will message.


[cloud_firestore] snapshots().listen not getting called

I'm trying to perform a query and to get real time update to that query, the first time i launched it worked but it always returned me an old value, then i restart the app and the query isn't getting called at all, and it's not giving me any errors.
To Reproduce
Steps to reproduce the behavior:

Set rules to give permissions to read that documents.
2.-
Additional context
cloud_firestore: ^0.13.5
Flutter doctor
[✓] Flutter (Channel stable, v1.17.0, on Mac OS X 10.15.4 19E287, locale en-MX)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.45.0)
[✓] Connected device (1 available)

• No issues found!

issue exist
Closing as duplicate of https://github.com/FirebaseExtended/flutterfire/issues/2225


Missing "matched_services"...

Before May 18, when I tried search query i get "matched_services" that contain what matched with my query like below.
{
          "ip": "*.*.*.*",
          "services": [
            {
              "port": 80,
              "service_name": "HTTP",
              "extended_service_name": "HTTP",
              "transport_protocol": "TCP"
            },
            {
              "port": 139,
              "service_name": "NETBIOS",
              "extended_service_name": "NETBIOS",
              "transport_protocol": "TCP"
            },
            {
              "port": 443,
              "service_name": "HTTP",
              "certificate": "*******",
              "extended_service_name": "HTTPS",
              "transport_protocol": "TCP"
            },
            {
              "port": 445,
              "service_name": "SMB",
              "extended_service_name": "SMB",
              "transport_protocol": "TCP"
            },
            {
              "port": 3389,
              "service_name": "RDP",
              "certificate": "***",
              "extended_service_name": "RDP",
              "transport_protocol": "TCP"
            },
         ...
          "matched_services": [
            {
              "port": 80,
              "service_name": "HTTP",
              "extended_service_name": "HTTP",
              "transport_protocol": "TCP"
            },
            {
              "port": 443,
              "service_name": "HTTP",
              "certificate": "***",
              "extended_service_name": "HTTPS",
              "transport_protocol": "TCP"
            }

But now, search query with v2.2.2 python api doesn't have "matched_services"...
Is it bug or censys api nomore provide "matched_services"?

Hi @nkh8212,
We have not removed the matched_services field, would you be able to try again or provide us with an example to replicate this?

I was using a free key because my paid key had expired.
I'll experiment with getting my paid key back and open an issue again.


mssql collector reports last metric value even when mssql service is stopped

Hi,
We have observed a strange behavior hen using the mssql collector to monitor a SQL Express installation: even when the service is stopped the collector continues to report the last value for each metric that was collected as if the mssql service was running.
Can you please clarify if this intended behaviour and/or assist us?

Since I didn't know, I just tried looking at windows performance monitor, and the numerous performance counters are not set to zero when stopping the sql server service, so I guess the values just freeze in place and wmi_exporter has no way of knowing the service is down, as it does not actually connect to the server.
You would need an additional sql_exporter anyway since there are quite some things missing in the default wmi_mssql_ metrics.
If you want to know if your server is up, you could use sqlcmd to test connection, example via powershell:
Invoke-Sqlcmd -Query "SELECT 1" -ServerInstance "localhost\SQL2019" -ConnectionTimeout 3 | Out-Null;
Write-Host $?;

This will output True ✅  if the connection was successful or False ❌  if not.
⚠  DO NOT USE $LASTEXITCODE as thats 1 in both cases here for me.
You can easily write that as a mssql_up metric for the textfile collector. Maybe this helps.
I am currenlty evaluating myself if we use one of the several sql exporters found here on github, or query everything with sqlcmd and write all to .prom file to consume with wmi_exporter.

Thanks for the investigation @Mario-Hofstaetter! Agree that writing this to a file used by textfile collector would be a good and easy way to see if it is up. @pmvieira would that be sufficient for you? I think it'll be out of scope for including directly into the mssql collector.


Append a request header with a value from a secret

Description:
I'd like the ability to append a request header value from a secret (SDS) , this header represents the API key header and the secret represents the actual API key
The request_headers_to_add field in the RouteConfiguration uses the HeaderValueOption which only supports a literal value
Relates to https://github.com/envoyproxy/gateway/issues/4757
[optional Relevant Links:]

Any extra documentation required to understand the issue.

cc @mathetake @aabchoo

Sounds a bit risky.
Can you provide more details on the desired behavior (requirement) that you want to accomplish?

@adisuissa another use case: appending authentication header to the HTTP CONNECT request sent to the upstream forward proxy, like Squid. Currently, we can only set a literal value in tunneling_config.headers_to_add. Why do you think it's risky?
I could try to contribute this feature.


Gulp is not needed globally

I guess it will just work when bundled to web/ package and will eliminate warning that local gulp package was not found

Fixed thanks o @XeeD as of today.


Flaky test: GameStoreTest#GIVEN_created_WHEN_first_click_on_cell_THEN_state_started

The test sometimes fails because the game state may become WIN after the first click, but the test expects STARTED. We should also check other tests. Best to check is to run a test in repeat(1000) { ... }.

One of the possible ways of fixing the test is to create GameState with a predefined grid, such that clicking on the cell always leads to STARTED. Plus we could also precisely verify the entire grid state.
And we can also add a separate test for the case when clicking on the cell first time makes the state WIN.


src/ripple/beast/core/core.unity.cpp is used in nounity builds

Maybe it is just misnamed a bit, but all in all it's a cpp file that's including a bunch of other cpp files and so I'd say it probably shouldn't be used in nounity builds.

I agree. I did try to unroll this one as part of https://github.com/ripple/rippled/pull/2376, but it got to be a bit more than I wanted for that PR. I'll add this issue to my list. Thanks.

Another weird case is src/ripple/beast/unity/beast_hash_unity.cpp which also seems to include one of the few .c files in the ripple subfolder into a .cpp file - but then again apparently it has a C++ header file with namespace and everything. This really messes with trying to compile this separately...

As a side note, there are several xxhash implementations in this code base...


Add config folder contents to the dev_package target

The tree command on the package target and tree -l (follow symlinks) on the modified dev_package target both gives the same output and in conclusion
3 directories, 31 files


Yolox improve

Motivation

use ASFF/TOOD/REPVGG to improve YOLOX, achieve 44.5map yolox-s (need a table, coming soon)

Modification

Model Yolox components such as ASFF/ASFFSim/TOODS supports
Model of RepVGGBlocks backbone
Export support of reparameter and blade best practice for Yolox

Backward-Compatibility-Breaking (Optional)
Use cases (Optional)
Pull-Request Todo List

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
The documentation has been modified accordingly, like docstring or example tutorials.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: haiasd:white_check_mark: zouxinyi0625:white_check_mark: wuziheng:x: zxyzxy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


.sty not found

I am using package \usepackage{pythonhighlight} got following error,
File `pythonhighlight.sty' not found.

Did you download and copy the .sty-file to the same directory as your main LaTeX file?


[FEATURE] Add string split function to PPL

Is your feature request related to a problem?
Analysts would want to split a string using a delimiter. For example, when analyzing DNS logs, analysts would split the hostname using the "." delimiter.
What solution would you like?
A function that's similar to the split_part(str, delimiter, partNum) function in Spark SQL:

Splits str by delimiter and return requested part of the split (1-based). If any input is null, returns null. if partNum is out of range of split parts, returns empty string. If partNum is 0, throws an error. If partNum is negative, the parts are counted backward from the end of the string. If the delimiter is an empty string, the str is not split.

What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

@A-Gray-Cat
this is possible today using the parse command as shown here
source=accounts | parse address '(?<streetNumber>\d+) (?<street>.+)' | sort num(streetNumber) | fields streetNumber, street

Yeah. I'm aware of that function. Regex parsing can do what split can do, but for many folks, regex is just difficult to get right in the first run.
Having a dedicated string splitting function can make this experience much easier.
I agree with you that, since the use case is supported by parse, we can prioritize other tasks over this one.


Allow some of the internal limits to be configurable

For example:

Limit in Solid Queue to count jobs.
Time limit to consider scheduled jobs to be delayed

Hey @garyhtou, thank you so much for looking into this one, and sorry for the delay 🙏  I think I'd go with something a little bit more explicit 🤔  For example... internal_query_count_limit for the count limit. Hmm... that one is hard 😅  For the second one, definitely more explicit, as we want to convey that this is about scheduled jobs and not about anything else that can potentially be delayed. What about scheduled_job_delay_threshold? Or something like this.


Problem using wildcard in a dockerized application

Hi,
first of all i would like to thank you for this project which seems to have a bright future.
i have a small problem using it in a docker container,
i have a node and a react container running on port 8000 (for the node server) and 3000 for the react server.
I tried to set the first exemple from the readMe, when i run it in the browser manually it works perfectly http://localhost:8000/wildcard/myFirstEndpoint
but when i call it from react
(async () => {
const {msg} = await endpoints.myFirstEndpoint();
console.log(msg);
})();
the http request is http://localhost:3000/wildcard/myFirstEndpoint and i get a 404
can you give me some explanation on how the routes are build ?
you can check my repository here
https://github.com/worms19/wildcard-docker
might be helpful for a future boilerplate repo
thanks

Hi @worms19 !
Check out https://github.com/reframejs/wildcard-api#serverurl. Your frontend and backend don't run on the same port hence you have to tell Wildcard where the server lives.
Let me know if that works.

can you give me some explanation on how the routes are build ?

See, https://github.com/reframejs/wildcard-api/blob/master/docs/how-wildcard-works.md#how-wildcard-works. Let me know if something is not clear.

first of all i would like to thank you for this project which seems to have a bright future.

Thanks :-) - ideas on how I can make more people know about Wildcard?

@brillout thanks for the quick reply

Let me know if that works.

i didn't read far enough on the documentation it works perfectly

Thanks :-) - ideas on how I can make more people know about Wildcard?

i can tell you're pretty much everywhere (reddit, medium), maybe you could try to run a demo app on a youtube videos (videos courses are quite popular nowaday) but you better have a lot of free time in front of you.
I'm gonna try to build something out of this project, i'll try to give you feedback on it
you have a star from me and good luck for the future

i can tell you're pretty much everywhere (reddit, medium), maybe you could try to run a demo app on a youtube videos (videos courses are quite popular nowaday) but you better have a lot of free time in front of you.

Yea I'm not sure how I can popularize Wildcard without spending too much time on it.
It's just silly that people are doing all this GraphQL stuff when they don't really need it. Kinda my responsibility to get to word out there that RPC is enough for like 95% of the time.

I'm gonna try to build something out of this project, i'll try to give you feedback on it

That would be awesome, I'm def looking for feedback :-)

you have a star

Thanks hehe :)


Working with Schema.org with i18n routes

Details
Hello,
I'm working on a website that has i18n. I have eg-ar, and eg-en locales and I use the prefix strategy so for the root page I have /eg-ar and /eg-en/ paths.
When I tried to use the breadcrumb which by default generates items automatically based on schma.org (which is also automatically generated by the module). I noticed that it renders the Home link twice, once as / and another as /eg-ar/ (or /eg-en/ depending on the current locale). I'm not an SEO expert and have some questions

Does this mean that schema.org spec also has this setup? If so, is this the expected behavior?
If it's the expected behavior for schema.org spec, how to pass a baseUrl that contains the current locale to breadcrumbs
Does the breadcrumb component adds any SEO improvement? or its main feature is the auto-generating of link and if I were to manually do those I could build my own component with no drawbacks.

Hi, this would be an issue with the useBreadcrumbItems composable provided by the Nuxt SEO module itself.
In terms of general I18n, in the latest minor I've introduced a much better I18n integration which will properly seperate your WebSite entries based on your locales.

Does this mean that schema.org spec also has this setup? If so, is this the expected behavior?

This is a bug.

Does the breadcrumb component adds any SEO improvement? or its main feature is the auto-generating of link and if I were to manually do those I could build my own component with no drawbacks.

It's for the Rich Results snippet, see https://developers.google.com/search/docs/appearance/structured-data/breadcrumb. This may or may not improve click-through-rates or ranking.


Add path to the C library in the OCaml archive

e.g. pass -Lfoo when using foo/libstubs.a
Fix (at least) the compilation of cmxs files when C libs are in subdirectories

@avsm mentioned that it is probably a bad idea to do that when the archive is installed. I will fix my build system instead.

Commenting for reference:
The path given to the link_stubs tag, e.g. link_stubs(src/libfoo.clib) is the path in the source tree. In a typical topkg situation, all stub libs are installed directly in the package libdir, and their location under it can be overridden via Pkg.clib.
So the path of the clib has nothing to do with the path that needs to be included to find the installed library. It's certainly possible to add a parameter to link_stubs to denote this path, but then it would have to live in two places, both in _tags and pkg.ml.
For now, I would prefer to simply have a convention of not installing stubs in subdirs of libdir.


Initial Plans table

Resolves https://github.com/konveyor/virt-ui/issues/56

@vconzola we can't really use flex or grid, it's just a table row which sort of behaves like flex. The column widths will adapt to the sizes of their contents, so let's make sure we get those just right first, then we can maybe set minimum widths on the contents of certain cells to persuade it a little.

@gildub you have a rebase conflict here, and I have some final minor feedback above. After that I think this is ready to go!

We also need to follow up after this to add the filters above the table, but I think it's ok to leave that for another PR. https://github.com/konveyor/virt-ui/issues/76

@mturley As we discussed in our 1-1, can we right align the button and kebab column and set a min column width so that the Target providers column header is never truncated? Also, I don't know if this is PF ok, but can we reduce the size of the text above the progress bar to 14 px to increase the space between the status message and the number of VMs?

Yeah, @vconzola @gildub I'll make a commit you can cherry-pick to address these things, it'll be easier than repeating everything we discussed

@vconzola could you please do another review?

@gildub It's looking good. There's one more change I'd like to see, if possible. @mturley If we set a min width on the status column and changed the Button label to "Start" would that elongate the progress bar. It still looks kinda weird with the message and VM summary so close to each other. If you'd like I can open a separate issue for this and we can come back to it later.

@vconzola,
I've defined the status column at 30% cell width to give it more room.
I've also put the "Start migration" on 2 lines (you might not like it, just trying options), which makes the action column smaller but keep the full meaning.
I thought about reducing action column spacing (padding and margin) but I haven't found yet how to apply that only to last cell (--pf-c-table--Cell--PaddingLeft, etc).
There is also an issue when resizing the display to smaller size as the Kebab moves under the button, which looks a bit strange. Although we can force the two (button and kebab) to stay inline, that would make that column kinda of big compare to the other ones. One solution would be to have all actions inside the kebab or to split them into two columns (but it then gives more space just for the actions, unless shrinking their spacing if possible).
It seems there is an overall issue with the Progress bar inside a table.
This is look worse when making the display smaller because the progress bar, in essence, needs horizontal space no matter what, so it's always going to be a fight against the other columns.
Looking for a better fit, what about using a Donut utilization chart instead?
+1 to take it from here to another PR so this one doesn't block the VMs migration plan details.

The donut chart variant takes up way too much vertical space IMO, making the rows super tall also looks strange. I think maybe what we can do is:

put a CSS class on the progress bar component and give it a static minimum width, either that or put some significant margin/padding to the right of the status message and the left of the VM count message, to force that div to grow.
modify the CSS class I already put on the actions column to add white-space: nowrap which I believe will prevent it from ever wrapping onto multiple lines. We can probably also do the same thing for the status text above the progress bar.
We can maybe change "Start migration" to "Start" or "Migrate"?

Yes, let's open a new issue so we can merge this and unblock the next PR.

@mturley, yeah the donut takes a bit of vertical space, not ideal either.
Looking forward to implement your recommendation.

@vconzola, could you please ack so we can take from here. Thanks

Actually @gildub I think if you grab this commit that addresses everything Vince has pointed out so far: https://github.com/konveyor/virt-ui/commit/283f25ce5960bde7b28b61ef70ae262a6d80b65f

@mturley, nice the no-wrap property, that's it!

Ok @vconzola sorry for the noise, this is ready for a final review for reals this time. I think it's looking good

@gildub @mturley This looks great and handles responsiveness very well. Nice work guys.


New Version: Racket.Racket version 8.12

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @custardtwist,

This PR's version number 8.12 has 2 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version 8.11.1, which has 3 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 726)

@stephengillie Yes, 8.12 is the latest version. 8.11.1 was a bugfix release for the previous version.


[US] Adding Details View

As an application user, I want to click on a row in the table so that a side panel opens that will show all of the row's data.

MoSCoW
Risk
Story Points

Must
Medium
5

Acceptance Criteria

Ability to hide/unhide the side panel
Ability to display a Details View of a certain row
Ensuring that all existing attributes are presented on the side panel with the proper data from the database.

Risks

Adding complexity to the Virtual Model Definition

Tasks

[x] Adjust the ListView to have a hideable side panel
[x] In the ListView table, on row click, open the side panel
[x] When the side panel is open, show all the attributes
[x] For each attribute, populate with data from the database
[x] Component testing using cypress

Figma Mockup

Demo Video
https://github.com/WillTrem/UInnovate/assets/74876532/314f2300-174e-4528-b57f-68911ed2024f

This looks great! Congratulations!
Signing off!


DelayShow

Hi, I really like this plugin, just wonder why it's not possible to delay (before) showing the tooltip, or is it? Is there a plan to include that at any point?

@Kadrian
Just create a new feature delayShow in v0.9.4

or
<ReactTooltip delayShow={1000} />


PaginationGenerator next and previous are the wrong way around

next() should be the newer link and previous() should be the older link; they are currently the wrong way around.

This is actually right, what is incorrect is $previous_next passed to the content_type page view.

This is fixed by 7b9d0fa52a4d01c10bbd48ac620ddd2a468d6b0e


ENG-5590 using UUID.randomUUID() to generate a random responseID

Using UUID.randomUUID() generate a random responseID. This will generate a different response ID for each event that requires it. This will not randomly increase. Example ids below from a sample app run on a Pixel4a
"responseId":"a75bc0623a972800"
"responseId":"ff88266abd62000"
"responseId":"95a7c502ed7a0400"
"responseId":"1806cad5bb204800"
"responseId":"4e5f8e4723d47000"
"responseId":"f25588839fffdc00"

The md5 hash usage in the requestId is not a sensitive location. RSA folks (@dbroeckelmanpost, @quacke-nid) please take a look and let me know. Thanks.


Unexpected behaviour when VR is applied to cosmological zooms with baryons

Hi, I recently started working with cosmological zooms of Milky Way-mass haloes and I wanted to apply VELOCIraptor to the data in the same way as I do it for full cosmological boxes.
When I run VELOCIraptor on zooms to obtain the catalogues, I always find that VR struggles to correctly compute the properties of the central halo and of one of the subhaloes.
What I keep finding instead is that in all  the zoom simulations,

Problem A. an object with the largest halo mass in the simulation, which is for some reason not classified as central, has too low stellar mass and too large r200.
Problem B. an object with a very small halo mass, which should not but is classified as central, has far too large stellar mass. This object also has a negative r200. [UPD: this problem is now gone]

Below is a simple python script that shows what is going on in the VR catalogue in one of the zoom simulations. The object with problem A (problem B) is the last one in the last (first) row of the output.
In [1]: import numpy as np

In [2]: import velociraptor as vr

In [3]: data = vr.load("halo_halo_10_0037.properties") # Load VR catalogue

In [4]: halo_mass = data.masses.mass_200crit  # Fetch halo masses

In [5]: stellar_mass = data.apertures.mass_star_30_kpc # Fetch stellar masses

In [6]: r200 = data.radii.r_200crit # Get r200

In [7]: sort_idx = np.argsort(halo_mass) # Sort according to halo mass

In [8]: stellar_mass.to("Msun")[sort_idx] # Show stellar masses
Out[8]:
unyt_array([0.00000000e+00, 0.00000000e+00, 5.70441663e+09,
            0.00000000e+00, 0.00000000e+00, 0.00000000e+00,
            0.00000000e+00, 0.00000000e+00, 0.00000000e+00,
            0.00000000e+00, 0.00000000e+00, 0.00000000e+00,
            0.00000000e+00, 0.00000000e+00, 0.00000000e+00,
            0.00000000e+00, 0.00000000e+00, 0.00000000e+00,
            0.00000000e+00, 0.00000000e+00, 0.00000000e+00,
            0.00000000e+00, 0.00000000e+00, 0.00000000e+00,
            2.72257719e+06, 0.00000000e+00, 2.57739048e+05,
            1.74264642e+06, 2.02618167e+08, 5.25602289e+08], 'Msun')  # [The largest halo has too low stellar mass]

In [9]: r200.to("kpc")[sort_idx] # Show r200
Out[9]:
unyt_array([-1.00000000e+00, -1.00000000e+00, -6.91574142e-02,
            -1.94654432e+02,  8.12557225e+01,  2.44313447e+01,
             1.32149940e+01,  6.28237402e+00,  8.00385528e+01,
             2.38922979e+01,  3.74907068e+00,  3.73819116e+01,
             1.03565331e+01,  1.64424960e+01,  1.15966419e+01,
             4.34248021e+01,  2.39968011e+01,  2.79605862e+01,
             3.40829283e+01,  3.33680927e+01, -2.75381537e+01,
             1.10373186e+02, -2.33397677e+01,  5.57266117e+02,
             3.56556864e+02,  3.61359039e+02,  1.77724087e+03,
             1.29139616e+03,  8.10693203e+02, 4.50728670e+03], 'kpc') # [r200 seem to be very large for all (sub)haloes]

In [10]: data.structure_type.structuretype[sort_idx] # Show structure types (to see what is central and what is not)
Out[10]:
unyt_array([10, 10, 10,
            10, 20, 20,
            20, 25, 20,
            35, 20, 30,
            20, 30, 30,
            20, 20, 30,
            30, 30, 10,
            30, 10, 20,
            25, 25, 15,
            15, 15, 20],  dtype=int32, units='dimensionless')  # [the largest halo is not a central]

Visualisation of the problematic (sub)haloes
The gas surface density and dark matter mass surface density of the object with problem A are

The gas surface density and dark matter mass surface density of the object with problem B are

To reproduce the bug

Path to the snapshot /cosma7/data/dp004/dc-chai1/zooms/batch1_12_05_21/halo_10/snapshot_halo_10_0037.hdf5
Path to VR config /cosma7/data/dp004/dc-chai1/vrconfig_3dfof_subhalos_SO_hydro_final2_zoom.cfg
Path to VR /cosma7/data/dp004/dc-chai1/VR_ICRAR_27_March/VELOCIraptor-STF/build/stf
VR version: master branch (64de17bff6925f47f3ebe8f8195108801d661d95, March 24, 2021)

VR was compiled as:
cmake -DVR_USE_GAS=ON -DVR_USE_STAR=ON -DVR_USE_BH=ON -DVR_MPI=NO
I tried compiling with  -DVR_ZOOM_SIM but that didn't help.
I think that the problems occur because I am using a VR config that was originally designed for full cosmological boxes (not zooms). The solution will therefore most likely involve tweaking a few parameters in the VR config.
@MatthieuSchaller

Thanks for the full report Evgenii.
Indeed, this setup differs in two was from our normal runs:

zoom
different set of units.

However, the problem is somewhat reminiscent of the centering issues we have had in the past. This was at the time fixed by demanding that the haloes are always centred on a DM particle and not just any kind.
Could something like that be at play here once more?

Also, do we believe A & B should be a single object with the correct total stellar, gas and DM mass?

Also, do we believe A & B should be a single object with the correct total stellar, gas and DM mass?

@MatthieuSchaller the two figures below should clarify what is going on

UPDATE!
I figured what was causing part of the problem. When I was specifying the units for the Hubble parameter in the config file, I wrongly assumed that its units follow the standard cosmological convention, which is km/s/Mpc. However, since the value of H0 actually needs to be provided in internal units and my internal units in the simulation are [M,L,V] =  [1e10Msun, km/s, kpc], what I should have used instead is km/s/kpc.
Now the problem B is gone. But the problem A remains. That is, the object with the largest M200 still has too low stellar mass, too large r200, and is not identified as central. However, its origin can now be interpreted differently.
The object's dark matter surface density distribution is the same as what I provided in the description.

However, now the second-largest object is the central, which seems to have the correct stellar mass and r200.

Its position coincides with that of the largest object.

I am now guessing that if we simply ignore the first object, everything will become correct. It appears that VR is thinking of the whole zoom-in region as if it was an additional halo, which is why we have a halo with an incredibly large r200 (problem A).

Would the remaining issue be fixed by compiling in zoom mode and switching on the parttype 2 particles in the config?

Changing Input_includes_extradm_particle=0 to Input_includes_extradm_particle=1 does not help (does not help = problem A is still there)
Changing Halo_core_phase_significance=5.0 to Halo_core_phase_significance=2.0 does not help either (althogh it has effect on the masses of substructued.
I also tried running with -DVR_ZOOM_SIM (and with Input_includes_extradm_particle=1) but in this case VR seg-faulted.

The problem with too large r200 and assigning large halo masses to haloes which, based on the density field, should not have them seems to be not unique to the example I showed above. Here is another one

Probably a more interesting and illustrative example can be found in /cosma7/data/dp004/dc-chai1/zooms/batch1_12_05_21/halo_5 (snapshot snapshot_halo_5_0037.hdf5)
Below are images of two of the three largest (by M200_crit) DM haloes. Clearly, in both cases, r200 and M200 are too large.

@EvgeniiChaikin have you tried a more recent master? Also, when changing units, did you also update G, not just H0?
@edoaltamura could you share the code version, modules, VR config and full parameter file you use for the XL cluster zooms?

Hi @EvgeniiChaikin and @MatthieuSchaller, see below the setup I use.
 \
    -DVR_USE_HYDRO=ON \
    -DVR_USE_SWIFT_INTERFACE=OFF \
    -DCMAKE_CXX_FLAGS="-fPIC" \
    -DCMAKE_BUILD_TYPE=Release \
    -DVR_ZOOM_SIM=ON \
    -DVR_MPI=OFF
  make -j

Note, I do recommend compiling with MPI due to some negative density issues and better performance, just haven't done that myself.
Then run with: -C /cosma/home/dp004/dc-alta2/snap7/xl-zooms/hydro/vrconfig_3dfofbound_subhalos_SO_hydro.cfg with the same modules imported.
I see you have compiled with -DVR_ZOOM_SIM=ON, but there might be also some zoom-specific parameters to tweak in the .cfg. I will take a look at your config file, but do feel free to try with/copy mine if it helps!

@EvgeniiChaikin have you tried a more recent master? Also, when changing units, did you also update G, not just H0?
@edoaltamura could you share the code version, modules, VR config and full parameter file you use for the XL cluster zooms?

Yes, I changed the gravity constant too. (otherwise there would be no decected haloes at all)
I haven't used the most recent master yet; should definitely give it a try!

Hi @EvgeniiChaikin and @MatthieuSchaller, see below the setup I use.
 \
    -DVR_USE_HYDRO=ON \
    -DVR_USE_SWIFT_INTERFACE=OFF \
    -DCMAKE_CXX_FLAGS="-fPIC" \
    -DCMAKE_BUILD_TYPE=Release \
    -DVR_ZOOM_SIM=ON \
    -DVR_MPI=OFF
  make -j

Note, I do recommend compiling with MPI due to some negative density issues and better performance, just haven't done that myself.
Then run with: -C /cosma/home/dp004/dc-alta2/snap7/xl-zooms/hydro/vrconfig_3dfofbound_subhalos_SO_hydro.cfg with the same modules imported.
I see you have compiled with -DVR_ZOOM_SIM=ON, but there might be also some zoom-specific parameters to tweak in the .cfg. I will take a look at your config file, but do feel free to try with/copy mine if it helps!

Thanks Edo! I will check your version tomorrow. Hopefully, VR won't be complaing anymore!

I have just tried the following. Explaining everying step by step.

Switch to most recent master

bfd106824c003b831c41779f76d09f8981c3926a (May 26, 2021)

Ensure all modules are loaded

[dc-chai1@login7c build]$ module list
Currently Loaded Modulefiles:
 1) cmake/3.18.1   3) intel_comp/2020-update2   5) parmetis/4.0.3-64bit   7) fftw/3.3.8cosma7
 2) ucx/1.8.1      4) intel_mpi/2020-update2    6) parallel_hdf5/1.10.6   8) gsl/2.5

Compile as recommended by @edoaltamura

cmake .. -DVR_USE_HYDRO=ON -DVR_USE_SWIFT_INTERFACE=OFF -DCMAKE_CXX_FLAGS="-fPIC" -DCMAKE_BUILD_TYPE=Release -DVR_ZOOM_SIM=ON -DVR_MPI=OFF

Build

make -j

Copy @edoaltamura config to a new file (vrconfig_edo.cfg) and adjust the units of lengths, G, and H0 because my zooms have kpc instead of Mpc

[dc-chai1@login7c dc-chai1]$ diff vrconfig_edo.cfg /cosma/home/dp004/dc-alta2/snap7/xl-zooms/hydro/vrconfig_3dfofbound_subhalos_SO_hydro.cfg
25,27c25,27
< #assumes input is in 1e10 msun, kpc and km/s and output units are the same
< Gravity=43000.0211349 #for 1e10 Msun, km/s and kpc
< Hubble_unit=0.100 # assuming units are km/s and kpc, then value of Hubble in km/s/kpc
---
> #assumes input is in 1e10 msun, Mpc and km/s and output units are the same
> Gravity=43.0211349 #for 1e10 Msun, km/s and Mpc
> Hubble_unit=100.0 # assuming units are km/s and Mpc, then value of Hubble in km/s/Mpc
30c30
< Length_unit_to_kpc=1.0
---
> Length_unit_to_kpc=1000.0
156a157
>
[dc-chai1@login7c dc-chai1]$

Try running VR on one of zooms

export OMP_NUM_THREADS=1

COSMA_PATH="/cosma7/data/dp004/dc-chai1/"

${COSMA_PATH}/VR_ICRAR_27_March/VELOCIraptor-STF_zoom/build/stf \
-C ${COSMA_PATH}/vrconfig_edo.cfg \
-i ${COSMA_PATH}/zooms/batch1_12_05_21/halo_5/snapshot_halo_5_0037 \
-o ${COSMA_PATH}/zooms/batch1_12_05_21/halo_5/halo_halo_5_0037 -I 2

Result. seg-fault

[  26.253] [ info] main.cxx:439 Baryon search with 1 threads finished in 4.647 [s]
[  26.253] [debug] search.cxx:3983 Getting Hierarchy 764
[  26.253] [debug] search.cxx:4016 Done
[  26.258] [ info] substructureproperties.cxx:5025 Sort particles and compute properties of 764 objects
[  26.258] [debug] substructureproperties.cxx:5037 Calculate properties using minimum potential particle as reference
[  26.258] [debug] substructureproperties.cxx:5040 Sort particles by binding energy
[  27.292] [debug] substructureproperties.cxx:5065 Memory report at substructureproperties.cxx:5065@long long **SortAccordingtoBindingEnergy(Options &, long long, NBody::Particle *, long long, long long *&, long long *, PropData *, long long): Average: 1.330 [GiB] Data: 1.261 [GiB] Dirty: 0 [B] Library: 0 [B] Peak: 1.459 [GiB] Resident: 1.260 [GiB] Shared: 8.379 [MiB] Size: 1.360 [GiB] Text: 3.289 [MiB]
[  27.292] [debug] substructureproperties.cxx:42 Getting CM
[  27.308] [debug] substructureproperties.cxx:320 Done getting CM in 15 [ms]
[  27.308] [debug] substructureproperties.cxx:4599 Getting energy
[  44.721] [debug] substructureproperties.cxx:4711 Have calculated potentials in 17.412 [s]
[  44.750] [debug] substructureproperties.cxx:5012 Done getting energy in 17.441 [s]
[  44.750] [debug] substructureproperties.cxx:338 Getting bulk properties
[  46.806] [debug] substructureproperties.cxx:2195 Done getting properties in 2.056 [s]
[  46.807] [debug] substructureproperties.cxx:3215 Done FOF masses in 124 [us]
[  46.972] [debug] substructureproperties.cxx:3232 Get inclusive masses
[  46.972] [debug] substructureproperties.cxx:3233  with masses based on full SO search (slower) for halos only
[  47.012] [trace] substructureproperties.cxx:3313 Building trees for SO search
[  51.105] [trace] substructureproperties.cxx:3332 Max search distance is 1003.220 in period fraction 0.010
Segmentation fault

Clearly something is wrong, but I don't see what...

@EvgeniiChaikin the segmentation fault was a problem in the code, not on your setup (but your setup helped surfacing the problem). I've pushed a fix for it on the issue-78 branch which fixes the problem (I double-checked running exactly like you specified in your commands above). I've created a PR to double-check that nothing is broken, and after that I'll merge directly to the master.

The fix is now on the master branch after the PR successfully passed all the checks.

Thank you very much @rtobar!
My seg fault is gone too! And running with -DVR_ZOOM_SIM=ON produces all haloes with plausible masses and sizes!
I am therefore closing this issue because everyting has been fixed.
Thank you all again!

Fantastic. Thanks all.


⚠ Bofrost.es has degraded performance

In a88e0ae, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1492 ms

Resolved: Bofrost.es performance has improved in 78d97f7 after 15 minutes.


allowedStyles option

Thanks for the awesome work 👍
I understand the reasons for not allowing inline styles but for some wysiwyg editors they are hard to avoid completely.
I was thinking rather than an all or nothing approach, it could be cool to have something similar to the allowedClasses option, where some (safe?) inline styles can be allowed.
Like..
  allowedStyles: {
    'table': [ 'width', 'border-style' ]
  }

I couldn't find an option for this other than allowing all styles, for now I have come up with a work around... which could probably be improved on...
transformTags: {
    '*': function(tagName, attribs) {
        if (attribs.style) {
            let allowedStyles = ["width"];
            let cleanedStyles = [];
            allowedStyles.forEach(function(element) {
                let regex = new RegExp(element + ":[\\s|\\S]*?;");
                let matches = attribs.style.match(regex);
                if (matches) {
                    cleanedStyles.push(matches[0]);
                }
            });
            attribs.style = cleanedStyles.join(' ');
        }
        return {
            tagName: tagName,
            attribs: attribs
        };
    }
}

There are many ways of escaping CSS that are valid and would have to be
understood by a robust implementation of this. Conversely we'd need to make
sure those mechanisms could't fake it out... or introduce disallowed URLs
as sources for background images... etc. A full CSS parser would probably
be required.
On Tue, Aug 29, 2017 at 8:30 AM, Codemzy<EMAIL_ADDRESS>wrote:

Thanks for the awesome work 👍
I understand the reasons for not allowing inline styles but for some
wysiwyg editors they are hard to avoid completely.
I was thinking rather than an all or nothing approach, it could be cool to
have something similar to the allowedClasses option, where some (safe?)
inline styles can be allowed.
Like..
allowedStyles: {
'table': [ 'width', 'border-style' ]
}
I couldn't find an option for this other than allowing all styles, for now
I have come up with a work around... which could probably be improved on...
transformTags: {
'': function(tagName, attribs) {
if (attribs.style) {
let allowedStyles = ["width"];
let cleanedStyles = [];
allowedStyles.forEach(function(element) {
let regex = new RegExp(element + ":[\s|\S]?;");
let matches = attribs.style.match(regex);
if (matches) {
cleanedStyles.push(matches[0]);
}
});
attribs.style = cleanedStyles.join(' ');
}
return {
tagName: tagName,
attribs: attribs
};
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/sanitize-html/issues/158, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fY2fwuS6oZC0LJYNek1n9F2h2HXuks5sdARbgaJpZM4PF2S0
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com


Creates wrong path

$ pip show symstore
Name: symstore
Version: 0.3.3
Summary: publish PDB and PE files to symbols store
Home-page: https://github.com/symstore/symstore
Author:
Author-email:
License: MIT
Location: c:\users\thomas\anaconda3\lib\site-packages
Requires:
Required-by:

I added the following file, only zipped for github,
itcXOP2-64.zip to our symbolserver with symstore.
Now I can't access it with Visual Studio/CDB. The reason seems to be that the generated path is different in symstore (python) vs symstore (microsoft).
symstore.exe test itcXOP2-64.xop

$ ls -lR test
test:
total 0
drwxr-xr-x 1 thomas 197121 0 Oct  5 15:41 000Admin/
drwxr-xr-x 1 thomas 197121 0 Oct  5 15:41 itcXOP2-64.xop/
-rw-r--r-- 1 thomas 197121 0 Oct  5 15:41 pingme.txt

test/000Admin:
total 4
-rw-r--r-- 1 thomas 197121 51 Oct  5 15:41<PHONE_NUMBER>
-rw-r--r-- 1 thomas 197121 49 Oct  5 15:41 history.txt
-rw-r--r-- 1 thomas 197121 10 Oct  5 15:41 lastid.txt
-rw-r--r-- 1 thomas 197121 51 Oct  5 15:41 server.txt

test/itcXOP2-64.xop:
total 0
drwxr-xr-x 1 thomas 197121 0 Oct  5 15:41 5AA043B37D000/

test/itcXOP2-64.xop/5AA043B37D000:
total 476
-rwxr-xr-x 1 thomas<PHONE_NUMBER>76 Oct  5 15:41 itcXOP2-64.xop*

E:\test-ms>"c:\Program Files (x86)/Windows Kits/10/Debuggers/x64/symstore.exe" add /s . /l /f e:\itcXOP2-64.xop /o /t test

$ ls -lR test-ms/
test-ms/:
total 0
drwxr-xr-x 1 thomas 197121 0 Oct  5 15:41 000Admin/
drwxr-xr-x 1 thomas 197121 0 Oct  5 15:41 itcXOP2-64.xop/
-rw-r--r-- 1 thomas 197121 0 Oct  5 15:41 pingme.txt

test-ms/000Admin:
total 4
-rw-r--r-- 1 thomas 197121 52 Oct  5 15:41<PHONE_NUMBER>
-rw-r--r-- 1 thomas 197121 53 Oct  5 15:41 history.txt
-rw-r--r-- 1 thomas 197121 10 Oct  5 15:41 lastid.txt
-rw-r--r-- 1 thomas 197121 55 Oct  5 15:41 server.txt

test-ms/itcXOP2-64.xop:
total 0
drwxr-xr-x 1 thomas 197121 0 Oct  5 15:41 5AA043B37d000/

test-ms/itcXOP2-64.xop/5AA043B37d000:
total 477
-rwxr-xr-x 1 thomas<PHONE_NUMBER>76 Apr 20 00:01 itcXOP2-64.xop*
-rw-r--r-- 1 thomas 197121     44 Oct  5 15:41 refs.ptr

The microsoft version generates 5AA043B37d000 whereas this program generates 5AA043B37D000.
As URLs are case sensitive the symbol search fails.

There is a fix here now: https://github.com/symstore/symstore/commit/dcab9b4f648678278e002b80461bc50bf08b80a7
Some post-mortem notes. The reason that this issue have not been noticed previously, is because the part of the hash that is supposed to be in lower cases, is based on PE files 'size of image'. If 'size of image' in hex only contains 0-9 numbers, that there is no difference between lower or upper cases.
I'm planning to include this fix into the next release.

Thanks for the fix. Is there an ETA for a release?

Unfortunately, I don't know an exact ETA, but I hope I can sit down someday this week and do a release.

Very nice!

I've just hit this as well, when do you expect to be able to do a release?
Unfortunately I didn't notice until after we'd created a few hundred items with the wrong path :(


Update artifacts and start template with enterprise-gateway naming

I did not change references to elyra messages that will be displayed during ansible script run.  If we intend to expose these publicly, then we should change those references (and variable names, etc.) as well.
I believe about the only thing that remains elyra might be the remote user/passwordless ssh we configure.
Also removed the setting of LOG_FORMAT since the default we set in enterprise gateway is better anyway.

PR Merged


Custom styling is missing from export

Thanks for quickly fixing the recent export fix!
With the new update, custom styling is not being exported correctly. See example below:
Screenshot from Obsidian editor/reader:

Screenshot of the exported file:

As you can see, the header colors are different.
PS. Love your plugin :)

Interesting, where are these custom colors defined?

Interesting, where are these custom colors defined?

Using the style settings plugin: https://github.com/mgmeyers/obsidian-style-settings
I'm using the Things Theme if that helps you any

Okay, style settings should be supported so I will check it out. Could you copy paste your style settings by any chance? There should be an "Export" button at the top of the style settings options page. Just copy paste the output here. Thanks!

Here you go:
{
  "things-style@@mobile-black-background": false,
  "things-style@@h1-color": "#F94144",
  "things-style@@h2-color": "#F3722C",
  "things-style@@h3-color": "#F9C74F",
  "things-style@@h4-color": "#90BE6D",
  "things-style@@h5-color": "#43AA8B",
  "things-style@@h6-color": "#277DA1",
  "things-style@@active-line": true,
  "things-style@@em-color": "#74EBE6",
  "things-style@@text-highlight-bg-l": "#9EEF68",
  "things-style@@text-highlight-bg-d": "#D700D9",
  "things-style@@strong-color": "#FF5454"
}

Also experiencing a complete freeze after exporting to HTML. Think I'll create a new issue and post a quick loom

Okay thanks! I have experienced a complete freeze a few times when exporting to a network drive that wasn't responsive, it may be related but not sure.

Hmm interesting. I’m using Syncthing, but exporting to local folder...
I just tested it again with Syncthing turned off - same issues!

On Sep 13, 2023, at 11:22 AM, Nathan George @.***> wrote:
Okay thanks! I have experienced a complete freeze a few times when exporting to a network drive that wasn't responsive, it may be related but not sure.
—
Reply to this email directly, view it on GitHub https://github.com/KosmosisDire/obsidian-webpage-export/issues/214#issuecomment-1716989419, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHH4PBJ4PMP5CFB46OT3M63X2FCYHANCNFSM6AAAAAA4VPA3KM.
You are receiving this because you authored the thread.


Implementing a wider selection of variable transformations

Discussed in https://github.com/pymc-labs/pymc-marketing/discussions/442

Originally posted by iraur November 23, 2023
Hello!
I was wondering if anyone has considered implementing alternatives for variable transformations, namely Weibull PDF for adstock where the (cumulative) decay rates are given by :
$$G_{k,\lambda}(t) = \frac{k}{\lambda}\Big(\frac{t}{\lambda} \Big)^{k-1}e^{-(\frac{t}{\lambda})^k}$$
and the Hill function for saturation, where transformed spend in week $t$ is given by :
$$x_t^{\textrm{transf}} = \frac{x_t^\alpha}{x_t^\alpha + \gamma^\alpha}$$
The main motivations for this are :

Weibull PDF offers a lot more flexibility, compared to geometric adstock which is sometimes criticised for being too simplistic. As a result it can lead to better performing models (see this white paper from Ekimetrics for example)
Weibull PDF is preferred by business stakeholders as it can better model real life marketing effects
We started out our MMM journey using Robyn, Meta’s open-source MMM package, which uses Weibull PDF and Hill - it would be great for pymc-marketing and Robyn to align on these transformations (more direct comparison, QA checks, etc)

We have attempted to write a scrappy first version of these functions below and would very much appreciate any comments or feedback (still getting up to speed on pymc so appreciate your patience on any silly mistakes here!) 🤗
import pymc as pm
from pymc_marketing.mmm.transformers import batched_convolution

def weibull_pdf_adstock(x, shape, scale, l_max: int = 30, axis: int = 0):
    shape = pm.floatX(pt.as_tensor(shape)[..., None])
    scale = pm.floatX(pt.as_tensor(scale)[..., None])

    w = shape * pt.power(scale, -shape) * pt.power(pt.arange(l_max, dtype=x.dtype) + 1, shape - 1) * pt.power(pt.as_tensor(math.exp(1)), -pt.power((pt.arange(l_max, dtype=x.dtype) + 1)/scale, shape))
    w = (w - pt.min(w.eval()))/(pt.max(w.eval()) - pt.min(w.eval()))
    return batched_convolution(x, w, axis=axis)

def saturation_hill(x, alpha, gamma, x_marginal=None):
    inflexion = np.dot(np.array([1 - gamma, gamma]), np.array([pt.min(x, axis=0), pt.max(x, axis=0)])) # linear interpolation by dot product

    if x_marginal is None:
        x_scurve = x**alpha / (x**alpha + inflexion**alpha)
    else:
        x_scurve = x_marginal**alpha / (x_marginal**alpha + inflexion**alpha)
    return x_scurve

Thanks! cc. @louismagowan

I'd be happy to help out here
For the other adstocks, they use the sum for normalization (i.e x / x.sum()). Would that be something to implement for weibull too.
I think there's a gain of having sum for normalization in order to keep the sum of the input the same
What was the rationale behind this difference? Would there be a benefit of leaving these as option to the user as well?


=3.3.1 POST command/download newline differences

Handle newlines different for POST command/download based on qBitTorrent version:
qBitTorrent < v3.3.1: urls separated by %0A, & must be converted to %26
qBitTorrent >= v3.3.1: urls separated with normal newlines (not converted to %0A); looks like multipart formatting.

After qBittorrent v3.3.1 API7 (included):
POST /command/download HTTP/1.1
User-Agent: Fiddler
Host: <IP_ADDRESS>
Cookie: SID=your_sid
Content-Type: multipart/form-data; boundary=---------------------------6688794727912
Content-Length: length

-----------------------------6688794727912
Content-Disposition: form-data; name="urls"

https://torcache.net/torrent/3B1A1469C180F447B77021074DBBCCAEF62611E7.torrent
https://torcache.net/torrent/3B1A1469C180F447B77021074DBBCCAEF62611E8.torrent
-----------------------------6688794727912
Content-Disposition: form-data; name="savepath"

C:/Users/qBit/Downloads
-----------------------------6688794727912
Content-Disposition: form-data; name="cookie"

ui=28979218048197
-----------------------------6688794727912
Content-Disposition: form-data; name="label"

movies
-----------------------------6688794727912--

Multipart support depends on HTTParty multipart support which is currently in limbo.

Just FYI, we implemented @jwagener's HTTMultiParty as a workaround years ago, and it's been working perfectly ever since (in production!).


Remove old docs before building new ones

I discovered that jsdoc generator is not aggressive and leaves leftovers. Now we delete old docs before generating new ones (this change will be included in all other repos PRs for documentation)

Thanks!