How to install?

  I see pipenv files, but don't know how to actually run this.

Yea, in general a setup documentation would be pretty nice/handy.

Yes totally agree, that part is missing! There are a lot of manual steps which I haven't got around to writing up.
Moving forward perhaps docker would be the way to go.

Hi @AlexNaga, I stumbled over your project while on a research adventure. I'd like to use your project for my thesis, also. So I won't need to reinvent the wheel…
So far I successfully installed all dependencies. In the next step I will try to dig my way through the code, to get it running. Upon success I will provide a setup documentation.

Hello,
What are you investing in your thesis?
On Mon, May 4, 2020, 17:33 Markus Kwaśnicki<EMAIL_ADDRESS>wrote:

Hi @AlexNaga https://github.com/AlexNaga, I stumbled over your project
while on a research adventure. I'd like to use your project for my thesis,
also. So I won't need to reinvent the wheel…
So far I successfully installed all dependencies. In the next step I will
try to dig my way through the code, to get it running. Upon success I will
provide a setup documentation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlexNaga/rpi-people-counter/issues/2#issuecomment-623535804,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLI7XNCQI3X5VYNVFE6ZCLRP3N5JANCNFSM4J2EDR7Q
.

Well, my goal is to visualize the PAX count (as I say for People ApproXimatelly :-) over a building floor plan (Open Street Map) for different use cases (e.g. emergency evacuation assistance system and such).

Very interesting!
I've written a thesis of a similar topic which you may find of some value:
http://www.diva-portal.se/smash/get/diva2:1335277/FULLTEXT01.pdf
I would however recommend you not to try and get the current state of this
project up and running since it will be very time consuming. There are a
lot of redundant code.
This mean that you're better off inventing the wheel again since right now
the wheel isn't even round. I
could upload the rewrite to a branch if you're interested to continue
coding since it's not yet done.
On Mon, May 4, 2020, 21:58 Markus Kwaśnicki<EMAIL_ADDRESS>wrote:

Well, my goal is to visualize the PAX count (as I say for People Appro
Ximatelly :-) over a building floor plan (Open Street Map) for
different use cases (e.g. emergency evacuation assistance system and such).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlexNaga/rpi-people-counter/issues/2#issuecomment-623674033,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLI7XP4Q67XFLJQCSICZPLRP4M7XANCNFSM4J2EDR7Q
.

In fact, I found your GitHub repository from exactly that document of yours :-)
Of course, I would like to have a look in your code rewrite.

As I said I'm trying to write down the setup instructions. Until now I got this far, see: https://github.com/mrkskwsnck/rpi-people-counter/blob/missing_setup_instructions/README.md
But now I'm stuck with the scanner. After starting it up it says:
$ pipenv run python main.py
Error: Could not access WiFi adapter.
expected str, bytes or os.PathLike object, not NoneType
[========                                          ] 17%   4min 34s leftTraceback (most recent call last):
  File "main.py", line 70, in <module>
    main()
  File "main.py", line 53, in main
    wifi_devices_count = data[1]
IndexError: list index out of range

I also tried it running via sudo as I thought it might have something to do with permissions for accessing the monitoring interface wlan0mon.
Any suggestions would be appreciated <3

So I used Kali for RPI that comes with nexmon included, if I remember
correctly.
Here's some post install steps that I had written down.
1.1 Configure the RPI
kalipi-config
1.2 Change root password
1.3 Change timezone and keyboard layout

Generate new SSH keys

dpkg-reconfigure openssh-server

Update the software

apt update && apt upgrade

Install additional tools (the wireless metapackage)

apt install kali-linux-wireless
5.1 Install Avahi
apt install avahi-daemon
5.2 Enable Avahi
Avahi daemon can be enabled to start automatically in the system start
with the systemctl command below.
systemctl enable avahi-daemon

Restart
$ reboot

Auto login SSH
Enter the command on local machine:
$ ssh-copy-id<EMAIL_ADDRESS>

Install pipenv

pip install pipenv

Enable MongoDB
$ systemctl enable mongod.service

10.1 Install MQTT Broker (Mosquitto)
apt install software-properties-common python-software-properties
apt-add-repository ppa:mosquitto-dev/mosquitto-ppa
apt-get update
apt install mosquitto
10.2 Start Mosquitto
service mosquitto start

Install Redis

apt install redis-server

Install piwheelshttps://www.piwheels.org

Install Node and NPM
$ wget -O - https://raw.githubusercontent.com/sdesalas/node-pi-zero/master/install-node-v.lts.sh
| bash

Install Uncomplicated Firewall (ufw)

apt install ufw
$ ufw enable
All commands starting with $ are run as normal user.
All commands starting with # are to be run as root user (superuser).
On Tue, May 5, 2020, 18:32 Markus Kwaśnicki<EMAIL_ADDRESS>wrote:

As I said I'm trying to write down the setup instructions. Until now I got
this far, see:
https://github.com/mrkskwsnck/rpi-people-counter/blob/missing_setup_instructions/README.md
But now I'm stuck with the scanner. After starting it up it says:
$ pipenv run python main.py
Error: Could not access WiFi adapter.
expected str, bytes or os.PathLike object, not NoneType
[========                                          ] 17%   4min 34s leftTraceback (most recent call last):
File "main.py", line 70, in
main()
File "main.py", line 53, in main
wifi_devices_count = data[1]
IndexError: list index out of range
I also tried it running via sudo as I thought it might have something to
do with permissions for accessing the monitoring interface wlan0mon.
Any suggestions would be appreciated <3
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlexNaga/rpi-people-counter/issues/2#issuecomment-624160810,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLI7XLC46ZX5HBV6JYPSV3RQA5QRANCNFSM4J2EDR7Q
.

As I said I'm trying to write down the setup instructions. Until now I got this far, see: https://github.com/mrkskwsnck/rpi-people-counter/blob/missing_setup_instructions/README.md
But now I'm stuck with the scanner. After starting it up it says:
$ pipenv run python main.py
Error: Could not access WiFi adapter.
expected str, bytes or os.PathLike object, not NoneType
[========                                          ] 17%   4min 34s leftTraceback (most recent call last):
  File "main.py", line 70, in <module>
    main()
  File "main.py", line 53, in main
    wifi_devices_count = data[1]
IndexError: list index out of range

I also tried it running via sudo as I thought it might have something to do with permissions for accessing the monitoring interface wlan0mon.
Any suggestions would be appreciated <3

So I used Kali for RPI that comes with nexmon included, if I remember
correctly.
Here's some post install steps that I had written down.
1.1 Configure the RPI
# kalipi-config

1.2 Change root password
1.3 Change timezone and keyboard layout

2. Generate new SSH keys
# dpkg-reconfigure openssh-server

3. Update the software
# apt update && apt upgrade

4. Install additional tools (the wireless metapackage)
# apt install kali-linux-wireless

5.1 Install Avahi
# apt install avahi-daemon

5.2 Enable Avahi
Avahi daemon can be enabled to start automatically in the system start
with the systemctl command below.
# systemctl enable avahi-daemon

6. Restart
$ reboot

7. Auto login SSH
Enter the command on local machine:
$ ssh-copy-id<EMAIL_ADDRESS>

8. Install pipenv
# pip install pipenv

9. Enable MongoDB
$ systemctl enable mongod.service

10.1 Install MQTT Broker (Mosquitto)
# apt install software-properties-common python-software-properties
# apt-add-repository ppa:mosquitto-dev/mosquitto-ppa
# apt-get update
# apt install mosquitto

10.2 Start Mosquitto
# service mosquitto start

11. Install Redis
# apt install redis-server

12. Install piwheelshttps://www.piwheels.org

13. Install Node and NPM
$ wget -O - https://raw.githubusercontent.com/sdesalas/node-pi-zero/master/install-node-v.lts.sh
| bash

14. Install Uncomplicated Firewall (ufw)
# apt install ufw
$ ufw enable

All commands starting with $ are run as normal user.
All commands starting with # are to be run as root user (superuser).

I created a pull request with my setup instructions based on Raspbian. So far it seems to work well. Despite the fact, that on Raspbian you need to compile WiFi driver/firmware by yourself, in order to make scanning in monitor mode possible. In hindsight I would everyone else recommend to use Kali Linux in the first place, as it is also based on Debian.
As you recommended, I decided to look for a different (simpler) approach to scan for wireless devices for my project. However, your project helped me a lot to understand how to build a scanning device.

Yeah I won't be able to upload the rewrite of the code until next week.
You can email me if you have anything you want to discuss regarding scanners :)

Closing due to the installation process added to the readme.


label and annotate s2i tekton build

FYI.. this requires an rbac update to label the image
e.g. https://github.com/kubevirt/kubevirt-tekton-tasks/blob/main/docs/tasks-rbac-permissions.md
Describe the behavior changes introduced in this PR
Linked Issues?
resolves # <-- Use this if merging should auto-close an issue
related to # <-- Use this if it shouldn't
Testing Instructions
Please include any additional commands or pointers in addition to our standard PR testing process.
@redhat-cop/mdt

/test 4.10-e2e-openshift

/lgtm

/lgtm too. Are your concerns resolved @mpryc ?


Using hyprcursor in KDE

Hi. I'm working on adding SVG cursors to the KDE compositor kwin:
https://invent.kde.org/plasma/breeze/-/merge_requests/433
and just saw your work. I'd prefer to use your format, and eventually make it a freedesktop.org standard.
There are a few concerns using it in KDE though:

The metadata file is in Hyprlang format, which is unfamiliar to KDE devs/users.

From https://github.com/hyprwm/hyprcursor/blob/main/docs/MAKING_THEMES.md it seems the actual format used is similar enough to a INI/desktop file, so can we use that instead?
Changes needed:
a) Add section headers, e.g. "[General]"
b) Same-named entries like
define_size = 64, image64.png
define_size = 32, image32.png

need to be merged:
define_size = 64, image64.png; 32, image32.png

I'd like to deploy the hyprcursor format of a cursor theme side-by-side with the xcursor format. So it would be good if manifest.hl is merged into xcursor's index.theme (which already has fields for name and description), so a cursor theme can have one index.theme file and both cursors and hyprcursors dir for traditional and cursor-shape-v1-enabled apps.

Instead of librsvg, in KDE we might want to use QtSVG to render SVG files. So can we have:
a) An interface to only get the metadata part, i.e. the SVG/PNG filename, hotspot, etc., but not doing the actual rendering.
b) A build option to only build that interface, not introducing librsvg and cairo dependencies.

What's wrong with hyprlang? If it's "similar enough", why change it to ini?

So, basically, if no manifest.hl, try to find metadata in index.theme?

a) SVG/PNG files are compressed and never saved into a file, so a memory buffer would make more sense IMHO. Especially with svg, it'd be trivial to just pack the svg into a string.
b) makes perfect sense if a) is implemented

What's wrong with hyprlang? If it's "similar enough", why change it to ini?

There's nothing wrong. Just introducing one more config file format to KDE seems, er, less ideal. (OTOH I do realize that using INI would introduce one more config format to Hyprland.)
And I suspect this might become an obstacle when we try to make it a FDO standard.

So, basically, if no manifest.hl, try to find metadata in index.theme? edit: fwiw, having both index.theme and manifest.hl is perfectly fine

That would be fine. I'd like to keep changes minimal to how cursors are deployed (/usr/share/icons directory, index.theme, etc.), so existing code like cursor settings UI can still work with minimal changes.

a) SVG/PNG files are compressed and never saved into a file, so a memory buffer would make more sense IMHO. Especially with svg, it'd be trivial to just pack the svg into a string. b) makes perfect sense if a) is implemented

Yes, that would work.

mmm fdo, let's just say I dislike how the fdo operates.
anyways, I can do 2 and 3 tomorrow, 2 is pretty trivial and for 3 I can make a MR so that you can make sure the API will suit your needs.
For point 1, it's kinda worth noting that this would result in there being two formats accepted, .hl and .ini, which means both dependencies (hyprlang and... whatever ini parser..? idk any libraries there) would be pulled by distros regardless... Even if KDE would only utilize .ini, distros most likely will still compile hyprcursor with support for both (because why wouldnt they, all themes atm are .hl)

Considering it again, I think you don't need to do any change about 2. KDE will just read name and description from index.theme (which has proper l10n translations) as it currently does, not using these fields from manifest.hl.
For 1, how about we all use JSON instead? Unlike INI, it does have a standard. And it's more expressive than INI.
Disregarding FDO. Although Hyprlang makes sense for Hyprland config files, and INI makes sense for KDE config files. But if we see hyprcursor as a common cursor format, that would be adopted by many cursor themes, and used by multiple compositors, then IMHO it makes sense to use a popular config format like JSON.

What about yaml or toml instead of json?

That's good too. I'd slightly prefer json because this is not end-user-facing, so we don't really need the user-friendness from those formats.

YAML I wouldn't want to see, I think it's a mess and does nothing to help the "yet another config lang" situation here. TOML I think would be a good candidate as it's just standardised INI and works nicely for basic flat structures like here.

I've added support for toml via toml++. We use toml in hyprpm anyways and thus I've decided to use that.
It's similar (almost the same ?) to INI, and I am familiar with the deps. Plus, I dislike json. It's ugly :P

I've made a MR for the raw data API.

Is there anything left to do here from my side?
on a tangent, https://youtu.be/fpf_psv42aY?si=Kz952IuEtck4SMIi&t=305 at 5:05, yet another example of how KDE could benefit from hyprcursor :)

Is there anything left to do here from my side?

No. Unfortunately things are a bit stagnant on my side. Sorry for that.

on a tangent, https://youtu.be/fpf_psv42aY?si=Kz952IuEtck4SMIi&t=305 at 5:05, yet another example of how KDE could benefit from hyprcursor :)

Yes. That is actually one of the motivating examples why I want to ship SVG cursors in KDE. The other one is so the user can choose arbitrary cursor size instead of a few pre-defined sizes.

No. Unfortunately things are a bit stagnant on my side. Sorry for that.

Don't worry, I was just asking so that if everything is done, I can close this as completed :)
Should there be anything wanted / needed by KDE, feel free to open another issue. Thanks for considering hyprcursor :)


adapter.js should not pollute the global namespace

From https://github.com/webrtc/samples/issues/297
ibc commented on Dec 13, 2014
No one in the world should define a global function called trace() (and so many other window globals defined in adapter.js).
adapter.js seems to ignore any new design pattern and JS module system (AMD, CommonJS, etc).
Would you accept a PR removing all the global stuff and providing a library suitable to be loaded by any existing JS module loader without polluting the window global namespace?
Sam Dutton
Owner
samdutton commented on Dec 13, 2014
You have a point :).
adapter.js worked OK in the context of the demos it was initially built for but, of course, will cause namespace pollution in larger projects.
At the very least it would be great to refactor it from an object-oriented approach with no globals.
@juberti @KaptenJansson @phoglund WDYT?
Iñaki Baz Castillo
ibc commented on Dec 13, 2014
I mean something that can be loaded with npm install webrtc-adapter (and borwserified with browserify), and Bower, and so on.
Iñaki Baz Castillo
ibc commented on Dec 13, 2014
Let me know if you are interested and I will provide a PR.
Justin Uberti
Owner
juberti commented on Dec 14, 2014
yeah, I think it's time for adapter.js to grow up. Would you mind changing the title of this issue to be more descriptive?
And please provide an initial PR for the adapter - we'll then have to do a subsequent PR for changing all of our sample code.
Iñaki Baz Castilloibc changed the title from Please build modern JS to Make adapter.js a modern JS library (module systems aware) on Dec 14, 2014
Iñaki Baz Castillo
ibc commented on Dec 14, 2014
I will start in my own repo by creating a JS "project" (not a single adapter.js file), which will include Grunt/gulp tasks for linting, browserifying, etc.
Let me some time please, I will notify it here when done.
Sam Dutton
Owner
samdutton commented on Dec 14, 2014
Thanks for doing this!
adapter.js will be happy to get some attention :).
…
Justin Uberti
Owner
juberti commented on Dec 14, 2014
Before spending too much time, it would be good to get a high-level overview of what you expect it to look like in the end. I am all for avoiding global namespace pollution but I want to make sure we're not biting off too much here.
Christoffer Jansson
Owner
KaptenJansson commented on Dec 14, 2014
SGTM
Will
willscott commented on Dec 15, 2014
FYI - I've begun maintaining a CommonJS library to do something similar: https://www.npmjs.org/package/webrtc-adapter
Iñaki Baz Castillo
ibc commented on Dec 15, 2014
@willscott found that yesterday, I will do something similar, but including a new JS namespace to place in there all the util functions exposed by the current adapter.js.
Will
willscott commented on Dec 15, 2014
@ibc If I understand, you want to see createIceServer & similar explicitly exported, right?
My motivation was to be able to pull in adapter via browserify, and have it find what's in scope via typeof rather than probing navigator, which isn't available for firefox addons.
If the intermediate utility functions are useful, I'd be happy to see those exported.
Iñaki Baz Castillo
ibc commented 26 days ago
Hi, finally I've taken a different approach. It covers more than what adapter.js provides as it is a full wrapper over the RTCPeerConnection interface (along with all the other WebRTC interfaces and functions). It is loadable via Node+browserify, AMD, RequireJS, etc, and it does not pollute the global (window) namespace, even stuff like RTCPeerConnection, etc. Instead it provides a single entry module called "rtcninja":
https://github.com/ibc/rtcninja.js
The documentation is still missing however.
Justin Uberti
Owner
juberti commented 26 days ago
Thanks. 2KLOC seems kind of heavy though for this shim lib. I think we probably want to have something more like willscott's webrtc-adapter.
Perhaps the documentation will make the value more clear, but I was also somewhat wary of the various timers I saw in your implementation.

Next step on this should be to make webrtc a global object that exposes .detectedVersion, .detectedBrowser, .trace, .attachMediaStream, .reattachMediaStream.
After that, we should clean out legacy cruft like createIceServers.

createIceServers should be obsolete in FF37. @jesup?

Does this include not overwriting window.RTCPeerConnection, window.RTCSessionDescription etc.?
IMHO that should be optional. If there is another library using WebRTC on a page it should be able to do its own fixing / detection.

the point of adapter.js is hiding those differences -- you can still get the original browser from webrtcDetectedBrowser et al. Works like charm for simplewebrtc (or rather it's lowlevel modules)

There are better and independent modules to get the exact browser version and platform such as bowser.
rtcninja uses it and exports it, and it does not create any global (but window.rtcninja if no JS modern loader is detected).

This will be covered in #164.

Fixed in #164.


Re-enable ShulkerBoxTooltip integration

This pull request re-enables the integration with ShulkerBoxTooltip.
The crash was caused by one of my dependency being included as transitive.
I set transitive to false in Gradle and updated the README of SBTT accordingly.

Already merged duplicate pull request


Is this project alive?

If yes, what is its state?
If not, do you know other similar project?

It's pretty dead.  I wrote it to scratch an itch but I ended up solving it differently.
Maybe try https://github.com/rustbridge/neon?

Thanks!
But I'm searching a way to execute a .js file from rust code. The idea is to have a server in Rust and let user to write plugins in JS.

Perhaps you can integrate https://github.com/servo/rust-mozjs somehow?

Ohhh.
Yes! I will research it. Thank you very much! :D

David, any luck? I'm considering Rust with JS. Memory management, framework, performance, and concurrency looks great. Interesting combination, since JavaScript is Garbage Collected and Single-threaded, yet Rust is Owned/Borrowed and Multi-threaded. Both high-level Syntax, though.

Sorry  @xoftware but I stopped my project. But the best shoot (I think) is the spider monkey engine. They are using it now for servo browser, then I think you can use it. @bnoordhuis gave me the link here. Maybe you can see how servo implement it.
Resources:
https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey
https://github.com/servo/rust-mozjs
https://github.com/servo/servo

@xoftware what about emscripten with rust?

I recently stumbled across this v8.rs repository while looking for V8's C++ bindings exposed to Rust-land so that I could do V8 C++ stuff from Rust across the Rust/C++ boundary. I eventually found the Deno rusty_v8 project https://github.com/denoland/rusty_v8 which did what I wanted. I think adding a link to https://github.com/denoland/rusty_v8 on the readme and/or archiving this repository would be a good idea to point other Google travelers to the right place.

Done. Interesting that people still end up here.


Requiring module error

Hello i am getting this error when i import import {TabView} from 'showtime-tab-view';
Error: Requiring module "node_modules\showtime-tab-view\src\index.tsx",
which threw an exception: TypeError: undefined is not a function

@abdullahIsa can you share an example with me?

@abdullahIsa can you share an example with me?

I am following the example, using latest version "showtime-tab-view": "^0.1.2" whenever i enter the screen to see the result of DiscussionProfileTest it just throws errors regarding showtime-tab-view
import React, {useCallback, useState} from 'react';
import {StatusBar, Text, View} from 'react-native';
import {useSharedValue} from 'react-native-reanimated';
import {TabView} from 'showtime-tab-view';
import {TabFlashList} from './TabFlashListComponent';

const StatusBarHeight = StatusBar.currentHeight ?? 0;
const TabScene = ({route}) => {
  return (
    <TabFlashList
      index={route.index}
      data={new Array(20).fill(0)}
      estimatedItemSize={60}
      renderItem={({index}) => {
        return (
          <View
            style={{
              height: 60,
              backgroundColor: '#fff',
              marginBottom: 8,
              justifyContent: 'center',
              alignItems: 'center',
            }}>
            <Text>{`${route.title}-Item-${index}`}</Text>
          </View>
        );
      }}
    />
  );
};

const DiscussionProfileTest = () => {
  const [isRefreshing, setIsRefreshing] = useState(false);
  const [routes] = useState([
    {key: 'like', title: 'Like', index: 0},
    {key: 'owner', title: 'Owner', index: 1},
    {key: 'created', title: 'Created', index: 2},
  ]);
  const [index, setIndex] = useState(0);
  const animationHeaderPosition = useSharedValue(0);
  const animationHeaderHeight = useSharedValue(0);

  const renderScene = useCallback(({route}) => {
    switch (route.key) {
      case 'like':
        return <TabScene route={route} index={0} />;
      case 'owner':
        return <TabScene route={route} index={1} />;
      case 'created':
        return <TabScene route={route} index={2} />;
      default:
        return null;
    }
  }, []);

  const onStartRefresh = async () => {
    setIsRefreshing(true);
    setTimeout(() => {
      console.log('onStartRefresh');
      setIsRefreshing(false);
    }, 300);
  };
  const renderHeader = () => (
    <View style={{height: 300, backgroundColor: '#000'}}></View>
  );

  return (
    <TabView
      onStartRefresh={onStartRefresh}
      isRefreshing={isRefreshing}
      navigationState={{index, routes}}
      renderScene={renderScene}
      onIndexChange={setIndex}
      lazy
      renderScrollHeader={renderHeader}
      minHeaderHeight={44 + StatusBarHeight}
      animationHeaderPosition={animationHeaderPosition}
      animationHeaderHeight={animationHeaderHeight}
    />
  );
};
export default React.memo(DiscussionProfileTest);

looks like it's related to Reanimated, what's your version of reanimted?

using latest "react-native-reanimated": "^3.1.0"

@abdullahIsa  gotchat, it doesn't support Reanimated v3 yet because of there is a issue with this, i'm have already created a issue report here https://github.com/software-mansion/react-native-reanimated/issues/4405, so you have to downgrade it to v2 before you can use it.

@abdullahIsa gotcha, it doesn't support Reanimated v3 yet because Reanimated has an issue with this, i'm have already created a issue report here software-mansion/react-native-reanimated#4405, so you have to downgrade it to v2 before you can use it.

ah i see but dont think i can downgrade as other things using it too, once fixed please ping here will test out, thanks.

yeah so incompatibility with Reanimated v3 and react-native-gesture-handler > 2.8.0 makes a difficult case for adoption.
Great lib tho

Good news! Reanimated has already fixed this issue, but they haven't released the fix yet! I'm already updated example code and tested it, working well.
If you guys need it, just use this patch and it will work!
https://github.com/showtime-xyz/showtime-tab-view/blob/v0.1.4/example/patches/react-native-reanimated%2B3.1.0.patch


missed -java version of 2.0.17 release

it seems: https://rubygems.org/gems/psych

Wat! I'm really sorry about that.  I'll cut a -java gem shortly.

Pushed. Really sorry about that.


⚠ Fortnite has degraded performance

In 9792f3d, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1493 ms

Resolved: Fortnite performance has improved in 6da0f8d.


requirements:bugfix - now returns error instead of panicking when fail

Signed-off-by: Ian Cardoso<EMAIL_ADDRESS>
- What I did
- How to verify it
- Description for the changelog

Fix lint errors @iancardosozup


can't add and remove layer

run into error:
[object] has other than one matching layers

@wwwtyro this is becoming a critical issue for Shan's app.
The issue is related to the layer needing to register a new Program etc, it always seemed hard to fix in PhiloGL since it uses a declarative approach for setting up programs making it a little tricky to figure out how to add and remove them after initialization.
This should be much more straightforward to fix using luma.gl once the ongoing port is complete.
@heshan0131 We might want to extend the example code to add and removes layers in a way that reproduces the issue.

:+1: It'd be great if I can get a reproducible example.

@wwwtyro I will edit the example code, and do a pull request?

That'll work, or you can just make a branch and push it and tell me where it's at so I can try it. :)

yes will do

@wwwtyro
example added in
https://github.com/heshan0131/deck.gl/tree/dynamic-layer-example

Thanks! I'll make sure it works when I get luma.gl working with deck.gl. :)

@heshan0131 Please retest this on the performance branch. Layer handling has been rewritten so if it is still an issue let us fix it in the right place.


rename the main Enter key to Return

A keyboard does not have 2 enter keys although some manufacturers got lazy and did not understand they were different.
The enter key on the main section is actually the Return key (carriage return, hence the arrow you sometimes see, a through back to type-writers)
https://en.wikipedia.org/wiki/Enter_key
UK Keyboard

US Keyboard

Proposed solution
Rename the main Enter key to Return

That's a good point.  I renamed the key from "Enter" to "return sign".
Thanks.

its the small things in life 😄
Thanks

any chance you can put a tail on it, looks like a delete/backspace key


Error when trying out the books' code samples

I have the following error  when executing the books' code samples using Node.js version 14.16.0 with NPM version 6.14.11 on Powershell 7.1.3  Windows 10 :
`PS C:\Users\uzabu\dev\learn\microfrontend> npm run 01_pages_links

<EMAIL_ADDRESS>01_pages_links C:\Users\uzabu\dev\learn\microfrontend
concurrently --names "decide ,inspire" "mfserve --listen 3001 01_pages_links/team-decide" "mfserve --listen 3002 01_pages_links/team-inspire" "wait-on http://localhost:3001/product/porsche && opener http://localhost:3001/product/porsche"

[decide ] Error occurred when executing command: mfserve --listen 3001 01_pages_links/team-decide
[decide ] Error: spawn cmd.exe ENOENT
[decide ]     at Process.ChildProcess._handle.onexit (internal/child_process.js:269:19)
[decide ]     at onErrorNT (internal/child_process.js:465:16)
[decide ]     at processTicksAndRejections (internal/process/task_queues.js:80:21)
[inspire] Error occurred when executing command: mfserve --listen 3002 01_pages_links/team-inspire
[inspire] Error: spawn cmd.exe ENOENT
[inspire]     at Process.ChildProcess._handle.onexit (internal/child_process.js:269:19)
[inspire]     at onErrorNT (internal/child_process.js:465:16)
[inspire]     at processTicksAndRejections (internal/process/task_queues.js:80:21)
[2] Error occurred when executing command: wait-on http://localhost:3001/product/porsche && opener http://localhost:3001/product/porsche
[2] Error: spawn cmd.exe ENOENT
[2]     at Process.ChildProcess._handle.onexit (internal/child_process.js:269:19)
[2]     at onErrorNT (internal/child_process.js:465:16)
[2]     at processTicksAndRejections (internal/process/task_queues.js:80:21)
[2] wait-on http://localhost:3001/product/porsche && opener http://localhost:3001/product/porsche exited with code -4058
[inspire] mfserve --listen 3002 01_pages_links/team-inspire exited with code -4058
[decide ] mfserve --listen 3001 01_pages_links/team-decide exited with code -4058
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>01_pages_links: concurrently --names "decide ,inspire" "mfserve --listen 3001 01_pages_links/team-decide" "mfserve --listen 3002 01_pages_links/team-inspire" "wait-on http://localhost:3001/product/porsche && opener http://localhost:3001/product/porsche"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>01_pages_links script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\uzabu\AppData\Roaming\npm-cache_logs\2021-04-27T07_42_34_137Z-debug.log`

Hi @uzabumuhire,  I've update the concurrently dependency. The previous version had a similar error (https://github.com/kimmobrunfeldt/concurrently/issues/232) that your report. Can you check if this works for you now.

After pulling your updates, I was still having the same issue after some search on the internet I was able to resolve the issue by adding  C:\Windows\System32 to the System PATH environment variable (Source : https://stackoverflow.com/questions/57054403/problem-with-npm-start-error-spawn-cmd-enoent)

Glad that it works. Thanks for the info @uzabumuhire

@naltatis,  thanks a lot for taking your time to help me.


Use generator service to register Azure Storage destination type.

What this PR changes/adds
This PR aims to fix the addition of Azure Storage destination type by the generator service in the initialiaztion of the extension.
Why it does that
With the removal of sourcetypes properties from the dataplane selector logic, now the destination type must be registered during the boot and so this change allows that for the Azure Storage type.
Further notes
Tests were added for existing class to promote quality and consistency.
Linked Issue(s)
Closes https://github.com/eclipse-edc/Technology-Azure/issues/236

@bmg13 pls fix the dash check and then we can merge


dynamic import not correct

Describe the bug
inside @web3-react/walletlink-connector package, it dynamic import walletlink

and it gives an incorrect result it wrap one more layer with default

and if do import out of node_modules, it's alright

Reproduction
import { WalletLinkConnector } from '@web3-react/walletlink-connector';

new WalletLinkConnector({
    supportedChainIds: ALL_SUPPORTED_CHAIN_IDS,
    url: '/',
    appName: 'some name',
})

System Info
System:
    OS: macOS 11.5.2
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 87.66 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 17.0.1 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 8.1.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 95.0.4638.69
    Firefox: 93.0
    Safari: 14.1.2
    Safari Technology Preview: 15.0
  npmPackages:
    @vitejs/plugin-legacy: ^1.4.3 => 1.6.2
    @vitejs/plugin-react-refresh: ^1.3.6 => 1.3.6
    vite: ^2.3.7 => 2.6.13

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

while that works well in production build, so did i miss some development configs?

ok, it works well after adding this config


fix(ngAnimate): ensure that minified repaint code isn't removed

Closes #9936
1.2 / 1.3 / 1.4

LGTM.  @Narretz does this work for you?

@petebacondarwin @Narretz's code was having issues with #11336. So this PR won't be compatible with the issues he's facing. However this issue still does exist for other versions of Angular and this patch corrects that.

I'll check in 30 minutes 😊
----- Ursprüngliche Nachricht -----
Von: "Pete Bacon Darwin"<EMAIL_ADDRESS>Gesendet: 31.03.2015 08:43
An: "angular/angular.js"<EMAIL_ADDRESS>Cc: "Martin Staffa"<EMAIL_ADDRESS>Betreff: Re: [angular.js] fix(ngAnimate): ensure that minified repaint codeisn't removed (#11469)
LGTM. @Narretz does this work for you?
—
Reply to this email directly or view it on GitHub.

Fine by me
On 31 Mar 2015 08:04, "Matias Niemelä"<EMAIL_ADDRESS>wrote:

@petebacondarwin https://github.com/petebacondarwin @Narretz
https://github.com/Narretz's code was having issues with #11336
https://github.com/angular/angular.js/pull/11336. So this PR won't be
compatible with the issues he's facing. However this issue still does exist
for other versions of Angular and this patch corrects that.
—
Reply to this email directly or view it on GitHub
https://github.com/angular/angular.js/pull/11469#issuecomment-87968655.

Ah, I didn't see this was a new issue. Interesting, I never had this problem in Firefox before. At least not as visible.

MERGED
Landed as:
https://github.com/angular/angular.js/commit/c55a494433e619aad0c7ef9fddadc0b3fdf53915
https://github.com/angular/angular.js/commit/d5c99ea42b834343fd0362cfc572f47e7536ccfb
https://github.com/angular/angular.js/commit/b041b664752e34a42bbc65e02bf0009f0836c50c


#144 separate k3s and rke2 template execution

Closes: #144
This PR refactors the gen_release_notes command.
It is trying to accomplish a few things:

Remove unused code
Better separate rke2 and k3s.
Move the logic out of the templates
Refactor the Markdown table white spaces

After adding the Markdown table white spaces changes, I'm not sure if it's worth all the extra code and complexity, I'd like some comments on it, you can compare the changes  here: https://github.com/rancher/ecm-distro-tools/compare/be986befcaac41e6dad12da2a936d42fd13daac2..731b8ceb8974ec6377e5655f68a097db778dbbe7

Except for the comments, I think this PR works and can be merged. @Tashima42 , if you don't care for the markdown table utility, definitely feel free to remove it, especially since it'd make this PR even simpler.


FormController $setValidity documentation: signature doesn't match implementation signature

I'm submitting a ...

[ x ] bug report

Current behavior:
Documentation for $setValidity shows its signature as
$setValidity();
Implementation actually takes three arguments: the validation error key, the state to set the key to, and a "controller" third argument that (based on how it's used) appears to be the controller for the edit form.
Expected / new behavior:
Please update documentation to match implementation. A code snippet would also help to clarify the meaning of the arguments.
Angular version: master branch

I was about to update the docs, when I realized I don't know any good usecase for this method. It updates the validity on a form (and parent forms), but without affecting the control.
Has anyone used this or has ideas on how this could be useful?

@gkalpak Could it be that this issue is about what you're saying? https://github.com/angular/angular.js/issues/13202

We should probably update the docs regardless of any considerations about setting child form validity. The whole form / model API is messy in this regard.
What's the 3rd argument anyway? The parent controller?

What's the 3rd argument anyway? The parent controller?

It is the child controller (the one that has/had the error).

It looks like this function is only useful in propagating the state of a control (ngModel) to its parent form(s).

Not sure what you mean. This function is for updating the error/validity of a form (based on that of a child controller; NgModelController or FormController) and propagate that upwards (e.g. in parent FormControllers). Just to be clear, we are talking about FormController#$setValidity(), not NgModelController#$setValidity(). (The latter one is properly documented.)

TBH, I only see this being useful for internal use (i.e. to be called from the control's $setValidity method in order to propagate the error/validity upwards). I can't think of another, external usecase (but maybe I am missing something).

Not sure what you mean.

I think we mean the same. $setValidity is usually called when a child control (ngModel) changes validity and informs the parent form.
We still can add the info that this is a function for internal use (as it has been exposed in the API for a long time)

Yeah, we probably mean the same. (But I would like to point out that there are two different $setValidity methods, even if they are created using the same helper and thus share a big part of their implementation. 😛)
OOC, @fixermark do you have a usecase for FormController#$setValidity()? Were you trying to use it? What for?

We have a situation where we are using a directive to modify the validity of a form element, and as a side-effect of checking the validity of that element, we also need to modify the validity of a second element that determines its validity based on the value of the first element.


Rename memory settings

@TomNicholas said in https://github.com/tomwhite/cubed/issues/160#issuecomment-1533846337

As an aside I find it a bit confusing that max_mem sometimes means "maximum memory the computation is allowed to use" (i.e. actual usage) and sometimes means "maximum memory the computation is projected to use" (i.e. a conservative estimate). For example the DAG visualization just shows me the number I entered as max_mem, which isn't particularly helpful here. It would be more helpful if it showed two numbers, i.e. projected_memory_use / max_allowed_mem, so I can see if I will exceed it or how much headroom I have.

See also #58

How about renaming max_mem to allowed_mem and required_mem to projected_mem?

I'm toying with the idea of whether to include reserved_mem in the allowed_mem and projected_mem numbers.
As shown on https://tom-e-white.com/cubed/computation.html#memory, "reserved_mem is the amount of memory reserved on a worker for non-data use - it’s whatever is needed by the Python process for running a task". The reason to include it in allowed_mem is because when you specify memory limits in runtime systems like Lithops, Modal, and Beam you give a total for what your tasks need, which includes the part for holding the arrays in memory, and the non-data part.
If we did this, then we'd have the following invariant:
measured_mem <= projected_mem <= allowed_mem

rather than
measured_mem <= projected_mem + reserved_mem <= allowed_mem + reserved_mem

This would mean that figures for projected_mem in the visualization and so on might be a bit harder to unpick because they would include reserved_mem, but if we showed that value too then maybe it would be OK.
Note that the user has to specify reserved_mem, since it varies by runtime, and basically has to be measured (by calling measure_reserved_memory). We might be able to come up conservative defaults for different runtimes though (e.g. I just measured Lithops as 38MB).
(Also perhaps reserved_mem would be better named runtime_overhead_mem?)
Any thougths @TomNicholas?

The reason to include it in allowed_mem is because when you specify memory limits in runtime systems like Lithops, Modal, and Beam you give a total for what your tasks need, which includes the part for holding the arrays in memory, and the non-data part.

This makes sense. Many users won't immediately appreciate this distinction, and will probably first try just setting allowed_mem to their total RAM. (cubed could maybe then even measure the system RAM, and default allowed_mem to 80% of it or something...)

we'd have the following invariant

I like that a lot.

This would mean that figures for projected_mem in the visualization and so on might be a bit harder to unpick because they would include reserved_mem, but if we showed that value too then maybe it would be OK.

I don't think that's a big problem. You could imagine showing a color-coded filled line chart, something like this

It's a bit awkward also because runtime_overhead_mem is projected rather than measured. If you were able to measure reserved_mem (or "overhead_mem") dynamically it would look more like this

(gist for both plots)

(Also perhaps reserved_mem would be better named runtime_overhead_mem?)

I prefer that term, though reserved does get across the idea that you set it rather than it being measured. Really it's the "reserved memory for runtime overhead" isn't it.

Great diagrams! I have tried to add something like this #180.


Feature make a pre-release on request

Idea: create a pre release on the subcommand createRelease with the -preRelease Flag

done by command awesome-ci release create ...


fix assert connected timeout

Pull Request Test Coverage Report for Build 512952718

15 of 15 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+11.5%) to 69.492%

Totals

Change from base Build 507708059:
11.5%

Covered Lines:
164

Relevant Lines:
236

💛  - Coveralls

Pull Request Test Coverage Report for Build 512952718

15 of 15 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+11.5%) to 69.492%

Totals

Change from base Build 507708059:
11.5%

Covered Lines:
164

Relevant Lines:
236

💛  - Coveralls


Meta-issue: TeachableAgent loose ends

This meta-issue covers the following tasks wrapping up details of TeachableAgent:
### Tasks
- [ ] #450
- [ ] #532
- [ ] #251
- [ ] #329

All sub-tasks completed.


JKCalendarScrollView issues with month change

Hi,
First of all this is a great library and thanks for that.
I've used JKScrollview in my app and it works fine until expand and collapse happen.
When expand or collapse happen, calendar view month change is not changing (actually previous ad next functions triggers).
I've checked why is this happening and it seems like index issue. Could you please assists me on this issue how to resolve it.
Regards,
Isuru

Hi,
Thank you for your praise!
Because I can't re-generate this issue, please provide your code and more details.


Add logic control to raw method and tests for corresponding codes

overview
Make the raw method more stable, and write some tests for it.
task

[x] when the query method is not defined, it should return a promise;
[x] when the query method is not a function, it should return a promise;

issue
As I have pointed out, when I

git clone
cd feathers-elasticsearch
npm install
npm test

it always fails. I figure out the reason: my machine runs 5.5.1 ES but in test file, api version is set to be 2.4 since my process.env.ES_VERSION is undefined. After I ran ES_VERSION=5.5 npm test, things are all good. I am not sure if we should integrate this into our test file, or point it out at somewhere in docs.

The whole point of setting ES_VERSION is that we can run the tests against multiple versions of ES and the environmental variable is pretty good way of dealing with it on Travis. I agree that we should point it out in the documentation.

@jciolek Thank you for your detailed comment : )


Week view behaves weird on offline mode

If the device has no internet connection, the user will always get a message with no internet connection. I think he should only get it if he tries to sync or if he reorders a specia week which is not present in offline globals.

Done.


Fast socket2

This is mostly a crazy experiment to see what perf gains can be achieved by bypassing Writable. If anyone wants to play around with profiling and experimenting with this.
Interestingly enough there doesn't seem to be much performance to gain here.
http - keepalive x 5,795 ops/sec ±1.78% (275 runs sampled)
undici - pipeline x 8,520 ops/sec ±1.48% (274 runs sampled)
undici - request x 12,594 ops/sec ±0.63% (277 runs sampled)
undici - stream x 13,935 ops/sec ±0.58% (279 runs sampled)
undici - dispatch x 14,553 ops/sec ±0.76% (276 runs sampled)
undici - noop x 22,110 ops/sec ±1.09% (257 runs sampled)

http - keepalive x 5,757 ops/sec ±1.54% (275 runs sampled)
undici - pipeline x 8,751 ops/sec ±1.51% (276 runs sampled)
undici - request x 12,279 ops/sec ±0.53% (280 runs sampled)
undici - stream x 13,631 ops/sec ±0.48% (280 runs sampled)
undici - dispatch x 14,388 ops/sec ±0.40% (279 runs sampled)
undici - noop x 20,883 ops/sec ±0.69% (265 runs sampled)

Something around 5%? Probably not worth it. Have you tried going with fs.write and writev and bypassing the wrap?

Have you tried going with fs.write and writev and bypassing the wrap?

I can use those to write to the handle? Was not aware of that.

You can use those to write to any file descriptor.

question is though how to access the socket fd

hm ,seems significantly slower 😕


Check if error via ListKeys of Storage Interface.

Signed-off-by: Tomoya Fujita<EMAIL_ADDRESS>/kind bug
What this PR does / why we need it:
Minor error check via LiseKeys of Storage Interface call.
Which issue(s) this PR fixes:
N.A (it just fixes debug log for user experience.)
Special notes for your reviewer:
I am not sure what kind of verification process or procedure required to this PR.
Could you provide the guidance about it?

lgtm, but seems e2e failed by accident, may need retest

@Congrool can you kick the test on your side? not sure how to kick it though.

I also cannot trigger retest. But you can git commit --amend --no-edit --sign-off and push to github to trigger manually.

/approve


Kk snow edc

@ako This is the draft PR for the external database connector - Snowflake updates. It's not ready to publish yet - I've got a few questions that I must answer before finalizing the work. Can you take a look at the Files tab and let me know?
FYI @mx-awsdevteam @quinntracy

@ako Thank you for your help! The PR is now ready for your review.
@mx-awsdevteam Please also let me know if you have any comments.

@rik-bos Here's the PR, @ako must still approve it so it's not finalized yet but FYI.


Burban writing style guide

Attention (CC)

Summary of changes
Description of the changes included in this pull request. Steps to follow to confirm the feature is completed or the bug/issue is resolved. Are there any dependencies? Does this code break any existing functionality or introduce any risk?
Motivation & Context

Feature: [ticket number](link to project management ticket)
Issue: #<add number> _references to issues
Related: #<add number> add any related pull requests numbers

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[x] Documentation Update
[ ] Other (please describe):

Does this introduce a breaking change?
Breaking change is defined as "A fix or feature that would cause existing functionality to change."

[ ] Yes
[ ] No

Checklist:

[ ] My code follows the code style of this project.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added/changed necessary documentation (if appropriate)
[ ] All new and existing tests passed.

How Has This Been Tested?

Screenshots (if appropriate):
Further comments

hi, WSG is ready for review!


Dataset Provenance for Authenticity Confidence

Is your feature request related to a problem? Please describe.
Verifiable version history of a dataset can install confidence for authenticity and quality. Stakers and dataset consumers can be more confident in a dataset if they can view a verifiable version history that includes the genesis, revision, and updates to a dataset.
Describe the solution you'd like
Provenance tracking for datasets is missing on Ocean Market. I envision a simple drop-down UI that shows the version history of the dataset, including detailed information for each commit, timestamps, and perhaps signed tags by the owner to validate updates.
An example of dataset versioning is nicely done with datalad: http://handbook.datalad.org/en/latest/index.html
Describe alternatives you've considered
Current alternative solutions for ensuring dataset quality include verification with 3box profiles. A simple in-house solution that doesn't depend on external software would be to include a version history of a dataset. This would work well with other authentication sanity checks.
Additional context
Following up on the conversation from OceaDAO Townhall 24, Mar

This is a good topic but unfortunately no easy solutions. Thank you for your feedback


small typo in docs :black_nib:

small typo in django-admin docs.

This looks correct as is. The proposed change makes it a demonstration of kwargs rather than args.


added boundingRadius property to Body type

👍 thanks!

Thanks to you for this typing! :) I'm making good use of it.

Good to see someone getting use from it! I never got to use it in the end, it may be fairly outdated though as you may have noticed.

Don't worry, CannonJS itself didn't change since mid of 2016 haha.


Ripple Color per tab

It would be useful to override the Ripple Color for a specific tab. In a pre-alpha version, this was possible because the Ripple was part of the Tab; but I had to move the Ripple outside of the Tab (so overflow visible works on Android), thus I removed the per-tab Ripple Color Configuration.
A dark ripple is often better with lighter background colors.

This issue can be closed, yeah?

@wswoodruff Maybe a bit of confusion here because there are two Ripple Animations: the Background Transition backgroundColor and the Touch Feedback rippleColor. This issue is related to the Touch Feedback, which currently isn't configurable for each Tab.
I'd like to add this as a feature, but currently don't have the time to add features to this project.

Ah, yeah I do notice a little funkiness there sometimes, it's not bad though, I'm using react-native-material-bottom-navigation in 2 recipes on my react-native boilerplate [strangeluv-native] :+1:
Have you noticed an issue where a FlatList or SectionList will render on top of the tabs if it has enough items to reach the bottom? I'm running into that on react-native 0.43.4

@wswoodruff Sorry, I don't use SectionList / FlatList in conjunction with react-navigation. Normally this shouldn't happen. I hope react-navigation didn't change the way the tabBarComponent integrates into react-navigation.

Whoo! Boy react-navigation v1.0.0-beta.9 is a dooozy. There's some breaking changes with config, specifically 'Navigation options are now flat', so header: { tintColor: 'green' } is now headerTintColor: 'green'. They just pretty much made a [bunch of breaking changes]

@wswoodruff I'm meaning the way react-navigation internally renders the BottomNavigation, not only the configurations. The docs are already updated to reflect the new way react-navigation handles configurations. By going through the code of react-navigation and react-native-tab-view, the tab contents shouldn't overlay the navigation component, since the TabView has flex: 1 and the navigation component a distinct height.
If this only happens with the BottomNavigation from react-native-material-bottom-navigation and not with react-navigation's integrated Bottom Navigation, please open a new issue.

They actually just fixed the issue with [this commit] (5 commits after the v1.0.0-beta.9 release). I checked out that commit and the issue was gone

@wswoodruff Ah, thank you. :) Relieved to hear that.

Closed with release of v1.0.0


[WORKING ON] Elastic search compatibility

I'm currently working on a possibility to store data either in the database or in a elastic search cluster.
If you have idea or input regarding this feature, please share your thoughts here.

First off, Thanks for working on this.
Some input:

The ability to augment the ES mapping from the standard mapping you might come up with when serializing an APIRequestLog to an ES cluster. Not sure if that is even worth while but might be something to consider.
Ensure data sending to an ES cluster uses the bulk API. However, this depends on volume though.
The ability to use a method override/signal to potentially filter out data I wouldn't care to send to an ES cluster.
A nice to have would be document/create commands to orchestrate management of a cluster. If this requires leaning on another library I think that's fine.

Thanks again.

Hey @avelis,
Thanks for your input.

The ability to augment the ES mapping from the standard mapping you might come up with when serializing an APIRequestLog to an ES cluster. Not sure if that is even worth while but might be something to consider.

For the first shot, I keep everything really simple and as I'm still not a super user of ES (almost never used it before) I don't really see exactly what you mean by that but I'm sure if it's needed we can work on it later

Ensure data sending to an ES cluster uses the bulk API. However, this depends on volume though.

How do you see that ? because I planned to send an event to ES every time a log is generated, otherwise it means that I have to store it somewhere, check how many messages I have and then trigger the bulk

The ability to use a method override/signal to potentially filter out data I wouldn't care to send to an ES cluster.

Is it not how it works already ? We just change where you register the log right ?

A nice to have would be document/create commands to orchestrate management of a cluster. If this requires leaning on another library I think that's fine.

I'll keep it for later if you don't mind, again, I'm not an experimented user of ES so I wouldn't try to go too far for the first version if you don't mind


Authorization Error

Hey everyone,
thanks for this nice tool. I would like to use alpaca paper trading data, but I always get the following error:
nats.aio.errors.NatsError: nats: 'Authorization Violation'
I am just running a basic example with (triple checked) valid api key and api secret for the paper api. Could you point me in the right direction to resolve this issue?
I am using the latest backtrader, Python 3.7 and pipenv.
Any help is appreciated. Thanks!

Why it isn't possible to use alpaca-backtrader-api in paper mode as it described in readme ?? And in fact the code contains paper API urls... But I get nats.aio.errors.NatsError: nats: 'Authorization Violation' error and I want to use just paper-api without access to polygon


Release 3.4.4rc1

Coverage remained the same at 82.688% when pulling 5fec94ca8f92b8ead59f384b75234dd307511092 on czpython:release/3.4.4rc1 into bfbf505c9d5fba858294653bed1ea240ddc5adcb on divio:release/3.4.x.

Coverage remained the same at 82.688% when pulling e9c4c98652dd62f2ef784aaf0c396a38f3bd2109 on czpython:release/3.4.4rc1 into 44ed34b3904c5e81bb024eeab04a00e36c017286 on divio:release/3.4.x.


[Bugs] Run error in ArchLinux

As titled.
I was going to port it to ArchLinux AUR. https://aur.archlinux.org/packages/animeclient-bin
When I run the app,I've got errors:
A JavaScript error occurred in the main process
Uncaught Exception:
Error: Cannot find module 'discord-rpc'

Then I used npm to add discord-rpc ,it run well.
BTW,will you please use logo.png instead of logo.ico in index.js?

I recently got into arch, i will try to fix the error and let you know when its done :)

I rebuilt the app on Arch. It seems to be working fine now. Please let me know if you get any errors.

[2.0.4] is OK now.


Hostname/IP does not match certificate's altnames

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I get this message for some sources added to Uptime Kuma:
Hostname/IP does not match certificate's altnames: Host: sub.example.com. is not in the cert's altnames: DNS:*.duckduckgo.com, DNS:duckduckgo.com

I don't understand what it has to do with the DuckDuckGO?! Sources have their valid wildcard certificates set on their windows server. It happens two/three times per day for only one or two minutes as Down status?
I did troubleshooting on the network side and didn't find anything related.
Could it be related to Kuma?
Where I can find logs of above error for more details?
🐻 Uptime-Kuma Version
1.18.5
💻 Operating System and Arch
Ubuntu 20.04
🌐 Browser
Chrome
🐋 Docker Version
No response
🟩 NodeJS Version
No response

Either the SSL for the domains were setup wrongly or your uptime-kuma setup or monitor has something wrong.
Either way need more information on details.
Could you provide an example of a monitor as well as how you started uptime-kuma?
With docker? Without?

Hi this is a recurring issue every now and then since the instalation. I run too in docker/caprover. Please let me know what details to provide to help fix this.

Yes well we checked. The websites have nothingh in commong to the gitlab server mentioned. Especially not on a ssl level.

Plus for now for months was not occuring and now is back again. The only resolution to this is to temporarily removed ssl check.


Generic functions that have a function as a parameter expand inferred type instead of narrowing

TypeScript Version<EMAIL_ADDRESS>
Search Terms: Generic Type Inference Function Parameter Union
Code
interface Foo {
    foo: number;
}

declare function acceptUnion(foo: Foo | number): void;
declare var foo: Foo;

declare function erroneouslyExpands<T>(value: T, func: (t: T) => void): T;

 // Errors, TS infers the result as `Foo | number`
const test: Foo = erroneouslyExpands(foo, acceptUnion);

Note: This case is common when passing comparators into functions.
Expected behavior:
T should be inferred as Foo.
Actual behavior:
T is expanded based on what func accepts.
Playground Link: https://www.typescriptlang.org/play/index.html#src=interface Foo {
    foo%3A number%3B
}

declare function acceptUnion(foo%3A Foo | number)%3A void%3B
declare var foo%3A Foo%3B

declare function erroneouslyExpands<T>(value%3A T%2C func%3A (t%3A T) %3D> void)%3A%20T%3B%0A%0Aconst%20test%3A%20Foo%20%3D%20erroneouslyExpands(foo%2C%20acceptUnion)%3B%0A
Related Issues:

https://github.com/Microsoft/TypeScript/issues/16107
https://github.com/Microsoft/TypeScript/issues/24747 may reduce to this issue..?

See #14829

However, fixing this causes #19576 to again break. I seems that having #19576 be an error is the lesser of two evils, but it would be nice to somehow keep around contra-variant inferences when the only co-variant inference we have is a never resulting from a freshly created array literal.

I'm still having issues with this, I think.  You can see the error on this Playground example.  Note that bad gives a compiler error.
The problem that lead me here is that const s: BehaviorSubject<number[]> = new BehaviorSubject([]); is an error if I turn strictFunctionTypes on.  ("Type 'BehaviorSubject<never[]>' is not assignable to type 'BehaviorSubject<number[]>'.") I can't imagine that we really want everybody to have to write new BehaviorSubject([] as number[]) all over the place.  Am I missing something?


Embla Carousel - Add tag to project

Hello Michael @michaelrambeau ,
I’d be happy if you would add the React tag to Embla Carousel. Like you suggested some time ago, I’ve created a mono repo so the React package is now part of the core package 📦 .

Thank you for your time!
David

Great move David @davidcetinkaya thank you for letting me know!
OK I'm doing that, it will be online in a few hours, when the site data is rebuilt (it happens everyday at 6:00 Japan time if you're curious, 21:00 GMT)
I can see you use a lot your mobile to browse the app, so if you found anything wrong with the layout, let me know.
It also makes me think that I'd like to add offline access to the app.
One last thing, a few days ago I added the ability to click on project's avatar to open the project details page, to increase the "clickable area", let me know if it makes sense... or if the clickable area is still too small on mobiles, thank you!

Thank you!
Of course, I will let you know if I find more stuff that may need some attention. I need an answer to this question when you have a moment.
Cheers!
David

Hello again David @davidcetinkaya
I was checking your repo and I'm curious about the strategy you followed to set up the mono-repo: it seems you didn't use Lerna and Yarn workspace and there is only one single package.json, so it means there is only a single embla-carousel package on NPM, if my understanding is correct?
(which is not a problem to me, React packages being only devDependencies)

Hello Michael @michaelrambeau!
Your observation is correct 🙂. It’s a single NPM package where you choose to import which Embla implementation you desire (each implementation is compiled into a separate file with rollup.js), as described in the docs. The alternatives like Lerna and similar felt a bit overkill. At least for now. I might change the approach in the future if I run into issues but right now things are working fine and it feels simple.
You’re right. I should brag about the small bundle size more 😅. Thanks for the idea! And I’m glad you like the cats 🐱. What’s your thoughts about the 3D time pickers at the far bottom?
Thanks Michael!
Best,
David

Closing this as resolved. Thanks for your help Michael @michaelrambeau!

Best
David

Hello David @davidcetinkaya
It's your day of glory, you are number 2, congratulations! 🎉
I'm curious about what happened, did you promote Embla on some media?

Hello Michael @michaelrambeau,
Thanks for sharing this! Yes, after rebuilding the documentation and releasing v.3, I decided to share it on reddit. It seems like people appreciate it 😊.
Cheers!
David

That's cool David! @davidcetinkaya
By the way, I can see badge on your README.md... did you know we provide badge too?
Like the one here: https://github.com/tannerlinsley/react-query
We provide different variations (daily, weekly, monthly numbers)
But you may think that it adds to much clutter to your README.md so you can refuse this offer, no problem!

Thank you Michael! @michaelrambeau
Yes I've checked those badges out and they look awesome. At the time of writing Embla Carousel isn't gaining many ⭐ per day (excluding the peak right now caused by the reddit post) so maybe it could have a negative impact on visitors, I don't know. But if people start sharing it more and it gets more traction over time, I'll definitely add a badge.
Thanks Michael 🙂.

Hello Michael (@michaelrambeau),
That's cool, didn't even know javascript weekly mentioned it. Thanks a lot 🙂. The newsletter suggestion is great, go with that. Appreciate it a lot Michael.
Have a great day and thanks a lot. We'll probably be in touch soon again 🙂.
David

Hi @michaelrambeau,
I hope you're doing great 🙂?
Would you consider adding the following tags to Embla Carousel?

React
Vue
Svelte

Because Embla Carousel now supports all of these libraries/frameworks and has wrappers for them. All wrappers are part of the same monorepo.
PS. For whatever reason the React tag was added way back but seem to have disappeared.
Thank you for your time!
Best,
David

Hello David @davidjerleke I'm fine, it's good to see you!
Sorry for the miss about the tags, I'm fixing, it will be online in 24 hours.
I've just noticed Embla is the trendiest slider of the week on Best of JS, it's impressive!

Thank you very much for the help @michaelrambeau ⭐. I'm closing this as resolved:

On a side note
Is this how you prefer to add tags? That is, creating an issue in this repository? Because I browsed bestofjs for a while looking for information about how to request new tags and didn't find anything. It could just be me getting old though so if that's the case, you can ignore this side note 😉.

Thank you for having checked David @davidjerleke
You are 100% right, there is no "official" way to suggest corrections about tags, so contacting us via GitHub is fine.
Or you can join the Discord server where we are supposed to discuss with the other members, you'd be welcome!
(it's a very quiet place actually!)
I should add a note on the top page.
And I guarantee you are not getting old... at least not as much as me 😄
Sharing the screenshot to celebrate your success (number 2 over the last 12 months)

Thank you for your swift response.
I like your "time to celebrate" initiatives, it's very encouraging 🙂.

Or you can join the Discord server where we are supposed to discuss with the other members, you'd be welcome!

Thanks! I'll make sure to join your Discord when I have some input, feedback, or questions next time.
Cheers,
David


ENH: add basic support for M values (parsing, checking dim, accessing values)

With GEOS main (future GEOS 3.12), there is improved support for M values. Those can now be stored under the hood in the CoordinateSequence. The M value isn't used for much (quite similarly as Z values), but we could provide basic functionality to parse/create geometries with M values, check if a geometry has M values and access the M value.
For example, that would support the workflow of parsing WKB/WKT with M and extracting that value. The parsing now works with GEOS main:
>>> geom = shapely.from_wkt("POINT M (1 2 3)")
>>> geom
<POINT M (1 2 3)>

But there is currently no way to access this value 3.
Some possible places to add support for M values:

Add a has_m function and class attribute (mimicking has_z). We should probably ask upstream to add a GEOSHasM to the C API.
Add an include_m keyword in get_coordinates. Currently this function defaults to return 2D, but can return 3D if you specify include_z=True

I was wondering how useful is the M coordinate, as I've never seen it actually used in the wild :)

Add a has_m function and class attribute (mimicking has_z). We should probably ask upstream to add a GEOSHasM to the C API.

As calling has_z and has_m in a row can be wasteful, It might be a good idea to also have a function has_zm that will either return Nx2 bool array (has z, has m, per geom) or  a N int8 array (first bit=has z, second bit=has m, per geom)

Add an include_m keyword in get_coordinates. Currently this function defaults to return 2D, but can return 3D if you specify include_z=True

In the same way, include_z: int | bool could be considered (2 bits), although have a confusing name, could be backwards compatible with include_z: bool without the addition of extra parameter.
Also we need to consider if there is code where coord_dim=3 was considered equivalent to has_z. which now might not be the case with XYM geometries.

The M-value is user-defined, but the most common use is to store time. M-values have had long-standing support within several GIS geometry models, so I'd be happy to see it here eventually too. PostGIS has some good inspiration, for example a LineString [Z]M could add timestamps to each coordinate to create a trajectory (see ST_IsValidTrajectory, ST_DistanceCPA and ST_ClosestPointOfApproach). Other functions like ST_SetEffectiveArea return effective areas from each coordinate to the M-values.

BTW, I've started some of these tasks. Adding has_z is trivial. Adding .m to Point is nearly done. The bigger challenge that I'm looking at (and might need help with) is reworking much of the internal logic related to the 3D coordinates, whether they are XYZ or XYM. For instance https://github.com/shapely/shapely/blob/eeef37fcf398afeb6342d37b63037dc398fa1a5c/src/geos.c#L1007-L1009
The argument dims can be 3, but this doesn't distinguish between the two variants. Should this change to have has_z and has_m parameters instead?
Also the Python signatures, e.g. shapely.points. This could be one of several things:

non-breaking:
points(coords, y=None, z=None, indices=None, out=None, m=None, **kwargs)
breaking, possibly yielding unexpected results:
points(coords, y=None, z=None, m=None, indices=None, out=None, m=None, **kwargs)
breaking, but safer for users to identify fix:
points(coords, y=None, *, z=None, m=None, indices=None, out=None, m=None, **kwargs)

If we are introducing breaking changes for shapely 2.1, should warnings be introduced to the 2.0.* series?
Simpler shapely.Point could be refactored something like this:
shapely.Point(1, 2, 3)  # POINT Z (1 2 3)
shapely.Point(1, 2, 3, 4)  # POINT ZM (1 2 3 4)
shapely.Point(1, 2, None, 4)  # POINT M (1 2 4)

More complicated shapely.LineString:
shapely.LineString([(1, 2, 3), (5, 6, 7)])  # LINESTRING Z (1 2 3, 5 6 7)
shapely.LineString([(1, 2, 3, 4), (5, 6, 7, 8)])  # LINESTRING ZM (1 2 3 4, 5 6 7 8)
shapely.LineString([(1, 2, None, 4), (5, 6, None, 8)])  # LINESTRING M (1 2 4, 5 6 8)
shapely.LineString([(1, 2, 4), (5, 6, 8)], some_option=x)  # LINESTRING M (1 2 4, 5 6 8)

the last example would specify a yet-to-be-decided keyword to signal that the third coordinates are M ordinates (the default would remain Z).
Opinions and example prototypes welcome!

As @idanmiara noted, include_z is already used for get_coordinates in src/coords.c, so it would be consistent to add include_m for retrieving coordinates.

Wouldn't it be easier to modify include_z to accept some enum | bool so we don't need to add include_m to so many functions? This would allow the include_z to properly so only the function that actually parse it would need to be changed.

Also the Python signatures, e.g. shapely.points. This could be one of several things:

non-breaking:
points(coords, y=None, z=None, indices=None, out=None, m=None, **kwargs)
breaking, possibly yielding unexpected results:
points(coords, y=None, z=None, m=None, indices=None, out=None, **kwargs)
breaking, but safer for users to identify fix:
points(coords, y=None, *, z=None, m=None, indices=None, out=None, **kwargs)

If we are introducing breaking changes for shapely 2.1, should warnings be introduced to the 2.0.* series?

I think that we should choose a non-breaking change and at a later version (after we issue a warning ahead) to make all the parameter rename/reorder all at once, there are many relevant cases, just to name one example - ops.transform vs shapely.transform

Looking at low-level C and Cython functions, there are several functions that have the arguments unsigned int dims and assume it is 2 for XY and 3 for XYZ. But obviously this will need to change for 2.1. These functions are:

PyGEOS_CoordSeq_FromBuffer(...) (see src/c_api.h and shapely/_pygeos_api.pxd)
coordseq_from_buffer(...) (see src/geos.h)
coordseq_to_buffer(...) (see src/geos.h)
several force_dims_*(...) functions (see src/geos.c)

Some options to replace the single unsigned int dims arg are:

2 args: int hasZ, int hasM (currently done for #1760)
2 args: bool has_z, bool has_m. Not only is this compact, but if needed unsigned int = (unsigned int)has_z + (unsigned int)has_m + 2; will be one of {2, 3, 4}.
1 arg as an enum like CoordDim as suggested above by @idanmiara. Unlike GEOSGeomTypes, GEOS does not have any built-in enums like "GEOSCoordTypes" to use.

For the low-level (private) coordseq-related APIs (coordseq_from/to_buffer et al), I would go with 2 args, given that this is consistent with the equivalent GEOS functions (and then I personally don't care whether we use the exact same as GEOS with int hasZ style, or bool has_z)
For the public Python-level methods like transform, that's another issue though. There it might indeed be useful to work with a single argumen. Although then we also need to use one with a different name than the current include_z, so that's not necessarily smoother compared to adding a include_m. But longer term a single argument might be the nicest option.

I'll add a major application for the M dimension--linear referencing. Linear referencing is a common data structure for linear asset data such as roadways, pipelines, etc. Much of this involves tabular operations, though some features include management of M-enabled linear geometries as well.
As a transportation data scientist this is quite common in my work. I've developed a package called linref which provides some features related to this data structure. To this point I've used a bulky workaround, building on the MultiLineString geometry type and managing the M dimension in parallel.
Some of my collaborators on that project and I would be excited to have more complete support for M dimension data in shapely and would be happy to support the effort. The work-around module is a bit dated and limited but functional and may be a reference point for potential related features. In particular, the features of greatest value would be extending the existing linear geometry support for interpolate and project methods to account for the M dimension. Additionally, calibration of the M dimension.
Looks like there will be some opportunities to expand in this direction for 2.1 and beyond? Let me know if this is something I can support.


chore: bring release job in line with old maven-publish job

When the mvn commands for building and publishing were moved from the maven-publish workflow to the release workflow, they were changed, which causes the release workflow to fail to publish the metal-java JARs to GitHub Maven.
This updates the release job to use the mvn commands that were in the maven-publish job so that we can successfully release this SDK.


[Proposal] Field types & column types naming

Priority: low low low
I think a very intuitive way to name the field and column type would have been using the column names used in Laravel migrations.
We need to keep the names we have now for backwards-compatibility, of course, but we could create these new fields, too, as aliases of existing fields.
PROs:

easier to understand what each field type represents
dead-simple to generate CRUDs from migrations (we would just place a field/column with the same name as in the migration), either as a setFromMigration() method (alongside setFromDb()) or to have the fields and columns written one-by-one; generating CRUDs from migrations should be simpler than from DB, because there's only one migration type;
consistency in naming with other Laravel parts;

CONs:

not a big fan of the names for migrations...

What do you guys think? Pasting the table with all the names below so it's easier.

Command
Description

$table->bigIncrements('id');
Incrementing ID (primary key) using a "UNSIGNED BIG INTEGER" equivalent.

$table->bigInteger('votes');
BIGINT equivalent for the database.

$table->binary('data');
BLOB equivalent for the database.

$table->boolean('confirmed');
BOOLEAN equivalent for the database.

$table->char('name', 4);
CHAR equivalent with a length.

$table->date('created_at');
DATE equivalent for the database.

$table->dateTime('created_at');
DATETIME equivalent for the database.

$table->dateTimeTz('created_at');
DATETIME (with timezone) equivalent for the database.

$table->decimal('amount', 5, 2);
DECIMAL equivalent with a precision and scale.

$table->double('column', 15, 8);
DOUBLE equivalent with precision, 15 digits in total and 8 after the decimal point.

$table->enum('choices', ['foo', 'bar']);
ENUM equivalent for the database.

$table->float('amount', 8, 2);
FLOAT equivalent for the database, 8 digits in total and 2 after the decimal point.

$table->increments('id');
Incrementing ID (primary key) using a "UNSIGNED INTEGER" equivalent.

$table->integer('votes');
INTEGER equivalent for the database.

$table->ipAddress('visitor');
IP address equivalent for the database.

$table->json('options');
JSON equivalent for the database.

$table->jsonb('options');
JSONB equivalent for the database.

$table->longText('description');
LONGTEXT equivalent for the database.

$table->macAddress('device');
MAC address equivalent for the database.

$table->mediumIncrements('id');
Incrementing ID (primary key) using a "UNSIGNED MEDIUM INTEGER" equivalent.

$table->mediumInteger('numbers');
MEDIUMINT equivalent for the database.

$table->mediumText('description');
MEDIUMTEXT equivalent for the database.

$table->morphs('taggable');
Adds unsigned INTEGER taggable_id and STRING taggable_type.

$table->nullableMorphs('taggable');
Nullable versions of the morphs() columns.

$table->nullableTimestamps();
Nullable versions of the timestamps() columns.

$table->rememberToken();
Adds remember_token as VARCHAR(100) NULL.

$table->smallIncrements('id');
Incrementing ID (primary key) using a "UNSIGNED SMALL INTEGER" equivalent.

$table->smallInteger('votes');
SMALLINT equivalent for the database.

$table->softDeletes();
Adds nullable deleted_at column for soft deletes.

$table->string('email');
VARCHAR equivalent column.

$table->string('name', 100);
VARCHAR equivalent with a length.

$table->text('description');
TEXT equivalent for the database.

$table->time('sunrise');
TIME equivalent for the database.

$table->timeTz('sunrise');
TIME (with timezone) equivalent for the database.

$table->tinyInteger('numbers');
TINYINT equivalent for the database.

$table->timestamp('added_on');
TIMESTAMP equivalent for the database.

$table->timestampTz('added_on');
TIMESTAMP (with timezone) equivalent for the database.

$table->timestamps();
Adds nullable created_at and updated_at columns.

$table->timestampsTz();
Adds nullable created_at and updated_at (with timezone) columns.

$table->unsignedBigInteger('votes');
Unsigned BIGINT equivalent for the database.

$table->unsignedInteger('votes');
Unsigned INT equivalent for the database.

$table->unsignedMediumInteger('votes');
Unsigned MEDIUMINT equivalent for the database.

$table->unsignedSmallInteger('votes');
Unsigned SMALLINT equivalent for the database.

$table->unsignedTinyInteger('votes');
Unsigned TINYINT equivalent for the database.

$table->uuid('id');
UUID equivalent for the database.

My 2 cents: I understand and see the posibilities of having this name schema (for example for the numeric fields), and that would do a +1 to the feature
BUT I really think we have a MVC issue here, those names are great for database schemas, but not really enought for daily "presentation" of form fields, for example maybe you use a varchar text for storing an IP address, but a "addField" varchar for that would need a lot of custom attributes for it to validate. Same goes  for an image cropper, that in the end is just a text field, but needs a lot of frontend customization.
So support for a setFromMigration would be nice as an adition, but I think the long term is having a lot of custom components; not a component per attribute of course, read-only, etc, but image croppers, master-detail views, markdown/wysiwyg editors, etc...
Please excuse my bad english :disappointed:

What @MarcosBL said. Those names are great for databases as those are the column types. But i think for the "addField" we should use what is closer to HTML's "input" types.

Thanks for the input, guys, you make a good point. Let's keep them in mind when we make new fields, but renaming them to fit this pattern... let's drop that like it's hot.
Cheers!


Need use  ServicePort in http01

Hi!
I'm use SLB (server load balancing) server, all my k8s nodes give slb fixed nodeport 32001, so the slb IP:80 can guest the 32001.
(all my web site program use 32001 nodeport.)
so, when i use helm to deploy a web server (whatever is traefik or kong or nginx or something else), cert-manager will build pod name like cm-acme-http-solver-8fjtx, it will listen port 8089, and random build nodeport 32 ***.
I want fixed nodeport 32001 ... can let me slb work again, for cert-manager self-check.
I look the document :  https://cert-manager.io/docs/reference/api-docs/#acme.cert-manager.io/v1alpha2.ACMEChallengeSolverHTTP01Ingress
but the API don't realy have servicePort field.
Here is the kubectl apply -f le-staging.yaml ouput :
error: error validating "le-staging.yaml": error validating data: [ValidationError(ClusterIssuer.spec.acme.solvers[0].http01): unknown field "servicePort" in io.cert-manager.v1alpha2.ClusterIssuer.spec.acme.solvers.http01, ValidationError(ClusterIssuer.spec.acme.solvers[0].http01): unknown field "serviceType" in io.cert-manager.v1alpha2.ClusterIssuer.spec.acme.solvers.http01]; if you choose to ignore these errors, turn validation off with —validate=false

Here is My ClusterIssuer yaml :
apiVersion: cert-manager.io/v1alpha2
# kind: Issuer
kind: ClusterIssuer
metadata:
  name: letsencrypt-staging
  #namespace: default
spec:
  acme:
    # You must replace this email address with your own.
    # Let's Encrypt will use this to contact you about expiring
    # certificates, and issues related to your account.
    email<EMAIL_ADDRESS>    server: https://acme-staging-v02.api.letsencrypt.org/directory
    privateKeySecretRef:
      # Secret resource used to store the account's private key.
      name: letsencrypt-staging
    # Add a single challenge solver, HTTP01 using nginx
    solvers:
    - selector: {}
      http01:
        ingress:
          serviceType: NodePort
          servicePort: 32001 # don't exists API field ,But the document said need to use this
          class: nginx

Help ..

Even if this field did exist, the setup you're working towards will not work - your ingress controller (i.e. ingress-nginx, contour, etc) should be listening on port 32001 - an Ingress resource that is created during the HTTP01 solving process is what actually routes traffic to the Service running on port 8089 (the acmesolver). Only a single Service can have a single nodePort, so if you were to try and do what you're doing above, your actual webserver that serves actual traffic for your website would not be accessible, as the acmesolver would be using port 32001.
You can read more on how ingress-nginx works here: https://kubernetes.github.io/ingress-nginx/how-it-works/
Hope that makes sense! Take a look at our "Securing nginx ingress" tutorial here too, as it will talk you through the entire process end-to-end: https://cert-manager.io/docs/tutorials/acme/ingress/


Should show "Column not found" instead of "Provided label column can't be null" when column not found

How to reproduce it
run an mlnet experiment with unexist label name

Hi @LittleLittleCloud , could you provide the build to verify this issue?
I can repro this issue on ML.NET latest version: 16.7.2 (preview feed):

@vzhuqin
Yeah sure, I'll let you know when its ready


Bump julia1.8

Updates Julia version for compatibility with Julia 1.8

Expect some fails in the tests due to updates in ATools. These are dealt with in PR #33


Sync topnav and fonts to Hugo site for release 2.1

[description]

make changes to topnav template and styling to conform to hugo site for 2.1 release
implement Graphik font in docs to conform with hugo site for 2.1 release

cc: @jseldess

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/8ade77de1b5c2c3933ef1020487dee7d3574ffdf/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b7c2eb63e5f2eae87a789da179586d4c9cc00514/

@jseldess I'm assuming some link checks will break here since the Hugo pages aren't yet deployed

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e46c7f8d6a9edd43bedde4ab51a0dd760ab538f7/

LGTM, though as discussed, it'll be good to polish some font sizes, etc., in the coming days.


Authentication_UI

This is Authentication UI .

Please specify the npm script to run the code:
npm start or npm test

Please specify the npm script to run the code:
npm start or npm test

Sorry For that, it was npm start...


Fix variable names in readme

The docs refer to input which is a variable that does not exist.

Great, thanks!


[EHN] Functorial Data Migration

I implemented functorial data migration between CSets which the functor represented by an object dictionary and a hom dictionary.

Thanks @slibkind, this will be a great feature to have.
Looks good to me. The only issue I see is that this code won't handle data attributes. Let me know whether you'd like to do that extension or whether I should.

Thanks, Evan! I'll add that extension this evening

To some extent we can treat data attributes like homs. For example, this code has the correct output:
@present TheoryLabeledGraph <: TheoryGraph begin
  ELabel::Data
  VLabel::Data

  vlabel::Attr(V,VLabel)
  elabel::Attr(E,ELabel)
end
const LabeledGraph = ACSetType(TheoryLabeledGraph, index=[:src, :tgt, :vlabel, :elabel])

@present TheoryLabeledDDS <: TheoryDDS begin
  Label::Data
  label::Attr(X, Label)
end
const LabeledDDS = ACSetType(TheoryLabeledDDS, index=[:Φ, :label])

d = LabeledDDS{Int}()
add_parts!(d, :X, 4, Φ=[2,3,4,1], label=[1, 2, 3, 4])

LabeledGraph{Number, Int}(d, Dict(:V => :X, :E => :X),
                Dict(:src => id(X), :tgt => :Φ, :vlabel => :label, :elabel => :label))

A nice extension would be to infer the data attribute types from a dictionary of attributes, but I'm stumped about how to proceed.


The type or namespace name 'ExpectedException' could not be found

By using NUnit 3.0, this error occours because the ExpectedExceptionAttribute has been removed. So instead it should be using a construct like the Throws Constraint.
Use this Assert.Throws<Exception>(()=>MethodThatThrows()); instead of this  [ExpectedException(typeof(Exception))] to test for exception
We can see more about NUnit exceptionAsserts in here: https://docs.nunit.org/2.5/exceptionAsserts.html

This is still happening

I Confirm that, to repro just use Unity 2021.3 I think, I fixed it like that
[Test]
public void DuplicateVersionString() {
Assert.That(() => fsVersionManager.GetVersionedType(typeof(VersionedModelDuplicateVersionString)),
Throws.TypeOf());
}
[Test]
public void MissingConstructor() {
Assert.That(() => fsVersionManager.GetVersionedType(typeof(VersionedModelMissingConstructor)),
Throws.TypeOf());
;
}


I get error everytime i try to download a file

whenever i start to download a post, it says:   System.ArgumentNullException: Value cannot be null. (Parameter 'html')
at HtmlAgilityPack.HtmlDocument.LoadHtml(String html)
at PatreonDownloader.Program.DownloadMedia(HttpClient client, CookieContainer cookies, List`1 posts) in E:\Development\PatreonDownloader\PatreonDownloader\Program.cs:line 167
at PatreonDownloader.Program.Main(String[] args) in E:\Development\PatreonDownloader\PatreonDownloader\Program.cs:line 83
can you msg me on discord with a potential fix? Duddy#0028

I think what happened here is that it found a post that doesn't have any text content, and apparently the content property is null in those cases, as opposed to an empty string.
I've released an update, it shouldn't crash anymore. Please try this and close the issue if no more issues appear.

I think what happened here is that it found a post that doesn't have any text content, and apparently the content property is null in those cases, as opposed to an empty string.
I've released an update, it shouldn't crash anymore. Please try this and close the issue if no more issues appear.

i will try it now

i will try it now

worked!
thanks

worked!
thanks


CI: Automatic vX tag update workflow

See https://github.com/lycheeverse/lychee-action/issues/169#issuecomment-1454895532
Let's try how it works!

Thanks!

Let's try it with the next release. ⭐

Let's try it with the next release. ⭐

If you edit the last release, it should also trigger the action!

Hum, I went to https://github.com/lycheeverse/lychee-action/releases/edit/v1.6.1, changed the release description and hit Update Release. The pipeline didn't run for some reason. 🤔 Maybe I misunderstand what "editing a release" means and changing the text is not enough.

That is exactly what the docs says as well:
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#release
Webhook events and payloads link:

They even link to an example telling you how to adjust the description in the web UI. :)
As long as you were not dealing with a draft, it should have triggered I think. 🤷

Late feedback, but it worked with the latest 1.7.0 release. 🚀
Logs here: https://github.com/lycheeverse/lychee-action/actions/runs/4726601856/jobs/8386343900
Thanks @tooomm.


Replace NuGet references with known security risks.

The referenced NuGet package SixLabors.ImageSharp 3.1.4 has a known high security risk.
This can be solved by update to 3.1.5.
Same goes for DeepCloner 0.10.40 but they do not have a fixed version yet.

Thanks @swtrse, it's fixed now.


Supported file formats

Hi!
What are the supported file formats?

see https://github.com/hMatoba/piexifjs/issues/28

As this repository is not supported anymore, I added png and webp support to this fork.
https://github.com/asc0910/exif-library


Add ERC: Consolidated Dynamic Storage (CDS)

Reference implementation:
https://github.com/wisecameron/ConsolidatedDynamicStorage/blob/main/contracts/CDSMinimal.sol

Will fix all remaining auto check failures tomorrow


Add support for thread names stack artifacts.

Summary
Add support for thread names in the callstack artifact.

Release Notes Entry

/backport to release/7.x

/backport to release/7.x


library creation fails due to missing parameter

These parameters are missing in the argument reading section. They need to be added otherwise the user has to use exactly the same header as the provided example.
https://github.com/nf-core/metaboigniter/blob/fca2ab5c508fc8179380ff747eab68b6db8aa93f/bin/createLibrary.r#L12-L15

This should have been fixed by the latest PR


Fixed issues in Dockerfile.

A command still referred to Consul Template 0.7.0 despite downloading 0.8.0.
Alpine Linux uses apk to install packages, not opkg-install.

@jimmcslim
Thank you for your patch.


Spelling and semantic fixes in README.md

Just a few spelling and semantic fixes in the README file.
Cheers,
Aaron (insp3ctre)

Great PR, thanks!

 Happy to contribute.
Cheers,
Aaron (insp3ctre)


TSN model was trained on Kinetics datase with Resnet50

hello ,
If you have the model, would you like to share it, thank you very much

hi,dear,
could you have the kinetics400 datasets ?
or just look the issue
https://github.com/yjxiong/temporal-segment-networks/issues/278
or tell me the configure/distribution ？
see the issue


Use .Release.Name in frontend.memcached.address to match service

What this PR does: Fixes the mapping between the query-frontend and the memcached created for it. The other memcached instances spun up in this chart use .Release.Name.
Which issue(s) this PR fixes:
Fixes #
Checklist

[x] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@nschad Totally open to that change. This PR was to fix a bug we ran into where the query-frontend doesn't properly start if you try to enable this due to the mismatch of what the service is and what this was setting, so was wanting to fix that without changing the world.
Would you want me to edit this PR to make that larger change or could that potentially be a follow up PR with this one still merged as an intermediate bug fix?

Would you be up to merging this PR as an intermediate bug fix given it's broken presently? I can follow up this PR to make that larger change at a later date.

I  don't feel comfortable doing this. What are your thoughts on this @kd7lxl ?

For context, this is the current behavior without the change.
level=warn ts=2022-03-16T15:41:16.49671671Z caller=resolver.go:133 msg="IP address lookup yielded no results. No host found or no addresses found" host=v1-cortex-memcached-frontend.cortex.svc.cluster.local
level=error ts=2022-03-16T15:42:16.49494025Z caller=resolver.go:86 msg="failed to lookup IP addresses" host=v1-cortex-memcached-frontend.cortex.svc.cluster.local err="lookup v1-cortex-memcached-frontend.cortex.svc.cluster.local on <IP_ADDRESS>:53: no such host"
level=warn ts=2022-03-16T15:42:16.495017428Z caller=resolver.go:133 msg="IP address lookup yielded no results. No host found or no addresses found" host=v1-cortex-memcached-frontend.cortex.svc.cluster.local
level=error ts=2022-03-16T15:43:16.502803721Z caller=resolver.go:86 msg="failed to lookup IP addresses" host=v1-cortex-memcached-frontend.cortex.svc.cluster.local err="lookup v1-cortex-memcached-frontend.cortex.svc.cluster.local on <IP_ADDRESS>:53: no such host"

because
$ kubectl -n cortex get service
NAME                                   TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)             AGE
v1-cortex-compactor                    ClusterIP   <IP_ADDRESS>   <none>        8080/TCP            8d
...
v1-cortex-store-gateway-headless       ClusterIP   None             <none>        9095/TCP            8d
v1-memcached-blocks                    ClusterIP   None             <none>        11211/TCP           8d
v1-memcached-blocks-index              ClusterIP   None             <none>        11211/TCP           8d
v1-memcached-blocks-index-metrics      ClusterIP   <IP_ADDRESS>   <none>        9150/TCP            8d
v1-memcached-blocks-metadata           ClusterIP   None             <none>        11211/TCP           8d
v1-memcached-blocks-metadata-metrics   ClusterIP   <IP_ADDRESS>   <none>        9150/TCP            8d
v1-memcached-blocks-metrics            ClusterIP   <IP_ADDRESS>   <none>        9150/TCP            8d
v1-memcached-frontend                  ClusterIP   None             <none>        11211/TCP           45h
v1-memcached-frontend-metrics          ClusterIP   <IP_ADDRESS>   <none>        9150/TCP            45h

Can you tell me the name of your helm-release?

I believe it's simply v1.

@elliottmb Okay I think this pr would fix the problem. My thoughts appear to be wrong


CodeExpression: add a programmingLanguage property

Curently CodeExpression component does not have a programmingLanguage property which it inherits from Code as does CodeChunk.

This is still an issue and means that CodeExpressions can not be executed because when they are sent to Executa, they lack programmingLanguage property that allows them to be delegated to the correct language interpreter (currently they are returned unexecuted):

Note that currently, Encoda is encoding code expressions in HTML with the programming-language="r" attribute, which is inconsistent with data-programminglanguage="r" that is used for code chunks:
<stencila-code-expression programming-language="r" itemscope=""
  itemtype="http://schema.stenci.la/CodeExpression">length(subset(data2, Lot==1 & Time==0)$value)<output
  slot="output">3</output>
</stencila-code-expression>

Thanks for this @alex-ketch, what should the HTML attribute be for both CodeChunks and CodeExpressions: programming-language or data-programminglanguage?

Right now there's a discrepancy, but ideally it should be programming-language for both.
Will log an issue in Encoda to make the change.


Token in args [Headless]

It would be good to implement the possibility of entering the token directly in the CLI
like: node headless --token-raw "xxxxxxxxxxxxxxxxxxxxxxxx"

Has been addressed in #135 but not pulled into the main branch yet.


Removed Java tab and improved Php template

Description
This is a small pull request to set up the repository for starting to allow using the php Umple template instead of JET.
Had to change one template to allow Umple and JET to produce the same output for php.
Outlined in Issue #708, but does not fix much of the spacing issues in Java, to make it easier for everyone still working on Java templates. Further, similar, changes will need to be performed on Java before Umple templates for Java will be able to pass.
Tests
Changed one Php test, to not expect a tab, and removed unnecessary spaces between two functions.
Also changed all Java tests that included String outputString = ""; in their toString() function to not use a tab.

Looks good to me
@vahdat-ab if it is OK by you I will let you merge it

great. Thank @Diusrex