Fetch metadata in batches

This fixes an issue with Postgres and instances with >= 32768 certificates
available to a user.
CertificateDataService.findAllValidMetadata(names) puts all cert names in a
single IN (...) SQL query, which JdbcTemplate turns into a query with a bind
variable for each cert name (IN (?, ?, ?, ...)). This exhausts Postgres's
per-query bind variable limit.
Fixes https://github.com/cloudfoundry/credhub/issues/340
The PR also includes

Local DB automation
Fixes I needed to make existing tests pass with Postgres and MySQL
A commit adding a couple of @DirtiesContext annotations, helping with resource consumption

We have this story prioritized on our backlog.

Thank you @andreasf, we have performed an initial review of the issue and this PR. It makes a lot of sense. We do notice that the following commits are not essential to the fix of the issue described & might require further validation on our part:

https://github.com/cloudfoundry/credhub/pull/346/commits/7714b81edc6a54464214cdb3be1895b940b0f775: This commit is solving a test run issue that we do not observe in our testing environment. Further, we should consider having these datasource params be configurable. We need to validate this change further.
https://github.com/cloudfoundry/credhub/pull/346/commits/8b4e58c0f40ad6b6673c3fbcbc8810fc9ba59dda: Currently, CredHub is only run with Java 8 in credhub-release. We will likely need to bump to Java 11 at some point, which will require a more systematic effort. For now, making CredHub run on Java 9 is not a prioritized effort & not essential to the fix.
https://github.com/cloudfoundry/credhub/pull/346/commits/9ec4d7faf4ce8ee139fceffaab7adb3cd6b96475: Could you describe the specific issues that this change would solve? Does it make the tests less flaky, faster, more resource-saving? We also need to look into this further.

To expedite delivering the fix to the issue, could we ask you to extract these 3 commits as 1 or more separate PRs (these 3 commits are helpful but will require more time to validate)? So that this PR only contains the commits essential to the fix (the other 2 commits look great, and probably can merge soon)?
@hsinn0 and I

We cherry-picked part of the commits  to create separate pull requests as follows:

https://github.com/cloudfoundry/credhub/pull/396
https://github.com/cloudfoundry/credhub/pull/399


Merge j17-preview branch to replace previous internal APIs and older shaded artifacts

The j17-preview branch is the current focus on development work, as it contains changes within the drivers to align to the new driver APIs with a focus on the DriverAdapter approach.
Development discussions and questions about the j17-preview branch merge should be handled on this issue. Strategic and practical reasons for the j17-preview branch merge will be explained here as well.
Foundation

Using shaded artifacts has been necessary in the past to isolate different versions of the same driver within a common runtime. For example, the DataStax Java driver 1.9 (the original NoSQLBench CQL driver), and the 3.* version could not co-exist within the same package and class structure.
Driving towards JPMS clean packaging requires the removal of shaded artifacts, as shading is a solution to the problem above before component-level encapsulation were explicitly supported by the JLS and JVM specs. Now that JPMS is effectively delivered in modern Java versions, using it is the surest and most supportable route to modular components. This is fully supported within the internal Java APIs used to load resources and classes as well.
Common dependencies for advanced features like a groovy runtime create further packaging/bundling issues, as these have package names which extend outside the scope of CI/CD tooling (user docs, etc).
Recent (CQL) driver versions have externalized the dependency management of elements like gremlin and/or groovy so that package builders can manage these concerns without having to resort to shading.
Recent refactorings around the internal driver APIs has introduced some change surface which needs to be tested further before being merged fully into the main branch.

With the inclusion of driver 4, (https://github.com/datastax/java-driver), the complexity of bundling these clients together without the benefit of JPMS mechanisms became a forcing function. While it may/might be possible to address runtime compatibility with these using the previous shading mechanism, it has reached a level of complexity which is not sustainable for the benefit provided.
Solution

In order to provide some early adopter testing of the NB CQLd4 driver and to de-risk ongoing testing operations, a reorganized (focused, cleaned up, ...) branch of NoSQLBench is being developed on j17-preview branch with releases tagged as pre-release only.
Build the j17-preview branch using Java 17 as the latest LTS release, allowing the project to come up to modern JLS features.
(on the j17-preview branch) Stop relying on the older driver versions. Remove any older driver which depends on shading.
(on the j17-preview branch) Do the work to bring the advanced testing features which wrapped the APIs of driver 1.* and 3.* to the 4.* driver.
(on the j17-preview branch) This branch will have the remaining drivers relying on the ActivityType drivers migrated over to the new DriverAdapter API. Both types are supported concurrently in the runtime, but the main merge of j17-preview branch should only happen once each and every driver is moved over to the DriverAdapter API.

The main branch is now the primary workspace for all the API improvements mentioned above. Closing this issue, as there are other more specific issues which track the remaining work.

The main branch (now nb5 development) has only nb5 APIs and compatible driver active. Shaded artifacts are by-default not included.


[Feature Request]: put highlight items at the top

Feature Description
It can either be a mode where only the highlight keywords are only visible and the rest is hidden or just put highlights at the top and the rest after it

At the top of what?

Sorry, at the top of each page, when we are going through the pages, each page has items that match the keywoards, it would make sense to put them at the top so that you can look at them and go to the next page

Yeah I can do that :)

44566515bb2ec0e288244e3fe603113cdb25eece


Added test scenario for no variable

i.e. no qualified counters for the limits

Seems safe enough... Merging


Debug why Checkboxes in docs aren't the right color

Increase selector specificity because it was losing to the ui icon definition
Closes https://jira.corp.adobe.com/browse/RSP-1795
✅ Pull Request Checklist:

[ ] Included link to corresponding Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Team:

This looks good but I have a question
Do we know why this broke when the new focusring was added?


[Bug] Visual preview not displaying anything

Insomnia Version: 5.10.1
Operating System: OS X 10.13

Details
For basic response with headers
Server: nginx/1.13.4
Date: Fri, 10 Nov 2017 09:41:32 GMT
Content-Type: text/html; charset=UTF-8
Transfer-Encoding: chunked
Connection: keep-alive
X-Powered-By: Nette Framework
X-Frame-Options: SAMEORIGIN
Access-Control-Allow-Origin: *
Access-Control-Allow-Headers: Authorization,Content-Type,Accept,Origin,User-Agent,DNT,Cache-Control,X-Mx-ReqToken,Keep-Alive,X-Requested-With,If-Modified-Since,X-Api-Key
Access-Control-Allow-Methods: GET,OPTIONS,PUT,POST,DELETE
X-UA-Compatible: IE=Edge,chrome=1

and html in body visual preview is empty(completely white) even though when switched to source code I see the expected result

Can you open the console (View > Toggle DevTools > Console) and let me know if you see any errors?

As of today I am seeing the same issue.
Previews on Source Code and Raw are working - though raw is a bit slow.
View > Toggle DevTools > Console doesn't throw any errors.
Insomnia Version 6.3.2 (<IP_ADDRESS>88)
Calling from macOS Mojave 10.14.2
Date: Wed, 02 Jan 2019 18:05:54 GMT
Server: Apache/2.4.33 (Ubuntu)
Cache-Control: no-cache, no-store, must-revalidate, max-age=0, proxy-revalidate, no-transform
Expires: Wed, 02 Jan 2019 18:05:54 GMT
Vary: Accept-Encoding
X-Frame-Options: sameorigin
Pragma: no-cache
Transfer-Encoding: chunked
Content-Type: text/html; charset=UTF-8

Hey guys. I am facing the same issue here.
I can try with any public endpoint, the result seems always the same.

I was unable to reproduce this.. hmm.  Any ideas on how I could reproduce this?
https://www.irishtimes.com

https://www.symfony.com

Hey @dimitropoulos
I am trying to figure it out. I asked some other guys to try it too and they couldn't reproduce it as well.
It's a fresh new installation, so I am not sure what can be wrong. I tried to disable all browser's extensions just to be sure (as I don't know if they cna be related to the embedded browser), but still without the preview.

@dimitropoulos
I managed to make it work. I followed some old and not understandable instructions (resetting the internet, restart the laptop and reinstalled Insomnia), and it magically worked.

glad you got it working!  if you are able to reproduce it in the future don't hesitate to reach out :)


chore: read chainId from block

Proposed Changes

CouponOnboarding + SnapshotProposalContract were changed to read the chainId from the block instead of reading it from the storage, the idea is to better handle the chain splits and keep the contracts operating.
block.chainid (uint): current chain id -> https://docs.soliditylang.org/en/v0.8.0/units-and-global-variables.html#block-and-transaction-properties

The coverage will be fixed in another PR.


Revert switch to jaeger

It was discovered that the image we wanted to use for jaeger tracing has problematic memory profiles by default. Instead of trying to tune the images and find a production-ready deployment scheme, this PR reverts the changes as a starting point -- particularly to unblock release. When a better approach to running with Jaeger is developed, we can go forward with this transition.
cc: @objectiser

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

for #5782

@douglas-reid  Are the automatic tests good enough to validate this? Is any additional / manual testing required?

The existing tests should be sufficient. We we're testing manually with this setup during the first release testing day.

I can manually verify from a daily build when next in the office.
Peter can also verify quickly.

Everything passes except flaky e2e-pilot tests.
Unfortunately that means there are no zipkin tests that I see.

Based on this recent comment it looks like zipkin has the same issue - so maybe would be better to unrevert this change and we will try to get a fix for jaeger asap?

i mentionned it yesterday too (that regpatrol/jose saw the same issue with zipkin)
what is odd is it was working before so something else changed ?
re test: pilot is where “addons” are tested afaik

https://github.com/istio/istio/pull/5840   switching back to jaeger.


docs: update OpenAI notebook

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

/azp run

Codecov Report

Merging #1999 (54e58f9) into master (161e1b1) will decrease coverage by 75.34%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1999       +/-   ##
===========================================
- Coverage   87.03%   11.69%   -75.34%
===========================================
  Files         306      306
  Lines       16062    16062
  Branches      827      827
===========================================
- Hits        13979     1878    -12101
- Misses       2083    14184    +12101

see 278 files with indirect coverage changes

/azp run

/azp run


Website not working

Hey,
I just wanted to mention that I recently was using the time-lapse app on streamlit app, and for whatever reason is no longer working for me. When trying to load the page, it now states, "Error running app. If this keeps happening, please [contact support).
https://geo.streamlit.app/?page=Create Timelapse
Brett

https://streamlit.gishub.org


Pdf2json remove

Removed pdf2json module and all its references, as we decided to have pdfminer as the (for now) only pdf extractor.

Some pdf2json references can still be found on the Dockerfile. As this is coming from AxaTechLab repository, I'm not sure about what to do with that.

Thanks @marianorodriguez !


Add support for typed arrays

Is your feature request related to a problem? Please describe.
Currently MikroORM works fine with Node.js Buffer. They are handled correctly in the EntityDTO type returned by the toObject method. Also BlobType can store them as expected in the database.
So far so good. But unfortunetely in browser applications we can only work with typed arrays. I'm working with a binary format (CBOR) in my API and data is transferred as Uint8Array since that is supported in both Node.js and browsers.
Describe the solution you'd like
I propose to support at least Uint8Array in BlobType and EntityDTO. Since Node.js Buffer basically is a Uint8Array and it usually is the drop-in  replacement when browsers are involved.
Describe alternatives you've considered
Right now two extra conversion steps (incoming and outgoing) are necessary to make things run smoothly. The database side is not that bad actually. I created a custom type that is based on BlobType. The ougoign data on the other hand requires more boilerplate code if I don't want to do without type safety - which is one reason I picked MikroORM in the first place.
Additional context
I already made the rather trivial changes locally and can provide the necessary PR if that is helpful.

I am not entirely sure what you mean, PR welcome for sure.


Announce v2 release

After we merge in V2 and transfer ownership to the drivendataorg, plan and execute public marketing of V2 release.

Any time line on this? I'm happy to use a near-finished template for now, but would be good to get the ccds tool uploaded to pypi...

See blog post here!


Finish autocomplete logic for after where

Description

Finish autocomplete logic for suggestions after the base query ( source = index )
Fix to handle query change and search differently depending on if it's from application analytics

Issues Resolved
Resolves #431
Fixes #471
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #480 (255ff45) into main (47f917d) will increase coverage by 13.81%.
The diff coverage is n/a.

:exclamation: Current head 255ff45 differs from pull request most recent head 58de0d2. Consider uploading reports for the commit 58de0d2 to get more accurate results

@@              Coverage Diff              @@
##               main     #480       +/-   ##
=============================================
+ Coverage     59.23%   73.05%   +13.81%
  Complexity      273      273
=============================================
  Files           205       38      -167
  Lines          7156     2119     -5037
  Branches       1403      235     -1168
=============================================
- Hits           4239     1548     -2691
+ Misses         2715      408     -2307
+ Partials        202      163       -39

Flag
Coverage Δ

dashboards-observability
?

opensearch-observability
73.05% <ø> (ø)

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

Impacted Files
Coverage Δ

...rds-observability/common/constants/autocomplete.ts

...lytics/components/config_components/log_config.tsx

...y/public/components/common/search/autocomplete.tsx

...lic/components/common/search/autocomplete_logic.ts

...ability/public/components/common/search/search.tsx

...ervability/public/components/explorer/explorer.tsx

...ponents/explorer/hooks/use_fetch_visualizations.ts

...ds-observability/common/constants/custom_panels.ts

...blic/components/explorer/hooks/use_fetch_events.ts

...nts/common/helpers/ppl_docs/functions/condition.ts

... and 157 more

Continue to review full report at Codecov.

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


Sort pull requests in pr list, and ensure all pull requests are rendered

📝 Description
This diff makes two changes:

Sort the PR's, first on repo name and then on pr number.
Fix a bug that resulted in one (the last) PR not being displayed

🔗 Issue ID(s): TDX-754, TDX-756
📋 Checklist

✅ Lint checks passed on local machine.
✅ Unit tests passed on local machine.

Made this diff from the same branch as the previous pr by mistake. The only changes here are the ones in the pr package.


convert pytorch model directly to caffe model

Hi,
is there some way to convert pytorch model directly to caffe model?  Thanks :)

Hi raymon-tian:
you can check pytorch2caffe.py which could convert resnet50 from torchvision models directly into caffe model.

Hi, marvis, thank your for your great work, i'm trying to convert pytorch pth file to caffemodel with your demo, but i get error like this

i am trying to figure it out but i have no idea where should i begin, so i want to check which version of pytorch and caffe on your OS, or did i miss any installment tips which you didn't mention in the ReadMe?
Many thanks.

@BowieHsu  seems your model is loaded in GPU mode, in my demo the model and variable are all in cpu mode. BTW, pytorch needs to install from source.


DocArray example adjustment

It's me again! 😂
small adjustment to the notebook, and added it into index.rst

Please don't hesitate to contribute your PRs, we welcome it warmly. 🥳

@jupyterjazz when you has completed your pull request, please give me a message 🤝

@SimFG I moved the example as you suggested so if you don't have any other requests let's merge it 🚀


Nuget source url should not be fully qualified

[No story]
Seems like the nuget source URL was resolving to https://nuget.pkg.github.com/BESTSELLER/index.json/v3/index.json for some reason. Hope removing the index.json will make it valid..

Okay, this does not work.
That GitHub action is forcing /v3/index.json to be in the nuget source URL, which does not work with GitHub packages.
Curious why you would provide a method to set a custom URL, yet make assumptions about the way URLs MUST look.
I manually pushed the packages I need, for now.

That action is archived, so I guess a swap to something else is going to be needed at some point anyway. Guess just a dotnet CLI based approach would be fine?


Update notebooks api changes expand

This PR:

Updates some API calls that were used in the example notebooks
Hot fixes a leftover from a previous rebase

In the second notebook, can we make cell [8] print(slide.level_properties(0)) show a reduced/expandable representation?

Looks great, thank you 👍


Can we add two response schemas in RAML

Hi Kurtpa,
In my project all POJO class are extending one parent class "ResponseEdo".
In my controller there is one method  whose  return type is "ResponseEdo" but the actual it returning child class ,here I am typecasting the child class to parent class "ResponseEdo" .
As the return type is "ResponseEdo" so I am getting schema of "ResponseEdo" in RAML but actually it is  returning child class.
Is there any to add two schemas in API response of RAML?

As far as i'm aware RAML does not support multiple response schemas for the same content type. Jsonschema does support polymorphism but at the moment there is no way for the generator to group all child classes together
In the Java -> RAML path,  the generator is using the method signature of the controller to decide which class to assign to the api call. You would need to put the child class as the return type of the controller method. If you need a signature with the parent for some other reason, you could possibly hide your controller method behind an interface and in the interface declare the parent class as the return type.

okay's thanks for sharing knowledge about RAML.


Custom CSS properties & styling distributed children

I have a custom element that serves as a wrapper for page content.  I am intending for this element to style its distributed children to ensure a consistent theme.  I've come across the following issue in the style portion of my dom-module.  The CSS snippet below renders paper-checkbox as shown in the attached image.
#contentWrapper > ::content paper-checkbox {
      --paper-checkbox-checked-color: red;
      --paper-checkbox-checked-ink-color: red;
      --paper-checkbox-unchecked-color: red;
      --paper-checkbox-unchecked-ink-color: red;
      background: red;
}

As you can see, the background (made red for testing purposes) renders as expected.  However, none of the custom properties seem to work.  Am I doing something wrong?

This is currently not supported.  The custom property shim only allows assignment of variables in its host element (the owner of the local dom root), not its light parent tree (where you put the ::content rule).
Minimum repro: http://jsbin.com/ruyicin/edit?html,console,output
For now, the only workaround we have to give you is to find the elements you want to style, manually set their custom style (e.g. element.customStyle['--paper-checkbox-checked-color'] = 'red';) and call element.updateStyles().
Workaround applied to above example: http://jsbin.com/zigolo/edit?html,console,output
We have ideas on how to improve the shim to support this, so we'll keep the issue open to track addition of that feature.

@kevinpschaaf Is it possible to watch for changes to distributed children?  Then I could modify element.customStyle and call element.updateStyles() more dynamically.

We are working on a "distributed children changed" event/callback right now that will definitely help your use case. You can follow this issue https://github.com/Polymer/polymer/issues/1773 for status.

@kevinpschaaf Awesome.  Thanks for pointing me to that other issue.

My situation involves a custom element that acts as a wrapper for page content.  The custom element includes all the header, search, nav, and footer elements.  Page content comes through as distributed children.  In light of the 1.1 release, the following is a better (hopefully temporary) workaround for me to enable my custom element to theme its children:
Polymer(
	{
		is: 'my-custom-element',
		ready: function () {
			var ss_link = document.createElement('link');
			ss_link.rel = 'import';
			ss_link.href = '/path/to/my-shared-styles/my-shared-styles.html';
			document.head.appendChild(ss_link);

			var ss_style = document.createElement('style', 'custom-style');
			ss_style.setAttribute('include', 'my-shared-styles');
			document.head.appendChild(ss_style);
		}
	}
);

It would be great if the docs made mention of this limitation


Collapse toggling bug

Hi.
This buggy behaviour applies to this demo page also, so to reproduce...
Clicking between the last two accordion links (Key Features and Source) will show correct collapse/expand toggle behaviour, but if you then click back to the first (How To), you will notice that the last one (Source) is still open, it has not collapsed.
If you play around with it you will notice this happens a lot (eg click first, then second, then back to first: second is still open).
NOTE: if you remove the .active classes from the markup, the problem is gone. You can then use:
$('#myTab a:first').tab('show'); // Select first tab
to show the first panel BUT this only works with the normal, bootstrap/desktop panel, it doesn't also trigger the first responsive accordion (should do).
PS, it would be good to have auto collapse/expand as an option. I would prefer to not use it as it causes unexpected page jumps during collapse when you use more realistic amounts of text in the panels.
Thanks

This is true! I am having this problem as well. I am surprised that this has slipped by the wayside.

+1

When this script calls collapse('show') on the item that has an active class, twitter bootstrap doesn't read the data-parent attribute from the html that was just made.
https://github.com/twbs/bootstrap/issues/16360
https://github.com/twbs/bootstrap/issues/8400
It looks like an object that specifies the parent has to be passed through with this call.. I am looking into this now.

I have the same problem

I solve this bug by this code:
            $('.panel-collapse').on('show.bs.collapse', function (e) {                 $(e.target).closest('.panel').siblings().find('.panel-collapse').collapse('hide');             });

Same issue here. +1
Klainer's workaround is ok.

I try fix it with @Klainer and I clicked other tab the tab, but close collapse all the tab and don't show clicked tab.

Same issue here, and trying Klainer's fix gave me the infinite loop mentioned by equiman

Is there any updated fix for this issue..!


Creating Gordon Surfaces

Discussed in https://github.com/gumyr/build123d/discussions/830

Originally posted by jsbroks December 19, 2024
I'm wondering what is the best way to create a Gordon Surface?
Here is what I have so far but as you would expect I getting Error building a non-planer face with the provided exterior.
import csv
import numpy as np
from build123d import *

def read_heightmap(filename: str):
    with open(filename, 'r') as f:
        reader = csv.reader(f)
        heightmap = []
        for row in reader:
            heightmap.append([float(val) for val in row])
    return np.array(heightmap)

heightmap = read_heightmap('heightmap.csv')

points = []
for x in range(heightmap.shape[0]):
    for y in range(heightmap.shape[1]):
        points.append(Vector(x, y, heightmap[x,y]))

x_splines = []
y_splines = []

for x in range(heightmap.shape[0]):
    points = [(x, y, heightmap[x,y]) for y in range(heightmap.shape[1])]
    x_splines.append(Spline(points))

for y in range(heightmap.shape[1]):
    points = [(x, y, heightmap[x,y]) for x in range(heightmap.shape[0])]
    y_splines.append(Spline(points))

boundary_wire = Wire.make_wire([x_splines[0], y_splines[0], x_splines[-1], y_splines[-1]])

show_object(Face.make_surface(boundary_wire))
```</div>

From @mnesarco
TiGL[1] has an OpenCascade based Gordon surface builder that is very good (c++). Christophe Grellier (@tomate44) ported it to FreeCAD's Curves WB[2] long time ago in Python. It can be used as a reference to port it to build123d. I am in the process of reorganizing the implementation as a standalone python module, but it is WIP and i can't make compromises on that [3]
[1] https://dlr-sc.github.io/tigl/
[2] https://github.com/tomate44/CurvesWB/blob/master/freecad/Curves/gordon.py
[3] https://github.com/mnesarco/MarzWorkbench/blob/master/freecad/marz/curves/gordon.py


Delete badge

sometimes there is some human error cause badge assign to specific users (like assign wrong required skills ). Event though the problem fix but there is no way to revoke the given badge, is there anyway to remove the badge?

@kstan79 thank you for reaching out. Badge achievement corresponds to badge's requirement so once all of the required skills are achieved then the badge is awarded. If a skill achievement was reported by mistake then it can be easily removed by deleting one or more of Skill Events (Navigating to Project -> Users -> Select a User -> Performed Skills)
https://skilltreeplatform.dev/dashboard/user-guide/users.html#performed-skills
After the deletion of a Skill Event for one of the badge's required skills the badge achievement will be removed by the system automatically. I hope this helps!

nice it fixed! However it maybe good for you guys add FAQ in documentation page, explain how it work! Thanks!


how to use face stable diffusion  colab to train sdxl lora?

how to use face stable diffusion  colab to train sdxl lora?Is there a tutorial here?

the notebook is fairly simple, it's the same as dreambooth, prepare the dataset, rename the images and start training


Update C2345

__declspec(align(0)) no longer generates C2345, added a high power of 2 to trigger the error.

#sign-off


[Fedora] cups.service.in: Use 'notify' service type and run after net…

…work.target

If the service is defined with 'simple' type, the result of 'systemctl' is 0 regardless of actual startup result, because it reports success/failure of forking process (even before cupsd is started). This way errors due bad configuration or programming errors are masked during systemctl invocation.
The 'notify' type depends on executable sending a 'Im running' type of message to systemd after successful start and systemctl's return code depends whether this message came or not, which solves the issue.
The service needs to be started after all units needed for network.target are activated. This prevents starting cupsd before we have ports ready.

Fedora bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1153660 (adding network.target)
https://bugzilla.redhat.com/show_bug.cgi?id=1088918 (change of the service type)

The next patches depends on the code in this PR, so I'll commit them into the same branch to prevent conflicts between PRs.

Thanks for the merge and coding style fix!
I really try to follow coding style, even if doesn't look like that :) .

@zdohnal No worries :)


Mod Options

Mod options need to be implemented for the following:

Enable/Disable the mod itself (Checkbox)
Enable/Disable each budget item (Checkbox)
Control the percentage offset of consumption for capacity (Slider)

This should be simple and straightforward, but with me being new to C# and a familiar with C++, I will need to examine the the callback methods used for GUI operations.

Implemented the first changes, but the slider doesn't quite work as expected.  The minimum granularity of the slider is too large to offer any accurate offset positions and the slider doesn't display the value that it's set at.  Either I will need to implement my own GUI element, test out the Text Input, or just leave it as is.


Added possibility to see end2end lineage for processes.

Signed-off-by: Cristian-Mihai.Iliescu<EMAIL_ADDRESS>Description
Added possibility to see end2end lineage for processes.
Added test for it.
Testing
Tested with local lineage graph

I don't see any particular reason why the gradle FVT failed. We have intermittent failures - see #6130 . Something timing/sequence related perhaps as we're failing to find an entity. Rerunning the verification here


Add variables for energy prices

This PR adds investment variables suggested by @gunnar-pik for energy prices at the relevant levels.
FYI @IAMconsortium/common-definitions-energy

@danielhuppmann Thank you for this. Just wondering, why do we continue to use USD_2010? I mean, why 2010, not 2015 or 2020? Perhaps we can specify it with a tag file so that we can change it later?

Thanks, but not sure if this would make much difference in practice...
And you can easily convert for analysis. With pyam, it's as simple as
df.convert_unit("USD_2010", "USD_2020")

using conversion factors from the iam-units package.
If you still feel strongly about this, please start a new issue.

At least 2010 is modern! GCAM still uses 1975 dollars because the model was first made in the early 1980s 😆

Wow, didn’t know this about GCAM…
OK, now understood.


python312Packages.ldap3-bleeding-edge: init at <IP_ADDRESS>7, ldeep: 1.0.60 -> 1.0.61, python312Packages.pywerview: 0.6.1 -> 0.7.0

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 340166 run on x86_64-linux 1

12 packages built:

ldeep
ldeep.dist
netexec
netexec.dist
python311Packages.ldap3-bleeding-edge
python311Packages.ldap3-bleeding-edge.dist
python311Packages.pywerview
python311Packages.pywerview.dist
python312Packages.ldap3-bleeding-edge
python312Packages.ldap3-bleeding-edge.dist
python312Packages.pywerview
python312Packages.pywerview.dist


remove will remove the element before its successor is loaded in unlink

remove starts off by removing it from elements
https://github.com/Ralith/sieve-tree/blob/460fd6aa4a416a9de1614b7a0da625b29300cbb3/src/lib.rs#L223
but later calls unlink. unlink tries to load the successor of the deleted element (let successor = elements[element].next;) but this is too late.
It looks like moving let elt = self.elements.remove(id); after unlink should work fine.

Something like this should reproduce it
#[test]
fn remove() {
    let mut tree = SieveTree::<1, 2, Bounds<1>>::new();
    let bounds = Bounds::point([0.]);
    let id = tree.insert(bounds, bounds);
    assert_eq!(tree.remove(bounds, id), bounds);
}

Fixed, thanks!


Bikeshed: top-level function wrap for outside code.

Observe the following: https://github.com/ThePhD/sol2/issues/466#issuecomment-319497147
Specifically:
int lua_openMy_Object( lua_State* L ) {
     sol::state_view lua(L);
     sol::table my_object_table = lua.create_table();
     my_object_table.new_usertype<test>("test",
          sol::constructors<test(), test(int)>(),
          "value", &test::value
     );
     int number_of_things_pushed_to_stack = sol::stack::push(lua, my_object_table);
     return number_of_things_pushed_to_stack;
}

This seems like a blahstorm waiting to happen for people who do not understand the Lua stack. We need a function that allows someone to lift the grubby details up. I propose something like sol::wrap. Example:

sol::table My_Object_requires ( /* any stuff you want */, sol::this_state L ) {
     sol::state_view lua(L);
     sol::table module = lua.create_table();
     module.new_usertype<test>("test",
          sol::constructors<test(), test(int)>(),
          "value", &test::value
     );

     return module;
}

int lua_openMy_Object( lua_State* L ) {
     return sol::wrap( L, My_Object_requires );
}

Note the lack of knowing what a stack is and how it should work. Also note that we do this internally, we just never published that internal detail (it's currently called sol::stack::call_into_lua and sol::stack::call). Do we publish this so people have an "easy out" when working with raw Lua functions?

We'll just use sol::stack::call_lua and document it in an example.

Added docs: http://sol2.readthedocs.io/en/latest/api/stack.html#stack-call-lua
Used in DLL example: https://github.com/ThePhD/sol2/tree/develop/examples/require_dll_example

Would it be possible to provide a example of calling sol::stack::call_lua which passes additional arguments besides sol::this_state to the function sol calls.  something like
static
sol::table
initialiseModule(sol::this_state state, std::string string)
{
}
{
std::string string{"Hello World!!!"};
return sol::stack::call_lua(state, 0, initialiseModule, string);
}
as with gcc (10.1.1) I have been unable to get this working?
Thanks.


Use letsencrypt free certs to not get unsecured warning

http://goroutines.com/ssl

Good idea, but the domain name(s) specified with -hosts need to be
externally visible for this to work as Let's Encrypt has to verify them by
looking for a specific response from that domain. It might be interesting
to add it as an option but at that point you should just use the
letsencrypt client.
On Sat, Jan 30, 2016 at 9:32 AM Kyle Varga<EMAIL_ADDRESS>wrote:

http://goroutines.com/ssl
—
Reply to this email directly or view it on GitHub
https://github.com/jakebasile/ssltunnel/issues/1.

You can just make it under localhost. Avoids SSL warning
On Saturday, January 30, 2016, Jake Basile<EMAIL_ADDRESS>wrote:

Good idea, but the domain name(s) specified with -hosts need to be
externally visible for this to work as Let's Encrypt has to verify them by
looking for a specific response from that domain. It might be interesting
to add it as an option but at that point you should just use the
letsencrypt client.
On Sat, Jan 30, 2016 at 9:32 AM Kyle Varga<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

http://goroutines.com/ssl
—
Reply to this email directly or view it on GitHub
https://github.com/jakebasile/ssltunnel/issues/1.

—
Reply to this email directly or view it on GitHub
https://github.com/jakebasile/ssltunnel/issues/1#issuecomment-177260410.

--
Kyle Varga

What I mean is that letsencrypt needs to verify your domain from their servers. They can't verify that      localhost means your machine because localhost isn't defined by DNS, so it can't access your system.
An idea would be to add the generated self-signed cert to your local trust store, but that's both kind of creepy and a little out of scope I think.


Missing currency in input when paying with account balance

Followup on https://github.com/opencollective/opencollective-frontend/pull/7530
Steps to reproduce:

Go to submit expense page
Select a collective as a payee and "Account Balance" for the payout method
See missing currency

It actually breaks the form, the "Next" button seems disabled

Note that this was initially reported in the feedback


Table 33 - age 1 q values missing

For the models Phi TV selectivity.

Fixed this with this download:
s3_quick_download(c("models/2022.01.24_compWeight_HarmonicMean/2022.01.24_compWeight_HarmonicMean.rds",
                    "models/2022.01.27_tvSelect_phi_extralow/2022.01.27_tvSelect_phi_extralow.rds",
                    "models/2022.01.28_tvSelect_phi_low/2022.01.28_tvSelect_phi_low.rds",
                    "models/2022.01.29_tvSelect_phi_high/2022.01.29_tvSelect_phi_high.rds"))

[ ] Andy
[ ] Kelli

Kelli I don't know if you're compiling now, if not let me know and I won't make a box for you next time


Brownfield - Urgent Fix for App Attach VM Deployment

Overview/Summary
URGENT Fix: Brownfield for App Attach VM was not configuring the VM via post deployment script as it was using a URL for Github that was not the "Raw content" version thus the PS1 on the VM was an HTML download.  Adjusted the URL to correct the issue. This impacted all previous deployments since it was added to the accelerator.
This PR fixes/adds/changes/removes

Variable with Post Deployment URL now correct

Breaking Changes
N/A
Testing Evidence
Re-deployed from updated code via a Template Spec and script downloaded and executed successfully. Easily noted by script's output log and the configuration for the VM was correct with added App Attach Tools.
As part of this Pull Request I have

[X] Read the Contribution Guide and ensured this PR is compliant with the guide
[X] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[X] Checked for duplicate Pull Requests
[X] Associated it with relevant GitHub Issues
[X] (AVD LZA Team Only) Associated it with relevant ADO Items
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[X] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Docs etc.)

@JCoreMS thanks for your contribution.


Getting an "error: 'list' object has no attribute 'items' downloading a video from Rumble

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
US
Provide a description that is worded well enough to be understood
Downloading this video with yt-dlp is resulting in something that looks like a python error
Rumble URL:
https://rumble.com/v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html
Exact yt-dlp options I used:
yt-dlp --write-thumbnail -S res,ext:mp4:m4a --recode mp4 -o "/tmp/%(title)s.%(ext)s" https://rumble.com/v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html
Simplified yt-dlp options I tried:
yt-dlp https://rumble.com/v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html
Exact error I got, either way I tried it:
[Rumble] Extracting URL: https://rumble.com/v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html
[Rumble] v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html: Downloading webpage
[RumbleEmbed] Extracting URL: https://rumble.com/embed/v2ezmdm
[RumbleEmbed] v2ezmdm: Downloading JSON metadata
ERROR: 'list' object has no attribute 'items'

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
% yt-dlp https://rumble.com/v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html -vU
[debug] Command-line config: ['https://rumble.com/v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html', '-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (pip)
[debug] Python 3.10.11 (CPython arm64 64bit) - macOS-12.6.2-arm64-arm-64bit (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: ffmpeg 6.0 (setts), ffprobe 6.0
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-11.0.1
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[Rumble] Extracting URL: https://rumble.com/v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html
[Rumble] v2hl1om-is-a-trumpkennedy-2024-ticket-around-the-corner-on-occams-razor-ep.-281.html: Downloading webpage
[RumbleEmbed] Extracting URL: https://rumble.com/embed/v2ezmdm
[RumbleEmbed] v2ezmdm: Downloading JSON metadata
ERROR: 'list' object has no attribute 'items'
Traceback (most recent call last):
  File "/opt/homebrew/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1518, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1594, in __extract_info
    ie_result = ie.extract(url)
  File "/opt/homebrew/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 694, in extract
    ie_result = self._real_extract(url)
  File "/opt/homebrew/lib/python3.10/site-packages/yt_dlp/extractor/rumble.py", line 167, in _real_extract
    for height, video_info in (ext_info or {}).items():
AttributeError: 'list' object has no attribute 'items'

Duplicate of #6612. Already fixed in master.


No Typescript types for react-tsparticles

When importing the particle component as described in the docs:
import Particles from "react-tsparticles";

I don't get any types for the props of the component. I can however import like the following:
import Particles, {IParticlesParams} from "react-tsparticles";

and then use those types for the options, but it's not ideal.
Typescript Version: 4.0.3
react-tsparticles Version: 1.17.22

I don't understand what you are expecting to be exported, the default export is one and it's easier to have the Component exported.
The IParticlesParams is a type that is not always needed to be imported since an object with a valid structure is accepted by the options parameter.
If you can explain further the issue I can evaluate it better.

I am not expecting anything to be imported, but rather that the default exported component's props are typed, which they arent for me

The options attribute is typed, also the obsolete params is still supported and typed.
I don’t really understand what is the problem
export interface IParticlesProps {
    id?: string;
    width?: string;
    height?: string;
    options?: ISourceOptions;
    params?: ISourceOptions;
    style?: CSSProperties;
    className?: string;
    canvasClassName?: string;
    container?: RefObject<Container>;
}

These are the properties exported from 1.18 (1.17 has less optional and a more complex options type).
Can you try using 1.18 (beta for now) and tell me it the issue persist?

I saw you merged 1.18 so I just upgraded to that and the problem persists. I created a codesandbox so you can see it in action: https://codesandbox.io/s/nifty-tesla-06sg1?file=/pages/index.js

Thanks! I've understood and found the issue. It will be solved in 1.18.1 that will be released later today.

@matteobruni Great, thanks alot for that :)

1.18.1 is out, let me know if everything works fine now

@matteobruni the issue persists in 1.18.1 as you can see in the same codesandbox: https://codesandbox.io/s/nifty-tesla-06sg1?file=/pages/index.js

I tried autocompleting on CodeSandbox but it doesn't work fine even in simple HTML tags like h1.
WebStorm instead had a good autocompletion in 1.18.1.

the reason for that was because @types/react was not installed, installed it now and it is autocompleting stuff on the h1 tag, but still not on the Particles Component. I also have the same experience in vscode. Do you get types on the Paricles component in Webstorm?
https://codesandbox.io/s/nifty-tesla-06sg1?file=/pages/index.tsx

I've done more tests with VSCode, the demo project inside this repository was working fine, but I've found some tricks to make it work for other projects by adding some JSDoc comments.
I'll release it later today with the fix, this should be the definitive solution (at least I hope it is)


[do we need it?]   particular button events ?

instead of validating buttons with id and event type in events file

instead we can add buttons in following files with proper events!!
src/buttons/unban.ts
or
src/events/unban.button.ts

@ArnavK-09 Mmm, I'm not quite sure what you mean. Are you suggesting we place these button files inside the /src/buttons directory?
I'm not against it, but the reason we place these button callbacks in /src/events/interactionCreate is because they're all subsets of the interactionCreate Discord event. We'd still need to forward them somehow, preferably in a decentralized fashion.

Well, isn't that the same thing you do with slash commands already? They're also triggered by interactionCreate, but forwarded to the individual files in src/commands. A src/buttons is my number 1 feature request.

@OfficialCRUGG  Ah, I understand what you both mean now. Yes, I think that'd be an excellent feature, especially if there's an automatic way to pass the state to those.
We'll definitely look into supporting this!

Yup

Discussed in discord
Also just realised there is template to reach similiar results
https://github.com/Wave-Play/robo.js/tree/main/templates%2Fdiscord-bots%2Fmrjawesome-dev-toolkit-js


A personal body of knowledge - Integrate Chatpal and Smarti

Motivation
Until now, Smarti was focusing on creating a common, anonymized body of knowledge for a team based on its conversations.
Real user interaction has shown that people don't care much about creating coherent conversations but write in a continuous stream.
It is crucial that even in these cases, Smarti can deliver value in order to accelerate the conversation.
Suggestion
With Chatpal being a part of Rocket.Chat, it provides an explicit search with a focus on individual, message-based knowledge. Thus, considering Chatpal as an additional knowledge source in Smarti seems reasonable. This way, Smarti can provide an implicit, NLP-backed, federated search with the personal body of knowledge by Chatpal.

Technical remarks as discussed with @westei
The Chatpal-provider is similar to a conversational provider: The query to be executed shall be generated by Smarti, with all bells and whistles about partial matching, weighting and so on. This potentially complex query has to be issued to the Chatpal index.
However, the Chatpal index must not be consumed directly: The Rocket.Chat Search Backend adds restricting parameters based on the accessibility of rooms.
Consequently, Chatpal "only" needs to expose a method to the widget which receives the prepared query and returns search results.

FYI this is the main story for the v0.8.* releases of Smarti. Currentl it is planed out in two releases:

0.8.0 release with all features that are possible without changes in the current Rocket.Chat Search implementation and the Chatpal
0.8.1 release that depends on the acceptence of Recket.Chat Search pull requests and releases of those in a Rocket.Chat release as well as additional features for the Chatpal provider

@westei can you let us know the chatpal RC PRs? We can merge them into our fork at least

@mrsimpson the two issues (#278 and #288) are now implemented (they do require assistify/Rocket.Chat#601). With that work on this project has completed. So I suggest to also markt this meta issue as complete after review.
The release of  #278 and #288 is now scheduled as part of 0.9.0


BUG: <dark/light mode fix>

Description
Light/dark mode icon can be switched when clicked i.e there should be one icon and a function that changes the icon when it's clicked depending on the switch.
If the icon is on dark mode, when clicked, the icon should change to light mode icon.
Something like this:
https://user-images.githubusercontent.com/69390818/216075003-a49dd932-ac27-4388-93b0-6f2df3242e04.mov
Screenshots
No response

Sorry @i-am-rita this issue doesn't concern this repo

Hello, @i-am-rita I would love to hear more thoughts on this. Does it affect UX in any way?

Hi Njong392,
Yes, it does to me.
A user should not have to move their mouse away from one of the modes to
change it when he/she can stay on one and toggle it.
The light/dark mode acts like a switch. In a house, there isn't a separate
switch for light or dark, it's the same switch.
I hope I was able to explain well.
Kind regards,
Rita
On Sat, 4 Feb 2023 at 12:35 pm Njong Emy @.***> wrote:

Hello, @i-am-rita https://github.com/i-am-rita I would love to hear
more thoughts on this. Does it affect UX in any way?
—
Reply to this email directly, view it on GitHub
https://github.com/Njong392/Abbreve/issues/239#issuecomment-1416729504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQRNDYUST3SX3LBLKFOXAYDWVY5JFANCNFSM6AAAAAAUN2GNKA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I think in this case we can consider the distance between the buttons. They are close together and would require the same amount of clicks from the user to switch between modes. The UX between both scenarios will not be too different. I understand your point and am trying to do some research.

Ok, thanks.
Rita
On Sat, 4 Feb 2023 at 12:55 pm Njong Emy @.***> wrote:

I think in this case we can consider the distance between the buttons.
They are close together and would require the same amount of clicks from
the user to switch between modes. The UX between both scenarios will not be
too different. I understand your point and am trying to do some research.
—
Reply to this email directly, view it on GitHub
https://github.com/Njong392/Abbreve/issues/239#issuecomment-1416732957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQRNDYVYJTN6OFOKWDFIFVTWVY7SPANCNFSM6AAAAAAUN2GNKA
.
You are receiving this because you were mentioned.Message ID:
@.***>

thought about this too
i might open a PR on this soon

thought about this too
i might open a PR on this soon

Please don't open a PR yet. I still looking into this.

thought about this too
i might open a PR on this soon

Please don't open a PR yet. I still looking into this.

no prop
you wont merge it it will just be there pending on when you finish looking into it

Hey @Njong392 I Think this is a good idea to have only one button to change the theme instead of having two buttons. I hope you will look into it.

This issue really skipped my mind already. I'll have a response by the weekend hopefully. In the meantime, who should I assign it to?

You can assign it to me

In my opinion, having one button controlling the modes would be fantastic for UX. Thanks !

Hi @i-am-rita , if no one is working on this, can you please assign to me.  will create a single button that allows the user to toggle between dark and light mode, resulting in the same output as the selected theme.


ERROR: http://mirror.envygeeks.io/alpine/v3.5/main: DNS lookup error

Hi,
I am using the following docker-compose.yml file

version: '2'
services:
site:
command: jekyll serve
image: jekyll/jekyll:builder
volumes:
- .:/srv/jekyll
- ./vendor/bundle:/usr/local/bundle
ports:
- 4000:4000
- 35729:35729
- 3000:3000
-   80:4000

Trying to run the following commands to create my new site:

docker-compose run site jekyll new site

This fails with the following errors:
C:\projects\jekyll\test2>docker-compose run site jekyll new site
ERROR: http://mirror.envygeeks.io/alpine/v3.5/main: DNS lookup error
ERROR: http://mirror.envygeeks.io/alpine/v3.5/community: DNS lookup error
ERROR: http://mirror.envygeeks.io/alpine/edge/community: DNS lookup error
ERROR: http://mirror.envygeeks.io/alpine/edge/testing: DNS lookup error
ERROR: http://mirror.envygeeks.io/alpine/edge/main: DNS lookup error
Am I missing anything, how can I resolve this issue?
tnx!

Spending 4 hours on an issue yourself, writing 1 single post on Github and 2 minutes later you find the problem youself.
I was using the wrong image :)
It should be image: jekyll/builder:latest

Glad you were able to figure it out on your own. :smiley:

We might have a build problem, we should be aliasing jekyll/jekyll:builder to jekyll/builder:latest at build time, as for the DNS resolution error, we removed that almost 6 months ago when the bill for the mirror got too high, so we reverted to Alpine's mirror when they finally got funding to get a CDN.


Configuration for AWS SSO

Hello! I am trying to get djangosaml2 working with AWS SSO. I have configured it based on your docs, but haven't got it to work yet. Is there a known configuration required to be compatible with AWS SSO?
Thanks!

Sure It Will work, what's the problem you're encountering?

Thanks for confirming it should work. After I log into SSO, I get the following from my django app:
Authentication Error.
Access Denied.
Here is my SSO attribute mapping:

I sent detailed information to AWS Support and one of the things they pointed out was the following:
_"From the above, we can see that the application is requesting name-id’s format to be set to “entity”.
→ urn:oasis:names:tc:SAML:2.0:nameid-format:entity
But from the screenshots of your attribute mapping (attr.png), the uid attribute is mapped as “unspecified”. As of now the only support values for nameid-format are: unspecified, emailAddress, persistent and transient."_

see djangosaml2 logs
format should not be unspecified but uri as I see in urn:oid: ...
attribute MUST be mapped in attributes-map
use firefox saml2-tracer to debug a response and see which attributes comes and in which format, these must be mapped in your SP, otherwise djangosaml2 won't detect the attributes and the ava dictionary will be void

Thanks. Sorry the attribute map I posted earlier was not the latest. Here is what I currently have.

I am confused about the name-id format that AWS support is referencing. They are saying my SP is using "entity" which is not a supported option. However, I can't find where in djangosaml2 this is configured.
Sorry, I am completely new to SAML2, do I need to specify the "uri" format for the other "Subject" and "uid" attributes as well?

I will also try to use firefox saml2-tracer.

Actually, I previously captured an HAR file with Chrome. I assume that is similar to what saml-tracer provides? This is where the issue of the name-id format came up after AWS support reviewed the HAR file.

Post AWS Saml response here as references, let's take a look in it

OK, I will do a fresh capture.

OK, I found out something interesting. If I am already logged into AWS SSO and go to my saml2/login page, it looks like it is actually logging in, but then fails when it redirects to /accounts/profile:
User XYZ authenticated via SSO.
Redirecting to the RelayState: /accounts/profile/
Not Found: /accounts/profile/
Not Found: /accounts/profile/

I went back and deleted the user from the admin_user table and then tried again after logging out of SSO (the way I originally tried it). Then it fails with this:
Saving the session_id "{'_db': {'id-rpOTWhybM5AtZKBIr': '/accounts/profile/'}}" in the OutstandingQueries cache
Unsolicited response
XML parse error: Unsolicited response: id-rpOTWhybM5AtZKBIr
Received SAMLResponse when no request has been made.
Traceback (most recent call last):
	  File "/usr/local/lib/python3.6/site-packages/djangosaml2/views.py", line 343, in post
	    outstanding_queries)
	  File "/usr/local/lib/python3.6/site-packages/saml2/client_base.py", line 812, in parse_authn_request_response
	    xmlstr, AuthnResponse, "assertion_consumer_service", binding, **kwargs
	  File "/usr/local/lib/python3.6/site-packages/saml2/entity.py", line 1455, in _parse_response
	    response = response.loads(xmlstr, False, origxml=xmlstr)
	  File "/usr/local/lib/python3.6/site-packages/saml2/response.py", line 534, in loads
	    "Unsolicited response: %s" % self.in_response_to)
	saml2.response.UnsolicitedResponse: Unsolicited response: id-rpOTWhybM5AtZKBIr
	Forbidden: /saml2/acs/
	Forbidden: /saml2/acs/

If this is not enough, I will dig into the HAR file to look for more detailed information.

Change in settings login_redirect_url... Se Django official configuration for this

Thanks, I will look. Is this to address the "Not Found: /accounts/profile/" problem or the "XML parse error: Unsolicited response: id-rpOTWhybM5AtZKBIr" problem?
BTW, here are the attributes in the SAMLResponse:
<saml2:AttributeStatement>
<saml2:Attribute Name="uid" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:unspecified">
    <saml2:AttributeValue xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:type="xsd:string">XYZ email</saml2:AttributeValue>
</saml2:Attribute>
<saml2:Attribute Name="urn:oid:0.9.2342.192003<IP_ADDRESS>" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
    <saml2:AttributeValue xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:type="xsd:string">XYZ email</saml2:AttributeValue>
</saml2:Attribute>
</saml2:AttributeStatement>

OK thanks, I understand why I am getting the "Not Found: /accounts/profile/" in the first scenario and how I can fix it by setting LOGIN_REDIRECT_URL.
For the second scenario, where I am getting the "XML parse error: Unsolicited response: id-rpOTWhybM5AtZKBIr" error. Does this have to do with a session or attribute issue?

No, it's a response not related to any request...
You can even configure allow_unsolicited_response to True but I'd try to investigate some more of I were you

OK, thanks. Any suggestions where to look? I believe this happens every time, not just randomly.

take a look here:
https://github.com/IdentityPython/pysaml2/issues?q=is%3Aissue+aws+is%3Aclosed
remember, it MUST works!

@peppelinux Setting allow_unsolicited to True does fix, or at least work-around the problem. So far I haven't been able to find a good explanation why this might be needed if the user starts from the login on the SP. I think I understand why this would be necessary if the user started from the IdP.

do you have configured samesite cookie middleware?

I looked up information on this and since I am using django 3.1, it doesn't
seem I have to include this middleware directly. However, I am still not
clear if there is some samesite configuration needed that is different from
the default for this to work. Can you elaborate on how this could be
related to my problem?
On Wed, Apr 28, 2021 at 3:14 AM Giuseppe De Marco @.***>
wrote:

do you have configured samesite cookie middleware?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/peppelinux/djangosaml2/issues/265#issuecomment-828208334,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4PJ4JI6ZM6O24S5PL35FDTK6YUPANCNFSM43PHBPMA
.

--
NOTICE: The information contained in this e-mail message is intended for
the exclusive use of the addressee(s) and may contain proprietary
information. Any proprietary information contained herein should be handled
per executed NDAs. If you are not the intended recipient of this message,
you are hereby notified that any form of dissemination of this
communication is strictly prohibited . Please note that any views or
opinions presented in this email are solely those of the author and do not
necessarily represent those of the company. Finally, the recipient should
check this email and any attachments for the presence of viruses. The
company accepts no liability for any damage caused by any virus transmitted
by this email. NSV Inc, Suite# 306, 7540 Windsor Drive, Allentown, PA, USA,
www.nsvtech.com http://www.nsvtech.com

djangosaml2 comes with a samesite cookie Middleware, do you have configured It in your project?
You find an example in official docs

@terrylimnsv do you have any update on this issues?

@peppelinux Not yet, I had to put it on temporary hold. Hopefully I can get back to it later this week.

try to update djangosaml2, we have some important bugfixes related to samesite cookies and django version used

@peppelinux I updated from from 1.0.5 -> 1.1.5, but it did not fix my problem when 'allow_unsolicited': False.
I am using django 3.1 and have the following SameSite cookie config in settings.py as per the documentation:
MIDDLEWARE = [
'django.middleware.security.SecurityMiddleware',
'django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.common.CommonMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware',
'djangosaml2.middleware.SamlSessionMiddleware',
]
SAML_SESSION_COOKIE_NAME = 'saml_session'
Are there any other settings I may be missing?

Looks really strange
Which python version?
py37?

@peppelinux I am using Python 3.6.7. Do you recommend something different?

Yes, use python3.8 and if you have problems in your environment with this python version use pyenv

OK, I will try 3.8. My environment is a docker container, so it should not be an issue.

Have you tried bumping the log level to debug?  If so what errors do you
see?  Is it "unsolicited request"?
On Thu, 6 May 2021 at 15:50, terrylimnsv @.***> wrote:

OK, I will try 3.8. My environment is a docker container, so it should not
be an issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/peppelinux/djangosaml2/issues/265#issuecomment-833585227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB2CY2IAPIPGK72IR56H6LTMKUD7ANCNFSM43PHBPMA
.

When you get the error do you see both saml_session and sessionid cookies?
If so, do they both have samesite=None set?
On Thu, 6 May 2021 at 16:30, terrylimnsv @.***> wrote:

Changing to 3.8 did not seem to help. In this scenario, I am already
logged into AWS SSO and I click on my application on the SSO page. I get
re-directed to my django app which shows:
Authentication Error. Access Denied.
Here is the log:
[image: image]
https://user-images.githubusercontent.com/79230193/117324856-338d7700-ae5e-11eb-993c-568db63f735c.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/peppelinux/djangosaml2/issues/265#issuecomment-833616215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB2CYYDGEBSL67NQ7CFABDTMKYYFANCNFSM43PHBPMA
.

OK, I think you are onto something. I looked at the response header and I see saml_session with the following warning:
This Set-Cookie was blocked because it had the "SameSite=None" attribute but did not have the "Secure" attribute, which is required in order to use "SameSite=None"
I don't know if it matters, but to capture this info from the Chrome Developer Tools, I started from my django app this time.

Ah, this will be it.
Presumably you just need to set:
https://docs.djangoproject.com/en/3.2/ref/settings/#std:setting-SESSION_COOKIE_SECURE
On Thu, 6 May 2021 at 16:54, terrylimnsv @.***> wrote:

OK, I think you are onto something. I looked at the response header and I
see saml_session with the following warning:
This Set-Cookie was blocked because it had the "SameSite=None" attribute
but did not have the "Secure" attribute, which is required in order to use
"SameSite=None"
I don't know if it matters, but to capture this info from the Chrome
Developer Tools, I started from my django app this time.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/peppelinux/djangosaml2/issues/265#issuecomment-833634232,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB2CY3QLFCINXS6QU7XXV3TMK3RTANCNFSM43PHBPMA
.

See:
https://github.com/peppelinux/djangosaml2/blob/master/djangosaml2/middleware.py#L74
On Thu, 6 May 2021 at 19:11, Lyndon Garvey @.***> wrote:

Ah, this will be it.
Presumably you just need to set:
https://docs.djangoproject.com/en/3.2/ref/settings/#std:setting-SESSION_COOKIE_SECURE
On Thu, 6 May 2021 at 16:54, terrylimnsv @.***> wrote:

OK, I think you are onto something. I looked at the response header and I
see saml_session with the following warning:
This Set-Cookie was blocked because it had the "SameSite=None" attribute
but did not have the "Secure" attribute, which is required in order to use
"SameSite=None"
I don't know if it matters, but to capture this info from the Chrome
Developer Tools, I started from my django app this time.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/peppelinux/djangosaml2/issues/265#issuecomment-833634232,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB2CY3QLFCINXS6QU7XXV3TMK3RTANCNFSM43PHBPMA
.

Doocuumentaaatiooooon :)
this MUST be there!
@terrylimnsv would you like to do a PR for this?

@peppelinux, I apologize if I missed this in the djangosaml2 documentation. I just looked again and I didn't see any reference to SESSION_COOKIE_SECURE. Maybe I am looking in the wrong place.

@terrylimnsv no you're right, that's misding in djangosaml2 docs, I would like to Asl you this contribution. Feel free

@peppelinux Thanks for checking! I am new to GitHub and not clear on what you are asking regarding this contribution.

Don't worry, I'll keep this issue opened untill I'll made an update to djangosaml2 documentation, to include this important detail
thank you @lgarvey


Doesn't work with Youtube Music

Phone: Google Pixel 3
OS: Android Pie 9 (april update)
Issue: It works with Play Music, but doesn't with YT Music. I saw a comment displaying the same issue on the XDA developers article regarding this app, but the user had Android Q 10.0.

Can confirm issue -- logcat attached. The app does seem to be doing its job...
logcat.log

Yeah same here, the app works with my standard music player but not with the YouTube Music app. Android 8.1.0 on an LG


Chown the builds before checkout

This PR adds a
chown -R  buildkite-agent.buildkite-agent /var/lib/buildkite-agent/builds

before doing a checkout.
This works around permission issues caused volume mounts leaving files owned by root in the checkout.
I think the long term aim should be https://github.com/buildkite/buildkite-aws-stack/issues/30, as it fixes the underlying security issue and not just the symptom, but it needs more thorough testing.

This fixes #30


C-language code-block lexing error with specific comment syntax "//@"

I'd reported this originally here - https://github.com/sphinx-doc/sphinx/issues/9548 - but it turns out that Sphinx is using pygments highlighter which independently shows the same behaviour I reported there. So copying the issue over.
Describe the bug
The following code block causes a C-language "lexing error" to be reported and highlighting skipped.
.. code-block :: c

    int div(int x, int y)
    //@requires y >= 0;
    {
        return x / y;
    }

On the other hand, if I just remove the trailing semicolon on the comment line, the error goes away. i.e. the below version lexes fine.
.. code-block :: c

    int div(int x, int y)
    //@requires y >= 0
    {
        return x / y;
    }

.. and the following lexes fine too -
.. code-block :: c

    int div(int x, int y)
    // requires y >= 0;
    {
         return x / y;
    }

Since the line in question is a comment line, what characters feature within the comment line should not be of any consequence for lexing the block as "C".
How to Reproduce
Include the example code blocks shown in any rst documentation processed by Sphinx. The following note by @jakobandersen shows how to reproduce it using pygmentize.

Thanks for reporting, though the lexing and highlighting is under the hood done with Pygments. It seems that running
pygmentize -l c -F raiseonerror test.c emulates what Sphinx does, and for your first example I indeed get an error:

*** Error while highlighting:
pygments.filters.ErrorToken: @
   (file "/home/jla/.local/lib/python3.8/site-packages/pygments/filters/__init__.py", line 782, in filter)
*** If this is a bug you want to report, please rerun with -v.
int div(int x, int y)

Expected behavior
All the indicated C-language code blocks should lex fine since the line in question is a comment line.

Hi, I'm a student from the University of Adelaide. I'm working in a team of 5 on contributing to an OSS project. The duration of the project is 8 weeks. We would like to tackle this issue, if it is still available. Our current proposed timeline aims to have the issue fixed at the start of May. The lecturer for this course has already approved this issue as appropriate for our project.


Mass transfer feature

Once in a while, you want to send more NFTs to various addresses or transfer more NFTs to your second wallet. We don't have a way to do it!
Can we please think about having a mass transfer feature?
Where it can be used

giveaway and competitions
when transferring more NFTs to another wallet

It will make things faster!
@kodadot/internal-dev, how hard is it to implement that?

@kodadot/internal-dev, how hard is it to implement that?

we already have something similar with mass listing, surely we can reuse some of that code
design is needed cc @exezbcz

sure, on it

Design
available in: https://www.figma.com/design/JXNtryaNYhY0XJNArzvPQx/Handoff?node-id=2734-21138&node-type=canvas&t=oxF8FbhJCCMbV6sW-11

shares a lot from transfer component, offer notification system, signing. Main part where the items are displayed in the modal is taken from the success modal when mass buying
Toast when selecting items

here is a small change: new button for transfer alongside the list items button

they are both medium buttons, check the padding
buttons follow button style guide

After clicking transfer
you are presented with a modal that has a list of items

its the same list you have when mass buying
transfer to:

same as a transfer modal for single nfts

don't forget about different states for the input
more here:

https://www.figma.com/design/JXNtryaNYhY0XJNArzvPQx/Handoff?node-id=1287-18959&node-type=canvas&t=oxF8FbhJCCMbV6sW-11
Sign and toast

there won't a be congrats modal, there is not many converting actions after the transfer - it means it will be a progress toast, and then congrats

same as offers just with different copy and in the congrats toas there is a link for the transaction
other

let me know if i have missed anything
mass transfer is disabled on mobile since you cannot hover over the card and select it.


Do not ignore preventOverrun in CronJob

Fixes #195

@Hexagon Could you take a look? It looks like after the overrun protection is enabled with the true flag, it never gets released.

Any updates on this? Is something wrong with the PR?

@boristian yes, functionality appears to be broken on @hexagon end

Hello! Cannot find anything wrong with croner, tested both<EMAIL_ADDRESS>used by toad-scheduler, and<EMAIL_ADDRESS>which is the latest iteration.
Tested using this code:
import { Cron } from<EMAIL_ADDRESS>
// Demo blocking function
const blockForAWhile = (ms) => new Promise(resolve => setTimeout(resolve, ms));

// (Optional) Callback to be triggered on a blocked call
const protectCallback = (job) => console.log(`Call at ${new Date().toISOString()} were blocked by call started at ${job.currentRun().toISOString()}`);

Cron(
    "*/2 * * * * *",
    {
        protect: protectCallback // <- Using a callback for overrun protection, but `true` would work just as well
    },
    async (job) => {
        console.log(`Call started at ${job.currentRun().toISOString()} started`);
        await blockForAWhile(6000);
        console.log(`Call started at ${job.currentRun().toISOString()} finished ${new Date().toISOString()}`);
    }
);

With this output
codespace@codespaces-56134a:/workspaces/croner master [?]> deno run test.ts
Call started at 2023-11-13T19:21:44.003Z started
  Call at 2023-11-13T19:21:46.002Z were blocked by call started at 2023-11-13T19:21:44.003Z
  Call at 2023-11-13T19:21:48.001Z were blocked by call started at 2023-11-13T19:21:44.003Z
  Call at 2023-11-13T19:21:50.002Z were blocked by call started at 2023-11-13T19:21:44.003Z
Call started at 2023-11-13T19:21:44.003Z finished 2023-11-13T19:21:50.005Z
Call started at 2023-11-13T19:21:52.001Z started
  Call at 2023-11-13T19:21:54.006Z were blocked by call started at 2023-11-13T19:21:52.001Z
  Call at 2023-11-13T19:21:56.007Z were blocked by call started at 2023-11-13T19:21:52.001Z
Call started at 2023-11-13T19:21:52.001Z finished 2023-11-13T19:21:58.002Z // <- Callback runs before actually unblocking, that's why this call is out of order
  Call at 2023-11-13T19:21:58.007Z were blocked by call started at 2023-11-13T19:21:52.001Z
Call started at 2023-11-13T19:22:00.002Z started
  Call at 2023-11-13T19:22:02.007Z were blocked by call started at 2023-11-13T19:22:00.002Z
  Call at 2023-11-13T19:22:04.006Z were blocked by call started at 2023-11-13T19:22:00.002Z
Call started at 2023-11-13T19:22:00.002Z finished 2023-11-13T19:22:06.004Z

@boristian @kibertoad Can you check the comments i made below?
    it('allows preventing CronJob execution overrun for async tasks', async () => {
      let counter = 0
      const scheduler = new ToadScheduler()
      const task = new AsyncTask('simple task', () => { // <- 1. The callback isn't flagged with async, is this correct?
        counter++
        advanceTimersByTime(6000) // Do this really delay the return of this async function?
        return Promise.resolve(undefined)
      })
      const job = new CronJob(
        {
          cronExpression: '*/2 * * * * *',
        },
        task,
        {
          preventOverrun: true,
        }
      )
      scheduler.addCronJob(job)

      expect(counter).toBe(0)           // Counter is expected to be 0, yes
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      expect(counter).toBe(2) // 5 seconds elapsed, but advanceTimersByTime in callback is advanced 6 seconds... Counter should still be 1,
      advanceTimersByTime(1000) // somewhere around now
      advanceTimersByTime(1000) // or now - will the second callback call
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      expect(counter).toBe(5)         // Counter should be around 2 here, not 5
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      advanceTimersByTime(1000)
      expect(counter).toBe(7)
      scheduler.stop()
    })
    ```


When importing module using substreams.yaml, substreams is looking for binary in the incorrect folder

Example:
I have two substreams (erc721 and uniswap) with the following folder structure:
- erc721
  - substreams.yaml
  - target/.../substreams_erc721.wasm
- uniswap
  - substreams.yaml
  - target/.../substreams_uniswap.wasm

When I import the erc721 substream in uniswap's substreams.yaml, and build, I see the following error:
~/go/bin/substreams protogen ./substreams.yaml --exclude-paths="sf/ethereum,sf/substreams,google"
Error: reading manifest "./substreams.yaml": error loading imports: importing "../erc721/substreams.yaml": failed to convert manifest to pkg: failed to read source code "./target/wasm32-unknown-unknown/release/substreams_erc721.wasm": open ./target/wasm32-unknown-unknown/release/substreams_erc721.wasm: no such file or directory

However, if I copy erc721/target/.../substreams_erc721.wasm to under uniswap/target/.../substreams_erc721.wasm the error goes away. I think substreams is looking for the binary in the main substreams.yaml instead of the imported substreams.yaml.
See https://github.com/messari/substreams/blob/2ef4b9278c8a6d372c48e1df8c4be869081f481b/uniswap-v2/substreams.yaml#L8 or https://github.com/messari/substreams/commit/2ef4b9278c8a6d372c48e1df8c4be869081f481b

Fixed by e36a3f87ec38699868b6befe8573bbde09d694cc


Show Axiophytes Only

"Show axiophytes only" should also be in the 'Find Records for Sites' search box.
Axiophytes are a list of indicator species.  The list changes rarely (2 or 3 years).  The list is established by the County Recorders.

Would require filtering of the data returned from NBN.  Can NBN queries accept a list of acceptable returns?


Configure OpenSSH according to Mozilla Infosec guidance.

We need to configure OpenSSH to have more secure settings that can be shared by multiple versions of OpenSSH installed on different platforms. We need to create configuration in /etc/ssh/sshd_config similar to proposed below:
HostKey /etc/ssh/ssh_host_ed25519_key
HostKey /etc/ssh/ssh_host_rsa_key
SyslogFacility AUTHPRIV
AuthorizedKeysFile      .ssh/authorized_keys
KexAlgorithms curve25519-sha256@libssh.org,ecdh-sha2-nistp521,ecdh-sha2-nistp384,ecdh-sha2-nistp256,diffie-hellman-group-exchange-sha256
Ciphers chacha20-poly1305@openssh.com,aes256-gcm@openssh.com,aes128-gcm@openssh.com,aes256-ctr,aes192-ctr,aes128-ctr
MACs hmac-sha2-512-etm@openssh.com,hmac-sha2-256-etm@openssh.com,umac-128-etm<EMAIL_ADDRESS>AuthenticationMethods publickey
LogLevel VERBOSE
Subsystem sftp  /usr/lib/ssh/sftp-server -f AUTHPRIV -l INFO
PermitRootLogin No
PasswordAuthentication no
ChallengeResponseAuthentication no
UsePAM yes
PrintMotd no

with additional Diffie-Hellman moduli. More information on Mozilla Infosec guide site under  Modern (OpenSSH 6.7+).

Everything looks reasonable from my point of view.
One thing which wonders me, is do we really need to enable verbose mode? Our logs might be super huge after that option and less readable.. I understood enabling this option for particular issue investigation, but setting it globally is a bit of over-productive and useless.

Tested configurations:

Azure/AWS x Ubuntu/RedHat

on-prem CentOS

No issues found.


Undertake a code review

Code Review (500-700 words – 5%)

Undertake, as part of a team activity, an inspection of a medium-size code segment using either a Walkthrough or Inspection as described in the lecture on Software Verification

Document the following:

[ ] List of faults detected
[ ] Description of each fault
[ ] Who was assigned to fix each fault
[ ] How the fault was fixed

Begun doing a code review as a team.
Went through each view, component, libriary and discussed the game loop.
Found the following issues: https://github.com/maxxxxxdlp/eecs-448-pixelland/issues/46 and https://github.com/maxxxxxdlp/eecs-448-pixelland/issues/47

Fixed by @nathanrdodson in 1586f80476654ffb0acb56fc33cde567f5cafb86
Thanks!


Unpin pipeline docker container version

https://github.com/cmu-delphi/forecast-eval/pull/245 pins the pipeline docker container to the last working version before the Ubuntu 22.04 transition started. A working container relies on arrow changes https://github.com/cmu-delphi/covidcast-docker/pull/12 and https://github.com/cmu-delphi/covidcast/pull/607 to be made.

Pin Ubuntu 20.04 so we can build and use latest image.


Adding support for Namespacing in the LeftAndMain menu

Adding support for Namespacing in the LeftAndMain menu. Required replacing backslashes with hyphens in three locations: X-Controller header (when changing admin page), assigning a class to the list item, and in the '$Code' variable sent to the menu (for the id attribute of the list item).

Sorry about the lack of response! This has since been resolved so this PR doesn't need to be merged.


fixing npm install to npm install --production

This might break existing guys task if someone incorrectly specifies dependencies as devDependencies. Hence closing this PR instead will make same changes to master, wait for stabilization then take in next release.


Add portable schemas to the roadmap

R: @chadrik @TheNeuralBit
And is it possible that we don't have documentation for schemas yet or is it just too hard to find?

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Adding Reuven since he authored a ton of schema stuff in Java.

@TheNeuralBit @reuvenlax unfortunately I don't have visibility into future plans for schemas. I originally came here to learn more and didn't find anything. I was able to locate the meetup presentation though. This is an important topic for Beam adoption. It would be great if someone could provide more details as suggested by @kennknowles


Caveflyer: Enemy may spwans next to player and the game finish in single step

There is a small chance that the enemy will spawn next to the player and move toward the player at the same time.
This fix will redirect the enemy's direction to other axes in that case.
p.s.
I generated environments with 5s timeout for an hour and it never occurred again.

This looks good to merge
We should look to add testing soon to check that this compiles correctly


PyCharm 2021.1.1 fails to give parameter hinting

Ctrl+P shows no parameter hinting below
Ctrl+Q does not show anything.

@wuyuanyi135
I can reproduce the problem.
There is a related issue.
https://github.com/koxudaxi/pydantic-pycharm-plugin/issues/267#issuecomment-821832674

PyCharm 2021.1 detects types of decorators. PyCharm 2020.3 doesn't work the feature.

I will fix it.

But, strawberry.type doesn't have type annotation of return value :thinking:
def type(
    cls: Type = None,
    *,
    name: str = None,
    is_input: bool = False,
    is_interface: bool = False,
    description: str = None,
    federation: Optional[FederationTypeParams] = None,
):

This line affects analyzing the type.
cls: Type = None,

@wuyuanyi135 I think this should be resolved now

@BryceBeagle Thanks. Is there plan for a new release of the plugin?

@koxudaxi Does CD trigger when a release is tagged?

@BryceBeagle
CD is triggered when a tag is pushed.
And, CD creates a release.

Let's make a release for @wuyuanyi135 👍

@wuyuanyi135
https://github.com/strawberry-graphql/strawberry-pycharm-plugin/releases/tag/v0.0.8
You can download the compiled plugin on the page.
Also, The version will be published in the Jetbrains market after Jetbrains approves the plugin within two business days

@koxudaxi @BryceBeagle  Many thanks 👍 ! I can confirm the issue is fixed in v0.0.8. Closing.


Add install_dev_tools support for arm macs

Describe the behavior changes introduced in this PR
Makes the install_dev_tools script handle arm64 macs.
Linked Issues?
resolves #519
Testing Instructions
Remove your .venv and re-run make dev-env. Make sure all the necessary dev tools are present.

make commands still work on x86 w/ this change :)

/LGTM

/lgtm
won't need CI e2e tests on it as it's not really related to e2e. Merging.


Rubocop

Continue #116 and add Rails 5 in travis.

I hope you don't mind, I pulled your changes into #116 and fixed the gemfile for rails 5 so travis can actually run. Thanks for your help :)


Update libraries

Basically fetch-lib. This is a marginal diff which is basically just stripPrefix, but either the rebuild of alertmanager from source or the new traefik corrects the catalogue index, so we may as well.
Also fetch-lib for prometheus_scrape, which closes #105

For example ^^


[embree3] Udpate embree to 3.13.5

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Feature avx512 is tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static
x64-osx
Features avx,avx2,core,sse2,sse42 are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static

Thanks for the update!


Input box text is unreadable with a bright accent color

See image :-)

Elsewhere in Obsidian, all the text on the accent color background (like buttons) is black. That would be one possibility, but I also think the accent color is not the right choice for the input box, because it's used for buttons and similar UI elements.
How about the input box would simply be the same color as the background of your text? I think this would be the most logical and also nicest looking option :-) (Maybe you could choose to use the accent color for the text to make it stand out from "regular text".)
Otherwise this looks like a cool plugin, but it's just unusable for me now :-)
(I'm currently using Copilot, but I don't like its default visual settings that can't be changed.)

v1.8.8
I added Chatbox Background Color and Chatbox Border Color colorpicker.
Please let me know if adjusting the chatbox appearance will make it more readable.
Thanks

Oh yeah, much better! Now I can actually use it :-)
A minor cosmetic thing is that the "Start typing…"  (There is no #FFFFFF in the default or Minimal theme, they use grays.) I would personally prefer it to be dark gray so it would look more like a subtle hint and not stand out so bright. Of course how it looks also depends on your background color setting, and it's a minor niggle anyway :-)
Thanks for the quick fix, I appreciate it! 🙏

v1.9.0 now has more options to customize the chatbot interface.

If you have <v1.9.0, please perform the following instructions:

Go to Community Settings and click on the folder's icon:

Close Obsidian completely.

Find the bmo-chatbot folder and delete data.json.

Restart Obsidian.

Love it, thank you! 🙏


Misleading error message when inserting OBJECTs and auto-casting fails

Problem Statement
Given a table with a REAL column inside an OBJECT:
CREATE TABLE a (b OBJECT(DYNAMIC) AS (c REAL));

Trying to insert a BOOLEAN fails:
from crate import client

connection = client.connect(["http://localhost:4200"])

cursor = connection.cursor()
cursor.execute("INSERT INTO a VALUES (?)", [({"c": True},)])

The failure itself is ok, but the error message is this one:
crate.client.exceptions.ProgrammingError: SQLParseException[Cannot cast value `[{c=true}]` to type `object`]

We were debugging a more complex real-world case, where this error message was hard to understand for the following reasons:

It doesn't describe the actual reason. {c=true} itself is a valid object, so Cannot cast to type object put us onto the wrong track that something was wrong with the object itself (in our case, the object was much more complex).
It doesn't name the target column on which the error occurred. One cannot clearly identify the offending column if there are several OBJECT columns with identical/similar subcolumns.
If the OBJECT value has a large number of attributes, it is not clear for which of the attributes the casting failed

Possible Solutions
Depending on what is technically possible, change the error message, so that it says something in the direction of:
Cannot cast value `true` to type `real` for column b['c']

Considered Alternatives
No response

Thx a lot for reporting this issue, it has been fixed and will be available with the next hotfix release(s)


Laravel 8.x Shift

This pull request includes the changes for upgrading to Laravel 8.x. Feel free to commit any additional changes to the shift-95966 branch.
Before merging, you need to:

Checkout the shift-95966 branch
Review all pull request comments for additional changes
Run composer update (if the scripts fail, add --no-scripts)
Clear any config, route, or view cache
Thoroughly test your application (no tests?, no CI?)

If you need help with your upgrade, check out the Human Shifts. You may also join the Shifty Coders Slack workspace to level-up your Laravel skills.

:information_source: Laravel 8 removed the public/web.config file. The presence of this file creates a potential security risk for projects not using Microsoft IIS as their web server.
Shift removed your file since it did not appear to be modified from the default version. If you are using Microsoft IIS as your web server, you may undo this commit by running git revert ec66887a.

:information_source: Laravel 8 streamlined the Exception Handler into a single register method to make it easier to add custom logic around rendering or reporting errors within your application.
While this change is not required, Shift recommends comparing yours against the Laravel 8.x version as an opportunity to reduce code and be fully up-to-date.

:information_source: Laravel 8 introduced class based factories. Shift automated this change. However, if you were using a community package for factories or other custom code, you should review your factories for additional changes.
If you encounter an issue, you may revert this commit and temporarily install the laravel/legacy-factories package. You may then automate the conversion to class based factories using the Shift Workbench.

:x: To match the other auth middleware, Laravel 8 now allows multiple guards for the RedirectIfAuthenticated middleware.
Shift attempted to automate these changes, but detected customizations. You will need to compare your app/Http/Middleware/RedirectIfAuthenticated.php  with the default Laravel 8 version to finalize the upgrade.

:information_source: Laravel 8 added a uuid column to the failed_jobs table to support the new job batching features. Shift detected you have a migration for this table and created a migration for this new column. Be sure to run php artisan migrate to complete your upgrade.

:information_source: Laravel 8 added new ENV variables for AWS_USE_PATH_STYLE_ENDPOINT, LOG_LEVEL, and LOG_DEPRECATIONS_CHANNEL. While these are defaulted within their respective configuration file, you may take this opportunity to set them accordingly for your environments.

:warning: Shift upgraded your configuration files by defaulting them and merging your true customizations. These include values which are not changeable through core ENV variables.
You should review this commit for additional customizations or opportunities to use new ENV variables. If you have a lot of customizations, you may undo this commit with git revert 9cb296a9 and make the config file changes manually.

:warning: Laravel 8 removed the token driver from the API authentication. This driver was not very robust and was removed from the documentation. Laravel now recommends using Sanctum.
However, you may continue use the token driver and migrate to Sanctum at your convenience.

:information_source: Shift updated your dependencies for Laravel 8. While many of the popular packages are reviewed, you may have to update additional packages in order for your application to be compatible with Laravel 8. Watch dealing with dependencies for tips on handling any Composer issues.
The following dependencies were updated by a major version and may have their own changes. You may check their changelog for any additional upgrade steps.

[ ] laravel/ui

:warning: Laravel 8 now requires PHPUnit 9 which made some configuration changes to its config file. You may automate these changes by running vendor/bin/phpunit --migrate-configuration or comparing your phpunit.xml to the Laravel 8 version and merging any changes.

:warning: Shift upgraded your NPM dependencies and scripts for Laravel Mix 6. You will need to run npm install locally to ensure your dependencies are up-to-date and compatible.
If you encounter any issues, be sure you have the latest LTS or stable version of Node installed. If the problem persists, review the Update to Mix 6 docs for any additional steps.

:information_source: Laravel 8 reintroduced the app/Models folder by default. This is an optional change. Laravel and the artisan commands will automatically detect if you are using the app/Models folder or not.
If you wish to modernize your application to use the app/Models folder, you may run the Namespace Models Shift for free.

:information_source: Laravel 8 supports running your PHPUnit tests in parallel. Depending on your system, this may decrease run times as much as 70%.
To run your tests in parallel, you may use the php artisan test --parallel command, or use the underlying vendor/bin/paratest runner.

:warning: Laravel 8.x has reached end of life. It no longer receives bug fixes or security updates. Shift recommends continuing to upgrade to the latest version (Laravel 10.x).


Ported to

Hi,
I love this theme and I have ported it to ES-DE.
I’ve given credit and linked to your GitHub page  in the theme description and acknowledgements.
Let me know if you want this documented any differently.
https://github.com/RobZombie9043/elementerial-es-de


does not work with adblock plus, not even adblock list

see title. ABP says its an invalid format

use mirror host for that not github list

I believe that's what I used