Multiple improvements

I reformatted the logger to give consistent logging and added levels for successful downloads and failures.

Those changes include a new format and new colours, green for successes and blue for debug/info

Failed downloads now get automatically deleted
Download progress can be monitored live because successes and failures are written in a separate file
The started download threads are now returned and joined after all of them were started, when they are all done the successes and failures will be written again, but sorted
Added a shell script to rerun the program until there are no failures (this will work because failed downloads will no longer be kept)

Thank you for the PR.
I tested it on Windows and everything works from what i can see.


Add build_tag to set_build_info_metric

Currently, we return only revision of a service in libmetrics_build_info:
https://github.com/neondatabase/neon/blob/ffd146c3e519ace5f14b619ca63f00ca9dda4b56/libs/metrics/src/lib.rs#L92-L100
See a grafana dashboard (with deployed revisions):
https://neonprod.grafana.net/goto/EQUiO5iSg?orgId=1
To make it more useful, let's add a build_tag to the metric, then it'll easier to find a corresponding release in https://github.com/neondatabase/neon/releases
Required changes:

Add BUILD_TAG env var support to all services
Add  build_tag to libmetrics_build_info

For Rust at least, I think the BUILD_TAG should be baked into the binary, not supplied via env var to the process.

I think the BUILD_TAG should be baked into the binary, not supplied via env var to the process.

It is baked into the binary.
Env var is used to supply the version on the build stage.

I would like to pick this task!
@bayandin
Can you review the draft PR #5576?
Thanks in advance!

@bayandin
Can you take some time to review the PR #5576?


Missing CSRF token in comments template

mezzanine/generic/templates/generic/includes/comment_form.html
Line 3 should be:
<form method="post" id="contactform" action="{{ comment_url }}#comment">{% csrf_token %}
From Mezzanine 4.1.0

That template doesn't even exist.
The one that renders the form uses fields_for tag, whose corresponding template provides csrf_token.


doc: fix links for custom scalar page

TODO:

[ ] If this PR is a new feature, reference an issue where a consensus about the design was reached (not necessary for small changes)
[ ] Make sure all of the significant new logic is covered by tests
[ ] Rebase your changes on master so that they can be merged easily
[ ] Make sure all tests and linter rules pass
[ ] Update CHANGELOG.md with your change

Thanks!


not working on opera mini

I create a simple page with list bind with API call
its working with all mobile browsers but not in opera mini
$http({  method: 'GET',  url: APIRequest.getListData(), timeout:6000})
.success(function(response) {
//code
}).error(function(){
//code
});

We don't officially support Opera Mini, so this is expected behavior.
Since I'm curious, what's the error message for the app? ( stacktrace)


Aerial Fishing Spot Tiles Occasionally Showing Incorrectly - Fishing Plugin

Type
Incorrect behavior
Operating System
Windows
OS Version
Win10
Bug description
Occasionally when aerial fishing, RL will incorrectly highlight fishing spots that aren't there any more. They start to build up after a long session and can be distracting. This might be caused by tabbing to another app then back to RL. I'm not sure.
If a real fishing spot spawns on top of a falsely marked one, it doesn't automatically clear it. Teleporting away doesn't clear them either. Logging out and back in again doesn't seem to clear them consistently, but seems to work sometimes.
Coming back to type this-- I just did a little experiment. I uninstalled ver 2.4.0 (which I was testing my fps with at ToA) and reinstalled 2.5.0. I must have been aerial fishing on 2.5.0 before because I came back to see different falsely marked fishing spots. I believe they were saved from another session, but idr when I was last there.
I also just noticed that logging out for a few minutes cleared 1 of the mis-marked spots. So weird.
The attached image is from a session using 2.4.0. In my 2.5.0 session the mis-marked spots are all different, none overlapping.
Screenshots or videos

RuneLite version
1.9.3
32 bit 2.4.0
and
1.9.3
32 bit 2.5.0
Logs
No response

Duplicate of #7702


Will squash it out when merging.


Us018 add variable dates data

This PR is associated with US018 for loading variable event dates.
Use this to load in the variable data for the PR on Response operations UI.
Changed the loading of CSV's so that all the details are loaded from one file rather than multiple.
Added and changed details in the CSV to reflect the CSV spreadsheets provided by the business.

@benjefferies i've fixed the integration tests and got them all passing, could you give this another go?

 US018: INT Load variable event dates for a collection exercise [M]

@benjefferies This PR works in conjunction with Emilio's other PR in the integration tests repo. It'll work with them


[QUESTION] How to display an error with the max constraint?

Hello!
Zod has a max constraint that can make sure the string does not have more than the expected number or characters.
When using SuperForms, this constraint is turned into a maxlength attribute.
The problem is that the field just refuses to let you enter more characters, which is not SuperForms' fault, obviously. But using this attribute does not give the user any information as to why he cannot input more characters, because no error message given to the Zod schema is displayed.
Is there any way to workaround this situation and make sure the user can input a wrong value, obtain a helpful message and correct his input (as with the other constraints)?

Hi, unless it works with the customValidity option on the field, I think you must disable the constraint and set a custom error message with setCustomValidity. The $errors store could be helpful there, when displaying a friendly error message.

Thanks for the quick answer! The customValidity option does the trick indeed of not setting the maxlength, but the problem is that I would need to wire the Zod checks by myself, which would clutter my code. I love using Superforms precisely to make all boilerplaty non-business code disappear 😅
I'll see what I can do! Thanks for pointing me in the right direction, and thanks again for bringing us SuperForms 👏

Thank you for your kind words. :) Do you really need to handle the Zod checks yourself? If you're using the validators option, it should be pretty much automatic.

After reading your first answer, I thought that I would need to call the validate method of the Zod schema for each field myself, and use the result on my own to display it. But I was wrong (hopefully!).
As you suggested after, I just have to keep the validators option (that I was already using) and remove the {...$constraints} that I applied to the <input>. By doing that, my fields are correctly validated, the error messages do appear correctly and the user can type what he wants in the field and receive some feedback as to what is wrong 👍 This is exactly what I wanted, thanks for posting your second message 🙏
The only difference is that the <input> no longer uses the HTML attributes. I can live with that. because on top of my initial problem I gain something great: the possibility to have multiple regexps!
That was initially an issue for me, as I wanted the user to have correct feedback to guide him during the input. Just saying "your password is incorrect because it does not as the correct format" is not very friendly. I prefered being able to tell him "Missing a digit, missing a special character...".
I initially browsed the issues here and saw that the topic had already been discussed. I then settled with only one regexp in my Zod schema, because anyway only one would appear in the pattern attribute.
But now that I don't use this attribute anymore, I can have multiple regexps (and bonus: they are really short and simple), each one with its own error message (just tested it and it's great!).
So, once again, thank you very much 😁


No such file or directory: 'requirements.txt'

windows 本地环境安装 pip install torch==1.10.1+cpu torchvision==0.11.2+cpu torchaudio==0.10.1 -f https://download.pytorch.org/whl/cpu/torch_stable.html
之后执行  pip install --use-deprecated=legacy-resolver  -r requirements.txt -f https://download.pytorch.org/whl/cu113/torch_stable.html 报错  No such file or directory: 'requirements.txt'
这种报错如何处理

Hello @LillyChen,
thanks for using TabLLM!
The requirements.txt file is provided in the t-few project. Please be sure to first change into the t-few project directory before executing the pip install ... command.
I hope that helps!


🪲[BUG] Pico-PIO-USB does not build with pico-sdk version 2.0

Describe the bug you encountered:
Perform the following steps

Install the pico-sdk version 2.0.
Run the get_deps.py script to install the Pico-PIO-USB library in TinyUSB
In the Pico-PIO-USB directory, run git checkout -b main origin/main to get version 0.6.0
Attempt to build this program
Build failed.

usb_rx.pio.h[Ln 176, Col 29]:redefinition of 'pio_sm_set_jmp_pin'
pio.h[Ln 1450, Col 20]: previous definition of 'pio_sm_set_jmp_pin' was here

...
What did you expect to happen instead?
Build should succeed
...
What actually happened in your case?
Build failed.
...
Additional information
There are 3 issues in Pico-PIO-USB that I identified:

The file usb_rx.pio.h is auto-generated. It should not be checked in.
The file usb_tx.pio.h is auto-generated. It should not be checked in.
The 2.0 version of pico-sdk defined its own version of pio_sm_set_jmp_pin.

Deleting the files in 1 and 2 above and removing the function definition of pio_sm_set_jmp_pin from file usb_rx.pio fixes the problem.
Other information: The TinyUSB library in pico-sdk version 2.0 includes an older version
of Pico-PIO-USB instead of 0.6.0.

Solved by #126


[Subtask] Unify PostgreSQL container management with ContainerSuite

Describe the subtask
Subtask of #2765
Parent issue
#2765

I'd like to work on this, would you assign this to me?

@xiaozcy, someone was already working on this. Is there something else you would like to work on?

@xiaozcy, someone was already working on this. Is there something else you would like to work on?

@justinmclean, got it. I took a cursory look at the PR and it's exactly the same way as I thought before.


[WIP] Feature/messaging/local disk start processor

Fixes
Issue #14
Description
Provide an implementation of Task IBlobStorage.StartProcessorAsync(string channelName, IMessageProcessor messageProcessor) for LocalDiskMessenger

There are some areas that still need needs to be improved
I'll add comments on this PR

Thanks a lot!


10.1 is not support by invenio-cli

Is your feature request related to a problem? Please describe.
The latest version 10.1 is not supported.
invenio-cli init rdm -c v10.1

I got:
The v10.1 branch of repository https://github.com/inveniosoftware/cookiecutter-invenio-rdm.git could not found, have you made a typo?

The branch has been created: https://github.com/inveniosoftware/cookiecutter-invenio-rdm/commits/v10.1

Deployed locally, tested common user flows: search and record creation with files.


A4TECH.Bloody7 version V2022.0805

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

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

Microsoft Reviewers: Open in CodeFlow

Duplicate of #89606

Close with reason: See above;


I can't load favorite

I'm use Exhentai
When loading favorite I can see the error 'The World is Big and the panda site alone'
This has not happened until a few days ago.
I logged in again using cookie but this was not resolved.
This works by changing to e-hentai, but does not work again by changing to exhentai.
Changing to e-hentai works, but changing it to exhentai does not work again
logcat-2018-02-13-20-24-22-470.txt

Please make sure, both in e-hentai and exhentai, Settings->EH->EHentai Settings->Front Page Settings is "List View".

Thank you for your help


[wlm] Fixes retrieve config.json error (#2212)

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

@frankfliu we will do a rebase to 0.29.0


Feature/hg sprint1

Added use case file directory "use cases"

What is this PR for?

Rejecting, no obvious purpose for this pr


Add menu indicator for mode button state

mode button toggles between "activate" and "enable" behaviours for the step buttons.

Done


IGAPP-560: Send signal if page is closed

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
This is definitely not an optimal solution but I did not find an official react-navigation way to do this and it is relatively simple.
Other ways I could think of:

Add a hoc to all route containers and send on unmount
Somehow add react-navigation focus listener to all screens and send if page is focused again: https://reactnavigation.org/docs/function-after-focusing-screen/#triggering-an-action-with-a-focus-event-listener
Not sure how we could exclude app state changes then though...

What do you thin about this? This does not handle navigation from city content to landing and vice versa, i.e. all navigating that either removes more than one screen at once or just replaces the current screen. Not sure if I should include it.

With short research i found it would be possible to do screen tracking with
* use onStateChange in the NavigationContainer [here](https://reactnavigation.org/docs/screen-tracking/) -> mostly what you have done, and i think it is the best solution, maybe modify and use routeName instead of index, if this works also and/or better

* or use a redux middleware for screen tracking [here](https://reactnavigation.org/docs/2.x/screen-tracking/#screen-tracking-with-redux)

I guess you thought about it, but just wanted to drop it.
Your other suggestions:
1. Add a hoc to all route containers and send on unmount: Think this could be a nice solution too, would also handle app closes properly i guess?

2. Somehow add react-navigation focus listener to all screens and send if page is focused again: This would not handle if the app is closed, but it would be possible to handle this seperatly i guess. Do not like this option though, because we do not really want to listen on "new focues".

Also the Offers-Page is not working properly.
Open Offers -> GoBack ('close' Signal is sent)
Open Offers -> Click on e.g. Sprungbrett ('close' signal is not sent)

@SteffiStuffel  So I think we are talking about little different things here. Our current tracking approach is to send two signals for user navigation:

OPEN_PAGE if the user navigates to a NEW route (e.g. by clicking on a category or the events or offers navigation item
CLOSE_PAGE if the user navigates BACK, i.e. closing a route that was opened previously
This differs from the approach in the docu as there is just one signal sent if the currently opened screen changes. This is e.g. visible in the example you described when opening the sprungbrett. No screen is closed but the offers screen just goes to the background, therefore no close signal is sent.

Maybe (or even probably) the approach from the docs (screen tracking) would make more sense, but I think it is very difficult to implement (how do we get the url of the new page?) and conflicts with the agreement with JPAL.

We can discuss this in a call as well if you want :)

No need. I just unterstood it wrong (cause i do not really get the usecase of this, but none of my buisness ;) ) and if this is the intened behaviour, i'm perfectly fine with your solution.

We can discuss this in a call as well if you want :)

No need. I just unterstood it wrong (cause i do not really get the usecase of this, but none of my buisness ;) ) and if this is the intened behaviour, i'm perfectly fine with your solution.
Just one other thing i noticed: Open langugage change does not send a "Open-Page" signal, but when navigating back a "close-page" signal is sent. (If this is also intended, just start ignoring me smile )

Not intended, good catch. Will fix it. And don't get me wrong, I think your proposed solution would normally be the better approach. But our architecture is sometimes a bit complicated and kind of prevents it.


fix(Quiz) Don't highlight stroke when setting "showHintAfterMisses" to false

The docs mention in multiple places that setting "showHintAfterMisses" to false will disable hint stroke highlighting. This isn't the case and instead highlights after every mistake. This commit fixes the issue.

Really good catch! Could you add a test for this too?

All good, I've added the test

:tada: This PR is included in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Refactor

/cc @opscode-cookbooks/release-engineers

@yzl @schisamo Can either of you rebase this so I can merge it in?

@tas50 I will rebase and retest with Chef 12.


Command for updating information

Added a command for updating person's information

Oh yeah I didn't even notice that wildcard import!
I also totally agree that it would be more efficient if the remove operation could be avoided but I discovered too late that the Person is implementing a read-only interface. I didn't want to start removing this aspect so this was the only solution that I could come up with.
Thank you for your review!


Remove 'Scroll to Bottom' from import as service does not support it

As per https://github.com/adobe/spacecat-api-service/pull/487#discussion_r1752547565, the option is not used.
Expected Behaviour
Only have the "Scroll to Bottom" checkbox if the service uses it.
Actual Behaviour
Apparently, the input is ignored, so do not present it to the user.
Reproduce Scenario (including but not limited to)
Run imports with it on and off - any differences?
Steps to Reproduce
Platform and Version
Sample Code that illustrates the problem
Logs taken while reproducing problem

The flag is still mentioned here (main code), so hesitating to pull it out too quickly.

Looks like both scrollToBottom and customHeaders are not (yet?) supported by the service:
https://github.com/adobe/spacecat-api-service/pull/494

Looks like both scrollToBottom and customHeaders are not (yet? Jira) supported by the service: adobe/spacecat-api-service#494

customHeaders is no longer included in options object but as its own object instead which is already supported by the Bulk Import UI AFAIK.

@atopper The only checkbox that should be exposed is enableJavascript then?

Since 1 checkbox in 'options' looks weird, I suggest moving 'timeout' to the Options as well (technically they make up the 'options' object):


repair uwsgi error

Hi everyone,
this PR modifies the error of the wsgi.py file, which may lead to a failure to run.because migrating the settings.py dict.
The first time to PR :)

Hi @xsseng congrats on the first PR! Merged.


Default display-name's acceptTranspilerName to true

This was a change that has been queued for v4.
Addresses #436

Merged #440 instead.


Support for particle system scaling/sizing in world space

Allows particle system billboards to be sized in world space via sizeInMeters property that was added and cascaded to billboards during update/creation process.

Thanks for the pull request @richard3d!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Thanks for the PR @richard3d!  We'll try to look at this in the next week or so

Yikes! I just pushed a change to fix the merge conflict with CONTRIBUTORS.md can someone advise why my latest push caused AppVeyor to fail? I would really like to get this PR in :)

Thanks again for your contribution @richard3d!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @richard3d!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @richard3d!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

@richard3d the AppVeyor failure may have been a fluke. I would try resolving the merge conflict, merge in master and it'll re-run the CI.

Thanks again for your contribution @richard3d!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

@lilleyse can you review this?

@richard3d does Merijn Wijngaard also work at Prominent Edge and is he covered by your CLA? Could you add Merijn and yourself to the Corporate CLA section?

I pushed a few small tweaks in https://github.com/AnalyticalGraphicsInc/cesium/pull/7746/commits/e1907350369e6692440a8406224ae83bea0bf9af.
I'm noticing some strange behavior in one of the Sandcastle examples that uses particle bursts. sizeInMeters seems to be exposing some other bug where the particle bursts fill the whole screen. Local sandcastle

@richard3d does Merijn Wijngaard also work at Prominent Edge and is he covered by your CLA? Could you add Merijn and yourself to the Corporate CLA section?

Never mind, I think Merijn was accidentally added twice to CONTRIBUTORS.md. I've made the correction.

Thanks again for your contribution @richard3d!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @richard3d!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @richard3d!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

@lilleyse @criyson opened the same change as this in https://github.com/CesiumGS/cesium/pull/8603.  Do you think we can get this one merged, or is the weirdness you saw when testing a showstopper?

Do you think we can get this one merged, or is the weirdness you saw when testing a showstopper?

Whatever this was, I don't see it anymore. My theory is it was one of those bugs where hidden points/billboard would either take up the whole screen or stay in the center of the screen when hidden.

I updated CHANGES.md https://github.com/CesiumGS/cesium/pull/7746/commits/53400d843e6a43e488b1af7513dbf67e3678aa2e. Good to merge whenever CI passes.

Thanks @lilleyse!
And thanks @richard3d and @criyson!


Tools in Lite Mode

Don't render images in Lite Mode.
https://lite.crimethinc.com/tools

[ ] hero image


Get an error while launching

Getting an error after typing 'python autopilot.py'
Traceback (most recent call last):   File "autopilot.py", line 1, in <module>     from dev_tray import tray   File "C:\Users\fouso\Desktop\EDAutopilot-master\EDAutopilot\dev_tray.py", line 3, in <module>     from dev_autopilot import autopilot, resource_path, get_bindings, clear_input   File "C:\Users\fouso\Desktop\EDAutopilot-master\EDAutopilot\dev_autopilot.py", line 293, in <module>     keys = get_bindings(PATH_KEYBINDINGS)   File "C:\Users\fouso\Desktop\EDAutopilot-master\EDAutopilot\dev_autopilot.py", line 280, in get_bindings     binding['key'] = SCANCODE[binding['pre_key']] KeyError: 'DIK_▒'

Hey there,
You seem to missing necessary keybinds for the program to work.
You must go to your controls and make sure you have set primary keyboard
keys for all of the following:

In 'Flight Rotation':

Yaw Left
Yaw Right
Roll Left
Roll Right
Pitch Up
Pitch Down

In 'Flight Throttle':

Set Speed To 0%
Set Speed To 100%

In 'Flight Miscellaneous'

Toggle Frameshift Drive

In 'Mode Switches':

UI Focus

In 'Interface Mode':

UI Panel Up
UI Panel Down
UI Panel Left
UI Panel Right
UI Panel Select
UI Back
Next Panel Tab

In 'Headlook Mode':

Reset Headlook

On Sat, 4 May 2019 at 11:20, fousol<EMAIL_ADDRESS>wrote:

Getting an error after typing 'python autopilot.py'
Traceback (most recent call last): File "autopilot.py", line 1, in
 from dev_tray import tray File
"C:\Users\fouso\Desktop\EDAutopilot-master\EDAutopilot\dev_tray.py", line
3, in  from dev_autopilot import autopilot, resource_path,
get_bindings, clear_input File
"C:\Users\fouso\Desktop\EDAutopilot-master\EDAutopilot\dev_autopilot.py",
line 293, in  keys = get_bindings(PATH_KEYBINDINGS) File
"C:\Users\fouso\Desktop\EDAutopilot-master\EDAutopilot\dev_autopilot.py",
line 280, in get_bindings binding['key'] = SCANCODE[binding['pre_key']]
KeyError: 'DIK_▒'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/skai2/EDAutopilot/issues/14, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA765HIJMILCEQ2S5E6MCWLPTWLRPANCNFSM4HKY7Y3A
.

Also unbind the shift key to someting else, as that gave me an error even after setting up the keybinds

@aradalienzzzz that is odd, the shift key should work just fine. What did you bind shift to? If you could send me your autopilot.log it would also help.

I unbind all numpad keys and it succesfully launched.

Roll Left
Roll Right
Pitch Up
Pitch Down


refactor: drop classvar in error

drop non class var in error.py

:tada: This issue has been resolved in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


List all shopcart under a user

As a [role]
I need [function]
So that [benefit]
Details and Assumptions

[document what you know]

Acceptance Criteria
Given [some context]
When [certain action is taken]
Then [the outcome of action is observed]

Only one shopcart per user so we don't need this one anymore.


Use preevy GH plugin to notify users on new environment

Preevy GH plugin post a comment on the PR with details of the preview environment.

done


Make the Mod count easter egg show "nice" at 420

Make the Mod count show "..nice" when the count is and 420, not only 69.

base on 1.17 and we'll talk


Add Kansas LTC scraper

New Tableau scraper for KS LTC data (ks_ltc_scraper.py)
Reorganize LTC data, moving scraped data files into a new ltc_states directory and organzing scraper scripts by platform

Sorry for the large diff, which is all from reorganizing everything. The new bit is ks_ltc_scraper.py.

TableauScraper doesn't always work, for reasons I never figured out because it involved going way too deep into the depths of Tableau web's internals. But when it does, it's pretty slick.

TableauScraper doesn't always work, for reasons I never figured out because it involved going way too deep into the depths of Tableau web's internals. But when it does, it's pretty slick.


Fixing go {lint,vet} warnings

I have tried to rebase the changes on latest master. hope that went fine.


FakeQuant Node

I use TF quantization-aware training, and the fakequant nodes will be added to the model.
Therefore, the converted TFLite model contains FakeQuant node. Does the Arm NN support the FakeQuant operation?
Because the error show the following messages

Hi @ivs-pychen
ArmNN supports Fake Quantization but the TfLiteParser does not support it.
If you want to try to add support for Fake Quantization to the TfLiteParser we do welcome contributions, for more information on contributing to Arm NN please see the Contributing page on the MLPlatform.org website, or see the Contributor Guide.
Best regards,
Mike

OK, Thank you.


Question: How to properly use google.api.httpBody with gRPC-JSON transcoder to accept arbitrary content?

I am trying to accept arbitrary data via a post request, but I am unable to progress
relevant envoy config:
name: envoy.filters.http.grpc_json_transcoder
typed_config:
  '@type': type.googleapis.com/envoy.extensions.filters.http.grpc_json_transcoder.v3.GrpcJsonTranscoder
  proto_descriptor: '/data/descriptor_set.pb'
  services:
    ['HealthCheck', 'AuthService', 'TrelloSlackBot']
  auto_mapping: false
  request_validation_options:
    reject_unknown_method: false
    reject_unknown_query_parameters: false
   print_options:
     add_whitespace: true
     always_print_primitive_fields: true
     always_print_enums_as_ints: false
     preserve_proto_field_names: false

service definition:
service TrelloSlackBot {
  rpc Help(HelpRequest) returns (String) {
    option (google.api.http) = {
      post: "/v1/trelloslackbot-help",
      body: "raw_body"
    };
  }
}

message HelpRequest {
  google.api.HttpBody raw_body = 1;
}

I have tried altering the service definition in various ways (e.g. using google.api.HttpBody as the message input and specifying "data" as the body mapping target, not specifying a mapping target etc). The above configuration results in 500 internal server error when posting anything to that URL.
[optional Relevant Links:]
The documentation describes this briefly and based on this comment / example and this pull request 10110 I think I am doing this correctly but I cannot get it to work. I am unable to find any other reference for how to proceed and I'm not sure what debugging steps I could take so any help is greatly appreciated.

it seems to be related to this longstanding issue: #16629

We actually had to work around this by adding a subdomain for option(http) services.
Separate filter chains let us provide separate common_tls_context.alpn_protocols. Our grpc alpns are only "h2" and our option(http) services only advertise 'http/1.1" to work around this. In the fullness of time we'll put h2 back in the option(http) services' alpn list but this at least works around the issue for us.
I don't know if you have the option to vary your hostname or your port but if you can do either of those things you can separate your handlers (and bonus, you can remove the grpc-JSON transcoder from your grpc filter chain since you split the regular-http vs grpc paths)


Parallel environment with multiprocessing.Pipe?

Hi there, thank you so much for sharing the code! I was trying to run parallel environment using pipe as done in: https://github.com/lcswillems/rl-starter-files. However, that doesn't seem to work. I got EOFerror for all process other than the main one. Any idea why and how to fix it? Thanks in advanced!
Here's the relevant code:
class ParallelEnv(gym.Env):
    def __init__(self, envs):
        assert len(envs) >= 1,
        self.envs = envs
        self.observation_space = self.envs[0].observation_space
        self.action_space = self.envs[0].action_space

        self.locals = []
        for env in self.envs[1:]:
            local, remote = Pipe()
            self.locals.append(local)
            p = Process(target=worker, args=(remote, env))
            p.daemon = True
            p.start()
            remote.close()

    def reset(self):
        for local in self.locals:
            local.send(("reset", None))
        results = [self.envs[0].reset()] + [local.recv() for local in self.locals]
        return results

    def step(self, actions):
        for local, action in zip(self.locals, actions[1:]):
            local.send(("step", action))
        obs, reward, done, info = self.envs[0].step(actions[0])
        print(obs.shape)
        if done:
            obs = self.envs[0].reset()
        results = zip(*[(obs, reward, done, info)] + [local.recv() for local in self.locals])
        return results

    def render(self):
        raise NotImplementedError

When opening an issue, the first thing that you should do is always to provide some details about which platform you are on, and what command you used to run the program.
The error could happen for a number of reasons. The EOFError likely means all the other processes crashed, but they aren't able to log an error to the console.
This is one example:
https://github.com/lcswillems/rl-starter-files/issues/48
You might be able to log the output of the crashing processes like this:
https://stackoverflow.com/questions/1501651/log-output-of-multiprocessing-process
Alternatively, you might have better luck with the RL code found in the babyai repository:
https://github.com/mila-iqia/babyai

Thank you for the prompt response. Here's what I did:

git cloned rl-starter-files
pip3 install -r requirements.txt
git clone https://github.com/lcswillems/torch-ac.git
cd torch-ac
pip3 install -e .
cd ..
git clone https://github.com/maximecb/gym-miniworld.git
cd gym-miniworld
pip3 install -e .
cd ..
added import gym_miniworld in envs.py
run PYOPENGL_PLATFORM=egl CUDA_VISIBLE_DEVICES=0 PYTORCH_JIT=0 python -m scripts.train --algo a2c --env MiniWorld-TMaze-v0 --save-interval 100 --frames 10000000 --procs 16 --recurrence 4 --frames-per-proc 4 --log-interval 20 --lr 1e-4 (running on linux through ssh with offscreen rendering)

The error comes from this line: https://github.com/lcswillems/torch-ac/blob/85d0b2b970ab402e3ab289a4b1f94572f9368dad/torch_ac/utils/penv.py#L8. I'm pretty sure it's different from lcswillems/rl-starter-files#48 because I was able to receive output from the main process.

[Update] it seems all other processes crashed at this line: https://github.com/maximecb/gym-miniworld/blob/d640e5bb143e218aa1cb4127f8251b495a34fd9a/gym_miniworld/miniworld.py#L1083

Did it crash inside the resolve method, or when doing the call? Can you print frame_buffer just before the call?

Sorry for the delay. Here's the output of printing frame_buffer:
<gym_miniworld.opengl.FrameBuffer object at 0x7f9e1a425fa0>

It crashed inside the resolve method at this line: https://github.com/maximecb/gym-miniworld/blob/d640e5bb143e218aa1cb4127f8251b495a34fd9a/gym_miniworld/opengl.py#L300-L308

This is probably related to your graphics card or driver. I can't really help you debug it because I have no way to test on your machine. If you're doing training or running on a remote machine, you might have better luck running the code with xvfb: https://github.com/maximecb/gym-miniworld/blob/master/docs/troubleshooting.md#nosuchdisplayexception-cannot-connect-to-none


'package' wrapped so IDEs don't go on keyword red alert

Just changed it to string hash access.
Good that there's a deprecation warning now.

nice.
Thanks!
... can you get actionhero as well?  There's a similar recent change.

> npm publish
+<EMAIL_ADDRESS>


Do not use sles15sp1o

What does this PR change?
This official image is causing problems: we are not able to
deploy when this image is used in the suse clients.

I think we need to change it here too:
https://github.com/uyuni-project/sumaform/blob/8529d14433458bae1e6865e4cebf11f85a009ec8/backend_modules/libvirt/host/user_data.yaml#L142
And review this file! Because mayeb the if statement must contain both images in the comparison

What is the problem exactly?

What is the problem exactly?

We don't exactly know it, Ricardo and us are trying to figure out, you can join the party in #galaxy-qa-review-40 or #galaxy-qam-review :)
The thing is that is failing with this error, when using this image:
13:47:33  Error: Error: couldn't retrieve IP address of domain.Please check following:
13:47:33  1) is the domain running proplerly?
13:47:33  2) has the network interface an IP address?
13:47:33  3) Networking issues on your libvirt setup?
13:47:33   4) is DHCP enabled on this Domain's network?
13:47:33  5) if you use bridge network, the domain should have the pkg qemu-agent installed
13:47:33  IMPORTANT: This error is not a terraform libvirt-provider error, but an error caused by your KVM/libvirt infrastructure configuration/setup
13:47:33   timeout while waiting for state to become 'all-addresses-obtained' (last state: 'waiting-addresses', timeout: 5m0s)
13:47:33
13:47:33    on /home/jenkins/jenkins-build/workspace/manager-4.0-qam-setup-cucumber/results/sumaform/backend_modules/libvirt/host/main.tf line 65, in resource "libvirt_domain" "domain":
13:47:33    65: resource "libvirt_domain" "domain" {

I'd recommend having this merged only as a last resource. Fixing the root cause would be preferable.
In the mean time running testsuites from this branch is feasible as a temporary workaround.

@moio I would strongly prefer to revert the default images, to use the images working, not breaking master.
Did you consider that some external people must not be aware of this issue and our master is broken for them?
It happened in the past, and we started bad moves, creating tags and branches that took too long, @Bischoff will agree on that.
Better to revert all the default images changed on the last commit, then work in a branch to assure that all the new default entries work in all our scenarios, then when all is good and safe we merge to master.
At that moment in QAM environment, sles15sp1o and opensuse150o break the deployment stage.

Did you consider that some external people must not be aware of this issue and our master is broken for them?

I just tried again and, in my setup, deployment of sles15sp1o, opensuse150o and centos6o all work as expected.
I will need your help tomorrow to reproduce the problem reliably/safely and then continue from there.

Did you consider that some external people must not be aware of this issue and our master is broken for them?

I just tried again and, in my setup, deployment of sles15sp1o, opensuse150o and centos6o all work as expected.
I will need your help tomorrow to reproduce the problem reliably/safely and then continue from there.

This is the main.tf I was using before reverting to custom images. Jenkins history already erased the logs shown with this main.tf
@rjmateus is aware about them.
In the controller for instance I had:

[2020-08-14T12:36:00.890Z] module.controller.module.controller.module.host.null_resource.provisioning[0] (remote-exec):      Comment: Desired working directory "/root/spacewalk/testsuite" is not available
SUSEManager-4.0-qam.tf.txt

Did you consider that some external people must not be aware of this issue and our master is broken for them?

I just tried again and, in my setup, deployment of sles15sp1o, opensuse150o and centos6o all work as expected.
I will need your help tomorrow to reproduce the problem reliably/safely and then continue from there.

This is the main.tf I was using before reverting to custom images. Jenkins history already erased the logs shown with this main.tf
@rjmateus is aware about them.
In the controller for instance I had:
[2020-08-14T12:36:00.858Z] module.controller.module.controller.module.host.null_resource.provisioning[0] (remote-exec):      Comment: Swap /extra_swapfile failed to activate. Added new entry to the fstab.
....
[2020-08-14T12:36:00.890Z] module.controller.module.controller.module.host.null_resource.provisioning[0] (remote-exec):      Comment: Desired working directory "/root/spacewalk/testsuite" is not available

Be sure to use those official images in those modules: server, proxy, controller
SUSEManager-4.0-qam.tf.txt

@moio I would strongly prefer to revert the default images, to use the images working, not breaking master.
Did you consider that some external people must not be aware of this issue and our master is broken for them?

SUSE is a open source company.  Sumaform is open source.
There is no reason to hide this issue.  Honesty is in our core values, and make the others trust on us ;-)

SUSE is a open source company. Sumaform is open source.
There is no reason to hide this issue. Honesty is in our core values, and make the others trust on us ;-)

I'd say it's not about hiding. Think about our developers. As I said at the chat, if they start using master and find it's broken, I will be one of the persons they will ping to ask what's going on.
master should just work.
We can make this public and visible with an issue for SLE (if there's an issue there, as it seems), and another one on this repository to use the official images again when SLE is fixed.

master should just work.
We can make this public and visible with an issue for SLE (if there's an issue there, as it seems), and another one on this repository to use the official images again when SLE is fixed.

I agree with your 2 points: I also want master stable, and ping SLE people if there is something to fix there.
concerning this thread, there is no evidence that my changes are affecting test suite deployments (which is what I
have extensively tested).
Please, if someone thinks that these changes prevent sumaform deployment, show evidence, not guesses or feelings.
I have been deploying with this branch the whole last week with no issues.

No worries @calancha, I'll be looking into @srbarrios's specific configuration today and I hope to be able to come out with an exact explanation, then I will report here.

It smells like qemu-agent not installed in cloud init. I agree with @moio this should be a last resort only. I'd vote for closing this PR only if we fail debugging that.

In my understanding this is addressed by #767, can you confirm @Bischoff?

In my understanding this is addressed by #767, can you confirm @Bischoff?

My PR #767 removes the deployment problems when sles15sp1o is used. However, there could be other problems linked to sles15sp1o in secondary features phase. I just don't know, it would need investigation.
To give you a taste of it:

[https://ci.suse.de/view/Manager/view/Manager-4.0/job/manager-4.0-cucumber-PRV/541/#showFailuresLink](4.0 Provo), with "sumaform" images, 4 failures
[https://ci.suse.de/view/Manager/view/Manager-4.0/job/manager-4.0-cucumber-nue/108/#showFailuresLink](4.0 Nuremberg), with "official" images, 41 failures

In the second case, many failures are linked to CentOS or to Ubuntu, but some might be on sles15sp1.

Let's revisit this after Friday then

@calancha we don't need anymore this PR isn't it?

Closing out-of-date PR (we are already using sles15sp1o successfully).


Support nbformat 4.0 (ipython 3.0) #135

nbformat 4.0 is the current ipynb files format for ipython 3.0
We use to support only nbformat 3.0 which belongs to ipython 2.0
Confusing, I know!

Looks good to merge


[#noissue] fix klocwork issue

Codecov Report

Merging #3359 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3359      +/-   ##
==========================================
+ Coverage   40.36%   40.36%   +<.01%
==========================================
  Files        2132     2132
  Lines       70018    70027       +9
  Branches     9682     9684       +2
==========================================
+ Hits        28260    28266       +6
- Misses      39039    39042       +3
  Partials     2719     2719

Impacted Files
Coverage Δ

...s/dbcp/interceptor/DataSourceCloseInterceptor.java
0% <0%> (ø)
:arrow_up:

.../interceptor/DataSourceConstructorInterceptor.java
0% <0%> (ø)
:arrow_up:

.../dbcp2/interceptor/DataSourceCloseInterceptor.java
0% <0%> (ø)
:arrow_up:

.../interceptor/DataSourceConstructorInterceptor.java
0% <0%> (ø)
:arrow_up:

...karicp/interceptor/DataSourceCloseInterceptor.java
0% <0%> (ø)
:arrow_up:

...tor/metric/datasource/DefaultDataSourceMetric.java
75.75% <0%> (-3.04%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

...m/navercorp/pinpoint/test/OrderedSpanRecorder.java
72.11% <0%> (-1.93%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
72.5% <0%> (-1.67%)
:arrow_down:

...inpoint/profiler/sender/AsyncQueueingExecutor.java
76.08% <0%> (-1.09%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ec48f4...17c4c2c. Read the comment docs.


Replace mailtrap by maildev

In order to avoid rate limits of mailtrap, we should replace mailtrap by any nodejs based mail (ie maildev)
It should be included by cp-local so that the production stack is not impacted

@butlerx complexity-wise, could cp-local spin up a maildev server ?

that's relatively simple adding a mail server container wouldn't take much.
Would we want it to actually send an email or just to kill all mail?

sorry i misunderstood. adding maildev should be super quick as it already has a container.
so in the docker-compose we can just add this and modify all the mailtrap references in the configs to maildev container

@heyitsmaimai handled that without much issues :) https://github.com/CoderDojo/cp-local-development/commit/caa8eb64e8ea46e1e67174daac386df6cd2d047b


Updated Application Auto Scaling pass role policy

Small updates to the Application Auto Scaling permissions configuration in the stand-alone tutorial based on updates in the controller ReadMe (https://github.com/aws-controllers-k8s/sagemaker-controller/pull/130/commits/194bee023a1eb4af2af6a5ad175bf26e255df7ef)
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Save fit parameters to a file

It would be useful to be able to serialize the results from the fits so that we can use them afterwards. I should work out what the simplest way of doing this is.

I can use ASDF for this, which is like a cross between YAML and FITS.

Docs of asdf python package
Also, see asdf-astropy
And the documentation of the ASDF format

So it turns out that ASDF cannot be used out of the box, since it uses yaml.safe_dump under the hood, which does not work on arbitrary python objects. I would have to write a special Converter for it, which looks like quite a lot of work.
Therefore, I am looking into other options:

Just use pickle on the MinimizerResult object. This may or may not work. And it would mean that the save file can only be read by the exact same python version that was used to create it.
JSON. This might be better, since the Parameters class (which is the most important component of MinimizerResult) has .dumps() and .loads() methods that convert to/from json representations.
Alternatively (or additionally) we could just save the XYConic object instead of the lmfit result. This might be enough for Vanessa's purposes.

I have now decided not to try and serialize the lmfit result, but just to save the parameters and their uncertainties

That seems to work at least partially. For example
params:
  eccentricity: 1.0830661105211852
  r0: 7.5053539040889685
  theta0:<PHONE_NUMBER>6800825
  x0: -0.32146936319570624
  y0: -1.3853933155243527
uparams:
  eccentricity: 0.05258668094648568
  r0: 0.5690418171170513
  theta0: 1.4801671974934552
  x0: 0.5755166597405751
  y0: 0.5344204040883483

But I think I need to save the dat points as well

Fixed in b0650bf42018c8bf2a693fafe371a279d7d2c8d0


[Bug] Padding/Margin issues with containers

Description
If containers are next to each other, they overlap when published.

If scrolling is disabled within the container, they don't overlap

Important Details

Version: [Cloud /]
OS: [MacOSX]
Browser [MS Edge]

@Nikhil-Nandagopal This is not happening anymore.
I just created an application with a container similar to the screenshot posted above. - https://app.appsmith.com/applications/5ff452b30d17d27a05365ac3/pages/5ff452b30d17d27a05365ac5

@Nikhil-Nandagopal This is not happening anymore.
I just created an application with a container similar to the screenshot posted above. - https://app.appsmith.com/applications/5ff452b30d17d27a05365ac3/pages/5ff452b30d17d27a05365ac5


Use exec to replace the shell

I'm not a huge fan of replacing the shell by modifying .bashrc. I don't think this is really having a huge impact- the biggest one is that exit (or ctrl+d, etc.) will behave unexpectedly if that is done. Using exec would help, although I'm not sure that's a great solution either.

Hey Alex, I would agree with you add that one of the easiest ways to bork a machine is to set the default shell in a bashrc file. Ansible, scp, rsync, and other tools just won't work anymore against said machine.
Might we consider:

To persist a change to the default shell set it in /etc/passwd which in this case lives in https://github.com/rht-labs/enablement-framework/blob/main/codereadyworkspaces/stack/Dockerfile#L142
To temporarily change the shell by just type zsh, bash, fish, etc...
Add a "don't try this at home" disclaimer to the existing step

the reality is a bit more complicated as well ..
CRW - injects sh as the default when you open the shell regardless of whats set in /etc/passwd
the whole zsh vs bash - is basically ME (bash) and @springdo (zsh) - people who like different shells ;) which i think is ok and healthy since it covers 95% of what people use today on the linux shell generally.
i dont think exec really helps because of the "sh" thing - i certainly don't have any issue with ctrl-d
a better "fix" might be to figure out CRW default shell setting (i have not found the magic yet), and have IT launch it. WDYT ?

Hah, actually... Whichever method that CRW uses to create the terminal... .bashrc is not sourced at startup :-p
Doing some actual testing, using exec does change things (ctrl+d exits zsh and brings me back to bash if I don't use exec, while it exits the terminal if I use exec)... but that's a moot point.
Closing this, continuing this discussion in chat.


feature request: opacity slider for crosshatch fills

I sometimes find myself wanting to be able to create a crosshatched box that sits on top of an arrow, but without the arrow being seen inside the box itself: this can be done by creating a background-filled box with no border and then another box on top as usual, but it is a bit cumbersome.
I was wondering if it would be possible to add this as a feature (say a separate checkbox/slider for the "fill background" opacity) which behind the scenes can do the same (create a no-border box the color of the background with configurable opacity sitting behind the box the user is creating) allowing non-solid boxes to hide other objects/text/... if so desired

I find myself doing the same hacks from time to time, but I'm not sure adding a separate UI for this (and increasing complexity) is is warranted.
But maybe we can come up with some power-user workflows. Here are some ideas:

Support HEX,HEX in the input, so you can pick a background color, then type ,fff at the end (→ 4c6ef5,fff), and it would give you this:

double-click (or cmd/ctrl click...) the hachure/cross-hatch button to force it into non-transparent background:

This will only allow fully non-transparent, white background, but I think that's fine.

Support (2), but actually expose it as a checkbox in the color picker menu:

So effectively what you suggest. I think a single on/off might be ok. We will be soon redesigning the color popup, so we may consider this at the same time.

thanks for the ideas, of the above to be honest if you think this is too niche of a feature maybe (1) would be good for the time being to have less UI work? It also would allow people to be even more colorful having crosshatches of one color against a different color background.
If you instead thought this was a feature that would be good to expose then having a color picker for the box background and a slider for the background opacity would give the most flexibility, but of course that might be effort you'd prefer to invest in other areas so whatever you think works best.
On the other hand, maybe a more general feature would be that when you group shapes and you click on the group, you could see/change all the individual component shapes colors/transparencies (say the left sidebar would have all the individual shapes sidebars separated by a line, and a scroll bar in case there are many)?
This way one could group the two boxes (or even additional boxes/shapes) and yet still be able to influence their individual colors, seems like this might be less ad-hoc and possibly more useful for shape libraries but of course even more work to do...


Crashing due to tank things

Idk know the name of these thigns but they are like ultimate liquid tanks but they crash my game when I'm just in the Mekanism tab scrolling through. I am very irate as the current version makes the Project Ozone modpack pretty damn easy but I cannot do anything if I search for "Mekanism" in the NEI tab and it just crashes because of the tank thingies rendering.  My PC is a fairly decent non-clunky potatoe pc so I do not know why this would crash the game).

Set PrefilledPortableTanks to false


SCTP support capabilities

Hi,
Does Pion SCTP support direct SCTP (RFC 4960) or does it only do SCTP over UDP?
Thanks

You are right. Pion/sctp only implements a subset of RFC 4960 required for WebRTC, not designed for the standard / kernel space SCTP.


Add Prometheus Operator

https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack

https://picluster.ricsanfre.com/docs/prometheus/

Configured and running!


partial indexing

Is there a way to tell it to only index a single folder ?
When i get up to 100,000 pages its slow and so instead i would like to tell pagefind to index a folder when i know that new html is in that folder.

👋 Hey @gedw99
There isn't just yet, but I am hoping to get to that very soon. The existing issue for this is at #71 with the latest update here: https://github.com/CloudCannon/pagefind/issues/71#issuecomment-1401301921
Until that work is done, what I have seen people do is use Pagefind's multi-site mode to index a site in a couple of sections, and then search both in the browser. It does come out to a little more bandwidth to load two indexes, but it can help cut down build speed.
I'll close this issue out but feel free to drop any comments on the main issue for this 🙂


Add .fsti extension to auto-mode-alist

It's missing

What's the full list of extensions? fst, fsi, fsti?

Just fst and fsti (fsi is F#)

Done, thanks!

fsi is also a valid extension for F* interfaces.

Maybe we should rename them then ...

Why.

Because that extension is already taken by a much more popular tool, which can be very confusing.

For instance have a look at this: https://github.com/github/linguist/pull/2590

I am personally in favour of using .fsti for files which contains F* code and .fsi for things which is F# or common syntactic subset.

Is there any reason to open files in the common subset as F* files in Emacs, though? Wouldn't we want to leave these to fsharp-mode?

Is there any reason to open files in the common subset as F* files in Emacs, though?  Wouldn't we want to leave these to fsharp-mode?

If they are F# is there any reason not to open them as F#? :)

@catalin-hritcu I agree, but I'm not an F* programmer, so I can't really say :) As a package developer, I'd rather reduce the overlap with other packages; users should feel free to add the file association themselves. I'll add a note in the README.


refactor: use community keccak

What Does This PR Do
Update black version for pre-commit to avoid version error.
Delete self-written implementation of keccak_256 hashing and replace it with pysha3 community package.
How Is The Changes Tested
Generated random bytestring of length n using secrets.byte_tokens(n) and asserted old and new implementation hashes. Tested with different n [0; 500], 200 times for each. New version works ~150 times faster.
Notes: Older version had wrong behaviour when hashing bytestring of length >135.

This is the re-open version of PR https://github.com/virtualeconomy/py-vsys/pull/158 to merge into branch develop


Reviewed No.9 - What happens inside the pipeline function? (TensorFlow)

To #390
Modified!
No.9 - What happens inside the pipeline function? (TensorFlow)
According to No.8

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Only run spell check and link check if changes are relevant

@BrianMMcClain This is one idea I had to speed up certain builds - if there are not changes to content or theme (configuration-only basically), then we should not need to run spell or link check. Let me know what you think...

FYI I think because we haven't merged this to main yet, it's still expecting status for the two checks?


CUPS w/ Samsung ML printer "filter failed" / "Broken pipe" error

I installed CUPS on my x86 router box, which runs IPFire, and connected my old Samsung ML-2510 printer to it by USB. However after successfully setting it up by following the IPFire CUPS guide, I'm unable to even get the test page to print.
I had made this post asking for help at the IPFire community forum, but this appears to be out of their realm of expertise.
So far, I have tried using both the v1.9 PPD along with its bundled rastertospl filter from HP, and the v2.0.0 PPD and its bundled rastertoqpdl filter from OpenPrinting, both without success. Additionally, I'd also tried various generic drivers (IPP Everywhere Printer, PCL Laser Printer and PDF Printer) that came along with CUPS itself, and even though when using these latter generic drivers will show the jobs as "completed at DATETIME" in the CUPS web UI job queue, and no errors are logged to /var/log/cups/error_log, they in fact still don't print the test page requested.
Here are some info from lpstat:
[root@ipfire ~]# lpstat -t
scheduler is running
no system default destination
device for Samsung_Printer: usb://Samsung/ML-2510%20Series?serial=3V61BKCQA03376Y.
Samsung_Printer accepting requests since Sat 30 Jan 2021 06:31:36 PM EST
printer Samsung_Printer is idle.  enabled since Sat 30 Jan 2021 06:31:36 PM EST
	Rendering completed
Samsung_Printer-15      root              1024   Sat 30 Jan 2021 06:31:28 PM EST

And:
[root@ipfire ~]# lpstat -l
Samsung_Printer-15      root              1024   Sat 30 Jan 2021 06:31:28 PM EST
	Status: Filter failed
	Alerts: job-completed-with-errors
	queued for Samsung_Printer

I've enabled --debug-logging as suggested in error_log, and extracted out the relevant log lines for one cycle of failed printing, which I've attached here: cups_job10_error_log.txt. I'm hoping you guys would be able to help me out here.

hello @jyscao
can you please provide us the ppd file you are using. It must be your /var/ipfire/cups/ppd/Samsung_Printer file.
Thanks :-)

@surajkulriya thanks for following up.
Here are the 2 Samsung specific PPDs I'd tried:

v1.9 from HP
v2.0.0 from OpenPrinting.

And here's one generic PPD I tried.


[Bug] Dropzone not working

Bug report
What I did
Added dropzone as crud field
      CRUD::field([
        'label' => 'Image Gallery',
        'name' => 'image_gallery',
        'type' => 'dropzone',
        'withFiles' => array(
          'disk' => 's3',
          'path' => $storagePath
        ),
        'tab' => 'General'
      ]);

What I expected to happen
??
What happened
On the FE I get [object Object] error, and also this error
2

#2 upload (ajax) (21:06:34)
8.3.10
20.61ms
2MB
POST admin/product/dropzone/upload
No query results for model [App\Models\Product] 0
vendor/laravel/framework/src/Illuminate/Database/Eloquent/Builder.php#506
Illuminate\Database\Eloquent\ModelNotFoundException
503
504
505
506
507
508
509
510
        }

        if (is_null($result)) {
            throw (new ModelNotFoundException)->setModel(
                get_class($this->model), $id
            );
        }

What I've already tried to fix it
use dropzone locally without s3 button, if the images are already uploaded, dropzone shows the previewed images.
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there?
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
8.3.10

### PHP EXTENSIONS:
Core, date, libxml, openssl, pcre, sqlite3, zlib, bcmath, bz2, calendar, ctype, curl, dba, dom, hash, FFI, fileinfo, filter, ftp, gd, gettext, gmp, json, iconv, intl, SPL, ldap, mbstring, session, standard, odbc, pcntl, exif, mysqlnd, PDO, pdo_dblib, pdo_mysql, PDO_ODBC, pdo_pgsql, pdo_sqlite, pgsql, Phar, posix, pspell, random, readline, Reflection, mysqli, shmop, SimpleXML, soap, sockets, sodium, sysvmsg, sysvsem, sysvshm, tidy, tokenizer, xml, xmlreader, xmlwriter, xsl, zip, Zend OPcache

### LARAVEL VERSION:
<IP_ADDRESS>

### BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.3.5
backpack/crud: 6.7.37
backpack/devtools: 3.1.5
backpack/generators: v4.0.5
backpack/pro: 2.2.17
backpack/theme-tabler: 1.2.11```

Hey @tomjrjones
Do you mean it works on a local disk but not on s3?

@karandatwani92 It works on neither, get the above error when uploading to local disk or s3


Search is not working on Bootstrap 4.3.1 documentation

Search is not working on Bootstrap 4.3.1 documentation
Windows 10
Chrome 81
Anything i searches it does says "ABC" is not found etc.
https://getbootstrap.com/docs/4.3/components/modal/

https://github.com/algolia/docsearch-configs/issues/1951#issuecomment-628585979

Should be fixed now.


notebook build errors -- unexpected operator

While building, the log shows repeated
"/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator"

The error looks like it might be an expecting-bash-but-ran-in-sh problem:

http://stackoverflow.com/questions/3411048/unexpected-operator-in-shell-programming
http://stackoverflow.com/questions/2011160/unexpected-operator-error

Docker build log:
Step 12 : RUN Rscript -e 'install.packages("mallet", repos="http://cran.rstudio.com/")'
 ---> Running in e8c819c09034
/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
trying URL 'http://cran.rstudio.com/src/contrib/mallet_1.0.tar.gz'
Content type 'application/x-gzip' length 4460706 bytes (4.3 MB)
==================================================
downloaded 4.3 MB

/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
* installing *source* package ‘mallet’ ...
** package ‘mallet’ successfully unpacked and MD5 sums checked
** R
** inst
** preparing package for lazy loading
** help
*** installing help indices
** building package indices
** testing if installed package can be loaded
/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
* DONE (mallet)

The downloaded source packages are in
	‘/tmp/Rtmp8dDq5e/downloaded_packages’
Updating HTML index of packages in '.Library'
Making 'packages.html' ... done
 ---> 83e37ef1de7b
Removing intermediate container e8c819c09034
Step 13 : RUN Rscript -e 'devtools::install_github("agoldst/dfrtopics", quiet = TRUE)'
 ---> Running in 81731fc33920
/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
Installing 10 packages: DBI, dplyr, lattice, Matrix, R6, Rcpp, RcppEigen, stringi, stringr, tibble
Updating HTML index of packages in '.Library'
Making 'packages.html' ... done
Warning messages:
1: In utils::install.packages(pkgs, repos = repos, type = type, dependencies = dependencies,  :
  installation of package ‘Rcpp’ had non-zero exit status
2: In utils::install.packages(pkgs, repos = repos, type = type, dependencies = dependencies,  :
  installation of package ‘stringi’ had non-zero exit status
 ---> 3a2475da200c
Removing intermediate container 81731fc33920
Successfully built 3a2475da200c

https://github.com/jupyter/docker-stacks/issues/210
https://github.com/ContinuumIO/anaconda-issues/issues/1063

Two resolutions:

datascience-notebook:54552d36cf0a works for me. I can plot again!

And:

With irkernel 0.7 and conda r-base 3.3.1-3, I can crash the kernel by simply entering x <- 1 in a new notebook. The error on the console is the same as what @poplav stated above.
Switching back to r-base 3.3.1-1 resolves the issue. It seems that conda 3.3.1-3 build is the culprit again.

Our RUN Rscript -e lines may be failing due to Rscript exec happening through sh rather than bash. Still not sure why it worked on all our earlier builds and then just stopped working -- perhaps jupyter, perhaps anaconda.
We might try mapping the command to bash using the RUN command form:
RUN ["/bin/bash", "-c", "echo hello"]

so:
RUN ["/bin/bash", "-c", "RUN Rscript -e etc etc etc"]

as per:

https://  github.com/docker/docker/issues/  8100
https://  github.com/docker/docker/issues/  7281
https://github.com/docker/docker/pull/7489/commits/55dc4f2b94f68e9e1b4e73b1f21cf5151f86e8f8

Nope. Looks like the Jupyter R is just damaged (probably as per 120 ?). Running from jupyter bash:
jovyan@717fbe331266:~/work$ Rscript -e 'install.packages("mallet", repos="http://cran.rstudio.com/")'
/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
trying URL 'http://cran.rstudio.com/src/contrib/mallet_1.0.tar.gz'

...and running from Jupyter R repl:
> install.packages("mallet", repos="http://cran.rstudio.com/")
trying URL 'http://cran.rstudio.com/src/contrib/mallet_1.0.tar.gz'
Content type 'application/x-gzip' length 4460706 bytes (4.3 MB)
==================================================
downloaded 4.3 MB

/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
/opt/conda/lib/R/bin/R: 12: [: Linux: unexpected operator
* installing *source* package ‘mallet’ ...


'The following Vite config options will be overridden by SvelteKit' message is hidden

Describe the bug
If you set a value in vite.config.js that SvelteKit needs to control, like base, a message is printed:

I had to add clearScreen: false to the Vite config to take that screenshot, because normally the message is immediately hidden by Vite's welcome message. Not sure if there's a way to either preserve the message or print it immediately after the welcome message?
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-pmx9s3?file=vite.config.js&terminal=dev
Logs
No response
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /bin/yarn
    npm: 7.17.0 - /bin/npm
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.60
    @sveltejs/kit: next => 1.0.0-next.373
    svelte: ^3.46.0 => 3.49.0
    vite: ^3.0.0 => 3.0.0

Severity
annoyance
Additional Information
No response

It can be printed after the welcome message by listening for Vite's http server start, and log after a setTimeout. Me and dominik did this for printing qrcode's after the welcome message.


Improve handling of From: Headers without parseable email addresses (fixe

Improve handling of From: Headers without parseable email addresses (fixes 3814)
http://code.google.com/p/k9mail/issues/detail?id=3814

where should i add testcases? Do we have some automated testing stuff like junit?

Yup, junit works; there's a few tests in tests/src/.  Mockito's also available, if you need that.


Fix edit to remove link not working

Fixed Issues
$ https://github.com/Expensify/App/issues/28344
Tests

Go to a chat and add link like

https://www.google.com

Edit the link from

[https://www.google.com](https://www.google.com)

to
https://www.google.com

Save it and verify that link, https://www.google.com, is displayed as plain text

https://github.com/Expensify/expensify-common/assets/117511920/a5bb7a43-7dd2-4496-85df-3f656b06df9e
QA
Same as tests

@lakchote Can you help to merge the PR? So that I can get the hash to bump version for the App. Thanks!

@eh2077 done!


Error with aot build

I'm submitting a ...  (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I'm using ngx-datatable in a library.
This library is used in another project. During the build I have this error :
ERROR in TypeError: Cannot read property 'provide' of undefined.
at @angular\compiler-cli\src\ngtools_impl.js:150:14
Expected behavior

Reproduction of the problem
I'm using ng-packagr to build & pack angular library.
I noticed that this bug happens when 'providers' is used in datatable.module
If the provider is set at the component level, the error disappears
What is the motivation / use case for changing the behavior?
Remove the error for aot build.
Please tell us about your environment:

Table version: 0.8.x
Tried with 8 to 10

Angular version: 2.0.x
4.4

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]

Please reopen if this is still an issue.


Cannot go to General Setting page

http://localhost:3000/admin/settings/site`
gives

Why is that?

Fixed, please check again.
It was a windows problem.

Hi @owen2345 ,
I follow all step (http://camaleon.tuzitio.com/installation.html) again but it also happened.
What was I wrong here?

Please check this:
http://camaleon.tuzitio.com:8080/documentation/category/40757-developer-docs/installation-1

It worked, thanks @owen2345


[release/1.6 backport] update to go1.21.3, go1.20.10

relates to https://github.com/containerd/containerd/pull/9180#issuecomment-1758276111

(partial) backports of:

https://github.com/containerd/containerd/pull/8957
https://github.com/containerd/containerd/pull/9178
https://github.com/containerd/containerd/pull/9072
https://github.com/containerd/containerd/pull/9221

Hm.. so, I guess I completely interpret the original intent in reverse; and "build binaries with go1.21" had the intent to "verify binaries also work with go1.21", but not to build (and release) binaries with go1.21.
Let me move this to draft to prevent trigger-happy merges, pending further discussion

Sorry for the confusion on that. I probably should have added more context to https://github.com/containerd/containerd/pull/9180.
It was a direct cherry-pick of https://github.com/containerd/containerd/pull/9167. The immediate reason for the change was to fix flaky tests with windows-2019 and go 1.19. I'm not sure if the original intention was to solve more than just that problem.
There's some more context in slack (but that really should have translated into the codebase itself). https://cloud-native.slack.com/archives/CGEQHPYF4/p1695915034690419

No worries! I should read more carefully 😂 It was just the PR title that confused me, and I somehow completely interpreted it as the reverse (thought we were building with 1.21, but all the test continuing on go1.20.. which felt "off")
I'll prepare another / alternative PR to update to go1.20.10 (for the security fix); I'll keep this one in draft for now, for when we're ready to update to go1.21  😄


User Interface becomes unresponsive when running many tasks

I'm running Mesos 0.22.1 and Marathon 0.9.0. I have ~15 applications deployed, most with 1 - 5 tasks. However, one application has 750 tasks. When trying to click on that app, to see the list of tasks the UI becomes unresponsive. Running Firefox, I'll eventually get a dialog asking if I want to Stop the script or continue.
I will eventually need to run thousands of tasks but I'm concerned based on what I'm seeing so far.

Hi @mindscratch, we are aware of this issue and want to fix it in the next months.
The API call that is used in the GUI returns all tasks of the app at once. That overwhelms the UI at the moment. Marathon itself should be fine with this number of tasks.
We want to create a new API call which allows paging through the tasks and update the GUI accordingly.

Also note: this PR which will be shipped in the upcoming Marathon v0.11.0 will already help a lot https://github.com/mesosphere/marathon-ui/pull/105

@kolloch @pierlo-upitup thanks, looking forward to the improvements.

Hey @mindscratch, your problem should be solved with the newest version of the UI by mesosphere/marathon-ui#105 . Before all existing tasks were rendered, even if there weren't displayed.. Now only the displayed ones are rendered. I just scaled up easily to 3000 tasks and I think it can be much more. Only problem can be a slow network connection.

@aldipower sounds good, thanks for your work on it.


Separate standalone components Part 1

Re #10152
As we discussed in today's standup, we are introducing a new top folder src/standalone as the place for components which are directly integrated to VS Code (through extensin.node/web.ts) and no other primitive components should depend on src/standalone.
I'll also move the rest of webviews into standalone but that would require changes to the build scripts so I'll do that in second PR.

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for feature-requests.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #10527 (a8db252) into main (684354b) will decrease coverage by 0%.
The diff coverage is 100%.

:exclamation: Current head a8db252 differs from pull request most recent head e6c8e42. Consider uploading reports for the commit e6c8e42 to get more accurate results

@@          Coverage Diff           @@
##            main   #10527   +/-   ##
======================================
- Coverage     71%      71%   -1%
======================================
  Files        466      466
  Lines      27909    27909
  Branches    4699     4699
======================================
- Hits       19840    19833    -7
  Misses      6196     6196
- Partials    1873     1880    +7

Impacted Files
Coverage Δ

src/standalone/activation/activationManager.ts
100% <ø> (ø)

src/standalone/devTools/serviceRegistry.ts
100% <ø> (ø)

src/standalone/import-export/importTracker.node.ts
85% <ø> (ø)

...one/recommendation/extensionRecommendation.node.ts
87% <ø> (ø)

src/telemetry/types.ts
100% <ø> (ø)

...nsion-side/dataviewer/dataViewerCommandRegistry.ts
72% <ø> (ø)

src/extension.node.ts
76% <100%> (ø)

src/standalone/activation/globalActivation.ts
92% <100%> (ø)

.../standalone/activation/workspaceActivation.node.ts
56% <100%> (ø)

src/standalone/api/api.ts
45% <100%> (ø)

... and 21 more


$scope.$on function is not running

Hi everybody, I'm trying to use $scope.$on and $rootScope.$broadcast in order to use a variable of a controller function in another controller function. I have already use this for other purpose, so my issue is the $scope.$on function isn't executing, I tried to make alert inside it but I didn't get any alert, I don't know what is the problem. Here is a snippet of my code to illustrate the issue. Thankx for advance :)
`app1.controller('autoCompleteCTRL',function($scope,$rootScope,$http) {
$scope.$on("myEvent3", function (event, args) {
    alert(1);
    $scope.SearchNearPoint= args.value;

    $http({
    method: "get",
    url: "path/to/file.php",
    dataType: 'json',
    params: { 'userText': $scope.searchText},
    async: false

    }).success(function(data, status, headers, config) {
    $rootScope.suggestions = data;

    }).error(function(data, status, headers, config) {

    });

 });

app1.controller('BasicCenterController', function($scope, leafletData,$http) {
$scope.$root.$broadcast("myEvent3", {
        value: startLatLng

    });

});`

Is it the problem because I use $scope.$on function before $scope.$root.$broadcast?

Hi, your issue reads like a support question. This github repo is only for bug reports and feature requests. Please try one of these channels instead: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question

Oki I understood. Thankx


fixing issue #68

fixing the approval_prompt issue correctly

Coverage remained the same at 100.0% when pulling af46092a7d3480f6ce72fd6d1841cea3dd72c73c on mxdpeep:fix68 into 4c927e4b567db026e4bac1b0ad0ffc5bda468074 on thephpleague:master.

Coverage remained the same at 100.0% when pulling af46092a7d3480f6ce72fd6d1841cea3dd72c73c on mxdpeep:fix68 into 4c927e4b567db026e4bac1b0ad0ffc5bda468074 on thephpleague:master.

Coverage remained the same at 100.0% when pulling af46092a7d3480f6ce72fd6d1841cea3dd72c73c on mxdpeep:fix68 into 4c927e4b567db026e4bac1b0ad0ffc5bda468074 on thephpleague:master.

Released in version 3.0.1.


fix(error & syntax): Optimize error handling

What ❔

Before the if statement, the else keyword is used, which is incorrect. The unwrap_or_else method should be used before the if statement to handle possible errors.

Unwrap() is used to handle possible errors, which may cause the program to crash when an error is encountered. A better approach is to use the ? operator to propagate errors so that the function can return an error when it encounters it, rather than crashing.

I added two new error types StorageAccessFailed and FailedTransactionCheck to the CircuitBreakerError enumeration
Why ❔

Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[ ] Code has been formatted via zk fmt and zk lint.
[ ] Spellcheck has been run via zk spellcheck.

Thank you for your contribution, unfortunately I am going to have to close it because it kind of breaks the way we are doing error handling currently. TLDR we want our services to go down fast on irrecoverable errors such as database going down and .unwraps() currently do exactly this for us. In the future we plan to improve this by cleaner shutdowns.


PSA: Rename AEAD tag length macros

This PR renames the AEAD tag length macros that were renamed in the PSA crypto API spec between Beta 3 and 1.0.0
Fixes parts of #3278
Status
READY
Requires Backporting
No, PSA only
Migrations
Yes, This brings them in line with PSA Crypto API 1.0.0
PSA_ALG_AEAD_WITH_DEFAULT_TAG_LENGTH -> PSA_ALG_AEAD_WITH_DEFAULT_LENGTH_TAG
PSA_ALG_AEAD_WITH_TAG_LENGTH                 -> PSA_ALG_AEAD_WITH_SHORTENED_TAG

#3984 Is dealing with an overlapping set of macros. Once the relevant parts of this PR are merged into that one (which I've already requested), I'll drop the overlapping changes, and refocus this PR on renaming PSA_ALG_AEAD_WITH_DEFAULT_TAG_LENGTH -> PSA_ALG_AEAD_WITH_DEFAULT_LENGTH_TAG
PSA_ALG_AEAD_WITH_TAG_LENGTH -> PSA_ALG_AEAD_WITH_SHORTENED_TAG, which would have been out of scope in the first place.

#3984 Is dealing with an overlapping set of macros. Once the relevant parts of this PR are merged into that one (which I've already requested), I'll drop the overlapping changes, and refocus this PR on renaming PSA_ALG_AEAD_WITH_DEFAULT_TAG_LENGTH -> PSA_ALG_AEAD_WITH_DEFAULT_LENGTH_TAG
PSA_ALG_AEAD_WITH_TAG_LENGTH -> PSA_ALG_AEAD_WITH_SHORTENED_TAG, which would have been out of scope in the first place.

I've pushed a new version of the PR that removes the overlaps with #3984

I've pushed a new version of the PR that removes the overlaps with #3984

I've rebased this PR on top of develoment.

Rebased PR again, and updated to account for new occurrences of PSA_ALG_AEAD_WITH_TAG_LENGTH added in #4010 and #4012


Suggestion: Allow use of ⌘+Enter on OS X to advance

This would be much more intuitive / ergonomic for mac users imho as opposed to ctrl-enter, and mimics gmail keystroke behavior on mac browsers. :)

That sounds like a good idea, but since I don't have a Mac, the Ctrl+Enter code is spread out all over the place, and the cmd key has different keycodes in different browsers, I'm going to let someone else have at it.
Shouldn't be too hard technically though, just a lot of testing and searching required.


ModuleNotFoundError: No module named 'dapr.version' (dapr-python 1.8.0)

Expected Behavior
No error!
Actual Behavior
youngp@DESKTOP_frlucas state_store % python3 state_store.py
Traceback (most recent call last):
  File "/Users/youngp/repos/samples/python-sdk/examples/state_store/state_store.py", line 8, in <module>
    from dapr.clients import DaprClient
  File "/usr/local/lib/python3.10/site-packages/dapr/clients/__init__.py", line 20, in <module>
    from dapr.clients.grpc.client import DaprGrpcClient, MetadataTuple, InvokeMethodResponse
  File "/usr/local/lib/python3.10/site-packages/dapr/clients/grpc/client.py", line 40, in <module>
    from dapr.version import __version__
ModuleNotFoundError: No module named 'dapr.version'

Steps to Reproduce the Problem

Install the latest dapr pkg (1.8.0
Run state_store.py sample

Release Note

RELEASE NOTE:

This breaks our code when upgrading to 1.8.0

I saw the same thing last night. I had to pin my version to 1.6.0 to get back up and running.

Thanks not sure why this happened but will fix!

I saw the same thing last night. I had to pin my version to 1.6.0 to get back up and running.

Why not pin the latest 1.7.X?

I saw the same thing last night. I had to pin my version to 1.6.0 to get back up and running.

Why not pin the latest 1.7.X? @DarqueWarrior

1.7.x works too. I pinned 1.7.0 for my app.

I have yanked version 1.8.0 and will release 1.8.1 momentarily.

Released 1.8.1 with the fix to ensure everyone pulls in the correct version. 1.8.0 is yanked and can no longer be installed.
Fix is here: https://github.com/dapr/python-sdk/commit/8f64c3ea259a1dd713308706e4a46f7b27b3e97b

Please note that the Metadata API and Configuration API depend in >=1.8.0 depend on proto changes that are only in Dapr Runtime 1.9.0. So you cannot use those Alpha APIs unless you upgrade to the new runtime release as well.

@espenairmine @ngiind @DarqueWarrior @youngbupark fixed now. Sorry about that!

I was in a hurry building my demo for Ignite so I pinned to the last known working version is all.


Moves the Git Diff window tree selection panel into the toolbar

The Git Diff window uses a lot of horizontal space while displays have more vertical space.
This is particularly annoying because any horizontal space taken by some other component is not usable by the lines of source code being diffed.
This PR moves the 'tree selection panel' into the toolbar thereby adding some 2 horizontal lines of code.

+1

Agreed. Works for me even on a 1280x1024 monitor showing project tree and properties:

And there is empty space in the toolbar, that could be reclaimed.


Fix production issue in pr for New UI Example Page

Fixes  #1209
Describe the changes you have made in this PR -Changed file extension from jpeg to jpg.
Refer: https://github.com/rails/sprockets-rails/issues/304
Screenshots of the changes (If any) -

Coverage increased (+0.01%) to 84.279% when pulling 80653caeaaf7c1552a00aa402dfff8990d3d7c95 on harshgujrati:New-Ex-Pg into 52726469cf2ccc42ed2aa48d8818cb22ce7f96d1 on CircuitVerse:master.

@tachyons  please review it


chore(pagination): use en dash instead of hyphen

replaces hyphen (-) with en dash (–) in example pagination content

@shinytoyrobots @alisonjoseph HTML-only change - Wanted to put this in v10? Thanks!


$__timeGroup not working with joins

$__timeGroup macro is replacing the name of the column with quotes here. This is causing some issues on queries that require table name specification before the column, very common pattern when joining tables.
For example, using a_ts as an alias for our table inside the $__timeGroup fails
This query works
SELECT
  $__timeGroup(created, 1d),
  action as metric,
  COUNT(a_ts.id) AS value
FROM
  table_name a_ts
GROUP BY 1,2
ORDER BY 1, 3 desc
limit 10

This query does not work and fail with message  Unrecognized name: a_ts.created at [2:38], invalidQuery
SELECT
  $__timeGroup(a_ts.created, 1d),
  action as metric,
  COUNT(a_ts.id) AS value
FROM
  table_name a_ts
GROUP BY 1,2
ORDER BY 1, 3 desc
limit 10

Naming the table is important in case joining tables and having ambiguous column names for example queries like
SELECT
  $__timeGroup(a_ts.created, 1d), # we need to have a_ts before created at since this column is present in both tables
  action as metric,
  COUNT(a_ts.id) AS value
FROM
  table_1 a_ts
  LEFT JOIN table_2 ts ON ts.id = a_ts.monitor_id
GROUP BY 1,2
ORDER BY 1, 3 desc
limit 10

@zoltanbedi any updates on the following? It really renders the plugin unusable on basic time grouping queries.

Hi @HaddadJoe,
the error message you are seeing here is from BigQuery can you please show me how the sql statement should look with alias that works on BigQuery but not in Grafana?

yes of course, here's how grafana plugin is translating this into BQ's query syntax
SELECT
  TIMESTAMP_SECONDS(DIV(UNIX_SECONDS(`a_ts.created`), 86400) * 86400),
  COUNT(a_ts.id) AS value,
  action as metric
FROM table_1 a_ts
LEFT JOIN table_2 ts ON ts.id = a_ts.monitor_id
GROUP BY time, action
ORDER BY time
LIMIT 10

As mentioned above, this returns Unrecognized name: a_ts.created at [2:38] and yes this is definitely coming from bq because the grafana plugin is building the query wrongly here.
Query should look like
SELECT
  TIMESTAMP_SECONDS(DIV(UNIX_SECONDS(a_ts.created), 86400) * 86400) as time,
  COUNT(a_ts.id) AS value,
  action as metric
FROM table_1 a_ts
LEFT JOIN table_2 ts ON ts.id = a_ts.monitor_id
GROUP BY time, action
ORDER BY time
LIMIT 10

Notice how the first column after the select does not have the single quotes around it. Let me know if you need any help, it's pretty easier to replicate.

create two tables on bq related with a "foreign key" (there's no enforcement of constrain on bq but I'm sure you get my point). Make sure you have timestamp in that table
setup latest grafana v9.2.6
configure bq plugin
try to query group by timestamp using the $__timeGroup(a_ts.created, 1d) syntax

@zoltanbedi here we go, https://github.com/grafana/google-bigquery-datasource/pull/155. Any expected timelines for cutting a new release? thanks for the help here!

@HaddadJoe I'll release this tomorrow once https://github.com/grafana/google-bigquery-datasource/pull/156 merged.

Sorry it took longer to release it but 1.2.3 is out with your PR. Thanks again @HaddadJoe.

tested works fine, feel free to close that issue. thanks for the collab here 💪


[WIP] Envoy-based ingress controller

This is a first-pass on Envoy-based ingress controller (addressing #6).
I'm PR'ing this while still WIP to start gathering feedback and set this in the right direction.
What we have so far:

Ingress controller running in the context of the manager (via manager ingress command).
Extending the platform/kube/Controller to sync ingress resources.
Adding ingress watcher that reconfigures Envoy based on the given ingresses.
Envoy clusters made up of single upstream (service's DNS name).

What's still missing:

Represent ingress rules in the Istio model, so that proxy/envoy isn't coupled with k8s objects.
Support for path regex in ingress (under the limitations of what Envoy supports)
Constructs envoy clusters using SDS + pod endpoints.
Update ingress status with the proxy's "public" IP address (node IP?).
TLS termination.
Testing.

@zcahana could you please sign the cla?

@rshriram I applied to join the ibm-contributors group, waiting for it to get approved.

More comments for the TODOs:

Constructs envoy clusters using SDS + pod endpoints.
Yes, we should avoid using kube DNS since we have SDS. Don't worry about CDS for now, it's easy to refactor for that later.
Update ingress status with the proxy's "public" IP address (node IP?).
GCE LB ingress does this because it manages firewall rules and IP allocation. NGINX ingress does not AFAIK and you have to create a separate service of type LoadBalancerIP pointing to NGINX deployment. Let's do what NGINX ingress does.
TLS termination.
Don't worry about it. We will do one big swoop to enable TLS termination.
Testing.
Take a look at test/integration/driver.go. I think it has all the pieces to reuse.

I've rewritten the ingress controller implementation such that Ingress API objects are converted to Istio model rules (currently, same config.RouteRule, but with a different kind - IngressRule), which are then transformed to Envoy configuration. There's still only a little re-use of existing Envoy configuration code. This can be refactored later when the package stabilizes and the common parts become clearer.
Note that due to the major changes that already went to master, I've rebased my branch and restructured the commits so that they're more reviewable. The commits from the first round are thus gone :).

Jenkins' presubmit build quickly fails with the following error (link):
+ go get ./...
<EMAIL_ADDRESS>2<EMAIL_ADDRESS>go: not found
[Pipeline] }
[Pipeline] // dir
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // node
[Pipeline] End of Pipeline
ERROR: script returned exit code 127
Finished: FAILURE

Not sure what's causing this... any chance it's some recent changes to istio-testing repo? @sebastienvas

Please rebase your PR.

Jenkins job manager/presubmit started

Jenkins job manager/presubmit failed

Jenkins job manager/presubmit started

Jenkins job manager/presubmit passed

thanks @sebastienvas , build now passes.

There's at least one bugfix on the way, and potentially another one. I'm currently running/debugging this locally.

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit started

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit started

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit started

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed

With the latest round of commits, I've verified that basic ingress function works (i.e., ingress controller forwards ingress traffic to correct backend service according to defined routes). In practice, the backend service's reverse proxy is still blocking the request from hitting the actual service (due to #150), so this is still not usable end-to-end, but probably solid enough for first-pass --> merging.

Together with #177 - fixes #6


fix: handles abort signal in response lifecycle

Problem: the native http/https 'request' method only handles aborting (through AbortSignal) during the request lifecycle not the response lifecycle. It means that if the AbortSignal is triggered while we are processing the response data, it will be ignored an we will the result as if nothing was aborted.
Example:
Considering this code:
import https from 'https'
import { addAbortSignal } from 'stream'

const abortController = new AbortController()
const req = https.request('https://www.google.com', { signal: abortController.signal })

req.on('response', (res) => {
  console.log('res')
  // Fix
  // addAbortSignal(abortController.signal, res)
  const chunks = []
  res.on('data', (chunk: Buffer) => {
    console.log('data', chunk.length)
    chunks.push(chunk)

    // Option 4: request and response "error" events are NOT triggered
    // abortController.abort()
  })
  res.on('end', () => {
    console.log('end', Buffer.concat(chunks).length)
  })
  res.on('error', (e) => console.error('RES ERROR', e))

  // Option 5: request and response "error" events are NOT triggered
  // abortController.abort()
})
req.on('error', (e) => console.error('REQ ERROR', e))

// Option 1: request "error" event is triggered
// abortController.abort()

req.end()

// Option 2: request "error" event is triggered
// abortController.abort()

// Option 3: request "error" event is triggered
// setTimeout(() => abortController.abort(), 5)

If we execute the code as is, we will have no error because we don't abort so we will see the body length as a result.
If we comment out the abortController.abort() for options 1, 2 or 3 then we will see the AbortError in the request error event listener which is what we expect
If we comment out the abortController.abort() for options 4 or 5 then we won't see any error and we will finish processing the response and display the body length as a result
If we comment out the fix addAbortSignal(abortController.signal, res) which basically just destroy the res stream when the abort signal is triggered, then it will trigger our request and response error event listener.

In docker-modem:
In the buildRequest method:

when we expect a stream as a result (context.isStream !== true), we directly call the callback with the stream so there is no problem. Also I think it's not the responsibility of this method to handle an abort event that could happen after we call the callback.
when we don't expect a stream, we have the pattern described above to read the response content before calling the callback which means that if the request is aborted during the response processing, the callback will be called with success parameters instead of an error.

The fix:
Same as the example above, we just need to make the response stream "abortable" so that the request ends with an error.

@apocas Any opinion on this ?

Great stuff 👍
Will ping when published.


Adding PinotCrypter interface

If we want the option to store data in encrypted form in remote pluggable storage, we need ways to decrypt/encrypt that data. Adding the initial interfaces that will take care of that

Codecov Report

Merging #3115 into master will decrease coverage by 10.16%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##             master   #3115       +/-   ##
============================================
- Coverage     70.06%   59.9%   -10.17%
  Complexity        4       4
============================================
  Files           936     938        +2
  Lines         43527   43541       +14
  Branches       5904    5904
============================================
- Hits          30499   26084     -4415
- Misses        10973   15517     +4544
+ Partials       2055    1940      -115

Impacted Files
Coverage Δ
Complexity Δ

...inot/common/segment/crypt/DefaultPinotCrypter.java
33.33% <33.33%> (ø)
0 <0> (?)

...inot/common/segment/crypt/PinotCrypterFactory.java
81.81% <81.81%> (ø)
0 <0> (?)

...lection/iterator/FloatSelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...ion/iterator/LongArraySelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...com/linkedin/pinot/minion/metrics/MinionMeter.java
0% <0%> (-100%)
0% <0%> (ø)

...nion/events/DefaultMinionEventObserverFactory.java
0% <0%> (-100%)
0% <0%> (ø)

...ection/iterator/DoubleSelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...on/iterator/FloatArraySelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...selection/iterator/IntSelectionColumnIterator.java
0% <0%> (-100%)
0% <0%> (ø)

...ot/minion/events/EventObserverFactoryRegistry.java
0% <0%> (-100%)
0% <0%> (ø)

... and 281 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10f8242...fc7bd4c. Read the comment docs.


use the latest version of unittest-helper jar

The latest version of unittest-helper is 0.5.1
On the other hand, the unittest-hlper 0.11.3 uses 0.3.0
Should upgrade it.

superceded by #115


Dev

Merge incomplete API wrapper feature. It success for Owner API and User API. It may change in the future. Merge it for now


Supporting php7

With the upcoming release of php7 all libraries under php-http should support php7 IMO, WDYT ?
(The change will be to remove php7 from the allow_failure in travis file in all libraries)

As long as PHP 7 is not released, allowing failures on it allows us to not worry about it when it fails because of a breaking change in the consecutive builds of PHP 7 or release candidates.
When PHP 7 is released, it is indeed recommended to remove it from the allow_failures.

PHP 7 is in RC state. which means i would not expect much to happen that could affect us here. and if it does we should report it to the PHP 7 maintainers so they can fix it.
I am +1 on the proposal.
Also, seeing that currently the build fails, we really should work on fixing the build for 7. There might be some issue that would lead to a BC break for us to be able to fix it (like doctrine has a problem with the class called String as PHP 7 says this is a reserved word, and they have a class for annotations for that...).
@joelwurtz could you work on fixing the PHP 7 build? we can then still decide whether we allow failures or not - but the stuff that is failing now is likely to keep failing. Maybe its just the phpspec version we use https://travis-ci.org/php-http/httplug/jobs/86556672

I think the current break stuff in PHP7 comes more from phpspec than this library (see https://travis-ci.org/phpspec/phpspec/phpspec/jobs/87006391)

@joelwurtz you are right, failing is because phpspec. AFAIK phpspec guys already work on a fix.


php comments

Thanks for the package, in many ways is better to hable code with php than js, Im having some problems hope you can help me, I trying this code but it doesn't divide into tags anything
<code class="hljs php">
// Instantiate the Highlighter.
$hl = new \Highlight\Highlighter();
$code = file_get_contents('some_ruby_script.rb');
</code>

Hope you can help me

I don't see any echo statement? 🤔

the echo is fine, for other codes it works, please see the first image, that's the render (echo) Im getting

I'm sorry, but all provided examples aren't even valid PHP code, no opening / closing brackets, no echo statement (or shorthand thereof), I just can't reproduce your problem!
Please provide a complete code example, which version you are using, etc 🙃

Also, just from an image, I can't tell you what went wrong. Please post your example markup (= the code being rendered by PHP)!