No module named 'codechecker_api_shared' when CodeChecker Server

Hi There
When I applied the command: CodeChecker server -w ./ws -v 8555, I met the following error message:
Couldn't import module for subcommand 'server'... ignoring.
Traceback (most recent call last):
File "/Users/pengli/CodeChecker-byd/build/CodeChecker/cc_bin/CodeChecker.py", line 110, in main
subcommands[subcommand])
File "/Users/pengli/CodeChecker-byd/build/CodeChecker/cc_bin/CodeChecker.py", line 39, in add_subcommand
command_module = cmd_spec.loader.load_module(module_name)
File "", line 407, in _check_name_wrapper
File "", line 907, in load_module
File "", line 732, in load_module
File "", line 265, in _load_module_shim
File "", line 696, in _load
File "", line 677, in _load_unlocked
File "", line 728, in exec_module
File "", line 219, in _call_with_frames_removed
File "/Users/pengli/CodeChecker-byd/build/CodeChecker/cc_bin/../lib/python3/codechecker_server/cmd/server.py", line 27, in
from codechecker_api_shared.ttypes import DBStatus
ModuleNotFoundError: No module named 'codechecker_api_shared'
usage: CodeChecker [-h] {} ...
CodeChecker: error: invalid choice: 'server' (choose from )
Any idea about it?
Thanks
Peng

I searched the entire code base, and did not find "codechecker_api_shared" module, can you please advise if it is missed?
Thanks
Peng

@lipeng28 There was a huge code refactoring in the v6.12.0 release where we moved the generated thrift API stubs to python/nodejs modules. See: #2570. So codechecker_api_shared module can be found on the pypi (https://pypi.org/project/codechecker-api-shared/) and it is listed in the python requirements files (https://github.com/Ericsson/codechecker/search?l=Text&q=codechecker_api_shared) and it will be installed to your virtual environment on make venv or make venv_dev targets. I think your problem is that you are using an old virtual environment where this module is not installed. Could you please try to run the following commands:
# Clear your directory.
make clean

# Create a Python virtualenv and set it as your environment.
make venv  # This command will install the codechecker_api_shared module too.
source $PWD/venv/bin/activate

# Build and install a CodeChecker package.
make package

@csordasmarton I tried your instructions, however, it does not work. I still met the following error:
ModuleNotFoundError: No module named 'codechecker_api_shared'
Thanks
Peng

@lipeng28 Could you please provide us a few more information:

Your OS type and version.
Please run the following commands inside your codechecker directory and give us the outputs:

git rev-parse HEAD
rm -rf venv && make venv
pip list | grep codechecker

@csordasmarton
OS type and version: MacOS version 10.15.5
git rev-parse HEAD: e59c1b47dd821aed702bf8701d07264fe25b9985
rm -rf venv && make venv
pip list | grep codechecker
codechecker-api        6.27.0
codechecker-api-shared 6.27.0
Looks like codechecker-api-shared is installed, however, CodeChecker server does not work fine.
Best,
Peng

Hi,
Is is possible that the virtualenv is not sourced and the codechecker-api modules are installed on the system level too (v6.27 is an older version)?
rm -rf venv && make venv
source venv/bin/activate
pip list | grep codechecker

I did not found this commit on the master branch:
git show e59c1b47dd821aed702bf8701d07264fe25b9985
fatal: bad object e59c1b47dd821aed702bf8701d07264fe25b9985

Best,
Gyorgy

@csordasmarton
it works now.
As for the git head, It is because I had my own repo to support my own extension.
Thanks
Peng


Null apikey error happens too fast

It looks like if an API key is not provided, the chat appears in a "sent" state with no error, even though I can see that an error occurs internally. This is likely a React issue where the error state updates too quickly.
To reproduce, select Anthropic and don't provide an api key.

Fixed by registering listeners immediately in #172


vscode debug values should take into account source map names

The ES5 transpilers such as Typescript and babel translate nested this into _this1, this2 e.t.c
the source map generated by transpiler contains what names should be used as the values but this isn't taken into account via inline values or hover in the editor. To get the actual values, a user has to do guess via console on what the generated property is actually called.
I believe the experience can be improved. Happy to send a PR if you guide me on how to get source map names from the debug adapter.
@roblourens @isidorn

We have this issue: https://github.com/Microsoft/vscode/issues/12066 and I think it would be implemented entirely in the debug adapter. A PR would be awesome but I'm not sure exactly what would go into it.

Okay closing this issue then.


Enable secrets cache

Hello,
I would like to raise the idea of having a cache for polling the secrets.
My use case is having the same secret on multiple namespaces.
With the current implementation, the external secrets controller fetch AWS many times, one for each namespace, although they all use the same secrets manager entry.
Adding such a feature will reduce the API calls which will reduce cost.

I think it's a good idea to have some kind of cache. Also if you do something like this:
apiVersion: kubernetes-client.io/v1
kind: ExternalSecret
metadata:
  name: hello-service
spec:
  backendType: secretsManager
  data:
    - key: hello-service/credentials
      name: username
      property: username
    - key: hello-service/credentials
      name: password
      property: password

It will fetch the hello-service/credentials secret twice which seems a bit unnecessary :)
It has to be properly implemented so it doesn't allow bypassing privileges.
It raises some questions of when to poll for changes? If we have two different secrets looking at the same secret, but they are created with 6 hours in between and the polling is every 12 hours. Will one have potentially 6 hour old data as it's a cache hit or how to deal with that.
It would require some refactoring though

I've created some draft PR as my prposal to provide this feature.
Let me know what do you think.
Tests should still be added.

@Flydiverny  ?


Graphical glitch with Skybox

Not a big issue, however if the skybox changes, the bottom right part of the minimap will change to the same color as the skybox. As an example, here's a picture of the bottom of the minimap while in my PoH.

Now whenever I leave my PoH the bottom right section looks blue instead of black.

This happens regardless of if the Skybox plugin is enabled, the only difference is the color. If the Skybox plugin is disabled it will turn a beige color.

@iawhd the texture should be expanded a bit to account for that transparent space. I believe I have fixed this on the vanilla sample pack but I am not too sure.

Yeah I believe the vanilla sample pack worked just fine, its only the dark theme pack that this is happening on. Again, not a big deal, but it does look a little annoying.

@melkypie I've resolved this in the new 1.5 version


Improve email notifications

Email notifications should be optional.
There should be option to send only failed jobs notifications (skip passed jobs).

Fixed. See updated Apidoc: http://apidoc.me/gilt/svc-sundial/latest#enum-notification_options


docs(samples): Add Dataflow TextIO snippet

Adds a snippet for a Dataflow doc in progress
Also adds a snippets dataflow/snippets subdirectory with POM file

SGTM - please visit the internal site github/ and register.  Join Googlers, GoogleCloudPlatform, and GoogleAPIs


change perms on this tasks or cluster will break

https://github.com/redhat-cop/automate-tower-ha-dr/blob/0e9efb33db60dd47482bb9eb7db08e25519eef91/tower-dr-standup.yml#L40 to 750

It was mentioned this causes tower to crash. That 750 are the correct permissions. Need to test.

Closing based on functionality added in https://github.com/redhat-cop/automate-tower-ha-dr/pull/17


Unable to run migrations in github workflow when automating test runner

Description
I am trying to automate the test runner in my github workflow and it will exit with an error on ace commands. Running the test runner locally works as expected.
Not sure if I should have posted this here or in the adonis repo itself.
Even if there is a decent example of setting up running tests with the db for a github workflow, that would be greatly appreciated.
Package version
^4.0.0
Error Message & Stack Trace
Error: Command failed with exit code 1: node ace migration:run
    at makeError (/home/runner/work/portal-api/portal-api/node_modules/execa/lib/error.js:60:11)
    at handlePromise (/home/runner/work/portal-api/portal-api/node_modules/execa/index.js:118:26)
    at processTicksAndRejections (internal/process/task_queues.js:[95](https://github.com/DirectLink-AI/portal-api/runs/5773496600?check_suite_focus=true#step:7:95):5)
    at runMigrations (/home/runner/work/portal-api/portal-api/japaFile.ts:32:3)
    at run (/home/runner/work/portal-api/portal-api/node_modules/japa/build/src/SlimRunner/index.js:107:9) {
  shortMessage: 'Command failed with exit code 1: node ace migration:run',
  command: 'node ace migration:run',
  escapedCommand: 'node ace "migration:run"',
  exitCode: 1,
  signal: undefined,
  signalDescription: undefined,
  stdout: undefined,
  stderr: undefined,
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}

Relevant Information
Just to note, I have a .env.testing file for any necessary environment variables for my application
I have in my workflow already set up postgres, node, and installed the dependecies:
name: validate-pull-requests
on:
  pull_request:
    branches:
      - dev
      - staging
      - main
jobs:
  lint-test-build:
    runs-on: ubuntu-latest
    services:
      postgres:
        image: postgres
        env:
          POSTGRES_USER: root
          POSTGRES_PASSWORD: root
          POSTGRES_DB: test
        ports:
          - 5432:5432
        options: >-
          --health-cmd pg_isready
          --health-interval 10s
          --health-timeout 5s
          --health-retries 5
    steps:
      - name: Checkout code
        uses: actions/checkout@v2

      - name: Setup Node
        uses: actions/setup-node@v2
        with:
          node-version: '14.19.0'

      - name: Install Dependecies
        run: npm install

      - name: Run Linter
        run: npm run lint

      - name: Run Tests
        run: npm run test

      - name: Run the build
        run: npm run build

and here is my japaFile in order to set up the test dbs and run the migrations:
import 'reflect-metadata'
import pgtools from 'pgtools'
import execa from 'execa'
import { join } from 'path'
import getPort from 'get-port'
import { configure } from 'japa'
import sourceMapSupport from 'source-map-support'

process.env.NODE_ENV = 'testing'
process.env.ADONIS_ACE_CWD = join(__dirname)
sourceMapSupport.install({ handleUncaughtExceptions: false })

const dbConfig = {
  user: 'root',
  password: 'root',
  port: 5432,
  host: 'localhost',
}

const handleDBCallback = (error, result) => {
  if (error) console.log(error.message)
  else console.log(result)
}

async function createTestDbs() {
  pgtools.createdb(dbConfig, 'test', handleDBCallback)
  pgtools.createdb(dbConfig, 'dl-test-1', handleDBCallback)
  pgtools.createdb(dbConfig, 'dl-test-2', handleDBCallback)
}

async function runMigrations() {
  await execa('node', ['ace', 'migration:run'], { stdio: 'inherit' })

  await execa('node', ['ace', 'migration:run', '-c', 'dl-test-1'], { stdio: 'inherit' })

  await execa('node', ['ace', 'migration:run', '-c', 'dl-test-2'], { stdio: 'inherit' })
}

async function rollbackMigrations() {
  await execa('node', ['ace', 'migration:rollback', '--batch=0'], { stdio: 'inherit' })

  await execa('node', ['ace', 'migration:rollback', '-c', 'dl-test-1', '--batch=0'], {
    stdio: 'inherit',
  })

  await execa('node', ['ace', 'migration:rollback', '-c', 'dl-test-2', '--batch=0'], {
    stdio: 'inherit',
  })
}

async function startHttpServer() {
  const { Ignitor } = await import('@adonisjs/core/build/src/Ignitor')
  process.env.PORT = String(await getPort())
  await new Ignitor(__dirname).httpServer().start()
}

function getTestFiles() {
  let userDefined = process.argv.slice(2)[0]
  if (!userDefined) {
    return 'test/**/*.spec.ts'
  }

  return `${userDefined.replace(/\.ts$|\.js$/, '')}.ts`
}

/**
 * Configure test runner
 */
configure({
  files: getTestFiles(),
  before: [createTestDbs, runMigrations, startHttpServer],
  after: [rollbackMigrations],
})

I am really sorry, but I realized I was missing a configuration variable in my .env.testing file when I finally added it back in, I got it working.
Thanks!


cut out animation frames to make faster attacks

reopened because it's worth looking into

god this was such a good idea


WIP: Add fuzzer for HCLK_IOI3 PIPs

This PR solves #891
It's WIP as it is still missing some PIPs that drive the BUFIO and BUFR from ILOGIC clock or through the IOI tile.
Currently it solves 149 PIPs out of 196, and the solved ones include all PIPs that are used by Litex on Arty.
The current approach generates the netlist in a partly random fashion and uses Vivado only for P&R and bitstream generation. The fuzzer could be optimized by adding a todo list based rerouting script to Vivado.

Red CI due to timeout. Without targeted rerouting we want be able to get all of the PIPs solved as they appear to rarely in the specimen


Reduce barrier to entry

This looks like a great framework!  There is always a but right ;)
I have a few existing nodejs applications that server out routes via express.  I would love to serve my api docs as swagger and recently started looking at my options.  This lib is awesome if you are starting a new application with node/express and build this in.  However if you have an existing application it requires extensive rework.
One idea would be to use the middleware framework, imho something like this would make it much easier to get started, and also easier to remove, why would I ever remove right ;)
Existing route, which would need to be completely reworked
  // get all uses
  app.get(
    function(req, res) {
      var users = User.getUsers(err, users)
        res.json(users);
      });
  });

New route with a simple middleware function to add swagger, everything else remains intact
  // get all uses
  app.get(
    '/api/users',
    swagger({
    "description" : "Operations about pets",
    "path" : "/users",
    "notes" : "Returns all users",
    "summary" : "Find all users",
    "method": "GET",
    "parameters" : [],
    "type" : "User",
    "errorResponses" : [],
    "nickname" : "getUsers"
    }),
    function(req, res) {
      var users = User.getUsers(err, users)
        res.json(users);
      });
  });

Maybe this is outside the scope of this project as the design has already been done.  Could also be that is was considered and thrown out due to reasons I don't understand. Maybe something like this is not possible.

+1 Could probably even take it further and do away with redundancies and have swagger figure out from express the HTTP method and path.

Take a look here for the proposed direction for version 3.0:
https://github.com/swagger-api/swagger-node-express/issues/180#issuecomment-83224535

This is handled in #209


Apple pay with stripe in ionic 3

const success = await MyPaymentProvider.authorize(response.paymentData);
Please give more information about above line as i can't see this function in the code

This is just an example. Your payment provider would be Stripe, so use whatever SDK method they have that takes payment data to authorize payments. You will have to find this out from Stripe’s own documentation.

Can you please 'makePaymentRequest' in this function return with stripe token as well so

@darshansarvaiyasa i am also facing the same problem did you find any solution?

@darshansarvaiyasa i am also facing the same problem did you find any solution?

Yes, in this function 'makePaymentRequest' return base64 token string, if plugin also return stripe token then problem solve.

@darshansarvaiyasa i know that but i don't know whether they are planning to do so or not so in meantime did you find anything to convert the paymentData response to stripe acceptable token?

You can share me your code I will send you the code with updated corrections.


De-duplicate model URLs in CAM-KP Frontend

CAM-KP Frontend currently duplicates model URLs (because of how they are stored in the backend). There should be some way to deduplicate them, but that will require some thinking.

I've implemented some basic de-duplication logic in PR #138, but I haven't yet found an example that finds a duplication within the top hundred nodes, so I haven't been able to test this yet.


Redirection Issues?

Hi there, i'm don't know if i'm wrong or if this behavior is just weird to me.
There is a settings to be complete with the plugin (Account component), and this settings is the "Redirect to",  but the plugin is ignoring the "redirection" settings, instead the plugin is always redirecting  to a previous page.
This is the normal behavior?
There is a way to "fix" this?
Thank you

This is normal behavior only when the user has hit a page that requires authentication. Otherwise the normal redirect is used.


About routing_response and reference_line

I have some confused about the relationship between routing_response and reference_line,
What I understand is that routing_response contains the information is like Segment,just contains such "start_s" and "end_s".
And we get  reference_line_info from routing_response, reference_line_info contains some lane_way_point information, such as "SL" information. My question is how can we get points information just according to routing_response.
From hdmap or what?
Apollo生成道路的主要思想是通过在道路上撒点，点之间进行平滑生成路线。那么撒点这个步骤在哪里完成（可以具体到代码实现）？
Thank you very much for taking my question seriously.

routing_response
routing response decide which road to go, and the data store in
https://github.com/ApolloAuto/apollo/blob/4f7fbcf8e472da72b8c196208bf10dd93edf09f0/modules/routing/proto/routing.proto#L49-L64
reference_line
Then use the RoadSegments and hdmap to generate reference line. More details will be added later.
https://github.com/ApolloAuto/apollo/blob/3711be4317b62fffa548f91cb383269d4cc25987/modules/planning/reference_line/spiral_reference_line_smoother.cc#L169-L181

Thank you for your detailed explanation. it helps me a lot.


feat: Allow extra data to be saved to secret

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from tumido by writing /assign @tumido in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
/lifecycle rotten


buildGoModule: warn if -trimpath or -mod= is used, remove explicit usage

Description of changes
Follow up to https://github.com/NixOS/nixpkgs/pull/276685
Adding warnings if -trimpath or -mod= is explicitly set, as these flags are controlled by the builder.
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.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

@ofborg eval

Rebased to resolve merge conflict. Also removed newly added -trimpath from gcs and jq-lsp.

@kalbasit @SuperSandro2000 @jnsgruk Could someone merge this?

This can go to master instead of staging.


Update to What's new for Windows Desktop client - May release

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

@sethmanheim : This PR seems to contain useful information. Will we see that information added via an internal (windowsserverdocs-pr) commit?


Revert "(PUP-5001) Remove Ubuntu 14.04 acceptance node defs"

Reverts puppetlabs/puppet#4293

/cc @johnduarte


Mac M2 model.export_coreml('.mlmodel') Unable to export model

This is my code:
import turicreate as tc
image_folder = 'image1/训练集'
data = tc.image_analysis.load_images(image_folder, with_path=True)
data['label'] = data['path'].apply (lambda path: 'D_4015' if 'D_4015' in path   else 'DB_013')
data.save('diller.sframe')
train_data, test_data = data.random_split(0.8, seed=2)
model = tc.image_classifier.create(train_data, target='label')
predictions = model.predict(test_data)
metrics = model.evaluate(test_data)
model.export_coreml('diller.mlmodel')
But an error occurred while exporting the model
Error code:
WARNING:root:scikit-learn version 1.1.2 is not supported. Minimum required version: 0.17. Maximum required version: 0.19.2. Disabling scikit-learn conversion API.
zsh: illegal hardware instruction  /opt/anaconda3/envs/diller/bin/python
How can I solve this problem? Apple computers do not support Apple modules, which surprised me.

M2 cannot return scikit born version 0.19.2; I tried many methods in the evening and asked a great god to help me solve this problem. I'm new


hypernetworks don't working

I try to apply the hypernetworks from the novelai in the AUTOMATIC1111 settings, but they don't change anything.
In this colab hypernetworks work fine.
https://github.com/daswer123/stable-diffusion-colab

No, everything works as if they were not turned on.

I did this, still doesn't work. And as I said above, the colab from daswer123 link works great. But it is worse than yours in that it takes a long time to set up.

And perhaps a clarification. I throw the weights from the novelai on the google drive where the SD is installed. So for novelai, you need main weights, vae weights, yaml config and the weights of the hypernetworks themselves.
I do not use Model Download / Load, since it only allows one file to be loaded.

You will get better answers about hypernetwork opening a discussion in the AUT1111 repo https://github.com/AUTOMATIC1111/stable-diffusion-webui/

Same problem.  Hypernetworks don't work. No error messages.

Thanks so much for fixing that problem! You are my hero. @TheLastBen

happy to help


Drill down: empty grid when data source is an array of objects

I was writing the same issue exactly in the same time! I provided a patch in a pull request for it... ;)

:thumbsup:


plumbing/transport: allow AdvertisedReferences being called multiple times

AdvertisedReferences is now part of transport.Session.
It is allowed to be called more than once.
It is allowed to be called before and after FetchPack/SendPack.
Implementations cache its result.

Current coverage is 76.88% (diff: 0.00%)

Merging #165 into master will decrease coverage by 0.78%

@@             master       #165   diff @@
==========================================
  Files            88         88
  Lines          5489       5489
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           4263       4220    -43
- Misses          766        816    +50
+ Partials        460        453     -7

Powered by Codecov. Last update 592b538...41a2b4a


changing background {video attached}

Closes
Closes #1570

Description

Screenshots
https://github.com/OSCode-Community/OSCodeCommunitySite/assets/119532943/9017214c-2d4c-4b0c-8216-111f7f543964

Note to reviewers

Checklist:

[x] I have linked the PR to the correct issue.
[x] I have read the Contribution Guidelines
[x] I have built and tested the changes, and they do not break or show any errors.

@Divesh2003 PR ready to merge now 🚀


Make multi-chan timeseries workflow into public-facing resource

closed by https://github.com/holoviz-topics/examples/pull/418


Update version

Coverage decreased (-0.02%) to 90.695% when pulling 89ce41f336d0ba9bd4db7d0fbf3609eaad827fb8 on 1.3 into 1333dd8f0d671ed48306989f4218afa7a2ea5dcd on master.

Coverage decreased (-0.02%) to 90.695% when pulling 89ce41f336d0ba9bd4db7d0fbf3609eaad827fb8 on 1.3 into 1333dd8f0d671ed48306989f4218afa7a2ea5dcd on master.

Coverage decreased (-0.02%) to 90.695% when pulling 89ce41f336d0ba9bd4db7d0fbf3609eaad827fb8 on 1.3 into 1333dd8f0d671ed48306989f4218afa7a2ea5dcd on master.

Coverage decreased (-0.02%) to 90.695% when pulling 89ce41f336d0ba9bd4db7d0fbf3609eaad827fb8 on 1.3 into 1333dd8f0d671ed48306989f4218afa7a2ea5dcd on master.

Coverage decreased (-0.02%) to 90.695% when pulling 89ce41f336d0ba9bd4db7d0fbf3609eaad827fb8 on 1.3 into 1333dd8f0d671ed48306989f4218afa7a2ea5dcd on master.

Coverage decreased (-0.02%) to 90.695% when pulling 89ce41f336d0ba9bd4db7d0fbf3609eaad827fb8 on 1.3 into 1333dd8f0d671ed48306989f4218afa7a2ea5dcd on master.


ZK filter: waiting for full packet in buffer before decoding

Signed-off-by: Zhewei Hu<EMAIL_ADDRESS>Commit Message: ZK filter: waiting for full packet in buffer before decoding
Additional Description: Sometimes ZK filter did not fetch all the data of a certain request or response from buffer at once (since buffer may not contain all the data). Then, ZK filter tried to parse the metadata (len, xid, opcode, etc) from all later parts of the data fetched from the buffer the same way as the first part. Since all later parts of data does not include the metadata, only pure raw data, the decoder error will be recorded. After the discussion, we think it is easier to wait for the all the data for a certain request or response in buffer before decoding.
Risk Level: low
Testing: unit tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
Fixes #Issue: #25024

cc @rgs1 and @JuniorHsu, please review, thanks!

I think another way to implement is to take the advantage of StopIteration. If the data in the buffer is not enough, the filter just return StopIteration without drain the buffer and start from the first filter, until the data in the buffer is enough.
In this case, we do not need to introduce a new filter status and do not need the infinite loop.

cc @fishcakez, just updated the code, thanks!

@snowp gentle ping as you're the code owner who's able to merge :)

One last thing: did you manage to bake this with real ZK traffic?

One last thing: did you manage to bake this with real ZK traffic?

Yes, I tested this patch on two clusters.

One cluster's decoding errors got eliminated with this patch. And there is no latency spike.
Another cluster still had very few decoding errors with the patch (will look into it later). The latency raised a bit, not sure how the "stop and buffer" mechanism affected the latency. I think the latency measures the time ZK service processing the request, should not be directly related to the filter's behavior? Since the filter tries send the entire packet to ZK service.


Issues of Equivariant KPconv

Hello,
Thanks for your contribution.
I have an issue:
How did you convert the kpconv to an equivariant version, is there any changes you made to their structure besides the quaternion representation?
Best Regards,
Judge

Hi Judge,
we convert kpconv to the rotationally equivariant version by rotating the "kernel points" X times and extracting these rotated features into an extra dimensions. Please refer to the paper for the details.


Non-standard callback arguments

Why doesn't this library return error as the first argument and result as second argument, as is the standard in node?
I'm asking because I tried promisifying the library and got an error because promisification libraries expect that first argument is an error, as they should.

Not for any good reason unfortunately. This code was originally written to be plugged into an older application which expected return values of this type (coincidentally that application has also converted to promises and now uses gamedig through a legacy wrapper).
We may be able to support both promises and nodejs-standard callbacks by sniffing the argument parameter counts. I'll give it a shot tonight if I have a spare moment.

If that is not possible, a new major version with correct style could be released and nothing should break.

Fixed in 7b9fe2161cc54c7d7c36a50105aec5b8c6f12862 v0.2.26
Give it a shot!

Cheers, it works.
cat servers.txt | \
  R -r --js --import gamedig \
  'split(":")' \
  '([host, port]) => gamedig.query({ type: "rust", host: host, port: parseInt(port) })' \
  '(p) => p.catch(() => {})' | \
  R --slurp -o table --compact 'map(pick(["name", "map"]))'
┌───────────────────────────────────────────────────┬────────────────┐
│ name                                              │ map            │
├───────────────────────────────────────────────────┼────────────────┤
│ Armoton Suomi - Merciless Finland - armoton.fi    │ Procedural Map │
│ Pelipalvelin.com Rust server                      │ Procedural Map │
│ Hubu ServeBeer                                    │ Procedural Map │
│ [FI/EU] Rakeservers.net - Vanilla - Last wipe 4.3 │ Procedural Map │
│ SE/DK/NO/FI - Road to Valhalla                    │ Procedural Map │
│ Firestarters [FI/EU]                              │ Procedural Map │
│ FRT Rust Server                                   │ Procedural Map │
│ FIN Rust public server                            │ Procedural Map │
│ Entajuu.com Rust01                                │ Procedural Map │
└───────────────────────────────────────────────────┴────────────────┘


[camera_windows] Use CameraAccessDenied error code for camera permission errors

Background
The camera plugin reports issues to the Flutter app by throwing CameraExceptions. As per this proposal, we would like to standardize the CameraException's code value so that common errors can be handled consistently across platforms.
This change makes the Windows camera plugin set the error code to CameraAccessDenied when camera permissions were denied. This is a breaking change as previously the error code would've been camera_error or capture_error.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@yaakovschectman Good question! You can pop open the Details link next to each failure to learn more about what happened:

publishable - Looks like my change is using an invalid version number. I'll need to address this before merging this pull request. That's my mistake.
submit-queue - We want the main branch to build and pass tests at all times. Whenever this isn't the case, we should stop merging new things to main until the branch is fixed. Common failures on main branch include flaky tests, or, test failures from post-submit tests (these are tests that aren't run on pull requests until after they have been merged). If you haven't already, I recommend reading through this: https://github.com/flutter/flutter/wiki/Tree-hygiene


Added amazon_freertos_common, boards, rx_driver_package (v125)

Signed-off-by: Dinh Van Nam<EMAIL_ADDRESS>Update the lastest FIT version v125
Description
Add rx_mcu_boards (includes amazon_freertos_common, boards, rx_driver_package) in vendors/renesas
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

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

/cbmc run checks

/bot run checks

/cbmc run checks


Add advancement generator

This branch is missing the requirements property, because I'm not sure how to structure that in a compact way. Worst case it would just be a simple text field.

I also noticed that this breaks the apply_bonus function, but since there is a refactor planned I think I can live with that.


C# Flow Creation

Hey there! Very cool project.
I'm quite interested (as I'm developing most of my semantic kernel applications in C#) in how the C# ecosystem is planned to be integrated into this.
If there isn't a framework now, I'm just trying to figure out what the general plan is. I see a few commits on running promptflow flows within C# but looking at the commit logs its a little unclear to me how to get started at this point.
Might be too early but wanted to get this question in to understand the roadmap a bit better if possible, and to understand what contributions might need to made to get this running.

Support for C# flow experience is work-in-proess, we will reply back here when we have an concrete ETA to share.
Could you describe your scenario a bit more in detail?
What do you plan to do by itegrate promptflow and semantic kernel?

Hi @wangchao1230 just checking in if there are any updates around this?
My team is also interested in doing evaluations on our C# Semantic Kernel project via prompt flow.
I'll include a high-level description of what we'd like to do.  I'm happy to include any more details or provide any suggestions on implementation as well.
Brief Description of Our Project

We have a repository of OpenAI Style plugins.  These are exposed at endpoints like https://service.com/api/.well-known/ai-plugin.json.   These are added simply using Semantic Kernel in C#.
For orchestration we simply use OpenAI Function Calling via Semantic Kernel

Work tried so far
Our initial attempt was to use Python Semantic Kernel.   Expose our C# plugins via OpenAI Plugins like mentioned above.  Then create Prompt Flow "Tools" by using python SK to add them to the kernel and call a completion with them.  This fails because python Semantic Kernel has no support for OpenAI plugins.
How We'd like to Test with Prompt Flow

Evaluating Single Plugins:  These evaluations would take a list of questions + ground truths.  We'd then provide the Kernel with the Single OpenAI style plugin and evaluate the results.
Evaluating Multiple Plugins:  Same as above but providing the Kernel multiple plugins
Evaluating Orchestration:  These would evaluate how well the Planner chooses the right plugins.

This image shows our plan to include SK Plugins via python SK.  This was not possible due to the lack of OpenAI Plugin support from python SK.   Ideally we'd like to be able to do something similar to this, but each C# Semantic Kernel plugin could be exposed as a Prompt Flow Tool.

We have a first version of Promptflow.Net now, and we are verifying its functionality & quality with Microsoft Internal Customer. We will post update here if it's ready to share with broad scope. Thanks for reaching to us!

Thanks @wangchao1230, that's great news.  Looking forward to the updates!

I am interested as well, if there are any updates on this.

Any status on Promptflow.NET? @wangchao1230

Also interested

Also interested too!

Yes!

Very much keen on it

Likewise, as well. As an Microsoft Technology Center Architect, I typically work with 2 - 3 enterprise customers each week. I can you assure you there is a tremendous amount of interest in expanding AI Studio, specifically Prompt Flow, to include C# development teams. As you move into the Enterprise App space, you'll find that teams tasked with constructing GenAI and Intelligent Apps are typically grounded in C# and Java. Had hoped to hear about PF C# at Build, but heard no mention. Question: Is this still an active project?

Yes. It's still an active project. The promptflow .Net experience is under internal review process to get it public preview.
However I could not provide an ETA of when it will be come publicly available now.

@wangchao1230 is there any way to sign up for a private preview?

Any Update from Prompt flow .Net?

There should be support for mixed-language-flows.
The individual nodes should be able to be written in different programming languages. Something like this could be implemented by supporting Jupyter Notebook Kernels.

[!NOTE]
the flow schema seems to only support a global language: element, but not for individual nodes.

@wangchao1230 has there been any update? There's level of interests in keeping codebase in .NET.

Any updates on this? We are investing quite a bit of time in prompt flow and semantic kernel (dotnet version), and feel like there should be an integration point between these two.

Any status on Promptflow.NET?

I'm going to pile on....looking for more info on PF .Net

@wangchao1230 Can you please provide an update on this?


mhlo: fix CMake build for tosa pipeline

The include directories for building the MHLO and TOSA interop is
incorrect, causing the CMake build to fail.  This patch fixes the
include statements.

cc: @jpienaar This is likely to break the Bazel build, so I've marked this as a Draft PR, but let me know if there are any small fixes I can make so that the bazel build will pass.

Sorry, I just noticed this review. I'm going to assign to Jacques as a more appropriate person. As you predicted, this breaks the bazel build without further modification

Thanks @tpopp!  At the cost of sounding naive, how do I trigger a bazel build in the mlir-hlo repo (in the hopes of fixing this myself)?  From what I see, the README includes instructions for CMake builds but not Bazel builds.

I missed this one too (unfortunately the person most active on this section has been out for a month and will be out for one more).
@burmako  for visibility and there is the pending issue to externalize the cmake CI.
This repo is a direct mirror of what's in OpenXLA/XLA and TF, the bazel build instructions for the former probably applies here (but current recommendation there is to use Docker, which I'm not particularly found of as build step). @GMNGeoffrey for visibility too given current dependencies.


feat: add role for SRE bot

This PR adds a role and policy for the SRE Bot in it's account to access read resources inside the org account for trusted advisor and the org list.

We should look at the delta in the S3 bucket


nixos/sshd: Current command vanished from the unit file, execution of the command list won't be resumed.

Describe the bug
I am deploying the same NixOS configuration (with NixOps) from and to the following 2 commits on nixos-20.03-smallchannel:

https://github.com/NixOS/nixpkgs-channels/commit/e8cf1234d77ac2e9121306f46c64aca5c6910396
https://github.com/NixOS/nixpkgs-channels/commit/3ffd8dc105283a69557335d4620124f4eff2a280

For every deploy back and forth these 2 nixpkgs commits, I get the following warning:
May 12 13:13:42 myhostname systemd[1]: sshd.service: Current command vanished from the unit file, execution of the command list won't be resumed.

Expected behavior
No warning should be logged
Additional context
The warning is not logged anymore when overwriting the change introduced in commit 96ba0ca283607977620d31809d52396232074806 by adding this in my config:
systemd.services.sshd.stopIfChanged = mkForce true;

Notify maintainers
@edolstra
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.40, NixOS, 20.03pre-git (Markhor)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.4`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixos modules affected by the problem
module:
  - openssh
  - nginx
  - and more …

I understand the comment made:
https://github.com/NixOS/nixpkgs/blob/9110f85c91dc6ca300d54010213be3308fa9aa73/nixos/modules/services/networking/dhcpcd.nix#L195
I wish we could avoid the warning but I guess it is inevitable when using stopIfChanged = false;, isn't it?


[WIP] Bump up to iron 0.6 and other iron libraries

Merge this after https://github.com/iron/body-parser/pull/79 has been published.

Restarted build

merge ?

Done!


Multi-threaded results after running Bench Average are not normal

The multi-threaded results after running Bench Average are lower than the normal benchmark scores should be.
This is probably a bug to do with multi-threading

Affected Versions: 0.13.1 and 0.13.0

Disabling Bench-Average for now because of how inaccurate it is.


Consumed Electrical or Primary for Hot Water or Heating is Unknown (4 Entities)

Before submitting a new issue

[ yes] I redownloaded the latest release and restarted Home Assistant
[ yes] I've checked known issues
[ yes] I've enabled debug logs and attached the relevant logs to this ticket

Problem description
After today's update (March 12, 2024) I noticed that the energy consumed is no longer displayed.
The entities:

sensor.home_device_0_vc_20cs_1_5_n_de_ecotec_plus_consumed_electrical_energy_domestic_hot_water
sensor.home_device_0_vc_20cs_1_5_n_de_ecotec_plus_consumed_electrical_energy_heating
sensor.home_device_0_vc_20cs_1_5_n_de_ecotec_plus_consumed_primary_energy_domestic_hot_water
sensor.home_device_0_vc_20cs_1_5_n_de_ecotec_plus_consumed_primary_energy_heating
have the status unknown.
For example, I had entered the primary_energy entities in the energy dashboard for gas consumption.
I use a Vaillant VC 20CS/1-5 (N-DE) ecoTEC plus with VR921.

Hello,
i have the same problem since 2 days. Yesterday, after a restart, it was back, but no data arrived. Now I restarted again and now no more data arrives.

I think this might be an issue/"update?" at Vaillant, and not specific to this integration. I also noticed that the information was missing starting March 11th, and the myVaillant app also shows no consumed power information for the current day. Today, yesterday's values suddenly appeared.

This issue describes the same behavior:
https://github.com/signalkraft/mypyllant-component/issues/133
It’s probably better to merge them.

I will close the issue.
The data is available again today without me having done anything.


Add reflection configuration for Actuator endpoints

After the fix for #992, we're seeing that the actuator-webmvc sample fails with the following behavior:

all 15 endpoints are detected by actuator
the CustomNoComponent endpoint is created as a bean as expected
fetching localhost:8080/actuator/customnc fails with a 404

It seems that reflection information is registered for the Custom endpoint, because it is annotated with @Component:
  {
    "name": "com.example.demo.Custom",
    "allDeclaredFields": true,
    "allDeclaredConstructors": true,
    "allDeclaredMethods": true,
    "allPublicMethods": true,
    "allDeclaredClasses": true,
    "methods": [
      {
        "name": "<init>",
        "parameterTypes": []
      }
    ]
  },

On the other hand, nothing is registered for the CustomNoComponent endpoint - actuator cannot look for a @ReadOperation-annotated method and won't register that as an endpoint.

Fixed by 7298292a6c3287


Port to Python 3.5+ using six

Fixes #137

Thanks for the pull request, @dstufft! I've created OSPR-1233 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
We can't start reviewing your pull request until you've submitted a     signed contributor agreement     or indicated your institutional affiliation.     If you like, you can add yourself to the AUTHORS      file for this repo, though that isn't required.     Please see the CONTRIBUTING     file for more information.

I can't reproduce the test failures on Python 3.x locally (to be honest, running those tests just deadlocks for me locally).

@dstufft Thanks so much! Keep us posted on the progress with the contributor agreement.  I understand HPE might be difficult...

Hi @dstufft thanks a lot for this work. I personally won't be able to look for a couple of days, but would be happy to if that would help.
CC @jzoldak

Hi @dstufft I looked at this today, and think it's this expectedFailure regression in py 3.5.0. Travis doesn't appear to have 3.5.1 and above, just 3.5.0 and 3.5-dev.
Because these have been expected failures for a long time, we can consider skipping them instead.

@dstufft @benpatterson FWIW in another project that I happen to be working on that has nothing to do with this one, I had set the python version to 3.5.1 in the travis.yaml and travis downloaded and installed that version successfully. So you might want to try out 3.5.1 after all.

@dstufft Are you working on the contributor agreement, or should we get this ported another way?

Let me poke my manager again to see what the status of that is.

@dstufft I'm looking to show bok-choy some love and had Python 3 compatibility on my list. Any update on whether we'll be able to merge your PR into the codebase?

Sorry, we had some internal rearrangements and my manager switched in the process of that and I sort of forgot about this. I asked my new manager about it and she's going to figure it out.

To be clear, if you want to recreate this work and just make your own PR I'm not going to feel bad about it. The changes were largely mechanical in nature either way.

@dstufft During a hackathon here at edX yesterday and today an impromptu team here was able to bust out a number of improvements that we've been wanting to make with bok-choy, including redoing the python 3 upgrade work.
Thanks again so much for your inspiration/encouragement to get this done.
Not sure if we'll be able to close the loop on publishing to PyPI today as we're heading into demos in a few minutes. But if not today then early next week for sure.

Implemented in #168

Sounds good, thanks a lot!

@dstufft sorry, forgot to publish the changes.
Python 3 support is now available in version 0.6.0 up on PyPI.


New commer. Can you help me to find the scratch folder. I need the recon all results. Thanks!

I have run the docker by the command with preproc mode.  And I noticed that it generated a scratch folder. How can I store it or find it? I need the results in it. Thanks very much.

Hi @ZhyGao,
I think currently the only way to retain the contents of the scratch directory when using a container environment is to include the --debug flag; that will move the scratch directory to the mounted output directory, and prevent its erasure upon completion.
If access to intermediate data is necessary, then a single-button-press application may not be the best fit for you. For things like recon-all results (which isn't part of the preproc analysis level, not sure the source of that discrepancy), personally I'd rather have FreeSurfer run in its own App, and then an App like this would load the derivatives of that App and utilise them, rather than running FreeSurfer internally within its own container. But I think the domain is still some way away from properly standardising how such things will work in the BIDS ecosystem.
Rob

Thanks for your reply. I have stored the scratch folder.
And I have any other questions.

I found a script which is written 3 years before. In this script, they generated the sub-XXX(participant_label)parc_XXX(parcellation)"level-participant_connectome.csv&meanlength.csv". But now I generated the sub-XXX(participant_label)desc_XXX(parcellation)"connectome.csv&meanlength.csv".
Are they the same just changing the name with the update?
Similar to sub-${participant_label}/anat/sub-${participant_label}_parc-${parcellation}_indices.nii.gz. But in my anat folder. I have these files.

And also the desc_desikan_lookup.txt. I need the par-deskin_lookup.txt.

I am unsure what the difference between "parc" and "desc" is. how can I generate the "parc" files?
Waiting for your help, thanks very much.

Are you describing data that were generated using a prior version of this script? I expect that it is likely just a name update, but I don't get to play with this very often so I don't recall the full change history very well any more. For participant level connectomes there's nothing major that comes to mind.
It would be preferable for the software version to be added as a suffix to the derivatives directory name, and for run levels to then be sub-directories within that (at least I think that's supposed to be the structure), in which case you'd know what version of the App was used to generate each. Unfortunately it looks like I don't currently write the version number anywhere else in metadata.
This is almost certainly just a name change. I first wrote this App just after BIDS became a thing, well before BIDS derivatives. I only transitioned to dseg once it became part of the specification.
The lookup table does not vary across subjects, so from memory at some point I started exporting it to the derivatives root directory (if it doesn't yet exist) rather than writing the same thing in every subject directory.

Hello, Lestropie.
Here's a new question. Can you help me?
I process a test data like this in dwi file, And it works.

But when I use my real data. There's a error making the script stoped.
The data list in dwi and the error information are as below.

This error has been reported many times; unfortunately one issue with reporting errors via screenshots is that they are not text searchable by others.
I can only elaborate on the error message generated. As per the README page, the ability to correct EPI susceptibility distortions in the input data is requisite, due to subsequent use of the ACT framework. This App currently only supports correction of those distortions through the FSL topup method, which relies on having spin-echo EPI images (which can come either from DWI b=0 images or the fmap/ directory) with phase encoding contrast: different phase encoding directions and/or readout times. In the absence of such data, EPI susceptibility distortion correction can't be performed, and so the whole process aborts.
Using an alternative solution for EPI distortion correction for processing datasets where such image information is absent would be nice (see #81, #101). Unfortunately this App has fallen well down my priority list, and TBH its design does not lend itself particularly well to such extensibility.


for testnet API invalid highest/lowest/ATH prices depending of number of candlesticks

Version
v0.0.85(55d7027)
RPi version installed on ARM VPS (Oracle)

Description
using testnet api binance notice problem with highest, lowest and ATH values
the larger number of candlesticks the value is growing completely out of the sky, like cumulating some calculation error
for example for pair BTCUSDT (Interval/Limit)
1h/100, highest price: 150535.61, lowest price: 20000.00
1h/30, highest price: 100000.00, lowest price: 20000.00
1h/15, highest price: 75308.67, lowest price: 52607.35
1h/5, highest price: 64685.51, lowest price: 58754.41
1h/1 highest price: 62839.21, lowest price: 61807.95
behavior the same for other pairs, although different values correspnding to the pair

To Reproduce
using Binance testnet API with RPi version
it's fresh installation without any tweaks, only added testnet API keys to the .env file
docker-compose -f docker-compose.rpi.yml up -d --build

Expected Behaviours
proper highest/lowest/ATH values

Screenshots

Additional context
not sure if it's only testnet API bug and/or RPi version

Testnet prices are different than "real market" and transactions are scoped to that testing environment.
As users have a big amount of assets on test account creation and volume are very low for each pairs, it's easy for anyone to pump and dump BTC.
You can check on testnet graphs directly.

yeah, tested installation of the bot on different platforms, same behavior everywhere.
yeah, checked they have no mercy :-(

Thank you for help
At least got some practise with docker :-)


Adding Should I Deploy app

This app retrives https://shouldideploy.today/ response and show it in tidbyt

I have read the CLA Document and I hereby sign the CLA


Elastic 5

Upgrading Elastic to v5 🎉
Upgrading Cloudformation to yaml 🎉
Updating AMIs to Ubuntu 16.04 LTS 🎉

Maybe in the future we should create an AMI with amigo and publish it cc @satterly

Awesome work. Thanks! 👍


Change ordering for  Multisig Transaction list

Closes #339
This PR changes the ordering when retrieving a list of multisig transactions by changing ordering parameter of the request sent to the Transaction Service.

This change only affects GET /v1/chains/:chainId/safes/:safeAddress/multisig-transactions.
MultisigTransactionDetailsMapper._getRejectors also uses this datasource function but the ordering doesn't affect this function's behavior.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 92.217%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
4076

Relevant Lines:
4317

💛  - Coveralls


Icons are gone

I'm using vs code v 1.6.0 and extension v 3.0.3
I updated my vs code today to this version and all the icons are gone.
I've been through this before and followed the instructions from this comment (https://github.com/robertohuertasm/vscode-icons/issues/281#issuecomment-245881786) and every thing worked fine, but now i cant use the Icons Update command.

Already tried to reinstall the extension.

just use the icons theme to enable it again

Hi @viniciusld, icons are now supported via the official API. Please read the readme for more info on this matter.

@robertohuertasm didn't know about that. Thank you!


Q: Continuous Batching without Turbomind?

Hello!
I wanted to ask: Do models like Deepseek Coder v2, which don't have turbomind support, also have continuous batching support or how does it work with them?

PyTorch Engine also supports continuous batching :) cc @grimoire


Added pool part for repo_git

Per https://github.com/openstack/openstack-ansible/commit/99ffcf3e0f0560475f86075d45c729a5c4159434,
the repo server now utilizes a git server. This commit
adds the git server pool part and monitor so the f5-config.py
script will construct the f5 components necessary for this new
git server.
Connects https://github.com/rcbops/rpc-openstack/issues/1625

@busterswt Thoughts on this? If I remember right you guys have looked over changes to the f5 scripts in the past.

LGTM 👍


[TS] Boolean metadata annotation not allowed

What happened?
Trying to set gcp.cloudrun.Service annotation like so :
...
metadata: {
  annotations: {
    "autoscaling.knative.dev/maxScale": "...",
    "run.googleapis.com/cloudsql-instances": "...",
    "run.googleapis.com/cpu-throttling": true
  },
},

But Typescript complains Type 'boolean' is not assignable to type 'Input<string>'.ts(2322).
When trying to pass it as string we get the following error after a pulumi up :
[
  {
    "@type": "type.googleapis.com/google.rpc.BadRequest",
    "fieldViolations": [
      {
        "description": "Invalid value 1. Annotation run.googleapis.com/cpu-throttling must be a boolean",
        "field": "spec.template.metadata.annotations"
      }
    ]
  }
]

Steps to reproduce
import * as gcp from "@pulumi/gcp";

new gcp.cloudrun.Service("default", {
  location: "us-central1",
  template: {
      spec: {
          containers: [{
              image: "gcr.io/cloudrun/hello",
          }],
      },
  },
  metadata: {
    annotations: {
      "run.googleapis.com/cpu-throttling": true
    },
  },
});

Expected Behavior
Be able to pass boolean type to metadata annotation.
Actual Behavior
Cannot pass boolean type to metadata annotation.
Output of pulumi about
No response
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Thanks for the issue @nicaura! I've raised this with the team to be prioritized and added to our work stack. This odd because the docs from google imply that annotations take string, string - and even point directly to K8s docs that explicitly say that both keys and values in annotations must be strings.
Might be worth noting that upstream the annotation field is TypeMap which is apparently map[string]any (not map[string]string).

I think string-only values are by design here. FWIW the following code
import * as gcp from "@pulumi/gcp";

new gcp.cloudrun.Service("default", {
  location: "us-central1",
  template: {
      spec: {
          containers: [{
              image: "gcr.io/cloudrun/hello",
          }],
      },
  },
  metadata: {
    annotations: {
      "run.googleapis.com/cpu-throttling": "true"
    },
  },
});

returns a different error now:

Annotation 'run.googleapis.com/cpu-throttling' is not supported on resources of kind 'Service'. Supported kinds are: Revision

So maybe the original error is fixed? Are you still using this code to try again @nicaura?


Some tables missing important date indexes

A few tables are missing date indexes. Think all the entity models could do with indexes on created and updates dates. Also activity 'created' dates. There may be more.

Coming back to this, Think we should await a specific reason before specifically applying indexes all over the place.


multiSchema + introspection: add --schemas param to the db pull command

Currently, the only way to specify the array of namespaces to introspect is in the datasource block inside a Prisma schema.
This would be required for full support of multiSchema with db pull --url.

Note: the new --schemas param (which refers to the database schemas supported by the multiSchema preview feature) is incredibly similar to the currently existing --schema param (which refers to the location of a custom prisma schema).
How about we introduce a new --multi-schemas param instead of --schemas?

What should happen when the given Prisma schema already contains some database schemas? I think we should inform the user by printing a warning with the following templated message:
`Introspecting with database schemas ${MULTISCHEMAS_FROM_CLI}, which override the database schemas defined in your Prisma schema: ${MULTISCHEMAS_FROM_PRISMA_SCHEMA}`.

Example when running prisma db pull --multi-schemas public,something with the following Prisma schema:
generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["multiSchema"]
}

datasource db {
  provider = "postgresql"
  url      = env("TEST_POSTGRES_URI")
  schemas  = ["base", "transactional"]
}

We should see the following warning:
Introspecting with database schemas `public`, `something`, which override the database schemas defined in your Prisma schema: `base`, `transactional`.

Docs note: we can pass multiple schemas to the db pull command in two ways:

Using commas without a space: prisma db pull --multi-schemas public,something
Using repetitions of the --multi-schemas param: prisma db pull --multi-schemas public --multi-schemas something

Both examples are equivalent to const multiSchemas = ["public", "something"]

How about we introduce a new --multi-schemas param instead of --schemas?

No way to avoid that, unfortunately as "schema" just means multiple things in our space. We prefer to stay consistent in the naming - which means this should be schemas.

What should happen when the given Prisma schema already contains some database schemas? I think we should inform the user by printing a warning with the following templated message [...]

Right now this can just override the existing values. Outputting a warning might be helpful, but should be handled as an additonal feature and not make the existing request more complex.

Closed via https://github.com/prisma/prisma/pull/17034.


<class 'pyscf.pbc.scf.hf.RHF'> does not have attributes  rsjk

When running example 35-gaussian_density_fit.py, I get the error or warning:
<class 'pyscf.pbc.scf.hf.RHF'> does not have attributes  rsjk

That's not unique to example 35. Every PBC calculation starting in 2.0 has
that message since Qiming's implementation of exact 4-center periodic SCF
("RSJK").
On Fri, May 28, 2021 at 7:21 PM Verena Neufeld @.***>
wrote:

When running example 35-gaussian_density_fit.py, I get the error:
<class 'pyscf.pbc.scf.hf.RHF'> does not have attributes rsjk
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/962, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJCZMC7GMBAZ44B2W55DTQAQQNANCNFSM45XUYJ2Q
.

Fair enough, thanks :). I guess it would be less confusing to not have this warning or write a different kind of warning if the warning is indeed deliberate.


Plan to GA aws related features

Which version of the AzCopy was used?
10.3.3
Do you have any plan to GA AWS to Azure copy features? Azure Data Factory once asked us about whether there is a way to let users migrate data from AWS to Azure.

@rishabpoh was discussing this just a few hours ago.  I'd suggest you email him to discuss.

Closing this since v10.5.0 says it is already GA.


Execution hangs after a period if time with ctd-amplified mode

Describe the bug
Tried to generate test cases using the below command on pdl common-externalid-service
tkltest --config-file ./test/data/pdl/common-externalid-service/tkltest_config.toml --verbose generate ctd-amplified
The cmd execution starts but after a period of time the execution gets stuck and the test cases are also not generated.
Attachments
Have attached the output log.
Environment information

CLI version
OS: Windows

Screenshots
common-externalid-service_CombinedTestGenerator_output.log

Fixed as part of recent general enhancements to sequence parsing. The fix should be obtained using the download_lib_jars.sh  script.

Resolved by running on latest version of the code


Whitelist authorization is only checked on login

If a user gets logged in and the whitelist is update to not include them, the user continues to be allowed to do things (create more projects) and stay in the system.
Steps:

Admin: Set access control to unrestricted
User: Login
Admin: Remove all projects User has access to
User: Should see they have no projects now.  Can create more if they want. (this works fine)
Admin: Change access control to restricted, allow only [admin's github username].
User should be locked out at this point because they have no projects && are not in the whitelist.

Actual Result:

User is still logged in. (bad)
If User log out they will not be allowed in again. (good)
If User instead create a new project, they now have a project are now "legitimately" allowed in. (bad)
Thus, it is impossible for Admin to reliably kick User out. (very bad)

v0.19.2
Verified fixed


Basic refactor of extractrepo/repotester. [phab:D2316]

Created by danielmoy-google at 2018-05-17 20:45:37:

danielmoy-google wrote at 2018-05-17 20:50:21:

Merge branch 'master' into refactor

craigbarber commented inline (context lost during migration) at 2018-05-17 22:33:10:
My suggestion for this future TODO would be to preserve the orthogonality of the extraction stuff, and therefore maintain separation between testing for indexing vs extraction.
Also, there might be some overlap between the verifier DSL and an indexing smoke test, could be worth looking into.

craigbarber commented inline (context lost during migration) at 2018-05-17 22:33:10:
There's a "file walk with me" joke somewhere in here ;)

craigbarber commented inline (context lost during migration) at 2018-05-17 22:33:10:
It may be worth abstracting this into a helper function so that when it's parameterized later you only have to re-factor it in one place.

craigbarber commented inline (context lost during migration) at 2018-05-17 22:33:10:
This could just be the Java programmer in me speaking, but it feels like this concern is being duplicated with the configPath parameter in the ExtractRepo function above. One alternative would be to export an Extractor type and give it a ConfigFile property.

craigbarber commented inline (context lost during migration) at 2018-05-17 22:34:15:
I don't see anything specific to Git about this function. Consider instead: NewDefaultExtractor()

danielmoy-google commented inline (context lost during migration) at 2018-05-17 22:51:53:
I'm not sure r.e. the duplication.  One is a default path to look for in the external repo, the other is a file explicitly passed in used to override the default repo bit.
Is the concern here having separate type and function New...? Where the solution is just to have type DefaultExtractor struct { ConfigFile string }?

danielmoy-google commented inline (context lost during migration) at 2018-05-17 22:51:53:
Yea I'm thinking of actually having a smoke test for indexing as well... so extending this framework to support indexing and extracting, but in addition to not in leiu of doing separate verifier tests.

danielmoy-google commented inline (context lost during migration) at 2018-05-17 22:51:53:
ack, though I think I'd rather do that abstraction with the refactor, not ahead of time.  I'm not settled on exactly what form it'll take.

danielmoy-google commented inline (context lost during migration) at 2018-05-17 22:56:03:
ah my merge is failing.  this is done locally

craigbarber commented inline (context lost during migration) at 2018-05-17 22:58:42:
My thinking is that the config file path and its default are separate concerns, not necessarily needing separate variables. One is just the default state of the same variable. The duplication comes from having two variables essentially representing a config file path.

danielmoy-google wrote at 2018-05-17 23:25:52:

addressing comments

danielmoy-google commented inline (context lost during migration) at 2018-05-17 23:37:29:
so we chatted on this offline and it roughly made sense.
one follow-up though - one advantage to having separate default vs passed-in-as-path config, is that you could rely on the default structure for most repos, but selectively point certain repos at different configs, which may actually be worthwhile in the future.

danielmoy-google wrote at 2018-05-17 23:44:50:

addressing comments

danielmoy-google commented inline (context lost during migration) at 2018-05-17 23:45:38:
Ok my proposal now is to just go back to the way you had it before.  When we actually start to have repos with separate configs, we can figure out what the correct thing to do there is.

craigbarber commented inline (context lost during migration) at 2018-05-18 00:00:36:
Makes sense :)

craigbarber accepted this revision at 2018-05-18 00:01:22

danielmoy-google closed via commit 50603419a6482bcb007c7bfe4f8018ccfa77c251 at 2018-05-18 16:24:06


machine types and IOPS

I thought we had some of this in our docs but looks like we removed it or never added it.
We should be transparent that we use the n1 standard series and the c5 series in GCP and AWS respectively for CC. We should also document that we use EBS and Persistent disks and the the IOPS associated with each node size in AWS.
Inspired by this question https://cockroachlabs.slack.com/archives/CJX8V9SJ2/p1614090139010400

Closing in favor of CC-3698


free-softare to free-software + Modding+

I wanted to fix a typo in the contents and include the engine Modding+ since I've used that engine a lot

I'm a bit concerned about the stability of this engine but I wouldn't be as surprised if it gets in there.

it was one of the first engines alongside kade, but the input absolutely sucked, and was honestly just outclassed by kade, though ive heard disappointing+ is decent

I think the input in version 4.0.0 is noticeably better than in 2.2.0. Plus, DIsappointing+ does have some really good touch-ups.
Also, I hope people also notice the typo of "free-softare" in the code of the README because I really want that typo to be fixed to "free-software", as minor as it is (typos tend to slip our minds until we see them, after all).


hoverctl status should tell you if running webserver or proxy mode

It's actually rather hard to discern this information during runtime. There aren't even messages in the bootlog.

@sherl0cks Can you make sure that you have configured hoverctl to use the remote target: https://docs.hoverfly.io/en/latest/pages/tutorials/advanced/remotehoverfly/remotehoverfly.html
You should then be able to use hoverctl status

@JohnFDavenport I have no issue with hoverctl status and remote targets. hoverctl targets works well, nice job on that. the issue is with the output of the command, which as of 0.14.2, does not tell me whether hoverfly is running as a webserver or a proxy. See below example:
$ hoverctl start
Hoverfly is now running

+------------+------+
| admin-port | 8888 |
| proxy-port | 8500 |
+------------+------+

$ hoverctl status

+------------+----------+
| Hoverfly   | running  |
| Admin port |     8888 |
| Proxy port |     8500 |
| Mode       | simulate |
| Middleware | disabled |
+------------+----------+

$ hoverctl start webserver
Hoverfly is now running as a webserver

+----------------+------+
| admin-port     | 8888 |
| webserver-port | 8500 |
+----------------+------+

$ hoverctl status

+------------+----------+
| Hoverfly   | running  |
| Admin port |     8888 |
| Proxy port |     8500 |
| Mode       | simulate |
| Middleware | disabled |

Quite agree with the suggestion, we can change Proxy port to Webserver port if hoverfly is running in webserver mode, or print a new entry shows Proxy type : reverse (webserver)

I'm in favor of a new entry, I think it's more declarative.


zellij: fails to build by not finding a dependency

repro:
{
  inputs.dream2nix.url = "github:nix-community/dream2nix";

  outputs = { self, dream2nix }@inputs:
    let
      dream2nix = inputs.dream2nix.lib.init {
        # modify according to your supported systems
        systems = [ "x86_64-linux" ];
        config.projectRoot = ./. ;
      };
    in dream2nix.makeFlakeOutputs {
      source = ./.;
    };
}

in flake.nix and run nix build .#zellij.
The error:
error: no matching package named `pretty-bytes` found
location searched: registry `crates-io`
required by package `strider v0.2.0 (/build/pcbnlwx4a4r25h4zjkyl0gznigs93kf1-source/default-plugins/strider)`

pretty-bytes is in Cargo.lock, but it seems it's not getting vendored for some reason.

Thank you for opening the issue!
Here is a more self contained example:
{
  inputs.dream2nix.url = "github:nix-community/dream2nix";
  inputs.zellij.url = "github:zellij-org/zellij";
  inputs.zellij.flake = false;

  outputs = { self, dream2nix, zellij }@inputs:
    let
      dream2nix = inputs.dream2nix.lib.init {
        # modify according to your supported systems
        systems = [ "x86_64-linux" ];
        config.projectRoot = zellij ;
      };
    in dream2nix.makeFlakeOutputs {
      source = zellij;
    };
}

Currently, the error when running nix build .#zellij on the example flake is:
zellij> building
zellij> Executing cargoBuildHook
zellij> ++ env CC_x86_64-unknown-linux-gnu=/nix/store/pja9g36cy32z3d51942jqk91a6l2d5nv-gcc-wrapper-10.3.0/bin/cc CXX_x86_64-unknown-linux-gnu=/nix/store/pja9g36cy32z3d51942jqk91a6l2d5nv-gcc-wrapper-10.3.0/bin/c++ CC_x86_64-unknown-linux-gnu=/nix/store/pja9g36cy32z3d51942jqk91a6l2d5nv-gcc-wrapper-10.3.0/bin/cc CXX_x86_64-unknown-linux-gnu=/nix/store/pja9g36cy32z3d51942jqk91a6l2d5nv-gcc-wrapper-10.3.0/bin/c++ cargo build -j 4 --target x86_64-unknown-linux-gnu --frozen --release --package zellij
zellij> error: package collision in the lockfile: packages zellij-client v0.30.0 (/build/source/zellij-client) and zellij-client v0.30.0 (/nix/store/p4inwc3sjl85cyhq1dcasizw61mdl5ix-source/zellij-client) are different, but only one can be written to lockfile unambiguously

zellij now builds & runs fine on main.

Awesome, thanks @yusdacra for also checking again after all these fixes.


[code-completion] Don't try to equate/convert GenericTypeParameterTypes when computing type relations of completion results

For call argument completion, if the expected type and completion result type are
both GenericTypeParameterTypes, comparing them doesn't make sense as we can't account
for their different contexts at this point in the code, and hit assertions in the
constraint solver if we try.
For now, don't attempt to add a type relation for this case.

Resolves rdar://problem/38153332.

This is https://github.com/apple/swift/pull/15411 minus the extraneous commits.

@swift-ci please test

Cool, thanks for fixing the crash! Perhaps in the future we can do something smarter here where we take the requirements into account?

Yeah, agreed. I'll clone the radar to track actually being correct here.

Build failed
Swift Test OS X Platform
Git Sha - d9ca14203d38c7294a66083d5ca3e12428bc352a

@swift-ci please smoke test OS X


One to one relation not working - opposite field complaint

Bug description
I'm modelling a User with roles such as 'student', 'customer'.
A student can be tied to one customer in a "tutoring" ( mentoring? ) relationship.
A customer can tutor several students.
This is my model:
model User {
  id            Int @id @default(autoincrement())
/** irrelevant fields... */
  roles         Role[]
  tutorId       Int?
  tutor         User? @relation(name: "StudentCustomer", fields: tutorId, references: id)
  // students      User[] @relation("CustomerStudent", references: [id]) // buggy so commented out.
}

This is the error message I get any time I run a migration:
Error: Schema parsing
error: Error validating field `tutor` in model `User`: The relation field `tutor` on Model `User` is missing an opposite relation field on the model `User`. Either run `prisma format` or add it manually.
  -->  schema.prisma:148
   |
147 |   tutorId       Int?
148 |   tutor         User? @relation(name: "StudentCustomer", fields: tutorId, references: id)
149 |   // students      User[] @relation("CustomerStudent", references: [id])
   |

Expected behavior
A no error migration? I've tried naming/not naming the relation, having the fields and references in and out array, having tutorId after/before tutor, none works.
I've also tried the documentation's example:
  successorId Int?
  successor   User?   @relation("BlogOwnerHistory", fields: [successorId], references: [id])

This is not working either.
Ultimately I need to be able to store multiple self relation to the User model.
Environment & setup
Prisma 2.17
RedwoodJS v0.27
nvm 14.0.0

You missed the third needed field which is in case of documentation is
model User {
  id             Int     @default(autoincrement()) @id
  successorId Int?
  successor   User?   @relation("BlogOwnerHistory", fields: [successorId], references: [id])
  predecessor User?   @relation("BlogOwnerHistory")
}

@njilrem thanks for the answer.
I'm not sure I follow, predecessor is necessary in the case of the documentation but how is it relevant in my use case? Does it mean that named relations would require a third field, only identified as a part of the relation through @relation("name") ?

Ok, got it now, fixed.

Ok, got it now, fixed.

how to fix, Can you please share your code? thanks

@leanbkhcm  can't recall if that was enough, but this is the bit that's related to this issue in my current code and it's since been working with it:
model User {
  # ... some fields
  tutorId       Int?
  tutor         User? @relation("CustomerStudents", fields: tutorId, references: id)
  # ... some other fields
}

If it can help.

Please open a new issue @RafaelCss - this issue is closed.

References should exist on both sides, this code that I wrote:
`model User {
id String @id @default(auto()) @map("_id") @db.ObjectId
email String
name String?
username String @unique
password String
profileImage String?
createdAt DateTime @default(now())
updatedAt DateTime @updatedAt
refreshToken RefreshToken[]
}
model RefreshToken {
id String @id @default(auto()) @map("_id") @db.ObjectId
token String @unique
createdAt DateTime @default(now())
updatedAt DateTime @updatedAt
userId String @db.ObjectId
user User @relation(fields: [userId], references: [id])
}`


how to store Box<[u8]> or Vec or bytes::Bytes?

as title
and I use Box<[u8]> as primary_key ?

It is not supported in v0.1.0, we will support in the next version, thanks!


Dependency Updates: devDependencies, Renovate, etc.

Description/Context
Many of our dependencies are significantly out of date. For example, we use postcss-import v12, whereas v16 is the latest; we use sass-loader v8, whereas v14 is the latest.
It would be nice to update dependencies automatically, e.g., with Renovate. However, may not want Renovate to auto-merge major version updates—even if tests pass—of things that make it into our production bundle, since major version updates could affect browser support.
Plan/Design
My thoughts are:

First, group our package.json dependencies into devDependencies and dependencies. Things that make it into our production bundle (jquery, lodash, etc) would go in dependencies. Build and dev tools (webpack, eslint, etc) would be devDependencies.
While doing the above, audit our dependencies. Some are unused (for example—as far as I can tell—walk is unused). Remove unused dependencies.
Configure renovate to update stuff.

I would not recommend auto-merging major updates to dependencies.
We may not want to auto-merge updates to hugo-bin-extended.

Devil's advocate for auto-updating hugo-bin-extended:
The Node based Hugo package here is only used for local development. Having it auto update would mean that we would become more aware of issues with Hugo more quickly, rather than finding out about things when we go to update the version we use in production. Although, there's also something to be said about simply pinning to the version in production. That way, everything you test locally should behave the same in production.

Renovate was configured for this repo in https://github.com/mitodl/ocw-hugo-themes/pull/1270. Minor dependencies are being updated and auto-merged. I believe hugo-bin-extended is in the queue to be auto-merged soon.

That way, everything you test locally should behave the same in production.

I believe this is the main motivation for disabling auto merges for hugo-bin-extended. If I were to develop something based on a newer feature of an auto-updated Hugo and it fails on RC, checking the Hugo version might not be my first instinct.
I can see the potential in the auto-updates. Perhaps when we have a more thorough suite of e2e tests, we can automate the version updates on both ends i.e. course publisher image image and npm package.
I've created https://github.com/mitodl/ocw-hugo-themes/pull/1357 which should prevent the renovate auto-merges for hugo-bin-extended once merged.


Fix issue with sheet.close and sheet.open

The two functions could be called when the given sheet was in the correct state. Updated px.sheet tests.

Codecov Report

Merging #42 into develop will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop      #42     +/-   ##
==========================================
+ Coverage    94.24%   94.35%   +0.1%
==========================================
  Files          124      124
  Lines         1965     1967      +2
==========================================
+ Hits          1852     1856      +4
+ Misses         113      111      -2

Impacted Files
Coverage Δ

src/px-script/main/sheet/index.js
100% <100%> (+2.19%)
:arrow_up:

Continue to review full report at Codecov.

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

Better naming for subheading? Doc sub heading for Sheet is example x2.
Handle .has-scroll might need a OS prefix, Mac is a bit jumpy, 16px seems right, this change probably belongs in another feature/task/issue and handleScrollbar() could be better named.

The class that is added to the body when handleScrollbar() runs is called .has-vScroll, should we change that aswell? Maybe rename handleScrollbar() to findScrollbar \ validateScrollbar \ hasScrollbar or something, i'll make an issue for it on Jira.

Better naming for subheading? Doc sub heading for Sheet is example x2.
Handle .has-scroll might need a OS prefix, Mac is a bit jumpy, 16px seems right, this change probably belongs in another feature/task/issue and handleScrollbar() could be better named.

The class that is added to the body when handleScrollbar() runs is called .has-vScroll, should we change that aswell? Maybe rename handleScrollbar() to findScrollbar \ validateScrollbar \ hasScrollbar or something, i'll make an issue for it on Jira.
Related: scrollbar widths

Jira issue 👍


[BUG] Waybar uses raw width of output mode, which is wrong when rotated

Correct my if I'm wrong, this is just my suspicion. I have an external screen configured to be rotated by 90 degrees, and waybar seems to take the raw width instead the actual width from the perspective of the workspace or whatever: The middle part or almost at the right border and the right part of the bar is not displayed at all.

Nevermind, this works as expected after exiting and starting sway again. I had only soft reloaded at this point.


[embedded] Avoid a crash on location-less SIL functions in PerformanceDiagnostics

Turns out that SILGen can generate calls to things like swift_readAtKeyPath, which end up being location-less SILFunctions. Let's avoid crashing on those in PerformanceDiagnostics. The attached test case previously failed with:
Assertion failed: (DebugScope && "no scope/location"), function getLocation, file SILFunction.h, line 1086.

rdar://125724841

@swift-ci please test

@swift-ci please test macOS platform

@swift-ci please test macOS platform


[test/SourceKit] Add a test case for a crash that is already fixed

@swift-ci smoke test and merge


Setup instructions are now explicitly clear

Fixes #33
It can be difficult for users unfamiliar with Swift Package Manager and the Vapor toolbox to add and use dependencies to a project. To reduce friction toward adopting the software these instructions make no assumption about users’ knowledge and provide explicit instructions.

Sorry about that, I've taken the liberty to edit it in a cleaner style, so that comment is obsolete.
Also, neither vapor build, nor vapor xcode will run the package resolution, which is mandatory to have a working project in the end, so I changed the second step too.
Thanks for the update though! We needed a gentle push to be newbie-friendly in the README after all.


Mocking interface with more than 1 generic type does not work

Hi,
thanks for this awesome lib. Unfortunately, I ran into a bug.
When I try to mock interface looking for example like this:
interface Example<INPUT : Any, OUTPUT : Any> {     fun foo(key: INPUT): OUTPUT }
Build ends with an error:
e: ...shared/build/generated/ksp/android/androidDebugUnitTest/kotlin/io/mockative/GeneratedMocks.kt: (3, 219): Expecting a top level declaration
GeneratedMock.kt looks like this
internal fun <INPUT, OUTPUT> mock(@Suppress("UNUSED_PARAMETER") type: kotlin.reflect.KClass<com.example.mockativetest.Example<INPUT, OUTPUT>>): com.example.mockativetest.Example<INPUT, OUTPUT> where INPUT : kotlin.Any where OUTPUT : kotlin.Any = com.example.mockativetest.ExampleMock<INPUT, OUTPUT>()
Looks like a syntax error, there should be just one where, following generics should be separated by , :)

Hi @spartanmarty
Apologies for not getting back to you sooner.
I have just released version 1.2.0 of Mockative, which features a complete rewrite of the code generation using KotlinPoet, which should resolve your issue.

Hi again
I discovered an issue in 1.2.0 causing the generated code to be invalid. I have resolved this issue in 1.2.1, which is on its way out.


Catboys?

What happened with the Catboy API is it possible for it to be re-added or is there any alternatives?

Sadly the Catboys API Website got shut down, so i can't add it again. (the old url of the API: catboys.com )
i don't know of any alternative APIs, which I could add to the plugin.


[][broker/client] Initial implementation of message filtering

Motivation
Broker-side filtering of messages can reduce the network load significantly when running certain types of highly-filtered jobs. Example usecases are flink filter-pushdown, displaying data over websocket, and other network-limited actions.
So far, the list of available filters is pretty small, but this framework should make writing new ones relatively easy.
Modifications
I've modified the pulsar protocol such that clients can send a spec for filtering to the broker. The broker's Consumer class builds a filter from the arguments sent as well as the class name. I've gone with reflection because that makes for easy extension and customization of the available filters.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

the new tests in MessageFilteringTest/SimpleTypedMessageProducerConsumerTest should pass
all other tests should be unaffected by the changes

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)

a new function is added to the consumer to allow for constructing a message filter

The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)

a new field was added to commandsubscribe to send info about the message filter

The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

once this is in a more stable state, an entry in the actual docs is probably necessary.

If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Hi all.
This is my first large pulsar PR, and given the contents of it I've got quite a few questions/discussion points about the implementation (in no particular order)

my biggest worry is that the way I've done filtering (acking filtered-out messages on the broker) fails in some strange case
additionally, the performance of running the filter inside the jetty thread-loop might be questionable, but I feel like running it in the consumer send function would be worse (as I think that's single threaded?).
reflection to create the filter classes isn't pretty, but it allows for easy user extension. Is that reason enough to have it?
also, what kind of filters should pulsar itself be in the business of providing? I only created a few to prove out the feature, but they also cover a relatively large amount of my use-case.
Is there a better way to send over arguments to the filter? I used KeyValue since it was convenient, but sending serialized objects of some sort would be much nicer (and necessary for flink integration, as far as I can tell)

For changes that introduce new features or protocol changes, it's recommended to submit a PIP (design doc) for discussion to the mailing list. PTAL for examples at https://github.com/apache/pulsar/wiki

For changes that introduce new features or protocol changes, it's recommended to submit a PIP (design doc) for discussion to the mailing list. PTAL for examples at https://github.com/apache/pulsar/wiki

You know, "should I open a PIP too" should have made my question list. I'll start putting one together.

I've drawn a PIP up but couldn't figure out how to actually contribute it. It doesn't look like you can do wiki PRs.

@seeday You can put the PIP in a Google Doc or a gist and send an email to dev@ mailing list. Once the thread is started, one of the committers will take the content and add it to Pulsar wiki.


cilium status: fix --brief to print less when cilium isn't running

Fixes: #2880
Signed-off-by: Nirmoy Das<EMAIL_ADDRESS>

test-me-please

Ginkgo CI tests need to pass before merging. Initiated the tests. Thanks

test-me-please

Can't understand why Ci fails :/
/home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:373
Expected
: "...      STATU..."
to equal               |
: "...     STATUS..."
/home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/src/github.com/cilium/cilium/vendor/github.com/onsi/gomega/internal/assertion/assertion.go:35

@nirmoy looks like potentially another case of https://github.com/cilium/cilium/issues/4255. I'm working on a fix.

test-me-please

test-me-please

test-me-please


Trovi 2.0: Implement read endpoints

Trovi 2.0: Implement read endpoints
Implements the ListArtifacts and GetArtifact endpoints. The original task called for auth to be implemented by this point,
but since this task was prioritized first, auth has been left as a TODO.
There is also a basic test to confirm that the models will properly serialize to JSON,
but better testing will be implemented while the code is in review.
Changes

Implements an API app
Creates some basic response types which make error responses a little easier
Implements read endpoints as classes with a static get function, ensuring that only GET requests are valid
Fixes an unforeseen bug in URNField, which did not handle null values
Adds an abstract base APIModel, which defines how models will be serialized to JSON.
All models which appear in API responses now implement this class.

Defines an api_items property which defines the structure of how a model will be translated to a JSON API response
Defines a to_json property which defines how specifically the model will be serialized.
The implementation in the base class adheres strictly to the properties of api_items
and will be the correct choice in almost every case.

Adds some JSON Protocol type definitions for use in hints

NOTE
This PR is currently a little weird because it is based off of the work in #3, which has not been merged into main yet. I will pull it out of draft state once that work is on main so that the commits on this branch are not so redundant.

 Trovi 2.0: Implement read endpoints

I merged #3, so you can try rebasing against the main branch again to clean up that aspect of the history :)

Read endpoints are 100% in the state I want them to be in. Might be a good idea to give them another once-over. The write-endpoints PR that is coming soon will be based on this work.


Make a couple of fixes to ComInterop as suggested by .NET team

PR Summary
This is a small change suggested by the .NET team. See the conversation here: https://github.com/dotnet/runtime/pull/40234#discussion_r464648065.
Quoted:

we consider VT_PTR as unsupported throughout the runtime and specifically handle IntPtr/UIntPtr to be VT_INT/VT_UINT. I think that part needs to stay as-is.
Update SetAsByRefInt/SetAsByrefUint to take int/uint instead of IntPtr/UIntPtr

PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@PoshChan please remind me in 1 day

@rjmholt, this is the reminder you requested 1 day ago


[Question] Fort IDs

I was playing around with some map object dumps, and I would want to ask if anyone knows what is the formar for a fort_id string
It seems to be a string with 32 characters (maybe a md5 of something? Cell id?), a dot, and a number
Does anyone have precise information? Also spawn point ids work the same way (i guess).
Thanks :D

Closing this issue, let me know if this is still relevant.


Add checks for missing submodules like ansible-ca-store

Add check for missing submodules like roles/ansible-ca-store and roles/ansible-docker-ubuntu.
Otherwise it can be quite troublesome to only realise this when the new AMI is built and run without the right certificates.

include_role and import_role is weirdly defined. What we should be hoping for is to fail the Ansible task as long as the task file or entire tasks directory is missing, which is the case when we forget to do submodule update within this repository. Unfortunately this cannot be achieved for this Ansible task as of yet.
If we look at the code from branch stable-2.7 (latest stable at this point of time):
https://github.com/ansible/ansible/blob/07927edfac185b50d85904cd959214082388c833/lib/ansible/playbook/role/__init__.py#L242
https://github.com/ansible/ansible/blob/07927edfac185b50d85904cd959214082388c833/lib/ansible/playbook/role/__init__.py#L225
even if we were to set tasks_from in include_role, it wouldn't work since it needs the tasks directory to be created, and if not, it simply skips loading tasks for the given role, rather than raising an error.
As a sidenote, it also has weird behaviour when tasks_from  is set to a non-default value (default value is main):

If left as main, even if tasks directory is created and filled with other files, it passes even if main.yml / main.yaml / main.json doesn't exist.
On the other hand, if set as some non-default value, if tasks directory is created but empty, it will fail and print the "Could not find specified file in role" error message.

As such, this is more likely to be resolved through custom hackish checks.


Add basic CI for current Python and Django versions

This PR updates the CI to build against current versions of Python and Django.
The Travis integration is currently broken: maintainers, you'll need to follow the steps here to get it working: https://docs.travis-ci.com/user/migrate/open-source-repository-migration
Then you'll need to do something like git commit --amend --no-edit && git push -f to trigger it.
I've tested the CI config on my fork - see allanlewis/django-username-email#1.

Updated after Django 4 changes.

@allanlewis Thanks for taking the time to do this!
@tmm Do you think you would be able to migrate the repo to travis-ci.com sometime?

@allanlewis nice one! @ataylor32 will do.

Thanks, all - glad I've been able to contribute in a small way 😊
That said, I can't see a build from this repo for the merge...

@allanlewis I am the owner of this project now (@tmm transferred it to me). I don't have a plan on travis-ci.com. There is a free trial plan, but not a free plan. Would you be open to modifying your code to use GitHub Actions instead of Travis CI?

See #25.