V4.0 and Higher Casts Params to 1st Specified Type using Typing Syntax

Hi,
This is my first time ever submitting a bugfix request so please take it easy on me :) FYI, longtime fan of this work... My company has converted to use this exclusively for our automated testing. We leverage custom backend libraries in Python and use typing in params to show the accepted formats.  I ran into this issue when bumping from 3.1.2 -> 4.0 on up.
Consider the below files
Supporting library Lib.py:
class Lib:
    '''
    Does cool stuff
    '''

    def __init__ ( self ):
        pass

    def check_param_as_str ( self, param : (str or list) ):
        '''
        Accepts param as type str or list -- asserts when passed in str
        '''
        if type(param) == str:
            raise AssertionError ("Ran into type<str> when expected type<list>")
        return param

    def check_param_as_str_without_typing ( self, param ):
        '''
        Accepts param as type str or list -- asserts when passed in str
        '''
        if type(param) == str:
            raise AssertionError ("Ran into type<str> when expected type<list>")
        return param

and test.robot:
*** Settings ***
Library  ${CURDIR}/Lib.py

*** Test Cases ***
WITH TYPING: Pass in Str
    ${a_str}  Set Variable  some param
    ${param} =  Run Keyword And Expect Error  Ran into type<str> when expected type<list>  Check Param as Str  ${a_str}
    Log  ${param}

WITH TYPING: Pass in List (SHOULD Pass)
    ${a_list} =  Create List  more params
    ${param} =  Check Param as Str  ${a_list}
    Log  ${param}

WITHOUT TYPING: Pass in Str
    ${a_str}  Set Variable  some param
    ${param} =  Run Keyword And Expect Error  Ran into type<str> when expected type<list>  Check Param as Str Without Typing  ${a_str}
    Log  ${param}

WITHOUT TYPING: Pass in List
    ${a_list} =  Create List  more params
    ${param} =  Check Param as Str Without Typing  ${a_list}
    Log  ${param}

On 3.1.2  < version < 4.0, the test cases fully pass and typing works as expected.
On version >= 4.0, the test case 'WITH TYPING: Pass in List (SHOULD Pass)' fails. It appears that using typing with style param:(<type> or <type>) will look to cast the param to the first provided .  Whereas style param: (<type>, <type>) will not effect the param.
It should be noted, when running this code exclusively against Python, latest, the behavior acts like 3.1.2  < version < 4.0.
I am going to try to attach some relevant screenshots / files. Thank you!

First of all, thanks for kind words! I've had pretty rough few days and compliments are appreciated even more than normally. This is a pretty interesting issue because there's a bug in your example but there's also a bug in Robot related to this. I'll explain both below.
The bug in your example is that you cannot specify that argument would have multiple possible types like param: (str or list). Although it reads like it could work, the type hint is evaluated in Boolean context so that its value is str if str is considered true and list otherwise. Since types are true this is exactly same as just using param: str. It's easy to demonstrate that on the Python interpreter:
>>> def func(arg: (str or list)):
...     pass
...
>>> func.__annotations__
{'arg': <class 'str'>}

The right way to specify that an argument has multiple possible values is using typing.Union like Union[str, list]. With Robot Framework you can actually also use a tuple like (str, list), but with Python 3 code Union is generally recommended.
Why the code worked with versions prior to RF 4.0 is that earlier type conversion was done only if the given argument was a string. Thus when you effectively had type hint like param: str and called it with a list, there was no conversion.
The bug in Robot is that it does unnecessary conversion if an argument has multiple possible types and the given argument is one of them. Instead of passing the argument as-is, it converts it to the first possible type. Thus if you'd have type hint Union[str, list] and call it with a list, the list is converted to a string. This will be fixed in RF 4.0.1 (#3897) that I hopefully get released still today. The fix is already in RF 4.0.1 beta 1 so you can install that if you want to test this already before. In this particular case a workaround that works also with RF 4.0 would be using Union[list, str].
Could you test does Union work for you as explained above?


Add Carpool Vote phone number to all rider SMS notifications

We'd like riders to be able to access the self-service portal even if they are not online. This would have to be done via an operator on the phone line.
Please add:
User support:<PHONE_NUMBER>

@sasjkia Please verify

@dmilet this now works - thank you.


REDCap Integration

We are in preliminary talks about how to integrate DRIVE with a REDCap database that will pull in medical center data and other research data. Although this work is only very vaguely defined at the moment, this ticket covers the work involved with getting our data to them and data back from them in some sensible manner

Slack comment from @danfunk :
I've got a read only account created for Dave, but I need to work with Tom Martin to open up a port to the server so that Dave can connect remotely to the database to pull data.  That is in process and I've pulled Dave into the conversation with Tom to hopefully help things move forward quickly.


Data with ui-mask not being stored.

Hey! I am trying to use ui-mask, and then store the data in local storage.
All the fields that I don't use ui-mask are stored, but the ones I do use, are stored as null;
The fields without the mask just don't have the ui-mask and ui-mask-placeholder lines.
Am I doing anything wrong here?
Here is an example:

                    License Plate
                    <input type="text"
                           name="placa"
                           ng-model="dadosAIT.placa"
                           ng-minlength="7"
                           required
                           placeholder="License Plate"
                           ui-mask="AAA-9999"
                           ui-mask-placeholder ui-mask-placeholder-char="_">
                    This field is required!

Thanks!

Are you trying to store "invalid" values in local storage? For instance, a valid license plate would be ABC-1234 according to your mask. Are you trying to store a partially filled out license place, such as ABC-____?

No! I am verifying that, and only storing good data.

The only thing that is slightly wrong is that you don't need the ui-mask-placeholder or the ui-mask-placeholder-char attributes but that shouldn't be affecting the local storage process. Although since I don't know what that code looks like it's impossible to be certain.
I would just look into your storage code and see what the model value for license plate is at the time of saving to local storage. Maybe you need to tweak when that occurs...


Bottlerocket nvidia

1. Issue, if available:
If the AMI family type used is Bottlerocket and the instance type is an NVIDIA GPU instance the drivers are not available in the generic Bottlerocket AMI. The lookup for arm64 architecture was also dependent on GPU instances but they're not mutually exclusive (eg G5g).
2. Description of changes:
This adds a lookup for Bottlerocket AMIs to use the -nvidia variant along with arm64 AMI when needed.
3. How was this change tested?
I based the code off the existing AL2 lookups and ran linting and basic tests but did not have the infrastructure in place to deploy it into a live cluster.
4. Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[ ] No

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

Awesome work! 💯

but did not have the infrastructure in place to deploy it into a live cluster.

Let's hold on this until we can get this manual test done.

I've tested manually.
Scaled up an nvidia gpu request on both amd64 and arm64 w/ Bottlerocket 🚀


Sorting is bugged

I have the following imports:

I press Ctrl+S to save the file and this is what I get:

This is my settings.json:
"[typescript]": {
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "editor.formatOnSave": true,
  "editor.codeActionsOnSave": {
    "source.organizeImports": "explicit",
  },
},

@garysassano doe fix work correctly for you in the terminal using eslint --fix. If so, can you please provide me with a GitHub repository I can clone with a minimal setup that demos what you are seeing.

Had to change the codeActionOnSave block:
"[typescript]": {
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "editor.formatOnSave": true,
  "editor.codeActionsOnSave": {
    "source.fixAll.eslint": "explicit"
  },
},

I wrongly assumed that it would be possible to only apply eslint rules related to imports instead of fixing all, but it's not actually feasible.


Issue on Unit 3

Running the Colab notebook default values in Unit 3 is causing an error to happen when trying to train the model.
Running this line of code:
!python train.py --algo dqn  --env SpaceInvadersNoFrameskip-v4 -f logs/
After what appears to be sucessfully running all cells above produces the error message below:
Traceback (most recent call last):   File "/content/rl-baselines3-zoo/train.py", line 1, in <module>     from rl_zoo3.train import train   File "/content/rl-baselines3-zoo/rl_zoo3/__init__.py", line 6, in <module>     import rl_zoo3.gym_patches  # noqa: F401   File "/content/rl-baselines3-zoo/rl_zoo3/gym_patches.py", line 82, in <module>     patched_registry.update(gym.envs.registration.registry) TypeError: 'EnvRegistry' object is not iterable
Is this a known issue or is their a solution to fix this?

I was able to get it training using the following code below. I didn't test everything else that succeeds training yet.
`%%capture
!sudo apt-get update
!apt install python-opengl ffmpeg xvfb swig cmake
!pip3 install pyvirtualdisplay
import os
os.kill(os.getpid(), 9)
Virtual display
from pyvirtualdisplay import Display
virtual_display = Display(visible=0, size=(1400, 900))
virtual_display.start()
!git clone https://github.com/DLR-RM/rl-baselines3-zoo
%cd /content/rl-baselines3-zoo/
!pip install -r requirements.txt
!pip install gymnasium[accept-rom-license]
!python train.py --algo dqn --env SpaceInvadersNoFrameskip-v4
`

i try to downgrade python to 3.7
but pip, distutils.util, etc..
Python library version dependency issues.
please let me know how to fix it

@simoninithomas

I think this is because SB3 switched to gymnasium.
Replace the pip install of gym[atari,accept-rom-license] with:
!pip install gymnasium[atari,accept-rom-license]
Then replace your command line run of train with:
!python train.py --algo dqn  --env ALE/SpaceInvaders-v5 -f logs/
(I think frame skipping is now passed as an argument in gymnasium.make() so it's not included in the environment name?)

I think the solution will be to use specific branch !git clone https://github.com/DLR-RM/rl-baselines3-zoo --branch v1.8.0

i follow your advice fix code
!pip install gymnasium[atari,accept-rom-license]
!python train.py --algo dqn --env ALE/SpaceInvaders-v5 -f logs/
but i show error like this
Traceback (most recent call last): File
"/content/rl-baselines3-zoo/train.py", line 1, in <module> from
rl_zoo3.train import train File
"/content/rl-baselines3-zoo/rl_zoo3/__init__.py", line 6, in <module>
import rl_zoo3.gym_patches # noqa: F401 File
"/content/rl-baselines3-zoo/rl_zoo3/gym_patches.py", line 82, in <module>
patched_registry.update(gym.envs.registration.registry) TypeError:
'EnvRegistry' object is not iterable ```

2023년 4월 22일 (토) 오전 4:22, dylwil3<EMAIL_ADDRESS>작성:

> I think this is because SB3 switched to gymnasium.
>
> Replace the pip install of gym[atari,accept-rom-license] with:
>
> !pip install gymnasium[atari,accept-rom-license]
>
> Then replace your command line run of train with:
>
> !python train.py --algo dqn --env ALE/SpaceInvaders-v5 -f logs/
>
> (I think frame skipping is now passed as an argument in gymnasium.make()
> so it's not included in the environment name?)
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/huggingface/deep-rl-class/issues/286#issuecomment-1518246964>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AG235R7XAXUGUHFNJI3I34TXCLNA3ANCNFSM6AAAAAAXAOK5AE>
> .
> You are receiving this because you commented.Message ID:
><EMAIL_ADDRESS>>

--
*한준구*
teacher / creator
M 010.3193.1108
<EMAIL_ADDRESS>

+1

The new notebook:  https://colab.research.google.com/github/huggingface/deep-rl-class/blob/main/notebooks/unit3.ipynb


Make a working calculator using Javascript

Make a working calculator using javascript  which can help in making mathematical calculations easily

@ashutoshkrris  can i work on this ??

A calculator is already present in our website @mohit200008


describe use of .password file

Change our docs to instruct users to use a .password file rather than hard coding the pw in their .tds files.

Should we retain the documentation about hard coding it?

Oh I just saw yours! I can drop the manual steps into your branch, but yours looks great.


diff

I use grgit.show and want to see full path to changed files, but instead I receive only top level folders names that contains changed files. Is there any way to get a list of changed files (relative or full path to files)?

Forgot to test show on subdirectories, turns out it's a bug (tracking in grgit#78). I'm pushing a fix up now but won't be releasing right away.

This is getting releases in 1.3.1 (in a few minutes)

Technically fixed in grgit 1.4.1, but bumping gradle-git's dependency on grgit as part of the gradle-git 1.3.1


Diagnostic Scope: AddTag function should be NoOp for Sampled out activity

As part of this PR, making sure AddTag should add information in the activity, only if it is not sample out activity otherwise it should be NoOp.

API change check
API changes are not detected in this pull request.

@lmolkova:  Would you please offer your thoughts?

LGTM! Is it possible to adda test for it?

added test


Updates the team member names.

Test change for updating team member names.

Test change for updating team member names.


Allow for third-party plugins to be loaded into DevToys

What feature or new tool do you think should be added to DevToys?
Allow for custom-written dlls to be loaded into DevToys and expand the amount of features available.
Why do you think this is needed?
That allows for devs to centralize their self-written, specialized codes into a single platform.
If an optional marketplace were to exist, these libraries could be shared with a larger audience, and further expand the amount of tools available.
Solution/Idea
The implementation could be fairly simple.

Create a new abstract class (or interface) representing the structure a would-be plugin should be created against:

This class would require a plugin-writer to provide basic data, such as Name, description and GUID, along with delegate implementations that would be called in specific points in the DevToys lifetime.
 public abstract class DevToolsPluginBase
{
	public abstract string Name{get; set;}
	public abstract string ShortDescription{get; set;}
	public abstract string LongDescription{get; set;}
	public abstract string Guid{get; set;}

  // .. Other mandatory data
	public delegate void OnClick (object state);
	public delegate void OnLoad (object state);

	public virtual OnLoad OnClick {get;set;}	 // Event called when the plugin icon is clicked
	public virtual OnLoad OnLoad {get;set;}	 // Event called when the plugin finishes loading
}

A simple reflection could be used to list the dlls in a previously-provided path, and dynamically load them, provided they contain  type(s) that inherit from the abstract class above.

Comments
Although my experience with WFM is limited, I've done this implementation before in a number of applications, so I'd love to help!

Hi,
Thanks for suggesting this. Here are some great news:

We're working on DevToys 2.0
This new version will support extensions
You can see a preview of our Extension Manager in this pull request: https://github.com/veler/DevToys/pull/901
Every tools in DevToys 2.0 are extensions
We will have an API for it. There will be a documentation. The API is still having some breaking changes at the moment, which is why we do not invite people to try to make extensions for now.
In a few months from now, when we will have a Preview release for DevToys 2.0, we will publish a documentation and invite people to make extensions if they wish.

Hope this help :)

Duplicate with #146


Add first-class email_open event

Need to define the fields for this event. Maybe checkout the fields in the Sendgrid data model?
Also: should the user's email address be put in as a dedicated field, or should it go into uid?
Once this is done, then think about:

Redshift and Postgres table definitions and any recipes/cubes
Adding this to the no-js wizard

We support ingest of email events from MailChimp and Mandrill already, with SendGrid (#1161) on the way, so this should be covered now, closing...


Game gets harder over time

The longer the game goes on, the more dangerous the environment gets

Idea is to limit game time


[AutoPR] recoveryservices/resource-manager

Created to accumulate context: recoveryservices/resource-manager

Build output:


Various improvements to ammonite-sshd:

Modern OpenSSH clients will refuse to accept an ssh-dss server key; generate using RSA instead
Enable keyboard-interactive authentication as well as password-based
Allow password-based and/or publickey-based authentication

Merged this into https://github.com/lihaoyi/Ammonite/commit/b80b7638454cafdfff68368793614b5e5f81a454


Ability to name spaces

This was briefly discussed in chunkwm, and lately mentioned in #116
After looking into this previously it appears that the display name visible in mission-control is simply an enumeration of available user-spaces - thus a space is not assigned a name that we can alter, you'd have to intercept and change the code that draws the mission-control overlay.
This is not something I'll bother looking into, however..
As the query system in yabai outputs a lot of useful info (more to come as of writing this) it would be possible to add a command yabai -m space <space_sel> --name or --alias or something, that would map the space with the given mission-control index to some name.
The mission-control index would only be used from the users side of things - the actual mapping would store the internal space id such that the name would persist through movement. names would then become a valid option for SPACE_SEL.
This sort of a system could then be used together with a query command inside the config file to map a range of spaces during startup.
I imagine this would be something like:
# config stuff..
# ...
# ...
#
# 1. get all spaces
# 2. filter away native-fullscreen spaces (or maybe name them as well, if you want)
# 3. map range of spaces based on their current mission-control index to a range of names

More generally speaking, do you think it makes sense to allow unique names as selectors for windows, spaces and displays and either disallow existing selectors as names or require them to have a certain prefix?

I'm not sure yet what the best option is - but I'd be fine with having selectors as reserved words, and everything else the user can use as their own names.

I've had some thoughts on designs for this, and I think labels fit yabai best. They are easy to implement and offer versatility when used in combination with jq or shell scripts.

Any entity (window/space/display) should be able to get a per-domain unique label assigned to it
The default label should just be an empty string
A label may be anything that does not begin with a digit and is not a valid selector for the entities domain
Labels should be unique within each domain, and assigning an existing label to another entity of the same domain should remove the label from its previous entity

I was looking around SkyLight.framework and noticed the procedures SLSSpaceGetName and SLSSpaceSetName (and also SLSSpaceCreate and SLSSpaceDestroy). Did you try these already, @koekeishiya?

The name in these functions simply refer to the internal uuid of said space. SLSSpaceCreate/Destroy are functionally equivalent to CGS versions as far as I have been able to tell.

Did my first initial take on this, allowing the user to label a space, and refer to that label in every command that takes a SPACE_SEL. Labels are not currently returned in space queries, but it will eventually.
This can later be expanded to work on windows and displays.

Is there a way to remove the label after assigning it to a space?

Not currently. I've created a new issue.


feat: add getGasFeeData for Eth chain adapter

add getGasFeeData to EthereumChainAdapter to return just gas fee info. This will be used to resolve an issue where eth thor trading gas calculation function call can fail in some cases causing the estimated quote to fail.

closes #786

Both this and buildCustomTx are big duplications. If not improving the existing functions to handle the slight differences, can we at least re-use the logic added here in the getFeeData function? I just feel like these additions are just an easy route but not the most ideal.

@kaladinlight After giving it a second look, you're right. Re using the logic that is duplicated inside of getFeeData is a better way to handle this. I'll make that change.


IDA9 not support

I'm using IDA9, but it doesn't work

I have the same issue but I fixed it by
100c100,103
<             dump_type = idaapi.BWN_HEXVIEW
---
>             try:
>                 dump_type = idaapi.BWN_HEXVIEW
>             except:
>                 dump_type = idaapi.BWN_DUMP
418c421,424
<             dump_type = idaapi.BWN_HEXVIEW
---
>             try:
>                 dump_type = idaapi.BWN_HEXVIEW
>             except:
>                 dump_type = idaapi.BWN_DUMP

I have no idea about idaapi, but it seems like my idaapi doesn't idaapi.BWN_HEXVIEW.
I don't know if it's a common problem worth an update.
@L4ys  Thanks for checking this issue.


Refactor: centralize Lesson related API calls into a global Lesson store

There are couple of places we make API calls and some of them are tied to a certain page and components.

List of lessons & activities and detail information
Completion state of lesson & activities

We would want to have these to be centralized and stored so could be queried anytime, anywhere, with or without network connectivity.

PR - Code refactoring for Lesson and activities related function #80


Sikka blocking Google!

I see a number of these in my logs:
Sikka: IP Blocked. <IP_ADDRESS>
Sikka: Closing current connection [the connection id from meteor.connection]

When I do a whois lookup on <IP_ADDRESS> I see that it belongs to Google.
Now I wonder, am I blocking google bots? Does Sikka have adverse affects on SEO?
Additionally, it would be really nice to see exactly why Sikka has blocked an IP.
I doubt Google has some kind of vendetta against me so it has been DDoS'ing my small meteor site :)

Pure Meteor is not SEO friendly, you need something like phantom.js or server-side-render.

This is not about seo, this is about sikka blocking legitimate requests from the most legitimate ip block on the internet.

@serkandurusoy just increase the ip limit 20 is really low. 100+ should be good. But an option to white list well known IPs is better...


New version: KeeperSecurity.Commander version 16.11.20

Pull request has been created with WinGet Updater :rocket:
Microsoft Reviewers: Open in CodeFlow

Verify:

@ stephengillie Your verification screenshot seems to be from another PR. This isn't the KeeperSecurity Commander.

Apologies - I have a bug where sometimes an approval will kick off a validation, and then I post the screenshot into the PR somewhat on autopilot.


Highlight best hand in token poker

Call out the digits that contribute to the hand, to make it clearer what the best hand found was.

this is done in card-poker


Test3

Closes/Fixes/Resolves

Explain the feature/fix

Does this PR introduce a breaking change

Screenshots

View Screenshots

Ready-for-merge Checklist

[ ] Expected files: all files in this pull request are related to one feature request or issue (no stragglers)?
[ ] Does the change have appropriate unit tests?
[ ] Did tests pass?
[ ] Did you update or add any necessary documentation (README.md, WHY.md, etc.)?
[ ] Was this feature demoed and the design review approved?

Kindly specify the names of env u want to specify in the given format [dev,stage,qa]

["dev"]


multipleOf validation throws DivisionImpossible error for Decimal class, if one of the operands is much bigger than the other.

Modulo operation between two instances of decimal.Decimal classes can throw an decimal.InvalidOperation: [<class 'decimal.DivisionImpossible'>] error if the bigger operand is 28 orders of magnitudes bigger than the other (for a % b we get an error if a > b * 10^28). Relevant stackoverflow thread.
I believe this exception should be caught and tried again with a better precision value. I understand that decimal.Decimal isn't used by json.load. Still, decimal.Decimal is being heavily used by Singer taps and targets for some reason.

That's something that should be done outside jsonschema -- as you say, jsonschema doesn't deserialize things. It supports Decimals if they've been loaded properly, but it's up to you to set their precision to what you need them to be before handing them to validation, jsonschema can't really know what precision is appropriate.


BackupEvent query not returning list of backup events

I believe this has something to do with gt not functioning how we expect.
where: {
      gt : lastSummaryDate
}

The query this performs is
SELECT `id`, `machine`, `rsyncExitCode`, `rsyncExitReason`, `transferSize`, `transferTimeSec`, `backupTime` FROM `BackupEvents` AS `BackupEvent` WHERE `BackupEvent`.`backupTime` = '2018-05-30 00:00:00.000 +00:00';

INSTEAD of backupTime > `'2018-05-30 00:00:00.000 +00:00'

Yeah, I had copied that from existing code and it must have never gotten converted to the new Sequelize. That should read:
  [Op.gt] : lastSummaryDate
}```

to use Op.gt, you need to define Op = Sequelize.Op inside the file

required sequelize and defined Op using the above definition, this gets all of the backupEvents since lastSummaryDate. I also found a typo 'backupsEvents' instead of 'backupEvents' in the .then() function chained to the database promise


Feature enhance examples

added some new examples and also add a test suite for the example folder ( to be sure the the examples work)

Coverage decreased (-0.09%) to 93.168% when pulling 4502d79520d32f8d3a5fd8326476e43e3913f546 on feature-enhance-examples into 7de89f2027ca29df9b3b571b5044c02c18aa4bcf on develop.

Coverage increased (+0.3%) to 93.552% when pulling c5c80a040c554465c82cb624d2f3721a772b6f77 on feature-enhance-examples into 7de89f2027ca29df9b3b571b5044c02c18aa4bcf on develop.


[PAIShare] Ignore error of illegal YAML in template source.

Coverage decreased (-0.0003%) to 51.26% when pulling 66a9b905d0d5b11e63187529262d7846acff0de6 on shaocs-paishare into 4012543d9c73fec96967a9d8137bdf7274abfd69 on paishare.


Issue with Pose Landmark Detection Segmentation Mask on iOS/iPadOS 17.4.1

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
Yes
OS Platform and Distribution
iOS, iPadOS 17.4.1
Mobile device if the issue happens on mobile device
iPhone 13 mini, iPhone 13, iPhone XR, iPhone SE 2, iPhone SE 3, iPad pro
Browser and version if the issue happens on browser
No response
Programming Language and version
C++
MediaPipe version
v0.10.9
Bazel version
No response
Solution
Pose
Android Studio, NDK, SDK versions (if issue is related to building in Android environment)
No response
Xcode & Tulsi version (if issue is related to building for iOS)
No response
Describe the actual behavior
Crash when using "segmentation mask" on iOS
Describe the expected behaviour
We would like to have the "segmentation mask" working
Standalone code/steps you may have used to try to get what you need
We're experiencing an issue when using the Pose Landmark Detection (https://developers.google.com/mediapipe/solutions/vision/pose_landmarker) with the "segmentation mask" enabled on iOS and iPadOS 17.4.1. This feature works without any problems on Windows and Android. However, on iOS, enabling the segmentation mask causes the app to crash.

The crash occurs randomly after varying usage times. Sometimes, it works fine for several minutes, but often crashes within a few seconds. The issue seems to happen more quickly on older devices, but using a newer device doesn't guarantee stability.

We would appreciate any assistance or advice you can provide in addressing this issue. Please let me know if you require any additional information.

Other info / Complete Logs
Attached is a backtrace captured at the moment of the crash:

(lldb) bt
* thread #57, name = 'mediapipe/0', stop reason = EXC_BAD_ACCESS (code=1, address=0x11b5ec010)
  * frame #0: 0x000000018f49a404 vImage`vConvert_PermuteChannels_ARGB8888_CV_vec + 276
    frame #1: 0x000000010548ec20 MediaPipeLib`CreateCVPixelBufferForImageFrame(std::__1::shared_ptr<mediapipe::ImageFrame>, bool) + 592
    frame #2: 0x000000010548e584 MediaPipeLib`CreateCVPixelBufferForImageFramePacket(mediapipe::Packet const&, bool, CFHolder<__CVBuffer*>*) + 248
    frame #3: 0x0000000104b0aff4 MediaPipeLib`mediapipe::ImageFrameToGpuBufferCalculator::Process(mediapipe::CalculatorContext*) + 136
    frame #4: 0x0000000105412740 MediaPipeLib`mediapipe::CalculatorNode::ProcessNode(mediapipe::CalculatorContext*) + 776
    frame #5: 0x00000001053eb4e0 MediaPipeLib`mediapipe::internal::SchedulerQueue::RunCalculatorNode(mediapipe::CalculatorNode*, mediapipe::CalculatorContext*) + 292
    frame #6: 0x00000001053eafa8 MediaPipeLib`mediapipe::internal::SchedulerQueue::RunNextTask() + 232
    frame #7: 0x000000010540aa00 MediaPipeLib`mediapipe::ThreadPool::RunWorker() + 372
    frame #8: 0x000000010540a40c MediaPipeLib`mediapipe::ThreadPool::WorkerThread::ThreadBody(void*) + 120
    frame #9: 0x00000001e5425a90 libsystem_pthread.dylib`_pthread_start + 136

@tyrmullen Do you have suggestions here?

@cavisions did you find any fix or workaround?


Create New Organisation - Replace wording on button after the Organisation profile is created from "Check the feed" to "Check the Help Board".

Create New Organisation - Replace wording on botton after the Organisation profile is create from "Check the feed" to "Check the Help Board".
Steps to reproduce:
Pre-requisite: User is logged in

In stagig from Help Board, click on plus icon
Select "Organisation"
"Create New One"
Enter Organisation name, contact email (not used yet), address, select any option under "what are you looking for", Select "Type of Industry", check boxes "By signing...."
"Create Profile"

Current Result: Current name on second botton is "Check the Feed"
Expected Result: Replace name with "Check the Help Board" to be consistent with page's name.

I see that this has been already fixed and it is in production

Please see comment above


OpenVPN Instances: Add more DNS options (which exist in "Servers")

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
Windows networking behaviour is strange - therefore openVPN offers some options for workarounds (existing Option field in "Servers"):
block-outside-dns  (Prevent DNS leaks)
Describe the solution you like
Add this option also to Instances - e.g. to Miscellaneous / Options

It's probably better to add these kind of options to client specific overwrites, given their not applicable for all (see also notes in https://openvpn.net/community-resources/reference-manual-for-openvpn-2-6/).

There are many "windows client only" networks. Here is a global configuration the right solution. As I understand the documentation non-windows clients will ignore it. So exceptional clients (non-windows) will not be harmed.

I support this request. Preventing DNS leaks is a required feature for us to switch to the new OpenVPN instance model. Feature parity is probably expected by many people too.
The current (23.7.10-1) implementation of OpenVPN instances already includes Windows-only options, such as bypass-dhcp or bypass-dns. So adding block-outside-dns to the list seems logical.
Client specific overrides do not fit our use case either, since all of our clients are Windows. IMO, the option should be made global.
Thanks!


chore:Prep release 1.18.0

This is automated release prep. Remember to update CHANGELOG.md to capture changes in this release. Please review changes carefully before merging.

[ ] Updated CHANGELOG.md

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (6b8fe49) 79.38% compared to head (152802a) 79.49%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1515      +/-   ##
============================================
+ Coverage     79.38%   79.49%   +0.10%
  Complexity      755      755
============================================
  Files            93       93
  Lines          2877     2877
  Branches        307      307
============================================
+ Hits           2284     2287       +3
+ Misses          494      492       -2
+ Partials         99       98       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


UnicodeDecodeError: 'utf-8' codec can't decode byte 0x8b in position 6485: invalid start byte

Hi,
Thanks for the great program, I've had no issues at all running through the pipeline aside from one minor hiccup.
I've corrected 5 datasets with TC - all of which ran perfectly fine, however, one data set (ONT directRNA, mapped with minimap2) is continually outputting this error below. I have tried running this on  a HPC, and two local machines, with different version of TC, but to no avail.
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x8b in position 6485: invalid start byte
I have cleaned two other replicates with the same -j -g -v files as I am using for this one. Both were also mapped with the exact same options on minimap2, sorted via samtools.
I'm having suspicions that there could be something funny going on in my minimap2 output .sam file....
Any help would be appreciated
Cheers
Dean

Hi Dean,
That is certainly odd- I haven't run into that type of error before! Your guess about the Minimap2 SAM file seems plausible. Maybe there is a character in there somewhere that TranscriptClean or one of the libraries it uses is not expecting to be there? Would you mind sending me a sample of the offending SAM file (with header) that causes the problem? Hopefully I can help pinpoint the issue.
All the best,
Dana

Thanks for the swift response. Yeah that was my reasoning too? I've attached 4000 lines from the .sam file - let me know if the whole thing would help.
samplesam.txt

I ran TranscriptClean on your file in a couple of different modes, and it finished successfully each time. Were you able to reproduce your error with the file you sent me specifically?

Also, I googled your error message, and I'm getting hits related to files being gzipped when programs were not expecting them to be. Can you send me your full TranscriptClean command in case I can spot anything unexpected?
Best,
Dana

Hi Dana,
I omitted by variant file, thinking that perhaps it was causing the problem given it was the only gzipped input - and voila, the issues disappeared. I noticed that when I ran my other two replicates, I was receiving the "no variant file provided ......" message, despite a .vcf being provided - so it seems the vcf was corrupted. Now for whatever reason the program just didn't want to progress past this . I'm chalking this up to issues with my VCF file and not TranscriptClean so I'll go back and investigate my vcfs as I believe something went wrong when I merged them in the first place.
Thanks a lot for your quick replies, I'm going to go ahead and close this now.

Ok, sounds good. If anything else comes up, feel free to let me know!


Check for import nonexistent module

🔍 Search Terms
check import nonexistent module
✅ Viability Checklist

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
Now according to https://github.com/microsoft/TypeScript/issues/20767, we should write import {} from "./nonexistenmodule" to import nonexistent module.
But, unused module will be removed by IDE or some other import optimize tools. So this code will be removed.
📃 Motivating Example
I don't know.
💻 Use Cases

import module to run it without use symbols of this module
ensure to check nonexistent module error

https://github.com/microsoft/TypeScript/issues/20576#issuecomment-355403887

Duplicate #40450


AVS.InstallPackComplete version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

1st attempt at wincreate upload and install

Thank you.  Will I be advised when the process is completed?

From: msftbot[bot] @.>
Sent: Thursday, May 26, 2022 5:06 PM
To: microsoft/winget-pkgs @.>
Cc: Crazy Tech @.>; Mention @.>
Subject: Re: [microsoft/winget-pkgs] AVS.InstallPackComplete version <IP_ADDRESS> (PR #62119)
@mqdbhttps://github.com/mqdb,
The check-in policies require a moderator to approve PRs from the community.
Our moderators are community volunteers, please be patient and allow them sufficient time to review your submission.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/winget-pkgs/pull/62119#issuecomment-1139162964, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVBTHN5WU7BT6ODBD4H7ISDVMAG2BANCNFSM5XCUDTFA.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you so much for this information. This was my first approach to this and I feared I was not entering the right info.  Should I make the mods to the manifest and resubmit or just adjust the manifest at GitHub? Please forgive my ignorance.
Thank you so much!
Joe
COO, Lead Designer

Toxic Karma, LLC
📠 800.949.0434tel:+18009490434
📳<PHONE_NUMBER>tel:+14806867314
Toxic Karma: Video Poetryhttps://toxickarma.com/

From: ItzLevvie @.>
Sent: Friday, May 27, 2022 5:00:39 AM
To: microsoft/winget-pkgs @.>
Cc: Crazy Tech @.>; Mention @.>
Subject: Re: [microsoft/winget-pkgs] AVS.InstallPackComplete version <IP_ADDRESS> (PR #62119)
@ItzLevvie requested changes on this pull request.

—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/winget-pkgs/pull/62119#pullrequestreview-987415540, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVBTHNY7CZNLADJ5BQO37GLVMC2OPANCNFSM5XCUDTFA.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you so much for this information. This was my first approach to this and I feared I was not entering the right info. Should I make the mods to the manifest and resubmit or just adjust the manifest at GitHub? Please forgive my ignorance. Thank you so much! Joe COO, Lead Designer * Toxic Karma, LLC * 📠 800.949.0434tel:+18009490434 * 📳<PHONE_NUMBER>tel:+14806867314 * Toxic Karma: Video Poetryhttps://toxickarma.com/
________________________________ From: ItzLevvie @.> Sent: Friday, May 27, 2022 5:00:39 AM To: microsoft/winget-pkgs @.> Cc: Crazy Tech @.>; Mention @.> Subject: Re: [microsoft/winget-pkgs] AVS.InstallPackComplete version <IP_ADDRESS> (PR #62119) @ItzLevvie requested changes on this pull request.
________________________________ In manifests/a/AVS/InstallPackComplete/<IP_ADDRESS>/AVS.InstallPackComplete.locale.en-US.yaml<#62119 (comment)>:
@@ -0,0 +1,13 @@
+# Created using wingetcreate <IP_ADDRESS> +# yaml-language-server: $schema=https://aka.ms/winget-manifest.defaultLocale.1.1.0.schema.json + +PackageIdentifier: AVS.InstallPackComplete +PackageVersion: <IP_ADDRESS> +PackageLocale: en-US +Publisher: Online Media Technologies Ltd. +PackageName: AVS4YOU Programs Installation +License: https://www.avs4you.com/license-agreement.aspx ⬇ Suggested change
-License: https://www.avs4you.com/license-agreement.aspx +License: Proprietary +LicenseUrl: https://www.avs4you.com/license-agreement.aspx
________________________________ In manifests/a/AVS/InstallPackComplete/<IP_ADDRESS>/AVS.InstallPackComplete.locale.en-US.yaml<#62119 (comment)>:
@@ -0,0 +1,13 @@
+# Created using wingetcreate <IP_ADDRESS> +# yaml-language-server: $schema=https://aka.ms/winget-manifest.defaultLocale.1.1.0.schema.json + +PackageIdentifier: AVS.InstallPackComplete +PackageVersion: <IP_ADDRESS> +PackageLocale: en-US +Publisher: Online Media Technologies Ltd. +PackageName: AVS4YOU Programs Installation +License: https://www.avs4you.com/license-agreement.aspx +ShortDescription: AVS4YOU Programs Installation 5.2.2 ⬇ Suggested change
…
-ShortDescription: AVS4YOU Programs Installation 5.2.2 +ShortDescription: Get all AVS4YOU software in 1 installation package. — Reply to this email directly, view it on GitHub<#62119 (review)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVBTHNY7CZNLADJ5BQO37GLVMC2OPANCNFSM5XCUDTFA. You are receiving this because you were mentioned.Message ID: @.***>

I think that I have completed the requested changes.  Thank you again for the help. What's next? 😸


Like model update

###Changes

getRepliesToComment: -> getRepliesToComment:parameters:.
SBClient now updates content which it is requesting to like/unlike, including on a failed request.

:+1:


Make it clearer that user can manually enter name, phone, address in Send screen

Description
Changes:
Magnifying glass icon in Send input replaced with “To”
Search field placeholder copy updated
When user enters a phone number, update the label to say "Send to Phone Number"
When user enters an address, update the label to say "Send to Address"

Other changes
Tested
Tested on Android and iOS
Related issues

Fixes #225

Backwards compatibility

Hey @gnardini Is there any update on above observations ?

Everything looks expected! Thanks!


Make compatibile with Python 2

See http://python3porting.com/cextensions.html for more details.

I left a few small comments regarding white space, etc.

Fantastic!  @mcstro thank you very much for your contribution---it is a very nice first pull request.  I am going to release a new version of the library with your changes shortly.
I hope that the library is useful for you.  If you don't mind me asking, what are you using natural neighbor interpolation for?

I just pushed version 0.2 of the library.  You should now be able to install it using pip.
See: https://pypi.org/project/naturalneighbor/


Missing parameter of beta

Hi, in the original VQVAE paper, the commit_loss is defined as
(quantize.detach()-x) ** 2 + beta * (quantize - x.detach() ** 2)

where the beta is usually to be 0.25. But the commit_loss is defined as the following in your implementation:
F.mse_loss(quantize.detach(), x)

So I wonder if the parameter beta is set to be 1 by default or if the second term is missing? Thank you very much.

@Corleone-Huang Hello Corleone! So the beta would correspond to the commit_loss_weight https://github.com/lucidrains/vector-quantize-pytorch/blob/master/vector_quantize_pytorch/vector_quantize_pytorch.py#L543

Hi @lucidrains, seems that the commitment loss in your code refers only to the first term of the equation:
(quantize.detach()-x) ** 2 + beta * (quantize - x.detach() ** 2)
Does the second term appears in you code somewhere? If it does, can you please help me find it? I tried and found no references to it.
If it doesn't - what "moves" the codebook towards the encoder's output? The commit loss seems to "move" only the encoder outputs.

In this repository, the codebook is updated based on exponential moving averages.


Explaination Part

Fixes Issue🛠

Closes #1366
Description👨💻
In this i add explaination section to define armstrong number that what exactly it is
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷
Before
https://github.com/Rakesh9100/CalcDiverse/assets/168094736/156c6c72-0ce0-439c-bbe1-96f96bfe6d7b
After
https://github.com/Rakesh9100/CalcDiverse/assets/168094736/6a3c1c50-b289-4af0-bff2-4412317942e7

Do not create a PR until you have been assigned to an issue @puneet426

ok


Rename EntityBuilder methods

Closes #199

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (7854dfd) 100.00% compared to head (b9d9823) 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##              main      #212   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          124       124
  Lines         8645      8648    +3
=========================================
+ Hits          8645      8648    +3

Impacted Files
Coverage Δ

crates/modor/src/app.rs
100.00% <ø> (ø)

crates/modor/src/lib.rs
100.00% <ø> (ø)

crates/modor_graphics/src/components/camera.rs
100.00% <ø> (ø)

crates/modor_graphics/src/components/frame_rate.rs
100.00% <ø> (ø)

crates/modor_graphics/src/components/material.rs
100.00% <ø> (ø)

crates/modor_graphics/src/components/model.rs
100.00% <ø> (ø)

...modor_graphics/src/components/render_target/mod.rs
100.00% <ø> (ø)

crates/modor_graphics/src/components/texture.rs
100.00% <ø> (ø)

...es/modor_graphics/src/components/texture_buffer.rs
100.00% <ø> (ø)

crates/modor_graphics/src/components/window.rs
100.00% <ø> (ø)

... and 12 more

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


Install Error /w requirements.txt

Followed instructions, got error
Installing collected packages: yapf, tensorboard-plugin-wit, pyasn1, ninja, lmdb, einops, dlib, addict, tensorboard-data-server, six, rsa, pyyaml, pyparsing, pyasn1-modules, protobuf, oauthlib, numpy, networkx, MarkupSafe, llvmlite, kiwisolver, importlib-resources, grpcio, future, fonttools, cycler, colorama, charset-normalizer, cachetools, absl-py, werkzeug, tqdm, tifffile, scipy, PyWavelets, python-dateutil, opencv-python, numba, markdown, kornia, imageio, google-auth, contourpy, scikit-image, resampy, requests-oauthlib, matplotlib, librosa, google-auth-oauthlib, filterpy, face-alignment, tb-nightly, facexlib, basicsr
Running setup.py install for dlib ... error
error: subprocess-exited-with-error
× Running setup.py install for dlib did not run successfully.
│ exit code: 1
╰─> [9 lines of output]
running install
C:\Users\chlyw\anaconda3\envs\video_retalking\lib\site-packages\setuptools\command\install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
warnings.warn(
running build
running build_py
running build_ext
  ERROR: CMake must be installed to build dlib

  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: legacy-install-failure
× Encountered error while trying to install package.
╰─> dlib

Fixed that, but now get this >>
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>python3 inference.py --face examples/face/Carlin.mp4 --audio examples/audio/Carlin.wav --outfile results/Carlin.mp4
Traceback (most recent call last):
File "C:\Users\chlyw\Desktop\video-retalking\inference.py", line 11, in
from third_part.face3d.extract_kp_videos import KeypointExtractor
File "C:\Users\chlyw\Desktop\video-retalking\third_part\face3d\extract_kp_videos.py", line 6, in
import face_alignment
ModuleNotFoundError: No module named 'face_alignment'

I've now tried re-installing and still get same face_alignment error.

Error Log:
Successfully installed MarkupSafe-2.1.2 PyWavelets-1.4.1 colorama-0.4.6 face-alignment-1.3.5 filelock-3.10.1 imageio-2.26.1 importlib-metadata-6.1.0 jinja2-3.1.2 lazy_loader-0.2 llvmlite-0.39.1 mpmath-1.3.0 networkx-3.0 numba-0.56.4 numpy-1.23.5 opencv-python-<IP_ADDRESS> packaging-23.0 pillow-9.4.0 scikit-image-0.20.0 scipy-1.9.1 setuptools-67.6.0 sympy-1.11.1 tifffile-2023.3.21 torch-2.0.0 tqdm-4.65.0 typing-extensions-4.5.0 zipp-3.15.0
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>python3 inference.py --face examples/Carlin.mp4 --audio examples/Carlin.wav --outfile results/Carlin.mp4
Traceback (most recent call last):
File "C:\Users\chlyw\Desktop\video-retalking\inference.py", line 11, in
from third_part.face3d.extract_kp_videos import KeypointExtractor
File "C:\Users\chlyw\Desktop\video-retalking\third_part\face3d\extract_kp_videos.py", line 6, in
import face_alignment
ModuleNotFoundError: No module named 'face_alignment'
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>python3 inference.py --face examples/video/1.mp4 --audio examples/audio/1.wav --outfile results/1.mp4
Traceback (most recent call last):
File "C:\Users\chlyw\Desktop\video-retalking\inference.py", line 11, in
from third_part.face3d.extract_kp_videos import KeypointExtractor
File "C:\Users\chlyw\Desktop\video-retalking\third_part\face3d\extract_kp_videos.py", line 6, in
import face_alignment
ModuleNotFoundError: No module named 'face_alignment'
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>python3 inference.py --face examples/video/1.mp4 --audio examples/audio/1.wav --outfile results/1.mp4
Traceback (most recent call last):
File "C:\Users\chlyw\Desktop\video-retalking\inference.py", line 11, in
from third_part.face3d.extract_kp_videos import KeypointExtractor
File "C:\Users\chlyw\Desktop\video-retalking\third_part\face3d\extract_kp_videos.py", line 6, in
import face_alignment
ModuleNotFoundError: No module named 'face_alignment'
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>pip install face-alignment
Requirement already satisfied: face-alignment in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (1.3.5)
Requirement already satisfied: opencv-python in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from face-alignment) (<IP_ADDRESS>)
Requirement already satisfied: tqdm in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from face-alignment) (4.65.0)
Requirement already satisfied: numpy in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from face-alignment) (1.23.5)
Requirement already satisfied: numba in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from face-alignment) (0.56.4)
Requirement already satisfied: scikit-image in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from face-alignment) (0.20.0)
Requirement already satisfied: torch in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from face-alignment) (2.0.0)
Requirement already satisfied: scipy>=0.17 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from face-alignment) (1.9.1)
Requirement already satisfied: llvmlite<0.40,>=0.39.0dev0 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from numba->face-alignment) (0.39.1)
Requirement already satisfied: importlib-metadata in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from numba->face-alignment) (6.1.0)
Requirement already satisfied: setuptools in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from numba->face-alignment) (67.6.0)
Requirement already satisfied: packaging>=20.0 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from scikit-image->face-alignment) (23.0)
Requirement already satisfied: lazy_loader>=0.1 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from scikit-image->face-alignment) (0.2)
Requirement already satisfied: networkx>=2.8 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from scikit-image->face-alignment) (3.0)
Requirement already satisfied: PyWavelets>=1.1.1 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from scikit-image->face-alignment) (1.4.1)
Requirement already satisfied: pillow>=9.0.1 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from scikit-image->face-alignment) (9.4.0)
Requirement already satisfied: tifffile>=2019.7.26 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from scikit-image->face-alignment) (2023.3.21)
Requirement already satisfied: imageio>=2.4.1 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from scikit-image->face-alignment) (2.26.1)
Requirement already satisfied: sympy in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from torch->face-alignment) (1.11.1)
Requirement already satisfied: jinja2 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from torch->face-alignment) (3.1.2)
Requirement already satisfied: filelock in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from torch->face-alignment) (3.10.1)
Requirement already satisfied: typing-extensions in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from torch->face-alignment) (4.5.0)
Requirement already satisfied: colorama in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from tqdm->face-alignment) (0.4.6)
Requirement already satisfied: zipp>=0.5 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from importlib-metadata->numba->face-alignment) (3.15.0)
Requirement already satisfied: MarkupSafe>=2.0 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from jinja2->torch->face-alignment) (2.1.2)
Requirement already satisfied: mpmath>=0.19 in c:\users\chlyw\anaconda3\envs\video_retalking\lib\site-packages (from sympy->torch->face-alignment) (1.3.0)
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>python3 inference.py --face examples/video/1.mp4 --audio examples/audio/1.wav --outfile results/1.mp4
Traceback (most recent call last):
File "C:\Users\chlyw\Desktop\video-retalking\inference.py", line 11, in
from third_part.face3d.extract_kp_videos import KeypointExtractor
File "C:\Users\chlyw\Desktop\video-retalking\third_part\face3d\extract_kp_videos.py", line 6, in
import face_alignment
ModuleNotFoundError: No module named 'face_alignment'

FIXED. Bunch of issues. Had to launch using python not python3, had to uninstall and reinstall Torch /w Cuda.

Fixed that, but now get this >>
(video_retalking) C:\Users\chlyw\Desktop\video-retalking>python3 inference.py --face examples/face/Carlin.mp4 --audio examples/audio/Carlin.wav --outfile results/Carlin.mp4 Traceback (most recent call last): File "C:\Users\chlyw\Desktop\video-retalking\inference.py", line 11, in from third_part.face3d.extract_kp_videos import KeypointExtractor File "C:\Users\chlyw\Desktop\video-retalking\third_part\face3d\extract_kp_videos.py", line 6, in import face_alignment ModuleNotFoundError: No module named 'face_alignment'

@bbecausereasonss could you tell me how you fix this question?


Build error under Node 16

Node-LMDB should be bumped to at least 0.9.5
See https://github.com/Venemo/node-lmdb/issues/190

Done in https://github.com/nimiq/jungle-db/commit/f02dfd07aac7acda7802244404a7afb17027bf46


Request new module: Module to provision Bastion service on AWS

Related user story
Module to provision Bastion service on AWS - #480
Type of module
terraform
If requesting a terraform module, what is the target cloud platform?
aws
What is the purpose of the module?
Module to provision Bastion service on AWS
What is the category of the module?
infrastructure

Provided repo - https://github.com/cloud-native-toolkit/terraform-aws-bastion
@Sivasaivm You have been added to the repo with Maintainer access


[feat]: Ability to use 2 or more storage devices

Prerequisites

[X] I have searched for existing issues that already suggest this feature.

Describe the Feature Request
At the current time StartOS only allows one storage device. I would like to install bitcoin core. The ledger is almost 500GB so I would like to have it installed on a secondary storage device
Describe the Use Case
Backups, more storage, ability to install bitcoin ledger on a larger storage device, etc.
Describe Preferred Solution
Allow StartOS to recognize secondary storage devices.
Describe Alternatives
None.
Anything else?
Thanks to the devs who developed StartOS, it just needs a little improving.

Closing as a duplicate of #2060 - please search issues before creating new ones to ensure it has not already been requested


[SMALLFIX] Remove unused code

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15363/
Test PASSed.


fix(instrumentation): replace api base url hardcoded value from watsonx

Fixed #2163
I am a beginner in learning python, I don't know how to make or add test for this changes. I need guidence.

[ ] I have added tests that cover my changes.
[X] If adding a new instrumentation or changing an existing one, I've added screenshots from some observability platform showing the change.
[X] PR name follows conventional commits format: feat(instrumentation): ... or fix(instrumentation): ....
[X] (If applicable) I have updated the documentation accordingly.

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


When I refresh browser, show system checkbox turns on

Rancher versions: 5/24
Steps to Reproduce:

Uncheck Show system
Refresh browser

Results:  Show system checkbox turns back on

Same thing as the other issue..

Version - 5/30
Verified fixed


Clean up metaprogramming

Summary

is_* structs prefer SFINAE over partial specialization. This means that things like scalar_type for Eigen matrices are now specialized and pull out the T::Scalar value.

is_* base structs inherit from std::false_type in scal and then their specializations inherit from either std::true_type or std::interal_constant<bool, value>.

Removes value_type because it was redundant to scalar_type. It was only used in two places in the math library.

Some of the tests were looking at std::vector<const T> and std::vector<T&> which are both illegal C++11

The is_* functions use std::decay_t<T> s.t. const and ref types of the check will evaluate to true. I don't know of anywhere in the library where we want to check something is a ref of a type specifically.

Tests
This is a refactor so idt any new tests are necessary
Side Effects
The meta methods are now a bit more well defined. is_vector will now not work for std::vector<const double>, though that is also illegal in the C11+ standards
Checklist

[x] Math issue #1308

[x] Copyright holder: Steve Bronder

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Closing this till I have time to come back to it


Could you change the RP Type from "Playing A Game" to "Listening To"?

It's not really a bug but rather a small 'context issue'.

This has already been addressed and, to my knowledge, it's currently not possible using the standard Rich Presence API (apps like Spotify or Twitch use custom integrations).

But, like. Can you try do like an custom API for it, or it's impossible.

@kkevinm but how did these bots did it?
Custom APIs?

What I meant is that it's not possible with the Rich Presence API, for whatever reason, but possible using the more general Discord API. So yeah, that could be changed, but no guarantees for now.

OK then.
On Sat, Sep 26, 2020 at 10:43 PM Kevin M<EMAIL_ADDRESS>wrote:

What I meant is that it's not possible with the Rich Presence API, for
whatever reason, but possible using the more general Discord API. So yeah,
that could be changed, but no guarantees for now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kkevinm/iTunes-Discord-RP/issues/41#issuecomment-699504655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQQ4WG7CHTBV7I4K2ETXPBDSHX42HANCNFSM4RXM6QQQ
.

What I meant is that it's not possible with the Rich Presence API, for whatever reason, but possible using the more general Discord API. So yeah, that could be changed, but no guarantees for now.

Oh okay


results depend on CPU architecture?

The final test in findreplace.jl was originally developed on a machine with SkylakeX CPU architecture (my work server), and it fails on Haswell-architecture CPUs (like my laptop and the GitHub CI runners).  To make the test pass, the architecture is detected and a different set of results are compared if SkylakeX is detected vs. Haswell.
This is not ideal!!  The root cause should be identified and eliminated, if at all possible.

PR #91 addressed this issue at the test level

PR #93 runs different tests for Haswell and SkylakeX
PR #94 adds testing on different operating systems

Julia 1.7.0 completely breaks the existing test due to major differences in how BLAS/LAPACK are handled (now uses LibBLASTrampoline).  CPU architecture is no longer accessible in the same way.


feat: Configure src/ directory via Vite middleware config

Description
Similar to how Vite configs support assetDir and publicDir options, we'd like to be able to specify a srcDir instead of every project requiring entry files in ./src/.
Additional context
This is to support efforts to incrementally adopt hydrogen and support use cases like monorepos. Examples of this include implementing Hydrogen in the existing brochure2 app and an effort to consolidate the linkpop-web and linkpop repos into a monorepo.
Before submitting the PR, please make sure you do the following:

[ ] Add your change under the Unreleased heading in the package's CHANGELOG.md
[ ] Read the Contributing Guidelines
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[ ] Update docs in this repository for your change, if needed

Hi @RcKeller ! Thanks for bringing this up. Can you have a look at #626 and tell me if that would be enough for your use case? We are currently evaluating the approach we should take.


Money

This file could be useful for people to put algorithms that have to do with money.

You can create the "money" folder


Add unit tests and CI

This PR contains the following changes:

Add pytest-based unit tests
Add Github Actions CI for unit tests

I can squash before merging :)
Closes #72

Thanks for helping out with this! Not super familiar with pytest but the test code looks good to me!

@gerblesh @akdev1l  do we actually want to add a test run to the spec file?  I would think the unit tests would be run on PR in github actions CI

@gerblesh @akdev1l do we actually want to add a test run to the spec file? I would think the unit tests would be run on PR in github actions CI

If you had to ask me, I would say just github actions. Having it in the spec file would be redundant

I'm also planning on implementing some integration tests for the project soon as well so that'll likely be in the build/test action, so that would likely fill the role @akdev1l was mentioning


fix file permissions when generating profiles

File permissions were incorrect when building on macOS.  @vladbarosan could you try this on Windows to ensure it works?  I don't have access to a Windows machine at present.

Can one of the admins verify this patch?


Warn message at start

Hello,
I have allways this warn message on my server start.
I test with a multiverse core stable version but I have allways this.
Thank you very much
PS: I'm on 1.15.2 paper build - 220
`

[Multiverse-Core] "Multiverse-Core v4.1.1-SNAPSHOT-b777" has registered a listener for org.bukkit.event.entity.EntityCreatePortalEvent on method "public void com.onarandombox.MultiverseCore.listeners.MVPortalListener.entityPortalCreate(org.bukkit.event.entity.EntityCreatePortalEvent)", but the event is Deprecated. "Server performance will be affected"; please notify the authors [Rigby, fernferret, lithium3141, main--, dumptruckman].

`

see #2000 it's not an issue to worry. Thanks!


STORY: Create Endpoint for Program

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 (wireframes, business rules, etc.)?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a user I want to see a list of valid programs so that I can include the correct data for my query
Acceptance Criteria
Given a user has selected the url
When the url is executed
Then the system will return program records in JSON format
And for each record will display:

Program Code
Program Description
Comp Parameter Code
Program Group Code

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Dev Tasks

[x]  Create Program entity
[x]  Create Program DTO
[x]  Create ProgramMap
[x]  Create Module
[x]  Create Repository
[x]  Create Controller and add endpoint
[x]  Create Service to return program data to the Controller
[x] Update routes.ts and app.module.ts files
[ ]  Unit tests

Testing
Test Case: TC102
Tester: @davidkwartler
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?
[ ] Has the user story been accepted by the Product Owner?
[ ] Is the user story ready to be deployed to the test/staging environments?

Not sure why the user story has unit type in it? IS this just a copy paste error? @JorjaComer

Created endpoint that returns program data, will work on unit tests on Monday
Need to confirm MATS should not be included in results and if so remove it


refactor DefaultTrainer.from_checkpoint

Only had a brief look at the changes, I think this should work in principle.
But tests are still failing:
Traceback (most recent call last):
  File "/home/runner/work/torch-em/torch-em/test/trainer/test_default_trainer.py", line 81, in test_from_checkpoint
    trainer2 = DefaultTrainer.from_checkpoint(
  File "/home/runner/work/torch-em/torch-em/torch_em/trainer/default_trainer.py", line 195, in from_checkpoint
    trainer_kwargs = cls._get_trainer_kwargs(load)
  File "/home/runner/work/torch-em/torch-em/torch_em/trainer/default_trainer.py", line 175, in _get_trainer_kwargs
    train_loader=load("train_loader"),
  File "/home/runner/work/torch-em/torch-em/torch_em/trainer/default_trainer.py", line 126, in __call__
    return self.load_loader(name)
  File "/home/runner/work/torch-em/torch-em/torch_em/trainer/default_trainer.py", line 154, in load_loader
    ds = self.init_data[f"{name}_dataset"]
KeyError: 'train_loader_dataset'

Please make sure that they pass.

I still don't see why we can't use **kwargs in __call__. Looks good otherwise.

This would fail in case someone needs a special dynamic_kwarg that happens to also be called optional...
It's an edge case, sure...
The updated, simplified version of this does not have this issue anymore: https://github.com/FynnBe/torch-em/pull/1
It builds on top of this PR, but changes the Deserializer a bit to be more analogous to the new Serializer, which is used in _build_init...

Ah ok, I see what you mean. Quite a corner case indeed, but good to catch it here.

The updated, simplified version of this does not have this issue anymore: FynnBe#1

Cool, I will have a look once you think it's good to go and create it as a PR here, for now I think the changes in this one are good to go and I'll merge.


Suggestion: Convert type hints to checks in method

Scalar type hints for arguments and return value could be converted into checks within the function itself, e.g:
function foo(string $bar) : string {

  if (is_string($bar)) {
    throw new \InvalidArgumentException(__FUNCTION__.' expects paramater 0 to be a string');
  }

  $return_val = 1;

  if (!is_string($return_val)) {
    throw new \UnexpectedValueException(__FUNCTION__.' expects return value to be a string');
  }

  return $return_val;
}

Yup, you're right. If you want to work on this I'd accept a PR with tests that adds this the package.
Or you could just use this more feature rich convertor that already supports that functionality: https://github.com/jaytaph/Transphpile


text is corrupt when this plugin is on

$ cat 1234.sh
echo Final aim $str
if [ -n "$str" ]
	then
			# Run
			$TERMINAL &> /dev/null &
      notify-send new_TERMINAL
    # Raise
	else wmctrl -i -a $str
fi

When I open this file with the plugin loaded, I see this

Though the buffer content itself is not corrupt, only the render is. I can copy and paste lines to other buffer and see they are correct

Your file is mixed indentation with spaces and tabs. And shiftwidth and tabstop are not set to the same value.
Either, format your file correctly (and also set expandtab if you want to use spaces, or noexpandtab if you want to use tabs), or at least set them to the same value.

I understand it. But still when someone happens to have such not very sane settings, perhaps accidentally, he doesn't expect the text to be hidden.


Don't be lazy to fetch miq_task_id, it's needed for icon

Reproducer
ChargeBack -> Reports -> Saved Chargeback Reports -> Expand
Addressing error like:
Error caught: [ActiveModel::MissingAttributeError] missing attribute: miq_task_id
app/models/miq_report_result.rb:39:in `status'
app/presenters/tree_node_builder.rb:132:in `get_rr_status_image'
app/presenters/tree_node_builder.rb:89:in `build'
app/presenters/tree_node_builder.rb:29:in `build'
app/presenters/tree_builder.rb:436:in `x_build_single_node'
app/presenters/tree_builder.rb:412:in `x_build_node'
app/presenters/tree_builder.rb:441:in `x_build_node_dynatree'
app/presenters/tree_builder.rb:202:in `block in x_get_child_nodes'
app/presenters/tree_builder.rb:201:in `map'
app/presenters/tree_builder.rb:201:in `x_get_child_nodes'
app/presenters/tree_builder.rb:483:in `tree_add_child_nodes'
app/controllers/application_controller/tree_support.rb:26:in `tree_autoload_dynatree'

@miq-bot add_label ui, bug

@martinpovolny @h-kataria  @dclarizio please review

looks good, verified in UI, fixes load error in Chargeback Reports tree

I wanted to write a test for this one but did not get to it the whole week. There seems to be no test at all for this tree. So maybe next time...


Styles selects and input numbers

Adds a temporary style for selects and customises number input.

CR: @xavijam
CC: @piensaenpixel

Oh cartodb_id appears with the system style :P.

:+1: :dancer: :dancers: :100: :top:

Damn, didn't notice… fixing it.

Fixed! :relaxed:


Fix sandbox sync when original tree contains reactions

This is an attempt at fixing #96. Since patching the sandbox copy when the original tree changes triggers reactions and there doesn't seem to be a way to prevent it from happening at this point, the original tree is now cloned whenever there is a change. This is, of course, much more expensive than applying patches, but the way I'm using the sandbox only requires a single sandbox for the entire tree whereas before I was cloning the tree several times in different places. Also, correctness is more important than performance and perhaps we can eventually find a way (perhaps with some help from the MobX team) to make patches work.
The implementation feels a bit ugly, but I'm not sure how make it more elegant. I'm open for any suggestions. :-)

https://github.com/xaviergonz/mobx-keystone/pull/114 works well and the idea implemented in this PR is flawed, so I'll close it.


Review and set minimum and remove maximum rails version in manageiq related repos and gems

From: https://github.com/ManageIQ/manageiq-ui-classic/pull/7054

We should move all of our repositories to supporting the minimum supported rails version of manageiq and not set a maximum version with the following exception:

standalone gems that can run without manageiq: they can set their own minimum and maximum rails versions but will need to be maintained/updated when we need to change manageiq's rails version.

manageiq-schema currently requires the maximum version be equal to the minimum supported manageiq version because when you generate migrations, it will version the migration based on the newest version of rails that is compatible with the schema repo, so we have to mirror the "release" version of manageiq there. That version is also the version the schema repo will test against.

In the past, we've had the follow repos/gems that required us to "bump" the rails or rails subgem dependencies and it seems like they should be reviewed against the guideline above (or have this guideline revised) so we don't do this everytime we upgrade rails:

manageiq-gems-pending (AR, AS)
manageiq-schema (rails, requires special treatment, see above)
inventory_refresh(AR)
manageiq-postgres_ha_admin (AS)
activerecord-id_regions (AR, AS)
manageiq-providers-openstack (AS)
ovirt_metrics (AR)
vmware_web_service (AS)
manageiq-ui-classic (rails)

I believe this is all of them.
The difficult part of this issue is if you drop the maximum version, travis will run tests against the latest version by default and not be testing the version we might actually be using so you need to fix the test matrix first.

cc @agrare since there are a number of provider specific things here.

to supporting the minimum supported rails version of manageiq

This is going to be tough for some of these, since a number of these gems are used by multiple released versions of MIQ.
At a minimum ovirt_metrics, inventory_refresh, and vmware_web_service all use the same version for ivanchuk, jansa, and master.
I think these gems should leave the dependencies relatively open and only require something specific if it is needed by that gem.  E.g. hypothetically if vmware_web_service used something which was dependent on AS >= 5.2.0 then we can cut a new version with that dependency but otherwise I don't see the problem allowing e.g. ~> 5.0 if what that gem needs out of ActiveSupport would work on any of those versions.

Thanks @agrare.  I totally agree.  I would lump the referenced gems into the first category:

standalone gems that can run without manageiq: they can set their own minimum and maximum rails versions but will need to be maintained/updated when we need to change manageiq's rails version.

If you can run them in irb to do their thing, they can maintain their own minimum/maximum version but like you said, leaving them open ended is best although they need to have pretty good testing to ensure they work with various versions.
If we have a jansa, ivanchuk, etc. release branch in the repo, it should probably be in the other category and only set a minimum rails version to match manageiq and leave the maximum very open.  If we have a repo that has these branches but needs to work on other versions of rails, we might have to move them to being a real gem.
None of this is set in stone and honestly, the only real problem is where we're very pessimistic and set a maximum rails version to the current version we support... setting 5.3 for rails right now means we can only run up to 5.2 since 5.3 doesn't exist and 6.0 can't be used.  I'd prefer we remove these max rails version constraints where it makes sense.


Normalize to Unix-style path separators

This addresses issue #99 and is hugely based on the tar implementation.

👍
Thanks!

My pleasure


Prohibition of event spoofing

Q
A

Is bugfix?
✔

New feature?
✔

Breaks BC?
❌

Tests pass?
✔

Fixed issues
❌

See the added test to get more info

Maybe we should throw an exception in that case, because users may rely on this spoofing and will not know this does not work

👍


Added docs for stream scan and scanMerge

These hadn't been added yet so I took a stab at it. TBH I'm a bit out of my depth, so I looked at the flyd docs for some help. Their scan docs appear to have typos so I tried to straighten it out. scanMerge is here

@JAForbes would you be willing to take a look?

I think it's all spot on, I just added some ideas / notes to the diff.  But you could merge as is and be 100% accurate.

@JAForbes thanks! If someone wants to act on your feedback I'll happily merge that PR.


Accept null for optional user data and don't treat accidentallyOptional instance fields as possibly null

First, null should be accepted as user data for optional fields. Optional columns are NULL-able, so accepting T | null | undefined is pretty reasonable.
Other change relates to accidentallyOptional in model instances not being treated as nullable, so instead of T | null, we will have just T (same as old seqelize models). Accidentally optional is basically specifying loopholes that never got exploited. The main question is whether we want to validate these fields as nullable as well - this line.
Without these changes, constructs like this cannot pass type checks:
const plan = await models.v4.plan.findOne({
  where: { id: createBrandedValue(1) },
});

if (!plan) {
  throw new Error();
}

await models.v4.plan.create({ ...plan });

This is needed for https://github.com/UN-OCHA/hpc_service/pull/2445

94def03 makes sense to me
The change in e2cc91d means that there's effectively no difference between required and accidentallyOptional as far as the way it's treated in types goes. I'm not neccesarily opposed to this, but then should we just get rid of accidentallyOptional entirely, or do we want to still list them differently to note that we may want to make schema changes?
Beyond this, if we do make a change to this type, we also need to make a change to the validation to ensure that all accidentallyOptional fields are guaranteed to be set when we read, and throw an error if not. This can be done here: https://github.com/UN-OCHA/hpc-api-core/blob/develop/src/db/util/validation.ts#L147
(it's one of the rare occasions in our codebase where the io-ts validation is separate from the types)

That is why I committed separately :)
My thinking was that required and accidentallyOptional are the same, and that accidentallyOptional indicates we may want schema changes. After all, that is how I make decisions where to put the column:

NOT NULL: required
NULL: Make a query in DB <column_name> IS NULL and

If there are no records returned, it's accidentallyOptional
If there are any records, it's optional

I have also commented about validation, and included the link to the same line of code, just used future-proof commit hash, instead of develop in path :)
If we only keep 94def0343bae5e7f85a38aaefd90bc0c91d068e3, given a little exercise that I used in opening comment
const plan = await models.v4.plan.findOne({
  where: { id: createBrandedValue(1) },
});

if (!plan) {
  throw new Error();
}

await models.v4.plan.create({ ...plan });

that would mean we cannot clone columns marked as accidentaly optional, and would need to provide what developer thinks is a default value. For example, with plan, we need to provide value for restricted.
const plan = await models.v4.plan.findOne({
  where: { id: createBrandedValue(1) },
});

if (!plan) {
  throw new Error();
}

await models.v4.plan.create({ ...plan, restricted: false });

Providing defaults to accidentallyOptional might be fine for boolean and string columns, but what if we want to clone some other model, which has foreign key marked as accidentallyOptional, what default value do we provide in such cases.
In CONTRIBUTING.md, there is a section Special handling of "accidentally optional" fields where I see that you meant more loose definition of accidentallyOptional, so I'll let you decide on this

The HPC schema has a number fields defined that were accidentally left "nullable" across its tables. Sometimes there is data that is null and sometimes it's just the schema that is incorrect and a migration would fix the issue.
To account for this, models can be defined in a way that allows developers to specify which fields are "accidentally optional". When this is done, the static type-checking requires this field to be specified when creating new rows for a table, but when reading data from the table, it is defined as possibly null.

Right, I think i envisaged that accidentallyOptional would also handle the cases where the majority of the data was there, and the intention of the property was almost certainly for it to be required, but some data has slipped through the cracks.
To be honest... this concept was envisaged at a time where we didn't have any run-time type-checking of the data being returned from the database (and in particular the presence of fields that were optional but probably shouldn't be). So to protect the code, the idea was that we'd mark these fields as possibly null.
Now that we however have runtime type-checking of all data and all fields when read from the database, if we treat  accidentallyOptional technically in the same manner as we treat required, then we'd at least get runtime validation of these fields, and if we encounter data that is later missing from these fields, we'll get a validation error and not be able to read the data, rather than allowing a value to slip through the type-safe boundary.
This is still a vast improvement from just assuming possibly null fields are non-null, but it does mean that there's a chance that there is data we wouldn't be able to read. That also being said, accidentallyOptional does offer a nice migration path for fields from optional to required, as it's the only field type that requires a field when creating an entity, but is possibly null when reading. So e.g. you can first prevent any more null values being written, while still being able to read them, and then write a migration using the models, and then later make the field NON NULL and change the type to required.
So I see our options as some combination of the following things:

make accidentallyOptional technically equivalent to required, just treated as an annotation effectively.
get rid of accidentallyOptional and just use required for these fields. Adding a TODO annotation to those fields with incorrect constraints.
rename accidentallyOptional to futureRequired, or introduce another field type of that name, that requires fields when creating entities, but has the rows potentially null when reading.

I'm also not sure what the best path forward is, maybe (1) and (3)? Having the field as a different type does allow for easy identification of fields that need their constraints changed. We can also add unit-tests that check the constraints of the fields against the definitions at some point too.

Another thing we could do is tackle DB constraints soon. Do we have a ticket to track this work? It should be easy to search for columns marked as accidentallyOptional and write a migration for those that have no rows with NULL in given column.
I plan to define all (or most) of the models in hpc-api-core soon, so I could also write the migrations, so we can switch these columns to required. We can then remove accidentallyOptional or treat it as "I don't want this column to be nullable any longer`

Yeah that sounds good too! I don't think we have a ticket though?


improve UX of the main Jobs by adding accordion style interface

Overview
From the epic in #765 and a focus to improve the UI-UX of Jobs, this item is focused on improving the general UI-UX of the main Jobs page.  Once there is 20+ Jobs on a single page, it becomes hard to visualize and digest.  This proposal is to add an "accordion" style interface to view Jobs.
An example of that can be found here: https://www.networktocode.com/training/private/
I've also seen an implementation that could possibly be used for this from @jvanderaa and @josh5276
User Story
As Ozzie the Operator, who views and executes A LOT of Jobs,
I want to streamline the Job experience for myself and my other team members,
So that we can focus and view only the Jobs of a given "category" or "type" (not to be confused with a new feature that allows user-defined categories; this is analogous to the current "sections" that exist on the Jobs page).
I will know this is done when ...

Each Jobs section can be rolled-up/down using an accordion-like view using an arrow or plus sign
There is an "easy" button at the top that allows a user to expand all or hide all Jobs
There is a setting defined in settings.py?? that allows me to state a default of hide all or show all Jobs.

Implemented in next (v1.3.0) by #1232.


⚠ Fortnite has degraded performance

In 8678098, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1502 ms

Resolved: Fortnite performance has improved in 311f7f8.


[PENDING] Windows standalone binary

Dear Mayank,
Could you be so kind to generate .exe for the rest of us who use Windows w/o Node?
I’m told that either Nexe or EnclosureJS might help you to achieve that.

Err… Hello?

Six months passed. Err… Mayank, hello?

Standalone binaries can be tricky w/ node, and are generally not recommended. Any particular reason you need a .exe for this instead of just being able to install node?


Server starting but Players 0 / 0

Hello,
I'm trying to setup an Ark Server (one instance) on a dedicated server running centos 7.
Everything seems fine until I run the status command :

Running command 'status' for instance 'main'
Server running:   Yes
Server PID:   8995
Server listening:   Yes
Server Name: MusketeersServer - (v310.68)
Players: 0 / 0
Active Players: 0
Server online:   No
Server build ID:   5036335

I've waited an hour and it doesn't change. When I go to my steam servers, it shows during around 1 second with the game name and 0/0 players then disappear. So I think it's not a firewall issue.
This is what i get for the used port :

lsof -i -P -n
COMMAND    PID  USER   FD   TYPE DEVICE SIZE/OFF NODE NAME
ShooterGa 8995 steam   10u  IPv4 392273      0t0  UDP *:27015
ShooterGa 8995 steam   16u  IPv4 394257      0t0  TCP *:32330 (LISTEN)
ShooterGa 8995 steam   18u  IPv4 391638      0t0  UDP *:7778

Can someone help me ?

Server online: No suggests that the server can't communicate with the Steam master server.


Reorder members

Classes are more readable if their elements are ordered by visibility (public first, then internal, then protected, and last private).
At least with Java IntelliJ can do the job.

I don't personally care, as I use the structure view in intellij (view 7) which will order by visibility anyway if you want it to. But if you want to change them around be my guest :)

Most members were already ordered properly. In some cases it was not possible to get the desired order because of dependencies between members. So, you were right not to care ;-)


[Question] shulkers

isn't it too op to be able to have 4 shulkers inside a bundle?
maybe add it as an option in a config file?

a i might have forgotten about that
On Thu, Jun 16, 2022, 03:16 Shardion @.***> wrote:

Hmm, it appears Bundles can't contain Shulker Boxes in the first place?
—
Reply to this email directly, view it on GitHub
https://github.com/Shardion/unstackabundles/issues/1#issuecomment-1157128583,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP32RUQIRS56I55MUDS3CQTVPJ56PANCNFSM5Y4JAULQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix typos in README.

Thank you. I'll merge after the green light.

Green light!

:vertical_traffic_light:


Reset not working

Hi,
When i am trying to reset the form, it does not reset multi select fields..

Sorry changed the state now it's working..


Add GNSS Metadata Field (Pro) - Hosted Feature Service Failed

I;m getting the error below when trying to run the Add GNSS Metadata Field (Pro) - Hosted Feature Service tool on a feature layer in our ArcGIS Online site:

Start Time: Thursday, August 10, 2017 11:33:36 AM
Running script AddGNSSFieldsPythonAPI...
Failed script AddGNSSFieldsPythonAPI...
Traceback (most recent call last):
File "C:\ArcGIS\ArcGIS_Desktop_Tools\collector-tools-master\CollectorUtils\add_update_gnss_fields_python_api.py", line 17, in
from arcgis.gis import GIS
ImportError: No module named 'arcgis'
Failed to execute (AddGNSSFieldsPythonAPI).
Failed at Thursday, August 10, 2017 11:33:37 AM (Elapsed Time: 0.88 seconds

Please make sure that you have the ArcGIS API for Python installed. Here are instructions to do this through ArcGIS Pro.

Thanks @danmaps, That worked. I had this working at the previous version of ArcGIS Pro. I don;t remember adding the Python package, but must have. I apologize for submitting this as an issue. I should have read the documentation closer.


As a deepface api user, i should use gunicorn server

currently, we are running deepface api with flask run command. this is not recommended for production purposes.
python api.py

i should be able to run it with a production server such as gunicorn. in that way, we will be able to scale it with many workers instead of single core.
#!/usr/bin/env bash
gunicorn --workers=1 --timeout=3600 --bind=<IP_ADDRESS>:8000 "api:create_app()"

moreover, please add a shell script (.sh) to run service easily in scripts folder.
finally, api usage should be changed in dockerfile.

we should not forget to add gunicorn dependency in requirements.txt and setup.py

done


Add release rating endpoint

Discogs introduced a new endpoint: https://www.discogs.com/developers/#page:database,header:database-release-rating

New endpoint added in 0.8.0


Add S3 logging to TensorFlow's logging system

This fix is an attempt to help the issue raised in #15159 where there is no logging in S3 file system and it is not easy to debug to diagnose.
This fix adds S3 logging to TensFlow's logging with
LogLevel::Info -> INFO
LogLevel::Warn -> WARNING
LogLevel::Error -> ERROR
LogLevel::Fatal -> FATAL

This fix is related to #15159.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

@drpngx Thanks for the review. The PR has been updated. Please take a look.

I see, sorry I didn't spot that, but it looks like s3_logging is more like aws_logging (including classes and tag). Could you make that change? Move s3_logging to aws_logging and change everything from s3 to aws in that file and change the tag name, Thanks.

Thanks @drpngx. The PR has been updated with naming changed. Please take a look.

Thank you!
Jenkins, test this please.

Sigh. This seems to trigger an extra log message that foils the debug.

ERROR: offline_analyzer output didn't match expectation: 2017-12-27 23:08:24.429427: I tensorflow/core/platform/s3/aws_logging.cc:53] Initializing Curl library
ERROR: dump_dir flag is empty.
Expected output: ERROR: dump_dir flag is empty.
================================================================================

Looking at the test, I think we should suppress that line from the input (any log info) before we compare with the goldens. Could you do that?

Jenkins, test this please.

/CC @gunan out of disk. I'm wondering if we need to add more quota.
devmapper: Thin Pool has 968455 free data blocks which is less than minimum required 983040 free data blocks. Create more free space in thin pool or use dm.min_free_space option to change behavior

Jenkins, test this please.

Looks like it still has an issue, could you check locally?
Testing offline_analyzer

ERROR: offline_analyzer output didn't match expectation: 2017-12-29 00:07:58.941632: I tensorflow/core/platform/s3/aws_logging.cc:53] Initializing Curl library
ERROR: dump_dir flag is empty.
Expected output: ERROR: dump_dir flag is empty.

Thanks @drpngx. The PR has been updated. The previous failures should have been fixed now.
There is a new failure of GPU Python3 — Internal CI build failed without any detail. Maybe it is from infrastructure failure and is unrelated?

Jenkins, test this please.

Jenkins, test this please.

Jenkins, test this please.

Woohoo!


BUG impossible to open Kali..

Hi,
im unable to open kali after installing commando.
and why this is the version 2018.3 and wsl 1 ?
the error is when opening kali-gui.exe:

and when i open kali.exe no tools is installed !?
I really want to use this distro to deploy this during internal engagement.. but if it this to hard to deploy without problem :/
Version
Windows 10 Pro Activated 21H1

@littlestewart you might want to make sure that you have run apt update && apt upgrade -y. Additionally take a look at some of Kali's metapackages so you have all of the tools. I'd recommend starting with kali-linux-default and also kali-desktop-xfce as a start. If you are going in somewhere where you wont have internet connectivity you might try kali-linux-large.
As for why it's 2018.3, we do not manage that chocolatey package, it is maintained by Offensive Security, and they havent updated in a while. As for WSL1 I havent found a good way to reliably auto-install WSL2 since it seems to be only supported on the newest version of windows each time (rather than 1909+)..
To be honest I havent looked at this package in a while, so if there are any more bugs please post them here :-)


[Onboarding] No results returned screen

Apparently there are no artists called Marinda :). (I was looking for Marina and made a typo)
What shall we show? I can stick in a simple label, but let me know if you have thoughts @katarinabatina

Ah great question @mennenia

Done!

it looks slightly darker, but the lighter color in our set was too light :S

Looks correct to me!