perf: Copy files instead of sending over HTTP if Wave app and Wave server are running on the same machine. #982

This is a rough proposal for further discussion. If approved, will finish the rest of the sync/async upload methods.
Rationale: When uploading files, we send them over HTTP with serialization/deserialization overhead that slows the whole method. However, most of the Wave apps run both Wave server (waved) and Wave app on the same machine which means a simple file copy should be enough.
Perf testing
HTTPX 0.16.1 is the original version from the time of creating issue #982.

File size
HTTPX 0.16.1
HTTPX 0.23.1 (latest)
cp (Mac)

1GB
~8.5s
~4.3s
~14ms

10 GB
10min timeout
~40s
~10ms

Although we have public/private dirs, the concept of serving files and figuring out what is the correct URL can be troublesome for non-software engineering users so I would suggest keeping the current q.site.upload as is.
This change would impact most (if not all) Wave users who use q.site.upload.
Wdyt @lo5?
Closes #982

Thanks for the suggestions @lo5! A combination of A and C seems like the way to go for me. Will do.

The PR is done.

Tested against both Windows and Unix.
Made async (to not block the main thread).

@lo5 can you please let me know what you think about naming? (H2O_WAVE_NO_COPY_UPLOAD, H2O_WAVE_WAVED_DIR). Once resolved, can be merged.


Remove JSONB property with patch

Objection supports partial updates (patches) of JSONB columns through field expressions. However I can't seem to delete a property without doing an update by getting the existing value and excluding the properties to delete.
Would it be possible to support deleting a property by supplying a value of undefined?
As of now giving an undefined in the fieldExpression generates no query.
Example:
const params = { tags: { deleteMe: true } };
const person = await Person.query().insert(params).returning('*');
await person.$query().patch({ 'tags:deleteMe': undefined });

This does not update the deleteMe field of the tags column, and the value is still true. Is setting a fieldExpression to undefined a reasonable approach to this or is there a better way to remove a JSONB property with a patch operation?

Did you ever find the solution? Please let me know. Similar issue.

There's no shortcut syntax for this but you can always use raw sql:
await person.$query().patch({ tags: raw("tags - 'deleteMe'") });


single thread executor with named thread

Hi, I think a single thread is enough to run the purge task and I have named it (named thread helps me when analyzing problems)

@matheusgg ?


KAFKA-6324: Change LogSegment.delete to deleteIfExists and harden log recovery

Rename delete() to deleteIfExists() in LogSegment, AbstractIndex
and TxnIndex. Throw exception in case of IO errors for more informative
errors and to make it less likely that errors are ignored, boolean is used
for the case where the file does not exist (like Files.deleteIfExists()).
Fix an instance of delete while open (should fix KAFKA-6322 and
KAFKA-6075).
LogSegment.deleteIfExists no longer throws an exception if any of
the files it tries to delete does not exist (fixes KAFKA-6194).
Remove unnecessary FileChannel.force(true) when deleting file.
Introduce LogSegment.open() and use it to improve encapsulation
and reduce duplication.
Expand functionality of LogSegment.onBecomeInactiveSegment()
to reduce duplication and improve encapsulation.
Use AbstractIndex.deleteIfExists() instead of deleting files manually.
Improve logging when deleting swap files.
Use CorruptIndexException instead of IllegalArgumentException.
Simplify LogCleaner.cleanSegments() to reduce duplication and
improve encapsulation.
A few other clean-ups in Log, LogSegment, etc.

@junrao, maybe you can review this.

@junrao removed the stray comma and kept the fileType as per your last comment.

FAILURE
5034 tests run, 1 skipped, 52 failed.
--none--

FAILURE
5034 tests run, 1 skipped, 52 failed.
--none--

FAILURE
5034 tests run, 1 skipped, 52 failed.
--none--

Looking into the test failures.

@junrao, this is ready for review. The PR description is up to date.

lgtm

All tests should be passing now.

Test failure (UserClientIdQuotaTest.testProducerConsumerOverrideUnthrottled) is unrelated.

@ijuma : Thanks for the latest patch. LGTM.

Merging to trunk.

Is this fix supposed to avoid the NFS silly renames issue?  I am currently testing Kafka 2.7 on NFS filesystem and encountered the same error message when deleting a topic.


update the connection button at every connection change

If you want to try the PR before the merge this is the pages link: https://gdorsi.github.io/BangleApps/

This looks great - thanks!
No worries if you don't feel like it, but actually as there's a connect button, you could replace it with a 'Disconnect' one when the page is connected to the watch :)


Author profile sidebar jitters with hover

When hovering over the author sidebar profile, the opacity transition seems to cause it to jitter.

Good eye! Now fixed in f75727466a91847b2268a7736c14d09d9efc8652, which is a small patch to _sass/sidebar.scss from mmistakes (https://github.com/mmistakes/minimal-mistakes/issues/583). Reminds me that I need to do a refactor/patch upstream from mmistakes...


Added Search Bar in the Tools Section

Added the Search Bar in the Tools Section .
Solved issue #547 .
Video :
https://github.com/jayk-gupta/web-resources-project/assets/127319225/70b01545-95d5-4c6e-8176-1f688732bebf

We need to add in for all sections and implement on the search bar already present

Ok I will make a PR of that.
Just a small request. Can you make the PR I will create as level 3 . It would really help.


MAPREDUCE-7063: Update Logging to DEBUG Level

Merged to trunk by @aajisaka

Thanks @vbmudalige for closing this PR.


Author of the "vision" page's quote

There is a nice quote in the beginning of the "Vision" page. Its author should be named for clarity purpose.

@nebali  could you help on this one ?

This PR is the only source I've found so far: https://unitimes.media/auther/12279/
Phrased slightly differently here: https://www.omise.co/omise-funding-led-by-globalbrain
So, might just be credited to OmiseGO?

I'm looking for the source, but let's just go with OmiseGO for now. I don't know if we'll be able to track down any one person that wrote it.

Revised to credit OmiseGO team; see: https://github.com/OMGnetwork/guide/blob/master/docs/vision.md; closing this issue 👍


Memory leak when calling list(...) on OrderedDict

Description
There is a memory leak when calling the list function with an OrderedDict. Even after the runtime has been shut down, objects of type PythonGetMemberBinder, PythonType, FunctionDefinition, FunctionCode etc. remain referenced and will not be garbage collected.
Steps to Reproduce
(Repro project available here: https://github.com/spkl/IronPythonMemoryRepro)

In a .NET 6 program, create a ScriptEngine and ScriptScope.
Create a ScriptSource from a file containing the following code:

import collections

def MyMethod(dummy):
    od = collections.OrderedDict()
    for i in range(100):
        od[i] = []

    for v in list(od.values()):
        continue

    return dummy

Execute the ScriptSource.
Call the MyMethod function.

Repeat this 50 times.

Expected behavior: The memory footprint is stable. (For reference: After the first execution, it is ~50MB).
Actual behavior: The memory footprint exceeds 300 MB after 50 repetitions. Forcing a garbage collection does not change it significantly. (With a larger, more complex example, we have observed ~600 MB of additional, uncollectable memory after 30 script executions.)
Workaround: Instead of for v in list(od.values()):, use for v in [x for x in od.values()] or for v in od.values():.
Versions
The issue is reproducible with both 3.4.0 and 3.4.1.

Thanks for the well-prepared report. I was able to reproduce the problem using the latest development version.

The underlying cause of this problem is that the conversion of an object (here an OrderedDict) to an enumerable (here, in the list constructor) happens in the default code context. After a successful conversion (here: a resolved call to __iter__), the call site dispatch table gets a rule for that specific type of the object (OrderedDict), so that subsequent iterations of object of the same type are resolved faster.
After the runtime shutdown, the dynamic type for OrderedDict no longer exists, however, the corresponding rule in the call site of the default context still references that type. A new runtime instance will create a new dynamic type for OrderedDict, which is a different type instance than from the previous execution. The default context is static and lives till the end of the hosting process, so those zombie types from previous executions linger around.
I have not seen the code responsible for the call site cache management, but from reading the docs of the DLR I vaguely remember that the cache site is limited (in number of entries, not memory size). Therefore those stale dispatch rules will eventually get replaced by newer ones, so technically it is not a memory leak, more like memory swelling. Indeed, if I run the provided example, say 500 times, the memory footprint reaches a sort of a plateau around 2GB.
Unfortunately, OrderedDict is a large class, so the effect of the swelling is very visible. Preferably, the default context is not used for dispatching of dynamic types, for any of the dynamic calls/conversions. Conversion to an enumerable is just one example. However, using the actual context may not be the best solution either: each module has its own context. So in my assessment, executing everything in the actual context would increase memory usage and slow down the execution, as modules could not share learned dispatch rules. So perhaps having zombie dynamic types is a reasonable price to pay for the performance gain, especially if the those types are small. This may be a common case for most types implementing some part  of the Python protocol (like a type with __index__), but enumerable types (i.e. types with __iter__ or __getitem__) tend to be large.
Ideally, those dynamic calls would be done in a runtime-scope context. Unfortunately, IronPython does not have the runtime-scope context; the choice is either the static global (default) context, or the actual (module) context. Perhaps the context of the builtin  module could serve as the runtime-scope context, but implementing this across the whole codebase would be a lot of effort.


CreateTemp creates files with 0666 perms

For an upcoming major update.
Previously, some files created were made by os.CreateTemp which made files with 0600 permissions, which meant the administrator was required to use sudo to access and modify these files. CreateTemp makes a file in the same way (so that it is guaranteed to have a unique filename) and is meant to be used in place of os.CreateTemp.

Actually, I was wrong, this should be considered a bug. It is not backwards-incompatible and it does not change behavior except to rectify unwanted behavior.


Guildbook "out of date" on Burning Crusade Classic 2.5.2

Thanks for the latest update but you forgot to update the BCC .toc file.

Guildbook uses just a single .toc file as curse doesnt support the multi toc stuff yet, afaik anyways.
I have however updated the interface version in the toc which i did miss.

I am the author/maintainer of Skillet (for retail) and Skillet-Classic (for BCC and Classic Era). Curse has "sort of" support for multiple .toc files. You can use the Blizzard feature of multiple .toc files (addon.toc, addon-BCC.toc, addon-Classic.toc, addon-Mainline.toc), see this post and when you upload the file to Curse, you can edit the file and indicate which version(s) it supports.
I believe the BigWigs Package Manager can also be configured to generate multiple source files and upload different files for different flavors of World of Warcraft to a single Curse project, but in my Skillet / Skillet-Classic case, I have two different projects. The Skillet-Classic file can be installed on both Classic Era and Burning Crusade Classic.
Let me know if you have questions about how this all works and I'll do my best to help.


Decide needed metadata for models and specify corresponding markdown formats

Issue description / feature objectives
Currently the markdown files allow user-friendly editing, but the files only capture a subset of the json functionality.  Some things would be good to specify directly in the markdown somehow, e.g. items in the OSCAL metadata (title, version) and list of controls to be included by a profile.
Caveats / Assumptions
The split functionality already has top level info in profile.json and catalog.json, so it is natural to have generically named profile.md and catalog.md, etc., in the top level markdown directories.
Completion Criteria

[ ] Decide what metadata are needed for each top level model
[ ] Determine if simple markdown structures such as lists are suitable, or if it needs to be more complex yaml
[ ] Change all the -generate routines to create an initial top level .md file for later editing
[ ] Change all the -assemble routines to use this metadata and go direct from markdown to json

As an example, for profile the list of imports is needed.
Each import requires:

an href
optional include-all flag
include-controls -> list of controls
exclude-controls -> list of controls

Even this is a subset of profile functionality, but it may be sufficient for most use-cases.  trestle currently does not support pattern matching for control inclusion - for example.
Is this complexity enough to use yaml instead of markdown?

For catalog, in the main markdown directory for catalog, we can have a catalog.md file containing the metadata associated with it.
There are 4 required fields in metadata, out those 2 needs to be specified in the markdown, and 2 can be autogenerated by trestle -

title: should be in catalog.md
last-modified: auto-populated by trestle while generating catalog.json
version: should be in catalog.md
oscal-version: auto-populated by trestle while generating catalog.json

The group ids are already captured currently as name of the directory containing the controls. Sub-groups can also be specified in a similar way using sub-directories. The group title is currently part of each control markdown file inside the group directory. For this to work, all controls for a group should have the same title in the control markdown, else it will result in an error. Same with sub-group titles also. Only subgroups can be supported this way, and not sub-controls. If sub-controls are required to be supported then some other mechanism need to be devised.
A template file should be used to specify the structure of control markdown for adding new controls, including control description (parts/subparts), guidance, parameters, any props, etc. Parameters and props may be better captured in yaml header and others as markdown content.

For profile, we can have a profile.md file in the top level markdown directory having profile title and version metadata (as described above for catalog). In addition to this a profile also needs to specify imports.
There can be one on more sections in profile.md corresponding to imports. Each import section should capture -

href: to specify catalog or profile URL
a list of control ids to be imported

We may not support include-all, exclude controls, or patterns. This should fine for the current requirements.

I'm leaning toward yaml so it can handle more general structures and we don't need to specify a specific format or items supported.
Profile needs multiple imports and potentially complicated include/exclude structures.  As long as the dictionaries can be applied unambiguously to the model it should be ok.  Error otherwise.
backmatter is another potentially important piece and I think it would also work better with yaml than markdown.
A key point is that catalog assemble does not require a catalog.json file, but profile assemble does require one.
Also, ssp assemble doesn't require an existing ssp json, but it will load and modify it if it exists.  It uses the existing ssp to extract the components, and will set the import_profile href properly.  Otherwise the import_profile is REPLACE_ME.
In summary, current behavior allows loading all needed info from existing json for profile-assemble and ssp-assemble, but catalog-assemble currently does not check for an existing catalog json file.  A simple way to get around all this is to allow an optional catalog.json file with metadata filled in.
catalog.yaml ->
metadata:
  title: "the title"
  version: "1.001"

profile.yaml ->
metadata:
  title: "the title"
  version: "1.001"
imports:
  -
    href: my_url1
    include-all: true
  -
    href: my_url2
    include:
      - ac-1
      - ac-2
    exclude:
     - ac-3
     - ac-4
merge:
  combine:
      method: merge
  as-is: true

Yet another option is to let all this happen in trestle create, and allow it an optional yaml file to inject into the created json model.  Then allow catalog assemble an optional catalog.json file.
But it isn't a huge burden just to use the normal trestle create and then edit the json file.
Either way - I think catalog assemble should allow an optional json file to use as the base for adding groups and controls.

Using YAML would be beneficial for us just because it is a well-defined format that user has to comply with. The conversion between JSON and YAML is also much easier. Whereas markdown is pretty much a free form text where nothing stops the user of doing something unexpected.
On another hand markdown is more "user-friendly" if the user does not know YAML structure. Fixing the user-related errors could be tricky and might made such "YAML" unreadable (we can of course add git push check to validate YAML structure). The big advantage of markdown here is that user can preview it directly in the Github to see if it is valid.
Representing a Profile imports in markdown could have an identical structure to the YAML.
Imports:

Import 1:

href: some_url
include-all: true

Import 2:

href: some_url
include:

ac-1
ac-2

exclude:

ac-3

We can also use md tables for such lists:

include
exclude

ac-1, ac-2
ac-3, ac-4

Decision was made per #1008 simply to provide a reference json catalog when doing catalog assemble.  This means all -assemble tools will have a reference json file.  Some editing of json will be needed in the overall editing cycle to provide metadata and other items but the edits should be relatively straightforward and rare.


⚠ Turnos 24x7 has degraded performance

In b0ece15, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 5283 ms

Resolved: Turnos 24x7 performance has improved in e687388.


Force Rewrite to Include Name Arg

Hi!
When calling rewrite_intl_messages I need to ensure that Intl.message calls always define the name argument. Currently, name will be added only when the args argument exists. I need the name to be added regardless of whether args exists.
The solution is simple and I can PR a fix if this is something you are willing to accept.
Thanks.

Sorry, this had gotten filtered down and I didn't notice it. Yes, I'd be happy to accept that as an option. So are you using rewrite_intl_messages on an ongoing basis? Generally I think people have just used it as a way of migrating away from transformer usage.

Closing in favor of #61.


Code on main repository

@yasmensarhan27 HI, your code is still not in the main repository.

merged now
the algorithm markdown file and the python code file

thank you


Make exception messages more descriptive for cases when test suites or flows are incorrectly disposed

The problem
When TeamCityTestSuiteBlock or TeamCityFlowWriter instances are disposed, a check is made if no child suites/tests/flows are open. If there are open children, an exception is thrown. The exception message is not very descriptive, e.g.:
Some child test suite writers were not disposed.
System.InvalidOperationException
   at JetBrains.TeamCity.ServiceMessages.Write.Special.Impl.Writer.TeamCityTestSuiteBlock.DisposeImpl()
   at JetBrains.TeamCity.ServiceMessages.Write.Special.Impl.Writer.BaseDisposableWriter`1.Dispose()
   at JetBrains.TeamCity.ServiceMessages.Write.Special.Impl.TeamCityWriterImpl.Dispose()
...

which is not very useful to diagnose the problem.
The proposal
The proposal is to add open suite / test / flow names to the exception message.

Hello @matt-richardson,
Could you please take a look at this PR and check that the changes in TeamCityTestSuiteBlock.cs will help with diagnosing the problem you described in #34?


程序每次启动都没有下载历史记录,不知道历史记录去哪里了

如题,我是win系统,motrix每次启动,记录里的历史列表都是空的,不知道什么情况.
有遇到和我一样问题的同学吗?

我也遇到了，而且更可怕的是内存吃了 1.7G，弃疗

目前的版本因为 aria2 的 API 设计原因，没有保留历史下载记录的，重新启动应用就会清空

"Persepolis Download Manager" is also base on Aria2, but it has a download history.  It make a sqlite DB file in the APPDATA folder.
Maybe Motrix can do it in the same way.

下载大文件的时候到 80% 崩溃了，然后重启发现没有记录，无法继续……而且不能从 .aria2 文件中恢复

I also wish Motrix could keep download history as I think it's a core function of any download manager

I also wish Motrix could keep download history as I think it's a core function for any download manager


build-a-pluggable-web-widget-part-1

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: build-a-pluggable-web-widget-part-1
Document link: create-a-pluggable-widget-one.md
My Issue/Suggestion
Where the text reads: "As part of the widget scaffolding, the generator builds the widget for the first time. You can do this yourself by running npm run build.
There is also a watcher available that will rebuild your widget as you make changes to files. Start the watcher by running npm start."
it would be good to mention one must first navigate to the widget-folder, eg: cd TextBox. Otherwise, the npm-commands cannot find the package.

Hi @masebok ,
Thank you for reaching out. You are totally correct, it would be smoother if the doc specified where to start the builder and watcher. I've added your details explaining that users must do it inside the widget's directory.
Thank you for improving Mendix's documentation,
Connor


arm32 build issue with some libs

Describe the bug
When building for arm32 those libs do not build:
# ec2;\
# ec2-instance-connect;\
# polly-sample;\
# ssm;\
# ssm-contacts;\
# ssm-incidents;\
# ssm-sap;\
# sagemaker;\
# sagemaker-a2i-runtime;\
# sagemaker-edge;\
# sagemaker-featurestore-runtime;\
# sagemaker-geospatial;\
# sagemaker-metrics;\
# sagemaker-runtime;\
# rds;\
# rds-data;\
# securityhub;\
# quicksight;\
# iot;\
# redshift;\
# redshift-data;\
# redshift-serverless;\
# pinpoint;\
# pinpoint-email;\
# pinpoint-sms-voice-v2;\
# glue;\
# lightsail;\
# s3-crt;\
# s3;\
# s3control;\
# s3-encryption;\
# s3outposts;\
# identity-management;\

Expected Behavior
build as they do with arm64, x86-64
Current Behavior
error.log
Reproduction Steps
yocto (master) + meta-aws and enable building of all libs
Possible Solution
No response
Additional Information/Context
No response
AWS CPP SDK version used
1.10.30
Compiler and Version used
x86_64-poky-linux-g++ (GCC) 12.2.0
Operating System and version
yocto master on ubuntu 22.04.

Easy reproducible example:
https://github.com/thomas-roos/yocto_example/tree/aws-sdk-cpp_issue_2242

cmake -DCMAKE_CXX_FLAGS="-Wno-psabi" helped to fix some ABI warnings but still those - do not build:
ec2
s3-encryption
identity-management

ERROR:
...
aws-cpp-sdk-ec2-integration-tests/CMakeFiles/aws-cpp-sdk-ec2-integration-tests.dir/EC2Tests.cpp.o:(.rodata+0xc0): first defined here
| /home/ubuntu/yocto/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.24-r0/recipe-sysroot-native/usr/bin/arm-poky-linux-gnueabi/../../libexec/arm-poky-linux-gnueabi/gcc/arm-poky-linux-gnueabi/12.2.0/ld: tests/aws-cpp-sdk-ec2-integration-tests/CMakeFiles/aws-cp
p-sdk-ec2-integration-tests.dir/EC2TestsDualStack.cpp.o:(.data.rel.ro+0x8): multiple definition of `typeinfo for Aws::Endpoint::DefaultEndpointProvider<Aws::Client::GenericClientConfiguration<true>, Aws::Endpoint::BuiltInParameters, Aws::Endpoint::ClientContextParameters>'; tests/a
...

full log (for ec2): error_ec.log

Can you try updating to the latest version of this sdk or remove unity from your build? Since this issues was opened we broke up the unity build, so it should now build on 32bit systems. You still might have problems with ec2, but please let us know if you run into those problems

Hi, build now works fine. But just one thing would be great if possible.
Only on arm32 it seems that absolute paths are in the generated cpp files.
Yocto is complaining about that - would be great if those were like relative paths.
Also I do not know why this does not happen with arm64?
Such as in this file: generated/src/aws-cpp-sdk-vpc-lattice/ub_VPC-LATTICE.cpp
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspaceResourceAssociation.cpp>
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspaceState.cpp>
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspaceType.cpp>
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspacesIpGroup.cpp>
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspacesPool.cpp>
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspacesPoolError.cpp>
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspacesPoolErrorCode.cpp>
#include </home/ubuntu/data/yocto_master/build/tmp/work/cortexa15t2hf-neon-poky-linux-gnueabi/aws-sdk-cpp/1.11.395/git/generated/src/aws-cpp-sdk-workspaces/source/model/WorkspacesPoolSession.cpp>


Developer (C#, JavaScript)

Job title: Software Developer
Job level: Intermediate or Senior
Company name: Xero
Referral limit: 4
Link to job description: http://app.jobvite.com/m?3VHUKkw5
Contact info: @quintuschen or 陈果 on WeChat

Job fulfilled, 1 referral made but not successful.


Document how to have the "Map view" button

I tried to have the "Map view" button on a local instance of the UI, but I wasn't able to have it displayed for queries that returns polygons ; but it's working well on the OSM Planet dataset on the demo instance.
Is it a feature that needs to be configured at the dataset level?
What is the best way to do it?
Is the map already part of the UI, or is there a way to provide a path to use to point to the location where the map is hosted?

The map is a separate project: adfreiburg/qlever-petrimaps. There currently is no way to configure the "Map view" button for custom datasets. Currently the "Map view" button is shown for some predefined datasets if the last column's datatype is WKT literal. But let's use this issue to collect what you'd need to use it:

Configuration on a dataset level

Enable the "Map view"
Set the URL of the map deployment

Is the heuristic "the last column is of type WKT literal" sufficient for you?

@Qup42 Thanks a lot for your anwser here!
Yes, this would be what we would like to see.
Regarding the configuration, it could be great to have it documented somewhere also once it's configurable :)
Regarding the heuristic, it's good enough for now, yes.
In the future, maybe having a way to define the name of a column or so could be great.
If there is a column that have the name that is in the configuration, and if it contains WKT litterals, then it shows the button ; if the column is missing, but if the last one contains some WKT litterals, use that one as fallback, and if not, don't display the button.

Thanks @Qup42.
I understand the actual mechanism which makes the button appear or not, but I don't see how we can specify the link of the petrimaps instance when the button appears.
Maybe it's hard coded at the moment in the UI, on a path where you know you deployed the tool on your side?

Maybe it's hard coded at the moment in the UI, on a path where you know you deployed the tool on your side?

Yes, the path to the petrimaps instance is currently also hard coded.

Yes, this would be what we would like to see. Regarding the configuration, it could be great to have it documented somewhere also once it's configurable :)

Of course :)

Regarding the heuristic, it's good enough for now, yes. In the future, maybe having a way to define the name of a column or so could be great. If there is a column that have the name that is in the configuration, and if it contains WKT litterals, then it shows the button ; if the column is missing, but if the last one contains some WKT litterals, use that one as fallback, and if not, don't display the button.

Thanks for the input. I'll add that to the list. Having the data not in the last column unfortunately seems to require a change to the map tool as well.

Hi @Qup42,
I hoped updating my data would display the button, but I failed :/
Here is my Qlever test instance.
When I run the following request, returning object typed as wktLiteral as last parameter, button (even with a false link) is not displayed.
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
PREFIX geo: <http://www.opengis.net/ont/geosparql#>
PREFIX ign: <http://data.ign.fr/def/geometrie#>
SELECT ?label ?geometry WHERE {
  ?s ign:geometry/geo:asWKT ?geometry .
  ?s rdfs:label ?label .
}
ORDER BY ?label

Do you know why?

Your dataset isn't in the (currently hardcoded) set for which the map view is considered. The dataset must be in this predefined set and the last column must contain WKT literals.

Oh, I hadn't seen this list.
Magic, the button appears :)
Is this filtering useful or is a hack that stuck? Can we delete it?

Looks like a hack that stuck to me, yes. I'd opt for a simple switch per dataset as described here. That would make the filter obsolete.
I'll try to discuss this internally tomorrow, so that we can move forward with fixing it.


Added docs on login URLs/view/template for permissions

Coverage remained the same at 82.604% when pulling 90c121a644de54be98af7e9438db40ac3a391006 on evildmp:docs-permissions into 1ef83443e7d8be8439ae1f04bb9460246f000eb0 on divio:release/3.4.x.

Coverage remained the same at 82.604% when pulling f2fdc54a59db67f62fc9e8602ab221abe93481a0 on evildmp:docs-permissions into c195fc9fad07029f83765276489a5f1a0678fd0a on divio:release/3.4.x.

@FinalAngel Not sure what happened the first time, please try again!


Add Solana teleburn address

Example output:
$ ord teleburn 4e27441f9c9e535868bf4778179c23ae306c7f0608e439b5f8f1a4953d5badf3i0
{
  "ethereum": "0xcc6251db97730972a4e1875afa27f07600dbcdcc",
  "solana": "Ekq8Ewm6dpkjrTiznHKiBBa6t6faP87Dhjpgan6fdFUC"
}

Implementing #1705

@orenyomtov is it worth updating the docs to explain this process a bit?

@orenyomtov we merged the ethereum teleburn address branch and this got closed. Can't reopen because it was based on. that branch. If you want just open a new PR and we'll review it


os time change

when os time change，Cron will not determine whether to execute a job based on the current time

you can use *cron.Cron.Entries()[0].Next  to set cron next excute time,also can processing time change


Hotfix/calendars swift

This change is

Reviewed 22 of 22 files at r1.
Review status: all files reviewed at latest revision, 17 unresolved discussions.

byuSuite/Apps/Calendars/controller/CalendarsEventDetailViewController.swift, line 23 at r1 (raw file):

        super.viewDidLoad()

        tableView.estimatedRowHeight = 44.0                 //Arbitrary value, set to default cell height

Create extension

byuSuite/Apps/Calendars/controller/CalendarsEventDetailViewController.swift, line 87 at r1 (raw file):

                    //When user allows the feature to access their calendar.
                    event.calendar = store.defaultCalendarForNewEvents
                    let controller = EKEventEditViewController()

Check constructor

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 78 at r1 (raw file):

    //MARK: Custom Methods

    private func loadEvents(startDate: Date, months: Int) {

make month default

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 79 at r1 (raw file):

    private func loadEvents(startDate: Date, months: Int) {
        let dateFormat = DateFormatter.defaultDateFormat("YYYY-MM-dd")

private let

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 89 at r1 (raw file):

                        //Sort just the events coming back. Because we know this small back are within a set date range it will not put self.events out of order since we sort by date.
                        //Default to today's date just to prevent crashing but will likely never come back as nil
                        let sortedEvents = events.sorted { return $0.startTime ?? Date()  < $1.startTime ?? Date()  }

look at inlining

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 16 at r1 (raw file):

class CalendarsViewController: ByuViewController2 , UITableViewDataSource {

    @IBOutlet private weak var tableView: UITableView!

Add //MARK

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 68 at r1 (raw file):

        CalendarsClient.getCalendarCategories { (categories, error) in
            self.spinner.stopAnimating()
            if let err = error {

Flop these

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 73 at r1 (raw file):

                var tempCategories = categories ?? [CalendarCategory]()
                //Add Academic Calendar to Categories
                tempCategories.append(CalendarCategory(dict: ["CategoryId": ACADEMIC_CAL_ID, "Name": ACADEMIC_CAL_NAME]))

Save the Dictionary as a private static final

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 75 at r1 (raw file):

                tempCategories.append(CalendarCategory(dict: ["CategoryId": ACADEMIC_CAL_ID, "Name": ACADEMIC_CAL_NAME]))
                //Sort Categories alphabetically by name
                tempCategories.sort { return $0.name ?? "" < $1.name ?? "" }

Check if default is needed

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 75 at r1 (raw file):

                tempCategories.append(CalendarCategory(dict: ["CategoryId": ACADEMIC_CAL_ID, "Name": ACADEMIC_CAL_NAME]))
                //Sort Categories alphabetically by name
                tempCategories.sort { return $0.name ?? "" < $1.name ?? "" }

Instead of using sort, use sorted, and just set the self.calendarCategories

byuSuite/Apps/Calendars/model/CalendarCategory.swift, line 20 at r1 (raw file):

    init(dict: [String: Any]) {
        categoryId = dict["CategoryId"] as? String
        let temp = dict["Name"] as? String

inline

byuSuite/Apps/Calendars/model/CalendarCategory.swift, line 23 at r1 (raw file):

        name = temp?.decodedHTMLEntities()
        if let categoryId = categoryId {
            iCalFeed = String(format: "%@%@", I_CAL_FEED_BASE_URL, categoryId)

Use regular swift notation. Talk to me about this one

byuSuite/Apps/Calendars/model/CalendarEvent.swift, line 13 at r1 (raw file):

private let DATE_FORMAT = DateFormatter.defaultDateFormat("yyyy-MM-dd HH:mm:ss")
private let STRING_DATE_FORMAT = DateFormatter.defaultDateFormat("MM/dd/YYYY hh:mm a")

Use lower case Y

byuSuite/Apps/Calendars/model/CalendarEvent.swift, line 64 at r1 (raw file):

    func generateEkEvent(store: EKEventStore) -> EKEvent? {
        guard let eventTitle = title,
            let startDate = startTime,

Double check if these are required

byuSuite/Apps/Calendars/model/CalendarEvent.swift, line 77 at r1 (raw file):

        if event.startDate == event.endDate {
            event.endDate = event.endDate.addingTimeInterval(1.0)   //Add one second to avoid bug in EKEventEditViewController

Test to see if this is still a bug

byuSuite/Apps/Calendars/model/CalendarsClient.swift, line 18 at r1 (raw file):

        request.startAsynchronous { (response) in
            if response.succeeded, let dataArray = response.getDataJson() as? [[String: Any]] {
                let categories = dataArray.map({ (dict) -> CalendarCategory in

inline

byuSuite/Apps/Calendars/model/CalendarsClient.swift, line 35 at r1 (raw file):

                completionHandler(nil, response.error)
            } else {
                guard let result = response.getDataJson() as? [[String: Any]] else {

Flop this one

Comments from Reviewable

Review status: 14 of 22 files reviewed at latest revision, 17 unresolved discussions.

byuSuite/Apps/Calendars/controller/CalendarsEventDetailViewController.swift, line 23 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Create extension

Done.

byuSuite/Apps/Calendars/controller/CalendarsEventDetailViewController.swift, line 87 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Check constructor

There doesn't seem to be a constructor that takes parameters.

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 78 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
make month default

Should I not worry about passing in months then, when I call this method?

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 79 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
private let

Done.

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 89 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
look at inlining

Done.

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 16 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Add //MARK

Done.

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 68 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Flop these

Done.

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 73 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Save the Dictionary as a private static final

Done.

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 75 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Check if default is needed

Done.

byuSuite/Apps/Calendars/controller/CalendarsViewController.swift, line 75 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Instead of using sort, use sorted, and just set the self.calendarCategories

Done.

byuSuite/Apps/Calendars/model/CalendarCategory.swift, line 20 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
inline

Done.

byuSuite/Apps/Calendars/model/CalendarCategory.swift, line 23 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Use regular swift notation. Talk to me about this one

Done.

byuSuite/Apps/Calendars/model/CalendarEvent.swift, line 13 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Use lower case Y

Done.

byuSuite/Apps/Calendars/model/CalendarEvent.swift, line 64 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Double check if these are required

They are indeed.

byuSuite/Apps/Calendars/model/CalendarEvent.swift, line 77 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Test to see if this is still a bug

Done.

byuSuite/Apps/Calendars/model/CalendarsClient.swift, line 18 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
inline

Done.

byuSuite/Apps/Calendars/model/CalendarsClient.swift, line 35 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Flop this one

Done.

Comments from Reviewable

Reviewed 8 of 8 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 78 at r1 (raw file):
Previously, ecbrady (Erik Brady) wrote…
Should I not worry about passing in months then, when I call this method?

Yup. Don't pass it in

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions.

a discussion (no related file):
Look into search with me

Comments from Reviewable

Review status: all files reviewed at latest revision, 2 unresolved discussions.

byuSuite/Apps/Calendars/controller/CalendarsEventsViewController.swift, line 78 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Yup. Don't pass it in

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: 19 of 24 files reviewed at latest revision, 1 unresolved discussion.

a discussion (no related file):
Previously, romrell4 (Eric Romrell) wrote…
Look into search with me

Done.

Comments from Reviewable

Reviewed 5 of 5 files at r4.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Classes/Reusable/UI/ByuSearchViewController.swift, line 11 at r4 (raw file):

import UIKit

//TODO: Read and verify comments

Resolve TODO

Comments from Reviewable

Review status: 23 of 24 files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Classes/Reusable/UI/ByuSearchViewController.swift, line 11 at r4 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Resolve TODO

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r5.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


form validation not work

I'm submitting a...

[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior

currently form validation not work. error message not showing. empty form submit without validation.
Expected behavior

required message should be showing.

Can we use (FluentValidation)[https://www.nuget.org/packages/FluentValidation] for form validation ?

On 5/23/2018 3:55 PM, Md. Aftab Uddin Kajal wrote:

Can we use
(FluentValidation)[https://www.nuget.org/packages/FluentValidation] for
form validation ?

I will check later and share my opinion. In the meantime you can use
anything you want. we can refactor code anytime.

@kajal-alpha : It is working on my end. Can you please check and provide more info about how to reproduce the problem?


fix the issue with allowing articles to exist in multiple sections

@miltontony ready for review 😞

👍 🍰 silly bug! 🐞


CI failure: provisioning K8s 1.11 VMs failed due to invalid Docker version

Build link: https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/3448/testReport/junit/k8s-1/11/K8sKafkaPolicyTest_Kafka_Policy_Tests_KafkaPolicies/
Output:
 k8s1-1.11: [init] using Kubernetes version: v1.11.0
    k8s1-1.11: [preflight] running pre-flight checks
    k8s1-1.11: I0727 14:08:06.152813    3503 kernel_validator.go:81] Validating kernel version
    k8s1-1.11: I0727 14:08:06.152858    3503 kernel_validator.go:96] Validating kernel config
    k8s1-1.11: 	[WARNING SystemVerification]: docker version is greater than the most recently validated version. Docker version: 18.03.1-ce. Max validated version: 17.03
    k8s1-1.11: [preflight/images] Pulling images required for setting up a Kubernetes cluster
    k8s1-1.11: [preflight/images] This might take a minute or two, depending on the speed of your internet connection
    k8s1-1.11: [preflight/images] You can also perform this action in beforehand using 'kubeadm config images pull'
    k8s1-1.11: [preflight] Some fatal errors occurred:
    k8s1-1.11: 	[ERROR ImagePull]: failed to pull image [k8s.gcr.io/kube-controller-manager-amd64:v1.11.0]: exit status 1
    k8s1-1.11: 	[ERROR ImagePull]: failed to pull image [k8s.gcr.io/pause-amd64:3.1]: exit status 1
    k8s1-1.11: [preflight] If you know what you are doing, you can make a check non-fatal with `--ignore-preflight-errors=...`

No logs from the test are needed; the error is not related to Cilium; it is an issue with dependencies installed in the VM.

Looks like google container was not correct, not sure if we need to deep more on this, I don't think so

@eloycoto what does "google container was not correct" mean? Please clarify.


open link to ofxaddons instead of showing tooltip

instead of showing a tooltip on the "Addons" label, maybe we should use an icon (like the glyphicon/halflings "share" icon) and this technique https://github.com/jlord/git-it-electron/pull/35 to open a window to ofxaddons.com in a browser.

Good idea, looks nice :)
The technique was already used but got broken somehow...


[SPARK-45475][SQL][3.5] Uses DataFrame.foreachPartition instead of RDD.foreachPartition in JdbcUtils

This PR cherry-picks https://github.com/apache/spark/pull/43304 to branch-3.5

What changes were proposed in this pull request?
This PR is kind of a followup for https://github.com/apache/spark/pull/39976 that addresses https://github.com/apache/spark/pull/39976#issuecomment-1752930380 comment.
Why are the changes needed?
In order to probably assign the SQL execution ID so df.observe works with this.
Does this PR introduce any user-facing change?
Yes. df.observe will work with JDBC connectors.
How was this patch tested?
Manually tested.
Was this patch authored or co-authored using generative AI tooling?
Unit test was added.

Merged to branch-3.5.


Not working for created() lifehook

The docs doesn't show the content for created() lifehook.
    /**
     * This is the created() lifehook of VueJS. It is activated and executed during the creation of views.
     * This function fetches the list of InterMines from the InterMine Registry API endpoint and pushes it to an array for further usage.
     * It also takes input query parameters from the URL.
     * @param {String} search - URL query parameter which acts as the search term
     * @yields {Array} The array containing the details of each InterMine instance
     * @yields It also allows us to use the search tool by passing query parameters from any other external website
     */
    created () {
      let vm = this
      axios.get(`https://registry.intermine.org/service/instances`)
      .then(response => {
        vm.minesList = response.data.instances
        response.data.instances.map((mine) => {
          this.selectIntermines.children.push({
            text: mine.name,
            url: mine.url,
            neighbourhood: mine.neighbours
          })
        })
      }).then(() => {
        this.selectAll()
        vm.progressDialog = false
        if (this.$route.query.search) {
          this.searchTerm = this.$route.query.search
          this.searchMine()
        }
      })
    }

That's because Vue hooks are implicitly excluded from generated documentation (https://github.com/Kocal/jsdoc-vuejs/blob/master/index.js#L61).
IMO, it's a non-sense to document lifecycle hooks, because they are internal to your component and do not provide any value.

However, you are free to open a PR if you want hooks to be documented.
But this functionnality should be configurable and disabled by default.


includeEnumInFeildComment is mispelled

it is just a name mispelling. The word "field" was written as "feild".
includeEnumInFeildComment should be includeEnumInFieldComment

Thank you for your comment.
Fixed in v1.0.2.


Easy slug snippet

Created a snippet that can create a URL friendly slug without the use of any regex, and will be easy to learn for newbies of javascript.

@Trinityyi mam , it's not that similar code,  I reviewed that code and that was converting the string to slug using regular expression.
But if you look at my code it is not using any regular expression,  I wrote this snippet for those people who don't understand regular expressions.
Please review the code again mam

@Tinokura What happened sir,  is there any mistake?

@Creatoon Implementation details are irrelevant. We only keep one snippet for each piece of functionality.


provide AutoUpdate for desktop installer

This issue has been migrated from:
https://github.com/mike-goodwin/owasp-threat-dragon-desktop/issues/101
There is a branch for applying auto-updates using squirrel.  It should be cross platform for Windows, MacOS and Linux.
File app/app.js has been modified to not use autoupdate via PR OWASP/threat-dragon-desktop#102  because it was causing problems, but this modification can be undone when autoupdate is working:
//electron autoupdate
//Note: autoupdate has been disabled until this issue has been satisfied:
//      https://github.com/mike-goodwin/owasp-threat-dragon-desktop/issues/101
// app.run(['common', 'dialogs', 'electron', 'VERSION', require('./app/config.autoupdate')]);

This can probably be looked into again, as MacOS signing is done and Windows signing will be done in the very near future.

With both MacOS and Windows signing in place, this can be considered for version 2.0. It is not a pre-requisite for the version 2.0 release, but it would be nice to have.
It depends on having the electron wrapper in place

It may be easiest  to use electron's auto-updater:
https://www.electronjs.org/docs/latest/tutorial/updates

Decided to use electron/update-electron-app


[release-1.5] Fix the cert rotation setup for istio agent.

This is an automated cherry-pick of #21274

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio_release-1.5
fa904d67b94811fc0653570c27c44d2e3b084fa4
link
/test integ-telemetry-k8s-tests_istio_release-1.5

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

/test integ-telemetry-k8s-tests_istio_release-1.5

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e-bookInfoTests-envoyv2-v1alpha3_istio_release-1.5
fa904d67b94811fc0653570c27c44d2e3b084fa4
link
/test e2e-bookInfoTests-envoyv2-v1alpha3_istio_release-1.5

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

/test e2e-bookInfoTests-envoyv2-v1alpha3_istio_release-1.5


Running infinitly

Hello
I try to implement the extension in our platform but the display is very very very slow...
The extension is loading to infinity ...
I follow all the things mentioned in the ReadMe.

I have the same problem. On clicking "Generate", get infinite loading and no results back

Dear @J-Day and @Baatof,
We kindly request your assistance in checking for errors in the Cloud console. Specifically, we would like you to:

Navigate to the BigQuery interface.
Check the project history section for any errors.
If no errors are found, open the dev tools console and share any error logs displayed there.

Hi
@J-Day ,
Have you check the permissions and the console logs? Usually it's due to configuration.
We'll try to track error messages for these.


Remove the unnecessary cast for main_t, NULL and argv

Summary
I have try my best to fix the style issue, so please ignore the rest style warning.
Impact
Minor, code refactor
Testing
Pass CI

@pkarashchenko could you merge this patch?


Test mask rcnn failed

I only have one gpu, so my test mask rcnn script is :
python tools/test.py configs/mask_rcnn/mask_rcnn_r50_fpn_1x_coco.py checkpoints/mask_rcnn_r50_fpn_1x_coco_20200205-d4b0c5d6.pth --out results.pkl --eval bbox segm
but it run failed and reported error as below:

My environment info as follows:
pytorch: 1.6.0+cu101
torchvision: 0.7.0+cu101
gcc: 5.4.0
cuda: 10.1
Addition：I test/train FasterRcnn successful, it's not any error, but MaskRcnn will crash when I test or train
I don't know why, is there anyone can help me ? Thank you in advance！

Hi! I do not seem to be able to reproduce your problem. Please provide more information by following the template of reporting an error.

Hi! I do not seem to be able to reproduce your problem. Please provide more information by following the template of reporting an error.

Sorry for not following the template，now my new description is as follows：
My target is to test maskrcnn by one gpu

What command or script did you run?
python tools/test.py configs/mask_rcnn/mask_rcnn_r50_fpn_1x_coco.py
checkpoints/mask_rcnn_r50_fpn_1x_coco_20200205-d4b0c5d6.pth
--show-dir mask_rcnn_r50_fpn_1x_results

Did you make any modifications on the code or config? Did you understand what you have modified?
I'm not change any thing, only make a data dir and checkpoints dir to place dataset and your trained models

What dataset did you use?
coco2017

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.

You may add addition that may be helpful for locating the problem
I install pytorch by pip

If applicable, paste the error trackback here.

If there is anything else I shoud add, please let me know，thank you for your attention！

This does seem to be a problem. It is due to the Conv2d in mmcv.ops being deprecated. We will fix it soon.
For now you can fix it by editing this file. Add Conv2d to line 5 and remove line 6.

This does seem to be a problem. It is due to the Conv2d in mmcv.ops being deprecated. We will fix it soon.
For now you can fix it by editing this file. Add Conv2d to line 5 and remove line 6.

I did what you said and it worked. Thank you！


Cluster constants doesn't handle phi nodes properly

Cluster constants uses incorrect insertion locations for phi nodes. This can lead to invalid IR.

fixed by 96f43a660680ed789ed7202bfdbbbc7412023f6b


Use secure SHA-512 and AES-256 by default

SHA-1 is insecure and deprecated Using AES-256 reportedly improves resistance to certain quantum attacks.

+1

You can change the hash and encryption method.  However, the default settings allow for reasonable backwards compatibility which is needed by most users.


Validation endpoint

Replaced by:
https://github.com/ultrasoundmoney/reth-payload-validator/pull/1


Support use of children: Seq[ReactElement] within Props

Currently only children: ReactElement* will be converted to a curried parameter.
The use of varargs is a problem as:

It forces children to be the last parameter.
It means you can't use default values for any of your parameters.

Allowing children: Seq[ReactElement] would solve these two problems.

This may be a breaking change for users that are currently using children: Seq[ReactElement], so targeting the 0.7.0 release.

I just checked and this actually works. Closing.


Improvement of flash message.

Problem
For now the flash message is displaying using bootstrap simple alert message. Particularly for this there is an ISSUE, that is it's not automatically disappeared, need to manually close it or simply refresh the page for closing this, and it's directly placed in webpage as content which is very annoying and directly effect to the UX.
And also the flash message displaying very largely at the top of the screen with big size. That effect the UI of the Web site.
The Issue is here --->
https://github.com/user-attachments/assets/0b29caa7-7ca0-44a3-81b0-5368f34bb2a3
Solution you'd like
I want to replace this flash message with Bootstrap Toasts. For this the advantage is, it disappeared automatically after 5 second, You can also close it by clicking the cross button on it, which is impact the User Experience directly because it's not directly placed on the website content, Just appeared on the screen corner and disappeared automatically, like normal modern website looks like.
Like this -->
https://github.com/user-attachments/assets/d3b77738-8530-46e2-94af-006c8f8b46bb
Please assign me this ISSUE, I want to add this features Improvement.

@Suvadip-sana i wanted you to display error message in more creative ways, like " oops! looks like you have entered wrong password " or smtg like that, error msg has to be displayed on top itself
can you do that

I am not talking about error message. I am old about flash message, which flash the message just one time after its occuring.
I think you also try to say the same.
If it's that then yes. I am capable to do this. just messages need to update.
Is the flash message displaying format is left remain same or change it as I said earlier?
Please clarify this. @Soujanya2004

If you assign me this task. Please give it label because it's need lot more work to update all type of message and also implement the Toasts flash. @Soujanya2004
Thank You.

@Soujanya2004 please clarify this.

@Suvadip-sana here you go

@Soujanya2004
It's ready.
Should I create a PR? I also change the messages as you said.
https://github.com/user-attachments/assets/ac915c5b-ccce-40a4-8715-8dcfcf428a49


Error reported when install the environment

https://github.com/EricGuo5513/HumanML3D/blob/dbe45f03d2573c0e6fb20add93e6b95cfc37bac1/README.md?plain=1#L39
I believe this line should be conda env create -f environment.yaml. Correct me if I'm wrong.

Thanks! Have been corrected.


Stacking text commands

Per: https://github.com/Khan/KaTeX/issues/676
Sorry for the delay on this one. Been busy lately :(
Note: This does not include Main-BoldItalic.

@rrandallcainc I'll try to look at it this weekend.

@kevinbarabash Thanks for the quick feedback. Will take a look tomorrow or the wekeend.

@kevinbarabash This should be ready for another look. I opted to keep the old testing method as it looked more verbose/readable. I did end up adding the tests you suggested however.

Thanks for the review @kevinbarabash

@rrandallcainc great work!


Issues/339 export csv report

This PR is for issue #339 . It's nearing completion and should be completed sometime tomorrow. It is far enough along however for review and feedback.
Summary of this PR
This PR makes it possible to export the ALM Cumulative report (alm_report.csv) to Zenodo. In doing so, it provides a generic way to export data out of Lagotto.
At a high level the following things happen:

Reports are written to disk via ReportWriter using the rake report tasks. When they are written they are logged to ReportWriteLog.
Zip archives are created using ReportZipper.
report:export_to_zenodo is a new task  that will take the most recently exported ALM combined stats report, and it upload it to Zenodo in the background using Sidekiq and a DataExportJob

At a slightly lower level...

Behind the rake report:all_stats and related rake tasks is theReportWriter. This is a high level class that knows how to create specific reports using lower level classes. As it writes these reports to disk it logs this to ReportWriteLog.

The ReportWriteLog is used to keep track of what reports were written and when. It also keeps track of the file path for the report on the local file-system.
ReportWriter and ReportWriteLog are intended to minimize the number of places where code has to worry about report filenames, filename conventions, where files live, etc.

The rake report:zip task is still intact as well with the following changes:

It uses a new object, ReportZipper,  to handle knowing how to zip up certain reports and what files to include in them.

Therake report:export_to_zenodo task handles generating the Zenodo export, queuing up a ZenodoDataExport job, and then a background worker will pick it up and do the work.
Exceptions during the export are logged as Alert(s).

Running/testing this locally
Get a Zenodo Sandbox account set up

Get a Zenodo Sandbox account here: https://sandbox.zenodo.org
Get a personal token / API key by going to https://sandbox.zenodo.org/account/settings/applications/ once you've logged in and generate a new Personal access token
Update your .env file to have both ZENODO_API_KEY=... and  ZENODO_URL=https://sandbox.zenodo.org environment variables.

_Note: when ready to go to production the same steps will have to be repeated on Zenodo's main site: https://zenodo.org
Update your env variables
New ENV variables have been added that you'll need to configure in you .env file. You can find them in the .env.example file. You'll need:
CREATOR=Public Library of Science
ZENODO_KEY=YOUR_KEY_HERE
ZENODO_URL=https://sandbox.zenodo.org/api/
SITENAMELONG="PLOS ALM"

Checkout and run migrations
This adds couple new tables to the DB to keep track of reports written to disk.
bin/rake db:migrate

Run reports, zipping them up, exporting to Zenodo
Next, run the following rake tasks:
bin/rake report:all_stats report:zip report:export_to_zenodo

Note: Make sure you have Sidekiq up and running otherwise the Zenodo export won't happen.
Check "My Uploads" in Zenodo's Web Interface
Go to https://sandbox.zenodo.org/deposit/
On the right hand side you should see an upload listed. If you don't see an upload listed, then there are two things to check:
First thing to check
Look for an Alert record to have been created outlining any exceptions from talking with Zenodo
Second thing to check
Look at the last  ZenodoDataExport record to see if it has a starting_export_at and finishing_export_at fields set. If both of these are nil then they either haven't started or they started and failed (see Alerts); if starting_export_at is populated, but not finishing_export_at then the export is in progress. If finished_exporting_at has  value then the file is on Zenodo, try to refresh your browser.
Things to note, discuss
READMEs in the Zip
The alm_report.zip includes a README.md file. It gets pulled in from docs/readmes/alm_combined_stats_report.md. Its contents need to be fleshed out.
Date  in the zip filename
I have not included the alm_report.zip filename. This was only out of lack of time. It isn't huge effort to get in there, but I wanted to move on to the next issue and start making headway.  I figure we can always circle back and in this in later if need be.
Removing files after export
Right now the files exported (alm_report.zip) is not removed after export. The Zenodo export only knows about the zip file so it could remove that when done. However, there are other files left on the system.
Let's chat and see if we should have a cron job remove any/all files older than X days or if we want to this be integrated into the system in another way. The benefit of keeping the file around for a short period of time is that it may not be available on Zenodo right away and if there are any particular issues there will be a grace period for someone to investigate on the server w/o having to re-run a long a potentially long running report.
Zenodo Gem
The Zenodo gem's Pull request 7 is the last one we're waiting on. Once that is merged in and released we can use the official Zenodo ruby gem.
I think these changes will be merged in this weekend at the latest, but if they're not, we'll want to have ALM fork my repository until they are. Right now, the Gemfile uses my fork of the Zenodo ruby gem.

@mfenner, I've updated the PR description and pushed all of my changes. There are two things to note at the bottom of the PR description that we'll probably want to chat about tomorrow.
Let me know if yo run into any issues running or reviewing the code. Thanks,

Rebased in the latest changes on master and force pushed this branch.

@zdennis, I have worked through the pull request. Per your comments:

I have updated the README, in sync with the text we use for the Zenodo description
Data in the zip filename. We can leave it out for now.
removing files after export: I decided to remove the data folder, and store the reports in tmp/reports, all other files we generated temporarily in tmp/files. See #378.

@mfenner, thanks for reviewing, make those updates, and merging in!


Documentation request: test data + model order

Is your feature request related to a problem? Please describe.
I am concerned about how to evaluate the performance of the models as the documentation does not clearly explain how the test data is split. I also would like to estimate better how long to run iterations if I have already experienced with a related data set.
Describe the solution you'd like
I suggest adding to documentation:

how is test data split. E.g. is it from the end, randomly or based on auto-selection of columns?
how are the iterations generated. It would seem they might be in fixed order, or my data is fairly similar and leads to it. For example, if iteration 201 performed best, then when I re-train I could stop after iteration 201 next time, if I know this. If they are not in fixed order, then I know to let my next dataset run a bit longer.

@torronen thanks for the feedback

how is test data split. E.g. is it from the end, randomly or based on auto-selection of columns?

Depends on the size of your dataset. either 5-fold cv or hold-out (10% for test), all randomly. Considering the size of dataset you have, it will be hold-out

how are the iterations generated.

It's based on the performance of each trainer. If one trainer has a higher improving speed, it will be assigned with more time budget. The improving speed is evaluated via ECI, you can refer to the following paper for detailed info
https://export.arxiv.org/pdf/1911.04706
@briacht @luisquintanilla Maybe we can add those to the doc in the next release?

@luisquintanilla Can we please add a work item to Docs to talk about how Model Builder splits up the data?

Thanks for this feedback @torronen . I've added an issue to our backlog dotnet/docs#25091 to track this work.

This is being tracked in Docs now so I will close this issue here.


Set up a CI system (travis or circle)

Tests are there and working through test kitchen. Let's wire this thing up to CI and get a badge on the README.

@brandoconnor Good to see that you open this issue.
There is already a small group of people (@ozbillwang @ncs-alane to name a few from top of my head) who are interested in this effort for Terraform modules we maintain.
"Discuss/propose a CircleCI workflow" is already on the top of my list of things for the next week.
The question I'd like to get solved before copy-paste any solution to all modules I raised here. What do you think?

Excellent. I'm working on this through TravisCI setup currently and have been leveraging @ncs-alane 's kitchen-terraform. It's nearly working for a skeleton AWS module project. It'd be good to see a couple CI products tried (codeship, travis, circle) to see how well they do and if they support the requirements.
Assumptions/requirements:

the AWS (or whatever cloud provider) account used for testing will probably need to be paid for by a maintainer if it falls outside of free tiers. Maybe having a separate account we all can access and use for this purpose makes sense? Not sure. Let me know if anyone has ideas around eliminating cost.
The CI system should be 100% free. I think they all are but those free tiers likely have their own quirks and details that might matter.
secret/encrypted variables (environment variables are fine) are a must for credential storage. Sounds like this is possible with some differences on the edges for each of the platforms I've listed.
coming from the CM space, test kitchen is great and I think kitchen-terraform + awspec has been an all too familiar experience for me. It's not the only option but a good one. Interested to hear opinions on what people prefer.

I don't necessarily think we ALL need to take the exact same approach but it would be nice to have one or a few prescriptive paths for testing and TDD that infra developers can easily start a project with. Let's mosh in a chat room if we have enough to discuss.

AWS costs should not be an issue as long as we use dedicated AWS account just for running tests and keep an eye on a bill. I will get it covered/sponsored.
While I agree that there can be multiple alternative approaches to do testing it will be better if there is the default (aka recommended) way which fits most cases.
All other points you listed require some thinking. :)
Till next week!

@brandoconnor - Here you can see how I am using CircleCI 2.0 with Terraform - https://youtu.be/MmFu-oII9_k?t=15m . This is pretty similar to how I am thinking to use it for other modules.

:wave: I have been using TravisCI's beta bulid matrix feature to test kitchen-terraform against different versions of Ruby and Terraform; it works quite well.
There is also an AWS plugin for InSpec that is under development: https://github.com/chef/inspec-aws. It's not yet ready for mainstream usage but this effort could be a motivation to spur or participate in the development of that project.

https://github.com/terraform-aws-modules/terraform-aws-security-group/issues/4 - I've just made more "actionable" issue for a smaller module.

Resolved in PR https://github.com/terraform-aws-modules/terraform-aws-alb/pull/5. Closing.

As a final note, I'm using some personal account credentials for testing here. If you have an employer sponsor who we could transition to using, let me know and we can find the right way to add those to the Travis config instead. I've disabled the nightly tests as they sporadically fail (this terraform version isn't officially supported but does usually work).

I don't have coupon codes available at the moment, but I will try to get some before AWS re:invent (btw, are you going there?).
Maybe you can adjust TravisCI to build only if you are a committer to not spend $$$ so fast.

I won't be there this year (just started contracting full time) though I've gone in the past. Have fun if you're going!
Yea I'll just limit who and how often the build runs. It finishes in just a couple minutes anyway so I haven't seen any cost just yet.


NoSpacesInsideParenthesis - allow multi line with comments

PHP 7.1.1 / Fixer 2.x
<?php

$a = $b->test(  // do not remove space between '(' and this comment
    $c,
    $d
);

currently the space is removed, RFC so the fixer will allow space between ( and a comment.

makes sense, yet behaviour changing. it needs config option

👍 although I would consider this a bug, not a new feature.


Limit who can call notifications/govdelivery endpoint to Granicus IP addresses

As a VANotify system administrator
I want to secure my granicus webhook enpoint
So that anyone on the internet is NOT allowed to POST a request to it.
Context:
At the moment Granicus does not implement any security in their webhook calls.
As a workaround we should limit who can call our granicus webhook endpoint based on IP address.
Notes:
AWS Application Load Balancer allows creation of rules on each of the listeners to handle incoming traffic.
We can create a rule that will look at the path (notifications/govdelivery) and source IP address. If source IP address matches Govdelivery addresses it will forward traffic to our target group.
A second rule will match only path and will return a fixed response 404.

Discuss during kickoff if the VA Environment is ready, if not we can discuss options to split this story - TW, VA

Govdelivery addresses:
<IP_ADDRESS>/24
<IP_ADDRESS>/24
<IP_ADDRESS>/28
<IP_ADDRESS>/24
<IP_ADDRESS>/24
<IP_ADDRESS>/25
<IP_ADDRESS>/24
<IP_ADDRESS>/24

Pending ESECC Review.

Configuration to our dev & staging ALB added.
However Granicus is using IP addresses in staging environment that are not covered by the above list.
Keeping the IP addresses list in sync might be difficult.

Story is cancelled upon consideration as we are concerned about keeping IP address list and blocking in sync which if there is an out of syncness may cause serious issues in production.


Refactor matrix exports

Description

[x] Refactor mapping into saner, potential key-based representation
[x] Conditionally fall back to NumPy arrays?
[x] Ensure JAX training example still works
[x] Resolve #296

Checklist

[ ] Add tests
[x] Lint
[ ] Update docstrings

This should help in going from a matrix representation to the more conventional MD formats:
$ ipython
Python 3.9.0 | packaged by conda-forge | (default, Nov 26 2020, 07:54:06)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.27.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: run readme.py
Warning: importing 'simtk.openmm' is deprecated.  Import 'openmm' instead.
/Users/mwt/software/openff-units/openff/units/simtk.py:36: UserWarning: The openff.units.simtk module is deprecated. Use openff.units.openmm instead.
  warnings.warn(

In [2]: vdw = out['vdW']

In [3]: original = vdw.get_force_field_parameters()
WARNING:absl:No GPU/TPU found, falling back to CPU. (Set TF_CPP_MIN_LOG_LEVEL=0 and rerun for more info.)

In [4]: from jax import numpy as np

In [5]: modified = original * np.array([1.1, 0.5])

In [7]: modified / original
Out[7]:
DeviceArray([[1.1, 0.5],
             [1.1, 0.5]], dtype=float32)

In [8]: vdw.set_force_field_parameters(modified)

In [10]: vdw.get_force_field_parameters()
Out[10]:
DeviceArray([[3.7396364, 0.0547   ],
             [2.9144862, 0.00785  ]], dtype=float32)

In [12]: original
Out[12]:
DeviceArray([[3.3996694, 0.1094   ],
             [2.6495328, 0.0157   ]], dtype=float32)

In [13]: modified
Out[13]:
DeviceArray([[3.7396364, 0.0547   ],
             [2.9144862, 0.00785  ]], dtype=float32)

From here one could export out to an MD engine using the parameters that were tuned with matrix operations. The modifications here (increasing sigma by 10% and decreasing epsilon by 50%) are arbitrary; imagine if that operation was replaced by more complex operations, possibly in a different framework like JAX or PyTorch!


Outdated packages number on output

Closes #20

Thanks for your contribution. :-)
After review, please squash your commit on remove issue reference on commit comment.

Tests are failing, could you please take a look?
Note: I suggest you to ping me when it's done. Maintainers don't get notification for just commit on PR.

@Soullivaneuh should be ok now, sorry for the repeated mistakes ^^'

Thank you. :+1:
Will merge it later.

@EmanueleMinotto Available on v1.1.0.
Sorry for the delay.


analysis_options exclude isn't working, behavior differs between flutter analyze and dartanalyzer

Excluding files still displays analyzer warnings in IntelliJ and in dartanalyzer,  but not with flutter analyze
Here's my analysis_options.yaml file:
analyzer:
  exclude:
    - lib/exclude_me/*.exclude.dart
  strong-mode:
    implicit-dynamic: false

Flutter 1.9.1+hotfix.4,
Dart SDK 2.5.0
Steps to reproduce

clone https://github.com/johnpryan/analyzer_exclude_bug
run flutter analyze (no errors)
run dartanalyzer .
observe dartanalyzer displays warnings, even  though ànalysis_options.yaml`  excludes the file.

Related: #25551. The now closed dup #34053, also listed these related items: #28463, #25551, #29430

Good find, thanks. I think this is a dupe of #25551


Fix footer not positioned on bottom of page #54

Fixes #54 iPad mini: footer on Connect page
Issue can also be seen on desktop by zooming out so the page content is smaller than the page height.

After some CSS adjustments, footer is locked to bottom of page:

It also looks like the Gemfile.lock file was not updated when the jshint gem was added recently, so those updates are also included currently

@lizzyaustad thank you for your contribution!


Blog post fix

@peff @slopes321
Please review and accept, if it matches, my fixes for wrong commit images and references in the text for git Undoing Merge blog post.

The changes are more extensive here than I expected; I thought we just needed to give a new number to the duplicate C8. Not that changing more is necessarily wrong, but what made you decide to renumber the other commits?

I just wanted to keep all images consistent: all commits now looks in expected time order  and all images have the same basic root pattern (1->2->3->4->5), which looks better, than (1->2->15->4->5).
Basically, at first i made those images with "new number" 13. Than i understood, that 13 is used on other images and I should use some unique number (> 14 in our case). Then i looked at the images and understood, it is confuses, why 15 is between 2 and 8. So, finally i decided if change something - change it to be good completely, not just a t crutch.
That's it...

Thanks for explaining. I guessed it was something along those lines, but wanted to be sure.
I gave a brief read-through of the content and AFAICT the updates are all sane. Thanks for working on this. I'm sure the image-tweaking was a bit tedious. :)

yeah, especially in tricky GIMP :)

guess now #830 may be closed as well?


feat(watcher): make watcher can be executed

this PR resolves #5

:tada: This PR is included in version 1.0.0-dev.1 :tada:
The release is available on:

GitHub release
v1.0.0-dev.1

Your semantic-release bot :package::rocket:

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

v1.0.0
GitHub release
v1.0.0

Your semantic-release bot :package::rocket:


Remove router api

Description
Briefly describe the changes made in this Pull Request.
Additional context (optional)
Add any additional context or information about the changes made in this PR.
How to test
Provide clear and concise instructions on how to test the changes made in this PR.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.62%. Comparing base (0b0e83b) to head (8afbd8f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #391      +/-   ##
==========================================
+ Coverage   89.37%   89.62%   +0.25%
==========================================
  Files         118      119       +1
  Lines        3463     3441      -22
  Branches      773      769       -4
==========================================
- Hits         3095     3084      -11
+ Misses        337      326      -11
  Partials       31       31

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Migrate GetDelegateDto to zod

Summary
This migrates the validation of GetDelegateDto to zod.
Changes

Remove GetDelegateDtoValidationPipe and associated schema
Add and use GetDelegateDtoSchema schema
Infer GetDelegateDto from schema
Add tests accordingly

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

10 of 10 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 93.432%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
6340

Relevant Lines:
6553

💛  - Coveralls


Cypress test. Reset password notification.

Motivation and context
Cypress test for #3140
Increase coverage.
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Added a dummy email. So as when specifying the email with which the user is registered an error
[Thu Apr 29 06:18:11.449787 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]   File "/opt/venv/lib/python3.8/site-packages/django/core/mail/backends/smtp.py", line 102,
in send_messages
[Thu Apr 29 06:18:11.449790 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]     new_conn_created = self.open()
[Thu Apr 29 06:18:11.449793 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]   File "/opt/venv/lib/python3.8/site-packages/django/core/mail/backends/smtp.py", line 62, in open
[Thu Apr 29 06:18:11.449797 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]     self.connection = self.connection_class(self.host, self.port, **connection_params)
[Thu Apr 29 06:18:11.449801 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]   File "/usr/lib/python3.8/smtplib.py", line 253, in __init__
[Thu Apr 29 06:18:11.449805 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]     (code, msg) = self.connect(host, port)
[Thu Apr 29 06:18:11.449808 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]   File "/usr/lib/python3.8/smtplib.py", line 339, in connect
[Thu Apr 29 06:18:11.449811 2021] [wsgi:error] [pid 401:tid<PHONE_NUMBER>75008] [remote <IP_ADDRESS>:34008]     self.sock = self._get_socket(host, port, self.timeout)

is issued. Perhaps this is the correct behavior since there is no smtp server.


Rebuild against MySQL 8.3

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge/mysqlclient Ready for review.
I've ran into some issues with requiring mysqlclient to be rebuilt for 8.3 else I get a very old build that didn't depend on mysql-devel and which isn't usable due to:
E   sqlalchemy.exc.OperationalError: (MySQLdb._exceptions.OperationalError) (2059, "Authentication plugin 'caching_sha2_password' cannot be loaded: /opt/conda/envs/analysis-productions/lib/plugin/caching_sha2_password.so: cannot open shared object file: No such file or directory")

ping @conda-forge/mysqlclient


Not Available status for Sense 2

Hi,
Not sure if this is actually an issue, or expected, but both versions are listed as incompatible with the Sense 2

Just wondering if there's any plans to bring it to the Sense 2, or not?

I'd be also interested in this! I'd also be happy to help if something needs testing.

It likely just needs the Sense 2 checkbox ticked in Fitbit Studio. I'll try to do this soon.

I just checked and the latest version of Heart of Frogg should support the Sense (at least the first one). It's using the latest SDK and Fitbit Studio isn't showing me a checkbox for the Sense 2 so I'd assume it's supposed to work...? I just looked at your screenshots again and they don't actually say it's unsupported, it's just warning you that it's a private app (since I haven't officially submitted it to the Fitbit store). What happens if you tap the "Install" button?

For the "Sense 2", the install button is disabled. So clicking it sadly does nothing.
What you say is true though, in the app it doesn't say anything about support, but in the browser version it says it only supports "Sense" and "Versa 3".

I just poked around the Fitbit developer forums and apparently the Sense 2 does not have support for this party apps for some reason. Maybe they'll add support later on, but for now, Heart of Frogg cannot work on a Sense 2. Sorry. https://community.fitbit.com/t5/Sense-2/Are-third-party-apps-available-on-Sense-2/td-p/5244355


useSupabaseUser returns the user on the server (eg. middleware) but isn't populated on frontend

Version
@nuxtjs/supabase: 0.3.5
nuxt: 3.4.2
Reproduction Link
Can't really give a reproduction since it seems to happen on any environment and requires email magic link
Steps to reproduce

Get sent a magic link through a forgot password process
Click the link, you should be logged in on the website and the user should be available through the useSupabaseUser() method
Click the link in the email again, if you have middlewares checking for the value of user, they should work fine, but your frontend won't be able to get the user through useSupabaseUser()

What is Expected?
The user should be available on the frontend
What is actually happening?
The user isn't available on the frontend but only on the server
This behavior is very weird, but basically when logging the user value from the middleware, the user is populated on the server, when the middleware runs on the client, it's also populated, but after the hydration process, the user becomes null.
Here's a snippet of my console:

export default defineNuxtRouteMiddleware(async (to, _from) => {
  const user = useSupabaseUser();
  if (!user.value) return notLoggedIn(to.path);
  console.log("user =", user.value);

  if(process.client){
    setInterval(() => {
      const user = useSupabaseUser();
      console.log("user.value == null ?", user.value == null);
    }, 100)
  }

});

The same behavior occurs every time I refresh the page and can only be fixed by deleting the cookies.

Same happens to me

I had the exact same issue, FYI the workaround I found (not ideal but it works 🤗 ) :
// In my global authentication middleware
export default defineNuxtRouteMiddleware(async (to) => {
  // check is user is landing on your site after authentication funnel
  const isComingFromAuthentication = to.hash.includes("access_token");

  if (isComingFromAuthentication) {
    /*
    Explicitly post SIGNED_IN event to nuxt/supabase session api route.
     */
    const { data } = await useSupabaseAuthClient().auth.getSession();

    await $fetch("/api/_supabase/session", {
      method: "POST",
      body: {
        event: "SIGNED_IN",
        session: data.session,
      },
    });
    const { data: userData } = await useSupabaseAuthClient().auth.getUser();
    useSupabaseUser().value = userData.user;
  }
});

Also facing the same issue which unfortunately is a show stopper on a current project. If I console.log user.value it is null but when I console.log the user object instead and check the ObjectRefImpl object manually user data is present. Weird

I came up with a middleware solution that works. Maybe it also helps someone else:
export default defineNuxtRouteMiddleware(() => {
  const user = useSupabaseUser()
  const client = useSupabaseAuthClient()

  setTimeout(async () => {
    if (process.client && !user.value) {
      const { data } = await client.auth.getUser()
      user.value = data.user

      if (!user.value) {
        navigateTo('/login')
      }
    }
  })
})

I came up with a middleware solution that works. Maybe it also helps someone else:
export default defineNuxtRouteMiddleware(() => {
  const user = useSupabaseUser()
  const client = useSupabaseAuthClient()

  setTimeout(async () => {
    if (process.client && !user.value) {
      const { data } = await client.auth.getUser()
      user.value = data.user

      if (!user.value) {
        navigateTo('/login')
      }
    }
  })
})

Thanks @bart, worked for me!


RasterRef should not read HDFS scheme with GDAL reader.

And enforce other logic like can't read a gdal only format (mrf, jp2) outside the gdal reader.
Needs a close look and maybe more extensive unit tests. Possibly also more extensive / detailed error messages.
For #314
This also relates, from the user perspective, to #267

Still having some class not found errors in travis. Unsure why.

@vpipkt I think something's wrong with the implementation, because in this test case the scheme resolution should not be attempting to use the GDAL reader if it's not available, yet it's still selecting it over the default reader.
https://travis-ci.org/locationtech/rasterframes/builds/580393491#L832-L866

@vpipkt Not 100% sure what's going on here, but don't want to be extra sure the JVM fallback reader still gets selected in non-GDAL scenarios. Wonder if we should leverage this sort of logic?
https://github.com/locationtech/rasterframes/blob/36be70056fa01001bc3df006a4ea5655640c2d58/.circleci/config.yml#L126
Also odd: not seeing it here:
https://circleci.com/gh/s22s/rasterframes

Now you can see in Circle CI, it is passing. https://circleci.com/gh/s22s/rasterframes/tree/fix%2F314

@vpipkt Once this weirdness is resolved, should we be trying to use this (if memory serves, written by Azavea folk!)?:
https://gdal.org/user/virtual_file_systems.html#vsihdfs-hadoop-file-system

@metasim yep, by @jamesmcclain

@metasim yep, by @jamesmcclain

I am here to help if you run into any trouble with that.  Please be advised that some users have run into issues when trying to access HDFS filesystems through GDAL from a JVM process.  The problem appears to be related to the fact that GDAL spawns its own inferior JVM to interface with HDFS, and that is done from within a JVM process running native GDAL code (more here).

@jamesmcclain Thanks for the background color... sounds to me like we shouldn't go that route if we don't have to.

@jamesmcclain Thanks for the background color... sounds to me like we shouldn't go that route if we don't have to.

You are welcome


fixed bug in namespace lookup breaking import aliases (#5667)

Pull request opened by the merge tool on behalf of #5667

Merged in d9c0b4f176dce18719a1c461e5570f15272b74c3


Weird behaviour with dropdown menus

I've been working on implementing some dropdown menus with Tailwind and I'm seeing some weird behaviour with other dropdown menu controls.
I'm sure I haven't done something correctly so would appreciate any advice anyone can provide.
When the menu is collapsed, everything looks as expected:

When the menu is expanded, controls for other dropdown menus float above the menu, even though no z-indexs have been set:

Any help would be much appreciated!

Can you provide a sandboxed example?
That would be easier to help 🙂

Echoing @MichaelDeBoey's comment; if you can provide the code there's a lot more we can do to help.
Quick glance makes it look like the red boxes either have a z-index set or have position: absolute set which creates a stacking context and elevates them around the surrounding elements.
Since those red boxes appear after the dropdown in the DOM, they will have a higher effective z-index if the z-index is not actually set on either the drop down or the red boxes.
I suspect the fix for you in this case is just to assign a z-index of any value to dropdown area. I'd probably pick 20 or 30 if using the Tailwind defaults, so you still have some room above it for things like modals as well as below for anything in the main content area that needs a z-index but should be positioned under the dropdown.

I've done up a Codepen with minimal viable css, and applying the z-index to the dropdown content works as you have suggested, but applying the z-index to my site doesn't entirely fix the problem - sad face!
Applying the z-index has fixed the dropdown the red controls, but not the dropdown in the nav bar:

The markup & less file with customisations are available in this Gist, if you have any more suggestions as to what I could be doing wrong, it would be appreciated.
Btw, totally loving Tailwind - I've never been able to design a site that looks this good (and nothing like a Bootstrap site) before!

Hey @MichaelDeBoey, @adamwathan any chance you'd be able to help take a look into what might be going on here? Thanks in advance!

@kubacode Can you provide a small project (GitHub link) with only this page please? I'll take a look for you then 😉

@MichaelDeBoey That would be great - I've setup a quick repo here for you. Thanks again for helping me out 😄

@MichaelDeBoey were you able to take a look into this?

@kubacode Haven't had the time yet unfortunately 😕
Will try to take a look at it tonight 🙂

@kubacode Took a quick look and the problem I found was that header and the main container have the same z-index.
If you add an z-index (ie z-10) to the header, your problem will be fixed 🙂

@MichaelDeBoey Thats fixed it - thanks for taking a look!

@kubacode would you mind sharing your code with me? I'm trying to implement dropdown in tailwind too


Update foxit-reader.json

Original url returned a 404 error as there's something wrong with the versioning in it.

#3998


$IsWindows is read-only since PS 6.0

Opening vsts-promotepackage-task\vsts-promotepackage-task\vsts-promotepackage-task.ps1 in latest PS VS Code extension gives an error like this:
Starting from PowerShell 6.0, the Variable 'IsWindows' cannot be assigned any more since it is a readonly automatic variable that is built into PowerShell, please use a different name.
Am opening a pull request

The reason for this is because IsWindows did not exist in earlier versions of Powershell. It is only written to if it is not populated already (indicating it is on a newer version of Powershell that populates it automatically).

Yes, got it. The issue is, that setting it in PS > 6.0 is giving an exception that says
Starting from PowerShell 6.0, the Variable 'IsWindows' cannot be assigned any more since it is a readonly automatic variable that is built into PowerShell, please use a different name.
So i thought would be better not setting it at all :)
See Pull Request #50

@renevanosnabrugge  @NightOwl888  what do you think about that?