Integrate the Mailchimp Mailing list code

Is your feature request related to a problem? Please describe.
Need to make the Announcement mailing list sign up work.
Describe the solution you'd like

[x] Create Mailchimp account with ISC ID
[x] identify the code that can be used on the ISC website (there are various options on the Mailchimp site)
[x] Add it to the Stay Connected section on Home page
[x] -Add it to the Stay Connected section on community Page
[ ] Add it to the Stay Connected section on Contact Us page

Additional context
Add any other context or screenshots about the feature request here.

The Mailchimp account is created. Will work this afternoon on getting the embedded code required for the website.

This is the code from Mailchimp.

<input type="email" value="" name="EMAIL" class="email" id="mce-EMAIL" placeholder="email address" required>

<input type="text" name="b_0363fb06e6a59d48c6e260f1b_97224b5ff2" tabindex="-1" value="">

<input type="submit" value="Subscribe" name="subscribe" id="mc-embedded-subscribe" class="button">

Works here and here.
Created a new {{< contact-form >}} tag for the form placement.
Currently on the plain HTML, in the future to be improved to Ajax JS way to make the process of subscription fancier.

The process is seamless - thanks @voborgus
I note though, it doesn't do the usual "confirm your subscription from your email" process. Is that something we need do you think?

I don't know, seems this is the default behavior of the MailChimp.
And maybe we should insert something for GDPR compliance (or not :) )

Contact us page now live. Closing this issue. Will open a new (non MVP issue) to investigate email confirmation with Mailchimp.


Add Bootstrap Icons to alerts docs

Adds examples of using Bootstrap Icons in the alerts component with flexbox utilities. Takes it a small step further and suggests custom SVG sprites as well just in case folks use an extensive set of alert/icon combinations.
Closes #33536, fixes #33480.

@mdo we could add this in 5.0.0 if you prefer.

@mdo we could add this in 5.0.0 if you prefer.

I hope so

Thanks for proofreading, @XhmikosR, my Bad!


pss: add function to get salt length from RSA PSS keys

To create proper RsaPssParams it is necessary to determine the salt length specified for the signing key. Add a function to get it from signing keys.

Looks good, thanks!


Notification permission not requested

On Android 13, the app has to properly declare and request notification permissions. This app doesn't do that, and therefore notifications are disabled.

Should be fixed in release 8.4, sorry about that!
Google Play update should roll out soon, or you can get the apk here: https://github.com/tom-anders/Easy_xkcd/releases
Let me know if there are still problems


do not overwrite default actions

within init() the default actions object is overwritten
and becomes an array. This works onces but fails after
calling another init()

Thanks very much for this. Fixed in 10e4012.


Update CAMPD Header

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[x] Does this have a dev link been provided in the design?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a CAMPD user I want the header to display "Maps & Graphs" and "CAMPD APIs" So that I can navigate the site
Acceptance Criteria
Given that a user has accessed CAMPD
When the CAMPD main menu functionality is displayed
Then the system displays the following items: Home, Data, Maps & Graphs
Given that a user has selected Maps & Graphs
When the drop down is displayed
Then then system displays: Coming Soon
Given that a user has selected Data on the CAMPD menu
When the drop down is displayed
Then the system displays the following items: Data Home, Custom Data Download, Bulk Data Files, CAMPD APIs
Given that a user has selected Data on the CAMPD menu
When CAMPD APIs  is selected
Then the user is directed to the following url: ----Need URL
Given that a user has navigated to the Data Home Page
When the functionality is displayed
Then the page content is consistent with the Data Home Page content document
Given that a user has navigated to the Data Home Page
When Learn more about CAMPD APIs is selected
Then the user is directed to the following URL: --- NEED URL
Given that a user has accessed the CAMPD home page, Data Home page, or CAMPD About Page
When the meta tag content is displayed (meta description, keywords)
Then the meta content is consistent with the content document (see context)
Development Notes
The CAMPD Main menu items will be: Home, Data, Maps & Graphs
The child items for Data will be: Data Home, Custom Data Download, Bulk Data Files,  CAMPD APIs
Utility Nav items will remain the same. Resources, help/support, regulatory partners
update the cam api section on the data home page to reflect CAMPD APIs content
Update Meta Tag information to have CAMPD APIs instead of CAM API.
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Testing
Test Case:
Tester:
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Is the user story ready to be deployed to the test/staging environments?

Dependency:

Need confirmation from the EPA that CAM API will be changed to CAMPD apis.
Need link to CAM API portal

@JanellC We discussed and here are the final edits to the a/c for this ticket.

The header menu should be changed from "CAM API" to "APIs"
The "CAM API" box title on the Data Home page should be changed to "APIs"
The "Learn more about CAM API" button should change to "Learn more about CAMPD APIs"

Let me know if you have any questions.

@tlageman-epa I have a couple of follow up questions. See below.

Is the link available for us to direct a user out to the API portal?
Does the team need to update any meta content that mentions CAM API? If so, should any mention of CAM API in the meta content be replaced with “APIs” or “CAMPD APIs”?

@JanellC

@maheese should have the link to the CAM API portal
Yes, let's go ahead and update the meta content as you suggested.

@tlageman-epa I updated the meta content for the CAMPD home page, Data Home page, and CAMPD About Page in the content documents. Let me know if you have any questions. Also, I will be on the look out for the link to the CAM API portal. Thanks

TC540 passed @tlageman-epa . This ticket is ready for 508 testing @aprematta


Adding coverage support

Thinking about setting up code-climate for this repo

Someone who's an owner on this repository needs to goto here - https://lima.codeclimate.com/github/giraffi/fluent-plugin-amqp
and find out what the REPO_TOKEN is so the above change can had the right token applied.
Or give me owner rights for a little while and I'll set it up :-)

Ok, I'll check how to enable it.

I've added https://codeclimate.com/github/giraffi/fluent-plugin-amqp as open source project.
And, according to GIthub's help,  Open Source project unfortunately we don't yet support service hooks for these repositories.
Can I merge this branch to check ?

Not sure what the service hooks are needed for, and merging in at the moment may cause the build to fail. But why not.


Library API

Hello, I’m really interested in this project, and have already been trying it out in a few of my own. One thing I was wondering is if you would be interested in exposing a library API for the WITX syntax tree? Obviously witx-codegen Is understandably focused on generating WASI bindings, but I’m interested in extending to generate bindings for other formats, such as C headers with cbindgen. I’m happy to deal with the library API being very unstable and changing each version.

Hi!
Sure, any new and useful backend that would be welcome!
Can you clarify what you mean by library API? C headers and Typescript definitions?

Well by library API I mean that there is a crate with a library function like fn parse(witx: &str) -> Result<WitxAst> where WitxAst is some type representing a witx document.
I think ideally (though would take longer to build), would be something like syn but for witx, where you can easily parse and modify the syntax.
I essentially want to use WITX as an Interface Description Language for generating API bindings for my Rust library for C applications to be able as well as WASM from a single definition.

Isn't it what the witnext crate does?
The way types are represented is a little bit messy compared to the ASType type we have in witx-codegen, but if you are looking for something very low level, this may fit the bill.

I hadn't heard of that crate, thanks for sharing! That is what I'm looking for.

Hey, do you happen to know where the source code for witnext is? The repository points to the wasi repo but I can’t find it there.


Apply transform to other data is not visible in Registration widget

Should be visible whenever the registration data is not 4D and we are not doing MoCo:

Fixed in master


Planner / Assign + Reassign not working

Just hangs at the selection screen for Quick Venture / Hunt / Field Exploration until it errors out.
I'm on the german client btw.

Hello, this should be fixed in testing version of AutoRetainer, please check it and reopen if necessary


pact-node fails to launch mock server on windows

We are using pact-node (v6.16.0) bundled with pact-karma (2.1.5) and pact-web (5.9.0)
When trying to run the pact tests on a windows machine, I'm seeing the following error
[2018-05-07T22:23:19.539Z]  INFO<EMAIL_ADDRESS>on xxxxxx: Created 'standalone\win32-1.38.0\bin\pact-mock-service.bat service --port '3110' --consumer 'consumer' --prov
ider 'provider' --log 'C:\.....\pacts\stubserver-integration.log' --pact_dir '.....\pacts' --pact-file-writ
e-mode 'overwrite' --host 'localhost'' process with PID: 8688
[2018-05-07T22:23:24.034Z]  WARN<EMAIL_ADDRESS>on xxxxxx: Pact exited with code 1.
[2018-05-07T22:23:24.036Z]  INFO<EMAIL_ADDRESS>on xxxxxx: Removing Pact with PID: 8688
ERROR: The process "8688" not found.

When trying to run that bat file directly, I get the following error
<EMAIL_ADDRESS><EMAIL_ADDRESS>No such file or directory -- UI (LoadError)

Any pointers on why this is happening?

Hmm could you please share your Windows environment details?
Which shell are you running it on?

@bbarke was there any issues with your install process? There's a postinstall that runs and downloads the binary files needed for your OS.  Maybe just try to reinstall it using npm uninstall @pact-foundation/pact-node; npm install @pact-foundation/pact-node.

This is on Windows 10 with the regular windows terminal.
It looks like the issue was because the location/path of the project had a space in it, for example:
C:\MY UI\angular-project\ it was failing because it didn't handle the space and tried to run 'UI as an argument, resulting in 'no such file or directory -- UI'

@bbarke wait, pact-node didn't work because of the space or just the .bat command?  Because if it's the former, there's definitely an issue which shouldn't happen.  Can you please confirm?

Yes I can confirm this. I renamed a working pact project path to have a space in it, and pact failed to launch. I changed the name back to to not have a space, and it started working again.

@bbarke my question is, are you talking about using the command line to hit the .bat file, or using the pact-node CLI/API?  Because if it's the latter, that's a bug that needs to be fixed.

I'm using karma-pact, which uses the node CLI/API to start the pact stub server.

huh, okay.  This might be a bug then.  I'll have to investigate further,
but I'm guessing it has something to do with the shell script running that
wrapping it in quotes.
On Thu, May 10, 2018 at 9:44 AM Brent Barker<EMAIL_ADDRESS>wrote:

I'm using karma-pact, which uses the node CLI/API to start the pact stub
server.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pact-foundation/pact-node/issues/90#issuecomment-387907644,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjA5EtTjiYxoIcoLnD1R6eD0v7YVb03ks5tw39MgaJpZM4T1vcy
.

@mboudreau, if quoting is indeed the issue, then the investigation here might be useful: https://github.com/pact-foundation/pact-js/issues/160

Yep, spot on. I'll fix this today if I can.
On Thu., 10 May 2018, 11:49 am Timothy Jones<EMAIL_ADDRESS>wrote:

@mboudreau https://github.com/mboudreau, if quoting is indeed the
issue, then the investigation here might be useful:
pact-foundation/pact-js#160
https://github.com/pact-foundation/pact-js/issues/160
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pact-foundation/pact-node/issues/90#issuecomment-387926344,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjA5PPSF0SPMF8kAbN7RimpiB_GiKOPks5tw5zAgaJpZM4T1vcy
.

@bbarke very sorry I haven't gotten to this yet.  It's been hectic at work with GDPR :/

@bbarke good news, version 6.18.3 is going out as we speak with the windows path fixes.  Please let us know how you go.  Cheers.

Thanks Michel!

Thank you!

We are running pact-node 6.19.6 karma-pact 2.1.5 and pact-web 5.9.1
Is there a max path length requirement? I seem to be having issues with running pact tests with directories that are long.  After I change the length of the directory names, pact-node starts up successfully.
After running on windows machine with long path names:
Creating Pact Server with options:
    consumer = ux,
    provider = core,
    logLevel = debug,
    dir = C:\Users\aserie\source\repos\my-long-repo-name\long-workspace-name\apps\app1\pacts,
    log = C:\Users\aserie\source\repos\my-long-repo-name\long-workspace-name\apps\app1\logs\pact.log,
    port = 1234,
    host = localhost,
    pactFileWriteMode = overwrite,
    ssl = false,
    cors = false
[2018-07-16T14:34:57.615Z]  INFO<EMAIL_ADDRESS>on L-19412-1: Created 'standalone\win32-1.52.1\bin\pact-mock-service.bat service --consumer 'ux' --provider 'corerunner' --log-level 'DEBUG' --pact_dir 'C:\Users\aserie\source\repos\my-long-repo-name\long-workspace-name\apps\app1\pacts' --log 'C:\Users\aserie\source\repos\my-long-repo-name\long-workspace-name\apps\app1\logs\pact.log' --port '1234' --host 'localhost' --pact-file-write-mode 'overwrite'' process with PID: 6308
(node:10772) DeprecationWarning: Tapable.plugin is deprecated. Use new API on `.hooks` instead
[2018-07-16T14:35:04.536Z]  WARN<EMAIL_ADDRESS>on L-19412-1: Pact exited with code 1.
[2018-07-16T14:35:04.536Z]  INFO<EMAIL_ADDRESS>on L-19412-1: Removing Pact with PID: 6308
ERROR: The process "6308" not found.

Simply making the directory smaller will start pact-node without it exiting with code 1.
Shorter directory name: C:\Users\aserie\source\repos\my-repo\workspace\apps\app1...


Wireshark SSLKEYLOGFILE decryption failed

I'm new to quinn. Maybe it is a wireshark bug?

When enable_keylog and set SSLKEYLOGFILE=/tmp/keylog.file env, keylog.file generates ok.
When set wireshark preferences TLS -> (Pre)-Master-Secret log filename to keylog.file,
wireshark shows [Failed to create decryption context: Decryption (checktag) failed: Checksum error]

BTW, Is there any other way to debug quinn ?

What version of Wireshark are you using? Only the latest pre-release snapshots typically have support for the latest QUIC draft. You can see more information here: https://github.com/quicwg/base-drafts/wiki/Tools#wireshark

Wireshark Version 3.2.5 (Git commit ed20ddea8138)
Quinn use the latest master commit
Thanks a lot. I'll have a try.


Drag and Dropping

Hey there!
It would be awesome if you guys add the feature where you can drag and drop the files/items into like discord or somewhere!

@DashCruft The feature is already present in Files. It's just that dragging isn't very fast currently. You need to drag a bit slower to see it work.

and can you guys solve that in future updates?

@DashCruft Drag and drop should be available with v0.23.1.


build-docs gh action checkout teams

Proposal for how to include management SDK in build-docs github action.
How it works:

Checks out fiftyone as normal
Checks out main branch of fiftyone-teams under fiftyone/fiftyone-teams

needs deployment key secret to checkout private repo (see below)
harder to correspond branches dev->dev or releaseX->releaseY, so for now just use main which should be latest release.

Uses -t fiftyone-teams option to generate_docs.bash to link relevant folders in before building.

What would be needed to test and finalize it:

Create an ssh key
Upload private key as a secret to fiftyone, called TEAMS_SSH_PRIVATE_KEY
Upload public key as a deployment key to fiftyone-teams with read only access.
Delete generated ssh key from local machine

I think it is safe because:

deployment key only has read only access to the single repo
private ssh key is contained within github secret and nowhere else

Alright I fixed and tested it - now it will build properly with fiftyone-teams/main linked in
(it will probably keep failing until fiftyone-teams release branch is merged into main)


feat(error-handler): Updated handleError method

Added optional stackTrace argument

Codecov Report

Merging #66 (4692ced) into main (a9ed375) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #66   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          325       325
=========================================
  Hits           325       325

Flag
Coverage Δ

elementary
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

packages/elementary/lib/src/core.dart
100.00% <100.00%> (ø)

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


Support all node versions

Node versions supported:

latest stable
latest LTS
v8
v7
v6
v5
v4

Travis runs the tests against each version.
Jest v20+ does not pull in babel-polyfill by default so that had to be done via the setupFiles option in the Jest config to support running the tests under older versions of Node requiring some polyfills used in eslint-summary-formatter codebase e.g. String.padStart()

Coverage remained the same at 97.849% when pulling f8fe4bb0dbf46121940eeadcc592f5000eab2afa on support-all-node-versions into e0e65126d5251061b03a74835c30c60912f97258 on master.


Subtle undocumented metrics changes between 1.22 and 1.23

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
A subtle metrics change in the istio 1.23 release caught us out.
If we have a Service Entry, and Virtual Service for an external host (which we do a lot, as we use tls origination to get the http metrics) historically destination_service_namespace used to be unknown, which makes sense, as the workload isn't in a namespace.  We had a bunch of prometheus matchers based on the expectation that is unset for off-cluster destinations, so here is a metric example from 1.22.4:
istio_requests_total{reporter="source",source_workload="insurance-partner-service",source_workload_namespace="insurance-partner-service",source_app="insurance-partner-service",destination_workload="unknown",destination_workload_namespace="unknown",destination_service="www.moneysupermarket.com",destination_canonical_service="unknown",destination_service_name="www.moneysupermarket.com",destination_service_namespace="unknown",response_code="200",response_flags="-"} 133

However, on 1.23 - destination_service_namespace now seems to be set to the namespace that the ServiceEntry / VirtualService resides in, eg:
istio_requests_total{reporter="source",source_workload="insurance-partner-service",source_workload_namespace="insurance-partner-service",source_app="insurance-partner-service",destination_workload="unknown",destination_workload_namespace="unknown",destination_service="www.moneysupermarket.com",destination_canonical_service="unknown",destination_service_name="www.moneysupermarket.com",destination_service_namespace="insurance-partner-service",response_code="200",response_flags="-"} 1767424

It'd be good to know if this change is intentional or not.
The documentation here just says:
"Destination Service Namespace: This identifies the namespace of destination service."
So i think it's a bit ambiguous/subject to terminology overload, yes the service is defined in that namespace (eg via a service entry) but there is no "service".
Version
1.23.0

Additional Information
No response

maybe not intended, but setting to SE namespace looks reasonable from my perspective

Yeah I think i agree if we're saying the service in source|destination_service_ metrics isn't kubernetes service, it's any "service"; eg a serviceEntry too.
Definitely worth confirming if it was intentional or not, and i'm guessing if not it highlights some missing tests around metric labels for service entries.

This was intentional; it was part of a bug fix here: https://github.com/istio/proxy/pull/5617

Fair enough, @keithmattix do you want me to close this? is there anything to do here?
As a user it'd be good to see any metric label changes in release notes but unsure how programatically you can ensure that happens

Ah yeah we don't have auto release notes for the proxy repo; that's a gap. But yeah you can probably close this


Add params support for client, contact and project endpoints

Add updatedSince to client, contact and project and client support for
contact and project.

Thanks for your contributiion!


[BUG]: Cannot import drizzle-orm/libsql on intel mac

What version of drizzle-orm are you using?
0.28.6
What version of drizzle-kit are you using?
0.19.13
Describe the Bug
Also, installed:
"@libsql/client": "^0.3.4",
"better-sqlite3": "^8.6.0"

And just importing it in my server files with import { drizzle } from 'drizzle-orm/libsql' crashes the server with:
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

Error: Cannot find module '@libsql/darwin-x64'
Require stack:
- /Users/hugo/Projects/vet-project/node_modules/libsql/index.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1077:15)
    at Module._load (node:internal/modules/cjs/loader:922:27)
    at Module.require (node:internal/modules/cjs/loader:1143:19)
    at require (node:internal/modules/cjs/helpers:110:18)
    at Object.<anonymous> (/Users/hugo/Projects/vet-project/node_modules/libsql/index.js:26:24)
    at Module._compile (node:internal/modules/cjs/loader:1256:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
    at Module.load (node:internal/modules/cjs/loader:1119:32)
    at Module._load (node:internal/modules/cjs/loader:960:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:169:29) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/Users/hugo/Projects/vet-project/node_modules/libsql/index.js' ]
}

Node.js v18.16.1

Expected behavior
Able to import with import { drizzle } from 'drizzle-orm/libsql'
Environment & setup
Intel Mac, running catalina 10.15.7, node v18.16.1 and npm 9.5.1

update: removing all node_modules in the monorepo, and making a clean install fixed it


minor UI improvements

remove useless alert
@Leemoonsoo
creating an interpreter setting is absolutely user-intended action and there's no side effect,
so I guess there's no need of showing alert.

stretch max notebook list height
@minahlee did a great job, but I found maximum height of the list is quite small in practice.

Looks good to me!


fix: catch errors from zwave driver contructor #61

Fixes #61

Pull Request Test Coverage Report for Build 420472223

0 of 17 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 27.899%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ZwaveClient.js
0
17
0.0%

Totals

Change from base Build 414937517:
-0.02%

Covered Lines:
1865

Relevant Lines:
6802

💛  - Coveralls


Remove Data Qualifiers

Closes #
Description Of Changes
This PR implements the full removal of anything related to DataQualifiers.
Code Changes

[x] hunt down every instance of Data Qualifiers and remove it

Steps to Confirm

[ ] automated checks confirm
[ ] website visualization works as expected
[ ] generation scripts work

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
[ ] Documentation Updated
[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

looks good @ThomasLaPiana. Discussed in standup, seems like this work should not go out immediately, should wait until next release to make sure it lands safely.

for sure, this will take lots of testing and checking! Still working through issues in Fides since the Pydantic upgrade also has a lot of effects


chore: bump jsonrpsee

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.103%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
930

Relevant Lines:
1161

💛  - Coveralls


Movement System

This system would handle the player's movement, including dashing, double-jumping, and moving around the game environment. It would also handle collision detection and physics, ensuring that the player moves smoothly and fluidly through the game world.
First thing first since we will have different characters that move it would be nice to have a character class. This character class is just the sprite of the character, width and height, and collission Im pretty sure arcade has something built in for that.
class Character:     """ Character boilerplate """     def __init__(self, ...):          """ Character init"""
Next thing is implementing WASD movement W for jump (player can jump again for double jump) S to duck (if double tapped S go down platform) A for strafe left D for strafe right.
We need to also make sure to use the sprite sheet to animate everything. (if there is art)

Also we want character states. That would make life easier. For example when player jumps the state is airborn.

I vote we also add in arrow controls, so right, left, up, down too, and spacebar for jump. Some players prefer arrows and some prefer wasd, so I think we can just implement both

I think we can just implement both

might be just better to do a settings page for it that way people around the world with different keyboard layouts can set there own to whatever.

Finished since #15


Add is_pydantic_dataclass to public API

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
We are incrementally changing our validators from mutiple other libs to pydantic. Part of our strategy depends on detecting which validator we need to use based on the "schema" that was provided.
For pydantic dataclasses we would like to use pydantic._internal._dataclasses.is_pydantic_dataclass, but would rather not use a any internals that might be subject to breaking changes.
Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[X] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

Selected Assignee: @samuelcolvin

Calling pydantic.dataclasses.is_pydantic_dataclass on pydantic dataclasses returns False due to '__pydantic_validator__' not being in __cls.__dict__. Anyone else facing the same problem?

Seems to be working for me, how are you creating your Pydantic dataclass ?
import pydantic

@pydantic.dataclasses.dataclass
class PydanticDataclass:
    bar: str

print(pydantic.dataclasses.is_pydantic_dataclass(PydanticDataclass))
# > True

That's odd, your code is working for me. However, the following is not
from pydantic.dataclasses import dataclass, is_pydantic_dataclass

@dataclass
class DummyDataClass:
    value : int = 0

dd = DummyDataClass()
print(is_pydantic_dataclass(dd))

Doh... I've called it on an object and not the class itself... Is there a function to check if an instance of a dataclass is a pydantic dataclass object?

Sure, you can use the builtin type to get the class of any object. So, on your example you'd do:
dd = DummyDataClass()
print(is_pydantic_dataclass(type(dd)))

I see, thanks. But maybe it's worth noting that Python's native is_dataclass function returns True for both, classes and instances of a dataclass.

Seems stdlib checks if we are passing a class or an instance https://github.com/python/cpython/blob/4f04172c9287c507f1426e02ddfc432f1f3ade54/Lib/dataclasses.py#L1306
Might make sense to implement this on pydantic as well, I'd suggest you open a feature request


Add process.env parameters for Push configurations

Can you add a note that all of these are optional if you aren't using push? This is a lot of new parameters which could be very confusing for new users.


Migrate Deprecation Policy page from metapackage to qiskit_sphinx_theme

This file in the metapackage defines the content for https://qiskit.org/documentation/deprecation_policy.html.
It would be better if this page could be refactored into an .html file that could be kept here and deployed straight to IBM COS.
Converting to html and removing the sphinx build from the equation will make this easier to maintain and allow us to develop more complex frontend features in futures that are not well supported by .rst

closing this as direction has changed and it makes more sense to keep this page in it's current location in the metapackage


email not working with attributes query anymore

Providing the attributes filter in the 1.3.0 version doesn't return the emails anymore.
GET ?attributes=userName,emails will only return id and userName.

Hi @spech66, thanks for raising this!
I was able to reproduce the issue and it appears to have been caused by changes introduced in PR #56.
I've now released v1.3.1 which should resolve this issue and return to the expected behaviour of including the entire emails attribute value when explicitly specified in the request.
Thanks,
Sam


moretrees causes Luanti to crash

Hi,
I have installed moretrees yesterday and everything worked properly.
Today, I'm facing a luanti crash :
2024-12-28 11:16:25: ERROR[Main]: ServerError: AsyncErr: Lua: Runtime error from mod 'moretrees' in callback item_OnPlace(): C:\games\luanti\bin\..\mods\xcompat/src/functions.lua:25: attempt to call field 'sapling_on_place' (a nil value)
2024-12-28 11:16:25: ERROR[Main]: stack traceback:
2024-12-28 11:16:25: ERROR[Main]:       C:\games\luanti\bin\..\mods\xcompat/src/functions.lua:25: in function 'sapling_on_place'
2024-12-28 11:16:25: ERROR[Main]:       C:\games\luanti\bin\..\mods\moretrees/node_defs.lua:448: in function <C:\games\luanti\bin\..\mods\moretrees/node_defs.lua:447>

relevant code parts:
https://github.com/mt-mods/xcompat/blob/7a136dc7393dcb8d50d34f55aa357b15811cbc59/src/functions.lua#L23-L25
called from the moretrees mod here:
https://github.com/mt-mods/moretrees/blob/ac3e30ae862334ba59d3fe2e1843bb65e0e5a6b5/node_defs.lua#L448

not relevant since thats only triggered if default exists

i thought per chance they might be using https://codeberg.org/cora/default_lite, but that defines the function


podcast:email Is it really necessary?

RSS 2.0 already has the channel level elements managingEditor and webMaster.
webMaster is defines as:

Email address for person responsible for technical issues relating to channel.

It seems suitable or the current purpose described for <podcast:email>:

This is a channel-level element. An email address that can be used to verify ownership of this feed during move and import operations. This could be a public email or a virtual email address at the hosting provider that redirects to the owner's true email address.

I don't have a strong opinion on which should take priority, managingEditor and webMaster.
Source: https://www.rssboard.org/rss-specification

I agree with the redundancy here, but I feels it's necessary in this particular case.  Existing emails in feeds are going to be all over the map with regards to accuracy.  This feels like a perfect time (during namespace implementation) to say that this element is the one tied to transfer locking contact, so people know that it's has to be valid and can't be a junk address.
The one thing I'm open to here is possibly renaming it to something like podcast:contactEmail or something like that.

I think you could expand this to support additional email types, and you might want to consider the "type" approach. For example:
<podcast:email type="contact">
<podcast:email type="advertising">
<podcast:email type="verification">
Then, if type is missing, it will be assumed the default contact email for everything.

I like this a lot.  Having it fallback to everything in case of no attribute is a great approach.

There could also be a recommended fallback order for developers. Kind of like how summaries/descriptions "work" in many apps. (For example, Google Podcasts uses descriptions in this order: <googleplay:description> ➜ <itunes:summary> ➜ <description>.)

Yes that is good too.  I do similar things as well across the parser.
How about the following:  We keep a separate tag for verification and just rename it to:
<podcast:ownerVerification>[email address]</podcast:ownerVerification>
That tag is so critically important that I think having it stand alone and be unambiguous, with no attribute requirements is best.
And, that allows for changing your idea to something even more flexible, like this:
<podcast:contact type="feedback" method="link">[link to a forum or chat room]</podcast:contact>
<podcast:contact type="advertising" method="email">[email address]</podcast:contact>
<podcast:contact type="abuse" method="email">[email address for DMCA takedowns, etc.]</podcast:contact>

So, there can be different contact methods for each of the areas of concern.

I like the podcast:contact type=""
one question is this an open ended tag so you could add what ever. say type="livestream" method="link" or is it limited to predefined types?

I like the method approach since that can help better point people to the right place.

@Lehmancreations Something like that would be better to use alternateEnclosure tag.

Yes that is good too. I do similar things as well across the parser.
How about the following: We keep a separate tag for verification and just rename it to:
<podcast:ownerVerification>[email address]</podcast:ownerVerification>
That tag is so critically important that I think having it stand alone and be unambiguous, with no attribute requirements is best.
This looks great to me, gives it a clear unambiguous purpose.

And, that allows for changing your idea to something even more flexible, like this:
<podcast:contact type="feedback" method="link">[link to a forum or chat room]</podcast:contact>
<podcast:contact type="advertising" method="email">[email address]</podcast:contact>
<podcast:contact type="abuse" method="email">[email address for DMCA takedowns, etc.]</podcast:contact>

So, there can be different contact methods for each of the areas of concern.
No strong feelings here. I see a minor overlap with #14 (contact vs social media information), but it is hard to tell.


feat: implement feeds endpoint

Summary:
Implements:

https://github.com/MobilityData/mobility-feed-api/issues/32
https://github.com/MobilityData/mobility-feed-api/issues/35
https://github.com/MobilityData/mobility-feed-api/issues/51
fixes the broken tests by introducing environment variables during the tests and fix endpoints

Expected behavior:
Screenshots from Postman requests against local API:

a7">

There are also some query parameters to implement, namely limit and offset.
filter and sort have been delegated to https://github.com/MobilityData/mobility-feed-api/issues/63
I suggest either you mention clearly in this PR and in the issue that limit and offset are not included, and not close the issue after the PR is closed. or you create a new issue to cover these parameters.

There are also some query parameters to implement, namely limit and offset. filter and sort have been delegated to #63 I suggest either you mention clearly in this PR and in the issue that limit and offset are not included, and not close the issue after the PR is closed. or you create a new issue to cover these parameters.

Added limit and offset to this PR. filter and sort can be tracked separately as these apply to most endpoints

@cka-y we saw that you added changes to provider field in feat_15.sql. The OpenAPI spec still shows providers as a array of strings on BasicFeed. I've changed it to provider as a string


Updated - WP.AlternativeFunctions: Add more functions to filesystem functions list

Closes #1265

[x] Check the current state of WP_File_System and remove any functions from the list which don't have a WP_File_System counterpart.
If there is no replacement function, the sniff would throw a non-actionable error/warning, which effectively is just noise.
[x] For functions which do have a WP counter-functions (outside WP_File_System), set those up in separate groups with an informative warning/error message.
[x] Remove file_get_contents() from the list as it is already checked in another group in the same sniff (with logic to prevent triggering on opening of local files) and we don't want to throw duplicate notices.

@sandeshjangam Just checking - will you have a change to update the PR in the next few days ? I'd like to include this in WPCS 3.0.0, but if not, I'll just move it out of the milestone.

@jrfnl Trying to finish it today itself. I will update you here.

@jrfnl Changes done as per suggestions and added test cases also.

Thanks for the feedback. All changes are done now.


Add configurable keepalive

Signed-off-by: shikoku1<EMAIL_ADDRESS>

@shikoku1 please apply the second commit as a fixup to the first one, to keep the history clean


Add getting started section to README

README Update
Please add a getting started section to the README so folks know how to start with this project!

Added Getting Started section in README file.


networking-cilium fails to start on K8s >= v1.20 Seed clusters

How to categorize this issue?
/kind bug
What happened:
networking-cilium fails to start on K8s >= v1.20 Seed clusters
How to reproduce it (as minimally and precisely as possible):

Create a Seed cluster with K8s.

Apply the<EMAIL_ADDRESS>ControllerRegistration with spec.deployment.policy=Always

Make sure that the corresponding extension Pod fails to start with:

$ k -n extension-networking-cilium-k264k logs gardener-extension-networking-cilium-7cd8b5b457-w56t7
{"level":"info","ts":"2021-04-14T16:04:29.370Z","logger":"controller-runtime.metrics","msg":"metrics server is starting to listen","addr":":8080"}
{"level":"error","ts":"2021-04-14T16:04:29.374Z","msg":"Could not add controllers to manager","error":"could not create Gardener client: error discovering kubernetes version: unsupported kubernetes version \"v1.20.5\"","errorVerbose":"error discovering kubernetes version: unsupported kubernetes version \"v1.20.5\"\ncould not create Gardener client\ngithub.com/gardener/gardener-extension-networking-cilium/pkg/controller.(*actuator).InjectConfig\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/pkg/controller/actuator.go:74\nsigs.k8s.io/controller-runtime/pkg/runtime/inject.ConfigInto\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/runtime/inject/inject.go:68\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).SetFields\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/manager/internal.go:228\ngithub.com/gardener/gardener/extensions/pkg/controller/network.(*reconciler).InjectFunc\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/gardener/gardener/extensions/pkg/controller/network/reconciler.go:75\nsigs.k8s.io/controller-runtime/pkg/runtime/inject.InjectorInto\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/runtime/inject/inject.go:144\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).SetFields\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/manager/internal.go:243\ngithub.com/gardener/gardener/extensions/pkg/controller.(*operationAnnotationWrapper).InjectFunc\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/gardener/gardener/extensions/pkg/controller/reconciler.go:56\nsigs.k8s.io/controller-runtime/pkg/runtime/inject.InjectorInto\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/runtime/inject/inject.go:144\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).SetFields\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/manager/internal.go:243\nsigs.k8s.io/controller-runtime/pkg/controller.NewUnmanaged\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/controller/controller.go:107\nsigs.k8s.io/controller-runtime/pkg/controller.New\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/sigs.k8s.io/controller-runtime/pkg/controller/controller.go:74\ngithub.com/gardener/gardener/extensions/pkg/controller/network.add\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/gardener/gardener/extensions/pkg/controller/network/controller.go:82\ngithub.com/gardener/gardener/extensions/pkg/controller/network.Add\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/gardener/gardener/extensions/pkg/controller/network/controller.go:77\ngithub.com/gardener/gardener-extension-networking-cilium/pkg/controller.AddToManagerWithOptions\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/pkg/controller/add.go:49\ngithub.com/gardener/gardener-extension-networking-cilium/pkg/controller.AddToManager\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/pkg/controller/add.go:59\ngithub.com/gardener/gardener-extension-networking-cilium/cmd/gardener-extension-networking-cilium/app.NewControllerManagerCommand.func1\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/cmd/gardener-extension-networking-cilium/app/app.go:104\ngithub.com/spf13/cobra.(*Command).execute\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/spf13/cobra/command.go:844\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/spf13/cobra/command.go:945\ngithub.com/spf13/cobra.(*Command).Execute\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/spf13/cobra/command.go:885\nmain.main\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/cmd/gardener-extension-networking-cilium/main.go:30\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:204\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1374","stacktrace":"github.com/go-logr/zapr.(*zapLogger).Error\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/go-logr/zapr/zapr.go:128\ngithub.com/gardener/gardener/extensions/pkg/controller/cmd.LogErrAndExit\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/gardener/gardener/extensions/pkg/controller/cmd/cmd.go:32\ngithub.com/gardener/gardener-extension-networking-cilium/cmd/gardener-extension-networking-cilium/app.NewControllerManagerCommand.func1\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/cmd/gardener-extension-networking-cilium/app/app.go:105\ngithub.com/spf13/cobra.(*Command).execute\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/spf13/cobra/command.go:844\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/spf13/cobra/command.go:945\ngithub.com/spf13/cobra.(*Command).Execute\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/vendor/github.com/spf13/cobra/command.go:885\nmain.main\n\t/go/src/github.com/gardener/gardener-extension-networking-cilium/cmd/gardener-extension-networking-cilium/main.go:30\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:204"}

Anything else we need to know?:
Actually this issue should be fixed with https://github.com/gardener/gardener-extension-networking-cilium/pull/36.
See https://github.com/gardener/gardener-extension-networking-cilium/blob/08d8e287b0a5c4e6d24d65c8de9f66b1f41c8d8f/vendor/github.com/gardener/gardener/pkg/client/kubernetes/client.go#L234
So it should be enough to cut a new minor release to fix this issue (the last minor release is from 23 Nov 2020).

Gardener version (if relevant): v1.20.3
Extension version: v1.4.0

So it should be enough to cut a new minor release to fix this issue (the last minor release is from 23 Nov 2020).

@DockToFuture , are there any plans to cut a new minor release for networking-cilium?

The commit is currently tested on dev as soon as its validated it will be released.

New version was released.

Thank you @DockToFuture !

thanks! Awesome!


[ISSUE #3503] fix the consumeOffset will be set as 0 when getMessage returns null

Make sure set the target branch to develop
What is the purpose of the change
#3503
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.03%) to 53.446% when pulling 7191932a1d816aedf75e7fe99e2fbdc731b75b89 on cserwen:bugfix_offset_0 into 4506f34e24714ec4d6ac37babd8f096632fd6b1c on apache:5.0.0-alpha.

@odbozhou Can you help me to review this pr?

BTW, getMessageTmpResult == null contains two cases, such as defaultMessageStore is shutdown and defaultMessageStore is not readable. The detail causes of these two cases is lost. We can improve it in the future.

BTW, getMessageTmpResult == null contains two cases, such as defaultMessageStore is shutdown and defaultMessageStore is not readable. The detail causes of these two cases is lost. We can improve it in the future.

Yes! I agree with you.

@duhenglucky  Can you help me to review this pr?:blush:

@RongtongJin  Why this pr was closed?

@RongtongJin Why this pr was closed?

Sorry, because the 5.0-alpha distribution has a rollback, I pushed a new 5.0-beta branch (deleted the original branch), resulting in the automatic close of PR.

@cserwen I reopen the pr

@RongtongJin Why this pr was closed?

Sorry, because the 5.0-alpha distribution has a rollback, I pushed a new 5.0-beta branch (deleted the original branch), resulting in the automatic close of PR.

OK, I got it.:smile:


can't run the project

Please upload files in the directory of drive,otherwise I cant run the project.

Hello, @liuziyuan827!
What files do you mean? Are you talking about pretrained models?
You can take the pretrained models from here and put it together with the project.


Single file static XVIZ Log

Hi, First of all, this is a great project and I would really like to make use of it, thanks for making it open source.
I was wondering if it is possible to pack all the single frame files into one large file? Which then has some kind of header at the front so that the client can skip to the correct frame using range headers, like a video stream. Is this implemented already or are you planning on implementing this?

We do not currently have this but we had some thoughts around it.  Can you describe how you want to use these files?
The basic idea was to move our multi-file on disk format into a "glb" like format where chunk 0 is a JSON index, chunk 1 is the metadata, and the rest of the chunks are the frame/updates.  So a client can read the front of the file then any chunk they want based on the time range they want to see.  We could also update the index to include which streams are in which chunk, such you can further focus reading based on the chunks.

I see the this single file as serving a few purposes

Convenience
Streaming
Optimization

Each case leads to a different internal structure.  I looked over at glTF related issues on streaming and it is on the radar, but not really a target for the format.  The scope for general glTF is different and our usage models the video stream closely, so it would be handy for an HTTP server.
In general 1 & 2 align well for our case and is definitely worth doing.
https://github.com/KhronosGroup/glTF/issues/1238
https://gist.github.com/javagl/bfde5cfab4240843120ed6eb38f4af87

Thanks for the answers!
My usecase would be converting logs offline from the rosbag format to XVIZ and then storing them in some http accessible storage, like S3. This would then not require any special server. Of course this would also be possible with all the single frame files, but in my opinion it's easier to manage and cleaner if there is only one file per log.

My solution:

gzip each frame.glb, store its sizes before & after compression
concatenate the gzipped files orderly into a single large file, store it in S3 (setting Content-Encoding: gzip header)
put the two sizes of all frames into the meta file, or a separate index file
write a custom XVIZFileLoader using the index info in meta file to calculate the offsets (both before & after compression), and use http Range header to fetch part of the large file or you can download/stream the whole file from beginning to end. (use the sizes after compression to calculate the offsets for the http request range, use the sizes before compression to calculate the offsets for frame data in the decompressed result. gzip decompression is handled automatically by the browser)

Using gzip compression makes the result file significantly smaller. The key point is that the large file concatenated from individually compressed gzip files is still valid as long as you know the offsets to split it into what you want.

@MikeChou this would be a great example to have added or at least we could like to any project you have that implemented this


Enhanced covers again

Still dissatisfied with the covers, I redid them once again.
Much better now, still the exported JPEGs show "ringing artifacts" (i.e., noise) around the black letters, while the original PNGs (cover_big.v4.png and downscaled from it cover.v4.png) don't.
Exporting them to JPEGs with a software, which allows to control the JPEG compression ("image quality") may alleviate that at the cost of the file sizes.
P.S.: I always wondered, why PNGs are not allowed for the cover and example images; it would be nice to be able to arbitrarily use either a JPEG or a PNG for these images.

Exporting them to JPEGs with a software, which allows to control the JPEG compression ("image quality") may alleviate that at the cost of the file sizes.

Done per PR #106.


Attempt to fix race and nullptr ctd in Hooks::UpdateCombatControllerSettings

#2
trying to test build with this changes

The nullptr check in remove_these should be unnecessary, I have played extensively with the lock only implementation and didnt have any issues, in fact I dont even notice either of these pointers ever being null after adding the mutex
Are you certain this check is necessary?

I catch a CTD if use  lock, but without this checking for nullptr, but it is rarely, and not clear how reproduce (1 CTD after 1 week of playing).
We can to not add this check, but then I advise adding logging for this case - to simplify the search for the reason if receive feedbacks with crashes.

Skyrim SSE v1.6.640
CrashLoggerSSE v1-12-1-0 Dec 22 2023 02:20:56

Unhandled exception "EXCEPTION_ACCESS_VIOLATION" at 0x7FF9C652E3E3 Acheron.dll+006E3E3	movups xmm0, [rbx+0x28] |  E:\andb_skyrim_3\projects\Acheron\src\Acheron\Hooks\Hooks.cpp:225 ?UpdateCombatControllerSettings@Hooks@Acheron@@CAXPEAVCharacter@RE@@@Z)

SYSTEM SPECS:
	OS: Майкрософт Windows 11 Pro v10.0.22631
	CPU: GenuineIntel 13th Gen Intel(R) Core(TM) i9-13900KF
	GPU #1: Nvidia AD103 [GeForce RTX 4080]
	GPU #2: Microsoft Basic Render Driver
	PHYSICAL MEMORY: 25.91 GB/63.78 GB
	GPU MEMORY: 12.40/14.66 GB

PROBABLE CALL STACK:
	[0] 0x7FF9C652E3E3  Acheron.dll+006E3E3	movups xmm0, [rbx+0x28] |  E:\andb_skyrim_3\projects\Acheron\src\Acheron\Hooks\Hooks.cpp:225 ?UpdateCombatControllerSettings@Hooks@Acheron@@CAXPEAVCharacter@RE@@@Z
	[1] 0x7FF7230DF32C SkyrimSE.exe+060F32C	mov eax, [rsi+0x204] |  sub_7FF6C5DFF2FF_60F32C -> 37350+0x10C	mov eax, [rsi+0x204]
	[2] 0x7FF72373C628 SkyrimSE.exe+0C6C628	mov ecx, [rbx+0x0C] |  sub_7FF6C645C575_C6C628 -> 69378+0xD8	mov ecx, [rbx+0x0C]
	[3] 0x7FF72373CBF1 SkyrimSE.exe+0C6CBF1	mov r15d, eax |  sub_7FF6C645C575_C6CBF1 -> 69380+0x361	mov r15d, eax
	[4] 0x7FF72373AF7A SkyrimSE.exe+0C6AF7A	movzx eax, byte ptr [rbx+0xA74] |  BSFile::seek_140C6ADB0_C6AF7A -> 69344+0x8A	movzx eax, byte ptr [rbx+0xA74]
	[5] 0x7FF723715E4D SkyrimSE.exe+0C45E4D	mov rcx, [0x00007FF725AD6898] |  StartAddress_0_140C45E10_C45E4D -> 68445+0x3D	mov rcx, [0x00007FF725AD6898]
	[6] 0x7FFAE8A2257D KERNEL32.DLL+001257D
	[7] 0x7FFAEA28AA58    ntdll.dll+005AA58

[13:27:39] [warning] Hooks::UpdateCombatControllerSettings attackedActor = nullptr
[13:27:39] [warning] Hooks::UpdateCombatControllerSettings targetActor = nullptr

related code:
https://github.com/AndbGame/Acheron/blob/d70963a1fcfb78c36e530b8d433198662e60d2d6/src/Acheron/Hooks/Hooks.cpp#L225

By the way, I remembered. The above CTD occurred while saving and loading the game in the middle of the battle


Add option to define asset folder and file name

It would be quite useful if the plugin allows devs to define the asset folder destination and the related file name (both for the json and html).

Do you mean a folder within the "asset" folder?

As much as I remember the plugin generates a folder which is located in a fixed location. It would be great if you could define that location yourself including the actual file name of the output.


[Skype for business]  integration with SFB

Bot Info

Bot handle<EMAIL_ADDRESS>App ID: 6f1d5233-80e1-4318-8467-e5a364076516
SDK Platform: .NET
SDK Version: 4.7
Active Channels: Skype for business
Deployment Environment: Azure Bot Service

Issue Description
Hello, I really need some help with an issue. I have a working .NET C# bot working on Telegram integration but not with Skype for Business.
I have configured Skype for Business on Office 365 Premium trial and added the integration in Bot Framwork with two powershell command
I can see the new user in my office 365 and add it into my Skype client as end user
But the bot is still offline.... it looks like the discovery is not working? It was done 5 hours ago
Does anyone knows why my bot is not working with Skype for business? How to troubleshoot? How to force the discovery?
Code Example
Standard example from Azure App Service
Reproduction Steps

Expected Behavior
Bot to talk
Actual Results
Bot is offline on Skype client.... Not connected!!!

Have you tried talking to the bot?  I contacted the SFB team and they said presence is not supported.  You can see the bot so discovery is working.

Thanks the issue was the domain...

Hello guys, my problem is similar to this one, my bot is perfectly functional on MS Teams using azure enviroment..
i've conected the bot with the two powershell cmdlet and it's been created but its still offline (the presence unknown message is not shown!!!) its just OFFLINE, and when i try to talk to it just doesn't work....
MS app ID: a1743f7b-83db-4ce5-ae88-cda52c220be0
RunspaceId     : 68f56c68-b242-440a-af0c-6a63cab9e9d5
Id             : 02d65798-eb7e-4ead-ac0a-114e1d6b7c29


Replace encoder_weights: str | None = "imagenet" with pretrained: bool=True

The encoder_weights parameter in the model initializers is a bit ambiguous/strange, especially given its lax type hint.
At least to me, it looked like you could pass strings to represent other weights (e.g., “coco”, “instagram”, “ssl”, etc.) at first glance, but it actually turns out that any non-null argument results in the timm weights for the particular encoder being loaded. This is because timm really expects a boolean flag for this and --- by extension --- torchseg.encoders.get_encoder() only performs the appropriate checks required to comply:
https://github.com/isaaccorley/torchseg/blob/47a4dd67eb503671c788740178dae8113bf55938/torchseg/encoders/__init__.py?plain=1#L62-L83
I understand that this had some value in smp because it had multiple weights for some models, but timm has only one, so maybe it would make sense to change our API to reflect this.

I just saw that right above the block I referenced, the value of encoder_weights is actually used to look-up MixTransformer weights. I'm not sure where these are being pulled from though...

This was left in for backwards compatibility. I agree with this though. Particularly because with timm you load the weights through the encoder/model name like vit_b16_224.mae. We can likely add a new pretrained arg and then deprecate the weights arg.


[Question] Is there a way to use the script just to create ISO ?

Due to known Virtual Box bug  in Mac OSX Catalina 10.15.2, I intend to use VMWare.
Is it possible to use this script just to generate the ISO and use it with VMWare ?

Thanks for the feedback. Actually not using VMWare. Closing stalled issues.

would like this for esxi


socket.setdefaulttimeout crashes SSL connection.

socket.setdefaulttimeout(60) crashes SSL connection and throws ERR_SSL_PROTOCOL_ERROR when trying to reach any of https sites. When using default value or socket.setdefaulttimeout(None) all works fine.
Using libmproxy (mitmproxy 0.11.3).

Sorry, but reading your report I have no idea what your issue is. Where/Why/How do you set socket.setdefaulttimeout? Please use our bug report template.

My bad.
I'm running custom implementation of controller.Master class with specified ...-ca.pem certificate.
I'm trying to sniff my browser https traffic so I'have configured it to use this custom mitm proxy.
All was fine when I have used default values of socket timeouts, but when I changed default socket timeout value via socket.setdefaulttimeout method I start getting ERR_SSL_PROTOCOL_ERROR in my browser.

Sorry, but I cannot do anything here if you don't provide an example that I can reproduce. If you cannot share your custom implementation, you need to break it down to a minimum test case.

Please try -> this <- example. You should do some steps manually, such as set up proxy in your browser or navigate to https site through it.

The root cause seems to be https://github.com/pyca/pyopenssl/issues/168. Sorry, this seems to involve too many changes to justify the effort at the moment.

It seems to be true. Thanks, I will be watching this issue.


added function to reset the Catalog paths

Quick kludge to add a function to reset the CAT_PATH_* dirs.
You should be able to call set_cat_paths(a_new_path) after importing and away you go.
But I haven't actually tested that -- there may be some more reconfiguration that needs to be done,
But as it is -- nothing's different than it was if you don't call that function

Side note -- It would be better if the:
Fresh parameters: how long until model output avialability will be refreshed
for find_availabililty() if requested
FRESH
stuff was part of the catalog yaml file, rather than here.
-CHB


fix: issue 55 problems when filename with path is passed to --config

It turns out that the findUp method cannot handle a full path to a file as input. When this
is passed in, it redudantly or completely incorrectly prepends the current working directory
to the front of the returned path.  Work around is to split any path information off of the
input path and pass the path portion into the method as opt.cwd.
Closes #55

Also, it might be a good idea to add a test that covers this use case

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Logout link showing "adminlte::adminlte.log_out" instead of "Log out"

I researched here and did't find anything, so I am posting... I was doing tests on localhost and everything was fine. The issue started when I moved the application to the host. Do you have any ideia about what could be happening? Thanks!

I already do that without any problems .

Can you post your code @niltonssjr

Thanks a lot, @Lucky-Loek ! I didnt' find #207 before, it worked perfectly.


Add gRPC support to tutorials

Hey @mortendahl and @jvmncs,
This PR should be ready to review. It adds a section to each tutorial show they can be run with gRPC.
Thanks,

does it make sense to move the tutorials directory to the root of the repo?

Good point, let me do that. Thanks


Strip leading country code without leading plus

Given a country like Honduras with a country code of 504, if a number is formatted like (504) 2221-6592, it will not parse as it's an international string, however it does not start with a plus or the international prefix.
Now the territory will check if it starts with the country code before parsing and strip the country code when it is prefixed.

Thanks for the PR :smile:

Thanks for the merge!


My results are lower than yours.

When I train the model with consistent_teacher_r50_fpn_coco_180k_10p_2x8.py on one GPU, the result is too low. And I didn't change the parameters.
`[>>>>>>>>>>>>>>>>>>>>>>>>>>] 5000/5000, 43.4 task/s, elapsed: 115s, ETA:     0s2023-05-20 11:33:56,083 - mmdet.ssod - INFO - Evaluating bbox...
Loading and preparing results...
DONE (t=1.21s)
creating index...
index created!
Running per image evaluation...
Evaluate annotation type bbox
DONE (t=17.84s).
Accumulating evaluation results...
DONE (t=5.17s).
2023-05-20 11:34:22,052 - mmdet.ssod - INFO -
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.123
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.197
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.126
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.067
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.142
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.156
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.342
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.342
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.342
Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.146
Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.359
Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.496
[>>>>>>>>>>>>>>>>>>>>>>>>>>] 5000/5000, 43.1 task/s, elapsed: 116s, ETA:     0s2023-05-20 11:36:22,808 - mmdet.ssod - INFO - Evaluating bbox...
Loading and preparing results...
DONE (t=1.24s)
creating index...
index created!
Running per image evaluation...
Evaluate annotation type bbox
DONE (t=15.87s).
Accumulating evaluation results...
DONE (t=6.67s).
2023-05-20 11:36:48,355 - mmdet.ssod - INFO -
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.098
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.165
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.099
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.051
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.113
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.125
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.305
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.305
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.305
Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.131
Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.315
Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.446
2023-05-20 11:36:48,895 - mmdet.ssod - INFO - Exp name: consistent_teacher_r50_fpn_coco_180k_10p_2x8.py
2023-05-20 11:36:48,898 - mmdet.ssod - INFO - Iter(val) [180000]	teacher.bbox_mAP: 0.1230, teacher.bbox_mAP_50: 0.1971, teacher.bbox_mAP_75: 0.1262, teacher.bbox_mAP_s: 0.0674, teacher.bbox_mAP_m: 0.1415, teacher.bbox_mAP_l: 0.1562, teacher.bbox_mAP_copypaste: 0.1230 0.1971 0.1262 0.0674 0.1415 0.1562, student.bbox_mAP: 0.0984, student.bbox_mAP_50: 0.1653, student.bbox_mAP_75: 0.0991, student.bbox_mAP_s: 0.0513, student.bbox_mAP_m: 0.1129, student.bbox_mAP_l: 0.1249, student.bbox_mAP_copypaste: 0.0984 0.1653 0.0991 0.0513 0.1129 0.1249
wandb: Waiting for W&B process to finish... (success).`

Sorry, this config is supposed to be run on 8 GPUs with 2x8 meaning samples_per_gpu=2 and the total number of GPUs = 8.

Thanks, is there any config supposed to be run on one GPU? Or, what parameters should I changed in the config?
I have changed the parameter —gups and --gpu-ids in the tools/train.py to set the one GPU.
Thanks!

Dear @yuan738,
Thank you for your insightful question. Currently, the semi-supervised method is heavily dependent on a large batch size, and as a result, reducing the number of GPUs could significantly impact performance. Unfortunately, we have not yet found an effective solution to this issue.
One potential workaround could be to implement "gradient accumulation," or you might consider using fp16 to increase the batch size for a single GPU. Moreover, given the limitations of single GPU training, you might find that a 1:4 label-to-unlabel ratio is too ambitious and could consider adjusting it to a 1:1 ratio. However, please note that the performance might still be subpar with a single GPU setup. We acknowledge this challenge and will strive to address it in our future work.
Best Regards,

hi @Adamdad
i train 'consistent_teacher_r50_fpn_coco_180k_10p.py' config using 4 GPUs, at iteration 16000, the result as below
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.113
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.190
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.113
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.063
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.133
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.143
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.323
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.323
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.323
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.145
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.346
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.439

it seems the mAP is too low.

Dear @zimenglan-sysu-512 ,
I recommend increasing the ratio of labeled to unlabeled samples, such as a 1:1 ratio. Currently, the learning rate is based on 8 labeled samples (1 labeled sample per GPU). If you are training with fewer GPUs, you may need to adjust the batch size for labeled samples or decrease the learning rate to match your setup.
It's important to keep in mind that performance may still be lower with a reduced GPU setup. We are aware of this challenge and will make efforts to overcome it in our future endeavors.
Best.

hi @Adamdad

Dear @zimenglan-sysu-512,
Yes, setting data.sampler.train.sample_ratio to [2, 4] or [3, 3] should work well. The first number represents the number of labeled samples, while the second number represents the number of unlabeled samples. Therefore, you will be using 2 or 3 labeled samples per batch per GPU. Make sure
data.samples_per_gpu = sum(data.sampler.train.sample_ratio).
Best,

hi @Adamdad

We only has a file called configs/consistent-teacher/consistent_teacher_r50_fpn_coco_720k_fulldata.py. No config provided for 360k training for full data.

We only has a file called configs/consistent-teacher/consistent_teacher_r50_fpn_coco_720k_fulldata.py. No config provided for 360k training for full data.

thanks, another question, where to find configs/consistent-teacher/base.py file?

Dear @zimenglan-sysu-512 ,
Sorry for the confusion. the configs/consistent-teacher/base.py has been renamed to configs/consistent-teacher/consistent_teacher_r50_fpn_coco_180k_10p.py. I just changed it in the README as well.
Best

hi @Adamdad
two questions here:

from the log file, learning rate is keeping the same through the training phrase, why?
what is the difference between the 36w and 72 iterations? how about the performance of mAP? since only have few gpus e.g. 4, training 72w iterations takes more than 10 days to finishing even using fp16.

Dear @zimenglan-sysu-512,
Yes, setting data.sampler.train.sample_ratio to [2, 4] or [3, 3] should work well. The first number represents the number of labeled samples, while the second number represents the number of unlabeled samples. Therefore, you will be using 2 or 3 labeled samples per batch per GPU. Make sure
data.samples_per_gpu = sum(data.sampler.train.sample_ratio).
Best,

using 4 GPUs, and batch size is 6 in which sample_ratio = [2, 4], the result is below:
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.381
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.542
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.409
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.211
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.411
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.484
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.570
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.570
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.570
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.343
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.609
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.740

Dear @zimenglan-sysu-512
The results you provided are amazing. It would be very helpful if you could share the config and checkpoint. This experiment could be extremely useful for people with less GPUs. You can start a pull request or send the file to me.
Great 😃

Dear @zimenglan-sysu-512 The results you provided are amazing. It would be very helpful if you could share the config and checkpoint. This experiment could be extremely useful for people with less GPUs. You can start a pull request or send the file to me.
Great 😃
hi @Adamdad , i will send the config, log and checkpoint files to u. please check the qq email.

hi @Adamdad
why in the fulldata config, the backbone r50 freeze the BN layer, but the 10% config does not ?

hi @Adamdad why in the fulldata config, the backbone r50 freeze the BN layer, but the 10% config does not ?

dear @zimenglan-sysu-512
could you share your config file with me? It will be of great help to me, I only have 2 GPUs and get a bad training results.

dear @zimenglan-sysu-512
could you share your config file with me? It will be of great help to me, I only have 2 GPUs and get a bad training results.
my qq mail is<EMAIL_ADDRESS>


Improvements

Several unrelated improvements.
Ping @beckettsean.

:+1: CI green and all changes are either related to better conventions (rspec) or increased spec coverage (rails 4.1/4.2 and newer rubies).
:shipit:

@toddboom another influxdb-rails PR

@mvz definitely good changes - thank you!


Enable (move?) the data preparation code outside the view

Goal: place data preparation work (like manipulating data, search-integration etc.) into a separate file so the module stays simple.
Notes

we should probably wait to see what dnn 8 does in this regards, so our solution is not too different?
if possible, we should also allow this external thing (call it the controller) to deliver alternate views back if we wants to...

With our move to Project X, this may become unimportant.

Believe this becomes unimportant. Search not solved yet, but I assume most people don't care as they don't use DNN search. Close.


Scam :(

Hey dude like 99.9% sure this dude is selling your bot. https://hqtriviawebsocketb.wixsite.com/website
@Exaphis  Sorry to bring bad news.

Thanks for letting me know.


Secure Content Spec Implementation

Hi @martinweismann
This is a preliminary work from the secure content implementation, which still misses the encryption file format and some minor fixes that I'll continue to work on. Hopefully it's building properly and unlocks your work of preparing the new release. Let me know if you have any questions.
I've included this on top of your RefactorPackageID branch, but haven't tested the merge with the prior version from the secure-content repo, which was an old commit.

@martinweismann I've just added support for the encrypted file header.

@martinweismann These are the points I've collected. Let me know if I've missed anything.

[ ] Add a proper return value field to the callbacks
[ ] Update CKeyStoreResourceData to replace string path by a PPackageModelPath
[ ] Implement default random number generator callback using std:: implementation
[ ] Refactor CModelReaderWarning into CModelWarning
[ ] Add warning mechanism to Writer and advice when using the local RNG call
[ ] Writer to implement access methods to set warning level (strict mode)
[ ] Add example to SDK with proper encrypted file, code to throw implementation on encryption parts and point to UnitTest_EncryptionUtils (to avoid binding to libressl)
[ ] Make sure CResource::SetPackagePart and CPackagePart::SetPath are working as expected with a new test
[ ] Memory leak investigation

@martinweismann I've notice calling SetPackagePart on a resource before it gets added to a build item does not work, not sure whether that is expected.
I have added an example of how to construct secure content based on the cube example. Let me know if you think we should include a reading example as well.

@martinweismann I've notice calling SetPackagePart on a resource before it gets added to a build item does not work, not sure whether that is expected.
No, this should work. I am adding a top level checkbox su woe do not forget.

I have added an example of how to construct secure content based on the cube example. Let me know if you think we should include a reading example as well.
Yes, we should also have a reading example.


This should probably be a metapackage

There is already a 7za-feedstock, that I believe is the same as this.  We shouldn't have both as real packages.

I wish that the linux port at https://anaconda.org/bioconda/p7zip was also included in such a metapackage.

7za is windows only


Add baseline TDA techniques

I decided to add a new repository that implements all baseline influence function techniques.


Reading in raw template library

I am generating template_hash with python's hash function which should return an integer but for some reason doesn't always. This causes a problem in preprocess_rollout.py in read_csv in _filter_dataset. The same problem could occur with mixed types in reaction_hash and other columns. I fixed this by using low_memory=False.

Thanks for the feedback.
First, the python hash function is always returning an integer, it is not possible to define a __hash__ function for a class that isn't returning an integer (you will get a TypeError exception, if you try).
Secondly, I don't think that using the hash() function for creating the template (or reaction) hash. Have a look at the hashlib module.
The template library for the public data was created with this module https://github.com/reymond-group/CASP-and-dataset-performance/blob/master/template_utils/amol_utils_rdchiral.py
the reaction hash is created by the method generate_reaction_identifier(), and the template hash is created by the method hash_template().


GII CRUD: flash messages after actions

I think it would be nice having flash messages in crud after common actions were performed.
For example, after deleting something, I would expect to read: "The item was successfully deleted", and after creating or updating: "The item was created/updated successfully".
Or something like that.

Ok the idea seems very good , I manually do it in my projects.
As I have familiarity with gii and made a simple extension that helps you in it to the core do it for you .
https://github.com/lukBarros/legba-gii
I will try to work it in the appropriate design for such it is:
https://github.com/yiisoft/yii2-gii
Keep informed if progress in this encoding.

duplicated https://github.com/yiisoft/yii2-gii/issues/26

@pana1990 thanks!


Setting table_type = 'EXTERNAL_TABLE' builds a 'MANAGED_TABLE'

Sorry to open one more...
When I create an external table it ends up being created as a managed table.
            my_table = TableBuilder(
                            table_name='my_table',
                            db_name=table['DatabaseName'],
                            storage_descriptor=storage_descriptor,
                            partition_keys=partition_keys,
                            parameters=parameters,
                            table_type='EXTERNAL_TABLE',
                            owner='root'
                            ).build()

However when I look in the metastore postgresql database for that table:
# select "TBL_TYPE" from "TBLS"  where "TBL_NAME" = 'my_table';
   TBL_TYPE
---------------
 MANAGED_TABLE
(1 row)

fwiw, create table WITH (external_location = xxx) works fine from the presto client and creates the EXTERNAL_TABLE type in the database.
I'm still looking for a root cause or work-around, but thought I'd log what I've run into while I'm looking.   Your examples and tests don't include creating an external table.

Unfortunately, although I can query a created in presto manually, I get 0 rows back from an identical table created (with a different name) using this client.  So even though the "SHOW CREATE TABLE" command outputs are now identical, and the TBL_TYPE is now correct, there is something else not quite right - still investigating.

Unfortunately, although I can query a created in presto manually, I get 0 rows back from an identical table created (with a different name) using this client.  So even though the "SHOW CREATE TABLE" command outputs are now identical, and the TBL_TYPE is now correct, there is something else not quite right - still investigating.

To follow up on the investigation into the 0 rows issue after the table was re-created.  That one example I was testing appears to have had an underlying storage issue.  Once that was resolved, the queries worked.  I've gone back and recreated 75 of my tables now with the extra Table Parameter set, and that switched them all to EXTERNAL_TABLE when I did so and they started working.

To follow up on the investigation into the 0 rows issue after the table was re-created.  That one example I was testing appears to have had an underlying storage issue.  Once that was resolved, the queries worked.  I've gone back and recreated 75 of my tables now with the extra Table Parameter set, and that switched them all to EXTERNAL_TABLE when I did so and they started working.

Hi @rotten thank you for opening this issue!
Actually, we had not realized that this problem was occurring in Hive Metastore Server. We have also faced some strange behaviors in the tables due to the EXTERNAL/MANAGED option thought.
As far as I know, after digging in the code and testing the possibilities this is a bug from Hive Metastore Server.
We will create a new method that encapsulates the behavior of adding the option {'EXTERNAL': 'TRUE'} so we can work around this issue for now.

We addressed the issue in the linked PRs and released version 1.0.4 with all due improvements :smile:
Thank you again @rotten for bringing it up!


Numpy optimizations

thanks, what is the advantage of these changes? this is not a primary function

Using zeros_like instead of zeros is I think a better implementation as it takes care of making type consistent arrays and may reduce memory usage in certain cases. As I said, it is a minor optimization, but can be useful nevertheless.

thanks, I didn't know that. since this function is used for replicating the paper and doesn't have tests for it, I'd rather be safe and not change it, but thanks for the info


elaborate on using DPROD and PROV together

https://ekgf.github.io/dprod/#detailed-level-between-datasets and https://github.com/EKGF/dprod/tree/main/examples/data-lineage say that you can also use PROV for lineage.
But there's no integrated example showing how DPROD and PROV can be used together.

See also #76


Drugs and GC scrappers update along minor improvements

Updated DrugsScrapper and GCScrapper to avoid issues.
Optimization of Scrapper to use the cached links already fetched by previous runs to skip the link gathering.
Optimization of the chrome driver and handling of more options, of which some are necessary on windows (binary and driver locations) and others optional (headless, page loading mode, and pdf download folder).
One missing requirement: scipdf.

Thanks for contributing to the open-source community!


Run tests requiring Distributed internals separately

Some tests require Distributed internals and installed in developer mode. Since we have no good way to install it with rapids-dask-dependency, we now run those separately from the rest.

Thanks for reviews @charlesbluca @raydouglass @vyasr !

/merge


share link with image in whatsapp

DONE
https://github.com/Yuri-Lima/SharePay/commit/4be7f0d411395b68ae8c2eb5e031d26bb8e50132


Implement Signature Validation

The library should have a straightforward way to validate signatures of incoming requests from Nexmo's SMS API.

fixed in #261


Docs: Bootstrap

Improvement Idea:
Remove tsdx from the "Start a New Project" section
Why:
Consuming tsdx as is currently leads to a few security alerts being opened (for e.g., set-value, etc) for dependencies of that project. The project itself seems to be falling behind on updating insecure dependencies which forces downstream users to rely on workarounds such as yarn resolutions.

Fixed by #2044. Thanks @orta!


ignore/omit XMLTV sources with channels containing invalid characters

Problem:
I notice i am unable to even import an xmltv source which has a channel  with a semicolon
XML syntax error on line 275: invalid character entity &h (no semicolon)

Solution:
For threadfin to automatically ignore/omit these lines in xmltv files. (possibly with notification)
I know that i can remove it manually, which i will do. But it seems like it would be easy to implement.

@princinv can you add an example of the line causing the issue?   ...based on that issue, I think it might be saying that you're missing a semicolon.

Please use a valid XML file. I will not be fixing it to allow for edge cases


missing check for degenerate shape - zero length path

Need to add a check to catch zero length paths.  I believe we already cover zero area shapes today.

test case is here...
https://foss-eda-tools.googlesource.com/third_party/shuttle/mpw-four/slot-030/+/refs/heads/main/oas/caravel_000410da.oas

Handled in EF-3498 / mpw-7a release


chez-scheme: 9.5.6 -> 9.5.8

Description of changes
https://github.com/cisco/ChezScheme/releases/tag/v9.5.8
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build chez


Zoom option for TabNew

Two general approaches:

Use Zoom Client URL Schemes to trigger desktop/mobile client to join the room you just jumped into. Unfortunately, the behavior depends on how you've set the "Join different meetings simultaneously on desktop" user option. If on, you'll accumulate meetings on desktop; if off, you'll switch meetings (probably better because it's more managed by Comingle).
Use Zoom webapp. This works but is limited to the features of the web client (notably, no drawing on screen share, and no background removal). No iframe required, so we can directly control the interface, e.g., clicking the "join audio call" button (if user agrees to it at Comingle level) and muting/unmuting when you're in multiple meetings. Quality is notably lower than desktop client, but I guess this is a limitation of Chrome...

Unfortunately, upon further consideration, it's not really practical to use a separate Zoom meeting for each Comingle room,  because of the limit of one concurrent meeting per host. This would make it pretty impractical to general the number of simultaneous meetings, while avoiding conflicts with meetings users have otherwise scheduled.
A possible fallback approach would be to use breakout rooms in a single meeting, and have a bot that re-assigns people to breakout rooms according to their behavior in Comingle. But this doesn't seem to be supported by the API. So we'd need to actually drive a (headless?) host that clicks the relevant buttons... Doable, but difficult.

New approach: like current YouTube option, just detect URL and give it a type.  No room generation (which seems impossible), but allow people to enter room and password. Then, when rendering, give user a choice among above.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97317

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97598

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97613

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97651

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/110199


Move RPC streaming notes from metric brief section to notes section

Noticed while investigating https://github.com/open-telemetry/opentelemetry-demo/issues/969.
I'm making an assumption that the brief should be a more concise, basic definition.
Changes
Moved RPC streaming notes from metric brief section to notes section.
Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[x] CHANGELOG.md updated for non-trivial changes.
[x] ~schema-next.yaml updated with changes to existing conventions.~

@open-telemetry/specs-semconv-approvers, @open-telemetry/specs-semconv-maintainers can we get that merged?


Task: Conduct UX Research

Overview
As Web3Studio, we need to find "sponsor users" and conduct the research, so that we can be confident in our choices for what to build for the devKit and SXSW.
Assumptions

The people we can find in a week are going to give us the input we need to get things right and be confident in our conclusions.

Acceptance

List of users we agree are good.
Interviews and other activities (based on the dependency task below) conducted and documented.

Tasks

[ ] Conduct interviews
[ ] Document and share results

Will conduct interviews in next sprint. (Feb 11)


Update bin\dbatools-buildref-index.json : SQL 2019 CU2

Added newly released SQL 2019 CU2  to bin\dbatools-buildref-index.json

[ ] Bug fix (non-breaking change, fixes # )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Approach

Commands to test

Screenshots

Learning

awesome, thank you!


Unnecessary Document information is announced upon focusing on Bot response message.

Is it an issue related to Adaptive Cards?
No.
What is the PWD impact?
Visually challenged users who rely on screen readers will be impacted if Unnecessary Document information is announced upon focusing on Bot response message.
What browsers and screen readers do this issue affect?
Others
Are there any code-based customization done to Web Chat?
No, I am using Web Chat without any customizations except "styleOptions".
What version of Web Chat are you using?
Latest production
Which area does this issue affect?
Others or unrelated
What is the public URL for the website?
https://airtuserexpdev.azurewebsites.net/Dashboard
How to reproduce the issue?
Step 1: Open the above URL in Edge Browser.
Step 2: AIRT Dashboard page will appear, Navigate to “Chat with Sage” chat box at the bottom of the page.
Step 3: Enable NVDA and Navigate to Bot response message.
Step 4:Verify whether unnecessary document information is announced upon focusing on Bot response message or not.
What do you expect?
Unnecessary Document information should not be announced upon focusing on Bot response message.
NVDA expected response: "Chat History, bot said "Message."
What actually happened?
Unnecessary Document information is announced upon focusing on Bot response message.
Observation: NVDA announcing as "Chat History, document 1 of 1, bot said "Message".
Do you have any screenshots or recordings to repro the issue?

Did you find any DOM elements that might have caused the issue?
No response
MAS reference
https://microsoft.sharepoint.com/:w:/s/accessibility/EXdDUZzldv5MgwDaMHnTzcMBcT8opKGncYjU-gCJ4uqQdg?e=9nuVmt
WCAG reference
No response
WAI-ARIA reference
No response
Adaptive Card JSON
No response
Additional context
No response

#A11yMAS;#A11ySev3;#MAS1.3.1;#HCL;#Accessibility;#FeatureBotframeworkwebchat-Apr23;

When there are more than 1 message in the chat history, the "1 of 1" will become "1 of X".
As the chat history could become very long after the conversation started, we think this is necessary information for the PWD the know where their cursor is on.


Release 0.5.0

@zhangthen please use create a merge commit

Codecov Report

Merging #849 into master will increase coverage by 5.3%.
The diff coverage is 40.78%.

@@             Coverage Diff             @@
##             master     #849     +/-   ##
===========================================
+ Coverage     32.77%   38.07%   +5.3%
- Complexity      939     1006     +67
===========================================
  Files           228      220      -8
  Lines          9066     8446    -620
  Branches       1098     1016     -82
===========================================
+ Hits           2971     3216    +245
+ Misses         5749     4846    -903
- Partials        346      384     +38

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/io/seata/core/protocol/ResultCode.java
75% <ø> (ø)
4 <0> (?)

...ava/io/seata/core/constants/ConfigurationKeys.java
0% <ø> (ø)
0 <0> (?)

...core/protocol/transaction/BranchReportRequest.java
80.68% <ø> (ø)
16 <0> (?)

...seata/core/exception/AbstractExceptionHandler.java
0% <ø> (ø)
0 <0> (?)

...ol/transaction/AbstractTransactionRequestToTC.java
50% <ø> (ø)
1 <0> (?)

.../protocol/transaction/GlobalLockQueryResponse.java
0% <ø> (ø)
0 <0> (?)

...re/src/main/java/io/seata/core/rpc/RpcContext.java
0% <ø> (ø)
0 <0> (?)

.../src/main/java/io/seata/core/protocol/Version.java
0% <ø> (ø)
0 <0> (?)

...ol/transaction/AbstractTransactionRequestToRM.java
50% <ø> (ø)
1 <0> (?)

...ore/protocol/transaction/GlobalStatusResponse.java
0% <ø> (ø)
0 <0> (?)

... and 410 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 9531156...c2efff8. Read the comment docs.


oref0-autotune-test.sh script add

Added oref0-autotune-test.sh script to make testing oref0 autotune easier 😄. Script arguments allow the user to specify:

OpenAPS Directory (--dir)
Nightscout URL (--ns-host)
Start Date (--start-date)
End Date (--end-date)
Number of Runs (--runs)

Note that the pump profile is pulled from the following location: /settings/profile.json in users's specified OpenAPS Directory. Pro Tip: this is the only directory structure requirement, so you could setup PARENT_FOLDER/settings/profile.json anywhere without other OpenAPS files.
Also note that End Date and Number of Runs are not required parameters, but the script will default to the day before today as the end date and 5 runs, so you may or may not want to use those.

Example Usage: ./oref0-autotune-test.sh --dir=openaps --ns-host= --start-date=2016-12-9 --end-date=2016-12-10 --runs=2

Merging now, will test on my end tomorrow!


fix: allow starting two dart_frog instances at the same time

Status
READY
Description
Resolves #568.
New behaviour
https://user-images.githubusercontent.com/44524995/227979052-2ed2c563-b60f-4825-9161-a8a0fa6f5771.mp4
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[X] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

I can't figure out why the CI is failing for Windows and Ubuntu. Working as expected locally. I need to investigate it further. If anyone knows why could this be let me know!

Closing in favour of #599.


Leanpub Book Description

Here's what we're working with on Leanpub for when the book goes live.

Here's some examples of what I consider to be ideal:
https://leanpub.com/cpp20
https://leanpub.com/psprimer
https://leanpub.com/howcanitestthis (this one has a good example with multiple authors)

I emailed Leanpub about adding a Digital Forensics category to Computers and Programming so we can properly categorize this book on their website.

I emailed Leanpub about adding a Digital Forensics category to Computers and Programming so we can properly categorize this book on their website.

This was added and we will be the first Digital Forensics book 😎

Posting this here for future reference for me: http://help.leanpub.com/en/articles/6266784-do-i-need-a-reader-membership-to-make-purchases
I want to make sure the book is free for everyone. The steps are laid out here to do just that.

Posting this here for future reference for me: http://help.leanpub.com/en/articles/6266784-do-i-need-a-reader-membership-to-make-purchases
I want to make sure the book is free for everyone. The steps are laid out here to do just that.

This is done. Closing this issue now.