wip

This adds the boolean flag spec.security.configureMeshNetworkPolicyRules to
the SMCP, with the default value of true.
If it is true, current behavior is preserved, i.e., the operator creates
and manages the NetworkPolicies that allow communication between the
control plane and namespaces that are members of the mesh.
If it is false, the operator doesn't create such resources. The user is
entirely responsible for managing the NetworkPolicies and troubleshoot
any issue that might occur as a result of opting out of this feature.

/test all

@rcernich Not sure I get it. The NetworkPolicy must be created in the control plane so that member namespaces can communicate with the control plane ns. This PR prevents this NP to be copied to the member namespaces upon user request. But I think the original NP in the control plane ns must not be touched.
About chart changes... Are you saying we don't need to make changes in globals and shouldn't care about conversion v1 <-> v2? I'm a bit confused :)

Hey @jwendell, I meant in the code where we process the objects produced by the charts, e.g. here, you might do a check of the setting in smcp and skip processing if the object is of kind NetworkPolicy.

/test all

didn't we decide on a global on-off switch? shouldn't that mean we don't create any NetworkPolicy resources? I think that's clearer tbh
that would then also mean it should probably live straight under spec.security because it affects both control plane and data plane

didn't we decide on a global on-off switch? shouldn't that mean we don't create any NetworkPolicy resources? I think that's clearer tbh
that would then also mean it should probably live straight under spec.security because it affects both control plane and data plane

I updated the patch. Please take a look. It is a global switch.

didn't we decide on a global on-off switch? shouldn't that mean we don't create any NetworkPolicy resources? I think that's clearer tbh
that would then also mean it should probably live straight under spec.security because it affects both control plane and data plane

I updated the patch. Please take a look. It is a global switch.

Sorry, I didn't see you moved it to .spec.security already. Re: naming: I think spec.security.manageNetworkPolicy is good
This is missing a few places where we create NetworkPolicy resources:
$ rg NetworkPolicy resources/helm/v2.1/
resources/helm/v2.1/gateways/istio-ingress/templates/networkpolicy.yaml
3:kind: NetworkPolicy

resources/helm/v2.1/istio-control/istio-discovery/templates/networkpolicy.yaml
2:kind: NetworkPolicy

resources/helm/v2.1/istio-telemetry/kiali/templates/networkpolicy.yaml
3:kind: NetworkPolicy

resources/helm/v2.1/istio-telemetry/grafana/templates/networkpolicy.yaml
3:kind: NetworkPolicy

resources/helm/v2.1/istio-telemetry/tracing/templates/networkpolicy.yaml
4:kind: NetworkPolicy

resources/helm/v2.1/mesh-config/templates/networkpolicy.yaml
2:kind: NetworkPolicy
21:kind: NetworkPolicy

resources/helm/v2.1/istio-telemetry/prometheus/templates/networkpolicy.yaml
3:kind: NetworkPolicy


vlan filtering, validator: a trunk port must have > 1 vlan tag

Using the currently defined schema, a trunk port must feature
at least 2 trunk tags, otherwise, it is impossible to identify the
correct port type when reading from NetworkManager via libnm.
This patch updates the validator accordingly.
Signed-off-by: Miguel Duarte Barroso<EMAIL_ADDRESS>

Coverage decreased (-2.4%) to 75.704% when pulling 205b15b70d6dc1aa1e259beb6923d66df77e6d3c on maiqueb:vlan_filtering_update_validator into f70fe88bc00d386be117b73dcd7b5f879112a0f3 on nmstate:master.

CI fails, please take a look

Using the currently defined schema, a trunk port must feature
at least 2 trunk tags, otherwise, it is impossible to identify the
correct port type when reading from NetworkManager via libnm.

Sorry for the late question, but why is it impossible? A linux-bridge is supposed to allow such configuration. Is it an NM problem? Can you explain this?

I expected this, the objective of this PR is mostly so we can discuss this.
When I read the port settings, I get an array of BridgeVlan objects - [0].
How can I know from those objects if the port is an access / trunk port ? My proposal is to do it based on the number or tags configured on a port: trunks have > 1 tag.
Let's hear you out; I'm open to any other alternatives.
[0] - https://valadoc.org/libnm/NM.BridgeVlan.html

Maybe I am missing something but here is my attempt to do it:

If it is pvid and untagged, it may be an access or a native of a port.
— if there is another tagged, it is a native, otherwise it is an access.

Limitations based on the above:

Do not support setting a trunk mode with no trunks at all.

Maybe I am missing something but here is my attempt to do it:

If it is pvid and untagged, it may be an access or a native of a port.
— if there is another tagged, it is a native, otherwise it is an access.

Limitations based on the above:

Do not support setting a trunk mode with no trunks at all.
Do not support pvid without untagged and vice versa.

Fair enough.
I'll update #576 accordingly once I can spare the time.


Add Send a Telegram Message GitHub Action

GitHub Link: https://github.com/appleboy/telegram-action
Marketplace: https://github.com/marketplace/actions/telegram-notify

@appleboy your changes have been merged, thanks for your contribution 👍


Support :SSL{Min,Max}Version config options

These are necessary to get the tests passing with LibreSSL 3.8.1+, which dropped support for TLSv1.0 and TLSv1.1 for security reasons.
This updates the tests to use TLSv1.2 on OpenBSD.  This is only strictly necessary on OpenBSD 7.4+, but it will work fine in previous versions as well.

@seki This issue this fixes is blocking the update of the Ruby CI OpenBSD machine to OpenBSD 7.4, so I plan to commit it in a couple of days unless there are objections.


[bug] [levels] doesn't handle multi levels scenes

Using foundry 11.
While streaming a scene with levels, the "stream view" only shows the top ans not each levels

Duplicate of #21
Unfortunately Levels is designed to only work with a single selected token.


XivAlexander not appearing in the minimized taskbar tray on windows 11

Hi!
I just changed to windows 11, and after reinstalling ff and xivalex, i noticed that xivalex does not appear on the minimized system tray, so i cannot access the logging/control window at all

I noticed this about a month ago, when my opcodes stopped working and I wanted to go edit them. It didn't use to be the case, where the minimise to tray setting still correctly stops showing the program in the taskbar, but now no longer has it show in the tray. For reference, I'm in a recent version of Windows 11 as well.
Being unfamiliar with the config files to be able to unset that setting manually, I did a full reinstallation. Then I did some more testing with fresh installs while messing with the settings and the config files.
For now, my workaround is to just not use the minimise to tray setting. So I manually minimise the small window whenever I start the game, and leave it be in the taskbar.

mine isn't showing up in task manager or minimize tray, after uninstalling, and reinstalling like 5 times.

@utopianrampart Make sure to delete the configuration files.  That worked for me.
But minimize to taskbar tray still doesn't work.


GPU-enabled trees don't improve accuracy

The CPU model performance:
[0]     train-logloss:0.686354  eval-logloss:0.68647
[10]    train-logloss:0.631807  eval-logloss:0.632971
[20]    train-logloss:0.594666  eval-logloss:0.596735
[30]    train-logloss:0.567844  eval-logloss:0.570785
[40]    train-logloss:0.547611  eval-logloss:0.551368
[50]    train-logloss:0.532089  eval-logloss:0.536725
[60]    train-logloss:0.520107  eval-logloss:0.52544
[70]    train-logloss:0.510285  eval-logloss:0.516222
[80]    train-logloss:0.502186  eval-logloss:0.508758
[90]    train-logloss:0.495478  eval-logloss:0.502596
[100]   train-logloss:0.489662  eval-logloss:0.497406

The same model and parameters set with GPU enabled:
[0]     train-logloss:0.686314  eval-logloss:0.686437
[10]    train-logloss:0.736006  eval-logloss:0.739857
[20]    train-logloss:0.754894  eval-logloss:0.763834
[30]    train-logloss:0.713751  eval-logloss:0.721646
[40]    train-logloss:0.798925  eval-logloss:0.814037
[50]    train-logloss:0.759044  eval-logloss:0.776334
[60]    train-logloss:0.740096  eval-logloss:0.758636
[70]    train-logloss:0.738714  eval-logloss:0.760426
[80]    train-logloss:0.734791  eval-logloss:0.757961
[90]    train-logloss:8.16676   eval-logloss:8.2664
[100]   train-logloss:11.2758   eval-logloss:11.3626

The model parameters:
params = {}
params['gpu_id'] = 0
params['updater'] = 'grow_gpu'
params['booster'] = 'gbtree'
params['objective'] = 'binary:logistic'
params['max_depth'] = 7
params['eta']  = 0.025
params['min_child_weight'] = 5
params['silent'] = 1
params['eval_metric'] = 'logloss'
params['subsample'] = 0.80

Environment info
Operating System: Ubuntu 16.04
Compiler: c++ (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609
Package used (python/R/jvm/C++): python3
xgboost version used: 0.6
Installed from source

The commit hash (git rev-parse HEAD)
98ea46153223ce3539da1e8e18d9858f3d88639c

Logs -- probably not relevant

NVIDIA Driver Version: 375.51, GPU: Tesla K80

Python package:

The python version and distribution
Python 3.5.2 (default, Nov 17 2016, 17:05:23) [GCC 5.4.0 20160609] on linux

The command to compile and install:

cmake .. -DPLUGIN_UPDATER_GPU=ON -DCUB_DIRECTORY=/home/ubuntu/cub
cd ../python-package; sudo python3 setup.py install

Steps to reproduce

Start training with logloss evaluation metric on CPU only
Start training same model with GPU enabled

params['gpu_id'] = 0
params['updater'] = 'grow_gpu'

Get decrease in the model accuracy instead of improving

Found solution:
Use integers, or increase values by multiplying by 1e6,
-- small value floats don't allow to make accurate splits due to low precision.

We have encountered the same issue. Converting to integers works for us.
@RAMitchell can you explain a bit more for this?


orchestrator/restart: Reset restart history when task spec changes

When a service reaches the limit on the number of restart attempts and then it is updated, currently the limit is still enforced. This isn't the right behavior, because restarts before the service was updated shouldn't count against the new version. There may have been a problem with the old service spec that is fixed with the new spec.
See https://github.com/moby/moby/issues/34007

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "reset-restart-attempts" git@github.com:aaronlehmann/swarmkit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842353965688
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

(Should this go into 17.06.1?)

I don't think this is a regression since 17.03, so including in 17.06.2 would be ok with me. It's a good one to have though, as it improves the (perceived) stability of swarm mode.
(open for input though if this should be prioritised, and is able to make it in time)


[feature]: support for other data types

checkbox type
date type
multi-select type
image type

At the moment, some of the HTML input field types (https://www.w3schools.com/tags/tag_input.asp) are supported out-of-the-box:
E.g.: date, color:
fields:
- name: color
  type: color
- name: reviewDate
  label: Review date
  type: date

Checkbox is handled a little diffrently so I need to implement it separately.
For image and multi-select - I'll think how to implement that in a nice way :)

I've just published version 1.0.3 with native checkbox support :)

How you imagine handling image type?

Do you mean how should we write the image type metadata in the file ? something like this maybe property: [[image_file]] ?
Oh, and also an option to specify image size in the query

Oh, I thought about some nice selector for choosing the image, displaying it is much simpler :)

Was thinking the dataview way, but yeah, your way is much better and simpler.

Ok, new release allows to show image based on frontmatter field, in future versions maybe I'll add some selector to choose image.


Removed Nimble Dependency

Removed Nimble dependency for the library

Added XCTestErrorReporter to use the native XCTest functionality for fail
Added XCTWaiter to use the native XCTest functionality for wait

Added testTarget with Nimble and Quick as dependencies
Removed OHHTTPStubs dependency for RubyPactMockServiceStub

Gotcha: this is a breaking change, so anyone that is using the MockService initializer without pactVerificationService and errorReporter will fail. They can easily fix this by adding the extensions that can be found in NimbleErrorReporter
TODO - Update documentation before merging if this PR is approved
resolves #57

@rubencagnie thanks heaps for this! I'm still in the midst of reviewing the changes. I'll try to finish it over the weekend.

@andrewspinks - I addressed your comments + replied to some of them

@andrewspinks - I took another look, and found that just using XCTFail should just do the trick. I tested it in the project itself and in a sample project I made. I just pushed my latest code (which is actually mostly deleting files...)

@rubencagnie Apologies, but the build failed with some linting issues:

pact-consumer-swift/Sources/Reporting/ErrorReporterXCTest.swift:5:1: warning: Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace
pact-consumer-swift/Sources/MockService.swift:37:1: warning: Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

@andrewspinks - sorry about that. Fixed it and pushed

@rubencagnie build failed again with Build input file cannot be found: 'pact-consumer-swift/Sources/Reporting/XCTestErrorReporter.swift' (in target 'PactConsumerSwift iOS' from project 'PactConsumerSwift')
On travis, the build is run using ./scripts/build.sh

@rubencagnie There were still some build failures for this PR due to a linking issue with XCTest framework on some versions of iOS.
Reason: no suitable image found.  Did find:
	/usr/lib/swift/libswiftXCTest.dylib: mach-o, but not built for iOS simulator

I took a look into it and it appears to be caused by a bug in XCode. Updating the XCode version in the .travis.yml file to xcode11.4 appears to fix the issue. There are also some minor merge conflicts with master.

@andrewspinks - I’m really sorry. Recently changed jobs and with the pandemic (and 2 small children) I just haven’t gotten around to it

@rubencagnie no worries. I have a 9 month old. Getting the time to even read emails can be a challenge!

Tell me about it.

Codecov Report

Merging #87 into master will increase coverage by 0.58%.
The diff coverage is 88.00%.

@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
+ Coverage   88.98%   89.56%   +0.58%
==========================================
  Files          11       11
  Lines         890      930      +40
==========================================
+ Hits          792      833      +41
+ Misses         98       97       -1


Convert Network Selection Dialog to Activity

#513

Apologies for the long delay in checking.
The new activity does look better and means there's no need for scrolling to see all the options (at least on high res phones). Need to double check it scrolls correctly on a lower resolution phone. Would be great to have the selections colour coded like metamask:

However the network selection on dapp browser should still be a dialog box, with a title 'Select Network'. The difference is that you're selecting a single network rather than choosing a filter. It can be an activity, but this doesn't make sense for the UX.

@James-Sangalli

Why is this approved when the ask is to make it into an activity and it ended up still a dialogue?

@justindg

If this is about the pop up in dapp browser, Please don't show the networks not enabled in Settings, instead, tell the users to enable more network in Settings.

@justindg
it should 𝑛𝑜𝑡 be possible for the user to choose a network not previously enabled in Settings when the user browses a dapp, no matter how much convenience it brings, even if the browser can correctly identify the user should be accessing a network not previously enabled in Settings.
If you prioritize consitency, you get convenice. If you priorize convenience, you get neither consistency nor convenience.

I would approve if this is done @JamesSmartCell

Only list the enabled networks in the dialogue.
Hint that more networks are available in settings (yes, I know a message is not the best design)
The word "Ethereum" is made into a button, with a browser "refresh" icon followed by the text "Ethereum".

Just to reduce confusion: the spec @justindg was following was to convert all dialog into activity, and the previous operation of the network picker in dapp browser didn't restrict based on network filters.
Ok I have a design solution for the button; pending a redesign which fulfils the requirement.
I'll update the PR.

@James-Sangalli

Why is this approved when the ask is to make it into an activity and it ended up still a dialogue?

@justindg

If this is about the pop up in dapp browser, Please don't show the networks not enabled in Settings, instead, tell the users to enable more network in Settings.

It is not a dialog, it is an activity.

It makes sense for the filters to be a separate activity but not for the network selector in dappbrowser.

It is not a dialog, it is an activity.

@James-Sangalli When you gave justing https://github.com/AlphaWallet/alpha-wallet-android/issues/513 do you mean:
𝑎) the network selector in Settings should be changed to an activity;
𝑏) the network selector in dapp should be changed to an activity;
I think Justin took your intention as 𝑎+𝑏 when you only intended 𝑎
I can't remember well but I must have lacked precision and was laconic too in my first start-up. It takes time to build that skill but I think the short cut is to speak verbosely at the outset and learn to make it terse later.

#513

The issue clearly says that it should be an activity, also, this issue was created long before network filters were made as it was done in Jan, before we had the multi chain UI.
The network selection page should still be an activity like this PR addressed.

With the UX overhaul it now makes sense for network select to be an activity. Will merge after testing.

The network selection page should still be an activity like this PR addressed.

Which is the selection page?
𝑎) the network filter in Settings
𝑏) the network selector in dapp

@JamesSmartCell Please remember to delete the branch and close the issue


PAN-2775 Update attr_encrypted to 3.1.0

Update our fork of attr_encrypted with the main attr_encrypted release 3.1.0 plus Noah's 2017 commit. This is necessary because 3.1.0 has fixes for Rails 5.x compatibility.

Does 3.1.0 still allow to do searching on the table (i.e. a per table salt instead of per row)?

I believe so.
On Fri, Jan 10, 2020, 9:12 AM Noah Kochanowicz<EMAIL_ADDRESS>wrote:

Does 3.1.0 still allow to do searching on the table (i.e. a per table salt
instead of per row)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rewardops/attr_encrypted/pull/1?email_source=notifications&email_token=AAENKFZVOF7XUNO6WCHYJK3Q5B64BA5CNFSM4KFAAHGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIUA6FQ#issuecomment-573050646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAENKFZ3ZRI75VIRDPOHBMDQ5B64BANCNFSM4KFAAHGA
.


fix: [Selected Fluent Tab from extra tabs popup is not displaying in Tab Headers] in [Blazor Server APP]

Hi Team,
we are using Fluent UI version 3.0 in Blazor Server Application.
We have used Fluent Tabs as mentioned in demo site https://www.fluentui-blazor.net/Tabs but Fluent Tab selected from extra tabs popup not displaying as active tab in Tab Headers.
Please find below screenshots from demo site.

Help on this will be highly appreciated!
Thanks,
Rajesh

🐛 Bug Report

💻 Repro or Code Sample

🤔 Expected Behavior

😯 Current Behavior

💁 Possible Solution

🔦 Context

🌍 Your Environment

OS & Device: [e.g. MacOS, iOS, Windows, Linux] on [iPhone 7, PC]
Browser [e.g. Microsoft Edge, Google Chrome, Apple Safari, Mozilla FireFox]
.NET and FAST Version [e.g. 1.8.0]

Hi, this is 'by design'.
The underlying web component does not have a way to scroll/move, so we can't get the actual selected tab into view.
I'll check if it is possible to mark the active tab on the overflow.


Mac M1 pro segmentation fault on import

🐛 Describe the bug
Hello! Can not import torchvision on my mac. Get an error:
libc++abi: terminating with uncaught exception of type std::length_error: vector
as well as segmentation fault.
code:
import torch
from torch import nn
from torch.nn import functional as F
from torch.distributions.normal import Normal
from torch.distributions.kl import kl_divergence
from torchvision.utils import save_image

it also fails just with from torchvision.utils import save_image
Similar issue: https://github.com/pyg-team/pytorch_geometric/issues/4419
But not a single proposed solution works...
Versions
PyTorch version: 1.10.2
Is debug build: False
CUDA used to build PyTorch: None
ROCM used to build PyTorch: N/A
OS: macOS 12.1 (arm64)
GCC version: Could not collect
Clang version: 13.1.6 (clang-13<IP_ADDRESS>.3)
CMake version: Could not collect
Libc version: N/A
Python version: 3.9.10 | packaged by conda-forge | (main, Feb  1 2022, 21:25:34)  [Clang 11.1.0 ] (64-bit runtime)
Python platform: macOS-12.1-arm64-arm-64bit
Is CUDA available: False
CUDA runtime version: No CUDA
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
Versions of relevant libraries:
[pip3] numpy==1.21.5
[pip3] torch==1.10.2
[pip3] torchaudio==0.11.0
[pip3] torchvision==0.12.0
[conda] numpy                     1.21.5           py39h47fb9ce_0    conda-forge
[conda] torch                     1.10.2                   pypi_0    pypi
[conda] torchaudio                0.11.0                   pypi_0    pypi
[conda] torchvision               0.12.0                   pypi_0    pypi

TorchVision doesn't properly provide support for this at the moment. We have a todo to investigate at #5171 but we don't currently have the bandwidth to resolve.
We'll definitely need to fix once when we investigate. Also by briefly checking the linked issue but I'm not sure what exactly it's invoking the libc++abi when you simply import from torchvision.utils import save_image. I wonder if that's linked to the @torch.no_grad() decorator. Could you test this out?

@datumbox could you provide me with a code? Idk what is decorator.
But I solved this problem by downgrading to pip install "torchvision==0.11.3"

@axmav I was wonder what would happen if you try to import a dummy method like this:
import torch

@torch.no_grad()
def dummy():
    return 0

@axmav I was wonder what would happen if you try to import a dummy method like this:
import torch

@torch.no_grad()
def dummy():
    return 0

yes, this works on 0.12.0

I also had this issue on m1. Fixed by using pip install "torchvision==0.11.3"

doesn't work, still get segmentation 11, on Mac M2


Willing to PR to refactor the library (before contributing new features)

Hi, before PR for features, I want to firstly PR for refactoring. This is because, from the software engineering perspective, clean code is more maintainable and easy to add new features.
If I do the PR, will you merge it (given good quality code etc)? :)
Sample areas that I may work on:

[ ] Adding unit and integration tests
[ ] Code cleanup

Code cleanup example: In the code below, there are 4 items returned, but unpacked to only 3 variables

Ah yes I'll definitely and very happily merge them!
And I agree that the first thing to do would be create good unit tests.


feat: #1081 grant application admin component test

tested the breadcrumb, page title, sub title, submit button is disabled initially
tested the validation error and message for user id field
tested breadcrumb item redirect on click
tested cancel button redirect on click

Hi Jota, looks really good, just small cleanup comment, thanks so much!


Update Module.java

Hello forecaster

hi

What is the point of this? They both have the same outcome lol

Cleaner code i will merge it when back to coding


josh-cooter sprint challenge--user-interface submission

@Chrisgins tagging here just in case it doesn't send notification like we talked about in Slack. Going to take a walk and then try some stretch work.


Added some context to error messages.

Added some context to error messages that bit me when trying this out.

Hey, thank you!
Thanks for make all this available, from my very limited testing, it looks like this could really help us!

Thanks for make all this available, from my very limited testing, it looks like this could really help us!

More than happy to help. Any chance you can give an overview of your use case? (asking mainly out of curiosity 😄 )
Also, I think your commits are missing the DCO signoff - we'll need those before merging.

Ok, I "signed off" the last commit, is that alright?
Investigating if we (https://avassa.io) could use this as our abstraction for HSM/TPM for the cases there are e.g. TPM modules available in the hardware.
Our general use case is to enable our customers to distribute applications (containers) to their edge locations, e.g. retail stores, buildings, factories etc.

Ok, commits are now signed off.

Excellent, thank you!

Investigating if we (https://avassa.io) could use this as our abstraction for HSM/TPM for the cases there are e.g. TPM modules available in the hardware.
Our general use case is to enable our customers to distribute applications (containers) to their edge locations, e.g. retail stores, buildings, factories etc.

Nice, that's pretty much the space Parsec was made for. If you'd like to give any feedback, get some help with anything Parsec-related, or just to be involved in general, we're quite active on our public Slack channel and community meeting.


How to run the script several times?

Hello! How can I run the script several times on a page?
Now it starts only in the first block
https://jsfiddle.net/drtvader/467e3rys/10/
PS: I do not really understand why the script itself did not start, at least I should know how to run several times on the page

var elements = $(element);

for(i=0; i < elements.length; i++){
  const ps = new PerfectScrollbar($(elements)[i]);
}


Use got instead of request [Breaking Change]

Breaking Change
Use got instead of request what has been deprecated

~> #33


[feature] deploy app

No unique id
No dependent service setting
Vxnet setting, need call DescribeVxnet api

solved #100


Redis visual client, develop based on winform, can easily view and retrieve data from the server

A Redis visual client, developed based on winform, can easily view and retrieve data from the server and facilitate debugging.

@R-Mahmoudi Can you send information about the Redis visual client?

@Ali-YousefiTelori  Redis is a high-performance key-value database. The emergence of redis largely compensates for the lack of key/value storage such as memcached , and can play a very good supplementary role in relational databases in some occasions. It provides Java, C/C++, C#, PHP, JavaScript, Perl, Object-C, Python, Ruby, Erlang and other clients, which are very convenient to use On the PLC side, accessors including Mitsubishi, Siemens, Omron, AB and Modbus Tcp clients support two modes, short connection mode and long connection mode. Now let’s explain the principle.
Short connection : Each read and write is a separate request, and it will be closed after the request is completed. If the port of the server only supports a single connection, the port can be reused by other connections after closing, but under frequent network requests, It is prone to exceptions, and other requests will be unsuccessful, especially in the case of multi-threading.
Long connection : create a common connection channel, all read and write requests are completed through this channel, in this way, the read and write performance is faster, even multi-thread calls will not be affected, and there is a synchronization mechanism inside. If the port of the server only supports a single connection, then this port is occupied, such as the port mechanism of Mitsubishi, and the Modbus tcp port mechanism of Siemens. The following code uses persistent connections by default, which has higher performance and supports multi-thread synchronization.
In the short connection mode, each request is a separate access, so there is no trouble of reconnection. In the long connection mode, if this request fails, the next request will automatically reconnect to the server until until the request is successful. In addition, try to make all reads and writes judge the success of the result.

@R-Mahmoudi I understand, We have to move this issue to caching system repository. We are in serialization.
Thanks.


There is a concurrent issue in StoreStatsService

The issue tracker is ONLY used for bug report and feature request. Keep in mind, please check whether there is an existing same report before your raise a new one.
Alternately (especially if your communication is not a bug report), you can send mail to our mailing lists. We welcome any friendly suggestions, bug fixes, collaboration and other improvements.
Please ensure that your bug report is clear and that it is complete. Otherwise, we may be unable to understand it or to reproduce it, either of which would prevent us from fixing the bug. We strongly recommend the report(bug report or feature request) could include some hints as the following:
BUG REPORT

Please describe the issue you observed:

What did you do (The steps to reproduce)?
StoreStatsService#getSinglePutMessageTopicSizeTotal and StoreStatsService#getSinglePutMessageTopicTimesTotal are not thread safe  which could return a different instance under high concurrent scenario ,normally this  won't happen ,because the cost of creating a AtomicLong and put it into a map can be ignored ,but shit happens ,i think this should be fixed
What did you expect to see?
StoreStatsService#getSinglePutMessageTopicSizeTotal and StoreStatsService#getSinglePutMessageTopicTimesTotal will always return the instance
What did you see instead?

Please tell us about your environment:
Max OSX
Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):

    public AtomicLong getSinglePutMessageTopicTimesTotal(String topic) {
        AtomicLong rs = putMessageTopicTimesTotal.get(topic);
        if (null == rs) {
            rs = new AtomicLong(0);
            putMessageTopicTimesTotal.put(topic, rs);
        }
        return rs;
    }

Close as it's been fixed.


unsubscribe_from_all is giving error

When I try to use drip.unsubscribe_from_all(email) it gives me an error. The same occurs when I try "unsubscribe".
TypeError
memoryview: a bytes-like object is required, not 'NoneType'
c:\users\ekers\appdata\local\programs\python\python38\Lib\gzip.py in write, line 270

Updating this line: https://github.com/Ishamyyl/drip-python/blob/7b8f65dcfe5c9cb8757eb50bea1659804da67c1b/drip/api/subscribers.py#L128
To:
return self.session.post(f'subscribers/{email}/unsubscribe_all', json = {})
should fix the problem.


add file unique option and add more tests

Codecov Report
Base: 53.48% // Head: 64.24% // Increases project coverage by +10.76% :tada:

Coverage data is based on head (61d23c7) compared to base (549d545).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main       #7       +/-   ##
===========================================
+ Coverage   53.48%   64.24%   +10.76%
===========================================
  Files           9        9
  Lines         531      551       +20
===========================================
+ Hits          284      354       +70
+ Misses        219      162       -57
- Partials       28       35        +7

Impacted Files
Coverage Δ

dump/job.go
64.57% <100.00%> (+10.30%)
:arrow_up:

storage/local/local.go
47.05% <100.00%> (ø)

storage/s3/s3.go
92.85% <100.00%> (+35.71%)
:arrow_up:

storage/storage.go
85.71% <100.00%> (+8.29%)
:arrow_up:

driver/mysql.go
63.85% <0.00%> (+3.61%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Recognize the TLS Feature certificate extension

https://crt.sh/?id=11323316&opt=cablint
"ERROR: Unknown Extension: <IP_ADDRESS>.<IP_ADDRESS>.24"
This should not be an error.  See RFC7633.

Fixed in e4204c2b11c5cb5e77791a062d24e0d2776d7fff


refactor: rename product to project

Rename product to project, as that is less confusing for a new user.

Tested local, with my local db. Works fine.


Add convenience functions for planets + moons. Add build_time function.

Added some convenience functions:

name_by_planet_id
id_by_planet_coords
id_by_moon_name
id_by_moon_coords

Added calculation of build time in seconds based on resources.

I added a new commit to this PR and merged the current dev aswell.
Should prevent merge conflicts.

Now we need to update the readme aswell

closed due to inactivety


is there any c# or c++ SDK to connect the camera and get images frame by frame?

is there any c# or c++ SDK to connect the camera and get images frame by frame? thank you

@anguoyang It's not there. But new projects can be found here: https://azure.github.io/Vision-AI-DevKit-Pages/docs/projects/

@anguoyang there is an API for capturing image https://azure.github.io/Vision-AI-DevKit-Pages/docs/train/
but it won't get images frame by frame. only when you send the API will it capture one frame

@anguoyang I have same interest. I believe there is a project to tag images under the R&D samples.


[Bug] Session fingerprinting erroneously failing the check

Describe the bug
I log in to the admin area, change a setting and then I get the error message "Authentication failed (206)" - then I log in again, apply the setting and less than 30 seconds later I have to log in again.
How to reproduce

Login to Admin area
System
Settings
Email
Edit the template

Expected behavior
Attitude transfer
Screenshots
https://ibb.co/9GB06tk
FOSSBilling version
FOSSBilling 0.6.8

Hello,
It would be helpful to have more information such as your browser, if you're using any extensions that may modify your user agent or client info, if your IP address is frequently changing, ect.
Additionally have you tried to see if the same behavior happens on a different browser / connection?
What about a private window?

Hello,
It would be helpful to have more information such as your browser, if you're using any extensions that may modify your user agent or client info, if your IP address is frequently changing, ect.
Additionally have you tried to see if the same behavior happens on a different browser / connection? What about a private window?

Hello,
I use Firefox with the following extensions Shoop, BetterTTV and uBlock.
I've already tried it with Edge and Chrome, but always the same thing.
I don't change my IP address and I don't work with a VPN.

Okay, so it's very unlikely to be a problem with your browser and is most likely being caused by a security system in FOSSBilling.
The new 0.6.9 release has some improvements that can help us troubleshoot this, can you please update to that, reproduce the issue, and then check /data/php_error.log to see if there's anything similar to this message:

Session ID someSessionID has potentially been hijacked as it failed the fingerprint check. The session has automatically been destroyed.

If it does, then you can edit your config.php file and change 'perform_session_fingerprinting' => true, to 'perform_session_fingerprinting' => false, as a workaround.

I had to update the version manually, it currently works after I deactivated the fingerprint setting.
There is currently no php_error.log.

I also ran into the same issue, but when testing FOSSBilling on Caddy server. All works without changes on Nginx.
I made it work by changing 'CSRFPrevention' to false in the config.php - but have no idea what is wrong with Caddy.

it currently works after I deactivated the fingerprint setting.

Okay, thank you.
Feel free to leave it like that for the moment, I'll work to add implement a method for us to properly gather more info on what's causing the fingerprint to fail and I'll give you instructions at that point, but the existing method of session fingerprinting isn't super advanced nor robust so having it disabled isn't a significant reduction in security.

I also ran into the same issue, but when testing FOSSBilling on Caddy server. All works without changes on Nginx.
I made it work by changing 'CSRFPrevention' to false in the config.php - but have no idea what is wrong with Caddy.

The "Authentication failed (206)" error is fairly generic. In the case of this bug report they were seeing it because their session failed the fingerprint check, causing it to be destroyed under the assumption of a session hijacking.
If disabling the CSRF prevention option fixed it for you, then that's another issue, but I can't think of any way Caddy would cause these issues as it's just an additional token that's added to API requests.

Hi @JstnKlbt,
The newly released 0.6.10 release has added debugging for this issue which can be enabled via the config file like so:
Then it'll print info to the error log when a fingerprint is rejected which will help us identify where the issue is occurring.
Keep in mind, session fingerprinting must be enabled once again before this is functionality is going to happen.
    'security' => [
        'mode' => 'strict',
        'force_https' => true,
        'session_lifespan' => 7200,
        'perform_session_fingerprinting' => true,
-       'debug_fingerprint' => false,
+       'debug_fingerprint' => true,
    ],


PHP Doctrine with SQL Server Always On

Hello,
I am developing an application using Symfony 3 (php 7.1) with SQL Server database with AlwaysOn. AlwaysOn is a SQL server support for high-availability and disaster recovery, and Doctrine 2 is not working well with this paradigm.
I've developed a CRUD to test it for the infrastructure section at my job that implemented this database. At the first time, it was working correctly and persisting ok. Although, when they change the database for other one, like when happens if the first one stops working, automatically go to the other database with the same structure. When it happens I receive an error message from doctrine:

Fatal error: Invalid handle returned. in C:\xampp\htdocs\testesql\vendor\doctrine\dbal\lib\Doctrine\DBAL\Driver\PDOConnection.php on line 43

It looks like Doctrine caches something that is not working with this paradigm of database. I am using Microsoft SQL Server driver (pdo_sqlsrv). Could someone help me? I would be glad.

@fabionvs any way to reproduce this? What are you passing to the connection?

when they change the database for other one, like when happens if the first one stops working, automatically go to the other database with the same structure. When it happens I receive an error message from doctrine:

I can't understand the wording in this bit: can you rephrase it?

Hello @Ocramius, thank you for your help!
The SQL Server database was implemented in a cluster. And, when the main database stops working, or if they change the instance, it goes to another backup instance while using the same connection. And, for testing propourses they changed the instance to see if the application still works, but is not working after they changed the instance.

That's most likely something that needs fixing within pdo_sqlsrv, not in doctrine, sorry :-\
Closing as can't fix here.

Note: removed the Bug label because this is a pdo_sqlsrv issue that isn't a bug in doctrine itself.


Change type login and register

Explain your changes
Suppose there is a related issue with enough detail for a reviewer to understand your changes fully. In that case, you can omit an explanation and instead include either “Fixes #XX” or “Updates #XX” where “XX” is the issue number.
Checklist

[x] I have read the “Pull requests” section in the contributing guidelines.
[ ] I agree to the terms within the code of conduct.

🛟 If you need help, consider asking for advice over in the Kinde community.

@trunges21, Please see my comments on the :point_right: corresponding pull request the same comments apply here. Thanks :+1: .


Xampp can not start after include extension=mongodb.so in php.ini

Hi everyone,
I installed this driver successfully and got the file mongodb.so but when I am trying to use it with Xampp - add the line extension=mongodb.so into php.ini file . But Xampp can not start, it can start fine without this line.
Can please someone look at this issue.
Many thanks.

I checked error log file but didnt see anything.

Closing this as a duplicate of https://github.com/mongodb/mongo-php-driver/issues/247.


Crashes Hyprland

If hyprland is already running and the plugin is loaded everything is fine but the plugin prevents hyprland from starting up.
I noticed that in the hyprland crash report in the Plugins: section unlike the hyprgrass plugin that shows up as
hyprgrass (horriblename) v0.8.2
hypr-dynamic-cursor shows up as just
()
but It's not worth creating a separate issue for.
I tested hypr-dynamic-cursors on it's own, so it's not a conflict between plugins.
Hardware:
Nvidia optimus laptop (intel + nvidia gpu)
--------------------------------------------
   Hyprland Crash Report
--------------------------------------------
Oops

Hyprland received signal 11(SEGV)
Version: a425fbebe4cf4238e48a42f724ef2208959d66cf
Tag: v0.45.0
Date: 2024-11-09
Flags:

Hyprland seems to be running with plugins. This crash might not be Hyprland's fault.
Plugins:
	 ()

OS details:
Kernel: 6.11.7
os-release:
	ANSI_COLOR="1;34"
	BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"
	BUILD_ID="24.11.20241111.dc460ec"
	CPE_NAME="cpe:/o:nixos:nixos:24.11"
	DEFAULT_HOSTNAME=paragon
	DOCUMENTATION_URL="https://nixos.org/learn.html"
	HOME_URL="https://nixos.org/"
	ID=nixos
	ID_LIKE=""
	IMAGE_ID=""
	IMAGE_VERSION=""
	LOGO="nix-snowflake"
	NAME=NixOS
	PRETTY_NAME="NixOS 24.11 (Vicuna)"
	SUPPORT_URL="https://nixos.org/community.html"
	VARIANT=""
	VARIANT_ID=""
	VENDOR_NAME=NixOS
	VENDOR_URL="https://nixos.org/"
	VERSION="24.11 (Vicuna)"
	VERSION_CODENAME=vicuna
	VERSION_ID="24.11"

Backtrace:
Backtrace:
	# | /etc/profiles/per-user/nix/bin/Hyprland(_Z12getBacktracev+0x46) [0x6a5306]
		getBacktrace()
		??:?
	#1 | /etc/profiles/per-user/nix/bin/Hyprland(_ZN13CrashReporter18createAndSaveCrashEi+0xb84) [0x5da4d4]
		CrashReporter::createAndSaveCrash(int)
		??:?
	#2 | /etc/profiles/per-user/nix/bin/Hyprland(_Z25handleUnrecoverableSignali+0x5f) [0x51dfcf]
		handleUnrecoverableSignal(int)
		??:?
	#3 | /nix/store/3bvxjkkmwlymr0fssczhgi39c3aj1l7i-glibc-2.40-36/lib/libc.so.6(+0x40620) [0x7faad2047620]
		??
		??:0
	#4 | /nix/store/78wd06hvg7siqarf28pa6n9vz88798kf-hypr-dynamic-cursors-0-unstable-2024-11-10/lib/libhypr-dynamic-cursors.so(_ZN15CHighresHandler6updateEv+0x107) [0x7faab4094367]
		??
		??:0
	#5 | /nix/store/78wd06hvg7siqarf28pa6n9vz88798kf-hypr-dynamic-cursors-0-unstable-2024-11-10/lib/libhypr-dynamic-cursors.so(_ZN15CHighresHandlerC1Ev+0x6d) [0x7faab4094a4d]
		??
		??:0
	#6 | /nix/store/78wd06hvg7siqarf28pa6n9vz88798kf-hypr-dynamic-cursors-0-unstable-2024-11-10/lib/libhypr-dynamic-cursors.so(_ZN15CDynamicCursorsC1Ev+0x39) [0x7faab4078ff9]
		??
		??:0
	#7 | /nix/store/78wd06hvg7siqarf28pa6n9vz88798kf-hypr-dynamic-cursors-0-unstable-2024-11-10/lib/libhypr-dynamic-cursors.so(pluginInit+0x723) [0x7faab4099aa3]
		??
		??:0
	#8 | /etc/profiles/per-user/nix/bin/Hyprland(_ZN13CPluginSystem10loadPluginERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE+0x8d4) [0x7e1d44]
		CPluginSystem::loadPlugin(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)
		??:?
	#9 | /etc/profiles/per-user/nix/bin/Hyprland(_ZN13CPluginSystem19updateConfigPluginsERKSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaIS6_EERb+0x30f) [0x7e27bf]
		CPluginSystem::updateConfigPlugins(std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > const&, bool&)
		??:?
	#1 | /etc/profiles/per-user/nix/bin/Hyprland(_ZN14CConfigManager17handlePluginLoadsEv+0x60) [0x5a81d0]
		CConfigManager::handlePluginLoads()
		??:?
	#11 | /etc/profiles/per-user/nix/bin/Hyprland(_ZN11CCompositor12initManagersE18eManagersInitStage+0xabb) [0x531cfb]
		CCompositor::initManagers(eManagersInitStage)
		??:?
	#12 | /etc/profiles/per-user/nix/bin/Hyprland(_ZN11CCompositor10initServerENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEEi+0x868) [0x53aa48]
		CCompositor::initServer(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, int)
		??:?
	#13 | /etc/profiles/per-user/nix/bin/Hyprland(main+0xc4f) [0x6fc2df]
		main
		??:?
	#14 | /nix/store/3bvxjkkmwlymr0fssczhgi39c3aj1l7i-glibc-2.40-36/lib/libc.so.6(+0x2a27e) [0x7faad203127e]
		??
		??:0
	#15 | /nix/store/3bvxjkkmwlymr0fssczhgi39c3aj1l7i-glibc-2.40-36/lib/libc.so.6(__libc_start_main+0x89) [0x7faad2031339]
		??
		??:0
	#16 | /etc/profiles/per-user/nix/bin/Hyprland(_start+0x25) [0x4df2a5]
		_start
		??:?

Log tail:
Log tail:
[LOG] EGL: | with modifier INVALID (0xffffffffffffff)
[LOG] EGL: GPU Supports Format Y216 (0x36313259)
[LOG] EGL: | with modifier INVALID (0xffffffffffffff)
[LOG] EGL: 47 formats found in total. Some modifiers may be omitted as they are external-only.
[LOG] Creating the ProtocolManager!

[LOG] Creating the ThreadManager!
[LOG] Creating CHyprCtl
[LOG] Hypr socket started at /run/user/1000/hypr/a425fbebe4cf4238e48a42f724ef2208959d66cf_1731878276_1489924466/.socket.sock
[LOG] Creating the InputManager!
[LOG] Creating the HyprRenderer!
[LOG] DRM driver information: i915 v1.6.0 from 0 description Intel Graphics
[LOG] DRM driver information: nvidia-drm v0.0.0 from 0 description NVIDIA DRM driver
[WARN] NVIDIA detected, please remember to follow nvidia instructions on the wiki
[LOG] Creating the XWaylandManager!
[LOG] Creating the SessionLockManager!
[LOG] Creating the HyprDebugOverlay!
[LOG] Creating the HyprNotificationOverlay!
[LOG] Creating the PluginSystem!
[LOG] Loading plugin /nix/store/78wd06hvg7siqarf28pa6n9vz88798kf-hypr-dynamic-cursors-0-unstable-2024-11-10/lib/libhypr-dynamic-cursors.so which is now present in config

damn, always forget to test loading the plugin via plugin = and not via hyprpm or similar. Should now be fixed with this commit, but would be great if you could confirm this.
The problem now is that the version on nixpkgs is not yet updated, so this will be a problem for everyone else using v0.45.0 and the plugin from nixpkgs. So I don't really know what to do there (I don't use nix).
If anyone cannot update to this commit, you may disable hyprcursor support (i.e. set plugin:dynamic-cursors:hyprcursor:enabled to false), which would avoid this crash, at the cost of not having high-res shake to find shapes.

@VirtCode thank you for your fast fix, for other people who want to get it immediately, use this:
      plugins = [
        (pkgs.hyprlandPlugins.hypr-dynamic-cursors.overrideAttrs (oldAttrs: {
          src = oldAttrs.src.override {
            rev = "81f4b964f997a3174596ef22c7a1dee8a5f616c7";
            hash = "sha256-3SDwq2i2QW9nu7HBCPuDtLmrwLt2kajzImBsawKRZ+s=";
          };
        }))
      ];


speedtest-cli shows too slow speeds

Testing with the Flash web site shows a download speed of more than 700 Mb/s. Running speedtest-cli against the same server never goes above 200.
Is there is a problem with how data is read from the server so that it is unable to read data at a fast rate?

Check the readme

....so, basically it's not useful.


ChunkMeta Code Review - World Object Reference Storage

Code Location:
https://github.com/CivMC/CivModCore/blob/cab0ef422d9dffc081b271cc35b7a1b6f3dab68a/paper/src/main/java/vg/civcraft/mc/civmodcore/world/locations/chunkmeta/ChunkCoord.java#L37

Issue:
This is a treemap, but a hashmap may be more efficient in the case of numeric keys.

I'm closing this out as something that we won't fix. It could be revisited in the future, but it's not within our capacity to work on right now.


fix: wrong word for docs/source.md

fix Word Error
What type of PR is this?

/kind bug

Any specific area of the project related to this PR?

/area documentation

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Needs the commit to be signed-off

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: 906844ff867255de1c350729f04c25f07be5b295

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

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


enforce camelCase outputs

depending on how you name folders on the makes/ folder, the outputs are going to look like-this, or likeThis, or whatever you name the directories.
for consistency with makes built-ins, we should translate directory names to camelCase: https://levelup.gitconnected.com/converting-a-string-to-camelcase-in-javascript-f88a646a22b4
so people can have directories like /makes/my-derivation/main.nix, but execute m .myDerivation
nix has no built-in, and nixpkgs does not have a function in lib.strings, so we should create our own in this function: https://github.com/fluidattacks/makes/blob/27b669a539c6be8caffbbfab23da07a8c87a1f13/src/modules/outputs/custom.nix#L16 (turn name into camelCase before appending to the list)

example of the casing incoherence:

@kamadorueda I am interested in working on this issue. Can I work on it?

@kamadorueda I am interested in working on this issue. Can I work on it?

@dsalaza4 is the current maintainer

Hi, @dsalaza4! Can I work on this issue?

@0xt3j4s Sure! If you have any questions do not hesitate to ask!

This is the relevant source code:
https://github.com/fluidattacks/makes/blob/main/src/evaluator/modules/default.nix#L91

Hey @dsalaza4, in the description of the issue, it is given that I need to change the name to camelCase in this file.
While you had given a different source code that is quite similar to the one shown in the issue, though in a another directory.
Can you confirm the name conversion to camelCase should be done in which directory or branch?

@0xt3j4s As this issue was created years ago, the source code has changed significantly. The file mentioned initially no longer exists.

Oh, Ok. I got it and will work accordingly.

Not relevant to us right now. We will most likely favor allowing users give their jobs whatever name they want


Use full versioned package.son as dependency

This fixes #18

Test are failling due to new global objects Set, Map and Intl availables in Node.js 0.11, this will need to be fixed soon so tests can pass on more recent versions of Node.js.

@piranna getting around to doing some work on this library for you 👍
mind rebasing with master? I pulled in @othiym23's work ... should have landed that a while ago, where has time gone.

mind rebasing with master?

Done ;-) I think there should be some other changes like usage of a more tradition (err, result) callback, let's see what we can do here :-)

Or emit error events instead of using a logger.

@piranna looks like a few standard style nits breaking the build, if you don't mind patching :)

@piranna looks like a few standard style nits breaking the build, if you don't mind patching :)

Done :-)

Any updates on this?

@piranna this is a fairly critical component in npm Enterprise, so I need to be mindful of the API change.
I sat down and started working on a merge that gives you the functionality you're looking for, while changing the underlying contract less -- with this first pass, for instance, the bin is no longer working.
I'll work on getting something landed soon, but I'd like to find a compromise that gets you the functionlaity you're looking for, while changing the underlying API less drastically.

I don't remember adding backwards incompatible changes, but if so, we can increase semver...

Tests are failing due to the next error, but I'm not able to identify it... :-/
test/remote-ls-test.js .............expected { Object (dependencies) } to deeply equal { Object<EMAIL_ADDRESS>}
test/remote-ls-test.js .............................. 13/14
  RemoteLS > ls > happy path works as expected
  not ok Error: Callback was already called.
    at:
      line: 839
      column: 36
      file: node_modules/async/dist/async.js
    stack: |
      node_modules/async/dist/async.js:839:36
      Request._callback (lib/remote-ls.js:9:2495)
      Request.self.callback (node_modules/request/request.js:200:22)
      Request.<anonymous> (node_modules/request/request.js:1067:10)
      IncomingMessage.<anonymous> (node_modules/request/request.js:988:12)
      endReadableNT (_stream_readable.js:913:12)
    test: happy path works as expected
    message: 'Error: Callback was already called.'
    source: |
      if (fn === null) throw new Error("Callback was already called.");

total ............................................... 13/14

  13 passing (3s)
  1 failing

---------------|----------|----------|----------|----------|----------------|
File           |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
---------------|----------|----------|----------|----------|----------------|
 lib/          |    93.65 |    80.65 |      100 |    98.25 |                |
  config.js    |    83.33 |       75 |      100 |    83.33 |              5 |
  remote-ls.js |    94.74 |    81.48 |      100 |      100 |                |
---------------|----------|----------|----------|----------|----------------|
All files      |    93.65 |    80.65 |      100 |    98.25 |                |
---------------|----------|----------|----------|----------|----------------|

Hey @piranna it looks like this is drifted away from master a bit; unfortunately we had to land a few releases for internal tools at npm. Do you still have a need for this feature?

Yes, I'm still interested on this changes. I have just rebased my changes on top of master.


on mobile, the measuring character (the h), shows up above the input dialogue

this has also been fixed

i hope


Need to add cssselect2 as a dependency

requirements.txt is now:
reportlab
lxml
tinycss2>=0.6.0
cssselect2>=0.2.0

and without it, it fails with:
E   ModuleNotFoundError: No module named 'cssselect2'

I'm working on adding cssselect2 to conda-forge, and then adding it as a dependency here


run examples on windows

Objective

Running examples on Linux in CI timeoutLinux is back!
But hey we can run examples on windows too!

Solution

Run examples on windows daily
I also added a 30 minutes timeout so that when it explodes, it doesn't explodes in 6 hours (the default timeout)
And simplified the linux examples by not requiring a custom feature set

Added the Critical tag so we can merge things reliably. This LGTM (even if it is a bit of a hack), although I'm also curious about the cache.

I changed the PR to run examples on windows on a daily cron. It's still interesting to have the result, but not worth it to run it on every commit (and that way it'll be easy to copy/paste if there's a failure again)

I wonder if there's other tests that could be moved to a daily only run

IMO we should try to run all of the examples daily on all platforms. I don't have a strong preference about whether that's done in this PR or a follow-up.

I suspect the examples should be run on the bors try/staging commits too, even if not run on every commit within a PR?
For example, being able to minimise windows on windows is both quite an important property to preserve, and apparently an easy property to lose.


Occlusion Culling?

Hello,I'm interested in using Godot to create a 3D game, but I've heard that there in't occlusion culling.
I saw a comment from late last year about it being added in the next release, but when asked, they wouldn't answer on whether that meant 3.2, or 4.0, etc...
Sorry if this has been asked before, I couldn't find anything that seemed like this while searching through closed/opened issues.

https://github.com/godotengine/godot/issues/22048#issuecomment-421475081
it's on the roadmap. and will be added when able.
https://github.com/godotengine/godot-roadmap/blob/master/ROADMAP.md

Also note, these sorts of questions shouldn't be asked here. Please look into the various community channels for places to ask questions like this. Issues are for posting about bugs/suggestions.

Duplicate of #22048.
I've also added "Occlusion culling" to the name of #22048 to avoid further duplicated issues


Update QrScanner.tsx

Close #1240

Summary by CodeRabbit

New Features

Enhanced QR scanner to support NOVA WALLET QR codes, automatically formatting and setting addresses when detected.

cant import from nova, still get "Invalid QR code."

@Nick-1979 how do you try to add watch only account from account?

I just tried to attach a QR-attached account from nova

@AMIRKHANEF  have u tried to add an account from Vault as watch only?

Solved.


Missing command for building the UI in the Readme

The top-level Readme has a missing command for building the UI
User must run npm i ( npm install) first.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I have added the domainPrefix as a stack prop as the TODO mentioned.

 I have added the domainPrefix as a stack prop as the TODO mentioned.

Thanks, I can still see the ""@aws-cdk/aws-apigatewayv2-alpha": "^2.40.0-alpha.0"," in package-lock.json though. Please remove this file from the PR as well.

Thanks for all your effort!


feat: Included covid-19 self assessment app

How you feel - covid-19 self assessment app
Because of the fact that the Covid-19 is more and more present, and it’s very similar to other infections such as flu or cold, we’d like to provide people with a web-based self-assessment tool. How You Feel app allows you to perform self-checkup to determine the possibility of SARS-Cov-19 infection, as well as a cross-reference for alternative reasons for the combination of symptoms. It gives the user a number of recommendations one should take into consideration in the nearest future.
Couple things:

The app will not track - ever - do not ask for adding that feature.
The app will not send the symptom data anywhere

:/ don't see anything.
As mentioned in the contributing guide, please have an issue filed first, have it accepted, then assigned to you before submitting a PR :)
This will help us discuss any changes/requirements before work is done on it!


chore: dotnet targets

[x] Add .editorconfig and improve .gitignore files to ensure more consistency
[x] Move package definition and shared properties between the two projects in .props and .targets to ensure long-term consistency
[x] Switch from Net Standard 2.0 to net6.0 and net7.0 to be able to use null tracking (huge step forward for maintenance)
[x] Review the package structure to include the native dlls (Don't need anymore to manually copy them to the bin folder for consumer of the package) [Windows only]
[x] Update Readme.md related to installation instructions
[x] Update function description in the code to fix some mistakes
[x] Refactor exception messages
[x] Fix the name of the dll to import from libprqlc.dll to prqlc.dll and explicitly set ASCII encoding
[x] Update Readme.md related to how to use it (was already outdated)
[x] Make PrqlCompilerOptions immutable
[ ] Include native dlls for Linux and OSX
[ ] Handle selection of runtime native dlls in the DllImport (not sure, it's really needed)
[ ] Set github-actions

Checked with:
dotnet build prql-net.sln -p:version="0.10.1" -c Release /p:ContinuousIntegrationBuild=true --nologo
dotnet test PrqlCompiler.tests\PrqlCompiler.Tests.csproj -c release --runtime win-x64  --nologo
dotnet pack PrqlCompiler -p:version="0.10.1" -c Release --include-symbols --no-build --nologo

Integration of the package into an external project is successful: https://github.com/Seddryck/DubUrl/tree/feature/prql

@Seddryck this looks great! Thanks for making so much progress.
If we can add a GHA, we can definitely merge...

Where did you get the .editorconfig from? Is this your own personal preferences or some common convention?
Someone could argue that the .gitconfig is a bit excessive (since it has ignores for stuff that we don't use), but that is okay. I realize it is from some template somewhere.
Not sure there is much benefit to using Directory.Build.props and Directory.Build.targets here since we only have two projects and I've mostly seen these in huge monorepos. That's okay though, its probably just me who haven't seen the light yet. 😂
I intentionally targeted netstandard2.0 in order to get compatibility with both .NET and .NET Framework. Targeting net6.0 and net7.0 we lose compatibility with the old .NET Framework. I am not really opposed to that though, because I really like modern .NET with nullable reference types. NET 8.0 just got released though, so you should target that.
If we're dropping .NET Framework then maybe it would be time to start considering the LibraryImport attribute which was introduced in .NET 7 to replace the DllImport attribute
If we drop netstandard2.0 you should use file-scoped namespaces too.
Now that you load the DLL from ..\..\..\..\target\release\prqlc.dll I guess it can be tricky to use this in a real world project where you wouldn't have your project in that directory. The README says it will look in /bin/Debug/net7.0/. That seems contradictory.
I don't know why you explicitly set ANSI coding, maybe you can enlighten me?
Does this do anything for us?
We do have a GitHub Action at:
https://github.com/PRQL/prql/blob/main/.github/workflows/test-dotnet.yaml
We have some problems which I have not been able to figure out, if you're good at FFI in .NET maybe you could take a look?
When there is an error, we get wrong values in the Start and Stop properties in the Span, as well as the StartLine, StartCol, EndLine and EndCol properties in the SourceLocation.

(just to clarify, while I appreciate the comments & questions from @vanillajonathan , the main thing we'd want in order to merge is to integrate with our GHA, so future changes can't break the code. Other discussions are very welcome, but not a gate for merging. Thanks!)

The .editorconfig is a classical one is there anything that is shocking?
The .gitignore is also a classical one but it has been extended with some stuffs that could be weird for this project, I confess. In fact, I added these two files because I add some issues with the light .gitignore that was on the repo. But feel free to suggest other versions. I'm not married with them.
Regarding the drop of .netstandard, it's related to nullable types. Not sure that many people have a project targeting .NET Framework and depending on a package still not published on Nuget 🧐 It's a balance between maintainability and legacy backward compatibility.
Yep, I'll add .NET 8.0 on my next edit. TBH was waiting official release to prevent use of RC SDK.

Regarding DllImport or LibraryImport, I was planning to switch when also targeting Linux and MacOS. About ANSI, just try with Unicode to see what's happening, tests were failing in my side. I didn't investigate further but at least now, it's visible and people who don't like it can take a look by themselves.

I'm not following you regarding the path from which the prql library is loaded.
I think I'm just using the relative path to include the dll in the package. The runtime import is not related to this path. As mentioned above, I used this package in another project without usage. So except if I'm really lucky and I've the same structure, I doubt that I really have a relative path dependency when I load the library at runtime. Could you be more explicit where I made a mistake?

I'm not familiar with GitHub actions and workflows and took a bit of time to check GitHub docs around that. Should be ok soon if it's not too complex to deploy them.

Thx for the heads up on the hidden issues with FFI. I'll add test to check this after the work on the GHA.
I'm clearly not a specialist around FFI but who is?

(FYI we changed some names to fix a name conflict on the rust side — so to avoid you any issues I merged main into your branch + resolved conflicts. Only a few lines. Hope that's OK)

@Seddryck

The .editorconfig is a classical one is there anything that is shocking?

I haven't looked much into it so I don't know if there is anything shocking in there. I was wondering if it was based upon your or someones personal preferences, because I think we should follow upstreams .NET conventions.

About ANSI, you could just try with Unicode to see what's happening, the tests were failing on my side. I didn't investigate further but at least now, it's visible and people who don't like it can take a look by themselves.

I think CharSet.Ansi is not needed, because the documentation seems to indicate that it is the default anyway.

I'm not following you regarding the path from which the prql library is loaded.
I think I'm just using the relative path to include the dll in the package. The runtime import is not related to this path. As mentioned above, I used this package in another project without issues. So, except if I'm really lucky and I've the same structure in the other project, I have troubles to consider that I really have a relative path dependency when I load the library at runtime. Could you be more explicit where I made a mistake?

Maybe it is not a problem then, I just saw:
<None Include="..\..\..\..\target\release\libprqlc_lib.dll">

and was thinking maybe that could somehow be a problem, but now that you say it, yeah I think you're right, it's not where it's loaded from at runtime.
I just load it a runtime from the build directory because it was easy but I don't know if that is the way to go, because I don't know what the best practices are for when using FFI libraries. Maybe it is to embed it, I don't know.

Thx for the heads up on the hidden issues with FFI. I'll add test to check this after the work on the GHA.
I'm clearly not a specialist around FFI but who is?

Yeah, this FFI is really tricky.

@max-sixty , I'm checking that today and also the GHA.

I dropped a message on Discord (channel #bindings) regarding the upload to NuGet.org

I dropped a message on Discord (channel #bindings) regarding the upload to NuGet.org

Awesome, thanks for getting the GHA working!
I just set up a NuGet account and set up a Gravatar profile for<EMAIL_ADDRESS>Shall I add you to the NuGet account so you can test it?

@max-sixty There was a bit of discussion on Discord. I think we need to consider whether to merge this and continue with mono repo, or split the repository by language.

@max-sixty There was a bit of discussion on Discord. I think we need to consider whether to merge this and continue with mono repo, or split the repository by language.

I'll check out discord — sorry I've been off for a while.
I wrote up a quick schematic for whether when I think we should separate into a new repo, interested in feedback

Factor
Rationale
Example

Does someone want to sign up to maintain a repo?
A different repo is harder for the core team to maintain
tree-sitter-prql is well maintained

Can it change independently from the language?
If it's in a different repo, it can't be changed in lockstep with the language
prql-vscode is fine to change "behind" the lang

Would a separate repo invite new contributors?
A monorepo with all the rust code can be less inviting for those familiar with other langs
prql-vscode had some JS-only contributors

Is there an convention for a stand-alone repo?
A small number of ecosystems require a separate repo
homebrew-prql needs to be named that way for a Homebrew tap

I believe the library binaries are now attached to the GitHub release. What do you want to do with this PR?
I think monorepo is too much of a maintenance burden and would prefer move into another repo.

shields.io have badges that can be placed in the README.md file.
https://shields.io/badges/nu-get-version
https://shields.io/badges/nu-get-downloads

I do believe that the easiest is to move forward with a specific repo dedicated to these bindings.

Yes, sorry if I was unclear — if @Seddryck is up for maintaining it, I'd very much welcome another repo containing the dotnet bindings. It would be a great contribution to the PRQL project. (And FWIW I don't think it would require much maintenance...)
What would be next to make progress?

I won't be able to work on this next 10 days but will take it back after.
I agree that I'm not expecting a lot of maintenance.

Can you add net8.0 too?

Yes surely, I'll work on this during the next days.

Hey @Seddryck, hows it going?

Hey @Seddryck!

Hey @Seddryck, hows it going?


Create Work Merging service

Part of https://github.com/wellcometrust/platform/issues/1010

Receives updates from Works table
Service which looks at the list of identifiers on a work, makes a decision about how to merge and does so.
Outputs merged records to topic.

Implement merging solutions for (in-order):

https://github.com/wellcometrust/platform/issues/1008
https://github.com/wellcometrust/platform/issues/1009
https://github.com/wellcometrust/platform/issues/1364
https://github.com/wellcometrust/platform/issues/1365

Closing as out of date.


get_modified_clean_environment: Make it possible to keep vars by prefix

get_modified_clean_environment: Make it possible to keep vars by prefix

do you confirm that the new function makes visible all the env variables matching with CUDA_PATH?
i mean, if on the system we have defined CUDA_PATH, CUDA_PATH_V11_2 and CUDA_PATH_V10_3 all of them are visible as before right?

Should be. This probably needs end to end tests

This broke my builds for some reason.


Switch to phpredis/phpredis

Seems like the package predis/predis is not maintained anymore. We can switch to the php extension phpredis/phpredis.

predis/predis is maintained again.


Latest version of VSCode Insiders fails to load and execute tests

Starting today I am unable to run tests in the web environment.
The latest VSCode Insiders version (34f8428e4b0e3bf822f4b9e8b4e3e0cab69ef2) does not load in the browser.
The console log contains a lot of "Duplicate definition of module" entries. Also the page tries to retrieve non-existent files from the server (/static/build/out/vs/platform/product/common/product.js, /static/build/out/vs/platform/window/common/window.js).
Log is attached

I also experience an issue. Last week i bootstrapped my first web extension like mentioned here:
https://code.visualstudio.com/api/get-started/your-first-extension
I chose "Typescript Web Extension" and was able to test this with the provided npm script "run-in-browser". When i tried the same today (freshly created from the wizard) i just get a blank screen. There seem to be some files missing:

With today's version of the insiders build, the errors shifted to the following
localhost-1720430718020.log

I am running into the same issues. Would be great if there is a solution.

VSCode issue.

Hi, I encountered the same problem recently, I browsed the issue here and used the stable version to temporarily solve the problem.
I noticed @slavek-kucera mentioned that this is an upstream issue from vscode insiders, however I didn't find the relevant issue tracker in the issue list there (probably due to too many issues for me to find it), can someone tell Where can I track the progress of this issue?

@Frederisk Make sure you are running 0.0.56
https://github.com/microsoft/vscode/issues/220171


[New API] : Pexels API

Brief Description 🤓
Access high-quality stock photos and videos with various search options.
How will you develop it?🤩
react,js
Deployment
None
Select the open source program you are contributing to
Other
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINES of this project.

Greetings @AmruthaPariprolu🎀 ! , We are excited to have you dive into another issue with us!🚀
Your involvement in our project has been invaluable, and we're confident that your skills and insights will help us conquer this challenge😇.  Use command /gssoc24 to self assign the issue.  This issue represents an exciting opportunity for us to improve and enhance our project, and we are thrilled to have you on board.
We request you to follow CONTRIBUTING GUIDELINES.
Looking for your PR soon!
In case of any issues, you can contact us on Discord.❣

/gssoc24

EVERYTHING IS UNDER DEVELOPMENT.


[material-ui][Menu] highlight from keyboard breaks on webkitgtk

Steps to reproduce
Steps:

Open this link (https://codesandbox.io/p/sandbox/quirky-bartik-8wk4ff?file=/src/index.tsx) in a webkitgtk browser (ex. Epiphany/Web on on GNU Linux,
2.Click on Age Select dropdown, press cursor up/down keys, highlight appears and selection moves with keys
3.Click on Test button (which does nothing)
Click on Age Select dropdown again, press cursor up/down keys, highlight no longer moves with keys, but the selection, still does, as evidenced when hitting Enter

Current behavior
When clicking the Test button and then the Age Select dropdown, the highlight does not move with cursor up/down keys
Expected behavior
The highlight should appear consistently in a Select dropdown, even if another control is clicked.
Context
Consistent highlight in Select dropdown (and menus) after another MUI control is clicked in a webkitgtk browser.  Note that this issue does not appear to be an issue in Chrome or Firefox.  The reason I care about webkitgtk is that it used by Tauri.
Your environment

  System:
    OS: Linux 6.8 Linux Mint 21.3 (Virginia)
  Binaries:
    Node: 20.18.0 - /usr/bin/node
    npm: 10.9.0 - /usr/bin/npm
    pnpm: 9.5.0 - /usr/bin/pnpm
  Browsers:
    Chrome: 131.0.6778.69
  npmPackages:
    @emotion/react: latest => 11.13.3
    @emotion/styled: latest => 11.13.0
    @mui/core-downloads-tracker:  6.1.7
    @mui/material: latest => 6.1.7
    @mui/private-theming:  6.1.7
    @mui/styled-engine:  6.1.7
    @mui/system:  6.1.7
    @mui/types:  7.2.19
    @mui/utils:  6.1.7
    @types/react: latest => 18.3.12
    react: latest => 18.3.1
    react-dom: latest => 18.3.1
    typescript: latest => 5.6.3

Issue manifests in Epiphany (Gnome Web)  at https://codesandbox.io/p/sandbox/quirky-bartik-8wk4ff?file=/package.json:3,1-10,27
Search keywords: webkitgtk highlight

Thanks for the report @apicize. I can confirm this also happens in Safari (also webkit). We already have an issue for tracking this: https://github.com/mui/material-ui/issues/42451. I'm closing this one as a duplicate and adding this case to the original issue.


Nuitka pre-built binaries have problems on Linux cross-distro

The pre-built binary for v0.0.2 in releases was built on Debian GNU/Linux 9. When attempting to run the same binary on Arch Linux, the following error occurs:
$ ./darkbox
./darkbox: error while loading shared libraries: libpython3.5m.so.1.0: cannot open shared object file: No such file or directory

This seems to be a naming issue, cross-distro with shared libraries, just need to investigate this further... Should be an easy fix
Note: this is a Linux specific issue

Not an optimal solution, but this can be solved by statically linking the libraries into the binary, removing the reliance for it to exist on the system


Add new site (ryanair.com) to sites.json

After easyjet hack more people will be looking to remove personal details from other airlines.
Hope this helps.

Just a minor issue to make it good.
Thanks for your contribution!


HashMap test

Not all TODOs are removed, to be continued

Huge, thx!
Please prepare yourself mentally that with #578 the n >= length() et al. checks will disappear because length() is now O(n). This is the right direction. I'm sure, other libs act the same to keep the performance of the core operations at the limit of the possible.


How To Use Lambdas To Manipulate Through A Collection Of Objects in Java.

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[ ] Have you used the correct folder and file structure? - see Contribution Guidelines

[ ] Is your article properly formatted in Markdown? - see Github Markdown guide

[ ] Have you used our preferred heading structure? - H3 (###) and up

[ ] Does your article follow any of the suggested structure formats? - see example formats

[ ] Is your article over 750 words? - Tool: Wordcounter.net

[ ] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[ ] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[ ] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[ ] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[ ] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[ ] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[ ] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[ ] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[ ] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[ ] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #issue-number

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.
This closes #3729

Hi, @kingsleynwafor54. Welcome to the EngEd programme. Thanks for submitting your first PR.
Please run your article through a grammar checker such as Grammarly before submitting to ensure your article doesn't contain spelling or grammatical errors. This was an area picked up on when you submitted your topic suggestion so you should heed our reviewer's advice as your article won't be eligible to be reviewed unless you fix these.

We are closing this for now to allow you the time to work on this. Please take advantage of our resources that can help improve your article.

@kingsleynwafor54, we are closing this to give you more time to fix the issues that I mentioned that still exist. Once complete, feel free to reopen.

Good day! @louisefindlay23  I have done the correction and if I may indulge you to view it when opened. Thanks a lot!

@louisefindlay23  I made the corrections for over 3weeks and you didn't even look at it to tell me whether I'm making a progress or not and now you say I should go and make corrections, where should I please?

👋🏼 good afternoon and welcome to the program - although our reviewers often help point out glaring issues - they DO NOT provide revision services.
We ask student contribute polished (ready to publish) articles.


Platformer example: the tophat gets stuck in the roof after jump

Problem occurs on Windows, but not on Linux.

A possible cause of this issue is a different behavior of th.delta:
On Linux:
2
2
4
2
2
1
2
2
2
4
1
2
1
1
1
1

On Windows:
1
0
0
0
0
0
8
0
0
0
5
0
4
2
0
0
0

It seems that the th.deltas are the same on average, but not instantly, due to the implementation of the system timer.
Having th.delta == 0 is not very good for many reasons (for example, one may want a numeric derivative like vel = (pos - prevPos) / th.delta).

My "quick and dirty" solution is twofold:

Add a delay in window.cycle(): for round(std.clock()*1000 - start) <= 0 {/*Delay*/}.
Use larger steps in player.move(): const steps = 10.

I could try detecting when player is stuck, but that is not the issue here.
I will try adding vsync to tophat and enable it by default.

After some experimenting, I got this working on Windows. Not sure it's the best solution.
player.um.txt
(+ a delay for non-zero th.delta)

@vtereshkov does this still happen to you with the frame limiter?

I didn't see zero th.delta anymore. However, the player may still get stuck in the blocks (though quite rarely).


Bidirectional beam search tensorflow implmentation

Problem Description :
I read the paper named "Bidirectional Beam Search: Forward-Backward Inference in Neural Sequence Models for Fill-in-the-Blank Image Captioning"  Paper-Link.
I checked the official Tensorflow website for a feature similar to this one but didn't find any.
So is it possible we add this feature to Tensorflow or are there already similar implemented ones?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?


User guide branch

Edited the introduction paragraph to be more focused on healthcare establishments.
Edited parameter list to have a restriction column instead of a description column.
Compressing information to make the user guide more compact (i.e. moving up the information about what INDEX is from two pointers to one sentence in the description of the command).
Cleaning up awkward phrasing in the user guide.
Edited the constraint message for ward numbers.

Codecov Report
Base: 62.69% // Head: 62.69% // No change to project coverage :thumbsup:

Coverage data is based on head (f4e8226) compared to base (288a4e1).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #183   +/-   ##
=========================================
  Coverage     62.69%   62.69%
  Complexity      666      666
=========================================
  Files           118      118
  Lines          2356     2356
  Branches        295      295
=========================================
  Hits           1477     1477
  Misses          769      769
  Partials        110      110

Impacted Files
Coverage Δ

...in/java/seedu/address/model/person/WardNumber.java
70.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Disconnects after reading Discord message

Trying to link a discord server with an IRC server via ZNC instance. Server it's running on is running Debian Jessie (8.5), and both the ZNC and Discord connections are functional. However, immediately upon receiving a Discord message it throws an error and ceases to function.
Error thrown:
BOTNAME@SERVERNAME:~/discord-irc$ npm start -- --config /home/BOTNAME/discord-irc/discord.json

><EMAIL_ADDRESS>start /home/BOTNAME/discord-irc
> node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"

26 Aug 22:29:29 - SEND: PRIVMSG #DISCORDCHANNELNAME :<04BOTNAME> Discord -> IRC test.
/home/BOTNAME/discord-irc/node_modules/irc/lib/irc.js:932
    if (!this.conn.requestedDisconnect) {
                  ^
TypeError: Cannot read property 'requestedDisconnect' of null
    at Client.send (/home/BOTNAME/discord-irc/node_modules/irc/lib/irc.js:932:19)
    at dequeue (/home/BOTNAME/discord-irc/node_modules/irc/lib/irc.js:962:22)
    at wrapper [as _onTimeout] (timers.js:274:14)
    at Timer.listOnTimeout (timers.js:119:15)

npm ERR! Linux 3.16.0-4-amd64
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "start" "--" "--config" "/home/BOTNAME/discord-irc/discord.json"
npm ERR! node v0.12.15
npm ERR! npm  v2.15.1
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"'.
npm ERR! This is most likely a problem with the discord-irc package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs discord-irc
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls discord-irc
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/BOTNAME/discord-irc/npm-debug.log

Debug log:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/node',
1 verbose cli   '/usr/bin/npm',
1 verbose cli   'start',
1 verbose cli   '--',
1 verbose cli   '--config',
1 verbose cli   '/home/BOTNAME/discord-irc/discord.json' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose node symlink /usr/bin/node
5 verbose run-script [ 'prestart', 'start', 'poststart' ]
6 info prestart<EMAIL_ADDRESS>7 info start<EMAIL_ADDRESS>8 verbose unsafe-perm in lifecycle true
9 info<EMAIL_ADDRESS>Failed to exec start script
10 verbose stack Error<EMAIL_ADDRESS>start: `node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"`
10 verbose stack Exit status 1
10 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/lib/utils/lifecycle.js:217:16)
10 verbose stack     at EventEmitter.emit (events.js:110:17)
10 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/lib/utils/spawn.js:24:14)
10 verbose stack     at ChildProcess.emit (events.js:110:17)
10 verbose stack     at maybeClose (child_process.js:1019:16)
10 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1091:5)
11 verbose pkgid<EMAIL_ADDRESS>12 verbose cwd /home/BOTNAME/discord-irc
13 error Linux 3.16.0-4-amd64
14 error argv "/usr/bin/node" "/usr/bin/npm" "start" "--" "--config" "/home/BOTNAME/discord-irc/discord.json"
15 error node v0.12.15
16 error npm  v2.15.1
17 error code ELIFECYCLE
18 error<EMAIL_ADDRESS>start: `node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"`
18 error Exit status 1
19 error Failed at the<EMAIL_ADDRESS>start script 'node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"'.
19 error This is most likely a problem with the discord-irc package,
19 error not with npm itself.
19 error Tell the author that this fails on your system:
19 error     node dist/index.js "--config" "/home/BOTNAME/discord-irc/discord.json"
19 error You can get information on how to open an issue for this project with:
19 error     npm bugs discord-irc
19 error Or if that isn't available, you can get their info via:
19 error
19 error     npm owner ls discord-irc
19 error There is likely additional logging output above.
20 verbose exit [ 1, true ]

JSON config:
{
        "nickname": "botname",
        "server": "ZNC.server.address",
        "discordToken": "TOKEN",
        "channelMapping": {
                "#discordchannel": "#ircchannel"
        },
        "ircOptions": {
                "userName": "zncuser/server",
                "realName": "something",
                "password": "zncpassword",
                "port": "0000", // custom port (ssl), this comment doesn't exist in the file
                "autoRejoin": "true",
                "autoConnect": "true",
                "secure": "true",
                "selfSigned": "true",
                "certExpired": "true",
                "sasl": "true",
                "floodProtection": "false",
                "floodProtectionDelay": "1000",
                "retryCount": "10",
                "retryDelay": "10000",
                "debug": "true",
                "showErrors": "true"
        }
}

Sorry for the late answer. Does this still persist? Seems like it's an issue with node-irc and its connection to your ZNC server.

@yenfleet are you still experiencing this issue?


Deprecate EmptyContextInterface, InputInterface::setAllowEmpty & allowEmpty

Deprecate magic logic for auto attach a NonEmpty validator with breakChainOnFailure = true
Instead append NonEmpty validator when desired.
$input = new Zend\InputFilter\Input();
$input->setContinueIfEmpty(true);
$input->setAllowEmpty(true);
$input->getValidatorChain()->attach(new Zend\Validator\NotEmpty(), /* break chain on failure */ true);

Close https://github.com/zendframework/zend-inputfilter/issues/12

Since ZF 2.4 there are lot of bugs and headaches about the Required VS AllowEmpty VS ContinueIfEmpty matrix
This simplify the problem by removing the last two and forcing the user to attach the NotEmpty validator when he/she wants that behavior.

Any delegation needs to happen in add new minor version; as such, it needs to be targeted at 2.6.0.

I propose to do this on next patch because I found this a thing to be done ASAP. Anyway the most important thing is if we all agree this should be removed

Any deprecations MUST happen at a new minor release. After we get the other issues ironed out, we can do a 2.6.0 release with these changes.

Rebased on top of develop

Anyway I think deprecations should be backport to the lowest version for early adoptions of workarounds.
As soon as the developer avoid deprecated features as soon he/she will upgrade to major versions with less effort.

I think we have two classes of deprecatations.

New features which make obsolete old one. This should be deprecated in a new minor because includes a new feature.
Removal of code without new feature. This could be deprecated at any moment because there is no new code related to it.

Ready for 2.4.8 LTS


Why package polygon_base don't have polygon_base.so?

Why package polygon_base don't have polygon_base.so? How I can add this plugin to the .sdf ?

Please open questions like this on https://answers.ros.org, which is our central Question and Answer site.  You'll get a better answer there, and it will be searchable for the future.
Make sure to include a lot of information on what platform you are using, which ROS distribution you are using, and the exact steps you took.


nccopy failes to convert a particular nc3 file to an nc4 file

Originally reported by Kent at the HDF group.  The e-support ID is 26542.  See the message for the file Kent provided, it's too large to attache to this issue.

Use the current nccopy(<IP_ADDRESS>) to covert the NetCDF-3 file to netCDF-4 file, then
$ nccopy -k4 -d1 outfile-n3-GLAH outfile-n4-GLAH
Have:
NetCDF: HDF error
Location: file nccopy.c; line 1444
The platform is CentOS 7.1

Findings by @dmh:
I have recreated the error using cygwin.
A couple of things I noticed so far:

It appears to actually create the output netcdf4 file.
A diff of the metadata appears to show that the netcdf3 metadata
and the netcdf4 metadata are consistent except that the netcdf4
metadata is missing all attributes named 'CLASS' or 'NAME'.
Weird.

=Dennis Heimbigner
Unidata

Follow-up from Kent:
I did a little more investigation of the original HDF5 that may be helpful.
In the original nc3 file, the attribute CLASS=IMAGE. This is a special convention for an HDF5 high-level image file.
I think nccopy fails when calling H5DSattach_scale. In the function H5DSattach_scale, it will return FAIL if this dataset is an image(CLASS=IMAGE) or table(CLASS = TABLE).
HDF5 IMAGE and TABLE cannot have dimension scales.
One solution for this problem is to check if the attribute CLASS is IMAGE or TABLE, don't attach the scale if finding the variable is actually HDF5 image or table.

In my investigation, I discovered that for some reason, the netcdf-c/libsrc4 code is not successfully extracting certain attributes from the hdf5 file. Now to find out why.

I created a program to do some testing. The program is at the end.
The interesting thing is that doing ncdump gives this:
netcdf tst_gh189 {
variables:
int BROWSE_Image_00 ;
data:
BROWSE_Image_00 = _ ;
}
Note that the attribute "CLASS" is not listed.
 So somewhere, the "CLASS" attribute is either suppressed by HDF5 library or by the netcdf library.
Test program:
-#include <nc_tests.h>
#include <stdio.h>
#include <stdlib.h>
#include <netcdf.h>
#define FILENAME "tst_gh189.nc"
int main(int argc, char *argv)
{
/ Minimal demonstration of github issue 189 */
int ncid,varid;
char data[5] = "abcd";
if (nc_create(FILENAME, NC_CLOBBER|NC_NETCDF4, &ncid)) ERR;
if (nc_def_var(ncid, "BROWSE_Image_00", NC_INT, 0, NULL, &varid)) ERR;
if (nc_put_att(ncid,varid,"CLASS",NC_CHAR,4,data)) ERR;
if (nc_close (ncid)) ERR;
SUMMARIZE_ERR;
FINAL_RESULTS;
}

Could the answer be that "CLASS" is a protected word for attributes in HDF5 and should not be allowed for a netCDF attribute name?

Dennis and I've had a couple more exchanges with Kent and the HDF group and this is indeed the case; I'm probably going to close this out since the behavior is both explained and not going to change.  I'll re-open the issue if further discussion dictates it.
I'm hesitant to tell people they can't use CLASS across the board, but documenting this somewhere that this is a reserved word for netcdf4 files wouldn't be a bad idea.  Ideally we'd have a way to return a reserved-keyword error, and I suppose we could add code to do that, but it seems low priority until we actually observe this "in the wild".

Note that we'd have to maintain the list of reserved keywords ourselves; from talking with Kent, HDF5 does not raise any sort of flag or error code beyond FAIL in this case, and so does not indicate the reserved-keyword problem.

Ward, My proposal on this is as follows:

Get a list of all the known HDF5 reserved attribute names.
Assuming there is a way to read them using some HDF5 procedure,
add special code to extract the attributes that way.
There is some performance cost for this because it would need to ask
for these special names on every variable and every group.
=Dennis

On 1/15/2016 10:34 AM, Ward Fisher wrote:

Dennis and I've had a couple more exchanges with Kent and the HDF
group and this is indeed the case; I'm probably going to close this
out since the behavior is both explained and not going to change. I'll
re-open the issue if further discussion dictates it.
I'm hesitant to tell people they can't use |CLASS| across the board,
but documenting this somewhere that this is a reserved word for
netcdf4 files wouldn't be a bad idea. Ideally we'd have a way to
return a reserved-keyword error, and I suppose we could add code to do
that, but it seems low priority until we actually observe this "in the
wild".
Note that we'd have to maintain the list of reserved keywords
ourselves; from talking with Kent, HDF5 does not raise any sort of
flag or error code beyond |FAIL| in this case, and so does not
indicate the reserved-keyword problem.

—
Reply to this email directly or view it on GitHub
https://github.com/Unidata/netcdf-c/issues/189#issuecomment-172026996.

Ok, So I found out what is going on. In the file libsrc4/nc4file.c,line1750, the following atttributes names are checked and ignored: REFERENCE_LIST, CLASS, DIMENSION_LIST, NAME, COORDINATES, NC_DIMID_ATT_NAME.
It turns out that this is documented in the file: ''docs/file_format_specifications.md''
The claim is that "...This attribute is created and maintained by the HDF5 dimension
scale API".
This information was in the old netcdf.texi, (''docs/old/netcdf.tex''),  but apparently got lost in the move to doxygen.
So, we need to restore this info to the current documentation,

Given that the file isn't created/copied, is there a problem with how the attribute names are being checked?  I'll copy over the old info into the new documentation on Tuesday; I've been working on tightening up the documentation anyways.

Closing this out now for housekeeping reasons, will open a new, related issue specific to the change we're making.  Also tagged with revisit to keep it from falling through the cracks.


Revert "u-boot-iot2050: Add patch to re-write vars of invalid board i…

…nfo"
This reverts commit 7d3eec8a9bf7c09e538bf43dcc0e3aacc1a8d012.

For the commit log: "It turned out to not solve the problem completely. Instead, invalid environments will now be cleaned so that reprogramming is properly triggered on the next boot."

ok.  I have added to the commit message