Add ability to save changes to changeset regardless of validation status, and then validate on changeset.execute()

So after building out all of Ghost's new validation functionality using this, I ran into an issue with this library that forced me to re-write a private method (which is bad practice 😁 ). Essentially, I found out that my changeset didn't contain the most recent changes, because only valid changes were actually persisted to the changeset.
This isn't a bug, because it says in the readme:

The idea behind a changeset is simple: it represents a set of valid changes to be applied onto any Object

Here's the specific use-case for why this broke my refactor: the validation flow in Ghost works by validating when the particular input is de-focused, and then all inputs are validated when the form is submitted. After all changes are validated (using the changeset.validate) method, then the changes are persisted to the underlying model.
The problem arises because if the user de-focuses an input without the content of said input being valid, the actual input value doesn't exist in the changeset, resulting in an incorrect error message displayed (e.g. the please enter a value error message other than the invalid format for value message).
After reading through the Ecto Changeset documentation, it seems that the changes are set on the changeset regardless of validation status (it differentiates "casting" from "validation"). I could be reading this wrong though.
I understand that not everyone would necessarily want to use ember-changeset this way, but I think there could be an ability to do both at the same time, based on how the changeset is created initially (maybe a third parameter passed to the changeset constructor that enables/disables this new method of behavior?)
In terms of code changes, it's pretty minimal.
Thoughts?
/cc @poteto

The problem arises because if the user de-focuses an input without the content of said input being valid, the actual input value doesn't exist in the changeset, resulting in an incorrect error message displayed (e.g. the please enter a value error message other than the invalid format for value message).

I'm not sure if I'm misunderstanding, but in my mind this doesn't feel like how ember-changeset works - I'm assuming on blur / focus out you run some kind of action to validate(propertyName). If you then do {{#if changeset.error.propertyName}}, the validation attribute on that object should be the right error message (not that it is blank). For example, if you try out the changeset demo you'll see that it behaves in the way I just described it.
In ember-changeset I'm implicitly casting when the value is set, which I think is more robust on the client side. There isn't really any gain to be had from splitting it up.

@acburdine I had a similar issue using ember-changeset with ember-paper where a paper-input component matches the value passed to it exactly—if the value is invalid, the input wouldn't reflect what the user is typing because the changeset attribute wasn't updated.
I don't think this is a shortcoming of ember-changeset, but rather a quirk of my use-case when trying to integrate with another (very opinionated) library.
I worked around this by creating an intermediary component to handle the value of the input (versus the value of the changeset attribute).  This is specific to my situation, but maybe you could modify to work for you.
x-paper-input.js
export default Ember.Component.extend({
  errors: [],

  inputValue: Ember.computed.oneWay('value'),

  actions: {
    updateValue(value) {
      this.set('inputValue', value);
      this.sendAction('onChange', value);
    }
  }
});

x-paper-input.hbs
{{paper-input value=inputValue onChange=(action "updateValue") errors=errors}}

Usage
{{#with (changeset model) as |changeset|}}
  {{x-paper-input
      value=changeset.firstName
      onChange=(action (mut changeset.firstName)
      errors=changeset.error.firstName.validation
  }}
{{/with}}

@poteto I went and looked at the updated changeset, and was able to reproduce my error. Apologies for perhaps not explaining it well at first.
To reproduce the error on that twiddle, you must:

Have a validatePresence and another type of validation in the field
Start off with a blank value
Fill out the input with an invalid value
tab/focus out

After which the error message will also include <value> cannot be blank as well as the second validation, even though there is still a value in the field. (In the case of that twiddle, I had the user's last name be initially set to a blank string, then typed one letter, then tabbed out.

@acburdine I believe this is fixed, try it out again on the demo!

@poteto Sorry for not replying sooner. I tested it out in the demo and it fixed the issue! However, I need to apply the update to my PR in Ghost to verify it there, once that's done I'll go ahead and close this 😄
Thanks for getting it fixed so quickly!

@poteto Finally got around to updating ember-changeset in my PR, this issue has been fixed 😄


teleop_twist ROS2

ISSUE TEMPLATE ver. 0.4.0

Which TurtleBot3 platform do you use?

[x ] Burger
[ ] Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[ ] ROS 1 Melodic Morenia
[ ] ROS 1 Noetic Ninjemys
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[x ] ROS 2 Foxy Fitzroy
[ ] etc (Please specify your ROS Version here)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Intel Joule 570x
[ ] Raspberry Pi 3B+
[ ] Raspberry Pi 4
[ ] etc (Please specify your SBC here)

Which OS you installed in SBC?

[ ] Raspbian distributed by ROBOTIS
[x ] Ubuntu MATE (16.04/18.04/20.04)
[ ] Ubuntu preinstalled server (18.04/20.04)
[ ] etc (Please specify your OS here)

Which OS you installed in Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[x ] Ubuntu 20.04 LTS (Focal Fossa)
[ ] etc (Please specify your OS here)

Specify the software and firmware version(Can be found from Bringup messages)

Software version: [x.x.x]
Firmware version: [x.x.x]

Specify the commands or instructions to replicate the issue.

HERE

Copy and Paste the error messages on terminal.

HERE

Please describe the issue in detail.

HERE

I would like to find out where I can locate the file it is not is the GIthub.
Thank you and best regards

HI @elthef  :)
Do you mean teleop_twist_keyboard?
You can find it in /opt/ros/foxy/lib on your computer
Please refer to the image below

Thank you.

Closing this issue as there isn't recent activity. Please feel free to reopen when necessary.

Closing this issue as there isn't recent activity. Please feel free to reopen when necessary.


Add documentation

Add documentation

@mht-sharma @mfuntowicz I'll wait for you to merge this PR (I don't know Mohit if the doc is ready on your side) to test if the PR doc is properly generated, uploaded and retrieved. And then I'll open a PR in Optimum to add it to the main doc.

@mht-sharma @mfuntowicz I'll wait for you to merge this PR (I don't know Mohit if the doc is ready on your side) to test if the PR doc is properly generated, uploaded and retrieved. And then I'll open a PR in Optimum to add it to the main doc.

Thanks @regisss, I have done changes from my side.

@mht-sharma can you start drafting a release notes to publish along with the release? 🙂
Let's do a proper release early next week this way

@regisss @mht-sharma I'm not familiar with the documentation builder, @regisss do you want to review the high-level?
Otherwise i think we can merge this one, check the doc builder output and address in a second PR.
Approving for now to help moving forward.

@mfuntowicz It looks good on my side, I pushed a couple of commits to add what was missing. Let's merge it!
Then, I'll check if the PR doc works correctly (and fix it if necessary) and open a PR in Optimum to add the doc for Optimum Furiosa.


Button to refresh the run list page

If I am on the Run list page of my product and somebody storing some new runs to this product I have to refresh the whole page to see these new runs.
It would be great to introduce a new Refresh button for example beside the filter bar which will refresh the Run list page.

Thank you :-)


cached_download, but with custom file name

Hi there !
Thanks for building this very helpful interface. We are integrating hugging face to SpeechBrain to release all our pretrained speech models. However, we would like to be able to allow users to specify what will be the name of the file that they download. In practice cached_download stores the downloaded file with a very long and hashed filename. What is the best way to allow for a custom name (in the end, it's custom save dir + custom file name).
Thanks !

The issue is that we are using this hash to find out if we already have the file or not.
Hard to say how to work around this without seeing your use case – maybe just mv the file after downloading it in that case?

Well, doing a "mv" wouldn't be very clean in this case. The problem is that the model would be downloaded again and again every time the user restarts the script (or a slurm scheduler) even if already downloaded once. It's just not super "user friendly" but we can live with it.

What about a symlink? Otherwise feel free to open a PR if you have an idea!

I feel I will just go with your hash-style file names. It will just not be super transparent for users, but that's ok I guess.


Successfully run clinvar-ingest workflow with a full ClinVar Release file

Test the workflow against a full clinvar release file
Noted issues with timeouts and throttling inactive containers
Next steps:

MVP getting single run through end-to-end in Cloud Run Job

with manual trigger through gcloud
gcloud run jobs execute clinvar-ingest-workflow-py --region us-central1 --update-env-vars='directory=/pub/clinvar/xml/VCV_xml_old_format,host=https://ftp.ncbi.nlm.nih.gov/,last_modified=2024-01-07T15:47:16,name=ClinVarVariationRelease_2024-02.xml.gz,release_date=2024-02-01,released=2024-02-01T15:47:16,size=3298023159'

Connect the ClinVar FTP watcher to execute the Cloud Run Job (instead of the Workflow), with inputs passed as env vars instead of HTTP POST.

Notify upon successful run. Slack message (to ____ channel). Either send Slack message directly, or use Google Cloud Alerts.

Moving items 2 and 3 to their own respective tickets.


mergegateway broken in 1.0.2

Description:
when i have upgrade envoy gateway from 1.0.1 to 1.0.2 i saw that envoy gateway operator does not merge gateway anymore. And see a lot of deployment when i created kind: Gateway. Rolled back deployment fixed this problem.
Environment:
kubernetes: v1.27.3
Envoy Gateway 1.0.2

hey @Uburro can you share the errors from envoy gateway logs as well as the config ?

hey @Uburro can you share the errors from envoy gateway logs as well as the config ?

Hi
cm
apiVersion: v1 data:   envoy-gateway.yaml: |     apiVersion: gateway.envoyproxy.io/v1alpha1     kind: EnvoyGateway     extensionApis:       enableEnvoyPatchPolicy: true     gateway:       controllerName: gateway.envoyproxy.io/gatewayclass-controller     logging:       level:         default: info     provider:       type: Kubernetes kind: ConfigMap
Enoyproxy
spec:   logging:     level:       default: warn   mergeGateways: true   provider:     kubernetes:       envoyDeployment:         container:           image: company.cloud/mirror/io/docker/envoyproxy/envoy:distroless-v1.29.2           resources:             limits:               memory: 1Gi             requests:               cpu: 350m               memory: 1Gi         patch:           value:             spec:               template:                 spec:                   containers:                   - name: envoy                     ports:                     - containerPort: 10443                       name: https                       protocol: TCP                     - containerPort: 10080                       name: http                       protocol: TCP         pod:           affinity:             nodeAffinity:               requiredDuringSchedulingIgnoredDuringExecution:                 nodeSelectorTerms:                 - matchExpressions:                   - key: node-role.nebius.ai/msp-infra                     operator: DoesNotExist             podAntiAffinity:               requiredDuringSchedulingIgnoredDuringExecution:               - labelSelector:                   matchExpressions:                   - key: envoy-gateway                     operator: In                     values:                     - private                 topologyKey: kubernetes.io/hostname           labels:             envoy-gateway: private         replicas: 2       envoyService:         externalTrafficPolicy: Local         patch:           value:             spec:               ipFamilies:               - IPv4               ports:               - name: https                 port: 443                 protocol: TCP                 targetPort: 10443               - name: http                 port: 80                 protocol: TCP                 targetPort: 10080         type: LoadBalancer     type: Kubernetes   telemetry:     accessLog:       settings:       - format:           json:             authority: '%REQ(:AUTHORITY)%'             bytes_received: '%BYTES_RECEIVED%'             bytes_sent: '%BYTES_SENT%'             downstream_local_address: '%DOWNSTREAM_LOCAL_ADDRESS%'             downstream_remote_address: '%DOWNSTREAM_REMOTE_ADDRESS%'             duration: '%DURATION%'             host: '%UPSTREAM_HOST%'             message: '%LOCAL_REPLY_BODY%'             method: '%REQ(:METHOD)%'             path: '%REQ(X-ENVOY-ORIGINAL-PATH?:PATH)%'             protocol: '%PROTOCOL%'             request_id: '%REQ(X-REQUEST-ID)%'             requested_server_name: '%REQUESTED_SERVER_NAME%'             response_code: '%RESPONSE_CODE%'             response_flags: '%RESPONSE_FLAGS%'             upstream_cluster: '%UPSTREAM_CLUSTER%'             upstream_host: '%UPSTREAM_HOST%'             upstream_local_address: '%UPSTREAM_LOCAL_ADDRESS%'             user_agent: '%REQ(USER-AGENT)%'             x_forwarded_for: '%REQ(X-FORWARDED-FOR)%'           type: JSON         sinks:         - file:             path: /dev/stdout           type: File
Also i see when merge gateway works correct label gateway.envoyproxy.io/owning-gatewayclass in deployment
When merge gateway  does not work just
        gateway.envoyproxy.io/owning-gateway-name: spark-gateway-sparkcluster-e0tfrazvr7sang80t3
        gateway.envoyproxy.io/owning-gateway-namespace: sparkcluster-e0tfrazvr7sang80t3

it looks like merge gateway didnt kick in, any negative status in GatewayClass or Gateway ?

no
GatewayClass
  conditions:
  - lastTransitionTime: "2024-07-01T12:34:40Z"
    message: Valid GatewayClass
    observedGeneration: 2
    reason: Accepted
    status: "True"
    type: Accepted

Gateway
status:
  conditions:
  - lastTransitionTime: "2024-07-01T12:38:04Z"
    message: The Gateway has been scheduled by Envoy Gateway
    observedGeneration: 1
    reason: Accepted
    status: "True"
    type: Accepted
  - lastTransitionTime: "2024-07-01T12:38:04Z"
    message: No addresses have been assigned to the Gateway
    observedGeneration: 1
    reason: AddressNotAssigned
    status: "False"
    type: Programmed
  listeners:
  - attachedRoutes: 2
    conditions:
    - lastTransitionTime: "2024-07-01T12:38:04Z"
      message: Sending translated listener configuration to the data plane
      observedGeneration: 1
      reason: Programmed
      status: "True"
      type: Programmed
    - lastTransitionTime: "2024-07-01T12:38:04Z"
      message: Listener has been successfully translated
      observedGeneration: 1
      reason: Accepted
      status: "True"
      type: Accepted
    - lastTransitionTime: "2024-07-01T12:38:04Z"
      message: Listener references have been resolved
      observedGeneration: 1
      reason: ResolvedRefs
      status: "True"
      type: ResolvedRefs
    name: http
    supportedKinds:
    - group: gateway.networking.k8s.io
      kind: HTTPRoute
    - group: gateway.networking.k8s.io
      kind: GRPCRoute
  - attachedRoutes: 0
    conditions:
    - lastTransitionTime: "2024-07-01T12:38:04Z"
      message: Sending translated listener configuration to the data plane
      observedGeneration: 1
      reason: Programmed
      status: "True"
      type: Programmed
    - lastTransitionTime: "2024-07-01T12:38:04Z"
      message: Listener has been successfully translated
      observedGeneration: 1
      reason: Accepted
      status: "True"
      type: Accepted
    - lastTransitionTime: "2024-07-01T12:38:04Z"
      message: Listener references have been resolved
      observedGeneration: 1
      reason: ResolvedRefs
      status: "True"
      type: ResolvedRefs
    name: https
    supportedKinds:
    - group: gateway.networking.k8s.io
      kind: TLSRoute

but EXTERNAL-IP assigned to kind: service

intersting, when i have deleted all my gateway and deploy it again - all of them work ok. Where envoy gateway store metadata about do i need merged gateway or not? i see this one but it for pathing labels of deployemtn and services

Hi @Uburro, curious about your upgrade steps ?
Did you first remove existing EnvoyGateway ? and then reinstall it ? If is, then is related to https://github.com/envoyproxy/gateway/issues/2637. The lots of deployment you saw, are probably got created right after the uninstall steps.

Hi shawnh2. I deployed through fluxcd in helm release  controller. It similar helm upgrade -i ....
Do i need delete envoy gateway after that install int when i want to deploy new version of operator?


Pressing select all button does not do anything

Using the select all button does not select all, making it hard to select each and individual song.

The select all button is working for me. I have reproduced it as working by selecting my playlist, selecting my wanted track, pressing the Select All icon and downloading.

I tried to do what you said but pressing select all does not do anything. I'm on Marshmallow with a Nexus 6P if that helps.

It looks like this is an issue with the two-pane mode. As a temporary fix, try cranking your DPI down so the two-pane view doesn't activate.

Just fixed this. https://github.com/Arcus92/PlayMusicExporter/pull/9 The latest APK is available here until the PR gets approved. https://github.com/ddonuts4/PlayMusicExporter/releases/tag/0.0.1


Fixed bug where tips "Learn More" didn't update upon changing languagae.

Fixes #686

Nice! Should be possible to make a lint rule that checks for this, but it would require some heuristics to figure out what is allowed (constant strings, arrow functions, regular functions ig) and what is disallowed (function calls, most things).


Views are always scrolling over the end when applying min-h-screen-safe

The min-height: -webkit-fill-available; part of min-h-screen-safe makes the page be scrollable over the end, even if the content is occupying less than half of it.
I'm using a calc instead of the utility class to overcome this issue, but would it make sense to have a class that doesn't imply adding that part?

Thanks for the report. Where are you experiencing this? Desktop vs mobile & which browser? I don't mind removing WebKit fill or making it its own helper—I've only needed the WebKit fill for PWAs on mobile but haven't experienced any overscroll outside of that


adds migrate command and ansible base image scaffolds

The operator-sdk migrate command adds go source and other files necessary to
convert an ansible operator to a hybrid operator.
The scaffolds used for migrate are also made available for use in creating
the ansible-operator base image through
commands/ansible-operator-base/main.go.

I'll add docs for this, but want to vet the general approach first, and also ensure we are happy enough with the contents of main.go that we're ready to start recommending that people use this.


K78 patch 1

A slightly dull strategy but hopefully have indented correctly and have tried writing tests.

That's a pretty neat one! I refactors slightly, merged with upstream etc... :) Thanks for the contribution! :+1:


IGNITE-15363 Update Ignite Spring Data Dependency Snippet

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

All spring data extension dependencies are marked as provided based on discussion - [1]. Current documentation shows all dependencies required by spring data extension. Why should we remove them?
Also the current documentation already shows the correct versions available in the table. It helps to match version of extension with supported versions of Ignite.
[1] - https://lists.apache.org/thread.html/r0c97640b9c5f11a02fb6d5ea39ffb26edc0fd8215a479fa46e5d54fa%40<dev.ignite.apache.org>

@ololo3000 Can you please confirm then that the docs are in up-to-date state and this PR can be closed?
@Nikita-tech-writer From my point of view - yes, they are. At least i see no reasons for the current changes.

Close the PR due to the extensions section was added to the site.


Opening an editor on windows raises a PermissionError

I'd say that the problem is that editor is owning the temp file and, at the same time, tries to open another process owning it. See:

Oops sorry, that was using inquirer.
This one is using editor directly:

Well, this is a clear bug report, but hmm, I don't quite understand what's going on here.
Please bear with me - my knowledge of how Windows Python I/O differs from other systems is small, but I need to know this, as I have other utilities I know don't work.
Can you send me just the text of the tracebacks, please?
I'm determined to solve this one, if I can!

No need for the stack trace, I am fairly sure I understand what's going on.
I believe that in Windows, you can't open a file you already have open.  I think this is even in the Python documentation somewhere.
I can sort this out pretty easily.  I'll come up with a version to test!

If it serves you, I also don't use windows personally, but you can get a dev vm for free from windows website.


WW-5219 Moves TestNG related classes into TestNG plugin

Also moves test classes under testng package
Fixes WW-5219

Coverage increased (+0.002%) to 50.709% when pulling 5a3d2c4eb9c8cf9ed74c53ab2c79c05868dc70c6 on WW-5219-move-testng into<PHONE_NUMBER>2325a84bc11e48fdadc4fb7987db5f on master.


Update recon to a more recent release.

Recon was using a very old version, this updates it to the latest stable release.

let me close this and re-submit, some experimenting snuck in here.


one more level of try

these inner calls aren't wrapped in try so they could fail

@keybase/react-hackers


impdp

Dears
I need to import a dump file to the DB,
I  changed to
/u01/app/oracle/product/11.2.0/xe/bin
when I try to execute ./impdp
I get this error
UDI-00013: Message 13 not found; No message file for product=RDBMS, facility=UDI
UDI-00019: You may need to set ORACLE_HOME to your Oracle software directory

Thank you for your interest,
wnameless/oracle-xe-11g
but I tried a different way, by using the command:-

When you want to work with exp and imp and not expdp and impdp, go into your container with
docker exec -it oracledb /bin/bash
from
https://github.com/kekru/docker-oracle-xe-11g-extra-tablespace
and it worked successfully,


Notify service missing in HA

So I did install everything according to the instructions and somehow got it working, but I cannot get the "notify" service. It seems that either the service is not created or I do something wrong.
I have installed HASS.Agent locally on a Win11 Client

and also in HA:

Notification test on the client is successful and also this test

The only thing I see and assume is that I do not have some kind of notify entry in MQTT and don’t know if this is supposed to be there or how in the world this notify service for the client is created within HA

According to the documentation I assumed that it should be created automatically when I add the device via autodetect. The device pops up and I can add it but no service is created.
I found the old hass.agent notification integration but apparently it is deprecated and I should use this one instead.
Can anyone help?

Are you running the latest HAOS core update (2024.5.0)?
I had notifications previously working and once I updated to that version the notification entity wasn't being created.

Same issue here, 2024.5 breaks HASS.Agent.

Ah, there's a new version here:
https://github.com/hass-agent/HASS.Agent/releases/tag/2.1.0-beta1

Ah, there's a new version here: https://github.com/hass-agent/HASS.Agent/releases/tag/2.1.0-beta1

That version also doesn't work fyi but is actively being maintained. I've raised this issue over there as well

Are you running the latest HAOS core update (2024.5.0)? I had notifications previously working and once I updated to that version the notification entity wasn't being created.

Yes, I am running 2024.5.0.
Unlucky coincidence running Hass.Agent for the first time while being on a beta.
And there I was so happy having finally found the solution I was looking for. 😄
So, can we expect someone to fix this? Community here doesn‘t seem very active.

Are you running the latest HAOS core update (2024.5.0)? I had notifications previously working and once I updated to that version the notification entity wasn't being created.

Yes, I am running 2024.5.0. Unlucky coincidence running Hass.Agent for the first time while being on a beta. And there I was so happy having finally found the solution I was looking for. 😄
So, can we expect someone to fix this? Community here doesn‘t seem very active.

It's active, you need to go here: https://github.com/hass-agent/HASS.Agent/issues/75

Oh wow, I didn't know it had been forked. Thanks guys!

Anyone were able to solve this? I have MQTT enabled, device discovered but no notify service it registered

Same problem here...also with new beta4 version

Read the thread again

Try latest HA Agent for Windows (2.1.0)
and
HASS Agent Integration 2.1.0-beta1
It is now working for me with those versions.

My notify also did not exist on my older setup.
After updating my HASS.Agent windows and reinstalling the latest integration and restarting HA I still did not see notify.
Renaming my device in the windows HASS.Agent (Configuration : General : Device Name) and adding it to my HASS.Agent integration via my IP address worked.

How hard is it to just read the thread? There are literally 5 links to the alive project in this thread


FIX: Narrow scope of style rule for GitHub & GitLab link shortening

Fixes #1156

I though when a rc was released we should stop merging to main until the final release. If you think otherwise, I'll merge the next one as usual.

IMO we should definitely keep merging things that are fixing potential bugs or regressions. I feel like merging new features is more on the fence and depending on whether it's complex enough to warrant a second RC.
Given that we don't release often though, I'm generally a fan of just merging things and getting the code out there. Or even better we could shoot for a more regular release cadence like once a month.


Create target mask as bool

Avoids "UserWarning: Support for mismatched key_padding_mask and attn_mask is deprecated. Use same type for both instead." from Casanovo#174.
@wfondrie I don't understand the motivation of creating the mask with 0 and -inf previously, so please carefully double-check the change. I also can't find a version of generate_tgt_mask in the PyTorch repository.

There used to be an example the created a very similar function in the PyTorch docs. I think this should be good 👍


[SafeSnap] better expose execution status

Expected behavior
When there are transactions attached to a proposal, users should easily be able to see the execution status of the proposed transactions.
Actual behavior
Right now there are several issues:

You can't see the execution status unless you are logged in with your wallet and on the correct network.

The execution status is hidden at the bottom of the page, which means lots of scrolling for proposals with long descriptions and lots of included transactions.
The execution status is not communicated on the proposals view, meaning you have to navigate into each proposal to see its execution status.

Closing this issue, it was transferred on snapshot-labs/snapshot

@mktcode what's the status on this issue?

You can't see the execution status unless you are logged in with your wallet and on the correct network.

This is no longer the case in the new implementation.

The execution status is hidden at the bottom of the page, which means lots of scrolling for proposals with long descriptions and lots of included transactions.

Can easily be moved in current implementation even.

The execution status is not communicated on the proposals view, meaning you have to navigate into each proposal to see its execution status.

Not sure I easy that would be currently. In the new version should be pretty straight forward.


Document InputDialog

Environment

Documentation version: 7.0
Part of the documentation: Manual

Description of the bug or enhancement
See cuba-platform/cuba#1831

Plz rename INPUT_DIALOG_APPLY_ACTION and INPUT_DIALOG_REJECT_ACTION to INPUT_DIALOG_YES_ACTION and INPUT_DIALOG_NO_ACTION respectively.


BetterBotTags: Space before the [Webhook] and [Bot] tag, looking like [ Webhook] and [ Bot] instead.

Describe the bug
The blue [Bot] and [Webhook] tags have a little blue space to the left of them. A sort of spacebar or spacing.
This is likely caused by the disabled icon, because when the icon is enabled, this isn't an 'issue' anymore.
To Reproduce
Steps to reproduce the behavior:

Enable mod
Look at a bot or webhook
Observe

Expected behavior
Expected there to not be a spacebar :P
Screenshots
Mod enabled, icons enabled

Mod enabled, icons disabled

Mod disabled

Details (please complete the following information):

Addon Name: BetterBotTags
Addon Version: v2.0.0
Discord Release Channel: Stable

Than you for reporting it. Fixed in version 2.0.1 along side some other minor things, update when it asks you to.


Will you consider supporting Hangar in addition to Spiget?

Enhancement Type
Not sure
Describe the enhancement
Will you consider eventually supporting resource fetching from Hangar ( https://hangar.papermc.io/ ) in addition to Spiget?

That looks cool. IMHO that seems even better than Spiget since this seems to be under the PaperMC umbrella.

Making a note of API docs
https://hangar.papermc.io/api-docs

+1


Missing peer dependency in @nova-odm/annotations

Hi,
while attempting to include these libaries in a project with yarn2, the following error occurred:
➤ YN0000: ┌ Resolution step
➤ YN0002: │ @nova-odm/annotations@npm:0.8.7::__archiveUrl=[...]2Fnpm%2Fnpm%2F%40nova- \
odm%2Fannotations%2F-%2Fannotations-0.8.7.tgz doesn't provide @aws-sdk/client-dynamodb (p1924d), \
 requested by @nova-odm/auto-marshaller
[...]

Comparing the annotations package with the other ones, it seems the peer dependency to @aws-sdk/client-dynamodb is missing.
Best regards,
Alex

Hi,
It's easy to add it but the annotations package doesn't use anything from the @aws-sdk/client-dynamodb so that's why it doesn't have it as a peer dependency. Let me know if I need to fix this or if you managed to somehow solve this. Thanks!


Link to gcc_s and gcc if compiler is GCC version 5. Fixes issue #9593

Link to gcc_s and gcc if compiler is GCC version 5 to fix issue #9593

Can one of the admins verify this patch?

@tensorflow-jenkins test this please


Unexpected useFocusRing behavior

🐛 Bug Report
useFocusRing seems to say a focus ring is required even if it isn’t in some cases.
🤔 Expected Behavior
I expect

when hovering over the listbox initially, that useFocusRing returns false
If I start using the keyboard, that useFocusRing returns true,
If I go back to using the mouse, that useFocusRing return false

😯 Current Behavior
useFocusRing returns true in all 3 instances.
https://codesandbox.io/s/youthful-sanderson-f5wjkh?file=/src/Listbox.tsx
Notice that once the sandbox loads, and you initially hover over the listbox, the selection option has a focus ring. If you click outside the listbox, and hover back, the options no longer have the focus ring. Once you start using the keyboard, the focus ring appears again. However, if you go back to using the mouse, the ring remains visible (but it shouldn’t, because I’m not longer using the keyboard to navigate).
💁 Possible Solution
🔦 Context
I’m just trying to emulate Linear’s behavior, which is under the expected behavior
💻 Code Sample
https://codesandbox.io/s/youthful-sanderson-f5wjkh?file=/src/Listbox.tsx
🌍 Your Environment
Codesandbox, safari and chrome.
| Software         | Version(s) |
| ---------------- | ————— |
| react-spectrum   |
| Browser          |
| Operating System |
🧢 Your Company/Team
🕷 Tracking Issue (optional)

Hey, could you verify this against https://react-spectrum.adobe.com/react-aria/useListBox.html#example ? I think it's working how you expect.

Sorry I should have specified; as in the codesandbox, I set shouldFocusOnHover on useListBox

Ah, then this is behaving as expected. When you first load the page, we don't know what interaction mode you're in, so we assume keyboard. As you hover each item, it gets focused. Because we think you're in keyboard mode, it returns true. Then you use the keyboard, so it stays in keyboard mode, but now we have an interaction mode we know about. Finally, you start moving the mouse again. This doesn't count as an interaction mode change. Only clicking or taking an action with the mouse will change the interaction mode.
You can see what I mean by loading your example, click somewhere on the page, then hover the items. It shouldn't return true anymore.
We did want to move a use to mouse interaction just because they move the mouse since it's easy to bump a mouse and the focus ring shouldn't disappear because of that.

Ah, I see, that makes sense! Do you know of any way to achieve what I’m trying to do? We don’t have to change the interaction mode immediately on move; only once the mouse enters the boundary of a different item, you can switch the interaction mode to mouse, and have the focus ring disappear. Can I manually change the interaction mode on focus of a different section?

You'd be looking for https://github.com/adobe/react-spectrum/blob/main/packages/%40react-aria/interactions/src/index.ts#L19
It's definitely a use at your own risk situation though. We don't document it and it could change. It also causes a re-render in almost every one of our hooks (something we're working on). I don't think it hurts to be a little overly obvious with focus indicators. Most people using a mouse will click on something pretty quickly.


Do not preprocess automatically besides Option [1]

I noticed that for files containing tens of thousands of .cif files, the automatic preprocessing by CifEnsemble slows down the filtering process. Currently, cifkit in 1.0.2 will provide an option not to preprocess all the files automatically here. https://github.com/bobleesj/cifkit/issues/15

This option is available in cifkit 1.0.2. The requirement will be bumped once 1.0.2 is released.

This is implemented in the latest version.


Allow Recap partner update events to delete

I tried deleting a partner update job from the UI but encountered an error: https://app.honeybadger.io/projects/54497/faults/39527739#notice-summary

https://github.com/pulibrary/princeton_ansible/pull/344


Nameof pointer + tests

The versions (gcc, clang, and msvc) are not the final ones. It needs some documentation and various platform and version checks.

Thanks


BZ-1172069 add test-jar dependency for jbpm-console-ng-process-runtim…

…e-forms-client.
@cristianonicolai Please review it.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

+1

master: 647bfbb1b8eff267af1ff0619c76ab5c5b1b1747


Error 5005 on play H265 stream

Have you read the FAQ and checked for duplicate open issues? YES
What version of Shaka Player are you using? 2.5.10 & 2.5.20
Can you reproduce the issue with our latest release version? YES
Can you reproduce the issue with the latest code from master? YES
Are you using the demo app or your own custom app? YES
If custom app, can you reproduce the issue using our demo app? YES
What browser and OS are you using? Tizen/chrome
For embedded devices (smart TVs, etc.), what model and firmware version are you using? 4K modelés 2020
What are the manifest and license server URIs?
<?xml version="1.0" encoding="UTF-8"?><MPD xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="urn:mpeg:dash:schema:mpd:2011" xsi:schemaLocation="urn:mpeg:dash:schema:mpd:2011 DASH-MPD.xsd" profiles="urn:mpeg:dash:profile:isoff-live:2011" id="dash4k/4k_bbright_v11" type="dynamic" minBufferTime="PT5.000S" maxSegmentDuration="PT2.112S" availabilityStartTime="1970-01-01T00:00:00.000Z" publishTime="2021-02-19T17:50:45.473Z" minimumUpdatePeriod="PT1.920S" timeShiftBufferDepth="PT182.540S"><Period id="6079963961" start="PT1613727455.026S"><AssetIdentifier schemeIdUri="urn:eidr" value="md:cid:EIDR:10.5240%2f075a-8ef4-c981-07f1-C"/><AdaptationSet id="1" contentType="video" mimeType="video/mp4" segmentAlignment="true" startWithSAP="1" maxWidth="3840" maxHeight="2160" maxFrameRate="50" par="16:9"><SegmentTemplate timescale="90000" initialization="$RepresentationID$-Header.m4s" media="$RepresentationID$-0-T-$Time$.m4s" presentationTimeOffset="6079963961"><SegmentTimeline><S t="8726673925" d="172800" r="5"/><S d="183600"/><S d="172800" r="20"/><S d="174612"/><S d="172800" r="65"/></SegmentTimeline></SegmentTemplate><Representation id="v0_121" bandwidth="20000000" codecs="hvc1.2.4.L183.90" width="3840" height="2160" frameRate="50" sar="1:1" scanType="progressive"/></AdaptationSet><AdaptationSet id="2" contentType="audio" mimeType="audio/mp4" segmentAlignment="true" startWithSAP="1" lang="und"><Role schemeIdUri="urn:mpeg:dash:role:2011" value="main"/><SegmentTemplate timescale="90000" initialization="$RepresentationID$-Header.m4s" media="$RepresentationID$-0-T-$Time$.m4s" presentationTimeOffset="6079963961"><SegmentTimeline><S t="8726675196" d="172800" r="5"/><S d="184320"/><S d="172800" r="20"/><S d="174971"/><S d="172800" r="65"/></SegmentTimeline></SegmentTemplate><Representation id="v0_221" bandwidth="128000" codecs="mp4a.40.5" audioSamplingRate="48000"><AudioChannelConfiguration schemeIdUri="urn:mpeg:dash:23003:3:audio_channel_configuration:2011" value="2"/></Representation></AdaptationSet></Period><UTCTiming schemeIdUri="urn:mpeg:dash:utc:direct:2014" value="2021-02-19T17:50:45.473Z"></UTCTiming></MPD>

URL could be sent if needed

What did you do?

Play
What did you expect to happen?
Strat Playing
What actually happened?
Error PLA 5005.

URL could be sent if needed

Hi Reda,
You may have missed this note in the issue template:

NOTE:
You can send the URIs to<EMAIL_ADDRESS>instead,
but please use GitHub and the template for the rest.
A copy of the manifest text or an attached manifest will not be
enough to reproduce your issue, and we will ask you to send a
URI instead.  You can copy the URI of the demo app to send us the
exact asset, licence server, and settings you have selected there.

Please send us a URI that we can use to reproduce the issue.  Without this, we can't debug anything.  Thanks!

Hi Joey,
U r right, I sent the URI to the right adresse ,
best

5005 is INVALID_STREAMS_CHOSEN, an error code removed in v3.0; so this error can't happen in the latest release or in the latest code on master.
If I play on the latest release, I get CONTENT_UNSUPPORTED_BY_BROWSER.  Your content has HEVC (H.265) content, which isn't supported on many browsers.

Hi Jacob,
Thank you for your answer, could we get more précise idea about what is in the HEVC video track that is not uspoorted by the browser?  is it a codec options that we can adjust by changing encoding or packaging options?
Best.

Most browser don't support HEVC at all.  You can see on caniuse that almost none of the major browsers support this.  You would need to use AV1, VP9, or H.264.

This strang since I can play on the same device/browser (Tizen TV)  another stream with two tracks both are HEVC.
it is  just when I generat a stream with one track only, this error is generated and I wonder what is wrong with elemeinating on track any play a stream with on track. does that make sens?

I was testing on Chrome, it is possible Tizen does support HEVC.  I don't really have access to a Tizen to test with, can you try with the latest release (v3.0.8) to see if it works or if you get a different error?
https://shaka-player-demo.appspot.com/demo/
What may be happening is one of the streams are supported and the other is not.  So with two streams, we can play the supported one but if there is only one stream, that may not be supported.  For example, if one is a high profile and the other is low, then only one may be supported.

the strang thing is the two tracks one is low resolution and the other is 4K and Shaka start good with the low resolution and then switch to the 4K without any error. I I play the stream with one track 4K it gives me this error.
I can send you theanother link with the two tracks if that helps to detect the issue?

Sure, that would help.  Is the single-track manifest using the same content with just the other track removed?  Also, have you tried with v3.0.8?  What error, if any, do you see with that version?

yes the content is the same in two streams.
and no I couldn't intégrante and teste with the 3.0.8 version. (only 2.5.10 and 2.5.20)

Hi,
Have you get the send URI and tested it?
any idea about the root cause of the error with the first URI?

I got them, but I don't have any insight on why it wouldn't work.  Both manifests use the same codec string, so the support should be the same.  I can't test myself because it's not https and there are CORS errors.  If you could post these using https and with valid CORS headers to allow playback everywhere, then I could test myself.

I've tested again the first URI manifest for the mono-track this time without any real flux ( server don't stock any segements) behind and  still have the PLA5005 error.
This means that the PLA5005 is generated once get the manifest and parsing it, and when I parse the manifest file, it seems correct. do you get the same issue once parsing the manifest on the shaka player?

it seems my issue is realated to old one on Google groups:
https://groups.google.com/g/shaka-player-users/c/TzuJGrtqIbY
but I cann't find the bug track where it is resolved, can you help?

someting like this issue
https://groups.google.com/g/shaka-player-users/c/TzuJGrtqIbY and its fixe in dash_Parser.js
https://github.com/google/shaka-player/commit/7dc98a7d6b81c1bd245aa10f7b5c963ae5675382
?

Is the order of présentation in the manifest file (Audio id=1 then video id=2 ) can invoke this error?

Hi,
We are working on a complète log for the player with the stream.
Meanwhile I'll send you the new URI to teste from you side as well.
Best.

Hi,
have you get a look on the new URI (HTTPS)?
Best,

Thankx Jacob for your suggestion, it works fine.
De : Jacob Trimble @.>
Envoyé : mardi 9 mars 2021 19:39
À : google/shaka-player @.>
Cc : HOSNY, Reda @.>; Mention @.>
Objet : Re: [google/shaka-player] Error 5005 on play H265 stream (#3172)
First, the logs you sent contain multiple playbacks that have successful playback. This is confusing since it's hard to tell where playback sessions begin and end; there's also the possibility of multiple Player instances or having shutdown of another Player affecting a new Player. In the future, it would be easier if you only sent logs for a single playback session. There are also a lot of extra logs from your app that are a bit confusing, but I think I saw the problem.
There is a log line like Playershaka setConfigBeforeLoading maxHeightRestriction:false maxBandWidthRestriction:75000000. In the manifest, the only video stream has a larger bandwidth; so if you are setting a bandwidth restriction, we'll filter out that stream and have nothing to play. The two-stream manifest works since we can still play the low-resolution video.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/google/shaka-player/issues/3172#issuecomment-794286286, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AADY36D5XDJNF3SYSYLL7ADTCZMNDANCNFSM4X42W4TQ.

L'intégrité de ce message n'étant pas assurée sur internet, la société expéditrice ne peut être tenue responsable de son contenu ni de ses pièces jointes. Toute utilisation ou diffusion non autorisée est interdite. Si vous n'êtes pas destinataire de ce message, merci de le détruire et d'avertir l'expéditeur.
The integrity of this message cannot be guaranteed on the Internet. The company that sent this message cannot therefore be held liable for its content nor attachments. Any unauthorized use or dissemination is prohibited. If you are not the intended recipient of this message, then please delete it and notify the sender.

Glad it's working, is there anything else you had problems with?

for the moment no, Thankx for your support :-)


Add relay

@kroese NostrChat UI allows you to add new relays from settings page. You can add relays you like to use there. Closing this one.


FIX: wp.editor is undefined in latest version of gutenberg.

wp.editor is undefined when using the latest version of Gutenberg. This fix makes sure the wp.editor is loaded before the cgb. plugin.

Successfully published<EMAIL_ADDRESS>@lassemt thanks for the help.

No worries 🎉


Potential secutiry vulnerabilities in the shared library which realm-kotlin depends on.

Hi, @cmelchior , @rorbech , I'd like to report a vulnerability issue in io.realm.kotlin:cinterop-android:0.10.2.
Issue Description
io.realm.kotlin:cinterop-android:0.10.2 depends on 4 C libraries (.so) cross many platforms(such as x86-64, x86, arm64, armhf). However, I noticed that one C libraries is vulnerable, containing the following CVEs:
librealmc.so from C project openssl(version:1.1.1g) exposed 2 vulnerabilities:
CVE-2021-3711, CVE-2021-3712
Suggested Vulnerability Patch Versions
openssl has fixed the vulnerabilities in versions >=1.1.1l
Java build tools cannot report vulnerable C libraries, which may induce potential security issues to many downstream Java projects.
Could you please upgrade the above shared libraries to their patch versions?
Thanks for your help~
Best regards,
Helen Parr

Hi @HelenParr
Thank you for reporting this. It is correct that the current release uses OpenSSL 1.1.1g. It comes through a dependency from https://github.com/realm/realm-core.
Realm Core has already upgraded to OpenSSL 1.1.1n, so we should be able to easily fix this by updating to their latest version.


ACCUMULO-3971 Better doc for tracing

This adds an example of how to configure another SpanReceiver for tracing by the Accumulo server and by the client. Also documents the format of the trace table and some utilities.

This is all really great! +1 from me, would be good to hear from @billierinaldi  and/or @ericnewton if they have some time.
The unrelated formatting change will be squashed by me -- I noticed I goofed that up over the weekend.

This looks great.  Thanks, Dylan!  I have two suggestions.  One is about the sentence, "Span entries record full span information, whereas index and start time entries provide indexing into span information useful for quickly finding spans by type or start time."  Actually both the span and start time entries have the full span information, while only the index entries provide indexing.
The other suggestion is to mention other configuration properties that would need to be set for the zipkin span receiver in accumulo-site.xml and in the client configuration, such as:
<property>
  <name>trace.span.receiver.zipkin.collector-hostname</name>
  <value>localhost</value>
</property>
<property>
  <name>trace.span.receiver.zipkin.collector-port</name>
  <value>9410</value>
</property>

Thanks for spotting that sentence Billie, will fix tonight.  It's good to
throw in the rest of the Zipkin configuration as well (I was using the
default localhost and port).
On Mon, Aug 24, 2015 at 2:22 PM, billierinaldi<EMAIL_ADDRESS>wrote:

This looks great. Thanks, Dylan! I have two suggestions. One is about the
sentence, "Span entries record full span information, whereas index and
start time entries provide indexing into span information useful for
quickly finding spans by type or start time." Actually both the span and
start time entries have the full span information, while only the index
entries provide indexing.
The other suggestion is to mention other configuration properties that
would need to be set for the zipkin span receiver in accumulo-site.xml and
in the client configuration, such as:

trace.span.receiver.zipkin.collector-hostname
localhost

trace.span.receiver.zipkin.collector-port
9410

—
Reply to this email directly or view it on GitHub
https://github.com/apache/accumulo/pull/44#issuecomment-134325576.

Okay, this is set.  On a slightly related note, I see that Zipkin supports HBase, Cassandra and Anorm connections as backing stores for traces.  I wonder how much work it would take to get Zipkin's interface and/or trace-aggregating-statistics to use Accumulo as a backing store.

+1

I wonder how much work it would take to get Zipkin's interface and/or trace-aggregating-statistics to use Accumulo as a backing store.

Not an excessive amount of work: https://github.com/jaredwinick/accumulo-trace-zipkin


Fixit for adding public to a class removes final

Steps to reproduce

Create new class and declare it final
Mark it as instantiate with SafeDI
Error appears: Instantiable-decorated type must be public or open
Apply fixit, class declaration is now public class XYZ instead of public final class XYZ

Expected behavior
Fixit applies public class
Actual behavior
Fixit applies public final class

This is a bug in InstantiableVisitor.processModifiers. We should be adding to modifiers rather than setting them directly.


Bumped version to 2.2.1

Changes
Bumped package version to 2.2.1 for the release.

@chrjorgensen I'll make the release when you'll approve this 😄


Contact email has no speaker name or id

There's a bug in the contact form where the speaker name or id is not being linked to the message. Instead, the message shows Speaker: undefined.

I see another occurrence of this from Sept. 1, inside the Netlify data. Looks like that person was just using our tool to say hello, but it would be good to fix this asap so we're ready for making connections.


Add files via upload

Data storage and management: configured our database settings in the settings.py file, located in our project's main directory. We've opted for PostgreSQL as our database engine. For Database Setup, I created  a login group named "nolepatrol" with the password "SearchEngineTest" and granted the group the privilege to log in. Following this, I created a database named "test1" and assigned "nolepatrol" as the owner. This setup ensures proper database access control.
Django Admin Setup: I implemented code in the admin.py file, enabling us to manipulate and manage data stored in the LeakedCredential model through the Django admin site.
Model Definition: Our data structure is defined within the models.py file. I created a LeakedCredential model with specific fields, emphasizing data type choices and any special considerations.

Need to open an updated pr.


Write data from a long time ago into TSM

Hello, I hava some questions about write and query TSM.
To efficiently compact and store data, the storage engine groups field values by series key, and then orders those field values by time
So the data in  TSM files ordered by time, my confusion is  what happens if a data with a timestamp from a long time ago
write to influxdb, it will cause the TSM file reorder to put this data to current position ?  or just append to the TSM file, and if so, how to find this data  efficiently,

如果是删除分片中的数据，会执行很慢，应该是在做重排序吧，你们的环境是啥样的啊？有空交流下，我碰见了内存占用的问题

Data with past dates  will be indexed when written, and compaction will eventually merge TSM files.


Amount of uniquely named storage accounts.

You can have 200 uniquely named storage accounts according to https://docs.microsoft.com/en-us/azure/storage/storage-scalability-targets (250 for specific case when requesting through support). Tested on personal subscription.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@domino46 : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@robinsh Please review these proposed changes. Thanks.

#sign-off Please go and merge this, it's correct.
@domino46 Sorry for the delay, approving this now. Thanks for contributing to the accuracy of our documentation, I really appreciate it.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@domino46 I approved this, and that puts it in a queue to be merged into the master branch. When they merge it, they will close your PR, and it will be released in the next build. (I think the builds are at 10am and 3pm).
#sign-off again in case we need to after closing and reopening the PR.


BookStack as a VM on Hyper-V, How Much Storage?

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
This is really a question rather than an issue. I watched @ssddanbrown tutorials on installing BookStack in Ubuntu 20.04 on a Linode server, and he chooses the lowest plan which is about 50GB of storage. I have created mine on a Hyper-V Server with 80GB of disk space? This is going to be used as a digital notebook for every project we do. We are a paint manufacturer and craft tons of specials for customers that require pages of documentation (including photos). We plan on using this year after year from here on out so that we can maintain historical data. Is 80GB enough? What does everyone recommend for something that could potentially grow pretty large?
Thanks in advance! BookStack is, without a doubt, the absolute best locally hosted solution out there for this. And Dan offers it for free! I am such a fan boy now!
Exact BookStack Version
BookStack v23.05.2
Log Content
No response
PHP Version
8.2
Hosting Environment
Ubuntu 20.04
Dell PowerEdge Server using Hyper-V
Windows Server 2019

Hi @jimmie-jam, thanks for the kind comments!

Is 80GB enough? What does everyone recommend for something that could potentially grow pretty large?

It massively depends on usage patterns, active users, and the files & images you upload.
80GB seems like a reasonable starting point. For any of my own previous use-cases of BookStack, that would take many years to get to.
My advice would be to start with that amount, record the current free disk space, then check on it again after a few months to see how things are scaling to get an idea of how fast you'd eat storage.
Since you're virtualizing, I'd have thought scaling up shouldn't be a significant problem if needed anyway?
Will depend on your environment and hardware though.

@ssddanbrown,
You are the best!!! Thanks for the recommendations and best practices. It will be roughly 8 users, with 3-4 additional "viewers." I'm really going to limit the size of images and have some methods that the users can employ to keep photos light. But for the most part, text and table data would be the bulk of the content.
I like your suggestion to check in every couple of months to gauge usage. Superb idea!
Thanks do much again!

Happy I could help! Will therefore close this off.


Add warning regarding reusing wave ids in flight

I'm inspecting the source a little and it looks like this is the current situation:

if a wave at the end of the queue, possibly being actively transmitted, is added again with SYNC, it will be relinked to its start, and it will loop forever with no error returned.  This is because waveEndPtr points to its own upcoming next pointer from its last transmission. https://github.com/joan2937/pigpio/blob/accd69d2fdd3404a0c56416912d87b2619d58563/pigpio.c#L9918
if a wave already queued is added with SYNC, its next pointer will be re-mutated and everything on the queue after its previous position effectively dropped, with no error returned.  This is because its next pointer was previously holding a role in the chain of waves.  https://github.com/joan2937/pigpio/blob/accd69d2fdd3404a0c56416912d87b2619d58563/pigpio.c#L9912

Ideally it would be an error to transmit a wave with SYNC that is still queued to be sent.

Thank you.  I believe this addresses one issue exposed by in #457 but something else is going on there that I have not had the time to look into.  I'm suspicious that changes made for padded waves may have broken the normal waves.

@xloem thanks for this contribution.  I need to verify if this warns for the situation in #457.  I then need to decide if a warning is sufficient or if an error more appropriate.

This warning was intended to address this issue mentioned in #457 (and to imply your proposed solution):
  // send second pulse before first has ended; second pulse does not appear

I'm not sure if this warning is clear, but I believe it to be much better than nothing :-)
It would indeed be nice to review the wave sending for robustness.  Warnings in documentation are not the best solution in the end.


Remove failed experiment in cache updating.

This caused more perf issues hitting the site 20+ every four minutes than
it solved keeping the cache hot.

Merge Conflict

Interesting. There is no merge conflict.

Creating a new branch from master because I can't get this one to see sense.


IgnoreLogBackups does not completely ignore.

[X] Verified running latest release of dbatools?

[X] Verified errors are not related to permissions?

[X] Can duplicate in new/fresh PowerShell.exe session?

Operating System (Name|Version): Windows 2012

PowerShell Version: 3

SQL Server (Edition|Version): SQL 2014

Steps to Reproduce
Have database with log backups.
Delete most recent Differential backup so that command will fail.
Run Test-DBALastBackup with -IgnoreLogBackups and without -CopyFile.
See that while log backups were not restored they are shown in the output.

Can you include what error you are getting?


Cookie preferences - now editable

Now you can update the cookie preferences by reopening / revisiting the popup window (maby we should call it cookie preference window).
#5 this should make you happy :)

@steinhaug Many thanks for your suggestions and your pull request. I've made a few simple updates to allow the popup to be reopened.


reference --rkt flag of calicoctl node

Left some general comments about the README, plus a few comments about the changes you're added.

Yup, looks good.

Is that an official 'LGTM'?

Yes


In netrw mode, three quits are required to close vim

nvim --version: v0.8.0
Operating system/version: Linux archlinux 6.0.6-ach1-1
nvim +'checkhealth rnvimr':

rnvimr: health#rnvimr#check
========================================================================
## OS
  - OK: Name: Linux

## Ranger
  - OK: Version: ranger 1.9.3

## Python
  - OK: Version: 3.10.8 (main, Oct 13 2022, 21:13:48) [GCC 12.2.0]

## Pynvim
  - OK: Version: 0.4.3

## Ueberzug (optional)
  - OK: Ueberzug is ready

## RPC
  - OK: RPC echo: Neovim send "Give me five!" and receive "Give me five!"

Describe the bug
When ranger is used as a netrw replacement it does not immediately quit on q/:q.
The first q/:q causes the screen to flash once. The second one will close ranger and the last :q is required to close vim.
To Reproduce using nvim -u mini.lua
Example:
cat mini.lua
-- use your plugin manager, here is `packer`
require("packer").startup(function(use)

        use 'wbthomason/packer.nvim'

        use {
                "kevinhwang91/rnvimr",
                config = function()
                        vim.g.rnvimr_enable_ex = true
                end
        }

end)

Steps to reproduce the behavior:

Install rnvmr
Enable netrw replacement: vim.g.rnvimr_enable_ex = true
Open vim with vim .
Press q or :q (screen flashes once)
Press q or :q again (ranger is closed, empty vim buffer is visible now)
Exit vim :q

Expected behavior
Ranger and vim should close after pressing q or :q once (in netrw replacement mode).

Weird, only double :q for me to exit nvim.

Sorry i totally missed your answer 5 days ago. I just migrated my email, seems like i overlooked that one.
Its weird that you only have to quite twice. With the config and steps provided, on two different machines i need three. Also another friend with the same config just confirmed it on his machine as well.
Nevertheless, double :q also seems odd. Doesn't it? If you use ranger as a netrw replacement, you probably expect it to quit immediately after one :q not two. Seems a bit counterproductive to me.

Be reproduced in min config, I can't reproduce the issue because I have set up:
vim.g.loaded_netrw = 1
vim.g.loaded_netrwPlugin = 1

in my init.lua.
The code is written while I'm using vim-plug whose load order is different than packer.nvim, some of code is out of action. The issue has been fixed now, thanks.

Awesome, can confirm i only need two quits now to close vim. Thank you.
Do you know if there is a way to make vim quit just after one :q?
Its totally okay if its not the intended default behavior of rnvimr. It just doesn't feel right to me.

vim.api.nvim_create_autocmd('VimEnter', {
    pattern = '*',
    once = true,
    callback = function()
        local name = vim.api.nvim_buf_get_name(0)
        if vim.fn.isdirectory(name) == 1 then
            vim.api.nvim_create_autocmd('FileType', {
                pattern = 'rnvimr',
                once = true,
                callback = function()
                    vim.api.nvim_create_autocmd('WinLeave', {
                        buffer = 0,
                        once = true,
                        callback = function()
                            vim.schedule(function()
                                name = vim.api.nvim_buf_get_name(0)
                                local off = vim.api.nvim_buf_get_offset(0, 1)
                                if name == '' and off <= 0 then
                                    cmd('q')
                                end
                            end)
                        end
                    })
                end
            })
        end
    end
})

look redundant, but work fine.

Works like a charm. tyvm


Discuss Schema Implementation

Has the schema design been discussed? Possibility of a draft is included.
Are the initial tables created for the statistics tracking and/or player information?

The initial design of the schemas was created for statistics tracking, as well as player information.


cleaning up repo

What type of PR is this?

cleanup

What this PR does / why we need it:
clean up repo
Which issue(s) this PR fixes:
Fixes #35
Fixes #29
Fixes #28
Special notes for reviewers:
cleans up repo
Changelog input
 archives MobileConnect definition
creates CAMARA reference to Mobile Connect
leaves CAMARA only API definition

Additional documentation

@DT-DawidWroblewski happy to help, if you're ok I can upload a commit with the extra cleanup and my comments


GEODE-3283: Expose parallel import and export in gfsh

This pull request allows users to conduct parallel import and export of snapshots through gfsh. Because multiple snapshot files may be present on a given member for import, a "directory" option needed to be added to import, to import all snapshot files in that location (this is required for parallel import: the "file" parameter is ignored when "parallel" is true). For export, the same use of "directory" is needed, so that saving all snapshot files to a shared network storage device is possible. The "directory" parameter can also be used when "parallel" is false, but only files from the specified member will be imported (export will still be a single file). Note: parallel is only supported in partitioned regions, if "parallel" is set to true on other types of regions, it will be ignored.

I would like to get a better understanding of the new feature first. So before the "export data" is export data of a specific region on a specific member to a file. All three options are mandatory. Now it seems I can export data of all regions on a specific member, so I should be able to do this:
gfsh> export data --member=abc --dir=xxx
Would the existence of --dir option implicitly mean "--parallel" already? do we need --parallel at all?

@jinmeiliao Region is still required, so this would still not work (there is a test in my commit that confirms this):
gfsh> export data --member=abc --dir=xxx
Your observation about --dir being able to implicitly mean --parallel is correct. When I opened this up to the community to discuss, my read of the conversation was that since non-parallel export/import can work with a directory, it would be better to have it available to both. In addition, while we have not added the ability yet to export all regions at once, that is possible to do and could be added as a feature and would require a directory to be provided instead of a file, for both parallel and non-parallel export. How to implement the options was definitely an area of mixed opinions on multiple options, so still an area worth discussing to make sure we get to the best solution.

I have pushed new commits, which contain changes suggested by reviewers, these changes include:
-"parallel" option defaults to value of "true" is option specified with no value
-handle gracefully issue of gfsh command created on member with different version of geode (will result in failed import/export)
-refactoring to improve DRYness of gfsh commands
-Remove base class from tests (incorporate setup code in each test, individually)
-change "directory" option to "dir"

Note that travis found a code formatting problem that can be fixed by running spotlessApply.

I think a user might think that they need to use --file to give the filename and --dir to give the directory of that file. So ignoring the --file and just using --dir might be confusing. Since we want them to use one or the other, but not both, I think the command should enforce that instead of ignoring one of them.


Restyle Header Navigation and Breadcrumbs

View changes on GitPod
I've added the new header navigation in this MR - note that the CSS for this involves overriding some of Bootstrap's default nav styles, so it's a little involved in parts. I've also added the breadcrumb styles in this MR, as these are close to the header & don't have a seperate ticket.
Screenshots

Nice work James!
I wonder if the mobile menu transition could be improved. There's two areas that I feel could be improved:

The hamburger menu shifts left and right when clicked/pressed
Upon closing the page contents shift upwards. I wonder if using a menu that overlayed the contents would resolve it? Maybe one to chat through with the designer to see what the expectation is here.


API to enumerate all pin paths for a program

This can either be clubbed with ebpf_get_program_info() API which will also return all the pin paths for the specified program or it can be a new API to just query all the pin paths for a program.
The same is applicable for maps.

Related to issue https://github.com/microsoft/ebpf-for-windows/issues/364

Related to issue #364
To clarify: Relation comes from the fact bpftool is already able to do this, so porting bpftool should bring this feature to Windows?
Also mentioned during the call: even with bpftool, there would still be a need for an API version of the feature.

https://elixir.bootlin.com/linux/latest/source/tools/bpf/bpftool/common.c works as follows:
For each path under the bpf mount point:

Call bpf_obj_get() to get an fd
Call get_fd_type() to see if it's the type we're looking for (e.g., program)
Call bpf_obj_get_info_by_fd() to get the info about it

Related to issue #188


Added the numpy implementation of some small functions in the docs.

Summary
They are all simple function with a few lines of numpy.
Related Issues
PR Overview

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[ ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Maybe, we need a discussion about which functions are useful to inform their NumPy equivalences. In my humble opinion, functions that are not primitive and combinations of NumPy operations need to be documented as NumPy codes. The batch_dot and binary_crossentropy are good examples. I think that functions such as max, maximum, sum, less_equal does not need to be documented. They are too explicit to elaborate.

Maybe reading the numpy implementation is faster than reading the function description? Also, we can avoid in this way any doubt regarding the behavior.
Especially since the names of backend functions can be misleading. For example np.repeat correspond to K.repeat_elements, not K.repeat which is something different. So users might wonder if K.max does really correspond to np.max.


Separate antlr4 grammar file into lexer and parser files

https://github.com/ballerinalang/ballerina/blob/release-0.91/docs/grammar/BallerinaLexer.g4
https://github.com/ballerinalang/ballerina/blob/release-0.91/docs/grammar/BallerinaParser.g4
We've done this few releases back.


Fix tests by increasing limits to 1000

Fix the historical kline tests by setting all limits to 1000.

Thanks @0b100100


videos don't go show up in the seen videos historic

Script version
What version is the script on eg 2.9 , 3.1 ect
Describe the bug
A clear and concise description of what the bug is.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots or a video to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

The way everything is being displayed now with that other youtube window hovering youtube does not recognize us as being signed in while watching videos


Fix readme instructions for node.js LUIS sample

Fixes #639
Proposed Changes
Revamp readme to include better instructions on updating sample.

Collided with another checkin..


Cross-compilation of geometry shaders

We need to support translation of geometry shaders written in HLSL/Slang syntax over to GLSL for Vulkan.
There are a few distinct points to this:

Our current translation for varying array-of-structure inputs is apparently wrong, and leads to us trying to construct a value of a struct type from arrays for each of its fields, and then assign the result to an array of the structure type.

We need to detect a GS output stream in a varying signature, and emit it appropriately as GLSL globals

We need to detect Append calls on GS output streams, and turn them into an assignment to the appropriate global(s) followed by an EmitVertex call.

We need to translate RestartStrip calls on a stream into EndPrimitive calls.

This work has largely been done, as part of bringing up Falcor + Vulkan on the new code generator. We really need test coverage to double-check that things work, but I'm going to close the issue for now to not lead to any confusion about what is or isn't implemented.


I18n-js does not support symbol references in i18n translations

Given
en:
  common:
    test: 'Some string'
  javascript:
   something: :'common.test'

I see I18n.t('javascript.something') returning 'common.test', but the Ruby implementation will return 'Some string' because it resolves symbol references.

Hmmmmm
I am not really sure what you are talking about

The Ruby implementation of I18n will recursively resolve symbols until a string is returned. Asking for javascript.something yields :common.test, which is looked up again to yield 'Some string'.
i18n-js returns the symbol, not the string.

In JS world there is no "symbol"
Is that a documented feature for i18n (Ruby)?

Yes -- but the objective of this is to expose the translations on the server to the client right? So if the server sees the correct string, this should be resolved on the server side before the client sees the symbol.
It's called "reference keys" but googling doesn't turn this up. Have a look directly in i18n: https://github.com/svenfuchs/i18n/blob/v0.7.0/lib/i18n/backend/simple.rb#L78

If some can submit a PR for this, I can merge it.
If not, I will work on it after 3.0.0

Thanks @PikachuEXE!

I have added a recursive lookup to the I18n.js helper file here. However, it requires that the engine generates the translation js file that returns a symbol value that looks like {foo: ':bar'}  instead of the current {foo: 'bar'} .
I don't personally use i18n.js for the generation of translation js using its rake task, but am using a webpack plugin which converts the locales yml to js, and then using the I18n.js helper file for the lookup function.
I do not have spare time right now to look into the rake task and the fnando is doing a v3.0 release bump soon. Feel free to look into this issue.


Demo's domain looks dead?

Hello.
The domain http://aexmachina.info/ was closed.

Yup, the domain's been poached. Working on it now. Thanks.

Now it's malicious...


MindVision

@coldkevin1991 suggested implementing a Camera interface for MindVision hardware (see #57).
This would involve filling in the abstract camera template with code from the grab.py in the provided python_demo.zip. The mvsdk.py file would probably also be included in slmsuite as _mvsdk.py so the user does not need to copy this into the working directory.
We do not personally have MindVision hardware to test upon, so @coldkevin1991, it would be awesome if you could test (and implement the code if you feel comfortable with this).
Note that r0.0.2 makes some changes to Camera attributes, so working off of dev or after r0.0.2 is on main might be best.

MindVision.zip
Hi @ichristen, I have filled the camera template, but I am not sure all the function can work.
I also have Holoeye SLM, later I will let them work together.

Hi @coldkevin1991 ,
Sorry for my late reply.
I made some edits (mostly style) to your code. The main logic changes that I made was that the buffer will no longer be freed every .get_image() can and images are no longer saved (instead returned as a numpy array).
I have put your code on the mindvision branch with you as co-author. If you could test that branch (git checkout mindvision for a git installation) and make sure everything works on hardware, then we can clean up the code a bit more and pull things onto main. You should also feel free to make a fork of slmsuite, put your code on there, and then do a pull request from your fork.
Best,
Ian

Hi @coldkevin1991 ,
The MindVision code is staged for the next release #85 .
I don't have MindVision hardware, so the current state is untested (and noted as such in the docs).
Additionally, the mvsdk.py file appears to be originally from dddomodossola/nastroprint. Including this file into slmsuite would violate nastroprint's Apache 2.0 license, so I instead documented the process for a user to download and include this python header file.
Best,
Ian

Hi @ichristen,
The mvsdk.py file is originally from the official website:
https://www.mindvision.com.cn/category/software/demo-development-routine/


Step 6 Variables reference is missing.

In Step 3 we setup a few variables which worked fine till step 5. The variables used in steps 6 are different from what we used.  Specially $job = Invoke-SPOMigrationEncryptUploadSubmit -SourceFilesPath $sourceFiles -SourcePackagePath $spoPackagePath -Credentials $cred -TargetWebUrl $targetWebUrl
Common variable would be a nice approach.

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

ID: 3b01f633-962a-274d-1769-e008e7049f03
Version Independent ID: c63b4c45-f5c6-d0fb-e14f-095f79b1a415
Content: Upload on-premises content to SharePoint Online using PowerShell cmdlets - Migrate to SharePoint and OneDrive
Content Source: migration/upload-on-premises-content-to-sharepoint-online-using-powershell-cmdlets.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@pathade90 Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Flutter upgrade 3.13

Why? (ticket link or issue description)
We need to upgrade all the packages to the latest version
What was done and how?

Flutter is upgraded to 3.13+
All packages have been upgraded.
Deprecated properties have been removed.

Anything special? (optional)

Great job @hasibul-hasan-shuvo!


How to code sign and use multiple configurations for snapshot

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
https://github.com/KrauseFx/krausefx.com/blob/master/_posts/2015-06-18-run-xcode-7-ui-tests-from-the-command-line.md
Hi there, I the above article about uitests and snapshot and thought it was great. However, I am having some issues making it work when using multiple configurations. It asks me to select a provisioning profile for each configuration, but none of the existing provisioning profiles I am using are listed.
I was following some of the guidelines listed in the articles below to use multiple configurations for different environments (App Store, Beta, and Debug), which has worked great for me. Using fastlane is a breeze with multiple configurations as well.
https://medium.com/@danielgalasko/run-multiple-versions-of-your-app-on-the-same-device-using-xcode-configurations-1fd3a220c608
https://medium.com/@danielgalasko/change-your-api-endpoint-environment-using-xcode-configurations-in-swift-c1ad2722200e
This is my first time working with Cocoa Touch UI Testing Bundle, which I plan to use for some UI tests as well for using snapshot and frameit to automate screenshots. Since documentation for UI Testing Bundle with multiple configurations is scarce, I would appreciate if you have any resources you can point me towards to get this going.
Thanks

@fabiogomezdiaz Can you send a screenshot or put the output here for what you mean by...

It asks me to select a provisioning profile for each configuration, but none of the existing provisioning profiles I am using are listed.

I'm not exactly sure what you are seeing with this but if you can show me I may be able to help out more 😊


DA Http Endpoints

Expose add_blob and get_range from DaVerifier and DaIndexer in Nomos Http Api.

Looking good. Next step is to integrate it within some tests?

Yes, verifier and indexer services can now be used in out integration tests.


[Analytics] refactor analytics url

Summary of Changes (What does this PR do?)

refactor analytics urls

Is this change ready to hit production in its current state?

[ ] Yes
[ ] No

Status of maturity (all need to be checked before merging):

[ ] I consider this code done
[ ] I've tested this locally
[ ] This branch has been LGTM'ed by a reviewer

How should this be manually tested?

Fetch this branch locally
Create a python virtual env (if none exists) python3 -m venv ./virtual_env_name`
Activate the virtual env source ./virtual_env_name/bin/activate
Install requirements pip install -r requirements.txt
Start application flask run
Test all analytics endpoints. ensure the endpoints work and they contain the prefix /api/v1/analytics/.

What are the relevant tickets?

PLAT-498

Screenshots (optional)

Thanks @ja-odur
LGTM, I suggest that we make the necessary updates to the Analytics documentation here: https://docs.airqo.net/airqo-platform-api/analytics cc: @sserurich

Will work on this immediately after the PR merge


[Task]: deploy in k8s helm with postgresql db persistence

What needs to happen?
Hello,
There is little information on the web portal and all the documentation with the persistence process in postgresql db. What is the process and step by step to install apache hop in k8s, using helm, and configure a db postgresql for persistence. To manage the restarts of the pods in k8s without losing data, flows, configurations, and others.
Tnks!
Issue Priority
Priority: 2
Issue Component
Component: Hop Config

Hop does not have a DB persistence layer, everything is done in files.

How is the practice to persist the archives? From hop-web to hop-server?
How is the practice for restarts of the hop-server or hop-web pods, so as not to lose the information?

For hop web, you'll have to mount your project and environment configuration files to volumes or clone from a git repository when the pod starts.
Hop server doesn't persist anything either. You'll have to add or register your workflows and pipelines before you can run them there.
Let's see if we can improve that part of the documentation over the winter holidays.


packages check - not parsing parameters

Validating some of the checks and noticing a consistent fail for repoExists checks -
{
    "Name": "chronos health check",
    "Notes": "For use with Microservice Infrastructure",
    "Checklist" : [
        {
            "Name": "mesosphere repo present in yum",
            "Check": "repoExistsURI",
            "Parameters": ["yum","http://repos.mesosphere.io/el/7/noarch/"]
        },
....

based on packages.go it seems that the baseurl or name in /etc/yum.repos.d/mesosphere.repo should fit the parameters. But they return errors:
Repo with given Url not found:
        Specified: http://repos.mesosphere.io/el/7/$basearch/
        Actual: []

Using repoExists and the "name=" in mesosphere.repo
Repo with given Name not found:
        Specified: Mesosphere Packages for EL 7 - noarch
        Actual: []

Using the microservices-infrastructure vagrant box for testing.
I've also installed the RPM of distributive 0.1.

Awesome, thanks for testing! This one is particularly difficult, as I don't really have three package managers on my system. In fact, the code is only looking through /etc/yum.conf. Can you send me both your /etc/yum.conf and /etc/yum.repos.d?

TODO: Improve repo check to read from information outputted by yum & deb.

Fixed by ca099df4a606531607b7d25bc5e8d8b70bc3e0ba


Remove XML section

As management of Cortana in a MTR is no longer possible using the XML file but must be done in PowerShell remove the XML section from the document.

@cichur PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Rails 5 compatibility

Thanks to @osheroff for doing the heavy lifting in #68. But while that PR changed ActiveRecordShards to work only on Rails 5, this PR keeps compatibility with Rails 3.2 and forward.

cc/ @grosser, @osheroff and @jacobat

👍 looks not too crazy ... just sad to see more and more hacks being added :/

just sad to see more and more hacks being added

I don't like it any more than you do – but that's what comes from monkey patching a private API, I guess.
BTW I'm pretty confident that we can soon drop support for some older versions of Rails ;-)

The build for Rails 5 was red, so I chose to ignore your thumbsup :-) I’ve added 2 commits, including the one from #73.

all my 👍  are "on green" ;)
still looks ok to me ...

I saw a spec failing with this message:
NoMethodError: undefined method `mysql2_connection' for ActiveRecord::Base:Class

Re-running make the specs pass. That’s not necessarily a good sign.

this includes the dubious removal of the ActiveRecord::Base.establish_connection monkey-patch.  But after that I'm 👍

This branch was released as v3.7.0.rc1 and has been used in without problems for the last couple of weeks. Merging and releasing :sparkles:


fix test "registers a new user with password confirmation and logs in"

If the setting registration_include_name_field => true, then this test fails with:
Component has errors: "name"
Failed asserting that false is true.
at vendor/livewire/livewire/src/Features/SupportValidation/TestsValidation.php:109
at tests/Feature/AuthenticationTest.php:94

Just adding name as in the other existing registration test in this same file.

Hey @ch4r-ch4r, In order to get the tests to pass correctly, I need to test the PR against a branch on the main repo. So, I've changed the base to merge into this branch fix-test-registers-a-new-user and I've re-opened the PR here: [url] 👍


lib/src/cli: Add commit verb that validates /var

Adds initial validation of /var per: https://github.com/ostreedev/ostree-rs-ext/issues/159#issuecomment-1006033596

I enabled merge for this, but one thing that'd be nice to do after this is extend ci/integration.sh to try testing this.

I enabled merge for this, but one thing that'd be nice to do after this is extend ci/integration.sh to try testing this.

Thanks, I'll add that to my to-do list.


Switch from openssl to rust-crypto for HMAC/hash generation

Rebase of @luser 's work in https://github.com/rusoto/rusoto/pull/280 .

We'll also need to remove the OpenSSL instructions from the documentation.  There's a mention in the Requirements section of the README at least.

Per the convo in https://github.com/rusoto/rusoto/pull/280 we'll still need that for Hyper.

FWIW, ring has just been released on crates.io. I'm attempting to port rusoto over to ring, stay tuned for a PR.
https://twitter.com/BRIAN_____/status/765384547575836672

Ah, yes.  I looked at ring when #280 first came up, though I think at the time it declared itself Not Ready.  BUT HOW COULD I FORGET THAT VERY UP-FRONT DISCLAIMER :smile:

PR opened: https://github.com/rusoto/rusoto/pull/347.

Perhaps a possibility would be to add support for all three options? Either via compilation flags or by implementation of a shared trait?

Ah, yes. I looked at ring when #280 first came up, though I think at the time it declared itself Not Ready. BUT HOW COULD I FORGET THAT VERY UP-FRONT DISCLAIMER

I never have and never plan to say that ring is "Ready" or "Not Ready" like that, so please don't wait on indications of that sort.

Anyone have strong feelings on this over #347 ?

Nope. I was just trying to make building Rusoto on Windows less painful long-term (once hyper sorts out its SSL dependencies), and using ring should be just as good.

I'll leave this open a bit longer in case someone has more feedback, but I'm leaning to switching to ring in #347 .
@indiv0 being able to switch between crypto libs sounds like a good future enhancement, but I'm okay with us supporting just one for the time being.  If it's something you feel strongly about, please open a new issue.

Ok. I agree that it's less of a priority than just switching right now. I might open an issue for it once I have some free time.

Closing in favor of #347 .


Azure AD Users not appearing post Azure CP Configure

Have installed AzureCP v16 on our on-prem SharePoint 2013 Farm. However when trying to add users to a site, the on-prem AD account is showing, and the Azure AD account is not showing in the people picker with AzureCP.
In other words, I cannot select the correct Azure AD account which works without Azure CP with Name/Role Claims mapping which we have defined with this article : https://docs.microsoft.com/en-us/azure/active-directory/saas-apps/sharepoint-on-premises-tutorial#additional-resources

@sikandersa can you check the SharePoint logs and filter on Product/Area "AzureCP" to see what happens?

Directory Read all permission was already given

Can you send a screenshot of the permissions page in the Azure AD tenant?


Service account authorization without file at file system

We have this code running with success. We can read data from Google Sheets with this:
<?php

require 'vendor/autoload.php';

$service_account_file = 'service-account-credentials.json';
$spreadsheet_id = 'some-spreadshet-id';
$spreadsheet_range = 'A:B';

putenv('GOOGLE_APPLICATION_CREDENTIALS=' . $service_account_file);

$client = new Google\Client();
$client->useApplicationDefaultCredentials();
$client->addScope(Google_Service_Sheets::SPREADSHEETS_READONLY);
$service = new Google_Service_Sheets($client);

$result = $service->spreadsheets_values->get($spreadsheet_id, $spreadsheet_range);
var_dump($result->getValues());

Due to security policies, we can't have hard-coded credentials in files at the file-system. (We store and read sensitive information from Amazon's Parameter Store.)
Having the necessary information for authentication in memory (variables), how can we achieve the service account authorization?

Alright... It seems some inspection into the source code enabled us to find the answer we were looking for.
This is what we got working:
<?php

require 'vendor/autoload.php';

$service_account_file = 'service-account-credentials.json';
$spreadsheet_id = 'some-spreadshet-id';
$spreadsheet_range = 'A:B';

$credentials = json_decode(file_get_contents($service_account_file), true);

$client = new Google_Client([
    'client_id' => $credentials['client_id'],
    'client_email' => $credentials['client_email'],
    'scopes' => [GoogleSheets::SPREADSHEETS_READONLY],
    'use_application_default_credentials' => true,
    'signing_key' => $credentials['private_key'],
    'signing_algorithm' => 'HS256',
]);

$service = new Google_Service_Sheets($client);

$result = $service->spreadsheets_values->get($spreadsheet_id, $spreadsheet_range);
var_dump($result->getValues());

Yep... it is still reading the file system to get the credentials, but we will change it. The important thing is that now we know how to create the necessary settings for the service account authentication.

In your application server's user profile issue this directive
export GOOGLE_APPLICATION_CREDENTIALS='/...path to your JSON
credentials.../credentials.json'
In your php application's Google Client API connection, include this
$client->useApplicationDefaultCredentials();
This will invoke the JSON file credentials without exposing the location or
contents in your source code.
On Thu, Dec 10, 2020 at 2:08 PM 'J Bruni' via greekub<EMAIL_ADDRESS>wrote:

I have this code running with success. I can read data from Google Sheets
with this:
useApplicationDefaultCredentials();
$client->addScope(Google_Service_Sheets::SPREADSHEETS_READONLY);
$service = new Google_Service_Sheets($client);

$result = $service->spreadsheets_values->get($spreadsheet_id, $spreadsheet_range);
var_dump($result->getValues());

Due to security policies, we can't have hard-coded credentials in files at
the file-system. (We store and read sensitive information from Amazon's Parameter
Store

.)

Having the necessary information for authentication in memory (variables),
how can achieve the service account authorization?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
, or
unsubscribe

.

--
Bryan Brodie
Appimatic LLC
804-591-0947

Thanks @appimatic
We managed to get it working as intended.
We read the credentials from Amazon Parameter Store. The security policies for our app do not allow the storage of any of its sensitive credentials anywhere hard-coded in the whole file system.
There is no need to invoke JSON file credentials.


Fixes mistakes and missing words in Czech datafile

Czech language is flective, therefore only lemma in dictionaries is enough.
Example for month June = červen
1st June = 1. června
June 1st = červen 1.
After the fix:

In [3]: dateparser.parse(u'6. června 2000', languages=['cs'])
Out[3]: datetime.datetime(2000, 6, 6, 0, 0)

Fixes cover:

correct weekday name abbrevs
all word forms for weekdays and month names
other small fixes

@janrygl thank you! can you please also add tests covering the changes you made?

Back to merging fixes. Updated with test.
Thank you for contributing!