sync

Why are these changes needed?

Related issue number (if applicable)

Checks

[ ] I've run format.sh to lint the changes in this PR.
[ ] I've included any doc changes needed.
[ ] I've made sure the relevant tests are passing (if applicable).

sync


⚠ Bofrost.com has degraded performance

In bb91e7d, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 1197 ms

Resolved: Bofrost.com performance has improved in 3b6b4da.


Setting window minimized on app startup gives error on app closure preventing graceful app termination

Software Versions

Python: 3.9.4 (tags/v3.9.4:1f2e308, Apr  4 2021, 13:14:17) [MSC v.1928 32 bit (Intel)] on win32
OS: Windows 10 Pro (21H1 version)
Kivy: 2.0.0
Kivy installation method: pip install kivy[full]

Describe the bug
Normally, Kivy closes gracefully with no traceback or error information on the console. But recently, I added a functionality that starts up the kivy app with window minimized: There are two ways to do it (either gives out an error when app is closed):

calling the Window.minimize() function
Setting kivy Config like this: Config.set('graphics', 'window_state', 'minimized')

No problem on window being minimized, but when I close the app using any of the above methods enabled, Kivy consoles out this error message:
 Traceback (most recent call last):
   File "C:\Users\User\Desktop\Project\main_app\main.py", line 501, in <module>
     mainApp.run()
   File "C:\Users\User\Desktop\Project\main_app\.env\lib\site-packages\kivy\app.py", line 950, in run
     runTouchApp()
   File "C:\Users\User\Desktop\Project\main_app\.env\lib\site-packages\kivy\base.py", line 584, in runTouchApp
     stopTouchApp()
   File "C:\Users\User\Desktop\Project\main_app\.env\lib\site-packages\kivy\base.py", line 625, in stopTouchApp
     EventLoop.close()
   File "C:\Users\User\Desktop\Project\main_app\.env\lib\site-packages\kivy\base.py", line 186, in close
     self.stop()
   File "C:\Users\User\Desktop\Project\main_app\.env\lib\site-packages\kivy\base.py", line 198, in stop
     provider.stop()
   File "C:\Users\User\Desktop\Project\main_app\.env\lib\site-packages\kivy\input\providers\wm_pen.py", line 114, in stop
     SetWindowLong_WndProc_wrapper(self.hwnd, self.old_windProc)
   File "C:\Users\User\Desktop\Project\main_app\.env\lib\site-packages\kivy\input\providers\wm_common.py", line 112, in _closure
     oldAddr = func(hWnd, GWL_WNDPROC, cast(wndProc, c_void_p).value)
 ctypes.ArgumentError: argument 3: <class 'TypeError'>: wrong type

Expected behavior
Close gracefully without error and traceback
To Reproduce
Here is the code to reproduce this issue:
from kivy.app import App
from kivy.uix.button import Button
from kivy.config import Config
Config.set('graphics', 'window_state', 'minimized')

class TestApp(App):
    def build(self):
        return Button(text='Hello World')

TestApp().run()

Additional context
Here is the link to the same StackOverflow question: Kivy: Running the Kivy app with window minimized triggers an error on app closure. Why?

I'm not sure on the implications of this change fully, so I'm afraid to post a PR. If any Kivy devs are interested in a simple fix, it just looks like a simple issue of None being passed to a ctypes function. Changing wm_common.py from:
def SetWindowLong_WndProc_wrapper_generator(func):
    def _closure(hWnd, wndProc):
        oldAddr = func(hWnd, GWL_WNDPROC, cast(wndProc, c_void_p).value)
        return cast(c_void_p(oldAddr), WNDPROC)

    return _closure

to
def SetWindowLong_WndProc_wrapper_generator(func):
    def _closure(hWnd, wndProc):
        wndProc = cast(wndProc, c_void_p).value
        if wndProc is None:
            return cast(c_void_p(), WNDPROC)

        oldAddr = func(hWnd, GWL_WNDPROC, wndProc)
        return cast(c_void_p(oldAddr), WNDPROC)

    return _closure

fixes it for me and doesn't appear to cause any side effects.

Implementing this from Kivy 2.2.0 raises this exception for me:
\lib\site-packages\kivy\lang\builder.py", line 925, in _build_canvas
    setattr(instr, key, value)
  File "kivy\graphics\vertex_instructions_line.pxi", line 790, in kivy.graphics.vertex_instructions.Line.width.__set__
kivy.graphics.vertex_instructions.GraphicException: Invalid width value, must be > 0

This issue comes up when I try to import the Window module after this:
Config.set('graphics', 'window_state', 'hidden')
However only in my main application. On a scratch file, it works just fine.
Later in my main application, I am modifying some Window properties, but I set the window state to visible right after importing Window, so I have no idea why this error is coming up, and why it isn't coming up in the scratch file.
I just want the kivy window(s) to not open when starting the app up. It's weird because I'm pretty sure it only happens on Windows, not Mac.


docs: clarify intended purpose of branches

Based on a discussion in sprint meeting. Please LMK if this doesn't help, can close instead of merging.

I am more in favour of bringing main and v1 in sync, and then establishing a clear process for branch management using main as the canonical branch.

I'm also in favor of that but until then I thought this note in the README may help maintainers b/c the current branch management is confusing.


Remove all storage snippets

This deletes all storage samples. This is to accommodate moving them to java-storage (https://github.com/googleapis/java-storage/pull/1258).
Do Not Merge: This shouldn't be merged until https://github.com/googleapis/java-storage/pull/1258 and all sample references in the documentation have been changed to point to those.
I"m only creating this PR now so that SnippetBot can give me a convenient list of documentation pages I need to update

I addressed the two issues snippet bot found (cl 436575556). snippet bot isn't picking up that they're fixed, but they are! this should be good to go


Update sharable link when alpha changes

I have not tested but this should work.

@9t8
Thanks you for the PR! I have tested and it works well :)


CMake build task does not show progress

Brief Issue Summary
Executing CMake: Build command in VSC, results in progress being displayed on bottom VSC bar showing progress (and output in OUTPUT tab).
Executing Task with cmake build does not show progress (and output in terminal).
Is there a way, how to achieve same behavior from task?
Task
{
	"label": "build",
	"detail": "CMake template build task",
	"type": "cmake",
	"command": "build",
	"targets": [
		"all"
	],
	"group": {
		"kind": "build",
		"isDefault": true
	},
	"problemMatcher": [],
	"options": {
		"cwd": "${workspaceFolder}/build"
	},
	"presentation": {
		"reveal": "silent",
		"panel":"dedicated"
	},
}

by removing this setting in your build task "reveal": "silent" you should be able to see the Build progress in the terminal.
for example:

Yeah, but this way, I have to always manually switch to something else manually.


[Minor] Update hashbrown to 0.14

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks @Dandandan !


exclude submitter information in solr indexing

References
Add references/links to any related issues or PRs. These may include:

Fixes #9660

Description
add configurable option to hide the submitter name being indexed in solr for archived items.
Instructions for Reviewers
after apply-ance of the change the item which has been findable by the submitter surname or firstname should not be findable in the public search anymore (see issue for example). This only affects archived items; the submitter information is still indexed for InProgressSubmissions (workspace, workflow items) etc...
List of changes in this PR:

added configuration discovery.index.item.submitter.enabled: when disabled, the submitter information is not written to the file at all.
added configuration discovery.index.item.submitter.name: this sets the anonymized name for the submitter (e.g. N.N., John Doe, Max Mustermann or any other random name) which is written to the submitter facet instead of the original submitter Name
added two Integration Tests:

proving that when the configuration for the submitter name is set, the submitter name is written from the configuration and the original submitter does not appear in these fields
proving that when the configuration for the submitter indexing is disabled, no field starting with submitter is written to the indexed solr documents

Include guidance for how to test or review your PR. This may include: steps to reproduce a bug, screenshots or description of a new feature, or reasons behind specific changes.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome).
However, reviewers may request that you complete any actions in this list if you have not done so. If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[ ] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[ ] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[ ] My PR includes details on how to test it. I've provided clear instructions to reviewers on how to successfully test this fix or feature.
[ ] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[x] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

Please rebase this on main.  The IT failures you have in this PR were fixed on main in #9691 and are unrelated to your PR.

Hi @nwoodward ,
thanks for your tests and your note. I've added some comment for the properties in the configuration file. It should also be mentioned in the upcoming https://wiki.lyrasis.org/display/DSDOC9x/Discovery configuration reference for the next Release

Great. I tested again with the new configuration, and everything still works well. One thing I noticed this second time is if I set the discovery.index.item.submitter.name to a placeholder name and then run bin/dspace index-discovery -b then the indexing process will refresh the submitter information of old archived items to the new placeholder name. The reverse is also true in that if I had the placeholder name in the submitter metadata and then commented it out in the configuration and reindexed everything then the original item submitter name would reappear. I imagine this is expected behavior, but I figured I should mention it.


add support for mounting log folder

When using Docker, you should not have logs into the containers as they are identical with what is already available in the stdout.
To avoid the log to be created by the service, you should add this to your mycroft.conf:
  "logs": {
    "path": "stdout"
  }


Huge error on Prediction (NER)

I created my own vocabulary and tags, I ran the code with this parameters:
{
"batch_size": 2,
"buffer": 15000,
"chars": "vocab.chars.txt",
"dim": 300,
"dim_chars": 100,
"dropout": 0.3,
"epochs": 25,
"filters": 50,
"glove": "glove.npz",
"kernel_size": 3,
"lstm_size": 100,
"num_oov_buckets": 1,
"tags": "vocab.tags.txt",
"words": "vocab.words.txt"
}
After 8 hours of training I got my results:
Saving dict for global step 9534: acc = 0.96229786, f1 = 0.8532131, global_step = 9534, loss = 36.943344, precision = 0.8494137$
Saving 'checkpoint_path' summary for global step 9534: results/model/model.ckpt-9534
But when I tried to predict with the same test dataset, the model is not predicting, for example in test we have Bora B-NAME B-NAME and when I tried to pass the same sentence, the algorithm is not predicting that entity. The problem is the algorithm is saying that the accuracy is 0.962297 and f1 0.8532131, but in reality is detecting only 38%, so the accuracy is not 0.85 is only 0.38.
What could be the problem?
Do you have an idea?
Thank you!

See my answer in the sequence_tagging repo (which is the version of the code you're using I assume).


implement byte type

Implement the byte type and add to coercion logic.

Implement the byte type and add to coercion logic.

Maybe we should have uint8 and int8?

We adopted int8 and uint8 ini zng-beta and abandoned byes so this is done.  bytes[k] will give a uint8.

Per the previous comment, the change in #1394 was to remove byte from the Primitive Types in the ZNG spec and use uint8 in contexts where we'd previously envisioned byte would be applicable. This is not to be confused with the bytes (plural!) type that remains.
Also per the comment above, the ability to invoke the index operator in ZQL such that "bytes[k] will give a uint8" has yet to be addressed. That is tracked in #1421.


update base image

update base image to fix vulnerability problem in work image

/assign @pmorie

/assign @deads2k

/retest

/approve
/lgtm


[BEAM-1187] Improve logging to contain the number of retries done due to IOException and unsuccessful response codes.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify.
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @jkff

Changes Unknown when pulling f20a47c450f5a6b05cbb410506b233fee71dbb0c on lukecwik:beam1187 into ** on apache:master**.


Support MySQL Database Backend

The database backends are all based on SQL queries "compiled" using SQLC to Go code. So it shouldn't be too difficult to add another support for MySQL on top of Postgres and SQLite. Would anyone be interested in this? I ask because the purpose of the current backends are:

SQLite: Embedded local transient use
Postgres: Hosted scalable use

MySQL would be an alternative to Postgres, but is it as popular? Are there popular database services based on MySQL? For example, Planetscale is one (https://planetscale.com/).

Very interested

Sounds good, can I ask what the situation is that needs or prefers MySQL? Just curious to learn more!


Incomplete nov2016.zip_codes table

There are only 33k zip codes out of approx 43k.
Need to find more comprehensive source

So, just to give some context, there are approx. 9K "PO BOX" zip codes.  The initial load of data didn't appear to contain those values.  Additionally, it was admittedly an older dataset., so there may be some newer "Standard" ZIP Cods which were added after the dataset was compiled.  We have purchased a copy of recent data from a supplier, and I'll be loading that data into the existing table to fill in the gaps.

@misteranderson if you could  load it on carpool_v2.0_live and carpool_v2,0_test that would be great!
I'll get a dump from one of them to include in the fresh db creation script.

What is the plan for the missing latitude/longitude coordinates for the 10,050 zip  codes that were added ?
Now the zip code can be found (so the user records don't get marked as Failed anymore), but the distances calculated with nov2016.distance() will be NULL, meaning they will never be included in any match. This is more insidious, because the system would appear to work properly, instead of identifying the records as Failed.
If we can't add the missing lat/lon info into zip_codes quickly, I'll have to modify the matching code to catch this exception too.

My thought is to use the Google geocoding API to geocode the remaining ZIPs programmatically; There could be a couple of approaches - geocode on demand (if we don't know the info, then look it up and store it back into the DB) - or batch geocode, for a few dollars (exceeding the free API threshold), I can have enough requests to get the remaining ZIPs filled in probably later tonight.

Turns out, I was able to fix it by just falling back (for now) to the slightly less precise value supplied in the data feed, but I had done my coalesce() wrong on the inintial insert - the data was '' instead of NULL.
Fixing with
UPDATE nov2016.zip_codes
SET
latitude  = CAST(  COALESCE(NULLIF(precise_latitude  ,''),src.latitude)  AS character varying(10))
,longitude = CAST(  COALESCE(NULLIF(precise_longitude ,''),src.longitude) AS character varying(10))
,latitude_numeric = CAST( COALESCE(NULLIF(precise_latitude ,''),src.latitude)  AS numeric)
,longitude_numeric= CAST( COALESCE(NULLIF(precise_longitude,''),src.longitude) AS numeric)
FROM
import.zip_code_database_commercial as src
WHERE
nov2016.zip_codes.zip = src.zip
;

Please let me know if this is now resolved.
At the same time, in the test db, I brought in the TimeZone as well..
carpool_v2.0_test=# alter table nov2016.zip_codes add column timezone character varying(50) null default '';
ALTER TABLE
carpool_v2.0_test=# UPDATE nov2016.zip_codes
carpool_v2.0_test-# SET
carpool_v2.0_test-# timezone = src.timezone from
carpool_v2.0_test-# import.zip_code_database_commercial as src
carpool_v2.0_test-# WHERE
carpool_v2.0_test-# nov2016.zip_codes.zip = src.zip
carpool_v2.0_test-# ;
UPDATE 42003

Looks good to me.


There's been layer exception spam lately

Description

Reproduction

Screenshots

also this thing:

Additional context
Happens when I ghost around, sometimes this comes with PVS not loading stuff properly

Cuffable seems to require sprite changes now where it didn't before; I had cuffable monkeys crashing the game when I merged the handcuff ECS stuff because they didn't have a layer


Generate types from local file

Requirements
PocketBase also allows you to export the schema for your collections to a pb_schema.json file. We could also use this as the "source of truth" for the collections instead of loading it directly from the database. While this file could be outdated it doesn't require you to set the admin credentials at all for public collections. This makes it easier to collaborate with other people, publishing only this schema file in your repository.

Closed via 4c65f58cf4eaac2e75e1f08dd68fddf0df367ef1


Ignore the .idea directory

What has been done

Added the .idea directory to .gitignore

Hey Bobby, thanks for sending this in. This, however, belongs in a global .gitignore file. .gitignore for projects are for project-specific ignore rules.
Here's more info: https://help.github.com/articles/ignoring-files/#create-a-global-gitignore
And here's my own file: https://github.com/driesvints/dotfiles/blob/master/.gitignore_global

Check! Thanks for pointing it out ;)


File comparisons don't work over SMB protocol and across folders

Please find attached my implementation of "Select Left" and "Compare" scripts which works across different folders and via Gnome's SMB pseudo paths. It would be great to have this working in your scripts. I had to rename the files to a .txt extension becuause GitHub is a Windows shop.
Compare.txt
SelectLeft.txt

Hi @snovotill. Thanks for your request.
I have fixed this in the last version. Could you test it again?
(Commit: https://github.com/cfgnunes/nautilus-scripts/commit/67fef95c9f283cfd408ea11dc0a08480745d5cce)

Hmm, I still get pop-up "You must select files or directories!" when I try to compare two files or two folders which are on a remote SMB share, via Scripts ‣ Directories and files ‣Compare with diff. On the other hand, the two script which I attached upon opening this ticket do work. I'm on Debian Trixie with Gnome-Wayland, but currently not quite Gnome 46 just yet. I'll debug this when Gnome 46 arrives, since the current Nautilus Scripts "API" is weirdly buggy. Until then, be well and stay relentless!

Hi @snovotill. Thanks for your feedback.
I have fixed this in the last version. Could you test it again?
Now, I've tested on Debian Trixie with Gnome-Wayland using samba server (local and remote).

Okay maestro, It's working perfectly now! Thanks for sharing =)
On another topic, I've attached a script for deleting files out of a PDF in case you're interested.
DeletePdfPages.sh.txt
ExtractPdfPages.sh.txt

Thanks for sharing the scripts.
About the scripts "DeletePdfPages.sh.txt" and "ExtractPdfPages.sh.txt" I use my scripts:

Document PDF -> PDF combine, split -> Split into single-page PDFs
and after:
Document PDF -> PDF combine, split -> Combine multiple PDFs

You sure do! And those are two scripts which I did not have to write myself, along with MANY others☺ so thanks!
...and if you implement the "select left" with "compare" (try the examples which I attached) then you will have achieved GOD-level on the topic of Nautilus (and other) Scripts.


chore: bump kind node image to v1.23.5

Fixes #429
Recent E2E tests fail with:
2022-04-21T11:56:47.995634245Z stdout F {"level":"fatal","ts":"2022-04-21T11:56:47.993Z","logger":"vsphere-source-webhook","caller":"sharedmain/main.go:313","msg":"Version check failed","commit":"8479058","error":"kubernetes version \"1.21.1\" is not compatible, need at least \"1.22.0-0\"

Proposed Changes

:bug: Bump and explicitly set kind node image to v1.23.5 as required by most recent deps

Pre-review Checklist

[ ] At least 80% unit test coverage
[ ] E2E tests for any new behavior
[ ] Docs for any user-facing impact

Release Note

Kind has been bumped in Upstram https://github.com/kubernetes-sigs/kind/releases/tag/v0.12.0. Do you mind pinning Node in upstream as well? https://github.com/knative/test-infra/blob/main/images/prow-tests/Dockerfile

Kind has been bumped in Upstram<EMAIL_ADDRESS>(release). Do you mind pinning Node in upstream as well? https://github.com/knative/test-infra/blob/main/images/prow-tests/Dockerfile

Not sure I understand what the ask is here, can you please elaborate again?

Don't worry about it. I will take care of it :+1:


How to serve multiple NLU models from an NLU-only server in 1.0.x?

Rasa version: 1.0.4
Issue:
Hi,
Thanks for the awesome work you guys are doing. We really appreciate the effort put in the recent releases of Rasa 1.0 and Rasa X.
I'm trying out 1.0.4 (upgrading from 0.14.4) and I am realizing that an important feature our deployments are relying on seems to have changed in 1.0.x: the ability for the NLU server to load and serve multiple models simultaneously.

In 0.14.4, we can serve multiple models by running the NLU server with a path pointing to a folder (e.g. ./models) containing multiple models using:
python -m rasa_nlu.server --path models
Then multiple models can be called (and loaded) by the NLU server by using a POST on /parse endpoint:
{"q": "Some text", "project": "my_project", "model": "nlu_model_01"}
{"q": "Some text", "project": "my_project", "model": "nlu_model_02"}

In 1.0.x, I can serve a single model using the command:
rasa run --enable-api -m models/nlu-20190603-135554.tar.gz
but it's unclear to me how serving multiple models can be achieved. The documentation  seems to mention a notion of tag: default@latest but I'm not sure if that provides the same functionality.

Note: it looks like the change was done through PR #3373.
Any guidance on this would be much appreciated.
Thanks!

correct, we took that functionality out, please see this issue https://github.com/RasaHQ/rasa/issues/3259 for a discussion, would love to hear your thoughts


feat: Add ability to refresh token automatically before expiration.

This pull request makes it possible to refresh the token when it is not expired but will expire in some period of time.
This can be used to address the issues described in #133, by allowing users to specify that tokens should be refreshed if they will expire soon. This is also necessary to fix GoogleCloudPlatform/cloud-profiler-nodejs#94
For earlier discussion of these changes, see PR #208.

Is there any update to the docs? How should this change be used?

Greetings!  You can learn how to us this in the reference docs:
https://google.github.io/google-auth-library-nodejs/classes/_auth_googleauth_.googleauth.html#getapplicationdefault

Greetings! You can learn how to us this in the reference docs:
https://google.github.io/google-auth-library-nodejs/classes/_auth_googleauth_.googleauth.html#getapplicationdefault

404


Add Hex String Conversion

Here's an implementation of an interesting function for converting hex strings into byte sequences:
function method IsHexDigit(c: char) : bool {
    '0' <= c <= '9' || 'a' <= c <= 'f'
}

function method ToHexDigit(c: char) : u8
requires IsHexDigit(c) {
    if '0' <= c <= '9' then (c - '0') as u8
    else ((c - 'a') as u8) + 10
}

function method ToHexByte(high: char, low: char) : u8
requires IsHexDigit(high) && IsHexDigit(low) {
    (ToHexDigit(high) * 16) + ToHexDigit(low)
}

function method convert(chars: seq<char>) : seq<u8>
requires |chars| % 2 == 0
requires forall i :: 0 <= i < |chars| ==> IsHexDigit(chars[i]) {
    // Determine number of characters
    var n := |chars| / 2;
    // Construct sequences
    seq(n, i requires i >= 0 && i < n => ToHexByte(chars[i*2],chars[(i*2)+1]))
}

This could be used, for example, when constructing EVM instances.  For example, this:
var vm := EvmBerlin.InitEmpty(gas :=100, code := [PUSH1,0x01,STOP]);

Could be rewritten as something like this:
var vm := EvmBerlin.InitEmpty(gas :=100, code := "0x600100");

Potentially, another option is to actually allow the code string to be written directly as a Dafny (hex) literal.

This might be useful, for example, for helping to certify proof objects.

Looks like a very good idea!


Feature request

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I've been using Aves for quite a while as my default gallery app and it's a great app, there are some feathers that i wanna request.
Size of image/video .. It's a very basic feature that every phone default gallery app has ... The app shows dimension and date, time but doesn't show size.
Location (where that image/video is stored in my phone storage)
Describe the solution you'd like
A clear and concise description of what you want to happen.
The solution I'm looking for is ... Make a another tab in more option for Details which shows size, dimensions, date, time and name of the file and where it's stored.
Something like this.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Maybe I misunderstood, but I believe everything you want is displayed at the top of the Info page, which is reachable with a swipe up from the viewer. I'm talking about this screen:

And you can come back to the viewer from the Info page by swiping down or using common buttons/gestures for "back".

Sorry my bad ... I didn't knew this existed ...
You should a little guide step for first timers

No worries. In previous versions there was also a button to reach that page, but it got removed to make the menu simpler. You're right that it would nice to provide some guidance. I'll think about it!


Sliding joints resistive heating error

In GitLab by @skahn on Feb 18, 2021, 15:45
Description
The sliding joint resitive heating was wrong.
Instead of joints in //
        ! Sliding joints resistive heating
        ! ---
        if ( i_cp_joints /= 0 ) then

            ! Total number of contact area (4 joints section per legs)
            n_contact_tot = 4 * n_tf_joints_contact* n_tf_joints * nint(n_tf_turn) * nint(n_tf)

            ! Total area of joint contact
            a_joints = tfthko * th_joint_contact * dble(n_contact_tot)

            ! joints resistive power losses
            pres_joints = rho_tf_joints * ritfc**2 / a_joints
        else
            ! Joints resistance to be evaluated for SC
            pres_joints = 0.0D0
        end if
        ! ---

The joints must be in series
        ! Sliding joints resistive heating
        ! ---
        if ( i_cp_joints /= 0 ) then

            ! Number of contact area per joint (all legs)
            n_contact_tot = n_tf_joints_contact * nint(n_tf_turn) * nint(n_tf)

            ! Area of joint contact (all legs)
            a_joints = tfthko * th_joint_contact * dble(n_contact_tot)

            ! Total joints resistive power losses
            pres_joints = dble(n_tf_joints) * rho_tf_joints * ritfc**2 / a_joints
        else
            ! Joints resistance to be evaluated for SC
            pres_joints = 0.0D0
        end if
        ! ---

Cheers
Seb

I am not sure anyone is ever likely to use the resistive coil model (although I thought that before and then Seb came along and spent months on it).

This change has already been made in sctfcoil.py (if loop at line 1862)


Cannot export TypeScript typing definitions for Skate-extended components

Using TypeScript 2.1.5, Skate 4.6.2.
When defining a component, if you have declaration: true set in your TypeScript compiler config, setting static get props() throws a compiler error:
[ts] Return type of public static property getter from exported class has or is using name 'PropOptions' from external module "c:/project/skatepark/node_modules/skatejs/src/ts-typings/api" but cannot be named.

To reproduce, clone the blaze-elements repo, and add "declaration": true to tsconfig.json's compilerOptions. Then run a build and it will fail.
/cc @Hotell, I'm not well-versed enough in TypeScript to debug where this needs resolved, but I think it's an issue with the type definitions?

@Hotell has been refining those lately. He'd probably know right off the bat what the issue might be.

hey @mattdsteele interesting. I'm core memberof that repo. That is probably some problem within blaze-elements setup which needs to be addressed.
When you try skate starter and add declaration: true to tsconfig.json no errors at all

Oh yes I remember now. We saw that error when we were pairing with @elmariofredo at work.
The error is occurring because we are not exporting that PropOptions so form compiler perspective, that's an error if you wanna produce public definitions ( he cannot create them properly if all needed interfaces, props are not exposed )

yup and there is issue for that https://github.com/wc-catalogue/blaze-elements/issues/126 we will address it soon in blaze-elements repo

Thanks for the info. So if my understanding is right, there's still work to do here; we should export PropOptions from src/index.d.ts to fix this? I can submit a PR if needed. Or is there another approach I should take?

not from ambient definition, that's generated stuff.
What needs to be done is this ( for instance in Alert component )
+export type AlertProps = Props & EventProps;
-type AlertProps = Props & EventProps;
+export type Props = {
-type Props = {
  color?: ColorType,
  isOpen?: boolean,
}
+ export type EventProps = {
-type EventProps = {
  onAlertClose?: ( ev: CustomEvent ) => void,
}
export class Alert extends Component<AlertProps> {
// ...
}

then tweak tsconfig
{
  declaration: true
}

Hmm, VSCode is showing the same error on the props getter even after exporting the types:

aha! right!
So compiler needs to be explicit because he cannot infer everything, which is good thing. Maybe we can try to improve generics within skate that inference would work.
Current fix is to strictly annotate return of props like this:
import {
 h, Component, prop,
+ComponentProps
 } from 'skatejs';

+export type AlertProps = Props & EventProps;
-type AlertProps = Props & EventProps;
+export type Props = {
-type Props = {
  color?: ColorType,
  isOpen?: boolean,
}
+export type EventProps = {
-type EventProps = {
  onAlertClose?: ( ev: CustomEvent ) => void,
}
export class Alert extends Component<AlertProps> {
static get is(){ /*...*/ }
+ static get props(): ComponentProps<Alert,Props> {
- static get props() {
    return {
      color: prop.string({
        attribute: true
      }),
      isOpen: prop.boolean({
        attribute: true
      })
    }
  }
// ...
}

Setting ComponentProps<Alert, Props> gave me this error:
[ts] Type '{ color: PropOptions<any, string>; isOpen: PropOptions<any, boolean>; }' is not assignable to type 'ComponentProps<Alert, Props>'.
       Types of property 'color' are incompatible.
         Type 'PropOptions<any, string>' is not assignable to type 'PropOptions<Alert, "brand" | "info" | "warning" | "success" | "error">'.
           Type 'string' is not assignable to type '"brand" | "info" | "warning" | "success" | "error"'.

Changing to ComponentProps<Alert, any> lets the compiler run, which is good enough to let me move forward. Happy to close this if you want to handle better generics in another ticket.

right the error is there because Alert uses string literals as type for color property to get proper autocomplete within JSX.
That type doesn't match of course to props.string() which returns string and string !== brand | primary.
Once again TS is right and forces us to write type safe code!
What needs to be done is to create custom prop helper for colors, which will also validate run time property mutation.
Here is a contrived example how it should look like:
import { h, Component, prop, ComponentProps } from 'skatejs';

// our lookupHelper for 'foo' | 'bar'
const lookupProp = prop.create<Lookup>({
  default: '' as Lookup,
  coerce: toNullOrString,
  deserialize: toNullOrString,
  serialize: toNullOrString,
  // run time validation yay!
  set(elem,data){
    const validValue = ['foo','bar'].filter(key=>key===data.newValue).length;
    if(!validValue){
      throw new Error(`${data.newValue} is not supported. Api allows 'foo' or 'bar'`)
    }
  }
})

export type Lookup = 'foo' | 'bar';
export type Props = {
  counter: number,
  lookup: Lookup,
}
export class App extends Component<Props> {
  static get is() { return 'my-app' }
  static get props(): ComponentProps<App,Props> {
    return {
      counter: prop.number(),
      lookup: lookupProp(),
    }
  }
  renderCallback() {
    return [

Hello

    ]
  }
}

export  function isEmpty (val:any): val is null|undefined { return typeof val === 'undefined' || val === null; }

function toNullOrString<T extends Lookup>(val:T){ return isEmpty(val) ? null : String(val) as T };

Result is fantastic! 💃

auto inference of static get props() just isn't possible in TS (generics cannot be used with properties and we cannot reference generic of instance within static method/prop) so you need to always explicitly use ComponentProps which is a good thing ( full type safety ).

So if there are any more question, feel free to close this ;)

Thanks!

no, thank you! thanks to this, missing "puzzle" to super powered type safe skate web components has been explored.
cheers! 🍺


RuntimeError: Could not infer dtype of NoneType

Hi,
I am trying to run CodeGPT (for text to code task), I followed exactly the same steps but I am getting this error:
Traceback (most recent call last):
File "run.py", line 653, in
main()
File "run.py", line 640, in main
global_step, tr_loss = train(args, train_dataset, model, tokenizer, fh, pool)
File "run.py", line 165, in train
for step, (batch, token_labels) in enumerate(train_dataloader):
File "/home/at5262/anaconda3/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 345, in next
data = self._next_data()
File "/home/at5262/anaconda3/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 385, in _next_data
data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
File "/home/at5262/anaconda3/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/home/at5262/anaconda3/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/home/at5262/CodeXGLUE/Text-Code/text-to-code/code/dataset.py", line 116, in getitem
return torch.tensor(self.inputs[item]), torch.tensor(self.token_labels[item])
RuntimeError: Could not infer dtype of NoneType
Traceback (most recent call last):
File "/home/at5262/anaconda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/at5262/anaconda3/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/at5262/anaconda3/lib/python3.7/site-packages/torch/distributed/launch.py", line 263, in
main()
File "/home/at5262/anaconda3/lib/python3.7/site-packages/torch/distributed/launch.py", line 259, in main
cmd=cmd)
subprocess.CalledProcessError: Command '['/home/at5262/anaconda3/bin/python', '-u', 'run.py', '--local_rank=0', '--data_dir=../dataset/concode', '--langs=java', '--output_dir=../save/concode', '--pretrain_dir=microsoft/CodeGPT-small-java-adaptedGPT2', '--log_file=text2code_concode.log', '--model_type=gpt2', '--block_size=512', '--do_train', '--node_index', '0', '--gpu_per_node', '1', '--learning_rate=5e-5', '--weight_decay=0.01', '--evaluate_during_training', '--per_gpu_train_batch_size=6', '--per_gpu_eval_batch_size=12', '--gradient_accumulation_steps=2', '--num_train_epochs=30', '--logging_steps=100', '--save_steps=5000', '--overwrite_output_dir', '--seed=42']' returned non-zero exit status 1.
I tried to run on Colab, and I am getting the same error

Hi, I can't reproduce the error. This error message indicts that there seems exist None during tokenizer processing. Would you please check if you are running the newest version of code with pytorch >= 1.4.0, transformers >= 2.5.0 and <= 4.0.0?

I have the same error

I have solved this bug. Updating transformers==2.5.0 to 4.0.0, it seems ok.

I reproduced this bug in text-to-code task. In my case, updating transformers does not work.
[Quick Solution]
1. Go to check the folder save/concode
2. Remove all the files begin with "train_blocksize_"
3. Rerun the training script
[Explaination]
I dived into details about the procedure and find that it is the preprocessing steps. The reason lies in reusing preprocessing files in "save/concode".
If there occurs error in generating preprocessed files in the beginning, like interpreting preprocessing steps in accident, you will get 2 (according to the default para) false files in  "save/concode". Later, if you do not remove them, you will keep getting errors when calling "data = [self.dataset[idx] for idx in possibly_batched_index]" at "File "/home/at5262/CodeXGLUE/Text-Code/text-to-code/code/dataset.py", line 116, in getitem".
If you remove these false files and get preprocessing steps rerun completely, the correct preprocessed files will be generated and the whole script can run normally.


Links doesn't display properly in comments

Links doesn't display properly in the comments.

you can close this


Migrate to<EMAIL_ADDRESS>

This PR adapts all places where we were using arg to invoke a wrapper function where we are using the new signature for 2.0.0. Also, it fixes a bug we had parsing arguments so that now when we pass something like now --debug ls it's going to work 👌
Please, test carefully!

Haven't tested but looks good aside from one comment.


Increasing allowed wasm bytecode size

Closes: #4171
What is the purpose of the change
Increase the max allowed wasm size to 2MB
Brief Changelog

Increased max allowed wasm size to 2MB

Testing and Verifying
Added an E2E test uploading a large file. The test fails without the change.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

I tested with our contract. It works.


docs: development and testing

Document how to develop and test this module.

Tests pass 🎉

@equinor/terraform-baseline An attempt at simplifying development and testing using a dev container 🙂
If you think it looks good, we can implement this across all baseline modules, see equinor/terraform-baseline#77.

Seems sensible :)


Update machine-learning-apps-text-analytics.md

We changed the schema so that the IDs are strings, not numbers. Updating docs accordingly.

Hi @LuisCabrer, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (luisca). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Add /examples folder

It would be nice to have an /examples folder, that would contain scripts with self-contained examples of usages of starknet.js
We have similar examples in our guides in documentation, but it never hurts to have more resources.
One good resource is @PhilippeR26 's examples repo here: https://github.com/PhilippeR26/starknet.js-workshop-typescript/tree/main/src/scripts
...but in our case, we would have only the latest relevant version examples!
Also, we would try and add these examples to run in our CI, so we are sure they are up-to-date and are not broken.
Some of the examples (besides the basic ones) we can add:

customizing your signer
adding your own provider

Can I work on this @ivpavici?

hello @ivpavici Please can I be assigned to this issue ?

hello @ivpavici I would love to work on this and be a contributor in your ecosystem

hi all!
Although this issue is not marked with "OnlyDust" for outside contributors, you are free to make a PR and we can review how it looks - if it's good we can reward it and assign additional guides!

hello @ivpavici can i work on this


CoC - Minor disagreement

From the CODE-OF-CONDUCT:

Likewise any spamming, trolling, flaming, baiting or other attention-stealing behavior is not welcome, and will result in your exclusion. This includes abusing support channels such as GitHub issues or social media to link to off-topic discussions or promote a personal fork or rival project. Sharing links to relevant tutorials, plugins, or issue workarounds is encouraged.

There are some people that are unsure as to what this means exactly (including me). It's a very open for interpretation rule. I also think I know why this was added, but I still think it doesn't belong in the CoC. It should either be rephrased, or removed.
Actually, talking about rival projects is a good thing. It's used to learn, to educate yourself. Sure; just jumping in a channel and spamming "screw sails, use this other project" would be wrong. But mentioning a project shouldn't constitute for reason of exclusion, ever.

@sgress454 Staple your bot to a wall please.

Negative, I am a meat popsicle.

The issue is a discussion point about a document. How can I do any of these things? Please disable this bot.

@RWOverdijk If you have a proposal as to how the code of conduct should be changed, you can make a PR to that document!

@RWOverdijk I'll hit you up on gitter


Added back button according to the UI

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes:  #294

Type of change

Adding back button to the video tutorials page would be easy for the users to navigate
Created back button in accordance with the existing UI

[X] New feature (non-breaking change which adds functionality)

Checklist:

[X] I have made this from my own
[X] I have taken help from some online resourses
[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
https://github.com/abhay-raj19/FitBody/assets/123291594/22a15d3e-c872-4cae-8bc4-578a5453449b

Changes made as requested and raised a new PR @


Dbz 175 snapshot refactor

Just the snapshot refactor portion of DBZ 175
move snapshot logic to AbstractSnapshotReader and SnapshotMySQLUtility
modify existing snapshotReader to use refactored functionality, and rename to SnapshotServerReader to distinguish it from the newly added (but as yet unused) SnapshotTableReader.

And now stepping back a bit to ask about the motives of this refactoring. Yes, the existing code is complex and difficult to follow, but IMO it also is really easy to debug and step through the various logical steps, and it's also easy to look at a log and understand what it was doing. If we break the logic up into lots of smaller methods, we're going to lose that higher-level understanding of the logic.
Why do we want to do the refactoring? IIUC, a significant reason is that we want to perform different kinds of snapshots, and the current code doesn't allow that. This is a good reason to do a refactoring -- are there other goals/reasons? The refactoring has to be guided by the goals.
Regarding that first goal, IMO the biggest problem with the current code is that SnapshotReader is doing two things: it's a asynchronous reader  and it implements the snapshot logic. These two things need to be separated, and IIUC we want to be able to use different snapshot algorithms for different purposes.
So, as a first step I might just try to separate those two behaviors. First, I might create a SnapshotAlgorithm interface that just had one execute() method, and we could have different implementations of that interface. We might make a first implementation named DatabaseSnapshotAlgorithm, and this would snapshot all of the tables. In fact, we might just try to move the current SnapshotReader.execute() code into the DatabaseSnapshotAlgorithm.execute() method.
Then, we could change SnapshotReader to use a SnapshotAlgorithm instance, and change the task to instantiate it with DatabaseSnapshotAlgorithm. All of this code should work exactly the same as the current code, and all test should pass.
Only then might we try to simplify the DatabaseSnapshotAlgorithm.execute() method. Here I think we could go in a couple of different directions. For example, at a really high level we might want the DatabaseSnapshotAlgorithm.execute() method to look like this:
try {
  db.acquireGlobalLock();
  if (!isRunning) return;

  db.startTransaction();
  db.readBinlogPosition();
  db.readDatabaseNames();
  db.readTableNames();
  if (!isLocked) {
    db.lockTables();
    db.readBinlogPosition();
  }
  db.generateDdlStatements();
  if (minimalLocking && isLocked) {
    ...
} finally {
  // do all the cleanup here, perhaps in a few steps to make sure it's done correctly
}

This is pretty darned attractive, but of course there are problems with simplifying this much. The current logic uses a fair number of variables used to track state shared across all of these steps, and my ultra-high-level ideal code neglects almost all of this (e.g., all of the checks to see if the logic was cancelled). We'd have to rectify that, but it's going to make the algorithm less ideal.
I think the biggest immediate win is to not try to simplify that much and to instead go step-by-step and try to pull out some of the multi-line chunks of code that would definitely be reusable. For example:
            final List<String> databaseNames = new ArrayList<>();
            sql.set("SHOW DATABASES");
            mysql.query(sql.get(), rs -> {
                while (rs.next()) {
                    databaseNames.add(rs.getString(1));
                }
            });

That could easily be pulled into a separate method, perhaps on the MySqlJdbcContext, so that the above could be replaced by:
            final List<String> databaseNames = context.readDatabaseNames();

There are a number of non-trivial loops and blocks that could probably be simplified this way without requiring a lot of passing of state. How far can we get while taking this simple approach.
THEN, we could create the TableSnapshotAlgorithm class by copying our new semi-simplified logic and then simplifying, not trying to reuse. It's not clear to me what that logic needs to be, so this approach outlines that logic first and only then can we see where the similarities lie.
WDYT? Am I way off base?

@rhauch I think you are not right path but let me propose one other think. We we really need to intemingle logic of anpashotting when globlac lock is not available with logic where it is not? Maybe it should be separated into two algortihm instances and it would clean the code a little bit too.

@jpechane, I would think that we'd have several algorithms, including the one you mentioned. Obviously the algorithms should be simple enough so that this is feasible. My concern is that we abstract it too much, we might make it really difficult to follow what the heck each is actually doing, or we build up too much state in those methods (e.g., locking) to make it difficult to reuse the methods.
Now that I think about it, the locking logic is complex but stateful, while most of the other steps actually are not very stateful (e.g., reading the tables in a database, reading the schemas for a list of tables, or copying the content of a list of tables, etc.). What if we were to pull out the locking logic into a reusable, stateful component, separate from the algorithms? Each algorithm could instantiate that, use the various methods as required, and then in the algorithm's finally block call the locking component's cleanup method that would "do the right thing" no matter which locks were obtained. Actually, @mtagle's snapshot utility class is this plus a bunch of the other non-stateful operations.

+1 for beginning with extracting some of the logic of SnapshotReader#execute() into separate methods (i.e. things which doesn't require the propagation of state). Then, in a next step, and regarding variables needed for keeping state, a context parameter object may be introduced which is passed around and keeps track of the required variables. Or, this could be extracted into a separate class, e.g. named SnapshottingRun which maintains that state as variables.
Taking a step back, one of my goals would be to explore whether these routines can be made re-usable and shared across connectors, as I'd assume that snapshotting logic may look similar for different RDMBS. But that's not something that'd need addressing with the change at bay. Focusing this refactoring onto what's needed in order to reasonably implement DBZ-175 seems like a good idea to me.


Theil–Sen estimator

Unfinished work

Coverage increased (+0.06%) to 75.315% when pulling c4fc871cbf64dde21a0271d0000a5e5871495e33 on TheilSen into f9f0828623d7540fa0b1df10d310d68a90846edb on master.

Coverage increased (+0.06%) to 75.315% when pulling 1d54acc35619de54b76bcaad4d6ea2f9413f0f5e on TheilSen into f9f0828623d7540fa0b1df10d310d68a90846edb on master.


No LICENSE and README

I would like to ask you to put the LICENSE file into this repo.

I took my first stab at it. Please let me know if it solves it for you or if you need some more information!

Super, thanks!


Downloads from PeerTube should use torrents, HTTP is rate limited

[x] Feature request (request for a new functionality)

I'm an administrator of a PeerTube instance, and I've noticed that youtube-dl has support for PeerTube. This is great!
However, it seems to download via a single HTTP connection, whereas PeerTube expects clients to use WebTorrent in order to share the bandwidth load. A client that uses only HTTP connections won't be as fast as a torrent-capable client, and a client that uses only a single HTTP connection will be hitting the server's per-connection rate limit, which is set to 800 kB/s by default.
It would be great if youtube-dl could discover the torrent link instead of the HTTP seed, to avoid overloading the single origin server.
Re test videos, there are many PeerTube instances listed at:
https://instances.joinpeertube.org/instances
My own instance is diode.zone, and a sample URL for a long video that might be a good performance test:
https://diode.zone/videos/watch/2f1544f6-3a3b-4cd0-bb72-c57d2e2d1e9b

Torrents are out of scope of youtube-dl.


Phase correction needs to work when # of pulses is smaller

Original report by Joseph Fowler (Bitbucket: joe_fowler, ).

phase_correct(), the 2015 version, can give ridiculous "corrections" when the # of photons is small. This needs to be improved.
First, do no harm.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Note that this never got resolved! I wrote on April 13 what I wanted to do, but I never did it.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Actually, issue #93 was fixed in April. This is separate: how to work on data with too few pulses for phase correct to operate. I'm going to put this on hold and leave it out of mass 0.6, but we can reopen it later if we find the perfect data set for the work.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Remove this from v0.6 milestone and wait until we have a good data set that we can test out ideas on.


Logitech Smooth Scrolling, Strange Behaviour

Atom 1.0.9: Windows 10
Hi, I recently bought the Mx Master mouse, from Logitech. It comes with software that enables smooth scrolling (the wheel spins freely so it doesn't feel great to have things still scroll in chunks). Now, when atom is inactive, and the mouse is hovered over top of the editor, this behaves as expected: the editor scrolls smoothly, without a set increment. However, once the editor is made active (clicked inside) it will only scroll in distinct chunks.
Hopefully someone finds a simple fix, however it is not critical,
Nico

I also looking for this feature for some time  but dont know how to make it.

Same here ;(


When running lost-pixel against project without config file an ugly error is thrown. Let's handle the error and throw a nice readable one-liner

this is indeed fixed @chriskalmar!


License type sync to KEB

Description
this provides a simple script along with the usage on how to synchronize ERS license type information to KEB via update instance endpoint. By default, it expects KUBECONFIG to be correctly set and port-forwards KEB database as a background process.
example usage:
$ ./license_type_sync.bash < ers_instances > log

Related issue(s)

following is the KEB query validating the effect of the script. License type is correctly populated.
broker=> select id, instance_id, state, type, finished_stages, provisioning_parameters->'ers_context'->'license_type' AS license_type, data->'migration_triggered' AS trigger, data->'sm_cluster_id' AS cluster_id, created_at from operations where instance_id = 'test-jw198' order by created_at;
                  id                  | instance_id |   state   |   type    |                                    finished_stages                                     | license_type | trigger |               cluster_id               |          created_at
--------------------------------------+-------------+-----------+-----------+----------------------------------------------------------------------------------------+--------------+---------+----------------------------------------+-------------------------------
 154198d8-d684-4c35-b5eb-bcd5310ee2e1 | test-jw198  | succeeded | provision | post_actions,,start,create_runtime,check_kyma                                          |              | false   | "3c2649f2-a22b-4c69-bf25-479a6e8fd7ab" | 2022-06-13 07:08:03.15981+00
 e3588037-ca67-41f9-9dd4-2363837fb2d0 | test-jw198  | succeeded | update    | ,cluster,migration,migration-check,remove-sc-migration,remove-sc-migration-check,check | "CUSTOMER"   | false   | "3c2649f2-a22b-4c69-bf25-479a6e8fd7ab" | 2022-06-13 07:30:50.540888+00


Better support for index template(s) + multiple indices

At the moment, when we use index templates but have static documents (i.e. in a JSON file), we need to use a hack to make this work by creating a dummy index to which we can "attach" the json file.
We should provide better support, so users can just specify index templates and JSON file(s) (which need to include an action and meta-data line to determine the correct index).

Closing in favor of #366 but we should take care there that this scenario works.


Nested class may be generated before its parent class

A sporadic issue, as a result - nested class gets overriden by its parent class code (i.e. removed).
In js it looks like:
TypeError: undefined is not a function
    at Class.Bridge.Class.define.constructor [as $constructor] (http://localhost:20000/Tests/TestSets.js:220:35)
    at Class.Bridge.Class.initCtor [as $$initCtor] (http://localhost:20000/Tests/bridge.js:842:29)
    at new Class (http://localhost:20000/Tests/bridge.js:972:33)

A workaround is to use [SerializationPriority(-1)] for the nested class to force it be serialized after the parent class (with SerializationPriority = 0 by default).

I guess it can be closed already


Please add the ability to whitelist/blacklist certain notifications

So, for a regular plugin, would just uninstall the plugin if it kept giving me annoying notifications that I don't want, but with VSCode, you don't have that option, because they are built into application itself.
Can you please create a way to silence the infuriating notifications that pop up all the time that no one wants? People are trying to work.

/duplicate https://github.com/microsoft/vscode/issues/41767

/duplicate https://github.com/microsoft/vscode/issues/41767

Please re-open this issue. This is the exact opposite of the issue you claim is a "duplicate".
That issue is asking about turning off notifications from plugins. I'm wanting to turn off notifications from VS Code itself.
Like for instance, how it seemingly creates a popup every time you close a terminal saying it exited with some error code or something that I don't care about and interrupts my workflow.
And there are a lot of notifications like that, and it is extremely annoying.
So, this is not a "duplicate", please unmark it as a "duplicate". This is its own issue.

Please re-open this issue. This is the exact opposite of the issue you claim is a "duplicate".
That issue is asking about turning off notifications from plugins. I'm wanting to turn off notifications from VS Code itself.
Like for instance, how it seemingly creates a popup every time you close a terminal saying it exited with some error code or something that I don't care about and interrupts my workflow.
And there are a lot of notifications like that, and it is extremely annoying.
So, this is not a "duplicate", please unmark it as a "duplicate". This is its own issue.

@zwhitchcox we have support in VSCode notifications to add a "Do not show again", which then will cause the notification to never show again. Can you tell for which notification you would like to configure to hide it?
I don't think that hiding all notifications is currently an option.

@zwhitchcox we have support in VSCode notifications to add a "Do not show again", which then will cause the notification to never show again. Can you tell for which notification you would like to configure to hide it?
I don't think that hiding all notifications is currently an option.

OK, I will start taking screenshots whenever I get one so I can provide you with examples. Thank you for your help.

OK, I will start taking screenshots whenever I get one so I can provide you with examples. Thank you for your help.

Ok

Ok

There's also the notification "Ignorance is bliss, we will not show any more notifications for this plugin" or something where they literally show a popup notification after you just asked them not to show any more popup notifications for something, I think it's "Suppress Errors".
I would even be ok if they made it where these notifications just fade away after a few seconds (although I would rather have the ability to turn them off), but I have to drop what I'm doing and dismiss the notification, and as a vim user, I do everything from the keyboard, and there is no way to dismiss them from the keyboard, so I have to actually drag my mouse, find where the cursor is, and get used to using a mouse again (your mind sort of has to calibrate how far moving your hand is going to move your mouse), and click on the X (and God help you if you miss the X, because who knows what that's going to do). And in the case of "Suppress Errors", right after that, there is another popup which shows up after a slight delay, and you have to exit that.
It just completely interrupts my rhythm, and especially when you're working on a complicated project, it is vital to eliminate all distractions you can. I mean, I guess it only happens a few times a day, and I guess the error above about the terminal process exiting is the only one you can't dismiss forever (or at least until you  close the program and open it again), but a few times a day over several years is a lot, especially when it interrupts your work flow.

There's also the notification "Ignorance is bliss, we will not show any more notifications for this plugin" or something where they literally show a popup notification after you just asked them not to show any more popup notifications for something, I think it's "Suppress Errors".
I would even be ok if they made it where these notifications just fade away after a few seconds (although I would rather have the ability to turn them off), but I have to drop what I'm doing and dismiss the notification, and as a vim user, I do everything from the keyboard, and there is no way to dismiss them from the keyboard, so I have to actually drag my mouse, find where the cursor is, and get used to using a mouse again (your mind sort of has to calibrate how far moving your hand is going to move your mouse), and click on the X (and God help you if you miss the X, because who knows what that's going to do). And in the case of "Suppress Errors", right after that, there is another popup which shows up after a slight delay, and you have to exit that.
It just completely interrupts my rhythm, and especially when you're working on a complicated project, it is vital to eliminate all distractions you can. I mean, I guess it only happens a few times a day, and I guess the error above about the terminal process exiting is the only one you can't dismiss forever (or at least until you  close the program and open it again), but a few times a day over several years is a lot, especially when it interrupts your work flow.

I would even be ok if they made it where these notifications just fade away after a few seconds

Most do, but not if an error is shown or input is asked for (via buttons). It is hard to know if a notification shows up because input is needed or the user really needs to be told or not.

and there is no way to dismiss them from the keyboard

That is just not true, there are tons of commands that allow you to do everything with regards to notifications:

We try really hard to allow you do operate notifications without a mouse, if something is missing please file a new issue.
I like that you posted a screenshot of a notification that you find annoying. Can you report it as individual issue so that we can think about alternatives? I somewhat agree that maybe notifications could selectively show closer within the component where it originates from (e.g. inside the terminal).

I would even be ok if they made it where these notifications just fade away after a few seconds

Most do, but not if an error is shown or input is asked for (via buttons). It is hard to know if a notification shows up because input is needed or the user really needs to be told or not.

and there is no way to dismiss them from the keyboard

That is just not true, there are tons of commands that allow you to do everything with regards to notifications:

We try really hard to allow you do operate notifications without a mouse, if something is missing please file a new issue.
I like that you posted a screenshot of a notification that you find annoying. Can you report it as individual issue so that we can think about alternatives? I somewhat agree that maybe notifications could selectively show closer within the component where it originates from (e.g. inside the terminal).

Ultimately, I think the solution should probably be to have the ability to disable all popups, which are notoriously annoying to users. I think the best solution for me personally would be to have them display in a little notification center in the sidebar. I personally would never look at them (and maybe this would cause more reported errors because users would not see something that would cause them to have errors). For me personally, I don't think I've ever seen a useful toast notification aside from maybe the ones that say "Would you like to install the extension for this file type?" (although a better solution, I think would be allowing users configurations for plugins and allowing them to sign in to VS Code, but  I realize that has its own challenges (and I realize there is a plugin for this too, but that solution is a little more complicated than just signing in)). But it would give users at least an option to display notifications in a less obtrusive way.
The next best solution would be a method to turn off all notifications.
The solution you gave of using a keyboard shortcut would be good, except you can't use the shortcut when the terminal is open, which is usually when that particular notification shows up, i.e., you're closing a terminal to create a new one, or you're consolidating all your terminals.
But I don't know, maybe this is a bug?

Ultimately, I think the solution should probably be to have the ability to disable all popups, which are notoriously annoying to users. I think the best solution for me personally would be to have them display in a little notification center in the sidebar. I personally would never look at them (and maybe this would cause more reported errors because users would not see something that would cause them to have errors). For me personally, I don't think I've ever seen a useful toast notification aside from maybe the ones that say "Would you like to install the extension for this file type?" (although a better solution, I think would be allowing users configurations for plugins and allowing them to sign in to VS Code, but  I realize that has its own challenges (and I realize there is a plugin for this too, but that solution is a little more complicated than just signing in)). But it would give users at least an option to display notifications in a less obtrusive way.
The next best solution would be a method to turn off all notifications.
The solution you gave of using a keyboard shortcut would be good, except you can't use the shortcut when the terminal is open, which is usually when that particular notification shows up, i.e., you're closing a terminal to create a new one, or you're consolidating all your terminals.
But I don't know, maybe this is a bug?

One other thing, I don't think there's anywhere else you could report the notification in the screen shot, because once you close the terminal, you obviously can't use that terminal to report that the terminal was closed.
Idk, though, maybe you're suggesting report it in the notification bar in the little status bar at the bottom of the window? I do think that would be a better place for it.

One other thing, I don't think there's anywhere else you could report the notification in the screen shot, because once you close the terminal, you obviously can't use that terminal to report that the terminal was closed.
Idk, though, maybe you're suggesting report it in the notification bar in the little status bar at the bottom of the window? I do think that would be a better place for it.

Also, btw, I hope I'm not coming across too negative, the product your team has made is amazing, and really, really good on 99.9999% of things, it's just this one thing that kind of grinds on my nerves over time, and unfortunately, that's the only time you're going to hear from me. But great work, and it's much appreciated!

Also, btw, I hope I'm not coming across too negative, the product your team has made is amazing, and really, really good on 99.9999% of things, it's just this one thing that kind of grinds on my nerves over time, and unfortunately, that's the only time you're going to hear from me. But great work, and it's much appreciated!

@zwhitchcox

But I don't know, maybe this is a bug?

As for keyboard support from terminal it would be good to file this as individual issue, I think we have a list of commands we allow from the terminal.

Ultimately, I think the solution should probably be to have the ability to disable all popups, which are notoriously annoying to users.

As I said before, some notifications may be critical for the flow of an extension. Just to give an example: an extension might require you to sign in but that failed and a notification is shown. If you hide this notification how would you know that something went wrong? In the end we cannot tell if a notification is important to show to the user or not. We have a heuristic by looking for error severity and the fact that buttons are added, but that might not be correct all the time.

@zwhitchcox

But I don't know, maybe this is a bug?

As for keyboard support from terminal it would be good to file this as individual issue, I think we have a list of commands we allow from the terminal.

Ultimately, I think the solution should probably be to have the ability to disable all popups, which are notoriously annoying to users.

As I said before, some notifications may be critical for the flow of an extension. Just to give an example: an extension might require you to sign in but that failed and a notification is shown. If you hide this notification how would you know that something went wrong? In the end we cannot tell if a notification is important to show to the user or not. We have a heuristic by looking for error severity and the fact that buttons are added, but that might not be correct all the time.

Ok, thank you for your suggestions. I think I'm going to close this issue and split it into 3 separate issues, because any of these ways could potentially have been a solution for me:

ability to close all notifications from terminal through keyboard shortcut #114268
ability to turn off the "terminated with exit code: 1" notification #114271
ability to keep all notifications minimized #46676 which you already mentioned.

Thank you for your help..you're a superstar!

Ok, thank you for your suggestions. I think I'm going to close this issue and split it into 3 separate issues, because any of these ways could potentially have been a solution for me:

ability to close all notifications from terminal through keyboard shortcut #114268
ability to turn off the "terminated with exit code: 1" notification #114271
ability to keep all notifications minimized #46676 which you already mentioned.

Thank you for your help..you're a superstar!

Thanks a lot, going on in individual issues is a good way forward 👍

Thanks a lot, going on in individual issues is a good way forward 👍


keyboardShouldPersistTaps missing from typings

react-native, react and native-base version

RN 0.54.0
R 16.3.0
NB 2.4.1

Expected behaviour
Typings present
Actual behaviour
Typings not present
Steps to reproduce (code snippet or screenshot)
<Tabs contentProps={{ keyboardShouldPersistTaps: "handled" }}>
    {tabComponents}
</Tabs>

Property 'contentProps' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes & Readonly<{ children?: ReactNode; }> & Read...'.

Screenshot of emulator/device
n/a
Is the bug present in both ios and android or in any one of them?
Yes
Any other additional info which would help us debug the issue quicker.
Props added in #1410 but not added to index.d.ts
Can be temporarily fixed by adding below typings file to project
declare module 'native-base' {
    namespace NativeBase {
        interface Content {
            keyboardShouldPersistTaps?: string;
        }

        interface Tabs {
	    contentProps?: NativeBase.Content;
        }
    }
}

Fixed with 2.4.2


Error: Mining phase failure

Hello everyone, I'm having problems running the mining phase. I'm using a computer with 60Gb of RAM and 16 CPU cores. When running the mining phase I get the error below.
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/raphael_assis4347/raphael/cc_net/cc_net/__main__.py", line 18, in <module>
    main()
  File "/home/raphael_assis4347/raphael/cc_net/cc_net/__main__.py", line 14, in main
    func_argparse.parse_and_call(cc_net.mine.get_main_parser())
  File "/home/raphael_assis4347/.local/lib/python3.8/site-packages/func_argparse/__init__.py", line 72, in parse_and_call
    return command(**parsed_args)
  File "/home/raphael_assis4347/raphael/cc_net/cc_net/mine.py", line 631, in main
    all_files = mine(conf)
  File "/home/raphael_assis4347/raphael/cc_net/cc_net/mine.py", line 341, in mine
    ex(_mine_shard, repeat(conf), hashes_files, *_transpose(missing_outputs))
  File "/home/raphael_assis4347/raphael/cc_net/cc_net/execution.py", line 200, in custom_map_array
    raise Exception(message)
Exception: 9 / 9 jobs failed while running _mine_shard
Will run cc_net.mine.main with the following config: Config(config_name='base', dump='2019-09', output_dir=PosixPath('data'), mined_dir='mined_data', execution='auto', num_shards=9, num_segments_per_shard=750, metadata=None, min_len=300, hash_in_mem=9, lang_whitelist=['pt'], lang_blacklist=[], lang_threshold=0.5, keep_bucket=['head', 'middle'], lm_dir=PosixPath('data/lm_sp'), cutoff=PosixPath('/home/raphael_assis4347/raphael/cc_net/cc_net/data/cutoff.csv'), lm_languages=['pt'], mine_num_processes=9, target_size='4G', cleanup_after_regroup=True, task_parallelism=-1, pipeline=['dedup', 'lid', 'keep_lang', 'sp', 'lm', 'pp_bucket', 'drop', 'split_by_lang'], experiments=[], cache_dir=PosixPath('data/wet_cache'))
Submitting 9 jobs for _mine_shard, with task_parallelism=16
Waiting on 9 running jobs. Job ids: 69378,69397,69400,69419...
Failed job 69516 (1 / 9): Job 69516 (task: 0) with path /home/raphael_assis4347/raphael/cc_net/data/logs/69516_0_result.pkl
has not produced any output (state: FINISHED)
Error stream produced:
----------------------------------------
2022-09-14 16:43 INFO 69535:cc_net.jsonql - preparing [<cc_net.dedup.DuplicatesRemover object at 0x7fd7313c8d00>, Classifier(bin/lid.bin), <cc_net.jsonql.where object at 0x7fd7313c8df0>, <cc_net.perplexity.MultiSentencePiece object at 0x7fd7313c8d30>, <cc_net.perplexity.DocLM object at 0x7fd7313c8e50>, <cc_net.perplexity.PerplexityBucket object at 0x7fd7313c8d60>, <cc_net.perplexity.DropKeys object at 0x7fd7313c8fa0>]

Waiting on 8 running jobs. Job ids: 69378,69397,69400,69419...
Failed job 69400 (2 / 9): Job 69400 (task: 0) with path /home/raphael_assis4347/raphael/cc_net/data/logs/69400_0_result.pkl
has not produced any output (state: FINISHED)
Error stream produced:
----------------------------------------
2022-09-14 16:43 INFO 69418:cc_net.jsonql - preparing [<cc_net.dedup.DuplicatesRemover object at 0x7fcee923a970>, Classifier(bin/lid.bin), <cc_net.jsonql.where object at 0x7fcee923aa60>, <cc_net.perplexity.MultiSentencePiece object at 0x7fcee923a9a0>, <cc_net.perplexity.DocLM object at 0x7fcee923aac0>, <cc_net.perplexity.PerplexityBucket object at 0x7fcee923a9d0>, <cc_net.perplexity.DropKeys object at 0x7fcee923ac10>]

I couldn't identify the problem by looking at the logs. In the process .log.err file it only contains the pipeline objects vector. Does anyone have any idea what it could be?
This is my configuration file:
{
    "dump": "2019-09",
    "hash_in_mem": 9,
    "num_shards": 9,
    "mine_num_processes": 9,
    "num_segments_per_shard": 750,
    "lang_whitelist": ["pt"],
    "lm_languages": ["pt"],
    "pipeline": [
        "dedup",
        "lid",
        "keep_lang",
        "sp",
        "lm",
        "pp_bucket",
        "drop",
        "split_by_lang"
    ],
    "execution": "auto",
    "output_dir": "data",
    "mined_dir": "mined_data",
    "target_size": "4G",
    "keep_bucket": ["head", "middle"],
    "cache_dir": "data/wet_cache"
}

After some analysis, I realized that I had memory overflow. Limiting to 1 hash in memory managed to solve the problem.


Output in .json

The script i am using yotse with outputs .json files. The following docstring fragment, extracted from the SystemSetup method of pre.py implies that this is supported:
output_extension : str (optional)
    Extension of the output files to be picked up by the analysis_script, e.g 'csv' or 'json'. Defaults to 'csv'.

However, running my code gives me the following error:
Traceback (most recent call last):
File "/gpfs/home5/fsilva/project_rl_quantum/experiments/purify_entangle_discard/simplified/potential_shaped_reward/0/yotse_rl.py", line 74, in
main()
File "/gpfs/home5/fsilva/project_rl_quantum/experiments/purify_entangle_discard/simplified/potential_shaped_reward/0/yotse_rl.py", line 67, in main
rl_example.run(step=0)
File "//home/fsilva/project_rl_quantum/yotse/yotse/run.py", line 377, in run
super().run(step, evolutionary_point_generation)
File "//home/fsilva/project_rl_quantum/yotse/yotse/run.py", line 126, in run
data = self.collect_data()
File "//home/fsilva/project_rl_quantum/yotse/yotse/run.py", line 201, in collect_data
data = file_list_to_single_df(files)
File "//home/fsilva/project_rl_quantum/yotse/yotse/run.py", line 79, in file_list_to_single_df
dfs = [pandas.read_csv(file, delimiter=' ') for file in files]
File "//home/fsilva/project_rl_quantum/yotse/yotse/run.py", line 79, in
dfs = [pandas.read_csv(file, delimiter=' ') for file in files]
File "/gpfs/home5/fsilva/rlquantumenv/lib/python3.9/site-packages/pandas/util/_decorators.py", line 311, in wrapper
return func(*args, **kwargs)
File "/gpfs/home5/fsilva/rlquantumenv/lib/python3.9/site-packages/pandas/io/parsers/readers.py", line 680, in read_csv
return _read(filepath_or_buffer, kwds)
File "/gpfs/home5/fsilva/rlquantumenv/lib/python3.9/site-packages/pandas/io/parsers/readers.py", line 581, in _read
return parser.read(nrows)
File "/gpfs/home5/fsilva/rlquantumenv/lib/python3.9/site-packages/pandas/io/parsers/readers.py", line 1254, in read
index, columns, col_dict = self._engine.read(nrows)
File "/gpfs/home5/fsilva/rlquantumenv/lib/python3.9/site-packages/pandas/io/parsers/c_parser_wrapper.py", line 225, in read
chunks = self._reader.read_low_memory(nrows)
File "pandas/_libs/parsers.pyx", line 805, in pandas._libs.parsers.TextReader.read_low_memory
File "pandas/_libs/parsers.pyx", line 861, in pandas._libs.parsers.TextReader._read_rows
File "pandas/_libs/parsers.pyx", line 847, in pandas._libs.parsers.TextReader._tokenize_rows
File "pandas/_libs/parsers.pyx", line 1960, in pandas._libs.parsers.raise_parser_error
pandas.errors.ParserError: Error tokenizing data. C error: Expected 6 fields in line 3, saw 10
After having a look at run.py, I believe that yotse does not actually (yet) support output in a format other than .csv. Are there plans to change this? Here is my yotse script:
import os
import shutil
from yotse.pre import Experiment, SystemSetup, Parameter
from yotse.run import Executor

def dumb_dist(min_value, max_value, number_points):
    return [0.]

def rl_input():
    rl_experiment = Experiment(
        experiment_name="EntangleMaster",
        system_setup=SystemSetup(
            source_directory=os.getcwd(),
            program_name="entangle_master_runscript.py",
            command_line_arguments={"--alpha": 0.1,
                                    "--elementary_link_fidelity": 0.88,
                                    "--n_trials": 10,
                                    "--single_gen_action": True,
                                    "--target_fidelity": 0.9,
                                    "--eta": 0.1,
                                    "--n_coherence": 10},
            # analysis_script="analyse_function_output.py",
            executor="python",
            output_dir_name="output",
            venv="~/rlquantumenv",
            num_nodes=1,
            alloc_time="04:00:00",
            slurm_args=["--exclusive"],
            modules=["2021", "Python/3.9.5-GCCcore-10.3.0"],
            output_extension="json"
        ),
        parameters=[
            Parameter(
                name="gamma",
                param_range=[0., 1.],
                constraints={'low': 0., 'high': 1.},
                number_points=1,
                distribution="custom",
                param_type="continuous",
                custom_distribution=dumb_dist,
            )
        ]
    )

    return rl_experiment

def remove_files_after_run():
    # remove files and directories
    dirs = [f for f in os.listdir(os.getcwd()) if (f.startswith(".qcg"))]
    for d in dirs:
        shutil.rmtree(os.path.join(os.getcwd(), d))

def main():
    def cost_function(f):
        return f

    experiment = rl_input()
    experiment.cost_function = cost_function
    rl_example = Executor(experiment=experiment)

    experiment.parse_slurm_arg('qiaopt_rl.py')

    rl_example.run(step=0)

    # clean up
    remove_files_after_run()

if __name__ == "__main__":
    main()

The script that it calls (entangle_master_runscript.py) makes calls to a bunch of repositories making it a bit of a hassle to run. I can make a minimal example if that's helpful.

Hey @FranciscoHS, thanks for reporting this. No need to make a minimal example (for now), I think it's quite clear what the issue is indeed.
In the script you supplied you commented out the analysis_script parameter, is that intentional?
Just to make sure I understand could you very briefly outline the behavior you would like to achieve? (like: you run your script, each job outputs a .json, you want yotse to combine this, or each job already has some analysis in it and you just use yotse to schedule the jobs)

I added a few changes in the branch connected to this issue.
@FranciscoHS could you check if this fixes your issue? Then I will add some proper tests for the entire functionality.
Also out of curiosity: Do you foresee using any other file extensions in your work? Then I could add support for those ones as well (currently only csv, json and pickle).

Alright sounds good.
Then last question: Would it be useful to have a way to not combine output at all?

I would say yes, but with very very low priority.

Support for pickle and json output  is now resolved in https://github.com/SURFQuantum/yotse/pull/26


Warning when building in iOS 11

When I build I get the following warning on line 21 of VYNFCNDEFPayloadParse.m:
"'NFCNDEFPayload' is partial: introduced in iOS 11.0"
I believe this can be fixed by an availability statement.

Thanks for reporting. It is fixed in the latest version.


Fix bdb log hole bug

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

We should set nextJournalId to the origin value, whatever the cause of the write failure is.

run starrocks_fe_unittest


Webpack production build yields "Invalid Regex" error with Webpack2

I have tried using transform-regexp-constructors and setting it to true and to false. No difference either way.
This doesn't occur in development. Any idea what if any change to webpack or .babelrc would be necessary to restore this functionality? It worked in the Webpack 1 setup.
Thanks for reading and for any advice!
Uncaught SyntaxError: Invalid regular expression: /(\\W|^)(?:(?:\\,|\\(|\\\uFF08)\\s*)?(?:on\\s*?)?(?:(this|last|next)\\s*)?(sunday|sun|monday|mon|tuesday|tues|tue|wednesday|wed|thursday|thurs|thur|thu|friday|fri|saturday|sat)(?:\s*(?:\,|\)|\）))?(?:\s*(this|last|next)\s*week)?(?=\W|$)/: Unterminated group
    at new RegExp (<anonymous>)
    at Object../node_modules/chrono-node/src/parsers/EN/ENWeekdayParser.js (ENWeekdayParser.js:12)
    at U (bootstrap 9726358…:19)
    at Object../node_modules/chrono-node/src/parsers/parser.js (parser.js:63)
    at U (bootstrap 9726358…:19)
    at Object../node_modules/chrono-node/src/options.js (options.js:1)
    at U (bootstrap 9726358…:19)
    at Object../node_modules/chrono-node/src/chrono.js (chrono.js:2)
    at U (bootstrap 9726358…:19)
    at Object../app/utils/rclReminder.js (rclReminder.js:2)

It's a known issue it looks like: https://github.com/babel/babel/issues/5564

It looks like the original reporter of the issue was also using the boilerplate from what I can tell: https://github.com/MatusSkerlik/Prezior

Closed as duplicate of 861


[Bug] Snapshotter unit tests which require root permissions are skipped in CI

Description
7 snapshotter unit tests require root permissions; however, the current CI setup is not running unit tests as root. This means these tests are being skipped. Moreover when running the tests locally with root permissions, the tests are still skipped.
Steps to reproduce the bug
Run unit tests with verbose output to see the skipped tests:
GO_TEST_FLAGS="-v -count=1" make test | grep "SKIP:"

Output:
--- SKIP: TestRemotePrepare (0.00s)
--- SKIP: TestRemoteView (0.00s)
--- SKIP: TestRemoteOverlay (0.00s)
--- SKIP: TestRemoteCommit (0.00s)
--- SKIP: TestFailureDetection (0.00s)
--- SKIP: TestOverlay (0.00s)
--- SKIP: TestOverlayOverlayRead (0.00s)

Run unit tests with root permissions results in the same behavior:
GO_TEST_FLAGS="-v -count=1" sudo make test | grep "SKIP:"

Output:
--- SKIP: TestRemotePrepare (0.00s)
--- SKIP: TestRemoteView (0.00s)
--- SKIP: TestRemoteOverlay (0.00s)
--- SKIP: TestRemoteCommit (0.00s)
--- SKIP: TestFailureDetection (0.00s)
--- SKIP: TestOverlay (0.00s)
--- SKIP: TestOverlayOverlayRead (0.00s)

Describe the results you expected

All of the snapshotter's unit tests should be ran in CI.
Running unit tests locally with root permissions should run all unit tests.

Host information

OS: Amazon Linux
Snapshotter Version:

soci version fe3fc93.m fe3fc9339a3fa5a3f602cfcbc21842bb8d3b9028.m

Containerd Version:

containerd github.com/containerd/containerd v1.6.24-36-gbf6f562cc bf6f562ccf2734f6f5e5723d043c45bb7d9620ac

Any additional context or information about the bug
The unit tests are taking dependency on testutil.RequiresRoot from github.com/containerd/containerd/pkg/testutil.

I think these are all because they create overlay mounts which can be done rootless since kernel 5.11. I think we should try only calling testutil.RequiresRoot if the kernel is lower (and verify that they do work without root on 5.11+).


Adding rich HTML output for All of GEOPHIRES.

New code for adding optional HTML output to all GEOPHIRES modules. It also offers an optional better text version of the output.
There is also one fix for the OUTPUT unit change function (but it doesn't solve the currency exchange issue).
Because this messes with the output format, the unit testing parsing routines are broken, so some of the unit tests fail. This creates a new issue to be resolved.

It made it easier to align the numbers, that is all – I am essentially padding the text on either side and allowing Python to handle the alignment rather than hand-padding the alignment with spaces on a line-by-line basis for alignment. Doing this lets me write the report lines in a generic loop rather than a custom-line-by-line format. More compact.
From: Jonathan Pezzino @.>
Sent: Thursday, April 4, 2024 8:52 AM
To: NREL/GEOPHIRES-X @.>
Cc: Malcolm Ross @.>; Author @.>
Subject: Re: [NREL/GEOPHIRES-X] Adding rich HTML output for All of GEOPHIRES. (PR #177)
@softwareengineerprogrammer commented on this pull request.

In tests/examples/example_SHR-2.out https://github.com/NREL/GEOPHIRES-X/pull/177#discussion_r1551734087 :

 Production Wellbore heat transmission calculated with Ramey's model

 Average production well temperature drop:              19.0 degC

 Flowrate per production well:                          55.0 kg/sec

 Injection well casing ID:                               7.000 in

 Production well casing ID:                              7.000 in

 Number of times redrilling:                             0

 Power plant type:                                       Single-Flash

                    ***RESOURCE CHARACTERISTICS***

 Maximum reservoir temperature:                        500.0 degC

 Number of segments:                                     1

 Geothermal gradient:                                    0.0500 degC/m

 Economic Model                               : Fixed Charge Rate (FCR)

What is the motivation for aligning the colons like this instead of the standard/regular way they are currently? I don't believe I've ever seen this style of textual representation for data.
—
Reply to this email directly, view it on GitHub https://github.com/NREL/GEOPHIRES-X/pull/177#pullrequestreview-1979987885 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AWGVYT4D2BCXPU6ZOBE6IIDY3VLGJAVCNFSM6AAAAABFVWRC26VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNZZHE4DOOBYGU .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AWGVYT4YBN2RN3ADKPBTTUTY3VLGJA5CNFSM6AAAAABFVWRC26WGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTTWAQ322.gif Message ID: @.*** @.***> >

This PR now includes the code for PTC and ITC.
It also corrects some issues in the output unit test files

@malcolm-dsider took a look at fixing failing tests per your request.
The extended economic profile test is failing because extended economic profile is no longer present in example1_addons.out: https://github.com/NREL/GEOPHIRES-X/pull/177/files#diff-d5df12751cf72e6cd45c0019dedbb262986aa5d2b7754d70799836a8bb8cb027L247-L300.
Can you please advise on whether this was intentional? (if so, we need to discuss because it would be backwards-incompatible)

Yep, that was a bug I introduced in the outputput. That bug is now fixed and it is correctly adding the outputs report for addons. I did a commit and push on the fix. But the unit tests are still failing – it looks like a parser issue?
From: Jonathan Pezzino @.>
Sent: Thursday, April 11, 2024 7:33 AM
To: NREL/GEOPHIRES-X @.>
Cc: Malcolm Ross @.>; Mention @.>
Subject: Re: [NREL/GEOPHIRES-X] Adding rich HTML output for All of GEOPHIRES. (PR #177)
@malcolm-dsider https://github.com/malcolm-dsider  took a look at fixing failing tests https://github.com/NREL/GEOPHIRES-X/actions/runs/8637350288/job/23679321834?pr=177  per your request.
The extended economic profile test https://github.com/NREL/GEOPHIRES-X/blob/46d18a783f1134f9873e77899b39a5bb2d009817/tests/test_geophires_x_client.py#L200  is failing because extended economic profile is no longer present in example1_addons.out: https://github.com/NREL/GEOPHIRES-X/pull/177/files#diff-d5df12751cf72e6cd45c0019dedbb262986aa5d2b7754d70799836a8bb8cb027L247-L300.
Can you please advise on whether this was intentional? (if so, we need to discuss because it would be backwards-incompatible)
—
Reply to this email directly, view it on GitHub https://github.com/NREL/GEOPHIRES-X/pull/177#issuecomment-2049593334 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AWGVYT3Y3XOUBLB52WQJYJ3Y4Z7GPAVCNFSM6AAAAABFVWRC26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBZGU4TGMZTGQ .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AWGVYTYCRYZMHFGWNSYBROTY4Z7GPA5CNFSM6AAAAABFVWRC26WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTT2FJH7M.gif Message ID: @.*** @.***> >


WIP WIP User auth

Only pushed PR atm to have something to link UI deploy hold to.

This change is

TODO:

Configure Deploy with GH App Keys
Configure Deploy with Key set

Current coverage is 20.00% (diff: 0.00%)

Merging #229 into master will decrease coverage by 0.13%

@@             master       #229   diff @@
==========================================
  Files            32         33     +1
  Lines          1167       1175     +8
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            235        235
- Misses          888        895     +7
- Partials         44         45     +1

Powered by Codecov. Last update 5c04a5b...3b2b282


Generate segmentation

Hello,
Is-it possible to save segmentations in a dicom file locally ? I see that we can generate reports but is it possible to do the same thing with segmentations ? I didn't find any examples.

Do you have the solution ? Save the segmentation as points and display it through points.