What would be cool for a project in the future would be a UWP version of the app launcher for people on W10M.

Copied from old repo:
"Basically, that would be called UXL Mobile and it would allow the user to launch Office Mobile apps from one place instead of having multiple tiles on their Start screen. This project would be something to think about eventually, but not right now. I want to focus on version 2.8 then work on the 3.0 rewrite. Once those are done, I can think about UXL Mobile again. Or the person could use it in Continuum."

Here's a list of Office URI schemes, including the URI's for Office software:
https://msdn.microsoft.com/en-us/library/dn906146

Actually, maybe I could make a web app have buttons to launch Office Mobile apps for UXL Launcher Mobile. That might work a little better for my purposes.

Windows 10 Mobile is kindof dead now, but in the future with foldable devices, this could be helpful. I'm thinking that maybe this could have some use on Andromeda.


in tag  v0.3.0-beta.4 webapp proxy is invalid

hi,i download v0.3.0-beta.4 source,then open webapp use webstorm , after run the webapp ,by cannnot send  request to proxy url successfull.
but ,when run the webapp ,the console log show [HPM] Proxy created: /  ->  http://xxxxx:8080/ ,
i cannot find the reason

The default proxy address of Davinci api is http://localhost:8080/, the path of the config file is
webapp/server/middlewares/addDevMiddlewares.js, you can change it to a proper value that corresponding to your exact server address.

Tks,I have do this,but it does not work,my server service is ok, Is it working properly in your environment?

Yes, make sure that you have installed npm packages properly.

hello，I found that the path in target cannot be configured with  http://localhost:8080/api/v3
To configure it to http://localhost:8080/ now it works.

mine too, I changed proxy in the addDevMiddlewares.js and termial did work, but request always http://localhost:5002
![BDFBB9EB-DF71-413c-AAC9-EE7CABB0A441](http

s://user-images.githubusercontent.com/45391716/90124653-56896f80-dd93-11ea-9f78-51e7e83ad252.png)


[Feature][Process Instance] Adding folder for process instance list

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Now, DS place all process instances in one list. In some scenarios, one project could have many process instances even exceed one hundred. In this case, user want to find some specific process is very inconvenience even have search and paging.

I think DS cloud add a FOLDER feature to manage process instances, just like windows file system, for prevent user create many deep folders, we could restrict the FOLDER depth.
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

There are currently no plans for that


Support multiple senders in connect_via

https://pythonhosted.org/blinker/#blinker.base.Signal.connect_via
We have a use case to send a 	signal to multiple senders, so we can do:
@some_signal.connect_via_multiple([sender1, sender2, ..., senderN])
def func():
	pass

# instead of
@some_signal.connect_via(sender1)
@some_signal.connect_via(sender2)
...
@some_signal.connect_via(senderN)

def func():
	pass

@jek Would like to get some opinion from you. Are you leaning on adding a new method connect_via_multiple that does the same thing as connect_via except that it accepts senders as a list, or updating the current connect_via to accept a senders as either a single sender or list of senders? I will be happy to throw in a PR.

I think it's fine to just use connect_via multiple times. If your specific use case requires a large number of these, it might indicate that some other refactor in your code might be appropriate, or you could use other strategies like a for loop or a custom decorator around this decorator.


Fix nixos-rebuild when cross building flakes

Description of changes
Fixes https://github.com/NixOS/nixpkgs/issues/166499
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg test nixos-rebuild

@ofborg test installer.simple

I think @wentasah's suggestion in https://github.com/NixOS/nixpkgs/issues/166499 is better: Not reexecing when the platform of the local system is different than the remote system.

This solution has the advantage that the same is done for non-flake systems. Also, when the rebuild fails for whatever other reason (out of disk space on localhost comes to my mind), it still might be useful to continue. So I'm not against this solution.

In any case, the solution should be the same for flake and non-flake systems. I'm a bit confused though, did this use case of a different build than a target host work previously with non-flake systems? I don't think it should've worked.
While I agree that it might be useful to still continue, some people rely on a different nixos-rebuild command, and would want it to actually fail when it doesn't build anymore. E.g. imagine someone tweaking their custom nixos-rebuild command, and it fails to build, they wouldn't want to use the normal nixos-rebuild then. I don't think continuing on a failed nixos-rebuild is useful, outside of it being caused due to a platform mismatch.

So if we want to reexec only when target and host platforms are the same, perhaps something like this can be used:
targetSystem=$(runCmd nix eval "$flake#$flakeAttr.pkgs.system")
if [[ $(nix show-config | sed -ne '/^system = / s///p') = ${targetSystem//\"/} ]]; then
    # ...
fi

I've updated the code to follow @wentasah's suggestion :+1:

cc @roberth


Refactor enable smb in determine-basal.js

So this is actually a step towards what I'm hoping to implement, but I think the enable_smb logic should be refactored to make it easier to read and add additional SMB toggles for more customize-ability in the future.
Unfortunately my oAPS rigs are broken (radios are dead), so I'm not able to test this in oAPS but I have this running in AAPS and it's running with no issue. Would welcome some testing to make sure I didn't accidentally change some logic.

Merging to dev to test there.


Why only evaluate on the final question for ConvFinQA?

Hey! I was curious why when evaluating ConvFinQA only the final question is the only one evaluated question for each example?
Seems all other questions are just appended as context to the model.

@MXueguang ran this experiment.

Hi @SeanNaren
We evaluated ConvFinQA for each question. using convfinqa_test_turn.json
The corresponding output file is
https://github.com/wenhuchen/Program-of-Thoughts/blob/main/outputs/convfinqa_test_turn_sc_s0_e-1_11_15_10_07.jsonl
I think forgot to update this line in this repo https://github.com/wenhuchen/Program-of-Thoughts/blob/64c6bea8d6391e76b7f1182d963bbfdcba0a908f/run_convfinqa.py#L88. Fixing it now.

This makes sense thank you! I think it makes sense to keep evaluating on the dev set, otherwise (unless I'm mistaken) you won't have labels to evaluate against.


go-sqlite3 did not depend on the .so?

I overwrite a bit source code of sqlite3,and want to replace the .so of system，but did not work. so,I use the ldd to check my application, can not find the SQLite3.so

Yes, please do.


InputEventJoypadMotion will trigger multiples on_value_changed with hslider

Godot version
3.4.4 (and 3.3.4, 3.5-rc6)
System information
Windows 10
Issue description
Hi all,
If you map ui_left and ui_left with the left joystick of a controller (xbox one in this case), when using a hslider, the on changed value event will be trigged multiples times.

e.g. a hslider with a step of 5%, with the stick, the value increase randomly from 10% to 20%, while using the direction cross will only trigger a 5% step.
https://user-images.githubusercontent.com/1162446/178392471-b41b5954-1482-45d8-adf1-8ad2027e1635.mp4
I think InputEventJoypadMotion should only be trigger once to match industry standard.
If this isn't a duplicate, I would be interested to take a look and create a PR if this is a good "first issue".
Thanks
Steps to reproduce
1 - Create a minimal project with a hslider
2 - Remap ui_left, ui_right to the left joystick

3 - Create a hslider
4 - Change the value with the joystick
Minimal reproduction project
hslider_issue.zip

Reproduced with a XBox One controller via USB on 3.x custom build. Doesn't seem like a "controller drift" problem

Up-to-date one, last commit in log is 3f02013

@Calinou hi, I wrote the wrong version (I tested both on 3.3.4 and 3.4.4), same behaviour on both version

Same with 4.0 alpha 10.
I think the problem isn't with the code for controller but how the slider reacts to stick's inputs.
If you push the stick to left/right and do not release, the slider will stop at some position (may have moved more than once), but sometimes when you release it, it will move again in the same direction.
So in this case, I think the desirable behaviour should be to detect delta is the same direction as previously received one, and therefore the slider should not react to it, and when inside the deadzone, consider it as released.

Could it be related to _gui_input? I remembered creating a custom carousel field. All action where trigged inside the _gui_input. But when using an analog stick, both action_pressed were trigger (both when using and released), resulting in a quite fast navigation
https://user-images.githubusercontent.com/1162446/178615774-71f480de-c82c-4b0f-8bb6-f9cf6058299f.mp4
func _gui_input(event: InputEvent) -> void:
	if event.is_action_pressed("ui_left"):
		_on_Previous_value()
		return

	if event.is_action_pressed("ui_right"):
		_on_Next_value()
		return

I fixed it by checking the raw strengh
func _gui_input(event: InputEvent) -> void:
	if event is InputEventJoypadMotion:
		if event.get_action_strength("ui_left") == 1:
			_on_Previous_value()
			return
		if event.get_action_strength("ui_right") == 1:
			_on_Next_value()
			return
		return

	if event.is_action_pressed("ui_left"):
		_on_Previous_value()
		return

	if event.is_action_pressed("ui_right"):
		_on_Next_value()
		return

https://user-images.githubusercontent.com/1162446/178615791-a9acd41c-6356-42d0-896b-dbc2c6129d6b.mp4
I saw the same behavior in slider.cpp
void Slider::_gui_input(Ref<InputEvent> p_event) {
	if (!editable) {
		return;
	}
	...
	if (!mm.is_valid() && !mb.is_valid()) {
		if (p_event->is_action_pressed("ui_left", true)) {
			if (orientation != HORIZONTAL) {
				return;
			}
			set_value(get_value() - (custom_step >= 0 ? custom_step : get_step()));
			accept_event();
		} else if (p_event->is_action_pressed("ui_right", true)) {
			if (orientation != HORIZONTAL) {
				return;
			}
			set_value(get_value() + (custom_step >= 0 ? custom_step : get_step()));
			accept_event();
		} else if (p_event->is_action_pressed("ui_up", true)) {
			if (orientation != VERTICAL) {
				return;
			}

			set_value(get_value() + (custom_step >= 0 ? custom_step : get_step()));
			accept_event();
		} else if (p_event->is_action_pressed("ui_down", true)) {
			if (orientation != VERTICAL) {
				return;
			}
			set_value(get_value() - (custom_step >= 0 ? custom_step : get_step()));
			accept_event();
		} else if (p_event->is_action("ui_home") && p_event->is_pressed()) {
			set_value(get_min());
			accept_event();
		} else if (p_event->is_action("ui_end") && p_event->is_pressed()) {
			set_value(get_max());
			accept_event();
		}
	}

Should we add someking of exception to check when an event input is of type InputEventJoypadMotion  ?

Slider inputs will likely need special handling to behave well with InputEventJoypadMotion. They should not be ignored – otherwise, you would only be able to use the D-pad to move a slider (which is poor UX).

I poorly express myself, I mean by exception, directly handling InputEventJoypadMotion as their own type on action inside the slider. Instead of behing managed by p_event->is_action_pressed("ui_up", true) should we check the get_action_strength when we handle a  InputEventJoypadMotion.
I not really familiar with the basecode so maybe there is a better solution to fix this behavior.

Maybe it's a bit too much but, what about introducing an InputEvent::is_action_held method with a delay parameter?
And then, gui components could optionally switch between is_action_pressed and is_action_held

I did a first quickfix based on what has been done in Viewport.cpp. Seems to work fine but I need to test further before creating a PR.
I would also like to improve the whole behavior, but I'm not sure if I should do it right now or create a proposal (Could @Calinou help my on this one, thanks) ?

If we quickly move the axis to the left or right, it should only increment by one step
If we held the stick to the left or right, it should increment the step until we unheld the stick (could be easier for user with motion issue)

I would also like to improve the whole behavior, but I'm not sure if I should do it right now or create a proposal

It's probably a good idea to create a proposal for this.
Ideally, sliders should behave like this with both keyboard and gamepad input. Thanks to the operating system, keyboards will automatically provide echo events, but gamepads don't, so this behavior needs to be manually implemented for gamepad input. I suggest using a delay of 0.5 seconds and a repeat rate of 20 repeats/second, similar to Windows' default rate for repeating keyboard input.
https://user-images.githubusercontent.com/180032/179607471-058e2611-76dd-48ea-a858-2726d9132ed0.mp4

Hello, it would be great to have an is_echo() function for gamepads, which would avoid this issue. Reproducing this in GDscript with timers in each interface code is feasible but generates a lot of unnecessary code.  This solution would enable gamedevs to use the function when needed (UI) and leave it be when not needed (gameplay).

Hello, it would be great to have an is_echo() function for gamepads, which would avoid this issue.

Gamepads never send echo inputs, so is_echo() would be invalid.
Instead, we should look into properly handling analog stick UI inputs out of the box. This is entirely feasible but requires manually implementing echo inputs in certain UI nodes (such as Slider and SpinBox).

I tried and add the same behavior to every node (slider, popup menu, item list, tree etc) but it seems more like a workaround (it is also my first PR so my lack of experience in c++ and Godot source code could be the real issue).
Even if a gamepad doesn't have an echo, could we implement a behavior inside Input to re-trigger an event. maybe for an UI exclusive mode?
Or we could list witch component to prioritize like the slider and popup menu and add the echoing

I understand that "Gamepads never send echo inputs, so is_echo() would be invalid" -> my point is that as a gamedev I need an is_echo() function (or equivalent) for gamepads in order to avoid detect input signals in certain situations (mainly UI). I can reproduce it with a timer in GDscript but it is tedious and ridiculous to reproduce for every game UI. Currently porting an interface game to Switch, I realize how important this is and if Godot is to be used for console games, this issue is a real problem.


Checkpoint

Your model is fascinating and I would like to test the model, can you provide the file about checkpoint please?
您好！我们对您提出的方法十分感兴趣。我们看一看实现效果，但是没有找到权重在哪里下载，可以提供一下吗？十分感谢！

You can find the link for our checkpoints under this section:
https://github.com/ShenhanQian/SpeechDrivesTemplates#demo

Thank you！


small chnge made  in the  cards (added box shadow)

Thanks for adding the shadow. It has become more awesome now. It has been merged.
I will be glad if you can leave a start to this repository.


Add ribbon filter

@ajpmaclean : : Please review.

@ajpmaclean : Indentation Fixed. No .txt file.
Please Review.

Thanks


Nullable Types

It seems it would be a good idea to make some of the values nullable. In particular, you have the ValueOrDefault extension method, which is great, but if for example, I need to get a DateTime, which happens to be empty, I will end up getting the 1/1/1, which I will then have to handle properly.
This would be even more important for ints & decimals, since they would be returning a 0, which I don't think would have the same meaning as no value.
Thank you!

Nulls are not a good way to report that there is no data in a particular field. For this, we give you the HasValue property to test whether Value/ValueOrDefault will return a default or a value. So, you could have something like this:
if (!result.Select(x => x.SomeDateTimeField).HasValue)
// do something about it

I don't agree with that, but I hear what you are saying.
However, since I am translating the data from a file to my own objects it's simpler to return a null for a value that is not there. This introduces an extra step.
Anyway, thank you for this library!
As an aside, it would be very helpful if there was some example code on reading a file. I had to navigate through the test classes to figure out how to use this library.

Thanks for the reply.
Value is a monad and similar to what is called an Option type in functional programming. It is meant to either return a value or none value.
Tell me a little more about your use case. For example, are you using HL7 or X12? How big is the file you need to parse? Are you using LINQ to HL7/X12 or Layouts? Can you share code samples?
We at least 4 different parsers, so I can perhaps guide you to the right one.
Would definitely be interested in understanding how are you using Machete so that we can make it better.

Interesting. I've never really encountered a monad, however, I see how it's useful here.
I'm right now trying to parse an X12/835 document just for display purposes, but eventually I will be saving some of it to my database.
I am working with an MVC/WebApi application where I'm having my users upload a document to our site, which we then parse and either display some data and save other parts. So far, what I'm doing is to pass the document to your parser, and then go through the document and convert it to concrete class. in the form of this class it makes it easier for me map it to my database, and display it to the user in the form of json. In particular,
I've attached some code I've written to test out using your parser. I made some mistakes using it (for example, later on I realized how to go through the LayoutList, which I was surprised has no enumerator), and I'm sure there are others.
Program.txt

fixed in release 1.0.236-develop

Hey, nycbauer please join our chatroom at https://gitter.im/PhatBoyG-Machete/Lobby?utm_source=share-link&utm_medium=link&utm_campaign=share-link

@nycbauer is this still an issue?


Benchmark run fails due to error: can not perform a '--user' install

Trying to run bert I'm hitting this error in nvidia's scripts:
Copying mlperf_loadgen.egg-info to build/bdist.linux-x86_64/wheel/mlperf_loadgen-4.0.egg-info
running install_scripts
creating build/bdist.linux-x86_64/wheel/mlperf_loadgen-4.0.dist-info/WHEEL
creating 'dist/mlperf_loadgen-4.0-cp310-cp310-linux_x86_64.whl' and adding 'build/bdist.linux-x86_64/wheel' to it
adding 'mlperf_loadgen.cpython-310-x86_64-linux-gnu.so'
adding 'mlperf_loadgen-4.0.dist-info/METADATA'
adding 'mlperf_loadgen-4.0.dist-info/WHEEL'
adding 'mlperf_loadgen-4.0.dist-info/top_level.txt'
adding 'mlperf_loadgen-4.0.dist-info/RECORD'
removing build/bdist.linux-x86_64/wheel
ERROR: Can not perform a '--user' install. User site-packages are not visible in this virtualenv.
make[1]: *** [Makefile.build:346: build_loadgen] Error 1
make[1]: Leaving directory '/home/jglauber/CM/repos/local/cache/4f4c6af97c2045cd/repo/closed/NVIDIA'
make: *** [/home/jglauber/CM/repos/local/cache/4f4c6af97c2045cd/repo/closed/NVIDIA/Makefile.build:184: build] Error 2
CM error: Portable CM script failed (name = build-mlperf-inference-server-nvidia, return code = 256)
Running as normal user or root user didn't solve it.
cm-repro logs:
https://gist.github.com/el8/2654d3638885bd3ac3d68782beecacea
https://gist.github.com/el8/ec3cafc4acd8ff3512943c33c00f6b28

Thanks, I'll retry with that instructions.
One question I had though - is the docker container mandatory to run the nvidia-closed benchmarks?

Yes, some of the Nvidia dependencies require the docker. It is technically possible to make them work outside of docker but that'll require manual download of cudnn, tensorrt etc.

Sorry for the delay, had to wait for to get the hardware back. I can confirm that using the instructions you pointed out and the nvidia container the --user error is gone.

That's great to know. I'm closing this ticket.


Rollup not running when same-object lookup nulled out

Background:
Same object rollup (counting number of child records on an immediate parent of same object), using CMDT and calling Rollup.runFromTrigger();.
Issue:
When child record has the lookup field cleared out, the parent record target rollup field is not decremented.
Repro:

Create a rollup from an object to the same object.
Fill in the lookup field and observe the rollup incrementing as expected
Clear out the lookup and observe that the rollup does not decrement as expected

Desired behavior:
Clearing out lookup should cause parent record rollup to recalc and update to correct count.

@systemsfixer thanks for submitting this one! Should be an easy fix.

@systemsfixer I'm wondering if you can send me a log with Is Rollup Logging Enabled set to true on your Rollup Control for this situation? I have a test that should be reproducing this issue using the following rollup metadata:
new Rollup__mdt(
      RollupFieldOnCalcItem__c = 'AnnualRevenue',
      LookupObject__c = 'Account',
      LookupFieldOnCalcItem__c = 'ParentId',
      LookupFieldOnLookupObject__c = 'Id',
      RollupFieldOnLookupObject__c = 'AnnualRevenue',
      RollupOperation__c = 'COUNT',
      CalcItem__c = 'Account'
)

But when I update the Parent Account field on an account, I am seeing the COUNT decrement accordingly on the old parent. A log will hopefully help to clear up the differences between the test I've written and the results you're seeing.

@systemsfixer I was able to identify and fix this issue - I will be looking to release the updated version in the next day or two


About Flow Training

I found that you haven't provide flow models, so if i want to retrain a flow model.
Are all training parameters the same with RGB training?
And do i need to train the model from scratch with random initialization?
Thanks~

We train flow model with the same hyper-parameters used in RGB training.
For initialization, we use ImageNet's weights. For the first layer, we average the weights in the channel dimension and then repeat it to the number of channels of the flow model to avoid the inconsistent of the weights' shape.

Very helpful. Thanks.


API key is hardcoded in library

It looks like the library artifact posted on maven does not perform the local.properties build step once included in a project.  It has your personal API key (AIzaSyDRD....DJPQPax5c) is inside the generated BuildConfig.java.
Rebuilding my project does not pull from my project's local.properties nor regenerate the lib's BuildConfig.java.  Meaning there is no way to use my own personal key using the maven artifact.
Love the library though :)

@tomblenz Thank you for pointing out this.
Let me check. I'll keep you posted.


Fix layout of button at Mailing Page

When we reduce width of the browser buttons in mailing form are not placed next to each other as expected.
Example: http://screencast.com/t/rbveceX2SuWj

Hi, PR created here.


Change min version to 3.7

Related to: https://github.com/pulumi/pulumi/issues/8131
Do not merge until release of pulumi 3.15.0

Closing this for now, we can revisit when we officially drop 3.6 support.


When a build fails, include the text from the last step in the lagoon UI along with the notification payload

The new Lagoon UI accordion gives a nice indication of where a build might have failed. From a developer perspective, it would be really great to have the text from the last failed section included with the build-failed notification payload.
In slack this could be a thread on the failed message containing the text, or perhaps included in a text file.

Some considerations however...

Does this log information contain potentially sensitive details such as keys - those would need to be redacted
Is the information actually available when the notification runs?
Do all the notifications (Slack, email, webhook) have a good way to display this info

So there could be reasons not to do this - then we should kill this feature request!

The information is not currently available when the notification fires, we would need to ship this information into the notification event.
Some build step failures could produce significant logs. Displaying it may not be very useful at all considering we dump the link to the build in the notification too. Our slack notifier doesn't support threading. Not sure about other notifiers, but they might not support it either (webhook doesn't even do the range of notifications, it just does build success/failures)

There is now a build step indicating a failure in the UI


99% Sync failing -- Problem Bootstraping

My pi seems stuck at 99% and I ran the following and get
sudo grep -i tor /var/log/syslog | nc termbin.com 9999
[Apr 13 13:00:03 raspberrypi systemd[1]: /lib/systemd/system/fail2ban.service:12: PIDFile= references path below legacy directory /var/run/, updating /var/run/fail2ban/fail2ban.pid → /run/fail2ban/fail2ban.pid; please update the unit file accordingly.
Apr 13 13:00:04 raspberrypi systemd[1]: Started RaspiBlitz Background Monitoring Service.
Apr 13 13:00:04 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: peers=
Apr 13 13:00:11 raspberrypi bonus.sphinxrelay.sh[5521]: # ok - copied fresh config.json & app.json into dist directory
Apr 13 13:00:51 raspberrypi tor[681]: Apr 13 13:00:51.000 [notice] New control connection opened from <IP_ADDRESS>.
Apr 13 13:01:11 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: peers=0
Apr 13 13:01:11 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: ZERO PEERS DETECTED .. doing out-of-band kickstart
Apr 13 13:01:11 raspberrypi _background.sh[5442]: #network.monitor.sh peer-kickstart
Apr 13 13:01:13 raspberrypi _background.sh[5442]: addressFormat='tor'
Apr 13 13:01:16 raspberrypi npm[5517]: 2021-04-13T12:01:16.691Z btcexp:app Loading historical data for chain=main
Apr 13 13:01:53 raspberrypi tor[681]: Apr 13 13:01:53.000 [notice] New control connection opened from <IP_ADDRESS>.
Apr 13 13:02:55 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: peers=0
Apr 13 13:02:55 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: ZERO PEERS DETECTED .. doing out-of-band kickstart
Apr 13 13:02:55 raspberrypi _background.sh[5442]: #network.monitor.sh peer-kickstart
Apr 13 13:02:57 raspberrypi _background.sh[5442]: addressFormat='tor'
Apr 13 13:03:12 raspberrypi tor[681]: Apr 13 13:03:12.000 [notice] We'd like to launch a circuit to handle a connection, but we already have 32 general-purpose client circuits pending. Waiting until some finish.
Apr 13 13:04:50 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: peers=0
Apr 13 13:04:50 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: ZERO PEERS DETECTED .. doing out-of-band kickstart
Apr 13 13:04:50 raspberrypi _background.sh[5442]: #network.monitor.sh peer-kickstart
Apr 13 13:04:52 raspberrypi _background.sh[5442]: addressFormat='tor'
Apr 13 13:06:34 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: peers=0
Apr 13 13:06:34 raspberrypi _background.sh[5442]: Blockchain Sync Monitoring: ZERO PEERS DETECTED .. doing out-of-band kickstart
Apr 13 13:06:34 raspberrypi _background.sh[5442]: #network.monitor.sh peer-kickstart
Apr 13 13:06:36 raspberrypi _background.sh[5442]: addressFormat='tor'
Apr 13 13:07:14 raspberrypi tor[681]: Apr 13 13:07:14.000 [warn] Problem bootstrapping. Stuck at 95% (circuit_create): Establishing a Tor circuit. (DONE; DONE; count 10; recommendation warn; host 03910F285A33F365838EC66EF2C2EF754D046760 at <IP_ADDRESS>:443)
Apr 13 13:07:14 raspberrypi tor[681]: Apr 13 13:07:14.000 [warn] 10 connections have failed:
Apr 13 13:07:14 raspberrypi tor[681]: Apr 13 13:07:14.000 [warn]  5 connections died in state handshaking (Tor, v3 handshake) with SSL state SSL negotiation finished successfully in OPEN
Apr 13 13:07:14 raspberrypi tor[681]: Apr 13 13:07:14.000 [warn]  3 connections died in state handshaking (TLS) with SSL state SSLv3/TLS write client hello in HANDSHAKE
Apr 13 13:07:14 raspberrypi tor[681]: Apr 13 13:07:14.000 [warn]  1 connections died in state handshaking (TLS) with SSL state SSLv3/TLS write finished in HANDSHAKE
Apr 13 13:07:14 raspberrypi tor[681]: Apr 13 13:07:14.000 [warn]  1 connections died in state connect()ing with SSL state (No SSL object)
Apr 13 13:07:26 raspberrypi tor[681]: Apr 13 13:07:26.000 [warn] Problem bootstrapping. Stuck at 95% (circuit_create): Establishing a Tor circuit. (DONE; DONE; count 11; recommendation warn; host 5790B62AD6C79A1092B30993A8F0F7F23A969FD6 at <IP_ADDRESS>:9001)
Apr 13 13:07:26 raspberrypi tor[681]: Apr 13 13:07:26.000 [warn] 11 connections have failed:
Apr 13 13:07:26 raspberrypi tor[681]: Apr 13 13:07:26.000 [warn]  5 connections died in state handshaking (Tor, v3 handshake) with SSL state SSL negotiation finished successfully in OPEN
Apr 13 13:07:26 raspberrypi tor[681]: Apr 13 13:07:26.000 [warn]  3 connections died in state handshaking (TLS) with SSL state SSLv3/TLS write client hello in HANDSHAKE
Apr 13 13:07:26 raspberrypi tor[681]: Apr 13 13:07:26.000 [warn]  2 connections died in state handshaking (TLS) with SSL state SSLv3/TLS write finished in HANDSHAKE
Apr 13 13:07:26 raspberrypi tor[681]: Apr 13 13:07:26.000 [warn]  1 connections died in state connect()ing with SSL state (No SSL object)
Apr 13 13:07:34 raspberrypi tor[681]: Apr 13 13:07:34.000 [warn] Problem bootstrapping. Stuck at 95% (circuit_create): Establishing a Tor circuit. (DONE; DONE; count 12; recommendation warn; host 9DA7EF9E9D6070CD8B7059B8A8CC5B41CEC02726 at <IP_ADDRESS>:9001)
Apr 13 13:07:34 raspberrypi tor[681]: Apr 13 13:07:34.000 [warn] 12 connections have failed:
Apr 13 13:07:34 raspberrypi tor[681]: Apr 13 13:07:34.000 [warn]  6 connections died in state handshaking (Tor, v3 handshake) with SSL state SSL negotiation finished successfully in OPEN
Apr 13 13:07:34 raspberrypi tor[681]: Apr 13 13:07:34.000 [warn]  3 connections died in state handshaking (TLS) with SSL state SSLv3/TLS write client hello in HANDSHAKE
Apr 13 13:07:34 raspberrypi tor[681]: Apr 13 13:07:34.000 [warn]  2 connections died in state handshaking (TLS) with SSL state SSLv3/TLS write finished in HANDSHAKE](url)

I tried restarting TOR and I also get this:
Apr 13 13:48:22.767 [notice] Tor <IP_ADDRESS> running on Linux with Libevent 2.1.8-stable, OpenSSL 1.1.1d, Zlib 1.2.11, Liblzma 5.2.4, and Libzstd 1.3.8.
Apr 13 13:48:22.768 [notice] Tor can't help you if you use it wrong! Learn how to be safe at https://www.torproject.org/download/download#warning
Apr 13 13:48:22.768 [notice] Read configuration file "/etc/tor/torrc".
Apr 13 13:48:22.774 [warn] Directory /mnt/hdd/tor/web80/ cannot be read: Permission denied
Apr 13 13:48:22.774 [warn] Failed to parse/validate config: Failed to configure rendezvous options. See logs for details.
Apr 13 13:48:22.774 [err] Reading config failed--see warnings above.

It seems to be a problem with TOR ( obviously ) but I am unsure how to correct this. Thanks

The logs: Directory /mnt/hdd/tor/web80/ cannot be read: Permission denied
The correction: https://github.com/rootzoll/raspiblitz/blob/da986f8c9ab40575d44b627ad6f39c6ccdcc7a68/home.admin/config.scripts/internet.tor.sh#L301
sudo chmod -R 700 /mnt/hdd/tor
sudo chown -R debian-tor:debian-tor /mnt/hdd/tor
sudo systemctl restart tor@default
sudo journalctl -eu tor@default

@ag2121-ag Please close this issue if this is solved.

This worked! thanks for the help!


An error occurs when attempting to create an account

Repro Steps:

Click "Create Account" button.
Select a subscription.
Select a resource group.
Type a name for the account.
Select an API.
Select a location.

Expected:
The account is created successfully.
Actual:
An error occurs.

This is Azure behavior. Try again.


[OpenlyBridge] Add bridge

Adds bridge for Openly

Please rebase for tests not to show red.
Also please add examples for detectParameters like I did in TelegramBridge:
https://github.com/RSS-Bridge/rss-bridge/blob/f5f0b77805452918314112b3fb3fc6022c342d56/bridges/TelegramBridge.php#L16-L20


FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory

made change in
/var/app/current/node_modules/sails/bin/sails.js
from
!/usr/bin/env node
to
!/usr/bin/env node --max_old_space_size=4096 app.js
how do you restart sails js and have this change take effect?

Hi @Gdockery,
Just wanted to let you know we are moving all Sails issues into one repository to help our devs keep a better eye on things.
~Cheers


Confusing English explanation for trend icons

Technical details

App version: 2.13.3 and later versions up to release/2.15.x

Describe the bug
The English language informational text in "Explanation of Statistics" shown above the trend icons "The following trends can be shown:" sounds like these are the only icons which can be shown and this is not true.
The German language text "Beispiele für die angezeigten Trends:" is much clearer that the icons are only examples.

English
German

Steps to reproduce the issue
Using Android, with locale set to English:

Open CWA
On Status screen scroll down to "7-Day Incidence"
Scroll right to "Vaccine Doses Administered"
Note the trend icon
Tap the "i" icon to display information text
Scroll down to the "Trend" section
Note the explanation concerning the trend icons

Expected behavior
Present accurately understandable information for trend icons in English.
For "Vaccine Doses Administered" if the trend is upwards then a green upwards-arrow is shown, however the information text in English seems to contradict that a green upwards-arrow could be shown.
Possible Fix
Translate the German text "Beispiele für die angezeigten Trends:" into English as:
"Examples of trends shown:"
instead of:
"The following trends can be shown:"
This affects the string statistics_explanation_trend_icons_title in Corona-Warn-App\src\main\res\values\strings.xml in Android. The same text is also used in iOS.
Additional context
This is a follow on from the issue https://github.com/corona-warn-app/cwa-documentation/issues/659 "Wrong arrow explanation for vaccination stats" and the https://github.com/corona-warn-app/cwa-documentation/issues/659#issuecomment-889061159.

Internal Tracking ID: EXPOSUREAPP-10831

Hi @MikeMcC399, thanks for your report. We have created an internal ticket and will raise this topic internally.
Internal Tracking ID: EXPOSUREAPP-10831

Corona-Warn-App Open Source Team

This issue was resolved by PR https://github.com/corona-warn-app/cwa-app-android/pull/4497 in Android release v2.15.1 released on Dec 20, 2021.

The suggested text "Examples of displayed trends" is now used.

@MikeMcC399 Thanks.

@MikeMcC399
Would you like me to cross check under iOS?

@Ein-Tim
You are always welcome to check the state on iOS. I will leave the issue open for the moment, although I assume the text was also changed by the translation team on iOS as well.

Text was changed for iOS as seen in repository.


Topics: add Simplicity

At the very least I need to add hashtags (I couldn't find the hashtags to link to specific sections of previous newsletters). There may be other stuff I've missed too.
However, I think we need a Simplicity topics page as it has cropped up in the newsletter a number of times now and so hopefully this is at least a start.

Concept ACK. Thanks!
I'd prefer a more detailed excerpt/detailed_summary, perhaps including information like:

Planned for Elements
Not proposed for Bitcoin directly in the near future (e.g. the next 24 months)
Easier proofs of correctness than Script
Maybe better resource accounting / better management of DoS risks
Native merklized scripting
Risks, e.g. more Ethereum-like losses due to poorly written contracts
Costs, e.g. non-jetted scripts might be more expensive than current scripts
More details about what jets are and why they're useful

In short, text that convinces me that this is worth considering (but without hiding any likely downsides).
@michaelfolkson Let me know if the anchor linking instructions in https://github.com/harding/bitcoinops.github.io/pull/15#issuecomment-668042421 are insufficient to address the links on this PR as well.

Concept ACK as well! I could have used this link last week as well

Thanks for this @michaelfolkson . Will review once you've addressed @harding's review comments.

@michaelfolkson - are you planning to update this PR?

Yes I will work on this after I've finished this month's transcripts section. Thanks for the reminder and apologies for delay.

Pushed an additional commit that addresses most if not all of @harding suggestions and adds the correct links to prior newsletters.

@michaelfolkson - quick reminder that there are some outstanding review comments on this and the build is failing because the intro paragraph is too long. Are you planning to work on this again?

@jnewbery: Yes I'll work on this Thursday and push update then (after Bitcoin Core PR review club) if that is ok.

Pushed latest changes but builds are failing (I haven't built locally yet). Will investigate.

@michaelfolkson

Is the limit for the intro paragraph 500 characters including spaces or excluding spaces? I think it is 500 characters including spaces...

Including spaces (but not the indentation).

Checks are passing now. It didn't like [SIGHASH_ANYPREVOUT][topic SIGHASH_ANYPREVOUT]. I don't know if this was because ANYPREVOUT was upper case or because ANYPREVOUT is an alias of NOINPUT and you need to use SIGHASH_NOINPUT as the topic. Anyway ready to be re-reviewed. I think I have addressed all of @harding comments.

I don't know if this was because ANYPREVOUT was upper case or because ANYPREVOUT is an alias of NOINPUT and you need to use SIGHASH_NOINPUT as the topic.

Yes, you need to use the shortname of the topic page, or the title if there's no shortname. The title in this case is SIGHASH_NOINPUT.

Merged in some more @harding suggested edits. I think this is at final review stage (or at least very close to it).

A reminder that this is ready to be reviewed. Latest @harding edits are merged.

ACK 931c5c1af42268b538b97242e0cabd8de6c262f3
If additional changes are requested by other people, I think maybe the simplicity.pdf link should be moved from the see_also section to the primary_sources section.  I don't think that's particularly important, though, and I feel bad about not re-reviewing this sooner as it is.


Implement a history of commands for command line

Describe the solution you'd like
When a user insert a command, the command is stored. Then when the user press the up or down arrow key, the system shows the latest commands.

Implemented

fixed by https://github.com/ldbc/gcore-spark/commit/0976e9d86aa61192579e8fce59d37d679e35bcdc


Issue working with k8s 1.19

What happened:
Have a k8s cluster setup in a vpc.  Installed the webhook and got an error "2020/11/04 19:11:15 http: TLS handshake error from <IP_ADDRESS>:12501: no serving certificate available for the webhook, is the CSR approved?"  Other service such as external-dns could not get proper permission.  The same setup works with 1.18.8
What you expected to happen:
The csr should be approved and external-dns get proper permission
How to reproduce it (as minimally and precisely as possible):
Set up a k8s cluster in a vpc.  install pod-identity-webhook on it
Anything else we need to know?:
Look like the webhook need an update as It should be setting the signerName since it is now required, but it's not:
https://github.com/kubernetes/client-go/blob/master/util/certificate/certificate_manager.go
Environment:

AWS Region:us-east-1
EKS Platform version (if using EKS, run aws eks describe-cluster --name <name> --query cluster.platformVersion):
Kubernetes version (if using EKS, run aws eks describe-cluster --name <name> --query cluster.version):1.19.0
Webhook Version:latest

Same for me on OpenShift 4.6 (k8s 1.19.0). I forked it,  added SignerName: "kubernetes.io/legacy-unknown", here https://github.com/aws/amazon-eks-pod-identity-webhook/blob/master/pkg/cert/request.go#L69 and built my own image of it to work around the issue until it is fixed.

The issue is fixed by PR #100


Use binding name, env flag instead of namespace id

Existing Design
Currently most kv commands require that you specify namespace id as a positional argument:
$ wrangler kv:key put 9d28bd2bbb2c4e489006d7d8afa61f07 "key" "value"

This is ergonomically awkward and will likely result in a lot of cursing, copy-pasting, and invocations of wrangler kv:namespace list or cat wrangler.toml to find the appropriate namespace id for a given command.
New Design
We can instead use the binding name for a given namespace in a project to provide a human-readable/grokable token for identifying a particular namespace in relation to the project. We can also benefit from environments here by allowing for default values (top level, see environments milestone) and accepting an --env flag:
$ wrangler kv:key put FOO "key" "value" [--env <environment>]

This means that given a toml that looks like this:
name = "worker-site"
type = "webpack"
account_id = "43434f8d0d9f52a395a1472b35f23439"
workers_dot_dev = true
kv-namespaces = [
	{ binding = "FOO", id = "9d28bd2bbb2c4e489006d7d8afa61f07" },
	{ binding = "BAR", id = "f5b309d7ebec48649ec0604facb26c7d" }
]

[env.staging]
kv-namespaces = [
	{ binding = "FOO", id = "c417513a199c4289a1a0c8385c58a0e7" }
]

[env.prod]
kv-namespaces = [
	{ binding = "FOO", id = "1f44a56679a040f0bb01ba39599ba704" }
]

the following command would put to the top level binding named "STATIC_CONTENT" (id: 9d28bd2bbb2c4e489006d7d8afa61f07)
$ wrangler kv:key put FOO "key" "value"

and this would put to the binding named "FOO" under env.staging (id: c417513a199c4289a1a0c8385c58a0e7):
$ wrangler kv:key put FOO "key" "value" --env=staging

Why not use the "title" of the namespace?
In the context of a Wrangler project, a user is more likely to be interacting with the binding they are using in the worker. There's also not a compelling reason to store namespace titles locally, so using title to refer to a namespace in this way would require an API lookup query on every command invocation, as opposed to the project toml parsing Wrangler already does as part of every command.
What if you want to work with a namespace you have not added to a project?
We should retain the namespace id argument as a flag and allow a -- positional argument for the binding name to allow a user to operate on a namespace that is not yet specified in their wrangler project:
$ wrangler kv:key put -- "key" "value" --namespace_id=f5b309d7ebec48649ec0604facb26c7d

Blocked until environments support (https://github.com/cloudflare/wrangler/milestone/12) is complete.

Could I propose an alternative to "What if you want to work with a namespace you have not added to a project"?
Perhaps we can make the syntax clearer/easier by allowing wrangler users to pass in a namespace_id in the first positional argument (much like namespace binding), and allow them to specify an optional -i/--id flag for when they want to use a raw namespace id. e.g.
wrangler kv:namespace rename <namespace_id> <new_name> -i

change positional binding argument to flag --binding OR --namespace-id

Closed by #542.


Update Data Prepper opensearch sink documentation

Description

Adds missing parameters to the OpenSearch sink
Remove outdated parameters from OpenSearch sink
Add example for using actions and template_content

Resolves #4771
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dlvenable: Any changes need for this one before we merge?


Python: Add Semantic-Kernel-Version header to Azure OpenAI requests for python

To help with debugging, the requests to OpenAI should include a Semantic-Kernel-Version header with the value "dotnet/version-number"

@matthewbolanos we have the following available in Python: if telemetry is enabled, we capture the SK version. Is this sufficient?

If we can match the header name, that'll make the queries a bit easier to run. I'm also concerned about name collisions since "version" seems pretty generic.

@matthewbolanos can you check the names and values used in the PR?


fixed warning of resolver = "1"

Running cargo build causes a warning the following warning:
$ cargo build
warning: virtual workspace defaulting to `resolver = "1"` despite one or more workspace members being on edition 2021 which implies `resolver = "2"`
note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest
note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest
note: for more details see https://doc.rust-lang.org/cargo/reference/resolver.html#resolver-versions

I think we should remove the workspace structure which would be a more long-term fix


Fix a bug in drogon_ctl

to resolve #425

I checked out the branch and tested it with a template containing one line of JavaScript that is longer than 8092 characters (about 20000) and it works now.


Cant get custom fmodel build cuz discord link not working

What's your discord username and are you in the discord.

My username is notimportant2004, yes i am

Sorry for the confusion, I didn't know you meant a discord server. I'm not in any server right now, my problem is that I can't access the jsonasasset discord server at all because I get this error when I try join.

https://discord.gg/RmEJuTar

it worked thanks


Decouple geom and attr spec logic away from R, towards generic JSON

[ ] Determine generic JSON specification definition
[ ] Create R functionality to translate
[ ] Create crosswalk between JSON specs and language specific (R, Py, Bash, SQL)
[ ] Update gaiaR functionality

TODO:
Elaboration of SQL approach

Robert Miller (Guest) can you update, use this issue as necessary. I would suggest high level approach, but use that ticket however you'd like to organize around this effort

and similar to above, example complexities for attributes
Variable source:
(one example with three pieces)

["dplyr::filter(staged,Defining Parameter=='Ozone')",
"dplyr::mutate(staged,geom_join_column=paste0(stringr::str_pad(State Code,width=2,pad=0),
3)stringr::str_pad(County Code,width=3,pad=0)),attr_start_date=as.Date(Date),attr_end_date=as.Date(Date),value_as_number=AQI,attr_source_value='Ozone',value_source_value=AQI)"]}

Third item to specify:
Lay out implications of staging source data in a database (postgis) rather than current approach of keeping in memory

approach to temporary tables; (?) add parameter to specify whether source data is persisted or wiped after
pros and cons of creating requirement that source data is already in a database table; dependency on this for translation processes
significance when pulling data from a larger source incrementally, e.g. from APIs, and how to account for that in our staging design

Fourth item:
adding clarity on the "phases" that were mentioned and how the specific functionality falls under each

Ingestion (CLI, ogr, others)
Translation (can we do this comprehensively in SQL?)
Extraction/population of exposure occurrence (arguably out of scope for this conversation)

@rtmill
What are your latest thoughts on this? SQL-only specs?


Could cn.hippo4j:hippo4j-config-spring-boot-starter:1.5.0-SNAPSHOT drop off redundant dependencies?

Hi, I found that cn.hippo4j:hippo4j-config-spring-boot-starter:1.5.0-SNAPSHOT’s pom file introduced 182 dependencies. However, among them, 12 libraries (7% have not been used by your project), the redundant dependencies are listed below.
More seriously, 4  redundant libraries have not been maintained by developers for more than 3 years (outdated dependencies).
1 redundant library has introduced an open source license conflict jakarta.annotation:jakarta.annotation-api:1.3.5. The dependent open source license is EPL 2.0(EPL 2.0 cannot be used by the project with license The Apache Software License, Version 2.0).
Reduce these unused dependencies can help prevent introducing bugs/vulnerabilities from dependencies with outdated and open source license conflict. Meanwhile, it can minimize the project size. To safely remove redundant dependencies, I constructed a complete call graph (resolved most of Java reflection and dynamic binding), and validated that they have not been used by the client code.
This PR cn.hippo4j:hippo4j-config-spring-boot-starter:1.5.0-SNAPSHOT for removing the redundant dependencies have passed the tests.
Best regards
Redundant dependencies
Redundant direct dependencies:
    org.springframework.boot:spring-boot-configuration-processor:2.3.2.RELEASE:compile [113 KB]

Redundant indirect dependencies:
    io.prometheus:simpleclient_common:0.11.0:compile [7 KB]
    com.tencent.polaris:polaris-circuitbreaker-api:1.7.2:compile [2 KB]
    org.apache.yetus:audience-annotations:0.5.0:compile [19 KB]
    org.springframework.security:spring-security-crypto:5.3.3.RELEASE:compile [78 KB]
    com.tencent.polaris:circuitbreaker-common:1.7.2:compile [32 KB]
    jakarta.annotation:jakarta.annotation-api:1.3.5:compile [24 KB]
    org.bouncycastle:bcpkix-jdk15on:1.64:compile [857 KB]
    org.springframework.security:spring-security-rsa:1.0.9.RELEASE:compile [19 KB]
    io.prometheus:simpleclient_httpserver:0.11.0:compile [10 KB]
    com.google.errorprone:error_prone_annotations:2.4.0:compile [13 KB]
    org.checkerframework:checker-qual:3.4.1:compile [207 KB]

Outdated dependencies
org.bouncycastle:bcpkix-jdk15on:1.64 (1278 days without maintenance)
org.apache.yetus:audience-annotations:0.5.0 (2099 days without maintenance)
jakarta.annotation:jakarta.annotation-api:1.3.5 (1349 days without maintenance)
org.springframework.security:spring-security-rsa:1.0.9.RELEASE (1220 days without maintenance)
Open source license conflict dependencies
jakarta.annotation:jakarta.annotation-api:1.3.5

Please resolve conflicts

Please resolve conflicts

Removing redundant dependencies will resolve the conflict


[Event Request] - Codeunit 99000792 "Create Prod. Order from Sale" - procedure OnCreateProductionOrderOnBeforeProdOrderLineModify

🕮 Describe the request
For a large customer project that we need to migrate to the cloud, we need the following event/customization.
Please add the following event

Additional context

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 12849.


Implement reading and writing boolean values

I know nothing in preforming server pings uses boolean values, but this would be super handy to have for handling responses with the forgeData section

forgeData isn't a part of minecraft's official protocol, hence mcstatus doesn't support it. However forgeData seems to just be a part of the status response, as just an additional key. This means that it's already being obtained and is accessible, why would having a bool in the reader/writer classes help?
Mcstatus probably isn't the place to introduce these kinds of changes, as you're not expected to use these classes in your code. These classes are considered internal, and they are not a part of the public API. This means they're only expected to be used for internal functionalities, and if they're not necessary for those, they're not needed.
If you do need this kind of change, either implement it locally within your project, or consider checking out https://github.com/py-mine/mcproto, which already supports this behavior, and where the reader/writer classes are indeed a part of the public API: see (this, and this).

I agree with ItsDrike: before merging this PR, you need to explain why we even need this as forgeData is just a JSON attribute which we should parse without touching the Connection classes.
From this comment, I can see how you use the Connection classes to handle forgeData (permalink to the code). However, you don't use it to handle forgeData! It's used for the mysterious attribute d in the response body (code)! This is undocumented in the protocol docs and with this name it's hard to find any information about what is it. I also thought that you modify the dict before passing to this function, but no. You pass it right after getting the status object.
We must be confident that we will even use the code in this PR, so you need to explain how will you use it. Currently, could you explain please what is d and what value it can have?

I concur with the other reviews too. We need to see examples of how this internal function would be used
@kevinkjt2000

CoolCat467 already sent a link to an example of where this is useful in #556. However our internal API doesn't need it, and we don't expect others to use the Connection class, as it's not a part of the public API. So in my opinion, this only makes sense if we agree to #556, otherwise this PR can be closed.
(Here's the link to that example from the issue: https://github.com/CoolCat467/StatusBot/blob/main/src/StatusBot/decode_mods.py, it seems that forgeData dict holds some data under "d" key, which is a string in JSON, however it essentially just represents binary data, and the buffer/connection reader/writer classes are indeed necessary to read it).
I wouldn't mind adding this, but again, only if we agree to also add a parser for the forgeData, since otherwise this doesn't really serve us any purpose considering it wouldn't be used internally, and we don't expect it to be used externally.

We must be confident that we will even use the code in this PR, so you need to explain how you will use it. Currently, could you explain please what is d and what value it can have?

Basically, the d field in the forgeData part of a ping response from a forge server is a UTF-16 string that represents binary data containing data like the forge mod loader network version, a big list of channels that all the forge mods use, and a list of mods the server has. Apparently forge ran into an issue a while ago with incredibly large modpacks sending a ton of data in those fields, so much that it exceeded the maximum message size the client can handle. To fix this, the d field was added, which basically acts like compressing the data so it doesn't exceed the max message size, and even if it tries to exceed the max message size, the server checks that and truncates the response if it would be too big.
More information is available in the docstrings in this file from forge itself: https://github.com/MinecraftForge/MinecraftForge/blob/42115d37d6a46856e3dc914b54a1ce6d33b9872a/src/main/java/net/minecraftforge/network/ServerStatusPing.java
Basically I need the ability to read boolean values because the compressed response contains boolean values. Then we could make a forgeData parser so to speak, but it wouldn't really be parsing just decoding the compressed response if the server has it. For older versions of forge, it just put all the data in the forgeData field and you didn't have to decompress it, so this wouldn't be needed for older versions.

Yes, I plan to add a forgeData reader, but I would love to hear feedback about where the interface should be.

Yes, I plan to add a forgeData reader, but I would love to hear feedback about where the interface should be.

Inside JavaStatusResponse class, of course.

Or if you are about where to put the parsing logic, I can send you an example of how I imagine this architecture in Discord.


偶尔buffering后就直接finished了。并且Speed很慢。

大多时候下是正常的，有时候会出现以下情况。 buffering没几下就finished了 没法继续播放。

看一下 你的语音啊  是不是 低于 15秒 造成的啊
是有这个问题的

我也遇到这个问题了。

有解决的吗

我也遇到了，有时候播放时正在缓冲突然停止了，就好像那个播放按钮没按过一样

上面的各位解决了吗？同样遇到了

https://github.com/douban/DOUAudioStreamer/pull/137 可以试试这个 fix

@LiweiDong @GaoHe-1 @inix @formeforyouok @DaokunBai @tonycn  可以提供复现的播放源和复现过程么

同问有解决的吗


[enhancement] clear out stages before re-render

As pointed out by @danlester, we should remove some files from the stages directory before re-attempting a qhub deploy (redeploy), as it would be more secure for the user as well as would decrease the chance of terraform complaining or erroring during deployment. e.g.

for example if an existing .tfstate file exists in stage 01 from before changing the project name, qhub deploy will error because it wants to refresh the older version of the state (if that old bucket was deleted).

As per #1010, currently, we have a way to discern which files exist only in the user space (now, untracked), so we can add a filter to remove some target files like the state ones.
Enhancements

Removal of .tfstate files from stages in each re-render
Add ability to the user to opt for removal, maybe a target arg? e.g qhub render --remove [01-terraform-state/<untracked-file>]
Notify user when creating and modifying/removing files (skip it with --disable-prompt as well)

c.c @costrouc @danlester

Closes #1062

We've had two meetings this week that discuss this issue. I'm against deleting anything that is encountered within these directories and instead having a "deprecated filenames/directories" list. We've talked about having a deprication approach. I'll have a PR that addresses this issue and up for review by end of day.

Cool, I will gladly review it


chore(deps): udpate kubewarden SDK

Use latest stable release of the kubewarden SDK. This release fixes an issue introduced by the prior release of the crate.

Once merged I'll tag a new release of the library and bump the dependency inside of policy-server and kwctl. I think both of them should consume the latest version of the KW SDK. This is something that has to be done in time for the 1.7.0-rc2 release

clicked on merge instead of approve, so I went ahead and tagged v0.11.1 in main.


Deploy failed on kubernetes 1.11.4  cause by: some request body already written

What did you do?
I can't deploy latest version kube-prometheus && prometheus-operator by normal way.
i'm using https://github.com/coreos/prometheus-operator/tree/master/contrib/kube-prometheus#customizing-kube-prometheus to deploy my own prometheus-operator.
the configuration file as follow:
local kp =
(import 'kube-prometheus/kube-prometheus.libsonnet') +
(import 'kube-prometheus/kube-prometheus-kubeadm.libsonnet') +
(import 'kube-prometheus/kube-prometheus-node-ports.libsonnet') +
(import 'kube-prometheus/kube-prometheus-static-etcd.libsonnet') + {

  _config+:: {
    namespace: 'monitoring',

    etcd+:: {
      ips: ['<IP_ADDRESS>','<IP_ADDRESS>','<IP_ADDRESS>'],
      clientCA: importstr '/etc/kubernetes/pki/etcd/ca.crt',
      clientKey: importstr '/etc/kubernetes/pki/etcd/peer.key',
      clientCert: importstr '/etc/kubernetes/pki/etcd/peer.crt',
      insecureSkipVerify: true,
    },

    prometheus+:: {
        replicas: 1,
    },

    kubeStateMetrics+:: {
      baseCPU: '200m',
      baseMemory: '200Mi',
      cpuPerNode: '25m',
      memoryPerNode: '50Mi',
    },

    alertmanager+:: {
      name: 'main',
      config: |||
        global:
          resolve_timeout: 5m
          wechat_api_corp_id: '1000002'
          wechat_api_url: 'https://qyapi.weixin.qq.com/cgi-bin/'
          wechat_api_secret: '*****************'
        route:
          receiver: 'wechat'
          group_by: ['alertname', 'job', 'instance']
          group_wait:      30s
          group_interval:  1m
          repeat_interval: 10m
        receivers:
        - name: 'wechat'
          wechat_configs:
          - send_resolved: true
            corp_id: 'ww5af28914ddf94b3a'
            to_party: '14'
            agent_id: '1000002'
            api_secret: '********************'
        inhibit_rules:
          - source_match:
              severity: 'critical'
            target_match:
              severity: 'warning'
            equal: ['alertname']
        templates:
        - '/etc/alertmanager/config/wechat.tmpl'
      |||,
      replicas: 1,
    },

  },
};

{ ['00namespace-' + name + '.json']: kp.kubePrometheus[name] for name in std.objectFields(kp.kubePrometheus) } +
{ ['0prometheus-operator-' + name + '.json']: kp.prometheusOperator[name] for name in std.objectFields(kp.prometheusOperator) } +
{ ['node-exporter-' + name + '.json']: kp.nodeExporter[name] for name in std.objectFields(kp.nodeExporter) } +
{ ['kube-state-metrics-' + name + '.json']: kp.kubeStateMetrics[name] for name in std.objectFields(kp.kubeStateMetrics) } +
{ ['alertmanager-' + name + '.json']: kp.alertmanager[name] for name in std.objectFields(kp.alertmanager) } +
{ ['prometheus-' + name + '.json']: kp.prometheus[name] for name in std.objectFields(kp.prometheus) } +
{ ['grafana-' + name + '.json']: kp.grafana[name] for name in std.objectFields(kp.grafana) }

when i run the command kubectl create -f manifests/ || true
I'll get the following error:
error when creating "manifests/0prometheus-operator-0alertmanagerCustomResourceDefinition.json.yaml": Post https://my-daemon:9443/apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions: unexpected EOF; some request body already written
error when creating "manifests/0prometheus-operator-0prometheusCustomResourceDefinition.json.yaml": Post https://my-daemon:9443/apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions: unexpected EOF; some request body already written
Unable to connect to the server: unexpected EOF; some request body already written
Unable to connect to the server: unexpected EOF; some request body already written
Unable to connect to the server: unexpected EOF; some request body already written
Unable to connect to the server: unexpected EOF; some request body already written
error when creating "manifests/prometheus-clusterRoleBinding.json.yaml": Post https://my-daemon:9443/apis/rbac.authorization.k8s.io/v1/clusterrolebindings: unexpected EOF; some request body already written

I can make the deploy successful by repeatedly executing this command kubectl create -f manifests/ || true and manual create missing yaml. but by this way, the CRD can not managed promethes resource; like stateful-set prometheus k8s or alertmanager.
my lb-server using Haproxy, configuration file as follow:
global
  log <IP_ADDRESS> local0
  user root
  group root
  daemon
  pidfile /root/haproxy.pid

defaults
  mode tcp
  log global
  retries 3
  timeout connect 5s
  timeout client 30s
  timeout server 30s
  timeout check 2s
  no option transparent

listen admin_stats
  mode http
  bind <IP_ADDRESS>:1080
  log global
  stats refresh 30s
  stats uri     /haproxy-status
  stats realm   Haproxy\ Statistics
  stats auth    ****
  stats admin if TRUE

frontend dnc-k8s
  bind <IP_ADDRESS>:6443
  mode tcp
  option tcplog
  log global
  default_backend dnc-k8s-proxy

backend dnc-k8s-proxy
  mode tcp
  balance roundrobin
  server <IP_ADDRESS> <IP_ADDRESS>:6443 weight 1 maxconn 1000
  server <IP_ADDRESS> <IP_ADDRESS>:6443 weight 1 maxconn 1000
  server <IP_ADDRESS> <IP_ADDRESS>:6443 weight 1 maxconn 1000

can u help me to resolve this?
What did you expect to see?
deploy prometheus-operator successfuly by kubectl create -f manifests/ || true
Environment

Prometheus Operator version:
Prometheus-operator version is v0.25.0

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.4", GitCommit:"bf9a868e8ea3d3a8fa53cbb22f566771b3f8068b", GitTreeState:"clean", BuildDate:"2018-10-25T19:17:06Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.4", GitCommit:"bf9a868e8ea3d3a8fa53cbb22f566771b3f8068b", GitTreeState:"clean", BuildDate:"2018-10-25T19:06:30Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:
install by kubeadm

0prometheus-operator-0prometheusruleCustomResourceDefinition.json.yaml
0prometheus-operator-0servicemonitorCustomResourceDefinition.json.yaml

I can create above yaml by kubectl create -f file-name.yaml
but when i create the CRD of prometheus and alertmanager i'll get error like this
error when creating "manifests/0prometheus-operator-0alertmanagerCustomResourceDefinition.json.yaml": Post https://my-daemon:9443/apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions: unexpected EOF; some request body already written
error when creating "manifests/0prometheus-operator-0prometheusCustomResourceDefinition.json.yaml": Post https://my-daemon:9443/apis/apiextensions.k8s.io/v1beta1/customresourcedefinitions: unexpected EOF; some request body already written

I have no idea how to deal with this problem pls help me to resolve this......  :(

@boxuan666 that looks interesting, but I think this is rather something to report upstream, it doesn't look like this has anything to do with the prometheus-operator itself, as you are just trying to apply a manifest.

@brancz i agree with that, but i can't find the any abnormal log or configuration in k8s cluster, did u meet this kind error before?

all right, now i'll close this issue
if i resolve this problem. i'll put the solution here

It is look  like the lb close tcp connecton ,  and the it must be a very long request .
pls check the lb configuration like  request timeout  for upsteam servers  and  clients
here is nginx stream example
server {
        listen 6443;
        proxy_pass dnctmaster;
        proxy_connect_timeout 1s;   # add  1s to 60s an try
        proxy_timeout 3s;  # add  3s to 60s an try
}

@jagjag this work for me!


start does not wait for the container

start must wait for the container when the runtime is running standalone
RUNTIME="./cc-runtime"
CONTNAME=random
BUNDLE=/home/user/bundle

${RUNTIME} --log runtime.log create --pid-file ${BUNDLE}/pid --bundle ${BUNDLE}  --console $(tty) ${CONTNAME}
${RUNTIME} --log runtime.log start ${CONTNAME}

ref https://github.com/containers/virtcontainers/pull/240


remove helpers or turn them into examples

@pastewka I never used these helpers and they are certainly not functionning (since they call PyCo ...).
I would remove them. Any objections ?

No objections, please remove. I've never used them and don't really know what they do. I think they are remnants from Till's time.


[BUG] v1 migration is meant for ammo but is migrating everything

ammo migrated away from using bonus_damage and bonus_attack so it wouldn't interfere with other bonuses. But the migration isn't checking the item's type, it's simply migrating all usages of those flags.

Fixed in 2.12.3


Add addresses in Winnemucca, NV

Status:

[ ] Research sources and propose best source (and it's data license)
[ ] Review proposed source
[ ] Approve source based on review comments
[ ] Add to OpenAddresses

NOTE: Proposed source notes should include basic column mapping to OpenAddress data model.

contacts listed:http://winnemuccacity.org/Contacts.cfm
emailed:
D. Stephen West .... City Manager/Engineer<EMAIL_ADDRESS>Roger Sutton ............ Public Works Supervisor<EMAIL_ADDRESS>

Justin,
We have no GIS department but we have a City utility master map that is maintained by the Humboldt County Assessor's office on ARCGIS. The locations of our utility facilities shown on the map have not been surveyed, however.
Steve

so that leads us back to here: https://github.com/openaddresses/openaddresses/issues/2449

K, let's close this one.


calling a localhost URL disconnects from OS server

Take for example you are in an election and need to jump quickly to another one. This is a usual case because counting the votes needs time therefore several elections are run parallel.
I thought I could easily change between elections by calling their URL. For example I am in an election that has this URL:
http://localhost:8000/assignments/7/
As I wanted to jump to the next one, I changed in the browser URL field the URL to
http://localhost:8000/assignments/8
and pressed return.
result: I get a pink message at the bottom of the browser that the connection to the server is lost. However, the desired election opened as I wanted, only the projector preview window is now collapsed (is was shown before).
So either the connection was not lost or what I did was a mistake (I would describe it then in the manuals) or it is a but that the state of the preview window is lost.

I also tried to open the currently running elections in separate browser tabs. When doing this I get the same problem.

The projector preview state is not saved, this is correct, but - i think - not important. But that the red message appears is interesting; I'll try to recreate this. But what you have done should work. This is like a reload but just with another assignment to show.

I can't reproduce it - tested also with portable version on Windows VM.

Hmm, I cannot recreate this. Just entering a new URL or using multiple tabs always works. Also with different apps like motions and users in parallel.
@emanuelschuetze May you can reproduce this with the populated db?

I still can reproduce this bug with the latest snapshot. Look at the bottom of the screen during second 4 and 5 of this video:
http://ftp.lyx.de/OpenSlides/bug3105.zip

Reproduced with firefox (chrome does not show it):
This it not a bug. The page reload caused a websocket disconnect and new connect to server. Firefox is not so fast with reload and shows for a short time on the "old" page the red disconnected message. But this is not dramatic for use. Usually, a user does not need to reload or manually change the url.
=> Won't fix

This bug has a milestone. The milestone can be removed because nothing will be done (WONTFIX or is no real bug).


[LIVE-6899] Bugfix - Force user agent in evm family

📝 Description
Fix faking a user-agent during Etherscan-like API requests to prevent triggering DDoS protection rejecting the requests.
❓ Context

Impacted projects: @ledgerhq/live-common
Linked resource(s): https://ledgerhq.atlassian.net/browse/LIVE-6899

✅ Checklist

[ ] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Closed because fix has been provided by the Etherscan team.


Should the zio-telemetry backends absorb the extra dependency?

I have a question about the zio-telemetry backend.
As it stands, it wraps an existing backend that works on Task into an enhanced one that does telemetry but then no longer works with Task (it exposes the extra dependency on Tracing.

The yielded effects are of type RIO[OpenTracing, *]

This means that you cannot use this backend as a drop-in replacement for all your code that needed a SttpClient (i.e. a Task-based backend) and everything has to be refactored to deal with the added dependency.
Would it be possible (or a good idea) to have the wrapped backend absorb the dependency (by taking it in at construction time), so that this gets hidden from users?

This sounds reasonable :) That's how I would do this normally, but maybe the "ZIO-way" is different.

@duxet Thoughts on this? If there are no objections, I can prepare a PR.


tested, changed time to date to align with discord format

@Balugrizzly , as mentioned I tested this out utilizing the #testing channel in discord and confirmed that creations, updates, and aggregate posts are being sent. Is there any further information that can be identified from the bot that is running in production? Or can the image it is running on be rebuilt if that is a potential problem?
I have made a small change anyway from TIME to DATE so that it aligns more closely with the discord formatting that has been implemented.

@Sarahtonein
Further information such as?
Debug information?
Logs?
Stdout output?
Image can be rebuilt but images are deterministic so rebuilding will have the same result if no changes where made
TBH i am unsure what you wish to do here

@Sarahtonein Further information such as? Debug information? Logs? Stdout output?
Image can be rebuilt but images are deterministic so rebuilding will have the same result if no changes where made
TBH i am unsure what you wish to do here

I have added further debug information to try and identify the issue, I noticed that the debug log was getting spammed by the handle_creations function, so I modified this. I am not having problems on my end though it is hard to diagnose further.

Why do you want to merge those into here?
Do you intend to run this in production? And derive something from it? If so could you tell me what?

Why was this closed?
Why is the branch not deleted?


Create Enumerable

Creating Our Own Enumerable
The following tasks are implemented in this branch:

[x] Create a class MyList that has an instance variable @list. ✔
[x] In MyList implement a method #each that yields successive members of @list and uses the MyEnumerable module. ✔
[x] Create a module MyEnumerable that implements the following methods (they should have the same functionality as methods in Enumerable): ✔

#all?
#any?
#filter

[x] Verify our solution: ✔

Hi @Abe1able and @IndieCoderMM,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because you did not follow the GitHub flow, thus creating the PR from the dev branch into the main branch, instead you followed Git-flow rather. Let us kindly fix this
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Avoid duplicate code to add headers

Since all my calls to the API needs to add custom headers, theres an easy way to avoid this?
var endpointClosure = { (target: TestApi) -> Endpoint<TestApi> in
    let endpoint: Endpoint<TestApi> = Endpoint<TestApi>(URL: url(target), sampleResponseClosure: {.NetworkResponse(200, target.sampleData)}, method: target.method, parameters: target.parameters)
    switch target {
    case .Customers(let u, let p):
        let credentialData = "\(u):\(p)".dataUsingEncoding(NSUTF8StringEncoding)!
        let base64Credentials = credentialData.base64EncodedStringWithOptions([])
        return endpoint.endpointByAddingHTTPHeaderFields(["Authorization": "Basic \(base64Credentials)"])
    case .Properties(let u, let p):
        let credentialData = "\(u):\(p)".dataUsingEncoding(NSUTF8StringEncoding)!
        let base64Credentials = credentialData.base64EncodedStringWithOptions([])
        return endpoint.endpointByAddingHTTPHeaderFields(["Authorization": "Basic \(base64Credentials)"])
    }
   case .Other(let user, password):
      //repeat the same... :-1:
}

In this case, you can take advantage from "Pattern Matching" in Swift:
switch target {
    case (let u, let p):
        let credentialData = "\(u):\(p)".dataUsingEncoding(NSUTF8StringEncoding)!
        let base64Credentials = credentialData.base64EncodedStringWithOptions([])
        return endpoint.endpointByAddingHTTPHeaderFields(["Authorization": "Basic \(base64Credentials)"])
    }

The code above is kind of "Wildcard pattern matching", should work for all cases which has the same signature. :wink:

Ok...I also run my tests this time, it won't work. I removed my answer

For now i can leave like this: :-1:
 switch target {
case .Customers(let u, let p):
    return endPointWithAuthHeader(u, p: p, endpoint: endpoint)
case .Properties(let u, let p):
    return endPointWithAuthHeader(u, p: p, endpoint: endpoint)
case .Login(let u, let p):
    return endPointWithAuthHeader(u, p: p, endpoint: endpoint)
}
}

func endPointWithAuthHeader(u: String, p:String, endpoint: Endpoint<NextVisitApi>) -> Endpoint<NextVisitApi> {
    let credentialData = "\(u):\(p)".dataUsingEncoding(NSUTF8StringEncoding)!
    let base64Credentials = credentialData.base64EncodedStringWithOptions([])
    return endpoint.endpointByAddingHTTPHeaderFields(["Authorization": "Basic \(base64Credentials)"])
}

If you need to put the same user and password for any kind of endpoint, why not just add auth header directly without switch cases ?

How can i add it without retreiving the username/password?

When i call the endPoint i pass a username/password stored in disk. Then i add those to the header. I dont want to retreive those directly from here. I have a  'user repository' where i store them.

OK, so if the username and password apply to every endpoint, you have a few options. The easiest is to store the username and password in a globally-accessible place. Then your closure would look like:
var endpointClosure = { (target: TestApi) -> Endpoint<TestApi> in
    let credentialData = "\(globalUserName):\(globalPassword)".dataUsingEncoding(NSUTF8StringEncoding)!
    let base64Credentials = credentialData.base64EncodedStringWithOptions([])
    let endpoint = Endpoint<TestApi>(URL: url(target), sampleResponseClosure: {.NetworkResponse(200, target.sampleData)}, method: target.method, parameters: target.parameters)
        .endpointByAddingHTTPHeaderFields(["Authorization": "Basic \(base64Credentials)"])

    switch target {
    // TODO: Customize any parameters on an as-needs basis.
    default:
      return endpoint
    }
}

There are other ways to do this, like using composition to encapsulate the username/password inside an object that also holds onto the Moya provider for you. We do something similar here, but for other purposes. You could also store the credentials in another type. There are lots of different options to do this well, but I'd first focus on getting it to work, and then make it awesome after :tada:
Does that help?

Ok thx you!


Version C header

Add the rust crate version information to the C headers

@t-nelson How about this?

@CriesofCarrots That's correct, they be updated to the latest rust package version during the header generation


Add erb linter

Also, you might want to enable some more Ruby flags for your linter here, like -w, etc. Have a look at the existing Ruby linter.

@w0rp thanks for the review, everything is working on https://github.com/w0rp/ale/pull/497/commits/b2522184110612869a1234134127a2e2efdee76c.

Yep, looks good to me. Cheers! :beers:


Update dependencies for security fixes

This updates the github.com/gogo/protobuf and replaces github.com/dgrijalva/jwt-go with github.com/golang-jwt/jwt/v4 as the former is deprecated.

/lgtm


NOT node not correctly evaluated in !x.Flag == false Linq translation

Note: query gets incorrectly translated, instead of throwing NotSupportedException.

Note: query gets incorrectly translated, instead of throwing NotSupportedException.

Fixed by #850


Deprecate WINEPREFIX

Start deprecating "WINEPREFIX" environment variable in favor of "PROTONPREFIX"

Added warning on commit 66e3f749b2daf4b064d83b803faf7fb156994853 on dev branch,
WINEPREFIX wont be removed / deprecated but the warning will be permanent. PROTONPREFIX is preferrable


fix: update google cloud dependencies that do not depend on rsa

What ❔
This PR updates the dependencies of google-cloud-storage and google-cloud-auth. The changes are as follows:

From google-cloud-storage = "0.12.0" to google-cloud-storage = "0.15.0"
From google-cloud-auth = "0.11.0" to google-cloud-auth = "0.13.0"

Relevant google-cloud changes: https://github.com/yoshidan/google-cloud-rust/pull/217
Why ❔

The primary reason for these updates is to address a security vulnerability associated with the rsa crate, as indicated by a recent cargo-deny check. The vulnerability (Marvin Attack, RUSTSEC-2023-0071) was detected in rsa v0.6.1, which is a dependency of google-cloud-storage v0.12.0. By updating to google-cloud-storage v0.15.0, we eliminate the use of the rsa crate, as the newer version of google-cloud-storage does not depend on it. Similarly, google-cloud-auth is updated for compatibility.
Cargo deny output:
error[vulnerability]: Marvin Attack: potential key recovery through timing sidechannels
    ┌─ /Users/dustinbrickwood/Documents/dev/dut/forks/foundry-zksync/Cargo.lock:759:1
    │
759 │ rsa 0.6.1 registry+https://github.com/rust-lang/crates.io-index
    │ --------------------------------------------------------------- security vulnerability detected
    │
    = ID: RUSTSEC-2023-0071
    = Advisory: https://rustsec.org/advisories/RUSTSEC-2023-0071
    = ### Impact
      Due to a non-constant-time implementation, information about the private key is leaked through timing information which is observable over the network. An attacker may be able to use that information to recover the key.

      ### Patches
      No patch is yet available, however work is underway to migrate to a fully constant-time implementation.

      ### Workarounds
      The only currently available workaround is to avoid using the `rsa` crate in settings where attackers are able to observe timing information, e.g. local use on a non-compromised computer is fine.

      ### References
      This vulnerability was discovered as part of the "[Marvin Attack]", which revealed several implementations of RSA including OpenSSL had not properly mitigated timing sidechannel attacks.

[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.
[x] Spellcheck has been run via cargo spellcheck --cfg=./spellcheck/era.cfg --code 1.

Don't forget to remove corresponding deny https://github.com/matter-labs/zksync-era/blob/ec5907b70ff7d868a05b685a1641d96dc4fa9d69/deny.toml#L11


Add Poetry

It would be nice to have the list of dependencies in this project in a Poetry's pyproject.toml file

Yeah I'm currently experimenting with mambaforge environments in another repo of ours. Condaforge has a kaldi build so that might kill two birds with one stone.


Mod Builder is unable to run game from common retail install directory

Mod Builder crashes when trying to run generals.exe within C:\Program Files (x86)\EA Games\Command & Conquer The First Decade\Command & Conquer(tm) Generals Zero Hour

Workaround: Put game into simple directory, for example C:\Generals\

Main issue is registry key. In The First Decade, registry value looks like so:
Computer\HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\Electronic Arts\EA Games\Command and Conquer Generals Zero Hour:InstallPath = C:\Program Files (x86)\EA Games\Command & Conquer The First Decade\
This is not the correct path to the game. The correct path is in different key:
Computer\HKEY_LOCAL_MACHINE\SOFTWARE\WOW6432Node\Electronic Arts\EA Games\Command and Conquer The First Decade:zh_folder = C:\Program Files (x86)\EA Games\Command & Conquer The First Decade\Command & Conquer(tm) Generals Zero Hour


Variable "type_" name consistency

In the code for handling logs, the "type" is shadowed as type_. That seems better than tipe, so this makes them consistent.
Please consider discussing complicated changes in Slack (join) or via email before submitting a PR
New Feature or UX change?

Sign up for the Elm slack at http://elmlang.herokuapp.com and join the #ellie channel
Alternatively, send an email to<EMAIL_ADDRESS>Have a conversation about your idea!
New features and changes to Ellie's UX are considered thoroughly and carefully. An unannounced pull request is not the correct way to propose them! Ellie is managed in the same spirit as Elm itself. You can find out more in this talk

Bug Fix?

[ ] Reference the issue your PR fixes
[ ] If your PR fixes an unreported issue, please open an issue first so that maintainers can have a chance to understand whether a PR is needed or if a deeper problem exists
[ ] If your fix includes a lot of changed lines or files then a maintainer may ask for a short explanation of why the changes your made are the best way to fix the problem

Thank you for contributing by fixing a problem with Ellie!
Performance Improvement?

[ ] Please include performance measurements if you can!

Thank you for contributing by making Ellie faster!

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Although doing a little checking, it seems like maybe tipe is the preferred. In any case, Subscriptions.elm has type_, so something should change to be consistent.

@coreyhaines can you sign the cla?

Signed!

Oops! Looks like I ended up deleting this file while doing the PureScript rewrite.


Allow more lines on scenario examples

I know I can set the allowed length of examples with:
RSpec/ExampleLength:
  Max: 19

Would it be possible to implement a specific limit to scenario examples (those examples used in feature specs)? For performance reason, especially when testing JavaScript, I tend to allow more than one example per scenario. Something like this:
RSpec/ExampleLength:
  Max: 5
RSpec/ScenarioLength:
  Max: 20

I'd be open to something like that (ideally not a totally new cop). I don't use scenarios so I probably wont implement that in my free time but if you want to propose a specific implementation solution I'd be fine with accepting a PR for this

If one would like to implement this, where would you start?

This issue also concerns RSpec/MultipleExpectations.

This is actually an issue for many of our cops (and I haven't been sure how to improve rubocop-rspec to fit the various use cases) so how do you feel about something like this:
---
AllCops:
  RSpec:
    Patterns:
    - _spec.rb
    - "(?:^|/)spec/"
    SpecTypes:
      Feature: 'spec/features/**/*'
      Unit: 'spec/unit/**/*'
      Scenario: 'spec/scenario/**/*'

# ...

RSpec/ExampleLength:
  Description: Checks for long examples.
  Enabled: true
  Max: 5
  SpecType:
    Feature:
      Max: 10
    Scenario:
      Max: 10

The idea here is that you could specify any arbitrary mapping of a spec type name to a file glob in the AllCops/RSpec/SpecTypes section. Then, in a cop that supports spec type overrides, you could use those spec type names you defined to define settings for just that part of your application. Seems like this would solve a lot more problems than adding new slightly different cops would.
@ehannes @bquorning @andyw8 thoughts on this approach?

As I recall it, rspec-rails will tag your specs with a type tag, e.g. type: :feature, type: :controller, type: :view etc. I wonder if these types could be used for the mapping you’re looking for.
I realize that not all people use RSpec together with Rails, so perhaps this is a bad suggestion.

@bquorning Good point. It's true that rspec-rails uses types in specs (see rspec-rails Directory Structure). It would be sad if this this gem was dependent on rspec-rails, so the question is if you can use this feature in rspec as well. I have not tried yet, but I found that you can add type: :special to specific examples in rspec (see [rspec --tag option] (http://www.relishapp.com/rspec/rspec-core/v/3-5/docs/command-line/tag-option)).

You can add metadata based on directory structure using config.define_derived_metadata: http://www.rubydoc.info/gems/rspec-core/RSpec%2FCore%2FConfiguration%3Adefine_derived_metadata
I guess the question is whether we want to force part of the configuration into people’s spec_helper.rb instead of their .rubocop.yml.

I tried the following:
# spec/support/integration_helper.rb
module IntegrationHelper
  # Includes things that integration tests need
end

# spec/integrations/test_class_spec.rb
describe TestClass, type: :integration do
  # Some examples that need IntegrationHelper
end

# spec/spec_helper.rb
RSpec.configure do |config|
# ...
  config.include IntegrationHelper, type: :integration
# ...
end

And it works like a charm.

I would rather add type: :whatever as I please to my specs than be forced to set that specific configuration in spec_helper.rb. Also, some users may use the type metadata for something else and forcing them to set the type according to the directory structure may break things for them.

If you use type, I guess you would get a .rubocop.xml file that looks something like this:
...
    SpecTypes:
      Feature: :feature
      Unit: :unit
      Whatever: :whatever

I'd rather just define the globs. It would be extremely hard to statically determine the types for different specs.

@backus With globs, do you mean the approach you proposed at first, or the approach @bquorning suggested (using the spec_helper.rb)?

@ehannes the explicit globs in the config file. IMO we couldn't reliably know what the different spec types are without loading the user's rspec environment and this gem is not in the business of dynamic analysis

I found a new feature in RSpec 3.3 today: aggregate_failures. This clearly requires a change to RSpec/MultipleExpectations since the purpose of it is to allow more than one expectation per example.

Yeah I added #233 for that reason

Should we move this issue from "Discussion" to "Enhancement"? "Explicit globs" seems like the way to go.

Sure!

I, too, would love this. Feature/Integration specs tend to be much longer so it forces you to either disable the :cop: altogether or bump up the Max setting, which eliminates any value from it.


#valueOfTokenNamed: is crappy

it is based on a string.
we should kill it.
and use message instead.
example in material design:
md.sys.color.shadow
this could be implemented with:
md sys color shadow
thus, getting the value of a token is simply done through message sending.
Now we need also a kind of infrastructure to specify the object hierarchies.
As an example, Style dictionary would be a good approach
see https://github.com/amzn/style-dictionary

@Nyan11 👀

We can also use slots for this and I would avoid chaining md sys color shadow
if this is just names.
The first things would be to hide the use of valueOfTokenNamed: from the user and get "normal" variables.

In Style Dictionary not all pathes are values.
For example:
color-background-red can be equal in Pharo to ^Color red.
But,
color-background do not return anything

Therefor the chaining of message is complicated because certain chains will return nothing

The current style sheet rules are like this:
self
	when: ToPressedLookEvent
	write: (self property: #'background-color')
	with: [ :e | e valueOfTokenNamed: #'color-primary-pressed' ]
	animation: nil.

I suggest to do something like this:
self
	when: ToPressedLookEvent
	write: self properties backgroundColor
	with: [ :e | e tokens colorPrimaryPressed ]
	animation: nil.

We can also use slots for this and I would avoid chaining md sys color shadow if this is just names. The first things would be to hide the use of valueOfTokenNamed: from the user and get "normal" variables.

I like chaining of message sends instead of flatten names because it matches a token organization.
to be discussed :)

The current style sheet rules are like this:
self
	when: ToPressedLookEvent
	write: (self property: #'background-color')
	with: [ :e | e valueOfTokenNamed: #'color-primary-pressed' ]
	animation: nil.

I suggest to do something like this:
self
	when: ToPressedLookEvent
	write: self properties backgroundColor
	with: [ :e | e tokens colorPrimaryPressed ]
	animation: nil.

self
when: ToPressedLookEvent
write: self properties backgroundColor
with: [ :e | e palette primaryPressed ]
animation: nil.
but I have to dig more

could you talk with marcus because I would like him to think about this?

Hello, did something happens about this?

I think not (:

No but it would be great to use slot to abstract this string base API.
@tinchodias could you ping marcus?


Generalize RobotWrapper.position and RobotWrapper.framePosition

Most of the demo/exercizes are writted assuming the "feet names" represent joints in the pinocchio model (see eg. https://github.com/stack-of-tasks/tsid/blob/master/demo/demo_romeo.py#L89, also the name in this script are misleading rf_frame_name is the name of a joint in the pinocchio model). However it is not always true, like for Talos where the feet are represented by frames.
This may cause confusion or create hard to find issue when using this scripts and changing robot model because the call to robot.position(data, robot.model().getJointId(ee_name)) still execute without error even if ee_name is the name of a frame, but in this case it return garbage.
I had to add the following method in my code to be able to deal with the fact that ee_name could be a joint or a frame:
def get_current_effector_position(robot, data, ee_name):
    id = robot.model().getJointId(ee_name)
    if id < len(data.oMi):
        return robot.position(data, id)
    else:
        id = robot.model().getFrameId(ee_name)
        return robot.framePosition(data, id)

I do not know what is your policy for checking user inputs when calling RobotWrapper methods, but I think that RobotWrapper.position() should check that the id is valid. (Either with an assert, warning message or throw an error as you prefer).

Agreed. Actually that's a bug that has been propagated through the scripts. My policy is to always use frames and never use joints, because anyway there's a frame associated to each joint, so there's no reason for not using frames. The piece of code you reported should actually be:
robot.position(data, robot.model().getFrameId(ee_name))

I'm gonna fix this in all scripts.
I also agree that some index checking in robot wrapper would be appropriate. I'll try and add them.

I added the asserts in 96f0c8f95fc3ac1ee946d3ad581bb515d12b9678


ForestDB RegularQuery api throwing CouchbaseLiteException intermittently

Hi all,
We're using couchbase-lite-android version 1.4-12 & couchbase-lite-android-forestdb version 1.4.-12 and we're getting the following error from crashalytics.  (Crashalytics apparantly didn't catch the exception message).  We also caught this while using 1.3.1-90.  I'm unable to catch a repro of this myself manually unfortunately.
Caused by com.couchbase.lite.CouchbaseLiteException
       at com.couchbase.lite.store.ForestDBViewStore.regularQuery(ForestDBViewStore.java:443)
       at com.couchbase.lite.View.query(View.java:358)
       at com.couchbase.lite.Database.queryViewNamed(Database.java:2457)
       at com.couchbase.lite.Query.run(Query.java:446)

@enicolasToast
Thank you for reporting the issue.
To identify the issue, we need more detailed information. Does crashalytics provide the detailed info?
Thanks!

@hideki It does not since we wrap the exception in a generic exception of our own.  I can change the logic to also log the internal couchbaselitexception message to crashalytics as well.  Would that be enough information for further investigation ?
Note: Since we can't reproduce this too easily, we would have to deploy this to our customers and wait for a bit (maybe a week) before we get data.

@enicolasToast Thank you for your support!

Would that be enough information for further investigation?

I can not clearly say yes or no.  Further information could help to investigate the issue.


No logo

Hi, I have made a logo that combines bus and the clock. Here it is:

Please provide the location at which I should put the logo.

Just put it at root thanks! If possible add it to the top of the README after the main header


@asyncapi/bundler vs asyncapi-bundler

Reason/Context
All other libraries all publish under AsyncAPI published under @asyncapi/xxx however this repository does not 🤔 Is there a specific reason for it?

@Souvikns is checking if some final fixes need to be done before we publish, this is why it is not there yet

I have changed the name to @asyncapi/bundler will update the documentation accordingly.
https://github.com/asyncapi/bundler/blob/649eae45b53690d39d61d54c024be1055561f5ef/package.json#L2

@asyncapi/bundler 0.1.0 is out in the wild
closing...