Update Set-SPExcelDataProvider.md

Formatted example

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Set-SPExcelDataProvider.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Package project yml configs overriding root project configs

Describe the bug
A clear and concise description of what the bug is. What command did you run? What happened?
I installed this branch of the zendesk_source package in my root project. This branch uses the custom collect_freshness macro we built in order to disable (or rather, automatically pass) freshness tests for source tables that the user has elected to disable (note: the source entity itself is not configured/disabled, but rather the staging models drawing from these source tables are disabled). When variables (such as using_ticket_form_history) are defined in the package's own dbt_project.yml file, the package-provided value overrides any value provided in the root project file. I am unable to disable/pass freshness tests (using dbt source snapshot-freshness --select zendesk.ticket_form_history) by setting the appropriate variable to false in my root project (but if i change the package-defined variable to false, it passes).
Note: I am only seeing this conflict with freshness tests. When i provide using_ticket_form_history in my root project (false in my project yml and true in the package yml) and execute dbt run, the respective staging models that are appropriately disabled by the variable. So perhaps this has to do with the collect_freshness macro being a part of dbt core rather than utils or anything?
Steps To Reproduce
In as much detail as possible, please provide steps to reproduce the issue. Sample data that triggers the issue, example model code, etc is all very helpful here.

Install the bug/disable-freshness branch of the zendesk_source package
Add the below to your root project file

vars:
  zendesk_source:
    using_ticket_form_history: false

Add the following to the dbt_modules/zendesk_source/dbt_project.yml

vars:
  zendesk_source:
    using_ticket_form_history: true

Run dbt source snapshot-freshness --select zendesk.ticket_form_history
Expected: freshness test should pass, since we have chosen to disable/automatically pass freshness tests for this table
Actual: freshness test fails (presuming you do not have this table or just have stale data)

Change using_ticket_form_history to false in dbt_modules/zendesk_source/dbt_project.yml

Run dbt source snapshot-freshness --select zendesk.ticket_form_history
This passes, which makes it seem like dbt is indeed grabbing the variable value from the package yml

Removing the variable from the package project yml allows me to disable freshness tests from my root project yml, but a good deal of our packages have these default-set variables in their dbt_project.yml files, so just trying to figure out if we need to remove those in order for this solution to continue to work.
Expected behavior
A clear and concise description of what you expected to happen.
(see bold text in steps to reproduce ^)
Screenshots and log output
If applicable, add screenshots or log output to help explain your problem.
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[x] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.20.0
   latest version: 0.20.0

Up to date!

Plugins:
  - bigquery: 0.20.0
  - snowflake: 0.20.0
  - redshift: 0.20.0
  - postgres: 0.20.0

The operating system you're using:
MacOS
The output of python --version:
Python 3.8.9
Additional context
Add any other context about the problem here.

@fivetran-jamie Thanks for the really detailed write-up! Sorry for the delay getting back to you. I've been able to reproduce the bug, it's just a real head-scratcher to figure out what's going on here.
I do think the peculiarities of the source freshness task have something to do with this one. Unlike other tasks, which generate a runtime context, compile SQL, and then execute that compiled SQL within a materialization, the freshness task jumps straight to the finish: find and execute a certain macro using a macro_context.
There are two things working right:

dbt picks the right macro, preferring fivetran_utils.collect_freshness over built-in dbt.collect_freshness
The macro_context does seem to be doing the right sort of var resolution (sometimes)

However:

I see different results when I pass the macro into dbt.clients.jinja.get_rendered vs. dbt.clients.jinja.MacroGenerator

I dropped a debugger in here:
        macro_context.update(context_override)
        if 'collect_freshness' in macro.unique_id:
            import ipdb; ipdb.set_trace()
        macro_function = Macro
        Generator(macro, macro_context)

When I just wrap the macro in get_rendered, it does the right thing (no error):
ipdb> from dbt.clients.jinja import get_rendered
ipdb> get_rendered("{{ fivetran_utils.collect_freshness('zendesk.ticket_form_history') }}", macro_context)
'| column         | data_type |\n| -------------- | --------- |\n| max_loaded_at  | DateTime  |\n| snapshotted_at | DateTime  |\n'

But when I follow the actual codepath, and execute the macro using self.connections.exception_handler, it clearly renders differently (raising an error):
ipdb> macro_function = MacroGenerator(macro, macro_context)
ipdb> with self.connections.exception_handler(f'macro {macro_name}'):
result = macro_function(**kwargs)
*** dbt.exceptions.DatabaseException: Database Error
relation "zendesk.ticket_form_history" does not exist
LINE 42:     from "jerco"."zendesk"."ticket_form_history"
^
When I try to just get the rendered variable value on its own, it doesn't appear to be accessible (??), but at least it shows me the correct value (`false`), as set in my root `dbt_project.yml`.
```python
ipdb> get_rendered("{{ var('using_ticket_form_history') }}", macro_context)
*** dbt.exceptions.CompilationException: Compilation Error in macro collect_freshness (macros/collect_freshness.sql)
  Required var 'using_ticket_form_history' not found in config:
  Vars supplied to collect_freshness = {
      "zendesk_source": {
          "using_ticket_form_history": false
      }
  }

I want to confirm whether I'm arriving at the right distinction: get_rendered does one thing, MacroGenerator does a different thing, and that's at the heart of the matter about picking the wrong var value. Even if that distinction holds, I imagine it's a really just symptom of some really low-lying subtleties of the way we're constructing contexts + templates in dbt/clients/jinja.py.
@gshank I think you know the most about rendering contexts today, so I'd be curious to hear if what I've laid out above seems like the right thread to pull on, or if I've totally missed the boat.

I took another look at this one, just to see if anything new has clicked.
I think this is not specific to the execution of source freshness, as opposed to what I said above. After all, in manifest.json, dbt renders meta.is_enabled for the source as true! It prefers the value of the variable set in the package dbt_project.yml over the value of the variable set in the root project dbt_project.yml, when it should be the exact reverse. If we simplify the issue to that—yaml properties are rendered with the wrong var precedence—we might have a better shot at getting to the root cause here.
As a separate matter, I do think there's a better way to achieve this functionality, rather than overriding the collect_freshness macro, which can yield some surprising behavior for end users (e.g. slack thread). It's possible to disable freshness for a specific source table by setting freshness: source; I see that Fivetran packages often do this for certain tables. So how about something like:
      - name: ticket_form_history
        description: Ticket forms allow an admin to define a subset of ticket fields for display to both agents and end users.
        freshness: "{{ ({} if var('using_ticket_form_history', true) else none) | as_native }}"

This prevents the freshness check from running at all for this table when using_ticket_form_history: false. (Unfortunately, it still doesn't resolve the bug in this issue: at parse time, dbt is rendering the var with preference for the package value over the root project's.)
Ultimately, I'd like to see us do unto sources as we've done unto other resource types (https://github.com/dbt-labs/dbt/issues/3662):

add a config property to sources, making it possible to set enabled for a given source, e.g. to a Jinja expression that leverages a var value
convert freshness into an honest-to-goodness config that can be set/overridden in the root project's dbt_project.yml (as an always-available last resort)

I am also running into what I think is the same issue when developing packages. Using the jaffle shop example on this branch.
Customers model enabled using the enabled_customers var within the models config
enabled_customers default value false set in package's dbt_project file.
enabled_customers set to true in root project, which imports the package.
Generic tests on customers model are also enabled using enabled_customers var. This is intended to stop the warning dbt would throw if the customers model was disabled in the root project:
[WARNING]: Test 'test.jaffle_shop.unique_customers_customer_id.c5af1ff4b1' (models/schema.yml) depends on a node named 'customers' which is disabled
With this setup, given enabled_customers is set to true in the root project, you would expect to be able to both run and test the customers model. Instead you can run the customer model but no tests are found. This suggests the vars value in the schema.yml seemingly is taken from the packages dbt_project.yml file while the model's config is taking the value from the root projects dbt_project.yml


Back-off pulling image "registry.k8s.io/gateway-api/admission-server:v0.7.1"

crictl pull registry.k8s.io/gateway-api/admission-server:v0.7.1

@duanhongyi Thanks for the issue, it's a known issue and will be fixed after this PR is merged: https://github.com/kubernetes/k8s.io/pull/5352

this should be resolved now!

This should be resolved now, please let us know if there's still a problem and we'll re-open. :+1:


<EMAIL_ADDRESS>hash check failed

Scoop Configuration

{
  "lastupdate": "2021-12-21T08:30:04.1821496+02:00",
  "SCOOP_REPO": "https://github.com/lukesampson/scoop",
  "SCOOP_BRANCH": "master",
  "aria2-enabled": true
}

scoop update output
Updating 'everything' (<IP_ADDRESS>9 -> <IP_ADDRESS>5)
Downloading new version
Everything-<IP_ADDRESS>5.x64.zip (1,7 MB) [==========================================================================================================================================================================================] 100%
Checking hash of Everything-<IP_ADDRESS>5.x64.zip ... ok.
install-context.reg (796 B) [=====================================================================================================================================================================================================] 100%
Checking hash of install-context.reg ... ERROR Hash check failed!
App:         extras/everything
URL:         https://raw.githubusercontent.com/ScoopInstaller/Extras/master/scripts/everything/install-context.reg
First bytes: 57 69 6E 64 6F 77 73 20
Expected:    8dae88b926055e99ed6bc9f851bb684ef791cf5213638330e822e5a6c3b9a98f
Actual:      6b27de9e61ff039bf42cd07f4da23034c4678a8968403913ba29fa5c5a3b457a

Please try again or create a new issue by using the following link and paste your console output:
https://github.com/lukesampson/scoop-extras/issues/new?title=everything%4<IP_ADDRESS>5%3a+hash+check+failed

The problem doesn't seem to have been resolved. The bot seems to be prematurely closing the related issues.
Below is my console log.
>scoop install everything
WARN  Purging previous failed installation of everything.
Uninstalling 'everything' ().
'everything' was uninstalled.
Installing 'everything' (<IP_ADDRESS>5) [64bit]
Loading Everything-<IP_ADDRESS>5.x64.zip from cache
Checking hash of Everything-<IP_ADDRESS>5.x64.zip ... ok.
install-context.reg (796 B) [==========================================================================================] 100%
Checking hash of install-context.reg ... ERROR Hash check failed!
App:         extras/everything
URL:         https://raw.githubusercontent.com/ScoopInstaller/Extras/master/scripts/everything/install-context.reg
First bytes: 57 69 6E 64 6F 77 73 20
Expected:    8dae88b926055e99ed6bc9f851bb684ef791cf5213638330e822e5a6c3b9a98f
Actual:      6b27de9e61ff039bf42cd07f4da23034c4678a8968403913ba29fa5c5a3b457a

Run scoop update and then try

I still have the exact same issue here.
scoop update does not fix it.


ubuntu-22.04 runner image 20230313.1 fails due to regression issue in Azure CLI 2.46.0

Description
This simple command fails (both locally and in github actions) while it was working before for months:
az aks update --name *** --resource-group *** --attach-acr ***

Downgrading to Azure CLI 2.45.0 works.
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Current runner version: '[2](https://github.com/jamixdev/g6-infra-as-a-code/actions/runs/4446029380/jobs/7805993806#step:1:2).[3](https://github.com/jamixdev/g6-infra-as-a-code/actions/runs/4446029380/jobs/7805993806#step:1:3)03.0'
Operating System
  Ubuntu
  22.0[4](https://github.com/jamixdev/g6-infra-as-a-code/actions/runs/4446029380/jobs/7805993806#step:1:4).2
  LTS
Runner Image
  Image: ubuntu-22.04
  Version: 20230313.1
  Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20230313.1/images/linux/Ubuntu2204-Readme.md
  Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230313.1
Runner Image Provisioner
  2.0.12[7](https://github.com/jamixdev/g6-infra-as-a-code/actions/runs/4446029380/jobs/7805993806#step:1:8).1

Is it regression?
20230305.1
Expected behavior
The command should succeed
Actual behavior
The command fails with:
Waiting for AAD role to propagate[###                                 ]  10.0000%
Waiting for AAD role to propagate[#######                             ]  20.0000%
Waiting for AAD role to propagate[##########                          ]  30.0000%
Waiting for AAD role to propagate[##############                      ]  40.0000%
Waiting for AAD role to propagate[##################                  ]  50.0000%
Waiting for AAD role to propagate[#####################               ]  60.0000%
Waiting for AAD role to propagate[#########################           ]  70.0000%
Waiting for AAD role to propagate[############################        ]  80.0000%
Waiting for AAD role to propagate[################################    ]  90.0000%ERROR: Could not create a role assignment for ACR. Are you an Owner on this subscription?

Repro steps

deploy aks with container registry attached
run az aks update --name *** --resource-group *** --attach-acr *** on the previously deployed aks as the subscription owner

Possibly related https://github.com/Azure/azure-cli/issues/25854

Hey @codepic! As you already found that issue connected with bug in Azure CLI 2.46.0. We won't downgrade it because of security and maintenance reasons. And as you already mentioned too you may use another version of AzCLI by installing it in runtime.


I can not open spyder 2.7. ValueError: Dependency has already been registered: pandas

Description of your problem
It seems that there are some conflicts with pandas. If I uninstall pandas, it works again. However, I need pandas.
Here is the log:
Traceback (most recent call last):
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\spyder.py", line 2367, in main
mainwindow = run_spyder(app, options, args)
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\spyder.py", line 2255, in run_spyder
main.setup()
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\spyder.py", line 779, in setup
from spyderlib.plugins.console import Console
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\plugins\console.py", line 30, in
from spyderlib.utils.environ import EnvDialog
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\utils\environ.py", line 17, in
from spyderlib.widgets.dicteditor import DictEditor
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\widgets\dicteditor.py", line 39, in
from spyderlib.widgets.dicteditorutils import (sort_against, get_size,
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\widgets\dicteditorutils.py", line 61, in
required_version=PANDAS_REQVER)
File "C:\Users\toluo\AppData\Local\Continuum\Anaconda\lib\site-packages\spyderlib\dependencies.py", line 68, in add
% modname)
ValueError: Dependency has already been registered: pandas
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
Versions and main components

Spyder Version:
Python Version:2.7.13
Qt Version:
PyQt Version:
Operating system:

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:

@ccordoba12 it seems we are registering pandas twice, maybe we should not error, but simply ignore this if already registered?, or replace with the one already set. Although it would be nice to make sure we are not doing this twice, so a warning would still be useful

This error was present in Spyder 2.3 but it was fixed a long time ago. Please update to our latest version (3.2.0) and try again.

@ccordoba12  You are right. Thank you!


Update typo in table of contents

Update typo "analaytics" in table of contents (it was driving me crazy)

@newstrider : Thanks for your contribution! The author, @, has been notified to review your proposed change.

Thanks!


enable metallb

This largish PR does the following:

enables load balancers using metallb by default on Packet CCM. It can be disabled by option, as listed in the README here
ensures it all syncs correctly
adds a make fmt command that formats the files
updates the README doc for running locally (usually during development) with the manifests

Some important notes about the load balancer

By default, a metallb speaker runs on all nodes, including control plane nodes. We may want to provide the user with an option to disable it. If so, future PR
I haven't looked into using it also for a control plane load balancer, or how it would conflict with one

ClusterAPi uses the spec released under ./deploy/releases to deploy packet-ccm, will it be updated as part of the release process?

ClusterAPi uses the spec released under ./deploy/releases to deploy packet-ccm, will it be updated as part of the release process?

Yes. Once this is merged in, we should cut a new release (I think v1.1.0 per semver) and then we can consume it.

https://github.com/packethost/packet-ccm/pull/56 adds a command make prerelease to prepare the releases.


BitHound Stinks

I think we can all agree that BitHound stinks. I think I found an alternative called Code Climate.
Pros:

Better than BitHound
Analyzes Line by Line
Graphs things
Intagrates with Atom

Cons:

We need to host ourselves
Might not be the same thing as BitHound.

@nywillb BUT I LIKE BITHOUND

@willweinstein :facepalm:

@nywillb HEY GUESS WHAT CODE CLIMATE HOSTED IS FREE FOR OPEN SOURCE
https://codeclimate.com/github/PlanHubMe/PlanHub

@thatoddmailbox YAY!

@thatoddmailbox One Problem:

^ That is a lot ^

@nywillb most are duplication errors in minified stuff (not our code) or compatibility but we don't support IE7 so.

@thatoddmailbox: Can we officially switch to code climate

Done!

Please get bitHound off Slack
Why are bitHound code checks still being done?

Yay! 4fcc71a!

@RobotXLabs re-opening because we still have to do slack and code checks.

Hi everyone!
I'm a developer at bitHound and we're always striving to improve our experience. I noticed you had a lot of PRs with pending statuses from bitHound. Unfortunately, we experienced some issues around that time causing that behaviour but it's since been resolved :-).
We're a young startup and any feedback you could give back regarding where we may have fell short would go a very long way. I hope that you might consider giving bitHound another shot, but, if not, Code Climate is a great choice as well.

@CSilivestru Thanks for asking! One of the things we found annoying with bitHound was that it generated a lot of errors that weren't really problems. For example, if you look at our bitHound page, you'll see that it complains jade is insecure, when the package it complains about (a dependency of a dependency!), uglifyjs, is not used by jade (or at we don't use it).
Also, we've found that the way Code Climate breaks down issues (it categorizes them into different types, like "Style", "Bug Risk", etc) is more helpful than marking "priority files".
Finally, and this may just be because you are a smaller startup, there isn't much documentation on bitHound. It could be possible to have it ignore packages, but there doesn't seem to be any documentation on your site, apart from a "FAQ" section that isn't easy to sort or search through. It would be better if were a separate site dedicated to configuration, troubleshooting, etc of bitHound.
If you resolved some (or all!) of these problems, we would definitely be open to giving bitHound a second try!

@thatoddmailbox Thanks for responding so quick! Great points and I'll try to explain each, in turn.
A nested insecurity is still, at the end of the day, making the module that depends on it insecure. This is a relatively new problem that's been showcased in the node and npm world but uglify providing an attack vector in jade could expose a vulnerability in your project :-). However, we understand that this is not always the case and so these cases ding your score a lot less as a vulnerability is more and more nested. In addition, status checks only fail on first level insecure dependencies because we also thought it would be a little too aggressive to fail an entire PR because of a nested dep vulnerability.
Priority files has always been something we've thought could be improved. We're currently exploring what that means but I understand the benefit of categorizing the different problems. We decided to start with categorizing a little less broadly and focus on insight 'types' such as Complexity, Ignored Errors, Duplicate, etc. However, I've passed this feedback back to the team.
We actually just deployed an update to our FAQ that helps to make finding some of our documentation a little bit easier. I would encourage you to checkout our bitHound-101 page here.
I hope this helps a little bit and please let me know if I can help in any way!


"ffmpegthumbnailer could not be found" while it's installed and available

Getting ffmpegthumbnailer could not be found in your path,\nplease install it to display video previews even when ffmpegthumbnailer is available.
# which ffmpegthumbnailer
/usr/bin/ffmpegthumbnailer

In the code there is a check for viu:
      if ! command -v viu &> /dev/null; then
        echo "ffmpegthumbnailer could not be found in your path,\nplease install it to display video previews"

which is a separate project. Once installed it's working without issues.

It doesn't look like it needs to be installed even. In my case I did:
sudo touch /usr/bin/viu
This assuming /usr/bin/ is in path (likely).
And it worked. After installing viu and also removing viu from touch nothing changed.


Datasets

We have multiple datasets that we need to prepare:
multimodal single-cell CRISPR screens:

[ ]  https://nature.com/articles/s41588-021-00779-1

[ ]  https://www.nature.com/articles/s41588-021-00778-2 (mixscape)

[ ] Christians Bocks dataset (we already have it)

[ ]  Hanifta large scale cite-se datasets with covid 19 (Chris has the data, large-scale CITE-seq, maybe goo to show scale)

Uni Modal KO datasets #17

[ ]  Norman et al.  https://science.sciencemag.org/content/365/6455/786.abstract (we have)

Normal scRNA-seq:

continues covariates to test auger ( https://science.sciencemag.org/content/367/6473/45, we have the data)
cross-species dataset with continuous-time but only perturbation (https://www.nature.com/articles/s41586-018-0657-2, we have the data)

RNA velocity perturbation analysis

[x] dataset which was analyzed augor protocol paper

[ ] Prepare the mixscape toy dataset and upload it into a function to the pertpy.datasets equivalent. (me)

Tracked in some GDoc


RqHeaders must be an interface

RqHeaders must be an interface and its implementation must be in its nested class RqHeaders.Base. Let's refactor it accordingly. The interface should look like:
public interface RqHeaders extends Request {
  Iterable<String> header(String name);
  Iterable<String> names();
}

@yegor256 sure, thanks, I added bug tag to it

@yegor256 thank you for the ticket reported, I topped your account for 15 mins, transaction 000-d5b9dcee

@lauresh this task is for you, keep these principles in mind, and don't hesitate to ask any technical questions
The budget of this task is 30 mins. This is exactly how much time will be compensated, when the task is completed. Read about our Definition of Done

@yegor256 I have question about Javadoc.
How can I understand release number (@since xxx) for my changes? Previous release version is 0.13.7 How can i understood next number (0.13.8 or 0.14)?
Thank you!

@lauresh just look here and pick the next major version: https://github.com/yegor256/takes/releases

@lauresh you're working with this ticket for the last 15 days. If it is not closed within the next 48 hours, it will be re-assigned to someone else, see No Obligations principle. This article should help if you're stuck; -30 to your rating, your total score is -60

@lauresh the task is overdue, and I have to re-assign it to someone else. Please, stop working with it immediately. In general, we're against overdue tasks, check this page; -60 to your rating, your total score is -180

@yamstranger please proceed,it's yours

@davvd  thanks

@davvd, @yegor256 ticket passed QA review.

@yegor256 please close this ticket. Thanks

@yegor256 @davvd please help me understand why this ticket is invalid, and why I was removed from this ticket?  Thanks

@YamStranger was there any code written by yourself? if so, in what PR?

@yegor256  Not. I did not wrote any line on code on this ticket. But to be honest - this ticket was not closed, only because @elenavolokhova did not closed it, and did not marked as "passed review". So when @davvd assign it to me - my task was close this ticket. And as I understand, if ticket assign to me - now I become responsible for it, and can rise my rating on this ticket, or loos some points(as @lauresh ), also difficulty of task described in "30 mins" label, nothing else matters. Is it correct? on  And I closed it by writing all info, what @elenavolokhova expected. After this ticket become "ready for closing", @davvd described payment info  for @longtimeago and  @elenavolokhova, and I was just removed from this ticket, and it was marked as "invalid".  So, according rules - please describe me - why I spend time to some ticket for closing it, and instead of any changing of  my rating  - it just become "invalid". P.S. I don't know what is "PR" means. I spend on this ticket 4 minutes.

@yegor256. As wrote @alice, I am not with your team any more. So, let finish this discussion. As I understand now - you pay only of code (of any project improvement), but not for "closing tickets". That is why this ticket was closed without adding any points for me. @yegor256 Is my understanding correct?


when do we support cudnn4?

cudnn4 natively support batch normalization, when do we support cudnn4?
Actually, theano acts very fast, it support cudnn4 only several days after it comes out.

@fayeshine, we will officially switch to Cudnn R4 when it is officially released. For now, it is a release candidate.

After commit 22ebf0a, you can build TensorFlow with Cudnn R3 and R4, after the following changes to your Cudnn library.
For Cudnn R3:
$ ln -s libcudnn.so.7.0 libcudnn.so.6.5
For Cudnn R4:
$ ln -s libcudnn.so.4.0.4 libcudnn.so.6.5
This is currently still an unofficial feature. TensorFlow will officially switch to R4 soon after the Cudnn R4 final version is released.
The performance with R4 is better than R2. There are still a number of memory and performance improvements we are working on. Please stay tuned for more changes in this area.
Thanks.
-XQ

@zheng-xq ,thank you! hope to see the official cudnn4 comes out soon! I'm really interested to see how theano and tensowflow acts on cudnn4. I believe tensowflow will compile much faster and hope it can also run the training faster than theano.

@zheng-xq However, when do we start to support cuda 7.5 or upcoming 8.0?

@zheng-xq , cudnn v4 now is officially released

We are working on upgrading to Cudnn R4, and test it on all platforms. We'll keep this thread posted.

@zheng-xq , and what about cuda 7.5?

@fayeshine I think cuda 7.5 is also experimentally supported right now. See https://github.com/tensorflow/tensorflow/blob/master/tensorflow/g3doc/get_started/os_setup.md#using-a-different-cuda-sdk-and-cudnn-versions


replace only way by fastest way

At the item "You lost your phone or got a new phone number" on https://docs.microsoft.com/en-us/azure/active-directory/user-help/multi-factor-authentication-end-user-troubleshoot we have an alternative step that is "ask your administrator to clear your settings" so secondary phone number is not the "only" way to get back into your account if your phone is lost or stolen.  But is the fastest since the user can authenticate him or her self without waiting the administrator.

@danieldbr : Thanks for your contribution! The author, @eross-msft, has been notified to review your proposed change.

Thanks for the edit. #sign-off


Update demo application and make it simpler

Demo application it's not working because of a change in Devise strong parameters, this PR fix that.
I also finished a lot of differents things to make the example simpler:
Tasks

[x] Bump Rails version
[x] Remove pg gem in favor of sqlite3 gem
[x] Fix Devise strong parameters config
[x] Update schema.rb
[x] Organize User model
[x] Remove unused gems
[x] Add flash messages to layout

Thanks so much for doing all this!


Run all tests for a Workspace with One command (Proposal)

Problem
I currently have a project with multiple modules. I'm using the new workspace feature in 1.18 and it works fantastically for getting dependencies and organizing my workflow better. One issue I'm having is actually testing all of the different modules at the same time.
go.work
go 1.18

use (
	./example
	./example/template
	./hello
	./hello/something
)

Given my current go.work above, I can test manually by running each of the use directories separately
How I run tests manually
go test ./example
go test ./example/template
go test ./hello
go test ./hello/something

Proposed Solution
Have a  single command that will run all of the tests for your given workspace.
There is currently a -workfile flag but I can't for the life of me find documentation or how to get that working.
My attempt at getting -workfile flag working
-  go-go-go git:(main) ✗ go test -workfile=go.work
no Go files in /Users/zachajon/Projects/delete_later/go-go-go

I'm completely open to all feedback and help that can be given! This is my first time posting an issue to an open source project like this and I'm nervous and super excited!!!

Duplicate of #50745


Add 4.0 craftables

Tidal Shadow:

[x] New weapon package
[x] Config in weapon package
[x] Run pipeline with added config
[x] Weapon key
[x] Shortcuts for weapon key
[x] Add weapon package to imports
[x] Weapon curve
[x] Weapon passive
[x] Update documentation

Song of Stillness:

[x] New weapon package
[x] Config in weapon package
[x] Run pipeline with added config
[x] Weapon key
[x] Shortcuts for weapon key
[x] Add weapon package to imports
[x] Weapon curve
[x] Weapon passive
[x] Update documentation

Rightful Reward:

[x] New weapon package
[x] Config in weapon package
[x] Run pipeline with added config
[x] Weapon key
[x] Shortcuts for weapon key
[x] Add weapon package to imports
[x] Weapon curve
[x] Weapon passive
[x] Update documentation

Finale of the Deep:

[x] New weapon package
[x] Config in weapon package
[x] Run pipeline with added config
[x] Weapon key
[x] Shortcuts for weapon key
[x] Add weapon package to imports
[x] Weapon curve
[x] Weapon passive
[x] Update documentation

Flowing Purity:

[x] New weapon package
[x] Config in weapon package
[x] Run pipeline with added config
[x] Weapon key
[x] Shortcuts for weapon key
[x] Add weapon package to imports
[x] Weapon curve
[x] Weapon passive
[x] Update documentation

closing in favor of #1643


Patch for deleting service accounts

Bring the patch from https://github.com/openshift-knative/eventing-kafka-broker/pull/1125 to main branch.

will we revert the work-around once actual issue is fixed?

will we revert the work-around once actual issue is fixed?

Yes. That's the idea.
I suppose the patch needs to be listed in generate-release like mentioned in https://github.com/openshift-knative/eventing-kafka-broker/pull/1125#issuecomment-2194236977


j2hcj rename to hcjify

j2hcj is inaccurate because it can take in not only jamo as input, but any values.

to be changed in 0.4


"Fix" example which calls regex.match() twice

Still ugly but calls regex.match('abc') only once. :sunglasses:

Heh that’s both cute and terrible at once. ;)
I wouldn’t replace the existing recipe but we could add your abomination as an additional example. :)

Thanks! I didn't get a notification about your commit so I missed it. For 2 years. 🙈


Mysql postgres relationship

Relevant information
Adding Rules to relate APM to Mysql and Postgres OHI entities

@gsanchezgavier should we close https://github.com/newrelic/entity-definitions/pull/1161 then right?

@gsanchezgavier should we close #1161 then right?

hey Ignacio 👋 I already reached out to Guillermo about this. I think we are safe to close it

tracked under https://issues.newrelic.com/browse/NR-146983


allow jwt signing algorithm per service

Allow to specify JWT signing algorithm per service (like idTokenSigningAlg in OIDC client)
Actually, RS512 and HS512 are CAS defaults, but IETF spec recommends or requires others, like RS256 and HS256.
These algorithms are often required by third-party RPs.
The PR is another attempt after PR 5193 but without breaking changes for existing setup.

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


LAN-176 - remove type coverage

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

Project coverage is 80.94%. Comparing base (795ca93) to head (a34897d).

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #31   +/-   ##
=========================================
  Coverage     80.94%   80.94%
  Complexity      111      111
=========================================
  Files            19       19
  Lines           572      572
=========================================
  Hits            463      463
  Misses          109      109

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


Include Lagrange kernel trace polynomial term into DEEP composition polynomial

As the title says, this PR includes the missing term(s) in the DEEP composition polynomial related to the Lagrange kernel trace polynomial. A few remarks are in order:

Although we open at $z, z\cdot g, \cdots z\cdot g^{2^{\log(tracelen)-1}}$, we end up having to add only one term to the DEEP composition polynomial. This is because we use a multi-point quotient instead of single point quotients. This comes at a slight degradation of security which should be accounted for (in a future PR) in the security calculator.
During proof generation, the quotient from the previous point is currently computed using a naive polynomial division. This should be benchmarked to see if we might need to use an FFT-based division procedure.
Currently, the code makes an assumption (but doesn't enforce it) that the Lagrange kernel column is always the last one in the auxiliary trace. This makes the code for computing the DEEP polynomial cleaner.
Some TODO's are contingent on #271 .

Rebased and updated the code in light of #271


Customize user profile

Some issues implementing a custom user profile.  Following documentation with this structure in my code:
# In myapp/models.py

from django.db import models

class MyProfile(models.Model):
    user = models.OneToOneField("auth.User")
    date_of_birth = models.DateField(null=True)
    bio = models.TextField()

I am getting an error:
AttributeError: 'str' object has no attribute '_default_manager'
Reading up suggests that the issue is the use of "auth.User" in quotes. When this is commented out, the problem goes away (except then I can't link to the user profile).
Shouldn't there be an import here to get to the auth.User model?

this project is death...
use another cms

Any suggestions for a CMS that is similar to Mezzanine? I really enjoy how simple the login/auth options generally are. I find things like django-cms to be over-engineered

Questions like this should be sent to the mailing list - not raised as issues. https://groups.google.com/forum/#!forum/mezzanine-users
In any case, you're right about the quotes not being needed. What you should do is:
from django.db import models
from django.contrib.auth.models import User

class MyProfile(models.Model):
    user = models.OneToOneField(User)
    date_of_birth = models.DateField(null=True)
    bio = models.TextField()

Thanks @molokov  -> I wasn't aware of the Google group.

Any suggestions for a CMS that is similar to Mezzanine? I really enjoy how simple the login/auth options generally are. I find things like django-cms to be over-engineered

i try use wagtail cms instead mezzanine


Another publicPath for production mode

Hi ,
I write npm run start for development mode. This command creates manifest.json file every time.
When I write npm run build the new manifest.json is not created. For development mode I have one  publicPath: 'http://localhost:2992/frontend/assets/' and for production mode i have another one. So I get a problem. On the local server everything is ok. . There is only a wrong  publicPath in manifest.json on production.
Is it possible to cteate the new manifest.json file after writing npm run build with publicPath for production?

Hi,
Did you try setting and checking the NODE_ENV in your webpack config? Then you could set a different path based on it being production or not.

Actually, I have three webpack.config files: for development, for production and the common one. I have tried with  NODE_ENV. But it doesn't help. The problem is that manifest.json is not rewritten after npm run build. The last manifest.json is in use after npm run start.

The issue was caused by MiniCssExtractPlugin.Wrong options. After that manifest.json is rewritten every time. My mistake.


Add UX support for HC detector

Issue #, if available:
Description of changes:
Add UX support for HC detector.
Preview for HC detector:

DetectorResult for HC detector:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

will fix UT while reviewing

current UT passed. will add more UT for new code.
Test Suites: 1 skipped, 51 passed, 51 of 52 total
Tests:       5 skipped, 230 passed, 235 total
Snapshots:   46 passed, 46 total
Time:        30.932s
Ran all test suites.
Done in 31.66s


why can't I re-use the same parser object?

Here is my nearley scripts and here is my test script.
Suppose I move the parser declaration outside of the forEach, then if I run two feed() calls the second one will be at offset 31. This leads me to two questions:

why is it at offset 31? Is it just chaining all of the inputs together when using feed?
is it advisable to not reuse the same parser object? my scripts could potentially be ~10k lines at most so I don't know if there's a huge performance hit from instantiating a parser 10k times.

Ah, this has come up before. The thing is, you can't think of a parser as a reusable object here, because a parser object maintains some "state" inside it. It's like how you can't really reuse a teabag because the process of using a teabag changes the teabag.
The reason feed() chains input is because nearley is a streaming parser: you can incrementally give it parts of a string and it'll parse it as it gets it. The use-case here is if you're reading a really long string from a file or from the network. One way to parse a long string is to first get all of it from the network, then parse it. But it's much nicer if you can start parsing the string as chunks of it get downloaded over a slow connection—and you get the benefit of not having to store the entire string in memory at once.
Are you using a separate parser for each line of code in your scripting language? I think you should, ideally, be using a single parser for an entire script, and write the parser so that it understands a multiline file. That's how all "real-world" languages are parsed. This lets you, for example, have a line break in the middle of a long statement without repurcussions:
console.log(
 thisIsAVeryLongStatementThatWillNotFitOnASingleLine();
);

Hope this helps!

Ideally, yeah, I would like to use a single parser for the entire script. I guess then it's just a matter of figuring out how to do that. I'm guessing I'll need to adjust my main node?
Also, since I'd be using a single parser... eventually, I wanted to implement variables. Variables would just be string substitution for me, so:
[TitleFont]=Arial, 16, BB, #000000
font=[TitleFont]

and
[TitleFont]=Arial
[TitleFontSize]=16
font=[TitleFont], [TitleFontSize], BB, #000000

would be equivalent (it's essentially just like a macro in C - copy/pasted all over the place). I was going to try to make it parse one statement at a time, and then do a bit of preprocessing on a string if I have any variables that are in the next statement, then apply them. I'm sure something like that is possible, but is there any docs on doing something like that?
Thanks! I'll close this issue regardless since I did get an answer.

Yeah, you can do something like:
main -> whitespace (line_of_code whitespace):*
to parse all the lines of a file.
I would do the replacement on the parsed structure, not the input string. Do something like:
macrodefinition -> "[" name "]" "=" value
value -> number | fontname | hexcolor | … | macrovalue

Then, once you have parsed the string, you can traverse the parsing output to find "macrodefinition"s and keep a dictionary of them. Then traverse again, searching for "macrovalue"s and replace them with the corresponding "macrodefinition" entry.
Nearley itself has a "macro" feature which you can read about in the readme; it works the way I described above.
withX[str] -> "XXX" $str

main -> withX["cow"] # this matches "XXXcow"

Cool. Thanks for all the help again, I'm pretty new to ... well, all of this. I wasn't sure if nearleys macros worked like I was expecting, but it sounds like it's something to look into. Looks like I'm going to have to restructure a lot of things to make it work, but better now than later.


Complete ref editing support

We are working on complete support for editing <element-citation> records.
Step 1: JATS4R representation: We define a strict schema for JATS4R for all available publication types. This schema must be valid JATS! Here's an example for modelling a journal article.
<ref id="r1">
  <element-citation publication-type="journal">
  <x specific-use="display">Townsend BR, Subasi E, Scherberger H (2011) Grasp movement decoding from premotor and parietal cortex. J Neurosci 31:14386 –14398</x>
    <person-group person-group-type="author">
      <name>
        <surname>Baumann</surname>
        <given-names>MA</given-names>
      </name>
      <name>
        <surname>Fluet</surname>
        <given-names>MC</given-names>
      </name>
    </person-group>
    <year>2009</year>
    <article-title>Context-specific grasp movement representation in the macaque anterior intraparietal area</article-title>
    <fpage>6436</fpage>
    <lpage>6448</lpage>
    <pub-id pub-id-type="doi">10.1523/JNEUROSCI.5479-08.2009</pub-id>
  </element-citation>
</ref>

[ ] figure out what publication types we want to support (currently journal and book are supported)
[ ] figure out requirements (required and optional fields for each supported publication type)
[ ] improve schema based on requirements (let's look at content from scielo, elife, pkp, erudit)
[ ] consider implementing other publication types once we have a complete specification for them (.e.g “book”, “letter”, “review”, “journal”, “patent”, “report”, “standard”, “data”, “working-paper”)

Step 2: We define an improved schema TextureJATS (internal schema, the model that the editor manipulates), which helps us realize our user interface in the best way. However it carries the same data, just modelled slightly differently. For instance instead of fpage,lpage,e-location-id we use a custom element content-loc wrapping these elements and providing an expclicit type attribute which states either print or electronic. This allows us to implement a switcher in the user interface, showing either fields for print publications or elocation-id for electronic publications. Also in TextureJATS all editable fields are non-optional. So there must always be at least an empty element present. This allows us to always render placeholders to be filled out or left blank.
In TextureJATS the example from above looks like so:
<ref id="r1">
  <element-citation publication-type="journal">
  <x specific-use="display">Townsend BR, Subasi E, Scherberger H (2011) Grasp movement decoding from premotor and parietal cortex. J Neurosci 31:14386 –14398</x>
    <person-group person-group-type="author">
      <name>
        <surname>Baumann</surname>
        <given-names>MA</given-names>
      </name>
      <name>
        <surname>Fluet</surname>
        <given-names>MC</given-names>
      </name>
    </person-group>
    <year>2009</year>
    <article-title>Context-specific grasp movement representation in the macaque anterior intraparietal area</article-title>
    <content-loc type="print">
      <fpage>6436</fpage>
      <lpage>6448</lpage>
      <page-range></page-range>
    </content-loc>
    <pub-id pub-id-type="doi">10.1523/JNEUROSCI.5479-08.2009</pub-id>
    <!-- empty elements (not optional anymore) -->
    <source></source>
    <volume></volume>
    <!-- lots of more empty elements, e.g. fields used with user publication types -->
    <chapter-title></chapter-title>
  </element-citation>
</ref>

[ ] improve schema as needed, which good decisions the editor implementations can be made simpler

Step 3: Provide transformers between JATS4R and TextureJATS

[ ] write transformer JAT4R -> TextureJATS -> JATS4R (importer and exporter)
[ ] write tests for all supported transform scenarios

Step 4: Improve Editor EditRef component.

[ ] Use a separate component for each publication type (e.g. EditJournalArticle)
[ ] Reusable components where possible (e.g. EditPersonGroup)
[ ] generate a display version on each change to the element-citation <x> element. (this could be acchieved with providing a simple html renderer (based on a template string)
[ ] Improve RefListComponent to use rendered <x> element for display

@Integral please work in element-citation-editing branch.

This is mostly in place. Closing this issue in favor of #238


Can't change WebSite settings while creating

Trying to remove or change php support/version on new sites that are created.
Using:
CreateOrUpdateSiteWithHttpMessagesAsync(resourceGroupName,
                siteName, new Site()
                {
                    Location = location.ToString(),
                    ServerFarmId = serverFarmId,
                    SiteConfig = new SiteConfig()
                    {
                        PhpVersion = "5.6",
                        PythonVersion = "3.4",
                        Location = location.ToString()
                    }
                }).Result

PhpVersion isn't changed. I also tried PhpVersion = "off" - doesn't work also
Tried to catch http request. Found that http request was sent with json data:
{
  "location": "JapanWest",
  "properties": {
    "serverFarmId": "%serverfarm id here%",
    "siteConfig": {
      "location": "JapanWest",
      "properties": {
        "phpVersion": "5.6",
        "pythonVersion": "3.4"
      }
    }
  }
}

but PHP and Python versions has not been changed

I've just removed the PHP version setting because it seems the "off" is the default one. phpVersion setting was working a half year ago anyway so something had to change recently.

Is this still an issue?  If so, can you please provide the .NET library and version you are using for this?
(Close on Oct 9 if no response)


Swift 6.0 update

Description of changes
This library has been updated to Swift 6.0
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the readme
[x] My changes generate no new warnings
[ ] I have added unit tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Addresses #26


Limit of unique scurity scopes

The SPO Limits article (https://docs.microsoft.com/en-us/office365/servicedescriptions/sharepoint-online-service-description/sharepoint-online-limits) states that the limit of Unique security scopes per list or library is 5,000.  However, this migration article mentions that the 5,000 recommended limit and 50,000 hard limit is per site. Can you help reconcile the apparent conflict?

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

ID: 842a924e-01b2-5aed-5b1a-c190431fe3fb
Version Independent ID: d6e9e76b-bcf4-e2d6-f268-5ca411b8e4f6
Content: SPO Migration API: Migrating shared files and folders
Content Source: docs/apis/migration-api-shared.md
Product: sharepoint
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@bernardlam I wrote a script to test the hard limit and ran it in two different tenants. The limit is (currently at least) 50,000. The script created list items and broke inheritance on them until it came up with the ServerException "You cannot break inheritance for this item because there are too many items with unique permissions in this list.".

@timdw  based on your test is the 50,000 hard limit per list or per site?

@timdw have you tried creating 2 (or more) lists in a site and adding 50,000 unique permission scopes to each list?

@bernardlam another list in the same site has more than 5,000 unique permissions which means the 50,000 limit cannot be per site.

I see. good to know! in this case should this article be corrected, since it mentions that the limits are per site, instead of per list?

Yes, and the SPO limits article updated too. Unless Microsoft are planning to enforce the 5,000 limit of course ...


add jets destroy as opposite of generator

This is a 🙋♂ feature or enhancement.
This is a 🧐 documentation change.

[x] I've added tests (if it's a bug, feature or enhancement)

The thing is, I didn't know how to write specs for this, as I couldn't find the corresponding specs for generate. I assume it'll be ok since jets piggy back this from rails.

[x] I've adjusted the documentation (if it's a feature or enhancement)
[x] The test suite passes (run bundle exec rspec to verify this)

Summary
This PR adds jets destroy command. Basically, the opposite of jets generate. The command calls behavior: :revoke instead of :invoke.
Context
See #rails-destroy.
Screenshot

Version Changes
TINY will suffice.

Neat! 👍

@tongueroo any suggestion for specs?

@aoozdemir For specs, maybe a CLI spec that does a noop? So the CLI portion is tested. Not a biggie though.
One thing with this PR is that these 2 classes are nearly identical:

https://github.com/aoozdemir/jets/blob/master/lib/jets/destroy.rb
https://github.com/aoozdemir/jets/blob/master/lib/jets/generator.rb

$ diff lib/jets/{generator,destroy}.rb
2c2
< class Jets::Generator
---
> class Jets::Destroy
17c17
<     Rails::Generators.invoke(@generator, @args, behavior: :invoke, destination_root: Jets.root)
---
>     Rails::Generators.invoke(@generator, @args, behavior: :revoke, destination_root: Jets.root)

So a bit of duplication here. Should be only one Generator class with a parameter.
Another thing is that jets destroy and jets delete are too similar. Will be confusing to remember. Understand that it's called rails destroy but rails doesn't have deploy and delete concepts, so that's probably why it's not an issue for it. Think it makes sense to not follow rails here. Thinking let's go with  jets degenerate instead; it feels clearer.

@tongueroo thanks for the reply! degenerate makes sense. 👍I'll dry the classes and do the appropriate changes for degenerate, then make a commit.

@tongueroo hi! sorry this took so long 🙂 I've added a simple cli spec here https://github.com/tongueroo/jets/pull/208/commits/3f697eac89d5b6bb836d7843ebd2e6f3c89262b3.

@aoozdemir Yes! So much better. Felt the same thing but been haven't had time to put thoughts together explaining it. Happy you figured it out!  Cheers, Tung

Will go out in next release

@tongueroo Hah! Amazing. Literally jumped out of the bed at 5 am, once I realised what could have I done instead. :) Thanks a lot.
Bests.


set() fails when NULLs are passed to it

According to the phpdoc of ClientInterface:
* @method mixed  set($key, $value, $expireResolution = null, $expireTTL = null, $flag = null)

When executing set("key", "value"), this is the resulting command from redis-cli monitor:
"SET" "key" "value"
However, when NULLs are passed in for the TTL values:
set("key", "value", NULL, NULL) - they are passed along as-is to redis:
"SET" "key" "value" "" ""
...which leads to an exception.

Possibly some missing code, or a bad refactor?
https://github.com/nrk/predis/blob/v1.1/src/Command/Command.php#L31
filterArguments just returns the arguments without modifications

@dannykopping you are not supposed to pass NULL values to SET (or any other Redis command for that matter) since everything in Predis gets converted to string when passed as an argument to commands. Those NULLs in the phpdocs for Redis commands are just for the sake of documenting optional arguments as there's no other way to achieve that.


Sentry client crashes entire process with "concurrent map writes" due to unsynchronized contextifyFramesIntegration

We are attempting to use the same Sentry client in a server with many goroutines, as we assumed this client was thread-safe.  However, we're getting "concurrent map writes" errors.

Is the Sentry client intended to be thread-safe?  Specifically, is it intended to be safe to concurrently send events to the client returned by sentry.NewClient?

If yes, then we have found a bug: the contextifyFramesIntegration in integrations.go is not thread-safe.  In particular, it has a cachedLocations variable which is a map, and access to this variable is not synchronized.  We're getting stack traces from the concurrent map writes errors that point to reads/writes of this map.

I noticed that sourceReader has a similar caching mechanism, but that one is protected by a mutex....
We're using the Sentry client from commit 510f01e3c1e95d434331b518fdc62b2adaf5cff8, but I don't see anything fixed in the latest master, either.

@james-johnston-thumbtack thanks for the report. I'll try to investigate it this week.


Add arm64 Visual Studio support

Add Arm64 support.  #193

Awesome ❤
Have you been able to verify this against VS2022 AMD64? If not I'll pull it locally and make sure it's good 👍

Yes, I verified that it functions correctly by running ARM version of Visual Studio 2022 17.5 on a Windows 11 VM running under Parallels Desktop 18 on my Mac Studio.


fix bug in hash computation for 2015-era badWhitespaceChainLinks

It turned out that after recent sigchain optimizations, we were using the sig payload to compute
chain link IDs. This is almost always correct, except for the ~30 links from badWhitespaceChainLinks.
In those cases, perform the "fix" operation which is just to strip off the trailing newline.

Is there time for a test?
Wasn't there already one of these links in the test vectors?

I tested it by hand... hmmm.... I'll work a little longer on it...
@patrickxb  what do you think about this second issue, that there's no reason to be rehashing the link IDs in the first place if we're reading out of cache. Do you think it would give us any CPU savings on long sigchains to skip the hash matching on a cache hit?

Ok, now the test suite exercises the fix for the bug...

Which line is the rehashing of link ids when coming out of storage?

Hmm, I see what you're saying. We aren't actually recomputing the hash, we're storing it on storage time. So that explains it why this behavior changed....


wrap resolved relation types with isbstorydata type

I'm not sure if its a good idea to include a dependency like storyblok-js-client. I'd rather like to keep a shallow copy of external types to keep future breaking change at a minimum. Also I've spent some time at that library and I am sure over the time there will be unexpected changes which I would like to avoid.

I'm not sure if its a good idea to include a dependency like storyblok-js-client. I'd rather like to keep a shallow copy of external types to keep future breaking change at a minimum. Also I've spent some time at that library and I am sure over the time there will be unexpected changes which I would like to avoid.

I replaced the dependency with a copy of the type, created with the type from the storyblok client package as a reference and compared to the actual API response. Was not possible to create it trough json-schema-to-typescript as generics are not yet supported


Attempt to fix a crash when tracking scrollview

To fix this issue with another approach : https://github.com/muukii/Rideau/pull/32

Just got the crash again :D Can we merge it ?

OMG
ok I'll merge it and then investigate it later.


Reduce 2 test datasets

Related: #531
WaterBottle.glb textures resolution have been reduced (365KB now)
dragon.vtu have been decimated (624KB now)

Should bluntfin.vts and palermo also been reduced ?


yew-router does not encode path parameters

Problem
When reading a path parameter from a route, yew-router does not decode escaped characters in the path segment, and passes them to the user as they are in the URL string.
When navigating via Link, path parameters are not encoded, leading to issues when path components have spaces, forward slashes, and other forbidden characters in them.
Steps To Reproduce
Example application:
use yew::prelude::*;
use yew_router::prelude::*;

#[derive(Clone, Routable, PartialEq)]
enum Route {
	#[at("/")]
	Root,
	#[at("/search/:query")]
	Search { query: String },
}

fn switch(route: Route) -> Html {
	match route {
		Route::Root => html!{
			<Link<Route> to={Route::Search { query: "foo bar baz".into() }}>
				{"Do search!"}
			</Link<Route>>
		},
		Route::Search { query } => html!{

{"You searched for: "}{query.clone()}

		},
	}
}

#[function_component]
fn App() -> Html {
	html!{
		<BrowserRouter>
			<Switch<Route> render={switch} />
		</BrowserRouter>
	}
}

fn main() {
	yew::Renderer::<App>::new().render();
}

Clicking the link navigates to Route::Search { query: "foo bar baz" }, whose handler just prints out the query. Clicking the URL goes to http://<IP_ADDRESS>:8080/search/foo bar baz which prints You searched for: foo%20bar%20baz.
Replacing the query with "foo/bar/baz" causes the link to go to http://<IP_ADDRESS>:8080/search/foo/bar/baz, which is an entirely different route.
Expected behavior
When creating a URL, yew-router should percent-encode characters forbidden in URL components. When parsing a route, yew-router should decode those percent-encoded characters before parsing.
Otherwise, it's quite unintuitive and inconvenient that Routes are not "round-trip safe", and that linking to one route may lead to a different route if the  path parameters contain slashes. This may also introduce a url injection vulnerability.
Environment:

Yew version: 0.20.0 (yew-router version 0.17.0)
Rust version: 1.66.1
Target, if relevant: wasm32-unknown-unknown
Build tool, if relevant: trunk
OS, if relevant: Not relevant
Browser and version, if relevant: Not relevant

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[X] I don't have time to fix this right now, but maybe later

I'm taking a look into this at the moment.

@ColonelThirtyTwo does #3187 seem to fix the bug for you?

@Jaffa-Cakes As the comments in that PR say, this only handles encoding. A fix for this issue would also need to decode the parameters when they are parsed.


Incomplete return value in DeviceV1_0::get_fence_status

Hello,
It seem to me that the current return value in DeviceV1_0::get_fence_status is incomplete compared to its "raw" equivalent DeviceFnV1_0.
The first one return VkResult<()> whereas the "raw" one return Result. We lose the information of the result value in the process which is not desirable due to the nature of the spec ( fence ), we usually want to do something like this:
let status = unsafe {
    self.state.device.get_fence_status(self.fence).expect("[ERR] Could not fetch fence status")
};

match status {
    vk::Result::SUCCESS => FenceStates::SIGNALED,
    vk::Result::NOT_READY => FenceStates::UNSIGNALED,
    vk::Result::ERROR_DEVICE_LOST => FenceStates::LOST,
     _ => FenceStates::UNKNOWN,
}

But currently, it is not possible. But replacing it with:
let status = unsafe {
    self.state.device.fp_v1_0().get_fence_status(self.state.device.handle(), self.fence)
};

Does allow to get the result wanted.
Faithfully yours,

VkResult<bool> might be appropriate here. Generally you'll be fine just checking is_ok(), though; if the device is lost you'll find out shortly regardless.

I think discriminating between your queue is still busy or all hell broke lose is still important :) . VkResult<bool> could fit the bill (provided that it is properly documented what are the Result equivalent).

Related: https://github.com/MaikKlein/ash/issues/41
https://docs.rs/ash/0.29.0/ash/version/trait.DeviceV1_0.html#method.get_event_status is a similar candidate where VkResult<bool> was used for indicating set/reset.


Grants for community building

I would love to drive more interest in the Denver meetups for @IPFS and @protoschool
We have had a few recent big turnouts to meetups and funds to foster and accelerate growth and support the present community are needed. (see #392 #395 )
Is there a way to apply for funding for community building officially? If not, are there plans for this?

@NukeManDan I’d like to better understand the need you see. Would you mind sharing some concrete examples of how grant funding would be spent and how this would translate into community growth?

@terichadbourne, thanks for the reply!
There are mostly direct (and a few indirect) types of costs that would be ideal to assist in:

Venue fees + rental equipment ( screes, projectors, whiteboards... )
Food + Supplies, Transportation + delivery
Marketing and promotion (flyers, ads,meetup.com fees  - and an easy to get on IPFS pro account)
Time and effort towards organizing, promoting, running, and producing media for and of the events

Organizers (like myself) run these events because they care, are passionate, and also get credibility/status in the community. But that only goes so far, especially when more than time is required to facilitate the events. That is why many seek official sponsors of events to help.
I think @IPFS should really consider at least a mechanism to request reimbursing direct costs of events. When all you are asking for is time, the barrier to entry for IPFS specific events goes way way down.  I suspect you will see a large increase in adoption of new groups with that mechanism in place! :grinning:

One possible way we may get outside funds into this effort is via donations that are then used to "bounty" meetups! I know the @gitcoinco team well, and think @protoschool would be an ideal fit for a gitcoin grant! Specifically here to fund production of tutorials from the community via bounties.
https://gitcoin.co/grants/
What do you think about that @terichadbourne?


Setting bot presence status to Online

Hi.
I don't know if this is still being maintained, but it happened to be useful for me (thanks for that).
I just wished the bot would show itself as Online while it is running.
Is that possible?
I'm on RC 5.4.2.
Thanks!

Joining in with this question. After upgrade of the RC server (from v.4.6) our hubot loses its online status. It just doesn't get updated. Looks like some change in driver <-> server communication. Any advice appreciated...


The ODM DataLoader does not upload all the data.

harish2rb[CodePlex] I have a file which has 530,782 records to be loaded into the DataValues table.
The ODM dataloader reads only a part of the data and then commits into the database.
On checking the Datavalues table, only part of the records are committed.

sreeder[CodePlex] Source Code updated. id: 50544
by sreeder
Sep 21 at2:03 PM
Update ODM DataLoader to gather data from larger excel files


Rename README.ms.txt to README.md

Change README filename to .md
This makes it easier to read the example

Thank you for your interest and for noticing this mistake !
It will be available in the next release (0.3.0) with automatic dependency injection :)


Why is XMLElement::_rootAttribute called "root"?

I cannot find any usage of "root attribute" term except this https://msdn.microsoft.com/en-us/library/system.xml.serialization.xmlrootattribute(v=vs.110).aspx Isn't it just "first attribute"?

It's just reflecting that the internal structure is a linked list - the API calls it FirstAttribute().

@leethomason Child elements are a linked list too but they use _firstChild. Is there any value in root for attributes?


Download PDF

You should do something with the formatting when downloaded to PDF.
When I download, all text comes normal, without the colors. Something that is very important.
It makes more difficult to read the code.
See for yourselves https://prnt.sc/kccpgm
So I had to manually copy-paste all docs, literally all them.

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

ID: fc2fe8f7-b023-5aa7-52c2-b9c1c5c13ded
Version Independent ID: 18c373d4-02f9-4cc6-656d-9cf4d11f6498
Content: C# Programming Guide
Content Source: docs/csharp/programming-guide/index.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for your feedback, @BelarminoVicenzo. @dend - Can you comment on this?

Given this is a general issue with docs.microsoft.com, I've moved it to https://github.com/MicrosoftDocs/feedback/issues/565, so it can get properly routed.
@dastaffo FYI


Differ between YAP and LOG information

General question: How do we differ between YAP and LOG information?
In the simulation this has been solved by routing the print() methods to the console, and the write() method to the SocketServer; which may not be ideal in the architecture, but allows for nice debugging.  In the target this is not possible, as both print and write are routed to the Serial interface.
Ideally the LOG information would be sent on its own YAP Data channel, meaning that the Logging Module should be able to write to the YAP Server somehow (only the App knows the YAP Server Instance, so it is not an ideal solution). And maybe only the INFO level Logs, not the DEBUG logs.
Other solution is turning Logs completely off on the target, and any information to be transmitted must be available for the App to read and re-route.
Originally posted by @gabryelreyes in https://github.com/BlueAndi/RadonAlcer/issues/2#issuecomment-1509939653

With #47 the logging will be disabled in case the socket server is started via command line. This is just a first step into this direction to make it easier for the user.


KCL-10481 - Properly track all necessary mutations when resolving com…

Abandoning in favor of https://github.com/kontent-ai/draft-js/pull/17 which should be more reliable


CA: Use openssl extensions instead of manually creating

Use openssl::X509::extension::* where available instead of manually building X509Extension.
Add ClientAuth to ExtendedKeyUsage

looks good (glad they finally got around to supporting these in rust-openssl), but I'd encourage you to test the results with zlint and other cert checkers (like the apple one on macs) to ensure they're compatible. I ran into some issues with this.

I think coyote still depends on a fork of rust-openssl too, you can probably eliminate that in Cargo.toml if this is supported by the core library now.

also, you're not going to want client auth for all certs. just the ones that specify client authentication. just making sure you're aware of this as it's a part of the ACME specification.
I'll be quiet now.

looks good (glad they finally got around to supporting these in rust-openssl), but I'd encourage you to test the results with zlint and other cert checkers (like the apple one on macs) to ensure they're compatible. I ran into some issues with this.

Recent openssl  (0.10.51) actually deprecates X509Extension::new() in favor of openssl::x509::extension::*.  (This change post-dates this PR)

I think coyote still depends on a fork of rust-openssl too, you can probably eliminate that in Cargo.toml if this is supported by the core library now.

It looks like Cargo.toml uses the stock openssl crate.

also, you're not going to want client auth for all certs. just the ones that specify client authentication. just making sure you're aware of this as it's a part of the ACME specification.

All of my letsencrypt certs are good for both Server and Client Auth,  and there's nothing in the CSR that would specify one way or the other.  Do you have a citation?  I can't find anything in RFC8555 one way or the other.

I guess there's no issue with setting extkeyauth to use clientauth. I was misremembering I suppose.
Still, check the certs with zlint and apple's cert validator. You're going to find that safari hates the certs this thing generates.
Have a nice day
------- Original Message -------
On Wednesday, April 26th, 2023 at 7:29 AM, Nathaniel Clark @.***> wrote:

looks good (glad they finally got around to supporting these in rust-openssl), but I'd encourage you to test the results with zlint and other cert checkers (like the apple one on macs) to ensure they're compatible. I ran into some issues with this.

Recent openssl (0.10.51) actually deprecates X509Extension::new() in favor of openssl::x509::extension::*. (This change post-dates this PR)

I think coyote still depends on a fork of rust-openssl too, you can probably eliminate that in Cargo.toml if this is supported by the core library now.

It looks like Cargo.toml uses the stock openssl crate.

also, you're not going to want client auth for all certs. just the ones that specify client authentication. just making sure you're aware of this as it's a part of the ACME specification.

All of my letsencrypt certs are good for both Server and Client Auth, and there's nothing in the CSR that would specify one way or the other. Do you have a citation? I can't find anything in RFC8555 one way or the other.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

I ran a generated cert through zlint and the only complaints were lack of "Authority Information Access" and "Certificate Policies"


Can't scroll down inside desktop app

I am not sure if this is a problem with the theme of the desktop app.  But when I use this theme, I can't scroll down inside the app.

Thanks, Fred. It's a problem with the new release, but I just committed a fix 👍
Hope it works now.


adding keerper security

I need help this does not worwk if  I add to the installomator script. Does anyone else have a working example?
keepersecurity)
name="Keeper Security"
type="dmg"
downloadURL="https://www.keepersecurity.com/desktop_electron/Darwin/KeeperSetup.dmg"
expectedTeamID="UBF8T346G9"
# Set blockingProcesses as appropriate for Keeper Security
blockingProcesses=( "Keeper Security" )
# Set forcefulQuit as appropriate for Keeper Security
forcefulQuit=YES
;;

I also have tried the following
name="Keeper Security" type="pkg" packageID="com.keepersecurity.mac.KeeperDesktop" downloadURL="https://keepersecurity.com/desktop_electron/Darwin/KeeperSetup.dmg" expectedTeamID="23A8QH23GX" blockingProcesses=( "Keeper" )

the app in the dmg archive is named "Keeper Password Manager.app"
You may want to specify appName in your label to define this if you plan to keep the name as "Keeper Security"
The team ID is also wrong, check out Wiki on how to obtain this.
forcefulQuit is unrecognized variable for installomator label therefore commented
https://github.com/Installomator/Installomator/wiki/Label-Variables-Reference
keepersecurity)
name="Keeper Security"
appName="Keeper Password Manager.app"
type="dmg"
downloadURL="https://www.keepersecurity.com/desktop_electron/Darwin/KeeperSetup.dmg"
expectedTeamID="234QNB7GCA"
# Set blockingProcesses as appropriate for Keeper Security
blockingProcesses=( "Keeper Password Manager" )
# Set forcefulQuit as appropriate for Keeper Security
# forcefulQuit=YES
;;

@asri-tm  thank you for the updated information, the installation script ran, but I was not able to find the application

I can confirm that now for a little while the provided section for Keepersecurity does work


Independent configurations for PageRootProvider

It corresponds to #1087 for AEM 6.2 and 6.3.

Coverage increased (+0.1%) to 29.48% when pulling d7d3de9f7e356814f63233847d80108624f43af8 on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

Hi, @davidjgonzalez. Sorry to bother you.
Do you have any idea why Travis had broken for this one?
I couldn't make head or tail of the last log message.
Thank you!

Overall the code looks good to me (though I have not tested it).  Considering the build was broken by documentation update, it might have just been a gremlin.
Reviewing the code, looks like some tab characters crept in - try converting those to spaces and committing, and see if the new build succeeds.

Thank you for reviewing the code, @HitmanInWis!
I've removed those weird extra tabs and I'm now waiting the build...

Coverage increased (+0.1%) to 29.48% when pulling d73f18b7cdd575c7b70989452b3d1832672522d1 on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

Coverage increased (+0.1%) to 29.48% when pulling d73f18b7cdd575c7b70989452b3d1832672522d1 on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

Coverage increased (+0.1%) to 29.48% when pulling 9095a238de0457b094be9616302972d8315ee6a3 on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

Coverage increased (+0.1%) to 29.48% when pulling a2dd6acc9966e1f96e211d3bc39f9696f152435e on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

Hi, @justinedelson
Do you have any idea about what could be the cause of the build error or how can I see the detailed error log?
Thanks!
Regards,
Daniel.

On the failing job page (https://travis-ci.org/Adobe-Consulting-Services/acs-aem-commons/jobs/270368586), there is a Raw Log link. In this case, since other jobs in the same run passed, I suspect it is errant and can be ignored.

Thanks for your reply, @justinedelson!
I had seen the raw log before, but I was expecting to find the surefire-reports or anything else with a more informative (or saner) messages than the Maven output...
Regarding the errant condition, I agree with you.
Thank you again.

@justinedelson @davidjgonzalez
QA'd this PR merged into latest master branch on AEM 6.2 and AEM 6.3

Tested with a fully functional demo site that depends on Page Root Provider
Tested an existing Page Root Provider config (now deprecated)
Tested deploying multiple Page Root Providers the new way
Tested mixing new Page Root Providers with the deprecated config
Tested adding/editing/removing Page Root Providers via Felix console

Everything works as expected.
In my opinion this is good to be merged.

@daniel-lima Please consider updating the documentation for PageRootProvider configuration of Shared Component Properties at https://github.com/Adobe-Consulting-Services/adobe-consulting-services.github.io/blob/master/_acs-aem-commons/features/shared-component-properties/index.md

@daniel-lima and @HitmanInWis - LGTM but can you do a quick change to make LOG -> log per the proj guidelines? Otherwise we can merge!

Coverage increased (+1.01%) to 30.376% when pulling 6e5c14e05ec062c5faa629e236dad2d4a835271a on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

Coverage increased (+1.01%) to 30.376% when pulling 6e5c14e05ec062c5faa629e236dad2d4a835271a on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

Coverage increased (+1.01%) to 30.376% when pulling 6e5c14e05ec062c5faa629e236dad2d4a835271a on daniel-lima:master into 9782ce30be1c72fe1017688db684c0ec3b4f5d09 on Adobe-Consulting-Services:master.

@HitmanInWis , thank you for testing everything.
Regarding the doc update, I was just waiting for the PR being approved and merged.
@davidjgonzalez , LOG renamed to log. Thanks!


Add link to matrix chat

closes #445
I chose to simply use vim.b.org_agenda_type after all, as it was way easier. If you think it obfuscates the logic too much, let me know.
edit: wait, I'll squash it myself

Sorry, completely botched this one somehow :man_facepalming:


PluginAdditionalOption.TextValue and other value properties are not persistent

Is it me or PluginAdditionalOption.TextValue and other value properties added in this PR  are not persistent?
I think defaultOption.TextValue = option.TextValue; and other assigments are missing.
// SettingsReader.cs

private static IEnumerable<PluginAdditionalOption> CombineAdditionalOptions(IEnumerable<PluginAdditionalOption> defaultAdditionalOptions, IEnumerable<PluginAdditionalOption> additionalOptions)
        {
            var defaultOptions = defaultAdditionalOptions.ToDictionary(x => x.Key);
            foreach (var option in additionalOptions)
            {
                if (option.Key != null && defaultOptions.TryGetValue(option.Key, out PluginAdditionalOption defaultOption))
                {
                    defaultOption.Value = option.Value;
                    // should add other value properties here
                }
            }

            return defaultOptions.Values;
        }

Originally posted by @waaverecords in https://github.com/microsoft/PowerToys/pull/28601#issuecomment-1783911338

@waaverecords
Can you please provide a poc of how to run into this bug.

Yes! @htcfreek
public class Main : IPlugin, ISettingProvider
{
    // ...

    IEnumerable<PluginAdditionalOption> ISettingProvider.AdditionalOptions => new List<PluginAdditionalOption>()
    {
        new PluginAdditionalOption
        {
            Key = "Textbox",
            DisplayLabel = "Textbox",
            PluginOptionType = PluginAdditionalOption.AdditionalOptionType.Textbox
        },
        new PluginAdditionalOption
        {
            Key = "Bool",
            DisplayLabel = "Bool"
            // PluginOptionType is Checkbox by default
        }
    };
}

Change the value of both settings.
Close PowerToys.
Open PowerToys.
Only the "Bool" option has its value persisted.
"Textbox" value is back to null.

@lin-ycv
I am sure this is the bug you hit with your initial settings implementation.

Yes, sounds like the same issue.
I didn't look too deep into it, basically just changed all the setting properties to public and use load to read the values.

Pretty sure only SettingsReader.CombineAdditionalOptions needs modifications.
I'll look into it and make a PR if it's the case.

Great. Then I will change assignment to you.

PR request -> https://github.com/microsoft/PowerToys/pull/29480

@jaimecbernardo
As the 0.75.0 release is feature closed I think we can consider to include this issue in a hot fix if we need one.

@jaimecbernardo
I have added the "Hot Fix" label. Hope you are okay with doing that.


Create an Upgrade Guide for RM3

We should document how the upgrade for RM3 will go:

Highlight what's new
Breaking changes and possible solutions (we may need a section for developers and another one for designers)
Encourage teams to work with a designer when in doubt

https://github.com/cengage/react-magma/wiki/Upgrade-Guide:-RM2-to-RM3


Issue with midis "Walls block ranged attacks" setting and simb's cover calculator

So, the situation:
I have a huge map (7000*9000px) The map contains far almost 2000 walls, and roughly 120 tokens. (I run highly automated sessions)
Whenever I have a token selected and then target another token, it takes serveral seconds, to "Page stopped responding" for the next midi step to take action.
I've checked the debugging window, with midis debuggin setting turned on and it clearly says that it it due to simb's cover calc.

(most of the Monster tokens it is calculating, are so far away, that they are not even part of the current fight.)
Each of this rows take between .5 and 2 seconds to apear. wich sums up, to the times i told previously.
I've done some tests and it seems to only occure, when I have the Setting
"Walls block ranged attacks" set to "simb's cover calculator", using the 4 corner method, from DND 5e.
This are my Midi Settings, if it helps, solving this problem. If further informations, or map-data is required, im here to help.
(Im a dev myself, i know the struggle of incomplete information :D)
fvtt-midi-qol-settings (2).json
Any other information about my world can be found in the MIDI Troubleshooting file, including version numbers and serveral settings.
fvtt-midi-qol-troubleshooter.json

Thanks for the detailed report.
Honestly, in this scenario I would just not use cover calculator.
2000 walls and 120 tokens is crazy (even if not using the calculator)
It will take long to calculate the cover as it's in effect doing a math calculation to see how many corner points of both tokens interesect with walls and other tokens. In order to do that I needs to calculate the position of the walls and tokens. Unfortunately it's not like game development where you just shoot a ray from one object to another.
Further to this, other than cover calculator, if you speak to any other user they would highly recommend not to run maps with so many walls and tokens. The amount of data that is processed with those amount of walls and tokens will cause it to run slow for most users, especially if they don't have a gaming rick and just a normal laptop or PC.


pydantic.mypy plugin error using pydantic_settings SettingsConfigDict

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
pydantic mypy crashes when providing a model_config SettingsConfigDict within the BaseSettings class. mypy continues to crash even after commenting out the model_config until the virtual environment is rebuilt.
Traceback:
$ mypy test.py --show-traceback
test.py:3: error: INTERNAL ERROR -- Please try using mypy master on GitHub:
https://mypy.readthedocs.io/en/stable/common_issues.html#using-a-development-mypy-build
Please report a bug at https://github.com/python/mypy/issues
version: 1.4.1
Traceback (most recent call last):
  File "mypy/semanal.py", line 6451, in accept
  File "mypy/nodes.py", line 1139, in accept
  File "mypy/semanal.py", line 1582, in visit_class_def
  File "mypy/semanal.py", line 1667, in analyze_class
  File "mypy/semanal.py", line 1694, in analyze_class_body_common
  File "mypy/semanal.py", line 1777, in apply_class_plugin_hooks
  File "[redacted]/.venv/lib64/python3.8/site-packages/pydantic/mypy.py", line 182, in _pydantic_model_class_maker_callback
    return transformer.transform()
  File "[redacted]/.venv/lib64/python3.8/site-packages/pydantic/mypy.py", line 443, in transform
    self.add_initializer(fields, config, is_settings)
  File "[redacted]/.venv/lib64/python3.8/site-packages/pydantic/mypy.py", line 804, in add_initializer
    base_settings_init_arguments = base_settings_info.names['__init__'].node.arguments
AttributeError: attribute 'names' of 'TypeInfo' undefined
test.py:3: : note: use --pdb to drop into pdb

When running the command with pdb to print the base_settings_info we get the following:
-> base_settings_init_arguments = base_settings_info.names['__init__'].node.arguments
(Pdb) base_settings_info
*** AttributeError: attribute '_fullname' of 'TypeInfo' undefined

Requirements to reproduce:
annotated-types==0.5.0
mypy==1.4.1
mypy-extensions==1.0.0
pydantic==2.1.1
pydantic-core==2.4.0
pydantic-settings==2.0.2
python-dotenv==1.0.0
tomli==2.0.1
typing-extensions==4.7.1

mypy config:
[tool.mypy]
files = ["api", "tests"]

plugins = [
    "pydantic.mypy"
]
strict = true

Example Code
from pydantic_settings import BaseSettings, SettingsConfigDict

class Settings(BaseSettings):
    API_HOST: str = "<IP_ADDRESS>"

    model_config = SettingsConfigDict(
        env_file=".env", env_file_encoding="utf-8"
    )

Python, Pydantic & OS Version
pydantic version: 2.1.1
        pydantic-core version: 2.4.0
          pydantic-core build: profile=release pgo=true mimalloc=true
                 install path: [REDACTED]/.venv/lib/python3.8/site-packages/pydantic
               python version: 3.8.6 (default, Oct 27 2020, 09:13:12)  [GCC 9.3.1 20200408 (Red Hat 9.3.1-2)]
                     platform: Linux-3.10.0-1160.15.2.el7.x86_64-x86_64-with-glibc2.2.5
     optional deps. installed: ['typing-extensions']

Selected Assignee: @Kludex

I've tried your provided code but didn't get the mypy error you've reported.
maybe it's something related to your environment

Hello 👋 ,
I have the same issue on my project with mypy and
"pydantic[email]==2.1.1",
 "pydantic-settings==2.0.2"

on Python 3.11.4.
The first time I run myapp it works fine but when I relaunch several times the error appears:
` base_settings_init_arguments = base_settings_info.names['__init__'].node.arguments
                                   ^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: attribute 'names' of 'TypeInfo' undefined
`

It also happens when I restart it multiple times (usually when there’s a modification to the file from what I’ve noticed)

@JeanArhancet Thanks for replying and creating the PR.
But I still couldn't reproduce the problem with
annotated-types==0.5.0
dnspython==2.4.1
email-validator==2.0.0.post2
idna==3.4
mypy==1.4.1
mypy-extensions==1.0.0
pydantic==2.1.1
pydantic-settings==2.0.2
pydantic_core==2.4.0
python-dotenv==1.0.0
tomli==2.0.1
typing_extensions==4.7.1

@dmontagu Do you have any idea here?

@hramezani In my PR I also reproduced the issue by adding additional test (https://github.com/pydantic/pydantic/pull/7002/files#diff-a3996a50e4d75365ab9f9aa132a8e25d8f5fa445d9cb10d215b6e31f08d24606R13-R21)
If you run this tests, do you get an error?

I can confirm that I am hit by this bug also and @JeanArhancet's PR fix it

Thank you @JeanArhancet for the changes. Much appreciated


Plugin Database Update

Update plugin database before installing Jenkins plugins (eliminates build failures).  See the following for additional information -

http://www.mymiller.name/wordpress/app/1515/
https://www.jeffgeerling.com/blog/2017/ansible-jenkinsplugin-module-and-404-plugin-not-found

Haha, I can't believe I kept getting hit by my own bug. Thanks @robbinscp !

Hmm, @geerlingguy @robbinscp I'm getting 404s during plugin install and using version 4.3.0 of this role. Did this fix go away or change in the last 2.5 years?
I found this variable which seems to default to a long time between updating plugin cache:
jenkins_plugin_updates_expiration: 86400
I set it to zero and am still getting 404 errors. I know this will go away if I wait a day or two, but I'd like to get a persistent solution to it. Can you suggest how I can configure this this role to prevent 404s? I am not concerned with it taking longer to run.
Thanks.

@timblaktu - I run a CI job that tests this role daily, and that job fails due to random 404s and 50x errors quite frequently. Unfortunately, the Jenkins plugin mirrors are historically unreliable. For a time, I was also building a Jenkins docker image that would install a couple plugins on a CI job, and that, too, would fail here and there for hours at a time. Then, mysteriously, it would work the next day... or three days later. Then it would break again in a week or two. Then work again.
That's kind of the status quo for Jenkins plugin installs, unfortunately.

Thanks @geerlingguy, I will check again in a few days and hope it starts working.
Now I just gotta figure out why none of my plugin dependencies are being installed.. I'll raise another issue. I know there's been several related issues on this, but now I'm seeing this problem even with jenkins_plugins_install_dependencies: true and jenkins_plugins_state: present and no plugin versions specified. Not sure why the dependencies aren't being installed. It's nothing to do with the 404 failure, bc I've commented our the failing plugin, and have done a complete re-provision from fresh OS install bootstrap then run my jenkins.yml playbook which includes your role..

@geerlingguy, what about 17 days? locks-and-latches plugin is still giving me 404 errors. I'm going to have to stop using this method (ansible jenkins_plugin module) for plugin installation if it's this unreliable. It would seem that all that's required is to place each plugin's .hpi file into JENKINS_HOME/plugins and restart. Perhaps I could just create my own jenkins plugin mirror, or just create a local cache of these hpi files that I need? Do you know of or have you experimented with alternatives?


Launch containers

Describe the results you received:
Part of containers failed to launch with error message
Error: no available IPv6 addresses on this network's address pools

Describe the results you expected:
Successful launch
Additional information you deem important (e.g. issue happens only occasionally):
docker ps hangs. After command sudo systemctl restart docker the docker daemon becomes a zombie process and only reboot helps to restore docker 'working'.
docker network inspect bridge sometime works, sometime gives error: "An error occurred trying to connect: Get http:///var/run/docker.sock/v1.21/networks/bridge: EOFnull"

Likely a duplicate of #18527

@alpo this issue should be resolved in docker 1.10.x and above. Can you test if you're still able to reproduce this on docker 1.10 or 1.11?

@thaJeztah After migration to CentOS 7 with  docker 1.9.1 and CoreOS alpha 1010.1.0 with docker 1.10.3 no more weird messages and deadlocks in both cases.

Closing this based on the above comment.


[incubator/fluentd-graylog]

What this PR does / why we need it:
This PR adds a new chart to the incubator designed to push kubernetes logs to graylog2 using fluentd and gelf plugin. It's adopts the already existing and maintained docker image from fluent/fluentd-kubernetes-daemonset.
This initial version of the chart is created by doing the minimal changes to the helm templates in incubator/fluentd-cloudwatch as it uses the same base docker images.

Hi @scottrigby is there anything more I need to do to get this reviewed? Please let me know.

@lachie83


BasePath processing is missing

According to the documentation
The PSR-7 Request object’s URI is itself an object that provides the following methods to inspect the HTTP request’s URL parts:

getScheme()
getAuthority()
getUserInfo()
getHost()
getPort()
getPath()
getBasePath()
getQuery() (returns the full query string, e.g. a=1&b=2)
getFragment()
getBaseUrl()
You can get the query parameters as an associative array on the Request object using getQueryParams().

Base Path
If your Slim application's front-controller lives in a physical subdirectory beneath your document root directory, you can fetch the HTTP request's physical base path (relative to the document root) with the Uri object's getBasePath() method. This will be an empty string if the Slim application is installed in the document root's top-most directory.

Slim v3 implemented the basePath processing as per https://github.com/slimphp/Slim/blob/3.x/Slim/Http/Uri.php#L204

It handled base path by making assumptions with REQUEST_URI which is unreliable and that’s why it was removed.

Slim-Psr7 is intended to be a "pure" implementation of PSR-7. As such additions such as these properly belong in Slim-Http, where getBaseUrl() already exists.


Two containers?

After a while of running, suddenly I have two containers of CrashPlan Pro running. How do I determine which one is primary and delete the duplicate one?
Screenshot attached. (note the different RAM usage)

Thanks in advance for any help.

Anybody?
Should I shut down/delete the one (..n-PRO1) showing less memory and CPU usage?

Why not stop it first - and see if everything works as normal before you delete it?

You should check the config of each instance and check which one backup the right set of files.  Do both instances are logged in to CrashPlan ?

Closing this issue.  Please re-open if needed.


Expired Slack invite URL

The slack invite link in the README has expired: https://semaphorecommunity.slack.com/join/shared_invite/enQtMzk1MzI5NjE4MjI5LWY3Nzk4ZGM2ODRmMDVjYmIwZGFhMWI0ZDYyOWIxMGI1ZjFlODU1OTZiZWM3OGVkZjBmMWRiNWYzNjA4MjM2MTA

Thank you for letting us know Jake, this has been fixed and the README link has been updated: https://join.slack.com/t/semaphorecommunity/shared_invite/enQtMzk1MzI5NjE4MjI5LTg3MDcyNTg0ZWFjNTI1Y2U1NGEyZDBlNjhlZGM0ZjcxMmMxZDY5ZGYzM2RkM2VhNzgzNzI3YTk5ZDIzZjdhNGM


Add queue support for phpnats

Currently phpnats supports Pub/Sub & Request/Response patterns to interact with NATs server.
Implement queue support according the docs here:
http://nats.io/documentation/concepts/nats-queueing/

Nats protocol documentation:
SUB
Syntax
SUB <subject> [queue group] <sid>\r\n

Where:

subject: The subject name to subscribe to
queue group: If specified, the subscriber will join this queue group
sid: A unique alphanumeric subscription ID

Description
SUB initiates a subscription to a subject, optionally joining a distributed queue group.
Example
To subscribe to the subject FOO with the connection-unique subject identifier (sid) 1:
SUB FOO 1\r\n

To subscribe the current connection to the subject BAR as part of distribution queue group G1 with sid 44:
SUB BAR G1 44\r\n

Protocol docs: http://nats.io/documentation/internals/nats-protocol/

@adriacidre :muscle: !!! Thanks for the complementary info.


Git file generator should support {{path.basename.parentfolder}}

It would be useful to be able to use {{path.basename.parentfolder}} template variable in git-files generator, to get parent folder name of chart deployed. Assuming one has the below folder structure one can easily obtain the parent folder while running the application set which can be used for the namespace variable in the destination.
apps
└─ namespace-name
        └─ app 1
        └─ app 2
        └─ app 3

eg 1:
path: kube-infra/dns-infra/*
path.basename.parentfolder: dns-infra
eg 2:
path: kube-infra/database/*
path.basename.parentfolder: database
applicationset.yaml below
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
name: kube-infra-setup
namespace: ci-cd
spec:
generators:

matrix:
generators:

git:
repoURL: git@github.com:xxxx/k8s.git
revision: HEAD
directories:

path: kube-infra/dns-infra/*
path: kube-infra/database/*
exclude:

list:
elements:

cluster: dev
url: https://xxx.xxx.xxx.xxx
template:
metadata:
name: '{{ path.basename }}'
spec:
project: xxxx
source:
repoURL: git@github.com:xxxx/k8s.git
targetRevision: HEAD
path: '{{path}}'
helm:
valueFiles:

'{{cluster}}-{{path.basename}}-values.yaml'
destination:
server: '{{url}}'
namespace: '{{path.basename.parentfolder}}'
syncPolicy:
automated:
prune: true
selfHeal: true
syncOptions:

CreateNamespace=true

@jgwest  i was planning to write code for this if this gets approved, please let me know your thoughts on it.

Thanks @jgwest that looks good, sorry for the delayed reply


Source from https://github.com/vscode-langservers?

Extracted language servers are maintained at https://github.com/vscode-langservers. Why not build off of those instead of building all of VS Code?

You are right.
The current approach is not efficient.
But if we want to extract other extensions from vscode, it will convinient.


Support for reading marathon location from zookeeper

@paulmach @mlerner any basic golang thoughts on this?

@paulmach ptal at very last commit for fixes


Bumping testcontainers and scalatest versions

Bumped testcontainers to latest 1.10.1
Bumped scalatest to latest 3.0.5

@scoquelin thank you!