Handle timezone-aware datetime objects when writing SPDX.

Fixes #766.

Looks like I didn't add the test dependency tzdata correctly, or missed something.  I'll look in the morning, or if someone wants to just ensure tzdata is installed as a test dependency everywhere it's needed, that should clear out the test failure.


Move typescript types into separate files

@helio-frota so, as the project grows it's probably a good idea to refactor the code so we can have all the TS types needed into a single file for a single source of truth kind of thing

@alexalikiotis I think we can hold this issue a bit. And see if the project will grow properly. :+1:

@helio-frota sure, I just opened this for the future


Unable to scroll inside InfoBox

If the content of the InfoBox overflows, you can't use the scrollbar to scroll
down because it will drag the map instead.

Any ideas?

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2 Jul 2010 at 9:38

Is there any solution for this? I am hitting the same problem on iOS devices when i try to scroll inside a InfoWindow


fix #224

Fixes #224
Changes proposed in this pull request:

change id to name
change url to path

to match spec http://frictionlessdata.io/specs/data-package/#licenses

Test output https://github.com/Stephen-Gates/data-package-examples/tree/master/one-semi-colon-file


Cannot run brackets due to segmentation fault on Debian 8

Prerequisites

[ ] Can you reproduce the problem with Debug -> Reload Without Extensions?
[*] Did you perform a cursory search to see if your bug or enhancement is already reported?
[ ] Did you read the Troubleshooting guide?

For more information on how to write a good bug report read here
For more information on how to contribute read here
Description
Brackets cannot run on Debian 8 after updating the master branch to latest code on brackets repository.
Steps to Reproduce

Pull the latest code on brackets repository
Checked out to jasonsanjose/cef_2171_linux branch in brackets-shell repository. Updated the control file to use libcrypt20 in installer/linux/control
Run 'grunt build' in brackets repository and 'grunt installer' in brackets-shell repository. Install the brackets .deb file from installer/linux folder.
Run brackets using command 'brackets' on command line.

Expected behavior: Brackets should successfully open and run.
Actual behavior: Brackets screen opens and suddenly closes and "Segmentation fault" prints on the screen.
Output of running brackets from command line :
prateek@prateek-debian:~/work/open-source/brackets-shell/installer/linux$ brackets
[0618/235112:ERROR:browser_main_loop.cc(163)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on.
ATTENTION: default value of option force_s3tc_enable overridden by environment.
[0618/235112:ERROR:renderer_main.cc(207)] Running without renderer sandbox
[0618/235112:ERROR:nss_util.cc(819)] After loading Root Certs, loaded==false: NSS error code: -8018
Segmentation fault
Output of dmesg afterwards :
[17577.767302] Brackets[28555]: segfault at 3600001 ip 00007fd8d347c614 sp 00007ffd638ef550 error 4 in libgobject-2.0.so.0.4200.1[7fd8d3449000+51000]
[17622.581046] Brackets[28753]: segfault at 3c00001 ip 00007fb8ee9ce614 sp 00007fffe7f652d0 error 4 in libgobject-2.0.so.0.4200.1[7fb8ee99b000+51000]
[19648.369458] Brackets[3063]: segfault at 2600001 ip 00007fc440674614 sp 00007fff4eb46b60 error 4 in libgobject-2.0.so.0.4200.1[7fc440641000+51000]
Versions
Please include the OS and what version of the OS you're running. : Debian 8
Please include the version of Brackets. You can find it under Help -> About Brackets (Windows and Linux) or Brackets -> About Brackets (macOS) : 1.10.0-17439 is the version printed while installing brackets
One more thing, do you have any plans to include libgcrypt20 (as mentioned in 10255 )? Any other separate stable build for debian 8 ?
Regards,
Prateek Sharma

@pratts jasonsanjose/cef_2171_linux branch is not a stable one, there are some issues with it, I would suggest you to use linux-1547 branch(now it supports native menus also), we are currently working on upgrading CEF version in Linux also, and we will remove the dependency of libcrypt11 in that upgrade.
If latest cef vesion has libgcrypt20 dependency then we will include it also.

Thank you for your reply @saurabh95
I tried the following 2 commands and had compilation errors while building brackets-shell.
prateek@prateek-debian:~/work/open-source/brackets-shell$ git checkout -b linux-1547 upstream/linux-1547
prateek@prateek-debian:~/work/open-source/brackets-shell$ grunt installer
Running "git:www" (git) task
Skipping fetch for www repo
Running "git:shell" (git) task
Skipping fetch for shell repo
Running "create-project" task
Building project files
Running "build-www" task
/home/prateek/work/open-source/brackets/dist
Running "build" task
Running "build-linux" task

In file included from appshell/command_callbacks.h:4:0,
from appshell/client_handler.h:14,
from appshell/appshell_extensions.h:27,
from appshell/appshell_extensions.cpp:24:
appshell/appshell_extensions_platform.h: In function ‘void* getMenuParent(CefRefPtr)’:
appshell/appshell_extensions_platform.h:76:57: error: ‘GTK_WIDGET’ was not declared in this scope
GTK_WIDGET(browser->GetHost()->GetWindowHandle()),
^
appshell/appshell_extensions_platform.h:77:9: error: ‘GTK_TYPE_VBOX’ was not declared in this scope
GTK_TYPE_VBOX);
^
appshell/appshell_extensions_platform.h:77:22: error: ‘gtk_widget_get_ancestor’ was not declared in this scope
GTK_TYPE_VBOX);
^
In file included from ./include/cef_app.h:42:0,
from appshell/client_app.h:13,
from appshell/appshell_extensions.h:26,
from appshell/appshell_extensions.cpp:24:
appshell/client_handler.h: At global scope:
./include/cef_base.h:178:11: error: ‘Lock’ in namespace ‘base’ does not name a type
base::Lock lock_;
^
appshell/client_handler.h:294:3: note: in expansion of macro ‘IMPLEMENT_LOCKING’
IMPLEMENT_LOCKING(ClientHandler);
^
In file included from appshell/appshell_extensions.h:27:0,
from appshell/appshell_extensions.cpp:24:
appshell/client_handler.h:160:16: error: conflicting return type specified for ‘virtual void ClientHandler::OnRequestGeolocationPermission(CefRefPtr, const CefString&, int, CefRefPtr)’
virtual void OnRequestGeolocationPermission(
^
In file included from ./include/cef_client.h:48:0,
from appshell/client_handler.h:12,
from appshell/appshell_extensions.h:27,
from appshell/appshell_extensions.cpp:24:
./include/cef_geolocation_handler.h:76:16: error:   overriding ‘virtual bool CefGeolocationHandler::OnRequestGeolocationPermission(CefRefPtr, const CefString&, int, CefRefPtr)’
virtual bool OnRequestGeolocationPermission(
^
In file included from ./include/cef_app.h:42:0,
from appshell/client_app.h:13,
from appshell/appshell_extensions.h:26,
from appshell/appshell_extensions.cpp:24:
appshell/client_handler.h: In member function ‘void ClientHandler::Lock()’:
./include/cef_base.h:175:19: error: ‘lock_’ was not declared in this scope
void Lock() { lock_.Acquire(); }
^
appshell/client_handler.h:294:3: note: in expansion of macro ‘IMPLEMENT_LOCKING’
IMPLEMENT_LOCKING(ClientHandler);
^
appshell/client_handler.h: In member function ‘void ClientHandler::Unlock()’:
./include/cef_base.h:176:21: error: ‘lock_’ was not declared in this scope
void Unlock() { lock_.Release(); }
^
appshell/client_handler.h:294:3: note: in expansion of macro ‘IMPLEMENT_LOCKING’
IMPLEMENT_LOCKING(ClientHandler);
^
appshell/appshell_extensions.cpp: In member function ‘virtual bool appshell_extensions::ProcessMessageDelegate::OnProcessMessageReceived(CefRefPtr, CefRefPtr, CefProcessId, CefRefPtr)’:
appshell/appshell_extensions.cpp:365:53: error: ‘class CefBrowserHost’ has no member named ‘GetDevToolsURL’
ExtensionString url(browser->GetHost()->GetDevToolsURL(true));
^
make: * [out/Release/obj.target/Brackets/appshell/appshell_extensions.o] Error 1
Warning: Task "build-linux" failed. Use --force to continue.

Aborted due to warnings.
prateek@prateek-debian:~/work/open-source/brackets-shell$

Fixed By: https://github.com/adobe/brackets-shell/pull/619


[DDW-431] Prevent multiple Daedalus instances per state dir

This PR implements a lock file mechanism that prevents multiple Daedalus instances from running against the same app state directory.
On the way the config / setup code has been simplified and improved due to the fact that we don't support starting Daedalus without nix-shell anymore.
How to test:
Open two nix-shell environments and run the app twice with NODE_ENV=development yarn start
Screenshots:

New error on startup when trying to start another instance:

Review Checklist:
Basics

[ ] PR is updated to the most recent version of target branch (and there are no conflicts)
[ ] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance tests are passing (yarn run test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn run dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn run package / CI builds)
[ ] There are no flow errors or warnings (yarn run flow:test)
[ ] There are no lint errors or warnings (yarn run lint)
[ ] Text changes are proofread and approved (Jane Wild)
[ ] There are no missing translations (running yarn run manage:translations produces no changes)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn run storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly documented and commented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-rendering
[ ] Any code that only works in Electron is neatly separated from components

Testing

[ ] New feature / change is covered by acceptance tests
[ ] All existing acceptance tests are still up-to-date
[ ] New feature / change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

@DominikGuzei we have the state directories separation ready in this PR: https://github.com/input-output-hk/daedalus/pull/1114#pullrequestreview-162083785 - big thanks to @cleverca22 🎉


avoid using a schema

How can I disable the use of a schema in .taplo.toml, I have a file called theme.toml, but I get errors because it automatically uses the schema of https://schemas.wp.org/trunk/theme.json, which is not required and does not correspond to my file, I have tried several ways and I can not exclude.
{"jsonrpc":"2.0","method":"taplo/didChangeSchemaAssociation","params":{"documentUri":"file:///D:/**/**/theme.toml","schemaUri":"https://schemas.wp.org/trunk/theme.json","meta":{"name":"theme.json","description":"WordPress block theme global settings and styles configuration file","source":"catalog","catalog_url":"https://www.schemastore.org/api/json/catalog.json"}}}

This should be fixed in the latest releases.


Fix bug when register is called before lang loading and allow implicit rules registration

Thanks for this. Would you mind adding some tests for the implicit rule registering?


list(bucket()) hangs with 100% CPU

From the docs:
b = bucket(['a1', 'b1', 'c1', 'a2', 'b2', 'c2', 'b3'], lambda x: x[0])

The following uses 100% CPU and never returns:
list(b)

Additionally, dict(b) fails with dictionary update sequence element #0 has length 0; 2 is required, leaving no way to inspect the opaque result without prior knowledge of all keys.
This occurs in all versions of more-itertools.

bucket itself is an infinite iterable. That's why it hangs.
However maybe bucket is only iterable by accident (it implements __getitem__ so it's iterable) - I'm not sure.
But yeah, bucket (as it currently is) isn't meant to be iterated over without knowledge of the keys.

Correct - this was discussed at the time of implementation.

Because __getitem__ is implemented, when calling list() or dict() on bucket(something) causes Python to look up 0, 1,, 2... on to infinity.
https://github.com/erikrose/more-itertools/pull/371 prevents this by explicitly defining __iter__. Now calling list() on bucket(something) will extract the keys from the iterable and return them.


Improvement / dataset metadata

There is currently no way to query for the metadata associated with a particular dataset. One can only look up metadata by tag, which requires some auxiliary knowledge about the metadata.
Changes proposed in this pull request:

Add a metadata property to a DataSet object to see all associated metadata.

This little improvement is needed for #1352 in order to ensure correct copying of metadata.
@astafan8

Comment: you might worry why snapshot is missing from RUNS_TABLE_COLUMNS. See the test. I think I made the correct choice, but perhaps we should upgrade the schema later.


Kafka Event Bus write-up

@mandy-chessell do you happen to remember who was going to write this one up?


Update retention-cmdlets.md

Updated description for ComplianceTag
Updated description for CompliancePolicy to call out label policies

#sign-off  Merging


workspace genome collection date formatting issue?

for many genomes, the collection date (at least as displayed on the website in the genome list, either production or beta) is only a year (has to do w/ HIPAA).   For example Mycobacterium bovis B2 7505, collection date is 2006, but if I add that genome to my workspace and look at the collection date as part of a genome group it says 12/31/05

Harry fixed this issue when fixing genome metadata.


Docker Image - Cannot Import Tensorflow

Hello
I built and run the image as described, with small modifications for mac M1 host and V2L target
DRP-AI Translator is v1.82 (https://www.renesas.com/us/en/products/microcontrollers-microprocessors/rz-arm-based-high-end-32-64-bit-mpus/drp-ai-translator)
docker build -t drp-ai_tvm_v2ma_image --platform linux/amd64 --build-arg SDK="/opt/poky/3.1.14" --build-arg PRODUCT="V2L" .

docker run -it --platform linux/amd64 --name drp-ai_tvm_v2ma_container -v $(pwd)/data:/drp-ai_tvm/data drp-ai_tvm_v2ma_image

However, there is some issue with tensorflow in the image, e.g.

root@cd50204bcf68:/drp-ai_tvm# python3
Python 3.8.10 (default, Mar 13 2023, 10:26:41)
[GCC 9.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow
[libprotobuf ERROR external/com_google_protobuf/src/google/protobuf/descriptor_database.cc:560] Invalid file descriptor data passed to EncodedDescriptorDatabase::Add().
[libprotobuf FATAL external/com_google_protobuf/src/google/protobuf/descriptor.cc:1986] CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size):
terminate called after throwing an instance of 'google::protobuf::FatalException'
  what():  CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size):
qemu: uncaught target signal 6 (Aborted) - core dumped
Aborted

Any suggestion to resolve this?

in the meantime, removing tensorflow import in compile_tflite_model.py seems to work, as tf is not used in that module.

Hello Jotham-san,
Our tool works only with Intel-based Linux and it does not support Apple M1-based environments.
This problem may be caused by the difference between the Intel and M1 environments, so I'm sorry, but could you try it in intel-based environment?


Develop resolve

This merges in feature/nightly, feature/h5_homology_dataset, and feature/amino_acid_tokenizers

"If you break it fix it!" - Alex


Cannot read property 'token' of undefined on FoundryVTT 0.8.6

Environment Details
Foundry VTT Version: 0.8.6
Operating System: [Linux]
How Are You Using Foundry: [Linux,Native Application (Electron), Chrome]
Which Game System: Dungeons and Dragons 5th Edition, version 1.3.3
Modules Enabled?: Yes
OS, Hosting, Browser (if applicable): Linux/Node/Chrome
Module Version: 0.3.0
** Description of the issue: **

What are you doing when this error happens?

After some test a use case it's after i made a attack while i'm in combat, I'm will try to investigate some module conflict.

Fixed in latest version.


How do you "forget" a public folder?

I haven't found a good way to do this: Following the guide, I added a user's public folder and confirmed I could see it in /keybase/public. Now, however, I'd like to remove it from Finder / the Keybase menu / etc.
What's the proper way to do this? It persists across kbfs remounts and moving to trash doesn't work.

I'm sorry, we don't have support for this yet.  Coming soon!

Ah, good to know. Thanks! I'll close this out since it sounds like it's already on your todo list.


Bdqc tutorial

Hi,
This is a tutorial on Blind Delegated Quantum Computing that I would like to contribute.
This tutorial requires basic knowledge of MBQC which I contributed in https://github.com/Qiskit/textbook/pull/58
This tutorial covers a whole topic on its own, but the MBQC is a building block. Think of this as having a tutorial on Quantum circuits, then another on one of the algorithms we can do as a quantum circuit.

@atharva-satpute the latest commit 2cd1eeb should have typos (including your edits) and few other mistakes fixed. Thanks

Thanks for your PR. As with your other PR (#58), I'm going to close it as we can't accept new content from the community at the moment. I think it would be great if you self-published this page too; I can help you set things up and/or leave review comments if you wish.
Please message me on the Qiskit Slack workspace if you want to chat.


Added Singleton Yasnippet

I grabbed this template by what Android Studio produces when you create a singleton class.

Hmm, Singleton is not often used pattern, it sometimes leads bad design and you can implement robust Singleton by using Enum. I am not so positive adding this. If there are others want this, please leave a comment so I'll merge it later.


Stream fails after 1-2 seconds

I'm running XBMC 13.1 on Android (ARM). This plugin loads, and plays 1-2 seconds of the news stream before crashing back to the XBMC menu.
It's an intermittent error. I would say 1 in 5-6 attempts at launching the addon with succeed, and the rest crash to XBMC or occasionally to the desktop.

On my Raspberry Pi running latest Raspbmc/Kodi (Raspian/DebianARM) the stream usually fails to start for me too. The plugin used to work reliably, but started having issues a while ago (not only with the most recent Raspbmc update, it started some time ago).

Seems to be working for me on Kodi 14.1 on ArchLinux. Haven't tested on the RPi yet though.
Are you still seeing this issue? How about a copy of your log file?

A log file would help if it includes any messages from the RTMP library. But since you say it is intermittent, my gut says it is the same issue that I was hinting at https://github.com/andybotting/xbmc-addon-abc-iview/issues/31#issuecomment-41284476. Updating the RTMP library to after 2014-03 might help.


Cannot use lambda parameters in Page Builder API elements

What went wrong? 🤔
Run the code below.
The terminal will show that the lambda parameters cannot be bound.
Now replacing the page with the Markdown equivalent:
page = Markdown(
    "<|{data}|table|style={lambda _, row: 'blue-row' if row % 2 else 'red-row'}|show_all|>",
    style={
        ".blue-row>td": {"color": "white", "background-color": "blue"},
        ".red-row>td": {"color": "yellow", "background-color": "red"},
    },
)

everything works properly.
Expected Behavior
tgb or Markdown should behave similarly. Meaning: the lambda works.
Steps to Reproduce Issue
import taipy.gui.builder as tgb
from taipy.gui import Gui

x_range = range(-10, 11, 4)
data = {"x": x_range, "y": [x * x for x in x_range]}

with tgb.Page(
    style={
        ".blue-row>td": {"color": "white", "background-color": "blue"},
        ".red-row>td": {"color": "yellow", "background-color": "red"},
    }
) as page:
    tgb.table("{data}", style=lambda s, row: 'blue-row' if row % 2 else 'red-row', show_all=True)

Gui(page=page).run(stylekit=False)

Workaround
Set the lambda function as an expression in a string:
tgb.table("{data}", style="{lambda s, row: 'blue-row' if row % 3 else 'red-row'}", show_all=True)

Version of Taipy
develop
Additional Context
Trace in terminal:
--- 4 warning(s) were found for page '/'  ---
 - Warning 1: Variable 's' is not available in either the '__main__' or '__main__' modules.
 - Warning 2: Variable 's' is not defined (in expression '{__lambda_5531718c857b4d30be287c5a1105ad15(s, row)}'):
Traceback (most recent call last):
  File "...\taipy\gui\utils\_evaluator.py", line 122, in _analyze_expression
    var_val[var_name] = _getscopeattr_drill(gui, encoded_var_name)
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "...\taipy\gui\utils\_attributes.py", line 26, in _getscopeattr_drill
    return attrgetter(name)(gui._get_data_scope())
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'types.SimpleNamespace' object has no attribute 's'

 - Warning 3: Variable 'row' is not available in either the '__main__' or '__main__' modules.
 - Warning 4: Variable 'row' is not defined (in expression '{__lambda_5531718c857b4d30be287c5a1105ad15(s, row)}'):
Traceback (most recent call last):
  File "...\taipy\gui\utils\_evaluator.py", line 122, in _analyze_expression
    var_val[var_name] = _getscopeattr_drill(gui, encoded_var_name)
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "....\taipy\gui\utils\_attributes.py", line 26, in _getscopeattr_drill
    return attrgetter(name)(gui._get_data_scope())
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'types.SimpleNamespace' object has no attribute 'row'

----------------------------------------------

### Acceptance Criteria

- [ ] Ensure new code is unit tested, and check code coverage is at least 90%.
- [ ] Create related issue in taipy-doc for documentation and Release Notes.

### Code of Conduct

- [X] I have checked the [existing issues](https://github.com/Avaiga/taipy/issues?q=is%3Aissue+).
- [ ] I am willing to work on this issue (optional)

the style property here should be bound to a callback but the parsing doesn't know about it


Create 205-Isomorphic-Strings.js

File(s) Modified:  205-Isomorphic-Strings.js
Language(s) Used: javascript
Submission URL: https://leetcode.com/submissions/detail/852204920/

Thanks, @Krishvora!


Added meta tags for improved (mobile) experience

This PR fixes the issue reported in https://github.com/spatie/laravel-health/issues/54. Currently on iOS phones (confirmed in Safari) the page scales down to two columns instead of one:

By adding the viewport meta tag the page will scale down correctly on mobile devices:

I also added the charset meta tag which helps displaying special characters correctly.

Thanks!


Should be patched in the latest commit!


SOLR-17367 - org.apache.solr.cli.PostTool is ignoring -params option

https://issues.apache.org/jira/browse/SOLR-17367

Description
Adding support for -params option in org.apache.solr.cli.PostTool
Solution
I modified the postData method so that it checks and appends -params value to the solr request
Tests
I implemented testRunCsvWithCustomSeparatorParam in org.apache.solr.cli.PostToolTest. The test creates a tsv and submits it to the test core. It then checks to see if the index contains exactly 1 document.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

thanks for the addition of the tests..  I will inline some comments....   one thing, did you run gradlew tidy?  that does some code formatting.

thanks for the addition of the tests.. I will inline some comments.... one thing, did you run gradlew tidy? that does some code formatting.

I did not 🤦♂.
I had a blast writing java for the first time in 10 years. I promise I'll try to contribute more to the project, as it's a key part in my job.

Looks like some great progress. A couple of questions! When I add it to CHANGES.txt, how would you like to be credited?

Bostoi
Thank you

thanks for the addition of the tests.. I will inline some comments.... one thing, did you run gradlew tidy? that does some code formatting.

I did not 🤦♂. I had a blast writing java for the first time in 10 years. I promise I'll try to contribute more to the project, as it's a key part in my job.

The only thing i'm not sure about is passing the params to the /extract component of the webMode.

One more thought which is that sometimes I find it easier to write a bats test instead of Java...   post.bats is used to test out the cli against a live solr...

by the way @bogdanstoik how did you find this bug?   Did you have a scripte using the -params?

by the way @bogdanstoik how did you find this bug? Did you have a scripte using the -params?

Yes. I was trying to load a tab-separated csv and it kept complaining about not finding the right columns. I ended up using curl to post my file directly to the update handler, then came back to check if's something in the tool.


Use ISO format datetime set at beginning of build

These are the modifications I've made on the JWST side.  I think having ISO format datetime makes them more easily sorted in the results.  And having the matrix and build info in the build directory instead of attached to each test directory makes sense for us too - less repeating information, and more easily-mapped output dir name to test name.
The format is essentially now:
repo-info
     \_______date-build-matrix-info
                      \_______________test-info

So instead of results in the repository organized like
01Dec2018T/test_nrs_fs_multi_spec2_test_n0_jenkins-RT-JWST-60_standalone/
01Nov2018T/test_nrs_fs_multi_spec2_test_n0_jenkins-RT-JWST-24_standalone/

one gets
2018-11-01_jenkins-RT-JWST-24_0/test_nrs_fs_multi_spec2_test_n0
2018-12-01_jenkins-RT-JWST-60_0/test_nrs_fs_multi_spec2_test_n0

The datetime was pulled into a global variable as for jwst, our regregression test runs take over 3 hours, and if they kick off at 11pm, generate_upload_params() could get called spanning 2 days, which means the same test run would get split over two directories based on two datetimes.  Better to have all results in one output dir.

If not run locally (not on Jenkins), the upload patterns look like
"2018-12-11_NOT_CI_jdavies_0/test_nrs_fs_multi_spec2_test_n0/"


ekf2: Allow use of RTK GPS heading

Adds feature requested by https://github.com/PX4/Firmware/issues/10284
Requires https://github.com/PX4/ecl/pull/497
Is inactive with default parameters.
Testing
See https://github.com/PX4/ecl/pull/497 for details of previous testing using replay.
Testing this branch requires setting EKF2_AID_MASK = 129 and EKF2_MAG_TYPE = 5. That will force the EKF to wait until GPS yaw is available before performing the yaw alignment. If EKF2_MAG_TYPE = 0, then the yaw will be set using the magnetomer intially and then reset to the GPS yaw when it becomes available.
Testing using replay was repeated and documented in the comments below.

Results from replay of log file
https://logs.px4.io/plot_app?log=d9bc48f9-3270-4ff9-8141-c33a006f9f9d

Note: The jump in heading innovations when turning are the result of the bug in PX4/Firmware that was present when the original log data was generated. This was been subsequently fixed upstream by
https://github.com/PX4/Firmware/pull/10346
Replay log here: https://logs.px4.io/plot_app?log=67bf7874-8a52-4c56-9baf-eb9d46985dc1

@bkueng Are you able to repeat the test you did here: https://logs.px4.io/plot_app?log=25380997-1d97-4e3c-8006-58fd4d4e52c5 and supply the data log?
This data will confirm the combination of recent driver changes and this PR provide adequate protection against gimbal lock and angle wrapping.

Here are 2 logs on this branch with enabled GPS yaw fusion (logging from boot until shutdown):

https://logs.px4.io/plot_app?log=1005486e-e7f4-4088-9b20-22c48036d22a
https://logs.px4.io/plot_app?log=51d7e1e3-8562-46d3-8b3c-960e008516b5

The second one contains the rolling/pitching towards the end of the log.

Inspection of the heading and GPS position and velocity innovations show that data is being fused correctly and the vehicle has a valid heading. Heading fusion stops as designed when the GPS antennas go through a bad orientation.
51d7e1e3-8562-46d3-8b3c-960e008516b5.ulg innovations

1005486e-e7f4-4088-9b20-22c48036d22a.ulg innovations - these are larger than for the previous log

One of the limitations with the interface used by this type of GPS is that there is no data to populate the s_variance_m_s field so the default observation noise of 0.5 m/s as set by the EF2_GPS_V_NOISE parameter is used at all time regardless of receiver signal quality. Something to look at for later improvement is scaling the noise using eph if the s_variance_m_s data is not populated.

Once we get an agreed method for messaging the heading offset, then this can be merged.

@dagar I've added the heading offset to the GPS message along with protection in the ecl library if the heading_offset is undefined.
The logs don't contain the additional heading_offset message, so the previous logs can't be used to test the change. Once you are happy with the changes, I will ask @beat to generate a new log when he has time.

Small failure ecl side.

@priseborough Please stop using @beat as alias, as I'm not the Beat Küng you are referring to. You should be using @bkueng for Beat Küng instead. That will for sure also speed up his replies. Thanks! :smile:

@bkueng That is correct. I am still working out the best way to achieve that. The easiest is to always pass through the heading from the receiver that has it. If both do, then blend it using the position accuracy ratio, given we do not have a heading accuracy value from the receiver.

@bkueng That is correct. I am still working out the best way to achieve that. The easiest is to always pass through the heading from the receiver that has it.

I think passing it through is good enough for now.

The heading_offset is 0 and the heading is NAN in that log. Did you upload the right file?

The heading is there:

The offset is indeed 0, but the GPS_YAW_OFFSET parameter is set to 90.
The problem is that the heading offset gets reset here: https://github.com/PX4/Firmware/blob/f1d08d90dadc77c60df502533f9b1cf94a868241/src/drivers/gps/gps.cpp#L707.

What program are you using to plot it? I am using FlightPlot v0.3.2 and it doesn't display, eg

Ah, you need to check the Markers checkbox in the upper left corner. Because there are NaN values, the lines are not plotted.

Other than 0 value for heading_offset, log data looks OK.

I've pushed a fix for the heading_offset reset.

Looks good to go. Once https://github.com/PX4/ecl/pull/497 is merged this needs to be updated to point to ecl master.

https://github.com/PX4/ecl/pull/497 has been merged so I'll clean up the commit history and push.

Well that didn't take long.

@priseborough I don't think we're there yet, but a rebase couldn't hurt to see how close we're getting.

We are now a lot closer on px4fmu-v2

The omnibus-f4sd build is in worse shape

@bkueng drop VTOL and FW from omnibus? We can fix any parameter interdependencies that have crept in.
https://github.com/PX4/Firmware/blob/master/cmake/configs/nuttx_omnibus-f4sd_default.cmake#L87-L93

@bkueng drop VTOL and FW from omnibus? We can fix any parameter interdependencies that have crept in.

Sounds good. Meanwhile we can quickly disable the topic listener (https://github.com/PX4/Firmware/pull/10533) to unlock this.

I've rebased. We are still 1345 bytes short on the px4fmu-v2 target.

@bkueng Could you have a look?

@priseborough you can disable any of the distance sensors: https://github.com/PX4/Firmware/blob/master/cmake/configs/nuttx_px4fmu-v2_default.cmake#L35

@bkueng Thanks! Do we have a more structured approach at fixing it?

One thing that concerns me about the module disabling: for fmu-v2, we are disabling several modules, but we are going to get to a stage where I do want to use one module in specific on the fmu-v2, I won't be able to do it because of flash overflow. Though I do see that the ekf2 and ecl modules keeping growing in size, but I don't see where exactly are able to, for example, ignore the build of some of each features. For example, this one in particular: not everyone would actually want to use the RTK heading, and eventhough the parameter is there, the code for this feature is going to be built. If this was a critical feature, I would say ok, this needs to go in. But I don't really see the scalability of this, while we keep adding new features to the estimator and ECL and don't have a way of filtering which features we want to build, keeping accumulating flash space without even wanting to use those same features.
IMHO, the way we add the build for these kind of features should be handled the in a similar way we do with modules and drivers we don't want to build for a certain target.

@dagar and @bkueng Now that this has been rebased and is now passing CI given the available flash space, can one of you approve it before the that flash gets eaten up by something else .
TXS

@priseborough would you mind giving some feedback on my comment? Thanks

@TSC21 it's a more general problem and ecl/EKF just happens to be the most recent victim. There's quite a lot of flash that could be regained from the drivers themselves with no loss of functionality. Ping me if you're interested.


When importing Angular AmplifyService it introduces ERROR in Cannot read property 'length' of undefined

Cross posting.  Is Amplify JS Still alive?
https://github.com/aws-amplify/amplify-js/issues/4920

@fireflysemantics Yes, it's very much alive and I've informed the team to take a look at your request.
We look at GitHub issues at first come first serve basis and the team will review the request accordingly.


[RESOLVED] Unavailable by ICMP ping

Problem started at 07:50:33 on 2023.06.20
Problem name: Unavailable by ICMP ping
Host: S100-Santiago
Severity: High
Operational data: Down (0)
Original problem ID: 28957
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23516&eventid=28957

Problem has been resolved in 11m 0s at 08:01:33 on 2023.06.20
Problem name: Unavailable by ICMP ping
Host: S100-Santiago
Severity: High
Original problem ID: 28957


Bug: Default settings are not loaded in the project

Default settings are not loaded in the project. When there is no SIMPLE_API_KEY setting provided, Django will throw an error.
  File "/Users/koladev/projects/test-project/venv/lib/python3.11/site-packages/rest_framework_simple_api_key/models.py", line 15, in APIKeyManager
    key_crypto = ApiKeyCrypto()
                 ^^^^^^^^^^^^^^
  File "/Users/koladev/projects/test-project/venv/lib/python3.11/site-packages/rest_framework_simple_api_key/crypto.py", line 20, in __init__
    fernet_key, api_key_lifetime = settings.SIMPLE_API_KEY.get(
                                   ^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/koladev/projects/test-project/venv/lib/python3.11/site-packages/django/conf/__init__.py", line 94, in __getattr__
    val = getattr(_wrapped, name)
          ^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'Settings' object has no attribute 'SIMPLE_API_KEY'

I'm a new contributor and I would like to work on this issue.
How can I recreate the error you posted above?

safsfasdf

Hi @cgelb
the issue has already been addressed in this PR #44
In the future, we will try to be more precise in the description of the issues and how the fixes are going.
Please, feel free to check any other issues in the project. :)


Send activate and deactivate command to ShellSpout

By sending activate and deactivate to ShellSpout it is possible to react on these events i.E. closing/opening resources

LGTM +1

I like that idea but in order to add this we need to address this to multilang implementations, and doc to announce new feature.
I'd be happy if @obiwabrakenobi follows up this issue, but if in case of not available, I can take it over.

i'll update the docs @HeartSaVioR

+1
I would like to see this made available so it can be used in pystorm/streamparse.

@obiwabrakenobi Any updates here? If you're not available I'll file an issue and take it over.

i'll finish it until the end of this week...is was busy on vacation :) @HeartSaVioR

@HeartSaVioR docu is updated, do you need a jira issue too?

@obiwabrakenobi Yes please. This is worth to be filed to issue.

@HeartSaVioR done. https://issues.apache.org/jira/browse/STORM-2294

@obiwabrakenobi
Please change your PR title to have STORM-2294 as prefix. If possible squashing your commits into one would be more appreciated.

well, i try to rebase, but not sure if that works properly

@obiwabrakenobi I'll try to take care of rebasing while merging.


Selected option in template

Using
{{option....}}
 one can access all properties of option to build the selection list display.
One can also use option-property or option-getter to get/build the ngModel value.
But then, one can't get a handle on the selected option.
It would be nice to have the selected option, if any, available in scope inside the directive so one can build a proper display of the selected option out of the modal.
Something like:
<some-element modal-select
      ng-model="myModel.someRelationship
      options="myOptions"
      option-property="id">

{{option.firstName}} {{option.lastName}}

Please select something

{{selected.firstName}} {{selected.lastName}}

</some-element>

What do you think?

hi @eskatos,
makes sense to me, and should be quite easy to implement.
thanks for the idea, I will add this feature ASAP.
best regards
Mauro


Libvirt: Dynamic port forwarding from host VM to guests

This PR address multiple guest support for CH/QEMU platforms with dynamic port forwarding.
cc: @anirudhrb @cwize1

@cwize1 Please help reviewing.


nixos-option: set platforms.all

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 132970 at 8f18af26 run on x86_64-linux 1

2 packages built successfully:

nixos-option
tests.trivial


[CARBONDATA-1181] Show partitions

Show table partition information

Can one of the admins verify this patch?

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2689/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/116/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/613/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2690/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/117/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/614/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/615/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2691/

Can one of the admins verify this patch?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/616/

Thanks for working on this issue. You always can use the same PR by rebasing the code with master and force pushing to this PR branch. Raising multiple PRs for same issue is not required.
rebase -uf  :


Fix date to timestamp conversion

Fixes the issue in processAnalyticsFromLogFile as reported by @AnthonyGiorgio by embedding date into utilities:

parse error: Invalid numeric literal at line 1, column 10
FSUM9364 Bad date conversion: 2023-10-06 10:42:29
Usage: date [-cu] [-d time] [+format]

Also embeds sha256sum, which would be used to verify the checksum matches the metadata.json's checksum

Note:
Another approach would be to embed coreutils (which includes date, sha256sum and many other tools). Another benefit of this is that we don't have to rely ascii workarounds like asciiecho (https://github.com/ZOSOpenTools/meta/blob/main/bin/zopen-build#L40) and we also can replace numcpus.rexx with nproc. It will increase size by 14mbs though.

This is fine but can you create another issue so that we can pick up fixes to date/sha256sum if they are required as part of the build process of meta?

Will do


Ability to add tags for item

Closes #99
Frontend:

Backend:

Seems to work great, small design flaw, it seems there's a line here even though there's no tags attached:

Also, it would be great if admins could manage the already created tags (and once you delete them, they detach from the already created items).

@Cannonb4ll everything done I think

Thanks! 🤝


mumble: 1.4.287 -> 1.5.517

Description of changes
Contains  #199111
We can wait for a stable (non RC) release of mumble, if the maintainers prefer that.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

I don't want to start that discussion here, but not sure how to reach out else.
I am @felixsinger writing from this account because I am unable to review this PR and write comments with my other account, while I am one of the maintainers for this package. I am able to review other PRs and others are able to review this PR as well. However, if you (@ajs124) blocked me somehow, please remove that. If you got a problem with me then please reach out to me and let's try resolving it, but blocking me and making me unable to review PRs doesn't seem very reasonable.

I talked to the Mumble developers. This release candidate shouldn't be used since it has issues which need to be fixed. Due to lack of time it takes longer than expected.


terminus drush behaving strangely in today's HEAD of master

$ ./bin/terminus --site=quicker-8 --env=dev drush status Drupal
This is now running "drush Drupal" instead of "drush status Drupal".
Argument handling seems to be shifting all the args over when there are multiple args on the command line.

Experiencing the same. Works fine on 0.9.3, but breaks as of v0.10.0.

Looks like this is the issue. https://github.com/pantheon-systems/cli/commit/a6068a2c656c90ae19d623436e21637e2fb85dcc

See #761 761


Consider further isolation of consensus related code

We already have primitive/consensus and client/consensus folders to keep together consensus related code.
Currently these folders contains code for protocols related to block authoring such as BABE, AURA, PoW and some shared utility code.
I propose to add to these folders everything related to consensus (thus also finality) such as grandpa and beefy crates.
Even though this is not paramount, I can't see any good reason to have these crates floating with the rest of the code.
In general  I think is a good practice to logically organize the code.
If there isn't any drawback I can easily take on this trivial chore.

If there isn't any drawback I can easily take on this trivial chore.

(commenting in the role of an outside observer)
https://github.com/paritytech/substrate/labels/I7-refactor
The question is: Can this task ("further isolation of consensus related code") be done by strict refactoring?
(Strict Refactoring: zero functional change)
Because even a strict refactoring can still introduce side-effects (things usually visible only in the machine-code, link long vs. short jumps etc.).
https://github.com/paritytech/substrate/labels/Z0-trivial
Refactoring should not be labeled as trivial - because it is not trivial.
(Should have own label, like Z9-The-Art-of-Code)

This is mostly about renaming


Attempting to upload a binary file stops the Docker image

Describe the bug
When attempting to hit a file upload endpoint that accepts a binary type, the docker errors and exits.
To Reproduce

Given this OpenAPI document:

openapi: 3.0.3
info:
  title: Spotlight Error Example
  description: "Spotlight Error Example"
  version: 1.0.0

servers:
  - url: https://localhost:4010
    description: Mock server (uses mock data)

paths:
  /file-upload:
    post:
      summary: Upload file
      description: Upload a file
      operationId: uploadFile
      tags: ["Files"]
      requestBody:
        content:
          multipart/form-data:
            schema:
              type: object
              properties:
                file:
                  type: string
                  format: binary
      responses:
        "201":
          description: Successfully uploaded file
          content:
            application/json:
              schema:
                type: object
                properties:
                  url:
                    type: string
                    format: uri
                    description: The URL from which the file can be downloaded.
              example:
                url: https://example.com/image.png

Run Start the docker image:

docker run --rm -v $(pwd):/tmp -p 4010:4010 stoplight/prism:4 mock -h <IP_ADDRESS> "/tmp/openapi.yaml"

Upload a file:

curl --location --request POST 'http://localhost:4010/file-upload' \
--form<EMAIL_ADDRESS>

See error:

$ docker run --rm -v $(pwd):/tmp -p 4010:4010 stoplight/prism:4 mock -h <IP_ADDRESS> "/tmp/openapi.yaml"
[2:07:48 PM] › [CLI] …  awaiting  Starting Prism…
[2:07:49 PM] › [CLI] ℹ  info      POST       http://<IP_ADDRESS>:4010/file-upload
[2:07:49 PM] › [CLI] ▶  start     Prism is listening on http://<IP_ADDRESS>:4010

/usr/src/prism/node_modules/split2/index.js:44
      push(this, this.mapper(list[i]))
                      ^
SyntaxError: Unexpected token ^ in JSON at position 623220
    at Transform.parse [as mapper] (<anonymous>)
    at Transform.transform [as _transform] (/usr/src/prism/node_modules/split2/index.js:44:23)
    at Transform._read (/usr/src/prism/node_modules/split2/node_modules/readable-stream/lib/_stream_transform.js:177:10)
    at Transform._write (/usr/src/prism/node_modules/split2/node_modules/readable-stream/lib/_stream_transform.js:164:83)
    at doWrite (/usr/src/prism/node_modules/split2/node_modules/readable-stream/lib/_stream_writable.js:409:139)
    at writeOrBuffer (/usr/src/prism/node_modules/split2/node_modules/readable-stream/lib/_stream_writable.js:398:5)
    at Transform.Writable.write (/usr/src/prism/node_modules/split2/node_modules/readable-stream/lib/_stream_writable.js:307:11)
    at Socket.ondata (_stream_readable.js:718:22)
    at Socket.emit (events.js:314:20)
    at addChunk (_stream_readable.js:297:12)

$

Expected behavior

Docker image not to exit
Example response returned:

{
  "url": "https://example.com/image.png"
}

Environment (remove any that are not applicable):

Docker version: 4
OS: MacOS
REST Client: Postman

I have the exact same error trace (same files and lines), but with a different unexpected token. I'm not uploading any files, but I'm not sure what is going wrong in the JSON being passed. Is there some way to see the complete "wrong" JSON, maybe in some logs?

I have the same error. The error does not seem to occur when the upload is less than ~100kb. The error does not occur when running outside of a docker container (e.g. npx prism mock my-api.yaml allows for essentially an unlimited binary upload)


gather/publish a list of values that rchain would like to convey in its content

Mission:
The RChain Cooperative’s mission is to build a decentralized, economically sustainable
public compute infrastructure. The platform boasts a modular, end-to-end design that commits to
correct-by-construction software and industrial extensibility. It’s multi year undertaking and involves
many different part of software components.
The cooperative attracts members that share the mission and the values of the cooperative:
● We make decisions. Decisions have to be supported at least three members.
● We are courageous. We are allowed to do the wrong thing, because otherwise, we can never
do the right thing either.
● We share profits according to SMART participation in results.
● We advance one another. We depend on our cohesion.
● We trust one another. We know that each and every one of us wants the best for the
cooperative.
● We take initiatives and respect those of others.If we see something we dislike, we respond
by making and spreading something we like, instead of pointing out what we dislike. We
need diversity in our activism and strive for it.
● We respect knowledge. In discussing a subject, any subject, hard measured data is
preferable. Second preference goes to a person with experience in the subject.
● We respect the time of others and the focus of the organization.
● We act with dignity. We’re always showing respect in our shaping of public opinion: respect
toward each other, toward newcomers, and toward our adversaries.
● We are long term.
The cooperative has to be managed in a way that the members believe in what they think is best for
the reaching the mission.


RelativePath in "args" is not work,but AbsolutePath can do.

Hello,
For a question , When I try to start execbeat by way of service in windows,I can't  get right result but "ERR An error occured while executing command: exit status 2" , then I change the style of args in execbeat.yml from RelativePath to AbsolutePath , I get the right result. example:
[ schedule: 0/1 * * * * ?
command: python
args: ip.py/
]    "ERR An error occured while executing command: exit status 2"
BUT
[schedule: 0/1 * * * * ?
command: python
args: D:\zork2.2.1\BDAgent\release\3rdPlugins\doubowen\testserver\ip.py
]    this way is right .I get right result.
the problem is that I must use RelativePath in my project .How can I do?
Thank you very much.

In the example with relativ path there is a backslash at the end.
Can you try the following options?:
args: .\ip.py
args: ip.py
Can you check also the log files for more details and post the log output here.

Thank you very much  christiangalsterer  ,due to I am in China so I received your retry latter.I'm sorry.
follower is the log named execbeat:
    path:execbeat/log/execbeat
    comment:
       2017-09-08T17:16:20+08:00 ERR An error occured while executing command: exit status 2
       2017-09-08T17:16:20+08:00 INFO Execbeat%!(EXTRA string=Executing command: [%v], string=)
 but now I solved this problem by user the relative path such as:
       args: "${path.config}/../../ip.py"
 and ip.py in  "../../"  base on execbeat  path
 it's very difficult for me to find the path ,however,with my friend help,I got it in execbeat.full.yml.

at the last,I want to share something:when  I user daemon to start execbeat ,no matter relative path or abs path .but when I use service to start execbeat by relative path ,maybe I should use the path above.
Thank your replay , @christiangalsterer :+1:


Add home page tests

Closes #81
Closes  #363
Changes

Update CI to Node 16 & workflows
Upgrade Cypress to v10
Addition of Percy to add visual testing (screenshots)
Addition of Cypress Dashboard

Also, I get API rate limitation when I launch the tests with yarn test. I'm not sure why.
Maybe you use the Cypress dashboard or Percy in local?
It actually does not happen for all the tests, only:
Functional tests
  home page
    Should display movies in the interactive search
  pricing page
    Should have working accordions in the faq section
Visual tests
  all tests

Same error for each test:
HttpError: The following error originated from your application code, not from Cypress. It was caused by an unhandled promise rejection.
[1]
[1]   > API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)
[1]
[1] When Cypress detects uncaught errors originating from your application it will automatically fail the current test.
[1]
[1] This behavior is configurable, and you can choose to turn this off by listening to the `uncaught:exception` event.
[1]
[1] https://on.cypress.io/uncaught-exception-from-application
[1]       at eval (webpack-internal:///./node_modules/@octokit/request/dist-web/index.js:86:27)

Also, I get API rate limitation when I launch the tests with yarn test. I'm not sure why 🤔 Maybe you use the Cypress dashboard or Percy in local? It actually does not happen for all the tests, only:

I'm pretty sure that's the call I make to github to fetch the stars. By default, you have a limited number of calls you can make (60 per hour), but in the CI I have the authenticated token which allows us to make a lot more :)

Your CI is still running on Electron 100, I saw you tried to add the browser: chrome option in the Browser tests step, was there any issue?
Other than that, everything is 💯

🦭

I had an issue where Percy was running twice for the same commit: one was ok, the second one was failing because it couldn't upload any screenshots.
I don't know if it's related, but after testing a lot of things it appears that everything works fine without the browser: chrome so I would vote to let it that way 🙈

bors merge

bors merge


Problem displaying GeoTIFF

I was trying to display a small DEM (epsg:4326) on the map. However, I noticed the DEM was being displayed at three locations with varying sizes. Only the smallest one at the upper left is the correct one.
Here is the notebook to reproduce the issue.

It works fine on my local setup but it's not even pushed to master yet, we need an ipyleaflet release before.
I will let you know when it's done, thanks for reporting the issue.

It should be fixed in v0.1.9.

It seems there is still some weird black background when displaying GeoTIFF with v0.1.9.
Here is the notebook to reproduce the issue.

Can you try removing da = xr.where(da==nan, np.nan, da)?

Fantastic! Removing that line solve the issue. I will be integrating xarray_leaflet into geemap shortly. Loading local datasets and cloud optimized GeoTIFF is a highly desirable feature. Thank you very much for your excellent work!

BTW, can the color ramp be customized?

Fantastic! Removing that line solves the issue.

I don't fully understand why removing this line makes it work though.

I will be integrating xarray_leaflet into geemap shortly. Loading local datasets and cloud optimized GeoTIFF is a highly desirable feature. Thank you very much for your excellent work!

Thanks! I'm excited to see what you will do with xarray-leaflet, I think that to have you using it is a great opportunity to improve it. I'll try to keep up with your feedback!

BTW, can the color ramp be customized?

Yes, for now it's just a matplotlib colormap so you can tweak it, see this example. Feedback is also welcome in #14.

I just figured it out.


[BIL-39] User Story 2.3: As a user, I want to view costs in a time series with daily granularity in the billing dashboard

This pull request contains:

New Billing API point GET /gcp-projects
"Project" renamed everywhere to "GCP Project"
New BillingCostByTime page & CostByTimeChart
Billing by Dataset removed as per discussion
StackedAreaByDateChart component with extra properties to make it more customisable
run npm update on node_modules/caniuse-lite

I will continue on adding new charts and convert the link from CurrentCost table to dialog, but in meantime we can merge into billing so @violetbrina you can work on the filters.


fix(pkFactory): use pkFactory for operations with insert as side effect

Description
As described in issue NODE-2275, the provided pkFactory is not being used for all _id generating methods. Or better said, all methods that can insert a document in some way.
It's being used for col.insertOne and col.insertMany, but not for findOneAndUpdate, bulk.insert or bulk.find.upsert.
This is my attempt to fix that.
Small reproduction to experience the issue yourself:
const Client = require('mongodb');

function CustomPKFactory() {}
CustomPKFactory.prototype = new Object();
CustomPKFactory.count = 0;
CustomPKFactory.createPk = function() {
return ++CustomPKFactory.count;
};

const host = 'mongodb://localhost:27017';
const options = { pkFactory: CustomPKFactory };

Client.connect(host, options, async (err, client) => {
  const db = client.db('pkFactory');
  const col = db.collection('test_custom_key');
  await col.deleteMany({});

  // ✓ expected _id: 1 | received _id: 1
  await col.insertOne({ a: 1 });

  // ⨉ expected _id: 2 | received _id: ObjectId(…)
  await col.findOneAndUpdate({ a: 2 }, { $set: { b: 2 } }, { upsert: true });

  const bulk = col.initializeUnorderedBulkOp();

  // ⨉ expected _id: 3 | received _id: ObjectId(…)
  bulk.insert({ a: 3 });

  // ⨉ expected _id: 4 | received _id: ObjectId(…)
  bulk
    .find({ a: 4 })
    .upsert()
    .update({ $set: { b: 4 } });

  await bulk.execute();
});

What changed?
Instead of simply using ObjectId to generate the id, the pkFactory is now being used.
Are there any files to ignore?
Not really, I only changed two files. Existing of 22 additions and 4 deletions. The change isn't that big.

fixes https://jira.mongodb.org/projects/NODE/issues/NODE-2275

Closing due to lack of response.


[Postgres/postgis] 3 or 4 dimensional geometries are shown as 2D

System information:

Ubuntu 18.04
DBeaver 5.3.5
Java version 1.8.0_201

Connection specification:

Postgres 9.6

Describe the problem you're observing:
3 or 4 dimensional geometries are shown as 2D
Steps to reproduce, if exist:
When executing a query that returns a 3 or 4 dimensional geometry like the following:
select 'MultiPointZ((1 2 3))'::geometry

the result is MULTIPOINT ((1 2)), losing the third dimension. Kind of irritating when you expect 3d geometry and only see 2 dimensions.

Recent PG JDBC driver allow to parse/generate PostGIS values directly.
This simplifies GIS support in DBeaver.
Multidimensional geoms support will be fixed in the next version, thanks for the bug report.


Listview widget does not scroll smoothly(scroll with glitch)

Steps to Reproduce

Run the attached sample on iPhone 14 simulator 16.2
List view does not scroll smoothly

Expected results: ListView widget will scroll smoothly without a glitch
Actual results: ListView widget scroll with glitch

Code sample

  ScrollController scrollController1 = ScrollController();
  ScrollController scrollController2 = ScrollController();

  @override
  Widget build(BuildContext context) {
    return MaterialApp(home: Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text("Title"),
      ),
      body: Container(
        child: Stack(
          children: [
            Positioned(
                left: 0,
                right: 0,
                bottom: 0,
                top: 0,
                child: Scrollbar(
                  controller: scrollController1,
                  child: ListView(
                    scrollDirection: Axis.horizontal,
                    controller: scrollController1,
                    children: [
                      Container(
                        width: 1500,
                        child: Scrollbar(
                          controller: scrollController2,
                          child: ListView(
                            scrollDirection: Axis.vertical,
                            controller: scrollController2,
                            children: [
                              Container(
                                width: 1500,
                                height: 1500,
                                child: Row(
                                  children: getChildren(),
                                ),
                              )
                            ],
                          ),
                        ),
                      )
                    ],
                  ),
                ))
          ],
        ),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    ));
  }

  List<Widget> getChildren()
  {
    List<Widget> children = <Widget>[];

    for(int i=0; i<10; i++)
    {
      children.add(Container(
        width: 150,height: 1500,
        child: Padding(
          padding: const EdgeInsets.all(5.0),
          child: Container(color: Colors.red.withOpacity(0.1 * i),),
        ),
      ));
    }

    return children;
  }

Issue replication video
https://user-images.githubusercontent.com/44917578/225917437-37b3551f-9e6f-4480-adc8-527f228309c3.mov

Running this example on https://dartpad.dev errors in the console:
Error: Assertion failed: file:///Users/brettmorgan/Documents/GitHub/dart-services/flutter-sdks/stable/packages/flutter/lib/src/widgets/framework.dart:2685:20
_debugCurrentBuildTarget == context
is not trueError: Error: Assertion failed: file:///Users/brettmorgan/Documents/GitHub/dart-services/flutter-sdks/stable/packages/flutter/lib/src/widgets/framework.dart:2685:20
_debugCurrentBuildTarget == context
is not true

@sashn-citiqprepaid
The above code was working fine as expected, please find the dartpad image as below

@iapicca
The specified issue only raised on iPhone devices and simulators. It was not replicated on Android, Web, Windows and Mac OS platforms.

@Sasikumar3595
I looked at the code implementation you shared and noted below as the hierarchy of widgets:
Container -> Stack -> children [Positioned] -> Scrollbar -> ListView -> [children] -> Container -> Scrollbar -> ListView -> [children] -> Container -> Row -> Container.
Which shows nested Scrollbar and ListView respectively. Would like to know the use case you are trying to achieve by having above widgets hierarchy. Are you looking to implement 2D scrolling ?
Can you also try by using SingleChildScrollView or something like ListView.builder and pass horizontal and vertical scroll directions and see if it helps or not ?

@Sasikumar3595
I tried your code sample and I have 2 observations and a question
observations

I don't experience any glitch or stutter

screen recording

partially related to the alleged issue, your code need optimization

flutter_lints is yelling about unnecessary containers and such
List<Widget> getChildren() ... is bad for performances  (see this issue and this SO question)

question
are you experiencing the issue scrolling with a mouse?
(if so the issue is very much related to this)

We created this sample to replicate the issue and we still facing issues on scrolling for iPhone simulator and iPhone devices. It looks like a glitch(video added on description #122886 (comment))

I'm not sure "'reproduces' on my machine" is a useful input for a bug report,
running your code sample the issue doesn't seem to occur (and I'm running a air m1, nothing fancy)

@iapicca
We prepared the above-attached sample for reproducing the issue and we checked with the stateless widget to resolve the issue but still, the glitch showed to us.
We suspect the issue is related to performance.
We have tested the issue on the below devices and the issue was reproduced - iPhone 14 simulator and iPhone SE real device.
We have tested this with the iPad simulator and it is working fine.
Note: We have tested the issue on iOS's latest version with flutter's latest version and the issue was only replicated on the iOS platform only, other platforms(Windows, MacOS, Android, Web) are working fine.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. If you find this problem please file a new issue with the same description, what happens, logs and the output of 'flutter doctor -v'. All system setups can be slightly different so it's always better to open new issues and reference the related ones. Thanks for your contribution.

Seriously ???

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. If you find this problem please file a new issue with the same description, what happens, logs and the output of 'flutter doctor -v'. All system setups can be slightly different so it's always better to open new issues and reference the related ones. Thanks for your contribution.

Seriously ???

@QuentinSc
when an issue is not reproducible gets closed (seriously).
If you are experiencing this issue I encourage you to open a new issue
with a minimal reproducible code sample (w/o 3rd party packages)


EROFS: read-only file system, chmod '/var/task/generated/prisma-client/runtime/query-engine-rhel-openssl-1.0.x'

Bug description
In #476 we established that deploying from windows to lambda (lambda) loses the executable flag on binaries since windows do not have file flags. Prisma client tries to add the executable flag at runtime and fails because the file system in the lambda environment is read-only.
However, in a user provided reproduction with nexus-prisma: https://github.com/leungandrew/nexus-prisma-serverless-express
We are also able to reproduce this when deploying from mac to lambda.
Notes:

I can see the executable flag on Linux binary when I look at it on mac:
https://user-images.githubusercontent.com/746482/85523654-e711c200-b624-11ea-9175-566af5d5c020.png

How to reproduce
Follow the steps outlined in https://github.com/leungandrew/nexus-prisma-serverless-express
I did attempt an isolated reproduction but was not successful (@pantharshit00 maybe you can also give it a go)

Possibly related, also non Windows: https://github.com/prisma/prisma-client-js/issues/476#issuecomment-647511761

Possibly related, also non Windows: #476 (comment)

It is the same exact issue, I produced the above repo for the reproduction steps for the issue in the above comment.  Let me know if I can help with more reproduction steps.

As we exclusively call the chmod logic, if the permissions of the engine are broken, it looked like it's a problem with serverless. And it turned out, it is:
https://github.com/serverless/components/issues/687
We can keep this open here until they have fixed the issue.


Verification Error pop up is shown with “ Your current balance is not high…” description even when invited user tries to verify the phone number while on-boarding

Frequency: 100%
Repro App version:  Play store build v1.0.1
Does not Repro version: App store build v1.01, Android Valora Mainnet build v1.0.0 (16)
Repro on device: Google Pixel Xl (8.1)
Pre-condition: User should have invited code.
Repro Steps:

Launch the Application and complete the walkthrough  screens.
Tap on create a new account and navigate to the Invite code screen.
Paste the invite code.
Tap on Start button
Observe.

Current Behavior: Verification Error pop up is shown
Expected Behavior:  Verification error pop up should not shown for invited user as they have Invited free i.e $0.25
Investigation:

User is able to verify phone number after onboarding.
Issue does not repro in IOS v1.0.1.

Impact: Invited user blocked to verify phone number while onboarding
Attachment: Invite_0_Balanceissue.mp4 (Bug observe at 06 sec)

I believe this has been addressed.

Verified with latest build Test flight build v1.2.0 (25) , Android internal build 1.2.0(1004294313) and found that issue is fixed.

Verification Error pop up is not shown with “ Your current balance is not high…” description when invited user tries to verify the phone number while on-boarding.

Device used: iPhone SE (14 iOS) , samsung galaxy m40(10)


Newton result field empty

Issue Label:

[X] Bug
[ ] New feature
[ ] Enhancement
[ ] New component

Description:
If its a bug make sure to include this section.
OS and OS version:

[ ] Mac
[ ] Linux 32 Bit
[X] Linux 64 Bit
[ ] Windows 32 Bit
[ ] Windows 64 Bit

OS Version:
Linux centos 3.10.0-1127.19.1.el7.x86_64 #1 SMP Tue Aug 25 17:23:54 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Issue is concerning Newton. Similarly to the lyrics issue, the result field seems to be empty when typing in any combo of operation and expression.

This is because of the following line https://github.com/alexanderepstein/Bash-Snippets/blob/97a1c0d6358264a50ca0c6e221b74ea1cb72b372/newton/newton#L115
The new base url is https://newton.now.sh/api/v2/
I have opened a PR with the fix, you can just change the URL in your local installation till then.


Document working with multiple zones in eureka

i google and read the doc and can't find some detailed description or example for config eureka server & client in multiple zones.
in cloud env, there are 2 zones in 1 region. i want to deploy eureka server1 in zone1, eureka server2 in zone2.
Service deployed in zone1, registered in eureka server1.
service deployed in zone2, registered in eureka server2.
for request route to zone1, prefer to route to service in zone1; and route to service in zone2 in case of service in zone1 is down
for request route to zone2, prefer to route to service in zone2; and route to service in zone1 in case of service in zone2 is down.
how to config the eureka server and client? Thanks.

Having the services register with the correct Eureka Server for their zone is just a matter of setting the URL to the Eureka server in eureka.client.serviceUrl.defaultZone.
To enable the case where a request to route to a service in another zone when the service in its zone is unavailable you need to make sure the Eureka servers in all zones are set up as peers of each other.  More information on that can be found in #1251.
To specify zone information you have two options.

If the hostnames of the services will be different in each zone you can set ribbon.eureka.approximateZoneFromHostname to true and zone information for a service will be calculated based on the host name.  (assuming you are using ribbon)
You can specify the zone for a service by setting eureka.instance.metadataMap.zone.

@ryanjbaxter
my eureka server and client config as below, eureka server 1 and server 2 replicated each other. eureka client in service or api-gateway (zuul), register for server 1 and server2.
eureka server1 config
eureka:
  instance:
    hostname: <IP_ADDRESS>
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/
  server:
    enable-self-preservation: false

eureka server2 config
eureka:
  instance:
    hostname: <IP_ADDRESS>
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/
  server:
    enable-self-preservation: false

eureka client config  (zone1 service)
eureka:
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/, http://<IP_ADDRESS>:8761/eureka/
  instance:
    preferIpAddress: false

eureka client config (zone1 api gateway: zuul)
eureka:
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/, http://<IP_ADDRESS>:8761/eureka/
  instance:
    preferIpAddress: false

how to config eureka server1 to zone1 and eureka server 2 to zone2?
at zuul, i config 2 URLs in serviceUrl.defaultZone, the 1st one is zone1 eureka server, the 2nd one is zone2 eureka server. if this config is enough when zone1 service is unavailable, the request is routed to zone2 service?
at zuul / service, how to config to route the request to service the same zone?
2. You can specify the zone for a service by setting eureka.instance.metadataMap.zone.
this way?
eureka client config  (zone1 service)
eureka:
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/, http://<IP_ADDRESS>:8761/eureka/
  instance:
    preferIpAddress: false
    metadataMap:
       zone: zone1

btw, how to config region info for eureka and config 2 zones under this region?
Thanks.

It looks like your Eureka server configuration is not correct.  You should configure them as follows.
eureka server1 config
eureka:
  instance:
    hostname: <IP_ADDRESS>
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/,http://<IP_ADDRESS>:8761/eureka/
  server:
    enable-self-preservation: false

eureka server2 config
eureka:

  instance:
    hostname: <IP_ADDRESS>
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/,http://<IP_ADDRESS>:8761/eureka/
  server:
    enable-self-preservation: false

In your Eureka Clients you should set defaultZone to the URL of the Eureka server in the same zone.  So for example if Eureka server 1 is in zone 1 and Eureka server 2 is in zone 2, your client deployed to zone 1 would look like
eureka:
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/
  instance:
    preferIpAddress: false
    metadataMap:
       zone: zone1

And you client deployed to zone 2 would look like
eureka:
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/
  instance:
    preferIpAddress: false
    metadataMap:
       zone: zone2

Since the two Eureka servers are set to be peers of each other they will replicate service information with each other.
On the clients when a request is made to a service we will use the zone information obtained from Eureka about the various services to determine which service to try first.

thanks, @ryanjbaxter
i try the following config and it works:
eureka server config is just as above and the eureka client config in service is below:
service1 in zone1
eureka:
  instance:
    preferIpAddress: false
    metadataMap:
      zone: zone1
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/, http://<IP_ADDRESS>:8761/eureka/
    preferSameZoneEureka: true

zuul1 in zone1
eureka:
  instance:
    preferIpAddress: false
    metadataMap:
      zone: zone1
  client:
    serviceUrl:
      defaultZone: http://<IP_ADDRESS>:8761/eureka/, http://<IP_ADDRESS>:8761/eureka/
    preferSameZoneEureka: true

then zuul1 will route request to services in zone1 first and if services in zone1 is down, will route request to services in zone2.

To document

@liangchouzhou I am working on updating the documentation and after going back and looking at the configuration you said worked, I am wondering why in your Eureka clients you are setting the defaultZone property to point at both Eureka servers?  Shouldn't you only need to point at the Eureka server in the same zone and let the peer replication functionality between the servers replicate the data to the other Eureka servers in the other zones?

ping @liangchouzhou ^^^

Closing due to lack of activity.  Please provide feedback and reopen.

@ryanjbaxter Beause if Eureka server in the same zone was shut down, so the function of replicate was broken.The Eureka clinets will look for another Eureka server  defaultZone:http://<IP_ADDRESS>:8761/eureka/, http://<IP_ADDRESS>:8761/eureka/

@ryanjbaxter Beause if Eureka server in the same zone was shut down, if your client set only one serviceurl, the client can't look for a stand by server.so you should set one more below:
defaultZone: http://<IP_ADDRESS>:8761/eureka/, http://<IP_ADDRESS>:8761/eureka/

@SeanWan1989 would doing this cause the Eureka client to register with both Eureka servers or does it just try the first one in the list?

@ryanjbaxter clinet will just try the first one in the list, if the first one was shut down, the client will auto register with the next one.

@ryanjbaxter client will just try the first one in the list, if the first one was shut down, the client will auto register with the next one.

@liangchouzhou can you write out your config eureka server and eureka client
i do not understand prettily contents which your had talked about  'service1 in zone1' and 'zuul1 in zone1'

@ryanjbaxter ask for you about the core concept of zone.  btw,what is different between zone and region?

@yaoyuanyy Each of region and zone is concept of AWS, you can follow this URL to learn about what is it. http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-regions-availability-zones.html

@eacdy lucky because of your help, i will study from it.  but now the question:'config eureka server & client in multiple zones'. i can not work it @liangchouzhou  @ryanjbaxter

@eacdy Firstly thank you very much.  you are right, i am a chinese who has been a high interesting with  spring-cloud. But i am a beginner for spring-cloud.  thanks again

@yaoyuanyy My Honor. My open source book may help you get started.https://github.com/eacdy/spring-cloud-book.
B.T.W, 我为啥要说英文？

@eacdy Knowledge is belong to all over the world. 感谢周立同学的分享，太棒了，马上拜读，在这里我学到了很多。Thanks everyone

I am trying to get this worked with client able to discover them within the same zone and hop over in case they are not available in thier current zone. I have set up all the properties as mentioened and its going to below place for resolution of the eureka service name and zone is not getting used. I am on latest edgware release
com.netflix.discovery.DiscoveryClient
public List getInstancesByVipAddress

@jprateekvmware please open a separate issue with a more detailed description on the problem.  In this case your configuration would be particularly useful.

Referring to @liangchouzhou comment https://github.com/spring-cloud/spring-cloud-netflix/issues/1327#issuecomment-245800603 and @SeanWan1989 comment https://github.com/spring-cloud/spring-cloud-netflix/issues/1327#issuecomment-262137314 . I had the similar usecase where I have to use the list of defaultZone and these comments were helpful for me. But unfortunately, I didn't find any Spring Cloud doc around it. @ryanjbaxter is it documented ?

If you dont see it in our current docs (http://cloud.spring.io/spring-cloud-static/Finchley.SR1) than no.  We always welcome PRs to enhance our docs.


face detection container stops running

The face detection recognition container stops running on its own after which I receive the ConnectionError that I mentioned in another issue before. What can I do to make sure it continues running?

Hmm the container shouldn't stop itself. Maybe you don't have enough memory?
Can you show me your output when you type below in your terminal?
free -m

docker ps -a

Where should I type this? If I go to Jupyter Lab again and open a terminal there it says free is not a command. When I try the second one this is the output I get

:

free -m and docker ps -a are Unix commands. You type them in your terminal.

This free command doesn't work for MacOS, right? Above is the output for docker ps -a. Do you have a solution for the container that stops running?

I can't say so much at this moment. I have to see the logs of the docker containers to be sure.
I was asking about the memory, since OOM is the most common cause of docker containers failing.

This was solved. On MacOS, you have to ease the hardware constraints (e.g., allocate more memory, cores, etc.)


global.XMLHttpRequest is not a constructor

I'm building a small library.

Using following versions:

 "request": "^2.83.0",
 "request-promise-native": "^1.0.5",

Injecting the package as follows:

import request from 'request-promise-native';

Building UMD package so it could be used all kinds of ways. Browser and Node. Using webpack as a bundle tool.
When I inject the output as script tag and use use instance of the library and call methods ( using request ) everything works fine.
When I used node repl to test when I require my output file I receive this.
TypeError: global.XMLHttpRequest is not a constructor

FWIW, I came across this error using node-fetch (since 2.1.2 the problematic code gets added when one uses import instead of require; I discovered this for myself after reading https://github.com/calvinmetcalf/rollup-plugin-node-builtins/issues/25 ; )... I suppose one might be able to avoid this through configuration of their Rollup/bundling pipeline by explicitly avoiding the browser field in package.json for Node code, etc., but I went with the quick-and-dirty solution of just using require over import for the responsible code...

I can’t explain your TypeError @peculiarity . It is something that originates from your particular environment.
In general, if you use request-promise-native client-side then you will have to polyfill many core node.js libraries. request-promise-native is written for node.js and thus it is not straight forward to use it outside of node. It is possible though. Many users reported that they successfully use it with webpack or browserify. However, I can’t provide official support unfortunately.


Consider reusable read transactions using ThreadLocal

mdb_txn_begin has notable effect on performance which can be avoided by reusing transactions per thread. This is only applicable for read transactions.
Example of how this might be implemented by Viktor Szathmáry.
private final ThreadLocal<Transaction> cachedTx = new ThreadLocal<>();

private Transaction acquireReadTransaction() {
  Transaction tx = cachedTx.get();
  if (tx == null) {
    tx = env.createTransaction(true);
  } else {
    cachedTx.set(null);
    tx.renew();
  }
  return tx;
}

private void releaseReadTransaction(Transaction tx) {
  if (cachedTx.get() == null) {
    tx.reset();
    cachedTx.set(tx);
  } else {
    tx.abort();
  }
}

It is not clear whether this code belong in lmdbjni or not, since its features are already quite sparse which is a good thing.

Ok to close this until we figure out a good way to host this kind of (higher level) functionality?

Reusing transactions means not freeing deleted pages - which is sort of a bad thing in case your db is almost full. Probably some sort of pool of  relatively short-lived transactions might be used with a set max lifetime.

Ah yes, read transactions are an actual snapshot of the database. Makes sense, good call.
Yes, some kind of timeout would be appropriate. Still not sure if this feature belong here. Maybe this could be built into LMDB?
Let's see if we can summon Howard :-)

He would say this makes no sence -and reusing old transaction objects means not getting recent changes - it a hack, and it's not appropriate in most cases.

Ok, let's not wake the bear ;-) We'll figure something out. Any PR's would be appreciated in the meantime.

PR stands for what exactly? ))

Pull Request.  But maybe not in this project. Keeping the API in lmdbjni as close as possible to native LMDB is a good thing?

Low level interface is better for performance and worse for adoption. Have you tried LMDB fork from Re-Open LDAP guys?

Agreed. Do you have link?

https://github.com/ReOpen/ReOpenLDAP ?

Exactly. I were unable to build it locally however. In contrast to original one.

Caution - Manual and FAQ is in Russian. They claim to improve a few things as well as code quality. Didn't run diff merge yet on it. Don't think they will offer any support to it. Let's just hope Putin will not cut Internet access in there ;-)

Haha :-) I need to read google translated version of the wiki first, but this is mostly OpenLDAP mods right?

Wiki doesn't contain much of a technical information - it's more about who they are, why forking, which parts of original code doesn't smell right to them. Useless stuff. But in sources there is libmdb folder.

Found it: https://github.com/ReOpen/libmdbx

May I ask how do you deal with constant  environment size in production ?

Set a overdimensioned value. With Linux, the OS will handle it.

Or do you mean big data? :-)

I meant when you set map size before initializing environment - it creates file of that size, and when map is full you need to stop all operations , close it and resize i,  then reopen.

No, memory/file will not be allocated upfront. Set a high value, close to max size of disk, and the OS will handle the rest. No problem.

With Linux that is. I think Windows eagerly allocate unused space.

Thanks, that's awesome.

on this subject:
are you thinking about some sort of object pooling mechanism?
on another subject:
I am playing around, doing some stupid tests, with Bookkeeper and
lmdbjni here https://github.com/mauricioscastro/rlmdb (for replicating
lmdb with java over yours lmdbjni) after having used it a lot in
basex-lmdb https://github.com/mauricioscastro/basex-lmdb.
now that I started thinking seriously about it and once I use lmdbjni
all over the place in basex-lmdb I started by extending Env class for
initial replicated environment setup, etc, but now that I need to extend
other stuff, but I face package protection issues for Database and
Transaction, thoughts? create a lmdbjni branch for me?
on Mr's Chu's character:
good to know other's participate on this feeling of mine where I see him
as a difficult person to deal with, I never needed to approach him, but
from the replies I read and the presentations I saw I almost feel
threatened, he he :)
Regards and again, thanks for lmdbjni!
On 11/27/2015 09:37 PM, Kristoffer Sjögren wrote:

Ok, let's not wake the bear ;-) We'll figure something out. Any PR's
would be appreciated in the meantime.
—
Reply to this email directly or view it on GitHub
https://github.com/deephacks/lmdbjni/issues/12#issuecomment-160227188.

--
Maurício Santiago de Castro
mauriciosantiagodecastro
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://br.linkedin.com/pub/mauricio-castro/17/275/a5b
http://www.delicious.com/mscastro
http://twitter.com/#!/mauricioscastro
mauriciosantiagodecastro@facebook
http://facebook.com/mauriciosantiagodecastro
https://mscastro.startssl.com/

Why not use Apache Ignite or Hazelcast for replication ?
30 Ноя 2015 г. 16:17 пользователь "Mauricio Santiago de Castro" <
<EMAIL_ADDRESS>написал:

on this subject:
are you thinking about some sort of object pooling mechanism?
on another subject:
I am playing around, doing some stupid tests, with Bookkeeper and
lmdbjni here https://github.com/mauricioscastro/rlmdb (for replicating
lmdb with java over yours lmdbjni) after having used it a lot in
basex-lmdb https://github.com/mauricioscastro/basex-lmdb.
now that I started thinking seriously about it and once I use lmdbjni
all over the place in basex-lmdb I started by extending Env class for
initial replicated environment setup, etc, but now that I need to extend
other stuff, but I face package protection issues for Database and
Transaction, thoughts? create a lmdbjni branch for me?
on Mr's Chu's character:
good to know other's participate on this feeling of mine where I see him
as a difficult person to deal with, I never needed to approach him, but
from the replies I read and the presentations I saw I almost feel
threatened, he he :)
Regards and again, thanks for lmdbjni!
On 11/27/2015 09:37 PM, Kristoffer Sjögren wrote:

Ok, let's not wake the bear ;-) We'll figure something out. Any PR's
would be appreciated in the meantime.
—
Reply to this email directly or view it on GitHub
https://github.com/deephacks/lmdbjni/issues/12#issuecomment-160227188.

--
Maurício Santiago de Castro
mauriciosantiagodecastro
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://br.linkedin.com/pub/mauricio-castro/17/275/a5b
http://www.delicious.com/mscastro
http://twitter.com/#!/mauricioscastro
mauriciosantiagodecastro@facebook
http://facebook.com/mauriciosantiagodecastro
https://mscastro.startssl.com/
—
Reply to this email directly or view it on GitHub
https://github.com/deephacks/lmdbjni/issues/12#issuecomment-160642334.

Humm... overkill I thought initially about Hazlecast. Dunno about
Ignite, but I am looking into it. A write through cache? hummm... I
remember considering Infinispan for this...
Too high level I thought, Bookkeper being more raw would allow me more
control over what's going on and perform better?
I will look at how embeddable Ignite would be.
Thanks.
 But since it messes with deleted pages and cause inconsistency I think the use case is quite limited. Maybe its possible to do something smart with the transaction id in LMDB 0.9.17.
Regarding the package structure, you could use delegation instead of inheritance.

There's also copycat which is a Raft implementation. The author claims it passes the jepsen test.

Won't fix.


Adjust logic on handlebars to avoid 'if true' statement on models

There was a small logic on handlebars causing to create if true statements on the golang sdk as we can see here
Description
with this new implementation the code is now like this:
before
func (o ListObjectsRequest) MarshalJSON() ([]byte, error) {
	toSerialize := map[string]interface{}{}
	if o.AuthorizationModelId != nil {
		toSerialize["authorization_model_id"] = o.AuthorizationModelId
	}
	if true {
		toSerialize["type"] = o.Type
	}
	if true {
		toSerialize["relation"] = o.Relation
	}
	if true {
		toSerialize["user"] = o.User
	}
	if o.ContextualTuples != nil {
		toSerialize["contextual_tuples"] = o.ContextualTuples
	}
	return json.Marshal(toSerialize)
}

after
func (o ListObjectsRequest) MarshalJSON() ([]byte, error) {
	toSerialize := map[string]interface{}{}
	if o.AuthorizationModelId != nil {
		toSerialize["authorization_model_id"] = o.AuthorizationModelId
	}
	toSerialize["type"] = o.Type
	toSerialize["relation"] = o.Relation
	toSerialize["user"] = o.User
	if o.ContextualTuples != nil {
		toSerialize["contextual_tuples"] = o.ContextualTuples
	}
	return json.Marshal(toSerialize)
}

References
#close https://github.com/openfga/go-sdk/issues/11
Review Checklist

[x] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

now that I noticed that has some problem on CI:
2023-09-24T06:24:39.4544560Z ##[group]Run ./scripts/clone_sdk.sh
2023-09-24T06:24:39.4544895Z [36;1m./scripts/clone_sdk.sh [0m
2023-09-24T06:24:39.4606088Z shell: /usr/bin/bash -e {0}
2023-09-24T06:24:39.4606328Z env:
2023-09-24T06:24:39.4606539Z   GITHUB_ORG_ID:
2023-09-24T06:24:39.4606767Z   GITHUB_REPO_ID:
2023-09-24T06:24:39.4606971Z   SSH_KEY:
2023-09-24T06:24:39.4607202Z   SDK_PATH: clients/fga-js-sdk
2023-09-24T06:24:39.4607440Z   KNOWN_HOSTS:
2023-09-24T06:24:39.4607640Z ##[endgroup]
2023-09-24T06:24:39.5003625Z Agent pid 1755
2023-09-24T06:24:39.6400279Z ./scripts/clone_sdk.sh: line 6: GIT_SSH_COMMAND:: command not found
2023-09-24T06:24:39.6429660Z Error loading key "(stdin)": error in libcrypto
2023-09-24T06:24:39.6430229Z Cloning into 'clients/fga-js-sdk'...
2023-09-24T06:24:39.6430559Z<EMAIL_ADDRESS>Permission denied (publickey).
2023-09-24T06:24:39.6430903Z fatal: Could not read from remote repository.
2023-09-24T06:24:39.6431102Z
2023-09-24T06:24:39.6431347Z Please make sure you have the correct access rights
2023-09-24T06:24:39.6431945Z and the repository exists.
2023-09-24T06:24:39.6443073Z ##[error]Process completed with exit code 128.

sounds like it cant pull the sdk repositories

Added https://github.com/openfga/sdk-generator/issues/212 to tackle to allow tests to pass for future PRs


JSONField bug: unexpected Django 1.7 behavior

https://bitbucket.org/schinckel/django-jsonfield/issues/55/jsonfield-returns-a-string-on-django-17

Decided to drop 1.7 support. It is no longer supported (https://www.djangoproject.com/download/#supported-versions) and this will fix this problem.


cpu: Refactor indirect predictor

This is the third patch set to restructure the branch predictor unit
Simplify indirect predictor interface. Several of the existing functions where merged together into four clear once. Those four are similar to the main direction predictor interface. 'lookup', 'update', 'squash' and 'commit'. This makes the interface much more clear, allows better functionality isolation and makes it simpler to develop new predictor models.
Change-Id: I6d6b43965b2986ef959953a64c428e50bc68d38e

Agree, lets wait for the tests.
Thank you @andysan

We have now finished running performance regressions using SPEC 2017 SimPoints, and can confirm that the IPCs reported by gem5 are not changed by this refactoring.


Build sphinx using either Markdown or a hybrid with rst

Do we want to keep using rst? I Sphinx supports markdown (see here also). Maybe, a hybrid between the two would be a solution?

If you feel strongly that this is the better way to go, please open an issue and describe the "hybrid" solution a bit more. If you are on the fence, I think we can just continue with the current workflow and keep focused on higher-priority tasks.
Originally posted by @damonbayer in https://github.com/CDCgov/multisignal-epi-inference/issues/224#issuecomment-2200413117

I semi-duplicated this in #235. Will close that one.

The parameters
We're set on having .qmd for literate programming for the moment, and unless/until there's closer Quarto integration for Sphinx, those docs will always need to be rendered to something (.md, .rst, etc) before being handed off to Sphinx.
My preference: use .md rather than .rst
I slightly favor .qmd -> .md and the rest of the docs written in regular .md, as that comes closest to a single unified coding style.
Status quo (.rst) would also be fine
I'm also happy to stick with the status quo .qmd -> .rst and everything else in .rst if others prefer that.
Question
@gvegayon: by "hybrid" did you mean a mix of (non-Quarto) .md and .rst? If so, what do you see as the potential benefits?

Tagging @CDCgov/multisignal-epi-inference-devs for visibility.

Agree with all of what @dylanhmorris said above.

I have now tried out Markdown-based sphinx doc sources via MyST-parser in another project. I like it and would support moving to it here.

Another reason to use MyST: https://github.com/CDCgov/multisignal-epi-inference/pull/330#issuecomment-2251477606
We should also investigate ways of taking advantage of intersphinx etc in tutorial links

Given enthusiasm from @damonbayer and me, I'm going to resolve this in favor of switching over to Markdown via MyST and rename the issue accordingly.


[Bug]: Testing Bug Report Template Functionality

Prerequisites

[X] I have read the Guidelines in reporting bugs.
[X] I have performed basic troubleshooting (if applicable).
[X] I'd be willing to address this issue myself.

What happened?
As requested, this report is the test the "Bug Report" template to ensure everything is working as intended. See attached screenshot - describing the footer of this website:

Steps to reproduce

Open the website in Chrome.
Scroll to the footer section of the homepage.
Observe the content and layout of the footer.

What browsers are you seeing the problem on?
Chrome
Device type.
Desktop
Code of Conduct

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

Nicely done @botgina . Thank you!

Closing this on the next PR @botgina


Remove config dependency from discovery

This is a follow-up to #3013. In #3629, we removed config  dependency from the discovery package partially. We need to decouple these packages completely.

#3629 did this.


CSS output

CSS/SCSS should contain one main .class where is {{url}} , because now url duplicate in each class name and make finale .css file much bigger. I know that I can create my template but this should be  by default, because 100 icon make 100xURL.

When we discuss consolidating lines for CSS rules, we start talking about CSS optimization which is outside of the scope of this project.
We recommend using a tool like csso after the sprite rules are generated. An example can be found in the README:
https://github.com/twolfson/gulp.spritesmith/tree/6.2.0#continuing-the-pipeline
Package: https://github.com/ben-eb/gulp-csso


Change MAP api keys

There are ACDH keys available.
This is for Mapbox.

@vronk where can I get the keys?

new key is available

The new generic API key works. It works for the maps in the login area at Courses Add/Edit. It doesn't work for the map on the main page.
First Leatlet needs to be on the latest version, see separate issue.

In the meeting of today we agreed to remove it from the milestone. This task is waiting for issue https://github.com/acdh-oeaw/dhcr-main/issues/66 to be completed and that takes some time, And currently doesn't have the highest priority.

It seems that changing the Mapbox API didn't work in the beginning, because there was a custom style used, which was linked to the old key.
Now I've migrated to a default Mapbox style. The style looks nice and I didn't see any unpleasant differences.
As well the new mapbox api key is used in dev.

Note to myself: Change global key when this goes into production and remove 2x env key

@PixlTracer  @IvdL22
Can you review this?
You could check that the map still works and if you're satisfied with the slightly changed style/design of the map:
https://test-dhcr.clarin-dariah.eu/

@patrickakk The map seems to work fine. I like the design as well, thank you :)

Implemented in 2023-09.
After this goes into production, global keys need to be changed, see above
@IvdL22 I've changed the label to "done" for you ;)

repository secret updated to new map api key.
env secrets removed.
all containers redeployed after changes.
everything works fine.