documentation fixes, perform all examples builds/runs in Dockerfile

the build is slow as I introduce a dependency on downloading BigBuckBunny - perhaps better to add a small video file to the repo that has audio (object tracking video already present does not, and this causes failures).

I get an error trying to build your dockerfile:
 ---> Running in 165462de4be8
Extracting Bazel installation...
ERROR: The 'build' command is only supported from within a workspace (below a directory having a WORKSPACE file).
See documentation at https://docs.bazel.build/versions/master/build-ref.html#workspace
The command '/bin/sh -c bazel build -c opt     --define MEDIAPIPE_DISABLE_GPU=1     --define no_aws_support=true     --linkopt=-s     mediapipe/examples/desktop/object_detection:object_detection_tensorflow' returned a non-zero code: 2```

Allen, most of the changes of this PR were implemented in https://github.com/google/mediapipe/commit/b133b0f2004c831c2be004291d3fcf7a378a48f4.
Building and running all examples might be too heavy for some people who only need a clean environment. We can put them as comments. What do you think?

Yes, good idea. Shall I revise the PR here or we do it internally?
On Thu, Jul 23, 2020, 01:26 Jiuqiang Tang<EMAIL_ADDRESS>wrote:

Allen, most of the changes of this PR were implemented in b133b0f
https://github.com/google/mediapipe/commit/b133b0f2004c831c2be004291d3fcf7a378a48f4
.
Building and running all examples might be too heavy for some people who
only need a clean environment. We can put them as comments. What do you
think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/mediapipe/pull/678#issuecomment-662583288, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADKOP47GL7IWVL36OCK533R44OLPANCNFSM4MYEKENA
.


Using the POC (amd64_19041)

Hi, I'm trying this on a VM running Win 10 19045 using an older version of bootmgfw from 2020 as recommended in #1.
(Almost) similar to that post, I had:

Mounted ESP
Replaced ESP:\EFI\Microsoft\Boot\bootmgfw.efi with said older version
Copied minram, maxram, system32 directories to ESP root
Copied bootmgr, BCD, efisys_noprompt.bin to ESP root
Copied bootmgr to ESP:\EFI\Microsoft\Boot\bootmgr.efi
bcdedit /import ESP:\BCD (I could not delete the boot directory BCD because it was "opened by another process")
Unmounted ESP, shutdown -r -t 0

However, I still got 0xC00004B4 from trying to launch \minram\bootmgr.efi. Do you have any ideas? I cannot seem to find any reference to this error code when decompiling bootmgfw, is this not thrown by it?

STATUS_FILE_NOT_SUPPORTED, somehow a newer bootmgfw is running.

Alright, I have changed the BCD settings to points to ESP:\EFI\Microsoft\Boot\bootmgfw.efi for good measure and I have gotten C0000010, which I understand is STATUS_INVALID_DEVICE_REQUEST from using the TH1 method on a VM https://github.com/Wack0/CVE-2022-21894/issues/1#issuecomment-1212353108_.
I'll try to figure things out, in the meantime do you happen to know any reference for these boot error codes? They seem to be distinct from NTSTATUS ones and Microsoft seems awfully quiet on these.

What do you mean? The Windows boot environment uses NTSTATUS codes, and converts to and from those when interoperating with EFI.

Right, they are. I was just baffled that I could not find them some on the usual reference https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-erref/596a1078-e883-4972-9bbc-49e60bebca55, and I also noticed some undocumented ones when disassembling bootmgfw

They should all be in ntstatus.h. i don't think that page ever included all ntstatus values in the first place.


Enforce Maven Dependency Check

Most of Opencast's modules use the maven-dependency-pligin to ensure dependencies are specified correctly:
https://github.com/opencast/opencast/blob/a67ece488617d23101e16357ab5da0c69105644b/modules/animate-impl/pom.xml#L88-L98
Some modules still have an exception though because no one found the time to fix them until now:
https://github.com/opencast/opencast/blob/a67ece488617d23101e16357ab5da0c69105644b/docs/checkstyle/maven-dependency-plugin.exceptions#L1-L17
You can check which dependencies are problematic by running mvn dependency:analyze in the module directory (you need one full Opencast build before you can do that). Note that you may need to specify exceptions since for things which are dynamically loaded at runtime (like the logger in the example above).

List of missing modules:

[ ] modules/admin-ui/pom.xml
[ ] modules/asset-manager-impl/pom.xml
[ ] modules/asset-manager-storage-aws/pom.xml
[ ] modules/common-jpa-impl/pom.xml
[ ] modules/common/pom.xml
[ ] modules/cover-image-impl/pom.xml
[ ] modules/db/pom.xml
[ ] modules/distribution-service-aws-s3/pom.xml
[ ] modules/external-api/pom.xml
[ ] modules/index-service/pom.xml
[ ] modules/oaipmh-persistence/pom.xml
[ ] modules/publication-service-youtube-v3/pom.xml
[ ] modules/scheduler-impl/pom.xml
[ ] modules/search/pom.xml
[ ] modules/search-service-impl/pom.xml
[ ] modules/security-shibboleth/pom.xml
[ ] modules/solr/pom.xml

For anyone working on this: Don't hesitate to submit pull requests fixing only some modules.

I will give this a shot. Do you prefer a pr for each module individually or rather many together?

Fewer PRs would be better IMO.

Yep, I agree. We don't mind if you split this – if you have one part running and have a problem, don't wait until everything is done – but if you work on it at one time anyway, putting multiple modules in one pull request is easier for everyone. Also, don't hesitate to ask if you have questions about dependencies.
Also, pull request #1886 already fixes the dependencies for search-service-impl. So, no need to work on that one.

Thanks! I've started working on publication-service-youtube-v3. It compiles (yay) but still need to test if it actually works... Draft pr to show what's going on in #1904

#1904 now has fixes for the following modules:

modules/admin-ui/pom.xml
modules/publication-service-youtube-v3/pom.xml
modules/search/pom.xml

Since my weekend is coming to an end I will leave it at this.


decomp: finish sunken-pipegame | sun-exit-chamber | beach-obs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 54.805%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
40807

Relevant Lines:
74458

💛  - Coveralls


Inconsistent labels in VEuPathDB ontology and ClinEpi web display terminologies

Annotation property, EUPATH_0001005 multifilter, is used to indicate the term type in the ClinEpi web display terminologies including multifilter now. So, we need to update the label and definition in the ontology.
label: term type
alternative label: termType
proposed definition: The annotation specifies how terms is used in generation search filters on the website. It has five options:

category: a term to organize the variables for exploration on the website
variable: a variable in the dataset
variable, derived: one term that is calculated, using the data in two or more other variables
multifilter: any value below it in the hierarchy will be displayed as a multifilter
value: the value of a multifilter

I'm making this change in the release candidate.


Create lyceefrancais.txt

Added french school in Vienna/Austria - Lycée Français de Vienne

@clews Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


Centcom & FTLDestination cleanup

In prep for #28075
About the PR

Names the centcomm map "Central Command"
Lets the TryAddFTLDestination() actually set all the fields of the component.
Does so with an overload so no breaky.
Hopefully I didn't forget anything.
Why / Balance

Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

Beat sloth to adding box4d

Code looks good, tested the PR to make sure there wasn't anything wonky with setting the CC FTL destination as enabled and it seemed to work fine.


Fix selector in heketi OCP template.

Fixes #197.

This change is

Thanks @jarrpa . So is the kube-template not broken? (Just asking because #191 changed both).

Doesn't seem like it. I don't see a selector field there, must be something exclusive to DeploymentConfigs. Also it worked when I tested it after my last PR.

https://github.com/gluster/gluster-kubernetes/pull/191/commits/50978ebe91e5126b2fd93021d20e26b589fc524a changed a selector in kube-templates as well, hence asking... But if you tested it, I'm fine. :-)

after more discussions and analysis ... LGTM


Use default kubernetes imagePullPolicy

For latest tag, it should use Always.
For tags other than latest, it will use IfNotPresent.
https://kubernetes.io/docs/concepts/containers/images/#imagepullpolicy-defaulting
Description
The summary of the proposed changes as long as the relevant motivation and context.
Issues
#597
Type of change
List the type of change like below. Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)

Dependencies
List the newly introduced 3rd party dependency if exists.
Tests
Describe the tests that you ran to verify your changes.

Target for 1.2-dev. We won't fix this for 1.1 release because IfNotPresent is proper policy for released version.

That is the default when policy is not specified, and tag is something else than "latest".

agreed with @mkbhanda , I think we can make it looks nicer by:
in values.yaml, leave the image.pullPolicy as a comment with a little bit explanation, which gives the user some hints that it's configurable, i.e.
image:
  repostitory: opea/abc
  tag: "latest"
  # Uncomment the following line to set image desired image pull policy if needed, as one of Always, IfNotPresent, Never
  # pullPolicy: ""

In the corresponding deployment.yaml, change the image policy settings as optional:
   ... ...
   {{- if .Values.image.pullPolicy }}
   imagePullPolicy: {{ .Values.image.pullPolicy }}
   {{- end }}

The helm lint fail is due to "helm dep build", refer to this:
https://github.com/helm/helm/issues/8036
Will need to add the repo first as @eero-t mentioned.

Checked the GMC CI chatqna.log, the output file is good with "[DONE]" ending. That's more like an issue from client side, or the "router" sevice of GMC.

GMC test:

Response check failed, please check the logs in artifacts
=> CI checks response before it has fully completed?

Gaudi device conflict, rerun should solve this.

ChatQnA, TGI & vLLM tests:

Shard 0 failed to start
RuntimeError: synStatus=8 [Device not found] Device acquire failed
=> CI Gaudi k8s device plugin error; pod resource request is satisfied, but device is missing?

Ok, the other CI issues are now fixed. The 2 remaining ones (audioqna + speecht5 on Gaudi) are both due to CI using stale opea/speecht5-gaudi:latest image with this issue:
 Traceback (most recent call last):
  File "/home/user/comps/tts/speecht5/dependency/speecht5_server.py", line 15, in <module>
    from comps import CustomLogger
  File "/home/user/comps/__init__.py", line 7, in <module>
    from comps.cores.proto.docarray import (
  File "/home/user/comps/cores/proto/docarray.py", line 10, in <module>
    from pydantic import Field, conint, conlist, field_validator
ImportError: cannot import name 'field_validator' from 'pydantic' (/usr/local/lib/python3.10/dist-packages/pydantic/__init__.cpython-310-x86_64-linux-gnu.so)

We'll let the PR merge in first even with issue opea-project/GenAIComps#948 to avoid huge CI tests cases involved in this PR


LCFS - Implement and Configure Webpack for Frontend ReactJS Project

Describe the task
This task involves implementing webpack in the frontend ReactJS project within the LCFS repository. If neccessary, eject the react-scripts build system in favour of webpack. Additionally, the webpack.config.js file needs to be configured to match the project’s specific build settings, ensuring optimal performance and build processes. Reference the webpack configuration currently being used in TFRS as a starting point, but ensure to use the latest version of webpack.
Purpose
The purpose of this task is to streamline and optimize the build process of the ReactJS frontend by implementing webpack. By configuring the webpack.config.js file according to the project’s requirements, the task aims to achieve faster build times, optimized asset handling, and an overall improved development and production workflow.
Acceptance Criteria

[ ] Implement webpack in the frontend ReactJS project using the latest version of webpack.
[ ] Configure the webpack.config.js file to align with the project's build settings.
[ ] Reference the webpack configuration currently used in TFRS as a guide, but ensure optimizations and updates are made to leverage the latest webpack features.
[ ] Ensure that the build process efficiently handles assets, code splitting, and optimization.
[ ] Test the build process to validate that the webpack configuration is effective and error-free.
[ ] Document the configuration and setup process within the project's documentation.

Additional context

During the configuration process, focus on achieving an optimized and efficient build process that caters to both development and production environments.
Stay abreast of best practices and webpack documentation to ensure the configuration adheres to modern standards.
Keep in mind that while the TFRS configuration serves as a reference, the goal is to utilize the most updated and optimized configuration possible with the latest version of webpack.

Hey team! Please add your planning poker estimate with Zenhub @AlexZorkin @jig-patel @prv-proton

Nothing for PO to test


hyprland.cong at line 1: Invalid config file

Hyprland Version
 System/Version info
os-release: NAME="Fedora Linux"
VERSION="39 (Workstation Edition)"
ID=fedora
VERSION_ID=39
VERSION_CODENAME=""
PLATFORM_ID="platform:f39"
PRETTY_NAME="Fedora Linux 39 (Workstation Edition)"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:39"
DEFAULT_HOSTNAME="fedora"
HOME_URL="https://fedoraproject.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora/f39/system-administrators-guide/"
SUPPORT_URL="https://ask.fedoraproject.org/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=39
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=39
SUPPORT_END=2024-11-12
VARIANT="Workstation Edition"
VARIANT_ID=workstation

Bug or Regression?
Bug
Description
I am having a config error after updating 0.36. Line 1 is looking fine to me but it looks like there's an error and I couldn't figure it out
`#AUTOGENERATED HYPR CONFIG.
#PLEASE USE THE CONFIG PROVIDED IN THE GIT REPO /examples/hypr.conf AND EDIT IT,
#OR EDIT THIS ONE ACCORDING TO THE WIKI INSTRUCTIONS.
monitor=,preferred,auto,auto
monitor=eDP-1,1920x1080@60,0x0,1
monitor=HDMI-A-1,1920x1080@60,1920x0,1`
here are the first few lines of my hyprland.conf file
How to reproduce
d
Crash reports, logs, images, videos

please attach your config as a file

please attach your config as a file

hyprland (copy).txt

Seems like your file encoded like UTF-8 with BOM, which is not working correctly, dunno. Save it like just UTF-8 or create new file and copy content from your file to it.
❱ cmp -b /home/ligthiago/Downloads/hyprland.conf /home/ligthiago/.config/hypr/hyprland.conf
/home/ligthiago/Downloads/hyprland.conf /home/ligthiago/.config/hypr/hyprland.conf differ: byte 1, line 1 is 357 M-o  43 #

Anyway, it's seems like hyprland not okay with byte order mark

as above, anything other than ascii / utf8 is not supported

Seems like your file encoded like UTF-8 with BOM, which is not working correctly, dunno. Save it like just UTF-8 or create new file and copy content from your file to it.
❱ cmp -b /home/ligthiago/Downloads/hyprland.conf /home/ligthiago/.config/hypr/hyprland.conf
/home/ligthiago/Downloads/hyprland.conf /home/ligthiago/.config/hypr/hyprland.conf differ: byte 1, line 1 is 357 M-o  43 #

Anyway, it's seems like hyprland not okay with byte order mark

thanks man I fixed it!!!! you were right my encoding was using utf-8 with rom


Service principal role assignment fails with "can't find" error

Describe the bug
When attempting to assign permissions to my AKS service principal, it fails with "Cannot find user or service principal in graph database for 'msi'. If the assignee is an appId, make sure the corresponding service principal is created with 'az ad sp create --id msi'" - even though I have confirmed that this is the correct service principal ID.
To Reproduce
Run the following:
CLIENT_ID=$(az aks show --resource-group aks-rg-name --name aks-name --query "servicePrincipalProfile.clientId" --output tsv)
echo $CLIENT_ID #to make sure the above command worked
az role assignment create --assignee $CLIENT_ID --role "Network Contributor" --scope /subscriptions/mysubID/resourceGroups/rg-name/providers/Microsoft.Network/virtualNetworks/myvnetname
Expected behavior
The role assignment to complete.
Environment summary
Running PowerShell on Windows with latest CLI version (2.11).
Additional context
I'm attempting to assign Network Contributor permissions to my Azure CNI configured AKS cluster so I can create an internal load balancer, as documented here https://docs.microsoft.com/en-us/azure/aks/internal-lb#use-private-networks

@tdevopsottawa
Why is this closed?  I encountered the issue, apparently the servicePrincipalProfile.clientId is not even registered in AAD App registrations!
using this cmd as instructed in lab...
aks create --resource-group akshandsonlab --name  --enable-addons monitoring --kubernetes-version $version --generate-ssh-keys --location

Experiencing the same. @tdevopsottawa why did you close the issue?

Experiencing the same. @tdevopsottawa why did you close the issue?

Same issue, why is this closed?

Same issue, why is this closed?

@tdevopsottawa

I just had this same error.  You could try reversing the order and create the ACR First, then attach it to the AKS on creation as shown here:
https://docs.microsoft.com/en-us/azure/aks/cluster-container-registry-integration
The set of the principle still errs out, but you should be able to then leverage the acr from your aks cluster.

I have this same error, attempting
az ad sp create-for-rbac -n "my-service-principle-name-svp" --role="Contributor" --scopes="$(az group show --name my-rg-northcentralus --query id -o tsv)"
az role assignment create --assignee "$(az ad sp list --display-name "my-service-principle-name-svp" --query '[0].appId')" --role "Contributor" --scope "$(az group show --name other-rg-northcentralus --query id -o tsv)"
Cannot find user or service principal in graph database for '"7b5fddf2-722c-4186-bd3c-110ae5a4569b"'.
If the assignee is an appId, make sure the corresponding service principal is created with 'az ad sp create --id "7b5fddf2-722c-4186-bd3c-110ae5a4569b"'.

I'm running into the same problem: I want to create a role assignment via a yaml pipeline.
When creating the role assignment, or querying for the id of the principal using az ad sp show --id thename  I get this error, although the pipeline has just created a resource (Azure Function) with the name 'thename'.

I had the same problem when using the normal login email address (like<EMAIL_ADDRESS>as the $CLIENT_ID or  for --assignee parameter. After using user principal name (like<EMAIL_ADDRESS>it was successful. You can find the user principal name in azure portal in the users list.

I am consistently experiencing the same issue when trying to assign a role directly after creating an App Registration and its Service Principal using the Azure CLI.
Is there possibly any way to wait until the Service Principal of the App Registration is available for role assignments or comparable actions?

@blgorman
Thanks I also had the same issue until I put all the role assignments after
subscriptionId=$(az account show --query id --output tsv)
echo subscriptionId: $subscriptionId
subscriptionName=$(az account show --query name --output tsv)
echo subscriptionName: $subscriptionName
tenantId=$(az account show --query tenantId --output tsv)
echo tenantId: $tenantId
DO ROLE ASSIGNMENT HERE NOT BEFORE THE ABOVE
az role assignment create --assignee $PRINCIPAL --role "AcrPull" --resource-group $ResourceGroupName
az role assignment create --assignee $PRINCIPAL --role "AcrPush" --resource-group $ResourceGroupName

I just had this same error. You could try reversing the order and create the ACR First, then attach it to the AKS on creation as shown here: https://docs.microsoft.com/en-us/azure/aks/cluster-container-registry-integration
The set of the principle still errs out, but you should be able to then leverage the acr from your aks cluster.


Input date test fix take 2

Attempts to fix https://github.com/aspnet/AspNetCore-Internal/issues/3131
We're still seeing flaky behavior for InputDateInteractsWithEditContext_NullableDateTimeOffset. It seems that the fix described in this conversation isn't sufficient. The screenshot captured from the new failure on release/3.1 shows that only some of the keystrokes got entered into the <input type=date>, or the element performed some normalisation behavior to coerce the entered value into something valid.
So, this PR uses the other proposed solution of applying the value through JS. I think this should behave reliably, since there's no sequence of keystrokes being simulated. We should only use this approach when strictly necessary though, since it's not as authentic a representation of actual user behavior. In this case I think it's reasonable since we're not trying to test the browser's <input type=date> normalisation behaviors.

The screenshot captured from the new failure on release/3.1 shows that only some of the keystrokes got entered into the <input type=date>, or the element performed some normalisation behavior to coerce the entered value into something valid.

I'm glad that the screenshot proved to be useful.


Default timeout / WaitTooLong

(some doc feedback)
I ran the example from the Python3 section. I did not set any timeout option, but yet I see those errors:
ERROR:root:on_close() called due to WaitTooLong:

And the message counter always returns 1.
Although the documentation says:

If the timeout option is set, it will raise a WaitTooLong exception after that many seconds.
If the timeout option is not set or is None (default), it will wait forever.

It's probably a doc/default value adjustement :)

The (async) Message Counter example on Python 3.4.3 contains these lines:
bot = telepot.async.DelegatorBot(TOKEN, [
    (per_chat_id(), create_open(MessageCounter, timeout=10)),
])

which explicitly set the timeout to 10 seconds. Once you send the first message (to initiate the counting), if you don't send another message in 10 seconds, it will raise a WaitTooLong exception and terminate the counting.
I re-test that example, and here is what I got:

Notice the time of messages. After the first three messages, I wait a bit longer to send another. The original counting was terminated, so it re-started from 1.
It is recommended that timeout is not None, because if the user leaves the conversation at any time, that thread or task will hang there forever.

Thanks for the feedback! I realize now that setting a timeout is the most appropriate!


Lifecycle management of scan result blobs - use filter if using rule to move to achive tier

Not sure if this is the appropriate location for this suggested documentation tweak but here goes:  If one is using rules to move scan result blobs from hot to cool to archive, filter the rule for moving to the archive tier to ensure the 'baseline_YYYY_MM_DD' blobs are excluded.  If the 'baseline_' blob is in the archive tier the VA scan for that database will fail (reference support case #2209070010003161).

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

ID: bb1c4c66-d581-f69a-91c4-3a68344ec5d0
Version Independent ID: 41e63908-f308-6b35-f6be-6dea45770889
Content: Store Vulnerability Assessment scan results in a storage account accessible behind firewalls and VNets - Azure SQL Database
Content Source: azure-sql/database/sql-database-vulnerability-assessment-storage.md
Service: sql-db-mi
Sub-service: security
GitHub Login: @DavidTrigano
Microsoft Alias: datrigan

@langstonj Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

Added a note about archive access tier not being supported. Thanks!

Hi @langstonj , thanks for contacting Microsoft! We've processed a change to modify the documentation. I'll go ahead and close out this Git Issue but please feel free to comment should you need anything further.
Thanks again and I hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#please-close
#reassign:CESANU


Bookstore: Styling

Hi there 👋,
Activities done:

[x] Added styling for the books page.
[x] Added styling for the categories page.
[x] Followed the given styling of the template.

@mukhammadrizooff Thank you so much sir!


license?

What's the license of this code? Is it BSD3 like most other Elm programs?
(Background: I'm doing logic programming, specifically I'm trying to extend this to a full minikanren implementation with added error handling. I want to use your code as a base so I don't have to reinvent all the wheels but I've got to assume it's not open source right now.)

The license is BSD3, as stated in the manifest, but it was remiss of me not to include the actual license file. (To think I even keep a script to automate licensing…)
Thanks for alerting me. A license file is included as of 4b847d5be27.

Thanks for clarifying! :D

Actually, can I clear something with you? I'm using your implementation as a base but changing it in some pretty radical ways. Would it satisfy you to have acknowledgement in my project's README? I'm not sure how I can include the license header anyway, since as a published library it'll be included in the doc comment. 🤔

Basically, I just want to make sure to give credit where credit is due, and for you not to feel slighted in any way. This is an awesome implementation and I'll be able to stand on the shoulders of giants using it 😀

Thanks for your consideration. Changes and overhauls are expected, since the kind of naïve unification performed by μKanren is rather ill-suited to hefty programs; it's more of an exercise in style.
The license header shouldn't be necessary, unless your project is haunted by legal banshees. As for attribution, there's no need to add anything explicit, particularly if elm-microkanren is a hard dependency; otherwise, you can always add a reference somewhere relevant in the source, at your leisure. (Although, to be frank, this library is a very nearly mechanical translation of the original Scheme implementation, and I feel that special mention is hardly due.)


Update to three.js r71

The changes to src/ files are pretty simple. I've refactored the build scripts a bit and also introduced a bower.json file to help maintain the three.js and ShaderParticleEngine libraries. Bower will install the libraries into a bower_components/ directory under libs/ and those directories are referenced by the symlinks I've added here.
There was a lot of trailing whitespace that my editor removed automatically. You can add ?w=1 to the pull request URL to have GitHub show you a diff with whitespace ignored.
I've tested the demos and examples and everything looks ok except for glamcity.html, which throws an error with the minified glam.min.js. It works with the debug glam.js but looks a bit strange (the building textures are much brighter for some reason). Hopefully you can look into that.
If you do decide to merge this, you'll have to complete the changes by adding the new libaries:

cd into libs/
Delete the existing three.js.r68/ directory
Run bower install to fetch the latest libraries (as specified in bower.json)

Brian I would prefer not to have Bower changes and Three.js lib changes in the same PR. I'm all for the idea of a Bower setup but I don't want that to get tied up with also upgrading to a new Three.js. I want both but we need them separate. Please submit a clean PR with just the Bower changes and I will get to that soon. Thanks


[Event Parameter Change] Codeunit 22 - "Item Jnl.-Post Line" - procedure CalcPosShares - Event OnAfterCalcPosShares

We would like the parameter GlobalItemLedgEntry added to the following event.
Codeunit 22 - "Item Jnl.-Post Line" - procedure CalcPosShares - Event OnAfterCalcPosShares
Original:
 local procedure CalcPosShares(var DirCost: Decimal; var OvhdCost: Decimal; var PurchVar: Decimal; var DirCostACY: Decimal; var OvhdCostACY: Decimal; var PurchVarACY: Decimal; var CalcUnitCost: Boolean; CalcPurchVar: Boolean; Expected: Boolean)
    var
        CostCalcMgt: Codeunit "Cost Calculation Management";
    begin
        with ItemJnlLine do begin
            if Expected then begin
                DirCost := "Unit Cost" * Quantity;
                PurchVar := 0;
                PurchVarACY := 0;
                OvhdCost := 0;
                OvhdCostACY := 0;
            end else begin
                OvhdCost :=
                  Round(
                    CostCalcMgt.CalcOvhdCost(
                      Amount, "Indirect Cost %", "Overhead Rate", "Invoiced Quantity"),
                    GLSetup."Amount Rounding Precision");
                DirCost := Amount;
                if CalcPurchVar then
                    PurchVar := "Unit Cost" * "Invoiced Quantity" - DirCost - OvhdCost
                else begin
                    PurchVar := 0;
                    PurchVarACY := 0;
                end;
            end;

            if GLSetup."Additional Reporting Currency" <> '' then begin
                DirCostACY := ACYMgt.CalcACYAmt(DirCost, "Posting Date", false);
                OvhdCostACY := ACYMgt.CalcACYAmt(OvhdCost, "Posting Date", false);
                "Unit Cost (ACY)" :=
                  Round(
                    CurrExchRate.ExchangeAmtLCYToFCY(
                      "Posting Date", GLSetup."Additional Reporting Currency", "Unit Cost",
                      CurrExchRate.ExchangeRate(
                        "Posting Date", GLSetup."Additional Reporting Currency")),
                    Currency."Unit-Amount Rounding Precision");
                PurchVarACY := "Unit Cost (ACY)" * "Invoiced Quantity" - DirCostACY - OvhdCostACY;
            end;
            CalcUnitCost := (DirCost <> 0) and ("Unit Cost" = 0);
        end;

        OnAfterCalcPosShares(
          ItemJnlLine, DirCost, OvhdCost, PurchVar, DirCostACY, OvhdCostACY, PurchVarACY, CalcUnitCost, CalcPurchVar, Expected);
    end;

[IntegrationEvent(false, false)]
    local procedure OnAfterCalcPosShares(var ItemJournalLine: Record "Item Journal Line"; var DirCost: Decimal; var OvhdCost: Decimal; var PurchVar: Decimal; var DirCostACY: Decimal; var OvhdCostACY: Decimal; var PurchVarACY: Decimal; var CalcUnitCost: Boolean; CalcPurchVar: Boolean; Expected: Boolean)
    begin
    end;

Change:
 local procedure CalcPosShares(var DirCost: Decimal; var OvhdCost: Decimal; var PurchVar: Decimal; var DirCostACY: Decimal; var OvhdCostACY: Decimal; var PurchVarACY: Decimal; var CalcUnitCost: Boolean; CalcPurchVar: Boolean; Expected: Boolean)
    var
        CostCalcMgt: Codeunit "Cost Calculation Management";
    begin
        with ItemJnlLine do begin
            if Expected then begin
                DirCost := "Unit Cost" * Quantity;
                PurchVar := 0;
                PurchVarACY := 0;
                OvhdCost := 0;
                OvhdCostACY := 0;
            end else begin
                OvhdCost :=
                  Round(
                    CostCalcMgt.CalcOvhdCost(
                      Amount, "Indirect Cost %", "Overhead Rate", "Invoiced Quantity"),
                    GLSetup."Amount Rounding Precision");
                DirCost := Amount;
                if CalcPurchVar then
                    PurchVar := "Unit Cost" * "Invoiced Quantity" - DirCost - OvhdCost
                else begin
                    PurchVar := 0;
                    PurchVarACY := 0;
                end;
            end;

            if GLSetup."Additional Reporting Currency" <> '' then begin
                DirCostACY := ACYMgt.CalcACYAmt(DirCost, "Posting Date", false);
                OvhdCostACY := ACYMgt.CalcACYAmt(OvhdCost, "Posting Date", false);
                "Unit Cost (ACY)" :=
                  Round(
                    CurrExchRate.ExchangeAmtLCYToFCY(
                      "Posting Date", GLSetup."Additional Reporting Currency", "Unit Cost",
                      CurrExchRate.ExchangeRate(
                        "Posting Date", GLSetup."Additional Reporting Currency")),
                    Currency."Unit-Amount Rounding Precision");
                PurchVarACY := "Unit Cost (ACY)" * "Invoiced Quantity" - DirCostACY - OvhdCostACY;
            end;
            CalcUnitCost := (DirCost <> 0) and ("Unit Cost" = 0);
        end;

        OnAfterCalcPosShares(
          ItemJnlLine, DirCost, OvhdCost, PurchVar, DirCostACY, OvhdCostACY, PurchVarACY, CalcUnitCost, CalcPurchVar, Expected, GlobalItemLedgEntry); <--ADDED
    end;

[IntegrationEvent(false, false)]
    local procedure OnAfterCalcPosShares(var ItemJournalLine: Record "Item Journal Line"; var DirCost: Decimal; var OvhdCost: Decimal; var PurchVar: Decimal; var DirCostACY: Decimal; var OvhdCostACY: Decimal; var PurchVarACY: Decimal; var CalcUnitCost: Boolean; CalcPurchVar: Boolean; Expected: Boolean; GlobalItemLedgEntry: Record "Item Ledger Entry") <--ADDED
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 41118.


[3.2] Add natural string comparison

Backport of naturalnocasecmp_to. This method is exposed in master branch but not in 3.2, the default string comparison operators are not very suitable for human consumption in many cases where natural order is preferred. Implementing this in GDScript can't be done with 3 or 4 lines of code and the code is aready in the engine.

Thanks!


The Host header is ignored?

I am unable to create a request using a relative path, even when I set the Host header.
requests.exceptions.MissingSchema: Invalid URL '/wiki/List_of_HTTP_header_fields': No schema supplied. Perhaps you meant http:///wiki/List_of_HTTP_header_fields?
I expect to be able to configure a Host header and use it in my request:
headers = {'Host':  'en.wikipedia.org/wiki' }

resp = requests.get("/List_of_HTTP_header_fields",  headers=headers)
resp = requests.get("/Virtual_hosting", headers=headers)

The domain name of the server (for virtual hosting), and the TCP port number on which the server is listening. The port number may be omitted if the port is the standard port for the service requested. (Mandatory since HTTP/1.1)

Thanks for this!
I don't know why you expect to be able to do this: Requests has never allowed this, and never documented that it would be possible. We very intentionally require that users provide absolute URIs so there is no ambiguity: for example, in your examples above should Requests be using HTTP or HTTPS?
This is just not how Requests works.

I expected this to work as it is part of the protocol (HTTP/1.1):

The most common form of Request-URI is that used to identify a resource on an origin server or gateway. In this case the absolute path of the URI MUST be transmitted (see section 3.2.1, abs_path) as the Request-URI, and the network location of the URI (authority) MUST be transmitted in a Host header field. For example, a client wishing to retrieve the resource above directly from the origin server would create a TCP connection to port 80 of the host "www.w3.org" and send the lines:

   GET /pub/WWW/TheProject.html HTTP/1.1
   Host: www.w3.org

I can see how this is a can of worms from a security perspective and it deserves a warning. But when I set a hostname manually following the spec I do expect it to work.
You know way more about the subject then I do, so I really appreciate your feedback. Either way the error messaging can be improved as I clearly did not intent to go here: http:///wiki/List_of_HTTP_header_fields (Check for forward slash character?)

Requests is a high-level HTTP library: it performs a number of transformations for you. Ultimately, if you want to be able to craft low-level HTTP requests like this, Requests is not the tool for you: its value comes entirely from providing high-level abstractions and transforming them into the low-level Python bytes format.
Specifically, if you do requests.get('http://host/some/path'), that will lead to Requests sending:
GET /some/path HTTP/1.1
Host: host

In general, Requests does not introspect user headers: they are considered to be opaque bytes from our perspective. URLs, however, are structured data, and thus make a much better way to provide us with information about the host you want.

That is cool, I did not realise Requests is automatically generating the Host header. I assumed the Host header was empty and Requests kept full path in the GET command.
I think this issue needs two action points:

Update Request Manual Custom Headers to include Host headers info:

Authorization headers set with headers= will be overridden if credentials are specified in .netrc, which in turn will be overridden by the auth= parameter.
Authorization headers will be removed if you get redirected off-host.
Proxy-Authorization headers will be overridden by proxy credentials provided in the URL.
Content-Length headers will be overridden when we can determine the length of the content.
Host headers will be removed, they are automatically generated by Requests.

Create seperate error to make Invalid URL more clear. Checking for single forward slash should make clear user is trying to use relative path? Something like No host found in URL

Host headers will not be removed: we will let you override them, we just don't let them affect our processing at all. But yes, a PR that adds a note about Host headers to that section is a good idea.
As to Invalid URL, we can certainly do it, but I'm not sure how useful it is. This really doesn't come up very often, as evidenced by the fact that this is the first time in the seven year history of this project that anyone has had this problem. Still, I'm happy to review a pull request adding that to see how it looks.


Code cleanup

After changing the motor drive from torque command to velocity command, we need to cleanup unused code.

I guess this will also involve fixing the "static" torque feedback mode.

CMakeLists.txt in tests dir can be removed


Question: how to add Textmate bundles?

Hello,
I was trying to add a Textmate language grammar definition file for Twig to get syntax highlighting. Twig is templating language for HTML/PHP.
Here are the steps I went through:

Convert the language bundle from the Textmate repo as described in the Atom docs
Convert CSON to JSON
Added this line to my config.tsx:

...
"ui.fontSize": "12px",
"language.twig.textMateGrammar": "[absolute path on my HD to]/twig/grammars/html (twig).json"

Restarted Oni as good mesure 😊

Is there anything I'm missing (the syntax highlighting still doesn't work...)? Once I get the correct workflow sorted out, I can reformat this question to serve as a reference for others.

@badosu you were right to suggest using the vim plugin–in general they contain more than just syntax highlighting but usually other extra features that helps vim understand that language e.g. folding, snippets.
In the future I will always first start with the vim plugin.


Download dataset

hi,
I want to build this project on my own on GCP and wanted to know where can i find all the dataset used in this project

Hi, Its available from S3 at this location s3://start-data-engg/data.zip. You can download it and upload to GCP or store where you prefer.
command to download to local fs

HI,
I am not able to access this bucket can you please provide the zip file in a google drive or something. Or can you send me the full procedure to download data


Current Time Marker

hi,
since there are not many (paid and free) alternatives out there for a scheduler in MudBlazor and this one is fitting my needs quite well, I really want to make this work.
unfortunately I've come across a few issues.
one of these is that the marker for the current time is not displaying at the correct spot.
with half hour view the marker is always jumping to the next half hour.
in 15 minute view it is always between the grid lines and for each quarter hour it jumps half a grid field ahead and then stays there until the time hits the next quarter hour.
I have another major issue, but I will open a new ticket for that.
thanks for creating this scheduler/calendar and I hope these issues can get fixed.

I can't recreate this problem - it always looks correct to me.
Do you see this problem in the example on the docs page?  If yes, then what browser are you using?

yes, in the docs page it is always jumping ahead between the lines, no matter what setting I use. for some reason in my project it jumps only on the lines, but still shows the incorrect time:

I tested the doc page with different browsers (Edge, Chrome, Firefox), time was 8:40 am:

could this be a localization issue? I'm in germany.
but I don't see why my project shows something different, while on the same machine (separate VM) it shows the same behavior on the doc page.
either way neither shows the correct time.

Yes, it is a localisation issue.  I will fix that.

thanks

I have inserted the fixed code in my project, but unfortunately the time is still not correctly showing, although now the marker seems to move with each minute.
Actual time here is 11:24:

It seems so be shifted by about 10 minutes now.
Sorry for bothering.

Do you still have this problem?  In the example you sent me it looks correct.  Do you have the problem in all browsers?

Yes, the problem is still there, but as written before, the marker now moves with each minute.

actual time was 16:20 (I think it is showing 16:35, so it is shifted 15 minutes with each minute)
tested in Firefox, Chrome and Edge, same result

That's very strange I can't replicate this problem.  Could you try debugging it yourself and see if you can find the problem.

As you might have figured out, I'm quite new to Blazor, but I managed to figure out, that the shift is always exactly half of the set interval, so +5 when in 10 minute interval and +30 when in 60 minute interval.
For now I fixed my indicator by changing TimelinePosition() in DayWeekViewBase.razor.cs
from this
var minutes = (int)Math.Floor(DateTime.Now.Subtract(DateTime.Today).TotalMinutes) - (TimelineRow() * (int)Calendar.DayTimeInterval);
to this
var minutes = DateTime.Now.Subtract(DateTime.Today).TotalMinutes - (TimelineRow() * (int)Calendar.DayTimeInterval) - (int)Calendar.DayTimeInterval / 2;
But obviously this is not a general fix.
I don't see why the localization is changing the outcome, since DateTime.Now already gives the local date and time.
I tried changing DateTime.Today to DateTime.Today.ToLocalTime(), which again puts the indicator right in the middle of the interval lines.
Maybe you can figure out more with this information.
If not I will leave it like that, because this page is not supposed to be used outside this timezone.

Where do you get this code from?  The code in TimelinePosition() has never included Math.Floor().
The localization changes the output because some cultures use , as a decimal point and some cultures use .
The fix I applied was to use TimelinePosition().ToInvariantString() in the StyleBuilder() function.


largeIcon and smallIcon from server?

Hi, is it possible send push notification with dinamically server-side smallIcon and largeIcon?

Not in the current version. If you have a limited set, why not store them all in the app and point to the correct one in your notification payload?

Thank's for reply. I've tried
"data" : {
"title" : "my title",
"message" : "my message",
"smallIcon" : "myicon.png"
}
where myicon.png is placed in platform/android/res/drawable but nothing.

Ah right, specifying the small/largeIcon is not yet supported in the payload of the notification, only when configuring it with registerPush({ })...
If you want, you could send a pull request with this feature.

@flaviox83 have you/are you doing this? If not, I will.

Sorry, I'm not a Java programming


NodeJS installation issue on MSYS2

Operating system and version:
Windows 10 Pro 64-bit Build: 1803

nvm debug output:

$ nvm debug
nvm --version: v0.33.11
$SHELL: /usr/bin/bash
$SHLVL: 1
$HOME: /home/tfbas
$NVM_DIR: '$HOME/.nvm'
$PATH: /usr/local/bin:/usr/bin:/bin:/opt/bin:/c/Windows/System32:/c/Windows:/c/Windows/System32/Wbem:/c/Windows/System32/WindowsPowerShell/v1.0/
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'GNU bash, version 4.4.23(1)-release (x86_64-pc-msys)'
uname -a: 'MSYS_NT-10.0 2.11.1(0.329/5/3) 2018-09-10 14:19 x86_64 Msys'
curl: /usr/bin/curl, curl 7.61.0 (x86_64-pc-msys) libcurl/7.61.0 OpenSSL/1.0.2p zlib/1.2.11 brotli/1.0.6 libidn2/2.0.5 libpsl/0.20.2 (+libidn2/2.0.5) libssh2/1.8.0 nghttp2/1.30.0
wget: /usr/bin/wget, GNU Wget 1.19.5 built on msys.
git: /usr/bin/git, git version 2.19.1
grep: /usr/bin/grep, grep (GNU grep) 3.1
sed: -e expression #1, char 9: Unmatched ) or \)
awk: , GNU Awk 4.2.1, API: 2.0 (GNU MPFR 4.0.1, GNU MP 6.1.2)
sed: /usr/bin/sed, sed (GNU sed) 4.5
cut: /usr/bin/cut, cut (GNU coreutils) 8.30
basename: /usr/bin/basename, basename (GNU coreutils) 8.30
rm: /usr/bin/rm, rm (GNU coreutils) 8.30
sed: -e expression #1, char 9: Unmatched ) or \)
mkdir: , mkdir (GNU coreutils) 8.30
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.6.0
nvm current: none
which node: which: no node in (/usr/local/bin:/usr/bin:/bin:/opt/bin:/c/Windows/System32:/c/Windows:/c/Windows/System32/Wbem:/c/Windows/System32/WindowsPowerShell/v1.0/)
which iojs: which: no iojs in (/usr/local/bin:/usr/bin:/bin:/opt/bin:/c/Windows/System32:/c/Windows:/c/Windows/System32/Wbem:/c/Windows/System32/WindowsPowerShell/v1.0/)
which npm: which: no npm in (/usr/local/bin:/usr/bin:/bin:/opt/bin:/c/Windows/System32:/c/Windows:/c/Windows/System32/Wbem:/c/Windows/System32/WindowsPowerShell/v1.0/)
npm config get prefix: bash: npm: command not found
npm root -g: bash: npm: command not found

nvm ls output:

$ nvm ls
            N/A
node -> stable (-> N/A) (default)
iojs -> N/A (default)
lts/* -> lts/dubnium (-> N/A)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.14.4 (-> N/A)
lts/carbon -> v8.12.0 (-> N/A)
lts/dubnium -> v10.13.0 (-> N/A)

How did you install nvm? (e.g. install script in readme, Homebrew):
curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.33.11/install.sh | bash

What steps did you perform?
$ nvm install 10.13.0

What happened?
Clang v3.5+ detected! CC or CXX not specified, will use Clang as C/C++ compiler!
Downloading https://nodejs.org/dist/v10.13.0/node-v10.13.0.tar.xz...
######################################################################### 100.0%
Computing checksum with sha256sum
Checksums matched!
( NVM gets stuck here and does not proceed with installation)

What did you expect to happen?
Successful installation of nodejs??

Is there anything in any of your profile files (.bashrc, .bash_profile, .zshrc, etc) that modifies the PATH?
.bashrc:
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm
[ -s "$NVM_DIR/bash_completion" ] && . "$NVM_DIR/bash_completion"  # This load$

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

$ curl -I --compressed -v https://nodejs.org/dist/
* STATE: INIT => CONNECT handle 0x600057368; line 1429 (connection #-5000)
* Added connection 0. The cache now contains 1 members
* STATE: CONNECT => WAITRESOLVE handle 0x600057368; line 1465 (connection #0)
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* STATE: WAITRESOLVE => WAITCONNECT handle 0x600057368; line 1546 (connection #0)
* Connected to nodejs.org (<IP_ADDRESS>) port 443 (#0)
* STATE: WAITCONNECT => SENDPROTOCONNECT handle 0x600057368; line 1598 (connection #0)
* Marked for [keep alive]: HTTP default
* ALPN, offering h2
* ALPN, offering http/1.1
* Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
* successfully set certificate verify locations:
*   CAfile: /usr/ssl/certs/ca-bundle.crt
  CApath: none
* TLSv1.2 (OUT), TLS header, Certificate Status (22):
* TLSv1.2 (OUT), TLS handshake, Client hello (1):
* STATE: SENDPROTOCONNECT => PROTOCONNECT handle 0x600057368; line 1612 (connection #0)
* TLSv1.2 (IN), TLS handshake, Server hello (2):
* TLSv1.2 (IN), TLS handshake, Certificate (11):
* TLSv1.2 (IN), TLS handshake, Server key exchange (12):
* TLSv1.2 (IN), TLS handshake, Server finished (14):
* TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
* TLSv1.2 (OUT), TLS change cipher, Client hello (1):
* TLSv1.2 (OUT), TLS handshake, Finished (20):
* TLSv1.2 (IN), TLS change cipher, Client hello (1):
* TLSv1.2 (IN), TLS handshake, Finished (20):
* SSL connection using TLSv1.2 / ECDHE-RSA-AES128-GCM-SHA256
* ALPN, server accepted to use h2
* Server certificate:
*  subject: OU=Domain Control Validated; OU=PositiveSSL Wildcard; CN=*.nodejs.org
*  start date: Aug 14 00:00:00 2017 GMT
*  expire date: Nov 20 23:59:59 2019 GMT
*  subjectAltName: host "nodejs.org" matched cert's "nodejs.org"
*  issuer: C=GB; ST=Greater Manchester; L=Salford; O=COMODO CA Limited; CN=COMODO RSA Domain Validation Secure Server CA
*  SSL certificate verify ok.
* STATE: PROTOCONNECT => DO handle 0x600057368; line 1633 (connection #0)
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Using Stream ID: 1 (easy handle 0x600057368)
> HEAD /dist/ HTTP/2
> Host: nodejs.org
> User-Agent: curl/7.61.0
> Accept: */*
> Accept-Encoding: deflate, gzip, br
>
* STATE: DO => DO_DONE handle 0x600057368; line 1695 (connection #0)
* multi changed, check CONNECT_PEND queue!
* STATE: DO_DONE => WAITPERFORM handle 0x600057368; line 1822 (connection #0)
* STATE: WAITPERFORM => PERFORM handle 0x600057368; line 1837 (connection #0)
* Connection state changed (MAX_CONCURRENT_STREAMS == 256)!
* multi changed, check CONNECT_PEND queue!
* HTTP/2 found, allow multiplexing
< HTTP/2 200
HTTP/2 200
< date: Tue, 30 Oct 2018 13:27:38 GMT
date: Tue, 30 Oct 2018 13:27:38 GMT
< content-type: text/html
content-type: text/html
< set-cookie: __cfduid=deae977e10b0dd0cddc5f69068454dab31540906057; expires=Wed, 30-Oct-19 13:27:37 GMT; path=/; domain=.nodejs.org; HttpOnly
set-cookie: __cfduid=deae977e10b0dd0cddc5f69068454dab31540906057; expires=Wed, 30-Oct-19 13:27:37 GMT; path=/; domain=.nodejs.org; HttpOnly
< cf-cache-status: EXPIRED
cf-cache-status: EXPIRED
< expires: Tue, 30 Oct 2018 17:27:38 GMT
expires: Tue, 30 Oct 2018 17:27:38 GMT
< cache-control: public, max-age=14400
cache-control: public, max-age=14400
< expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
< vary: Accept-Encoding
vary: Accept-Encoding
< server: cloudflare
server: cloudflare
< cf-ray: 471e386afa858046-CPT
cf-ray: 471e386afa858046-CPT
< content-encoding: gzip
content-encoding: gzip
<

STATE: PERFORM => DONE handle 0x600057368; line 2008 (connection #0)
multi_done
Connection #0 to host nodejs.org left intact
Expire cleared

I’m not sure what “MSYS2” is. The only way nvm currently works on windows is in WSL/BashOnWindows.
If it’s getting stuck at that part of the installation, it’s likely a bug with WSL, since others have been able to install node this way.

I’m not sure what “MSYS2” is. The only way nvm currently works on windows is in WSL/BashOnWindows.
If it’s getting stuck at that part of the installation, it’s likely a bug with WSL, since others have been able to install node this way.

Hi ljharb. MSYS2 is a Linux like subsystem for Windows, it seems to be running a custom build of Arch Linux from what I can tell. I've had success installing nvm on it before on a previous installation. I prefer using MSYS2 since it keeps all my libraries for different languages in one place.

Ah - arch linux doesn’t ship with enough stuff on it to install node. You’ll have to install some things - but I’m not sure which. nvm is actually available in AUR which i believe will address this for you.

@ljharb nvm is installed, it just fails to install node, it keeps getting stuck on the extraction of the package it seems.
Can not determine how many core(s) are available, running in single-threaded mode. Please report an issue on GitHub to help us make nvm run faster on your computer! Clang v3.5+ detected! CC or CXX not specified, will use Clang as C/C++ compiler! Local cache found: $NVM_DIR/.cache/src/node-v10.13.0/node-v10.13.0.tar.xz Checksums match! Using existing downloaded archive $NVM_DIR/.cache/src/node-v10.13.0/node-v10.13.0.tar.xz

Yes, but the packages required to build node won’t be installed unless you manually installed them, or unless the AUR package does it for you.

Ok seems there is an issue with uncompressing the tar during installation, there were multiple instane of "xz data compression tool for .xz and .lzma files" running under the task manager from all the attempts to install nodejs 10. Seems there may be an issues with MSYS2's Cygwin backend. NVM has worked before for me under MSYS2 on another installation. I will continue to attempt to fix this and post my findings on success on here.

Closing for now until I find a solution.
Seems Windows Defender since April update is causing issues with the folder protection feature, causing bash commands to fail under cygwin in msys to uncompress and write to disk. I've noticed it blocking even third party applications like games on Steam and open source applications like Krita and Blender.
Seems MSYS2's build of Arch has a special repository outside Arch, it's mainly for mingw packages. Like I've said creationix nvm has worked before under MSYS2 before without installion issue, so the culprit seems to be Windows security. I could use nvm-windows, however i'd like to keep it under my msys2 directory. Will update this once solution is found.

Can not determine how many core(s) are available, running in single-threaded mode.
Please report an issue on GitHub to help us make nvm run faster on your computer!
Local cache found: ${NVM_DIR}/.cache/src/node-v12.14.0/node-v12.14.0.tar.xz
Checksums match! Using existing downloaded archive ${NVM_DIR}/.cache/src/node-v12.14.0/node-v12.14.0.tar.xz
$>./configure --prefix=/c/Users/user/.nvm/versions/node/v12.14.0 <
which: no python2.7 in (/c/Users/user/bin:/mingw64/bin:/usr/local/bin:/usr/bin:/bin:/mingw64/bin:/usr/bin:/c/Users/user/bin:/c/Program Files/Java/jdk1.8.0_191/bin:/c/Program Files/dotnet:/c/Program Files/Microsoft SQL Server/130/Tools/Binn:/c/xampp/php:/c/ProgramData/ComposerSetup/bin:/cmd:/c/Program Files/nodejs:/c/BC5/BIN:/c/Users/user/AppData/Local/Microsoft/WindowsApps:/c/Users/user/AppData/Roaming/Composer/vendor/bin:/c/Users/user/AppData/Local/Programs/Microsoft VS Code/bin:/c/Users/user/AppData/Local/GitHubDesktop/bin:/c/Windows/System32:/c/Users/user/AppData/Roaming/npm:/usr/bin/vendor_perl:/usr/bin/core_perl)
which: no python2 in (/c/Users/user/bin:/mingw64/bin:/usr/local/bin:/usr/bin:/bin:/mingw64/bin:/usr/bin:/c/Users/user/bin:/c/Program Files/Java/jdk1.8.0_191/bin:/c/Program Files/dotnet:/c/Program Files/Microsoft SQL Server/130/Tools/Binn:/c/xampp/php:/c/ProgramData/ComposerSetup/bin:/cmd:/c/Program Files/nodejs:/c/BC5/BIN:/c/Users/user/AppData/Local/Microsoft/WindowsApps:/c/Users/user/AppData/Roaming/Composer/vendor/bin:/c/Users/user/AppData/Local/Programs/Microsoft VS Code/bin:/c/Users/user/AppData/Local/GitHubDesktop/bin:/c/Windows/System32:/c/Users/user/AppData/Roaming/npm:/usr/bin/vendor_perl:/usr/bin/core_perl)
./configure: line 3: exec: python: not found
nvm: install v12.14.0 failed

python 2.7 is a requirement by node itself, so you’ll have to have that installed.

i encounter the same thing as @ghost
stuck on checksum match


Parallax image is not showing

Hello team,
 Thanks for creating such fabulous library. I am using carousel to show the images in sections. I wrote following code.

_renderCarouselRow({item, index}, parallaxProps) {
  return (
      <View style={{flex:1}}>
          <ParallaxImage
              style={{ marginLeft: 2, marginRight: 2, flex:1}}
              source={{uri:item.image}}
              containerStyle={{width:sliderWidth - 30, height:300, backgroundColor:'red'}}
              parallaxFactor={0.4}
              blurRadiusStart={3}
              blurRadiusEnd={1}
              unBlurDuration={1000}
          />
          <Text style={{top:100,color:'black', alignSelf:'center'}} numberOfLines={2}>
              { item.title }
          </Text>
      </View>
  );
}

Here I can't see the image on the screen. I can see red background color. My image uri is correct. I can see only loading indicator. I tried by hardcoding the image path, but still no luck. Also in the view I can't see text also.
Am I missing something? Please guide me.

⚠ You did not follow the contributing guidelines!
As stated in these:

You need to fill out the issue template. This step is mandatory! Not doing so will result in your issue getting closed. Don't take this personally if this happens, and feel free to open a new issue once you've gathered all the information required by the template.

Unless you provide a Snack example that reproduces the issue, we won't be able to help you and the issue will be closed.
Also, I don't know how you did come up with the following props as they never were implemented: blurRadiusStart, blurRadiusEnd and unBlurDuration.

Closing as no further feedback was provided.

where the solution ?

I have the same issue


a few suggestions

Hi ocealot, thanks for your works on vfxediter. I've been using it for some time and I have a few suggestions.

Could you change the "Rotation" value from radians to degrees? Most of the time I need to rotate the effect by a specific angle, like 90° or 30° and it will be troublesome to enter the value in radians. Or could you add a button that toggles the values to degrees?
When I copy a curve from one option onto another option that already has a curve, the two curves are superimposed. Can a "paste and overwrite" button be added?
Sometimes I need to modify multiple emmiters/particles in succession. But repeatedly finding the next one in the list (especially when the total reaches fifty or sixty) makes me dizzy. Can you add "previous" and "next" options to the list of emitters, etc?
The library and favorite list is useful. Is it possible to add grouping or classification functions to them? This will be the icing on the cake.
Scdediter can read the sound files of monster's skill sound files, so it would be better if there is a monster(NPC) list like vfxediter. Now it only contains BGM.

I hope you can think about these. Thanks again for your work ^ ^

I'll look into it
There's now a "clear" button, so you can use that then "paste"
Good idea, I'll add it
I'll look into adding something like that
Unfortunately, SCD file paths don't work like that. A tmb might look like chara/action/mon_sp/m7003/mon_sp006.tmb which is clearly associated with monster m7003. scds are like sound/battle/mon/3101.scd, so they do no correspond to a unique npc


name changed to candy crush

Please change the title key to Candy Crush

Their is no title attribute I think you mean to say change title attribute to Candy Crush.

thanks


feat: add BoxError trait to simplify the way to create generic error

Which issue does this PR close?
Closes #626
Rationale for this change
See title.

What changes are included in this PR?
See title.

Are there any user-facing changes?
None.

How does this change test
Test by ut.

Great job!

The real problem of this issue is we have too many trivial errors, one solution I think of is to only define within mod.rs, add files within this module share this error.
Will create a separate issue to discuss how to do error handling error.


Border Layout

Preparation

[x] Existing defect backlog reviewed/evaluated, prioritized, and selected
[x] Functional requirements defined: functional test script is defined/written (not coded) and signed off by Dev, Design, ADA and Product leads
[x] Non-functional requirements defined (e.g. controlled vs. uncontrolled)
[x] Existing enhancements backlog reviewed/evaluated, prioritized, and selected
[x] Usability testing plan defined or deemed unnecessary
[x] Examples needed to properly demonstrate component features are defined and signed off by Dev, Design, ADA and Product leads

Development

[x] Component implemented in/ported to TypeScript
[x] 3rd-party dependencies removed
[x] JSS converted to CSS
[x] CSS upgraded to use characteristics
[x] Selected defects resolved
[x] Component API finalized
[x] Necessary refactoring completed
[x] Selected enhancements added

Testing

[x] Functional test script is implemented in Cypress and passes
[x] Code not covered by the functional test script is identified and reviewed by Dev & Design leads
[x] QA (v1-2 visual regression) test passes
[x] Automated Axe tests added and pass

Examples

[x] All examples present in TK1 are implemented as stories in Odyssey (ported from BitBucket)
[x] All examples needed are implemented as stories (either ported from BitBucket or created net new)

Deployment

[x] Codemods for migration created and verified

Reviews/Signoff

[x] Dev production review
[x] Design review
[ ] Product signoff
[ ] ADA review

Documentation

[ ] Dev docs content provided
[x] TK1→Odyssey migration guide provided
[ ] Design docs provided
[ ] Consolidated documentation produced by combining dev content, migration guide, and design docs
[x] Consolidated documentation published on Storybook
[ ] Consolidated documentation reviewed and updated according to user feedback

Figma symbols

[ ] New Odyssey Figma symbol created
[ ] Figma symbol references updated to use new Figma symbol

The code is generic with  elements.  element does not have semantic meaning; therefore names header and main in the design might be confusing as resemble semantic elements  and . Rename Header and Main. E.g Header=Top, Main= Main section. (Check with Content Designer to find the most appropriate name).
For building a website I would suggest to add and option for meaningful semantics so the structure of a document is semantically correct; therefore screen reader users can navigate from section to section.

Top: …
Main section: …
Left: … or …
Right: … or …
Bottom: …

Border Layout Form Composite and Border Layout Form Contacts Composite examples
A Layout component is accessible, however components inside the layout component are not accessible.

@maj-stella here you go: https://polymorphic-layout-component-gsek.uitk.pages.dev/?path=/story/core-layout-borderlayout--border-layout-all-panels
Let me know how that looks :)

In examples ‘Border Layout Fixed Panel’ and ‘Border Layout Contacts Composite’ there are buttons which are in <footer> (first screenshot below). Move buttons to the <main> as they are part of the main content and delete <footer>. If <footer> element needs to be on the page to showcase the structure of the document, put some additional information and links instead. Please see an example of footer in the second screenshot below.

Buttons in <main>

Example of content in <footer>


Update logo file format

Replace the png version with the SVG file.

A white background had to be added for the logo to display properly with a dark theme. There's currently no way of detecting what someone's github theme setting is (https://github.community/t/support-theme-context-for-images-in-light-vs-dark-mode/147981) but once there is, will update it to swap the font colour accordingly.


Alignment failure on images with large elevation change

The stereo process fails or produces poor results when called with AffineEpipolar alignment on images with large elevation changes.  This is probably caused by the initial affine computation performed on a zero-elevation datum.  A test case has been posted on Lunokhod2:  /home/smcmich1/data/steepDropTestCase
Run the command:
stereo_pprc M128306716LE.correctedMosaic.cub M128333857LE.correctedMosaic.cub noprojStereo --alignment-method AffineEpipolar
The output should include the following lines showing high error:
    Found interest points:
      left: 7650
     right: 5474
--> Matching interest points
    Matched 46 points.
    Inlier cluster:
      Triangulation Err: 2397.68 +- 336.924 meters

The input files are mosaics composed to two LRONAC image pairs each created with the ISIS noproj tool.  There is a 3000 meter drop about 3/4 of the way down the images.
There are cropped versions of the input files which do not contain the drop which work fine:
stereo_pprc M128306716LE.correctedMosaic.cropped.cub M128333857LE.correctedMosaic.cropped.cub croppedStereo --alignment-method AffineEpipolar
There are also pre-noproj mosaic versions of the files which work fine.
stereo_pprc M128306716LE.justCal.cub M128333857LE.justCal.cub stereo --alignment-method AffineEpipolar

This test case works fine now so a change since this problem was reported must have corrected it.


Release new version

We are getting a build error because of broccoli-caching-writer on version 2.3.1, I see that it is updated here to 3.0.3. We tested it with this new version and the build succeeds, can you please release a new version to fix this?

+1

Sure, I’ll release today.

done


armdev: pagefault with small address that not maped

I still get this pagefault with very address that is apparently not maped. when I execute docker run -d docker run -d --runtime=quark_d  nginx`, I got this
[INFO] [1/43c0180000|0] DFSC/IFSC == 0x5, FAR == 0x70, acces-type fault == r--, during address translation == false.
[INFO] [1/615590] hypecall: 8, paras: 43c019d9e8 0 0 0
[INFO] [1/43c0180000|0] VM: PageFaultHandler - FAR: 0x70, PC: 0x2b6cd70c2cd8,TTBR: 0x43c0401000, is-user: true, error code: 0x24.
[INFO] [1/615707] hypecall: 8, paras: 43c019d3c8 0 0 0
[DEBUG] [1/43c0180000|0] VM: Entered HandleFault.
[INFO] [1/615865] hypecall: 8, paras: 43c019cf78 0 0 0
[INFO] [1/43c0180000|0] sendsignalLocked, signal:SignalInfo { Signo: 11, Errno: 0, Code: 1 }, group:false
[INFO] [1/615972] hypecall: 8, paras: 43c019be78 0 0 0
[INFO] [1/43c0180000|0] sendsignalTimerLocked, signal:SignalInfo { Signo: 11, Errno: 0, Code: 1 }

It is a page fault with the address 0x70, which is too small.

I  think it is still some bugs related to libc or dynamic linker, when I execute a rust program build with musl, it is ok.

@CharlyYu This could be (probably) even related to #1099.

@CharlyYu This could be (probably) even related to #1099.

As a temporary workaround, please avoid using sys_clone at all. For example, if the docker image has a shell script as a entrypoint, it will call sys_clone with running the actual program.
Sorry I've been really busy recently and I can't help before Friday.

I think I have got the reason, MProtect changed the page flag, but I think some flag set is wrong, https://github.com/QuarkContainer/Quark/pull/1131 I will test PR and reopen it again.

this should be fixed.


add gh-page web link to description

it would be nice not to modify the url after opening the repo.
add the gh-page web link to the description and readme.

@rhoit i updated the readme but description should be updated by any of the admins


Fix MeasurementPeriodList validation

Description
Add the createdBy and updatedBy attributes on the  MeasurementPeriods of a Proposal.
I'm unsure this is the correct way to do it.
Motivation
Allow ingestion of measurement periods in the new backend.
Fixes:
#495
TODO

Add tests
Is it a problem that the timestamps are slightly different than the ones for the proposal?

Tests included/Docs Updated?

[ ] Included for each change/fix?
[ ] Passing? (Merge will not be approved unless this is checked)
[ ] Docs updated?
[ ] New packages used/requires npm install?
[ ] Toggle added for new features?

As for the test, probably just add a proposal with non-empty measurement peroid  to test/TestData.ts and use it in test/Proposal.ts

Thanks for the feedback, just added the test


document arithmetic special functions

This fixes #3821.
Related to #4480.

jenkins, test this please.

jenkins, test this please.

LGTM!


Extension incompatible with GNOME 44.0

I've upgraded to Ubuntu 23.04, which uses GNOME 44.0 . This extension is no longer enabled, because it is marked as incompatible with this version of GNOME.

a temporary solution: edit metadata.json, append 44 to shell-version

version 20
Added support for gnome version 44.

Thank you so much!


allow tab key input (fixes jaredreich/pell#66, satisfies WCAG 2.1.2)

Preventing Tab key input without providing an alternative (and advising the user of that alternative) violates WCAG 2.1.2 “No Keyboard Trap”.  That breaks Tab-based focus navigation, which breaks pell for many users.
This PR re-enables the default behavior for the Tab key, fixing jaredreich/pell#66 and eliminating the WCAG violation.
If an application developer has a more relevant behavior for the Tab key, they can:

[ ] implement their alternative behavior for Tab
[ ] prevent the default behavior of Tab
[ ] implement an alternative keyboard input for escaping the editor
[ ] advise users about their alternative for escaping the editor

Thanks @geppy

@julianguyen FYI

This is awesome! Thank you :D


Change the grades view for grade_entry_form to use assignment_content layout

Also add css to align the Release/Unrelease Marks buttons correctly, according to where they are in the browse view for submissions (http://<IP_ADDRESS>:3000/en/assignments/1/submissions/browse)

Thanks, @Sm1th! This is exactly what I was looking for.

View it via: http://<IP_ADDRESS>:3000/en/grade_entry_forms/1/grades
The grade_entry_form is rendered for a "Marks Spreadsheet" and does not seem to be for an assignment.


Fix logging fail when exception has unicode characters

Hello,
When my service raises en error (with some non-ascii characters in message), exception and its trace is not visible, because logging cannot process such message:
<WorkerContext [PS.storeFollowingEvent] at 0x7f0e141101d0>
Traceback (most recent call last):
  File "/usr/lib/python2.7/logging/__init__.py", line 859, in emit
    msg = self.format(record)
  File "/usr/lib/python2.7/logging/__init__.py", line 732, in format
    return fmt.format(record)
  File "/usr/lib/python2.7/logging/__init__.py", line 471, in format
    record.message = record.getMessage()
  File "/usr/lib/python2.7/logging/__init__.py", line 335, in getMessage
    msg = msg % self.args
UnicodeDecodeError: 'ascii' codec can't decode byte 0xd0 in position 0: ordinal not in range(128)
Logged from file containers.py, line 393
<IP_ADDRESS> - - [14/Nov/2017 19:29:07] "POST /storeFollowingEvent/252525252525/99999999999/1?moment=234567891 HTTP/1.1" 500 204 0.015099

Actionally, message is cyrillic:
ОШИБКА:  нулевое значение в столбце "id" нарушает ограничение NOT NULL
DETAIL:  Ошибочная строка содержит (1977-06-08 07:44:51,<PHONE_NUMBER>9, t, null).

When I add decode('utf-8'), problem is away, exception is printed OK.
Regards,
Evgeniy

Hi,
Thanks for the contribution! However, i'm a bit confused, since that file includes from __future__ import unicode_literals, so that formatting string should already be unicode, not ascii.
What version of nameko and python is this on? Could you post a minimal example to reproduce the error please?

I use Python 2.7.9 and nameko 2.8.0.
Here is reproducing code (reproduce.py) :
#!/usr/bin/python

# If I remove below line, UnicodeDecodeError goes away
from __future__ import absolute_import, unicode_literals

import logging

logging.basicConfig()

try:
	# I've got following utf-8 encoded string in exception from Postgres driver.
	# Actionally, it is cyrillic characters
	ba = bytearray([208,158,208,168,208,152,208,145,208,154,208,144])
	raise Exception(ba)
except Exception as ex:
	logging.getLogger().error ('error %s', ex, exc_info=True)

Right, now i'm with you. Had misread decode vs encode.
I think the best way to stay sane with regards to unicode is what e.g. ned batchelder calls the "unicode sandwitch". Basically, you want to decode your bytes as early as possible (and if required, encode back as late as possible)
In your example, since you know you are getting back bytes representing a utf8 encoded string,  i think you want to raise Exception(ba.dcode('utf-8'))

Closing this as unrelated to Nameko. Hope you fixed your issue @eugeneRover


Juniper JunOS Remote Code Execution

Juniper Networks has resolved multiple critical vulnerabilities in the J-Web component of Junos OS on SRX and EX Series devices. These vulnerabilities, when chained together, could allow an unauthenticated, network-based attacker to remotely execute code on the devices. The vulnerabilities affect all versions of Junos OS on SRX and EX Series, but specific fixes have been released to address each vulnerability. Juniper Networks recommends applying the necessary fixes to mitigate potential remote code execution threats. As a workaround, users can disable J-Web or limit access to only trusted hosts. Proof-of-concept (PoC) exploit code has been released, demonstrating the severity of these flaws and the urgency to apply the fixes.
CVE-2023-36844
CVE-2023-36845
CVE-2023-36846
CVE-2023-36847

changes verfied! 👍


Update Issue - Automate_undefined_68825

default description

Property
Value

Browser
Chrome 130.0

Operating System
OS X Sequoia

Resolution
1688x792

Screenshot Attached
Screenshot URL

Updated Screenshot Attached


Screen capture isn't capturing full page

Can we capture the full vertical height of the data pages? Seems we're cutting off the bottom of a lot of pages in our screen capture.

I'll look at this - the capture logic has been too simple for our use cases (I've been manually setting a capture browser window that works for most pages, and whitelisted some states that needed more but this isn't sustainable). Unfortunately I can't backfill this, but I can fix going forward.
@jdmaresco any other states off the top of your head that you've noticed this for?

I only looked at a couple – Colorado also had the issue: https://covidtracking.com/screenshots/CO/CO-20200324-140132.png
–jd maresco
linkedin ( https://www.linkedin.com/in/jdmaresco/ ) | twitter ( https://twitter.com/jdmaresco )
On Tue, Mar 24, 2020 at 5:26 PM, Julia Kodysh <<EMAIL_ADDRESS>> wrote:

I'll look at this - the capture logic has been too simple for our use
cases (I've been manually setting a capture browser window that works for
most pages, and whitelisted some states that needed more but this isn't
sustainable). Unfortunately I can't backfill this, but I can fix going
forward.
@ jdmaresco ( https://github.com/jdmaresco ) any other states off the top
of your head that you've noticed this for?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/COVID19Tracking/covid-data-pipeline/issues/7#issuecomment-603513605
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAKEG3ZBEMMW5QLYWXETHZTRJEQRJANCNFSM4LS7Y5TQ
).

This is now fixed for (most) screenshots starting early morning 3/25. There are still some states with poorly-behaving websites which may get truncated, and I can fix manually as we see them.

@julia326 FWIW, CA's seem to be truncated currently. The pictures from https://covidtracking.com/data/state/california/ look like   i.e. there's no visible numbers. Got worse over the last few days, presumably because there's more content at the top of the page.


docs: update installation guide

Signed-off-by: Navendu Pottekkat<EMAIL_ADDRESS>Updates installation docs and rearranges it.

please check CI errors


Remove expensive CSS query

issue goal: remove expensive CSS queries
On
https://github.com/SAP/techne/blob/v2.0/dist/css/techne-all.css#L165
the CSS query
[class*="tn-primary-nav--"] { }

That's a expensive query to perform and is equivalent to
.tn-primary-nav { }

which is the default query for CSS, performance on browser implementations is optimised for it. Then use the cascade as described on #659

There is an important difference in the two examples. Back to BEM, this shows a modifier for the component ...
.foo {
	border: solid 1px black;
	background-color: white;
}
.foo--highlighted {
	background-color: yellow;
}

In the markup, BEM notation (like Bootstrap) requires duplicate classes where foo gives you the base styles and the modifier class gives you the modifications ...

Adding the attribute selector means there is no need for the duplication in the markup ...
[class*="foo--"], .foo {
	border: solid 1px black;
	background-color: white;
}

If you look at the source you'll see that this was testing a concept. https://github.com/SAP/techne/blob/v2.0/src/styles/components/primary-nav.scss#L26
Not much attention has been given yet to component SASS structure and coding recommendations. As more components are built out we can assess the value. The attribute selector may not be useful or too hard to understand for some devs but I've found nothing to indicate this an expensive selector for performance being no worse than using multiple child selectors.

This is issue is not solved. Please don't close it.
Bootstrap is not using BEM
https://github.com/twbs/bootstrap/blob/v4-dev/dist/css/bootstrap.css#L4055
and Material2 is not using it either
https://github.com/angular/material2/blob/master/src/lib/card/card.scss
It is not a easy to type class names for developers and it creates a duplicated code as explained on #659
On a 1x1 benchmark it is more expensive
https://benfrain.com/css-performance-revisited-selectors-bloat-expensive-styles/
Comparing results of 10 and 20 tests
We have to ditch BEM notation since it doesn't bring any real value and create duplicate code

I'm fine removing the attribute selector. There are other ways to achieve the same thing in CSS.
But I strongly disagree about BEM. It brings A LOT of value and unless we align around another approach or I see a specific custom implementation recommendation, there is no reason to change our approach.
Please show me how you would define classes names for a component with multiple elements and multiple modifiers.

Component with multiple elements and multiple modifiers
.tn-component { }

    .tn-component-element1 { }
    .tn-component-element2 { }

    .tn-component-modifier1 { }
    .tn-component-modifier2 { }

/* card example */

.tn-card { }

    .tn-card-header { }
    .tn-card-content { }

    .tn-card-vertical { /* default, modifier optional */ }
    .tn-card-horizontal { }

and

 ...

 ...

@xak There you go


After an update to the latest version of HA and hacs-thermiagenesis no connection

I can no longer connect through the integration.
With a seperate modbus poller I can still just see the registers of my heatpump.
any advice on how to troubleshoot?

HA:
Home Assistant 2023.4.0
Frontend 20230405.0 - latest
Thermiagenesis: 3.1.1
Log info:
2023-06-01 15:42:32.649 DEBUG (MainThread) [custom_components.thermiagenesis.config_flow] Attempt connect...
2023-06-01 15:42:32.649 INFO (MainThread) [custom_components.thermiagenesis.config_flow] Other error
2023-06-01 15:42:32.650 INFO (MainThread) [custom_components.thermiagenesis.config_flow] 'bool' object is not callable


try to fix strings

Signed-off-by: anupritaisno1<EMAIL_ADDRESS>

Some of this isn't right and seems to be missing what the strings were trying to accomplish. It needs to take into the context the strings were used and probably needs to be split up more.


feat: add support for a model introspection provider as an input property

Description of changes
Adding a new input parameter where a 'modelIntrospectionSchemaProvider' can be passed in. This is a workaround for the fact that bundling deps is quite painful, and after spending some time on it here, we still weren't able to get the right matching deps for the full appsync-modelgen-plugin library. Instead, rely on that being passed in from the client use-location.
CDK / CloudFormation Parameters Changed
N/A
Issue #, if available
N/A
Description of how you validated changes
Unit + Functional tests on the changes and the outputs.
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)
[x] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Note: this may not end up working in any capacity. We will end up needing to invoke an async method for the injected codegen, so it'll need to pass in a Promise<string> though this won't work since we do all this work in the ctor. We may need some additional method to asynchronously build the meta.


Complex nested structures, like categories, trees, etc..

Hello, I'm going to add more tests for complex nested objects.
More: http://stackoverflow.com/questions/31137630/deserialise-json-payload-into-object-in-node-js

@gcanti thanks for fast reply


Finalizing step take long time

Hi.I've created 4 folders and copy some photos to each other.
Why after train my model finalizing step take long time and nothing occur?

How long? Can you share your dataset?

Closing due to lack of activity. If you're still hitting this issue please re-open and provided the requested information.


Rules Alert

Added a 'Show Rules' button as per #4

Thank you so much iktucker, this implementation is fantastic and is just what the game needed!


Integration with VOI

See also https://github.com/ubahnverleih/WoBike/blob/master/Voi.md for an API documentation.

closing due to inactivity. Let's reopen if someone finds an API to fetch trips.


zfs: fix evaluation and build on aarch64 systems

This is blocking: https://github.com/nix-community/nixos-images/pull/115
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

As of now latestCompatibleLinuxPackages points to broken version on aarch64-linux.

Yes. This is a convoluted mess:
boot.kernelPackages = boot.kernelPackages = lib.mkDefault (pkgs.zfs.override {
  removeLinuxDRM = pkgs.hostPlatform.isAarch64;
}).latestCompatibleLinuxPackages;

boot.zfs.removeLinuxDRM = true;

If you have a better idea... I don't have one.

I tried to test this PR with the following config, which I think is fairly standard.
boot.kernelPackages = config.boot.zfs.package.latestCompatibleLinuxPackages;
boot.zfs.removeLinuxDRM = true;

This still results in using 6.1, but also attempts to apply the patch which of course fails. This PR overrides the zfs package with removeLinuxDRM before it is passed to extraModulePackages, but this has no effect on the value of boot.zfs.package or pkgs.zfs, which are normally used to choose the proper kernel packages.

I made this more explicit in the option description. Please have a look.

friendly bump @RaitoBezarius @lopsided98

I will take responsibility.

Are there plans to backport this to 23.05?

We are too close to 23.11 that I can take care of this. You are free to open a backport PR as long as it complies with the backport policies.

hi, is this fixed for 23.11 ?

Yes, this is fixed in NixOS 23.11. You have to enable the boot.zfs.removeLinuxDRM option though: https://search.nixos.org/options?channel=23.11&show=boot.zfs.removeLinuxDRM


Marking visually selected areas?

Hey guys. We kinda like the idea of signature, and we are working on a small plugin for saving and restoring another kind of marks: marks to save and restore visually selected areas. It is young and small, but consistent so far and it is starting to get snappy, check it out there.
Wouldn't it be nifty to integrate it into signature, displaying colored marks locating the start / slash / the end of a selection, navigating across them etc.?
Should you be interested, please drop us a line and we'd be happy to push to your repo' :)

Hello @iago-lito, I'm subscribe to the unix philosophy of creating one tool for one job and ensuring that it does the job well. So much so that I considered splitting markers away from marks but didn't do so only because of a large portion of code being common to both.
Thus, I'm not in favor of merging two different tools offering completely different features. However, if you do anything from my side like making available some functions etc. do let me know and I'll see what I can do.


Align segment structure with openai/whisper

At the moment, the segments returned by faster-whisper are a subtly different shape to those returned by Whisper and also lack some of the information that JSON/verbose transcription results from that repo provide.
It appears that all of the information necessary to return these is in fact present in the transcribe function of faster-whisper, but they're simply not returned at present.
This PR adds those in and adjusts the naming of avg_log_prob to be consistent with Whisper.
The quirk I'm not certain of is whether it would be best to adjust the returned seek value on the other side of the VAD, but I don't do that at present.

I created a PR for this in the past (before VAD was implemented), but I only implemented avg_logprob (sorry @guillaumekln  typo there in my original PR) and no_speech_prob as I at that time I didn't think the other values will change things from a decision making perspective (whether to reject the output or not, or to assess the reliability of the output). In hindsight, I could see how knowing the temperature of the output can be useful (as temperature >0 may not be reliable).
I do wonder if VAD actually makes no_speech_prob (kinda) obsolete.....


Providing Rectified Extrinsic Transforms

Hello,
I'm trying to run a visual-slam system on KITTI-360, but having the issue that while rectified images are provided the extrinsic transforms in calib_cam_to_pose.txt are for the uncalibrated cameras.  Could you possibly provide the extrinsic transforms for the calibrated cameras?
Thanks!

Hi, the extrinsic transform for the calibrated cameras can be obtained using the following script:
https://github.com/autonomousvision/kitti360Scripts/blob/9defe218f4d05a1c920924f38ebfa8efc27318da/kitti360scripts/helpers/project.py#L35-L38


Add missing types

I add missing types and fix some typo.

Nice work, thanks! Since we're adding typed parameters and return types, which I think are only available from PHP 7.0 onwards, we probably need to bump the minimum PHP version in composer.json as well, and I should release this as a major version probably to indicate the backwards incompatibility.

No problem !
Let me know when you want to do the next release.If I have time, I want to help on this project.

Oh no need to close this PR! Happy for it to stay open.
Just suggesting that we need to change composer.json to reflect the new PHP version requirement.
Once that's added this can be merged into master, and then I can tag a new version release.


OneDrive File Picker does not work on safari and firefox

OneDrive File Picker does not work on safari and firefox, but it works well on chrome.
This issue can reproduce both on our product and oneDrive sample
Reproduce steps:
a) Open https://dev.onedrive.com/sdk/js-v7/js-picker-open.htm
b) Click Open from OneDrive button
c) Login with your account
Result:
The file picker will keep loading, cannot get reference files
Environment:
OS: Version 10.11.5
Safari: Version 9.1.1 (11601.6.17)
Firefox: 48.0a2
It's really a critical issue, it's impact our product's release.

hi @benjaminliugang, I tested with following 2x2 matrix:

Customer Account
Business Account

Safari
work
failed

FireFox
work
work

My test environment is:
Mac OS: 10.11.5
Safari: 9.1.1 (11601.6.17)
FireFox: 47.0.1
The failing case has been noticed. If it is not  your case, could you please provide more detail on the account you are using? Any browser console log and/or screenshot will be appreciated.

Already works for my case, just close the ticket.


Adding verification to the attribute 'collectionTime' in a client's ingestion request

The collectionTIme can't be older then current time - BEFORE_CURRRENT_COLLECTIONTIME_MS or newer than current time  + AFTER_CURRENT_COLLECTIONTIME_MS.
Also added error messages for the HTTP repsonse body which enumerate the following errors:

no tenantId, when required
collectionTime too old
collectionTime too new

The rest looks good. Once you sort out the Travis failure, I'll take another look and give final go-ahead. :-)

@ChandraAddala @VinnyQ @izrik @drenalin23  - I included @shintasmith feedback into the PR.  This PR will have updates for verifying collectionTime.  Future PRs related to this story will have updates to do the same checks for pre-aggregated metrics.

@ChandraAddala @shintasmith - I think i addressed all the concerns.  Please take a look.

@ChandraAddala & @shintasmith gave verbals lgtms


Is PinPoint only use Thrift to compress the data ?

There is a doubt that how does Pinpoint use the Thrift ? I have read Command.thrift , Pinpoint.thrift and shared.thrift in the project , but I didn't find any 'service' in the script . I can't find the TSocket be used in the project except of the Plugins .
Is that means PinPoint does not use the Thrift to implement RPC ? Is Pinpoint only use the Thrift's CompactProtocol to compress the data ?
Thanks for Pinpoint Team

Hi Eilikce,
you're absolutely correct. We only use Thrift internally for data compression. We wrap this with custom header before sending off to the collector. The collector mirrors this by using Thrift for the final decompression of the received data.

@Xylus
Thanks a lot .I'm just waiting the answer online .


add --generate-GDEF option

as discussed in https://github.com/googlei18n/glyphsLib/issues/427, this makes the generation of the GDEF table in the exported UFO features independent of the other option --no-preserve-glyphsapp-metadata (which corresponds to minimize_glyphs_diffs keyword argument of UFOBuilder).
A new argument generate_GDEF is added to the functions that build UFOs (we have a lot of them!), and a corresponding command-line option --generate-GDEF is also added to the glyphs2ufo script.
The default value of this new option is different between the console script (which defaults to False) and the programmatic interface (which defaults to True). This is in the same vein as the other boolean options like propagate_anchors and minimize_glyphs_diffs that similarly have opposite defaults between the API and the CLI.
This is for historical reason and to keep backward compatibility with tools (e.g. fontmake) that expect the ufo-building functions in glyphsLib to produce ready-to-export UFOs (i.e. without extra editor's metadata, with anchors propagated to composites and a GDEF table generated in the features).
On the other hand, the two companion console scripts glyphs2ufo and ufo2glyphs are designed to be used primarily for going back and forth between UFO and glyphs sources during the font development process, hence their defaults reflect that use case.
So doing fontmake -g MyFont.glyphs -o ufo is not the same as glyphs2ufo MyFont.glyphs. In the latter, the anchors are not propagated to composites, and there is extra data that is needed to reduce the diffs when roundtripping back to Glyphs format, and the features do not contain a GDEF table.
To obtain the same output as fontmake when using glyphs2ufo script one would need to additionally pass --propagate-anchors --generate-GDEF (and optionally --no-propagate-glyphsapp-metadata although the latter should be innocuous for the sake of building an OTF from the generated UFOs).
related fontmake issue https://github.com/googlei18n/fontmake/issues/457

lgtm. If we add more options, we should probably replace all the single options with an options dict or something.


twister: Add optional memory footprint report into twister.json

New Twister option --footprint-symbols allows to select which memory area's data to include into twister.json report as footprint object with detailed memory usage statistics generated by West in -t footprint mode.
The new option implies and extends --create-rom-ram-report, so there are three choices currently: 'ROM', 'RAM', and 'all'.
By default, the new option is disabled.
The new option allows to collect detailed footprint results consistently for each test suite instance in the test scope having all metadata from the Twister execution in the same report file. Next, the report file might be filtered and restructured up to the user's needs for further analysis and visualization, depending on a custom data storage schema.

an example snippet from twister.json after
./scripts/twister -p frdm_k64f -T tests/benchmarks/footprints --footprint-symbols all

the footprint.ROM and footprint.RAM objects are from each instance's build files 'rom.jsonandram.json` respectively.
{
    "testsuites":[
        {
            "name":"tests/benchmarks/footprints/benchmark.kernel.footprints.default",
            "platform":"frdm_k64f",
            .....
            "footprint":{
                "ROM":{
                    "symbols":{
                        "name":"Root",
                        "size":23640,
                        "identifier":"root",
                        "address":0,
                        "children":[
                            {
                                "name":"(no paths)",
                                "size":3216,
                                "identifier":":",
                                "address":0,
                                "children":[
                                    {
                                        "name":"mcux_mcg_off",
                                        "size":4,
                                        "identifier":":/mcux_mcg_off",
                                        "address":16389
                                    },
            .....
                               ]
                            },
                            {
                                "name":"ZEPHYR_BASE",
                                "size":15526,
                                "identifier":"/home/foobar/zephyrproject/zephyr",
                                "address":0,
                                "children":[ ... ]
                            },
                            {
                                "name":"WORKSPACE",
                                "size":1682,
                                "identifier":"/home/foobar/zephyrproject",
                                "address":0,
                                "children":[ ... ]
                            },
                            {
                                "name":"(hidden)",
                                "size":3216,
                                "identifier":"(hidden)",
                                "address":0
                            }
                        ]
                    },
                    "total_size":23640
                },
                "RAM":{
                    "symbols":{
                        "name":"Root",
                        "size":9867,
                        "identifier":"root",
                        "address":0,
                        "children":[ ... ]
                    },
                    "total_size":9867
                }
            }
        },
            .....
}

This is a hard No.
twister.json should not be a dumping ground for this type of data. I see this becoming a trend with this, benchmark data, coverage etc.

well, why so hard no for an optional feature ?
twister.json is a very useful container for extra data to analyze it consistently and at scale execution.

Data not related to test results should go into seperate files and should not be crammed in twister.json which is primarely for test results.

the footprint data is optional to include for some limited number of footprint benchmarks which are also tests, aren't they ?

This is a hard No.
twister.json should not be a dumping ground for this type of data. I see this becoming a trend with this, benchmark data, coverage etc. Data not related to test results should go into seperate files and should not be crammed in twister.json which is primarely for test results.

@nashif, an alternative implementation with footprint data in a separate json - #72470

close in favor of #72470 variant


Add support for atmos/camo image proxy

Hi,
is it possible to have atmos/camo image proxy ready with mattermost?
see:
https://docs.mattermost.com/administration/image-proxy.html?highlight=image proxy
greets

Hi,
I don't think this is the goal of this repository. We are currently dropping support for web image so I think we will not add more "side containers" like an image proxy.

Hi Pichouk,
thanks for your reply. I liked the All in One solution, I guess most ppl have a dedicated server for their mattermost installation. I suggested to include camo because currently i had problems with the builtin local image proxy in 5.8.
greets