allowing keyword arguments to pass to json3.read from oxygen.json

In order to parse json files which included inf and nan's it is required to pass along the keyword arguments.
oxygen.json is not able to do this yet, therefore adding this functionality.
I am leaving open the combination of custom struct + passing keyword arguments of the form json(request, classtype; keywordargs), this can be added later if required.

@shikon Thanks for the PR, great catch! Could you add that additional use case to this pr? If we're going to make these changes to this module, we should try to cover all the use cases. If you can't, I can add this use case + tests when I get some time this week.

@shikon Thanks for the PR, great catch! Could you add that additional use case to this pr? If we're going to make these changes to this module, we should try to cover all the use cases. If you can't, I can add this use case + tests when I get some time this week.

Sure I will do that later this week. Lets keep the PR open for now.

I will put a few extra commits with the additional use cases, but please double check if the tests pass. I am not able to use your run test file succesfully on my local computer and I don't see any CI/CD here on github.

Already in your build pipeline there are errors... Maybe just close this PR, if you have time later to show me how I can make all test pass then I can continue I already have something in mind.

There are errors that show up during the tests, but these are expected and shouldn't stop the tests from running. Aside from that, I've moved your test case into its own file. I moved it out because your tests don't require a running server in order to work.

I'd recommend cd'ing into the /test directory and opening a new julia repl there directly.
activate & instantiate
include("bodyparsertests.jl")
This will let you quickly run your specific tests in isolation from the other tests

There are errors that show up during the tests, but these are expected and shouldn't stop the tests from running. Aside from that, I've moved your test case into its own file. I moved it out because your tests don't require a running server in order to work.
1. I'd recommend cd'ing into the `/test` directory and opening a new julia repl there directly.

2. activate & instantiate

3. include("bodyparsertests.jl")

4. This will let you quickly run your specific tests in isolation from the other tests

Hi,
Thanks for the steps, I am experienced developing in julia ;).
The tests that I had in mind would require to be added the current file as they are going to use internalrequest and are a response.
I do need a little help still understanding what errors are intended, how can I judge that?

The easiest way to know if an error is expected is if the test is looking for a 500 response code. Otherwise, the test would be wrapped in a try...catch block.
That being said if it's possible to execute your test case without adding it to the server in the runtests.jl file, I would prefer that. This file is already pretty massive, and I would rather move any new tests to smaller modules. Since json() is a pure function, any test that can be performed using internalrequest() with the router and server can also be performed without it.

I thought you had some preference making the test in that way.
Then I will keep on doing it like I did.

I think its ready to be merged.
regarding local testing of runtest:
this is one of the errors I get

Codecov Report
Patch coverage: 100.00% and project coverage change: -1.31 :warning:

Comparison is base (d0c3b9f) 94.28% compared to head (d6d2243) 92.97%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   94.28%   92.97%   -1.31%
==========================================
  Files           9        9
  Lines         787      797      +10
==========================================
- Hits          742      741       -1
- Misses         45       56      +11

Impacted Files
Coverage Δ

src/bodyparsers.jl
64.28% <100.00%> (-35.72%)
:arrow_down:

... and 1 file with indirect coverage changes

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


Issue with the Structure3D preprocessing.

Thanks for making your repo public. We are trying to use the repository for processing Structure3D dataset to get the labelled dataset and 3d point clouds. However, the current preprocessing script throws an exception with the latest version of the dataset.
Can you please look into it?
zipfile.BadZipFile: Bad magic number for file header

No problem. I will look into it. However, as we mentioned, we don't have the latest version of Structure3D. Thus, we need to download it first. Additionally, you may post the whole failure log here in case we can't reproduce the failure.

Does the failure happen in some or all zip files? If just a few zip files are cropped, please give me the name of files. I can test or figure out workaround on these files.  And I notice that the latest download README.md says that:
"Structured3D_perspective_full_09.zip (corrupted: refer to https://github.com/bertjiazheng/Structured3D/issues/30)"
If there any connections with your issue?

Thanks for your response. Here is the complete log.
Traceback (most recent call last):
  File "/home/nikhilmadaan/anaconda3/envs/uni3drc/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/home/nikhilmadaan/anaconda3/envs/uni3drc/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/nfs/turbo/coe-chaijy/datasets/structured3d/Uni3DScenes/datasets/structured3d/__init__.py", line 338, in _mp_format_dataset
    = self._view2points(zip_reader, room_path)
  File "/nfs/turbo/coe-chaijy/datasets/structured3d/Uni3DScenes/datasets/structured3d/__init__.py", line 240, in _view2points
    cam_paras, attr_images = __class__.read_camera_and_image(\
  File "/nfs/turbo/coe-chaijy/datasets/structured3d/Uni3DScenes/datasets/structured3d/__init__.py", line 111, in read_camera_and_image
    color_image = cv2.imdecode(np.frombuffer(io.BytesIO(zip_reader.read(\
  File "/nfs/turbo/coe-chaijy/datasets/structured3d/Uni3DScenes/graphics_utils/stream_io.py", line 299, in read
    return self._zips_meta[z_idx].read(file_name)
  File "/home/nikhilmadaan/anaconda3/envs/uni3drc/lib/python3.8/zipfile.py", line 1475, in read
    with self.open(name, "r", pwd) as fp:
  File "/home/nikhilmadaan/anaconda3/envs/uni3drc/lib/python3.8/zipfile.py", line 1535, in open
    raise BadZipFile("Bad magic number for file header")
zipfile.BadZipFile: Bad magic number for file header

It happens for all the zip files. Please let me know if you want me to share something else to replicate the issue.

It doesnt seem to be related to the corrupted file, since we get this error for other files too.

No problem. I will try it. If I can't reproduce the error, I may give an alternative option that make you unzip the files by yourself, and the code process the data with unzipped files rather than raw zip files.

Could you give more details about your OS, python and zip file package information? And the downloaded Structured3D files could be decompressed with unzip or other similar commands?
I have tried but won't get the error. However, My OS is Win11. As I mentioned, an alternative option to read the unzipped files is a good solution. But I can't guarantee when I can finish it, maybe this weekend or end of July, depended on the workloads in my hand. Thus, if you ensure that the failure is from the python zip file package rather than the file itself (by unzip the file with other tools), I will promote the priority, for other people may have the same issue, due to difference in OS or python environment.

Thanks for looking into it. I tried modifying the loader as you suggested and it worked. Thanks.

Great to hear that. It will be very appreciated if you consider merging your modification into the main master via a pull request.

Hello, thank you for your public great work! I have met the same problem, and I tried to unzip the .zip following the above option but I encountered another problem:

unzip -d dataset/ Structured3D_perspective_full_00.zip
Archive:  Structured3D_perspective_full_00.zip
warning [Structured3D_perspective_full_00.zip]:<PHONE_NUMBER> extra bytes at beginning or within zipfile
(attempting to process anyway)
error [Structured3D_perspective_full_00.zip]:  start of central directory not found;
zipfile corrupt.
(please check that you have transferred or created the zipfile in the
appropriate BINARY mode and that you have compiled UnZip properly)

which seems to indicate that this is not a zip file format, so I can not unzip it. So how can i solve this problem? could you share your solution for this, Thank you !

Thanks for looking into it. I tried modifying the loader as you suggested and it worked. Thanks.


making eye assist page with the graph and text

I don't know what went wrong, but the eyeLogo is now gone. Please add it again to the EyeTrackingPage


feat: remove next dependency

About
This PR removes Next.js dependency so our components can be compatible with any React framework.

Just a reminder not to merge this until we have tagged 1.0.0 stable


Configure the new chef-fips pipeline to build using Jay's assets.

/cc @jaym @scotthain @schisamo @btm @chef/engineering-services

LGTM

Ok, with the understanding that the project.overrides hack here definitely in the "Bad Dog!  No cookie!" territory, I'm merging this.  The hope is that we introduce some native way to define custom omnibus project level or even repo level state/flags that can be passed to all software definitions.  We will need to solve that in the future.  We should also lock down overrides to not accept values that aren't explicitly part of the software DSL.
Do not cargo-cult copy-pasta this stuff!


Failed to access LDAP server over TLS

Abstract
My container based on version 2.4.51 failed to access our LDAP server via ldaps with the following error_log:
[Mon Dec 13 07:08:02.404750 2021] [authnz_ldap:info] [pid 8:tid<PHONE_NUMBER>11744] [client <IP_ADDRESS>:52618] AH01695: auth_ldap authenticate: user user01 authentication failed; URI /secret/ [LDAP: ldap_simple_bind() failed][Can't contact LDAP server], referer: http://<IP_ADDRESS>:8080/

Before 2.4.50 don't have the issue as the libldap-common package was installed.
Root cause
The missing /etc/ldap/ldap.conf file causes the missing "TLS_CACERT /etc/ssl/certs/ca-certificates.crt" configuration.
Because of this, the server cannot access using ldaps:// URL.
Conclusion
Is this an intended change in 2.4.51 or later?

Interesting... note sure why libldap-common isn't in 2.4.51, the Dockerfile difference is just the version change https://github.com/docker-library/official-images/pull/11057
The Dockerfile installs libaprutil1-ldap which has libldap as a dependency, which then has libldap-common as its dependency.
$ docker run -it --rm httpd:2.4.51 bash
root@e6828c8b35ed:/usr/local/apache2# apt search libaprutil1-ldap
Sorting... Done
Full Text Search... Done
libaprutil1-ldap/now 1.6.1-5 amd64 [installed,local]
  Apache Portable Runtime Utility Library - LDAP Driver

root@e6828c8b35ed:/usr/local/apache2# apt search libldap
Sorting... Done
Full Text Search... Done
libldap-2.4-2/now 2.4.57+dfsg-3 amd64 [installed,local]
  OpenLDAP libraries

root@e6828c8b35ed:/usr/local/apache2# apt search libldap-common
Sorting... Done
Full Text Search... Done

What 2.4.50 shows
$ docker run -it --rm httpd:2.4.50 bash
root@e8cbf9179ea8:/usr/local/apache2# apt search libaprutil1-ldap
Sorting... Done
Full Text Search... Done
libaprutil1-ldap/now 1.6.1-4 amd64 [installed,local]
  Apache Portable Runtime Utility Library - LDAP Driver

root@e8cbf9179ea8:/usr/local/apache2# apt search libldap
Sorting... Done
Full Text Search... Done
libldap-2.4-2/now 2.4.47+dfsg-3+deb10u6 amd64 [installed,local]
  OpenLDAP libraries

libldap-common/now 2.4.47+dfsg-3+deb10u6 all [installed,local]
  OpenLDAP common files for libraries

This is very similar to https://github.com/docker-library/php/issues/1228 -- the difference (and the thing that caused the breakage) is that in Debian Bullseye, libldap-common is only in Recommends instead of Depends, so it no longer gets installed automatically (thanks to our overuse of --no-install-recommends).
In this case, it's core functionality being broken, so IMO we should probably add it persistently (perhaps with a link to this thread so we can more easily remember/chase down why we added it).

Thank you for your prompt action!


Midi.Player.start not working

I'm having problems getting MIDI.Player.start to actually play the midi file. I'm free of console errors, so I'm not sure what's preventing the player from working.

Look at this issue https://github.com/mudcube/MIDI.js/issues/148 I am still struggling to implement my own MIDI player and having same problem.
I think it could be issue of "channel" set in the MIDI file. I have similar problem when I tried to play MIDI file generated using https://github.com/sergi/jsmidi. But, when I changed the channel while generating MIDI file it started playing in the MIDI player available with library (downloaded zip).


location=-1 exist

Description
I found a problem in xml file location is -1.
<chunk id="3" name="Initial Contents" .....>
-1
this is a bug？
note：test app was gfxbench 4.0.1 high-level Tests Car Chase
I also want to know  renderdoc will have a function that support to  translate rdc file to txt file？

not only in initial contents（chunk id="3"）

750
-1

not only in initial contents（chunk id="3"）
<chunk id="1766" name="glUniform4fv" ...>

-1

not only in initial contents（chunk id="3"）
<chunk id="1766" name="glUniform4fv" ...>
<int name="location" ...>-1<...>

The XML is just a representation of the RDC file, unless there's a problem with the replay I don't consider any contents a bug. E.g. if -1 is listed for the location parameter of a glUniform4fv call then that's what the application passed to the function.
I don't have any plans to provide other plain-text exporters, but it's relatively straightforward to write your own either in C++ or python using the access to the structured data.


pd.lib.infer_dtype infers bytes in Python3

ping!

?

@jreback ok with this?

oh whoops, need to add tests for strops

and add to this line: https://github.com/pydata/pandas/blob/master/pandas/core/base.py#L522
(test if you can)
squash as well. thxs

ok, in progress

hm, so to really be solid about this we'd have to test every single str operation against a bytes series
e.g., cat was failing because it has a sep argument that is '' if  None is passed in. That string will be a unicode string in python 3 which can't be used with a bytes object

Do we have to support bytes operations? I really just wanted the ability to tell if a Series contained bytes. I wasn't looking to add any new features.

no I don't think bytes are necessarily a big deal to support - in fact prob should raise on str type ops?

ok, i'll test that we raise on str access

ok, looks good. just add a release note and merge. ty.


Going through a portal breaks the "view origin" function [o]

I load into a creative world, and press [o], it shows my origin correctly
I go through a nether portal, and press [o] and is says 'you cannot have an origin'.
I go back through the nether portal and press [o] again with the same results.
changing origin with an orb of origin does fix, until I go through a portal
i have not tested ender portals yet, but the portals of the "TARDIM" mod also break it.
game version: 1.18.2, running Forge 40.2.0 with mod version <IP_ADDRESS>

Duplicate of #316


4th floor Diagram

Feature
Helps users understand the layout before they get there

Find where to add to bizport and add

4th floor plan for mobile: https://drive.google.com/a/codeforamerica.org/file/d/0B7h__NzTXxGUZlhCdHlFNzZ1QTQ/view?usp=sharing

Add this file to the CMS


TOTEMIC MOD NOT WORKING

I've playing Sevtech, at the stage where i need to do ceremonies mod does not work, the selector doesn't show up and i can't do anything. i tried downloading old versions but the problem is still the same.
The Mod just doesn't work and it's strange cause in youtubers gameplay it worked a couple months ago

In the Sevtech version that I am using it works fine. You need to sneak while playing an instrument in order to make the selector show up, maybe this is the problem?


Broken note formatting

This note (https://help.sumologic.com/docs/integrations/google/workspace/collect-logs/) looks weird:
Slack Message

Hey @nmohantyctr , I'll take this over

fixed


Flake8 comprehensions update

Motivation and Context
Public branch replacement for #1764.
Also updates clang-tidy CI job to Ubuntu 20 as 14 and 16 are deprecated.
How Has This Been Tested
CI
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I suggest merging this change now and addressing the lab tests separately. They appear unrelated as they are also on main.


Fix bug with parse.py

Fix bug with https://github.com/Filimoa/open-parse/issues/28

Minor request but could adjust your solution so it passes the failing test
Standard markdown is formatted with spaces to make it slightly more readable
| Year | Revenue | Expenses |
|---|---|---|
| 2022 | 100,000 | 50,000 |
| Row2Col1 | Row2Col2 | Row2Col3 |
| 2021 | 90,000 | 45,000 |

Your solutions outputs
|Year|Revenue|Expenses |
|---|---|---|
| 2022 | 100,000 | 50,000 |
| 2021 | 90,000 | 45,000 |


why this error?

onlyChild must be passed a children with exactly one child.

when i set TabNavigator.Item's selected to true

You're trying to render 0 or > 1 child in a component that wants one child as the error states. You should debug further.

Thanks! The problem has been solved! I didn't add any child…………
╮（╯＿╰）╭


Fixing typo in High Contrast Themes topic

Fixing typo in 'Detecting High Contrast in Previous Versions of Windows' block of 'Supporting High Contrast Themes' topic

+1


Drop  magic string handling for setting an attribute

Drop magic string handling when setting via property (color.red = "20%") or via set (color.set('red', '20%')).
Turns out color format takes percentages too. And it appears they handle things a bit different, like always serializing to 0 - 1 for percents. I'm thinking instead of all of this magic, we will parse CSS strings and such when creating a color object, but you have to work in numbers when modifying the color object directly.

Fixed in https://github.com/facelessuser/coloraide/pull/6


[Bug]: Can not create an instance of Chroma client

What happened?
When creating an instance of Chroma it throws an error. "Fetch is not defined"
/chromadb/dist/main/generated/runtime.js:17:24)
Versions
"chroma": "1.5.3",
"node" : "16.20.0"
Relevant log output
ReferenceError: fetch is not defined
    at Object.<anonymous> (/home/aqib/backend-11/vision/node_modules/chromadb/dist/main/generated/runtime.js:17:24)
    at Module._compile (node:internal/modules/cjs/loader:1196:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1250:10)
    at Module.load (node:internal/modules/cjs/loader:1074:32)
    at Function.Module._load (node:internal/modules/cjs/loader:909:12)
    at Module.require (node:internal/modules/cjs/loader:1098:19)
    at require (node:internal/modules/cjs/helpers:108:18)
    at Object.<anonymous> (/home/aqib/backend-11/vision/node_modules/chromadb/dist/main/generated/api.js:17:19)
    at Module._compile (node:internal/modules/cjs/loader:1196:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1250:10)

@aqibamir fetch is only natively in node18+. we are releasing this polyfill soon, https://github.com/chroma-core/chroma/pull/752, but it has not been released yet

Still seems to not be working

i was able to run this example in the repo with 1.5.3

any chance you could try this? just update package.json to point at 1.5.3 instead of file:../..

Getting:
Error: TypeError: fetch failed
    at ChromaClient.createCollection (/app/node_modules/chromadb/src/ChromaClient.ts:132:19)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

My package.json has: "chromadb": "^1.5.3",

@shamoons what version of node are you running?

@shamoons what version of node are you running?

Running v18.16.1

@shamoons are you running the docker backend?

Yeah - running the docker backend: docker-compose -f chroma/docker-compose.yml -p chroma up -d --build

@shamoons hmmmmm - we have nearly identical setups my node is at 18.13.0. But that shouldnt make a difference. Are there any logs on the server side, or just js?

Just the JS logs. It's a server-side express function

// src/functions/handleGoogleDoc.ts
import chromaClient from '../services/chroma.service';

export const handler = async (fileInfo: IFileInfo) => {
...

  try {
    const collection = await chromaClient.createCollection({ name: "docs" });

    const resp = await collection.add({
      ids: [fileInfo.id],
      metadatas: [{ ...doc.data }],
      documents: [text],
    })
    console.log({ resp })
  } catch (err) {
    console.log(err)
  }

  return
};

// src/services/chroma.service.ts
const { ChromaClient } = require('chromadb');
const client = new ChromaClient();

export default client;

My Dockerfile:
# Dockerfile
FROM node:18

WORKDIR /app

COPY package*.json ./

RUN npm install

COPY . .

EXPOSE 3000

CMD [ "npx", "nodemon", "src/server.ts"]

It looks like 1.5.3 doesn't have the latest merge that included the isomorphic-fetch @jeffchuber ?


Changing the adsize and showing different ads in listview.

I am using this library for adding adding Native Ads Express to listview via an adapter to my app . I'm extremely thankful for making this job easier to newbies but I need some help. I'm unable to change the AdSize as well as all the ads in the listview are same. Isn't it possible to show different ads in same listview?
mPointList = mDBHelper.getListPoints(table); adapter = new ListPointAdapter(this, mPointList); String[] testDevicesIds = new String[]{getString(R.string.testDeviceID), AdRequest.DEVICE_ID_EMULATOR}; adapterWrapper = new AdmobAdapterWrapper(this, testDevicesIds); adapterWrapper.setAdapter(adapter); adapterWrapper.setLimitOfAds(8); adapterWrapper.setNoOfDataBetweenAds(30); adapterWrapper.setFirstAdIndex(10); listDetail.setAdapter(adapterWrapper);
Please help.

@vivekkarn Hi!
I guess it is the thing you are looking for https://github.com/clockbyte/admobadapter/wiki/Cookbook#in-case-you-want-to-use-a-multiple-unit-ids

Pardon me, but that example is for recyclerview not listview.

@vivekkarn the same should work for AdmobExpressAdapterWrapper


Update README: fix docker image name.

I'll bundle this change with other docker related changes. Thanks @arnikz!


why do you use KubeMQ

Hi, I am reading your project and I love this project
I have a question, why do you choose KubeMQ in your project instead of others like RabbitMQ or Kafka, are there any special features that others dont have it. Can you share me it. I am wondering when I should choose KubeMQ instead of others like RabbitMQ or Kafka

Hi!
The choice of this technology was made because we wanted the installation of the broker to be in accordance with the IT guidelines we had for communication messaging. We had to stick to a technology that was the most standard in Kubernetes and that we did not have to resort to third-party libraries or plug-ins. Its handling is very similar to existing brokers and it didn't cost us much to take these concepts and apply them to our project. It can also connect with database technologies or Kafka and RabbitMQ.


[FEAT]: To download image in Google Drive

Issue Title: No option to save or Export drawings in Google Drive
closes #39
Type of change ☑
What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

How Has This Been Tested? ⚙
Tested it manually multiple times. Also while testing we have to add the Client ID and Client secret on the "useGoogleDrive" file to make it work properly.

@singodiyashubham87 Please check if this is correct
Thank you

@Bhumika1312 Which keys & secrets will we have to use here? You must specify this using proper comments.

@Bhumika1312 Secondly, create a new PR because you have exposed your API credentials in this PR.


gp.sh uses http instead of git protocol

Don't know if this affects anyone else, but the git protocol is sometimes blocked by firewalls for me.

I'm in favor of HTTPS instead of SSH as well because GitHub suggests it. I opened a similar issue for the pull-all script: https://github.com/Polymer/tools/issues/44

Outdated, from a previous use of this repo.


how can to change the pip source in a virtual environment, use the following command

python --version
Python 3.8.19

uv --version
uv 0.2.20

os--windows

C:\Users>uv pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple
error: unrecognized subcommand 'config'

thanks !

We don't have a config command right now. But you could add this to a uv.toml in that directory:
[pip]
index-url = "https://pypi.tuna.tsinghua.edu.cn/simple"

Happy to answer follow-ups!


feat: Github Enterprise Support

see #64
closes #64

:tada: This PR is included in version 2.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Log pronouns in Airtable

I’d like to run some stats on gender breakdown of those with streaks so we have a baseline for future improvement.
Would greatly appreciate if Scrappy could log this in the Slack Accounts table so I can do this.

Pronouns are now in Airtable! Note: some pronoun fields are not filled out because the user doesn't have pronouns in Slack and they've never posted in Scrapbook. I also created a filter that filters out everyone who hasn't posted in Scrapbook before.


Oembed update

hi @matthooks.  Thanks for such a great gem.  I noticed somewhat recently that Vimeo's API has changed.  My company uses your gem (mostly) for embedding.  Since Vimeo's now using oEmbed, I've gone ahead and swapped out your urls to accord with Vimeo's changes.  I've included documentation in the CHANGELOG.  Cheers!

Thanks for the PR!


Add "Forgot Password" Feature to Login Page

Implement a "Forgot Password" link on the login page. This link should direct users to a password recovery process, where they can enter their registered email address to receive instructions for resetting their password. This enhancement will improve user accessibility and overall experience.

Kindly assign this issue to me

Hey maintainers, I would love to work on this feature which is "forgot-password implementation in login page". I have forked the repo just to check the code structure and I have found out the .env files are there with the fork. There are credentials also. I don't think this should be like this. If so please let me know.

thank you @Mdraza78 for raising the issue. you can work on this issue ;)

@Mdraza78 How much time it will take ?

@theashikagupta @yashmandi Please assign this issue to me.

please assign me the task as for gssoc24 extd

you can assign this task to me..

Assigned @SatyamTiwari069

hey i have completed the isssue .. but it is not getting pushed...even i am not able to create a PR.. Please guide me so that i can push my work to you....

Major changes i did were
Backend Implementation
API Route for Requesting Password Reset:
Create an API route that accepts the user's email or username.
This route should verify the email/username, generate a password reset token, and send a password reset email to the user.
Example route:
js
Copy code
// Express.js route
app.post('/forgot-password', async (req, res) => {
const { email } = req.body;
const user = await User.findOne({ email });
if (!user) {
return res.status(400).send('User not found');
}
// Generate reset token
const resetToken = generateResetToken();
user.resetToken = resetToken;
await user.save();
// Send reset email
sendPasswordResetEmail(user.email, resetToken);
res.send('Password reset link sent to your email');

});
Token Generation:
You can use a library like crypto to generate a random token for password reset.
Example:
js
Copy code
const crypto = require('crypto');
function generateResetToken() {
return crypto.randomBytes(20).toString('hex');
}
Send Reset Email:
Use a mailing service like Nodemailer to send an email containing a reset link with the token.
Example:
js
Copy code
const nodemailer = require('nodemailer');
function sendPasswordResetEmail(email, token) {
const resetLink = https://yourapp.com/reset-password?token=${token};
const mailOptions = {
from<EMAIL_ADDRESS>to: email,
subject: 'Password Reset',
text: Click the link to reset your password: ${resetLink}
};
// Send email using nodemailer
}
API Route to Reset the Password:
Create a route to handle the password reset once the user clicks the link in the email.
Example:
js
Copy code
app.post('/reset-password', async (req, res) => {
const { token, newPassword } = req.body;
const user = await User.findOne({ resetToken: token });
if (!user) {
return res.status(400).send('Invalid token');
}
user.password = newPassword;  // Hash this password before saving
user.resetToken = undefined;  // Clear the reset token
await user.save();
res.send('Password updated successfully');
});
Frontend Implementation
Forgot Password Form:
Add a "Forgot Password?" link to your login page.
When clicked, this should navigate to a form where users enter their email.
Example:
jsx
Copy code
// React.js form
function ForgotPassword() {
const [email, setEmail] = useState('');
const handleSubmit = async (e) => {
e.preventDefault();
await fetch('/forgot-password', {
method: 'POST',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({ email })
});
};
return (

<input type="email" value={email} onChange={(e) => setEmail(e.target.value)} placeholder="Enter your email" required />
Submit

);
}
Reset Password Form:
Add a form that accepts the new password when the user clicks the password reset link.
Example:
jsx
Copy code
function ResetPassword() {
const [password, setPassword] = useState('');
const [token, setToken] = useState(new URLSearchParams(window.location.search).get('token'));
const handleSubmit = async (e) => {
    e.preventDefault();
    await fetch('/reset-password', {
        method: 'POST',
        headers: { 'Content-Type': 'application/json' },
        body: JSON.stringify({ token, newPassword: password })
    });
};

return (
    <form onSubmit={handleSubmit}>
        <input type="password" value={password} onChange={(e) => setPassword(e.target.value)} placeholder="Enter new password" required />
        <button type="submit">Reset Password</button>
    </form>
);

}
Security Considerations
Token Expiry: Set an expiration time for the password reset token (e.g., 1 hour).
Password Hashing: Always hash passwords before saving them to the database.
Email Confirmation: Send confirmation after the password has been successfully reset.

@SatyamTiwari069 What's the error ? Attach screen shot for clarity.

Assigned @Sourabh782


If there are multiple trx files, only render the results for the newest one for a given assembly

Since each test run actually emits a different file name by default when running dotnet test -l trx.
Make it the default too.
The option could be:

-l|--latest false (to render all, why would you want that?)

This should be the default behavior anyway and built-in, and now is!
We now keep the outcome of the most recently run test (by its constant test id) when processing multiple trx files, wihch are now all aggregated too. If you don't want aggregation, you can simply run the tool from each folder in turn multiple times (i.e. just run a trx file search first with powershell and pass --path for each).


Default y to x for translate and scale transformations

If only one value is provided for translate or scale transforms then the single value will be used for both the x and y value.
See https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/transform#scale and https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/transform#translate.
fixes #1

Thanks for the fix


Chromebook

This isn't working on my Chromebook
Version 116.0.5845.120 (Official Build) (64-bit)
I might be doing something wrong but I open crosh and type the command and it just says javascript is not an official command

ooooh!!! you need to open prodigy, press CTRL+SHIFT+J and enter the coding into the console!! Hope this helps!!
P.S js isn't a supported lang in crosh ;)


Front end issue

Found design issues, would like to fix it.

Extra white space
'Get quick and short summary for your desired wikipedia article' can be removed when results are given
Page URL not alinged properly

please add hactoberfest and hactober-acceped tags

please add hactoberfest and hactober-acceped tags

added

hey @dharmikjethva30 can you assign this issue to me i can work on it

hey @kartik3805 can you assign this issue to me i can work on it

Discussed UI changes already implemented


Allow the status to be exported as json

We would like to use the RDE env in a CI / CD environment, where we need to know if the environment is available before installing a next package.
The aem:rde:status is a textual description, it is non-parseable for an automated system.
Feature request:

Add a json output for the status command

@royteeuwen see #77 that adds json output for all commands in the next release.


Add extended DVI (XDV) to TeX.gitignore

Reasons for making this change:

XDV is the XeTeX-equivalent of DVI, an intermediary format that has to be converted before reading.

Links to documentation supporting these rule changes:

https://www.ctan.org/pkg/xetex

@niklasholm thanks!


The navigator is out of sync with the project

STR
in the default project, convert the div to a flex layout.
The navigator shows it not as a flex layout
Convert it back
navigator now shows it
why

how do you "Convert it back"?

click on 'pins' in layout system. Or use undo, I can repro with either

does reproduce with cmd + z to undo

This appears to be a duplicate issue that was fixed a while back by #1628


Problem with prod environment

Hi,
The bundle is working fine in the dev environment but nothing appears in the prod environment even if I cleared the cache.
What am I doing wrong ?
EDIT : It's definitely a problem with the load of the js file...
website.com/js/35a8e64_comments_1.js is working and not website.com/js/35a8e64_comments_1.js
Any idea ?
Thanks, Pete.

Can you be more specific? Are you getting an error? Did you check the logs?

No error in the page and in the logs. The include is just not appearing.

Do you have assetic enabled?  (Try making it last if it isnt)

app/console assetic:dump solved it ! Thanks :)


Add CI action to update lists weekly

Generate lists (top 100, 1000, and 10000) weekly and push back changes to
the repository.
Fixes #4
Signed-off-by: Khosrow Moossavi<EMAIL_ADDRESS>

Unfortunately we trigger the abuse heuristic now: https://github.com/mvdan/corpus/runs/2596564898?check_suite_focus=true
It worked for me locally, so maybe it's because my personal token has higher rate limits than the default CI one. If it fails again next weekend, I'll try swapping it out.

Interesting, I just tried with a brand new generated token locally and got the exact same error. There has to be some other factors in play here. This says:

For API requests using Basic Authentication or OAuth, you can make up to 5,000 requests per hour.
...
For users that belong to a GitHub Enterprise Cloud account, requests made using an OAuth token to resources owned by the same GitHub Enterprise Cloud account have an increased limit of 15,000 requests per hour.
...
When using the built-in GITHUB_TOKEN in GitHub Actions, the rate limit is 1,000 requests per hour per repository

So if your token works locally I suspect you are part of and enteprise account, otherwise it seems more weird than it is!
Even 1,000 should be more than enough for the CI, and 5,000 should be way more than enough for my local test. For verbosity I tried outputing ratelimit using vlogf and:
corpus on master ! took 8s ➜ GITHUB_TOKEN="xxxxxx" go run . -v -count 10000 > top-10000.ts
v
2021/05/17 13:20:01 querying first page of results for "archived:false is:public pushed:>=2020-05-18 language:go sort:stars"
2021/05/17 13:20:01 1 API called
2021/05/17 13:20:07 Ratelimit, cost: 1, limit: 5000, remaining: 4923
...
2021/05/17 13:22:00 1748/10000 done; querying page 24 with cursor Y3Vyc29yOjMwMA==
2021/05/17 13:22:00 24 API called
2021/05/17 13:22:07 Ratelimit, cost: 1, limit: 5000, remaining: 4900
...
2021/05/17 13:22:07 1822/10000 done; querying page 25 with cursor Y3Vyc29yOjQwMA==
2021/05/17 13:22:07 25 API called
non-200 OK status code: 403 Forbidden body: "{\n  \"documentation_url\": \"https://docs.github.com/en/free-pro-team@latest/rest/overview/resources-in-the-rest-api#abuse-rate-limits\",\n  \"message\": \"You have triggered an abuse detection mechanism. Please wait a few minutes before you try again.\"\n}\n"
exit status 1

As you can see I started with 4923 remaining and it broke when, realistically, I still had 4900 remaining in my quota.

All that said, it might worth exploring the possibility to even breaking down process further away in the code to have more graceful API call (maybe with some sleep in between?) Technically there's only one API call happening in the code, but very frequent.
This also says:

... The Search API has a custom rate limit. Unauthenticated clients are limited to 10 requests per minute, while authenticated clients can make up to 30 requests per minute.

But then again based on GraphQL rate limit our query is very flat and shouldn't cost too much.

@mvdan did you get a chance to try it with your personal token? Although I think it might be better to refactor a bit to more gracefully call GitHub APIs to make this work with any provided token. What do you think?

Swapped the token here, we'll see if it helps: https://github.com/mvdan/corpus/commit/209c5a85555d8af657bbf5b9fd993b356104c9bf
If it doesn't help, I might just decrease the limit to unblock the updates. I could give you write access if you want to try other methods, too :) For now, I want to see what the next run in three days does.

Nnnnnope: https://github.com/mvdan/corpus/runs/3216198836?check_suite_focus=true
@khos2ow do you want to attempt to fix this? If so, I can give you write access and you should be able to push to feature branches and trigger workflows.

I've also opened https://github.com/mvdan/corpus/issues/9 to track this, since this PR is merged.

@mvdan yeah, I'm gonna take a crack at it.


NTS 2022

Build microverse first capstone project using

HTML
CSS for styling
JAVASCRIPT for functionality
LATO font for text font
FONTAWESOME for font icon
Here is a video description of the project and the project process click here

Hi @Tobby8629,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are still some which aren't addressed yet.

Highlights

The webpage looks nice and very well implemented 👨🎨

Required Changes ♻

Please make sure you have no git conflict before submitting your project for review.

Check the comments under the review.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Please brother
I've spent weeks working on this project and I have just submitted my last pull request, these changes we can do solo and shouldn't cost me another week


Suggestion: Highlight the advantage of nuxt-compress compare to the normal compression system used by Nuxt

If I add nuxt-compress , what will be the advantage over https://www.npmjs.com/package/compression already set in nuxt
Could it be possible to add this in the Readme, in order to make it clear for next visitor interesting by this package
thanks ;-)

The first two lines of the README explain what the difference is. The compression middleware is at runtime, but this is during build time, so there's no runtime performance hit from compressing.

The first two lines of the README explain what the difference is. The compression middleware is at runtime, but this is during build time, so there's no runtime performance hit from compressing.

Ok I got it better.
I don't understand why the Core Team of Nuxt is not using nuxt-compress by default, in that case.
"Using nuxt-compress at build time and then, the compression middleware." => should be a feature in Nuxt

Ok I got it better.
I don't understand why the Core Team of Nuxt is not using nuxt-compress by default, in that case.
"Using nuxt-compress at build time and then, the compression middleware." => should be a feature in Nuxt

Maybe. It's hard to draw the line between plugins and core. This is a really tiny plugin so I'm happy to maintain it.

Maybe. It's hard to draw the line between plugins and core. This is a really tiny plugin so I'm happy to maintain it.


Put battery life into the advertisement

The most useful thing to see in a list of modules (gotten from advertisement) would be battery life. Could also advertise stim parameters.

p 87 - CyBle_GapSetLocalName


welcome pack

Create a "welcome pack" pdf containing

link to pombelist/slack/susan forsburgs page
link to documentation/faq
quick how to's for stuff e.g. how to do a sequence download, how to use the query builder, jbrowse (containing links to relevant faq/docs)
quick intro to gene/annotation pages
intro to community curation
introductory references (eg primer)

We should include this!
https://jcs.biologists.org/content/joces/121/11/1771.full.pdf

This was a nice idea of @Antonialock , I thought we even started to gather the materials together somewhere. It probably doesn't need to be a pdf but a new starter page with all of the useful materials gathered together?
Any way we should start a web page with  a list links that a lab head can point new starters to.
How to join pombelist
https://lists.cam.ac.uk/sympa/suboptions/ucam-pombelist
Slack
Publication
include pombase papers,
gentics primer,  https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4596657/
history https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4896181/
Antonias book chapter
Tutorials
https://www.pombase.org/documentation/JBrowse_quick_start
https://www.pombase.org/faq
https://www.pombase.org/community/fission-yeast-community-curation-project (page needs improving)
Importance of citing PomBAse https://www.pombase.org/about/citing-pombase

@manulera you can announce this and close?


Version number in IRC topic is wrong

Displays:
Topic: Current Version: 3.3.1 | Read the Docs: http://getbootstrap.com/css/ | To get help, show your issue in jsbin http://goo.gl/D7POz6 | Channel being logged at http://goo.gl/XOnP6p | Be good
oniijin set the topic at: Nov 26, 2014, 5:21 PM
Mode: +cnt
Created at: Apr 5, 2012, 6:09 PM

Should be:
Topic: Current Version: 3.3.2 | Read the Docs: http://getbootstrap.com/css/ | To get help, show your issue in jsbin http://goo.gl/D7POz6 | Channel being logged at http://goo.gl/XOnP6p | Be good
oniijin set the topic at: Nov 26, 2014, 5:21 PM
Mode: +cnt
Created at: Apr 5, 2012, 6:09 PM

Please message the channel operator about this. AFAIK, nobody on the Core Team is involved in operating the IRC channel.

Understood. Wasn't sure. Thanks for the feedback.


chore: merge rivet-gg/site in to docs/ folder

Changes

#1250  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


decryptWithArbitraryKeys

Fixes Issue #9

Oh, I am new to Git, I couldn't remove last 2 commits :(

Seems I have misunderstood data format :)


[Storage] az storage fs directory upload/download: Support adls gen2 file system directory upload&download

Description

Feature request from #15200
Testing Guide

az storage fs directory upload
az storage fs directory download
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage


Visualization with d3.js

Hi, I'd be happy to co-lead a session on d3. Coming from geography, I could put an emphasis on maps, but not necessarily.
Caroline is looking for a co-leader...?

Luc and Caroline are doing a session! Closed in a9edde3


After unstaking everything, Dashboard > My Courts is not empty

The subgraph response is actually different for someone who has never staked (it returns null) compared to someone who has unstaked everything.

Never staked
{
  "data": {
    "user": null
  }
}

Staked
{
  "data": {
    "user": {
      "tokens": [
        {
          "court": {
            "id": "1",
            "name": "General Court"
          },
          "staked": "40000000000000000000000",
          "locked": "0",
          "delayed": "0"
        }
      ],
      "totalStake": "40000000000000000000000",
      "totalDelayed": "0",
      "totalDisputes": "0"
    }
  }
}

Unstaked all
{
  "data": {
    "user": {
      "tokens": [
        {
          "court": {
            "id": "1",
            "name": "General Court"
          },
          "staked": "0",
          "locked": "0",
          "delayed": "0"
        }
      ],
      "totalStake": "0",
      "totalDelayed": "0",
      "totalDisputes": "0"
    }
  }
}

Request
{
  user(id: "0x...") {
    tokens {
      court {
        id
        name
      }
      staked
      locked
      delayed
    }
    totalStake
    totalDelayed
    totalDisputes
  }
}


Change to Track Volunteers Add Volunteers Button

The button for admins to add volunteers to an event is confusing.  The search bar makes it seem like you look them up there and then use the add button to add them in which is not the case.  We should remove the search bar on the table unless the table is paginated, and then just make the add volunteer button a standard button with clear naming as to what it does.

 CELTS #139: Change to Track Volunteers Add Volunteers Button


TODO

[ ] Revisit info caching and do it properly this time.

Check to see if flags were used or config options were changed before automatically using cached value.

[x] Integrate shellcheck with travis.ci

[x] Make a list of shellcheck error codes we can exclude.

[x] Look into optimizing get_de(), get_wm() and get_term() usage.
[ ] Remove some bloat.

Doing info caching properly will be difficult since we also have to cache values of options and then check to see if those options were changed. We're only caching GPU on macOS now so I'm just going to simplify the caching as much as possible.

neofetch use for "Theme" and "Icon" the Theme and Icon from kde-gtk-config:

Instead from here:

Additional Info: The Look and Feel packages are meta packages for Theme, Cursor, Icons and so on.
You might want to add that info to your neofetch, when Plasma5 is detected.

I'll look into it, thanks!

@ShalokShalom, the issue should be fixed now. Try the latest master.

@dylanaraps Where is the latest commit, can't seem to find it

Follow these instructions to install the latest unreleased snapshot:
https://github.com/dylanaraps/neofetch/wiki/Installation#latest-git-master-bleeding-edge

@dylanaraps Oh ok, the thing is that, I'm in the process of porting Neofetch to KaOS, I don't really know how to get the PKGBUILD out. I did before but with the AUR PKG

KaOS uses pacman right?
I'm packaging Neofetch for Arch Linux so I can help you out if you're having trouble.

Yes pacman, sure please

The existing PKGBUILDs should work just fine on KaOS right?
Latest Release:

aur page: https://aur.archlinux.org/packages/neofetch/
PKGBUILD: https://aur.archlinux.org/cgit/aur.git/plain/PKGBUILD?h=neofetch

Latest Unreleased Version:

aur page: https://aur.archlinux.org/packages/neofetch-git/
PKGBUILD: https://aur.archlinux.org/cgit/aur.git/plain/PKGBUILD?h=neofetch-git

All that should be needed is to use the existing PKGBUILDs.

Yeah, last time I worked with the Latest Release I'll go with the  Unreleased

Updated, thanks

@dylanaraps  Thanks a lot dude
KaOS use different names in same cases for its packages and it use mostly the split setup from upstream. If you are interested to build other KaOS packages, for some reason, here are some important notes:
https://kaosx.us/docs/package/#list-of-common-renames

No problem!
None of those renames affect Neofetch. :)

They are just examples, thanks a lot again. :)


ACCESS ESM1.5 CICE configuration

Once https://github.com/payu-org/payu/pull/483 has been merged, we should enforce (in QA checks) that esm1.5 configurations have a ice_history.nml file which contains an icefields_nml namelist. We also could enforce that cice_in.nml does not contain the icefields_nml namelist.

Not just merged, but in a released payu deployment. When we have that can you ping this issue @jo-basevi? Thanks!


Subdomain integration

Enable volumic integration (i.e assembly) on CellDomain containing only a non-dense index of mesv cells. There is currently an error (raised by an assert).
This will be very usefull for multiphysics problem where the equations depend on the domain.
Currently, we can handle this case by writing the same equations everywhere and use MeshCellData. But this is quite bad for ergonomy and performance.

Do you remember why https://github.com/bcube-project/Bcube.jl/blob/67f5fe6db20650887e6c0e858e86e51174f3cc43/src/mesh/domain.jl#L426 is necessary? It seems to me that we could just delete this here.

A simple Bcube.compute seems to be working fine. I need to check with FESpaces (and test functions).
using Bcube

n = 5
mesh = line_mesh(2 * n + 1; xmin = 0, xmax = 1)
sub1 = 1:2:(2 * n)
sub2 = 2:2:(2 * n)
dΩ = Measure(CellDomain(mesh), 1)
dΩ1 = Measure(CellDomain(mesh, sub1), 1)
dΩ2 = Measure(CellDomain(mesh, sub2), 1)
f = PhysicalFunction(x -> 1.0)
@show sum(Bcube.compute(∫(f)dΩ))
@show sum(Bcube.compute(∫(f)dΩ1))
@show sum(Bcube.compute(∫(f)dΩ2))

Do you remember why
https://github.com/bcube-project/Bcube.jl/blob/67f5fe6db20650887e6c0e858e86e51174f3cc43/src/mesh/domain.jl#L426
is necessary? It seems to me that we could just delete this here.

If you iterate over the DomainIterator, I think it's fine. The test was more a kind of reminder to not forget to think about the generalization of the concept of domains (for example : how to deal with the indexing relative to the mesh / domains / Multi-FESpace)


netbsd specific stuff for grabbing memory size

If sysctl -n hw.physmem is broken on 64bit systems then doesn't sysctl -n hw.physmem64 break the output on 32bit systems? Or am I totally wrong here?
Some more info would be nice. :)

sorry for not explaining more,
with 4GB RAM, the hw.physmem signed integer (32bit, not 64bit) is too small:
% sysctl hw.physmem
hw.physmem = -143978496
% sysctl hw.physmem64
hw.physmem64 =<PHONE_NUMBER>
from sysctl(7) for netbsd:
hw.physmem (HW_PHYSMEM)
The bytes of physical memory as a 32-bit integer.
hw.physmem64 (HW_PHYSMEM64)
The bytes of physical memory as a 64-bit integer.

Thanks for clearing that up.
LGTM :+1:


Add MAS build

This PR adds a new MAS build which removes usages of private API calls of OS X and can be published to Mac App Store.

[x] Disable auto updater
[x] Disable crash reporter
[x] Remove private API calls in libchromiumcontent
[x] Verify it can be published to Mac App Store
[ ] Add a guide on using MAS build

Fixes #249.

:100:

Shouldn't such build be a default one for mac?

Shouldn't such build be a default one for mac?

To be compatible with Mac App Store we have to disable some important features, I'll give a list of the features not available in MAS build as a part of this PR.

:+1:

:+1:

:+1:

I have successfully uploaded the MAS build to iTunes Connect and passed Apple's private APIs usage check, but currently we are in the middle of upgrading our OS X build machines and I'm not able to provide a public build for now.
I'm merging this branch since the mas build doesn't affect our normal build, a public build and guides on uploading to Mac App Store will be provided in other pull requests.

:+1:  Cool.

:metal:

👏

:100:

:+1: This is so awesome

:+1:

:+1:

Do we have the public builds for MAS somewhere?! Can't find them..

a public build and guides on uploading to Mac App Store will be provided in other pull requests.

:100:

#3108


Benchmark against Agner's methods

Just found his bin2hex_ascii and related functions: https://github.com/darealshinji/vectorclass/blob/master/special/decimal.h.

{128, 0, 48, 192, 47, 52, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 48, 0, 0, 0, 156, 1, 0, 0, 140, 2, 0, 0, 0, 0, 0, 0, 5, 0, 0, 0, 32, 0, 80, 96, 46, 99}
result =  8f0030cf2f340000000000000000000000000000300000009f0100008f0200000000000005000000200050602e63
encodeHexVec result is wrong


windows 7 builds with wrong computer name "vagrant-2012"

the windows 7 machines build with wrong computer name of "vagrant-2012", will submit a PR in a minute.

Fixed in master and develop branches.


WIP chore(litmus-portal): GitHub auth

What this PR does / why we need it:
This PR contains GitHub authorization to create a new user and store it in database
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #1824
Checklist

[x] Does this PR have a corresponding GitHub issue?
[ ] Have you included relevant README for the chaoslib/experiment with details?
[ ] Have you added debug messages where necessary?
[x] Have you added task comments where necessary?
[ ] Have you tested the changes for possible failure conditions?
[ ] Have you provided the positive & negative test logs for the litmusbook execution?
[ ] Does the litmusbook ensure idempotency of cluster state?, i.e., is cluster restored to original state?
[ ] Have you used non-shell/command modules for Kubernetes tasks?
[ ] Have you (jinja) templatized custom scripts that is run by the litmusbook, if any?
[ ] Have you (jinja) templatized Kubernetes deployment manifests used by the litmusbook, if any?
[ ] Have you reused/created util functions instead of repeating tasks in the litmusbook?
[ ] Do the artifacts follow the appropriate directory structure?
[ ] Have you isolated storage (eg: OpenEBS) specific implementations, checks?
[ ] Have you isolated platform (eg: baremetal kubeadm/openshift/aws/gcloud) specific implementations, checks?
[ ] Are the ansible facts well defined? Is the scope explicitly set for playbook & included utils?
[ ] Have you ensured minimum/careful usage of shell utilities (awk, grep, sed, cut, xargs etc.,)?
[ ] Can the limtusbook be executed both from within & outside a container (configurable paths, no hardcode)?
[ ] Can you suggest the minimal resource requirements for the litmusbook execution?
[ ] Does the litmusbook job artifact carry comments/default options/range for the ENV tunables?
[ ] Has the litmusbooks been linted?

Special notes for your reviewer:

This PR is closed because litmus-portal branch has been deleted. Please raise the same pr to master


nixos/usbStorage: adapt upsteam

Description of changes
https://github.com/torvalds/linux/commit/24eca2dce0f8d19db808c972b0281298d0bafe99

Things done

Built on platform(s)

[x] 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.05 Release Notes (or backporting 23.05 and 23.11 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.

Does this need to be made dependent on the kernel version? I'm becoming more and more convinced this option shouldn't be enabled by default, considering it has needed repeated tweaking.

Does this need to be made dependent on the kernel version?

No. Changes are already in the current 6.1 kernel.

I'm becoming more and more convinced this option shouldn't be enabled by default, considering it has needed repeated tweaking.

I presume there is a possibility of data loss if you shutdown your system (and drive) without flushing the SCSI cache.
Although I don't know anyone has ever encountered it.

No. Changes are already in the current 6.1 kernel.

We support older kernels than 6.1.

Ok, the more I look at this and the surrounding kernel patches, the crazier this option is.

Now the NixOS option is named incorrectly given that the attr name has changed from manage_system_start_stop to manage_shutdown.
Why is this specific to USB? The kernel patches are for all ATA / SCSI drives. But the original bug report never figured out why the problem was specific to USB disks.
Actually, why are we getting rid of manage_system_start_stop? It sounds like both of these attrs do relevant things. Based on the patch, it seems like both of these attrs do the exact same thing, except this new one only applies during shutdown instead of also during suspend and hibernate.
Do other distros do this? I really don't like that we have this thing that's been in flux for a while based on an unresolved upstream bug report that, as far as I can tell, no one actually understands.

Because scsi over usb is hacky and problems like this occur. For example discard_granularity is almost always incorrectly populated. I don't want to touch suspend as you can get away with not using suspend at all, but you need to use poweroff.
I am changing the naming. We can disable this by default, if you prefer.

@bb2020 You need to use mkRenamedOptionModule when renaming an option.
I would prefer for this to be disabled by default.

@ElvishJerricco


Embedding of a paste

Is your feature request related to a problem? Please describe.
Feature parity with Gist: Embedding of a single (or multiple) files:

Describe the solution you'd like
An URL I can put on my website to share a piece of code, i.e.

Things to consider: embedding of a whole paste, embedding of a single file.

CSP & XFO needs relaxing for the embed endpoint


How to change the object in the task of pick and place

hello, i failed to find whe way when i want to change the object in the task of pickandplace to see the performance.
i noticed that the object of pickandplace task is created by the function create_box (or _create_geometry in fact ) in the pybullet.py
It seemd  i can achived my goal through modefing the self. physics_client.createMultiBody. so i change the code between line 575 and 579,  added the filename where the object (i want to load, obj type) saved. But obviously my mehotd is not correct.
I faild to change the object, so i'd like to ask you how to do that if you have time to reply me.

I would advise to create you own task, following the custom env section of the doc. However, for a minimal effort, you can just modify this lines of PickAndPlace task
https://github.com/qgallouedec/panda-gym/blob/09ecfb6d57e0b516ae248ef1d54a4cc7b3db71f1/panda_gym/envs/tasks/pick_and_place.py#L36-L42
with, for example (if you want a sphere instead)
self.sim.create_sphere(
        body_name="object",
        radius = self.object_size / 2,
        mass=1.0,
        position=np.array([0.0, 0.0, self.object_size / 2]),
        rgba_color=np.array([0.1, 0.9, 0.1, 1.0]),
)

OK, thank you very much. l would have a try to create a new task


Use base merging for imports

This PR switches FBbt to the “base merging” system for imported ontologies.
In the process, it adds CL, SO, NCBITaxon, and CARO as imports, because we are already using a handful of terms from these ontologies in some of our definitions.

@matentzn Same as for FBdv, a double-check would be appreciated.
Especially on the issue of adding CARO, CL, SO, and NCBITaxon as imports.
As for FBdv, I’ve pushed a “mock” release PR (#1616), if you want to have a look at the generated artefacts. There’s nothing obviously wrong in them AFAIK.

Did you make sure there are no dangling classes?

No more than the ones we already had before with non-merged imports. And they are from ontologies we actually really import from (CL’ cell, PATO’s quality, SO’s sequence feature, GO’s biological_process, molecular_function and cellular_component), so their presence is actually expected.
The only extra dangling classes that resulted from the merged imports were a dozen of Uberon classes, hence the explicit exclusion.


Add missing depedency "einops" in pyproject.toml

einops is required in serve, should be part of pyproject.

Thanks!


How to add plugin icon to the bundle?

Hello,
is there anyway to add the plugin icon into the bundle during build?
Do i have to each time add the icon into the bundle myself?

Yeah, i guess only way is to add manually into the bundle.


'CustomUniPCSampler' object has no attribute 'make_schedule' when using hi-res fix

Hi there, really nice extension! Was wondering if this issue is known.
When trying to use UniPC Sampler with dynamic-thresholding and hi-res fix, it gives the following issue when it should start applyng the hi-res fix.
Traceback (most recent call last):
  File "G:\Stable difussion\stable-diffusion-webui\modules\call_queue.py", line 56, in f
    res = list(func(*args, **kwargs))
  File "G:\Stable difussion\stable-diffusion-webui\modules\call_queue.py", line 37, in f
    res = func(*args, **kwargs)
  File "G:\Stable difussion\stable-diffusion-webui\modules\txt2img.py", line 56, in txt2img
    processed = process_images(p)
  File "G:\Stable difussion\stable-diffusion-webui\modules\processing.py", line 486, in process_images
    res = process_images_inner(p)
  File "G:\Stable difussion\stable-diffusion-webui\modules\processing.py", line 636, in process_images_inner
    samples_ddim = p.sample(conditioning=c, unconditional_conditioning=uc, seeds=seeds, subseeds=subseeds, subseed_strength=p.subseed_strength, prompts=prompts)
  File "G:\Stable difussion\stable-diffusion-webui\modules\processing.py", line 908, in sample
    samples = self.sampler.sample_img2img(self, samples, noise, conditioning, unconditional_conditioning, steps=self.hr_second_pass_steps or self.steps, image_conditioning=image_conditioning)
  File "G:\Stable difussion\stable-diffusion-webui\modules\sd_samplers_compvis.py", line 179, in sample_img2img
    self.sampler.make_schedule(ddim_num_steps=steps, ddim_eta=self.eta, ddim_discretize=p.ddim_discretize, verbose=False)
AttributeError: 'CustomUniPCSampler' object has no attribute 'make_schedule'

I've tested with R-ESRGAN 4X+ and others upscalers, but all seem to suffer the same issue.
Thanks!

me to


Docovery: toward generic automatic document recovery

Paper link: http://dl.acm.org/citation.cfm?id=2643004

No data


us-co-el-paso

el paso county
http://adm.elpasoco.com/InformationTechnologies/GeographicInformationSystems/Pages/FreeDataCatalog.aspx

Parcel data only available as paid source: http://adm.elpasoco.com/InformationTechnologies/GeographicInformationSystems/Pages/LicensedDataCatalog.aspx

Too expensive.

More than $10,000 and no redistribution.

closing as we have a source: http://results.openaddresses.io/sources/us/co/el_paso


Navigation buttons to move left and right on a chart/graph

There were three options for Panning in Carbon, each with pros and cons:

Browser default scroll bar

this is the browser default scroll bar because there is too much data to show
would appear if we turn off Carbon resize
there is no return button option because all data is loaded
the consumer decides what the look back is and what “all data” means (the look back fetches more data, for example 1mo, 1 year, by consumer control)
this isn’t carbon, isn’t really panning, this is all browser
it would be arrow based navigation for keyboard accessibility
would work for any graph type

Return/left/right button control

Return/left/right button control
Doesn’t load all data in advance
Carbon will show loading while the consumer feeds more data
consumer provides answers each interaction to if there is more data outside of the arrows. we can dither the arrow if there isn’t more data outside the view. carbon will save the state of initial load and return will take them there.
We won’t disable resize.
Works for multiple graph types
Would be tab to navigate, enter to select
Text on the return button would require each consumer to define, or Carbon could do an icon and English-only Tooltip
Works better with zooming buttons

D3 panning

All data must be loaded in advance
no keyboard navigation
no indication you can pan
no return button option
Would disable resize because it is a requirement from d3 to use panning
only works for line and bar
We would have to disable zoom out
https://bl.ocks.org/mbostock/7ec977c95910dd026812

We cannot make our own scroll bar (browser is the only option) and we cannot have #2 or #3 together because we won’t know what a “lower step” is for each of the left/right buttons because everything is loaded. Ultimately, to be the most accessible, we will move forward with only left and right arrows to move by each lower step increments because panning from D3 offers no affordance you can click and drag. We will not have a return button at the moment because text is not well supported in this environment and there is not a good icon for return that is as universally understood as the navigation arrows. We can also dither the arrows based on knowing if there is more data to either side of the visible graph.
They will be placed in the upper right-hand corner of the graph aligning with edge of the y-axis.

Note to self: PaginationControl
{
	PREVIOUS: {
		key: "uid_previous",
		label: {
			// This can be locale based, by default this will be blank
			display: "Previous"
		}
		shape: {
			// Left arrow
		},
		isPreviousDisabled: false,
		previousClickEventHandler: () => {}
	},
	NEXT: {
		key: "uid_next",
		label: {
			// This can be locale based, by default this will be blank
			display: "Next"
		}
		shape: {
			// Right arrow
		},
		isNextDisabled: false,
		nextClickEventHandler: () => {}
	}
}

For "loading screen" we are opting for SVG feGaussianBlur.
https://bl.ocks.org/mbostock/1342359
Consumer input:
pan: {
    enabled: true,
    showControls: true
}

Instead of peppering code with conditionals, we could have a dictionary subconfig of settings that we need to enable/disable such as clamp and based on those properties we will set the values.
Before:
scale.x = d3.time
        .scale()
        .domain(config.axis.x.domain)
        .range([0, getXAxisWidth(config)])
        .clamp(true);

After:
// Create a dictionary
isPanningModeEnabled()
        ? {
              shouldClamp: false
          }
        : {
              shouldClamp: true
          };

// In axis.js
scale.x = d3.time
	.scale()
	.domain(config.axis.x.domain)
	.range([0, getXAxisWidth(config)])
	.clamp(consolidatedDictionary.shouldClamp);

We have a similar need to disable clamping, but don't need the navigation buttons because the ability to navigate will be from terra-timeline. Ideally, we would like an option to be able to disable that functionality similar to the above isPanningModeEnabled option.  Should we log a separate issue for this functionality or can our need be considered with this issue?

The controls and the mode dont need to be on the same PR but they need to support each other for consumers who dont need a timeline based panning or dont need all the functionality within timeline framework.
@Prashant-Kamboj was planning to work towards this, not sure how much progress he has done. Didnt  you have a prototype already for this?

I am working on this implementation and able to achieve some work is still pending for bar graph and the dateline indicator inclusion also.

Here are links to the arrow icons:
Left - https://github.com/cerner/one-cerner-style-icons/blob/master/src/icons/0026_chevronLeft_a.svg
Right - https://github.com/cerner/one-cerner-style-icons/blob/master/src/icons/0026_chevronRight_a.svg

Does this issue include the data point running off the edge of the viewable area?

@ryanthemanuel  @abhijit945  Can someone ELI5 how a graph under the stella-timeline would pan in response to navigation events? The PR above for adding the panning property added a demo into the site, but it looks like that demo is still destroying/remaking the graph upon navigation as opposed to actual panning.

The contribution is only disabling the animations and not the complete panning functionality. The full functionality of smooth panning is still being developed and is a major change.

Duplicate of https://github.com/cerner/carbon-graphs/issues/93
Closing this since more recent activity and comments are on that issue.


Problem when press "Enter" with invalid match

Hi.
when make the auto-complete invalid and press enter, <span class="atwho-query">  not remove. if use space, it work.
I tried on http://ichord.github.io/At.js/ with contenteditable, also having this problem.
Step:

Type :smileA (make it invalid)
press enter
simply type "a"
you may see the incorrect result already.

if use "space" instead of "enter", everything work.
How to solve this?
Thanks...

what's your config of at?


Weird duplication of root folder.

A weird folder showed up after the update (v0.7.2) which appears to duplicate the root folder.

Uhh Ohh.... Is the problem reproducible after Atom restart ? Can you try to disable the Hidden Folders option in the package preferences?

Restarting and/or disabling hidden folders doesn't have any affect.

Did you restart after disabling the Hidden Folders ? This is the only change since the last release I can think of causing something in this direction....

Yep. Restarted with it both on and off but has no effect.

Can you test another FTP server? To make sure that problem is related to one server or shows always.

If the problem is still persistent, please install the release from before this update and test if the issue really wasn't already there in the last version:
git clone https://github.com/mgrenier/remote-ftp.git
cd remote-ftp
git checkout 44aebc2195e4bf11fffebbb53008d682f7fa5656
apm install
apm link

Huh. I just opened the connection in filezilla to see c:/users/solus softworks/google drive/site backup which is my local root folder (not that directory downloading works but I already reported that). It appears that RemoteFTP may have synced the . directory (referencing the current directory) by mistake?


Support for Arduino Mbed OS Nano Boards

I am trying to use NimBLE with the Arduino Nano 33.
But currently it is not possible to use it.
The Arduino Nano 33 uses the nrf52840 chip. This chip itself is supported with the n-able Arduino core.
Will the Arduino Nano 33 be supported in the future or is there a way to still use NimBLE with it?

Hi @Xraydylan this issue would be better suiting for the n-able repo but I'll answer here anyway.
The bootloader on the nano33 is not currently supported (I'll look into this later). So flashing requires programming via a debugging tool using pogo pins or soldering wires to the SWD connections. If you want to do this you could use the generic 52840 board variant for the nano33.

@h2zero any updates on this?

Nothing to report yet, I haven't had time to get into it.

Okay thanks for the swift response :)


[Proposal] Disable CoreOS update-engine

We're having some issues due to CoreOS update-engine. Some machines has been upgraded to a newer CoreOS version. Immutability is important and this is causing upgrades to docker 17.09.1-ce that has serious issues like: #1135

I'm also thinking about a mapping of kubernetes version - CoreOS Release.
The main reason is follow the trusted versions of the official changelog and throw and warn if the coreOS release doesn't have the requirements:
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.8.md#external-dependencies
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.9.md#external-dependencies
In the other hand it will require a periodic maintenance of this mapping.

what about security updates then? can we somehow downgrade or pin versions while keeping base OS security updates?

It's how it is right now. I made a PR doing a feature flagging to remove auto updates.
Security updates are important but if they came with a downtime in my cluster I prefer to live in the risk during some time or add my own patch. In the other hand I prefer run a cluster update to patch a security issue, otherwise I'm broking immutability and running in issues like my current problem:
Right now I've a node pool in my cluster with 2 versions of docker running (1.12.1 & 17.09.1-ce) because a machine restarted after a coreOS update and I think this is a high risk I can't asume, half of my elastic cluster is down because of this.

Related: https://github.com/kubernetes-incubator/kube-aws/issues/1209

Thanks for bringing this up!
Just to be clear, I believe that you normally don't receive automatic updates as of today, already.
Please see my comment at https://github.com/kubernetes-incubator/kube-aws/pull/1241#pullrequestreview-112640826. TL;DR; We do miss disabling the update-engine, but we do have reboot-strategy: off. It practially worked like "updates are downloaded but never applied automatically".
Anyway, your change made in #1241 makes a lot of sense 😉I just wanted to make sure I'm not missing anything here.

@whereisaaron this is why I did #1201. The current issue it's that even with amiId setted some workers where updated. We are going to investigate the reasons of the reboot of some random machines. But for example, an aws maintenance reboot can potentially crash your node pool. We're also going to upgrade de docker 17.12, but also considering a dockerless cluster.

Hello All,
FWIW, I have been pinning our 'amiId' for over a year now with most of our kube-aws clusters.   I believe it's allowed our older clusters (sadly some are over a year old) to be more stable given everything you've noted above.
I initially started 'pinning' the amiId because I experienced intermittent strange/blocking issues after initializing a new (say Dev for example) cluster.  After deploying the initial cluster, and copying the new cluster config data (and changing names and subnet CIDR, etc.) to create a stage/prod cluster, the second cluster provisioning would fail with strange errors--this cluster had the exact same configs (just different names and subnets, etc) as the one that just deployed without any issues.  'Tracked it down to the CoreOS AMI incrementing in the time between a deploying a Dev cluster and a Stage cluster.  So even though I had checked the initial new cluster's config files into git, it could not be redeployed--due to the CoreOS AMI version being incremented behind the scenes.
It's great to see these issues discussed here, it's very helpful for me because when we've seen strange issues in the past, I wasn't sure what happened.  And I noticed that on our most recent kube-aws cluster upgrade ( v0.9.10-rc.3 ) that ami pinning is now the default and I wanted to ask about that.
Thanks!

We had a little scare last week when Container Linux was updated and almost broke our fleet.. thought it'd be relevant to share here https://blog.quentin-machu.fr/2018/11/12/the-moment-container-linux-was-updated-and-almost-broke-our-fleet/


[Enhance] Move ScanNet point alignment from data pre-processing to pipeline

In the original ScanNet pre-processing script for the detection task, the point cloud scene is aligned according to axis_align_matrix in the annotation (see here). However, ScanNet test set doesn't have such alignment matrix, so people don't do such alignment in the semantic segmentation task (because they need to submit the test set results to online benchmark, while det only test and reports on val set). In order for ScanNet seg and set to share one pre-processed data, we decided to move the alignment step to pipeline.

Modify ScanNet pre-processing scripts to save the axis_align_matrix to data_info.pkl
Support Box.rotate function when input is angle or rotation_matrix, add unittest for it

After discussing with Wenwei, we decided to extract both aligned and unaligned_bbox during pre-processing of ScanNet dataset. For the aligned bbox, they are stored in info with the same key as before (e.g. gt_boxes_upright_depth, dimension), while for the unaligned bbox, they are stored with unaligned_ marked keys (e.g. unaligned_gt_boxes_upright_depth).
As aligned bbox are stored with the same keys, we can keep most of the code unchanged (e.g. dataset.evaluate()). If we would like to support 3D instance segmentation on ScanNet in the future, we will need the unaligned_bbox infos.

I have checked the correctness of processed data by testing a pre-trained VoteNet on it and got mAP_25, mAP_50 close to the originally reported values (<1%). I think the data is OK to use.


suggestion for static sorted method on Seq

What do you think of replacing Seq#sorted() with a static method like this? Reasoning is that it's not a good idea to have a runtime ClassCastException if the elements of the Seq are not comparable, since we can't do implicit type constraints like in Scala. This one method would work for all subclasses of Seq as long as their sorted(Comparator) method returns the same instance as they are.
If you think it's good I'll remove the other one, rename this one properly and transform the test cases. Are there other methods that would benefit from a similar treatment (that throw ClassCastException)?

Current coverage is 95.97%

Merging #1165 into master will decrease coverage by -0.02% as of 247ad22

@@            master   #1165   diff @@
======================================
  Files           78      78
  Stmts         9063    9064     +1
  Branches      1712    1712
  Methods          0       0
======================================
- Hit           8700    8699     -1
- Partial         90      91     +1
- Missed         273     274     +1

Review entire Coverage Diff as of 247ad22

Powered by Codecov. Updated on successful CI builds.

Hi Martti,
it was a hard decision to make some of the API unsafe. I've thoroughly though about the safety topic while designing the API. There are too many of these methods. All numeric operations in collections (product, average, sum). All candidates xxx that have an xxxOption alternative might also fail, e.g. head, init, last, tail, reduce. The get method of all Values (e.g. Option, Try, ...) might throw NoSuchElementException - partially defined functions and so on. I bet there are more examples.
I chose an API design that is practicable in terms of conciseness. The unsafety of ordered collections is already from java.util.SortedSet. But there are also examples that were too unsafe and I removed them again. An example is the flatten() method which assumed that a List has type List<List<T>> for example. A static flatten method is possible but we don't have it right now.
These static methods are hard to maintain. They follow not the rules of inheritance. Every interface/class has to define their own. Regarding the increasing amount of collection impls this is really cumbersome work.
Let's keep it as it is - not optimal but practicable. There are other libs for Java that try to emulate real functional languages. But I think this far from being practicable. The code is hard to write and hard to read.
Are you fine with this or does it feel wrong?

Hi,
Ok, I'm not suggesting changing every method that can throw an exception, just maybe the ones that throw ClassCastException. This is because changing a type shouldn't lead to runtime exceptions. Of course Option#get() has to throw an exception if the option is not defined, this is the way it's in Scala too. But if you feel there are too many of these and it makes it difficult, let's keep it as it is.

One option is to just remove the sorted() method since you can do this and it's type safe:
Vector<Integer> vec = Vector.of(1,2,3);
vec.sorted(Comparator.naturalOrder());

If you change the type of vec to Object, the sorted call doesn't compile anymore.

You are right, that would be safe. But I want to provide the same API as introduced with Java 8: Stream.sorted()
From the Java 8 API docs:

For ordered streams, the sort is stable. For unordered streams, no stability guarantees are made.

We have stated this, too, at our interface:

throws ClassCastException - if this elements are not Comparable

Let's leave this behind us. It is not worth to put too much effort in it. Java needs inference - otherwise all looks just as a workaround, the one or the other way.
I will close this ticket.

Ok, no problem. I don't think following Java8 api's too closely is a very good idea, but I understand your point. :wink:

:+1:


Global Circuit Breakers

Currently Circuit Breakers are configured per core in solrconfig.xml. However, cores/collections do not live in isolation, and it could be that a cluster administrator would like to enforce circuit breakers for the entire cluster.
I'm not clear as to whether we need both cluster level and a core level pluggability. And would core-level breakers add to any cluster-level ones or override them for that core?
A potential design is to add this as a new plugin in solr.xml, and have them added in a new static context of CircuitBreakerRegistry. Then the isTripped logic would consult both the per-core list and the static/global list of breakers.

torvalds.dev is analyzing the ticket

torvalds.dev is analyzing the ticket

The issue you're dealing with is related to the configuration of Circuit Breakers in Solr. Currently, Circuit Breakers are configured per core in solrconfig.xml. However, cores/collections do not live in isolation, and a cluster administrator might want to enforce circuit breakers for the entire cluster.
The question raised is whether both cluster level and a core level pluggability are needed. And if core-level breakers would add to any cluster-level ones or override them for that core.
A potential design is to add this as a new plugin in solr.xml, and have them added in a new static context of CircuitBreakerRegistry. Then the isTripped logic would consult both the per-core list and the static/global list of breakers.
The files relevant to this issue are:

TestGlobalCircuitBreaker.java: This file contains tests for the pluggable circuit breaker implementation. It tests if the global circuit breaker is registered and if indexing trips the load average circuit breaker.

CircuitBreakerRegistry.java: This file contains the implementation of the CircuitBreakerRegistry. It keeps track of all registered circuit breaker instances for various request types. It is responsible for a holistic view of whether a circuit breaker has tripped or not. Circuit breakers may be registered globally and/or per-core.

Based on the information provided, here are some potential actions:

Review the current implementation of Circuit Breakers in Solr, specifically how they are configured per core and how they function.

Investigate the possibility of implementing cluster-level Circuit Breakers. This would involve understanding the implications of having both core-level and cluster-level Circuit Breakers.

Consider the design suggestion of adding a new plugin in solr.xml for Circuit Breakers and adding them to a new static context of CircuitBreakerRegistry.

Review the code in TestGlobalCircuitBreaker.java and CircuitBreakerRegistry.java to understand the current implementation and tests for Circuit Breakers.

If the design suggestion seems viable, plan out the steps for implementation and testing.

Remember to consider the potential impact on existing functionality and ensure that any changes are thoroughly tested.

nice. matches actual commits in the pull request:
https://github.com/apache/solr/pull/1919/files


Error while creating a copy of a Package Variant in the Edge repo

Follows a thread in Slack with @johnbelamaric
ubuntu@nephio-r1-e2e:~$
ubuntu@nephio-r1-e2e:~$ kpt alpha rpkg copy -n default edge01-6b26ca0f4fdf83212a73faff159bd013b41207ee --workspace=upfscaling
Error: Internal error occurred: error applying patch: conflict: fragment line does not match src line
ubuntu@nephio-r1-e2e:~$

Copy works for freshly created Package Variants
ubuntu@nephio-r1-e2e:~/test-infra/e2e/tests$ kpt alpha rpkg get --name free5gc-upf-test | grep packagevariant
edge01-563d796fdf5ce3690a6642c7b61b237855325aaf   free5gc-upf-test   packagevariant-1   v1         true     Published   edge01
edge02-2e99189bca68617a2e0b249e048834a71af86e2a   free5gc-upf-test   packagevariant-1              false    Draft       edge02

ubuntu@nephio-r1-e2e:~/test-infra/e2e/tests$ kpt alpha rpkg copy -n default edge01-563d796fdf5ce3690a6642c7b61b237855325aaf --workspace=upfscaling
edge01-b5c9f9b49026cd76ebd37f39c5ba4e34f833d8c4 created

We will need to upgrade our install to use that version, too, of course.

We have upgraded, I think this is fixed. Please reopen or create a new ticket if you see it again.


[1.x] Add endpoint for logout

Description
As per the title...
Example
No response

I will do it ISA.

Great! It's assigned to you now! 😉

Heya @Moemen-Gaballah
You may encounter an `Invalid credentials' error even with using a valid access token. I'm going to fix this today Inshallah.

@imdhemy
Good morning
Can you share a postman collection ?

You can find it here
https://www.postman.com/piscibus/workspace/siklid/collection/1271923-9f6f7d88-bfd4-446c-b2ed-79f4db0d195e?action=share&creator=1271923

@imdhemy
done store invalid token to cache (file) but when change adapter to redis I think some thing wrong
and I don't know where can add middleware in sympony ?
I will check it again evening or tomorrow.


macro block fails when the first line is a if statement

Macros causes a syntax error when it begins with if.
{%
  if true
  end
%}

Syntax error in test.cr:3: expecting token '%}', not 'NEWLINE'

  end

It works when some expression exists in first line.
{%
  1
  if true
  end
%}

crystal-0.24.1
Best regards,

{% if true
   end %}

and
{%
  if true
   end %}

Are fine though

Potential lead in parsing of expression suffix, can replicate with trailing ; which otherwise parses fine outside the macro
{%
  if true
  end; %}

Noting potential region of error
https://github.com/crystal-lang/crystal/blob/84288b7af588c7e00dc004ad21f5c9ec5be50f83/src/compiler/crystal/syntax/parser.cr#L289

This is working well in master.

Fixed by #6382


MSB0001: Internal MSBuild Error: Type information for Microsoft.Build.Utilities.ToolLocationHelper was present in the whitelist cache as Microsoft.Build.Utilities.ToolLocationHelper, Microsoft.Build.Utilities.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a but the type could not be loaded. unexpectedly null

I am having issues with the above error message when running build with the latest .nuget packages installed. I am trying to build a project, not solution. When I use Microsoft.Build version 14.3.0 I don't see this problem.
<?xml version="1.0" encoding="utf-8"?>
<packages>
  <package id="Microsoft.Build" version="15.1.548" targetFramework="net461" />
  <package id="Microsoft.Build.Framework" version="15.1.548" targetFramework="net461" />
  <package id="Microsoft.Build.Runtime" version="15.1.548" targetFramework="net461" />
  <package id="Microsoft.Build.Tasks.Core" version="15.1.548" targetFramework="net461" />
  <package id="Microsoft.Build.Utilities.Core" version="15.1.548" targetFramework="net461" />
  <package id="System.Collections" version="4.3.0" targetFramework="net461" />
  <package id="System.Collections.Immutable" version="1.3.1" targetFramework="net461" />
  <package id="System.Diagnostics.Debug" version="4.3.0" targetFramework="net461" />
  <package id="System.Globalization" version="4.3.0" targetFramework="net461" />
  <package id="System.Runtime" version="4.3.0" targetFramework="net461" />
  <package id="System.Runtime.InteropServices" version="4.3.0" targetFramework="net461" />
  <package id="System.Threading" version="4.3.0" targetFramework="net461" />
  <package id="System.Threading.Thread" version="4.3.0" targetFramework="net461" />
</packages>

To get this far I had to add assembly binding:
<dependentAssembly>
        <assemblyIdentity name="Microsoft.Build.Framework" publicKeyToken="b03f5f7f11d50a3a" culture="neutral" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>

Please help, I will be happy to provide more details.

Can you verify that Microsoft.Build.Utilities.Core.dll is in your output folder?

Thank you for responding. I have changed to use msbuild.exe from command line instead so moved on already.

I hit the same error after I installed:
Install-Package Microsoft.Build -Version 15.1.1012
But then I installed:
Install-Package Microsoft.Build.Utilities.Core -Version 15.1.1012
And things started working.
A little confusing...

Related SO question popped up: https://stackoverflow.com/questions/44249320/build-project-with-microsoft-build-api

Possible better solution for current versions of Microsoft.Build components:
https://stackoverflow.com/a/49886334/253938
https://github.com/dotnet/roslyn/issues/26029

Also see https://docs.microsoft.com/en-us/visualstudio/msbuild/updating-an-existing-application.

Thank you. Worked for me too. But I'm really confused, I'm using .NET Framework 4.7.2 for TargetFramework.


Update README.md

Dear Sean, I added our recent review "How to design a single-cell RNA-sequencing experiment: pitfalls, challenges and perspectives" to the "Experimental design" section. It provides an overview of the current available experimental and computational methods developed to handle single-cell RNA-sequencing data. Thank you very much!
Alessandra

Thanks, again!


[Pytorch][Relay] aten::_weight_norm implementation

This PR provides an implementation for the aten::weight_norm operator.
This operator is needed to build a relay representation of some models

@tvm-bot rerun


Increase QET cache and STXXL memory size

We're saving a lot of RAM with more efficient storage of the OPS/OSP
meta data and soon with vocabulary improvements. So we should put some
of that saved memory to good use.
Thus this commit increases the size of the query execution tree (QET)
cache ten fold. It also doubles the amount of memory STXXL may use.
When testing with Aqqu with a pure QLever backend this increases query
performance by at least an order of magnitude with minimal increase im
RAM use.
That said the 10-fold is currently quite arbitrary and likely still orders of magnitude less than what our virtuoso instance for Freebase has when combined with the varnish cache in front. Especially since we're caching just subtrees.

Ok running 8 Aqqu queries from wsptrain_tiny I need a 100000 cache size to get it down to an average of 16 seconds per Aqqu query on the first run and 1.6 seconds on the second run (the QLever queries are all below 20 ms). With a cache size of 1000 it's more than 23 seconds per Aqqu query on the first run and still 19 seconds on the second.
Note that this isn't a fair comparision to virtuoso since the Freebase loaded into our Aqqu Virtuoso is aggressively pruned and Aqqu generates more than twice as many candidates.
For the QLever process that's about 2 GB of additional RAM use which for Clueweb + Freebase is about 2%.

I also tried with a cache size of 100000 but that used too much memory running out of RAM even on vulcano.

I'm still figuring out a good cache size but let's keep this PR open to track it.

Ok so I'm still not sure this is the ideal size but 1000 subtrees works well and doesn't easily run out of memory.