MB-15131 - End-to-end tests for SC document review

Jira ticket for this change
Summary
This adds a test for a service counselor's complete PPM document review flow.
Since there are analogous methods for waiting for the review page of expenses and pro gear, I also refactored reviewDocuments, which was waiting on the Trip 1 text that is only present for weight tickets, to reviewWeightTicket().
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make client_run

Terminal 2
Start the UI locally.
make server_run

Terminal 3
Run the test individually:
yarn playwright test -g "can complete review of PPM shipment documents"
Run all the tests:
make e2e_test

Additional steps

Access the
Login as a

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

Messages

:book:
:link: MB-15131

  Generated by :no_entry_sign: dangerJS against b354a98f0d92ad68c8a305e1f3aa74cd2fbf21ea


Stalling at "Logging with Profile <my_profile_name>" on headless RHEL

Installed npm and all dependencies on headless RHEL. When running the tool with --no-sandbox option for my profile, the tool stalls at "Logging in with profile <my_profile_name>". Can't run in debug or GUI modes, since I SSH to a headless RHEL VM.
Tried DEBUG=aws-azure-login ./aws-azure-login --profile <my_profile_name> --no-sandbox and the tool creates the URL for the login page correctly, however it keeps returning "State not recognized endlessly".
I am sure the URL is correct because when I copy it to my local machine's web browser, it returns the page where I can select my roles successfully.
On a side note, when running the DEBUG=xxx, the tool does not ask for the password, so I hard coded the password in the config file, but haven't worked either.

As we had a similar issue; it seems MS AAD tends to inject a loop with a basic auth interception on the corporate sts domains ... monkey patching the authenticate injection on page worked for us.
I sadly could not yet come up with a proper interception in the states loop due to lack of puppeteer knowhow.

Same problem here on centos and ubuntu. with DEBUG=aws-azure-login aws-azure-login --no-sandbox -m cli what I got shows below.
@mowiena , I never get any password input, with or without DEBUG=xxx. could you share your command with parameter? Thanks.
aws-azure-login Error when running state "username input". Error: Execution context was destroyed, most likely because of a navigation.. Retrying... +107ms
aws-azure-login State not recognized! +1ms
aws-azure-login State not recognized! +1s

As we had a similar issue; it seems MS AAD tends to inject a loop with a basic auth interception on the corporate sts domains ... monkey patching the authenticate injection on page worked for us.
I sadly could not yet come up with a proper interception in the states loop due to lack of puppeteer knowhow.

Could you show me some details how you manage to make it work please? I am pretty sure we have the same issue here. Really appreciate it.

I had to build the functionality from scratch using python+splinter

@mowiena thanks a lot. I should check your repo before asking.

It still doesn't work for me... I'm running this on RHEL too (technically Amazon Linux 2023 - same as RHEL)


[BUG] ISM Plugin - Policy not apply to all concerned indices

Describe the bug
Hi everybody, I don´t know if this is a bug or not, but this is my problem:
I have an index_patter "my-index-*"
I have created a Policy in order to delete indices older than 2 weeks:
{
    "policy_id": "my-policy",
    "description": "Cluster Policy in oder to delete indexes that are older than two weeks",
    "last_updated_time":<PHONE_NUMBER>577,
    "schema_version": 13,
    "error_notification": null,
    "default_state": "Running",
    "states": [
        {
            "name": "Running",
            "actions": [],
            "transitions": [
                {
                    "state_name": "Delete",
                    "conditions": {
                        "min_index_age": "14d"
                    }
                }
            ]
        },
        {
            "name": "Delete",
            "actions": [
                {
                    "retry": {
                        "count": 3,
                        "backoff": "exponential",
                        "delay": "1m"
                    },
                    "delete": {}
                }
            ],
            "transitions": []
        }
    ],
    "ism_template": [
        {
            "index_patterns": [
                "my-index-*"
            ],
            "priority": 100,
            "last_updated_time":<PHONE_NUMBER>009
        }
    ]
}

Now let´s say I have 10 indices:
my-index-namespace01-2020-05-12
my-index-namespace02-2020-05-12
my-index-namespace03-2020-05-12
my-index-namespace04-2020-05-12
my-index-namespace05-2020-05-12
my-index-namespace06-2020-05-12
my-index-namespace07-2020-05-12
my-index-namespace08-2020-05-12
my-index-namespace09-2020-05-12
my-index-namespace10-2020-05-12

how can be that under Managed Indices just the first two are shown as managed by the policy?
Expected behavior
All the indices under "my-index-*" should be managed by the policy
OpenSearch Version
1.3.2, Helm Chart v1.11.1
Dashboards Version
1.3.2, Helm Chart v1.5.1
Thanks in advance to everyone who will have a look into it.

Hello @sevenval-admins, did you create the policy after the indices or the indices after the policy? If the policy was created after, then it will not be applied to the older indices.

If what you described is not related to the above, would you be able to provide some more information on how to reproduce it? I tried with 1.3.2 OpenSearch and when the indices were created after the policy, the policy was always applied successfully. Was there anything else special about those indices?

The Policy was created after, problem is solved, thanks to all you guys.

The Policy was created after, problem is solved, thanks to all you guys.

Hi sir
I have same yout poblem. How did you do

@downsrob So I need to apply the policy manually to the older indices. Are there any other ways to apply the policy to all older indices?

@downsrob please answer since there's no official opensearch curator


Pagination component generates react warning on first and last component

Get a react warning when Pagination reaches first or last page.
The bug in console from react:
"Expected onClick listener to be a function, instead got false."
"If you used to conditionally omit it with onClick={condition && value}, pass onClick={condition ? value : undefined} instead."
Steps to reproduce the bug

Use the Pagination component and look in console.
Click on first or last page.
Will happen each time activePage < 1 or activePage > pages

Expected behavior
Expect not to get any warnings from component.
Specifications

Version: 0.6.0
Browser: Chrome
OS: Windows

Additional context
Add any other context or screenshots about the bug.

Fix will probably be changing from conditionally omit to ternary in these two places:
https://github.com/equinor/design-system/blob/develop/libraries/core-react/src/Pagination/Pagination.jsx#L117-L120
https://github.com/equinor/design-system/blob/develop/libraries/core-react/src/Pagination/Pagination.jsx#L158-L161
Before:
activePage > 1 && ((event) => { onPageChange(event, activePage - 1) })

To:
activePage > 1 ? ((event) => { onPageChange(event, activePage - 1) }) : undefined

Thanks for reporting @oanylund !
We'll try to have a look at it asap.

Thanks for bringing this to our attention! I remember seeing Kent C. Dodds warn about using the conditionally omits instead of ternary operators – so this obviously proves his point.


Extract Dom Extractors for the state Result

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


LFS-1061 & LFS-1062: Files with URL special characters in their name can be uploaded but not downloaded or deleted

LFS-1061: Files with URL special characters in their name can be uploaded but not downloaded
LFS-1062: Files with URL special characters in their name can't be deleted
To test LFS-1061:

create a new questionnaire with one question of type file, or use Somatic variants in lfs runmode
create an instance of the questionnaire
locally create files containing special URL characters, e.g.: "test#1.csv", "final_results(?).txt"
upload the files to the file question
navigate to the subject chart or to the view mode of the form and try to download the files

Note: ? appears to be replaced with a different character depending on the browser in the name of the downloaded file (_ in Chrome, space in Firefox).
To test LFS-1062:

create a new questionnaire with one question of type file, or use Somatic variants in lfs runmode
create an instance of the questionnaire
locally create files containing special URL characters, e.g.: "test#1.csv"
upload the files to the file question, save
check .deep.bare.json for the form and see the file node is there
delete the answer
check .deep.bare.json for the form and see the file node has been removed (before these changes, only the value would be erased, while the file node woud STILL be there).

Another (unrelated) issue: trying to upload an empty file silently fails, the file isn't stored, but the request succeeds and the answer value is updated.

Also ', '%', '[' and ']' are causing the problem for 1061 and 1062. This is logged in LFS-1005.

If the above reported problems are going to be addressed in LFS-1005, I can approve this PR.


Setting selected property manually

Setting the selected property of a selectability drop down through jquery does not propagate to the drop down display until it is opened again.

Ultimately, it doesn't appear that there's a good (portable) way to handle this. jQuery's .val() method changes selectedIndex, which is impossible to observe reliably.
The recommended method is to use the following snippet:
$('select').val(foo).trigger('change')

Selectability will respond to the 'change' event and update accordingly.


Normalisation issues when extracting SDF values with ShapeNetCore V2

Hi, thank you for this awesome library! I have a question regarding the extraction of SDF values. From the example file you provide:
# Generate random points in the volume around the shape
# NOTE: ShapeNet shapes are normalized within [-0.5, 0.5]^3
p_vol = (np.random.rand(num_vol_pts, 3) - 0.5) * 1.1

I checked a few objects from the ShapeNetCore V2 dataset and noticed that they are not always bounded within [-0.5, 0.5] ^ 3. For example, the object 02942699/1ab3abb5c090d9b68e940c4e64a94e1e is within [0.115, 0.229, 0.56] and [-0.115, -0.171, -0.326]. Here's a picture of it:

It's hard to see, but the object slightly exceeds the upper face of the sampling volume. Also, it is not centred at 0.
Because papers that use SDF values for 3D reconstruction as DeepSDF suggest to normalise the object so that it fits a unit sphere, does it make sense to change the normalisation strategy to reflect that? Thank you.

Good catch! I didn't realize all the shapenet models weren't normalized to [-0.5, 0.5]^3. If you have the dataset downloaded, you could find the actual bounding box by taking max/min over each bounding box. If you do this, I can update the code. Something like
import os
import numpy as np
import point_cloud_utils as pcu

shapenet_root = "path/to/shapenet"
bbox_min = np.array([np.inf]*3)
bbox_max = np.array([-np.inf]*3)
for cat_name in os.listdir(shapenet_root):
    for shape_name in os.listdir(os.path.join.join(shapenet_root, cat_name):
        v, f = pcu.load_mesh_vf(os.path.join(shapenet_root, cat_name, shape_name, "model.obj")
        bbox_min = np.minimum(v.min(0), bbox_min)
        bbox_max = np.maximum(v.max(0), bbox_max)
print(f"ShapeNet range is {bbox_min} to {bbox_max}")

Thank you for providing the code! I had to use my own library because of the compilation errors on my Mac that we discussed in other Issues, but I used a similar script to the one you suggested. The bounding box is the following:
ShapeNet range is [-0.895247 -0.942603 -0.940616] to [0.827457 0.955908 0.851479]

It seems like a volume defined within [-1, 1]^3 should cover all the shapes.

Following up on my previous comment: once we solve #50 , it could be useful to reason around this normalisation task. I would change the bounding box definition as:
# Generate random points in the volume around the shape
# NOTE: ShapeNet shapes are normalized within [-1, 1]^3
p_vol = np.random.rand(num_vol_pts, 3)*2 - 1

(extracted on my M1!)
This way the bounding box wraps all the objects correctly in the ShapeNetCore dataset. But (please correct me if I am wrong) reconstruction models based on signed distances like DeepSDF could suffer in performance due to the very limited portion of the volume that the object actually occupies - I think this is why papers based on DeepSDF always normalise the objects so that they fit a unit sphere.
However, normalising objects in the range[-1, +1] makes reconstruction in the real world much harder, e.g. in robotic perception tasks. That is, by normalising inidividual objects within a [-1, +1] range we are effectively scaling each object by a different value. In real life, at inference, we are collecting a partial point cloud on the real object surface to infer a latent vector for the corresponding shape. This step is not possible if the DeepSDF model is trained on normalised objects, as we wouldn't know the scaling value to apply to the point cloud collected in the real world - as we don't know what object we are trying to reconstruct in advance.
I hope it makes sense and thank you for reading this. I'd love to hear your opinion about that!

You are right that normalizing will destroy object scale which is not what you want.
One strategy is to sample sample points within [-1, 1]^3 but at different densities. For example, one could sample the first N points from by perturbing points on the surface by Gaussian noise with varianbce proportional to a fraction of the bounding sphere, then sample the next N  by adding Gaussian noise with 2x the previous variance and so on.
In this case, the shapes are still normalized in [-1, 1]^3 but the sampling adapts to the shape.
@maurock could you give this a shot and see if it helps you?

Thank you very much, I will give it a try and close this issue.
Also, I have just sent you an email, I'm looking forward to hearing from you!


Handling of init.py Appears Incorrect

I am seeing D104 errors on __init__.py.
./src/example/__init__.py:1 at module level:
        D104: Missing docstring in public package
./src/example/templates/__init__.py:1 at module level:
        D104: Missing docstring in public package

Version of pydocstyle: 6.1.1

Why does this appear incorrect to you?

The more recent versions of pydocstyle are aware of __init__.py. I have no config settings, so it should go to the default and print D107. That code I can then add to an ignore line in the config. But I'm checking everything using the defaults first and found this error returned.
Am I mistaken about how pydocstyle should work in this case? If I am mistaken, under what conditions would a D107 be printed?

Forgive me, I see that there is a differentiation between public module and public package with public package being __init__.py.


change curve at Runtime through inspector doesn't work.

Godot version
Godot_v4.1.1-stable
System information
Win10
Issue description
change curve at Runtime through inspector doesn't work.
Steps to reproduce

new script

@export var curve : Curve
func _process(delta):
	if Input.is_action_just_pressed("ui_accept"):
		print(curve.point_count)

select new curve in the inspector and a arbitrary preset
run the scene, and notice that adding point in the inspector doesn't effect the print result.

Minimal reproduction project
N/A

I found that if I add @tool at the top, curve would be change in runtime, why is that?

after some digging, I found that when I add @tool and use a Curve resource in the disk, it would change in game when I modify the curve, also the signal "property_list_changed" and "changed" can only be listened in tool mode. So the problem become how to emit a signal from engine side which can be listened from non-tool mode?

Ok, looks like someone fixed it in main branch, so forget about it.


Physics Velocity Limits

fixes #14965

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15158/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15158/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15158/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15158/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15158/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15158/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15158/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15158/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15158/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15158/merge/testResults/webgpuplaywright/index.html

Can you update the havok wasm here as well?

It's was already udpated with these functions exposed.


[chore]  Clarify case normalization in db spec

Fixes https://github.com/open-telemetry/semantic-conventions/issues/886
Changes
Adding note about case normalization to following attributes:

db.collection.name
db.namespace
db.operation.name

The added note suggested in  https://github.com/open-telemetry/semantic-conventions/issues/886

It is RECOMMENDED to capture the value as provided by the application without attempting to do any case normalization.

Merge requirement checklist

[x CONTRIBUTING.md guidelines followed.
[ ] Change log entry added, according to the guidelines in When to add a changelog entry.

If your PR does not need a change log, start the PR title with [chore]

[ ] schema-next.yaml updated with changes to existing conventions.

I think probably needs a change log

CI fails with:
34s
Run make markdown-link-check
  make markdown-link-check
  shell: /usr/bin/bash -e {0}
fatal: No names found, cannot describe anything.
semantic-conventions@ /home/runner/work/semantic-conventions/semantic-conventions
└──<EMAIL_ADDRESS>

  ERROR: 1 dead links found in ./docs/attributes-registry/db.md !
  [✖] https://docs.datastax.com/en/cassandra-oss/3.0/cassandra/dml/dmlConfigConsistency.html → Status: 404
make: *** [Makefile:47: markdown-link-check] Error 1
Error: Process completed with exit code 2.

That link works from my browser - and the PR doesn't really change anything around it. Strange. Trying to figure this out.


Not compatible to the latest version of EnergyPlus

The current version is not compatible to the latest version of EnergyPlus (https://github.com/NREL/EnergyPlus/releases/tag/v9.1.0)

@antoine-galataud  Thank you for the patch for EP 9.1.0. I've committed it to the master branch. Please let me try on my Ubuntu 16 environment.
BTW. Don't you think the shar file of EP 9.1.0 for Linux (EnergyPlus-9.1.0-08d2e308bb-Linux-x86_64.sh) is corrupted ? I mean when I feed it to bash, the code was installed under /usr/local instead of /usr/local/EnergyPlus-9-1-0 because tar image included in the shar file is created under /usr/local/EnergyPlus-9-1-0. How did you solved it ?

@takaomoriyama I've faced the same problem. I haven't solved it yet, just applied a simple workaround: give /usr/local/EnergyPlus-9-1-0 when prompted so it's installed correctly. Symbolic links fail to install though, but it's not preventing it from running correctly.
I was planning to create an issue on NREL side, but didn't have time yet.

@takaomoriyama the issue has already been reported: https://github.com/NREL/EnergyPlus/issues/7256
I guess the fix will apply to 9.2.0

Thanks for the info.
I created a tentative patch https://github.com/IBM/rl-testbed-for-energyplus/blob/master/EnergyPlus/EnergyPlus-9.1.0-08d2e308bb-Linux-x86_64.sh.patch

@takaomoriyama I tested your patch and I confirm it fixes the install issue. Thx.


[Clean up] Remove the shouldShowOfflineMessage parameter from OptionsSelector

Problem
The shouldShowOfflineMessage parameter has been used only in one use case which has now been deprecated.
Solution
Remove the parameter

Done here already https://github.com/Expensify/App/pull/11173


docs: Sprint 20 docs updates

Structure
One idea for Docs is:
# GET STARTED
Overview
Quickstart (should be expanded, and link to most other sections)
Installation

# BASIC USAGE
Overview (should emphasize API server usage)
.cortexrc
model.yaml
Data Folder (covers where Cortex stores its files)
/libraries
    cortex.js
    cortex.py
/Model Sources
    Huggingface
    Cortex Hub
/Engines
    llama.cpp
    TensorRT-LLM
    ONNX

# CLI
...

Tasklist

[ ] Install and Uninstall instructions

https://github.com/janhq/cortex.cpp/discussions/1114
https://github.com/janhq/cortex.cpp/issues/1030

[ ] Cortex Data Structures

https://github.com/janhq/cortex.cpp/discussions/1115#discussioncomment-10558481

[ ] .cortexrc

What fields are in .cortexrc?
https://github.com/janhq/cortex.cpp/issues/1031

[ ] Stable, Beta and Nightly Builds

https://github.com/janhq/cortex.cpp/issues/1065
Need to document how we have different commands, data structures foe each ones

[ ] llama.cpp Params:

https://github.com/janhq/cortex.cpp/issues/1151
@nguyenhoangthuan99

[ ] Model Folder + model.yaml

https://github.com/janhq/cortex.cpp/issues/1154
@nguyenhoangthuan99

[ ] Logs: where do they go

https://github.com/janhq/cortex.cpp/issues/1153
@nguyenhoangthuan99

@dan-homebrew For Beta and Nightly version, I am not really sure that we should also provide commands for each version, as the users supposedly and encouraged to use the Stable version.

@gabrielle-ong I am transferring this issue to you for Sprint 21.


oathToolkit: 2.6.6 -> 2.6.7

Added update script.
Motivation for this change
Upstream update.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 121491 at 0dc3d793 run on aarch64-linux 1

1 package marked as broken and skipped:

bareos

4 packages skipped due to time constraints:

ceph (ceph-dev)
ceph-client
libceph
qemu_full

6 packages built successfully:

keysmith
krunner-pass
oathToolkit
passExtensions.pass-otp
plasma-pass
rofi-pass

I would rather not. The tarballs from GitLab do not contain a configure script, so the build gets more complicated. (see 483200692b134c3e8ec0a25d38dd3759f7e4f092 or what was removed in cf9bf19bf32f8866b19fca02705e02be204a601d)

but that would make updates way easier.

I modified the derivation to download its source from GitLab here. nix-update does not work. I believe this is because the format of the oath-toolkit's tags is quite awful.
Additionally the build process seems to obtain oathtool's version number from git, which is missing in the GitLab source tarball and oathtool --version prints the following:

oathtool (OATH Toolkit) UNKNOWN
Copyright (C) 2009-2021 Simon Josefsson.
License GPLv3+: GNU GPL version 3 or later https://gnu.org/licenses/gpl.html.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Simon Josefsson.


LXD Network ovs bridge - Error with interfaces ipv6 link-local address - "Only unconfigured network interfaces can be bridged"

https://github.com/lxc/lxd/blob/d59a112ff91d8914ca31ef536d6d9eeddbceaffb/lxd/networks.go#L659-L678
The template below is mostly useful for bug reports and support questions.
Feel free to remove anything which doesn't apply to you and add more information where it makes sense.
Required information

Distribution: Ubuntu
Distribution version: 17.4
The output of "lxc info" or if that fails:

Kernel version: 4.10.0-32-generic
LXC version: 2.0.8
LXD version: 2.12
Storage backend in use: lvm

Issue description
You can't add an interface to a lxd network openvswitch bridge, you get this error :
"Error : Only unconfigured network interfaces can be bridged"
Steps to reproduce

Install a fresh ubuntu then lxd
connect a net interface to a switch, as it get "UP" : for example eth0
Create a lxd network with openvswitch
lxc network create zone2 bridge.driver=openvswitch
Try to add eth0 to the ovswitch with lxc command
lxc network set zone2  bridge.external_interfaces eth0
You get an error :
"Error : Only unconfigured network interfaces can be bridged"

Analyze

This problem comes form those lines
This code detect an ip address, even if not configured ! Because every interface have at least a link-local address !

Workaround

sysctl net.ipv6.conf.eth0.disable_ipv6=1

It's usually a good idea to disable IPv6 on the underlying devices, just to avoid a kernel race condition where it may grab a global IPv6 right before it gets bridged, but yes, the check should be relaxed to allow bridging interfaces that only have an IPv6 link-local device.


Tests fail

writer test fails on writing the nested struct. Seems like the struct is skipped and fieldnummers get mixed up, see float 23.37 became int 23.
go version go1.6.3 linux/amd64
go test
--- FAIL: TestRoundTripStruct (0.00s)
	pickle_writer_test.go:373:
		---EXPECTED:(struct_export_test.Struct1)
		{{meow 23.37 0} 23 42.12 0 0}
		---GOT:(struct_export_test.Struct1)
		{{ 0 0} 23 42.12 0 0}
FAIL
exit status 1
FAIL	github.com/hydrogen18/stalecucumber	0.005s

This fails on 1.7 for me as well, I'll investigate it more soon.

The nested struct appears to not be getting handled properly

It looks like somehow the code is just encoding the outer struct only and skipping the nested one entirely.
I took the pickle output from the test and loaded it using python
pickle.loads(base64.b16decode('80027d2858090000004c61776e6d6f7765724a1700000058090000004475706c69636174654740450f5c28f5c28f752e'.upper()))
{u'Lawnmower': 23, u'Duplicate': 42.12}

In previous versions of go, the embedded structs fields basically just showed up as fields of the parent.
Now, the embedded struct is its own field.

This should be fixed  in cd9ec28f35da4da94a4bc35ac94d7427bc5b2c78 let me know if you have any more problems.

Great thanks!


Multidomain support for all models

@kazuakiyama This is the start of expanding all models to handle multidomain stuff intrinsically.
There are a few things that still need to change and if you want to try this out you will need to use
https://github.com/ptiede/ComradeBase.jl/pull/46
The interface isn't too different at a high level but for models that are intrinsically only defined in the spatial dimension
you can add frequency and time dependence by effectively adding tags on the model parameters.
For instance a blurred ring model can be made frequency dependent as follows
r0 = 1.0
index = 1.0
f0 = 230
m = smooth(modify(Ring(), Stretch(TaylorSpectral(1.0, 1.0, 230)), TaylorSpectral(1.0, 1.0, 230))
x = range(-5.0, 5.0, length=512)
g = RectiGrid((X(x), Y(y), Fr([86, 230, 345])))
intensitymap(m, g)

which will produce a (X, Y, Fr) image cube. The way this all works is that model parameters should now be extracted using the @unpack_param macro, which parses to the getparams function in ComradeBase.
For instance the mring model in the visibility domain is now defined as
@inline function visibility_point(m::MRing{T}, p) where {T}
    @unpack_params α, β = m(p)
    k = T(2π) * sqrt(u^2 + v^2) + eps(T)
    vis = besselj0(k) + zero(T) * im
    θ = atan(-u, v)
    @inbounds for n in eachindex(α, β)
        s, c = sincos(n * θ)
        vis += 2 * (α[n] * c - β[n] * s) * (1im)^n * besselj(n, k)
    end
    return vis
end

this will evaluate alpha and beta at the specific time and frequency values passed.
Note that for models that are natively frequency dependent, e..g, the scattering model, nothing should have to change.
So why did I take this approach? It was very simple to implement and worked without extensive changes to the code base. However, this is a proposal so we should brainstorm before I merge this!

Note the tests are failing because I am not using the specific branch of ComradeBase

@erandichavez, this will potentially impact some of your terrific work on multifrequency imaging. If you have a chance in the new year, reviewing this PR and checking if the implementation is sensible would be great.
This shouldn't impact your multifrequency branch other than I tweaked the implementation of the TaylorSpectral model a bit. The benefit is that this PR will allow any model to automatically work with multifrequency imaging with essentially only a single change to their code.

@kazuakiyama I consider the implementation essentially complete now. The remaining items are to add additional multifrequency and time variable tests, but I think this and Erandi's branch solves essentially every open item in #53.
I want to merge this sometime early in the new year, so getting some eyes in early 2025 would be super helpful.
P.S. I believe this branch is essentially functional right now, so if you want to try this for some BHEX multifrequency geometric modeling, that would be great!

Also, just as a fun side effect, this should make multifrequency geometric modeling possible immediately.


[Bug]: Opening the extension popup.html using will return new empty popup without any data how to get popup.html within the context of a specific tab/website

Minimal, reproducible example
const puppeteer = require('puppeteer');
const browser = await puppeteer.launch({
        // launch the browser in the headful mode
        headless: false,
        // add the extension to the browser
        args: [
        `--disable-extensions-except=${pathToExtension}`,
        `--load-extension=${pathToExtension}`,
        ],
    });
    await sleep(2000)
    // open a new browser tab
    let page = await browser.newPage();

  await page.goto('https://www.example.com/');
  await page.bringToFront();
  await page.goto(`chrome-extension://${extenstionID}/html/popup.html`)

Error string
will open new seprate page which is new empty page
Bug behavior

[ ] Flaky
[ ] PDF

Background
is there any work around to get the resultant produced by specific chrome extenstion on specific tab/website. resultant can be in form of popup in from of html should we any changes in manifest v3 file to get the data that should also be fine.
just need the resultant produced by extenstion on specific tab programticaly
Expectation
await page.goto(chrome-extension://${extenstionID}/html/popup.html) page should be within the context of a specific tab/website
Reality
popoup.html file used is not in  the context of a specific tab/website
Puppeteer configuration file (if used)
No response
Puppeteer version
22.6.5
Node version
v18.18.1
Package manager
yarn
Package manager version
1.22.21
Operating system
Linux

Here is an example of how to open an extension popup using Puppeteer https://github.com/oliverdunk/puppeteer-extension-experiments/blob/main/src/index.ts#L13

great the above code opens the pop up as expected.thanks


fix Permission denied without privileged flag

As a server admin I want to run the minecraft server without privileged flag.

The error was related to ubuntu 16.04 lts with recent docker + outdated seccomp profile - no idea why outdated, so there are multiple ways to come around.
How to check:
Using the official default seccomp profile from here: https://github.com/moby/moby/blob/master/profiles/seccomp/default.json
Tested it in docker run like this, just used a modpack image:
docker run --rm -it --security-opt seccomp=/path/to/seccomp/profile.json hello-world
Possible Fix:

Add the option to every invocation
Modify docker deamon config to use this default seccomp profile
Do a dist upgrade


Extension interferes with Brackets-File-Icons

https://github.com/drewbkoch/Brackets-File-Icons
If I use this extension along with your themes extension the icons for the Brackets-File-Icons stop rendering and become simple rectangles with an X in the middle:

As soon as I disable your extension this stops happening.  Any idea why?  I really like both extensions but can't use them together at present.

FWIW the maker of that other extension tells me he runs 2.0.10 of this extension w/o issue.  I am on 2.0.11 and do have this issue.

Hi @mfromin I'm not running into the same error, could you let me know the following:

What Theme you are using?
What Operating System?
What Brackets Version?

For example I am running:

Themes for brackets rubyblue
Ubuntu 15.04
Brackets 1.3

Theme:  Themes for Brackets Obsidian
OS: Mac OS X 10.10.3
Brackets:  1.3
If I simply uninstall Themes for Brackets and leave all other extensions in place this issue no longer happens.

Do you receive any errors in the console? Debug - Show developer tools I ufortunately do not have a mac so I cannot test this :cry:

I see one error related to the Bower extension (file permissions for a log) but nothing else.  I would note this also seems to mess with the icon that Brackets Git wants to put at the top of the Working Files list...

Notice the left icon (next to the cog wheel) is a smaller version of the same "rectangle X" icon.
Is there a way to get the prior release of your extension (2.0.10) to see if that eliminates the issue?

https://github.com/Jacse/themes-for-brackets/releases
I tagged some releases so you can download and install the zip.

Also see if this version of the file icons works https://github.com/AdriVanHoudt/Brackets-File-Icons/archive/1.3.zip
It is one of the pull requests that fixes a small error for the icons extension.

I know what is causing this now…
I wanted to change the font/size of the file names on the left side of Brackets.  I added this CSS to stuff.css:
.working-set-header-title,
.title,
#sidebar * {
font-size: 14px;
font-family: 'SourceCodePro-Medium', 'MS Gothic', monospace;
}
…and boom - things go bad for the icons.  Any idea how I can change the font & size using CSS that won’t cause this to blow up?
BTW - I really appreciate your help with this issue.  Bracket’s customizability is one of its best features but I appear to be good at breaking things!
--Michael

On Apr 30, 2015, at 9:57 AM, Eric J<EMAIL_ADDRESS>wrote:
https://github.com/Jacse/themes-for-brackets/releases https://github.com/Jacse/themes-for-brackets/releases
I tagged some releases so you can download and install the zip.
—
Reply to this email directly or view it on GitHub https://github.com/Jacse/themes-for-brackets/issues/125#issuecomment-97850551.

Just ditch the font-family
.working-set-header-title,
.title,
#sidebar * {
    font-size: 14px;
}

I actually wanted to change both the font and size...I don't care for the default font (getting old sucks).  After some playing this worked:
.working-set-header-title {
font-size: 14px;
font-family: 'SourceCodePro-Medium', 'MS Gothic', monospace;
}
.title * {
font-size: 12px;
font-family: 'SourceCodePro-Medium', 'MS Gothic', monospace;
}
#sidebar {
font-size: 14px;
font-family: 'SourceCodePro-Medium', 'MS Gothic', monospace;
}
Note the sidebar no longer has the asterisk (*) in the CSS.  This seems to solve the issue!


Google Analytics user opt-out feature

Hi!
Would it be possible to somehow allow users to opt-out from Google Analytics information collection? I have found this bit of code on Google Analytics webpage but I have no idea how to implement this myself. I am not sure that just by adding it before the gtag() function as they say would do the trick because I do not understand how would it know which user has opted out.
Thanks!

There are many third party services that offer this (GDPR-related) functionality and they can be integrated via the custom head/JS capability (see Customization page of the docs).
Alternatively the code can be edited to add your modifications or tracking disabled for all users of the site in params.toml.
It's not well suited to attempting to add dynamic tracking/cookie functionality into core Academic.

Sorry, I am struggling to add a custom head to insert a third-party cookie provider element (opt-in message with different privacy options) to my website. I have tried out to simply add the provided html script <script src="https://cmp.osano.com/example/example/osano.js"></script> to the partials folder under custom_head.html. Sadly, it the opt-in message does not appear on the website. Could someone help me out?


Exception on reload

(Required) What is your Quests version / build number (do not say "latest")?
3.0.1-b174
(Required) Please describe the problem and how to replicate it. Any console errors?
Run /questadmin reload with attached quests.yml
[10:28:03] [Server thread/INFO]: Wahrheit issued server command: /questadmin reload
[10:28:03] [Server thread/INFO]: [PLAYERCOMMAND} Wahrheit: [4374 64 543] => "/questadmin reload"
[10:28:03] [Server thread/INFO]: [Quests] There are new language phrases with the current version. They have been stored in /lang/en_new.yml
[10:28:03] [Server thread/ERROR]: null
org.bukkit.command.CommandException: Unhandled exception executing command 'questadmin' in plugin Quests v3.0.1-b174
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46) ~[server.jar:git-Spigot-edd0396-2a927e8]
	at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:141) ~[server.jar:git-Spigot-edd0396-2a927e8]
	at org.bukkit.craftbukkit.v1_12_R1.CraftServer.dispatchCommand(CraftServer.java:641) ~[server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.PlayerConnection.handleCommand(PlayerConnection.java:1386) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.PlayerConnection.a(PlayerConnection.java:1221) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.PacketPlayInChat.a(PacketPlayInChat.java:45) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.PacketPlayInChat.a(PacketPlayInChat.java:1) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.PlayerConnectionUtils$1.run(SourceFile:13) [server.jar:git-Spigot-edd0396-2a927e8]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_131]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_131]
	at net.minecraft.server.v1_12_R1.SystemUtils.a(SourceFile:46) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.MinecraftServer.D(MinecraftServer.java:748) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.DedicatedServer.D(DedicatedServer.java:406) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.MinecraftServer.C(MinecraftServer.java:679) [server.jar:git-Spigot-edd0396-2a927e8]
	at net.minecraft.server.v1_12_R1.MinecraftServer.run(MinecraftServer.java:577) [server.jar:git-Spigot-edd0396-2a927e8]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_131]
Caused by: java.lang.NullPointerException
	at me.blackvein.quests.Quest.updateCompass(Quest.java:168) ~[?:?]
	at me.blackvein.quests.Quester.checkQuest(Quester.java:2384) ~[?:?]
	at me.blackvein.quests.Quests.reloadQuests(Quests.java:1928) ~[?:?]
	at me.blackvein.quests.Quests.adminReload(Quests.java:662) ~[?:?]
	at me.blackvein.quests.Quests.questAdminCommandHandler(Quests.java:617) ~[?:?]
	at me.blackvein.quests.Quests.onCommand(Quests.java:606) ~[?:?]
	at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) ~[server.jar:git-Spigot-edd0396-2a927e8]
	... 15 more

(Optional) Can you provide an example quest from quests.yml using Pastebin.com?
https://pastebin.com/zyMDX5fu

https://www.spigotmc.org/threads/quests.47772/page-53#post-2614457
This user also can't run reload, but didn't provide much information.

Continued from #102, possibly related to #100
@SXRWahrheit Would you mind reposting your example quest? The Pastebin expired.

This one will last a month: https://pastebin.com/fiiHJLMP

Although I can't replicate the reload error, I did notice this in the startup log:
[13:41:53 ERROR]: [Quests] block-start: for Quest Cavernous Jaunt is not in prop er location format! [13:41:53 ERROR]: [Quests] Proper location format is: "WorldName x y z"
That said this is very likely a result of the fix for #100. Hopefully amending my fix will resolve both issues.

    block-start: prosperus 29.0 24.0 -84.0

I used the in-game editor to create that quest, too.

It seems I spoke too soon. In my rush, I simply misspelled the world name in my test environment.
Cannot replicate. I've tried /questadmin reload with and without a compass and in different worlds.
Please try this build, attempt to reproduce the error, and post back the debug messages from console:
https://www.dropbox.com/s/zgdlolaeu1be54u/Quests-3.0.1.jar?dl=0

Will give it a go tonight - internet at work is unstable right now.

Got to test earlier than anticipated. It looks like that fixed it! Now where can I grab a version that doesn't spam my logs with debug info? :P

Next dev build will contain this fix, minus the debug messages. Thanks for the report!
Unrelated to #100


Unreliable SD Card Initialisation

Right now I'm not 100% sure if this is a hardware fault due to me using my SD card slot as a ground connection for an oscilloscope, or an issue with how/when the SD card is initialised.
It seems to be the case that I can insert an SD card and have it work fine through device resets, but as soon as it is removed, or the battery is disconnected it will fail to work until I undergo an arcane ritual of slowly inserting the SD card while repeatedly resetting.
A vague guess is that something is preventing the card from entering SPI mode during the normal start-up procedure, and this can be fudged by not-quite-inserting-the-card. It looks like SPI mode requires the CS signal to be asserted (active low) during start up. Since the card will not exit SPI mode until a power cycle (which reset does not cause) it will work fine until power is otherwise lost.

It looks like dropping the SD card SPI interface speed from a 4X prescalar to an 8X prescalar has a huge (positive) impact on compatibility. I've made this change temporarily, pending a more correct fix to the SD card init.
Right now the prescalar change will half SD card speed (more or less) and it really feels quite sluggish. Since I've experienced the faster speed by jockying my SD card as described above, I'm confident we'll find a way to get back to 4X generally.

Not entirely correct speed change: https://github.com/Daft-Freak/32blit-beta/commit/1079793d99846c8924430ee3902778d5e3f83789

Probably don't need to switch back to the faster prescaler now that the SPI code is not awful. Hmm, the inefficiency there may have been the only reason that worked as it's waaaaay out-of-spec (120Mhz!).

I'm going to say that this was "fixed" by the various SD card optimisations making the faster clock unnecessary... and also that this issue was that clock being too fast, so reducing it wasn't a workaround, but correct.


translated CONTRIBUTING.md in korean

I translated this file into Korean so that many Koreans could be interested.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Would it be better to put the different translation to a separate file and name the file with a language suffix instead of mixing them together?
eg. CONTRIBUTE_cn.md(or _zh) for Chinese.

Thanks for contributing, we almost there, please keep the default English version as it is(do not delete it), and add a link for the translated version(_LANG-SUFFIX.md) to it somewhere( maybe at the beginning after the title?) , so developers can easily select the version they want.
I prefer this way.  How do you think? @taewankang @lianglia-apollo


feat(ci): move sqlsmith frontend to cron (IGNORE)

Trying different approach for conditional workflow from #4134. This PR won't be merged. Just to test things on ci. If it works well, rebase #4134 on it.

This approach works. Following up in main PR: #4134


Where can I contribute translations?

I want to contribute a translation. Is there a way to do so?

The language files are in this repository: https://github.com/Azanor/thaumcraft-beta


Image Component no working?

Environment
ios11.2  iponeX react-native0.55.3
Description
<Image
style={{width:200,height:300}}
source={{uri:'http://nimg-ts.jicai.com/image/mobile_img/banner/862f26c6-d673-4322-8c0e-6beac6d57a3c.jpeg'}}/>
Steps to Reproduce
no working? it is ok in Android,but do not work in iOS. why?

Same issue: RCTImage does not render anything
Environment:
  OS: macOS High Sierra 10.13.5
  Node: 8.11.1
  Yarn: 1.7.0
  npm: 6.1.0
  Watchman: 4.9.0
  Xcode: Xcode 9.4 Build version 9F1027a
  Android Studio: 3.1 AI-173.4720617

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: 0.55.4 => 0.55.4

@MacKentoch
I checked the image you are using and it does not exist.
Try this:
<Image
     source = {{uri: 'https://reactjs.org/logo-og.png'}}
     style = {width: 400, height: 400}}
/>

OMG I'm so...
I don't use https://facebook.github.io/react/img/logo_og.png but the uri I use does not work too.
Thank you @betiol , I feel like 😅(and gonna hide under my desk for the rest of the day 😂)

Haha happens. 😄

Closing as the original author did not respond with details.


Spyder won't launch

Issue Report Checklist

[ +] Searched the issues page for similar reports
[ +] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ +] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ +] Restarted Spyder
[ +] Reset preferences with spyder --reset
[ +] Reinstalled the latest version of Anaconda
[ +] Tried the other applicable steps from the Troubleshooting Guide

[ +] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
I just downloaded the latest version of Anaconda from anaconda website and installed it. When I start Anaconda Navigator, it takes 6 minutes to launch. When I click on Spyder, it won't launch. I just see in the beginning a couple of command prompt windows opening and closing very quickly and that's it.
Uninstalled and installed Anaconda again - didn't work.
Ran spyder --reset in Anaconda prompt, and the notification popped up "Python has stopped working".
Tried conda update spyder, but I see an error:
Collecting package metadata (current_repodata.json): failed

CondaHTTPError: HTTP 000 CONNECTION FAILED for url <https://repo.anaconda.com/pkgs/main/win-64/current_repodata.json>
Elapsed: -

An HTTP error occurred when trying to retrieve this URL.
HTTP errors are often intermittent, and a simple retry will get you on your way.

If your current network has https://www.anaconda.com blocked, please file
a support request with your network engineering team.

SSLError(MaxRetryError('HTTPSConnectionPool(host=\'repo.anaconda.com\', port=443): Max retries exceeded with url: /pkgs/main/win-64/current_repodata.json (Caused by SSLError(SSLError("bad handshake: Error([(\'SSL routines\', \'tls_process_server_certificate\', \'certificate verify failed\')])")))'))

I can access https://repo.anaconda.com/pkgs/main/win-64/current_repodata.json and https://www.anaconda.com from the browser.
I added Anaconda, python and spyder to the list of apps allowed through Window Firewall.
This is a company computer, however, everything's working fine for two of my colleagues, and I am the only person for whom it's not working.
What steps reproduce the problem?

Opening Anaconda Navigator
Clicking on Spyder

What is the expected output? What do you see instead?
Launch Spyder
Instead nothing happens
Versions

Spyder version:   3.3.6
Python version: 3.7.4
Qt version: 5.9.7
PyQt version:   5.9.2
Operating System name/version: Windows 10 Enterprise, version 1607

Hi @maiiasleptcova could you try running from an Anaconda prompt spyder --debug-info verbose and paste here what you get? Maybe with that we can get a better understanding of the issue here. Thanks!

thanks @dalthviz , here's the output
Usage: spyder [options] files

spyder: error: no such option: --debug-info

Oh sorry @maiiasleptcova my mistake, since you are running Spyder 3 you need to do the following:

Set the env variable SPYDER_DEBUG to 3 running this: set SPYDER_DEBUG=3
Run Spyder from the console.

That's what pops us:

@maiiasleptcova is anaconda the only python installation that you have? (it seems strange to me the path that's being referenced for the pythonw.exe executable).  Did you install anaconda just for your user or system wide?
Also, a couple of things that maybe could help:

Maybe you can run from a cmd and also from the anaconda prompt where python to check the path that the system is getting Python.
Try to create an env with conda from the anaconda prompt, something like conda create -n spyder spyder=3.3.6 (if it fails, then something beyond Spyder is happening here, and maybe is related to your Python installation itself).

Any new info it's greatly appreciated in order to understand what's happening here :)

@dalthviz yes, anaconda is the only python installation, installed only for one user (me)
from cmd:
INFO: Could not find files for the given pattern(s).
from anaconda prompt:
C:\Users\Sleptcom\AppData\Local\Continuum\anaconda3\python.exe
output from running
conda create -n spyder spyder=3.3.6
is:
Collecting package metadata (current_repodata.json): failed

CondaHTTPError: HTTP 000 CONNECTION FAILED for url <https://repo.anaconda.com/pkgs/main/win-64/current_repodata.json>
Elapsed: -

An HTTP error occurred when trying to retrieve this URL.
HTTP errors are often intermittent, and a simple retry will get you on your way.

If your current network has https://www.anaconda.com blocked, please file
a support request with your network engineering team.

SSLError(MaxRetryError('HTTPSConnectionPool(host=\'repo.anaconda.com\', port=443): Max retries exceeded with url: /pkgs/main/win-64/current_repodata.json (Caused by SSLError(SSLError("bad handshake: Error([(\'SSL routines\', \'tls_process_server_certificate\', \'certificate verify failed\')])")))'))

As before, I can access the links above from the browser without any problem.

@maiiasleptcova maybe your company uses a proxy (and your browser has it configured but such config is missing for your anaconda installation)? If that is the case, maybe you could try to config Anaconda to use it, although, I'm unsure of how that could be causing Spyder to not start :/
@ccordoba12 do you know if having a proxy set in a machine could cause Spyder to being unable to start?

do you know if having a proxy set in a machine could cause Spyder to being unable to start?

It shouldn't be an issue.

This is a company computer, however, everything's working fine for two of my colleagues, and I am the only person for whom it's not working.

I'm sorry @maiiasleptcova but that's all the help we can offer you. Please consult with your IT department because this looks like an issue with your computer and not with Spyder.


Generic opaque types leak concrete type information

Example:
// a.js
export opaque type A<T> = {};

// b.js
import { type A, } from './a.js';

declare var a: A<number>;

(a: A<string>);

This passes, thus leaking the fact that T is not used in concrete type.

@vkurchatkin Afaik 0.49 introduced only parsing opaque types, but not type system. So I guess flow doesn't care it yet.

Interesting! This was fixed in https://github.com/facebook/flow/commit/ca740562b3956f44035cf1f5d0badc66c2bf315b as a side effect of how we changed type checking for declared opaque types.
In case you are curious about why this was happening:
Before we added support for declared opaque types in libdefs, opaque type always carried around their underlying type information. When we instantiated a polymorphic opaque type, we made substitutions in this underlying type.
When we flowed one opaque type to another, we would check to make sure they were the same opaque type (they were both A defined in a.js in this case), and then we would compared their underlying types. Since, in this case, it never appeared in {}, this check would pass.
When we added support for declared opaque types, we had to consider the case where we didn't have an underlying type handy! To handle this, we explicitly keep track of the type params & their polarities. With this change, we saw that the A's were the same opaque type, we explicitly compared their type arguments (instead of their possibly missing underlying types).
Thanks for reporting this!

The fact that we check type args only for opaque types works around this issue, but I'm not sure the underlying cause is fixed. Since we rely on reason locations to determine in-file or out-of-file, an improperly repositioned type might cause opaque types to leak in practice.
Since generics are involved, I'm going to cc @mroch, who has some very long standing WIP to improve positioning for type apps. I don't have any hard evidence, but my gut says it's related.


chore(release): 2.66.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 141704806b95a717fadbd5e085d2cd3a67a6974e
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[q] Repository marked as ineligible for Hacktoberfest

Your query
...
Just checked on the Hacktoberfest progress and they've marked this repository as ineligible. Any ideas why?

Hi there @drentsoft
Unfortunately this a problem with the hacktoberfest system that's affecting PRs that were still under review after hacktoberfest ended. I've contacted the organisers of hacktoberfest about this and they acknowledge its a flaw in their logic but I'm still waiting to see if it will be fixed.
It may help if you report the problem at their email<EMAIL_ADDRESS>and maybe they will do something about it if more people say its affecting them!

Thanks @funbeedev I will do.


Code in apportionment option that apportions all ABC to one area

This should probably be done in the ApportionmentAppliedTo2017Data.rmd file, then the code can be duplicated to the RapportionmentFunctions file for posterity.  Need a function that will take the annual ABC for all areas and apportion to one management area only, and do this for each area in turn.

Done!


Add restyle-diff

Problem
pre-submission restyle is clunky or incomplete
Summary of Changes
try to make this less clunky
restyle-diff runs on a working tree. run it with a single argument: what to diff the current tree against in order to determine what should be run through the restyler

@hawk248 ?


Could not get CLTVersion

if CLT version is over 10.*, it could not get the version.
version = re.match(r'(\d+\.\d+\.?\d*)', version).groups()
and now latest CLT version is 10.14.
https://developer.apple.com/download/more/?=command line tools
https://github.com/nodejs/node-gyp/blob/872ee89a8541264ee2d96e162d410f478bba7815/gyp/pylib/gyp/xcode_emulation.py

I don't think that's your problem @tibbers622:
$ python
Python 2.7.16 (default, Mar 22 2019, 15:22:50)
[GCC 4.2.1 Compatible Apple LLVM 10.0.0 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import re
>>> version = "10.14"
>>> re.match(r'(\d+\.\d+\.?\d*)', version).groups()[0]
'10.14'

Can you give us some error output for a problem you're experiencing with this code? The \d+ should match both 10 and 14 just fine.

Fascinating because GYP3's code incorrectly returns 10.1
https://github.com/refack/GYP3/blob/master/gyp/xcode_emulation.py#L1407

I believe that #1890 fixed this. Please reverify.

@tibbers622 Should this issue be closed?

Closing in favor of #1927.  @tibbers622 If this has not been resolved, please jump in on #1927.


PDF entries in document information dictionary

Case-sensitive key. See PDF32000_2008.pdf reference doc (page 550, Table 317).
Key/Value pair for yaml key and PDF entry key.
Any key not in Table 317 will end up in "Custom Properties" section in the PDF document.

Functionality updated.
Need to read and extract relevant sections from PDF spec and use in documentation.


Pr473

What is this PR ? Looks like a copy of https://github.com/aerogear/aerogear-unifiedpush-server/pull/473 ?

@sebastienblanc copy of #473 + reduced logging on Travis

@kpiwko Ah ok :) Maybe easier if you PR against @edewit existing PR no ? But it's the same in the end

Right, that would make more sense.

@kpiwko hrm -> https://s3.amazonaws.com/archive.travis-ci.org/jobs/50640435/log.txt

@kpiwko @edewit any updates?

@kpiwko could you please update the description of this PR with what it was supposed to be + steps for testing?
I would appreciate.

@kpiwko closing...

@abstractj @matzew sorry, I had to unsubscribe myself from all Aerogear GH repositories to be able to track direct mentions.
I'll try to rebase the commit later this week and make travis happy on https://github.com/aerogear/aerogear-unifiedpush-server/pull/473


Use New Container Registration for Better Setup Experience

So the registration is way better now with the new container work. Helped me rework the SignalR Registration. The only thing I had to do was for AddRequestClient, I had to add a .AddSingleton as well, because I call the request client from a Singleton. Hope that's okay for me to add.

@phatboyg So I added the RequestClient singleton, similar to what you did here. But that's now causing the test to fail.

I ended up ditching the change for adding singleton, and instead injected IClientRequestFactory and used that.

I need to look closer at this given the latest changes to see what I'm missing so that you can use it in these samples. Otherwise, it isn't meeting the goal I set out for it.


Explore pulling traffic and counter data through Trafx

Description:
Determine if there is an API or other mechanism to pull traffic and counter data from Trafx into A&R to help automate data entry process for staff.
Acceptance Criteria:

[ ] Determine if there is an API available
[ ] Explore what is needed to pull/ingest data into our system
[ ] Get a walk through of Trafx from A&R team
[ ] Determine if any data manipulation may be required
[ ] Create future tickets as needed (e.g., ability to correct data once in system)

Development Checklist:

[ ] ...
[ ] ...
[ ] ...

Dependencies

Blocked by
Blocking

Relevant documentation as reference
Definition of Ready

[ ] Acceptance criteria are included
[ ] Wireframes are included (if applicable)
[ ] Design / Solution is accepted by Product Owner (if applicable)
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done

In progress:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] UI meets accessibility requirements
[ ] Unit tests are written
[ ] Work is traceable in GitHub
[ ] PR linked to ticket number
[ ] If needed/required - Dev adds flag/label to highlight any migration steps necessary prior to PROD deployment

Code review:

[ ] Code is peer reviewed and has passed CI/CD tests

QA:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Code is potentially shippable to the production environment
[ ] Functional features have been tested and passed by QA
[ ] UI components tested by designer
[ ] Code is deployed to PROD when moved to 'done' column (unless requested otherwise by PO)

PO Review:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Reviewed and approved by Product Owner

Notes:

This was discussed at the October 11 prioritization session. Additional notes from the conversation:

Explore what is possible
What does it look like to get the API
Determine if any data manipulation is required
Get a walk through of Trafx
Would need a way to correct the data (traffic counters are error prone)

Previous documentation done by RST to link to Trafx api:
p_&_r_visitor_statistics_report_and_dashboard_prototype_technical_document_v2.docx
code is in bcgov-c for now but some of you may have access and uses OpenSearch!
https://github.com/bcgov-c/rvs-card/blob/main/README.md

Previous documentation done by RST to link to Trafx api:
p_&_r_visitor_statistics_report_and_dashboard_prototype_technical_document_v2.docx code is in bcgov-c for now but some of you may have access and uses OpenSearch! https://github.com/bcgov-c/rvs-card/blob/main/README.md

Hey @LindsayMacfarlane!
I don't seem to have access to that first docx document (I made a request on Friday through that SharePoint Access Request screen it sends me to, but no change yet), and I get a 404 for the second link to the repo, so it must be hidden for me 😅
Just wondering if there's another way to access these or if you have the power to share them with me?

@davidclaveau - I've given you access to the word doc - let me know if that didn't work.
@JLWade - are you able to grant us access to this gitHub re. Trafx and RST? It comes up as a 404

https://github.com/bcgov-c/rvs-card/blob/main/README.md

the word docs might be the most useful - I dont think they were able to connect to TrafX in the end but did do some initial research on it

Thanks @LindsayMacfarlane  and @JLWade! I'll look through the doc and report my findings!

Findings:

Trafx does not have an API that we can use for integrating with A&R.
RST team was speculating a setup to import data from Trafx (and other data sources) into AWS OpenSearch to index and report visitor statistics.

This is probably overkill for data from Trafx alone, but could be considered if these other data sources are used for cross-referencing in A&R reporting

I will be speaking with the A&R team later this week about Trafx and how the data is exported/used by POs and entered into A&R.

One possible solution (so far) is a simple import system for .xlsx or .csv files, which would upload and allocate the data from a Trafx export into A&R.

Had a great walkthrough with Nicole, discussed TRAFx and the setup of the system:
TRAFx rundown:

There are several accounts that cover several different regions for BC Parks
Park Operators upload trail data from the TRAFx devices into TRAFx
TRAFx consolidates the data and allows for easy visualization of the data in their dashboard
This data can also be downloaded as an Excel spreadsheet and provides trail information by month and year
If there are several subareas and several trails on top of that, it can be available in a single export

Rough thoughts for implementation with A&R

TRAFx users take a bulk export of trail data and upload the spreadsheet to a designated page on A&R
This designated page sorts the trail data based on the user's access to parks (i.e. can only upload trail data for a park/subarea they have access to)
After uploading this data, A&R standardizes the report and distributes the trail data to their respective subareas
It would be preferred that, before saving this data, the user was asked to confirm the data once more before it's saved to a subarea, i.e. "you are adding X number of attendees to this subarea for Y month" and/or "you are overwriting data here: ________"
We would allows users to manually change these data points before/after it's uploaded, so it would be preferred to flag if the data was uploaded manually or by bulk import (bulkImport == true)

Some underlying issues:

We would most likely encounter issues with importing this information in bulk because there isn't a great way to connect trails with their subareas, outside of doing a string comparison

We could hard-code a data register in A&R to sort trails to their respective subareas, e.g. Buttle Lake trail always goes to Buttle Lake in A&R, but then we're responsible for maintaining that register manually with any changes.
We could also encounter issues with typos/user-input error
Also also, if there was a name change to a trail, we would need to update the trail data register on our end.
If every trail in TRAFx tracked what its subarea ID was in A&R, then a name change would be inconsequential.

Some thoughts and questions about creating a TRAFx importer in A&R. I'll look to create some tickets in the backlog for the team to discuss further!

@davidclaveau - great research and analysis. I'm curious about estimated effort/cost to implement before we move forward with anything.

@LindsayMacfarlane I can't say too much about the cost, but keeping the imported CSVs in an S3 bucket would be a very low cost from what I've been told.
As for the effort, that's probably a bit trickier - I'd be interested in others' input in backlog refinement or wherever we discuss this. To me, it's not a huge feature addition, but it does mean some bigger UI changes on the frontend for the importer landing page and confirmation page, and a fair bit of logic for the importer's data transformation/validation/input and any error handing. Also an update to the UI for showing manual vs. imported data.

Thanks again @davidclaveau for all your research and recommendations. Would love to take this back to Sally, Nicole and team to better understand next steps and how we may want to move this forward.


Establishing connection with username & password

Is there a way to establish a connection to Clickhouse having your username & password set? I tried both
http://username:mypass@myhost:8123
and
http://myhost:8123?user=username&password=mypass
and neither works.

There are username & password config options. Docs should probably have a note about that.


Add a new class to the container, when scrolled to the right with a left-pinned column

I'm submitting a ... (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I'm trying (for example) to drop shadow to the left-pinned columns only when the horizontal offset is greater than 0 (scrolled to the right) . To make this happen, I have to track the changes in the ng-reflect-offset-x attribute, with the CSS regex code as following. And it generates a fair amount of CPU usage to make calculations every single time the ng-reflect-offset-x value changes.
...
.datatable-body-row, .datatable-header {
	&[ng-reflect-offset-x^="1"],
	&[ng-reflect-offset-x^="2"],
	&[ng-reflect-offset-x^="3"],
	&[ng-reflect-offset-x^="4"],
	&[ng-reflect-offset-x^="5"],
	&[ng-reflect-offset-x^="6"],
	&[ng-reflect-offset-x^="7"],
	&[ng-reflect-offset-x^="8"],
	&[ng-reflect-offset-x^="9"] {
		.datatable-row-left {
			.datatable-body-cell:last-child,
			.datatable-header-cell:last-child {
				...
			}
		}
	}
}
...

Expected behavior
If you can add some class like has-offset-x to the ngx-datatable component when the offset-x > 0 (scrolled to the right), it will prevent the unnecessary CPU usage for calculation and make the component more flexible design-wise.
What is the motivation / use case for changing the behavior?
Performance.

I just found out that the ng-reflect- attribute is debugging purposes only. So I can't even do it with the ng-reflect-offset-x property in production. But the feature request is still valid.

Found the scroll output. Closing the issue.


CNF-6516: NTO: Enable PerformanceProfile

What this PR does / why we need it:
As part of the phase-3 of Node Tuning Operator adaptation to run in Hypershift environments ( #1651 ), Performance Profile controller should also be adapted.
Performance Profile controller uses PerformanceProfile as its API.
This PR modifies NodePool controller to allow Performance Profile controller to read PerformanceProfiles manifests using  the same procedure as NTO is already using with Tuned manifests
This PR contains the following changes, which are also outlined in this document openshift/enhancements#1244:

NodePool controller changes:

Modify code to reconcile the spec.TunedConfig field so it would not only reconcile Tuned manifests but also PerformanceProfile manifests. For each NodePool, propagate any Tuned or PerformanceProfile manifests from the ConfigMaps referenced in the spec.TunedConfig field into ConfigMaps in the hosted control plane namespace where NTO, and so Performance Profile controller,  can read them.

Which issue(s) this PR fixes
Fixes : CNF-6291
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[x] This change includes docs.
[x] This change includes unit tests.

/test all

/cc @dagrayvid  PTAL when you have some time
Thanks

/test all

/test all

/test all

rebase over last main changes containing  #1729

/test all

/test all

/test all

/hold
Should only be merged for 4.13

/retest

rebased

/retest

rebasing over main branch due to some renaming  #1802

/retest

/unhold

/retest-required

rebase over main

/retest

rebased

@enxebre @csrwng @dagrayvid   Please take a look whenever you have some time.
Thanks in advance

rebasing

rebase

/retest

/test e2e-aws

/test e2e-aws

@enxebre @csrwng @dagrayvid
Please take a look whenever you have some time.
Thanks in advance

/test e2e-aws

/test e2e-aws

/test e2e-aws

/retest-required

/test e2e-aws

/test e2e-kubevirt-aws-ovn

rebased over latest main and addressed some of the comments
Thanks @jparrill for your revision

/jira refresh

for easier review can you please split vendor changes into its own commit?

/approve

/hold

/lgtm cancel

/hold

/test e2e-aws

just rebased

/retest-required

/test e2e-aws

/test e2e-kubevirt-aws-ovn

/test e2e-aws

/retest-required

/retest

/test verify

/test verify

/approve

@jlojosnegros did another pass. The example perfprofile you have in the doc needed some tweaks for it to be accepted. You may want to replace it with something that does pass validation like the one you include in your e2e test:
https://github.com/openshift/hypershift/pull/1782/files#diff-8ac67e43decd1481bbd15bee0203d2c109cdc14c99fa2c1963d0a917d605453bR24

@jlojosnegros did another pass. The example perfprofile you have in the doc needed some tweaks for it to be accepted. You may want to replace it with something that does pass validation like the one you include in your e2e test: https://github.com/openshift/hypershift/pull/1782/files#diff-8ac67e43decd1481bbd15bee0203d2c109cdc14c99fa2c1963d0a917d605453bR24

Thanks @csrwng I have validated the one in docs after making the changes you pointed and it works.

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

/test e2e-kubevirt-aws-ovn

just rebased

/test e2e-aws

/lgtm

/retest-required

/hold cancel

/test e2e-kubevirt-aws-ovn

/hold cancel
/test e2e-kubevirt-aws-ovn

New changes are detected. LGTM label has been removed.

just a new rebase over main

/test e2e-aws

/retest-required

/test e2e-kubevirt-aws-ovn

/test ci/prow/e2e-aws

/test e2e-aws

We will need the lgtm again after the rebase  .    Can we avoid the rebase dance if all the mandatory lanes are passing ?

We will need the lgtm again after the rebase . Can we avoid the rebase dance if all the mandatory lanes are passing ?

Right now the problem is that not all mandatory lanes are passing.
e2e-kubevirt-aws-ovn has been failing lately

/test e2e-aws

/retest-required

/test e2e-aws

/test e2e-kubevirt-aws-ovn

/lgtm


Provide in-language examples for Resource Options

Our resource options are really powerful features, but not well documented. Just like our other programming model examples (e.g. .apply) we should provide language-specific examples of their usage. https://www.pulumi.com/docs/reference/programming-model/#resourceoptions

@lukehoban can you please provide a bullet list of use cases and code samples? Thank you.


Add minification step

this adds prepublishing minification,

resolvewithplus.mjs -5079, from 7282 to 2203
package.json -381, from 885 to 504
README.md -970, from 2911 to 1941
total, 6430 (~6.4Kb)

cc @Swivelgames the current "install size" is shown here as 11.4kB.  It will be interesting to see what size is shown after the update.

this is what npm printed out
npm notice
npm notice 📦<EMAIL_ADDRESS>npm notice === Tarball Contents ===
npm notice 750B  LICENSE
npm notice 1.9kB README.md
npm notice 504B  package.json
npm notice 2.2kB resolvewithplus.mjs
npm notice === Tarball Details ===
npm notice name:          resolvewithplus
npm notice version:       0.8.1
npm notice filename:      resolvewithplus-0.8.1.tgz
npm notice package size:  2.7 kB
npm notice unpacked size: 5.4 kB
npm notice shasum:        060ee6849d020768ec5cb315790b84be42f2cd91
npm notice integrity:     sha512-aO/aYbme7fDUb[...]0zT5dgbAEv1Rg==
npm notice total files:   4
npm notice
npm notice Publishing to https://registry.npmjs.org/
+<EMAIL_ADDRESS>


An issue of executing the script in run_start.sh

Hi,
First of all, your work is really interesting, and thank you so much for posting your codes.
I tried to execute the script you provided in run_start.sh and got the following error.
File ".../NeSy-CL/utils/training.py", line 237, in train
    tb_logger.log_accuracy(np.asarray(accs), mean_acc, args, t)
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (5,) + inhomogeneous part.

I tried to debug it and confirmed that the elements of accs were weird.
The type of accs is a tuple, but the items it contains have different data types, which is the cause of the above error.
([86.8], [86.8], [88.5], [88.5], tensor([0.9803, 0.9772, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000,
        0.0000, 0.9977, 0.9956, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000,
        0.0000, 0.0000]))

I'm still getting used to your repo and figuring it out, so could you let me know how to fix it?
Regards,
Jinyung Hong

Dear Jinyung Hong,
Thank you for your interest in our work and for spotting the error.
I am not sure yet what is causing this error, but for now I can only tell
you that we did not configure tb loggers and instead used wandb.
As for now, I would suggest to remove those loggers but I'm checking next
week what is causing this awkward result.
Best,
Emanuele Marconato
On Sat, 30 Sept 2023, 22:15 Jinyung Hong, @.***> wrote:

Hi,
First of all, your work is really interesting, and thank you so much for
posting your codes.
I tried to execute the script you provided in run_start.sh and got the
following error.
File ".../NeSy-CL/utils/training.py", line 237, in train
tb_logger.log_accuracy(np.asarray(accs), mean_acc, args, t)
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (5,) + inhomogeneous part.
I tried to debug it and confirmed that the elements of accs was weird.
The type of accs is a tuple, but the items it contains have different
data types, which is the cause of the above error.
([86.8], [86.8], [88.5], [88.5], tensor([0.9803, 0.9772, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000,
0.0000, 0.9977, 0.9956, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000, 0.0000,
0.0000, 0.0000]))
I'm still getting used to your repo and figuring it out, so could you let
me know how to fix it?
Regards,
Jinyung Hong
—
Reply to this email directly, view it on GitHub
https://github.com/ema-marconato/NeSy-CL/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANMY3ZD4LXOT7GAHWWRUNX3X5B4UPANCNFSM6AAAAAA5NYW3XY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi, Emanuele. Thank you so much for getting back to me so quickly.
Reflecting on your feedback, I initially turned off all base loggers, including csv_logger and tb_logger, by setting the default values of both --csv_log and --tensorboard arguments to False.
Then, I used wandb instead, which worked properly.
There may be some bugs in the processes while using both loggers.
Anyway, thank you so much for your feedback, and I will let you know if I have further questions.
Regards,
Jinyung Hong


schema: Remove redundant indices

Why are these redundant?  Is it because we don't use queries that rely on these indices?

According to PgHero, they are redundant due to existing uniqueness constraints.

Nice!


added LOG function

Hi yathura, i am relatively new to github but i know java programming and would like to contribute to your project. Do tell me if its correctly done. :)

Hi @jatingarg, I appreciate your effort to contribute and can congratulate you on a job wel done forking my repository and creating this pull request. Unfortunately this repository is still in (relatively) early development and I'm still actively changing the inner workings of the library (which makes it hard for other people to contribute to this repository). I have also not yet added a "how to contribute" section to the readme page, which might come soon (which will explain my wishes and expectations for contributors).
As you can see with issue #20 and pull request #30, I recently decided (and just today actually proceeded) to remove the function-specific classes in favor of generic base classes used by the helper methods in Helpers. So I'm sorry to say I will decline this pull request, but if you truly want to contribute, feel free to implement the log function to the Helper class (see issue #22 for more mathematical functions I still need to add).
I would like to ask you to create a branch other than development to perform your changes on (i.e. git checkout -b feature/22-description-of-your-changes) which would fit nicely into my workflow based on http://nvie.com/posts/a-successful-git-branching-model/ (and which will most likely be explained in the readme/wiki in the future).
Unfortunately I have only limited time to work on this personal project (and my other projects here on Github), so I can't say upfront when I will be working on it.

@yuthura, Thanks for all the suggestions, I will keep the conventions in my mind, Also as you said i will create a separate branch other than development for my changes and study the branching model. Also I will look into the Helper Class and see what can be done with the mathematical functions and come back to you with an update.And feel free to respond to it whenever u want, we all have time constraints.Keep up the good work!

Thanks for understanding @jatingarg, good luck and I'll be sure to check out any updates! :smiley:


X variable looping issue

If the same X variable is defined multiple times due to the looping of a QnA tree, the output value defaults to the first definition of the variable (actually, the same variable is defined multiple time in the document). The most recent value should overwrite all prior values (or rename old inputs to names other than the variable name).

Fixed by new variable handling.


Further increase TestCafe handleRedirect timeout

It's currently timing out a lot in CI, so let's try giving it some
more time.

Well what do you know, it actually passed CI in one go :D


Add to Kql validation the location of the error on the query

example
essage:
Template Id:95dc4ae3-e0f2-48bd-b996-cdd22b90f9af is not valid in Line:27 col:117 Errors:Missing expression, Code: 'KS006', Severity: 'Error', Location: '1659..1659'


puller(ticdc): add more log for puller

What problem does this PR solve?

Issue Number: ref #10181
What is changed and how it works?
In puller, print the context node for lastSlowestRange. eg:
	require.Equal(t, `[a @ 20] [b @ 20] [c @ 20] [d @ 20] [f @ Max] `, f.SpanString(spCD))

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/retest-required


Delete data from MongoDB/jskos-server?

Is there a way to delete data from jskos-server?

It's possible with Robo 3T

I'm using Robo 3T as well, but there should indeed be a better way of cleaning up the database. Maybe using custom scripts, similar to the script that manages local users in login-server?


[Feature Request] Add a way to set the BPM/timing effectclock beat uses in lua

Is your feature request related to a problem?
No
Describe the solution you'd like
For beat-based actor effects, it is to my understanding not possible to manually set the BPM/timing data that effectclock beat uses via lua. I'm requesting a way to make this possible.
As an example, effectclock beat will automatically sync actor effects to the beat of the currently playing song preview in ScreenSelectMusic. Similar functionality cannot be replicated on other screens, which might be wanted on, for example, a totally custom, lua-based select music screen (currently, effectclock beat would only sync to the custom screen's menu music in this case, to my understanding).
Describe alternatives you've considered
No response
Additional context
(Relevant conversation and Jose_Varela's proposed solution on the Outfox discord)

For now, there's now a new function inside the Song object that will perform the same operation called PlayPreviewMusic(). It'll send the song object to the soundmanager with all of its sync data so actors that have timing effects set can react to it.
local song = [your song object here]
song:PlayPreviewMusic()

Even if it's just a temporary/to be expanded on later thing, the idea behind this new function works for what I wanted to use it for, so I'll take it!
I think I've mentioned it in the Discord before a bit ago, but worth reiterating (especially now that I've tested/re-confirmed it on the 4.12 release), I've noticed a few things through testing it out:

The SampleMusicLoops metric is ignored if it's specified on the current screen, opting to always loop.
Sample music doesn't seem to play for a few specific songs in different packs (noticed, among others, on "birth", "Doki Doki", and "not eternity" from the DDR XX pack, along with "Broken" and "Heartbeat" in the Serenity pack). I assume this isn't theme specific, but since I'm not sure if any other themes (or the default one) use PlayPreviewMusic() at this time, I'm not sure how to go about testing this.

(log.txt, if needed)
But other than that, it seems like this new function is working well!


WIP Setup GitHub Actions

This closes #503 (replaces)
Since GitHub Actions is now out of beta, I've redone my branch, squashed my commits, and this is ready for further discussion. I figured a pull request would be appropriate so that changes can be reviewed.
For the moment windows builds are not passing. With node 8.x this appears to be a dependency version issue (although it is odd that this does not affect macos or ubuntu using node 8.x). With newer versions of node (10.x and 12.x) there seem to be other issues that may need actual fixes to the code.
CodeCov is setup in .github/workflows/nodejs.yml however to run correctly it will need CODECOV_TOKEN per the documentation https://github.com/marketplace/actions/codecov
.github/workflows/npmpublish.yml is setup to only run when a release is tagged in GitHub. The idea is tagging a release in GitHub would trigger this workflow to publish the package to npm. I do not know the specifics of yarn and the semantic-release package being used, but I imagine this may need to be changed to something else.

Closing in favor of https://github.com/okonet/lint-staged/issues/823, thanks anyway!


Syntax question

I've been messing with my colorscheme and I'm struggling to have brackets in object keys being colored like all other brackets.
For example,
obj[key] = 'newvLue';
Just colors the entire left side the same color on Gruvbox. I see the syntax for it is JavascriptProperty but I don't see how I can get the brackets to be the same color as other brackets. It's not a bug, so please don't feel like you need to implement this, just a question.
I see the syntax code was changed for javascriptComputedProperty. If if set matchgroup to javascriptBrackets it works. How can I properly refer to those brackets? Or anything really that is nested like that?

I found that bug too, it only happens when brackets are prefixed by other character:
This works fine:
const arr = [1, 2, 3]

this don't:
const el = arr[0]

I can't answer this kind of question.
I am also the struggle one~~
If there is a best practice or perfect behavior.
I will follow.
Feel free to discuss it here~
@FuzzOli87 This might be what you want to try
highlight link javascriptProperty javascriptBrackets

@othree the problem is brackets are not always colored. If you try my previous example you will see that only the brackets from the first example are colorized.

Yes, they have different meaning.
One is for object property. One is array.

Is there any way to get the object property brackets colored?

Try add
highlight link javascriptProperty javascriptBrackets


allow ip addresses in self signed cert

In addition to the hostname, we should also allow any "public" (i.e. non
bridge or loopback) addresses in the self signed certs we generate.
Closes #694
Signed-off-by: Tycho Andersen<EMAIL_ADDRESS>

Looks good to me.  I'm going to leave it to @stgraber to pull the merge trigger to make sure he sees and approves of the change in behavior.

On second thought I'd like to see whether this fixes the reported bug.


Is vite 6.x supported?

I am trying to add vike-node to a test project with vike 0.4.205 and vite 6.0.1 and I get a dependency conflict.
npm error code ERESOLVE
npm error ERESOLVE unable to resolve dependency tree
npm error
npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/vite
npm error   dev<EMAIL_ADDRESS>from the root project
npm error   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error   node_modules/vike
npm error<EMAIL_ADDRESS>from the root project
npm error     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error     node_modules/vike-node
npm error       vike-node@"*" from the root project
npm error
npm error Could not resolve dependency:
npm error peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error node_modules/vike-node
npm error   vike-node@"*" from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.

Probably yes, I'll take a look this weekend

@jasonhilldm should be fixed by 0.2.1

Thanks. I updated to vike-node 0.2.1 and vite 6.0.1 but when I run the test project I get the following error:
Error: No known conditions for "./client" specifier in "telefunc" package
Do you know if there are breaking changes in vite 6 that require some changes to telefunc setup?

@jasonhilldm Let me have a look.

I cannot reproduce. Can you publish a minimal reproduction?
(Closing this as the original seems to be fixed.)

Actually I could reproduce. Let me have a look.

@jasonhilldm https://github.com/vikejs/vike-node/issues/24

Thanks!


Retrieve list of departments from UCSB Curriculum API (not hard coded)

User Story
Basic: As a user, I can enter a quarter, department, and a course level, and see a list of courses.
More specific: As a user I can enter any department, not just ones that are hard coded.  The departments come from a query to the UCSB Curriculums API.
Acceptance Criteria

[ ] There is a backend method that can be used to query the list of departments.
[ ] The service that retrieves this list memoizes this list of departments so that it is only retrieved once per application startup (since it changes very infrequently).
[ ] On the basic course search form, the drop down for departments uses the values from this service

The API that you need for departments is this one:
https://api.ucsb.edu/students/lookups/v1/departments?includeInactive=false
It is documented here: https://developer.ucsb.edu/content/student-record-code-lookups
Here is some sample JSON for what you get back:
[
  {
    "deptCode": "ANTH",
    "deptTranslationShort": "Anthropology",
    "deptTranslation": "Anthropology",
    "collegeCode": "L&S",
    "divisionCode": "SOCSC",
    "orgCode": "ANTH",
    "inactive": false
  },
  {
    "deptCode": "ARTHI",
    "deptTranslationShort": "Hist of Art&Arc",
    "deptTranslation": "History of Art and Architecture",
    "collegeCode": "L&S",
    "divisionCode": "HUMFA",
    "orgCode": "ARTH",
    "inactive": false
  }
]

What you'll want to construct is a service that will go out to that API, grab all of the department info ,and then return that.


Unifies ALE to the EDL adapter by storing the cdl data in the metadata

Minor fix to the ALE adapter so that it stores the cdl data in the cdl property of the metadata dictionary, in the same way that the EDL one does.
It also adds one more ALE sample and related unittest.
Related issue: https://github.com/PixarAnimationStudios/OpenTimelineIO/issues/619

Codecov Report

Merging #621 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #621      +/-   ##
==========================================
- Coverage   81.69%   81.67%   -0.03%
==========================================
  Files          72       72
  Lines        2732     2729       -3
==========================================
- Hits         2232     2229       -3
  Misses        500      500

Flag
Coverage Δ

#py27
81.67% <ø> (ø)
:arrow_up:

#py36
?

#py37
?

Impacted Files
Coverage Δ

...ntimelineio/opentimelineio-bindings/otio_utils.cpp
87.05% <0%> (-0.16%)
:arrow_down:

...eio/opentimelineio-bindings/otio_anyDictionary.cpp
100% <0%> (ø)
:arrow_up:

...elineio/opentimelineio-bindings/otio_anyVector.cpp
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d32687...64f1444. Read the comment docs.

Any updates on this one? :)

@reinecke can you look again? I think all your review notes were addressed.


Add general description and basic information for missing instruments

Some instruments are missing a general description and basic information. Please add it.

I would like to work on this issue.

I'm new to to github. can someone please tell me how to check these issues?