Support for Google Colab

Setup
Browser: Google Chrome
Editor: VSCode
Description
Can I use GhostText with Google Colab?

@fregante Is there any form of documentation indicating how one might debug/add support for a site on which GhostText doesn't work? I'm interested in getting it to work on Colab.

"Support" is made of 2 parts:

detection (which in this case it could be URL-based, but ideally attribute-based)
read/write value

If the value an be set via DOM, like for a standard contentEditable, the second point would be easy:
https://github.com/fregante/GhostText/blob/a3ac72db5c05edbe0d4e2cef70c1a4fd5cdfd11b/source/ghost-text.js#L12-L29
In the more likely case where you have to access the website’s own script data, you'd have to go through the AdvancedTextWrapper, which communicates with the unsafe-messenger script to get and set the text:
https://github.com/fregante/GhostText/blob/a3ac72db5c05edbe0d4e2cef70c1a4fd5cdfd11b/source/unsafe-messenger.js#L33-L52
You can see the above piece of code uses a local CodeMirror property and then sends the value via events.

It looks like Google Colab uses Monaco editor, which is tracked here:

https://github.com/fregante/GhostText/issues/140

I'd definitely love to add support for it.

Support was added in v23.3.11 🎉
https://github.com/fregante/GhostText/releases/tag/23.3.11


Respect multiple decoration with @subscribe and @register. fixes #335

The motivation for subscribe is pretty simple. There are cases where published topics are handled by the same function.
For register it's a bit harder to find a use-case, but maybe this could be a good solution for some authorization problems. Or maybe services of multiple tiers with a unified client, where the simplest service just registers the same function for many URIs.

This PR makes sense for the subscribes for sure as-is, but it's currently an error if you register() two procedures on the same endpoint in the same session. I think (seperately) we should improve that behavior (I just filed #400 for these improvements).
That said, you could of course choose to register the same procedure at multiple (different) endpoints, so the code in this PR is correct. It might be ideal if it checked to ensure that there's not duplicate endpoints, but overall I think it's better to simply call the "real" register() as this code does and fix the bahavior of the underlying register() calls as per #400.
Sorry it took us so long to review this, and thanks for the patch!

(@oberstet this can be merged)


Minor proposals

This change is

@kant we cannot merge this PR at this point in time

We going to shrink the text in the readme file soon, so its not relevant any more. moving to operator repository.


*: support checking that an interval tree's keys cover an entire interval

/cc @mitake

@heyitsanthony lgtm, thanks! jenkins-cov and jenkins-proxy-ci aren't working?

they're working, just a long queue. merging...


Cleanup dependencies implementation, allow dependencies on JobID explicitly

Addresses #95

Note: also renames --afterok to --depend to be cluster-agnostic.


Fix param in vagrant

Coverage increased (+0.02%) to 76.197% when pulling 13eb25276a18affa4207453d9b4a953f2109bad2 on lukasmacko:ipam into 3921f925d27a4905f1f6b1a331eea26090ba685e on contiv:master.


Prevent leaking Database password in Logs

Hello,
Prevent leaking Database password in Logs
In Logs database password is visible in plain text. Please don't log any sensitive information like Password, SSH keys in logs. thanks.

That would be possible if the docker image stops using the setup command which would require a full coverage of environment variables.

duplicate of #1308

No, it's not a duplicate. This issue refers to the database password that leaks into the logs if semaphore starts initially based on the container.

I see, lot's of leaky passwords then. Hopefully it finds its way to being resolved.

@mjknoxie what passwords do you mean?

@fiftin detailed in https://github.com/semaphoreui/semaphore/issues/1308
I thought this report was a duplicate of 1308, but @tboerger said it's not, which is fine.

This issue can easily getting solved by avoiding the setup execution and relying on environment variables only. The other issue can only be resolved by some kind of ephemeral chroot environment which automatically cleans up.

Not sure if the op has retried the latest beta, but that closed #1308 for me. Thanks!

The wrapper script which generates the configuration and leaks initially the database password haven't been changed so far, that's why this issue is still open.

Hi @mjknoxie new release don't use files to store passwords and keys.  Semaphore uses Stdin to send passwords to Ansible.

@skywalker123p thank you, we removed printing DB creds in Docker container.


[fmt] Error when including using cmake

When I use find_package(fmt) then I get the following error when I try to reconfigure my project:
» cmake .
CMake Error at D:/path/to/vcpkg/installed/x86-windows/share/fmt/fmt-targets.cmake:87 (message):
  The imported target "fmt" references the file

     "D:/path/to/vcpkg/installed/lib/fmt.lib"

  but this file does not exist.  Possible reasons include:

  * The file was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and contained

     "D:/path/to/vcpkg/installed/x86-windows/share/fmt/fmt-targets.cmake"

  but not all the files it references.

Call Stack (most recent call first):
  D:/path/to/vcpkg/installed/x86-windows/share/fmt/fmt-config.cmake:27 (include)
  src/CMakeLists.txt:1 (find_package)

-- Configuring incomplete, errors occurred!

There were no issues when building the library. The path to fmt.lib should be D:/path/to/vcpkg/installed/x86-windows/lib/fmt.lib to be valid.
Package&version: fmt:x86-windows             3.0.0-1

This is specific instance of the problem described in https://github.com/Microsoft/vcpkg/issues/77 .
Currently we are solving it using the strategy described in https://github.com/Microsoft/vcpkg/issues/77#issuecomment-250028244 . In this case you need to add a -DFMT_CMAKE_DIR=share/fmt (see https://github.com/fmtlib/fmt/blob/master/fmt/CMakeLists.txt#L55 ) to the OPTIONS of vcpkg_configure_cmake and modify the debug targets as in (for example) https://github.com/Microsoft/vcpkg/blob/master/ports/openjpeg/portfile.cmake#L20 .
I think that also the assimp port is affected by the same issue.

Ah, you're right. I even saw that issue earlier but didn't quite make the connection. Closing.

I think it would be useful to have this issue open until the issue is solved for the fmt port in particular.

Yes, these sorts of GitHub issues are the only way we can track and find problems like this. Thanks for reporting it!


Is it possible to configure --shm-size=2g?

I would like to use testcontainers to start a solace pubsub container ala:
https://docs.solace.com/Solace-SW-Broker-Set-Up/Docker-Containers/Set-Up-Single-Linux-Container.htm
sudo docker run -d -p 8080:8080 -p 55555:55555 --shm-size=2g --env username_admin_globalaccesslevel=admin --env username_admin_password=admin --name=solace solace/solace-pubsub-standard
I've managed to expose the ports, and configure the environment variables, but is it possible to achieve the equivalent of --shm-size=2g?

Sorry, I didn't understand the syntax:
.with_kargs(shm_size="2g")
My bad.


Explicit note of underlying algo (LightGBM?) for Multiclass Boosted Decision Tree component

[Enter feedback here]
Docs for the Boosted Decision Tree regression and Boosted tree Two Class components call out explicit details of the underlying algo used (LightGBM). Would be useful to have Multiclass Boosted Decision tree component document follow similar pattern for consistency.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e49b8b9a-f407-63d2-12c7-5d6ad92662b5
Version Independent ID: a9767725-83c0-2cec-ab07-11828040c06e
Content: Multiclass Boosted Decision Tree: Component Reference - Azure Machine Learning
Content Source: articles/machine-learning/component-reference/multiclass-boosted-decision-tree.md
Service: machine-learning
Sub-service: core
GitHub Login: @likebupt
Microsoft Alias: keli19

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@cauldnz Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


CARDS-2013 - The wrong number of steps is displayed in paginated forms when the pagination variant is dots or text

See details in the description of CARDS-2013 on Jira.

This isn't good enough. The "current" page still goes to N+1, when the last page is saved (not in patient portal mode, but in the classical UI). Create a questionnaire with two pages, and filling it in goes:

in progress mode: half highlighted, half done and half highlighted, all done
in dots mode: first dot highlighted, second dot highlighted, no dot highlighted
in text mode: 1/2, 2/2, 3/2

The number of the "current" page after saving should also depend on the pagination variant.

Addressed in 293bd97. Also rebased on the latest dev.


"socket hang up" Error when sending SMS

In a recent project, we use the Nexmo module to create a call and send a SMS sequentially. When trying to send the SMS, we received the following error: socket hang up
The following is the debug log when sending the SMS
info: Request: { host: 'rest.nexmo.com',
Dec 06 08:29:33 fcm-solutions app/web.1:   port: 443,
Dec 06 08:29:33 fcm-solutions app/web.1:   path: '/sms/json?from=xx&to=xx&text=message&api_key=xxx&api_secret=xx',
Dec 06 08:29:33 fcm-solutions app/web.1:   method: 'POST',
Dec 06 08:29:33 fcm-solutions app/web.1:   headers:
Dec 06 08:29:33 fcm-solutions app/web.1:    { 'Content-Type': 'application/json',
Dec 06 08:29:33 fcm-solutions app/web.1:      Accept: 'application/json',
Dec 06 08:29:33 fcm-solutions app/web.1:      'User-Agent': 'nexmo-node/2.0.2 node/8.9.2',
Dec 06 08:29:33 fcm-solutions app/web.1:      'Content-Length': 155,
Dec 06 08:29:33 fcm-solutions app/web.1:      Authorization: 'Bearer {token}' } }
Dec 06 08:29:33 fcm-solutions app/web.1: Body: undefined
Dec 06 08:31:02 fcm-solutions app/web.1: [2017-12-06T16:31:02.497Z] error: problem with API request detailed stacktrace below
Dec 06 08:31:02 fcm-solutions app/web.1: [2017-12-06T16:31:02.497Z] error: { Error: socket hang up
Dec 06 08:31:02 fcm-solutions app/web.1:     at createHangUpError (_http_client.js:331:15)
Dec 06 08:31:02 fcm-solutions app/web.1:     at TLSSocket.socketOnEnd (_http_client.js:423:23)
Dec 06 08:31:02 fcm-solutions app/web.1:     at emitNone (events.js:111:20)
Dec 06 08:31:02 fcm-solutions app/web.1:     at TLSSocket.emit (events.js:208:7)
Dec 06 08:31:02 fcm-solutions app/web.1:     at endReadableNT (_stream_readable.js:1056:12)
Dec 06 08:31:02 fcm-solutions app/web.1:     at _combinedTickCallback (internal/process/next_tick.js:138:11)
Dec 06 08:31:02 fcm-solutions app/web.1:     at process._tickCallback (internal/process/next_tick.js:180:9) code: 'ECONNRESET' }

We believe it has something to do with using 2 instances of the Nexmo module, one for VAPI, the other for SMS.

Hi @tbass134, thanks for logging the issue. Can you also post a piece of code so I can try to reproduce this please?


zdt deploy

nb: moved command from manifest to procfile
todo: test this, since it's from a docker container so slightly different to existing process. also confusing NOTIFY_APP_NAME and CF_APP stuff.

@sakisv it's used in log groups and such (and is different to CF_APP in places eg https://github.com/alphagov/notifications-api/blob/master/manifest.yml.j2#L57) but I agree I think it's unnecessary.


Example training.conll, development.conll, and test.conll files?

Hello,
I have been trying to set up a simple test just to see if the code works but cannot figure out proper training.conll, development.conll, and test.conll files.
Can you please add an example or data directory with these files that you have used to train and test the network as I would like to run the exact replica of your work to get a better feel for it and for the discussions in your paper that I have on BiLSTM parsing for NLP.
If you could please add these files that you used to the repository then that would be a great help in this effort.
Thanks,
Lonnie

Unfortunatly, I cannot release the data you requested since it is protected by a commercial license  and therefore conflicts with the apache license.. Regarding the sample conll please refer to the link describing the conll format, there you will be able to find an example conll.

Dear sir , I have also made it workable , code is working well. I used conll format training and development data ... but one thing is not clear to me that .. what should the way to fit test data .. either these will be plain text or not. Sir could you please make me clear . Thank you.


Kitematic 0.7.2 for Windows - can not install

Hey Guys,
I was just trying to install the kitematic version 0.7.2 for Windows on Windows 8.1.
It started, but failes at about 22% with the following error message:
Setup Error
We're Sorry!
There seems to have been an unexpected error with Kitematic (+1)
C:\Users\tmichels\AppData\Local\Kitematic\app-0.7.2\resources\resources\docker-machine.exe -D create -d virtualbox --virtualbox-memory 2048 kitematic returned non zero exit code. Stderr: Error creating machine. Machine kitematic already exists. You will want to check the  provider to make sure the machine and associated resources were properly removed.
Although the virtualbox is defently NOT present, because I deleted it manually. 100% defently positivly sure.

I just fixed same issue by removing kitematic directory out of ~.docker\machine\machines\kitematic

Thanks to leafmulch's comment, I was able to bring up kitematic on my windoze box.
It is working fine and smoothly. Great job and thanks a lot.
I wouldn't have guessed to search in that corner for a solution.
We can close this issue now!

I ran into this issue too.  I had this issue also.  Finding a github comment is not the resolution that is best for the product.  When you click retry, it should remove this directory.

Hi Matt,
unfortunately it didn't help to click on "retry". I tried this .... many
times ;-)
But now, with the version Kitematic 0.7.4 for Windows this issue seems
to be solved.
Cheers
Thorsten
Am 10.07.2015 um 04:26 schrieb Matt Jones:

I ran into this issue too. I had this issue also. Finding a github
comment is not the resolution that is best for the product. When you
click retry, it should remove this directory.
—
Reply to this email directly or view it on GitHub
https://github.com/kitematic/kitematic/issues/706#issuecomment-120205195.

I'm having the same issue with 0.7.4, and I've deleted the folder ~.docker\machine\machines\kitematic.  I think this issue should be reopened.

Matt do you have other machines and certs already under ~/.docker?
It might be worth just moving the whole .docker directory to one side and try again. I sense that boot2docker, docker-machine, old kitematic and current kitematic aren't working well together.


nativemate: increase recursion depth

Fixes #1403
This is the maximum depth supported by ipc_message_utils .

Looks good to me, thanks!


AJ-596: TSV Streaming Download

I've tested this with a record type larger than 1gb and confirmed that it works even on a 100mb heap.  Changes are in place to make sure we stream from db to java.sql.Resultset, from Resultset to intermediate domain object, and from domain object to http response.  I can share a large json file you can use as input to the batch write API if you'd like to test for yourself.

See comments inline!
Also - there's some funky indentation here, could you format the changed files with Spotless? Obviously the Spotless integration isn't working seamlessly, we should make a follow-on ticket to make spotless more automated.

I see that in github, but it looks fine my IDE.  I've run spotless.


Different shortcuts for FancyZones Snap (instead of overriding Windows Snap's shortcuts)

Description of the new feature / enhancement
Add the function to bind custom keyboard-only (i.e. mouse-less) shortcuts to FancyZones snapping.
Scenario when this would be used?
Currently, the only way to bind keyboard-only shortcuts to FancyZones is to override the Windows Snap's shortcuts. FancyZones and Windows Snap both have their pros and cons, so people may want to have a shortcut handy to both of them in their workflow.
Shortcuts likely to be used for FancyZones if this feature were added:

⊞ Win+Shift+arrow, which will override the shortcuts to move windows across the monitors (which they may not have anyway) and to maximize-but-only-vertically,
⊞ Win+Ctrl+arrow, which will override the shortcuts to switch virtual desktops (which they may not be using anyway),
⊞ Win+Ctrl+Shift+arrow or ⊞ Win+Alt+arrow, which will keep all the originals.

Supporting information
No response

Duplicate of #13689

Thanks for your suggestion! This has been raised in #13689. To centralize the discussion thread, please go to that thread for discussion.

@ThoAppelsin Thanks for your suggestion. I've added it to #13689 since It'll be the enhancement tracker.


NameError: name 'exit' is not defined

Bug importance: minor
The exit() function is not defined in some cases apparently, sys.exit() should be used instead.
This should fix the problem where the program would signal a crash when it was actually closed with exit().

Fixed with 1.2.4


[material-ui] 49kb bundle size increase with the new Next.js integration package

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
If you use the new implementation for integrating mui in NextJs (pages router), as described here: https://mui.com/material-ui/guides/nextjs/
The bundle size increases with 49kb:

Can you provide a repo for me to check?

I can reproduce the issue. It happens with the latest version.

@laurenswuyts I think I found the fix. Can you replace the version of @mui/material-nextjs with https://pkg.csb.dev/mui/material-ui/commit/2d5cbbca/@mui/material in your package.json? Then remove .next folder, run yarn install and yarn build again.


[Tidy] Replace custom classNames with BS utility CSS classes

Description
Our long-term objective is to minimize the use of custom CSS in the static folder, favoring Bootstrap syntax instead. To achieve this, we can replace custom class names with Bootstrap utility CSS classes: https://getbootstrap.com/docs/4.0/utilities/borders/.
This PR shows one example on how to do so. However, I don't recommend replacing all custom class names with Bootstrap shortcuts, as it could make it harder for users to override styles.
Here are my current guidelines:

Use Bootstrap classes instead of custom class names when it's unlikely that users will want to customize the HTML element via CSS, or when there is an alternative method for customization.
Use custom class names for HTML elements that users are likely to want to customize e.g. all our main containers (page-header, control-panel, right-side, left-side etc.)
Not related to above: Remove any redundant extra div if you see one

See example here where we could simplify using BS shortcuts:  https://github.com/mckinsey/vizro/pull/910#discussion_r1865476510
Screenshot
Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
I have not referenced individuals, products or companies in any commits, directly or indirectly.
I have not added data or restricted code in any commits, directly or indirectly.

The only thing that's awkward here is the !important that's needed, and maybe if that's already used in bootstrap it doesn't even work well in practice? If that's the case then agree with what you say and I'm fine to just do this like:

Yes, that's why I wasn't more supportive of using native Bootstrap. When you use any of Bootstrap's utility classes, they automatically include the !important tag.
For example, if you write: html.Div(id="page-header", classNames="align-items-left)
Bootstrap will apply the following CSS:
#page-header {
   align-items: left !important;
}

This forces the user to also use the !important tag in their CSS to override it. Normally, user-defined CSS should take precedence since it is loaded last, but we can't predict how users will utilize vizro and the assets folder. For instance, we had difficulty styling some components of the dcc components because their CSS also used !important tags. This made it challenging to override styles due to conflicting orders and as a user you had to configure out the specificity.
Additionally, our users would need to inspect elements to differentiate between CSS properties from our static folder and Bootstrap's classes with !important tags, and their own styles. So I felt that approach would just lead to the user overusing the !important which in general is bad practice and would lead to confusion and potential errors 🤔

I've already merged this PR, but we can continue discussing our future strategy. I plan to replace all other custom class names with Bootstrap utilities for those elements that users are unlikely to want to customize.
For elements where users might want to apply custom styles, I'll leave the class names unchanged for now, pending our discussion above. 👍


Add Oracle Linux 6.6 i386 & x86_64 templates

Update Oracle Linux 6 templates to latest 6.6.
All have been tested and confirmed to work with veewee master branch + VirtualBox 4.3.26 on Linux x86_64.

can you consider merging templates like this: https://github.com/jedi4ever/veewee/blob/master/templates/CentOS-5.10/definition.rb


fix unmarshal err for blank field

What this PR does / why we need it:
fix unmarshal err for blank field
E1021 16:37:15.521443       1 control_plane_init_mutex.go:67] init-locker "msg"="Failed to get information about the existing lock" "error"="failed to unmarshal semaphore information: unexpected end of JSON input" "cluster-name"="capi-quickstart" "configmap-name"="capi-quickstart-lock" "namespace"="default"

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/lgtm
/approve


Implement token based authentication to eliminate Kudu's dependency on secret location

The runtime admin APIs need to support a token based authentication that does not rely on the master key.
The ability to store tokens in different locations (e.g. blob storage) increases the complexity and need to duplicate logic for systems like Kudu, since they must now be able to resolve the keys location in order to use the current authentication mechanism.
To address the issue, the following tasks need to be completed:

[ ] The security library (used for key encryption and decryption today) will be enhanced to expose token issuance and validation APIs. The token will be signed with the key resolved by using the same logic we follow to locate the current encryption/signing key for secrets.
[ ] Kudu will consume this new functionality and expose an API to be consumed by the portal (and other clients) to issue a token. This will replace the current call to get the master key.
[ ] The runtime will be enhanced to validate/authenticate this new token, when present, and appropriately set the authentication level.

@paulbatum @davidebbo @christopheranderson FYI, assigning this to the March milestone as, unless instructed otherwise, I'll be treating this as my highest priority at the moment (work is already in progress), since this is a requirement for some the items @krolson and @dariagrigoriu are currently working on.
@watashiSHUN FYI. I'll sync up with you on the Kudu work (David may have some comments as well).

All tasks completed.
Runtime changes merged with #1373


Correction for raw affiliation Department of Sport Science and Physical Education, École Normale Supérieure de Rennes, 35000 Rennes, France; Laboratoire M2S, Université de Rennes 2, 35000 Rennes, France

Correction needed for raw affiliation Department of Sport Science and Physical Education, École Normale Supérieure de Rennes, 35000 Rennes, France; Laboratoire M2S, Université de Rennes 2, 35000 Rennes, France
raw_affiliation_name: Department of Sport Science and Physical Education, École Normale Supérieure de Rennes, 35000 Rennes, France; Laboratoire M2S, Université de Rennes 2, 35000 Rennes, France
new_rors: 01m84wm78;015m7wh34;03rxtdc22;05b3p7p58
previous_rors: 01m84wm78;015m7wh34;03rxtdc22
works_examples: W4324146580
contact: 66119bf53eaecf71617430f26ae665e7:44055936dc94c89b6520b7eef4240c20 @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Eclipse Import: Timeout of 120000 reached waiting for exclussive access to file 'gradle-2.4-all.zip'.

I just get the error Eclipse Import: Timeout of 120000 reached waiting for exclussive access to file 'gradle-2.4-all.zip'. when I try to import my freshly generated libgdx project in Eclipse Mars.

``Gradle IDE <IP_ADDRESS>512211912-CI-B53` is currently installed.
I'm unsure where to find the mentioned erorr log. I don't see one in my project directory.
Thanks for looking at this!

It appears that the Gradle 2.4 distribution is no longer available. That's nice backward compatibility by Gradle :)
@tomski, could you update our stuff to use Gradle 2.10? Thanks!

So how should I go on @badlogic? Is there some easy way to change Gradle to newer version myself or do I have to wait for some bigger patch?

The 2.4 distribution is still available... That said, I am sure it would be fine to update.

@GiantCrocodile if you go into the /gradle/wrapper/gradle-wrapper.properties, you can change the version there.

Official Gradle download server is very slow at the moment. I get less than 20 kB/s. So it just timed out because of download speed. I just changed to some mirror and tried to use Gradle 2.10 but it fails because 2.2 is required @badlogic. Then I tried 2.9 like @MobiDevelop said and it works now.


Debouncing page has a link error

Fixed here https://github.com/makeabilitylab/physcomp/commit/ed8096a54d8408e9162f1ca70ec7afa8fc1956df


Parallel and Concat table support

Ideally, we will be able to decompose the table operations into operations working just with tensors and simply update the importer to support this. However, this is not really trivial and is also related to #627


full text search for media file, some permission checks

hide filter 'hidden' for delegates
don't show empty menu for delegates
fix full text filter for mediafiles (filter by file title (name given during import)

@tsiegleauq I'll merge it now. You can review later if you like.


fix(contribute): times_cont_diff got renamed

Fix the link to analysis/calculus/cont_diff

(The rename PR was apparently https://github.com/leanprover-community/mathlib/pull/12227)

yeah, it has been quite some time ago


Is 0.2.7 release correctly built?

After untar h30ga_0.2.7 package, tree is totally different than h30ga 0.2.6

Yes, it's ok.


Wrong results for miopenTransformTensor() from MIOpen 1.8

Using miopenTransformTensor() to compute y = 0.4 * y + 0.7 * x, and the answer is just not correct. By simple profiling, no any compute kernel is even launched on GPU when calling this API.

How was this test run? Via MIOpenDriver?

There is a ctest for tensor_transform. Currently we only support alpha = 1, beta =0 cases.
I will work on supporting arbitrary alpha/beta.

@ce1adon OK.. It means this API just supports y = x ..?

Currently yes @ghostplant

@ghostplant
Hey I'm working on this issue by providing better functionality and documentation. Would you like to describe a few more details or examples that you need from miopenTransformTensor()? What logic are you expecting this function to fulfill? Are you working on y = b * y + a * x with x and y in the same dimension, or you expecting some format transform in x and y at the same time (some details would be great)?
Thanks!

@ce1adon Hi, mostly includes b = [-1, 0, 1], a = [-1, 0, 1].
For example, I wanted to it for self-add and self-minus in the past.
Although miopenOpTensor() can do the similar thing, but it is 2-input based, so it is not always cheap to compute in terms of resource occupation.
I didn't feed y and x to be of different shapes before, but that function is supposed to accept cases like doing broadcast add/...

MIOpen needs a broadcast operation for sure. I think this specific change may be more narrow at first to encompass transforms on tensors of the same dimension. We can do another pass to implement broadcast tensor op.

@ghostplant @daniellowell
Thank you!
Let's make your expectation high priority and focus on fulfill it.

@ce1adon Thanks!

@ghostplant Do you mean something similar to TensorFlow's broadcast function?

yes, also similar to cudnnAddTensor.

We do have similar function: miopenOpTensor().
The difference of it with cudnnAddTensor() is that miopenOpTensor() handle a more general case c= beta* c + alpha1* a + alpha2* b.
You can also perform multiple(), min() and max() using this function.
To mimic cudnnAddTensor() behavior, you can arrange as below. Make sure those dimensions you want to broadcast equal to 1 and other dimensions matches the cTensor.
float alpha1 = 0;
miopenOpTensor(handle,
miopenTensorOpAdd,
&alpha1,
cDesc,
C,
&alpha,
aDesc,
A,
&beta,
cDesc,
C);

@ce1adon Hi, according to https://github.com/ROCmSoftwarePlatform/MIOpen/blob/master/src/ocl/tensorocl.cpp#L1192,
it doesn't support broadcast, and the runtime error output is:
MIOpen Error: /root/driver/MLOpen/src/ocl/tensorocl.cpp:1194: A and C Tensors do not match
terminate called after throwing an instance of 'std::runtime_error'
  what():
error: miopenOpTensor(global_cudnn_handle, miopenTensorOpAdd, &alpha, in_desc, input0, &beta, out_desc, output0, &beta, out_desc, output0) failed with error
file: /host/root/nnfusion_rt/rocm_codegen/nnfusion_rt.cpp
line: 441
msg: miopenStatusUnknownError

@ce1adon Sorry, I just verified that the input tensor to broadcast must be as the 2rd input argument.


Fix composer in encryption

Question
Answer

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed tickets
-

License
MIT

Doc PR
-

Codecov Report

Merging #217 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #217   +/-   ##
========================================
  Coverage      82.2%   82.2%
  Complexity      601     601
========================================
  Files            33      33
  Lines          1540    1540
========================================
  Hits           1266    1266
  Misses          274     274

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 93859a5...a05346e. Read the comment docs.


[ENHANCEMENT] Update relationships between tables

Summary
For SELECT statements:

[x] Tables with default joined should be updated to reflect so
[ ] Tables in which back referencing is not needed should be updated to remove relationships

For INSERT/UPDATE statements:

[ ] Joined insert/update should be wrapped inside a transaction

Example

[ ] Variables related entities

Actually this is not recommended as selectin is preferred over joined since optimisation for removing redundant rows is done in the database and not the CPU, whereas joined is more CPU intensive for joining two large tables.
Note that both select in load and joined are eager, so should only be used if needed.
On the other hand, the current repo does have an excessive number of selectin relationships, but without knowing the UI in advance, we should just leave the relationships as is.


position vendor prefix not working in Safari

I used position: sticky in my project, and it's working properly in Chrome, but turns out that it's problematic in Safari, I inspected the element, and position: sticky doesn't even show up. Would it suppose to be automatically transformed to position: -webkit-sticky ?

The issue seems to be within the inline-style-prefixer v2.0.5 dependency for safari versions 10+. For safari versions under 10, using position: sticky works for me. I've filed an issue with them here: https://github.com/rofrischmann/inline-style-prefixer/issues/126.

inline-style-prefixer has fixed this issue. We will need to migrate over to their next release when it happens to fix this issue.

Hey Stefan, so glad to meet you here. Thank you for keeping me posted regarding the issue we run into. I will try the latest version of inline-style-prefixer and see if it solves the problem.

Also should I use 3.0.3?

oh hey ZK. i didn't realize it was you! yeah, looks like the maintainer made another release to 3.0.3 that should have the fix. hope things are going well at op.


Feature request: Currency Customization

My server uses XPConomy with Vault plugin.  Is it possible to have Vane (especially Regions) use the Vault currency?  If not, is it possible to specify experience as the currency in the currency material?

I can certainly look into Vault integration. If it doesn't add a lot of technical debt and complexity i will add it in the next release.


Use supports X Browserslist query instead of exact versions list

In browserslistTargetHandler.js we have a few objects with browsers support data.
Instead of hardcoding this versions, we can use Can I Use data:
let dynamicImport = browserslist('supports es6-module-dynamic-import')

Reasons:

One source of truth
Less code in webpack
Future-proof for new browsers

@alexander-akait I found this improvement idea during the investigation of #14203

We try to do it before, but it was rejected due invalid browsers versions

Check out versions and check what browserslist returns, open a real browser and check it - failed

Is it because Can I Use data for some features was incorrect?

Yes

If they fix it, then we will return to the syntax you suggested, otherwise we generated invalid code, currently versions was checked by me manually using real browsers and as you can see no problems a long ago.

What about Can I Use data right now?

Still invalid checked a week ago

bump

@alexander-akait did they fixed Can I Use data? How we can fix it?

@ai Still not fixed :disappointed: Looks like it's not good so no one wants to fix it, let's reopen

What do we need to fix in Can I Use. I can send PR.

@ai these versions https://github.com/webpack/webpack/blob/main/lib/config/browserslistTargetHandler.js#L107-L126 was changed manually by me, so caniuse data should have almost the same versions, so if we fixed it we can use browserslist('supports es6-module-dynamic-import'), my previously check has other versions

@alexander-akait I decided to review a difference. I started from Chrome. ≥ 61 in Can I Use vs ≥ 63 in webpack.
I used Browserstack to load Chrome 61 and run ES tests from MDN. All tests except dynamic-module-imports were passed.
Why do you think Chrome 61 does not support ESM?
Maybe there is some bugs?

@ai As far as I remember when I tested I took information from MDN https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/import (63 there)

For dynamic import() we have exactly the same results:

https://caniuse.com/es6-module-dynamic-import
https://github.com/webpack/webpack/blob/main/lib/config/browserslistTargetHandler.js#L107-L126

Did I miss any difference?

Looks like fixed, because previously it was not 61, do we have supports x for each item from https://github.com/webpack/webpack/blob/main/lib/config/browserslistTargetHandler.js#L127 (i.e. const/arrow-functions/etc)

Looks like fixed

Let’s replace to Browserslist 🥳

do we have supports x for each item from

Yeap, we can use it everywhere since you already have Browserslist in dependencies.

@alexander-akait should we re-open it?

If there are still compatibility issues here to work out, can some output warnings or documentation be added to inform users in the meantime?
I have a query that uses supports es6-module-dynamic-import, and much to my confusion, I had to dig into the runtime code to find it wasn't using dynamic imports at all.

For dynamic import at least, it seems the versions agree between Webpack and CanIUse with the exceptions of UC and kaios.


Remove unused JWK error type

As noted here https://github.com/go-jose/go-jose/pull/130#pullrequestreview-2179088936
The error type errUnsupportedJWK is never used.
In addition, swapped out a use of fmt.Errorf() with the slightly more performant errors.New().

@jsha I'd like to hear from you on what the intention was here - is this an unused error we can drop, or should we actually use it?

Not a problem @jsha - happy to help. 👍


_static is ignored

It looks like that the _static folder is ignored by sphinx-revealjs and its content gets not copied to the final _build/revealjs/_static folder.
The folder itself is there, but it contains files only, which were added by extensions.
User defined content from _static can not be found.
A make html for the same project copies the user data from _static correctly to _build/html/_static. This works pretty well.
Any idea, how this can be fixed?
Tested with version 1.3.1 and sphinx 4.2.0.

Do you define reavealjs_static_path in your conf.py?
sphinx-revealjs use revealjs_static_path to ref static files folders (not html_static_path).
If there is not reavealjs_static_path in your conf.py, set your static folders.
# Example 1: simple usage
revealjs_static_path = ["_static"]

# Example 2: If you use same folders from html build
revealjs_static_path = html_static_path


Allow additional node affinity expressions in helm charts

This PR allows specifying custom node affinity expressions by configuring the new global.affinityMatchExpressions config value. To ensure this works as expected this PR also adds calls to the nodeaffinity helper to some pod templates that were missing it.
This is needed in situations where it would be desirable to ensure all istio components are running on a certain subset of available nodes.
Related to #9074.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: andraxylia
If they are not already assigned, you can assign the PR to them by writing /assign @andraxylia in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @cezarsa. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@cezarsa: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: cezarsa
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: andraxylia
If they are not already assigned, you can assign the PR to them by writing /assign @andraxylia in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Sticky navbar and search

Make navbar stick to the top. Same thing for search bar, maybe with filters.

Done


walk: only traverse hyperx strings

Make walking safe; prevents accidentally catching other template strings (e.g. https://github.com/sheetify/sheetify/pull/41). Thanks!

welp, was adding a regression test and there are none - was all my code. Thanks tho!


Doki Theme overwriting active tab when restoring window

Describe the bug
Doki Theme is replacing the latest active tab with a new tab when reopening closed Firefox windows.
To Reproduce

Open a Firefox window
Turn on "Restore previous session" in Settings > General > Startup
Open at least 1 tab
Close the window
Reopen Firefox

Expected Behavior
Reopening the window should restore the previous window's tabs, but Doki Theme is overwriting the active tab with the blank (new) page even if it previously had an open resource.
Environment

Firefox Developer Edition 91.0b3
Doki Theme for Firefox 11.0.1
Windows 10 Home 21H1 64-bit (build 19043)

Ah. I am able to reproduce the issue.
Wonder why I haven't noticed this issue.
FireFox is my daily driver.
I'll have some time later this week to look into the issue. Thanks for providing a detailed report!

I can provide a screen recording of the bug happening if you need it, it happens every time on my device. My workaround is to open a new blank tab before closing so it doesn't replace any of the tabs I'm using.

I'm sure it's Doki because I've tried disabling everything else, and it still happens. Screen recording here.

@kxmndz Yup, that's what happened to me as well. I was able to reproduce the issue, fairly confident it is an issue with the plugin. Thanks!
I also think I know why I didn't notice before you though.

My workaround is to open a new blank tab before closing so it doesn't replace any of the tabs I'm using.

I always have a blank empty tab open, I always switch to it when I am done with the browser so I can look at a pretty girl 😂

@kxmndz Aight, version v12.2.0 has been published to the marketplace and is available for update!
I've verified the fix, I closed all my tabs for you 😭
Thank you for taking the time to report the issue!


[Suggestion] New tutorial for setting up a custom server

The support page has a tutorial on setting up a server, however it is from 2016 and many things have changed.
For example, introduction of TLS support, the new server package and more importantly Debian and Ubuntu switching to the systemd init system as opposed to previous SysVinit. These changes greatly simplify the setup, but also make the old tutorial incompatible with newer installs.
If desirable, i would be willing to writeup a new guide to put on the wiki here possibly, that includes the whole setup.

I was already planning to write a server guide. Though thinking about it, it might be worth while to bounce some ideas around. I wanted to include the following in the guide:

Detailed installation on Debian and its derivatives (deb package) with all commands.
Basics of installing it somewhere else (satisfying python requirements, maybe examples for docker?)
Detail all the cmd switches, what they mean and whether you should use them.
How to make a systemd service unit with it.
Maybe include an example how to make TLS work with letsencrypt?

Do you think the guide should include something more or less than the stuff above?

Yeah, this is essentially what i was planning to do, with an addition of installing somewhere else.
Description of the command line switches is already in the docs, so the guide probably wouldn't need to go that into detail on them though.
I already have essentially the full install done on my server with following:

Install on Ubuntu from the server package
Systemd unit file
DNS and letsencrypt certificate
All running under a system user syncplay with explicit ACL to allow only syncplay user access to read the certs

The install is a few months old, so my memory is a bit rusty, but i have all the commands i used in bash history.


Remove any hardcoded references to std::experimental

As mentioned in the Python PR on the support lib, there were a few places in the generator where explicit references to std::experimental had been left behind. The options presented at that point to solve it were to either hardcode std::optional/std::nullopt everywhere or use the command-line parameter for the optional type to replace the aforementioned references.
This PR represents a third option, which is to replace the three instances of std::[experimental::]nullopt with code that does not specify any type:

Evaluate the optional type directly in a boolean context (false means != std::[experimental::]nullopt)
Default-construct an std::[experimental::]optional object with {} (which is equivalent to std::[experimental::]nullopt)

This might be an acceptable solution until we decide (or find time) to deal with the std::[experimental::]optional situation as a whole.

Code change is looks correct, thanks Leandro!
I just wonder, the update of the generator to generate the expected code, that happens in an other PR ?

@a4z I think you missed the change in src/main/scala/djinni/CWrapperGenerator.scala. That's where the output of the generator has been adjusted.

@a4z I think you missed the change in src/main/scala/djinni/CWrapperGenerator.scala. That's where the output of the generator has been adjusted.

True, somehow I saw it, but at the same time not ... thanks for pointing me to that!


Vergen doesn't respect configured location for build.rs

I notice that even when I set build = ...something in my Cargo.toml, vergen still produces a rerun-if-changed directive pointing to build.rs in the crate root directory. It would be nice if vergen would respect the current crate's configuration.
(See https://github.com/rustyhorde/vergen/blob/master/vergen/src/emitter.rs#L285)

Agree.  Didn't know that configuration existed.

So digging into this some, Cargo does not export the build script information via enviroment variables.  That means I can't detect a non-standard build script path automatically.   I can add the option to configure that path however.


can WebWarp and Webalign have a single shared filetype

It would be great if people could easily switch between webalign and webwarp and the workbench enables this, but the presence of wwrp filetype adds friction. Is it possible that both could use the waln extension and have a shared file?

Good point but we don't have time to prioritize this now as all content types have been created and reported. A change now would imply a lot of extra work.

we already have the waln filetype registered with WebWarp. WebWarp already uses waln as an input, this change is just to the extension placed on the file, and would improve user experience. Also the user would not see the waln or wwrp extension since the workbench hides this.

sweet thats good news, ill demo what i mean in the meeting tomorrow but I think it might be good to allow webwarp to save waln  files.  perhaps the default behaviour of the save button should be to save to the waln file that is currently open

Hmm, ooops, I think I misread the original question.

I think these requests (this one and the one on slack) should rather be revised as workbench-specific support requests. They only make sense in the context of the workbench, so we should call them like that. For example you wouldn't want to name all your images as .png on your hard drive just because software can recognize what's inside in other ways, but it makes sense inside the workbench, as users won't deal with the actual files there unless they really want to.
So yes, it's viable to add workbench-specific behaviors.

Amended :)

WA/WW/LZ: supplying a filename parameter disables renaming operations:

Save As.. button is not displayed at all
Save operation does not check filetype.


upstart missing env vars setup

An oversight caused upstart not allowing to configure env vars for services due to missing configuration in upstart's template. We need to add this support.

#10 solves this.


--rerunfailed only includes suites with failed tests in rerun.xml, which makes it impossible to merge with original.xml

If running a test with 3 suites pybot suite1 suite2 suite3 and a testcase fails in suite 2
pybot --rerunfailed original.xml only includes suite 2 in rerun.xml
Merging original.xml and rerun.xml is no longer possible
rebot --merge original.xml rerun.xml
[ ERROR ] Cannot merge outputs containing different root suites.
Is it possible to make --rerunfailed include the passed suites, without running them in the rerun.xml, so they can be merged to one report?

The problem is that the automatically created name of the top-level test suite changes depending on which suites are run. A simple solution is providing a custom name like --name Example in both runs.


Zero Install custom environment

I want to be able to say:
$ docker run -i -t /ipfs/QmBlah

and get a shell in some special environment.

@whyrusleeping so similar to #1 ?


Add support for frozen literals

Consider removing all typhoeus used in test cases, since it is not safe in frozen literals. https://github.com/typhoeus/ethon/pull/146


add gravitational constant and earch mass

I am adding here the universal gravitational constant and the earth's mass. These allow us to compute the Geopotential for deep atmospheres.

Looks like this wasn't merged and wasn't needed in the last half a year? I'll close it now and re-open when needed.


New ChatUI has Feature Parity with Chatbot UI

[x] Prompt Selection
[ ] Folders work
[ ] Persist Conversations
[x] Search
[x] Hide plugin keys button
[ ] Designer does design things
[ ] Color Palette
[ ] Persona Selection

Notes:

https://carbondesignsystem.com/

Need to support rendering markdown in the output.

Added optional RAG support with the chroma server


update oal example for gpu usage

allowing user to set "cpu": False for GPU training, if available

@LoryWangxx could you reopen the PR?


Remove all viper/cobra dependencies

The viper stuff for command-line args is handy, but not worth the half a billion random dependencies it pulls in.  Let's figure out a cheaper way to do this.  We could parse the toml manually, and probably jam together the ENV and CLI stuff in half a day.
In a very quick test, viper adds 6 megs to a stripped binary, versus about 200k for the toml parser by itself.

This may help: https://www.gmarik.info/blog/2019/12-factor-golang-flag-package/
Looks rudimentary and probably not quite suited to RAIS (no parsing TOML, for instance), but it could be a start


1974 wpml menu fix

Ticket: #1974
Issue
The last WPML language switcher menu item was picking up the prior item's classes
Solution
While we want to use the WP_Post object (as opposed to the Timber-based object) to pull classes from, we need to clear its value to prevent the issue from occurring
Impact
This fixes #1974 (including @mgarabedian related issue). Unlike #2009, this isn't WPML-specific
Usage Changes
None
Testing
I tested locally against a real WPML instance and verified the fix. I don't think new tests are necessary since this removes a bug vs. creating a feature

Coverage increased (+0.003%) to 94.266% when pulling 8b6cea82d6420415181b96c1834aa135ef44cce7 on 1974_wpml_menu_fix into 137aa59069a1cbca2fde8991dd43bd482c186834 on master.

This is ready for your review @palmiak !

@jarednova do we have some access to latest WPML? I only have some older version from some project.

@palmiak yeah, they gave me a dev copy to test against. I'll email you what I've got

Hi @jarednova does this commit fix issue #2020 ?

@roberrrt-s #2020 is resolved by a (pending) upgrade to WPML that will synchronize versions of Twig. Once it's released I'll test to verify and then (very loudly) annouce


provide ability to adjust deployment strategy

default value is choosen in such a way to make kubernetes replace one replica when there are 2 replicas running on 2 distinct nodes.
since these replicas are already using leader election a loss of 1 replica during rollout doesn't pose any issues.
minor: fixed a typo s/defautUpdateStrategy/defaultUpdateStrategy/
fixes: #140

Functionality was already tested as part of linked issue, the defaults and change is checked as below
// new defaults
> k get deploy rbd.csi.ceph.com-ctrlplugin -oyaml | yq -r .spec.strategy
rollingUpdate:
  maxSurge: 0
  maxUnavailable: 1
type: RollingUpdate

// no regression in typo change
> k get ds rbd.csi.ceph.com-nodeplugin -oyaml | yq -r .spec.updateStrategy
rollingUpdate:
  maxSurge: 0
  maxUnavailable: 1
type: RollingUpdate

// when driver spec is updated same is realized in the deployment
> k get deploy rbd.csi.ceph.com-ctrlplugin -oyaml | yq -r .spec.strategy
rollingUpdate:
  maxSurge: 25%
  maxUnavailable: 25%
type: RollingUpdate

@leelavg Can you please reorganize the commit so the first commit holds the API changes (plus generated changes) then a 2nd commit for the changed that are needed to support the new API field in the controller?


Make sure switching SGX_MODE work with just a "make clean"

What this PR does / why we need it:
A make clean didn't clean up some SGX_MODE-sensitive build artifacts which resulted in strange crypto test failures when switching from SIM to HW.  This small patch should fix that ..
Which issue(s) this PR fixes:
This should fix the problem Marcus was facing when testing PR #419
Special notes for your reviewer:
Does this PR introduce a user-facing changes and/or breaks backward compatability?:

maybe remove the changes in go.mod for a clean PR :)

aargh, i actually did do a git show but the page break was so that i saw the complete intended change but not the additional one. go.sum is now removed ...


Don't report error on 504 (timeout)

On an unstable Internet connection (for example in a train) the codeium requests sometimes timeouts returning a status code of 504. If this results in error message being spammed into neovim, it is very annoying.

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


Add @glimpse

Similar to glimpse() in tidyverse, @glimpse macro is like a transposed version of print(). I also added my favorite $ in the front of each row.
For example:
julia> df = DataFrame(
               a=1:3,
               b=1:3,
               f_string=["7", "8","9"]
            );

julia> @chain df @glimpse
Rows: 3
Columns: 3
$ a               Int64           1, 2, 3
$ b               Int64           1, 2, 3
$ f_string        String          7, 8, 9

However, @glimpse does not support invisible return. Therefore, the following action which is supported in R is currently not supported in Julia.
@chain df begin
  glimpse()
  select(1:3)
end

Thanks! Plan to review and merge this and the other commit in the next few days.


Home-Assistant integration

Home Assistant integration that goes both ways:

The Home Assistant loveace UI (dashboard) could benefit from a view that shows information from the tomorrow-app.
The data from Home assistants heating, light, ... systems could be used to calculate CO2 emissions.

Great idea. I'll add a feature for our app to allow for connecting externally.
One issue is that we currently don't get any data on individuals on our app (private-by-design), so the first action point (for now) is tough.


Jadeite handling project that's not committed

This is the front end half of https://github.com/GemTalk/Rowan/issues/378.
If a project isn't committed or otherwise all nice on disk, Jadeite breaks, and it would be nice if Jadeite was a bit more tolerant.
example case:
| proj pkg cls meth |
proj := RwProjectDefinition newForGitBasedProjectNamed: 'Tashkent'.
proj repositoryRootPath: '$ROWAN_PROJECTS_HOME/Tashkent'.
Rowan projectTools create createProjectFor: proj.
Rowan projectTools load loadProjectDefinition: proj.
Rowan projectTools write writeProjectDefinition: proj.
(RowanProjectService newNamed: 'Tashkent') refresh
stack:
a UserDefinedError occurred (error 2318), reason:halt, performOnServer: 'set -e; cd $ROWAN_PROJECTS_HOME/Tashkent; git --git-dir $ROWAN_PROJECTS_HOME/Tashkent/.git --work-tree $ROWAN_PROJECTS_HOME/Tashkent log -1 --pretty=format:%h ' stdout: 'fatal: your current branch ''master'' does not have any commits yet
' failed with status: 32768 errno: 0 errStr: nil
a UserDefinedError occurred (error 2318), reason:halt, performOnServer: 'set -e; cd $ROWAN_PROJECTS_
UserDefinedError (AbstractException) >> _signalWith: @5 line 25
UserDefinedError (AbstractException) >> signal @2 line 47
RwGitTool (Object) >> error: @6 line 7
[] in ExecBlock1 (RwGitTool) >> performOnServer:logging: @24 line 21
RwGitTool >> performOnServer:status: @4 line 4
RwGitTool >> performOnServer:logging: @2 line 4
RwGitTool >> performGitCommand:in:worktree:with:logging: @13 line 9
RwGitTool >> performGitCommand:in:with: @2 line 4
RwGitTool >> gitlogIn:with: @2 line 3
RwGitTool >> gitcommitShaIn: @4 line 5
RwProject >> repositoryCommitId @10 line 8
RowanProjectService >> repositorySha @3 line 3
RowanProjectService >> isSkew @6 line 3
RowanProjectService >> basicRefresh @9 line 6
RowanProjectService >> refresh @2 line 2
RowanProjectService class >> newNamed: @5 line 6
Executed Code @2 line 1
GsNMethod class >> _gsReturnToC @1 line 1

This is part of what Gunnar reported.  RowanClassServer >> update is also implicated.

@LisaAlmarode I'm not sure the second example is reasonable. The failure isn't related to the disk state. Rowan services presume that a class exists in the image, not just a definition. It gives a walkback because there is no class Baijan in the image.
Here's a slightly modified version of your example. If you run it and refresh open a browser, you'll see that Rowan globalNamed: 'Baijan' exists and update works fine. In your example it does not.
	| proj pkg cls projectService |
	proj := RwProjectDefinition newForGitBasedProjectNamed: 'Azer'.
	proj repositoryRootPath: '$ROWAN_PROJECTS_HOME/Azer'.
	Rowan projectTools create createProjectFor: proj.
	pkg := RwPackageDefinition newNamed: 'Baijan'.
	proj addPackage: pkg.

	cls := RwClassDefinition
		newForClassNamed: #Baijan
		super: 'Object'
		instvars: #(name address orderHistory)
		classinstvars: #()
		classvars: #()
		category: nil
		comment: 'a CustomerRecord holds the sales record for a customer.'
		pools: #()
		type: 'normal'.
	pkg addClassDefinition: cls.
	Rowan projectTools load loadProjectDefinition: proj.
	Rowan projectTools write writeProjectDefinition: proj.

I've got a fix for the first example. It will be in 3.0.42. Reassigning back to @LisaAlmarode  for review of second example.

I'm not sure what I did to move this to done last October, but assume I was OK with the fixes, and finally close this.


BCH Stats

Coverage decreased (-0.04%) to 83.255% when pulling 1949924c1c415dfbf99b6010b55269fc2f891d18 on JDonadio:stats/bch into 7a12b1761b3b99bdaee3dacde7c166f37bfc26ad on bitpay:master.

ACK


Add the text "my changes"

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Sorry, this doesn't make any sense and makes things worse. PR should be closed without merging.


Mouse cursor placement error

Depending on the indentation, the mouse cursor is wrong

This occurs whenever there are tabs. In terms of mouse positioning, a tab seems to be treated as multiple characters when it comes to positioning.
This also affects underlining keywords. The underline gets offset depending on the number of tab characters before that point on the line.

This is literally Neovim...


Add a clicking feature to show the planes name when clicking the picture

Card will flip on hover, will need to use JS to add the click to flip. Temporary fix


[Feature][ProcessInstance] Different Tenant can run process's task with different kerberos Keytab

Now  the keytab file is bind in ds backend config file,the datasource of hive can only config the principle.
But actual in production environment different tenant use it's own keytab file and principle to login into hadoop.
Describe the feature
so a new feature request is need to manage the tenant keytab to isolate the priviledge of data access
Describe the solution you'd like

tenant can upload it's own principle.
user of tenant use tenant's keytab to run task.

Additional context

Hi @geosmart , why not using impersonate feature to make this available ?

Hi @geosmart , why not using impersonate feature to make this available ?

I have a question, how do i get login user from SqlTask context ?
@xingchun-chen @dailidong

I have a question, how do i get login user from SqlTask context ?
@xingchun-chen @dailidong

I have a question, how do i get login user that run the SqlTask from task context ?
@xingchun-chen @dailidong
Update:
I think the proccess instance is what we need to auth so we can check running task with proccess instance.

sorry for reply so late,  does login user means tenant ?

I have a question, how do i get login user that run the SqlTask from task context ?
@xingchun-chen @dailidong
Update:
I think the proccess instance is what we need to auth so we can check running task with proccess instance.

sorry for reply so late,  does login user means tenant ?

I think its depend on customer usecase. But apparently, dolphinscheduler doesn't have auth at tenant level, user can select any tenant while saving jobs.
=> i think we shoud have some security feature like: assign tenant + run job with tenant user belong to.

I think its depend on customer usecase. But apparently, dolphinscheduler doesn't have auth at tenant level, user can select any tenant while saving jobs.
=> i think we shoud have some security feature like: assign tenant + run job with tenant user belong to.

When we running sparksql/hive/impala job then we can using impersonate feature to make sure only authen user can run jobs.
Currently i implement this feature for internal usecase.

When we running sparksql/hive/impala job then we can using impersonate feature to make sure only authen user can run jobs.
Currently i implement this feature for internal usecase.

@dailidong  how do you think ?

@dailidong  how do you think ?

Related issue: #4464

Related PR: #4593


Processes never end

I've added this code to launch from my frontend

func worker(category, city, userResult, sec string, w http.ResponseWriter) {
if err := run(category, city, userResult+"/"+sec+".json"); err != nil {
		fmt.Println("Oops:")
		os.Stderr.WriteString(err.Error() + "\n")
	} else {
		fmt.Println("Else!")
		fmt.Fprintf(w, "event: close\n\n")
		w.(http.Flusher).Flush()
		fmt.Println("Closed")
		// panic("error")
		//saveSearchToFireStore(userResult+"/"+sec+".json", email, documentName)
	}
}

http.HandleFunc("/api/google-search", func(w http.ResponseWriter, r *http.Request) {

        //code to get parameters
	worker(category, city, userResult, sec, w,)
}

Everything is working well but for some reasons, I have 5 subprocesses that never end when the search is done.

Any idea how I can end those processes ?

I assume your run function does the initialization of the app etc.
pass the request context there and once this context is done the processes should be cleaned up

@SegFault42 did my recommendation fixed your issue?

This is my run function

It just create the ctx and then run the runFromLocalFile function


Feature request: Show current wall time instead of elapsed time

Description
For long running jobs where individual tasks may take a long time or stall, Elapsed Time becomes hard to interpret. What would be more useful for me is a Current Time indicator. This would allow me, at a glance, to see if the jobs are still completing, or if they have stalled and haven't made progress in hours.
Code
One possible interface:
for job in ProgressBar()(create_jobs(), current_time=True):
   ...

which would output:
...
  1% (177 of 10001) |              | Current Time: 16:39:23 ETA: 1 day, 12:25:58
  1% (178 of 10001) |              | Current Time: 16:39:23 ETA: 1 day, 12:13:50
  1% (179 of 10001) |              | Current Time: 16:39:49 ETA: 1 day, 12:25:10

I can see how this can be useful, I'll add it as soon as I have some time to work on this library again :)
However... I would personally recommend running the actual job in separate processes (multiprocessing for example:  https://github.com/WoLpH/python-progressbar/issues/112#issuecomment-294985286
Or threads :)
Simply have the main process run a while True: bar.update(); time.sleep(1) or something so the progressbar itself won't stall.

Cheers Rick! I use your library everyday, it's saved me so much time writing progress output boilerplate.
Multi{processing, threading} isn't enough for this particular use-case. Here's a minimal working example for the kind of case where Current time would come in handy:
from time import sleep
from random import random
from progressbar import ProgressBar

def some_function():
    sleep(random() if random() > .01 else 1e4)

for i in ProgressBar()(list(range(10000))):
    some_function()

I may have a go at implementing this myself and sending over a PR. I have a couple of other features in mind. For example, given the following:

Q: which of those three terminals is the one running ./foo.py? An easy fix for this would be the ability to add a prefix text to ProgressBar's output, so that I could tell the separate jobs apart by seeing foo.py 1% (177 of 10001).

You could easily do that actually :)
import sys
import progressbar

bar = ProgressBar(widgets=[sys.argv[1], ' :: ', progressbar.ETA()])

Ah fantastic! I really should RTFM. Thank you for your help and code!

Cheers Rick! I use your library everyday, it's saved me so much time writing progress output boilerplate.

You're welcome :) Glad you like it

Multi{processing, threading} isn't enough for this particular use-case. Here's a minimal working example for the kind of case where Current time would come in handy:

I see your point and I agree rewriting the code can be tricky but I still think it's the better solution for your problem. Here's a (theoretically, untested) version of your code with threads:
import threading
from time import sleep
from random import random
from progressbar import ProgressBar

def some_function():
    sleep(random() if random() > .01 else 1e4)

class Worker(threading.Thread):
    def __init__(self, n):
        self.n = n
        self.i = 0
        super(Worker, self).__init__()

    def run(self):
        for i in list(range(self.n)):
            some_function()

worker = Worker(10000)
bar = ProgressBar(max_value=worker.n)
worker.start()
while worker.is_alive():
    bar.update(worker.i)
    worker.join(0.1)

Sorry, I over-simplified. In reality, some_function() will be something time sensitive or resource-bound (e.g.), hence why the calls can't be parallelised

I understand, but give the code a try. It might work for you :)
This code isn't really paralellising, it's simply splitting the user interface from the heavy calculations.


Add more math functions

More math functions should be added to expose more analytic capabilities

Acos(Double) : Returns the angle whose cosine is the specified number.
Asin(Double) - Returns the angle whose sine is the specified number.
Atan(Double) - Returns the angle whose tangent is the specified number.
Atan2(Double, Double) - Returns the angle whose tangent is the quotient of two specified numbers.
BigMul(Int32, Int32) : Produces the full product of two 32-bit numbers.
Cos(Double) - Returns the cosine of the specified angle.
Cosh(Double) : Returns the hyperbolic cosine of the specified angle.
DivRem(Int32, Int32, Int32) - Calculates the quotient of two 32-bit signed integers and also returns the remainder in an output parameter.
DivRem(n,n) : Calculates the quotient of two numbers
Exp(Double) : Returns e raised to the specified power.
IEEERemainder(Double, Double) : Returns the remainder resulting from the division of a specified number by another specified number.
Log(Double) : Returns the natural (base e) logarithm of a specified number.
Log(Double, Double) : Returns the logarithm of a specified number in a specified base.
Log10(Double) : Returns the base 10 logarithm of a specified number.
Max(Byte, Byte) : Returns the larger of two 8-bit unsigned integers.
Min(n, n) : Returns the smaller of two numbers
Pow(Double, Double) : Returns a specified number raised to the specified power.
Sin(Double) - Returns the sine of the specified angle.
Sinh(Double) : Returns the hyperbolic sine of the specified angle.
Sqrt(Double) : Returns the square root of a specified number.
Tan(Double) : Returns the tangent of the specified angle.
Tanh(Double) : Returns the hyperbolic tangent of the specified angle.

Also add Math.PI and Math.E
See https://msdn.microsoft.com/en-us/library/system.math_methods(v=vs.110).aspx

This issue is now in the SQLitePCL.core/raw/pretty repos


Is it possible to allow v2.* for psr/log ?

Overview/summary
Required version of psr/log can be updated so it won't be a conflict with other packages which uses a newer version of psr/log
Motivation

What inspired this enhancement?

When I tried to install this package on a Laravel 8 project, composer couldn't install it because my project uses psr/log 2.^
I downgraded psr/log version on my project since most of packages depends on it also accepts version 1.*
Area

[ ] Add any relevant Area: <area> labels to this issue

Checklist

[x] I have described this enhancement in a way that is actionable (if possible)

Unfortunately, I don't think we'll be able to bump the log version to 2 just yet - we're currently supporting PHP>=7.3, and as per the psr/log packagist page, v2 requires PHP>=8.0, so it would cause conflicts.
Since we won't be able to action this yet, I'll go ahead and close this issue, but please feel free to open more issues if you have any problems.

Hi,
need to use it with laravel 8 and psr 2.0, When it will be possible to use it with php 8? any timeline?


[Elasticsearch 2] Replaced usage of index_analyzer

index_analyzer has been removed in Elasticsearch 2.0. But the new way works in older versions as well so I think it's worth switching to the new way now.
https://www.elastic.co/guide/en/elasticsearch/reference/current/breaking_20_mapping_changes.html#_analyzer_mappings

Merged in d14150eb43030c38dddd58869564ea5b6e521f89. Thanks!

Reverted in 15ef4fcca74e9464b72619679bf5e13e6332ab29, as this causes TestElasticSearchBackend.test_query_analyser to fail on Python 2.7. (This failure wasn't shown in the checks here, because ES tests are set to allow-failures to avoid noise from false positives.)


schema/schemaspec/hcl: fix list of ref encode

Just realized we both fixed the same bug
https://github.com/ariga/atlas/pull/219/files


Package updates -- main

Summary and Scope

package version updates

Issue Type

updates

Prerequisites

[x] I have included documentation in my PR (or it is not required)
[x] I have tested this by building an image using this branch HEAD, or this is does not pertain to the node-images pipeline.

Idempotency
yes
Risks and Mitigations
None known

/backport lts/csm-1.5


Notifications during working hours

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
I would like to receive notification only during working hours.
During nights, my services are down for backup. I don't whant to receive notifications.
✔ Solution
In the Monitor screen, add 2 fields to configure a time window where notifications will not be sent.
Or the oposite, a time window where notifications will be sent.
❓ Alternatives
No response
📝 Additional Context
No response

I would prefer a more detailed notification rule structure, that could be built on an internal IFTTT.

That's actually the most-needed feature I'm waiting for -- I also do run nightly backups which of course trigger alarms when a service goes down for the (planned) backup.

I would need that for my company too, for the same cause (backups)

+1
Could be usefull to have "service time" per notification type.
eg: Only SMS on daytime, or another SMS number only in nightime.

确实有这种需要,比如公司内网项目,通过ddns映射到外网,每天重启路由器时,公网ip会随之变更,这个时候就会收到预警
希望加入工作静默时间的模块,在指定时间段区间不去请求和预警

+1

Maintenance Mode will be available in 1.19.0


Cannot find name 'Fluxxor' && 'Import Fluxxor' - multiple conflicting Intellisense detections

This is on a Mac.

Perhaps it is the * as Fluxxor?
import * as Fluxxor from 'fluxxor'

@jwulf would you be able to share a folder with me that shows the behavior.

Just encountered this also.
The code runs fine on tsc but error on vscode (ts language service). Here is how to reproduce:
// s.ts
export class Foo {}

// m.ts
import { Foo } from './s';
declare namespace Foo { };
export = Foo;

// c.ts
import * as Foo from './m';

console.log(Foo);

Reference:
https://github.com/typed-typings/npm-any-promise/pull/1

I tested this on Windows.
Code: 1.0  :tada:
Also tested with TypeScript@next using settings.json/typescript.tsdk: 1.9.0-dev.20160415

@unional thanks for the code test case. I am moving this to TS since the seems to be caused by the reexport in m.ts

This issue was moved to Microsoft/TypeScript#8140


Separate Configs

o___O

Ah ok. Makes sense. Can the directory names not be a lot be a lot shorter. Just 'react-native', 'react' etc?

What if they decide to change their defaults?

Any kind of review process go into this?...


added readme file for flask sample

@Chrisyhjiang looks like @raphaeltm already put in a README file. Is this PR still relevant?

@Chrisyhjiang sorry I should have let you know I did that. I did it when Prakash pointed out it was missing. Just wanted to get it out there asap.

@Chrisyhjiang unless there's something you think we should add to the existing readme, I think we can mark this as closed without merging.


feat: let micro handle the response encoding

micro is able to encode various data formats like: text, json, buffer and streams
Delegating the encoding of the response to micro opens up the use-cases where micro-post is applicable and also reduced redundancy of encoding implementations.
This branch includes an extension with which it is possible to pass a default micro RequestHandler as response handler function, profiting too of micros encoding capabilities.
Using the test suite I confirmed these changes are backwards compatible.

Thank you @zetaron!!


[Reminder] Add start-bot information to the README.md file

Add information about the start-bot.bat to the README.md file so even users with no knowledge about coding can use the bot.
Maybe add a gif that shows how to use the file?

Added.


Meshes change when hitting play

Can you confirm to us if this still happens? Otherwise we will close this issue


ogre2: explicitly request OpenGL 3.3 core profile context.

This change raises system requirements as it uses GLX 1.3 API and a GLX extension. Modern systems like Ubuntu bionic and focal satisfies these without issue.
Tested on bionic without GPU (QEMU) and focal with GPU.
Bug Report
Fixes issue #128 and similar issues.
Summary
Newer GLX APIs are used to explicitly request core profile GL context that Ogre2 uses.
Reference:
https://www.khronos.org/opengl/wiki/Tutorial:_OpenGL_3.0_Context_Creation_(GLX)
Reproduce:
Run ign gazebo shapes.sdf with the default ogre2 renderer on systems with non-AMD (e.g. Intel) mesa GL drivers or in situations where the hardware GPU is not accessible (e.g. QEMU VM).
The original code has a decent chance to fail due to it attempting to create OpenGL context in compatibility profile and the GL version of context created would be below the required 3.3.
Checklist

[X] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[X] codecheck passed (See
contributing)
[X] All tests passed (See
test coverage)
[ ] While waiting for a review on your PR, please help review
another open pull request
to support the maintainers

Note to maintainers: Remember to use Squash-Merge

done


scripts: Add destroy-storage option to reset tool

This will help for cleaning  volume

Reintroduce cluster_clean removed by @ktsakalozos (8173dce3)

Relate-to: https://github.com/ubuntu/microk8s/issues/488
Change-Id: I04a13f060b1c4beda9dd5006c435e5b2be12c385
Signed-off-by: Philippe Coval<EMAIL_ADDRESS>

Will be rebased once https://github.com/ubuntu/microk8s/pull/626 is merged


[Bug]: Backward incompatibility on use of label in CollectionPreferences.

Browser
Chrome, Safari, Firefox, Edge
Package version
v3.0.782
React version
18.1
Description
Resent change to https://github.com/cloudscape-design/components/commit/a5defd9e11e92cbe126a6093da15c80c6ba2b6c7#diff-9a4626222e43ed6f71fb6aa5a4fc8bff1727c66603314abd4888c51c55025741R65 have broke rendering of Preference panel due to new filtering logic where it assumes that label is string which is based on the interface (my assumption)
export interface ContentDisplayOption {
       id: string;
       label: string;
       alwaysVisible?: boolean;
   }

The issue is that we had use ReactNode as label before and since the current code assumes it's string it throws and error.
TypeError: e.label.toLowerCase is not a function
    at index.tsx:65:22
    at Array.filter (<anonymous>)
    at index.tsx:64:60

Source code
https://github.com/cloudscape-design/components/commit/a5defd9e11e92cbe126a6093da15c80c6ba2b6c7#diff-9a4626222e43ed6f71fb6aa5a4fc8bff1727c66603314abd4888c51c55025741R65
const sortedAndFilteredOptions = useMemo(
   () =>
     getSortedOptions({ options, contentDisplay: value }).filter(option =>
       option.label.toLowerCase().trim().includes(columnFilteringText.toLowerCase().trim())
     ),
   [columnFilteringText, options, value]
 );

Reproduction
import styled from "styled-components";
// Example of ContentDisplayOption
export interface ContentDisplayOption {
  id: string;
  label: string;
  alwaysVisible?: boolean;
}

function App() {
  const options = [
    {
      id: "first",
      label: "First Label",
      alwaysVisible: true,
    },
    {
      id: "first",
      label: (
        // ReactNode passed here

          ALSO LABEL

      ) as React.ReactNode,
      alwaysVisible: true,
    },
  ] as ContentDisplayOption[];
  return (

      {options.map((opt) => {
        return (

 Id: {opt.id}

 Label: {opt.label}

 Visible: {opt.alwaysVisible ? "Yes" : "no"}

        );
      })}

  );
}

export default App;

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems

Hi Nikolay,
per our API label accepts only string, there is no backward incompatible change since we didn't change the API.
your previous behaviour should still work though, you just cannot use the new filtering feature unless you follow our API and use strings for label

actually, we'll fix it anyway.


Issues with links to Reddit pages

Generally, relative links (e.g. <a href="/r/toolbox">) work fine for linking to places on Reddit from within the extension. However, if we're on mod.reddit.com, none of those redirects work, so we can't make a link for that. We need to standardize the usage of relative links in the extension so things don't break on mod.reddit.com but still respect the user's subdomain otherwise.
Brainstorm: maybe we could add a utility function for it like this?
TBUtils.link = relativeURL => TBUtils.isNewModmail ? `https://www.reddit.com${relativeURL}` : relativeURL;
// called like:
$(`<a href="${TBUtils.link('/r/toolbox/about/modqueue')}">`)

Looks like an elegant solution to me.


Refactor integration tests

Need to improve tests in terms of reusability and remove duplicate code

@kdhrubo I'm currently working on it


Design System Website: "All Components" page

Acceptance Criteria:

Just the Card view for the MVP
Sort capability

Alphabetical
Categories (alphabetical within categories)

Structure and Navigation

Microsite header
Site header
Footer
Hero banner
Pagination
Tabs
Side menu
Form stepper

Content layout

Container
Table
Details
Accordion
List (change from current name "List elements" to "List" move from utilities section)

Feedback and alerts

Badge
Notification banner
Circular progress
Callout
Modal
Tooltip

Inputs and Actions

Checkbox
Radio
Button
Button group
Icon button
Form item
Dropdown
File upload
Text area
Input
Chip

Utilities

Block
Divider
Grid
Spacer

Figma Design
Content

Closing as this has been completed


<EMAIL_ADDRESS>hash check failed

$ scoop install doom-retro
WARN  Scoop uses 'aria2c' for multi-connection downloads.
WARN  Should it cause issues, run 'scoop config aria2-enabled false' to disable it.
WARN  To disable this warning, run 'scoop config aria2-warning-enabled false'.
Installing 'doom-retro' (4.9.2) [64bit] from games bucket
Starting download with aria2 ...
Download: Download Results:
Download: gid   |stat|avg speed  |path/URI
Download: ======+====+===========+=======================================================
Download: 4e3e30|OK  |   4.5MiB/s|C:/Users/XXX/scoop/cache/doom-retro#4.9.2#https_github.com_bradharding_doomretro_releases_download_v4.9.2_doomretro-4.9.2-win64.zip
Download: Status Legend:
Download: (OK):download completed.
Checking hash of doomretro-4.9.2-win64.zip ... ERROR Hash check failed!
App:         games/doom-retro
URL:         https://github.com/bradharding/doomretro/releases/download/v4.9.2/doomretro-4.9.2-win64.zip
First bytes: 50 4B 03 04 14 00 00 00
Expected:    5e9c4e9a7aa0129d46119924691f73d23e3bc2ecc80adbdf049883d39a089b53
Actual:      d6b5b691798cce9ba9023b1554415b73715b377cf7d9df651bba27c025a19780

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/Calinou/scoop-games/issues/new?title=doom-retro%404.9.2%3a+hash+check+failed

username in the filepath above X'd out.


Update readme

Sorry @xeodou I don't fully get why that change is needed, can you explain the reason?
RS256 is RSA SHA256, saying RSA RS256 is redundant, isn't it?

yes, @ziluvatar It's my mistake. Thanks for your reply, I'll close the PR.


Unsupported Web3 Methods

(base) ➜  mev-inspect-py git:(main) ✗ ./mev inspect 13850154
Inspecting block 13850154
Skipping virtualenv creation, as specified in config file.
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/flashbot/.local/lib/python3.9/site-packages/click/core.py", line 1137, in __call__
    return self.main(*args, **kwargs)
  File "/home/flashbot/.local/lib/python3.9/site-packages/click/core.py", line 1062, in main
    rv = self.invoke(ctx)
  File "/home/flashbot/.local/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/flashbot/.local/lib/python3.9/site-packages/click/core.py", line 763, in invoke
    return __callback(*args, **kwargs)
  File "/app/mev_inspect/concurrency.py", line 18, in wrapper
    loop.run_until_complete(f(*args, **kwargs))
  File "/usr/local/lib/python3.9/asyncio/base_events.py", line 647, in run_until_complete
    return future.result()
  File "/app/cli.py", line 45, in inspect_block_command
    await inspector.inspect_single_block(
  File "/app/mev_inspect/inspector.py", line 56, in inspect_single_block
    return await inspect_block(
  File "/app/mev_inspect/inspect_block.py", line 68, in inspect_block
    await inspect_many_blocks(
  File "/app/mev_inspect/inspect_block.py", line 105, in inspect_many_blocks
    block = await create_from_block_number(
  File "/app/mev_inspect/block.py", line 31, in create_from_block_number
    block_timestamp, receipts, traces, base_fee_per_gas = await asyncio.gather(
  File "/app/mev_inspect/block.py", line 73, in _find_or_fetch_block_receipts
    return await _fetch_block_receipts(w3, block_number)
  File "/app/mev_inspect/block.py", line 110, in _fetch_block_receipts
    receipts_json = await w3.eth.get_block_receipts(block_number)
  File "/home/flashbot/.local/lib/python3.9/site-packages/web3/module.py", line 72, in caller
    result = await w3.manager.coro_request(method_str,
  File "/home/flashbot/.local/lib/python3.9/site-packages/web3/manager.py", line 203, in coro_request
    return self.formatted_response(response,
  File "/home/flashbot/.local/lib/python3.9/site-packages/web3/manager.py", line 168, in formatted_response
    raise ValueError(response["error"])
ValueError: {'code': -32602, 'message': 'Invalid params', 'data': 'invalid type: integer `13850154`, expected a string at line 1 column 8'}
command terminated with exit code 1

In the poetry file, it shows it is using web3=5.23 while I tried this version locally and there's no method called get_block_receipts. Has anyone met a similar problem before?

I've seen this before, and it is to do with the RPC you are using. Try using an ANKR RPC! :)

https://github.com/flashbots/mev-inspect-py/issues/319

#319

Hi ser, thanks for poiting it out. It went smoothly with ankr rpc :)
I am now testing it with my RETH node with trace API enabled. The strange thing is that the error seems have to do with get_block_receipts method in web3.eth module?

#319

Solved this problem by modifying the block number from int to dex in block.py
async def _fetch_block_timestamp(w3, block_number: int) -> int:
    block_json = await w3.eth.get_block(hex(block_number))
    return block_json["timestamp"]

async def _fetch_block_receipts(w3, block_number: int) -> List[Receipt]:
    receipts_json = await w3.eth.get_block_receipts(hex(block_number))
    return [Receipt(**receipt) for receipt in receipts_json]

async def _fetch_block_traces(w3, block_number: int) -> List[Trace]:
    traces_json = await w3.eth.trace_block(hex(block_number))
    return [Trace(**trace_json) for trace_json in traces_json]

Is it standard practice to pass block numbers as integers, as used in the codebase?


Publish 12/31/2020 10:30 AM PT

Docs Build status updates of commit d1ab85d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/addsadministration/New-ADServiceAccount.md
:white_check_mark:Succeeded
View (win10-ps)

docset/winserver2012-ps/activedirectory/New-ADServiceAccount.md
:white_check_mark:Succeeded
View (winserver2012-ps)

docset/winserver2012r2-ps/addsadministration/New-ADServiceAccount.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


--symbols and referenced project

Would it be posable to pass any specified symbols onto referenced projects?
I’m planning on having #if FABLE blocks in my common project. I can compile this project to javascript/.net no problems (using --symbols). When I try to compile the child project to javascript it blows up because FABLE isn’t defined - It tries to use json.net which is swapped out for json.stringify.
I've tried to create a Fable build configuration with FABLE symbol defined but is there anyway of using this from the fable compiler?

After adding the --refs option I was considering to allow multiple project compilation but this proved to be quite complex as I didn't have it in mind at the beginning :/
What I did change is that now you can put fableconfig.json in any directory (before it had to be in the same directory where fable was run). So if you want to compile multiple projects at once my advice is to put a fableconfig.json file in the folder of each project and then create a batch or bash script like:
fable path/to/project1/
fable path/to/project2/

Is it possible to see the project you're working on? Or can you post the steps you take to do the compilation? I'd like to understand better why can't you easily define FABLE symbol in the child project. Also, when you talk about the common project is this a project reference (like this) or a shared project?

I've posted up a demo project to https://github.com/davidtme/FableTest

Thanks for that! Now I understand the problem better :)