Cannot import name 'munchify'

Hi,
I am trying to run the basic examples in a Jupyter notebook (Python3), but I get this error:
ImportError: cannot import name 'munchify' from 'trading_ig.utils' (/usr/local/lib/python3.7/dist-packages/trading_ig/utils.py)
I tried installing munch via pip, but that made no difference.
Thanks in advance,
Erik

I have the same issue.
Interestingly, the offending line from munch import munchify seems to work fine on my command line too.

I think I found the issue and have it resolved.
In my project I need to install all the extra packages with pip install trading_ig pandas tenacity munch or poetry add trading_ig pandas tenacity munch.  Thereafter when spinning up the juptyer notebook fresh and rerunning, the previous error message is removed.
I think the issue is that in the pyproject.toml file the dependencies for pandas, tenacity, and munch are labelled as optional=true.  I'm sure there are reasons for this, but it does mean that as a user of this package, I need to install those separately.
This is something that could be fixed?
From pyproject.toml...
[tool.poetry.dependencies]
python = ">=3.6.1,<4.0"
requests = "^2.24"
pycryptodome = "^3.9"
requests-cache = "^0.5"
six = "^1.15"

pandas = { version = "^1", optional = true }
munch = { version = "^2.5", optional = true }
tenacity = { version = "^7.0", optional = true }
Sphinx = { version = "^4.0.3", optional = true }
sphinx-rtd-theme = { version = "^0.5.2", optional = true }

Thos imports are optional by design
I have created a new task (#216) to improve the error handling and docs for the optional imports


profile-card-DevCode24

@Dev-Code24 Thank you for your valuable contribution. As I found, they(Hacktoberfest) are verifying the code in the PR so I recommend that you add your code to a "code folder" as well to avoid any consequences (your code should be in a folder named after you). Sincerely apologize for the inconvenience.

@DiyaVj
ok, so I just have to put the code of my profile card into a folder and then request a PR instead of putting the link in the readme.md, right?


[FEATURE] ESC key should cancel editing

Description
There's a cancel button for canceling editing, but on devices that uses a physical keyboard, having key shortcut for this will make it easier to use.
Related to #186
Screenshots
No response

It's great having you contribute to this project by creating an issue Make sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community :nerd_face:

Just found this project, cool stuff! Going to start working on this, assignment is appreciated.

Just found this project, cool stuff! Going to start working on this, assignment is appreciated.

okay, thanks for wanting to contribute, Make sure you follow the rules 👉🏽here👈🏽, else your PR will not be accepted and will be closed.
Good Luck

Is it alright if I go on ahead and clear the input when ESC key is pressed during normal typing? It will cancel editing and clear the input if isEditing  otherwise it will just clear the input. Both will set typing to false.
I think that's standard behaviour.

Is it alright if I go on ahead and clear the input when ESC key is pressed during normal typing? It will cancel editing and clear the input if isEditing  otherwise it will just clear the input. Both will set typing to false.
I think that's standard behaviour.

No stick with the issue above, if you want that as an feature, create a separate issue🙌🏽


django-sekizai does not supported in django-cms>=3.6.0

Summary
Django-cms 3.6.0 requires django-sekizai>=0.7, but last published version of sekizai is 0.10.0, which does not have support for django>=1.11.
As we see this package in under your development and it is required package for django-cms.
@timgraham added support for django>=1.11 in this pull request https://github.com/divio/django-sekizai/pull/79.
Please, revise pull request and merge it.

https://github.com/divio/django-cms/blob/eeb1e4712b3866e243daf800c142e2199e4be9df/setup.py#L32

The new version of django-sekizai is published.


Lock down the access to the member variables in ADContext.

This PR locks down the internals of ADContext via appropriate API interface. This would be important to hide the details of ADContext when we move this out to a separate file.
(This PR is a part of https://bugs.swift.org/browse/TF-993)

@swift-ci please test tensorflow

Addressed the review comments. PTAL.

@swift-ci please test tensorflow

Addressed the review comments. PTAL.

To save our time, I added a NFC commit: https://github.com/apple/swift/pull/28567/commits/af9f0178444928a936739d5726e0828615ea03b2. Hope that's fine!

Addressed the review comments. PTAL.

To save our time, I added a NFC commit: af9f017. Hope that's fine!

Of course, thanks!

@swift-ci please test tensorflow


could I check the symbolic derivative equation of my residual function?

Hello, thanks for this great nonlinear optimization library.
I want to check the result of derivated equation of my residual function to debug it step by step.
Is it possible to see the results from symengine?
in case of the rotation parameters, does symengine derive by angle-axis paramterization?
my jacobian values by angle-axis parameterization and the jacobian values by symengine look quite different.

does jacobian_helpers.py related to the equation?
https://github.com/symforce-org/symforce/blob/main/symforce/jacobian_helpers.py

Yeah you can look at the symbolic expressions for the jacobians - assuming you're getting jacobians using Codegen.with_linearization or Codegen.with_jacobians, yes those call jacobian_helpers.tangent_jacobians internally.  You should be able to get the equivalent expression by calling tangent_jacobians on your expression yourself.  You could also look at the outputs field on the Codegen object returned from with_linearization or with_jacobians, which should include the symbolic expressions for any jacobians added by those methods.
The jacobians are computed with respect to the SO(3) tangent space (technically the coordinate space) at the input element - depending on what exactly you mean by the angle-axis parameterization, what you may be seeing is the difference between what we're computing:
$$\left. \frac{\partial}{\partial v} \left[ r(R.\mathrm{retract}(v)) \right] \right|_{v=0}$$
and this:
$$\left. \frac{\partial}{\partial v} \left[ r(\mathrm{from\_tangent}(\mathrm{to\_tangent}(R) + v)) \right] \right|_{v=0}$$

great! thanks for your answer!
I'll look into the details of jacobian_helpers.tangent_jacobians function.
thanks!


Do not allow users to create orphaned books

Describe the feature you'd like
Within Settings, include an option which forces non-admin users to choose a shelf for any new books they create, i.e. choosing an existing shelf is a mandatory step when a non-admin user creates a new book.
Describe the benefits this feature would bring to BookStack users
In a set-up where each non-admin user belongs to a role which only allows access to those shelves relevant to their team,
If you give a users permission to create books, you also give them permission to create books that are not associated with any shelf (and which their team members can't see), i.e. orphaned. Even if the book is moved into a shelf at a later time, it will not automatically have the right permissions. However, if it was mandatory for non-admin users to associate a new book with a shelf when it is first created, no books would be orphaned and team mates would automatically gain access. A setting to remove the 'Books' navigation link on the top menu bar for non-admin users would have the same practical effect.
Additional context
This is relevant where there are a large number of users divided into teams who don't want or need to see information related to other teams and where a strategy of a shelf per team/project has been implemented but where it is still desirable to allow any team member to create a new book within the shelves available to them. If there is a way of achieving this already, I apologise for not having been able to find it!

Thanks for the suggestion @thesik01.
To be honest I try not to introduce options/configuration for edge-case or specific workflows where possible.  Options around this relationship also get a little dicey since the book/shelf relation is more complex that your request accounts for since books can be assigned to mulitple shelves. Do know if you'd then desire further controls to limit that.

Even if the book is moved into a shelf at a later time, it will not automatically have the right permissions. However, if it was mandatory for non-admin users to associate a new book with a shelf when it is first created, no books would be orphaned and team mates would automatically gain access.

This is not how the permissions work at this time. Shelf permissions do not auto-cascade to books.

A setting to remove the 'Books' navigation link on the top menu bar for non-admin users would have the same practical effect.

Within the current release, you could achieve this via adding the following to the "Custom HTML Head Content" setting:
<style>
header .header-links a[href$="/books"] {
    display: none;
}
</style>

The overall request here very much overlaps with this existing issue #1171 so I'm going to therefore close this off as a duplicate.


Error: curl https://bitbucket.org/pypa/setuptools/raw/bootstrap/ez_setup.py | python returned 1 instead of one of [0]

Hi,
I just tried to install supervisord with the default config on a fresh debian jessie as well as ubuntu 14.04 and with both OS get the same error message:
curl https://bitbucket.org/pypa/setuptools/raw/bootstrap/ez_setup.py | python returned 1 instead of one of [0]
Looks like when I execute the command with sudo privileges directly in the console everything works fine. Anyone knows if this is a bug or am I doing something wrong?
Kind regards
hasc

I had the same issue today(same OS etc.).  However, running with sudo privileges didn't fix it for me. I poked around a bit and I think the canonical location of the ez_setup.py script has changed recently.  I updated line 101 in the params.pp file to:
$setuptools_url          = 'https://bootstrap.pypa.io/ez_setup.py'
This seems to have gotten me past that issue(and supervisord is running). YMMV

I ran into the same issue and was able to resolve it by setting that setuptools_url parameter to 'https://bootstrap.pypa.io/ez_setup.py' in my declaration of the supervisord class.  If the canonical location has indeed changed, it makes sense to change the default in params.pp.

Submitted a PR for this change:
https://github.com/ajcrowe/puppet-supervisord/pull/95

Fixed in #95


MERC-111 - Fix text cutoff in search dropdown

MERC-111 - Fix text cutoff in search dropdown

Unset height rule for the dropdown search bar, the results with going with the default height instead of being explicitly sized.

Before using Firefox

After using Firefox

@bc-chris-roper @bc-miko-ademagic

:+1:

I'm cool with this, but make sure that it's tested in all major browsers including IE. I have a feeling that it would've obviously been set for a reason, and typically form elements are annoying little dudes cross-browser. One of the browsers might need an explicit height set.

I tested on Chrome, Safari, and Firefox... I don't currently have IE but maybe @bc-arun can test?

What that issue looks like based on the screenshot isn't that the height is the problem on its own, but that the specified height combined with the font-size and padding of the box doesn't leave enough space for the text.
So another approach to fix it would be to just increase the height to what it ideally should be.

@mickr I verified it on IE 13 & 12. Looks good.

@bc-arun can you give this the green heart?

:green_heart:


ImportError: dynamic module does not define module export function (PyInit_bbox)

Hi, @roytseng-tw
I encounter the import problems (python3.5), need help. Thanks.
$ python3 tools/train_net_step.py --dataset coco2017 --cfg configs/e2e_mask_rcnn_R-50-C4.yml --use_tfboard --bs 4 --nw 4

Traceback (most recent call last):
  File "tools/train_net_step.py", line 25, in <module>
    from datasets.roidb import combined_roidb_for_training
  File "/home/yuekaiyu/code/Detectron.pytorch/lib/datasets/roidb.py", line 27, in <module>
    import utils.boxes as box_utils
  File "/home/yuekaiyu/code/Detectron.pytorch/lib/utils/boxes.py", line 52, in <module>
    import utils.bbox as cython_bbox
ImportError: dynamic module does not define module export function (PyInit_bbox)

Hi @KaiyuYue ,
were you be able to find the solution to this problem? I am having similar issue currently.
Thanks

I figured it out, the solution that worked for me was to make sure the  {root_dir}/lib/setup.py  is run by python3, also the names of .pyx files in the {root_dir}/lib/utils directory should be the same as the ones in the setup.py


Move grunt tasks to devDependencies

@listepo will this work in every case? devDependencies are not always installed, as in the case of recursive dependencies, and since sails-generate-new is a dependency of sails-generate, it seems like this might break normal usage of Sails.js

What is main idea of this PR ?
Why rc, sails, sails-disk need to be moved into production dependencies they need only for development ?


Make sure data are in order before displaying them

If we don't do this, the plotting library draws "backwards" lines which looks ugly.
Closes #16

Thanks!


Added section on JAR guidance and included new parameters in the table.

Why
At present, there is functionality available that allows the use of  JAR which makes sending requests to the /authorize endpoint more secure.
What
The suggested changes:

add a section to explain how you can use JAR to make the requests
add additional parameters to the table that summarises the request parameters

Jira ticket:  Jira ticket DTW-72 JAR guidance
The original reviews has been done here: https://docs.google.com/document/d/11b4iv9b3asimeRsaKs4w3DwMEVBWXYIDjvPToxd3KIQ/edit
Technical writer support
The reviews have been done by

Nathan Driver 14/11/23 (pre-i)
Christina Kyriakou 17/11/23 (2i)

How to review
Tell reviewers how to assess your changes.

This PR supersedes PR-136.


[FEATURE] parse file for list of changes attached to promotion record

sample code will be provided by nahuel

Duplicate of #35


treating '001' and '1' as same key

I tried to put data with key '001' and then '1'. When I try to get using same keys it always return same data which is inserted first.

as strings or as numbers? because as numbers they are the same.
> table.put('001', 'double oh one')
HashTable {}
> table.keys()
[ '001' ]
> table.put('1', 'just one')
HashTable {}
> table.keys()
[ '1', '001' ]
> table.get('1')
'just one'
> table.get('001')
'double oh one'
> table.get(1)
'just one'
> table.get(001)
'just one'
>

when you use primitive values (string, boolean, number) for the key, it uses the string representation; non-primitive values (everything else, descendent of Object) for the key, it uses the identity hash (built in to node, accessible from native code, a unique integer).
If you need to differentiate between '1' and 1, I suggest you use the package es6-native-map; it is based off of this package and adheres more to the ES6 Map standard. Here's an example:
> Map = require('./')
[Function: NodeMap]
> m = new Map
NodeMap { size: 0 }
> m.set(1, 'integer one')
NodeMap { 1 => 'integer one', size: 1 }
> m.set('1', 'string one')
NodeMap { '1' => 'string one', 1 => 'integer one', size: 2 }
> m.get(1)
'integer one'
> m.get('1')
'string one'
>

es6-native-map is on npm at https://www.npmjs.com/package/es6-native-map and on github at https://github.com/chad3814/es6-native-map

Thanks. For your quick response. I apologize to raise the issue before debugging myself. I actually was calling the hashtable incorrectly. Hence the issue occurred. Many thanks for your quick response.


Explanations are being truncated oddly

I suspect that this is a result of some code in reasoner.py that was dealing with a library(scasp) peculiarity that has been changed since they fixed #405.

This was resolved in 1.4.0-alpha


Clients do not gracefully handle server faults

When the server runs into halting exceptions, clients do not gracefully handle reconnection.
Video
Check discord: https://discord.com/channels/701873003679449108/701876824732598414/885877307665952809
Expected Behavior
When the server is restarted, existing clients should not experience desync.
Actual Behavior
After server failure, clients remain out-of-sync, leading to strange behavior.

The Heartbeat system in the upcoming rewrite of WorldQL has fixed this bug. Thanks for your report.


BaseModel with attribute "self" throws TypeError

Bug
For bugs/questions:

OS: Fedora 29
Python version: 3.7.2
Pydantic version: 0.21

from pydantic import BaseModel

d = {
    "self": 1,
}

class Model(BaseModel):
    self: int

Model.parse_obj(d)  # TypeError: __init__() got multiple values for argument 'self'

I'm not certain if this would be complex to support, but would be nice if models with self attributes would be possible.

probably possible if parse_obj was modified to not use __init__.

fixed in v1.


testing env needs to be able to read deployment storage but doesnt have write permission

if i try to use deployment storage in testing env for regular storage server, I get error when trying to create new book. I need a special read-only storage server for testing so i can read the delpoyment storage to tets migration.

just add a separate setting for production storage location


123377_ds - Thompson Creek surveyor efficiency sites

No action required, just documenting how we dealt with the revisited sites.
Thompson creek downstream sites (123377_ds_ef1, 123377_ds_ef2, 123377_ds_ef3) were revisited to estimate surveyor efficiency. These sites were enclosed, sample, fish tagged and then returned to the site (sept 14). 24h later (sept 15) the site was revisited, sampled, and the fish were scanned for pit tags to determine surveyor efficiency.
In the habitat_confirmations spreadsheet, 123377_ds_ef4, 123377_ds_ef5, 123377_ds_ef6 reference the revisited sites. We made separate reference numbers and site names for the revisited sites because they have a different sample dates as well as their own temp, cond, and turb data.
For example, 123377_ds_ef6 is the same site at 123377_ds_ef3. The site was enclosed, sampled, and the fish returned (123377_ds_ef3). 24 hours later the site was resample and the fish were scanned for pit tags to determine surveyor efficiency (123377_ds_ef6).
123377_ds_ef4, 123377_ds_ef5, 123377_ds_ef6 were added to the bottom of the spreadsheet so the reference number order was not changed.
Previously, in step 3 (Individual fish data), the different sampling days were differentiated by haul pass = 1 (sept 14) and haul pass = 2 (sept 15), but multiple haul passes are usually used to show that a site is clear of the fish (salvaging) so this was not the best way to document the data.  We changed 123377_ds_ef1, 123377_ds_ef2, 123377_ds_ef3 rows which had a haul pass = 2 to 123377_ds_ef4, 123377_ds_ef5, 123377_ds_ef6 respectively, now with haul pass = 1.

same thread here ? https://github.com/NewGraphEnvironment/fish_passage_skeena_2023_reporting/issues/85


Increase grpc buffer size

What this PR does / why we need it

Increases grpc buffer size

Checklist

[x] Update changelog in CHANGELOG.md.

For my curiosity -- why was this needed?


feat(validation) add uniqueness constraints to Credentials and ConsumerGroups of KongConsumers

What this PR does / why we need it:

Check uniqueness of items in Credentials and ConsumerGroups in KongConsumers because they are reference to other resource and same credentials/consumer groups will cause invalid Kong configurations.
Which issue this PR fixes:

fixes #5402
Special notes for your reviewer:

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Was this intended to be handled differently? The description suggests this should be a CRD-level CEL check on unique entries, but the actual code is in the admission controller.
I don't see a standard CEL utility for ensuring the uniqueness of list entries, but it looks like it's possible along the lines of https://github.com/kubernetes-sigs/gateway-api/pull/2370/files

Updated the description of the PR. The original intention was to add UniqueItems CEL on CRD, but the UniqueItems constraint is removed. So we have to move the constraints to the validation webhooks.


Twitter

schedule tweets on buffer

Feedback as of 8/18

for Monday the 22nd. I love that you're trying to integrate CARE principles, but CARE principles are all about Indigenous data sovereignty and governance, so they don't really match the quote. Is this tweet from the excel sheet of tweets? if so it definitely needs to be deleted!
Tuesday August 23rd: I've actually already tweeted this! Forgot to state that in the excel, sorry!
August 24th the page is from 2018, so maybe we make this a throwback thursday instead?
Friday August 26th, add a sunny arctic picture!
Monday the 29th, take away "research" in Arctic Research Center Data Science Team and just say Arctic Data Center team and tag @nceas_ucsb

@rzahabi There are two tweets scheduled for throwback thursday this week!

love August 30ths tweet!! so good! can you also add something about the course accepting applications next month?

three scheduled tweet look great. I think we should postpone the Friday one, and tweet about the course instead. the monday tweet about the course had an issue. and we need to tag a bunch of the organizations that we work with - not sure we can use buffer for that

accounts to tag found here  under the section: Accounts to tag for general retweets - also retweet these accounts when relevant


Inquiry: missing Language node data?

@szarnyasg
Hi, I generated a snb dataset using the following command:
./tools/run.py --parallelism 1 -- --format csv --scale-factor 0.003 --explode-edges --explode-attrs

I found a directory Person_speaks_Language in the generated dataset which might be used to contain edges connecting Person to the Language(languages a person speaks), but the problem is that I cannot find the Language node in the generated dataset:
./ldbc_snb_datagen_spark/out/graphs/csv/raw/singular-projected-fk [main]
% ls -lt static
total 0
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Organisation_isLocatedIn_Place
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Organisation
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Place_isPartOf_Place
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Place
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Tag_hasType_TagClass
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Tag
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 TagClass_isSubclassOf_TagClass
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 TagClass
./ldbc_snb_datagen_spark/out/graphs/csv/raw/singular-projected-fk [main]
% ls -lt dynamic
total 0
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Forum_hasModerator_Person
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Forum_hasTag_Tag
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Forum_hasMember_Person
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Forum
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Post_hasTag_Tag
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Forum_containerOf_Post
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Post_isLocatedIn_Country
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Post_hasCreator_Person
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_likes_Post
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Post
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_speaks_Language
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_email_EmailAddress
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Comment_isLocatedIn_Country
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Comment_replyOf_Post
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Comment_hasCreator_Person
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Comment_hasTag_Tag
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Comment_replyOf_Comment
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_likes_Comment
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Comment
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_workAt_Company
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_studyAt_University
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_isLocatedIn_City
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_hasInterest_Tag
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person_knows_Person
drwxr-xr-x  6 wenlinwu  staff  192 Aug  6 13:16 Person
./ldbc_snb_datagen_spark/out/graphs/csv/raw/singular-projected-fk [main]
% head dynamic/Person_speaks_Language/part-00000-21ea6ae4-b46e-41f9-8656-1275a34d91e4-c000.csv
creationDate|deletionDate|explicitlyDeleted|PersonId|LanguageId
2010-01-03T15:10:31.499+00:00|2019-12-30T00:00:00.000+00:00|false|14|fa
2010-01-03T15:10:31.499+00:00|2019-12-30T00:00:00.000+00:00|false|14|ku
2010-01-03T15:10:31.499+00:00|2019-12-30T00:00:00.000+00:00|false|14|en
2010-01-31T13:13:03.929+00:00|2019-12-30T00:00:00.000+00:00|false|16|pl
2010-01-31T13:13:03.929+00:00|2019-12-30T00:00:00.000+00:00|false|16|en
2010-02-12T22:05:24.513+00:00|2019-12-30T00:00:00.000+00:00|false|32|es
2010-02-12T22:05:24.513+00:00|2019-12-30T00:00:00.000+00:00|false|32|en
2010-03-24T18:14:04.882+00:00|2019-12-30T00:00:00.000+00:00|false|2199023255557|my
2010-03-24T18:14:04.882+00:00|2019-12-30T00:00:00.000+00:00|false|2199023255557|en

Hi, there is no language node: the language is encoded in a two letter code following ISO 639-1.

Hi, there is no language node: the language is encoded in a two letter string following ISO 639-1 codes.

@szarnyasg Sorry but I don't quiet get it. I thought Person_speaks_Language is an edge, is that right? If so, what's the  end point of this edge?

speaks and email are attributes: https://ldbcouncil.org/ldbc_snb_docs/schema.pdf
Their types are {String} and {Long String}. According to the specification, they are sets (i.e. they are unordered and their elements are unique) but you are free to represent them as arrays/lists.

speaks and email are attributes: https://ldbcouncil.org/ldbc_snb_docs/schema.pdf Their types are {String} and {Long String}. According to the specification, they are sets (i.e. they are unordered and their elements are unique) but you are free to represent them as arrays/lists.

Got it, thanks a lot!


Remove pattern validation for date normalization attributes

See #368 for the thread that originated this suggestion.
We have already removed hard-coded validation lists (i.e. for language codes), both because those code lists are updated more frequently than EAD, but also because it would allow people to use alternative code lists when appropriate.
Removing date normalization attribute patterns would fit with the second rationale and allow people to use alternative machine-processable date formats.  There are a lot available, actually, such as ISO 8601, a profile of ISO 8601 such as W3Cdtf, xs:date, or most significantly EDTF.
We can (and will) supply an external schematron for those who wish to continue to validate date normalization attributes with ISO 8601.  We could also create schematrons for validation in other date formats.
The following attributes would be affected:
@normal
@lastdatetimeverified
@standarddate
@notbefore
@notafter
@standarddatetime

@rockivist Where do you think it's best to incorporate information about using schematron to validate content within the TL? Within each of the relevant elements, in a preface, both?

@kshepher I think the preface needs to address in detail the validation functionality that has been removed to Schematron. Individual tag definitions should for the most part stick to what is represented in the schema.  We'll keep the schematron documentation mostly online - it will evolve over time too much for it to be incorporated into the print Tag Library.

That was my feeling as well. I'll leave this open to make sure it gets into the preface.

@tcatapano There is a problem in the schema re: this issue. We intended to remove the date normalization pattern from all of the attributes listed above in my first comment on the issue. However, as it stands now, the only attribute for which the pattern has been removed is @normal. Unless I am failing to remember a conversation and justification for why we should keep the pattern on the others, we need to revise the schema to convert those other attributes to token like @normal, and update the schematron to provide the appropriate pattern validation.
When updating the schematron, remember that the patterns for @notafter, @notbefore, and @standarddate don't include the time, but @lastdatetimeverified and @standarddatetime do.  And all of these omit the / range pattern from 8601 that is included in @normal.
@kshepher For now I have updated the TL to reflect how the schema currently stands. When Terry corrects the schema we can update then.

@rockivist my guess is that the patterns exist in the elements drawn from EAC-CPF and were retained. While I think it is better to remove the patterns from attributes of elements under <archdesc>, wouldnt it be best for values of dates and times in <control> to be, well, controlled?

@tcatapano I agree it makes sense to control at the least what's in control, esp if reflecting EAC-CPF. Was there a reason this would be too much of a burden for end users?

@tcatapano Yes, you're exactly right that the patterns persist in the EAC-CPF elements. Following your logic, we should keep the pattern validation on @standarddatetime and @lastdatetimeverified but remove it from @notbefore, @notafter, and @standarddate. It's not a perfect divide between control and archdesc, but it's pretty close.
Sound good?
@ruthtillman The decision to remove pattern validation for date normalization attributes was in response to community requests to allow alternative date normalization schemes.

@notbefore, '@notafter, and @standarddatenow have datatype token.@standarddatetimeand@lastdatetimeverified` unchanged

TL updated. See #472 re: adding patterns to schematron. Closing.


Prefer Void over () for return types of closures?

Wanted to bring this up here after looking at ReactiveCocoa/ReactiveCocoa#2059. Personally I prefer () but it's probably better to be consistent with Apple.

https://devforums.apple.com/message/1133616#1133616
http://ericasadun.com/2015/05/11/swift-vs-void/

I can open a PR if there's consensus

Consistency with Apple on this seems worthwhile to me. But I don't have strong feelings.

I much prefer () too. Apple have been wrong with their standards plenty of times in the past. :trollface:

For what it's worth, my vote is certainly for (). I fail to see the value of Void as an alternative, it obscures more than it clarifies.

Void is a typealias, so the debugger (for example) is always going to print () instead. I suggest ().
That said, I’m not sure this actually warrants an entry in the style guide. I think the document is at its best when it’s encouraging best practices for the project and/or the team, and I’m not sure that () vs. Void really matters.

Just my 2 cents: whatever the final decision will be, I think this should be in the style guide to enforce consistency.

I think this should be in the style guide to enforce consistency.

Consistency for consistency’s sake isn’t really in this guide’s charter. “Fewer debates about aesthetics” is the last item on the list for good reason, and since we haven’t had a debate about this on our team, there’s little to be gained by adding it here.
That said, if a team finds this guide useful but has some key differences, they should feel free to fork it & add whatever they see fit—this is, after all, only one team’s style guide, and thus not necessarily representative of any other team’s needs.
Or otherwise put: I don’t believe that there’s a one size fits all solution to code style, and I’d prefer not to act too speculatively—after all, who knows what the language will look like a week from now? :smile:

I’m going to close this out since there isn’t much to motivate a style guide rule.

:+1:  Thanks for the discussion.


Toggle chat window visibility from javascript

Hi! We've upgraded our chat client to version 1.0.0, and have noticed, that methods like toggleChat are not available from the WebChat object anymore. It seems only getSessionId, sendMessage and updateRules.
Is this function still available?

This does not seem possible from the React component either, is there a reason for it?

Any updates?


ios tweak request

hello. not sure if this in the right place because this github is the only link given in the tweak description. sorry if not.
tweak is wonderful.
would like the ability to disable on lock screen. it would use even less power and also give the ability to use other lock screen tweaks, like doodle, while this tweak is active.

We regret to inform you that we do not have plans to introduce new features for the tweak. The primary reason for this decision is that we are not generating sufficient revenue, and the sales data has not met our expectations. We also have other tweaks in development that require our attention.
If you’ve purchased the tweak and have questions, please contact us directly by tapping the author’s avatar for the email button.

I'm closing this issue since it's not related to the library code. If you have any further questions or concerns, feel free to leave a comment and I'll get back to you eventually.

Thank you for the updates. They are very much appreciated!! Adding you to my donate list once my next job is complete in the coming month or months. Sent from my iPhoneOn Aug 20, 2024, at 10:51 PM, Lakr @.***> wrote:
I'm closing this issue since it's not related to the library code. If you have any further questions or concerns, feel free to leave a comment and I'll get back to you eventually.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


[reveal][>=5.5.1 && <6.x] Configure on the fly not working

Test case:
http://codepen.io/liyaodong/pen/mROreZ?editors=1010
What should happen:

clickset js-custom-close-class,
open the Modal
when click custom close btn, the Modal should close

What happened instead:
Modal not close
foundation 5.5.0 doesn't have this issue
http://codepen.io/liyaodong/pen/OWbdRm?editors=1010

@liyaodong very sorry for the late response to your issue. We are moving on to Foundation 7, and there will be no more development or support for Foundation 5. I am closing this one out.


Improve the documentation of lib.extends and how it relates to overlays

Description of changes
lib.extends is the part of Nixpkgs that actually implements how overlays work, but its documentation was rather poor. This is an attempt to improve it.
This also relates to @roberth's suggested improvements to the lib.fix documentation in https://github.com/NixOS/nixpkgs/pull/242318 and @amjoseph-nixpkgs's suggested improvements to the extends documentation in https://github.com/NixOS/nixpkgs/pull/245825.
Things done

[x] Built the documentation and checked that it looks alright.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2023-08-17-documentation-team-meeting-notes-73/31853/1

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2023-09-28-documentation-team-meeting-notes-83/33599/1

I don't really have the time to work on this, but this has lingered for too long and is good enough. The docs are better than before and that's what matters most, so I'll just make the executive decision to merge this right now (thanks @9999years for the push!). If anybody thinks it should be improved still, please make a follow-up PR


Gz fuel server allows models with duplicate author and name

It seems two models with exactly the same name and author made it into the gazebo app portal:

Clicking each of them points to the same page https://app.gazebosim.org/anni/models/ur5_rg2.
This is causing a fair bit of issue in the Open-RMF side of things where we use model names to build a cache of fuel models and I believe it is probably not indended behavior?

@luca-della-vedova can you verify if this is still a problem. There was a caching issue on the Fuel server that was resolved last week which might have also fixed this. The URL for the ur5 model in the description no longer works for example.

However, I do see that the model count is 2 when searching for ur5 even though there's only one model shown

The way I'm using to verify the issue is to look at the models page, since the model has been uploaded fairly recently it appears in the page as one of the latest ones:

It looks like listing the model of just that owner shows there's one model, but if I list all models, it shows two ur5_rg2 models owned by anni.
$ gz fuel list -v4 -t model -o anni
Fetching model list from https://fuel.gazebosim.org...
Received model list (took 5ms).
https://fuel.gazebosim.org
└── anni
    └── ur5_rg2
1 owners, 1 models

$ gz fuel list -v4 -t model
Fetching model list from https://fuel.gazebosim.org...
Received model list (took 57653ms).
https://fuel.gazebosim.org
...
├── anni
│   ├── ur5_rg2
│   └── ur5_rg2

cc @nkoenig


era5cli v1.4.0: update meta.yml w/ grayskull

The built was broken because we changed to a pyproject.toml based setup.
As I am currently the main maintainer of era5cli, I have added myself as a maintainer of this recipe.
Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


Does titanium proxy support TLS 1.3?

Hi.
I am using titanium proxy as local proxy and it works well.
But i have some problems with tls.
My goal is to support only TLS1.2 and TLS1.3 versions.
So I added following code to upgrade the TLS version.
proxyServer.SupportedSslProtocols = SslProtocols.Tls12 | SslProtocols.Tls13;
My titanium proxy version is 3.1.1397 and dot net version is 5.0.
OS is windows 10 21H2 (19044.1826).
When i connect to tls check site, it says it does not support tls1.3

What's the problem?
If you don't use a proxy, tls1.3 is supported by Chrome

This is my internet setting and i checked tls 1.3.
And i also added tls 1.3 registry key but tls 1.3 is still not working.

Thank you for help.

proxyServer.SupportedSslProtocols means the supproted client SSL versions. (However there was a bug in it)
TWP requires (/earlier required) the same SSL/TSL version for the servers connection which was used by the client.
It seens the browsers are using TLS 1.2 by default.
Added a new property:
proxyServer.SupportedServerSslProtocols
If it is None, then TWP is using the same method as earlier.. if it is specified, then the specified value will be the supported server SSL versions.
You can try 3.2.2-beta+ (when it is released)


"InvalidFormat : Token malformed after retrieval." error when trying to get adminclient.

Server version - Build from mater -Aug 19
Not able to get admin client using apikey and secret key.
Following error returned:
E   ApiError: (ApiError(...), "InvalidFormat : Token malformed after retrieval.\n{'status': 400, 'detail': None, 'message': u'Token malformed after retrieval.', 'code': u'InvalidFormat', 'actions': {}, 'id': u'65ae6815-6f03-4447-9332-f57e0bcadfd3', 'links': {}, 'type': u'error'}")

If this only occurs during upgrade and the fix is to re-generate the keys, we don't need to fix this for the release.

This occurs whenever an apikey for a user who has logged in with github is used. re generating keys does not fix this bug.

This issue is not seen anymore on the latest server and we are able to start run validation tests on github auth enabled setup which uses user API keys.
Also tested with server build - v0.34.0-rc2. We are able to get authenticated successfully multiple times using the user api keys by setting api.auth.jwt.token.expiry to 5000 (5 secs).


only have the last 5 versions of our docs on the dropdown

Fixes #248

Mmmm this isn't doing what I intended. This is limiting the versions that docusaurus builds. What I want to have all the doc versions available (so we can eventually have an 'archive' page, but only the last 5 on the dropdown selector.


[SPARK-14833][SQL][STREAMING][TEST] Refactor StreamTests to test for source fault-tolerance correctly.

What changes were proposed in this pull request?
Current StreamTest allows testing of a streaming Dataset generated explicitly wraps a source. This is different from the actual production code path where the source object is dynamically created through a DataSource object every time a query is started. So all the fault-tolerance testing in FileSourceSuite and FileSourceStressSuite is not really testing the actual code path as they are just reusing the FileStreamSource object. This PR fixes StreamTest and the FileSource***Suite to test this correctly. Instead of maintaining a mapping of source --> expected offset in StreamTest (which requires reuse of source object), it now maintains a mapping of source index --> offset, so that it is independent of the source object.
How was this patch tested?
Refactored unit tests.

Test build #56621 has started for PR 12592 at commit ef65ad7.

Test build #56621 has finished for PR 12592 at commit ef65ad7.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56621/
Test FAILed.

Test build #56625 has started for PR 12592 at commit 48f1e6a.

Test build #56625 has finished for PR 12592 at commit 48f1e6a.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56625/
Test FAILed.

@zsxwing

LGTM except some nits and the style issue.

Test build #56762 has started for PR 12592 at commit 532a69c.

Test build #56762 has finished for PR 12592 at commit 532a69c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56762/
Test PASSed.

Merged build finished. Test PASSed.

LGTM. Thanks, merging to master.


Fix combining inlining and custom attributes

Inlined scripts were <script> tags whose tag.attributes property was undefined, and therefore tag.attributes.src was throwing an Uncaught TypeError: Cannot read property 'src' of undefined.
This change fallbacks script tags without attributes to use an empty string, technically allowing inlined script tags to still have custom attributes added (using a RegExp test for an empty string: /^$/).

Thanks for posting this. I just ran into the same problem. You may also need to add this to custom-attributes around line 31 to support adding attributes to inlined scripts.
tag.attributes = tag.attributes || {}

In my case I wanted to add a CSP nonce to the inlined scripts.

@numical it's important feature, shall I create different PR?

@phallguy, my hacky workaround :shit:
plugins: [
  ...
  {
    apply: compiler => {
      compiler.plugin('compilation', compilation => {
        compilation.plugin(
          'html-webpack-plugin-after-html-processing',
          (data, callback) => {
            data.html = data.html.replace(
              'src="legacy.bundle.js"',
              'src="legacy.bundle.js" nomodule'
            );
            callback(null, data);
          }
        );
      });
    }
  }
]

Yeah I think so. I've forked it for now and use that in my fork as well. Should cover all the possible scenarios.

@niieani  - sorry for how long it has taken for me to get this - truly
@phallguy, @VladimirGrenaderov  - thanks for your contributions.
I have incorporated your changes, plus tests, in v1.8.8.
@phallguy - for nonces I find the CSP webpack plugin effective.


DEV: Refactor the ChatChannel model

:construction: :construction: :construction:
Heavy WIP as of now

@martin-brennan if you work on slugs, take this PR into account please

Thanks for pointing this out @jjaffeux  -- At first thought I think based on this the new DMChannel and CategoryChannel would have their own generate_slug or similar methods that call Slug.for(name/title), then the slug itself would be saved on the main channel model. But, will do some experimenting.

What's left to be done on this from your point of view @Flink ?

What's left to be done on this from your point of view @Flink ?

I need to write some specs for the new models and the post migration. After that I think this first iteration should be ready :slightly_smiling_face:

I made few minor notes. I also tried it locally with existing records and it all seemed to work nicely :+1:
LGTM, let's talk of it tomorrow, thanks.

Everything we discussed about has been addressed I think :grin:


Merge docker-annotations into kubernetes-annotations.

Tests are complaining :(

Tests still failing unfortunately :(

Tests pass :100:


feat: initial version

Preview Architecture decision Record

closes #1

Here is a recording because it's kinda buggy right now
https://github.com/project-forms/project-forms.github.io/assets/39992/8f0b9527-3e1a-452f-8ef3-ef1783867044


Password complexity validation with ReDOS vulnerability

As I mentioned (in #24), the devise-security gem currently is likely (depending on what password_regex is used) to open up a vulnerability to ReDOS attacks.
Fixing this cannot be done without changing the means by which password complexity is set - it's the fact that a single regex is used that's the problem. This unavoidably means a breaking change.
I have implemented a complexity validator using what I believe is the most commonly required POSIX character sets (upper-case letters, lower-case letters, numerals, and punctuation marks/symbols). Client code can specify how many of each they want, or leave out any sets they don't want.
I also added a French translation, which is not great as French is not my native language, but I figured as I knew more French than German or Spanish I may as well add that in too while I was about it :)

@olbrich, I think I've addressed your feedback regarding character classes.
Also, as I was adjusting the tests, I found that the setup block in test/test_secure_validatable.rb did absolutely nothing. The reason is that validators accept their rules when the model is first loaded; they don't look up the config again when it's time to validate. (Incidentally, this is also true for the current master branch, too - the setup block does nothing. Try changing the value for password_regex in there - should break the tests, but doesn't.)
I considered making the validator accept a Proc, so that configs could be changed at runtime. However, this felt like deforming production code to make testing easier, which wouldn't be good. So instead, I added a unit test for the validator itself, with some dummy models to test various scenarios. See test/test_complexity_validator.rb.

👍 I like the approach of testing the validator independently.

I think I have addressed all second-round feedback..?


:arrow_up<EMAIL_ADDRESS>

:shipit:

Seems like the pull request can't be merged due to merge conflicts.

Let me clean that up and we can merge this then. Thanks for the bump, @jerone

The build failures don't seem to be related to this change, so I'm merging this.

:tada:


Bump iOS app version

This PR bumps the current iOS app version from 2.4.1 to 2.4.2.
Release: https://github.com/corona-warn-app/cwa-app-ios/releases/tag/v2.4.2

@Ein-Tim Thanks.


Pulse audio error

Nov 17 16:00:39  pulseaudio[1674]: [(null)] core-util.c: Failed to connect to system bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
Nov 17 16:00:39  syslogd: last message repeated 10 times
Nov 17 16:00:39  pulseaudio[1674]: [(null)] module-sndio.c: record channels changed:<PHONE_NUMBER> -> 0
Nov 17 16:00:39  pulseaudio[1674]: [(null)] oss-util.c: '/dev/dsp1' doesn't support full duplex
Nov 17 16:00:39  pulseaudio[1674]: [(null)] oss-util.c: '/dev/dsp2' doesn't support full duplex
Nov 17 16:00:39  pulseaudio[1674]: [(null)] authkey.c: Failed to open cookie file '/usr/home/user/.config/pulse/cookie': No such file or directory
Nov 17 16:00:39  pulseaudio[1674]: [(null)] authkey.c: Failed to load authentication key '/usr/home/user/.config/pulse/cookie': No such file or directory

has this been fixed with the latest release

In the pre-release the sequential error messages disappeared. I can control the volume, but if I choose on the system tray icon between the two hardware of my machines Realtek ALC255 internal analog and Intel Skylake HDMI than every time the Realtek works. But I think this already needs user manual setup and not the Trident's error.

Closing this issue.
Pretty much every package has had the pulseaudio option disabled now, and the pulseaudio->sndio sink is no longer used/needed by default any more.
After all those change, we have not been seeing any audio trouble on Trident (or at least none are getting mentioned/reported)


fix deprecated warnings for julia 0.4

Fix the deprecated warnings for julia 0.4 version

This looks awesome.  Thanks!


[WIP] Initial version - Documentation for Garbage Collection in Kubernetes

Fix #4322
@zacharysarah Here's the first draft based on my initial reading. I will keep updating this Pull Request as I dig deeper. As of now, I have the following questions to add more details to the document. It would be great if you can help me find answers for them while I go through the code and experiment with the setting (Checklist to keep a track of things for clear understanding :) ).

How exactly does the garbage controller delete the unused objects (Algorithm/more details)
Does Kube-Controller-Manager work in collaboration with Kubelet to delete unowned containers?
Are there any pointers on the internal working/workflow of the garbage collector control loop?
Are there any configuration parameters on Controller?
Pointers on internal workflow on how Kubelet coordinates with Kube controller manager in maintain consistency while cleaning up containers.  How the pointers are updated?
Eviction policy vs Garbage Collection in Kubelet and What's the future?
Tuning Garbage Collection in Kubelet. Are they any configuration parameter to change the timing of Garbage Collection.
What tool do we internally use to prepare visual workflows for a better understanding?

@daminisatya Heya! Thanks for opening this PR.
First: it looks like you're not yet a member of the Kubernetes GitHub org, but you've closed enough PRs to be eligible. If you'd like to become a member, I'm happy to sponsor you. You'll need an additional sponsor; feel free to ask in SIG Docs slack.
For more information, look at the new member issue template in k/org.
To answer your questions:

What tool do we internally use to prepare visual workflows for a better understanding?

Can you please say more about what you mean by visual workflows?

Questions 1-7

All of these are excellent questions. In this case, SIG API Machinery handles garbage collection, so we can ask them for review:
/sig api-machinery
/cc @kubernetes/sig-api-machinery-api-reviews
Note: If you ever need to find which SIG owns a particular piece of functionality, searching the kubernetes/community repository is a great way to find the owner. (That's how I found out that SIG API Machinery includes garbage collection in its charter).
Thanks again for opening this PR, and for digging in deeply. 🙇
@nikhita 👋 Tagging you here based on your comment on the original issue.
/assign @nikhita

@zacharysarah Thank you so much for recommending me to join the Kubernetes GitHub org. I would love too and will definitely raise a request for another sponsor!
Regarding,
1. What tool do we internally use to prepare visual workflows for a better understanding?

I wanted to know if there is any standard tool to prepare such images/workflows as follows. Found this over here, in our documentation.

kubernetes/community repository

Thank you so much for pointing me towards the community repo and the SIG-APIs group. Will definitely check this out and it's very helpful (Will answer most of my questions 😄).

@daminisatya 👋  Feel free to reopen this PR when you're ready to resume work.

sure @zacharysarah It's an ongoing work, i think it's good to be closed. I will reopen soon :D

Will reopen the issue soon and Update the PR. I have worked on it, mentioning to avoid any duplicate.

@daminisatya 👋

I wanted to know if there is any standard tool to prepare such images/workflows as follows.

Sorry for taking so long to reply! We ask for images in PNG or SVG format--but there's no standard tool. Feel free to use whatever works best for you! A diagram for garbage collection would be awesome. 👍


Provider spits out [DEBUG] messages in logs

What happened?
In v1.5.0 of the provider, spinning up resources creates debug log output:
Diagnostics:
  pulumi:pulumi:Stack (workshop-confluent-dev):
    2022/11/22 15:33:33 [DEBUG] POST https://api.confluent.cloud/org/v2/environments
    2022/11/22 15:33:33 [DEBUG] GET https://api.confluent.cloud/org/v2/environments/env-xqprvk
    2022/11/22 15:33:34 [DEBUG] POST https://api.confluent.cloud/cmk/v2/clusters
    2022/11/22 15:33:40 [DEBUG] GET https://api.confluent.cloud/cmk/v2/clusters/lkc-j3ojrq?environment=env-xqprvk
    2022/11/22 15:33:40 [DEBUG] GET https://api.confluent.cloud/cmk/v2/clusters/lkc-j3ojrq?environment=env-xqprvk

Steps to reproduce
n/a
Expected Behavior
n/a
Actual Behavior
n/a
Output of pulumi about
No response
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@jkodroff could you drop a github repro or gist with a program creating resources to repro?

https://github.com/pulumi/workshops/tree/main/confluent

This is happening with us as well 👍. Would love it if this became optional, as it is occasionally useful for debugging.

We have hundreds of topics/schema in terraform currently and trying migrate into Pulumi,
The amount of noise this generates makes the pulumi output unusable..
Can someone please help describe what needs to happen here ?
If no one is looking into it i'm happy to give it a try.

@FabioBatSilva the debug messages appear to be from the upstream provider's use of go-retryablehttp. Specifically, if you look at https://github.com/hashicorp/go-retryablehttp/blob/main/client.go#L609-L616 in that library, it seems to use the default logger instance which logs via stderr. So yeah the upstream provider either needs to provides its own logger or should set it to nil when constructing an instance of the HTTP client from go-retryablehttp.

Thanks @praneetloke.
I was able to get some log entries on the tf provider.
When running terraform with debug enabled i get :
2024-03-18T19:30:33.710-0400 [DEBUG] provider.terraform-provider-confluent: Reading Schema "xxxx-xxxxx/chesscom/xxxx.proto/latest": @module=provider tf_req_id=674347de-096d-5287-519d-d7afc4fbcb45 tf_resource_type=confluent_schema tf_rpc=ReadResource @caller=/terraform/terraform-provider-confluent/internal/provider/resource_schema.go:597 schema_id=xxxx-xxxxx/chesscom/xxxx.proto/latest tf_provider_addr=provider timestamp=2024-03-18T19:30:33.710-0400
2024-03-18T19:30:33.710-0400 [DEBUG] provider.terraform-provider-confluent: 2024/03/18 19:30:33 [DEBUG] GET https://xxxx.gcp.confluent.cloud/subjects/chesscom%2Fxxxx.proto/versions/latest
2024-03-18T19:30:33.941-0400 [DEBUG] provider.terraform-provider-confluent: 2024/03/18 19:30:33 [DEBUG] GET https://xxxx.gcp.confluent.cloud/schemas?subjectPrefix=chesscom%2Fxxxx.proto
2024-03-18T19:30:33.970-0400 [DEBUG] provider.terraform-provider-confluent: Fetched Schemas "xxxx-xxxxx/chesscom/xxxx.proto/latest": [...]: @caller=/terraform/terraform-provider-confluent/internal/provider/resource_schema.go:781 @module=provider schema_id=xxxx-xxxxx/chesscom/xxxx.proto/latest tf_resource_type=confluent_schema tf_rpc=ReadResource tf_provider_addr=provider tf_req_id=674347de-096d-5287-519d-d7afc4fbcb45 timestamp=2024-03-18T19:30:33.970-0400
2024-03-18T19:30:33.970-0400 [DEBUG] provider.terraform-provider-confluent: Fetched Schema "xxxx-xxxxx/chesscom/xxxx.proto/latest": {...}: schema_id=xxxx-xxxxx/chesscom/xxxx.proto/latest tf_provider_addr=provider

Is the parsing here  breaking the on pulumi side bc of the double [DEBUG] ?

@FabioBatSilva Ah I wasn't aware of that part in the bridge repo honestly.

Is the parsing here breaking the on pulumi side bc of the double [DEBUG] ?

Maybe this line since there are two ]s: https://github.com/pulumi/pulumi-terraform-bridge/blob/master/pkg/tfbridge/log.go#L94. You could try to reproduce that with a test in log_test.go and maybe propose a fix.

I tried to reproduce the issue with log_test but it seems ok, Maybe that is happening somewhere else..
I'v opened a PR to provide a logger for retryablehttp
https://github.com/confluentinc/terraform-provider-confluent/pull/374


Parameter 1 to MongoCollection::batchInsert() expected to be a reference, value given

Am I doing something, incorrectly?
			DB::collection('accounts')->insert(array(
				'name' => array(
					'first' => Input::get('fname'),
					'last' => Input::get('lname')
				),
				'email' => Input::get('email'),
				'company' => Input::get('coname'),
				'password' => Input::get('password'),
				'date_created' => new DateTime(),
				'date_updated' => new DateTime()
			));

Stack trace:
ErrorException thrown with message "Parameter 1 to MongoCollection::batchInsert() expected to be a reference, value given"

Stacktrace:
#23 ErrorException in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/jenssegers/mongodb/src/Jenssegers/Mongodb/Collection.php:59
#22 Illuminate\Exception\Handler:handleError in <#unknown>:0
#21 call_user_func_array in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/jenssegers/mongodb/src/Jenssegers/Mongodb/Collection.php:59
#20 Jenssegers\Mongodb\Collection:__call in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/jenssegers/mongodb/src/Jenssegers/Mongodb/Query/Builder.php:393
#19 Jenssegers\Mongodb\Collection:batchInsert in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/jenssegers/mongodb/src/Jenssegers/Mongodb/Query/Builder.php:393
#18 Jenssegers\Mongodb\Query\Builder:insert in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/app/controllers/HomeController.php:297
#17 HomeController:doNewAccount in <#unknown>:0
#16 call_user_func_array in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Routing/Controller.php:231
#15 Illuminate\Routing\Controller:callAction in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php:93
#14 Illuminate\Routing\ControllerDispatcher:call in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php:62
#13 Illuminate\Routing\ControllerDispatcher:dispatch in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Routing/Router.php:962
#12 Illuminate\Routing\Router:Illuminate\Routing\{closure} in <#unknown>:0
#11 call_user_func_array in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Routing/Route.php:109
#10 Illuminate\Routing\Route:run in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Routing/Router.php:1028
#9 Illuminate\Routing\Router:dispatchToRoute in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Routing/Router.php:996
#8 Illuminate\Routing\Router:dispatch in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Foundation/Application.php:776
#7 Illuminate\Foundation\Application:dispatch in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Foundation/Application.php:746
#6 Illuminate\Foundation\Application:handle in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/barryvdh/laravel-debugbar/src/Barryvdh/Debugbar/Middleware.php:34
#5 Barryvdh\Debugbar\Middleware:handle in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Session/Middleware.php:72
#4 Illuminate\Session\Middleware:handle in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Cookie/Queue.php:47
#3 Illuminate\Cookie\Queue:handle in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Cookie/Guard.php:51
#2 Illuminate\Cookie\Guard:handle in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/stack/builder/src/Stack/StackedHttpKernel.php:23
#1 Stack\StackedHttpKernel:handle in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/vendor/laravel/framework/src/Illuminate/Foundation/Application.php:642
#0 Illuminate\Foundation\Application:run in /Applications/XAMPP/xamppfiles/htdocs/mywebsite/public/index.php:49

Does exists any clean solution for this?

Any luck?

@Pezmc did you ever figure out how to resolve this issue?

Oh hey Frank, yes! It was something to do with having the wrong version of the Mongo extension for PHP installed. I think one might come bundled with it, BUT it's not the right version. Download a new copy (or try one different to what you already have) and you should be grand!

@Pezmc thanks buddy! Great to see you! Hope all is well. :)

hello , but this problem is still in hhvm mongofill .

@razzbee that is correct. mongofill was still on composer. Removing this from composer did the trick.

If I am not getting you wrong, I am not using the composer version of mongofill, I compiled the binary version.so I dont have mongofill in composer.

Have tried with and without mongofill however still getting this error, any suggestions?
"Parameter 1 to MongoCollection::insert() expected to be a reference, value given"
code
$user = new User();
$user->role = new \MongoId($roleId);
$user->title = $request->title;
$user->category = $request->category;
$user->__v = 0;
$user->save();

@ryuheiyokokawa this worked for me with mongofill and hhvm (only for inserts)
DB::connection('connection_name')->selectCollection('accounts')->insert(array( ));

@ryuheiyokokawa this worked out for me with mongofill and hhvm . but this works only for inserts
DB::connection('connection_name')->selectCollection("collection_name")->insert(arrray()).
For other operations DB::connection('connection_name')->collection("collection_name") works well

have you also tried eloquent with the ::create(array) method ?


[1.16.5] Tumblebee Bees choosing violence in the Atum

Version: dungeons_gear-1.16.5-3.1.15-beta.jar
Forge: 1.16.5 36.2.39
Atum: Atum-1.16.5-2.2.11.jar
I had found some tumblebee enchanted boots at some point during an atum start in my modpack.
At one point, I dodged an enemy and it spawned a bee that helped me and survived the fight. Later I realized the bee was highly aggressive towards villagers of the atum when I reached the nearby village. Due to the no pet damage I was unable to stop the bee constantly going in to stab the villagers and so had to resolve it by picking up the bee with carry on and then dunking it in a nearby oasis...
On the way there it seemed to be angry towards some of the passive animals like quails potentially also.
All of the atum mobs etc had already been placed in the effect blacklist list at this time.

Could you share your config list? This could happen if an entry in the list is not recognized (typo?). I have just tested locally with default configs. They were aggressive to cats (didn't add those to the default list, they are now), but not to villagers.

something like this:
	effectTargetBlacklist = ["guardvillagers:guard", "minecraft:bat", "minecraft:bee", "minecraft:chicken", "minecraft:cod", "minecraft:cow", "minecraft:dolphin", "minecraft:donkey", "minecraft:fox", "minecraft:horse", "minecraft:iron_golem", "minecraft:mooshroom", "minecraft:ocelot", "minecraft:panda", "minecraft:parrot", "minecraft:pig", "minecraft:polar_bear", "minecraft:pufferfish", "minecraft:rabbit", "minecraft:salmon", "minecraft:sheep", "minecraft:squid", "minecraft:strider", "minecraft:trader_llama", "minecraft:tropical_fish", "minecraft:turtle", "minecraft:villager", "minecraft:wandering_trader", "minecraft:wolf", "ars_nouveau:ally_vex", "ars_nouveau:blazing_weald_walker", "ars_nouveau:carbuncle", "ars_nouveau:cascading_weald_walker", "ars_nouveau:drygmy", "ars_nouveau:dummy", "ars_nouveau:familiar_bookwyrm", "ars_nouveau:familiar_carbuncle", "ars_nouveau:familiar_drygmy", "ars_nouveau:familiar_jabberwog", "ars_nouveau:familiar_sylph", "ars_nouveau:familiar_wixie", "ars_nouveau:flourishing_weald_walker", "ars_nouveau:summon_horse", "ars_nouveau:summon_wolf", "ars_nouveau:sylph", "ars_nouveau:vexing_weald_walker", "ars_nouveau:whelp", "ars_nouveau:wixie", "atum:camel", "atum:desert_rabbit", "atum:desert_wolf", "atum:quail", "atum:quail_egg", "atum:serval", "atum:stoneguard_friendly", "atum:stonewarden_friendly", "atum:villager_female", "atum:villager_male", "betterendforge:dragonfly", "betterendforge:end_fish", "betterendforge:shadow_walker", "cavesandcliffs:axolotl", "cavesandcliffs:glow_squid", "cavesandcliffs:goat", "charm:coral_squid", "charm:moobloom", "doggytalents:dog", "endergetic:booflo", "endergetic:booflo_adolescent", "endergetic:booflo_baby", "endergetic:puff_bug", "endermail:ender_mailman", "environmental:deer", "environmental:duck", "environmental:duck_egg", "environmental:fennec_fox", "environmental:koi", "environmental:slabfish", "environmental:yak", "farmingforblockheads:merchant", "iceandfire:amphithere", "iceandfire:hippocampus", "iceandfire:hippogryph", "iceandfire:hippogryph_egg", "iceandfire:myrmex_egg", "iceandfire:myrmex_queen", "iceandfire:myrmex_royal", "iceandfire:myrmex_sentinel", "iceandfire:myrmex_soldier", "iceandfire:myrmex_swarmer", "iceandfire:myrmex_worker", "iceandfire:pixie", "iceandfire:siren", "inventorypets:bed_pet_entity", "inventorypets:bill_gates_entity", "inventorypets:mini_quantum_blaze_entity", "inventorypets:mini_quantum_enderman_entity", "inventorypets:satya_nadella_entity", "inventorypets:siamese_entity", "inventorypets:steve_ballmer_entity", "losttrinkets:dark_vex", "mowziesmobs:baby_foliaath", "mowziesmobs:barakoan_player", "mowziesmobs:foliaath", "mowziesmobs:grottol", "mowziesmobs:lantern", "mutantbeasts:creeper_minion", "mutantbeasts:creeper_minion_egg", "mutantbeasts:endersoul_clone", "mutantbeasts:mutant_snow_golem", "mutantbeasts:spider_pig", "mysticalworld:beetle", "mysticalworld:deer", "mysticalworld:endermini", "mysticalworld:frog", "mysticalworld:hell_sprout", "mysticalworld:lava_cat", "mysticalworld:owl", "mysticalworld:silkworm", "mysticalworld:silver_fox", "mysticalworld:sprout", "outer_end:chorus_squid", "outer_end:spectrafly", "outer_end:stalker", "quark:crab", "quark:foxhound", "quark:frog", "quark:shiba", "quark:stoneling", "quark:toretoise", "savageandravage:creepie", "tombstone:grave_guardian", "villagertools:guard", "villagertools:reinforced_golem"]
it's kinda huge cause it's from my modpack. I might have to go in and test it around some of these other mobs to see if I indeed do have a typo in there somewhere.

Could you send me a latest.log? That often tells me if there's an issue with anything.

latest.log
sorry, github logged out so never noticed a reply. Um, thats the latest latest.log that I got.

Should be fixed in Dungeons Libraries 0.0.18-beta.
Summoned mobs will no longer attack passive mobs (except if their master does)


Decide on messaging service we'll use

How do we stay in touch with applicants via Email/SMS? Had a thread with the city but didn't seem like there was a clear recommendation.

Azure appears to have some kind of Communication Services tool: https://docs.microsoft.com/en-us/azure/communication-services/quickstarts/telephony-sms/send?pivots=programming-language-csharp. IDK if it works for us.
Amazon may have something too?
I'm just hesitant to get into another round of procurement w/ a new vendor.

These are Amazon's services:

Amazon Simple Email Service (https://aws.amazon.com/ses/)
Amazon Simple Notification Service for SMS (https://aws.amazon.com/sns/)

Good news is that we won't be sending that many emails and texts, so the spending won't be too high.

Everything we're using is on AWS, so SES / SNS are a good starting point. Need to discuss policies with City IT staff first, of course.

Just called John Doke & Atul from Seattle IT. Sounds like SES / SNS is the way to go. It can be procured as simply as flipping a switch. Other services will take WAY longer to procure; not worth it.
cc @bion @ndmckinley


Warning when user changes permissions of representative image

When a creator of a work changes the permissions from public to something less than public, they should see a warning message that the image will no longer be displayed as a representative image.

On the edit page of the FileSet
On the form add help text to Say something like. If this is or becomes your representative file Set this may impact the display of you work in search results


I there a way to package Jekyll with Electron?

Electron version: latest
Operating system: OSX

I was wondering if there is a way to package Jekyll with an Electron app, I would like to create a CMS for Jekyll that deploys to gh-pages

@jbonigomes It is possible to do so. You could include Ruby and Jekyll in your app and talk to them via a child process.
You may find more help for questions in the forum ✨

Thanks @jlord, btw, love your work

@jbonigomes Have you made some progress on this idea ? I've a similar need and was aiming to use https://github.com/mklabs/jekyll-preview to provide a basic preview feature.

Hi @mklabs, I am working on a different project at the moment, so I had to park this idea, however, in order to keep things slimmer, I was no longer considering Jekyll and Ruby, but Hexo instead.

Btw, nice project

Hey @jbonigomes thx :) didn't know about hexo, looks really realy cool.
I'll let you know if I make some progress on this idea.

Thanks @mklabs


Buttons don't behave correctly on mobile

What is the current behavior?

What changes are you suggesting?
Do this way

Additional information
I will do it soon

Using a mobile

Closed in https://github.com/github/docs/pull/1476


Implement ActivateCredential key attestation

A spec has been created under https://github.com/parallaxsecond/parsec/issues/370 to define the interface of the operations involved in key attestation. Protobuf protocols also exist, having been added in https://github.com/parallaxsecond/parsec-operations/pull/34 .
This issue covers the implementation of the new operations across the stack, plus backending them to the TPM provider. The task can be split into:

[x] Adding the operations in the interface ( https://github.com/parallaxsecond/parsec-interface-rs/pull/124 )
[ ] Adding the plumbing for the operations in the Parsec service
[ ] Implementing the required functionality in the TransientKeyContext in the tss-esapi crate ( https://github.com/parallaxsecond/rust-tss-esapi/pull/284 )
[ ] Implementing the operations in the TPM provider
[ ] Implementing support for the operations in the Rust client ( https://github.com/parallaxsecond/parsec-client-rust/pull/100 )
[ ] Implementing E2E testing for the new operations

While implementing some tests for the ActivateCredential attestation mechanism in the TSS crate I realized an important caveat that I don't think was very clear to me back when first pitching this implementation.
The API design that I came up with for ActivateCredential attestation accepts not only a key to be attested, but also an (optional) key to attest with. This latter one is at the core of the caveat. The issue is that the key used to attest with can't be just any decryption key, like the ones that Parsec clients can generate at any time and decrypt stuff with, but rather they must be restricted decryption keys, meaning they can't be used for decrypting stuff from users in general. Such keys are generally used to store other keys as its children, thus generally called Storage Keys, but using them for stuff like attestation via ActivateCredential is also allowed. The broader point is: decryption keys generated by Parsec clients via PsaGenerateKey cannot be used like this, so it's essentially impossible to create a chain of attestations in this way.
Two options here, looking forward are:

we implement a feature through which the admin can provision some keys and declare them in the config file, and these keys can be used by all clients to attest stuff (via a name?)
we implement a new operation specific to the TPM which would allow clients to provision Storage Keys like they would any other key, for the sole purpose of using them with ActivateCredential
These two options need not be exclusive.

Following a discussion on the Slack channel, the most sensible approach seems to be to the latter approach, but again with a generic operation (rather than TPM-specific). However, since there is no need or explicit use-case (for now) for supporting attestations with user-generated keys, this feature will be parked.

One item that was not covered by the initial checkboxes was error handling. Currently errors occuring during the ActivateCredential call are mapped to PsaErrorCommunicationFailure. Ideally, the relevant ones should instead map to more descriptive error codes.
A bit of design brainstorming should go on before this is put into practice to discuss questions like: "do we want these new error codes to be specific to key attestation?".
A checkbox for this work should now appear in the top comment.


Add pre-commit workflow

Implements linting with black, among other checks.
Based on https://github.com/CCI-MOC/openshift-acct-mgt and https://github.com/nerc-project/coldfront-plugin-api

@QuanMPhm review appreciated. Can't seem to be able to add you as a reviewer in the PR using the buttons.

Can't seem to be able to add you as a reviewer in the PR using the buttons.

I added the ops-core team to this repo and now we can tag Quan for reviews


Store CoreCommands as Denormalized Part of DeviceProfile

When adding a DeviceProfile, the CoreCommands are stored in a manner similar to a relational database. Rather than remaining part of the DeviceProfile document, they are persisted into their own collection (Mongo) or index (Redis). We need to stop this behavior. CoreCommands should be stored on the DeviceProfile itself and then serve as a template for the commands supported by Devices that inherit from the DeviceProfile.
MONGO
Looking at the AddDeviceProfile function, you can see that Commands are saved first before the DeviceProfile itself. This results in a list of DBRef entries that are associated to the CoreCommands property of the Mongo-specific data model, similar to a foreign key. Note that these commands are the only ones stored in this manner.
Eliminating this relationship will potentially allow us to remove code related to the commandTransform interface and its implementation in the metadata MongoClient.
REDIS
A similar approach is taken in Redis, which simply follows the pattern established in Mongo.
In both cases what we should wind up with once this work is done, given a fresh database, is a DeviceProfile saved in JSON that has all of its related data stored in one document without the need for additional lookups based on a relationship. The current command collection (Mongo) or index (Redis) where commands are stored should be empty.
Finally, as a follow up, I believe this lookup within the commandByDeviceID function can be removed. We are still at the point where a DeviceProfile is part of a Device. As you can see earlier in the function, the Device is loaded. All that needs to be done here is to find the CoreCommand entry corresponding to the commandID string parameter value.

Finally, as a follow up, I believe this lookup within the commandByDeviceID function can be removed. We are still at the point where a DeviceProfile is part of a Device. As you can see earlier in the function, the Device is loaded. All that needs to be done here is to find the CoreCommand entry corresponding to the commandID string parameter value.
Will we continue to generate ID for commands once they are part of the DeviceProfile itself? Why don't we use the command.Name and command.DeviceID/deviceName (we are going to add)  as a composed unique key  ?

@tsconn23 I am writing in order to be sure that I understand exactly what I am supposed to implement.
In the current implementation as I see we have:
dbmodel struct for Device  that includes a DBRef to the Profile.
While the db model struct for DeviceProfile  contains list of DBRefs for commands:
type DeviceProfile struct {
	....
	DeviceResources []DeviceResource  `bson:"deviceResources"`
	DeviceCommands  []ProfileResource `bson:"resources"`
	CoreCommands    []mgo.DBRef       `bson:"commands"`
}

As I understand I am supposed to replace
CoreCommands    []mgo.DBRef       bson:"commands"``
with
CoreCommands    []Command         bson:"commands"
As a result  the Commands will be saved in denormalized way (array of embedded command documents) that will contain uuid only (without -> Id bson.ObjectId bson:"_id,omitempty" ).
This uuid will be used temporary (just from temporary backward compatibility). When all tasks are done ..this uuid could be removed. Correct ? But removing this we will break down GetDeviceProfilesByCommandId(But is this function really needed?)
These commands as being embedded into the DeviceProfile struct itself will be maintained (created, updated, deleted) on restAddDeviceProfile/restUpdateDeviceProfile/restDeleteProfile_By_XXXX
Till now, I have been talking only for db structures - Device->Profile->Commands, which definitions come from the respected contracts models .
If I understand correct what i need to do is: when adding new device I need to get its deviceProfile that is already saved into the DB,
get its embedded(already denormalized) core-commands, and used them as a template for creating/saving absolutely new device.Commands along with saving the device itself.
And these device.commands saved in the DB will contain all the information from the device.profile.commands plus information of the device they belong to (DeviceID and DeviceName).
This new collection of commands will be saved using DBRefs (in case of mongo), not in denormalized way ? Is that true?
Once we have all this, all core-command service endpoints as: restGetCommandsByDeviceName, restGetCommandsByDeviceID  , even restGetAllCommands... will read directly from this new "device.commands" collection (considering https://github.com/edgexfoundry/edgex-go/issues/1274)
Having these two collections for commands: 1.profile.commads used as template to create 2.device.commads, if the profile.commads are updated this change needs to be propagated to all device.commands that use this profile. I suggest that the update/delete/create of the Commands should happen only through update of the DeviceProfile. Currently, as noted below, there are methods that manipulate the commands directly! (Btw, if we keep the device.profile.commands.uuid we could easily make reference between the device.commands and device.profile.commands? )
NOTE- in the current implementation there are endpoints for adding and updating , deleting Commands directly. This could lead to commands present into the database  that does not have any reference to any device or deviceprofile?

Yes, this is correct. I can add an agenda item to the next Core WG meeting with regard to removing the  endpoints you call out. As you say, I don't think anyone is using GetDeviceProfilesByCommandId and I agree that there is no use for directly manipulating commands for a given device independently from the profile. We will want a decision to remove these APIs for Edinburgh soon, even if your work isn't completed yet.

GetDeviceProfilesByCommandId is called only on  updateCommand and deleteCommandByID

restDeleteCommandById(in isCommandStillInUse)
restUpdateCommand
in 2 tests that test the function itself

So, if these is an agreement that it does not make sense to update/create/delete Commands aside from the device Profile (exception: delete device.commands when we are deleting the device instance) this will be the first thing I will do. (it will make my next steps easier)

Well, we'll need to discuss it in a Core WG call but I am nominally in agreement. I mean, those endpoints right now effectively don't do anything because we always look to a device profile for a device's commands. Someone could add/update commands all day long and it would have no effect. So we should just get rid of the endpoints.


Code fragmenting not correctly applied to code-block

Hello,
I'm trying to get code fragmenting work properly for a presentation. On some slides the fragments are not properly applied to the code block. I am not sure, if I did the markup right, but I can't find any mistake.
See the reproducer at https://gitpitch.com/nilsramsperger/vortrag-docker/dev#/
The behaviour can be seen on Firefox, Chrome and Safari.
A Bug, or just me?
Greetings
Nils

Hi @nilsramsperger thanks for reporting this issue. The problem here is that your code block has has not specified a code language hint and your shell commands are being incorrectly identified as dockerfile content (because docker appears repeatedly in the content).
What you need to do is simply add a code language hint on your code block(s). For example:

Note the addition of the word shell immediately following the opening triple backticks for your static code block.
You can learn more about code language hints here. I have also just updated the Code Block page on the Wiki to include a note about the use of code language hints.
Hopefully this will fix your problem. If you have a chance to test can you please update or close out this thread as appropriate.

Hello,
Thanks for the fast reply.
I just set language-hints for all my codeblocks and it worked with one exception.
One block is hinted with "Dockerfile". There the problem persists. According to the languages list on highlightjs.org it should be correct.
Using "dockerfile" instead makes no difference.
I updated the reproducer at https://gitpitch.com/nilsramsperger/vortrag-docker/dev#/

Thanks for this update, much appreciated. I've looked into this and there is definitely an incompatibility as you reported when dockerfile is in use. Until I have a solution to that problem I suggest you use text or shell as the lang hint on any dockerfile blocks that you are using. It will cause your Dockerfile content to render somewhat plainly, but code-presenting within your slides will be fully functional.

Thanks for the info. I will switch to bash as a workaround untill a fix is available.


Setting up hoverfly on all test cases loading remote context

Note to self: consider creating a new test or excluding an existing one from the simulated w3 payload to ensure that remote context loading still works in the real world.


Investigate using PostgreSQL NOTIFY and commit hooks

Is your feature request related to a problem? Please describe.
Currently this lib uses Django Signals to detect changes however Django does not send these events if you do bulk updates or raw SQL operations and if you have other clients connecting to the database making changes you also do not get any events.
Describe the solution you'd like
PostgreSQL support the concept of NOTIFY and LISTEN, since commit hocks can send NOTIFY messages it would be possible to have PG automatically inform all LISTENs of db changes regardless of were the changes came from.
I would like to investigate a little more how NOTIFY and LISTEN work in PG and how easy/hard it would be to add something to Django's DB manager that automatically adds migrations for ObservableModel classes to trigger the NOTIFY on commit.
As a first pass it could be interesting to implement a PG packed change channels layer that uses the LISTEN and NOTIFY api this could be an interesting option for people who do not want to deploy an additional redis.

I think a more general solution would be to use redis PUB/SUB as the redis is commonly used for caches anyway. This way it can work with any db.
What do you think?


[KBFS-1439] Sync blockJournal's in-memory map properly

Add a tag field to blockRefMap and populate it with
the ordinal. Use that to efficiently remove map
entries on a flush.
Move BlockServerDisk-specific logic out of blockJournal,
and remove more BlockServer-isms in it.
Clean up block_journal_test.go.
Move shutdown logic out of blockJournal and into
tlfJournal and BlockServerDisk.

Looks good mod mostly explanatory suggestions, thanks!

All comments addressed. Mind taking another look, since this has been rebased and has picked up some of your changes?

Hmm, the journal DSL tests look pretty broken.  In TestJournalSimple, for example, it seems like bob is getting a "block doesn't exist" error when trying to read the root block alice wrote via his journal.  Can you look into that before I take another look?

Hmm, that's unexpected! I'll look in the morning.

Okay, this is ready for another look.

Looks good to me pending CI and rebase, thanks!

CI failure is flake, filed: https://keybase.atlassian.net/browse/KBFS-1506


Revert "RR-667:  Add new subscription reminder summary endpoints"

Reverts uktrade/data-hub-api#4376 so I can write something to handle cases where there is nothing to summarize.

Closing as the FE is not using this endpoint yet - will add the fix in a later PR


FIX: Anchor links HTML template fix

I had a moment to try and fix this, so here's a start. It's not quite finished, but I'm pushing it in case it saves time for @AakashGfude. Feel free to push to this branch to finish it up, or throw this away and start from scratch :-)
This moves the printing TOC into the same div as the main content, so we re-use the table of contents.
The printing seems to work OK, with one exception: there's a horizontal scrollbar on the bottom, though it seems this is present on master anyway

@choldgraf looks good, thanks for starting the PR. There is a bit of misalignment with Contents as can be seen from the pic below.
(LEFT: This PR, RIGHT: master)

Will fix that, and we should be good to go.

@choldgraf have pushed a few CSS styles, i.e, alignment of Contents, overlapping in the footer. the horizontal scroll bar is also removed.
It looks good to me now.

It looks good to me - thanks for pushing the updates! Will merge and cut a quick release.


myks: 4.2.5 -> 4.2.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/myks/versions.
meta.description for myks is: Configuration framework for Kubernetes applications
meta.homepage for myks is: https://github.com/mykso/myks
meta.changelog for myks is: https://github.com/mykso/myks/blob/v4.2.6/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.2.6 with grep in /nix/store/l68i5kg13smybw74prk3nv29sdzb750x-myks-4.2.6
found 4.2.6 in filename of file in /nix/store/l68i5kg13smybw74prk3nv29sdzb750x-myks-4.2.6

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

myks

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/l68i5kg13smybw74prk3nv29sdzb750x-myks-4.2.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A myks https://github.com/r-ryantm/nixpkgs/archive/ff9b99d7719c5ff6f96444179c8045711df80261.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ff9b99d7719c5ff6f96444179c8045711df80261#myks

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l68i5kg13smybw74prk3nv29sdzb750x-myks-4.2.6
ls -la /nix/store/l68i5kg13smybw74prk3nv29sdzb750x-myks-4.2.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

myks

Maintainer pings
cc @kbudde @zebradil for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


Roll Packages from 23e56af4a622 to 8df9848e9f22 (2 revisions)

https://github.com/flutter/packages/compare/23e56af4a622...8df9848e9f22
2024-03-20<EMAIL_ADDRESS>[camera_android] Update README to encourage users to use CameraX implementation (flutter/packages#6353)
2024-03-20<EMAIL_ADDRESS>Roll Flutter from d31a85ba5c5e to b96c13d1e9ff (19 revisions) (flutter/packages#6359)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add compatibility for PySide6 6.5+

References and relevant issues
I understand the reasons for closing https://github.com/napari/napari/pull/6062
but it seems to me that this one particular line should be merged in regardless unless an upcoming fix is coming in 6.7.1+ that I am unaware of.
Description
There seems to be 2 more fixes that I haven't had time to go through, but I know in my testing that without this modification, Napari simply doesn't open with pyside 6.7+ locally.

Wait so you still need this with latest pyside6?
Because I don't (macOS), it works in a fresh env with pip install pyside6 and pip install napari:
napari: 0.4.19.post1
Platform: macOS-13.3.1-arm64-arm-64bit
System: MacOS 13.3.1
Python: 3.12.2 | packaged by conda-forge | (main, Feb 16 2024, 20:54:21) [Clang 16.0.6 ]
Qt: 6.7.0
PySide6:
NumPy: 1.26.4
SciPy: 1.13.0
Dask: 2024.4.1
VisPy: 0.14.2
magicgui: 0.8.2
superqt: 0.6.3
in-n-out: 0.2.0
app-model: 0.2.6
npe2: 0.7.5

OpenGL:
  - GL version:  2.1 Metal - 83.1
  - MAX_TEXTURE_SIZE: 16384

Screens:
  - screen 1: resolution 1680x1050, scale 2.0

Settings path:
  - /Users/piotrsobolewski/Library/Application Support/napari/nap-pyside6_fbd2ecd52d803b20ca985dc7a1dd14615e8ea70d/settings.yaml
Plugins:
  - napari: 0.4.19.post1 (77 contributions)
  - napari-console: 0.0.9 (0 contributions)
  - napari-svg: 0.1.10 (2 contributions)

I still get the test crashes, so I suspect seg faults are still around somewhere -- plus what @dalthviz showed with removing layers...

For reference, from pyside.org:

you are right. i don't know why it errored with me. i tried
napari version 0.4.19.post1
with a few different versions of pyside2 and pyside6 and it seems ok.


Add parse_uri() / parse_url() function

Summary of the changes / Why this improves CrateDB
Adding a parse_uri(<TEXT>) and parse_url(<TEXT>) function, that could resolve #11941.
References from other DBs

parse_url() from Snowflake
url functions from ClickHouse
parse_url() from Databricks

cr> SELECT parse_uri('https://testing.crate.io:4200/') as url;
+--------------------------------------------------------------------------------+
| url                                                                            |
+--------------------------------------------------------------------------------+
| {"hostname": "testing.crate.io", "path": "/", "port": 4200, "scheme": "https"} |
+--------------------------------------------------------------------------------+
SELECT 1 row in set (0.003 sec)

cr> SELECT parse_url('https://user:pw%26@testing.crate.io:4200/sub+space/sub%20space2/index.html?foo=bar&foo=&foo2=https%3A%2F%2Fcrate.io%2F%3Ffoo%3Dbar%26foo%3Dbar2%26foo2#ref') as url;
+---------------------------------------------------------------------------+
|  url                                                                      |
+---------------------------------------------------------------------------+
| {                                                                         |
|   "fragment": "ref",                                                      |
|   "hostname": "testing.crate.io",                                         |
|   "parameters":                                                           |
|     {                                                                     |
|       "foo": ["bar", null],                                               |
|       "foo2": ["https://crate.io/?foo=bar&foo=bar2&foo2"],                |
|     },                                                                    |
|   "path": "/sub space/sub space2/index.html",                             |
|   "port": 4200,                                                           |
|   "query": "foo=bar&foo=&foo2=https://crate.io/?foo=bar&foo=bar2&foo2",   |
|   "scheme": "https",                                                      |
|   "userinfo": "user:pw&"                                                  |
| }                                                                         |
+---------------------------------------------------------------------------+

CREATE TABLE uri_table (
   url TEXT,
   url_object OBJECT GENERATED ALWAYS AS parse_uri("url")
);

This would be currently rather hard to implement in CrateDB with UDFs as the GraalVM setup doesn't include the URL API
cc: @robd003
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

@seut / @jweberpm feedback appreciated

I've tried it on both 5.0-SNAPSHOT builds and even cherry-picking this to the 4.8 branch and both are working well!

@seut / @jweberpm feedback appreciated

Looks good from all I can see and judge, ok still missing changelog/docs but otherwise looks good for me.

Any chance we could get this merged into main before the 5.0 release is cut? @seut / @mfussenegger

Any chance we could get this merged into main before the 5.0 release is cut? @seut / @mfussenegger

I think this would need some rework (i.e. split up logic between URI and URL).
i.e the query parsing is rather URL specific and would probably fail for some URIs

Any chance we could get this merged into main before the 5.0 release is cut? @seut / @mfussenegger

Unfortunately not. We already kinda hit the feature freeze cut and as @proddata pointed out the PR needs some more work.

Thanks so much for this @proddata, Happy New Year!


Spelling and bash completion fixes

Codecov Report

Merging #1137 (c890bbc) into master (a62391e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1137   +/-   ##
=======================================
  Coverage   97.17%   97.17%
=======================================
  Files          67       67
  Lines        4248     4248
=======================================
  Hits         4128     4128
  Misses        120      120

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a62391e...c890bbc. Read the comment docs.