[Bug]: github reader quickstart throws error

Bug Description
When running the GithubReader MVP, there is a key error. Also, the demo needs to be updated to leverage the github client.
Version
0.10.20
Steps to Reproduce
# Example gihub repo reader
# %pip install llama-index-readers-github llama-index nest_asyncio httpx
# %env OPENAI_API_KEY=sk-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
# %env GITHUB_TOKEN=github_pat_xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

import nest_asyncio
nest_asyncio.apply()

from llama_index.core import VectorStoreIndex
from llama_index.readers.github import GithubRepositoryReader
from llama_index.readers.github.repository.github_client import GithubClient
import os

owner = "jerryjliu"
repo = "llama_index"
branch = "main"

github_client = GithubClient(github_token=os.environ["GITHUB_TOKEN"], verbose=True)
reader = GithubRepositoryReader(github_client=github_client, repo="repo",owner="owner")
reader.load_data(branch="branch") # throws

Relevant Logs/Tracbacks
Traceback (most recent call last):
  File "/Users/michael.berk/dev/mlflow/llama_index/testing/github_reader.py", line 123, in <module>
    load_data_from_branch()
  File "/Users/michael.berk/dev/mlflow/llama_index/testing/github_reader.py", line 72, in wrapper
    func(*args, **kwargs)
  File "/Users/michael.berk/dev/mlflow/llama_index/testing/github_reader.py", line 117, in load_data_from_branch
    documents = reader1.load_data(branch="main")
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/site-packages/llama_index/readers/github/repository/base.py", line 299, in load_data
    return self._load_data_from_branch(branch)
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/site-packages/llama_index/readers/github/repository/base.py", line 259, in _load_data_from_branch
    branch_data: GitBranchResponseModel = self._loop.run_until_complete(
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/site-packages/nest_asyncio.py", line 98, in run_until_complete
    return f.result()
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/asyncio/futures.py", line 201, in result
    raise self._exception.with_traceback(self._exception_tb)
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/asyncio/tasks.py", line 232, in __step
    result = coro.send(None)
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/site-packages/llama_index/readers/github/repository/github_client.py", line 361, in get_branch
    return GitBranchResponseModel.from_json(
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/site-packages/dataclasses_json/api.py", line 63, in from_json
    return cls.from_dict(kvs, infer_missing=infer_missing)
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/site-packages/dataclasses_json/api.py", line 70, in from_dict
    return _decode_dataclass(cls, kvs, infer_missing)
  File "/Users/michael.berk/opt/anaconda3/envs/1-dev/lib/python3.10/site-packages/dataclasses_json/core.py", line 172, in _decode_dataclass
    field_value = kvs[field.name]
KeyError: 'commit'

Details

dataclasses-json version: 0.6.4
This also fails: https://github.com/run-llama/llama_index/blob/5c53f41712785e5558156372bdc4f33a6326fa5f/llama-index-integrations/readers/llama-index-readers-github/llama_index/readers/github/repository/base.py#L537

If you change the repo to run-llama instead of jerryliu it works, otherwise you get a 301 response, moved

I'm getting the same error. I changed owner to run-llama, I updated llama-index to version 0.10.22, I tried to read langchain repo, and that one worked. I used the same settings for both. I tried some other repos and I got mixed results. For some it works okay, for others I get that error.

Same thing here.
I got the KeyError: 'commit' as well as KeyError: 'url' for a different repo

Same error here
KeyError: 'url'

Same error for me. KeyError: 'url'. The function fails at this point (in llama_index/readers/github/repository/github_client.py):
        response = (
            await self.request(
                "getTree",
                "GET",
                owner=owner,
                repo=repo,
                tree_sha=tree_sha,
                timeout=timeout,
                retries=retries,
            )
        ).text
        return GitTreeResponseModel.from_json(response)

The error was thrown at the GitTreeResponseModel.from_json(response).  Looks like the reason is that the repository I'm trying to ingest has submodules in it, which creates tree items without the url property.
I managed to bypass this by overriding the original implementation with this implementation:
root = (
            await self._github_client.request(
                "getTree",
                "GET",
                owner=owner,
                repo=repo,
                tree_sha=tree_sha,
                timeout=timeout,
                retries=retries,
            )
        ).json()
        root["tree"] = [path for path in root["tree"] if path.get("url") is not None]
        return GitTreeResponseModel.from_json(json.dumps(root))

If anyone wants add a fix to this function that'd be great!

There is a similar problem occurring with the following snippet at approximately 364 in github_client.py. It only impacts a subset of the repos I am attempting to pull.
After fiddling with it, I eventually defaulted to just cloning the repo and using the simple directory reader. I don't know if it's the stability of the GitHub API that's driving this or something else, but I'm moving on.
return GitBranchResponseModel.from_json(
            (
                await self.request(
                    "getBranch",
                    "GET",
                    owner=owner,
                    repo=repo,
                    branch=branch,
                    timeout=timeout,
                    retries=retries,
                )
            ).text
        )


Recipes not shown in JEI

Issue description:
In the last release the osmium compressor recipes are not shown in the JEI but they are working in the machine.
Version (make sure you are on the latest version before reporting):
**Forge: <IP_ADDRESS>84
**Mekanism: 1.12.2-<IP_ADDRESS>7
**JEI: 1.12.2-<IP_ADDRESS>9

Same for me


Palette option

choosing Bright or muted do not write to the ini, but greyscale does, changing back to any of the other two, deletes the line.

Bug elsewhere caused this. Fixed.


Bug: makeEffect entity impostor will get reused while async logic runs

Oops. Need to address this, and it could be tricky.
On one hand, could use an async iterator for effect iteration - but then we'd have to wait on each item before proceeding to the next, which wouldn't be necessary.
The core problem is entity impostor lives at the Archetype level, which works fine for synchronous iteration but once async usage is added it becomes a huge bottleneck to enforce only one iterator is using it at a time.
In fact this also breaks if nested iteration happens!
Best approach is probably proper pooling of EntityImpostors. I will need to identify some way to tell when an impostor can be returned to the pool - i.e. when it's still 'in use,' even in async cases.

I think a part of this will require migrating Archetypes to iterate over component arrays instead of impostors, and moving the impostor pooling and allocation into queries.


Sign in/sign up with Apple

How about Sign in/sign up with Apple suport for iOS apps

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 01f308c4-9c2a-06c6-d460-5b4b9ed3877f
Version Independent ID: 9ffaa9db-650b-a250-0e16-b450d69cc3f3
Content: Set up direct sign-in using Azure Active Directory B2C
Content Source: articles/active-directory-b2c/direct-signin.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@danielr11220 Thank you for your feedback . We will have this checked and update the thread.

@danielr11220 Currently you can use the custom policy flow to use Apple as Identity provider while sign in process.
Here is a sample on how to do it : https://github.com/azure-ad-b2c/samples/tree/master/policies/sign-in-with-apple
As far as the proper supported method via GUI, like any other IDP, not sure about any ETA on that.

@danielr11220 Here is the update from internal team :
Public preview for User Flows and Custom policies should be in about a month.
User Flows will be GUI approach.
Let us know if you have any questions.


support for running a Mendix app without routes

When trying to start a Mendix app using this buildpack, before this
commit you got this error and your app was not able to start:

INFO: Started Mendix Cloud Foundry Buildpack vX.Y.Z
Traceback (most recent call last):
File "start.py", line 1207, in
m2ee = set_up_m2ee_client(buildpackutil.get_vcap_data())
File "start.py", line 811, in set_up_m2ee_client
m2ee,
File "start.py", line 632, in set_runtime_config
"ApplicationRootUrl": "https://%s" % vcap_data["application_uris"][0],
IndexError: list index out of range
Terminated

This commit adds support for running a Mendix app without having any
routes mapped to this application.

[APP/PROC/WEB/0] OUT INFO: S3 config detected, activating external file store
[APP/PROC/WEB/0] OUT INFO: Successfully updated backup service
[APP/PROC/WEB/0] OUT INFO: Trying to start the MxRuntime...
...
etc.

Looks good

@pommi  - do we have an UP or DEP ticket for this PR? So that we can test and merge.

Closing in favour of #330


Adjust all tests of cloud-cap-samples to start test server inside describ

Hello CAP Documentation Team,
with #1068 the CAPire documentation on Testing with cds.test was adjusted in the section Running a CAP Server that the cds.test should be started inside a describe:
const cds = require('@sap/cds')
describe(()=>{
  const test = cds.test(__dirname+'/..')
})

Not all tests in https://github.com/SAP-samples/cloud-cap-samples/tree/main/test follow this best practice. I think this needs to be adjusted. For cap-sflight I've filed https://github.com/SAP-samples/cap-sflight/issues/1177.
Best Regards
Gregor

Hi Gregor,
Thank you for reporting the issue, I'll check and update as necessary.
Best regards,
Mahati


Exceptions and Typed errors

Hi! Thank you for the library, it's really helpful.
While investigating it I found that it's requires wrapping packageInstaller.createSession(apkUri).await() in try\catch block.
Have you considered instead of throwing exceptions to pass typed errors instead as you already do, but for all other cases too? It helps with understanding what could happen and react to it before it happens.
For me as user of library I can't really distinguish between InstallFailure.Exceptional and Exception been thrown.
One example would be - I encountered FileNotExists exception (when apk file was missing), while I expected some sort of InstallFailure

@nvkleban Thanks for an interesting question! This is quite complicated topic. In my opinion, as we're dealing with coroutines, await() function should rethrow all exceptions encountered, because this is an expected behavior by convention. Thus, actually, you'll never get InstallFailure.Exceptional as a return value.
However, as you noted, this is quite confusing. This peculiarity stems from core architectural choice of designing Session as an observable object to not be dependent on a concrete asynchrony implementation baked into frameworks or a language. I couldn't make Exceptional failure to not be a part of SessionResult.Error failure sealed hierarchy and instead to be a part of a separate one and maintain the same types for failures across modules. Sadly, I don't think Kotlin's type system allows this. Taking into consideration what I wrote in the first paragraph, I'm reluctant to introduce such a change. As a workaround, you can write your own await() wrapper which will catch exceptions and return them as InstallFailure.Exceptional objects, though I'll refrain from providing such functionality in the library.
Perhaps, the best compromise is to introduce a separate sealed hierarchy of failures specifically for coroutines, like I suggested above, though it'll be a major API change. What do you think?

Now that I think about the last suggestion I made, I'm sure that it also won't work out. So, we can't omit a type Exceptional from failures sealed hierarchy for await(). That leaves us with a question, whether to wrap exceptions or not. My stance is outlined in previous comment. If you have arguments against, I'm ready to consider them.

There is couple of my thoughts on the subject. Take it with the grain of salt.

Are all current exceptions should be exceptions? A lot of them are expected to happen sometimes, and could be a part of the sealed hierarchy. For i.e. mentioned above FileNotExists.
Are exceptions that should not happen ever (unless there is a bug in library) should be delivered in Exceptional to user as part of this sealed hierarchy? I think no as this situation should never happen and if it happens - I would rather prefer crash (fail fast approach). For i.e. cases where you look for extras in your own intents.
If there any cases where exceptions are not fit into 2 previous categories. As a library user do I even need them? Like what I can do with it. It may be as well just a Generic failure.
Exceptions are generally discouraged in Kotlin, while suspend function may rethrow any encountered exceptions CancelationExceptions, OOM, logic bugs etc. it doesn't inherently means that all other errors should be exceptions.

There is an excellent read for Roman Elizarov about exceptions in Kotlin

As a rule of thumb, you should not be catching exceptions in general Kotlin code. That’s a code smell. Exceptions should be handled by some top-level framework code of your application to alert developers of the bugs in the code and to restart your application or its affected operation. That’s the primary purpose of exceptions in Kotlin.

Thus, the same general advice applies to exceptions and coroutines: don’t use exceptions if you need local handling of certain failure scenarios in your code, don’t use exceptions to return a result value, avoid try/catch in general application code, implement centralized exception-handling logic, handle input/output errors uniformly at an appropriate boundary of your code.

P.S. It's totally not a blocker as I use the library like you mentioned in a doc, wrapping it in a try\catch, just a genuine question that appeared in my head while I was using it.

@nvkleban
So, in general we share mostly the same view.

All expected failures are handled with the existing sealed hierarchy, and unless there's faulty input, or bug in the library, or factors not controlled by it come into play, they should fall into those. To break down your example, Ackpine doesn't do file existence checks as it expects that checks are not needed (e.g. user input from file picker or freshly created file) or were made by library consumer, hereby FileNotFoundException is viewed as an exceptional outcome. I'm not even sure where it's being thrown. Well, now it may become a discussion about whether the library should check inputs, but if it will, then these expected failures would be wrapped as Generic.
I agree. Like I said, I view all exceptions originating in the library as unexpected ones. For example, I/O operations in general case are always expected to complete successfully. That's why, as I mentioned earlier, exceptions will never be delivered as Exceptional failures from await(), they will be thrown. There's some inconvenience because of that though when matching on a returned failure type, but I can't find an elegant solution. Maybe this is worth mentioning in the docs.
I don't think there are cases like this.
See above.


Support Serialize + Deserialize for dashu-ratio

string form

"-5/11"

object form

{
    "sign": true,
    "numerator": [5],
    "denominator": [11]
}

Well, sorry, seems not work with no-std

Thanks for your contribution, but the implementation seems over-complicate for me. Do you have any specific reason to serialize the rational number as a struct with three parts? I was planning to serialize the rational number just as a pair of integers.
Besides, you can take a look at #11, where I listed all thirdparty crates I planned to support, so you don't have to implement them yourselves.

How about dividing into the following three binary representations?

[ ] => 0 / 1
[i64, bytes<numerator>] => numerator / 1
[i64, bytes<numerator>, u64, bytes<denominator>] => numerator / denominator

Maybe VLQ is better than i64 and u64, since it also supports non-64-bit platforms.

I don't think we need to handle the VLQ implementation ourselves, it could be handled by serializers like the postcard. I already have serde support for UBig and IBig, we should reuse the implementation of that. (Although I am planning to refactor it to use byte array instead of the word array which is currently used).
What I have planned for the (binary) serialization formats is:

UBig serialized as a byte array (with given length)
IBig serialized as a byte array with the sign stored in the last byte
RBig serialized as a (named) tuple of (UBig, IBig)
FBig serialized as a (named) tuple of (IBig, isize, usize)

For text serialization, I would just use the format result of each type.

serde support for dashu-float and dashu-ratio has been added in https://github.com/cmpute/dashu/commit/d184f7cbb7654ccad692d7ff14d88c8e9f333e67, the improvements on serializing UBig and IBig will be added after releasing the next minor version, because that is a break change.
Thanks for your help but I will close this issue for now.


fix vGPUmonitor deviceidx is always 0

What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
deviceidx is always is zero.

Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@archlitchi PTAL


Private
Which package or tool is having this issue?
Oxygen
What version of that package or tool are you using?
~2023.10.3 (see Package.json below for versions)
What version of Remix are you using?
See Package.json below for versions
Steps to Reproduce

Ensure actions installed for Shopify for Hydrogen app
Check in code to Github (all permissions Shopify requested granted)
Examine action result
Fails on step "Add dependencies".
Does not publish.

Code is very close to the demo-store on the Hydrgen GitHub page.
Expected Behavior
I expect that the website, which runs fine locally to publish to Oxygen on commit via GitHub.
Since it has published fine before, I did not expect any issues.
However, I have since upgraded the project to closely match that latest demo-store code.
If it works locally, it should work (the same) when published. Any differences I personally would consider a bug and hope you guys would agree too.
Actual Behavior
Examining the GitHub action when deploying to oxygen, which failed at "Add dependencies" stage.  Error message below...
Run yarn
yarn install v1.22.21
info No lockfile found.
[1/[5]([URL]#step:6:6)] Validating package.json...
error hydrogen@2.1.[6]([URL]#step:6:7): The engine "node" is incompatible with this module. Expected version ">=1[8]([URL]#step:6:9).0.0". Got "16.20.2"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
Error: Process completed with exit code 1.

(note I have replace the potentially sensitive job urls with [URL].
Notes

In package.json engines is indicated as "node": ">=18.0.0"
Local version of node is 18.14.0 (same in project folder).
Not sure where it is getting node version 16.20.2 from at all
Tried deleting node_modules repeatedly
Tried npm audit fix and force to no avail
Tried clearing npm cache folders
Have previously successfully published so not sure what's changed
Also tried installing yarn for the first time, but it couldn't even run the project with yarn but npm worked fine.
I have uninstalled yarn from my system back to only using npm
Removed package-lock.json (after previous error mentioned mixing package managers)

My package.json
{
  "name": "hydrogen",
  "private": true,
  "sideEffects": false,
  "version": "2.1.6",
  "scripts": {
    "dev": "shopify hydrogen dev --worker --codegen",
    "build": "shopify hydrogen build",
    "preview": "npm run build && shopify hydrogen preview --worker",
    "e2e": "npx playwright test",
    "e2e:ui": "npx playwright test --ui",
    "lint": "eslint --no-error-on-unmatched-pattern --ext .js,.ts,.jsx,.tsx .",
    "format": "prettier --write --ignore-unknown .",
    "format:check": "prettier --check --ignore-unknown .",
    "typecheck": "tsc --noEmit"
  },
  "browserslist": [
    "defaults"
  ],
  "prettier": "@shopify/prettier-config",
  "dependencies": {
    "@headlessui/react": "^1.7.2",
    "@remix-run/react": "2.1.0",
    "@remix-run/server-runtime": "2.1.0",
    "@shopify/cli": "3.52.0",
    "@shopify/cli-hydrogen": "^6.1.0",
    "@shopify/hydrogen": "~2023.10.3",
    "@shopify/remix-oxygen": "^2.0.2",
    "clsx": "^1.2.1",
    "cross-env": "^7.0.3",
    "graphql": "^16.6.0",
    "graphql-tag": "^2.12.6",
    "isbot": "^3.6.5",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "react-intersection-observer": "^9.4.1",
    "react-use": "^17.4.0",
    "schema-dts": "^1.1.0",
    "tiny-invariant": "^1.2.0",
    "typographic-base": "^1.0.4",
    "js-cookie": "3.0.5"
  },
  "devDependencies": {
    "@playwright/test": "^1.40.1",
    "@remix-run/dev": "2.1.0",
    "@remix-run/eslint-config": "2.1.0",
    "@shopify/eslint-plugin": "^42.0.1",
    "@shopify/oxygen-workers-types": "^4.0.0",
    "@shopify/prettier-config": "^1.1.2",
    "@tailwindcss/forms": "^0.5.3",
    "@tailwindcss/typography": "^0.5.9",
    "@total-typescript/ts-reset": "^0.4.2",
    "@types/eslint": "^8.4.10",
    "@types/react": "^18.2.22",
    "@types/react-dom": "^18.2.7",
    "cross-env": "^7.0.3",
    "eslint": "^8.20.0",
    "eslint-plugin-hydrogen": "0.12.2",
    "postcss": "^8.4.21",
    "postcss-import": "^15.1.0",
    "postcss-preset-env": "^8.2.0",
    "prettier": "3.2.4",
    "rimraf": "^3.0.2",
    "tailwindcss": "^3.3.0",
    "typescript": "^5.2.2",
    "postcss-nesting": "11.2.2",
    "prettier-plugin-tailwindcss": "0.5.11",
    "react-icons": "5.0.1"
  },
  "engines": {
    "node": ">=18.0.0"
  }
}

Obviously need to be able to publish so would appreciate any help.
Thanks,
Dan.

Hi Dan, the package.json file I don't think determines what version of Node is run within the action runner. Instead look inside your Github workflow yml file. For example, our workflow says the node version should come from .nvmrc

Hi Dan, the package.json file I don't think determines what version of Node is run within the action runner. Instead look inside your Github workflow yml file. For example, our workflow says the node version should come from .nvmrc

You're right @blittle - there is some sort of weird yml file thing with node 16 and yarn inside.  But I have not the first idea how to edit such a document to e.g. use npm instead of yarn.  Is there a way to via Shopify e.g. CLI to get this updated or do I need to learn about these files?  How do I do it so I don't break the Oxygen server?
Do you know where on the Shopify (Hydrogen) website there exists documentation on this topic?  I can't seem to find any which might suggest I shouldn't be touching it.
So - not sure if bug i.e. it should be automatically updated based on current package.json environment, that I need to run a safe CLI command to refresh or that everyone who uses Hydrogen at some point will be forced to start editing this file to make sure up to date e.g. expected version of node
Maybe I'll just upgrade the node version and see if that fixes things.  Would be great if it used npm instead of yarn though.
Confusedly yours,
Dan.

Thanks again @blittle.  Between your example and some help from a popular AI tool, it's now working.
To resolve...

Changed the version of node from 16 to 18 (to match the one hydrogen uses).  I'll take a look at nvmrc at a later date once I've obviously learned some more about CI & GitHub.
To edit the yml file (in my case to move from a very old version of yarn to npm), this can be found in your project under the folder .github\workflows (easy to miss - I didn't realise it was even in my project).  Update to use npm (ask ChatGPT to do this for you while checking the current version - edit sensibly).  If you get an error about npm build, add npm run build instead.
In GitHub Actions you can see your action success (or failure) results which gives details of any issues.  Good error messages make it easy to diagnose issues.

Some documentation does exist and can be found here.
Action needed? Maybe consider adding to a troubleshooting page (add to failing oxygen deployments section) e.g. my project doesn't publish.  Then describe how to access Github actions, check errors and give example of node version as a change.  Perhaps with reasons why someone may wish to edit this file at all e.g. if it can do something helpful in common scenarios or might need to be updated periodically and if so what's likely needing to be updated.
Thanks,
Dan.


Please share vulnerable drivers on Virustotal

Hi Microsoft,
To perform some additional research and generate more Indicators of Compromise, it would be nice if you could share the source files with platforms such as Virustotal.
Right now, some of the items in the XML have zero hits on virustotal, so it is hard to analyze them .
E.g.
WinIO32B.sys
netfilterdrv.sys
bandai.sys
etc.
It would help the security community if you could share all files, at least on virustotal.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 6e8740b8-c1f2-4b88-3d4d-66cdbf20215f
Version Independent ID: 7986a684-b35d-06bd-be30-dfca9923cb9c
Content: Microsoft recommended driver block rules (Windows)
Content Source: windows/security/threat-protection/windows-defender-application-control/microsoft-recommended-driver-block-rules.md
Product: windows-client
Technology: itpro-security
GitHub Login: @jgeurten
Microsoft Alias: vinpa

Thanks @goosvorbook for taking the time to provide feedback on Windows! This idea is an interesting feature, and I want to make sure it gets directly into the hands of the engineering team. Fortunately, we've got dedicated channels for that very purpose. :) For more info, see Send feedback to Microsoft with the Feedback Hub app. I think it’s important that you submit it directly, because that allows you to receive notifications and more closely monitor the progress. I'm closing this issue as there's nothing actionable for docs at this time.


cls

Environment

Spring boot starter:

active directory spring boot starter
documentdb spring boot starter
key vault spring boot starter
media services spring boot starter
service bus spring boot starter
storage spring boot starter

OS Type: Windows/Linux/MacOS

Java version:

1.9
1.8
1.7

Summary

Reproduce steps

Expected Results

Actual Results

Accidentally raised this issue. Apologies.


Delayed variable evaluation until execution time

I am trying to find a solution to read the content of a file into an attribute during execution time. It is quite straight forward to do this during compile time using backticks or read the file in a ruby block.
My use case is:
Recipe 1.) Calls a process using a bash_block which produces a files with "special data" inside
Recipe 2.) Read the special data file and use the content read inside a template
So far I tried:
Backticks
specialData=`cat /tmp/specialData`

Problem:
Processed at execution time before the file is created
RubyBlock
Problem:
Reading the property in the ruby block works well enough, yet I haven't found a way to pass the value outside the ruby block. I tried using node.set['specialData']=... but this only seem to work inside the ruby bock.
Lazy
datafile=" /tmp/specialData"
template "/tmp/output" do
  source "output"
  variables(
    :specialData => lazy do
      if File.exist?(datafile)
        File.open(datafile, &:readline).chomp
      end
    end
  )
end

Problem:
Rather then the content read it shows Chef::DelayedEvaluator in the output file
Chef::DelayedEvaluator:0x00000003f519c0@/var/chef/cache/cookbooks/test/recipe.rb:229
Lambda
This method was suggested : https://tickets.opscode.com/browse/CHEF-1814
datafile=" /tmp/specialData"
template "/tmp/output" do
  source "output"
  variables(
    :specialData => lambda { `cat /tmp/specialData` }
  )
end

Problem:
Rather then the content read it shows Proc in the output file
Proc:0x000000028cd9b0@/var/chef/cache/cookbooks/test/recipe.rb:332 (lambda)
We are not able to generate the special data file using chef as this process is too complex to port, so we have to call the script which will create the special data file during the execution of recipe 1.
We are using chef version 12.14.89 on linux

Sorry this has sat idle for so long.
The syntax for lazy is:
template "foo" do
  variables( lazy {
    :specialData => begin
        if File.exist?(datafile)
          File.open(datafile, &:readline).chomp
        end
    end
  })
end

Does that work for you? In general I'd recommend posting to discourse or stack overflow for this kind of question, though.

probably needs to be more like this:
variables(lazy {
  h = {}
  h[:specialData] =
    if File.exist?(datafile)
      File.open(datafile, &:readline).chomp
    end
  h
})

otherwise i think you still wind up with a proc/lazy object as the hash value.

basically, you need to put a lazy around a function that returns a hash, not around a hash that contains lazy functions.

although actually you could use lambda, but then you need to .call the lambda in your template code (it is just code after all in the template resource).
^^^ this may actually be the best solution and CHEF-1814 may have been on the right track, but you can't do it without modifying your template to be lambda/proc-aware.

and this does now fall under a StackOverflow or Slack kind of question.  i don't think there's a bug in chef to fix here, its all just beating on ruby and chef repetitively until you get the right answer.


Add features to reduce dependencies

With my team we are evaluating httpmock; it really looks like a great project.
Anyway, our test compile-time increased substantially due to its number of dependencies.
Would it maybe be possible to reduce the dependencies using cargo features? For example, I guess isahc and some other "client-side" libraries are used only with a remote server; could they be made optional and enabled with a remote feature?

Thanks for creating this issue! By looking at the dependency tree, I think it should be possible to reduce the number of dependencies that are required for regular usage (i.e., non-standalone mode) by about 30%. Most dependencies seem to be added by isahc and async-std (coming in though async-object-pool). I'll look into it.

Making isahc optional would also let users remove OpenSSL from their dependency tree so their tests can work in rust-cross.

Could isahc be potentially replaced by hyper's client capabilities, seeming as the code, at least in the ping case, is async? (see src/api/adapter/mod.rs#L84-L108)


WiP Unmarshalling Error: unexpected element when invoking a service with

multiple parameters without explicit wrapper
This is an attempt to f.i.x #582. While the Unmarshalling Error: unexpected element goes away, the fix unfortunately causes another issue with unexpected namespace declaration on elements that pops up in nearly all other tests. So this PR is perhaps only useful to show what does not work. Any ideas to f.i.x #582 are welcome!

Replaced by https://github.com/quarkiverse/quarkus-cxf/pull/791


Changed value type to CodeableConcept for TNMcpuPraefix to match specification

According to the specification of the extension the value type must be CodeableConcept. This changes the type from Coding to CodeableConcept.

Thank you!


[Suggestion] Release lower level Pokemon of already caught Pokemon

Expected Behavior
Running the bot with parameter -c 150, it captures a Zubat with CP 160. I already have one with CP 170. Bot exchanges the Zubat with CP 160 even though i run it with parameter -c 150.
Actual Behavior
The bot keeps all Pokemon above CP 150, so many of them double or triple
Steps to Reproduce
Just run the bot

Being implemented right now at https://github.com/PokemonGoF/PokemonGo-Bot/pull/274. Because this is a duplicate I'll close it.


Object types do not support ABAP language version keyUser

Following object types actually do not support ABAP language version keyUser, but specify this in their enum values:

[x] CHKO
[x] CHKC
[x] CHKV
[ ] NROB

again, I'd argue to remove the language version from the object definitions, what if NROB sometime is supported in keyUser? Then the object definition must be changed, but its just another runtime that supports the object

the file format does not need to directly reflect the implementation in the system, eg. specific names and/or fields
🎪

the file format does not need to directly reflect the implementation in the system, eg. specific names and/or fields

Sorry, I don't understand what you mean

Also fixed with https://launchpad.support.sap.com/#/notes/3138884
This Note has been already linked in https://github.com/abapGit/abapGit/issues/5279


Correct status style for example app on readme

Thank you for sharing useful library.
I tried example app on readme but it failed because the status style was old.
I updated the style.

Hi @asukiaaa , thanks for your pull request!
Iron is currently being worked on to update to hyper 0.12. The example in the readme reflects the current iron code in the master branch of this repository. I think this should stay this way for people trying to use the code directly from this repository.
For crate-release specific examples, the corresponding documentation – currently https://docs.rs/iron/0.6.0/iron/ – should be consulted.
As soon as iron 0.7 is released, the readme and the published crate code should match again.


Add request/responce types for listVersions API

Add Versions request and responce types.
Added VersionDependency, VersionInfo and VersionInfos types, which will be used in List versions API.
Relates-To: OLPEDGE-1606
Signed-off-by: Liubov Didkivska<EMAIL_ADDRESS>

Please rename getters and setters according to openapi spec.


permission denied on CentOS - Podman emulator

Emulate Docker CLI using podman. Create /etc/containers/nodocker to quiet msg.
Trying to pull ghcr.io/pg9182/northstar-dedicated:1-tf<IP_ADDRESS>...
Getting image source signatures
Copying blob a99c34865d99 skipped: already exists
Copying blob 891382f67258 skipped: already exists
Copying blob 617ce86479e7 skipped: already exists
Copying blob 59bf1c3509f3 skipped: already exists
Copying blob 1d676022bd00 skipped: already exists
Copying blob aca1971a86b4 skipped: already exists
Copying blob d8e612b3ace8 skipped: already exists
Copying blob c49e47170caa skipped: already exists
Copying config 760369964c done
Writing manifest to image destination
Storing signatures
Northstar Dedicated Server - Docker (hostname: f7938bcd6dfe)
https://northstar.tf
https://northstar.tf/discord
https://github.com/R2Northstar/Northstar
https://github.com/pg9182/northstar-dedicated

Merging files...
Error: Failed to merge game files: merge titanfall: access "/mnt/titanfall/build.txt": stat /mnt/titanfall/build.txt: permission denied

CentOS 9 Stream

Check your permissions on the mounted titanfall directory on the host.

I am going to close this issue since there hasn't been a response. Feel free to re-open it if you are still having trouble.


Mule Soft Application Image building failing

STEP 1: FROM image-registry.openshift-image-registry.svc:5000/openshift/mule4@sha256:155d7b50801d6a58a408b89d29f9aec163de782881508c97832596a276ca845e
STEP 2: LABEL "io.openshift.build.image"="image-registry.openshift-image-registry.svc:5000/openshift/mule4@sha256:155d7b50801d6a58a408b89d29f9aec163de782881508c97832596a276ca845e"       "io.openshift.build.source-location"="/tmp/build/inputs"       "io.openshift.s2i.destination"="/deployments"
STEP 3: ENV OPENSHIFT_BUILD_NAME="sample-app-2"     OPENSHIFT_BUILD_NAMESPACE="openshift"
STEP 4: USER root
STEP 5: COPY upload/src /deployments/src
STEP 6: RUN chown -R 185:0 /deployments/src
STEP 7: USER 185
STEP 8: RUN /usr/local/s2i/assemble
: invalid option
error: build error: error building at STEP "RUN /usr/local/s2i/assemble": error while running runtime: exit status 1
error: the build openshift/sample-app-2 status is "Failed"

@Charbak, are you running this on Windows? I normally see that error when a script is copied into the container from Windows and the CRLF line ending are not compatible with Linux inside the container. Some code editors allow you to fix this on Windows, I believe.

@sohrab- I am using OC clinet running on windows 10 and pushing that to OCP cluster running externally to IBM cloud.
For application image creation, I cant see Docker file. From where it is getting the steps when executing
Build the image!
oc start-build buildConfig/sample-app --from-file=./target/sample-app-1.0.0-SNAPSHOT-mule-application.jar --wait --follow

@sohrab- I converted build.YAML and deploy.YAML under /openshift folder  to LF through Visual Studio. But still it is getting stuck
STEP 6: RUN chown -R 185:0 /deployments/src
STEP 7: USER 185
STEP 8: RUN /usr/local/s2i/assemble
: invalid option
error: build error: error building at STEP "RUN /usr/local/s2i/assemble": error while running runtime: exit status 1
error: the build openshift/sample-app-5 status is "Failed"
Can you pls highlight from where it is taking the steps. I cant find docker file under sample-app folder.

@sohrab- I again build the image and again trigger to create application image after converting all YAML to LF. It fails with below error. Can you check now.
STEP 8: RUN /usr/local/s2i/assemble
---> Moving Mule App to deployment directory...
mv: cannot move '/deployments/src/sample-app-1.0.0-SNAPSHOT-mule-application.jar' to '/opt/mule/apps/': No such file or directory
error: build error: error building at STEP "RUN /usr/local/s2i/assemble": error while running runtime: exit status 1
error: the build openshift/sample-app-6 status is "Failed"

Sorry, I think my instructions weren't clear.

Update only assemble script to convert the line ending
Rebuild the base image again
Build mule app and run OpenShift build

There is no Dockerfile because this using s2i.

@sohrab- Already did. But assemble script is failing.
Adding transient rw bind mount for /run/secrets/rhsm
STEP 1: FROM image-registry.openshift-image-registry.svc:5000/openshift/mule4@sha256:39e53fa55ac6bfb8fd4568fe3baca4f48b22be1240da10d15b87528010864832
STEP 2: LABEL "io.openshift.build.image"="image-registry.openshift-image-registry.svc:5000/openshift/mule4@sha256:39e53fa55ac6bfb8fd4568fe3baca4f48b22be1240da10d15b87528010864832"       "io.openshift.build.source-location"="/tmp/build/inputs"       "io.openshift.s2i.destination"="/deployments"
STEP 3: ENV OPENSHIFT_BUILD_NAME="sample-app-6"     OPENSHIFT_BUILD_NAMESPACE="openshift"
STEP 4: USER root
STEP 5: COPY upload/src /deployments/src
STEP 6: RUN chown -R 185:0 /deployments/src
STEP 7: USER 185
STEP 8: RUN /usr/local/s2i/assemble
---> Moving Mule App to deployment directory...
mv: cannot move '/deployments/src/sample-app-1.0.0-SNAPSHOT-mule-application.jar' to '/opt/mule/apps/': No such file or directory
error: build error: error building at STEP "RUN /usr/local/s2i/assemble": error while running runtime: exit status 1
error: the build openshift/sample-app-6 status is "Failed"

@sohrab- I fixed the issue this way. I added creation of the directory in docker file. Rebuilt the base image and run the application image and it works...
RUN \
  mkdir -p ${MULE_HOME}/apps && chown -R 185:root ${MULE_HOME}/apps \
  # Install Mule server
  && unzip /tmp/mule-ee-distribution-standalone-${MULE_VERSION}.zip -d /opt \
	&& ln -s /opt/mule-enterprise-standalone-${MULE_VERSION} ${MULE_HOME} \

But getting the exception as accessing the application.
/usr/local/s2i/run: line 15: /opt/jboss/container/maven/default//scl-enable-maven: No such file or directory
Starting the Java application using /opt/jboss/container/java/run/run-java.sh ...
/opt/run-java-options: line 2: $'\r': command not found
/opt/run-java-options: line 4: syntax error near unexpected token $'{\r'' /opt/run-java-options: line 4: wrapper_config() {'
INFO exec -a 'mule' java -javaagent:/usr/share/java/jolokia-jvm-agent/jolokia-jvm.jar=config=/opt/jboss/container/jolokia/etc/jolokia.properties -Xms94m -Xmx375m -XX:+UseParallelOldGC -XX:MinHeapFreeRatio=10 -XX:MaxHeapFreeRatio=20 -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90 -XX:MaxMetaspaceSize=256m -XX:MetaspaceSize=128m -XX:+ExitOnOutOfMemoryError -cp ".:/opt:/opt/mule-licence:/opt/mule-app-properties:/opt/mule/conf:/opt/mule/lib/boot/*" org.mule.runtime.module.reboot.MuleContainerBootstrap
/opt/run-java-options: line 2: $'\r': command not found
/opt/run-java-options: line 4: syntax error near unexpected token $'{\r'' /opt/run-java-options: line 4: wrapper_config() {'
Error: Could not find or load main class org.mule.runtime.module.reboot.MuleContainerBootstrap
I am using "mule-ee-distribution-standalone-4.3.0-20210224.zip" as runtime. I cant find exact "mule-ee-distribution-standalone-4.3.0-20210322.zip" in the distribution list. Is the problem due to that?

@sohrab- Can you pls see about this error after rollout deployment
Error: Could not find or load main class org.mule.runtime.module.reboot.MuleContainerBootstrap


[bug_5548] Added feature: mean time starting and ending time work.

Proszę o review.
https://gist.github.com/MarcinWyt/42459ea6103a1a38b7bb
https://bugzilla.stxnext.pl/show_bug.cgi?id=5548

Squash please.

Squash please.

Squash please.

Change commit name. Some bugs removed. doesn't give any information about new view.


Planning: the demo agenda

Digest https://github.com/apigee/registry-experimental/issues/187 into individual issues.

See https://github.com/apigee-apihub-demo/apigee-apihub-demo.github.io/issues


WIP net: tcp: Explicitly manage TCP receive window.

Challange the existing situation that "if application buffers data,
it's the problem of application". It's actually the problem of the
stack, as it doesn't allow application to control receive window,
and without this control, any buffer will overflow, peer packets
will be dropped, peer won't receive acks for them, and will employ
exponential backoff, the connection will crawl to a halt.
This patch adds net_context_tcp_recved() function which an
application must explicitly call when it processes data, to
advance receive window.
Jira: ZEP-1999
Change-Id: Id7255df3d4898e289a2d20e7a02fd5f3f8f05291
Signed-off-by: Paul Sokolovsky<EMAIL_ADDRESS>

This is trivial implementation of TCP receive window handling, to back discussion on how to do it properly in JIRA: https://jira.zephyrproject.org/browse/ZEP-1999 and on mailing list: https://lists.zephyrproject.org/pipermail/zephyr-devel/2017-April/007621.html

Maybe @andyross will want to take a look here.

@lpereira : Thanks for the ping. I actually wait for continuation of @andyross' review on https://github.com/zephyrproject-rtos/zephyr/pull/26 , and there's another which warrants his look ahead of this as it touches code written by him: #137.

Really not liking this API.  For one, it's asymmetric with respect to other net_context implementations.  You need to know that your context is a TCP connection and use it differently than you do other types.  Secondly, it's non-standard: other OSes don't do anything like this.
Third, I don't see value it provides.  The only reason TCP windows exist (for our purposes, anyway) is to prevent buffer overflows in the receiver.  I don't see how this helps the app do that.
If you want a way for the app to explicitly set windows, then (1) make it optional and (2) make it abstract.  How about something like Linux's TCP_WINDOW_CLAMP sockopt instead?

You need to know that your context is a TCP connection and use it differently than you do other types.

You need to know the type of a socket - STREAM or DGRAM to work with it properly.

Secondly, it's non-standard: other OSes don't do anything like this.

What OSes do you mean here? Those which can no longer work with a gigibyte of RAM, all those linuxes, windowses, and macosxes? The mechanism presented in this patch is a direct translation of the same mechanism of lwIP, which is not an OS, but a portable reusable IP stack which works with almost any RTOS out there, a golden standard in embedded IP stacks, used in probably 80% of projects, tested and tried for a decade or two.
All these points were given in detail in the RFC which is by the link given above: https://lists.zephyrproject.org/pipermail/zephyr-devel/2017-May/007637.html .
Note that in further discussion of that IRC, I subscribed to a research of doing it "how big OSes do": https://lists.zephyrproject.org/pipermail/zephyr-devel/2017-May/007637.html . But until that code is written and tested to give the needed effect, the simple changes presented in this patch are the only ones needed to correctly manage TCP control flow.

The only reason TCP windows exist (for our purposes, anyway) is to prevent buffer overflows in the receiver. I don't see how this helps the app do that.

Please see https://jira.zephyrproject.org/browse/ZEP-1999?focusedCommentId=18206&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-18206

what is the status of this one?

Can this be closed?

Thanks for removing 1.9 milestone. As this is a serious issue (blocker for large TCP xfers), I'd prefer to close this when an alternative patch will be available (which I hope to work on soon).

Superseded by https://github.com/zephyrproject-rtos/zephyr/pull/1002


remove need of @types/mailgen

add types directly to main package (removing the need of installing @types/mailgen)

@italodeandra Awesome again, thanks! 💯

Published as 2.0.15.


Add User Registration

Gather 3 fields upon registration.

Username
Email Address
Display Name (optional)

Upon creation a unique token string will be created for TOTP authentication. This will get emailed to them upon registration.

User Model:
{
username: String (required) (unique)
email: String (required)
displayName: String (optional)
token: String (required) (default: randomToken())
created: Date (required) (default: Date.now())
}


popover-append-to-body doesn't append to body

Hi,
popover-append-to-body setting doesn't append the created element to body, but rather it is still nested within the tooltip element... I seem to not be able to get z-indexes right. I have several tooltipped elements on vertical row. If I have set z-index for the tooltipped element and its being lower than tooltip's elements z-index, the tooltip stays behind the tooltipped element.

Mr. Kozlowski,
I forked your plnkr to the more recent:

angularjs. 1.3.15
angular-ui 0.13.0
bootstrap 3.3.4

It appears that popover-append-to-body no longer works in your example.  Am I using an incompatible set of versions? Or is this now a bug?  If so, I'll file a new bug.

Ross


Formalize the API Group in APIVersion

Motivation
Current API Version is mumoshu.github.io/v1alpha1, but this doesn't represent the current state of the project well.
Feature
Change the API Group to something like falco.security.cncf.io or whatever would be consistent with other CNCF projects. The sooner thing gets in the smoother the future updates will be.

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.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

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.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale


Trigger BeforeSave executes, however new data is not saved in database fields.

hi
Following Getting Started, everything appears to be correct in my implementation, including the fact that triggers are executed.
However, new data is not saved in the database.
Details
Trigger
public class EmpresasSetChangedAtTrigger : IBeforeSaveTrigger<Empresa>
    {

        public Task BeforeSave(ITriggerContext<Empresa> context, CancellationToken cancellationToken)
        {

            if (context.ChangeType == ChangeType.Modified)
            {
                context.Entity.ChangedAt = DateTimeOffset.Now;
            }

            return Task.CompletedTask;
        }

    }

Register DbContext in Startup
services.AddDbContext<TenantDbContext>((serviceProvider, optionsBuilder) =>
            {
                var CurrentTenant = serviceProvider.GetRequiredService<ICurrentTenant>();

                var TenantConnString = MultiTenantUtils.TenantConnStringFactory(CurrentTenant, _configuration);

                optionsBuilder.UseSqlServer(TenantConnString);

                optionsBuilder.UseTriggers(triggerOptions =>
                {
                    var TriggerAssembly = CurrentAssemblyUtils.GetCurrent();
                    triggerOptions.AddAssemblyTriggers(TriggerAssembly);
                });

            })
            //.AddScoped<IBeforeSaveTrigger<Nfe>, NfeSetChangedAtTrigger>()
            //.AddTransient<IBeforeSaveTrigger<Empresa>, EmpresasSetChangedAtTrigger>()
            ;

Any Tips ?

Hi @victorperez2911
So I assume that you're referring to the ChangedAt property, not getting persisted in the database?
I don't see a particular issue with your code. It may be that you have your mapping is incorrectly configured or that you have another trigger that Unsets the ChangedAt property.
Can you share the log output of when saving an Entity?
In appsettings.json:
"Logging": {
    "LogLevel": {
      "EntityFrameworkCore.Triggered.TriggerSession": "Information",
      "Microsoft.EntityFrameworkCore": "Information"
    }
  }

And in your startup class:
services.AddDbContext(optionsBuilder => {
    optionsBuilder.UseSqlServer(...);
    optionsBuilder.EnableSensitiveDataLogging(true);
})

This should show you that the trigger runs and that the Update statement includes the new timestamp.
Thanks

Hi
More details

I have only one Trigger.

And a found the root cause of the issue, the particular property is from a model base classe inheritance.

Implementing the Trigger from the model base class also does not work.

I'm using properties from base models currently with no issues. Can you share your logs as suggested before or ideally, share a repro of this issue? Thanks!

Problem solved.
We used INotifyPropertyChanged and INotifyPropertyChanged as change detection strategy in the Entity Framework and the base class did not implement the correct interface, and also did not raise the correct event in this property (ChangedAt) .
Thanks.


Why are some items missing from data.json?

Hi, thank you for this great work!
I have noticed some inconsistencies in the data.json file of the metagui dataset. Specifically, in the dev split:

There's no data dict associated with dataset/dev/dialog_0/turn_0/5.png
dataset/dev/dialog_0/turn_0/7.png is absent in the screenshot_history for that turn

Could you please clarify if these omissions were intended?
Thank you.

Sorry for replying so late, and thank you for your interest in our work.
These omissions are normal. That is because during annotation, we annotate some some action types that were not used during the training process. Also, due to some device errors, the view hierarchies for some screenshots are absent, which we deleted from the dataset.
Your can read this code for more information.


Allow pip-audit to report the current version of the vuln database

Is your feature request related to a problem? Please describe.
I'm integrating pip-audit with our build tool, Pants. Rather than run the tool every time, I'd prefer to run it again only when new vulnerabilities are discovered.
Describe the solution you'd like
I'd like to be able to call something like pip-audit --db-version and get back a unique identifier for the current version of the database (something like the etag for https://github.com/pypa/advisory-database/tree/main/vulns would work)
Describe alternatives you've considered

I can check the etag for https://github.com/pypa/advisory-database/tree/main/vulns myself every time
I can run pip-audit every time and ignore the potential performance improvements of cached results

Thanks for the request!
Something like this would be interesting, but I think it'd be difficult to do in the current form offered by our backing vulnerability services (PyPI and OSV): both pull from multiple vulnerability DBs and may have updates pushed to them at any point from multiple backing sources.
In particular, checking the ETag for the PyPA advisory database will probably be insufficient, since it's only one of several sources used for the vulnerabilities reported through the PyPI JSON API.
Could you say some more about your use case? pip-audit already does some internal caching of requests (and reuses pip's own cache), so repeated audits of the same dependency source should be relatively performant (especially if your dependencies are fully pinned.)

Closing as not currently actionable: not all vulnerability services are versioned, and our pre-existing caching is performant enough in the pinned deps case.


What is the region of the Cloud Shell?

Please be more precise, since this statement is missing an important information:
"Azure Cloud Shell has a limit of 20 concurrent users per tenant per region."
According to the Azure Products Site, the Cloud Shell is a non-regional service. Therefore it would be great to know, how the "region" in the sentence before is determined. Does it may align with the storage account which is the foundation for every Cloud Shell? If that´s the case, it would be great to clearly state that out - if not - please specify what the "region" is in that particular scenario.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8f7b8330-007e-d13b-ce2e-c9f1dd9ea06b
Version Independent ID: ace5ecf4-2021-63a7-c28f-73a11767f0be
Content: Azure Cloud Shell troubleshooting
Content Source: articles/cloud-shell/troubleshooting.md
Service: azure
GitHub Login: @maertendMSFT
Microsoft Alias: damaerte

@OliCSADoerr -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

#label:"cloud-shell/svc"

#reassign:sdwheeler


JSONPath Query results always have [ ] around them

Hi,
Templates that have JSONPath queries are not being properly returned. Some square bracket characters and quotes are being appended to the result. Maybe there is a missing step in the parsing of the JSONPath result?
The RML:

@prefix rr: http://www.w3.org/ns/r2rml#.
@prefix rml: http://semweb.mmlab.be/ns/rml#.
@prefix ex: http://example.com/.
@prefix xsd: http://www.w3.org/2001/XMLSchema# .
@prefix ql: http://semweb.mmlab.be/ns/ql# .
<#SubjectMapping> a rr:TriplesMap;
rml:logicalSource [
rml:source "simpleTestInput.json";
rml:referenceFormulation ql:JSONPath;
rml:iterator "$"
];
rr:subjectMap [
rr:template "http://example.com/Child/{first}/{last}";
rr:class ex:Child
];
rr:predicateObjectMap [
rr:predicate ex:BSN;
rr:objectMap [
rr:template "ex:{$..[?(@['ver']==1) ]['val']}";
];
].

The Json being mapped:

[{
"first"	: "Amarise",
"last"	: "Fernand",
"loves"	: "milk",
"birthday" : "2017-02-16",
"language"	: "Nederlands",
"age"	: "5",
"length"	: "5.5",
"BSN"	: {    "ver":1,
"val":"AaX"
},
"BSN2"	: {    "ver":2,
"val":"BbX"
},
"parents": "Gea and Hildebrand",
}]

The result:

(http://example.com/Child/Amarise/Fernand, http://www.w3.org/1999/02/22-rdf-syntax-ns#type, http://example.com/Child) [null]
(http://example.com/Child/Amarise/Fernand, http://example.com/BSN, ex:%5b%22AaX%22%5d) [null]

Notice the encoded characters around AaX
Thanks for any feedback!

Hi @syats,
The problem here is that the result of a JsonPath filter is always a list.
If you were to use "$.BSN.val" the result would be as you expected.
There doesn't seem to be a way to get a value out of a filtered list.
See also: https://github.com/json-path/JsonPath/issues/272
I guess this could be useful when you know for certain that a list can be filtered down to a singular item.
If you would really need to select items based on a filter like this you would have to move the filter in to a specific iterator and create a triples map specifically for creating triples based on that iterator.
Hope this helps.
Pano

Thanks @pmaria !
The RML spec isn't really clear about how to handle this case.
My suggestion would be that if the result of a JSONPath expression is a list, then the mapping gets applied to each element of the list. This is consistent with the evaluation of template "http://loc.example.com/city/{$.venue[*].location.city}" done in section 9.2 of the spec.
Do you think this could be implemented into carml?
(If you give me a pointer I can try implementing it myself and make a PR)
Victor

Hi @syats,
Interesting idea. Do you have a lot of datasets where you would need such functionality?
Currently, as RML is basically a direct abstraction of R2RML, and this isn't part of R2RML, it isn't in RML either.
So this would have to be a new construct. Something like a MultiTermMap that requires multiTemplate and multiReference perhaps. But this would have to be investigated further.
As we're currently not involved in the RML spec development I hesitate. We could add and develop such a construct as an extension in CARML, although since this is quite a fundamental extension, preferrably, this would be something to discuss with the RML spec developers.


Smarter thunder.jit decisions

Adds support for:
Have the user call thunder.jit but still use the strategy
fabric = Fabric(strategy=ThunderFSDPStrategy())
model = MyModel()
model = thunder.jit(model)
model = fabric.setup(model)  # this is now smart enough to know that model was already jitted

PoC:
import os
import thunder
import torch
import torch.distributed as torch_dist

world_size = int(os.environ.get("WORLD_SIZE", 1))
local_rank = int(os.environ.get("LOCAL_RANK", 0))
global_rank = int(os.environ.get("RANK", 0))
if world_size > 1:
    torch_dist.init_process_group(backend="nccl")
    pg = torch_dist.distributed_c10d._get_default_group()
device = torch.device("cuda", local_rank)
torch.cuda.set_device(device)

model = torch.nn.Linear(5, 10, bias=False, device=device)
x = torch.randn(2, 5, device=device)

def fwd_loss(m, x):
    return m(x).sum()

model = thunder.jit(model)
model._lc_cd.fn = thunder.distributed.fsdp(model._lc_cd.fn)

out = fwd_loss(model, x)

print(out)
if local_rank == 0:
    print("FN", thunder.last_traces(model)[-1].python())

Have the user compile an arbitrary function that includes the model
def fwd_loss(m, x):
    return m(x).sum()

fabric = Fabric(strategy=ThunderFSDPStrategy(jit=False))
model = MyModel()
model = thunder.jit(fwd_and_loss)
model = fabric.setup(model)
fwd_and_loss(model, ...)

Thunder doesn't support jitting twice here, so the user needs to disable the strategy's jit call since fabric doesn't know anything about fwd_and_loss
PoC:
import os
import thunder
import torch
import torch.distributed as torch_dist

world_size = int(os.environ.get("WORLD_SIZE", 1))
local_rank = int(os.environ.get("LOCAL_RANK", 0))
global_rank = int(os.environ.get("RANK", 0))
if world_size > 1:
    torch_dist.init_process_group(backend="nccl")
    pg = torch_dist.distributed_c10d._get_default_group()
device = torch.device("cuda", local_rank)
torch.cuda.set_device(device)

model = torch.nn.Linear(5, 10, bias=False, device=device)
x = torch.randn(2, 5, device=device)

def fwd_loss(m, x):
    return m(x).sum()

fwd_loss = thunder.jit(fwd_loss)
model = thunder.distributed.fsdp(model)

out = fwd_loss(model, x)

print(out)
if local_rank == 0:
    print("FN", thunder.last_traces(fwd_loss)[-1].python())

DDP is blocked until https://github.com/Lightning-AI/lightning-thunder/issues/94 is resolved


Small Correction - missing the letter "n"

Small Correction - missing the letter "n"
For servicePrincipals that correspond to a managed identity. Managed identities service principals can't be deleted in the Enterprise apps blade. You need to go to the Azure resource to manage it. Lear more about Managed Identity

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ed4aac4d-9b31-0161-0d57-e53d7bee0da9
Version Independent ID: 5bcb35ff-c2ba-a92f-4104-2b0a5677e911
Content: Troubleshoot common problem adding or removing an application to Azure Active Directory
Content Source: articles/active-directory/manage-apps/troubleshoot-adding-apps.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @kenwith
Microsoft Alias: kenwith

@jmstrupp-psjh
Thanks for your feedback! We will investigate and update as appropriate.

@jmstrupp-psjh
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for reporting! I just made a pull request to fix this. https://github.com/MicrosoftDocs/azure-docs-pr/pull/142679

Thanks for reporting! I just made a pull request to fix this. https://github.com/MicrosoftDocs/azure-docs-pr/pull/142679


DropBox integration

Hi, I have found in your documentation you are already supporting Dropbox. Found it in develop branch, so I tried, but I always get like empty white page after granting ckp application (instead of some file picker). In Chrome console I can see Uncaught ReferenceError: extensions is not defined  .  Do you have an idea what I am doing wrong ?  I Thx in advance.

Not sure ... feature is not ready for prime time, but it should work.
Do you have any KeePass files in dropbox?  If there are no files found then it probably shows a blank list.

I have an update to Dropbox integration in the https://github.com/perfectapi/CKP/tree/feature-premium-features branch.  That probably fixes the issue.

Dropbox integration is now in the live build on the web store.


add docs for @fastify/swagger and @fastify/swagger-ui

This is great thanks @dohaicuong ! I left a couple of comments on the Vercel deployment https://ts-rest-git-fork-dohaicuong-main-ts-rest.vercel.app/docs/fastify/openapi
other than that, this is great.

This is great thanks @dohaicuong ! A couple comments from me then it's perfect IMO.

Missing backtick on @fastify/swagger-ui first paragraph
Do you think it's worth explaining that you can just serve the openAPI doc as a JSON file, and this guide is how to server a Swagger UI if you fancy it
In Generating a OpenAPI Document can we link to the original @ts-rest/open-api docs so they're connected

Then lets get this merged thanks.

I have addressed your comment :+1:
Let me know if you want to make changes with the writing through...

@all-contributors please add @dohaicuong for docs


Upgrade the router onUpdate to take a startsWith string

Is your feature request related to a problem? Please describe.
Currently onUpdate is only given a callback and there is no way for the router to on call the callbacks based on filtration
Describe the solution you'd like
Add a second parameter that is a path filter. Then this callback with only happen if the path filter is met.
In this example, the function myFunc should only get called if the current path starts with "/mypath":
router.onUpdate(myFunc, '/mypath');

The filter could be a string or a regex to allow for better flexibility

Will be in next release


types(manifest): add missing scope and orientation

I found them missing but I directly edited the types files hoping it's enough 😬

Thanks!

Thanks!


Implement disable bundleId capability command

Closes #94
This PR includes #186
📝 Summary of Changes
Changes proposed in this pull request:

Create List and Disable BundleId Capability Operation
Implement Disable Bundle Id Capability Command

🧐🗒 Reviewer Notes
💁 Example
Usage
asc capability disable <bundle-id> <capability-type>
🔨 How To Test
asc capability disable com.exmaple.foo push_notifications

Once you merge #168, rebase this PR.


NOJIRA: Sort gitignore entries and remove duplicated entries

Several .gitignores had /target listed multiple times.
This removes that duplication and sorts the ignore list to make it easier to spot duplication in the future.

Coverage increased (+0.004%) to 22.994% when pulling 1da9688a3af87813e0d2a3aaef23d66f7235db5f on ChristianMurphy:NOJIRA-gitingore-cleanup into c5d425de9738e73432bf4c2d066b67782495b001 on Jasig:master.

Coverage increased (+0.006%) to 22.986% when pulling 4cc77b5a681d6225b6e1577549ba745fe143dc8e on ChristianMurphy:NOJIRA-gitingore-cleanup into 3933dcc7bc73ae3800a99988b084063ef1786bac on Jasig:master.

@ChristianMurphy -- Did you use a tool to do this?  It looks good.

@drewwills A couple tools built into the Atom IDE helped out.
https://atom.io/
https://github.com/atom/sort-lines
https://github.com/FloydaGithub/duplicate-removal


Issue of pixel spillover to next page

Hi, I am using v0.9.3 of html2pdfjs and finding difficulties with PDF generation. I am trying to build a page with specific DOM content.
const opt = {
      margin: [0.3, 0, 0.5, 0],
      filename: 'MySweetDocument.pdf',
      image: { type: 'jpeg', quality: 0.98 },
      html2canvas: { scale: 2 },
      jsPDF: { unit: 'in', format: 'a3', orientation: 'l' },
      pagebreak: { avoid: 'tr' }
  };
const content1 = document.getElementById('content1');
const content2 = document.getElementById('content2');

let pdfObject = html2pdf().set(opt);

await pdfObject.from(content1 ).toPdf().get('pdf').then(function (pdf) {
    pdf.addPage();
}).from(content2).toContainer().toCanvas().toPdf().save();

The content1 DOM has a text at the bottom without any bottom margin and padding. The DIV for content1 ends just right there.
However when I convert to the PDF, it spills over a thin line of 1 pixel to next page:

Has anyone faced this issue? I am not sure how to solve this because the spillover is not expected as the bottom side of the content1 ends right after the Report text and there literally no padding/margin between them.

Experiencing same issue. But in my case lower page has a part cut into upper page

you got any solution ? @ankur0101 @muneebshafiq

@muhammad-tayyab-tkxel well not really.
I just added margins in my content to move it to next page manually.
But didn't find any solution to make it fully dynamic.


Pip install doesn't work

Describe the bug
The straightforward pip install described in the installation directions fails.
To Reproduce
Steps to reproduce the behavior:

pip install graphein
Pip fails

Expected behavior
Graphein to install successfully
Screenshots
$ conda create -n test -qy pip && conda activate test && pip install graphein
Collecting package metadata (current_repodata.json): ...working... done
Solving environment: ...working... done

## Package Plan ##

  environment location: /home/matt/miniconda3/envs/test

  added / updated specs:
    - pip

The following NEW packages will be INSTALLED:

  _libgcc_mutex      pkgs/main/linux-64::_libgcc_mutex-0.1-main
  _openmp_mutex      pkgs/main/linux-64::_openmp_mutex-5.1-1_gnu
  bzip2              pkgs/main/linux-64::bzip2-1.0.8-h7b6447c_0
  ca-certificates    pkgs/main/linux-64::ca-certificates-2023.01.10-h06a4308_0
  certifi            pkgs/main/linux-64::certifi-2022.12.7-py310h06a4308_0
  ld_impl_linux-64   pkgs/main/linux-64::ld_impl_linux-64-2.38-h1181459_1
  libffi             pkgs/main/linux-64::libffi-3.4.2-h6a678d5_6
  libgcc-ng          pkgs/main/linux-64::libgcc-ng-11.2.0-h1234567_1
  libgomp            pkgs/main/linux-64::libgomp-11.2.0-h1234567_1
  libstdcxx-ng       pkgs/main/linux-64::libstdcxx-ng-11.2.0-h1234567_1
  libuuid            pkgs/main/linux-64::libuuid-1.41.5-h5eee18b_0
  ncurses            pkgs/main/linux-64::ncurses-6.4-h6a678d5_0
  openssl            pkgs/main/linux-64::openssl-1.1.1t-h7f8727e_0
  pip                pkgs/main/linux-64::pip-22.3.1-py310h06a4308_0
  python             pkgs/main/linux-64::python-3.10.9-h7a1cb2a_1
  readline           pkgs/main/linux-64::readline-8.2-h5eee18b_0
  setuptools         pkgs/main/linux-64::setuptools-65.6.3-py310h06a4308_0
  sqlite             pkgs/main/linux-64::sqlite-3.40.1-h5082296_0
  tk                 pkgs/main/linux-64::tk-8.6.12-h1ccaba5_0
  tzdata             pkgs/main/noarch::tzdata-2022g-h04d1e81_0
  wheel              pkgs/main/linux-64::wheel-0.38.4-py310h06a4308_0
  xz                 pkgs/main/linux-64::xz-5.2.10-h5eee18b_1
  zlib               pkgs/main/linux-64::zlib-1.2.13-h5eee18b_0

Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working... done
Collecting graphein
  Using cached graphein-1.5.2.tar.gz (185 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [45 lines of output]
      {'all': {'black',
               'flake8',
               'hypothesis',
               'interrogate',
               'isort',
               'nbstripout',
               'nbval',
               'pandoc',
               'pre-commit',
               'pycodestyle',
               'pydocstyle',
               'pytest',
               'pytest-cov',
               'pytest-xdist'},
       'dev': ['black',
               'flake8',
               'hypothesis',
               'interrogate',
               'isort',
               'nbstripout',
               'nbval',
               'pandoc',
               'pre-commit',
               'pycodestyle',
               'pydocstyle',
               'pytest',
               'pytest-cov',
               'pytest-xdist'],
       'extras': ['black',
                  'flake8',
                  'hypothesis',
                  'interrogate',
                  'isort',
                  'nbstripout',
                  'nbval',
                  'pandoc',
                  'pre-commit',
                  'pycodestyle',
                  'pydocstyle',
                  'pytest',
                  'pytest-cov',
                  'pytest-xdist']}
      error in graphein setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected end or semicolon (after version specifier)
          pyyaml>=5.1,<6.*
                ~~~~~~~~^
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

Desktop (please complete the following information):

OS: Ubuntu 22.04
Python Version: I tried this with fresh conda environments of both 3.8 and 3.10.
Graphein Version [e.g. 22]: pip head as of March 2, 2023

Thanks for the bug report. Seems like setuptools has dropped support for wildcard dependency versions. This has been resolved on master. You can install Graphein from GitHub pending a PyPI release with a fix.

Now resolved in 1.6.0
pip install graphein=1.6.0


feat: overhaul of client interfaces and scoped options

Codecov Report
Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (a789b86) 70.30% compared to head (0956ff8) 69.48%.

Additional details and impacted files
@@            Coverage Diff             @@
##            alpha     #185      +/-   ##
==========================================
- Coverage   70.30%   69.48%   -0.83%
==========================================
  Files          16       18       +2
  Lines         522      544      +22
  Branches      101      107       +6
==========================================
+ Hits          367      378      +11
- Misses        128      135       +7
- Partials       27       31       +4

Files
Coverage Δ

src/DeepgramClient.ts
100.00% <100.00%> (+8.33%)
:arrow_up:

src/index.ts
100.00% <100.00%> (ø)

src/lib/constants.ts
87.50% <100.00%> (+1.78%)
:arrow_up:

src/lib/helpers.ts
56.25% <100.00%> (-1.33%)
:arrow_down:

src/packages/ListenClient.ts
100.00% <100.00%> (ø)

src/packages/ManageClient.ts
68.42% <100.00%> (ø)

src/packages/OnPremClient.ts
70.00% <100.00%> (ø)

src/packages/PrerecordedClient.ts
60.00% <100.00%> (-1.30%)
:arrow_down:

src/packages/AbstractClient.ts
88.23% <88.23%> (ø)

src/packages/AbstractRestfulClient.ts
66.66% <87.50%> (+5.37%)
:arrow_up:

... and 2 more

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


Is paper available now ?

Thanks for your excellent work !
I wonder if this work's preprint version(eg. arxiv) is available now ?

Hi @yuhang9803,
I updated the README file 78f341aa and the preprint is now available at here. Please take a look. Thanks for asking!


Bug: Expand Button Logic on RATA Run Records

Expected Behavior
Only 1 flow run can have traverse data so once a traverse point is added to a flow run no other flow run should have an expand button to add traverse data
Note: The 1 flow run that has traverse data can have as many traverse points as the user wants

@mosesdeeCVP

Update screenshot once UI updates are completed:
Private Zenhub Image

Private Zenhub Image

User is able to create Rata Traverse as under any Rata Flow needed


cmake: animation support links to functions from libwebpdemux

Link to libwebpdemux as well, for pi

We should handle LOAD_WEBP_DYNAMIC case: maybe add a new LOAD_WEBPDEMUX_DYNAMIC and adjust IMG_webp.c for it?
The autotools side should be touched accordingly after this, too.

We need 2 handles then, one for libwebp.so.7 (as it is used now) and one for libwebpdemux.so.2? Right?

We need 2 handles then, one for libwebp.so.7 (as it is used now) and one for libwebpdemux.so.2? Right?

Yes, that's what I think.
Then again, libwebpdemux.so should be linked to libwebp.so, so: Can we just simply change the handle from libwebp.so to libwebpdemux.so for simplicity? (No, haven't tried myself yet.)

With current main (i.e. make sure to see that commit 1af8a539a is applied), the following autotools patch works for me (generated file changes not included.)
diff --git a/configure.ac b/configure.ac
index 3358d43..9b0be17 100644
--- a/configure.ac
+++ b/configure.ac
@@ -273,8 +273,6 @@ AC_ARG_ENABLE([xv], [AS_HELP_STRING([--enable-xv], [support loading XV images [d
  [], [enable_xv=yes])
 AC_ARG_ENABLE([webp], [AS_HELP_STRING([--enable-webp], [support loading WEBP images [default=yes]])],
  [], [enable_webp=yes])
-AC_ARG_ENABLE([webpdec], [AS_HELP_STRING([--enable-webpdec], [support loading WEBP images via libwebpdecoder instead of libwebp [default=no]])],
- [], [enable_webpdec=no])
 AC_ARG_ENABLE([webp-shared], [AS_HELP_STRING([--enable-webp-shared], [dynamically load WEBP support [default=yes]])],
  [], [enable_webp_shared=yes])
 AC_ARG_ENABLE([qoi], [AS_HELP_STRING([--enable-qoi], [support loading QOI images [default=yes]])],
@@ -534,25 +532,19 @@ if test x$enable_tif = xyes -a x$enable_imageio != xyes; then
 fi
 AC_SUBST([LOAD_TIF], $load_tif)

-load_webp=0
-if test x$enable_webpdec = xyes; then
-  webplib=webpdecoder
-else
-  webplib=webp
-fi
 if test x$enable_webp = xyes; then
-    PKG_CHECK_MODULES([LIBWEBP], [lib$webplib], [dnl
+    PKG_CHECK_MODULES([LIBWEBPDEMUX], [libwebpdemux], [dnl
         have_webp_hdr=yes
         have_webp_lib=yes
         have_webp_pc=yes
       ], [dnl
-        AC_CHECK_HEADER([webp/decode.h], [
+        AC_CHECK_HEADER([webp/demux.h], [
             have_webp_hdr=yes
-            LIBWEBP_CFLAGS=""
+            LIBWEBPDEMUX_CFLAGS=""
         ])
-        AC_CHECK_LIB([$webplib], [WebPGetDecoderVersion], [
+        AC_CHECK_LIB([webpdemux], [WebPDemuxGetFrame], [
             have_webp_lib=yes
-            LIBWEBP_LIBS="-l$webplib"
+            LIBWEBPDEMUX_LIBS="-lwebpdemux"
         ], [], [-lm])
       ])
     if test x$have_webp_hdr = xyes -a x$have_webp_lib = xyes; then
@@ -561,15 +553,15 @@ if test x$enable_webp = xyes; then

         case "$host" in
             *-*-darwin*)
-                webp_lib=[`find_lib lib$webplib.dylib`]
+                webp_lib=[`find_lib libwebpdemux.dylib`]
                 ;;
             *-*-cygwin* | *-*-mingw*)
-                webp_lib=[`find_lib "lib$webplib-*.dll"`]
+                webp_lib=[`find_lib "libwebpdemux-*.dll"`]
                 ;;
             *)
-                webp_lib=[`find_lib "lib$webplib[0-9]*.so.*"`]
+                webp_lib=[`find_lib "libwebpdemux[0-9]*.so.*"`]
                 if test x$webp_lib = x; then
-                    webp_lib=[`find_lib "lib$webplib.so.*"`]
+                    webp_lib=[`find_lib "libwebpdemux.so.*"`]
                 fi
                 ;;
         esac
@@ -663,11 +655,11 @@ if test x$enable_webp = xyes -a x$have_webp_hdr = xyes -a x$have_webp_lib = xyes
         echo "-- dynamic libwebp -> $webp_lib"
         AC_DEFINE_UNQUOTED(LOAD_WEBP_DYNAMIC, "$webp_lib")
     else
-        IMG_LIBS="$LIBWEBP_LIBS $IMG_LIBS"
+        IMG_LIBS="$LIBWEBPDEMUX_LIBS $IMG_LIBS"
         if test x$have_webp_pc = xyes; then
-            PC_REQUIRES="lib$webplib $PC_REQUIRES"
+            PC_REQUIRES="libwebpdemux $PC_REQUIRES"
         else
-            PC_LIBS="$LIBWEBP_LIBS $PC_LIBS"
+            PC_LIBS="$LIBWEBPDEMUX_LIBS $PC_LIBS"
         fi
     fi
 fi

It's okay to push separately. autotools and cmake are seaparate systems and won't cause merge conflicts.

I don't think using one library works on Windows.
Compiling the following code:
#include <stdio.h>
#include <windows.h>

int main() {
    void* h = LoadLibraryA("libwebpdemux-2.dll");
    printf("h=%p\n", h);
    void*r;
    r = GetProcAddress(h, "WebPDecodeRGBInto");
    printf("WebPDecodeRGBInto -> %p\n", r);
    r = GetProcAddress(h, "WebPDemuxGetFrame");
    printf("WebPDemuxGetFrame -> %p\n", r);
    FreeLibrary(h);
    return 0;
}

and running under wine gives:
#include <stdio.h>
#include <windows.h>

int main() {
    void* h = LoadLibraryA("libwebpdemux-2.dll");
    printf("h=%p\n", h);
    void*r;
    r = GetProcAddress(h, "WebPDecodeRGBInto");
    printf("WebPDecodeRGBInto -> %p\n", r);
    r = GetProcAddress(h, "WebPDemuxGetFrame");
    printf("WebPDemuxGetFrame -> %p\n", r);
    FreeLibrary(h);
    return 0;
}

I think we really need to use 2 library handles.

I don't think using one library works on Windows. Compiling the following code:
I think we really need to use 2 library handles.

Well, hell.. GetProcAddress is tied to the module handle, so you are correct.
Can you handle autofoo side too?

I'll try :)

This looks OK to me.


WA-529 Add task warning attributes on Task & API

JIRA link (if applicable)
RWA-529
Change description
See ticket
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ X] No

Not required anymore


So I kind of went a little overboard this weekend playing with this crate..

A summary of what I've done here & why:

Separated out the concept of storing tuples and coordinating with them, as the Linda language primitives are defined (out, in, rd, eval). Instead of blocking though, I've introduced futures, which will integrate nicely with Tokio for concurrency & networking.
Abstracted out the storage into a Store trait, replaced the vec implementation with BTreeSet to improve matching performance; as in database LIKE wildcard queries, prefixes will tend to full scan, but in general performance should be better. This required some changes to the tuple and element types, esp around how comparisons and equality work. The Store trait could be implemented with a database backend, so there are improvements that can be made here..
Removed tuple expiration times and randomization of query results -- for now. I think these are really good ideas but they should probably be expressed by implementing the Store trait, perhaps extending it to allow policy directives. I'm thinking about how to support that in such a way that's open to extension.
Some syntactical sugar (macros, helper functions, etc.). More tests.

I do apologize for the unwarranted rewrite of a good deal of things, and the amount of changes in this pull request. I've been wanting to write some code all week, and it was good getting back into Rust.. :) Let's call this a conversation starter -- feel free to land this of course, but also feel free to give me feedback and let me know what you'd like to see done differently.
Networking is next on my list... I'd like to put a simple interactive CLI frontend that supports Linda primitives, served up over SSH or websockets. Also want to try coordinating Actix actors with this stuff.
What do you think?

Thanks for your thorough input! Doing this would have taken me a couple months, because I'm still getting accustomed to Rust while working on the project. There are a number of concepts and patterns in your revision that I still have to internalize, so it may take me a little while to go through and understand everything you've changed and added.

Separated out the concept of storing tuples and coordinating with them, as the Linda language primitives are defined (out, in, rd, eval).
Instead of blocking though, I've introduced futures, which will integrate nicely with Tokio for concurrency & networking.

Great! Eliminating blocking access was high up on the priority list, right after replacing the underlying data structure. Initially I was looking at lock-free data structures and using atomics just short of actually implementing it. Futures I had not considered so far but they seem to tie very well into it, especially considering Tokio.

Abstracted out the storage into a Store trait, replaced the vec implementation with BTreeSet to improve matching performance; as in database LIKE wildcard queries, prefixes will tend to full scan, but in general performance should be better. This required some changes to the tuple and element types, esp around how comparisons and equality work. The Store trait could be implemented with a database backend, so there are improvements that can be made here..

Traits are a concept I really have to get used to use more. Abstracting storage as such didn't occur to me at all.
The underlying data structure was top on my priority list. Searching and matching with wildcards eluded me for a while, so I'm going to have a very close look at your implementation there.

Removed tuple expiration times and randomization of query results -- for now. I think these are really good ideas but they should probably be expressed by implementing the Store trait, perhaps extending it to allow policy directives. I'm thinking about how to support that in such a way that's open to extension.
Some syntactical sugar (macros, helper functions, etc.). More tests.

Agreed, expiration is not something the tuple itself needs to know about. I'm thinking about an event queue where automatic expiration can be scheduled while the tuple is being inserted into the space.

I do apologize for the unwarranted rewrite of a good deal of things, and the amount of changes in this pull request. I've been wanting to write some code all week, and it was good getting back into Rust.. :) Let's call this a conversation starter -- feel free to land this of course, but also feel free to give me feedback and let me know what you'd like to see done differently.

It does feel rather humbling to see how much need potential there already is for streamlining and future proofing at this early stage. Rust does allow for a lot of elegant tweaks that I have yet to learn and internalize.

Networking is next on my list... I'd like to put a simple interactive CLI frontend that supports Linda primitives, served up over SSH or websockets. Also want to try coordinating Actix actors with this stuff.

A server with CLI is a great idea that I'm looking forward to build. The library probably has room for a client implementation for an easy way to connect to tuple space servers.
Other than that one could even create a simple TCP or UDP protocol to allow for platform agnostic communication.

What do you think?

I really like that tuples can be initialized with a macro 'tuple!' now, couldn't get that to work myself so far.
Again, it will take a bit of time for me to go through all the changes to understand the mechanics behind them. So far I don't see anything I fundamentally disagree with.
Cheers!


Bump versions to SNAPSHOT

PTAL

If you have verified by grep that 1.14.0 and 0.32.0 have been updated to the SNAPSHOT versions in all pom locations, then LGTM


Adding in meeting report links to meeting page, etc.

Adding link to meeting report for fall 2020 meeting (and spring 2020 meeting when I noticed it wasn't there). Also, removed now outdated AGU info from meeting page.

This has very minimal changes, so not requesting a review.


Gear does not change

In ac_client.py, the GEAR does not change, so how can the vehicle move?

We were using the autoshift from AC. If you want to experiment with manual shift, some functionality needs to be implemented in ac_client. and in car_control.py. See here

Thank you. I have a new issue, that "reset" does not do anything. In Python log, I can see sensor_par Received: reset.

This usually happens if you don't have ContextManager installed. The reset functionality within the plugin comes from that mod.

I think I have followed the process to install.

I also launch AC from contextmanager since it seems easier than running from the game.

ah that's right. I will update the readme. We were using: Challenge -> hotlap which also works.

Thank you very much for the help.


[contour] Adds config key renderer to choose a renderer: default, software, OpenGL.

Not sure how this is possible with Qt, but it would be useful (for customers) to determine if the underlying hardware OpenGL implementation is prior 3.0 and then just default-select software rendering.
Note, the current "software" renderer is making use of the system provided OpenGL software renderer.
Why is Apple Software Renderer causing a SEGV? ;-(

Should help avoiding problems like mentioned in #563.


Fix: The typo where expected number was 6 instead of 5

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hi there,
Thank you for your contribution, however this is not a bug.
The challenge asks you to initialise a with a value of 5, yes. But the test expects a to be evaluated to 6 - which, with the a = a + 1 line after your initialisation, is correct.


IRetryAnalyzer bug in 6.8.17

In the version 6.8.17 the IRetryAnalyzer does not work correct. In the follow example shows a test that should run 3 times and every test should fail. In version 6.8.13 this works correct. In 6.8.17 and 6.8.21 the test rerun forever (until a java.lang.OutOfMemoryError: Java heap space)
public class RetryTest {

    public static class MyRA implements IRetryAnalyzer {

        int i=0;

        @Override
        public boolean retry(ITestResult result) {
            i++;
            boolean retry = i <= 2;
            System.out.println("retry " + retry + " count: " + i);
            return retry;
        }

    }

    @Test(retryAnalyzer = MyRA.class)
    public void hello() {
        System.out.println("hello");
        throw new RuntimeException();
    }
}

The Output is:
hello
retry true count: 1
hello
retry true count: 2
retry false count: 3
hello
retry false count: 4
retry false count: 5
retry false count: 6
hello
retry false count: 7
retry false count: 8
retry false count: 9
retry false count: 10
...

I can't reproduce this. Are you sure this isn't due to a thread safety issue in the test you wrote? What happens if you make i a final AtomicInteger?

Oh nevermind, it looks like this was fixed after the 6.8.21 release by https://github.com/cbeust/testng/commit/6c40ac103c04fac38274dafc077d55df12e9d8d7.


feat: void element tags in helpers are selectable between > and />

Description
Fixes #6649
Supersedes and closes #6658 and #6717
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

The second commit https://github.com/codeigniter4/CodeIgniter4/pull/6789/commits/e92116fbf3b7932fd6deb7474a77cc59db944ed3 is not refactoring.
Why don't you combine the first and second commit into one?

@kenjis I can modified CS-Fixer recommendation in this PR? or in other PR already fixed?

Now fixed.
If you rebase, the php-cs-fix error will be gone.

@ddevsr please rebase again

Thank you @ddevsr and @paulbalandan

link_tag
source
track
param
embed
See https://codeigniter4.github.io/CodeIgniter4/helpers/html_helper.html


Publish Windows targets as well

Could evlish provide Windows binaries online, in case any Windows users would like to try elvish as their command line shell of choice?

Thanks for the interest, but this is already tracked in #168.


Make HiveTable generic to support partition information supplied externally

I've added a commit with my suggestion for achieving backward compatibility via a separate 'GenHiveTable`.  The maestro api still needs to be updated though.

With the final suggestions I just made, this should work well.  👍
@vineethvarghese may want to have a look too.

Actually, I'd still like to do one last pass here.

Looks good, I think this is the best interface now, just a few final suggestions. :+1:

@vineethvarghese @rowandavies Can you please have a quick look at this PR

Done with my review.
Version also need to be bump and we should do a fast forward merge please. No merge commits.

That commit is closer - now rename AbstractPartionedHiveTable to GenericPartionedHiveTable and make it non-abstract and add the rest of the constructor arguments from GenericPartionedHiveTable as val arguments.  Unless that doesn't work for some reason?

@rowandavies if we make AbstractPartionedHiveTable  a  non-abstract class we end up in having override modifiers for all  members in PartitionedHiveTable or  we have to rename and pass them into AbstractPartionedHiveTable class constructor


Fix error message when passing an object literal

In the Excess Property Checks section in the Interfaces page, when an object literal is passed with a wrong key, the error message currently in the documentation isn't as verbose as the one actually displayed by TypeScript.
This PR fixes the error message.

Cool, yep


Add support for tests included in a test class via a trait

Laravel has a number of test case classes where the actual tests are defined in traits that are used within the class definition:

Update the test file parser to identify traits used within a Test class, and add any test methods within the trait as test items.

Test being executed from an included trait is reported in the teamcity logs as follows:
##teamcity[testSuiteStarted name='Illuminate\Tests\Integration\Database\EloquentTransactionWithAfterCommitTest' locationHint='php_qn://C:\Development\PHP\laravel\framework\tests\Integration\Database\EloquentTransactionWithAfterCommitTest.php::\Illuminate\Tests\Integration\Database\EloquentTransactionWithAfterCommitTest' flowId='12148']
##teamcity[testStarted name='testObserverIsCalledOnTestsWithAfterCommit' locationHint='php_qn://C:\Development\PHP\laravel\framework\tests\Integration\Database\EloquentTransactionWithAfterCommitTests.php::\Illuminate\Tests\Integration\Database\EloquentTransactionWithAfterCommitTest::testObserverIsCalledOnTestsWithAfterCommit' flowId='12148']
##teamcity[testFinished name='testObserverIsCalledOnTestsWithAfterCommit' duration='320' flowId='12148']

This could be tricker than originally estimated. Taking the example class in the above description, there are two traits used by the test. However, both of these include additional traits, down multiple levels:

Illuminate\Tests\Integration\Database\EloquentTransactionWithAfterCommitTests

Orchestra\Testbench\Concerns\WithLaravelMigrations

Orchestra\Testbench\Concerns\InteractsWithWorkbench

Orchestra\Testbench\Concerns\InteractsWithPest

Orchestra\Testbench\Concerns\InteractsWithPHPUnit

Orchestra\Testbench\Concerns\InteractsWithTestCase

Orchestra\Testbench\Concerns\InteractsWithTestCase

Orchestra\Testbench\Concerns\InteractsWithPHPUnit

Orchestra\Testbench\Concerns\InteractsWithTestCase

Orchestra\Testbench\Concerns\InteractsWithTestCase

Illuminate\Foundation\Testing\DatabaseMigrations

Illuminate\Foundation\Testing\Traits\CanConfigureMigrationCommands

The vast majority of these nested traits do not actually include test methods, but are just helper methods. Will need to recursively parse potentially quite deep nested traits.
Could look at caching traits, but that has issues of its own - traits aren't included in the patterns used by the file watchers, since they don't need to conform to anything defined in the phpunit.xml configuration. If the trait is modified by the user, the extension has no way of picking up those changes, unless we add new watchers for each trait parsed. Either option could introduce some significant performance degredation.


[BUG]:  blackarch signature

Athena OS version
Athna OS athena-rolling-x86_64
Which environment or technology are you using?

[X] Baremetal
[ ] VirtualBox Virtual Machine
[ ] VMware Virtual Machine
[ ] Hyper-V Virtual Machine
[ ] QEMU/KVM
[ ] Parallels
[ ] UTM
[ ] Container
[ ] WSL

Kernel version
6.2.11-lts-1-lts
Involved software
No response
Repro steps

Expected behavior

Actual behavior

Diagnostic logs
No response

before i was updated mirror on blackarch official but nothing happened. can i have solution

I uploaded a new ISO where I removed BA. Download it and try.

before i was updated mirror on blackarch official but nothing happened. can i have solution

Get it by https://github.com/Athena-OS/athena/releases/tag/v23.11 and let me know.


Move ComponentSelector from PowerAnalytics to InfrastructureSystems: PSY portion

This draft PR will get filled in as @tengis-nrl and I move the ComponentSelector features from PowerAnalytics to InfrastructureSystems.

Rebased onto psy4 and tests pass.