Broken plugin: extractor/senategov.py - New URL Structure

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS> (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United States
Description
The U.S. Senate changed the structure of its URLs for Akamai, including the code lookup. The ISVP delivery is still there, but the code lookup and structure is now different. The error log is below. This is confirmed changed on all Senate.gov websites as of the first week in June.
The new structure still calls the ISVP url from the iframe, for example, using
https://www.epw.senate.gov/public/index.cfm/2022/6/toxic-substances-control-act-amendments-implementation
The ISVP file is:
https://www.senate.gov/isvp?type=live&comm=epw&filename=epw062222&auto_play=false
You still use filename, however the comm lookup (epw) is now different, per the mapping on that page (https://www.senate.gov/isvp?type=live&comm=epw&filename=epw062222&auto_play=false). See below these nows for the new mapping, which now has a label and uses a new, seven-digit ID.
From the comm (epw), the filename (epw062222) and the new lookup, the new HLS stream is constructed as follows:
https://www-senate-gov-media-srs.akamaized.net/hls/live/2036783/epw/epw062222/master.m3u8
or
https://www-senate-gov-media-srs.akamaized.net/hls/live/SEVEN_DIGIT_ID/COMM/FILENAME/master.m3u8
The new mapping table is below. As with previous implementations, there appears to be a backup/fallback structure:
https://www-senate-gov-msl3archive.akamaized.net/environment/epw062222/master.m3u8
...but I have been unable to confirm as the backup doesn't appear to deliver a stream. But the primary works.
That structure is showing up on older URLs e.g from 2021
https://www.epw.senate.gov/public/index.cfm/hearings?ID=DE00984F-4EC0-4CD9-BA77-A781E4F94DA1
calls the following HLS:
https://www-senate-gov-msl3archive.akamaized.net/environment/epw031721_1/master.m3u8.
They have not finished updating all Senate committee pages. But it appears they are doing it (the above from 2021 def was not in that structure in 2021, per our database). However, that said, dk if it makes sense to keep old method around as a fallback just in case. It def is not on any new videos but dk the process for updating all the old pages or how long it'll take.
However, based on the URL, guessing it applies to Akamai's msl3...
 Confirm you are on the latest version using  yt-dlp -U
  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 647, in extract
    ie_result = self._real_extract(url)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/extractor/senategov.py", line 131, in _real_extract
    for entry in self._extract_f4m_formats(f4m_url, video_id, f4m_id='f4m'):
  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 1984, in _extract_f4m_formats
    res = self._download_xml_handle(
  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 965, in download_handle
    res = self._download_webpage_handle(
  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 833, in _download_webpage_handle
    urlh = self._request_webpage(url_or_request, video_id, note, errnote, fatal, data=data, headers=headers, query=query, expected_status=expected_status)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 790, in _request_webpage
    raise ExtractorError(errmsg, cause=err)

  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/extractor/common.py", line 772, in _request_webpage
    return self._downloader.urlopen(self._create_request(url_or_request, data, headers, query))
  File "/home/ubuntu/.local/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 3594, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 404: Not Found

The given URL doesn't exist anymore. Do you have another example?


Must be able to specify charset when creating InputSource from File

Describe the issue
The ForEachChildElementPipe has a charset attribute, that is not used. It would be appropriate to use that charset when processFile=true.
Reporter
Gerrit
Environment
7.7-snapshot

Additional Environment

DBMS: [e.g. Oracle, MSSQL, MariaDB, MySql, PostgreSQL]
Browser: [e.g. Chrome, Safari, Edge, Firefox]

Additional Context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

Decided to deprecate processFile, and remove charset attribute


[Decommission Hub][2024-08-31] aup.pilot.2i2c.org

Summary
AUP is not renewing for the next year.
I have emailed the hub representative to confirm that it is safe to delete the data on the hub. (See https://2i2c.freshdesk.com/a/tickets/2036).  I received an "out of office until August 26" message.  Once we receive confirmation, this issue can be completed.
Info

Community Representative: Claudia Roda
Link to New Hub issue: https://github.com/2i2c-org/infrastructure/issues/659
Proposed end date: 2024-08-31

Task List
Phase I

[ ] Confirm with Community Representative that the hub is no longer in use and it's safe to decommission
[ ] Confirm if there is any data to migrate from the hub before decommissioning

[ ] If yes, confirm where the data should be migrated to

[ ] Confirm a 2i2c Engineer has access to the destination in order to complete the data migration

[ ] If no, confirm it is ok to delete all the data stored in the user home directories

Phase II - Hub Removal
(These steps are described in more detail in the docs at https://infrastructure.2i2c.org/hub-deployment-guide/hubs/delete-hub/)

[ ] Manage existing home directory data (migrate data from the hub or delete it)
[ ] Manage existing cloud bucket data (migrate data, or delete it)
[ ] Delete the hub's authentication application on GitHub or CILogon (note CILogon removal requires the hub config in place)
[ ] Remove the appropriate config/clusters/<cluster_name>/<hub_name>.values.yaml files. A complete list of relevant files can be found under the appropriate entry in the associated cluster.yaml file.
[ ] Remove the associated hub entry from the config/clusters/<cluster_name>/cluster.yaml file.
[ ] Remove the hub deployment

TIP: Run deployer use-cluster-credentials <cluster_name> before running the commands below
helm --namespace HUB_NAME delete HUB_NAME
kubectl delete namespace HUB_NAME

Phase III - Cluster Removal
This phase is only necessary for single hub clusters.

[ ] Remove the cluster's datasource from the central Grafana with:

deployer grafana central-ds remove <cluster_name>

[ ] Run terraform plan -destroy and terraform apply from the appropriate workspace, to destroy the cluster
[ ] Delete the terraform workspace: terraform workspace delete <NAME>
[ ] Delete the terraform values file under the projects folder associated with the relevant cloud provider (e.g. terraform/gcp/projects/ for GCP)
[ ] Remove the associated config/clusters/<cluster_name> directory and all its contents
Remove the cluster from CI:

[ ] deploy-hubs.yaml
[ ] deploy-grafana-dashboards.yaml

[ ] Remove A record from Namecheap account

Definition of Done
A non-specific, pre-defined list of tasks that should be considered before marking the task complete.

[] All the tasks above have been completed
[] Existing functionality was not broken

I have requested the hub champion confirm is it is safe to delete the data on this hub.  Original email sent Aug 16 with a reminder email sent Sept 11.
If no response is received by Sept 18 I think we should proceed with the removal of the data.

Confirmation data can be deleted received: https://2i2c.freshdesk.com/a/tickets/2142


Update ubuntu install script for php 8.* (default for Ubuntu 22.04)

Updated the script to support and prefer PHP 8.* if available. This is the default PHP version on Ubuntu 22.04.
Tested on Ubuntu 22.04, Ubuntu 20.04 and Ubuntu 18.04.

@kevinfarrugia are you working on other changes as well?  If not, I'm happy to merge this once it's marked ready.

I still need to test it on 18.04 but don't expect problems. Will ping once tested.

Not sure I'd bother.  18.04 is EOL in 2 months so nobody should be doing new installs on it.

I managed to test it in the meantime and it worked well too. Ready to merge.


[Feature Request]: Refresh custom browser services.

Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem Description
When I add a new service "custom website/browser" and when I minimize the app "Ferdi" it doesn't refresh the custom website/browser. Let's say for example, I used it for voting discord bots and the vote refreshes every 12 hours. So when I click vote it says "You've already voted." and after 12 hours again it still says "You've already voted." even tho the vote is already available. Although when you restart the app it works. But I just want to suggest this feature so that we won't have to relaunch Ferdi every time we want to refresh the site.
Proposed Solution
When you add a custom service, it refreshes while running in the background or while it's minimized.
Alternatives Considered
It should refresh every 15-20 minutes or it depends on what you're going to set the refresh time for browsers.
Additional Information
No response

You don't need to restart Ferdi for this!
You can use View > Reload Service in the menus. Or, even simpler, just use Cmd+R which is the keyboard shortcut for the same.


Host Name Conflict

I do not get any response from any plug in, as I face a host name conflict.
Expected Behavior
Homebridge should run, the plug in should register and the data should be shown in HomeKit
Current Behavior
Homebridge runs, plugs ins are register and shown in HomeKit, but no data is transmitted.
Log shows Host Name Conflict
Steps to Reproduce (for bugs)
just restart the container
Versions

Homebridge: homebridge --version 0.4.44
Node:       node --version v.8.11.3
OS:            DSM 6.2-23739 Update 2

Configuration
{
"bridge": {
"name": "Homebridge",
"username": "DADADADAD",
"port": 51826,
"pin": "031-45-154"
},
"description": "This is an example configuration file. You can use this as a template for creating your own configuration file containing devices you actually own.",

"accessories": [
],

"platforms": [

    {
      "platform": "DysonPlatform",
      "name": "DysonPlatform",
      "email": "BBB",
      "password": "CCC",
      "country": "DE",
      "accessories": [
        {
        "ip": "EEE",
        "displayName": "PureCool",
        "serialNumber": "DDD"
      }
        ]
   },

   {
   "platform": "Netatmo",
   "name": "Netatmo Platform",
   "auth": {
     "client_id": "AAA",
     "client_secret": "ZZZ",
     "username": "YYY",
     "password": "XXX"
   }
 }

]
}
Homebridge Log / Command Output

Host name conflict, retrying with DSM216-1865
Registering new address record for <IP_ADDRESS> on docker0.IPv4.
Registering new address record for 2a04:4540:6d00:8000:211:32ff:fe55:3323 on ovs_eth0.*.
Registering new address record for <IP_ADDRESS> on ovs_eth0.IPv4.
^[cWithdrawing address record for <IP_ADDRESS> on docker0.
Withdrawing address record for <IP_ADDRESS> on ovs_eth0.
Host name conflict, retrying with DSM216-1866
Registering new address record for <IP_ADDRESS> on docker0.IPv4.
Registering new address record for 2a04:4540:6d00:8000:211:32ff:fe55:3323 on ovs_eth0.*.
Registering new address record for <IP_ADDRESS> on ovs_eth0.IPv4.

Your username is malformed, It must be a Mac address like:
"username": "CC:22:3D:E3:CE:34",


wesmile-showheb

Write a short description about your art piece, such as what your artwork represents and your experience drawing it!
-> I enjoyed making the art piece a lot. Always wanted to make computer generated art and this java flavour was a perfect way to start. I can say very confidently as a art student myself. This was an extremely surreal experience initiated by a sleepless IBDP students idle brain.

[x ] I have read the steps to getting a blot
[x ] I have read all the submission requirements and follow them
[ x] I am submitting art that...

[x ] is algorithmically generated (will meaningfully change each time the program is run)
[ x] is your own work that is not copied from somewhere else
[x ] is drawable through the editor(does not use outside tools)
[ x] doesn't call Math.random() (See the documentation on randomness)
[x ] is drawable on Blot itself (doesn't have lines overlap more than 5 times)

[ x] Doesn't overlap lines more than 5 times
[x ] Fits in the working area of 125x125mm
[x ] Doesn't rely on colour to look good

If you used based your art on something else, used a tutorial, or remixed it from something else, please link it here:
Lastly, link a picture of your art piece here:

/*
@title: wesmile
@author: Showheb
@snapshots: 1.png
*/
const width = 210;
const height = 297;
setDocDimensions(width, height);
const drawSpirals = (numSpirals, radiusIncrement) => {
for (let i = 0; i < numSpirals; i++) {
const angle = (i * Math.PI) / 8;
const radius = i * radiusIncrement;
const x1 = (width / 2) + radius * Math.cos(angle);
const y1 = (height / 2) + radius * Math.sin(angle);
const x2 = (width / 2) + (radius + radiusIncrement) * Math.cos(angle + Math.PI / 8);
const y2 = (height / 2) + (radius + radiusIncrement) * Math.sin(angle + Math.PI / 8);
drawLines([[[x1, y1], [x2, y2]]]);
}
};
const drawFace = (xCenter, yCenter, faceRadius, eyeOffset, eyeSize, irisSize) => {
const faceOutline = [];
for (let angle = 0; angle < 360; angle += 10) {
const rad = (angle * Math.PI) / 180;
faceOutline.push([xCenter + faceRadius * Math.cos(rad), yCenter + faceRadius * Math.sin(rad)]);
}
drawLines([faceOutline]);
const drawEye = (xCenter, yCenter) => {
    const eyeOutline = [];
    const irisOutline = [];
    for (let angle = 0; angle < 360; angle += 10) {
        const rad = (angle * Math.PI) / 180;
        eyeOutline.push([xCenter + eyeSize * Math.cos(rad), yCenter + eyeSize * Math.sin(rad)]);
        irisOutline.push([xCenter + irisSize * Math.cos(rad), yCenter + irisSize * Math.sin(rad)]);
    }
    drawLines([eyeOutline, irisOutline]);
};

drawEye(xCenter - eyeOffset, yCenter + 4);
drawEye(xCenter + eyeOffset, yCenter + 4);

};
const drawMultipleFaces = (numFaces) => {
for (let i = 0; i < numFaces; i++) {
const xOffset = Math.random() * (width - 20) + 10;
const yOffset = Math.random() * (height - 20) + 10;
drawFace(xOffset, yOffset, 8, 4, 2, 1);
}
};
const drawMoreSpirals = (numSpirals, radiusIncrement) => {
for (let j = 0; j < 3; j++) {
drawSpirals(numSpirals, radiusIncrement * (1 + j * 0.1));
}
};
drawMoreSpirals(80, 0.6);
drawMultipleFaces(20);
const drawEyelids = (xCenter, yCenter) => {
const eyelidWidth = 3;
const eyelidHeight = 1;
const drawEyelid = (isUpper) => {
    const eyelid = [];
    for (let angle = isUpper ? -90 : 90; angle <= (isUpper ? 90 : 270); angle += 10) {
        const rad = (angle * Math.PI) / 180;
        eyelid.push([xCenter + eyelidWidth * Math.cos(rad), yCenter + (isUpper ? 3 + eyelidHeight : -3 - eyelidHeight) * Math.sin(rad)]);
    }
    drawLines([eyelid]);
};

drawEyelid(true);
drawEyelid(false);

};
const drawEyelidsAroundAllFaces = () => {
for (let i = 0; i < 20; i++) {
const xOffset = Math.random() * (width - 20) + 10;
const yOffset = Math.random() * (height - 20) + 10;
drawEyelids(xOffset - 4, yOffset + 4);
drawEyelids(xOffset + 4, yOffset + 4);
}
};
drawEyelidsAroundAllFaces();
const drawMouths = () => {
for (let i = 0; i < 20; i++) {
const xOffset = Math.random() * (width - 20) + 10;
const yOffset = Math.random() * (height - 20) + 10;
drawLines([[[xOffset - 5, yOffset - 6], [xOffset + 5, yOffset - 6]]]);
}
};
drawMouths();

Hello! Sorry it's been so long. The main reviewer has been swamped lately, so I wanted to give you a few hints so you have the best chance of getting a blot.
It would be great if you could rename your files according to the naming scheme (directions here). This will allow an automatic preview to be made, which is super nice for reviewers.
Also, I noticed that your program currently generates a project larger than the size blot can handle. Blot's max size is 125x125, so you will need to scale it down to fit this size limit.
In terms of the actual project, it's really cool! It's certainly quite an interesting piece of art, and I think with a few tweaks we can get it to be fully "blot-worthy". Generally, people spend ~15-20 hrs getting a project to this level.
Some ideas I have are:

Currently, the faces have a chance of being completely overlapping. While I understand the artistic idea of having things overlap, when two of the faces overlap it makes for something that just looks muddled and won't draw great on the blot.
example:

Could you make it so that the faces have a little collision so they are spaced out slightly?
I would love to see the spiral in the middle change shape, size, etc.! It would make the whole piece feel even more dynamic
Could you add other elements of a face?

You're also the artist, so any ideas you have would also be great! I'm excited to see where you go with this!


Update login.js

Thanks but this is intentionally a redirect so we can change it after the fact.
We'll be doing another PR to change our url shortener domain in the future.


Fix code scanning alert - Pinned-Dependencies

Tracking issue for:

[ ] https://github.com/microsoft/ebpf-for-windows/security/code-scanning/521

False positive.


Empty registrationId Android

I am using Parse.com to send push messages to my Ionic app. Therefore, I am using their senderID.
var push = PushNotification.init({
	"android": {
		"senderID": "1076345567071"
	},
	"ios": {
		"alert": "true",
		"badge": "true",
		"sound": "true"
	}
});

The onRegistration event does fire for Android, but the registrationId field is empty, so I am unable to receive push notifications.

@brambogaerts Don't use the senderID from Parse. Use the GCM one you had to provide to Parse.
https://github.com/phonegap/phonegap-plugin-push#pushnotificationinitoptions

@macdonst Thanks for your speedy reply. However, I have created a new app from scratch to test with, and used my own senderID I provided to Parse, and I still get the same results...

As I was looking through the log items, I found this:
E/PushPlugin: execute: Got JSON Exception TIMEOUT
Internet connection is working.

@brambogaerts I am unable to reproduce your problem. Got a reproduction scenario/code I can look at?

Well, the problem fixed itself. Somewhere between cleaning all kinds of caches and reinstalling every part imaginable, I managed to get the correct ID. Thanks for your response!

Closing, re-open if you can reproduce.

Colleague of @brambogaerts here... Since we have bad wifi reception in the space we're working we setup a wifi using OS X's Internet Sharing. Somehow this is related to this problem as we got a real registrationId when we connected to a proper router... @macdonst any idea how that could be possible?

@thomasjonas sorry no. Networking is like black magic to me. Especially when you get into connection sharing and captive gateways.


feat: add clean up functions to custom run-command(s) executors

The goal of this change is to allow a dev to run yarn serve background:jobs (or equivalent) and have their docker container running ElasticMQ to stand up and upon killing the serverless-offline instance, stopping the docker container so the user does not have to do anything manually.
This draft is based on a different branch and has some excess work but I don't want to lose the ideas here. The tools/executors/workspace/run-command/impl.js v tools/executors/workspace/run-command/test.js really demonstrates the real issue here. To run the impl example, you can run yarn serve background-jobs. When you run Ctrl + C to send SIGINT, it kills the executor and you'll note that SIGTERM was sent. This is coming from somewhere in nx. I think it's this killCurrentProcess function  but I'm not super familiar with nx internals.
If you run node tools/executors/workspace/run-command/test.js, you'll notice that the SIGINT is properly captured in contrast. Without the ability to control the signal exits, I think this is going to be an impossible task to accomplish.

I've decided as a result of #57 that the docker file I'm trying to make happen belongs at the repo top-level and managed manually by developers instead of automatically. Closing as this is no longer relevant.


OS#16050168: fix jit crash for splitscope using 'with' statements

When 'with' statements are used in inside param scopes when param and body scopes are split, there will not always be a local closure generated. See attached test for an example case.
The IRBuilder would attempt to build a RegOpnd for the closure, and would assert on a nullptr sym. The backward pass would later AV on that nullptr sym.
This commit instead assigns nullptr to the param closure when there is no local closure, which is the behavior of InterpreterStackFrame::OP_BeginBodyScope()

@akroshg The local closure is not generated because of the logic in ByteCodeGenerator::ProcessScopeWithCapturedSym. The scope containing the captured sym is the 'with' scope, which is marked as MustInstantiate and thus can't merge.

LGTM


build broken on linuxbrew

Anyone care to help on https://github.com/grpc/homebrew-grpc/issues/94 ?
grpc on linuxbrew has been broken on linux for months:
This is the latest:
https://github.com/grpc/homebrew-grpc/issues/94#issuecomment-273184580
Help would be much appreciated!

Closing this as it is being tracked via the bug in the linked repository.


RFC: Row identities in MJ schema

Problem description
While MJ is built on a single CDP that includes data from many external sources, it relies on an administrative schema (defaulting to __mj, although it may be customized per installation) for managing its own state. The schema has tables for many internal entities, including the Entity table itself. As users set up their own MJ installation and make stateful changes (running CodeGen, creating views, adding favorites, or almost any other action in MJ) new records are created within the internal __mj tables as needed.
Our current strategy for managing database schema changes uses Redgate to identify and creates migration scripts based on both structure as well as data. The latter is important since the system configuration is largely expressed through data in the __mj tables. These tables generally use an autoincrementing numerical ID column for row identities.
New rows created for system purposes are often created with the next ID in the base sequence, while new rows created through user interactions generally have IDs beginning at a predetermined, high threshold (>=2500000) intended to avoid ID collisions. Despite this, there remains the potential (and there have been cases) of identity conflicts when upgrading MJ databases.
Our goal is to seek a solution that can manage or avoid having system records within the __mj schema collide with records added through user interaction.
Design constraints and considerations

[!NOTE]
TBD

Proposed solutions
The general expectation is that any meaningful change to the row identities within the __mj is a major change and users will likely have to re-setup their CDP databases.

Use KSUIDs for row identities

These 27-character identifiers offer uniqueness without managing address spaces, are portable base-62 strings, and allow binary sorting to preserve order even in a distributed system

[!NOTE]
More may be added once the problem is adequately understood and expressed

Selected solution

[!NOTE]
TBD

@AN-BC RFC as discussed. Please feel free to revise as needed

My current thinking would be that we:

Remove the current ID columns in all __mj tables
Replace them with a column defined as ID that is a KSUID type.
Ideally, we generate the KSUID as a SQL Server function so that we can embed this complexity in the spCreate procedure for each entity in the system
Make this type of primary key a first-class supported approach for entities as an alternative to IDENTITY column based primary keys so that application developers who want portability for their metadata (as well as users of MJ) can utilize this approach for their ID values
The only question I have is how this affects the user experience if the KSUID based ID field replaces the INT identity and therefore for anything user facing if there is a need for an ID value, it will be much harder to use. But perhaps there isn't a need for this?

@hiltongr @dray-mc  would be great to have both of your comments on this when time permits

This was implemented in major version 2.0.0


Remove all parts of ProjectApproval StreamForm

Fixes https://github.com/HyphaApp/hypha/issues/2583

Codecov Report

Merging #2635 (9018400) into main (9d7fdc3) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2635      +/-   ##
==========================================
- Coverage   81.00%   80.98%   -0.02%
==========================================
  Files         647      650       +3
  Lines       23000    22964      -36
==========================================
- Hits        18631    18598      -33
+ Misses       4369     4366       -3

Impacted Files
Coverage Δ

hypha/apply/funds/models/applications.py
84.19% <ø> (-0.12%)
:arrow_down:

hypha/apply/funds/tests/factories/models.py
100.00% <ø> (ø)

hypha/apply/funds/tests/test_admin_form.py
100.00% <ø> (ø)

hypha/apply/projects/forms/project.py
97.36% <ø> (-0.03%)
:arrow_down:

hypha/apply/projects/models/__init__.py
100.00% <ø> (ø)

hypha/apply/projects/models/project.py
94.89% <ø> (+0.26%)
:arrow_up:

hypha/apply/projects/tests/factories.py
98.98% <ø> (-0.10%)
:arrow_down:

hypha/apply/projects/views/project.py
92.96% <ø> (+4.04%)
:arrow_up:

...ons/0089_remove_project_approval_form_refrences.py
100.00% <100.00%> (ø)

...a/apply/home/migrations/0002_add_apply_homepage.py
100.00% <100.00%> (ø)

... and 4 more

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 9d7fdc3...9018400. Read the comment docs.

@frjo can we deploy this to test just to make sure it doesn't break anything.

Deploying to test now.

It works well for me.

Any idea why the Approving of a submission doesn't create a project on test? I tried on a couple of submissions.

@sks444 There is a setting PROJECTS_AUTO_CREATE that is set to "False" on test. It is set to "True" in ?dev.py` so you most likely have that on your local setup.
When PROJECTS_AUTO_CREATE is set to "False" you get a "Create Project" button in"Actions to take" so staff can do it manually.

@frjo Yes, I have PROJECTS_AUTO_CREATE set to True in my local. Wasn't aware of Create Project button, thanks! It works well.


IntelliSense problems if Flow plugin is used

VSCode Version: 1.11.2
OS Version: Linux Ubuntu 16.04

Hello, I opened issue here https://github.com/flowtype/flow-for-vscode/issues/97 and they said it's VS Code problem, maybe anyone can take attention on that? This plugin is very good, because it improves IntelliSense and we don't need to use .ts files to have improved IntelliSense, but the problem is, it somehow duplicates suggestions and some of them arranges to the end or at the middle which is really bad, but still is it better to have this with improved IntelliSense. I know Flow and TypeScript not the best friends since VS Code using TypeScript it makes zero sense to you for trying to fix it, but it will be very cool from your side.
P.S. If someone are lazy to check issue on Flow plugin which is above I can post here all the content from the issue I opened.
Thank you.

To save you digging around this is basically https://github.com/Microsoft/vscode/issues/15738 and https://github.com/Microsoft/vscode/issues/17068

Thanks @orta! Yes, #17068 tracks allowing built-in extensions to be disabled.
Closing as a duplicate of #17068


Python extension fails to activate

Edit: the error is
The number of constructor arguments in the derived class s must be >= than the number of constructor arguments of its base class

Web Browser: Chrome 83.0.4103.116 (Official Build) (64-bit)
Local OS: macOS 10.13.6
Remote OS: Clear Linux version 33420
Remote Architecture: x86_64
code-server --version: 3.4.1 48f7c2724827e526eeaa6c2c151c520f48a61259

I've loaded the Microsoft python extension. Syntax coloring works, but code completion does not. I also get the error "command 'python.setInterpreter' not found" when clicking on the python interpreter at the bottom of the window.  Remote server python is 3.8.3.

I'm having the same issue. I've tried reinstalling the extension and rolling back to an older version. No luck :cry:

This is actually an extension problem and this version works https://github.com/microsoft/vscode-python/releases/tag/2020.5.86806
Mine works after installing this one.
Source: Issue #1818

@EzzPrograms that works for me. Reverting to an old version enables the intellisense. thanks

Upstream issue: https://github.com/microsoft/vscode-python/issues/12465

@EzzPrograms can you post the version of the code server and python version you are using. I have downgraded still pylint shows import error

@EzzPrograms can you post the version of the code server and python version you are using. I have downgraded still pylint shows import error

This is the python extension version I’m using: https://github.com/microsoft/vscode-python/releases/tag/2020.5.86806
For code-server, it should be the latest one.

@EzzPrograms can you post the version of the code server and python version you are using. I have downgraded still pylint shows import error

This is the python extension version I’m using: https://github.com/microsoft/vscode-python/releases/tag/2020.5.86806
For code-server, it should be the latest one.

finally I got the python extension to work by disabling Jedi (jediEnabled:false) and by setting PYTHONPATH env var . The PYTHONPATH was not getting picked up from the .env file

@cmoog Lets revert this extension on the marketplace to the working version.

On code-server 3.8.0 I have to downgrade Python extension to version 2020.10 (the latest version is 2020.12) to make it work.

On code-server 3.8.0 I have to downgrade Python extension to version 2020.10 (the latest version is 2020.12) to make it work.

Same on code-server 3.8.1, have to downgrade to 2020.10. (Also the now latest 2021.1 still doesn't work).

On code-server 3.8.0 I have to downgrade Python extension to version 2020.10 (the latest version is 2020.12) to make it work.

I have the same issue, Have been looking everywhere for a solution, This finally fixed it for me ~ Thank You!

It seems that this bug will only appear in node 12，Node 10 is

Hey there - this issue is stale and the latest versions of Python/Jupyter work as of v3.10.2; closing.

Still an issue here. Activating the extension via the command Python: Select Interpreter does not show the window wherein I can select different Python executables. The status bar shows "Activating Extensions.." for a few seconds and disappears, but nothing happens after that.
code-server: v3.10.2
VS Code: v1.56.1
Commit: 387b12ef4ca404ffd39d84834e1f0776e9e3c005
Date: 2021-05-21T23:13:00Z
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.102 Safari/537.36

@jingw222 please open a new bug report issue with repro steps. Thanks!


always alert the window has crashed with version 1.46

since update to 1.46, it always alert the window has crashed when I run my program.

Moved product issue from vscode-docs.

/needsMoreInfo

Please specify the program you are running and are you using any extension for doing so.


Mobile friendly

I saw the template for the profile show is being naughty and isn't responsible.
I'll have a look at this later

Thanks. I look forward to the next PR


Added Check for flags in create.go

Since there can be only one flag at a time, This PR adds the check for the same.

I think check for binary should also be included

@mik-dass check for binary is happening here, https://github.com/surajnarwade/ocdev/blob/7ae180617297253182276f5f369d87a476df4a58/cmd/create.go#L54

@surajnarwade But the os.Exit and cmd.Help will be removed in future so I think we can add checkFlag++ in the if block now, otherwise someone removing the exit might forget it and checkFlag won't be incremented

@mik-dass , make sense, I have updated PR for the same

LGTM :)


Cache dependencies between crates published by the same owner

Right now, when a large workspace publishes its crates, we spend a lot of time rebuilding the same dependencies over and over again (e.g. https://github.com/aptos-labs/aptos-core). To reduce the load on docs.rs and speed up builds, we can cache those dependencies to avoid rebuilding them.
We are only planning to do this between crates owned by the same crates.io owner for security reasons; see https://discord.com/channels/442252698964721669/541978667522195476/996125244026794134 for a discussion of how this causes issues between crates. In practice, this should still be a giant speed up since most of our build times come from large workspaces.
In order to have deterministic time limits, we are going to count the time spent to compile a dependency against the time limit, even if we already have the dependency cached locally. To avoid running out of disk space we'll use an LRU cache that deletes crates when we have less than ~20 GB free on the production server. For reproducibility we are going to make the cache read-only, and wipe the target/doc directory after each root crate we build.
Here is the rough approach we're imagining:

Run cargo check --timings=json -Zunstable-options -p {dep} for each dependency of the crate we're building
If we can parse the JSON output, record the timings in a database. Otherwise, make a note to wipe the cache for everything we were unable to parse after this build finishes.
Run chmod -w on everything in target (except for the top-level directory, so cargo can create target/doc)
Subtract the timings of all dependencies from the time limit (even if some were already cached). If we were unable to parse the JSON, subtract the entire time needed in step 2.
Run cargo doc and upload to S3.
Delete target/doc.
repeat 1-6 for all new crates.io crates published by the same owner.
Wipe the cache and the database table the next time we call rustwide.update_toolchain, since the new nightly will be unable to reusse the cache.

Run chmod -w on everything in target

Tthis sounds like cargo check for each dependency ends up collecting everything we need for rustdoc. Also assuming we can make rustdoc work with /target/ dependencies readonly except the output directory.
Shouldn't this prevent any possible malicious changes that the crate author can do?
So we actually could cache the dependency for all owners?
Also the --timings=json logic and calculation feels like quite some complexity "only" for deterministic time-limits.

The most aggressive attack that @pietroalbini mentioned was:

We build some crate foo under control of the attacker

This has a build script that generates a bar_macro.rlib and associated files into the target-dir to make it look like bar-macro has already been built.

We build docs for another crate bar which depends on a proc-macro bar-macro

Cargo sees that (the attacker generated) bar_macro.rlib is up-to-date and uses it, now the attacker has code running in the rustc process during the build for another crates docs and can essentially do anything it wants to those docs.

Wouldn't that be prevented by only having /target/doc writable?
Or does cargo check not do everything cargo doc needs and we will add files?

Oh, I forgot one indirection in the first step, the crate foo depends on another crate foo2 under control of the attacker. The foo2 build script will be run during the cargo check step while the whole target-dir is writeable.

( sorry if I'm missing something, these build specifics are new to me )

Oh, I forgot one indirection in the first step, the crate foo depends on another crate foo2 under control of the attacker. The foo2 build script will be run during the cargo check step while the whole target-dir is writeable.

How would that be different with or without caching?

Without caching: if foo2 writes libbar_macro.rlib into the target directory, that will be wiped before we build the docs for bar, so there's no way for one build-tree to affect another build-tree (you only have to trust your dependencies, not everything else that has been built by the builder).

Ah, then I get it. Thank you for spending the time explaining!
Couldn't we keep the cache by top-level dependency and version? Of course this would mean duplicate storage of the output binaries, but the implementation could be quite simple

That means the possible process could be (without timings):
for each top-level dependency:

check if we have a cached target directory. If yes, copy it to the future doc-build target, skipping any duplicate files.
if we don't have a cached target, run cargo check dep@vers in an isolated env. Put the output into the cache, & copy it over to the doc-build target

When we did this for all top-level dependencies we can run cargo doc without any permission change since we don't put any output files into the cache. Also if we missed something from the cache, cargo doc will just build it.

I think the most effective way would be to just keep the same target directory only when the previous crate was published by the same person as the new crate. So, for example:

foo by @alice, uses a fresh target directory as no build happened before
bar by @alice, reuses the target directory of foo as it's also published by @alice
baz by @bob, removes old target dir and uses a fresh one as it's not published by @alice
foobar by @alice, removes old target dir and uses a fresh one as it's not published by @bob

That would solve the problem of big workspaces being publishes, as I assume they would all be published by the same user, and removes the chance of someone else sneaking something into someone else's docs.

Ah, so this would also depend on the queued order of the crates?

So it would only help specifically for these bulk crates that are quickly released after each other, and not for any other crates.

We have a hard-limit of max 1 day of caching because of updating the compiler. There are probably plenty of cases where a few related crates are published throughout the day which we would miss, but the really bad situations are when we have the mass publishes of workspaces with very deep dependency trees.

how would that look like with multiple owners? treat them as one (unique) new owner/team?

We can get the actual publisher of each release from the crates.io API (even for crates with multiple owners, if they're publishing a workspace at once I'd assume a single account will be used for all the crates).

FWIW this is painful enough I would be ok with omitting the "record the timings in a database" bit for now if it lets us get the caching in sooner, it can only have false positives (your crate intermittently builds when otherwise it would have always failed), not false negatives (your crate intermittently fails when it should have always succeeded).

I'm with you on both points, the first one we also already talked about, the second was clear to me.
What's slowing me down is mostly the iteration speed / debugging with docker on my mac, since I have to go through the web container for everything.
On top of the fact that the build process & rustdoc are definitely the parts I know the least, so much trial & error is needed.
While improving the dev experience is certainly possible, currently I'm trying to invest time into artifact caching here, after the CPU load problem was worked around & a possible fix is in a PR.

@syphar yup definitely, I wasn't trying to imply you weren't doing enough work ❤ for context @alice-i-cecile was asking how she could help speed up the build queue and I wanted to make sure the issue was up to date since she hasn't been sitting in on our meetings.

Yep, the Bevy community was curious about how we could help, and I was pointed here <3

@syphar yup definitely, I wasn't trying to imply you weren't doing enough work ❤

I wasn't trying to implying that you implied ;)
I myself feel slow when I'm working on the build-process, also visible when I want to dig into the other bigger build-process topic (adding builds into the DB before the build)

for context @alice-i-cecile was asking how she could help speed up the build queue and I wanted to make sure the issue was up to date since she hasn't been sitting in on our meetings.

ah, thanks for the explanation!
If needed, I can create WIP PR + some explanation where I am right now, if @alice-i-cecile wants to help out.

Sure, that sounds lovely :) I don't personally have a ton of time or expertise here, but I'll take a look and share it around the community: we have a lot of talented folks and care about improving things for the whole ecosystem.

I created the draft PR with what I already had: https://github.com/rust-lang/docs.rs/pull/2079


AS2 component has no configuration options

Describe the Bug
When adding the AS2 component from the Kaoto catalog to the canvas I have no configuration options. Test both as from and as to endpoint.
Steps to Reproduce the Bug or Issue
Add an AS2 component endpoint to the canvas and click it to open the config panel.
Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 0.30.0]

Looked at how this property was handled in the kaoto-ui. There seems to be a recommendation "Please use the source code editor to configure this property."
I think this issue is causing more components not be configurable ATM, e.g. the google sheets and calendar

duplicates #448


New: Add properties setters and getters

Resolves #88
This PR introduces setters, patchers, and getters for node/edge properties, along with implementing the Observer pattern to invoke view rerender upon property changes. The API has been significantly changed. Instead of writing unsafe code like
orb.data.getNodeById(0).property = x;

we now use
orb.data.getNodeById(0).setProperty(x);

and
orb.data.getNodeById(0).patchProperty(x);

Accessing properties is also modified, from
orb.data.getNodeById(0).x;

to
orb.data.getNodeById(0).getX();

Documentation needs to be updated accordingly. However, I believe it's more appropriate to do a complete rewrite upon the 1.0.0 release. Additionally, I want to implement tests after discussing all the details.
This PR is marked as a draft because I feel there are important aspects to double-check and review. However, it's open for all sorts of comments and advice!

Tests are failing because of the usage of the structuredClone() function, which was introduced in Node.js 17.0.0, while our tests are running on Node.js v16.20.2. Is it ok to update the Node.js version in the package.json to >=17.0.0, or should I use some custom function for deep cloning objects?

Tests are failing because of the usage of the structuredClone() function, which was introduced in Node.js 17.0.0, while our tests are running on Node.js v16.20.2. Is it ok to update the Node.js version in the package.json to >=17.0.0, or should I use some custom function for deep cloning objects?

Yep, we should update Node to some newer version for sure!


cypress for box plot

Signed-off-by: nidhisinghai<EMAIL_ADDRESS>Description
cypress test case for box plot which covers 18 test cases
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.
@abasatwar @spattnaik

Codecov Report

Merging #969 (c11e06e) into main (3e5d26b) will increase coverage by 15.82%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #969       +/-   ##
=============================================
+ Coverage     56.04%   71.87%   +15.82%
  Complexity      291      291
=============================================
  Files           262       42      -220
  Lines          8786     2311     -6475
  Branches       1984      240     -1744
=============================================
- Hits           4924     1661     -3263
+ Misses         3687      509     -3178
+ Partials        175      141       -34

Flag
Coverage Δ

dashboards-observability
?

opensearch-observability
71.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...lic/components/common/search/autocomplete_logic.ts

...ace_analytics/requests/queries/services_queries.ts

.../notebooks/components/helpers/modal_containers.tsx

...application_analytics/helpers/modal_containers.tsx

...analytics/components/traces/span_detail_flyout.tsx

...ic/components/common/field_name/field_type_name.ts

...e_analytics/components/services/services_table.tsx

...visualizations/visualization_specs/default_spec.ts

...omponents/common/helpers/ppl_docs/commands/rare.ts

...izations/count_distribution/count_distribution.tsx

... and 210 more

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


Add grant_uri_permissions, path_permissions tables to sysappdb

Right now I'm just storing the XML, which isn't really ideal.

Tagging for future.


Log more info about internal errors

In https://pydanticlogfire.slack.com/archives/C06EDRBSAH3/p1721433723530879 you can see an internal error where we don't produce enough information to determine what line of user code produced the issue. (We only get stack frames underneath the try: except: so can't see what user code led to the error.)  I think it would be better if we log a bit more frames up the stack when an internal error is produced to give us a better handle on debugging.
In particular, in this case, it seems to be an issue with magic formattting — seeing the source code could be very insightful.

@alexmojaki

it looks like there's something funny going on with 3.11. Do you know how to install multiple environments with rye? I tried skimming the docs and wasn't able to figure it out quickly, if you don't know where to look quickly / off the top of your head, I can dig more.

Here's an example of what this looks like:
Internal error in Logfire. Frames above traceback:   File "/Users/alex/Library/Application Support/JetBrains/PyCharm2024.1/scratches/scratch_222.py", line 5, in <module>
    logfire.info("log {'x':{'message'}}")

  File "/Users/alex/work/logfire/logfire/_internal/main.py", line 336, in info
    self.log('info', msg_template, attributes, tags=_tags, exc_info=_exc_info)

  File "/Users/alex/work/logfire/logfire/_internal/main.py", line 598, in log
    with handle_internal_errors():

  File<EMAIL_ADDRESS>line 158, in __exit__
    self.gen.throw(value)

  File "/Users/alex/work/logfire/logfire/_internal/utils.py", line 243, in handle_internal_errors
    log_internal_error()
Traceback (most recent call last):
  File "/Users/alex/work/logfire/logfire/_internal/utils.py", line 241, in handle_internal_errors
    yield
  File "/Users/alex/work/logfire/logfire/_internal/main.py", line 612, in log
    msg, extra_attrs, msg_template = logfire_format_with_magic(
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/alex/work/logfire/logfire/_internal/formatter.py", line 365, in logfire_format_with_magic
    chunks, extra_attrs, new_template = chunks_formatter.chunks(
                                        ^^^^^^^^^^^^^^^^^^^^^^^^

This is pretty hard to read. I'll see if I can improve it on Monday.

Specifically I want to try making it look like this:
Internal error in Logfire.
Traceback (most recent call last):
File "/Users/alex/Library/Application Support/JetBrains/PyCharm2024.1/scratches/scratch_222.py", line 5, in <module>
    logfire.info("log {'x':{'message'}}")
  File "/Users/alex/work/logfire/logfire/_internal/main.py", line 336, in info
    self.log('info', msg_template, attributes, tags=_tags, exc_info=_exc_info)
  File "/Users/alex/work/logfire/logfire/_internal/main.py", line 612, in log
    msg, extra_attrs, msg_template = logfire_format_with_magic(
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/alex/work/logfire/logfire/_internal/formatter.py", line 365, in logfire_format_with_magic
    chunks, extra_attrs, new_template = chunks_formatter.chunks(
                                        ^^^^^^^^^^^^^^^^^^^^^^^^

This is pretty hard to read. I'll see if I can improve it on Monday.

Is this stale, or... @dmontagu @alexmojaki ?

I have some WIP separate from this.


ament_package: 0.9.4-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository ament_package to 0.9.4-1:

upstream repository: https://github.com/ament/ament_package.git
release repository: https://github.com/ros2-gbp/ament_package-release.git
distro file: foxy/distribution.yaml
bloom version: 0.10.2
previous version for package: 0.9.3-1

Please hold this PR. I'd like to do releases for some other core packages before rebuilding the world.

Thanks @audrow !


Fix bundle --help

Fixes the issue mentioned in https://github.com/bundler/bundler/issues/4801

@homu r+

:pushpin: Commit 831c272 has been approved by indirect

:hourglass: Testing commit 831c272 with merge 4e85e96...

:sunny: Test successful - status


Code: Improve type hints for Creators

Changelog Description
Improve type hints for Creators
Additional info
This improves the type hints in my IDE (PyCharm). However, I wonder if these changes are worth it since typing seems to be quite a bit in flux in Py3 and I'm not sure whether all used here was available in older Py3 versions.
Note that the use of Union may make this Py3.5+ - removed the usage of Union in favor of Optional
Testing notes:

Creators should work across different hosts.

Fixed open issues, some more cleanup and merged in #696 which basically was me implementing some of the same things 🤦
Did a test publish myself, all seemed fine. Might be good to check again by someone else.
@iLLiCiTiT could you review the code style again? sorry about the long wait on this PR.

Can you make few smaller PRs? Right now github refuses to show differences and when something is loaded, it shows added files and lines.

Can you make few smaller PRs? Right now github refuses to show differences and when something is loaded, it shows added files and lines.

Github shows me very few changes actually - and the changes are small.

I'm not sure how to separate this further without creating a PR per file almost. Did you comment on the wrong PR or is Github doing something crazy for you?

Like 1 hour ago I saw at about 100 changed files...

Failing on linting.


Forge

This PR adds support for the MinecraftForge platform, the purpose of this is to be able to run LuckPerms without the need to use alternative platforms such as Bukkit / Forge Hybrids or Sponge.

Are there any news about how this is going?

Are there plans to make this actually happen?

Would be very nice since Velocity is currently debating adding forge support for modern FML

sadly, hybrid servers like mohist, magma, or spongeforge hate modpacks. i always had issues like players unable to login.
i tried going that route first so i could have a few plugins like luckperms.

Well personally, I'm running ~5 modpack servers on Magma 1.12, and had much more running without issues.
However as hybrids are nowhere near playable on 1.16+ for larger servers, having a native Forge mod would be much better

@LXGaming Thank you for resolving lots of my comments and merging master back in!
I will try to take another look soon, I think we're pretty close! :D

forge approved pull request  https://github.com/MinecraftForge/MinecraftForge/pull/8599
how have feedback?

how have link for download?
for testing mod

how have link for download? for testing mod

There isn't one, It's still being actively developed.

how have link for download? for testing mod

@joaoespanhol Please stop pestering - it'll be ready when it's ready. Asking for updates isn't going to make things progress any faster.


SIGSEGV from EVP_MAC_CTX_get_mac_size()

My test environment is OpenSSL 3.1.1 on Fedora.
James Muir reports it fails on the "master" branch.  (2023-Nov-27)
The example is a ctx that is ready but hasn't been used yet.
Calling EVP_MAC_init() fixes the problem.
Example file is a .txt because the web form said it didn't like .c
bug.c.txt

This seems like an expected error (or at least unsurprising behavior).  EVP_MAC_init initializes the context in which this instance of the EVP_MAC is used.  The documentation could be more clear here, but any use of the context structure prior to its initialization is effectively using uninitialized data.
In this specific case the segfault is occurring because EVP_MAX_CTX_get_mac_size is attempting to fetch the mac size from the underlying algctx struct, which has not yet been set up properly via EVP_MAC_init

The documentation could be more clear here, but any use of the context structure prior to its initialization is effectively using uninitialized data.

I don't think I agree here. This seems to occur after EVP_MAC_CTX_new() which should return a properly initialised (and empty) EVP_MAC_CTX object. Failing to initialise the mac with EVP_MAC_init is certainly an error, but I would not expect EVP_MAC_CTX_get_mac_size() to seg fault if you fail to do so.

I think what you are describing is exactly what is happening.  Calling EVP_MAC_CTX_new returns a new empty ctx.  Most notably that ctx has an internal algctx for which the cipher is set to NULL.  Its only after EVP_MAC_init is called through the call chain EVP_MAC_init->cmac_init->cmac_set_ctx_params, that the provider loads the cipher from the provided params and assigns a cipher to the ctx->algctx->cipher struct.
This seems to be the same pattern for other objects, so this same behavior may be present in other apis as well (just doing a visual inspection of the EVP digest code, I've not tested).
regarding the EVP_MAC api here, it would seem like the thing to do might be to augment cmac_get_ctx_params such that  an error is returned if the algctx cipher struct isn't set yet.  But that really just turns the segfault into an error case, the application fix would still need to be calling EVP_MAC_init.

But that really just turns the segfault into an error case, the application fix would still need to be calling EVP_MAC_init.

Right - That's all was saying. It should not seg fault, it should return an error instead.

we discussed this some on the "openssl-users" list.
I'm not sure if an error should be emitted.
the documentation says EVP_MAC_CTX_get_mac_size() should return 0 in this situtation.
https://www.openssl.org/docs/manmaster/man3/EVP_MAC_init.html

EVP_MAC_CTX_get_mac_size() returns the expected output size, or 0 if it isn't set. If it isn't set, a call to EVP_MAC_init() will set it.

That sounds right though.  A zero return code is in keeping both with the docs, and with the typical return code scheme openssl uses (i.e. 0 == something has gone wrong).
So it sounds like we're all on the same page then, that EVP_MAC_init is required for EVP_MAC_CTX_get_mac_size to not fail, and EVP_MAC_CTX_get_mac_size should return 0 if EVP_MAC_init hasn't been called on the context

Actually the implementation could be able to return the MAC size even after the set params call which sets the AES cipher. So it could be made to work even without the init() call.


Gnome 4X Compatibility

hey
any chance you can update the extension to use with Gnome 46+
thanks

I second this. Your extension is a fantastic tool for me on my work machine. I would love to be able to use it privately as well.
Thanks a lot for the hard work you have already put in for the community! ^^

??


Hudson Rock Cybercrime Intelligence Free Integration

Consider adding Hudson Rock's complimentary data to receive additional intelligence about the email address that was compromised in global Infostealer attacks.
Email sample<EMAIL_ADDRESS>Free API key and full documentation is available here: https://cavalier.hudsonrock.com/docs
Thank you.

@dmdhrumilmistry, The rate limit is 50 requests per 10 second

@dmdhrumilmistry, The rate limit is 50 requests per 10 second

Okay, will integrate it over the weekend


Survey: UI - display logged in user's surveys

Display survey instances that the user needs to fill along with their status
Change the survey link in the mailto link

Would Survey Subjectbe cleared than Survey Entity - your call.


Move test collector to Scarb

Testing: after merging https://github.com/software-mansion/scarb/pull/886 a nightly will be released which we will use. Tested locally, everything works for me
Introduced changes
I know that some of the code moving is unrelated, but it greatly reduced the cognitive power I had to use to write this PR

moved test collection and compilation to Scarb
moved structs related to raw and runnable test crates to separate files in forge and forge-runner (structs from test-collector are there too)
moved structs related to expected test result to separate file from test-collector to forge-runner

Checklist

[x] Linked relevant issue
[ ] Updated relevant documentation
[x] Added relevant tests
[x] Performed self-review of the code
[ ] Added changes to CHANGELOG.md

Cast doesn't work bcs of VersionedSierra serialization changes, consulting with @maciektr about possibly reverting these changes in Cairo's repo

Docs changed, not everywhere it is correct 💀

I'll be closing this PR anyways, since we have to retain old collector as well.


Feature/increase test coverage

Codecov Report

Merging #129 (c41645a) into main (541bbb4) will increase coverage by 6.56%.
The diff coverage is 100.00%.

:exclamation: Current head c41645a differs from pull request most recent head 2f21ea2. Consider uploading reports for the commit 2f21ea2 to get more accurate results

@@            Coverage Diff             @@
##             main     #129      +/-   ##
==========================================
+ Coverage   60.37%   66.93%   +6.56%
==========================================
  Files          74       74
  Lines        3846     3850       +4
  Branches      493      515      +22
==========================================
+ Hits         2322     2577     +255
+ Misses       1472     1228     -244
+ Partials       52       45       -7

Impacted Files
Coverage Δ

...ackages/cli/src/tasks/all/resolveTemplateConfig.ts
98.07% <100.00%> (+98.07%)
:arrow_up:

packages/cli/src/template/config.ts
98.30% <100.00%> (-1.70%)
:arrow_down:

packages/cli/src/node/packageManager.ts
19.59% <0.00%> (+19.59%)
:arrow_up:

packages/cli/src/tasks/all/moveOutput.ts
100.00% <0.00%> (+100.00%)
:arrow_up:

packages/cli/src/tasks/all/fetchTemplate.ts
100.00% <0.00%> (+100.00%)
:arrow_up:

packages/cli/src/tasks/all/loadGraphqlSchema.ts
100.00% <0.00%> (+100.00%)
:arrow_up:

packages/cli/src/tasks/all/installDependencies.ts
100.00% <0.00%> (+100.00%)
:arrow_up:

packages/cli/src/tasks/all/selectPackageManager.ts
100.00% <0.00%> (+100.00%)
: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 541bbb4...2f21ea2. Read the comment docs.


Update how-to-develop-emulator.md

If the "LimitToEndpoint = true" configuration option is not added to the CosmosClientOptions and the user is testing locally, the cosmos client never returns a response and just continually spins.  It appears that it just keeps trying to connect, but doesn't timeout correctly or has no limit.  I was able to reproduce in the cosmos db .net sample for createitem and was able to fix this issue by adding this option.

@sajeetharan

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


feat(HMS-1978): Add launch templates to GCP

This PR introduces a launch template dropdown to the GCP menu.
A big thank you to @amirfefer for implementing the dropdown in such a generic, clean, and clear manner!

Reads well, @ezr-ondrej or @amirfefer ?


Updated docs spo eventreceiver remove, Closes #4050

This PR Updates docs spo eventreceiver remove.
Closes #4050

Thank you @nicodecleyre
We'll review it ASAP!


Bump bokeh 2.4.2

Recent test failures caused by an update to the jinja2 package can be traced back to our pinned version of bokeh==2.3.3. This PR bumps bokeh to the latest version and fixes a related bug.

whoops, the changelog push with [skip ci] seems to have dismissed the running tests. They're still running here: https://github.com/askap-vast/vast-pipeline/actions/runs/2110641926


catkin_make failed

CMake Error at /opt/ros/kinetic/share/catkin/cmake/catkinConfig.cmake:83 (find_package):
Could not find a package configuration file provided by
"open_manipulator_msgs" with any of the following names:

make success when clone msgs and checkout branch

Hello, Robotis team, I ran into the same issue when building the open_manipulator package.  Have you restructured the package/metapackage to pull open_manipulator_messages out?
Would you update http://wiki.ros.org/open_manipulator to show "open_manipulator_msgs" as a dependency


Update d13-spearfish-sunday-literature.yaml

Separating the groups and cleaning up meeting time.

Separating? Is something missing?

I'm closing this as abandoned. Please re-open if anything changes.


Starting frame number ignored, flashing between types in scroll mode

Overview
Use the following code on any animation:
LottieInteractivity.create({ player:'#lottie_exploration', mode:"scroll",     actions: [     {         visibility:[0, 0.5],         type: "stop",         frames: [70]     },     {         visibility:[0.5, 0.6],         type: "seek",         frames: [75, 85]     },     {         visibility:[0.6, 1.0],         type: "stop",         frames: [95]     }     ] });
You will notice that the first action (stop on frame 70 for visibility 0 to .5) works correctly.
The second action (seek) will start at frame 0, not frame 75
And there will be a flash between the second action and the third action, even though one should end at 0.6 and the other begin at 0.6
...
Consuming repo

https://unpkg.com/<EMAIL_ADDRESS>
...
Labels

[ ] Add the Type: Bug label to this issue.

@samuelOsborne : animation here https://gist.github.com/andybrowncompassion/5dd379c161943d479a3d9363dcd182f5

@andybrowncompassion Seems to work ok for me - please check out this codepen:
https://codepen.io/Osbro/pen/abYZqwQ?editors=1010
Can't see any of the flashing problems


Balance druid sims, T8 resto set pieces showing higher DPS than Balance tier set.........

Like I said in title, its showing Resto tier set as DPS increase over Balance tier set pieces.

@Ifartstarz are you actually running the sim or just looking at EP values? Those are not an accurate measurement of DPS and only are used for ordering items in the menu

@Ifartstarz are you actually running the sim or just looking at EP values? Those are not an accurate measurement of DPS and only are used for ordering items in the menu

When I click on an item slot, its showing the resto gear higher

@Ifartstarz you shouldn’t rely on the item ordering since that won’t take into account item set bonuses and other stuff. The most accurate way to compare gear is to press the simulate button with different items worn


[server] Skip success request metric tracking for /health request

/health request from Router doesn't have a store name, so the store-level tracking would break with a gaint log stacktrace, which is bloating log file, and this is a regression happened a few months ago.

How was this PR tested?
CI

Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

I debugged further and this PR is not necessary.
The root cause is as follow with the version running in prod:

Fast-Client is using /health endpoint for conn warming.
Prod version doesn't have this code change: https://github.com/linkedin/venice/issues/798, which will bypass acl check.
Prod version will fail all the /health requests from fast-client.
When Venice Server fails the health request in acl handler, it won't setup serverStatsContext.isHealthCheck().
Without this flag, StatsHandler will go ahead to call serverStatsContext.successRequest, but it will fail since there is no store name.

Once we deploy #798 , we shouldn't see such issue in prod any more.


error messages when webhooks fail do not provide details

Watchman Version: latest
What were you trying to do?
receive webhooks on my local api with watchman running in docker (windows dev environment)
What did you expect to see?
error details. My assumption is that there is some more information on the issue with the http request that would make debugging simpler. If there are other debug flags that would help that would be enough but I didn't see anything configurable for that.
What did you see?
ts=2021-05-26T13:25:28.0110892Z caller=search_async.go:74 search="async: name watch 'Jane Smith' for company 0d55c2910a56d7566d75f234dc4431eb99f2ca6a found"
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x98235e]

goroutine 47 [running]:
main.callWebhook(0xc003e582a0, 0x28, 0xc001b15830, 0xc003e582d0, 0x2d, 0xc0019aa520, 0xd, 0x0, 0x0, 0x0)
        /go/src/github.com/moov-io/watchman/cmd/server/webhook.go:62 +0x24e
main.(*searcher).spawnResearching(0xc0003f8380, 0xc5b900, 0xc0003f3050, 0xc615e0, 0xc002008120, 0xc61620, 0xc002008160, 0xc6e460, 0xc0020080e0, 0xc5bc80, ...)
        /go/src/github.com/moov-io/watchman/cmd/server/search_async.go:90 +0x2ce
created by main.main
        /go/src/github.com/moov-io/watchman/cmd/server/main.go:184 +0x119a

How can we reproduce the problem?
add a watch with a webhook that will not succeed. This same error occurred for a few scenarios

ssl validation issues
timeout
connection failure (api not running)

@cushdan can you help me figure out which version of Watchman is being used? I'm seeing that exact line being a nil check, which wouldn't panic like your stack trace.
https://github.com/moov-io/watchman/blob/6d4b3332428916c5c2f9916025a2a6977df6c38e/cmd/server/webhook.go#L60-L63

I did wrap the response body Close() in a nil check, which could be a panic. See https://github.com/moov-io/watchman/commit/2403c40839d981016c070e78ea36ca2b1d104b22

I pulled the latest recently. This is what it shows up as in docker
moov/watchman | latest | 52c0d4cea379 | about 1 year ago | 161.44 MB
latest: Pulling from moov/watchman
Digest: sha256:1fe03d96f4a64b08c02109bcd0e4e8bde1a2b8898534f6880e95e27fc4f5a4fc
Status: Image is up to date for moov/watchman:latest
docker.io/moov/watchman:latest

sounds like you might have fixed it though? I can try and recreate it when it's available

https://hub.docker.com/layers/moov/watchman/latest/images/sha256-1fe03d96f4a64b08c02109bcd0e4e8bde1a2b8898534f6880e95e27fc4f5a4fc?context=repo
That is the most recent release, so I'm surprised it panic'd there. Perhaps that stacktrace was triggered by another error?
I can make a bugfix release with that nil check today.

v0.18.2 is out with that bug fix -- https://github.com/moov-io/watchman/releases/tag/v0.18.2


Enable static build option for color model functions

@r-abishek conflicts need to be resolved

@r-abishek conflicts need to be resolved

@kiritigowda Should be resolved now

@rrawther Thanks for notifying me, but it does not seem that I'm involved in this project in any way.
https://blog.github.com/2011-03-23-mention-somebody-they-re-notified/


Index out of range exception occurs in the md5 methods of BsUtil.cpp class

Details here: https://github.com/BearishSun/BansheeEngine/issues/201
Pull request with fix here:
https://github.com/GameFoundry/bsf/pull/86

This issue should now be resolved.


Github Actions to build linux & windows binaries?

Would be great to automatically compile windows and linux binaries with github actions. Wasn't able to get it to work with C++20, though.

Might actually have to specify "ubuntu-20.04" since "ubuntu-latest" is, in fact, not the latest: https://github.com/actions/virtual-environments
If linux builds won't work, at least windows builds might work using "windows-latest"

20.04 LTS would make the most sense for broad compatibility. Though an explicit apt upgrade/install might be necessary.
Regarding past attempts using GitHub, did you try passing in a flag to the compiler to enable C++20? GCC 10 should compile with the new standard if you pass this to it:
-std=c++20

Alternatively, there are additional GNU extensions on top of the 2020 spec that are enabled with:
-std=gnu++20

For what it's worth, I have yet to try this myself. I've only used VS 2019 locally. In theory, it should be all the same across any platform, but in reality there are small differences in MS C++20 because they added features to their IDE that were not finalized. It's not strictly conformant.

Here is a one-stop solution to compile and run PotreeConverter inside a docker container. I tested it on Ubuntu 18.04 and 12.04 and Mac OS X, works fine.
It doesn't exactly solve the problem of creating binaries/github actions, but it's very handy for all the folks having problems compiling on linux/Mac.
The Dockerfile at the end of the post is standalone. Here's how to use it to do potree-convert,
Build the docker image (takes quite some time):
docker build . -t potreeconverter --network host

Run the docker image it with your las files' folder mounted inside (adapt to your path):
docker run -it --rm -v '/absolute/path/to/folder/with/your/las/file/':/PotreeConverter/build/files potreeconverter bash

From inside the container, potree-convert (adapt filenames), and then exit the container:
./PotreeConverter files/input_file.las -o files/potree_octree_folder
exit

chown the file so you're the owner:
sudo chown -R your_username potree_octree_folder

I freezed the versions of the utilities/binaries used in this docker.
As of 23.11.2020, those are:

cmake 3.19.0
tbb git commit 141b0e310e1fb552bdca887542c9c1a8544d6503
PotreeConverter 2.0.2, git commit da93ec26411b82ffbf799daf5ac56608ef988bf8
Feel free to update them to more recent versions, but not guarantee it'll still work.

The Dockerfile:
# build environment
FROM ubuntu:20.04 as ubuntu

# install gcc, openSSL, git
RUN apt-get -y update
RUN apt-get -y install wget
RUN apt-get -y install git
RUN apt-get -y install build-essential
RUN apt-get -y install libssl-dev

# install cmake
RUN wget https://github.com/Kitware/CMake/releases/download/v3.19.0/cmake-3.19.0-Linux-x86_64.tar.gz
RUN tar -xzf cmake*.tar.gz
RUN mv cmake*/ cmake/

# install tbb
RUN git clone https://github.com/wjakob/tbb.git
WORKDIR /tbb/build
RUN git checkout 141b0e310e1fb552bdca887542c9c1a8544d6503
RUN ../../cmake/bin/cmake ..
RUN make
RUN make install
WORKDIR /

# compile PotreeConverter
RUN git clone https://github.com/potree/PotreeConverter.git
RUN mkdir /PotreeConverter/build
WORKDIR /PotreeConverter/build
RUN git checkout da93ec26411b82ffbf799daf5ac56608ef988bf8
RUN ../../cmake/bin/cmake ..
RUN make

# copy libtbb.so in PotreeConverter build
RUN cp /tbb/build/libtbb.so /PotreeConverter/build/libtbb.so

CMD ["sleep", "infinity"]

# Building:
# ---------------------
# docker build . -t potreeconverter --network host
# -> option --network host needed to access network for apt-get, wget and git
#
# Running docker image and converting a las file to potree format inside docker:
# ---------------------
# docker run -it --rm -v '/absolute/path/to/folder/with/your/las/file/':/PotreeConverter/build/files potreeconverter bash
# ./PotreeConverter files/input_file.las -o files/potree_octree_folder
# -> the potree octree folder will appear in /absolute/path/to/folder/with/your/las/file/potree_octree_folder
# -> outside of docker you will need to set yourself as the owner of the potree folder:
# sudo chown -R your_username potree_octree_folder
#
# Notes:
# ---------------------
# To ensure it works, I freezed the versions of the utilities/binaries used in this docker.
# As of 23.11.2020, those are:
# - cmake 3.19.0
# - tbb git commit 141b0e310e1fb552bdca887542c9c1a8544d6503
# - PotreeConverter 2.0.2, git commit da93ec26411b82ffbf799daf5ac56608ef988bf8
# Feel free to update them to more recent versions.

docker: Error response from daemon: invalid mode: /PotreeConverter/build/files.
Any idea why this error is coming up?
Thanks.

docker: Error response from daemon: invalid mode: /PotreeConverter/build/files.
Any idea why this error is coming up?
Thanks.

When does this happens? on docker build, docker run or when converting?
My guess is it is linked to the mounting of the las file folder in: docker run -it --rm -v '/absolute/path/to/folder/with/your/las/file/':/PotreeConverter/build/files potreeconverter bash
Did you specify a valid path for the las file folder? (in place of '/absolute/path/to/folder/with/your/las/file/')
Are you on Windows? you might want to check this SO answer

When does this happens? on docker build, docker run or when converting?
My guess is it is linked to the mounting of the las file folder in: docker run -it --rm -v '/absolute/path/to/folder/with/your/las/file/':/PotreeConverter/build/files potreeconverter bash
Did you specify a valid path for the las file folder? (in place of '/absolute/path/to/folder/with/your/las/file/')
Are you on Windows? you might want to check this SO answer

You absolutely don't have to run with a docker container if you just want to compile binaries on Windows or Linux. To me, it just adds an extra layer of abstraction where problems can occur.
Still, this isn't exactly on the topic of GitHub Actions. If you're still having trouble, search for a related issue or create a new one to help keep things tidy.

You absolutely don't have to run with a docker container if you just want to compile binaries on Windows or Linux. To me, it just adds an extra layer of abstraction where problems can occur.
Still, this isn't exactly on the topic of GitHub Actions. If you're still having trouble, search for a related issue or create a new one to help keep things tidy.

You absolutely don't have to run with a docker container if you just want to compile binaries on Windows or Linux. To me, it just adds an extra layer of abstraction where problems can occur.

You need quite a few packages to compile PotreeConverter (on linux at least), each of which creates a new chance of problem.
It seems to me that setting up a list of clunky packages and making sure everything works is exactly the kind of problems docker has been created for and why it's so popular.
Maybe it's easier on Windows. On Linux, getting PotreeConverter to compile took me a week scrolling through issues here, on TBB's github issues or stackoverflow. That's why I created this docker.
So, absent github actions and linux binaries, this docker goes a long way ;-)

You absolutely don't have to run with a docker container if you just want to compile binaries on Windows or Linux. To me, it just adds an extra layer of abstraction where problems can occur.

You need quite a few packages to compile PotreeConverter (on linux at least), each of which creates a new chance of problem.
It seems to me that setting up a list of clunky packages and making sure everything works is exactly the kind of problems docker has been created for and why it's so popular.
Maybe it's easier on Windows. On Linux, getting PotreeConverter to compile took me a week scrolling through issues here, on TBB's github issues or stackoverflow. That's why I created this docker.
So, absent github actions and linux binaries, this docker goes a long way ;-)

I made a pull request using an improved version of the above proposed docker image in pull request 586.
In the meantime, it is available in the forked repo.

@dddpt thanks for putting in the work on this.
Is it possible to use this container as a build environment for PotreeConverter so that the tools could extracted and used directly on a host?
I am in a position where I need to deploy PotreeConverter in an restricted environment that can have some changes made to it but not the free for all that is the current build mess.
(caveat I would need to run this on 18.04 but if the concept is sound I can put the effort into getting this working using your dockerfile as a roadmap)


Enum array gets output as []interface{}

Version
1.26.0
What happened?
When I select an enum array column in Postgres (pgx) with sqlc the generated type for that column is []interface{}. For some reason this doesn't reproduce in the playground.
I came across this issue but I am new to Go and I'm not quite sure what to do with this code, or if it is even relevant to the issue I am seeing: https://github.com/sqlc-dev/sqlc/issues/2116#issuecomment-1493299852
This is the output:
type UploadDataRow struct {
	ID       pgtype.UUID
	Variants []interface{}
}

Relevant log output
No response
Database schema
CREATE TYPE upload_variant AS ENUM (
    'VIDEO_4K',
    'VIDEO_4K_DOWNLOAD',
    'VIDEO_1080P',
    'VIDEO_1080P_DOWNLOAD',
    'VIDEO_720P',
    'VIDEO_720P_DOWNLOAD',
    'VIDEO_480P',
    'VIDEO_480P_DOWNLOAD',
    'VIDEO_360P',
    'VIDEO_360P_DOWNLOAD',
    'AUDIO',
    'AUDIO_DOWNLOAD'
);

CREATE TABLE upload (
  id   BIGSERIAL PRIMARY KEY,
  variants upload_variant[]
);

SQL queries
-- name: GetUploads :many
SELECT * FROM upload;

Configuration
version: "2"
sql:
  - engine: "postgresql"
    queries: "./app/data"
    database:
      uri: ${DATABASE_URL}
    gen:
      go:
        package: "data"
        out: "./app/data"
        sql_package: "pgx/v5"

Playground URL
https://play.sqlc.dev/p/e60a8ebfe202c5a91bdaaee9b3aef12b085f188ceec8e62b4503c59c6f1d553c
What operating system are you using?
Linux, macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

It seems like this might be related to not specifying a schema.sql file. Does that sound right?

I got everything generating correctly and now I understand the purpose of that code in the linked issue. Any custom types need to be registered with pgx or else there will be a runtime failure when pgx tries to read the value into the struct.
I did this as such:
func getDbConn(ctx context.Context) *pgx.Conn {
	conn, err := pgx.Connect(ctx, os.Getenv("DATABASE_URL"))
	if err != nil {
		log.Fatalln(err)
	}

	dataTypeNames := []string{
		"upload_variant",
		"_upload_variant",
	}

	for _, typeName := range dataTypeNames {
		dataType, err := conn.LoadType(ctx, typeName)
		if err != nil {
			log.Fatalln(err)
		}
		conn.TypeMap().RegisterType(dataType)
	}

	return conn
}


test(e2e): add progress steps e2e test

@chasestarr I think that makes perfect sense - how do you want to go about it? should we try that for the next component we cover with e2e tests?


How to remove LES WRIGHT 2021 from the Project?

You may not.
This project is Apache 2.0 Licensed, and Copyright notifications must be retained under the terms of this license.
(see: LICENSE)

As above. The software is free and useful, why expunge the author, except to plagiarize?


Resin-cli doesn't support http/https proxies

Resin CLI behind Firewall, doesn't use http, https proxies.
Upon investigation into firewall logs, its apparent that resin-cli client doesn't use http_proxy and https_proxy settings and tries to connect directly to internet
resin --help
ECONNREFUSED: request to https://api.resin.io/whoami failed, reason: connect ECONNREFUSED <IP_ADDRESS>:443
If you need help, don't hesitate in contacting us at:
GitHub: https://github.com/resin-io/resin-cli/issues/new
Forums: https://forums.resin.io
DEBUG=1 resin --help
Error
at ClientRequest. (/Users/user-name/.nvm/versions/node/v7.7.3/lib/node_modules/resin-cli/node_modules/node-fetch/index.js:133:11)
at emitOne (events.js:96:13)
at ClientRequest.emit (events.js:191:7)
at TLSSocket.socketErrorListener (_http_client.js:358:9)
at emitOne (events.js:96:13)
at TLSSocket.emit (events.js:191:7)
at emitErrorNT (net.js:1289:8)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickDomainCallback (internal/process/next_tick.js:128:9)
If you need help, don't hesitate in contacting us at:
GitHub: https://github.com/resin-io/resin-cli/issues/new
Forums: https://forums.resin.io

Behind enterprise Firewall: From the Firewall logs, it seems that my machine (source ip) is directly trying to connect to destination ip (api.resin.io) without even routing requests through proxy.
I setup a local proxy server(using charles proxy) at home yesterday and was able to confirm again that rein-cli doesn’t go through proxy at all.
The cli client should either have settings to configure http_proxy, https_proxy or pick those up from environment variables of terminal shell from where request is initiated. I believe it should be a relatively easy code fix.
Reference NPM: https://www.npmjs.com/package/https-proxy-agent

Hi @nkaul, thanks for this! To be clear: if you could configure resin-cli to send everything through an HTTP and HTTPS proxy everything would work fine for you, is that right?
Given that, yes, you're entirely right, we don't yet have built in support for proxies. Keep an eye on this ticket and we'll take a look at adding that in soon.

@nkaul the proxy support has been released in 5.9.1. Can you please give it a try and let us know how it works?


GDPR - Iubenda problem, cookies and payment process

Describe the bug and add screenshots
Hello, I have been using this module for a client on a project in the last years, and it worked so far without big issues.
Now I have something to ask as regards GDPR, Iubenda, cookies and the fact that it is impossibile to proceed with the payment if cookies are not accepted.
Our prestashop installation has this module and official iubenda module: https://www.iubenda.com/en/help/4338-cookie-solution-prestashop-plugin-installation-guide
I need to stress that Iubenda is the de facto standard here in Italy and maybe elsewhere (https://discovery.hgdata.com/product/iubenda).
When landing on our site the cookie banner asks to handle cookies, and if you "reject all" there is now way to pay with paypal, unless if you go back to cookie banner and accept "Experience" cookie, so that the yellow paypal button shows up and it is possibile to proceed.
These steps are complicated to understand and complete by our ecommerce users, which lead to lot of abandoned carts.
I contacted Iubenda and they told me that this is perfectly fine from their side because if paypal cookies are not accepted by the user it is normail that things stop working. So dead end.
All this issue is kinda new (last couple of months) and related, in my opinion, to the fact that iubenda adopted Google Consent Mode V2, which let technical cookies to be handle in a different way as before (as with Google Analytics).
We are not experiencing the same problem with the other Stripe module we adopt.
What do you think about this?
Can i guess that the problem could be related to Paypal API or the way this module interact with PayPal API?
Thanks for the support.
Expected behavior
I should be able to pay with paypal module even if cookies are rejected, since it should be a technical feature, not compromising the functionality of the ecommerce payment system
Steps to reproduce

visit https://www.colombomilano1911.com/en/ and do not accept cookies ("reject all")
put a product in the basket and proceed with the checkout,
reach the payment step (number 4) and select "Pay with Paypal": you will see that the yellow button is not available

click on the iubenda floating button on the left and accept "Experience" cookie

refresh page anche click again on PayPal radio: you will see that the yellow button is available

PrestaShop version(s) where the bug happened
<IP_ADDRESS>
PHP version(s) where the bug happened
7.3

Hi @masiorama
This issue is probably due to a conflict between both modules but we doesn’t know the gdpr you use.
Could you please contact our support team here to share with us more information : https://addons.prestashop.com/en/contact-us?id_product=1748
Do not hesitate link your support request to this issue because your explaination here looks clear.
Thank you.

Thanks @clotairer done.


display full option of combo box option after deselect option

Also have printed a node version in package.json for future use

Nice one

@jyooi this change is now published in<EMAIL_ADDRESS>👍


Windows See the current T&C in the wallet after you accept it.

It should be possible to see the current T&C in the wallet after you accept it.
Maybe add another T&C pop up when they have been updated?

Fixed as part of 'terms of service' navigation #96


Public repository

Hi, folks! I'm the lead programmer of Atomic Robotics (team 3805). I just thought I'd let you know that your repository is public. If this is intended, carry on-- see you at competitions! I know a lot of teams keep their code private, though, so I thought I'd let you know.
Good luck for this season, it's got some pretty unique challenges and I'm excited to see everyone's solutions for them.
~Rowan, 3805 software lead

Yes, that's intentional. Our code is always open source, but thanks for the heads up.


Bad Request 400 status code: "You must provide a model parameter" when attempting any openAI .call method

Greetings,
When using Node 18.1.0, or lower, on both Ubuntu 20.0.4 and Windows 10:
Attempting to use the method .call()  results in a 400 status code.

Here is my code:
import { ChatOpenAI } from "langchain/chat_models"
import { HumanChatMessage } from 'langchain/schema'
import dotenv from 'dotenv'
dotenv.config()

const chat = new ChatOpenAI({
    openAIApiKey: process.env.OPENAI_API_KEY,
    temperature: 0.9

})
async function getChat() {
    const response = await chat.call([
        new HumanChatMessage(
            "Translate this sentence from English to French. I love programming."
        ),
    ]);

}

getChat()

Things I have tried:

Instead of passing the openAIApiKey as a parameter with .call(), I instead ran package.json script: npm run dev which was defined as node -r dotenv/config index.js
Downgrading to Node 16.8.0 and setting NODE_ENV to experimental
Downgrading the version of langchain to 0.0.22

TEMPORARY SOLUTION
After changing my Node version to 18.2.0, the error does NOT happen!
What I suspect may be the cause:
Something with AxiosFetchAdapter...that's the only thing I can think of!

Hi
In general it's not recommended to stick with old minor versions of Node. We only support running Langchain on the most recent versions of Node.js 16.x, 18.x, 19.x

Still getting this error with node v18.16.0

Duplicate of #1010

same error


move sdk badge to Meltano Stats section, add maintainer ID

@rwfeather and @cjohnhanson - The location and width are still off but the rendering overall I think is improve and this PR is ready for review.
Related:

#785


rapids_cpm_spdlog properly specify usage of external fmt library

Description
Previously we specified the wrong options for spdlog to use an the external fmt library. This caused spdlog to use the internal version while we had other non-spdlog components uses the upstream fmt.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.

/merge


What are the reasonable results of the "three_key_questions" experiment?

Thank you for the wonderful project and for making the code publicly available.
As I am unfamiliar with projects involving LLM multiagents, this may be a simple question, but I would like to ask nonetheless.
I am testing with GPT-3.5 using the 'three_key_questions.ipynb'. In this setup, Alice is supposed to have a conversation apologizing to Bob. However, in my three trials, they(4agents) just keep playing various games lol. Even extending the 'episode_length' to 10 didn't change this outcome.
The technical report (https://arxiv.org/abs/2312.03664) didn't include the results of this experiment, so could you please let me know if this is replicating your results? (Including the version of GPT used).
I understand that, unlike typical multiagent reinforcement learning code, it might be difficult to replicate results in this case.
Sincerely,

I don't know what happens if you use GPT-3.5, but I believe I mostly observed the agents arguing with each other in this example.
We are hoping to be able to add example outputs for specific language models in the future.

Hi again, we just added an example of the experiment output here:
https://github.com/google-deepmind/concordia/blob/main/examples/three_key_questions_example_output.html


[v2] Button Component

Work-in-progress https://github.com/ZanderBrown/Framework7/tree/v2

@nolimits4web
What about adding this type of buttons?

@nolimits4web
Great! Thank you very much :)

When touch and hold two button together ripple stuck on one of them !


Standardise videos with a view component

Trello card
Trello-3571
Context
We are currently including custom HTML for YouTube videos at each occasion that we want to render a video on a page. This leads to inconsistencies between the various snippets of HTML.
Add YoutubeVideoComponent to standardise how we render YouTube videos and whitelist the compnent in the Markdown template handler so that we can use it in our content pages.
Changes proposed in this pull request

Standardise videos with a view component

Guidance to review
This may inadvertently fix an issue flagged in Google Search Console whereby only some of our videos are being correctly indexed. I've take the HTML snippet for the standard iframe from one of the indexed videos.
If this doesn't fix the issue (Google are vague about what constitutes a 'prominent' video) it may be worth removing the loading: lazy attribute as it could be that videos above the fold are being indexed and videos further down are not because they don't get loaded immediately.
Pages with videos on them:
Blog - Top Tips for Returning Teachers
Returning to Teaching
Teacher Training Advisers x 2
About TTT Events

Videos all look good to me @ethax-ross
Could you update the content guidance either before/after this is merged?
(cc @EllieNodder for info)


Adding visualizer package

Next Steps:

Add compatibility for single integer input in "ROIs to Plot" (reference lines 133-141 in data.py)
Fix compatibility for list input of ROIs in "ROIs to Plot" (reference lines 133-141 in data.py)
Reorganize visualizer > data > data.py classes into separate files (classes-s2p and classes-general)
Matplotlib contour map for s2p data should represent each ROI with different colors

The next steps listed above have been implemented, and errors with the line plots for event rel analysis have also been fixed.


Variables don't work in match

I have been battling for a while now with a schematron rule that does not seem to evaluate my variable correctly leaving me with no correct match.
My example and rule should always fail the assertion, and write out the message (as a "Test") ... as I want one question for every one reference (and visa versa) ... but the count always comes up as ZERO!
Where my rule is as follows:
<pattern id="Question">
  <rule id="Count" context="question/@id">
    <let name="href" value="concat('#', ancestor::parent/@id, '/', .)" />
    <assert test="false()">
      question/@id=[<value-of select="." />] has [<value-of select="count(ancestor::parent//reference[@href eq $href)])" />] references with reference/@href=[<value-of select="$href" />])
  </assert>
</pattern>

My input XML is as follows:
<parent id="GUID-ABCDEFG">
	<question>
		<references>
			<reference href="#GUID-ABCDEFG/GUID-12345"/>
			<reference href="#GUID-ABCDEFG/GUID-67890"/>
		</references>
		<questions>
			<question id="GUID-12345">...</question>
			<question id="GUID-67890">...</question>
		</questions>
	</question>
</parent>

My output when rendering outputs the variables correctly:

Question::Count:[[question/@id=[GUID-12345] has [0] references with reference/@href=[#GUID-ABCDEFG/GUID-12345]

As you can see, zero, when it should find 1 match.
If I count "all" the references, then it DOES give me 2!
Now, if I hard-code the count as a string:
<value-of select="count(ancestor::parent//reference[@href eq '#GUID-ABCDEFG/GUID-12345'])" />
Then I do get the correct count/result!!
On another strange note: If I put a syntax error in my rule, the exception that is thrown shows me the "content" of the variable, not the value:
count(ancestor::question//data-about[@href eq concat('#', ancestor::question/@id, '/', .))])': syntax error, expecting ','

Hi Bradley,
after fixing the bugs in your schematron code it generated the expected output for me (Tested with oXygen xml 18.1):
<schema xmlns="http://purl.oclc.org/dsdl/schematron" queryBinding="xslt2" xmlns:sqf="http://www.schematron-quickfix.com/validator/process">
  <pattern id="Question">
    <rule id="Count" context="question/@id">
      <let name="href" value="concat('#', ancestor::parent/@id, '/', .)" />
      <assert test="false()">
        question/@id=[<value-of select="." />] has [<value-of select="count(ancestor::parent//reference[@href eq $href])" />] references with reference/@href=[<value-of select="$href" />])
      </assert>
    </rule>
  </pattern>
</schema>

Output:
question/@id=[GUID-12345] has [1] references with reference/@href=[#GUID-ABCDEFG/GUID-12345])
question/@id=[GUID-67890] has [1] references with reference/@href=[#GUID-ABCDEFG/GUID-67890])

About your exception: when the code is buggy there exists no value because the resulting script is not even being executed. Not sure what you expected here?
Regards,
Patrik

Hi Patrik,
Thank-you for your reply and confirmation that it does work for you! It must be the version that I am using that does not work. I am using a .Net version (so also perhaps not the right area to have posted my question as well - so my apology for that!)
[assembly: AssemblyTitle("SchemaTron")]
[assembly: AssemblyDescription("The native ISO Schematron validator over XPath 1.0 query language binding.")]
[assembly: AssemblyConfiguration("")]
[assembly: AssemblyCompany("The XRouter Team")]
[assembly: AssemblyProduct("SchemaTron")]
[assembly: AssemblyCopyright("Copyright © The XRouter Team 2011")]
Can you point me in a direction for a .Net API version of Schematron that is functional, so that I may replace the one that I have perhaps?
Again, thanks for your help and confirmation!!!
All the best,
Brad


feature: add a option in lazyvim to set all plugins border

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
when I try to change float window border to rounded. I need to go through and modify the option for each individual plugin. If border is set within keymap,  such as diagnostic. I have to copy diagnostic_goto function and remap it only for add a border option.
local diagnostic_goto = function(next, severity)
  local go = next and vim.diagnostic.goto_next or vim.diagnostic.goto_prev
  severity = severity and vim.diagnostic.severity[severity] or nil
  return function()
    go({ severity = severity, float = { border = "rounded" } })
  end
end

map.set("n", "]d", diagnostic_goto(true), { desc = "Next Diagnostic" })
map.set("n", "[d", diagnostic_goto(false), { desc = "Prev Diagnostic" })
map.set("n", "]e", diagnostic_goto(true, "ERROR"), { desc = "Next Error" })
map.set("n", "[e", diagnostic_goto(false, "ERROR"), { desc = "Prev Error" })
map.set("n", "]w", diagnostic_goto(true, "WARN"), { desc = "Next Warning" })
map.set("n", "[w", diagnostic_goto(false, "WARN"), { desc = "Prev Warning" })

Describe the solution you'd like
Can we add a global option 'border' to LazyVim to control the 'border' parameter for all plugins. like colorscheme.
    {
      "LazyVim/LazyVim",
      import = "lazyvim.plugins",
      opts = {
        colorscheme = "catppuccin-frappe",
        border = "rounded",
      },
    }

Describe alternatives you've considered
I can config it myself, but I believe having a unified configuration option would be more user-friendly.
Additional context
No response

Do you want to have diagnostics with rounded borders? If so, you can try
return {
  {
    "neovim/nvim-lspconfig",
    opts = {
      diagnostics = {
        float = {
          border = "rounded",
        },
      },
    },
  },
}

opts = {
      diagnostics = {
        float = {
          border = "rounded",
        },
      },
    },

thanks you。It is worked and solved most of my issues。for the rest，I feel I can configure them myself。

Lazyvim also uses noice which has a preset for LSP borders. Test it out to see if that's also something that you'd like. You can configure it like this
return {
  "folke/noice.nvim",
  opts = {
    presets = {
      lsp_doc_border = true,
    },
  },
}

I don't know what other borders you'd like. For the floating terminal you could also see #1915.

Lazyvim also uses noice which has a preset for LSP borders. Test it out to see if that's also something that you'd like. You can configure it like this
return {
  "folke/noice.nvim",
  opts = {
    presets = {
      lsp_doc_border = true,
    },
  },
}

I don't know what other borders you'd like. For the floating terminal you could also see #1915.
Edit: and for the autocompletion menu you can see this post.

thank you。I already set border in which-key，cmp，noice，term and git plugins myself。It is not too hard. I believe that‘s all I need to set in lazyvim。

I believe there eventually will be an option like vim.g.lazyvim_ui_border