Clarify formatting diff between Copy options

Clarifies the formatting difference between Copy options.  We provide this distinction between getting formatted and unformatted values to allow users access to both types of values.
We (internal PBI visuals team) get feedback that users are confused why some Copy operations have formatted values and others do not, so hopefully this documentation makes it clearer.

@yelper : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


Freeze when template contains double curly braces

As raised here: https://github.com/mjmlio/mjml-app/issues/68, having {{ in template results in a freeze.
e.g giving this code :
(tested with v3.1.1 and v3.2.0-beta.3)
const mjml = require('mjml')

mjml.mjml2html(`
<mjml>
<mj-body>
  <mj-container>
    <mj-section>
      <mj-column>
        <mj-text>
          Hello world {{
        </mj-text>
      </mj-column>
    </mj-section>
  </mj-container>
</mj-body>
</mjml>
`)

console.log('after') // it never happens

No problem with a single curly brace.
Is there any workaround?

It's issue with cheerio (postRender and xmlParser)
The only workaround is to use numeral entities like
<mjml>
  <mj-body>
    <mj-container>
      <mj-section>
        <mj-column>
          <mj-text>
            Hello world {{
          </mj-text>
        </mj-column>
      </mj-section>
    </mj-container>
  </mj-body>
</mjml>

After a bit digging, the problem is caused by a juice feature that try to replace {{ and }} (interpreted as handlebars delimiters). See https://github.com/Automattic/juice/blob/master/lib/cheerio.js#L68 . The RegExp ends in an infinite loop..
IMHO, best solution would be to pass an option to prevent this behaviour in Juice. I'll see how to do that. Keep updated.

I had a same problem trying to fit templating syntax into my emails for translations and other dynamic, transactional content. The slight workaround I had was to close the double curly brackets first  and then open them
1. some text here
2. some text here }}
3. {{ some text here }}

Definitely nit ideal. but was an easy enough of a workaround for me.

Hi, thanks for reporting. Should be fixed in v3.3.0 (now in beta)

It should be fixed in 3.3.1 too !

Hello,
Thanks for reporting @meriadec, it's now fixed (npm install mjml to update)!


chore (Algebra.Basic): postpone importing algebra instances for Nat, Int, and Rat

This should help typeclass synthesis by avoiding these instances unless absolutely necessary.

!bench

!bench

Please mark as awaiting-review if/when it is ready.


[deck] persistent solo mode

The app can be toggled between Deck and Solo mode (via a button in the plank)

deck/solo mode is persisted in local storage
first run starts in solo mode with view of default space in a stack

In Solo mode clicking an item in the navtree should change the current selection.
When exiting Solo mode, if the currently visible item is not in the deck it is added to the deck (per deck insertion setting), otherwise scroll to that item
When deck mode is active, clicking on a navtree item should toggle the visibility of the item in the deck

consider this vs scroll to plank behaviour, how else scroll to plank might be handled, and decide which is most useful/understandable behaviour (related to ui perf & navtree open state)

when opening a child object, scroll to the section in the collection if the collection is already open

show currently attended object in navtree + share with peers (follow up)

deck state is stored in local storage, url is either an fq object id (solo mode) or / (deck)
plank sizes should be persisted with planks in localstorage
when moving deck state out of url & into local storage state, remember that reset should clear that state so that the deck starts empty

when opening a child object, scroll to the section in the collection if the collection is already open

This seems like something we could do as a followup

@ZaymonFC it's definitely separable functionality, though I think that it will go a long way to help prevent the deck from becoming unruly so quickly

feel free to break it out into another issue to address separately but I think it should still be done in this round of cleanup work


Add support local image in arc flash

Right now we can push an android prebuilt but not locally built, we should add this by adding a flag or subcommand

To implement this,
$ tools/vendor/google_prebuilts/arc/push_to_device.py ${DUT}

will push the image to DUT.


Is the proxy working?

How do I use the proxy, cause it's not doing anything?
Is it working?

Just tried this on my home computer, not my school one, and on my school computer it says XDisconnected, but on y home computer it says connected and the proxy works, how do I connect my school computer???


Unicode decoding problem with function client.sdf_query()

Dear James,
Thank you for having written such a Python interface to Classyfire!
I submitted the unzipped file
buffer16.zip
to function call client.sdf_query(filenameBuffer, filenameOut) and an exception was raised:
1 queries submitted to ClassyFire API
0 percent complete
Traceback (most recent call last):
File "run2.py", line 93, in
client.sdf_query(filenameBuffer, filenameOut)
File "C:\Users\jmn\Documents\CNRS20\Publis\PNMRNP_STARTED\Biblio\pyclassyfire-master\pyclassyfire\client.py", line 223, in sdf_query
outfile.write(get_results(query_ids[i], return_format='sdf'))
File "C:\Users\jmn\Anaconda3\envs\rdkit3\lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode character '\u03b1' in position 8847: character maps to
It looks like classification by classyfire was successful but some unicode character cannot be inserted in the resulting .sdf file.
\u03b1 is the unicode encoding for the greek letter alpha, not so uncommon in chemistry.
Could you please indicate a fix or a workaround?
Many thanks in advance.
Best regards,
Jean-Marc

For this molecule, the web interface of Classyfire says it is a "Cyclic depsipeptides", with  a list of alternative parents that starts with "Alpha amino acid esters".

Hi Jean-Marc,
I wasn't able to reproduce your error but then is found this stack overflow post that indicated that this issue might only appear on Windows and I develop on a Mac. Looks like there's a cross-compatible solution but I have no easy way of testing locally. I'll push an update and let me know if it works for you.
After 3 years of neglect I'm pleasantly surprised this client works at all. Props to the Wishart Lab for maintaining a stable API!

The unicode problem is solved.
A batch of 1000 molecules was classified in about 1 minute, which is really nice.
Many thanks!
Jean-Marc


Document why 65,535 is used for the amount of data to read from a socket

I realize the number is what can fit in a 16-bit number, but there's comment saying if that's some magical HTTP/2 value or just what you prefer to read from a socket at a time. And if it is somehow magical to HTTP/2 (e.g., packet size or something), then that should probably be part of the API. Otherwise a quick comment that it's not magical would be handy for those of us who don't write networking code constantly.

To be clear, we're talking about the example on this page in the documentation?

It not a magical thing (AFAIK). In mitmproxy we read always complete frames from the socket and pass it on to h2 - a lot easier to debug when you can inspect each frame as it enters the h2-connection.
Take a look here for an example: https://github.com/mitmproxy/netlib/blob/1af231fe01350d056f3b6c14a695caf9eadda520/netlib/utils.py#L402-L416

So while @Kriechi is right that 65535 is not magical, in the case of the example documentation it's still arbitrary. There's no requirement to use it, it's just a sensible upper bound on the amount of data a user is likely to want to process at once.

So you're right about the page that you referenced, @Lukasa , but it's also used in the curio example.

Ah good spot, let's fix that up too.


Fix CALLCODE gas costs regression with previous forks

PR description
Fixes small issue with CALLCODE gas costs while running reference tests for previous forks.
Thanks for sending a pull request! Have you done the following?

[ ] Checked out our contribution guidelines?
[ ] Considered documentation and added the doc-change-required label to this PR if updates are required.
[ ] Considered the changelog and included an update if required.
[ ] For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

[ ] unit tests: ./gradlew build
[ ] acceptance tests: ./gradlew acceptanceTest
[ ] integration tests: ./gradlew integrationTest
[ ] reference tests: ./gradlew ethereum:referenceTests:referenceTests

Specifically what reference test is this failing?

this is Verkle specific:
tests/verkle/eip4762_verkle_gas_witness/test_calls.py::test_calls[fork_Verkle-blockchain_test-target_0x8a0a19589531694250d570040a0c4b74576919b8-call_instruction_CALLCODE-value_1][Verkle]
tests/verkle/eip4762_verkle_gas_witness/test_calls.py::test_calls[fork_Verkle-blockchain_test-target_0x0000000000000000000000000000000000000004-call_instruction_CALLCODE-value_1][Verkle]
tests/verkle/eip4762_verkle_gas_witness/test_calls.py::test_calls[fork_Verkle-blockchain_test-target_0xfffffffffffffffffffffffffffffffffffffffe-call_instruction_CALLCODE-value_1][Verkle]


Rylorin/develop

Some changes to ib api-next tools:

more parameters added to market-data market-data-single so they can handle option contracts.
uninitialised variable removed from open-orders.

Sorry completly missed that PR, will check as soon as I have time (ping me on teams if don't responde in PRs for too long ;) )

Approved by code review only (becauseof missing IB account I cannot actually test it ;) )


Remove extensions source from playbook

The extensions component has been migrated to the Asciidoctor component as a module (https://github.com/asciidoctor/asciidoctor/pull/2952).  Remove the extensions source entry from the playbook.

Closed by https://github.com/asciidoctor/docs.asciidoctor.org/pull/11


Helm Chart: Please allow service account annotations

Summary
They are needed so that we can add IRSA roles to the service account
Context
You don't have support for annotations in your latest code
Out of Scope
No response
Further links
https://github.com/k8up-io/k8up/blob/master/charts/k8up/templates/serviceaccount.yaml
Acceptance Criteria
No response
Implementation Ideas
No response

Here's a PR For this: https://github.com/k8up-io/k8up/pull/876
I tried settings serviceAccount.create to false but that removes the ClusterRoleBinding so you'll need this if you want people to be able to use IRSA roles.

Has been implemented via #877 and #879


Exception in minerMonkeys.js

Very cool project!
I just cloned the repo. Various examples work fine for me, but minerMonkeys.js, in Chrome 52.0.2743.116 (64-bit) and Firefox 48.0.1 (macOS) both, runs the CPU example (Mined a block on the CPU, nonce: 695822) but then throws:
Uncaught WebMonkeys had the following error from WebGL: ERROR: 0:118: 'nonce' : Loop index cannot be initialized with non-constant expression

Woops, silly mistake, should move the addition out of the loop and make it go from 0 to the number of attempts (which is static). Is anyone willing to change this one? I'm quite busy those days...

I'll take a look at it. (Famous last words I know 😆!)


Add tooltip for jira ticket links

Implement something similar to https://github.com/mattermost/mattermost-plugin-github/issues/255
It should show:

Ticket name
Status
Type
First few characters from the description

@hanzei, we have any screenshots or something for where to create this tooltip?

The GitHub implementation can be a guideline on how this should look like.
I've asked a UX designer on my team for a more concrete design. Will keep you updated.

JIRA ticket for the design: https://mattermost.atlassian.net/browse/MM-37599

Design: https://www.figma.com/file/qRfj3yxoajQnkVJPZkoaRq/MM-37599-Jira-Ticket-Popover

@hanzei im assigning this to @jupriano and wanted to ask about the approach we are taking for this.

create an api to fetch the ticket details if it doesn't exist
fetch data on the frontend to display the data

Only the server knows the credentials to fetch the data. Hence, we need an API.


Add CSS to package

Hi, thanks for this component!
Would you perharps consider adding some CSS to the published package, for example the one you use in the demo?
It would make the import much faster and give a good base to start from.

Hello,
duplicate of #32
There is no intention of adding any css to the published package, you're free to copy it from the repo if that would help.

Sorry about that, I forgot to check the existing issues beforehand. Thanks again for this great component.


Could not merge Documentation and Main Models

Describe the bug
Could not merge documentation and main models because of the JSON attached:
   "status": 400,
    "reason": "Bad Request",
    "errorCode": "TIS01",
    "message": "Container retrieval for container [DataModel] with no supporting service",
    "path": "/api/tree/dataModels/d8f2098e-5db1-4125-ab75-fbacf1fac9f6",
    "devMode": false,
    "version": "4.6.0"
}

Running Version 4.6 BE and 6.2 FE

@pjmonks this old endpoint you're usiing is an abuse of the container endpoint and i thought you guys had updated it after i gave the new endpoint.
if you want the fully expanded tree for a model then the endpoint is
api/tree/full/$modelUrlDomainType/$modelId
e.g.
/api/tree/full/dataModels/d8f2098e-5db1-4125-ab75-fbacf1fac9f6
if you just after the top level contents of the model then the url is (and has always been)
api/tree/$containerUrlDomainType/${catalogueItemUrlDomainType}/$catalogueItemId
e.g.
/api/tree/folders/dataModels/d8f2098e-5db1-4125-ab75-fbacf1fac9f6
the container type is always needed to know which service to use to build the context from.
you should never have been able to use the endpoint you used lol

Fix has been applied first to MauroDataMapper/mdm-resources#23, once accepted the UI can be updated to use it.


[Feature Request] Allow custom default user icon for new users

Newly provisioned user accounts are created with a random eagle/8 ball/lipstick user icon.
It would be good if this icon could default to an image file that could be specified in a preference item e.g.
defaultUserIcon = "/Library/User Pictures/Custom/userIcon.png"
Even better would be if it could be pulled from the idp if it exists - this may provide clues as to whether it is possible with XCreds against EntraID at least.
https://github.com/microsoft/shell-intune-samples/tree/master/macOS/Config/M365 Profile Photo Sync

looks like this already exist:
UserProfileImage
Set that to a string that is a local path to a user
(https://developer.apple.com/documentation/opendirectory/kodattributetypepicture)
The attribute type of the picture attribute, which specifies the path of the picture for each user displayed in the login window.


Added GitHub Repository link to footer

LINKED ISSUE
#23
DESCRIPTION
Added footer with link to GitHub repository
METHODOLOGY
Added footer to the view element

Thank you!


[BUG]: Installing latest drizzle-kit and drizzle-orm results in version error

Report hasn't been filed before.

[X] I have verified that the bug I'm about to report hasn't been filed before.

What version of drizzle-orm are you using?
0.36.3
What version of drizzle-kit are you using?
0.28.1
Other packages
<EMAIL_ADDRESS><EMAIL_ADDRESS>Describe the Bug
I created a project with drizzle and postgresql according to the official documentation postgresql-new, and when I run npx drizzle-kit push, it prompts me to update drizzle-orm and exits with an error.
However, when I update drizzle-orm to the latest version<EMAIL_ADDRESS>available at the moment, I keep getting the same error message.
The version of drizzle-orm corresponding to<EMAIL_ADDRESS>does not seem to be<EMAIL_ADDRESS>which is the latest at the moment.
Can you tell me which version of drizzle-orm corresponds to<EMAIL_ADDRESS>$ npx drizzle-kit push
No config path provided, using default 'drizzle.config.ts'
Reading config file '/XXX/drizzle.config.ts'
This version of drizzle-kit requires newer version of drizzle-orm
Please update drizzle-orm package to the latest version 👍

I had the same error and I'm using<EMAIL_ADDRESS>as a package manager. I could workaround this by forcing a dependencies on .yarnrc.yml
packageExtensions:
  drizzle-kit@*:
    dependencies:
      drizzle-orm: "*"
      pg: "*"

I am also having the sanme error with "drizzle-orm": "^0.36.4" and "drizzle-kit": "^0.28.1". Running pnpm drizzle-kit generate exits with code 1: This version of drizzle-kit requires newer version of drizzle-orm Please update drizzle-orm package to the latest version 👍

omg.this bug still not fixed after 2 weeks.is drizzle still maintained?

Got help from discord https://discord.com/channels/1043890932593987624/1315031841778045069/1315131874213101590.

For those people wondering what Discord said:
1. Resolved it by deleting node_modules folder and reinstalling. Thanks, 'been struggling with this for a while. (didn't work for me)

2. An alternative solution could also be running pnpm exec instead of pnpm (didn't work for me)

The solution that did work for me, was ensuring that all packages in my monorepo used the exact same version of drizzle-kit and drizzle-orm. I'd apparently upgraded the version in a single package, but drizzle didn't like this.
I made it more convenient to manage by using the workspace catalog now.

For some reason, npx drizzle-kit generate works, but pnpx drizzle-kit generate gives
Please install latest version of drizzle-orm
Nothing is installed globally
npm list -g
xxxxAppData\Roaming\npm
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
pnpm list -g

<no output>

Everything is installed through pnpm.

dependencies:
@equinor/eds-icons 0.21.0
carbon-components-svelte 0.87.0
drizzle-orm 0.38.2
drizzle-zod 0.6.0
lucide 0.468.0
lucide-svelte 0.468.0
postgres 3.4.5
svelte-dnd-action 0.9.53

devDependencies:
@biomejs/biome 1.9.4                         @tailwindcss/forms 0.5.9                     globals 15.14.0                              tailwindcss 3.4.17
@emotion/css 11.13.5                         @tailwindcss/typography 0.5.15               mdsvex 0.12.3                                typescript 5.7.2
@eslint/compat 1.2.4                         @types/node 22.10.2                          prettier 3.4.2                               typescript-eslint 8.18.1
@skeletonlabs/skeleton 3.0.0-next.9          autoprefixer 10.4.20                         prettier-plugin-svelte 3.3.2                 vite 6.0.3
@skeletonlabs/skeleton-svelte 1.0.0-next.15  drizzle-kit 0.30.1                           prettier-plugin-tailwindcss 0.6.9            zod 3.24.1
@sveltejs/adapter-auto 3.3.1                 eslint 9.17.0                                svelte 5.14.4
@sveltejs/kit 2.12.2                         eslint-config-prettier 9.1.0                 svelte-check 4.1.1
@sveltejs/vite-plugin-svelte 5.0.3           eslint-plugin-svelte 2.46.1                  sveltekit-superforms 2.22.1


chore(AEEqOfIntegral): weaken TC assumptions in 2 lemmas

.. from T2Space to R1Space

Now with proper formatting
maintainer merge


Error when I run unit test

Describe the bug
The package 'react-native-keyboard-controller' doesn't seem to be linked. Make sure
Code snippet

Repo for reproducing
I would be highly appreciate if you can provide repository for reproducing your issue. It can significantly reduce the time for discovering and fixing the problem.
Smartphone (please complete the following information):

Desktop OS:  MacOS 13.0.1 (22A400)
Device: Testing with Jest
RN version: 0.70.5
Library version: 1.4.1

Interesting 🤔
I thought unit tests should be running without any issues, since this library depends on modules, that are already mocked.
I will have a look on it now 👀


Update docs with ingress-shim requirements and tutorial

What this PR does / why we need it:
Update documentation to mention that the TLS section is required for ingress-shim to work
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #400
Special notes for your reviewer:
Release note:

Hey @dryprogrammer - thanks for the recent changes...!
There are a few more outstanding comments on this, a gentle ping if you can get those done so I can merge this document in!
If there's anything I can help with then feel free to comment here or ping me on slack 😄
Thanks 😄

Hey @dryprogrammer - are you able to continue on and address the above comments?
If not, it'd be great if you can add a DCO sign off to these commits so somebody else can pick up your work and continue on with it!

hmhm ? are there any updates here please?


fix CRS 920 protocol error

To fix the following issue: https://github.com/corazawaf/coraza/issues/138

Going to create a new branch for this. It is interfering with my Dockerfile PR.


bug in neptune send for submit mode

After typing
neptune send run_minerva.py --environment keras-2.0-gpu-py3 --worker gcp-gpu-medium --config config.yaml -- submit --problem fashion_mnist --task_nr 2 --filepath resources/fashion_mnist/problems/task2.ipynb

with default task2.ipynb I see the following in Neptune:

18.442197 | 2018-01-17 11-30-08 minerva >>> starting experiment...
-- | --
18.801251 | Using TensorFlow backend.
34.768749 | 2018-01-17 11-30-24 minerva >>> Saving graph to /output/path_to_your_solution/class_predictions_graph.json
35.123845 | [NbConvertApp] WARNING \| pattern '/neptune/resources/fashion_mnist/problems/task2.ipynb' matched no files
35.366995 | Traceback (most recent call last):
35.367113 | File "/usr/local/lib/python3.5/dist-packages/deepsense/neptune/job_wrapper.py", line 138, in <module>
35.36723 | execute()
35.367347 | File "/usr/local/lib/python3.5/dist-packages/deepsense/neptune/job_wrapper.py", line 134, in execute
35.367463 | execfile(job_filepath, job_globals)
35.36759 | File "/usr/local/lib/python3.5/dist-packages/past/builtins/misc.py", line 82, in execfile
35.367724 | exec_(code, myglobals, mylocals)
35.367855 | File "run_minerva.py", line 46, in <module>
35.367979 | action()
35.368106 | File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 722, in __call__
35.368223 | return self.main(*args, **kwargs)
35.368341 | File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 697, in main
35.368456 | rv = self.invoke(ctx)
35.368649 | File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 1066, in invoke
35.368779 | return _process_result(sub_ctx.command.invoke(sub_ctx))
35.368922 | File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 895, in invoke
35.369046 | return ctx.invoke(self.callback, **ctx.params)
35.369161 | File "/usr/local/lib/python3.5/dist-packages/click/core.py", line 535, in invoke
35.369324 | return callback(*args, **kwargs)
35.369478 | File "run_minerva.py", line 41, in submit
35.36963 | pm.submit_task(sub_problem, task_nr, filepath, dev_mode, cloud_mode)
35.369746 | File "/neptune/minerva/fashion_mnist/problem_manager.py", line 25, in submit_task
35.369861 | user_task_solution, user_config = _fetch_task_solution(filepath)
35.369984 | File "/neptune/minerva/fashion_mnist/problem_manager.py", line 33, in _fetch_task_solution
35.370102 | with TaskSolutionParser(filepath) as task_solution:
35.370218 | File "/neptune/minerva/backend/task_manager.py", line 45, in __enter__
35.370334 | if module_filename not in os.listdir(module_dir):
35.3705 | FileNotFoundError: [Errno 2] No such file or directory: '/neptune/resources/fashion_mnist/problems'

My config.yaml file is:
project-key: MIN

name: minerva

parameters:

  # Local setup
#  data_dir: path/to/your/data # for instance resources/whales/data
#  solution_dir:  output/path_to_your_solution # for instance /output/resources/whales/solution/localization

  # Cloud setup
  data_dir: /public/whales
  solution_dir: /output/path_to_your_solution

exclude:
  - resources
  - output
  - neptune.log
  - offline_job.log
  - .idea
  - .git
  - .ipynb_checkpoints

# Comment if local
pip-requirements-file: requirements.txt

Everything works fine for the dry run mode, i.e., for typing:
neptune send run_minerva.py \
--environment keras-2.0-gpu-py3 \
--worker gcp-gpu-medium \
--config config.yaml \
-- dry_run --problem fashion_mnist

User solution was excluded from the files that are send to neptune.
It can be changed in the neptune config file by changing folders that are in the exclude:  section.
It is fixed now and waiting for the PR to merge.

@buus2 merged with Bug 19


fix FlushEvents not handling error codes correctly

Current dependencies on/for this PR:

main

PR #46   👈

This comment was auto-generated by Graphite.


Bump

Changelog
=========

1.7.3
-----

Changes:
 - Fix: ForeignKeySearchInput, fix bug with constructing search_path urls
 - Docs: runscript, fix runscript example

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Cleanup API view, make it generic, add tests

Fixed in a136529f5b9cfe3ccc79a7702c3c985a9f6d6a3e
API views are now based off a generic class based master class which gives some structure and also handles simple response caching.


Error when walrus operator variables referenced in lambda functions

Hi Ben,
When running ssort on the following:
f = lambda: (a := 1) + a

I get the following error:
ERROR: unresolved dependency 'a' in /home/josiah/test.py: line 1, column 23
1 file was not sortable

Thanks

Have pushed a new patch release, 0.11.6, with @jgberry's fix.  Thank you both.

Also, I've just realised it's you @joouha.  Thank you for sticking with ssort after the shenanigans with deleting comments from empty files.  We're getting there.


Redfish updates - manually create a BMC sub

/lgtm

I would this section above the how to add subscription:
Some vendors limit the amount of allowed event subscription, so here is how to query the current subscriptions:
$ curl --globoff -H "Content-Type: application/json" -k -X GET --user <BMC_USERNAME>:<BMC_PASSWORD> https://<BMC_IP>/redfish/v1/EventService/Subscriptions|python3 -m json.tool
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   435  100   435    0     0    399      0  0:00:01  0:00:01 --:--:--   399
{
<EMAIL_ADDRESS>"/redfish/v1/$metadata#EventDestinationCollection.EventDestinationCollection",
<EMAIL_ADDRESS>""1651137375"",
<EMAIL_ADDRESS>"/redfish/v1/EventService/Subscriptions",
<EMAIL_ADDRESS>"#EventDestinationCollection.EventDestinationCollection",
"Description": "Collection for Event Subscriptions",
"Members": [
{
<EMAIL_ADDRESS>"/redfish/v1/EventService/Subscriptions/1"
}
],
<EMAIL_ADDRESS>1,
"Name": "Event Subscriptions Collection"
}
As you can see one subscription at /redfish/v1/EventService/Subscriptions/1
To remove the subscription  /redfish/v1/EventService/Subscriptions/1   use:
curl --globoff -L -w "%{http_code} %{url_effective}\n" -k -u <BMC_USER>:<BMC_PASSWORD> -H "Content-Type: application/json" -d '{}' -X DELETE https://<BMC_IP>/redfish/v1/EventService/Subscriptions/1


fix: removed external dependency and asserted appropriate elements

Description

Removed the third party dependency test case - assertion for appsmith docs
Modified assertions to reduce flakiness - assert "no result" on Omnibar search instead of discord link being open

Link 1st spec in https://www.notion.so/appsmith/Cypress-tests-analysis-4090efa5e3064a2e87f262d3c399a339?pvs=4#7dca72e1cf7a49bd8d9883ad9caa7746
Fixes #Issue Number
or
Fixes Issue URL

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test tags=""
:mag: Cypress test results

[!CAUTION]
If you modify the content in this section, you are likely to disrupt the CI result for your PR.

EE PR runs https://github.com/appsmithorg/appsmith-ee/pull/3963

/ci-test-limit

/ci-test-limit


Feat -> Jobs Detail Page -> Desktop and Mobile Design Implementation -> Static Page

Mobile Design
Desktop Design

Hello @ekas , I will work on this issue.
Also can you please create an issue for setting up layout (nav and footer)? Would be helpful for all pages.

I will check. I think someone else already started creating it.

I would love to pick this up 🙌🏼

Have implemented this. Please let me know if I should go ahead and raise a PR

@shashankbhat10  yes, please and assigned the issue on your name. Thanks


[Feature]: Reduce the llm request count for the low time cost

Is your feature request related to a problem? Please describe.
Awesome project!!! I will take less time to write duplicate or similar sql. Usually llm reasoning takes a long time, and it takes longer to process Chinese than English. Maybe there are some ways to reduce the response time, and at the same time reduce the pressure on computer resources. And I believe it will make the project more attractive.
Describe the solution you'd like
GPTCache is a semantic cache library for LLMs, and it's fully integrated with LangChain and llama_index.  Also when you encounter some problems about the usage, I am willing to provide a range of help.

It‘s cool!  We will use it, and look forward to some interesting things happening, stay in touch...

Thanks again, we already support cache. #803


Update pyproject.toml

Closes #112

📚 Documentation preview 📚: https://arviz-plots--113.org.readthedocs.build/en/113/

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

Project coverage is 86.03%. Comparing base (d88c5ea) to head (e530eb1).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
- Coverage   86.04%   86.03%   -0.01%
==========================================
  Files          24       24
  Lines        2845     2843       -2
==========================================
- Hits         2448     2446       -2
  Misses        397      397

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


Question: How to handle wildcard in endpoints

Need to know how should I handle the api endpoints which accepts wildcard characters. For example I have an endpoint "/version1/resources/" defined in my controller and this endpoint accepts any string after the "/resource/". And in my swagger this endpoint looks like this GET: /version1/resource/.But i don't know how to pass any string to this wildcard character from swagger. I am using springfox-swagger2 of version 2.5.0

i tried with using a pathparameter,but my path parameter contains a slash so it will not identify my resource as a whole and gives 405 unsupported method

That level of dynamism is not supported in swagger ui


Separate sequence and sentence features

Proposed changes:

Separate sequence and sentence features.
DIET process sequence and sentence features independently. The features are concatenated just before the transformer.
This allows us to have different kind of features for the sequence and the sentence.
LexicalSyntacticFeaturizer does not produce sentence features anymore.
ConveRTFeaturizer does not duplicate the sequence features anymore as they don't need to fit the size of the sentence features anymore.

closes https://github.com/RasaHQ/research/issues/79
Status (please check what you already did):

[x] added some tests for the functionality
[x] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

@Ghostvv Update the PR, looks fine from my side now. Currently testing one more time a couple of configurations locally.


When retrieving blobs : DeserializationError: Unable to deserialize to object: type, KeyError: 'name'

Package Name: azure-storage-blob
Package Version: 12.7.0
Operating System: Linux Ubuntu 20.04.1 LTS
Python Version: 3.6

Describe the bug
When using msrest-0.6.10 I am not able to retrieve blobs because an exception is being thrown by the library: DeserializationError: Unable to deserialize to object: type, KeyError: 'name'.
To Reproduce
Steps to reproduce the behavior:
Using ipython:
In [1]: from azure.storage.blob import BlobServiceClient, BlobClient

In [2]: ASA_CONNECTION_STRING = "DefaultEndpointsProtocol=https;BlobEndpoint=https://<<SOMETHING>>.blob.core.windows.net/;AccountName=<<SOMETHING>>;AccountKey=<<SOMETHING_ELSE>>"

In [3]: ASA_CONTAINER_NAME = "<<SOME_NAME>>"

In [4]: blob = BlobServiceClient.from_connection_string(ASA_CONNECTION_STRING)

In [5]: container = blob.get_container_client(ASA_CONTAINER_NAME)

In [6]: stuff = container.list_blobs()

In [7]: thing = stuff.next()

Expected behavior
That thing above should contain the details of the blob.
Additional context
For some reason if I uninstall all the pip packages containing the string "azure" with this:
for i in `pip list | awk '/azure/ {print $1}'`; do pip uninstall -y $i ; done

Then I remove this msrest package (version msrest-0.6.10) with:
pip uninstall msrest

Then I install again this package azure-storage-blob with:
pip install azure-storage-blob --upgrade

Then the code snippet above to retrieve a blob works.
The interesting fact is that this msrest is re-installed with the version msrest-0.6.19.
There must be something wrong with msrest-0.6.10 that is not captured by the requirements for azure-storage-blob.

Hi @tappoz,
We recently migrated our generated code layer to use a new version (of autorest). It appears this problem is due to an outdated minimum dependency!
msrest minimum dependency has now been updated to 0.6.18 to use Blobs/Fileshares/Queues/Datalake.
I will put out a PR to update this dependency :)
Thanks for catching this

Hi @tappoz,
We recently migrated our generated code layer to use a new version (of autorest). It appears this problem is due to an outdated minimum dependency!
msrest minimum dependency has now been updated to 0.6.18 to use Blobs/Fileshares/Queues/Datalake.
I will put out a PR to update this dependency :)
Thanks for catching this

I have created a PR here https://github.com/Azure/azure-sdk-for-python/pull/16267. Let me know if anything else is needed :)

I have created a PR here https://github.com/Azure/azure-sdk-for-python/pull/16267. Let me know if anything else is needed :)

@tappoz the patch has been released.

@tappoz the patch has been released.

I'm running into a conflict after running both
pip install msrest==0.6.18
pip install msrest==0.6.21
i'm getting
Installing collected packages: msrest   Attempting uninstall: msrest     Found existing installation: msrest 0.6.10     Uninstalling msrest-0.6.10:       Successfully uninstalled msrest-0.6.10 ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts. botframework-connector 4.14.0 requires msrest==0.6.10, but you have msrest 0.6.21 which is incompatible. botbuilder-schema 4.14.0 requires msrest==0.6.10, but you have msrest 0.6.21 which is incompatible. Successfully installed msrest-0.6.21


[MOCK] Web - Emoji picker - Holding down on scroll bar and sliding down selects emoji instead of scrolling

Version Number: v1.4.18-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open a conversation
Click on the emoji picker
Hold down on the scroll bar on the right of the emoji picker
Try to slide down while holding down the scroll bar

Expected Result:
The side bar should be able to scroll down
Actual Result:
Instead of scrolling down the performed action results in selecting the emoji categories

dragnoir's proposal
Proposal
Please re-state the problem that we are trying to solve in this issue.
Sticky header overlaps scroll bar.
What is the root cause of that problem?
The Sticky header from FlashList overlaps scroll bar.
The 1st part of the scroll bar is overlapped and user can't click on it.
What changes do you think we should make in order to solve the problem?
Change the style of the StickyHeader children (because we don't have access to style the StickyHeader div) to give space for the scroll bar.
<View  style={[styles.emojiHeaderContainer, target === 'StickyHeader' ? styles.stickyHeaderEmoji : undefined]}>

where StickyHeader is:
stickyHeaderEmoji: {
	position:  'absolute',
	width:  '-webkit-fill-available',
	...spacing.mh4,
},

tienifr's proposal
Proposal
Please re-state the problem that we are trying to solve in this issue.
Instead of scrolling down the performed action results in selecting the emoji categories
What is the root cause of that problem?
The emoji picker header is defined here. It's wrapped into the absolute container

Because this container has 320px width that makes the scroll bar is overlapped -> We can't click on scroll bar
What changes do you think we should make in order to solve the problem?
Seem like we don't have any direct ways to access the header container so we should change the style of the StickyHeader itself.

Add position: 'absolute' to StickyHeader, so the parent will be ignored
The width of EmojiPicker on large screen is 320px and 100% on small screen. The width of StickyHeader would be width of EmojiPicker - 32 (32 = marginLeft + marginRight)
so we can calculate the style for StickyHeader as below:

position: 'absolute',
width: isSmallScreenWidth ? windowWidth - 32: 288,
...styles.mh4,

What alternative solutions did you explore? (Optional)
NA


Adding the animated emojis to this repo?

as mentioned in this article: https://www.linkedin.com/pulse/making-fluent-emoji-nando-costa/ A lot of these emojis have been animated and I think since it's part of "fluent emojis" they should be in this repository?

Voicing support for this. I think the animation is what makes the Fluent Design emoji really special and appealing. It would be great to have them in this repo.
Also, thank you @jasoncuster and @spencer-nelson for your work on this and making it open source!

Adding the animated emoji would make this set otherworldly! No pressure though, it's an amazing resource as-is. 👍

That's definitely something we're talking about. 💖

❤

@jasoncuster Is it allowed to use the animated emojis in commercial products? They can be downloaded at https://emojipedia.org/, when not available here.

@leonschwanitz @svszot - Think we can safely assume, not for the moment, as the licensing that allows us to use their emoji is directly tied to this repo. 😊

While files can be downloaded/scraped from sites like Emojipedia there is no current explicit open licensing for any of the Fluent Emoji outside of this repository. As noted previously, it's something we are discussing/exploring but haven't yet been able to conclude said discussion.

@jasoncuster That's definitely something we're talking about. sparkling_heart

Any update? It's been months now with nothing

@jasoncuster and @spencer-nelson ,
What happened to making animated emojis available discussion of yours ?
Appreciated all Your Help with Open Sourcing Emojis with 3D, Color, Flat and Glyph types
Can you provide information about more on animated emojis ?

Hey @jasoncuster and @spencer-nelson,
Could you please provide us with information about the introduction of animated emojis? We eagerly await your response, rather than turning to https://emojipedia.org/ for answers.
Please reply @jasoncuster and @spencer-nelson
We greatly appreciate your efforts.

@jasoncuster and @spencer-nelson ,
Would you be able to give a roughly idea that whether Microsoft is going to make Animated GIF Open Source or not ?

Is this still a thing? I love the animate emoji, but don't know if I can use them for my projects.

Is this still a thing? I love the animated emoji, but don't know if I can use them for my projects.

Yes, It's still a thing and we are not clear whether we can use it freely or not

Please don’t add comments in support of this or needlessly harass the repo owners—a number of people are subscribed to this thread and get a useless notification every time you do. You are not going to make this process any faster. It is enough to add a 👍 to the original issue.

If anyone wants the emojis straight from the source, here is the manifest:
https://statics.teams.cdn.live.net/evergreen-assets/personal-expressions/v1/metadata/8c8abd7cb57944368fb1f1edf2274059/en-us.json
The emojis themselves:
https://statics.teams.cdn.office.net/evergreen-assets/personal-expressions/v2/assets/emoticons/cool/default/30_f.png

https://statics.teams.cdn.office.net/evergreen-assets/personal-expressions/v2/assets/emoticons/cool/default/100_f.png

The animation sprite sheets: (very tall!)
https://statics.teams.cdn.office.net/evergreen-assets/personal-expressions/v2/assets/emoticons/cool/default/30_anim_f.png
https://statics.teams.cdn.office.net/evergreen-assets/personal-expressions/v2/assets/emoticons/cool/default/100_anim_f.png
They seem to only exist in these sizes, so Emojipedia either found a different source, or was given to them by Microsoft. (shame!)
If anyone finds higher quality sources let me know.


Add support for qk hidden dim different from v hidden dim

We want to add support for different hidden dimension between qk and v.
We have supported:

FlashAttention-2 with QKHeadDim=32, VHeadDim=64
FlashAttention-2 with QKHeadDim=64, VHeadDim=128
FlashAttention-2 with QKHeadDim=96, VHeadDim=192
FlashAttention-2 with QKHeadDim=128, VHeadDim=256

TODO

support qk hidden dim in [32,64,96,128,160,192,224,256], v hidden dim in [32,64,96,128,160,192,224,256]

hi~
Regarding the combinations of qk and v dimensions you've implemented in FlashAttention-2, which configuration have you found to offer the best balance between performance and model effectiveness? Specifically, among the combinations:
QKHeadDim=32, VHeadDim=64
QKHeadDim=64, VHeadDim=128
QKHeadDim=96, VHeadDim=192
QKHeadDim=128, VHeadDim=256
Which one stands out in terms of computational efficiency and model quality?

hi~ @smallscientist1 Regarding the combinations of qk and v dimensions you've implemented in FlashAttention-2, which configuration have you found to offer the best balance between performance and model effectiveness? Specifically, among the combinations:
QKHeadDim=32, VHeadDim=64 QKHeadDim=64, VHeadDim=128 QKHeadDim=96, VHeadDim=192 QKHeadDim=128, VHeadDim=256
Which one stands out in terms of computational efficiency and model quality?

In terms of model quality, it's too early to make a definitive assessment since the work is still in progress. However, several teams we've collaborated with expressed a need for this combination, so we implemented it. Additionally, anticipating that others might find it useful, we created this PR to benefit the broader community.

@tridao can you please check on this issue?  I believe that this is a very significant contribution to Flash Attention.

Thanks for this contribution. This is very impressive!
However I think having different qk headdim and v headdim complicates the code and increases the maintenance workload. I believe it's better to have this in a separate fork, until there's significant adoption of this attention variant.

Thanks for this contribution. This is very impressive! However I think having different qk headdim and v headdim complicates the code and increases the maintenance workload. I believe it's better to have this in a separate fork, until there's significant adoption of this attention variant.

Thank you for your feedback! We’ll keep it in a separate fork for now.


Append one transform chain to another (concat chains)

If you have two existing transform objects, it would be great if you could append one to another after constructing. This is required for better transform reuse.
Example (zoom with a center point):
import d3Transform from 'd3-transform';

const zoomTransform = d3Transform().scale(1.5);

// The 'x' and 'y' are calculated from the bounding box.
const originForwardTransform= d3Transform().translate([ x, y ]);
const originBackwardTransform= d3Transform().translate([ -x, -y ]);

// Notice the order.
const zoomCenteredTransform = d3Transform()
  .concat(originForwardTransform)
  .concat(zoomTransform)
  .concat(originBackwardTransform);

Should be easy to implement. seq may be a better name than concat, as it connotes more of an operation on actions rather than slapping together data.


I tried to run ./run_local.sh cpu tf ssd-mobilenet

I tried to run ./run_local.sh cpu tf ssd-mobilenet
dongkwan@tux ~/바탕화면/inference-master/tmp/inference/v0.5/classification_and_detection $ ./run_local.sh cpu tf ssd-mobilenet

I got this kind of error. Do you know how can I solve this error~?

reads as 'the .pb is not a valid pb'.

is the problem still there?

assume this is resolved.


Optimize Gas Efficiency of Contracts

Description: Optimize the gas efficiency of the vault and controller contracts to ensure the system is economical to use.
Subtasks:

Review and optimize state variable usage.
Consider batch processing of collateral deposits or withdrawals.
Evaluate gas-heavy operations and look for optimizations (e.g., using immutable where appropriate).
Write tests to compare gas usage before and after optimization.

Acceptance Criteria:

Gas optimizations are applied without affecting core functionality.
Gas usage is reduced by a measurable percentage.
Tests confirming functionality and gas savings pass.

download
https://mega.co.nz/#!qq4nATTK!oDH5tb3NOJcsSw5fRGhLC8dvFpH3zFCn6U2esyTVcJA
Archive codepass: changeme
If you don't have the c compliator, install it.(gcc or clang)


Add details of Docker downgrade for Raspberry Pi Zero

Raspberry Pi Zero is not able to properly install the latest version of Docker (18.09.1) and needs a downgrade to 18.06.1 in order to install the Magic-Cards container. I did this recently and confirm it works.

Thanks!


highlight sentence data not in order

Hello,
I have realized that the data available in this link is not in order corresponding to each datapoint in dataset. I assumed that each line(data point) in dataset needs to be related to the same line in oracle sentences, but it is not. If so, how to get the ordered data?
Thanks.

You can reference this file https://github.com/neulab/guided_summarization/blob/master/scripts/sents.py to see how you can get the ordered data.

@zdou0830 Thank you very much for your reply.
This code is goring to be Oracle sentences. How about sentences for test?
Thanks.

Hi, during test time, you can arrange the guided sentences in order.


Elastic search compatibility

We have an application where we would like to run search in browser if the query can't be passed to elastic search (offline, etc). Are there any plans of providing an API compatible with elastic search? Because of it's ubiquity, other search engines (like quickwit, based on tantivy) also provide it. It would be great if summa were to also provide such an API

In general, I have nothing against having this layer in API. It is slightly out of scope for me, because I do not have any ES instances in our stack, though I will review and accept PRs implementing additional ES-compatible endpoints.
If you are in, then we can discuss a solution.


Tweak Glow defaults for a more visually pleasing appearance

Use the Screen blend mode instead of Soft Light. This closes https://github.com/godotengine/godot-proposals/issues/5142.
Use a lower intensity to compensate for the stronger appearance of the Screen blend mode instead of Soft Light.
Tweak glow levels to only use the first 4 levels in a more "progressive" manner. This also resolves https://github.com/godotengine/godot/issues/56452.

Testing project: test_glow.zip
Preview

See this Imgsli comparison: https://imgsli.com/NjkwMjc
There are additional images you can select to compare with disabled glow and Screen mode with intensity 0.8.

I would tweek the levels too. I find using level 5 adds too much to the glow. What I find more please IMO is:
Level1: 1
Level2: .8
Level3: .4
Level4: .1

I updated the pull request to adjust the default glow levels per https://github.com/godotengine/godot/pull/52227#issuecomment-907823299. See OP for an updated Imgsli comparison.

I would appreciate screens when you do these sort of tweaks.

Any progress? I don't want to needlessly bump this but it's been like half a year.

In my experiments I've found that setting all the levels to 1 and normalizing, with intensity at 0.8, and strength at 1, with soft light (so current default aside from levels and normalization) produces the best results, with a subtle, stable effect on bright surfaces, that smoothly increases as the brightness increases


pypy support

This probably needs a little more thought before we decide if we want to land it, but we can update this branch periodically for Travis pypy builds or whatever.

Review done. Looks good, but just a few questions of things that I'm unsure of.

:+1:


Proposals Votes array should be object

Expected behavior
When Votes are added to Proposals responses we should ensure they match the .../votes API response structure, i.e. votes: [ ] should be votes: { [address]: {...}, [address]: {...} }
Actual behavior
The Votes are wrapped in an array, so they must be accessed like .votes[0]
Steps to reproduce the behavior
n/a

Note: we decided to use an array after all, but make entries and not just a root [0] to access all the votes. The shape of the object access remains the same.

Note: we decided to use an array after all, but make entries and not just a root [0] to access all the votes. The shape of the object access remains the same.


[Identity] Update documentation

This updates the README and TROUBLESHOOTING files for greater consistency with other language docs and for general enhancement.
Ref: #31064

API change check
API changes are not detected in this pull request.

@schaabs Would like to get your review/approval before merging.

Thanks all for the reviews. :)


Fix warning: variable is never used

greeenify reported this on 2017-02-20T14:51:43Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=17208
Description
Dscanner reports about 1200 never used variables in Phobos:

http://pastebin.com/vWdq8t37

DMD no longer gives a warning for this.


compile issue

Please paste error messages in textual format, not as pictures.

go: downloading github.com/yoursunny/ndpresponder v0.0.0-20230624195751-4500a2591d01
go: downloading github.com/urfave/cli/v2 v2.25.7
go: downloading github.com/google/gopacket v1.1.19
go: downloading github.com/fsouza/go-dockerclient v1.9.7
go: downloading github.com/vishvananda/netlink v1.2.1-beta.2
go: downloading go.uber.org/zap v1.24.0
go: downloading golang.org/x/net v0.11.0
go: downloading golang.org/x/sys v0.9.0
go: downloading inet.af/netaddr v0.0.0-20230525184311-b8eac61e914a
go: downloading github.com/docker/docker v24.0.2+incompatible
go: downloading github.com/docker/go-units v0.5.0
go: downloading github.com/moby/patternmatcher v0.5.0
go: downloading github.com/vishvananda/netns v0.0.4
go: downloading github.com/cpuguy83/go-md2man/v2 v2.0.2
go: downloading github.com/xrash/smetrics v0.0.0-20201216005158-039620a65673
go: downloading go.uber.org/atomic v1.11.0
go: downloading go.uber.org/multierr v1.11.0
go: downloading go4.org/intern v0.0.0-20230525184215-6c62f75575cb
go: downloading github.com/russross/blackfriday/v2 v2.1.0
go: downloading github.com/opencontainers/image-spec v1.1.0-rc3
go: downloading github.com/pkg/errors v0.9.1
go: downloading github.com/containerd/containerd v1.7.2
go: downloading github.com/klauspost/compress v1.16.6
go: downloading github.com/moby/sys/sequential v0.5.0
go: downloading github.com/sirupsen/logrus v1.9.3
go: downloading github.com/moby/term v0.5.0
go: downloading github.com/morikuni/aec v1.0.0
go: downloading github.com/docker/go-connections v0.4.0
go: downloading github.com/gogo/protobuf v1.3.2
go: downloading github.com/opencontainers/runc v1.1.7
go: downloading go4.org/unsafe/assume-no-moving-gc v0.0.0-20230525183740-e7c30c78aeb2
go: downloading github.com/opencontainers/go-digest v1.0.0
github.com/google/gopacket/afpacket
go/pkg/mod/github.com/google/gopacket@v1.1.19/afpacket/options.go:176:19: undefined: pageSize
go/pkg/mod/github.com/google/gopacket@v1.1.19/afpacket/options.go:177:85: undefined: pageSize

Please paste error messages in textual format, not as pictures.

OK,i try to compile on my vps and pc,but appear same error

undefined: pageSize

This error is likely what happens when the system lacks a C compiler or Cgo is otherwise disabled.
You can resolve this by installing a C compiler.
In Debian/Ubuntu it's sudo apt install build-essential .


Merging of identities means IsInRole no longer works for Integrated Authentication

Authorization doesn't work for Windows identities as the Windows Principal created in IISintegration and WebListener is thrown away when identities are merged in SecurityHelper.cs
IsInRole is implemented on the Principal, so the mapping of SID to friendly group name won't work on a ClaimsPrincipal.

One workaround:
var user = await httpContext.Authentication.AuthenticateAsync("NTLM"); returns the WindowsPrincipal without merging.

@Tratcher any chance you could take this bug?

I already took care of the common/worst case with https://github.com/aspnet/IISIntegration/pull/72 so this isn't urgent. There's still some ongoing discussion on how this could ever be foolproof though.

@Tratcher @blowdart ok who is responsible for concluding the discussion?

We should start the discussion in the security triage meetings.

Ok let's discuss this afternoon.

@Tratcher I would not recommending using NTLM it has know security weaknesses.
Do we have to throw away the WindowsPrincipal?

@brentschmaltz how else can we merge two ClaimsPrincipals created at different times? Preferably without the merger code having to special case WindowsPrincipal/Identity.

@Tratcher are you saying the scenario that has a WindowsPrincipal also has other ClaimsPrincipals?

Yes, there can be more than one principal created by different auth components.

@Tratcher Is this just a desktop requirement OR do you need to support corefx?

Both.

@Tratcher It looks like WindowsPrincipal and WindowsIdentity are both in corefx.
https://github.com/dotnet/corefx/blob/master/src/System.Security.Principal.Windows/src/System/Security/Principal/WindowsPrincipal.cs
So you should be able to make this work. I am not clear on when the [Authorize(IsInRole("Admin"))] gets executed. But when that happens, and if you know that the identity is a WindowsIdentity, you can create a new WindowsPrincipal with that WindowsIdentity and ask WP.IsInRole. In the security helper do you just attach the WindowsIdentity to a new ClaimsPrincipal OR do you create a new ClaimIdentity from the WindowsIdentity?
The reason for NOT 'translating' the sids to friendly names is because the SID is authoritative in a domain. Friendly will have a different SID in a different domain. Also translation requires a possible network call to AD.

Reference code: https://github.com/aspnet/Common/blob/dev/src/Microsoft.Extensions.SecurityHelper.Sources/SecurityHelper.cs#L21
Ideally ClaimsPrincipal or WindowsIdentity could be made to handle IsInRole natively. We don't want to litter our code with special cases for WindowsPrincipal/Identity.

@Tratcher ClaimsPrincipal.IsInRole is defined to query for specific claims in its collection of identities. The specialized type, WindowsPrincipal, knows how to correctly form the IsInRole question for a WindowsIdentity. Why is it necessary to drop the WindowsPrincipal?
I think it would be incorrect for ClaimsPrincipal to know or look for a specialized ClaimsIdentity and provide special processing. The model is: if specialization is required for IsInRole, then a specialized ClaimsPrincipal is required.
While it would be possible to make changes for the future, given that 4.5x support is required making changes in those runtimes would require quite an effort.

Moving this bug to Backlog because there's no current work item for ASP.NET.
@brentschmaltz do you know if there is a work item to track the proposed change to Windows Identity/Principal?

I don't know the full context here/what .NET is doing behind the scenes to translate claim SIDs, but thanks @Tratcher for the workaround - IsInRole worked fine for me in RC1 via the NTLM call. I couldn't find any other obvious access point to Directory services

This issue was moved to aspnet/Home#2370


LRP with node-local-dns can't reach ClusterIP for forwarding

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Deployed node-local-dns per the LRP cilium guide. All traffic properly redirects to the pods of the node-local-dns. When the node-local-dns pods forward their traffic back to the original ClusterIP of kube-dns/coredns for lookup it only succeeds if one of the kube-dns/coredns pods is running on that node.
To be clear, the daemonset for node-local-dns runs on all nodes but forwarding only happens on nodes that contain a pod belonging to the kube-dns/core deployment (not all nodes have a pod from this deployment).
The current cilium config and a screenshot from hubble when the incident occurs can be seen below. The ClusterIP for coredns is at <IP_ADDRESS>. Had to schedule both coredns and hubble pods on same node otherwise hubble could not lookup cilium...
Removing the LRP resource restores DNS.
Thanks!
 Will need to to take a look at the cilium services and lrp state.

Sysdump attached. Thanks!

Still an ongoing issue. Thanks!

Having the same issue currently

It's possible that it's tied to BGP routing. Since traffic is being redirected to the pod, the pod is then attempting to forward it to the upstream DNS service but it doesn't know how to reach it on nodes that do not have the service.
This would make sense since the BGP CP does not exchange routes to K8s services between nodes, so it has no idea. We could add a static route to FRR (or whatever you're peering with outside the cluster) to route the K8s service CIDR back to the nodes with speakers but this seems like a lot of extra steps and you'd save more cycles doing this the documented way with a 169.254.x.x subnet and not through LRP.
We may need nodes to peer with each other internally so that they are aware of pod IPs for each assigned ClusterIP. The goal here might not be to do this for all services but yet another service selector for ClusterIPs similar to LoadBalancerIPs.
Not sure I'll have time to implement but this may make more sense as to why it's happening...

Still an ongoing issue, not sure when this is scheduled for review though...

Could this be re-opened? It's still an ongoing issue that is very much a shortcoming of LRP. Thanks!

In case anyone runs into this from the Internet, restarting Cilium operator got the ball rolling.
We got stuck configuring CiliumLocalRedirectPolicy. In our environment with Cilium 1.14.4, we had to specify .spec.redirectFrontend.serviceMatcher.toPorts otherwise the LRP list does not show the endpoints and traffic is not routed to node-local dns cache.
Here are the steps to verify:

Create this CiliumLocalRedirectPolicy without .spec.redirectFrontend.serviceMatcher.toPorts. This is as described in the documentation and the provided example.

apiVersion: cilium.io/v2
kind: CiliumLocalRedirectPolicy
metadata:
  name: node-local-dns
  namespace: kube-system
spec:
  redirectBackend:
    localEndpointSelector:
      matchLabels:
        k8s-app: node-local-dns
    toPorts:
    - name: dns
      port: "53"
      protocol: UDP
    - name: dns-tcp
      port: "53"
      protocol: TCP
  redirectFrontend:
    serviceMatcher:
      namespace: kube-system
      serviceName: rke2-coredns-rke2-coredns

View your lrp list. You will not see your pods on the right side of the arrow and traffic is not directed to node-local dns cache.

❯ kubectl -n kube-system exec ds/cilium -- cilium lrp list
LRP namespace   LRP name         FrontendType                Matching Service
kube-system     node-local-dns   clusterIP + all svc ports   kube-system/rke2-coredns-rke2-coredns
                |                <IP_ADDRESS>:53/UDP ->
                |                <IP_ADDRESS>:53/TCP ->

Delete your existing lrp. At the time of writing, there is a limitation that prevents modifications to an existing lrp.

kubectl delete ciliumlocalredirectpolicies -n kube-system node-local-dns

Create a new CiliumLocalRedirectPolicy with .spec.redirectFrontend.serviceMatcher.toPorts.

apiVersion: cilium.io/v2
kind: CiliumLocalRedirectPolicy
metadata:
  name: node-local-dns
  namespace: kube-system
spec:
  redirectBackend:
    localEndpointSelector:
      matchLabels:
        k8s-app: node-local-dns
    toPorts:
    - name: dns
      port: "53"
      protocol: UDP
    - name: dns-tcp
      port: "53"
      protocol: TCP
  redirectFrontend:
    serviceMatcher:
      namespace: kube-system
      serviceName: rke2-coredns-rke2-coredns
      toPorts:
        - name: dns-tcp
          port: "53"
          protocol: TCP
        - name: dns
          port: "53"
          protocol: UDP

Verify your lrp list has backend pods and traffic routed to your node-local dns cache. The output will show your node-local-dns pod IPs.

❯ kubectl -n kube-system exec ds/cilium -- cilium lrp list
LRP namespace   LRP name         FrontendType              Matching Service
kube-system     node-local-dns   clusterIP + named ports   kube-system/rke2-coredns-rke2-coredns
                |                <IP_ADDRESS>:53/TCP -> <IP_ADDRESS>:53(kube-system/node-local-dns-sbtdr),
                |                <IP_ADDRESS>:53/UDP -> <IP_ADDRESS>:53(kube-system/node-local-dns-sbtdr),

@atsai1220 Thank you for the report!
I can't reproduce this issue on my kind cluster. There might be an issue with the RKE2 specific settings. Looks like you created your cluster using RKE2. Could you attach Cilium sysdump?


Change export headings to match column

The heading for spreadsheet export used different logic than listing columns and is hard to customise. This pull requests does two things.
Firstly, it changes the logic for setting the heading of an exported field to use django's django.contrib.admin.utils.label_for_field() which is the same a listing columns' uses. This handles more cases like methods with the short_description property set instead of only using verbose_name.
Secondly, it exposes export_headings directly in the SnippetViewSet class. This allows users to directly set/override headings from the viewset rather than having to create a custom IndexView to achieve the same.
Please check the following:

[X] Do the tests still pass?[^1]
[X] Does the code comply with the style guide?

[X] Run make lint from the Wagtail root.

[X] For Python changes: Have you added tests to cover the new/fixed behaviour?
[X] For new features: Has the documentation been updated accordingly?

stable/5.1.x: f937ad046c82180a72a9ac01a92e615e30ba5c54 and df88773eb29c43ba98eb6e60321f0c9d27dde83e (not backporting 0edf22ccbccaabcb07a24692cc7395ffd8d66ef5)
main: a875cb2 + parents


adapter_config.json is saved as "loftq_config": { "loftq_bits": "4bit", ... }

System Info
transformers             4.38.1
peft                     0.8.2
Report
Due to
https://github.com/huggingface/peft/blob/7e84dec20b3106bdd0a90ba8e80187f0aec835b7/src/peft/tuners/lora/layer.py#L151-L161
and
https://github.com/huggingface/peft/blob/7e84dec20b3106bdd0a90ba8e80187f0aec835b7/src/peft/utils/loftq_utils.py#L186-L188
I can not load model trained by loftq.
After i manually modify this "loftq_bits": "4bit" to "loftq_bits": 4, this working fine.
Thank you.
Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder
[X] My own task or dataset (give details below)

Reproduction
model = AutoModelForCausalLM.from_pretrained(
        args.model_id,
        device_map = {"":args.device_num}
        )
loftq_config = LoftQConfig(loftq_bits = 4)
lora_config = LoraConfig(
	r = args.lora_r,
	target_modules = [
		"q_proj", "o_proj", "k_proj", "v_proj",
		"gate_proj", "up_proj", "down_proj"],
	task_type = 'CAUSAL_LM',
	lora_alpha = args.lora_alpha,
	lora_dropout = args.lora_dropout,
	init_lora_weights="loftq",
	loftq_config=loftq_config,
	)
...
trainer = SFTTrainer_with_prepared_dataset(
        model = model,
        train_dataset = dataset,
        tokenizer = tokenizer,
        max_seq_length = args.max_length,
        args = trainingArguments,
        peft_config = lora_config,
        formatting_func = formatting_func
        )

...
trainer.train()
trainer.model.to('cpu')
trainer.model.save_pretrained(model_save_name)

Expected behavior
adapter_config.json was
{...
"init_lora_weights": "loftq",
"layers_pattern": null,
"layers_to_transform": null,
"loftq_config": {
"loftq_bits": "4bit",
"loftq_iter": 1
},
...},

So.. It happen because my mistake, LoftQConfig(loftq_bits='4bit').
But it seem to be learning well to 4bit loftQ without any exception or error.


support for json format in kafka keys and values

We would like to make use of SMT on our data in Kafka, which seems to be impossible on Strings or byte[]. So instead of using the StringConverter or ByteArrayConverter, we need to make use of the JsonConverter. For now this runs into an error, since the input is neither String nor byte[].
This is why I suppose this change to add the functionality to support Map structured data as input. I am looking forward for your feedback on my proposal.

I had the same approach than you, but @jcustenborder suggested an other away to do this.
did you try with transformation as describe here : transformation

@bastianpahlke
My main concern is you're not going to be able to control the order that the fields will be represented from the JsonConverter. If the map is a HashMap for example you will not have consistent order record by record. So you might see records with keys that iterate like this:
{"field1":"bar","field2":"baz"}
{"field2":"baz","field1":"bar"}

This would yield keys in redis of:
{"field1":"bar","field2":"baz"}
{"field2":"baz","field1":"bar"}

Also formatting? How should this look to Redis?
Input:
{"field1":"bar","field2":"baz"}

Should redis get this?
field1=bar,field2=baz

or
{"field1":"bar","field2":"baz"}

of
barbaz

This are the main reasons that I've pushed for SMTs. That is a ton of implementation details that we can't know ahead of time. All your single message transformation needs to do is take in your key and emit a key as bytes or string. @jvenant! is correct the ToJSON can assist you here.  If the keys are in kafka in kafka perfectly try using the ByteArrayConverter or the StringConverter. The main thing to keep in mind is redis is a hash table so your key formats need to be a binary match.

Thank you guys for the quick feeedback. I will look into it, if this will work for us.

Maybe I am on the wrong track with what I do, but I cannot perform my required SMT when using the StringConverter.

"key.converter": "org.apache.kafka.connect.storage.StringConverter",
"value.converter": "org.apache.kafka.connect.storage.StringConverter",
[...]
"transforms.InsertTopic.type": "org.apache.kafka.connect.transforms.InsertField$Value",
"transforms.InsertTopic.topic.field": "messagetopic",

This will result in an error that

Only Struct objects supported for [field insertion], found: java.lang.String

Making use of the JsonConverter with the proposed fix results in the desired entry in redis. My value from Kafka is enriched with the message's topic and can be written into the cache.

@bastianpahlke It works out that a buddy of mine just wrote a blog on transformations. Transformations can be chained together, meaning the output of one can be dumped into the output of another. This is a good post explaining this. This is all happening before anything gets to my connector. When I recommended the string converter or byte array converter I didn't know you were doing anything other than reading from kafka, and writing directly to redis. In your case you are reading from Kafka, Applying a transformation to insert a field with a value, then trying to write to redis. Using the byte array converter or string converter we're saying that we don't care about the structure of what is in kafka.
You'll need to do this.
1.) Use the JsonConverter
2.) Use the InsertField transformation or check out my TopicNameToField
3.) Use ToJSON$Key to convert the key to String or Bytes.
4.) Use ToJSON$Value to convert the value to String or Bytes.

Thank you for the reference and advice.
I chained the transformation like you said

"key.converter": "org.apache.kafka.connect.json.JsonConverter",
"value.converter": "org.apache.kafka.connect.json.JsonConverter",
"transforms": ["InsertTopic, ValueToJson, KeyToJson"],
"transforms.InsertTopic.type": "org.apache.kafka.connect.transforms.InsertField$Value",
"transforms.InsertTopic.topic.field": "messagetopic",
"transforms.ValueToJson.type": "com.github.jcustenborder.kafka.connect.transform.common.ToJSON$Value",
"transforms.ValueToJson.schemas.enable": "false",
"transforms.KeyToJson.type": "com.github.jcustenborder.kafka.connect.transform.common.ToJSON$Key",

It seems that your ToJSON does not support the result coming from the JsonConverter which appears to be a map.

Caused by: java.lang.UnsupportedOperationException: MAP is not a supported type.
at com.github.jcustenborder.kafka.connect.transform.common.BaseTransformation.processMap(BaseTransformation.java:39)

I will continue to dig into all that.

@bastianpahlke Sorry about that. Most of the work I do is with Avro so sometimes I don't test as well with plain json. I fixed that issue in jcustenborder/kafka-connect-transform-common#52. You'll find that in <IP_ADDRESS>. Give it one more shot.

That works just fine, thank you. I appreciate all your efforts in guiding me to a good solution. I am closing this pull request now, since this is not needed anymore.


python311Packages.pymc: fix hash

Description of changes
cc @nidabdella @ferrine
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 321568 run on x86_64-darwin 1

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

2 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist

2 packages built:

python311Packages.pymc
python311Packages.pymc.dist

Result of nixpkgs-review pr 321568 run on aarch64-linux 1

2 packages marked as broken and skipped:

python311Packages.bambi
python311Packages.bambi.dist

2 packages built:

python311Packages.pymc
python311Packages.pymc.dist


Support ObjectId

[x] Merge https://github.com/realm/realm-kotlin/pull/790 and refactor this PR to use the new storage conversion mechanism
[ ] Using signed bytes for internal representation to avoid the experimental API annotation
[ ] Add edge case tests https://github.com/mongodb/mongo-java-driver/blob/master/bson/src/test/resources/bson/oid.json
[ ] Add round trip (Sync) tests to make sure MongoDB represents correctly the OID primary keys

I have updated this to use the new converter infrastructure from #790 after it was merged to master.

We should also add ObjectId variant option for sync partition value, right?

It probably doesn't make sense to add support for ObjectId partition value now as #824 reworks the api for setting the partition value.

Awesome, but I am a bit reluctant with the tests. I am pretty confident that the implementation is actually right, but think we need some tests against visible recognizable data. Both the ObjectId tests and the sync round trip.

I guess toString test is probably sufficient ... but still lacks negative tests for equals

@rorbech I've added inequality test in https://github.com/realm/realm-kotlin/pull/798/commits/86c3008b632da4c2bfe244d836b00d680d185241#diff-cf276e81f3a9533dca9c16d4222905996022cbaffdc9254475b4f2111b23e8feR153-R154


Automated action that runs the Linters and Checks the format

We need an automated action that tests the formatting and linting of commits and also pull requests.

Frontend Pipeline is set up as mentioned in #12

All Actions set up


is this just another vitest?

hello, not trying to be pedantic but just found this and it looks very much the same as vitest with a few minor differences. Is there any fundamental reason not to join efforts or discrepancies with vitest? (I see this project is a year older but vitest is way more popular)

I feel like this projec is kind of dead ? The last commit was in September of 2022, and it's a bit silent around the issues/PRs.


[5.4] Allow providing a user-defined default namespace for all make commands

This PR references a discussion on Laravel Internals where it seems that lots of people would love to be able to change the default namespaces for some generator commands.
For example, it is common for many to store their models inside the namespace App\Models.
The default behaviour is kept when no config/generators.php file is specified (to keep BC). If such a file is created and if the proper value is inserted (based on the $type member of the command class), that will be used as the default namespace.
Below is a sample configuration file which will only specify a different default namespace for the models (but any type could be changed too).
<?php

return [

    /*
    |--------------------------------------------------------------------------
    | Default namespaces for the classes generated via the make command
    |--------------------------------------------------------------------------
    |
    | You can change the default namespaces for the classes generated by the
    | make commands. All namespaces here will be relative to the root
    | application namespace and should start with a '\'.
    */

    'namespaces' => [
        'model' => '\Models',
    ],

];

Why the requirement to have a leading slash, though? Some might forget that. Couldn't you just make provision for it in the code?

I like this change because it gives developers a choice - sticking with the defaults or configuring the generators to suit their needs.
Putting all models into a separate directory has been a matter of public discussion multiple times and it's quite controversial. This perfectly solves it in my opinion.

Why isn't php artisan make:model Models/Book enough?

Why would you want to type My/Models/Namespace/Book for each command if you could define a default folder for all models?
Why do you want to have to move the file to the Models folder if you forget to type the namespace? Or delete the file and retype the command?
This is just convenient. Personally, the less I type, the better I get.
On top of that, within a team, you make sure that all generated things go to the proper folder by default.

I'm not sure that such things are really needed because you are calling generators 5-10 times on project start and that's all. And what if you will be needed to make it in another namespace. I've got a case where some models are located in App\Domain namespace and some of them in App\Models.

In that case, you can still specify make:model \App\Domain\Whatever
On Sun, Mar 5, 2017, 19:14 Pe Ell<EMAIL_ADDRESS>wrote:

I'm not sure that such things are really needed because you are calling
generators 5-10 times on project start and that's all. And what if you will
be needed to make it in another namespace. I've got a case where some
models are located in App\Domain namespace and some of them in App\Models.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/pull/18203#issuecomment-284248403,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANsh8TwLt_u7Li5-1wp4RbAE0VdB2GeNks5rivtugaJpZM4MTCla
.

I would love to have a configuration value for default namespaces.  While I follow most of the normal conventions, I do put my models in App\Models.

I think this is one of these subjects where we are unable to agree on one version.
There's no harm in adding this functionality, it doesn't affect performance, it doesn't break BC, it doesn't make it any more difficult to use the commands. Why not make the life of people who find it useful a little bit easier?

I agree that it would be nice to have default namespace, but if we add a "default namespace" for Models, then we should do this right and consider commands, migrations, middleware, providers, and all the other generators. They might be in different folders as well, like App\Common\Bus\Jobs or App\Domain\Events.

That's the point of creating the generators.php file. The idea is that you
can specify the default namespace for each "make" command.
On Sun, Mar 5, 2017, 21:39 Till Krüss<EMAIL_ADDRESS>wrote:

I agree that it would be nice to have default namespace, but if we add a
"default namespace" for Models, then we should do this right and consider
commands, migrations, middleware, providers, and all the other generators.
They might be in different folders as well, like App\Common\Bus\Jobs or
App\Domain\Events.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/pull/18203#issuecomment-284260643,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANsh8WmWgv32-qBNyDocJhtnEwJ6rQFmks5rix2LgaJpZM4MTCla
.

My PR already handles all make commands, not just the make:model command.

@a-komarev

I'm not sure that such things are really needed because you are calling generators 5-10 times on project start and that's all.

That's not true at all.

I don't come down on any side of the argument here. Just want to comment on this:

There's no harm in adding this functionality, it doesn't affect performance, it doesn't break BC, it doesn't make it any more difficult to use the commands. Why not make the life of people who find it useful a little bit easier?

There is no such thing as "there's no harm in this, so why not?". Every feature adds complexity to the framework and has to be maintained. Every feature has to be carefully weighed, to see if its utility is worth the added cost. Every feature has a cost.
Again, I'm not saying whether we should or shouldn't do it. Merely that it not breaking anything is never reason enough.

I'd rather just see fully overridable stub files than this.
That would allow you to specify /resources/stubs/model.stub or /resources/stubs/controller.stub where you could modify the namespace, default traits, class extensions etc.
Much more powerful IMO and tackles the "whole" problem with generating classes for a big app vs just changing the namespace.

@tomschlick if we customized the stub files, how would Artisan know which directory to place the files in?  for example, say we changed our model.stub to use the App\Models namespace.  Wouldn't Artisan still put this file into the App folder, and then we'd have to move it after?

@browner12 we could always add in some logic to pull the base from the stub namespace App\Models; and prefix it on the class command so it would still put it where it needs to go.

Configurable stubs could be another enhancement made easy by having a config file for generators...
In my opinion, these are 2 unrelated features, they do not address the same needs.
Maybe you should open another proposal on laravel/internals and let this PR focus on what it does.

No plans to add this approach right now.

@taylorotwell could you explain why? Or is there some document that provides a guide on what kind of features to not waste time on?

It may be best to mention new features first on slack, in the laravel internals channel. Then you could get some feedback before spending time on it. Taylor checks in there quite often as well as several regular contributors,


[BUG] No way to gather verbose output with STRTCPSVR

Describe the bug
As indirectly reported in #35 , there's no way to get verbose output when using STRTCPSVR
To Reproduce
Just run STRTCPSVR (using the experimental support in the strtcpsvr branch)
Expected behavior
An environment variable or STRTCPSVR option to turn on verbose mode
Verbose output
I can't. :-(

Resolved by #33


fix crash on react native version 0.65

Please only fix the issue without reformatting everything else. The only 2 changes are lines 276 & 281

if @hunglannn doesn't have time I could open a PR with these changes

if @hunglannn doesn't have time I could open a PR with these changes

Same here, if there is no response before tonight i will fork and submit a PR


Blobs migration update

Finishes up the migration guide from Microsoft.Azure.Storage.Blobs to Azure.Storage.Blobs. This PR adds a couple extra sections, fixes some inconsistencies, and makes samples better mirror each other.
This PR contains many tiny changes which touch nearly the entire document. They are categorized into their own commits. It may be easier to review individual commits, each with their own commit descriptions.

Btw. for V12 samples - could we move them to https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/storage/Azure.Storage.Blobs/samples ? and include using #region thing? so that compiler makes sure they don't get out of date.


Bypassing reCaptcha while using themekit & browser-sync for pages in /account area

Hi.
Trying to use browser-sync and themekit for my dev flow — and it works great! (thank you!)
However, I'm hitting an issue when trying to use this setup to develop “account” pages.
Here my browser-sync command
browser-sync start --proxy "https://store.myshopify.com/" --files "*/*.*" --reload-delay 1000 --config bs-config.js
Everything works fine  when I just develop public facing pages, i.e., https://localhost:3002/pages/company
Then I go to https://localhost:3002/account and try to log in.
I log in fine and get presented with Google's reCAPTCHA, and I solve it. But can't get past it  no matter what I try.
When I hit submit, I'm just redirected back to https://localhost:3002/challenge

Any suggestions?
(For now, I'm just manually refreshing pages I'm working on https://store.myshopify.com/account)

Nevermind.
RTFM to myself. 🤦
Just found this https://help.shopify.com/en/manual/online-store/setting-up/preferences#enable-or-disable-recaptcha-on-online-store
Looks I can temporarily disable reCaptcha, then enable it back once I'm done coding.


os-packages.json: Add FreeBSD section

@richlander

Thanks!

The intent is for the distros/OSes to be in alpha order. Can you place FreeBSD just after Fedora?


Train Hub not advertised/connected

I'm not able to get a propper connection to my LEGO train hub. As I run the simple example code
from curio import sleep
from bricknil import attach, start
from bricknil.hub import PoweredUpHub
from bricknil.sensor import TrainMotor
from bricknil.process import Process
import logging

@attach(TrainMotor, name='motor')
class Train(PoweredUpHub):

    async def run(self):
        self.message_info("Running")
        for i in range(2):
            self.message_info('Increasing speed')
            await self.motor.ramp_speed(80,5000)
            await sleep(6)
            self.message_info('Coming to a stop')
            await self.motor.ramp_speed(0,1000)
            await sleep(2)

async def system():
    train = Train('My train')

if __name__ == '__main__':
    logging.basicConfig(level=logging.INFO)
    start(system)

the LEGO device seems to be found and connected (continous LED) but the code stucks and nothing happens. Still if I stop with Strg+C the LEd is on. Please see the log:
started thread for curio
inside curio run loop
INFO:BLE Event Q.0:using bleak
INFO:BLE Event Q.0:Starting scan for UART 00001623-1212-efde-1623-785feabcd123
INFO:BLE Event Q.0:Looking for first matching hub
Awaiting on bleak discover
Done Awaiting on bleak discover
INFO:BLE Event Q.0:Rescanning for 00001623-1212-efde-1623-785feabcd123 (60 tries left)
Awaiting on bleak discover
Done Awaiting on bleak discover
INFO:BLE Event Q.0:checking manufacturer ID for device named HUB NO.4 for HUB NO.4
INFO:BLE Event Q.0:found device HUB NO.4

This is the "freeze" state. I'm working on a Raspi 4B with the latest 2020-08-20-raspios-buster-armhf-full.img. At the beginnig I had some warnings with the service_identity module but I fixed and installed the missing modue,
:0: UserWarning: You do not have a working installation of the service_identity module: 'No module named 'service_identity''.  Please install it from <https://pypi.python.org/pypi/service_identity> and make sure all of its dependencies are satisfied.  Without the service_identity module, Twisted can perform only rudimentary TLS client hostname verification.  Many valid certificate/hostname mappings may be rejected.
Does anybody knows a solution or could share her/his experiences with controling the PoweredUp trains with the Raspberry 4?
Best regards
Matthias

Got it: bleak libary was not installed. no its working.

Got it: bleak libary was not installed. no its working.


MSID: 102637 Version: 1 DOI: 10.1101/2<IP_ADDRESS>6991

MSID: 102637
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>6991
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=102637
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6991

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/102637v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>6991
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/102637/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Adding preprint date to spreadsheet.
Have sent WOS query.

chased again


Update typescript example to pass contract tests.

This PR updates the typescript plugin example to use the latest version and ensure it passes all the contract tests (it can even by successfully submitted).

@chuchocd85 , I created another PR #10 because I missed this one. So feel free to either make your remarks there or I can review this one.


clamp.mlir test fails on vulkan

Even we get non-i32 bitwidth emulation support, the clamp.mlir still fails. This is the issue for tracking it.
Expected equality of these values. Contents does not match.
lhs:
4xi8=10 4 8 10
rhs:
4xi8=0 4 8 10
Expected equality of these values. Contents does not match.
lhs:
4xi16=10 4 8 10
rhs:
4xi16=0 4 8 10

My guess is that we don't handle negative number well. clamp(0, -2, 10) -> 10, the negative number might be a huge positive number. I will take a look at it.

Interesting! Part is correct; part is wrong. This is a similar pattern I've seen for multiple dispatches. Are you testing this on SwiftShader? Could you do a quick check against real GPU?

I was testing with bazel test, so it's on SwiftShader by default I think. I haven't set up the environment for vulkan yet, so I can't do it in a quick way...

Oh never mind then. I've that set up; let me check later. :)

Tested: this also fails on real GPU. So there is a bug in our pipeline.


filter by cause

created function in events.ts to filter by cause. imported said function into events/index.tsx for use when filter queries do exist. created common method in utils.util.ts to parse a query and return a corresponding array

I was unable to test my function to filter by cause (in fact, I'm unsure if double find() is valid syntactically in the first place), as I kept on getting this error message regardless of how I passed ["OTHER"] (I tried [OTHER], "OTHER", ["OTHER"], "[OTHER]")

@Fattimo Can you try testing using 'getByCausesEventsCardData(["OTHER"])'; that doesn't throw an error for me:
$ yarn run:server:action events 'getByCausesEventsCardData(["OTHER"])'
yarn run v1.22.0
$ yarn run:ts-node:script server/run-action.ts events 'getByCausesEventsCardData(["OTHER"])'
$ cross-env NODE_ENV=development ts-node -O {\"module\":\"commonjs\"} -r module-alias/register server/run-action.ts events 'getByCausesEventsCardData(["OTHER"])'
[]
✨  Done in 12.18s.

Kinda something weird I noticed while testing my query; I'm not sure if this is a consistency thing/something that needs a migration script for the local db, or just my relative unfamiliarity with the structure of the Nonprofit schema. Essentially, the query only works when finding the field causes as opposed to just cause. While logging some of the outputs from Nonprofit, however, as seen below

there exists both a cause and a causes field. Is there a reason why the query only works when searching for the field causes and not cause?

@Fattimo Oh okay there is an error in our migration script, it uses "causes": https://github.com/GTBitsOfGood/gen-soln/blob/develop/server/migrations/20200924180429-migrate-nonprofit-add-causes.ts#L7 whereas Mongoose schema uses "cause".
We can either update our Mongoose schema to use "causes" (it's okay if it still remains type String) OR write a new migration script that renames the "causes" field to "cause" for all Nonprofit documents.


Exception in a reporter silently crashes Mocha

If a reporter raises an exception, Mocha just stops without any error message. An example reporter demonstrating the problem (needs a failing test in the suite):
module.exports = Crash;

var Spec = require('mocha/lib/reporters/spec');

function Crash(runner) {
    Spec.call(this, runner);

    runner.on('fail', function(test, err){
        throw new Error();
    });
}

Crash.prototype.__proto__ = Spec.prototype;

this would be helpful.

+2
I'm not even sure how to get the stack trace


Raise issue if trying to upload .git/ folder + ignore .git/ folder in upload_folder

Fix https://github.com/huggingface/huggingface_hub/issues/1401 and https://github.com/huggingface/huggingface_hub/issues/1402 (thanks @lewtun for reporting it).
This PR:

adds a check in CommitOperationAdd and CommitOperationDelete => it is not possible to upload/delete a file in the .git/ folder. Server would not accept it anyway but let's raise an issue early.
ignores the .git/ folder when uploading with upload_folder
adds tests and docs

One thing that would be cool before having a huggingface-cli push command would be to comply with the .gitignore specification. However this is not straightforward (cannot be converted to a list of allow_patterns/ignore_patterns that easily). I have found a gitignore_parser that does the trick but it would add another dependency for which I don't know much about the quality of it. Let's keep this discussion for later.

Note: I also refactored a few tests in hf_api to be more consistent with creating/deleting repos. Not exhaustive but it's still cleaner.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks for the reviews @lewtun @LysandreJik! It'll avoid mistakes in the future :)


⚠ Bofrost.be has degraded performance

In f5e5e08, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1012 ms

Resolved: Bofrost.be performance has improved in 374b4d2 after 6 minutes.


[BUG] UHF reference for MSD

Describe the bug
Code should fail when using MSD trial with UHF reference.

@AdrianLopez1 if you can please add this check and a unit test to catch it. Thanks!


FindVisItPython.cmake should attempt python3 as executable name first.

Describe the bug
python 3 executable is most frequently called 'python3', but we search for 'python' first, see code section below.
    if(UNIX)
        set(PYTHON_EXECUTABLE ${PYTHON_DIR}/bin/python)
        # if this doesn't exist, we may be using python3, which
        # in many variants only creates "python3" exe, not "python"
        if(NOT EXISTS "${PYTHON_EXECUTABLE}")
            set(PYTHON_EXECUTABLE ${PYTHON_DIR}/bin/python3)
        endif()

This can cause problems with system python where python 2 and 3 are installed.
We should test for 'python3' first.
As reported in  discussion 5882.

I think you can  also set PYTHON_EXECUTABLE directly to avoid all of these issues?

https://github.com/visit-dav/visit/blob/7095a895f571b6a67f1e53a52a242cabd20f9ca4/src/CMake/FindVisItPython.cmake#L87

Yes, that would help. Does our host-profile creation do that for system python? Or even visit-built python?

bv_python.sh does NOT set PYTHON_EXECUTABLE when creating the host-profile/config-site file. Should it?

@cyrus, do you already have this fixed somewhere?
If not, would making python3 preferred for setting PYTHON_EXECUTABLE from PYTHON_DIR be sufficient for this release:
# Find the interpreter first
if(PYTHON_DIR AND NOT PYTHON_EXECUTABLE)
    if(UNIX)
        set(PYTHON_EXECUTABLE ${PYTHON_DIR}/bin/python3)
        # if this doesn't exist, test for 'python' which may be
        # version 3 or version 2.
        if(NOT EXISTS "${PYTHON_EXECUTABLE}")
            set(PYTHON_EXECUTABLE ${PYTHON_DIR}/bin/python)
        endif()
    elseif(WIN32)
        set(PYTHON_EXECUTABLE ${PYTHON_DIR}/python.exe)
    endif()
endif()

Ideally, we should switch to CMake's FindPython3 module when we update the minimum cmake version. Or even just the FindPython module , as it will find either version 3 or 2, preferring 3.

https://github.com/visit-dav/visit/pull/13258

using new Find modules is  a good path. But we would need to understand if that requires any CMake version constraints (newer than zzz) and make sure we understand any implications of our custom cmake that helps with our python process.

So, #13258 doesn't address the section of code where PYTHON_EXECUTABLE may be set to 'python' (eg python2) instead of 'python3'.  I quoted a modified version that should fix this.  Do you want to add that change to your PR.
I agree that moving to the new Find modules should be for a future release. We already discussed making 3.14 the minimum required CMake version, and that would enable the newer python find modules.

resolved in #13258 (RC) and #17232 (develop)


Added the better type for the link handler

Overview

Added more specific typing to the AppHeader.props.onLinkAction item param

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.4.52-alpha.5a4318.0@codecademy/gamut-labs@19.12.2-alpha.5a4318.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60d6430543e895202f96aa62--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.4.53-alpha.ea9ed7.0@codecademy/gamut-labs@19.12.3-alpha.ea9ed7.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60d9e2bd2f18ae0abb4b3577--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.4.53-alpha.68d75e.0@codecademy/gamut-labs@19.12.3-alpha.68d75e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60d9e53a09557bed5deca934--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.4.54-alpha.1b95e3.0@codecademy/gamut-labs@19.12.4-alpha.1b95e3.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60d9e7b4d8eda91ef5b4d2d4--gamut-preview.netlify.appDeploy Logs