Exception occurred while handling uri: 'http://localhost:5005/favicon.ico

hello, I install rasa X new version,run "rasa shell",It's error, I get this inspecting the core logs:
2019-07-30 13:23:09,608 [ERROR]  Exception occurred while handling uri: 'http://localhost:5005/'
Traceback (most recent call last):
  File "/home/duan/anaconda3/lib/python3.7/site-packages/sanic/app.py", line 893, in handle_request
    handler, args, kwargs, uri = self.router.get(request)
  File "/home/duan/anaconda3/lib/python3.7/site-packages/sanic/router.py", line 407, in get
    return self._get(request.path, request.method, "")
  File "/home/duan/anaconda3/lib/python3.7/site-packages/sanic/router.py", line 469, in _get
    raise NotFound("Requested URL {} not found".format(url))
sanic.exceptions.NotFound: Requested URL / not found
2019-07-30 13:23:17,398 [ERROR]  Exception occurred while handling uri: 'http://localhost:5005/favicon.ico'
Traceback (most recent call last):
  File "/home/duan/anaconda3/lib/python3.7/site-packages/sanic/app.py", line 893, in handle_request
    handler, args, kwargs, uri = self.router.get(request)
  File "/home/duan/anaconda3/lib/python3.7/site-packages/sanic/router.py", line 407, in get
    return self._get(request.path, request.method, "")
  File "/home/duan/anaconda3/lib/python3.7/site-packages/sanic/router.py", line 469, in _get
    raise NotFound("Requested URL {} not found".format(url))
sanic.exceptions.NotFound: Requested URL /favicon.ico not found

Thanks for raising this issue, @erohmensing will get back to you about it soon.

Hi there @pingyuan2016, sorry I seem to have missed this message. Can you post exactly what commands you're running and the full output (not just the error?)

Hello, I just reinstalled update version of Rasa and Rasa X few days ago. I have same problem when run rasa run -m models --enable-api --log-file out.log -m my model, it raises:
2019-08-21 14:51:58 INFO     root  - Generating grammar tables from /usr/lib/python3.6/lib2to3/Grammar.txt
2019-08-21 14:51:58 INFO     root  - Generating grammar tables from /usr/lib/python3.6/lib2to3/PatternGrammar.txt
2019-08-21 14:52:00 INFO     root  - Starting Rasa server on http://localhost:5005
[2019-08-21 14:52:09 +0200] [30698] [ERROR] Exception occurred while handling uri: 'http://localhost:5005/favicon.ico'
Traceback (most recent call last):
File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/app.py", line 893, in handle_request
handler, args, kwargs, uri = self.router.get(request)
File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/router.py", line 407, in get
return self._get(request.path, request.method, "")
File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/router.py", line 469, in _get
raise NotFound("Requested URL {} not found".format(url))
sanic.exceptions.NotFound: Requested URL /favicon.ico not found
[2019-08-21 14:53:09 +0200] [30698] [ERROR] Exception occurred while handling uri: unknown
Is there any wrong with my command or sth else in the new version Rasa?
Thank in advance for any recommendations or responses.

hi, I'm facing same issue. is there any update?

Can you post exactly what commands you're running and the full output (not just the error?)

I have this as well - [kind of fixed it]
Running Windows 10, fwiw.
When I initially execute rasa run there are no errors in the console.
Simply hitting the url : http://localhost:5005/ correctly show this in the browser:
Hello from Rasa: 1.2.5
However hitting that url in the browser displays the following error dump in the console and in out.log
Exception occurred while handling uri: 'http://localhost:5005/favicon.ico'
Traceback (most recent call last):
    File "c:\users\jonathan\appdata\local\programs\python\python36\lib\site-packages\sanic\app.py", line 893, in handle_request
        handler, args, kwargs, uri = self.router.get(request)
    File "c:\users\jonathan\appdata\local\programs\python\python36\lib\site-packages\sanic\router.py", line 407, in get
        return self._get(request.path, request.method, "")
    File "c:\users\jonathan\appdata\local\programs\python\python36\lib\site-packages\sanic\router.py", line 469, in _get
        raise NotFound("Requested URL {} not found".format(url))
sanic.exceptions.NotFound: Requested URL /favicon.ico not found

I don't think it's a big deal, it's simply not finding the favicon file when the Sanic server spins up.
I dropped a random one in the root of the project, restarted rasa and the error no longer appears.
However, the icon I placed there doesn't display in the browser as the favicon, and if I bookmark the url, there's a different favicon file next to the bookmark. :)

Hello, I face the same error. The command that I used:
 rasa run -vv -m models/20190926-143232.tar.gz --enable-api

And all output with error:
/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:526: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint8 = np.dtype([("qint8", np.int8, 1)])
/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:527: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_quint8 = np.dtype([("quint8", np.uint8, 1)])
/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:528: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint16 = np.dtype([("qint16", np.int16, 1)])
/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:529: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_quint16 = np.dtype([("quint16", np.uint16, 1)])
/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:530: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint32 = np.dtype([("qint32", np.int32, 1)])
/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/tensorflow/python/framework/dtypes.py:535: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  np_resource = np.dtype([("resource", np.ubyte, 1)])
/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/h5py/__init__.py:34: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
2019-09-27 10:56:48 DEBUG    rasa.cli.utils  - Parameter 'endpoints' not set. Using default location 'endpoints.yml' instead.
2019-09-27 10:56:48 DEBUG    rasa.cli.utils  - Parameter 'credentials' not set. Using default location 'credentials.yml' instead.
2019-09-27 10:56:48 INFO     root  - Generating grammar tables from /usr/lib/python3.6/lib2to3/Grammar.txt
2019-09-27 10:56:49 INFO     root  - Generating grammar tables from /usr/lib/python3.6/lib2to3/PatternGrammar.txt
2019-09-27 10:56:50 DEBUG    rasa.core.utils  - Available web server routes:
/conversations/<conversation_id>/messages          POST                           add_message
/conversations/<conversation_id>/tracker/events    POST                           append_events
/webhooks/rasa                                     GET                            custom_webhook_RasaChatInput.health
/webhooks/rasa/webhook                             POST                           custom_webhook_RasaChatInput.receive
/model/test/intents                                POST                           evaluate_intents
/model/test/stories                                POST                           evaluate_stories
/conversations/<conversation_id>/execute           POST                           execute_action
/domain                                            GET                            get_domain
/                                                  GET                            hello
/model                                             PUT                            load_model
/model/parse                                       POST                           parse
/conversations/<conversation_id>/predict           POST                           predict
/conversations/<conversation_id>/tracker/events    PUT                            replace_events
/conversations/<conversation_id>/story             GET                            retrieve_story
/conversations/<conversation_id>/tracker           GET                            retrieve_tracker
/status                                            GET                            status
/model/predict                                     POST                           tracker_predict
/model/train                                       POST                           train
/model                                             DELETE                         unload_model
/version                                           GET                            version
2019-09-27 10:56:50 INFO     root  - Starting Rasa server on http://localhost:5005
2019-09-27 10:56:50 INFO     root  - Enabling coroutine debugging. Loop id 104305912.
2019-09-27 10:56:50 DEBUG    rasa.model  - Extracted model to '/tmp/tmpl8k9zlqb'.
2019-09-27 10:56:50 DEBUG    rasa.core.tracker_store  - Connected to InMemoryTrackerStore.
2019-09-27 10:56:50 DEBUG    rasa.model  - Extracted model to '/tmp/tmpzs5ou51c'.
2019-09-27 10:56:50 DEBUG    pykwalify.compat  - Using yaml library: /home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/ruamel/yaml/__init__.py
[2019-09-27 10:56:55 +0200] [7665] [ERROR] Exception occurred while handling uri: 'http://localhost:5005/favicon.ico'
Traceback (most recent call last):
  File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/app.py", line 893, in handle_request
    handler, args, kwargs, uri = self.router.get(request)
  File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/router.py", line 407, in get
    return self._get(request.path, request.method, "")
  File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/router.py", line 469, in _get
    raise NotFound("Requested URL {} not found".format(url))
sanic.exceptions.NotFound: Requested URL /favicon.ico not found
[2019-09-27 10:57:14 +0200] [7665] [ERROR] Exception occurred while handling uri: 'http://localhost:5005/favicon.ico'
Traceback (most recent call last):
  File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/app.py", line 893, in handle_request
    handler, args, kwargs, uri = self.router.get(request)
  File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/router.py", line 407, in get
    return self._get(request.path, request.method, "")
  File "/home/nluu/.virtualenvs/Chabot_hotline/lib/python3.6/site-packages/sanic/router.py", line 469, in _get
    raise NotFound("Requested URL {} not found".format(url))
sanic.exceptions.NotFound: Requested URL /favicon.ico not found

Is there anyone have idea to fix this? Thank you

What are you sending when you receive the error?

Hello, I tried to open it in a browser, with address http://localhost:5005, it returned 'hello from Rasa 1.2.5' but http://localhost:5005/favicon.ico, it returned the error above.
But if I don't open the browser, and test in another terminal, for example I sent:
curl -XPOST localhost:5005/webhooks/rest/webhook -d '{"sender":"Me","message":"how are you?"}'

It worked.

Right, it's not a browser application, so that'l be why

hi,when i am type this command,"rasa run actions" ,i am getting this error,
C:\Users\mohammed tanveer.javeriabot>rasa run actions
2020-03-26 13:27:33.378025: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library cudart64_100.dll
2020-03-26 13:27:37 INFO     rasa_sdk.endpoint  - Starting action endpoint server...
2020-03-26 13:27:37 INFO     rasa_sdk.executor  - Registered function for 'find_facility_types'.
2020-03-26 13:27:37 INFO     rasa_sdk.executor  - Registered function for 'find_healthcare_address'.
2020-03-26 13:27:37 INFO     rasa_sdk.executor  - Registered function for 'facility_form'.
Exception occurred while handling uri: 'http://localhost:5055/webhook'
Traceback (most recent call last):
File "c:\programdata\anaconda3\lib\site-packages\sanic\app.py", line 918, in handle_request
handler, args, kwargs, uri = self.router.get(request)
File "c:\programdata\anaconda3\lib\site-packages\sanic\router.py", line 407, in get
return self._get(request.path, request.method, "")
File "c:\programdata\anaconda3\lib\site-packages\sanic\router.py", line 447, in _get
raise method_not_supported
sanic.exceptions.MethodNotSupported: Method GET not allowed for URL /webhook
Exception occurred while handling uri: 'http://localhost:5055/favicon.ico'
Traceback (most recent call last):
File "c:\programdata\anaconda3\lib\site-packages\sanic\app.py", line 918, in handle_request
handler, args, kwargs, uri = self.router.get(request)
File "c:\programdata\anaconda3\lib\site-packages\sanic\router.py", line 407, in get
return self._get(request.path, request.method, "")
File "c:\programdata\anaconda3\lib\site-packages\sanic\router.py", line 470, in _get
raise NotFound("Requested URL {} not found".format(url))
sanic.exceptions.NotFound: Requested URL /favicon.ico not found

@tanveerahmed668 Please create a new issue and fill out the issue template with all of the relevant information

sorry ,can you elaborate your answer

Please open a new issue instead of commenting on an old one if you think you have found a bug


[feat 🔥] Multi-Telended Insert

I completed these tasks for #79 issue

[x] CSV to SQL
[ ] XLSV to SQL
[x] Module system with class & full type safe
[x] Hard testing
[x] Insert progressing loader
[ ] Update usage documentation

Best regards,
@devlopersabbir

@shameellamba boss checks my multi-talented branch and I hope you will make a pull request on this branch with the XLSV to SQL task.
I have set up ap a multi-talented architecture project with resuing and comments, which will help you understand your step. and I hope you will feel comfortable with my code.
@gitdagray again I requested you to review my code 🙋🏻♂


Build source

Hello!
Trying to build from sources.
Go and Protobuf are installed successfully.
make requirements also passed without errors.
There are errors during make test:

--- PASS: TestADR (0.03s)
PASS
ok      github.com/brocaar/loraserver/internal/adr      0.041s
=== RUN   TestGatewayAPI
Given a clean database and api instance ERRO[0000] ping database error, will retry in 2s: pq: password authentication failed for user "root"
ERRO[0002] ping database error, will retry in 2s: pq: password authentication failed for user "root"

What is a next steps for installation? Calling packaging/package.sh returns me Unknown target
as GOARCH="amd64"

Please see the provided docker-compose.yml file for setting up a test environment, including test databases. The commands are explained here (including for packaging): https://docs.loraserver.io/loraserver/community/source/.

Wow! Adding export TEST_POSTGRES_DSN="postgres://loraserver_ns:your_pass@localhost/loraserver_ns?sslmode=disable"
variable to bash lets make test to pass all tests! Awesome! Thank you very much for awesome software!


PSR-20 - clock adapter

Do you need a PSR-20 adapter?
BTW, hope to see you tomorrow in Poznań

Change Log

Added

PSR-20 clock adapter

Hey!! Sorry, I somehow totally missed that one 🤦♂ I believe it wont hurt to have that adapter 👍


adding a screencast about error types

Still TODO:

[x] website updates
[ ] post screencast & update the youtube link

Deploy preview processing.
Built with commit 931a4ca0f647ca63f36d54ff9a8a33bbffd5ff8d
https://app.netlify.com/sites/goin5minutes/deploys/57dee2516686743fc5051cfe


fix cursor jumping at the end after every editor update

Currently the cursor constantly jumps at the end everytime you press a key in the editor
My changes is supposed to prevent that

using onBeforeChange may look "surprising" but it's what's in the demo:

@slorber awesome! Thanks so much for fixing this up, really appreciate it 😀

thanks to you @briandennis , your project was very useful inspiration for something I just published on git: https://github.com/slorber/react-reboot


Retrieving ALL Messages

How do I retrieve all in a chat given that it has probably over 100k messages?

I attempted using WebHistoryLength with no changes appearing
I could not find any paging implementation for messages() or a different way of retrieving older messages than the ones provided

How do I retrieve all in a chat given that it has probably over 100k messages?

I attempted using WebHistoryLength with no changes appearing
I could not find any paging implementation for messages() or a different way of retrieving older messages than the ones provided

Good question, I think it can be done using the history sync config field. I'll push it when cobalt 0.1 is ready.

I can confirm that it works, I'll push out this feature soon.
It will be available for everyone in Cobalt 0.1


[error] CUDA_ERROR_OUT_OF_MEMORY: out of memory

Hello,
I've update all the code to the latest version and re-compiled it this morning,
I tried to render a picture using the command:
bin/luisa-render-cli -b cuda ../../scenes/spaceship/spaceship/scene.luisa
but it gave an error again:
[2023-08-01 10:28:19.473] [console] [error] CUDA_ERROR_OUT_OF_MEMORY: out of memory [/media/liusheng/Projects/project/game_engines/LuisaRender/github/LuisaGroup/LuisaRender/LuisaRender/src/compute/src/backends/cuda/cuda_buffer.cpp:9]
     0 [0x000000000000]:  ::  + 0
     1 [0x000000000000]:  ::  + 0
     2 [0x000000000000]:  ::  + 0
     3 [0x000000000000]:  ::  + 0
     4 [0x000000000000]:  ::  + 0
     5 [0x000000000000]:  ::  + 0
     6 [0x000000000000]:  ::  + 0
     7 [0x000000000000]:  ::  + 0
     8 [0x000000000000]:  ::  + 0
     9 [0x000000000000]:  ::  + 0
    10 [0x000000000000]:  ::  + 0
    11 [0x000000000000]:  ::  + 0
    12 [0x000000000000]:  ::  + 0
已放弃 (核心已转储)

My video card is GTX 1070, with CUDA version "12.2.91, on ubuntu 22.04.2.
How could I fixed that ? Thank you!

Hi, @disini
The error is due to insufficient VRAM, which is relatively small on GTX-1070. The spaceship scene by default uses a wavefront path tracing integrator and can be too memory-consuming on your device. Please try to edit the scene file, change
render {
  ...
  integrator : WavePath { ... }
  ...
}

to
render {
  ...
  integrator : MegaPath { ... }
  ...
}

to use the less memory-intensive mega-kernel integrator.

OK, much appreciated for your help! ^_^


Add depth frames to stereo module

Added support for depth frames in the stereo module.
Depth frames are aligned to the stereo camera 0 (default or left)

depth frames are published under the ros topic /stereo/depth
format is CV_16UC1
debug_imshow() works only for stereo images
frame2cvmat() extended but backwards compatible

LGTM! :+1:


Training questions

Took me a while to get everything installed and working. Right now I am doing noise training with about 300 images. It is currently on run #53. Just curious if there is a set number of runs it goes through?

training step runs #200 epochs by default.

Another question about training: does the batch_size make any quality difference or is it just for performance tuning?

In waifu2x,  computing performance mostly not influenced by batch_size (but slow performance when batch_size=1).
I recommend small batch_size (2~4).
small batch_size: low memory usage. fast convergence. reducing overtting.

I have another question: how long did the entire training process take for your training set?

2x scaling model: 16 hours
Noise reduction model: 28 hours * 2
Total: around 72 hours
(on GTX760, 3000 images)

The facts brought up here (how long training takes, batch size recommendation, epochs to training end) should be added to the documentation - they would be EXTREMELY useful to someone like me trying to train their own model.

Could you please let me know how many iterations did you train it for? Also did you use all the current default settings when training your anime model?

See https://github.com/nagadomi/waifu2x/blob/master/train.sh
The default setting is defined at https://github.com/nagadomi/waifu2x/blob/master/lib/settings.lua
I was used 3000 images for training.
And I'm currently running the traning script for next version using dev branch code and 6000 images.

Thanks! Could you share what the next version is and what the differences are?

I released new version yesterday.
Summary of difference in training script:

Add data compression method for on-memory image (reducing memory usage for dataset)
Change loss function (MSE to Customized Huber Loss)
Improve some image processing functions
Add some data augmentation methods
Tuning default parameters
Improve code readability
Improve training time (4x~ faster)

Sorry, my english skill is very poor. Please refer the code for more details.

I would like to ask you a hypothetical question. If I have only one specific type of image I want to scale up, would it be possible to train the algorithm only on one perfect image with maximum resolution?


[VTOL] Broken Weathervane & broken MPC_YAW_MODE

Describe the bug
The weather vane does not work anymore in Mission (MC mode). Neither does MPC_YAW_MODE have an effect on the heading. So the drone ends up flying in all directions with the same heading, which is ugly to watch and can be dangerous in high winds.
To Reproduce
Tested in SITL on v1.11.0:

Launch Gazebo with make px4_sitl gazebo_standard_vtol__windy (by default 4m/s wind from South to North)
Execute some mission which contains navigation in MC mode (e.g. Dummy_MC_loop.zip)
Observe the heading.

Expected behavior
As stated in the docs (https://docs.px4.io/master/en/config_vtol/vtol_weathervane.html#mission-mode-behaviour). The drone should head into the wind. Also, MPC_YAW_MODE should behave as for a normal MC. I'm open to discussion whether the weather vane should have precedence over the heading calculations of MPC_YAW_MODE or not or if both should be active at the same time and you end up with a compromise. But certainly not changing the heading at all is not good.
Log Files and Screenshots
https://logs.px4.io/plot_app?log=41546a93-eaeb-45c8-af75-fcfd2af6b967 : always heading east, neither weathervaning nor heading towards waypoint.
https://review.px4.io/plot_app?log=5cd8b35e-390b-476c-85fb-30ac2fd3ff42 : high winds on a custom branch with minor modifications from v1.11-beta. The drone gets carried away because it can't fight the wind in its bad orientation. --> dangerous situation.
Drone (please complete the following information):
gazebo_standard_vtol

@ThomasRigi  I tried to increase the simulated wind speed to 10, and then execute take off, I can clearly observe that the drone's head is rotating correctly

In mission also at 10m/s there is the heading issue:
windy setup:
      <windDirectionMean>1 0 0</windDirectionMean>
     <windVelocityMean>10.0</windVelocityMean>
Log: https://logs.px4.io/plot_app?log=89f49258-1649-4af9-ad1c-39ba0271a870
But good to know that the weather vane works at least in take off and loiter mode. :)

Judging from the current code logic, the yaw generated by the weather vane will give priority to the yaw generated by the mission.
if (_ext_yaw_handler != nullptr && _ext_yaw_handler->is_active()) {
		_yaw_setpoint = _yaw;
		// use the yawrate setpoint from WV only if not moving lateral (velocity setpoint below half of _param_mpc_xy_cruise)
		// otherwise, keep heading constant (as output from WV is not according to wind in this case)
		bool vehicle_is_moving_lateral = _velocity_setpoint.xy().longerThan(_param_mpc_xy_cruise.get() / 2.0f);

		if (vehicle_is_moving_lateral) {
			_yawspeed_setpoint = 0.0f;

		} else {
			_yawspeed_setpoint = _ext_yaw_handler->get_weathervane_yawrate();
		}

	} else if (_type == WaypointType::follow_target && _sub_triplet_setpoint.get().current.yawspeed_valid) {
		_yawspeed_setpoint = _sub_triplet_setpoint.get().current.yawspeed;
		_yaw_setpoint = NAN;

	} else {
		if ((_type != WaypointType::takeoff || _sub_triplet_setpoint.get().current.disable_weather_vane)
		    && _sub_triplet_setpoint.get().current.yaw_valid) {
			// Use the yaw computed in Navigator except during takeoff because
			// Navigator is not handling the yaw reset properly.
			// But: use if from Navigator during takeoff if disable_weather_vane is true,
			// because we're then aligning to the transition waypoint.
			// TODO: fix in navigator
			_yaw_setpoint = _sub_triplet_setpoint.get().current.yaw;

		} else {
			_set_heading_from_mode();
		}

		_yawspeed_setpoint = NAN;
	}

Yes, since https://github.com/PX4/Firmware/pull/13782 WV is disabled when moving. The idea of WV how I understand it (yes, the docs would need some updating) is that the nose of the VTOL is always pointed into the (external) wind, as that is the dangerous factor for VTOLS, rather than the airspeed it picks up when moving lateral (wind can be at 15 m/s, normally VTOLs move laterally at 5 m/s or so). So it doesn't look nice when a VTOL is moving sideways and backwards, but compared to facing the wind from the rear/side in my perspective it's the lesser evil.
Happy for some discussion around this, and how we potentially could parametrize it better, or even change the logic (how about disabling WV completely if the estimated wind speed is below some threshold, and the confidence in the wind estimate is high enough?)

The problem with the current logic of just disabling the windvane when
moving laterally is that you never align in the first place. In all
simulated missions the vtol was never facing the wind, neither the relative
(including airspeed from moving) nor the external only. Instead it just
rolls and pitches, which is dangerous as it's not aligned.
In my opinion we can ignore mpc_yaw_mode at all times (as I agree that
facing the wind is more important), but weather vane should remain enabled,
also to readjust to changing wind directions.
As to whether it should only take into account only the external wind or
the relative including the airspeed induced by the drone moving, I think
the relative is more important. This is the one that instantaneously
affects the drone. The only small problem I see with this approach is that
when changing direction you are not aligned in the beginning. But to me
this is a minor issue compared to the case where the drone advances with
say 7m/s with 3m/s wind from the side and vaning only to the 3m/s side
wind.
A side effect of taking the relative wind is that the current WV logic
works. If you really want to consider only the external wind you need to
adapt the logics.
On Mon, 7 Sep 2020, 10:00 Silvan Fuhrer<EMAIL_ADDRESS>wrote:

Yes, since #13782 https://github.com/PX4/Firmware/pull/13782 WV is
disabled when moving. The idea of WV how I understand it (yes, the docs
would need some updating) is that the nose of the VTOL is always pointed
into the (external) wind, as that is the dangerous factor for VTOLS, rather
than the airspeed it picks up when moving lateral (wind can be at 15 m/s,
normally VTOLs move laterally at 5 m/s or so). So it doesn't look nice when
a VTOL is moving sideways and backwards, but compared to facing the wind
from the rear/side in my perspective it's the lesser evil.
Happy for some discussion around this, and how we potentially could
parametrize it better, or even change the logic (how about disabling WV
completely if the estimated wind speed is below some threshold, and the
confidence in the wind estimate is high enough?)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PX4/Firmware/issues/15673#issuecomment-688125034, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALPZHBLOZGWLXCIC6XWD7BLSESHJNANCNFSM4QS7IIZA
.


url fixed

@dawlysd : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@miwithro
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
#assign:@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@miwithro, the new URl doesn't seem to work. Can you check it/confirm? I see that the existing URL on the live page doesn't work also. There are a few possible problems with the syntax, including the ` marks and the fact that it's brackets+brackets instead of bracket+parentheses. However, even without those issue, I can't build a test URL that works from the supplied URL. Thanks. #hold-off

#sign-off

#sign-off

@miwithro, the new URl doesn't seem to work. Can you check it/confirm? I see that the existing URL on the live page doesn't work also. There are a few possible problems with the syntax, including the ` marks and the fact that it's brackets+brackets instead of bracket+parentheses. However, even without those issue, I can't build a test URL that works from the supplied URL. Thanks. #hold-off

I didn't touch the URL.  I just changed one word in the document.  I don't know who set it.

The issue that's addressed in the PR appears to be addressed in the default branch. We'll close the PR. If there are outstanding issues or questions, please add a comment or create a new pull request. Thanks.


NEXT-00000 - Improve order amount stats performance

1. Why is this change necessary?

If you have a large number of orders, the inner join from the order_transaction table to the state_machine_state takes quite a long time and slows down the query massively. We can simply fetch the required state id once in advance and save a lot of table joins and processing power
In group by and order by we can use the already calculated date value, to not calculate it again
We should only join the currency table, if we need the value from it
If the mysql.time_zone_name table is empty, CONVERT_TZ will produce a NULL value and mess up all the statistics. When that happens, we now fall back to order_date_time.
As we are in the admin context the default currency + the rounding rules have already been fetched on request into the context object, so we can just use the rounding rules from the context, no need to refetch them from the database

2. What does this change do, exactly?

Changed DashboardController to pass the context to the orderAmountService
Changed OrderAmountService to use $rounding from context, fix null timezone & prefetch the paid transaction id

3. Describe each step to reproduce the issue or behaviour.

Create a large number of orders
Load the main admin dashboard
Note that you will have to wait several seconds for the dashboard to load

4. Please link to the relevant issues (if any).

/

5. Checklist

[X] I have rebased my changes to remove merge conflicts
[X] I have written tests and verified that they fail without my change
[X] I have created a changelog file with all necessary information about my changes
[X] I have written or adjusted the documentation according to my changes
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hi @akf-bw,
Thanks for the PR! The optimizations look reasonable.
Could you share the size of the orders table where you noticed the performance improvement? That would help with context.

thanks again for your contribution @akf-bw and this great improvement 👍🎉💙


Single File Blazor Components

Building small, reusable components and composing them into larger components is a great way to build a web application that is used by many client side JavaScript frameworks today. It would be great for this to be brought to Razor/Blazor but to also blur the lines between the client and server too.
Following on from this thread talking about static file handling. Perhaps the Vue.js concept of single file components could be taken to the extreme, so Blazor could handle a HTML template, server side Razor, client side Blazor and CSS all in one file (or separated if you'd like but in my experience if you make your components small and reusable, a single file really works well) with the C# being shared and runnable on the client and server side. I envisage something like this:
ExampleComponent.cshtml
@Model Customer

<template>

HTML which can be rendered on the client using Blazor or the Server using Razor.

<EMAIL_ADDRESS></template>

The HTML would be prerendered on the server and show the customer name as 'Server Rendered'. When the event handler fires, the model text is changed to 'Client Rendered'.
There would need to be some sort of build system to extract CSS into a separate file, compile the C# code for the server using Razor but also for the client into web assembly format using Blazor. Finally the HTML template would also need to be compiled to C# as it is today for Razor but also for Blazor.
Making use of such a component could just use the web component standards, so it might look something like this:
ComponentImports.cshtml
@addComponents "ExampleComponent, MyAssemblyName"
@addComponents "*, MyOtherAssemblyName"

ExampleComponentUsage.cshtml
<template>

Using the component below

<example-component />
</template>

Thanks for the suggestion! I'll get some other folks on the team to discuss.
At the moment our plan for server-side rendering is to limit it to cases where you have .NET on the server. In that case we can trivially just execute the same components on the server like we do on the client. We don't need developers to specify different "templates" for server and client-side use, or at least not in general (there will be cases where you want to use DI to inject server variants of services that differ from client variants). Our existing architecture is designed with this in mind, so it should be very easy to enable, use, and have excellent performance.

Perhaps the Vue.js concept of single file components could be taken to the extreme, so Blazor could handle a HTML template, server side Razor, client side Blazor and CSS all in one file

We're already working with this in mind. The idea is that you can optionally have C# code-behind files for your components if you want, but equally well you can put your C# into a @functions { ... } block in the Razor .cshtml file and it works the same. As for how you'd declare CSS styles in the same file, well we haven't decided on a specific syntax for that but it follows that if you can do @functions { ... } for C# you should probably be able to do @style { ... } or @css { ... } or something like that.
So overall, your proposal is pretty close to what we are aiming for!
I'll mark this closed since I hope it addresses your points. Let me know if not!

That sounds really great.
It sounds like you are on the same page as me. So C# code is shared between the client and server but with the ability to run extra code on the client or server wher necessary.
I'm not sure how I feel about the use of more Razor @foo { } syntax instead of HTML wrapper tags. I personally think it would be one less thing to learn to use the standard template, script and style HTML tags instead of coming up with new Razor commands like @style.
Keep us posted, I'd really like to see a blog post showing some variants of this so the community can give feedback!

  I don’t trust anyone named Mohamed
Sent from my iPhone

On Feb 27, 2018, at 03:37, Steve Sanderson<EMAIL_ADDRESS>wrote:
Thanks for the suggestion! I'll get some other folks on the team to discuss.
At the moment our plan for server-side rendering is to limit it to cases where you have .NET on the server. In that case we can trivially just execute the same components on the server like we do on the client. We don't need developers to specify different "templates" for server and client-side use, or at least not in general (there will be cases where you want to use DI to inject server variants of services that differ from client variants). Our existing architecture is designed with this in mind, so it should be very easy to enable, use, and have excellent performance.
Perhaps the Vue.js concept of single file components could be taken to the extreme, so Blazor could handle a HTML template, server side Razor, client side Blazor and CSS all in one file
We're already working with this in mind. The idea is that you can optionally have C# code-behind files for your components if you want, but equally well you can put your C# into a @functions { ... } block in the Razor .cshtml file and it works the same. As for how you'd declare CSS styles in the same file, well we haven't decided on a specific syntax for that but it follows that if you can do @functions { ... } for C# you should probably be able to do @style { ... } or @css { ... } or something like that.
So overall, your proposal is pretty close to what we are aiming for!
I'll mark this closed since I hope it addresses your points. Let me know if not!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

 I'll get some other folks on the team to discuss.
At the moment our plan for server-side rendering is to limit it to cases where you have .NET on the server. In that case we can trivially just execute the same components on the server like we do on the client. We don't need developers to specify different "templates" for server and client-side use, or at least not in general (there will be cases where you want to use DI to inject server variants of services that differ from client variants). Our existing architecture is designed with this in mind, so it should be very easy to enable, use, and have excellent performance.
Perhaps the Vue.js concept of single file components could be taken to the extreme, so Blazor could handle a HTML template, server side Razor, client side Blazor and CSS all in one file
We're already working with this in mind. The idea is that you can optionally have C# code-behind files for your components if you want, but equally well you can put your C# into a @functions { ... } block in the Razor .cshtml file and it works the same. As for how you'd declare CSS styles in the same file, well we haven't decided on a specific syntax for that but it follows that if you can do @functions { ... } for C# you should probably be able to do @style { ... } or @css { ... } or something like that.
So overall, your proposal is pretty close to what we are aiming for!
I'll mark this closed since I hope it addresses your points. Let me know if not!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

deepness187, this is a github issue comment thread.  If you can see this, I think it would be best if you came onto this link:
https://github.com/aspnet/Blazor/issues/173
Because your comments are open to the  public and you may not appreciate them being public given what you have said and the place in which you have said it.

I think a general idea behind using the at syntax is that it lets you have a distinction between a pages javascript functions and the pages blazor functions.

@RehanSaeed I’m truly sorry that you have suffered a public and personal abusive attack from an individual just because of your name. I post this here because I would like github and the community to +1 to show support and to let that individual know vile personal attacks aren’t welcome.
@Eilon thanks for removing those comments, hope/trust the individual has been reported to github

I was just going to follow the "do not feed the trolls" mantra and ignore it as usual but thank you @grahamehorner.

@grahamehorner indeed I deleted the comments and reported them to GitHub. Certainly not acceptable behavior in this forum (or perhaps anywhere!).


WebCord blocks StreamKit overlay

Aknowledgements

[X] I have checked that there's no other issue describing the same or
similar problem that I currently have, regardless if it has been
closed or open.

[X] I can confirm that this is not an issue with the Discord website,
but it is a problem specific to the WebCord itself. I have tested
if this bug occurs on Chromium/Chrome or any other Chromium-based
browser that uses unpatched/upstream Chromium engine.

[X] I have tried running the build from the master branch and it does
not have any fixes implemented according to my issue.

[ ] My issue describes one of the unstable and/or not fully implemented
features.

[ ] I have found a workaround to mitigate or temporarily fix this issue
in affected releases (please write it in Additional context section
below).

Operating System / Platform
🐧 Linux
Operating system architecture
x64 (64-bit Intel/AMD)
Electron version
20.0.0
Application version
v3.7.1
Bug description
While WebCord is running the StreamKit Overlay fails to connect. If both WebCord and the native Discord client are launched at the same time then the overlay fails to connect to the native client until WebCord has been closed.
Additional context
I personally have 2 Discord accounts, my main running in WebCord and my second account in the native Discord client and my secondary account I use for streaming and use the overlay kit with it but because of this bug I have to close WebCord while I'm streaming.

https://user-images.githubusercontent.com/39580303/184503138-f085fb0f-8130-4c2c-91ac-417a9f14d202.mp4

I have a feeling this is caused when StreamKit is looking for Discord client via WS and finds WebCord's WebSocket instead of the Discord. As both Discord and WebCord should accuire one port from list of all, I believe ensuring that WebCord runs as the second client should fix/workaround your issue.

Also after WebCord is closed you should restart Discord as well to ensure it will catch the first available port from range.

Ah yeah you are right, launching native Discord first does make it work. Never thought the order would affect it

I've managed to do some "fixes" on my side, but Discord still keeps spamming WebCord's WebSocket.
I guess the one should report this issue on their side as StreamKit doesn't work when primary port is reserved by another app/WebSocket. A proposed solution on the Discord side would be to either limit how many requests it can send before switching to the next address or respond to any failure (close event) immediately.
You may think why WebCord uses WSS on the same port as Discord and the answer is simple: for communication with browser. While it doesn't support all requests, it can reply to some (like this). Even ARMCord has took the part of WebCord's code and uses it at the border of the legality (and it's sus they've locked up the conversation on this topic and not fixed the issue in the affected releases) to implement this feature.

I'll close this as wontfix, WebCord does not support requests that requires API to be resolved (SteamKit wants from WebCord to respond to AUTHORIZE request) and it is in general a bug on the Discord's side (I've described why I think that way in previous comment). As a side note, I believe setting up any WebSocket Server (without caring about the communication at all) has potential to be a denial-of-service for Discord client, blocking entire communication for stuff like RPC (based on WebSocket; Discord also uses IPC for RPC), invite links etc.

they've locked up the conversation on this topic and not fixed the issue in the affected releases – or at least respond why they didn't do that

If you can provide us with built binaries of the affected releases that fix your licensing complaints, I'll be more than happy to replace the release files with your new ones. The reason we haven't done it is because it's too much effort for little gain.

@lexisother oh I thought it would be just the case of re-running a specific job in GitHub, like I do in WebCord (before I used GitHub Actions, it was a pain for me to upload every distributable I've built and I couldn't even build some due to host limitations).
I wrote this not to force you to change that – at least this release provides both sourcemap and TS source (which contains the WebCord's license and sourcemap directly points to the TS file from JS), which I find as acceptable right now. I just wanted to hear from you the reasons why you couldn't do it.
I will also work on WS as Node module, so hopefully that will solve the most legal concerns and will keep ARMCord's implementation up-to-date with the WebCord's. I've also planned to pack it with the IPC, making possible to fully handle RPC requests done by both IPC and WS clients.
As of the future, maybe instead of copying my code, you could just request it as a module? I'll be happy to share my code with the other projects as long as they won't relicense my code (which MIT actually protects me from and with the modules, your code wouldn't be mixed with mine).

@3e849f2e5c I've reported the bug to Discord, hopefully they will improve their implementation of WebSocket client on StreamKit to solve this so it will ask Discord client once WebCord closes the connection.


add dummy serializeable interfaces

Adds dummy serializable interfaces to Increment and ModelAuxIncrement.
These are required by a recent change to oops, but since they are needed primarily for ensemble applications we don't actually needed them and so we can get away with just having these dummy implementations.

closes #4

@travissluka , my current local branch is develop_chen, which has been deleted from github. In order for me to update this OOPS change, are following commands I need to use? Just to confirm.
% git checkout develop  # switch to develop branch
% git pull  # update from remote/github develop branch.

@travissluka After I run "% git checkout develop", it says "Your branch is ahead of 'origin/develop' by 2 commits.", then I run "% git pull", it says
"Auto-merging src/oisst/Geometry/Geometry.h
CONFLICT (content): Merge conflict in src/oisst/Geometry/Geometry.h
Auto-merging src/oisst/Geometry/Geometry.cc
CONFLICT (content): Merge conflict in src/oisst/Geometry/Geometry.cc
Automatic merge failed; fix conflicts and then commit the result."
Below is a section of code from my local Geometry.cc, it looks like after I run "% git checkout develop", it did not really connect my local branch to the github/develop branch (which is up to date). What did I miss?
<<<<<<< HEAD
Geometry::Geometry(const eckit::Configuration & conf, const eckit::mpi::Comm & comm) : comm_(comm) {
//util::abor1_cpp("Geometry::Geometry() needs to be implemented.", FILE, LINE);
atlas::RegularLonLatGrid atlasRllGrid("Slat360x180");
atlasFunctionSpace_.reset(new atlas::functionspace::StructuredColumns(atlasRllGrid,
                              atlas::option::halo(0)));

=======
Geometry::Geometry(const eckit::Configuration & conf,
const eckit::mpi::Comm & comm) : comm_(comm) {
atlas::RegularLonLatGrid atlasRllGrid(conf.getString("grid"));
atlasFunctionSpace_.reset(
new atlas::functionspace::StructuredColumns(atlasRllGrid,
atlas::option::halo(0)));

8cb531dac33955291619f63d4d32085103569c12
atlasFieldSet_.reset(new atlas::FieldSet());
}

@loganchen39 This is because you had performed commits to your local develop branch for your previous PR, and so develop and the version of it sitting on github origin/develop diverged after the PR was merged in. git pull then tries to merge those two versions back together, but as you can see bad things happen. It's possible to fix it with a series of git commands, but your best is to just delete your local version of the repository and reclone it. Be sure next time to instead do your local work in a branch created from develop, not develop itself.


Dataform Execution Fails

Dataform Execution steps fail due to misconfiguration on the terraform.tfvars file.
In the line https://github.com/GoogleCloudPlatform/marketing-analytics-jumpstart/blob/main/infrastructure/terraform/terraform-sample.tfvars#L34, the project id was added by mistake.
Fix: Remove the project id in that line, and redeploy the solution by running terraform .. apply.

Added a comment in that line to remind the user of this restriction.
https://github.com/GoogleCloudPlatform/marketing-analytics-jumpstart/blob/main/infrastructure/terraform/terraform-sample.tfvars#L34


remove / before console.clear()

Description
This is just a small issue in the documentation , and since it is of environment setup its more problematic for beginners
Steps to reproduce
.
Additional context
No response
Hedera network
testnet
Version
v0.1.0
Operating system
macOS

Thanks for the issue and PR!


Feature/bphh 537/template sections and block display

Description
This PR implements the display components for requirement section and requirement block for edit template page

What type of PR is this? (check all applicable)

[ X] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📦 Chore (Release)
[ ] ✅ Test

Related Tickets & Documents
https://hous-bssb.atlassian.net/browse/BPHH-537

Steps to QA

Navigate to permit templates index page
Locate any existing requirement template and click edit (There should be some due to seeds)
Verify if sections and blocks are displayed

edit page is blank, clarification needed:

@bkmaibach Not all templates have seeded sections. Using rails console get get the template id for an existing section, i.e. RequirementTemplateSection.last.requirement_template.id. If a template section does not exist, run rails db:seed


Add version 2.4.2-pre.22 to xunit

Add another build for which the license information has not changed.

@zkoppert NuGet does not have a version 2.4.2-pre.22 for xunit. Is this the right version number?

Interesting! It's called out in one of our csproj files. Let me do some digging.


callbacks never called

steps to reproduce: make a failed URL request using etherKit, e.g. give it a url that isnt running a node
expected: callback is called with the Result containing .failure(theNetworkingError)
actual: callback is never called https://github.com/Vaultio/EtherKit/blob/master/EtherKit/RequestProtocols/URLRequestManager.swift line 35 just returns.
notes: I found another place where this can theoretically happen https://github.com/Vaultio/EtherKit/blob/master/EtherKit/RequestProtocols/RequestManager.swift line 45 , there may be others

Acknowledged! Thanks @pteasima!


tridentctl install --generate-custom-yaml: Create setup directory if not exist

tridentctl install --generate-custom-yaml fails if setup directory does not exist:
$ tridentctl install --generate-custom-yaml
YAML generation failed; could not write namespace YAML file; open /workdir/setup/trident-namespace.yaml: no such file or directory

I prefer the behavior that creates setup directory if not exist. WDYT?

This has already been fixed in 19.10 in 5441a38f60ec61f1b443ee9cc884d70477781eae

Great! Thank you.


add mdc_web_sass project to third_party/pkg

Adds a new package, mdc_web_sass that allows us to generate and use the Sass files from material-components-web

@RedBrogdon ready for another look

@RedBrogdon Since this is redistributing the material-components-web source code, I used the  same license file + copyright notice. (license)

LGTM.


Dispatcher: reversing packets with SVC destination address

Currently packet reversing is only used when creating replies for SCMP echo/traceroute/recordpath packets.
This works as long as the destination is not an SVC. In the case of SVC destination address, the dispatcher would need to:

Resolve SVC addresses
Possibly allocate more space for the address header, ie. SVC to IPv6 address.

NOTE: It is not clear if this is a desired feature and if SVC Multicast should be supported as it is a trivial amplification attack.

Service multicasts are no longer supported and the dispatcher will be deprecated soon (see scionproto/scion#3961). Closing this.

Service multicasts are no longer supported and the dispatcher will be deprecated soon (see scionproto/scion#3961). Closing this.


please add "copy path" to command palette

same to the title

Just to clarify, you meant... copy the path if the cursor is somewhere in the path even with out selection ?

I hope add "copy path" command, so i can copy the active file path.

Gotcha! Will do that 👍 Thanks for the suggestion!

Hey @DFVSQY I released a new version of extension with the copy function.
Other functions include converting ~ to home directory and home directory back to ~.
Test it out and let me know if you got any feedback! 😃

@cg-cnu I have a try, it works very well, thank you very much

You are welcome! Am closing this issue 👍


Duration field in resource report needs to be String

Somehow this got converted to a bare integer in Chef-15 refactoring
which causes the server-side to reject it, so reverting back to the
old format.
Closes #8532

Unclear to me how I managed to do this, I suspect I was playing around with the idea of pushing a bare float value with decimal seconds out in the resource_reporter and data_collector, but then decided to play it safe and reverted that change, but failed to restringify it here (the data collector also has nearly identical code, but correctly stringifies it) so that this was likely just a simple mistake in playing around with code.
Not caught simply because we had no test around this.


mill doesn't use COURSIER_CACHE

On WSL 2, mill will download cache to the project related directory ./coursier.
Setting bash env COURSIER_CACHE=~/.coursier, it seems not work.

This is probably because we override it for WSL without checking whether it was already defined: https://github.com/com-lihaoyi/mill/blob/067c7c7c9b020da0dcc14af1c122a7cddc4b0386/build.sc#L778


Wrong name for Diagnosis.BatteryVolt and other mistakes in the object structure "Diagnosis"

The "name" for the state Diagnosis.BatteryVolt is "Wirkungsgrad of that tower"
To Reproduce
Open object browser and view the objects
Expected behavior
The name should read something like "Voltage of Battery"
Screenshots & Logfiles

Versions:

Adapter version: Version from github (1.5.1?)
JS-Controller version: 6.0.9
Node version: 19.9.0
Operating system: Linux/Raspian

Kümmere mich drum, siehe auch Kommentar im PR

repaired in github, will be published in new version, thx for the hint!


✨ Add XX-plus-YY and XX-minus-YY gate support to ZX library

Description
This PR adds the last remaining unsupported gates to the ZX library.
In the process, it also introduces a new convenience function for integer division of parameter expressions that simplifies some code. (I even have the slight suspicion that this fixed a small underlying bug with integer division somehow not being carried out correctly because the integer would be converted to a PiRational. I am not 100% sure though.)
Fixes #343
Checklist:

[x] The pull request only contains commits that are related to it.
[x] I have added appropriate tests and documentation.
[x] I have made sure that all CI jobs on GitHub pass.
[x] The pull request introduces no new warnings and follows the project's style guidelines.

LGTM. I checked with the qiskit decompositions and everything seems alright with me. Just a tiny optimization but for me its fine to omit it.
Another thing that I thought of is that Z rotations can be fused with the Control of CNOTs before or after (same for X rotations and targets). But I think that happens in more places than the added gates so it can be handled separately.

Thanks for the feedback! I remember that I even tried that from the very beginning but I was getting some kind of weird results and couldn't get tests to pass.
Now that investigated a little more, I stumbled upon #486, which is kind of unfortunate and explains the errors I got on my first try.


How can I augment a property on the gulp command line?

In the Maven SonarQube plugin and the command-line "sonar-scanner" tool,  I can set the "sonar.branch" property, and that will use the legacy branching mechanism in SonarQube to use a project with the branch name appended to the project key and name, with a space between.
This Gulp plugin doesn't appear to do that, so I guess I'll have to dynamically set the project key and name properties to reflect the branch in use.  I see how the "projectKey" and "projectName" properties are set in the "gulpfile.js" file, but I will need to set these properties on the "gulp sonarscan" command line.  In fact, I have to more than set them, I have to simply append a value to them.  How can I do this from the "gulp sonarscan" command line?

I've figured this out.  I just had to change the gulpfile to check for an "argv" value.  Fortunately, setting the "sonar.branch" property from that was all that needs to happen.


Threshold-based replays need to send metadata in the replayed request

When replaying due to a write attempt, we want the replayed request to set a cookie on the client that will force replays for a period of time after that write. But if the replay is NOT a write attempt, and was a GET request triggered by this rule, it should NOT reset a cookie.
Currently, this is the behavior, and it results in all requests going to the primary region unless NO requests are made within the threshold.
To fix this, it would be helpful if the Fly route can accept something like the following response header:
Fly-Replay: region=iad, reason=exception
And then, on the replayed request:
Fly-Replayed: source=<instance id>; reason=exception; region=<region>
This is also needed to figure out when a replay has happened. Currently, we're using a hack that detects two timing values in Fly-Dispatch-Start.

I've just deployed a version that changes our set of headers like:

fly-replay’s value may now include state=<anything>
fly-replay-src: instance=<id>;region=<region>;t=<microseconds since epoch>;state=<optional state>
fly-dispatch-start’s value now includes instance=<id>

Done in #5


CSI Driver tokenRequests value incorrect when setting up from Cloud Shell

When you set up the CSI Driver from Cloud Shell the CSI Driver picks up a wrong value in the TOKENREQUESTS audience field. As a result, pods will not be able to mount any storage volumes and show the error message in the logs:
Pod event warning: MountVolume.SetUp failed for volume "xxx" : rpc error: code = Internal desc = failed to prepare storage service: rpc error: code = Internal desc = storage service manager failed to setup service: timed out waiting for the condition.
Workaround:

Manually delete the csidriver using kubectl delete csidriver gcsfuse.csi.storage.gke.io
Update the file deploy/base/setup/csi_driver.yaml and edit the property value audience
Recreate the csidriver using kubectl apply -f deploy/base/setup/csi_driver.yaml
In case there are running deployments, clean up the environment and redeploy.

Thanks for reporting this installation issue.
With this fix, you can now specify the project id when you install the driver, for exampe:
make install STAGINGVERSION=v0.1.2 PROJECT=<cluster-project-id>

Closing this issue.


README has incorrect commands

It appears as though the commands are incorrect here https://github.com/facebook/create-react-app#npm

Nevermind, I see that they are correct now.


How do I create very small SSD for learning how it works?

I don't want any VGG or GiganticResNexT-9000 base model. I want a few conv layers and train it with an easy dataset e.g. tracking a PS joystick that cannot deform or change color like a lizard.

Can you recommend article/course that teaches SSD so that you can implement it from the ground up?

Sorry, I was so late. In order to make a small SSD, you must first understand how SSD was created. I recommend that you read the paper a few more times to grasp the idea (no offense, I had to, too).
In short, you don't need that many output layers since their purpose is to learn the features of objects of different sizes. If your data only has small objects (i.e. ~ image_size / 10), you can only use the output from conv4 and conv7 (or just conv4). You may also want to read more about the receptive field to understand the effect of CNN's depth.


Publication dept affiliation CSV issue

Hi,
https://github.com/cepdnaclk/api.ce.pdn.ac.lk/blob/d736848f553d2b274d0e941cfece75042f70d313/python_scripts/publications.py#L114
This check is not working because the CSV has strings "yes" and "no" instead of True, False.
Thanks for taking a look!

Please merge https://github.com/cepdnaclk/api.ce.pdn.ac.lk/pull/31


Support for Typescript typings

This library doesn't have support for Typescript's type definitions, right?
Is it planned to add those?
I don't see any community-provided typings on DefinitelyTyped

My two cents:
for a little I bit I enjoyed flow.org and tried to add types (for flow) to paths-js (under a loose supervision from @andreaferretti).
Here's what's left https://github.com/mfirry/typed-paths-js
It's definitely doable to add Typescript typing to paths-js and actually somebody started doing it years ago (https://github.com/bdedardel/paths-typescript).
I don't think @andreaferretti is willing to do it, but It shouldn't be so hard and paths-js itself is not going to change that much so.

I could give a hand in writing typings (I'll also check the flowtypes you posted), but I haven't used the library much and doesn't seem the documentation is precise about expected types. I'll see what I can do when I have some time... I'm currently working on a fork of a react-native library which uses paths-js, and porting that from JS to TS and that's why I figured the type definitions for paths-js are missing

Hi @t-ricci-enhancers , in fact there are Scala JS bindings here, so the library is fully typed. The type system for Scala JS and Typescript are not the same, but it should be doable to translate from one to another.

the documentation doesn't seem to be precise about expected types

Can you point me to an instance of this? I have tried to be precise about the shape of expected inputs and outputs for all components, but maybe I missed something

Hi @andreaferretti nothing to be worrying about, I meant to say that the signatures of the methods and properties of returned object in the markdown documentation and examples aren't as detailed and precise as TypeScript's type definition would be. That's why I'd like to see type definitions

As I mentioned, I did the work for Scala JS, coming with definitions like this simple one or this more compelx one
You can take inspiration from there (or at least the peace of mind of knowing that the library is fully typed and does not do some strange dynamic metaprogramming trick behind your back). If you want to try writing types, I can give help when you get stuck

Yes, I've plenty of informations from your comments, thank you

For some (weird) reasons I had completely forgotten about ScalaJS bindings. Happy new year!


Documentation: Using OpenAPI Generator to Generate

Code for a Data Catalog Connector (in the Go Language)
Signed-off-by: Doron Chen<EMAIL_ADDRESS>

I approve it, but can you think about a shorter left side menu title?

How about simply: "Using OpenAPI Generator"  ?


Is it possible to remove a registered HTTP resource form uWS::SSLApp?

Is there a way to remove a registered handler of a specific resource path in uWS::SSLApp?
In my use case, I register handlers with the server to handle a given resource path on-demand. I also want to be able to remove such handlers from the server dynamically, so that the removed resource should not be accessible.
Following is the code snippet I use to register my handlers
_server->post(std::string(http_info._resource_path),                                   [http_info, this](uWS::HttpResponse<true> *uws_response,                                                     uWS::HttpRequest *uws_request){// My handler impl});

No but it's a feature I want. You can only add routes for now, but you can add them under a domain name (SNI) then remove the whole SNI domain name and add the routes you want again. So technically, yes for SNI it is possible but for regular non-SNI use it is not.
I want a simple way to remove routes in any case (both SNI and non-SNI).

I added removable routes. You remove the route by passing nullptr as handler. So app.get("/", nullptr) removes a prior app.get("/", some lambda here);

_server->post(std::string(http_info._resource_path), nullptr)


Adding runCmdShouldPass function with retry

Signed-off-by: Mohammed Zeeshan Ahmed<EMAIL_ADDRESS>What is the purpose of this change? What does it change?

This allows us to retry commands that can be flaky a few times
before declaring them as failures
How to test changes?

This should reduce failures in OpenShift CI

/assign @amitkrout
/assign @syamgk

:white_check_mark: odo build 2186 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/c044402294 by @mohammedzee1000)

:white_check_mark: odo build 2187 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/10e9f7804e by @mohammedzee1000)

:white_check_mark: odo build 2189 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/f40ec32a1c by @mohammedzee1000)

:white_check_mark: odo build 2193 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/09763c283a by @mohammedzee1000)

Workaround for https://github.com/redhat-developer/odo/issues/1504

/lgtm

/retest

/meow

/test e2e-core

@syamgk already done https://github.com/openshift/odo/issues/1504


'sliced-scroll'  query ES data  used   for a long time

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[x] elastic.v6 (for Elasticsearch 6.x)
Please describe the expected behavior
I run this 'sliced_scroll' ，
print logs :
Scrolled through a total of 13378 documents in 8.024080607s
Slice  0 received 663 documents
Slice  1 received 649 documents
Slice  2 received 685 documents
Slice  3 received 663 documents
Slice  4 received 700 documents
Slice  5 received 652 documents
Slice  6 received 690 documents
Slice  7 received 646 documents
Slice  8 received 664 documents
Slice  9 received 660 documents
Slice 10 received 713 documents
Slice 11 received 704 documents
Slice 12 received 611 documents
Slice 13 received 717 documents
Slice 14 received 662 documents
Slice 15 received 645 documents
Slice 16 received 658 documents
Slice 17 received 659 documents
Slice 18 received 686 documents
Slice 19 received 651 documents
i do not know why  it used so much  long time ....
How to get a large amount of data to reduce time ?

Without any data and code, I cannot do anything to help you.

I think you were just testing the sliced_scroll recipe, right? On a single server, sliced scroll will probably have worse performance than normal scroll. It's when you have a large cluster that sliced scroll will help.
Here's my results on a MacBook:
$ go run sliced_scroll.go -index=testdata -type=_doc -n=16
Scrolled through a total of 20000 documents in 7.002732099s
Slice  0 received 1100 documents
Slice  1 received 1337 documents
Slice  2 received 1338 documents
Slice  3 received 1347 documents
Slice  4 received 1317 documents
Slice  5 received 993 documents
Slice  6 received 1277 documents
Slice  7 received 1324 documents
Slice  8 received 1301 documents
Slice  9 received 1317 documents
Slice 10 received 1025 documents
Slice 11 received 1339 documents
Slice 12 received 1344 documents
Slice 13 received 1332 documents
Slice 14 received 1309 documents
Slice 15 received 1000 documents
$ go run sliced_scroll.go -index=testdata -type=_doc -n=4
Scrolled through a total of 20000 documents in 4.519559985s
Slice  0 received 8061 documents
Slice  1 received 3953 documents
Slice  2 received 4006 documents
Slice  3 received 3980 documents
$ go run sliced_scroll.go -index=testdata -type=_doc -n=2
Scrolled through a total of 20000 documents in 4.161829244s
Slice  0 received 12067 documents
Slice  1 received 7933 documents

thanks for your replay!
when i use curl -XGET http://xxxx:9200/index/_search?size=10000  ,the way  responsed  so quickly.
20000 docs spent 4 seconds normal?

Is there any better and faster way to get around 50,000 docs?

Doing that curl query takes 5s on my machine:
curl -XGET 'http://localhost:9200/testdata/_search?size=10000'
...
0.01s user 0.03s system 0% cpu 5.005 total

But you have to make sure that you don't spend time waiting for screen output. What are the numbers if you don't count that...?

The time it takes is very dependent on the data you save. E.g. do you have 10 bytes of JSON per document, or 2MB? There's so many different things that you cannot simply take a stop watch to make any statements about performance.

thanks!
The 4s is  used the 'sliced_scroll' demo , and  not included waiting for screen output.
10000 docs is around 3MB.


Introductions

Hi all:
As you are joining the team and getting acquainted with the project (and learning git).
Please do the following to add your name to the "Teams" section of the README.md.

clone this repository to your computer
Add your name and a short blurb about yourself in the README.md file
add the changes you made
commit your changes
push your changes back up to GitHub

Most likely this will block you from pushing. Just let me know about it this issue and I can add you as a contributor.

You should see your changes when you go to the repository page under the "Teams" section.
This task also serves as your understanding of Git and makes sure the settings on this repository are correct. So, please let me know if you run into issues.

yay @gitchrissi ! You're the first one! :tada:

Just realized. depending on when people clone the repository, when you try to push your changes you may be blocked for one of 2 reasons

Permissions (403 error): let me know your GH username so I can add you to this repository as a maintainer
Something about the remote having changes you don't have: if you keep reading the error message it's essentially telling you that you'd need to pull first before pushing again.

You may run into a merge conflict here depending on what lines were changed. Just let me know if you end up with problems here
The tl;dr is you need to open the README.md file and remove the >>>>>>>, =====, and <<<<< and clean up the entire file so you're happy with it. Then add, commit, and push again.

@mitchdolby your commits acd5408d606fa6e8c662998aef8a5452a626a39a and b26b25454429b163c75e9900d779cafc58a57898 didn't update the README.md file. It looks like you used the commit message for the text you were trying to put into the README.md file.

The master branch has been locked so, in order to update your name in the README.md file you also need to see the PR instructions in #4


fix(cli/windows): fixed trailing slash cmd issue

closes #67
@Mikachu2333 can you check that issue is fixed?

I'll check it.

Fixed.
But, the -s option goes wrong and play no role when output... Maybe I'd better open another issue for this qusetion rather than discuss in this pr?

This is a error of rust args() method, and clap parsing, I can't fix it https://github.com/rust-lang/rust/issues/72653

As a fix, just place -d option at the end of command

thanks for answering!


Misleading statement on XUA, EUA and BPPC

The statement starting from line 139 in IUA.b Rev 1.2, September 21, 2019 is misleading and may be wrong in its essence and shall be refined. The intention is not clear to me.
Currently it states:
The HTTP RESTful services may include user driven browser activity, downloaded applications, and automatic devices. The existing IHE ITI XUA Profile fills these needs for the SOAP transport based transactions. The existing IHE ITI EUA Profile fills these needs for various different transports within a single enterprise environment, including HTTP RESTful transports. The Basic Patient Privacy Consent (BPPC) Profile is associated with this profile and these other existing profile.

won't fix.


Process to publish the Plugin from starter kit to Azure Functions from Visual Studio

Kindly share the process to publish the plugin from starter kit in Visual Studio to Azure Functions.
https://github.com/microsoft/semantic-kernel-starters/tree/main/sk-csharp-chatgpt-plugin

The solution to this problem was provided by Rafferty over LinkedIn. Hence, closing this issue.


Add nodeSelector and schedulerName options to instancetype

What this PR does / why we need it: It adds nodeSelector and schedulerName fields to VMIInstancetype spec. With this additional field it's possible to change standard assignment of these fields.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): None
Special notes for your reviewer: None
Release note:

`nodeSelector` and `schedulerName` fields have been added to VirtualMachineInstancetype spec.

/test all

/test all

/retest-required

This goes against https://github.com/kubevirt/kubevirt/pull/8069. Do you want to join the conversation there?

This goes against #8069. Do you want to join the conversation there?

Thanks for highlighting this, I'll bring it up on the ML and community meeting before moving my deprecation PR forward.

As highlighted before by @xpivarc https://github.com/kubevirt/kubevirt/pull/8069 has now landed deprecating VirtualMachineInstancePresets for removal in a future v2 version of the core API. Are you sure you want to proceed with this given that?

@lyarwood I just updated the MR to also include the changes in the VMIInstancetype spec.

@VirrageS  Would it be sufficient for your needs to only include these changes for instancetype?

/cc @davidvossel
Thoughts on this David? It adds a different catagorey of attributes to the VirtualMachineInstancetypeSpec than was originally planned for but it does make some sense to me to have it here.

InstanceTypes exist roughly to describe the shape of the VM (small, medium, large, etc...) and preferences exist to tune that VM for a specific OS.

As I agree with this statement, I think it is also important to note that whenever you describe the shape of the VM you might want to specify for example the node selector "on which node this small/medium/large VM should be deployed" and schedulerName to say "how exactly this small/medium/large VM should be distributed". Because otherwise you need to remember that this "small VM" needs to be only scheduled on this given node selector, or should use this custom scheduler name. And when you, as a simple user, just simply references certain InstanceType for your VM you might not know the details of how exactly this particular VM should be handled. You only care about having this VM up and running.

@davidvossel could you respond? :)

/remove-lifecycle stale

/retest

Seems like all tests pass :)

@lyarwood Does this make sense? To me, at least the scheduler seems valuable.

@lyarwood rebased :) If you have time please take another look

/retest-required
/lgtm

@rmohr @rthallisey could you guys take a look to approve? Seems like all tests are passing

/retest-required


GH-339: RabbitMQ Stream Producer: Initial Support

Resolves https://github.com/spring-cloud/spring-cloud-stream-binder-rabbit/issues/339

If nothing more to add,

Not at this time.


get variant calling workflow from Goncalo (U. Mich.) (see epic # 129)

This is part of the epic #129 (onboarding the variant calling and alignment workflows into Dockstore).

Walt took this over https://github.com/DataBiosphere/topmed-workflows/issues/10


QUESTION: What is the difference between update_call and call_candid_as?

Both of them let me make calls to a specific canister ID as a specific sender. Is there a difference or are they doing the same thing?
@frederikrothenberger Thoughts?

Hi @saikatdas0790
call_candid_as uses update_call internally and handles the Candid encoding / decoding around the inputs / outputs. update_call only handles Vec<u8> as input and output.


Added Python solution to 206.Reverse Linked List

Looks like you've pushed an empty file @JoyKrishan. Please check.

Please check @Tahanima apu

Please check @Tahanima apu

Can you please change the folder name from 206_Reverse_Linked_List to 206_ReverseLinkedList to match the convention? Also, please delete the file Algorithms/Medium/7_ReverseInteger/Solution.py.

Great work! 🔥🔥🔥


Slider in slider

Hi,
I apologize in advance, I do not speak English well, translated as an interpreter. I need to implement such a task. In the slider place slides photo. But the problem is that when you try to make a svayp on the secondary carulleys, the main one is also scrolled. It may be possible to remove the main scroll from the area of the secondary sliders. Thank you.
Example of sliders in the slider https://codepen.io/Vlastelin/pen/EgwAqd

You must enable nested: true parameter on nested sliders


ClusterSink Resource not found after upgrade to 1.2.1

CC: @ahevenor
After upgrading to PKS 1.2.1, the ClusterSink resource is missing for clustersinks.apps.pivotal.io
We discovered while packaging the clustersink that opsman only executes add on spec jobs when a unique name is used. We addressed this eventually but have on confirmed that 1.2.1-build.9 uses the same name as previous tiles and therefore ends up not getting the clusterSink (but would on a fresh install). the final RC and GA versions address this problem
Workaround
I am having the following public-facing KB article published:
https://community.pivotal.io/s/article/ClusterSink-Resource-not-found-after-upgrade
Request
We would like to have this mentioned in

Known Issues
https://docs.pivotal.io/runtimes/pks/1-2/release-notes.html#v1.2.1-known-issues
Creating Sink Resources
https://docs.pivotal.io/runtimes/pks/1-2/create-sinks.html

Thanks for calling this out, @kroberts7! We've updated the docs with this info in the two locations you mentioned:

https://docs.pivotal.io/runtimes/pks/1-2/release-notes.html#v1.2.1-known-issues
https://docs.pivotal.io/runtimes/pks/1-2/create-sinks.html#define-cluster-sink


Addresses #10 / Update semver to the latest to fix CVE / Remove unnecessary dependencies

The PR does the following things:

Set the target to Node.js 12+

Removes all unnecessary polyfills

Replace is-core-module with require('module').Module.builtinModules, replace has with Object.prototype.hasOwnProperty.call

Why fewer dependencies is better: https://github.com/un-es/eslint-plugin-i/pull/11#issuecomment-1616631165

Update semver to the latest version

This fixes the following issue:

https://github.com/import-js/eslint-plugin-import/issues/2803
https://github.com/import-js/eslint-plugin-import/issues/2804
https://github.com/import-js/eslint-plugin-import/issues/2805
https://github.com/import-js/eslint-plugin-import/issues/2806
https://github.com/import-js/eslint-plugin-import/issues/2807
https://github.com/import-js/eslint-plugin-import/issues/2808
https://github.com/import-js/eslint-plugin-import/issues/2809
https://github.com/import-js/eslint-plugin-import/issues/2814

This also addresses the following PRs:

https://github.com/import-js/eslint-plugin-import/pull/2810
https://github.com/import-js/eslint-plugin-import/pull/2812

eslint-plugin-i is a fork of eslint-plugin-import that replaces the tsconfig-paths with the faster and more lightweight get-tsconfig, which also addresses the following issues:

https://github.com/import-js/eslint-plugin-import/pull/2447
https://github.com/import-js/eslint-plugin-import/issues/2770
https://github.com/import-js/eslint-plugin-import/issues/2771
https://github.com/import-js/eslint-plugin-import/issues/2782
https://github.com/import-js/eslint-plugin-import/issues/2783

If you want to replace eslint-plugin-import with eslint-plugin-i, just run following command in your projects:
# npm
npm install -D eslint-plugin-import@npm:eslint-plugin-i@latest
# yarn
yarn add -D eslint-plugin-import@npm:eslint-plugin-i@latest
# pnpm
pnpm add -D eslint-plugin-import@npm:eslint-plugin-i@latest

Ops, it seems that I forgot to remove those outdated Node.js versions from the CI matrix!

What benefits would we get from this do you think?

What benefits would we get from this do you think?

Less disk occupation due to smaller node_modules, faster npm i due to fewer packages needing to be installed, faster eslint . since polyfills are (and always will be) slower than native methods.

Can we drop is-core-module as well?

Can we drop is-core-module as well?
UPD:
Maybe we can't :(

@so1ve Actually maybe we can! The module.builtinModules is definitely available in Node.js 12.0.0+!

I'm not so sure for these, actually I was thinking to maintain this package separately from the upstream.
As you said, such changes make rebasing a bit complicated, if we do so it would mean we're really start "forking", but it would mean a lot of extra efforts and hurt the community I think.
But of course, I would like to listen from the community, I was trying to support same the same node versions as upstream but it's also boring and "ridiculous". So I like such changes personally, but also I'm in awe of this impactful project.
Let's wait for some more feedback for a while.

Additionally, less dependencies may reduce possible supply-chain attacks.

You can replace has with {}.hasOwnProperty.call, also.

@JounQin
Although ljharb hasn't been actively maintaining eslint-plugin-import for literally months while he is basically held that open-source project as a hostage, we should still try our best to stay close to the upstream.
I have rebased the PR and minimized the affected lines to ensure that there will be least merging conflicts in the future.

Thanks, let make it happen.

GitHub Action broke again.

Let's drop node 13 15 17 19

We also need to update the Node.js versions in appveyor config.
https://github.com/un-es/eslint-plugin-i/blob/616ac75b11731f41a3786bdbe2c7620d65df9ded/appveyor.yml#L8-L14

I guess we can delete that file because we don't use appveyor at all

We also need to update the Node.js versions in appveyor config.
https://github.com/un-es/eslint-plugin-i/blob/616ac75b11731f41a3786bdbe2c7620d65df9ded/appveyor.yml#L8-L14

I guess we can delete that file because we don't use appveyor at all

I don't know. IMHO the PR has contained enough changes. @JounQin What do you think?

I don't know. IMHO the PR has contained enough changes. @JounQin What do you think?

It's not used, then no change is needed.

For anyone may be interested, I just upgraded debug to v4 and removed unused resolve dependencies at v2.29.1. And I tried to upgrade minimatch to v9 but it fails on some cases like
https://github.com/import-js/eslint-plugin-import/issues/2900#issuecomment-1859477364
and
https://github.com/import-js/eslint-plugin-import/blob/ee5fadeffff68f2300bed7f67a310496cb969d61/tests/src/rules/no-internal-modules.js
I don't have enough time to work on this replacement, anyone interested can help to raise a PR which will be appreciated.

For https://github.com/import-js/eslint-plugin-import/issues/2900#issuecomment-1859477364, I think it should be changed to invalid cases instead.


draft init --dry-run --debug is somewhat useless

It's sorta useful in the fact that the information about the chart is displayed, but the actual information is not truly useful outside of debugging the chart. It would be cool if this could be improved to make it load-able via helm install, dumped to a .tar.gz or something.

I'd be happy to take a stab at this!

closing as irrelevant in master.


Publish 8.2-20230426230204+0000

Publish 8.2-20230426230204+0000

@bot-gradle test and merge

I've triggered a build for you.

Publish 8.2-20230426230204+0000

Publish


code mentions "code block above" or other directions

This code is creating a search results object. It's using the algoliasearchHelper library to do so. The code block above creates an instance of SearchResults, which takes two parameters state and hitsPerPage. State is the current state of our search (query, page number etc.) and hitsPerPage is how many items we want per page. It then returns this new SearchResults object with all its properties set as described in the documentation for SearchResults here https://www.algolia.com/doc/apireference/widgets/searchresultsreact/#SearchResultsProps .

actually usually the code block is below, so that could be confusing

Addressed on the API layer, should hopefully be fixed :)


Hard Clue: Blow a kiss between the tables in Shilo Village Bank. (Missing Step)

Clue plugin doesn't recognize this step. (Missing Step)

I have a fix for this. I will put a PR in a hot minute.


🚚 [Custom Loops] Thresholding refactor

Description

Refactors thresholding in custom loops.
[ ] TODO: Need to make similar changes to other configs if this design is approved

I wonder if the design could be simplified further. For example, what if we apply thresholding before normalization? In the validation loop, we could return the threshold value from ThresholdConnector.compute in the validation loop, and pass it to MetricsConnector.set_threshold. This would have the following advantages:

By explicitly passing the threshold values, the code becomes easier to follow.
We only need to set the threshold once after validation, and when loading the model from file.
We would not have to store the threshold value in the trainer, leading to a cleaner class implementation.
The logic in MetricsConnector.set_threshold would be simplified.
The value of the threshold would always be in the same (un-normalized) domain, making debugging easier.

Of course, we would have to confirm that everything still works as expected. We would also have to make some changes to the MetricsConnector. Specifically, the predictions would have to be collected after each step instead of at the end of the epoch.

The problem is that threshold is used by post_processing_connector and normalizer. Since it is accessed by a lot of modules, I thought it made sense to make it a part of the trainer instead of accessing it through self.trainer.threshold_connector.image_threshold. Basically remove one layer of access.

Closing as we are moving to fabric


native modules .node problem

Node-Resolve Plugin Version: 5.2.0
Rollup Version: 1.16.4
Operating System (or Browser): Linux
Node Version: 11.0

How Do We Reproduce?
yarn add atomic-counters
console.log(require('atomic-counters'));

Expected Behavior
const ac = require('./nameAsHashOfFileOrSomethingLikeThat.node');
// file .node in same directory as js file

Actual Behavior
const ac = require('../../build/Release/atomicCounters.node');
// that path is wrong, points to nothing

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. :beer:


can't install docker. Will it also work an other way?

I can't install docker. So is there also an other way to use this blog system?

Hey !
Yep there is another way but before we could try to find why docker isn't working on your system.
What's the error ?

I close the issue because it's not a bug I think.


index out of range panic,  runtime.heapBitsForAddr

What version of Go are you using (go version)?

$ go version
go version go1.15.12 linux/amd64

Does this issue reproduce with the latest release?
No
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/root/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/root/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build923979093=/tmp/go-build -gno-record-gcc-switches"

What did you do?

fatal error: index out of range
runtime stack:
runtime.throw(0x100d065, 0x12)
/usr/local/go/src/runtime/panic.go:1116 +0x72
runtime.panicCheck1(0x423e55, 0x100d065, 0x12)
/usr/local/go/src/runtime/panic.go:34 +0xdd
runtime.goPanicIndexU(0x3080200000, 0x400000)
/usr/local/go/src/runtime/panic.go:91 +0x45
runtime.heapBitsForAddr(...)
/usr/local/go/src/runtime/mbitmap.go:340
runtime.scanobject(0xc200000000000000, 0xc00008ee98)
/usr/local/go/src/runtime/mgcmark.go:1306 +0x3b5
runtime.gcDrain(0xc00008ee98, 0x7)
/usr/local/go/src/runtime/mgcmark.go:1143 +0x22b
runtime.gcBgMarkWorker.func2()
/usr/local/go/src/runtime/mgc.go:1981 +0x177
runtime.systemstack(0x0)
/usr/local/go/src/runtime/asm_amd64.s:370 +0x66
runtime.mstart()
/usr/local/go/src/runtime/proc.go:1116
goroutine 69 [GC worker (idle)]:
runtime.systemstack_switch()
/usr/local/go/src/runtime/asm_amd64.s:330 fp=0xc000221f60 sp=0xc000221f58 pc=0x4708e0
runtime.gcBgMarkWorker(0xc00008d800)
/usr/local/go/src/runtime/mgc.go:1945 +0x1be fp=0xc000221fd8 sp=0xc000221f60 pc=0x41fffe
runtime.goexit()
/usr/local/go/src/runtime/asm_amd64.s:1374 +0x1 fp=0xc000221fe0 sp=0xc000221fd8 pc=0x4726c1
created by runtime.gcBgMarkStartWorkers
/usr/local/go/src/runtime/mgc.go:1839 +0x77
What did you expect to see?
please help resolve panic
What did you see instead?
Panic after the program runs for a period of time

This won't be easy to investigate without a reproducer. Can you provide a (possibly short, auto-contained) one?
Also, usual questions:

Does your program use unsafe? What about cgo?
Does it print any warning when ran with the race detector enabled? -race?

Does your program use unsafe? What about cgo?
I used unsafe and cgo. I will remove the unsafe call and pressure test cgo.
Does it print any warning when run with the race detector enabled? -race?
I found many print places with the race detector enabled. I will fix these places.
I will try first, thanks

Thanks. As you know a program containing data races is not a valid Go program, and it's very very likely that this crash is caused by these data races (they can corrupt internal runtime structure). I'm closing here for now, but feel free to comment if you can still reproduce the crasher after fixing the program.


About the application

Hello, I'm a researcher from Renmin University of China(Peking, China). I appreciate your work here and try to apply your codes to refitting my lithic models. But I'm not very good at programming, and   have some difficulties in amending the codes related with ICP methods codes(Installing TEASE++ is a little hard for me on the Win11 platform). Would you like to offer some instructions? I am looking forward to your reply.

Hi @yanlingweihuili , thanks for your interests.
TEASER is usually better (especially when initial position is bad), but if you want to use ICP you can follow the instructions. Did you try to set 'icp' in the configuration parameters (as explained) and run the code? I used it some time ago and it worked, but as mentioned in the instructions, it might require a little tweaking of some parameters or wrapping some parts of the results.
If you can try to run it and explain what happens, I can try to guide you to fix the ICP-related part or add/correct some code.


[doc] readthedocs only shows latest (master) version, instead of released versions

This is more important, once we have more users as they surely want to read the documentation for a specific version.

I can switch between versions. Have you used the link on the main page? -> www.bam.de/weldx

The enforced "dark mode" in my browser did hide the choosing box ;-)


Any plans for ubuntu linux?

nice work. no plans for linux ubuntu?

It's a shell script. Why wouldn't it work?


tag regex doesn't handle attributes containing >

I've recently had a number of excel files fail when parsing the styles, with a fairly unclear error message:
TypeError: First argument must be a string, Buffer, ArrayBuffer, Array, or array-like object.
at fromObject (buffer.js:262:9)
at Function.Buffer.from (buffer.js:101:10)
at Buffer (buffer.js:80:17)
at utf8readc (.xlsx/xlsx.js:1717:52)
at parse_numFmts (.xlsx/xlsx.js:7046:23)
at parse_sty_xml (./xlsx/xlsx.js:7147:34)
at parse_sty (./xlsx/xlsx.js:12836:9)
at parse_zip (./xlsx/xlsx.js:17104:26)
at read_zip (./xlsx/xlsx.js:17393:9)
at Object.readSync [as read] (./xlsx/xlsx.js:17418:68)

I've tracked the issue down to the fact that the current regex used for detecting tags is /<[^>]*>/g, which can't handle <numFmt numFmtId="164" formatCode="[>0]General" /> as it matches the closing angle bracket inside the formatCode attribute and interprets that as the end of the tag. The regex should be updated to include any number of attributes, disregarding their content, prior to the end of the tag.

@MonochromeChameleon thanks for the report!   Do you know how were these files produced?  In your example, if you put the format code in Excel and save, the generated xml has the encoded >

I'm not sure exactly where the format code was coming from - I wasn't adding it myself! I'm using excel on a mac instead of on windows, which may have made a difference?