(MAINT) Bump for new release

Can one of the admins verify this patch?

@cdenneen I like this code, & we've created a ticket (QENG-6271) in order to do the work of setting up automation to run github_changelog_generator as a part of beaker-aws's release.
There's one hitch before we merge your code as-is though. Our release jobs will fail if the version doesn't change when they're run, so you'll need to take away the actual version number change you have in this PR. I know that you need it to be the number it is for the CHANGELOG to make sense, but we'll make sure that happens when we run the release job, or do whatever work is needed to make it consistent.

Last comment still current

I’ll take a look tomorrow.

 I can either submit that as a PR to your branch or we can close this PR for that one. Which would you prefer?

please submit PR back to this and will merge
On Mon, Mar 19, 2018 at 1:16 PM, Kevin Imber<EMAIL_ADDRESS>wrote:

@cdenneen https://github.com/cdenneen I'm going to spend some time on
this this week to create a rake task to make this easy to use for a job. I
can either submit that as a PR to your branch or we can close this PR for
that one. Which would you prefer?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/puppetlabs/beaker-aws/pull/9#issuecomment-374293174,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAr84ZKbSq95UVJiy5BWUpGx8bmjLOWBks5tf-fhgaJpZM4RC0mN
.

@cdenneen apologies for the big delay here. I've submitted a PR to your branch at your fork for the rake tasks: https://github.com/cdenneen/beaker-aws/pull/1. Let me know if you have any issues or comments with that there, & once that's done we can come back to this one to push it forward.

Can one of the admins verify this patch?


Add API endpoint to cancel a Renter's contract

resolves #3159

Would it make more sense to allow the user to set all the utility fields manually instead of just setting all the fields to false?


Recent Update broke usage in Vysor

The most recent update, while beautiful, has broken my workflow.  I use this all day, every day.  However, I can't take the time to pull my phone out to use it (nor the other phone apps I use throughout the day) and I rely on the speed/efficiency of using these apps via vysor.io.
Since the last update, the google authenticator is blank in the vysor.io window, and I have to use my phone to see the code, which is a much slower process.

You mean vyzor.io?
Why doesn't it work?

Yes, sorry.   I updated the post.
"Why" is that none of the content in the GA display shows up on the desktop.  It does on the phone, but something is blocking the content from getting to the desktop host via vysor.io.
I posted here since it was a GA update which broke GAs display via vysor.io.  Until the most recent update, GA worked great through vysor.io.  No other apps are behaving this way in vysor.io.  Therefore, it seems like it's probably something that GA updated which caused the break.  However, your suggestion is good - I'll send them a message as well.

I see. I still think you need to contact vysor.io about why it doesn't work. I don't know what method they use to share the screen, but it breaking may be a side effect of blocking an API for security reasons.

I believe it's to do with the same mechanism for not displaying the app content on the "recent apps" switcher. It's a security measure.. It is so malicious apps cannot abuse accessibility functionality and read your 2fa codes.

This seems like it's working as intended. Even if that means stuff like vysor can't work with GA. It's unfortunate, but if there were no abuse then we wouldn't need 2FA in the first place. Perhaps vysor will find a way to enable this use case.


[DRAFT] SNOW-1316977: Remove fallbacks ans raise not implemented error instead

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #NNNN

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

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


Fix the metadata for empty data table

Added default numDocsScanned, numEntriesScannedInFilter and numEntriesScannedPostFilter into empty data table

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b2f4448). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1935   +/-   ##
=========================================
  Coverage          ?   69.49%
=========================================
  Files             ?      830
  Lines             ?    40229
  Branches          ?     5394
=========================================
  Hits              ?    27957
  Misses            ?    10403
  Partials          ?     1869

Impacted Files
Coverage Δ

...core/query/executor/ServerQueryExecutorV1Impl.java
82.44% <100%> (ø)

Continue to review full report at Codecov.

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


Feature: Plugin Interface for Models

I was just working on project that has to use a plugin architecture ie the application will have a main core with multiple plugins extending the features for the application. Each plugin will have some data models that extend the core's EF Model.
I would like to suggest a feature that will make it simple to append Model and Relationship Definitions to a current DBContext
I know this is a long shot but will really appreciate if you take it into consideration

@alfkonee Thanks for the suggestion. Could you ellaborate more on how you envision this would work?

@alfkonee Are you talking about not knowing what models will exist in the DbContext until runtime?

 Well the feature I suggested is exactly as shown in  @Jetski5822  comment

I found this item sitting on my plate for investigation, but I think it either belongs in the backlog or closed, so clearing up milestone for re-triage.
The main point I gather from the conversation is to make it easy to have base models that are extended dynamically/programmatically, which is a valid and important scenario.
However, some of this is already possible, e.g. applications using ASP.NET Identity do it all the time by deriving from IdentityDbContext and overriding OnModelCreating(), etc. and customers can invoke reflection code to create an EF model as in Rowan's blog post. Although this is not necessarily as easy as it could be, lacking specifics on how to actually make it easier, I am not sure it makes sense to keep this open.
On the other hand some richer scenarios could be enabled by shadow entities which is already covered in #749.


File upload using multipart/form-data errors

I tried to Strawberry with FastAPI integration. Why is strawberry trying to do a json.loads on mulitpart_data? When I upload a file from the browser as a multipart/form-data I get the following error:
ERROR:    Exception in ASGI application Traceback (most recent call last):   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/uvicorn/protocols/http/h11_impl.py", line 366, in run_asgi     result = await app(self.scope, self.receive, self.send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in __call__     return await self.app(scope, receive, send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/fastapi/applications.py", line 261, in __call__     await super().__call__(scope, receive, send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/applications.py", line 112, in __call__     await self.middleware_stack(scope, receive, send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/middleware/errors.py", line 181, in __call__     raise exc   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/middleware/errors.py", line 159, in __call__     await self.app(scope, receive, _send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/exceptions.py", line 82, in __call__     raise exc   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/exceptions.py", line 71, in __call__     await self.app(scope, receive, sender)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__     raise e   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__     await self.app(scope, receive, send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/routing.py", line 656, in __call__     await route.handle(scope, receive, send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/routing.py", line 259, in handle     await self.app(scope, receive, send)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/starlette/routing.py", line 61, in app     response = await func(request)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/fastapi/routing.py", line 227, in app     raw_response = await run_endpoint_function(   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/fastapi/routing.py", line 160, in run_endpoint_function     return await dependant.call(**values)   File "/home/phat/projects/excel-file-manager/venv/lib/python3.10/site-packages/strawberry/fastapi/router.py", line 165, in handle_http_query     operations = json.loads(multipart_data.get("operations", {}))   File "/usr/lib/python3.10/json/__init__.py", line 339, in loads     raise TypeError(f'the JSON object must be str, bytes or bytearray, ' TypeError: the JSON object must be str, bytes or bytearray, not dict

Hi,
Can you give us some details: how did you implement the mutation, the query you are trying to do and how did you pass the variables?
The file upload specifications are a bit different, according to the GraphQL multipart request specification:
in the operations files must be replaced with null and in map For each file, the key is the file multipart form field name and the value is an array of operations paths.
it's harder to explain in words... but you can see some examples here in our tests hope those help to understand :)

Hey @Phat-Loc are you still having this problem?
Also meanwhile here I have updated the docs with some examples :)

It is working now. I looked at the test code for Strawberry and just made the same request via Javascript that the test code does. Thanks!


Add Redis Driver

Considering the necessity of employing storage other than arrays or databases, I took the liberty of crafting a driver to provide native support for Redis.
The advantage lies in not being confined to a single Laravel instance or the database, which already contends with its challenges of overload.
This minor alteration is a replica of the database driver, with adaptations for writing and reading Hash in Redis.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@potsky, can you please open an issue about the unique violations and we can look into it. The DB driver likely needs to perform upserts.


Meshes in linked blend file don't render

I linked some files together and while resulting scene renders correctly with Cycles, prman shows some errors after the render completes and the linked objects are missing. An test scene is attached. To reproduce, try rendering from Scene.blend.
Config:
OS X 15D21
PRMan 20.7 linked Tue Jan 26 11:42:50 2016 PST @1571626
PRMan-for-Blender: 0d377502
LinkingError.zip

Fixed.  FYI this was really an issue of duplis within duplis not working correctly.
Brian

Verified, working at 3f968ef. Thanks!


TransactionItemLayout UI breaks with long caption

The above screenshot is from the leather v6.42.1 production.

Thanks for adding a seperate issue for this. I think it is good to cover this in our redesign of this section.
https://github.com/leather-io/extension/pull/5536#discussion_r1651059637
I think it may make sense to have just icons to "increase fee" and "cancel transaction" so they take up less real estate. They could expand on hover to show "increase fee" or "cancel transaction" but I am not sure if that is even needed, it will be obvious to the user on the page they go to after clicking the icon.

https://github.com/leather-io/extension/pull/5536#discussion_r1654986771 adding this here for visibility. Not sure if this is still needed if that design is adopted with 5536.


Upgrade requests to 2.20.0

Pull Request Test Coverage Report for Build 172

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 169:
0.0%

Covered Lines:
342

Relevant Lines:
342

💛  - Coveralls

Pull Request Test Coverage Report for Build 172

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 169:
0.0%

Covered Lines:
342

Relevant Lines:
342

💛  - Coveralls

Pull Request Test Coverage Report for Build 172

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 169:
0.0%

Covered Lines:
342

Relevant Lines:
342

💛  - Coveralls

Pull Request Test Coverage Report for Build 172

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 169:
0.0%

Covered Lines:
342

Relevant Lines:
342

💛  - Coveralls

Pull Request Test Coverage Report for Build 172

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 169:
0.0%

Covered Lines:
342

Relevant Lines:
342

💛  - Coveralls


leftover DEBUG in diskover.py:365

There's a leftover use of DEBUG in indexAdd() but you're no longer passing DEBUG to the function.
This causes a fatal exception when that code is triggered.

I released version 1.0.4 tonight which should fix this issue. Please update and reply if it's fixed.

I've removed the debug code from Diskover, will close this issue.


Update note regarding 3D MSAA and foveated rendering

https://github.com/godotengine/godot/pull/83976 added support for 3D MSAA on the Compatibility renderer, but it also mentions that on platforms other than Android, foveated rendering will not work if MSAA is enabled.
I removed the note saying that 3D MSAA is not supported on compatibility and added a comment mentioning that foveated rendering won't work on platforms other than android if 3D MSAA is enabled and added the alternative/equivalent for desktop.

@BastiaanOlij can probably verify the information, since he's the author of the PR I mentioned

Almost good, indeed, foveated rendering + MSAA only works on Android (provided certain OpenGL extensions are available, but they will be on all current XR hardware). Otherwise MSAA rendering happens in a separate buffer, and as that isn't provided by OpenXR, foveated rendering is not enabled on that buffer.
Also the VRS_MODE alternative only works on Vulkan.

I forgot about VRS being Vulkan only! I believe now it should be good, but let me know otherwise.

Thanks!

Another section of the doc still has this text :(
    The Compatibility rendering method does **not** support variable rate
    shading.


normalize externalsType

Feel free to close it if I was wrong.

What kind of change does this PR introduce?
bugfix.
I was trying to set externalsType to script and found I got var instead. The option is filtered in lib/config/normalization.js file, which means it will always fall back on https://github.com/webpack/webpack/blob/master/lib/config/defaults.js#L178.

Did you add tests for your changes?
No

Does this PR introduce a breaking change?
No

What needs to be documented once your changes are merged?
Nothing

Thanks, for the next time a test case would be great.


Add wazero specific limitations

This PR add more documentations on wazero specific limits,
and impose the limitation in the module instantiation phase.
Also, this backfills tests for executeConstantExpression and buildGlobalInstances
and fixed a bug in the former.
The implementation on the function value stack height limit will be in a followup PR
as it would be rather large change.

@nullpo-head @codefromthecrypt thanks for the look!


TypeError: timer.unref is not a function

Do you want to request a feature or report a bug?
Bug, or just support request (sorry!)
What is the current behavior?
Lots of errors around TypeError: timer.unref is not a function when trying to run jest on my mocha tests.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal repository on GitHub that we can npm install and npm test.
Yeah just do this
test.js
test('thing', () => {
   var timer = setInterval(() => {}, 1)
   timer.unref()
})

And then run it with jest somehow.
 FAIL  tests/test2.js
  ● thing

    TypeError: timer.unref is not a function
      at Object.<anonymous> (tests/test.js:3:10)
      at process._tickCallback (node.js:364:9)

What is the expected behavior?
Not FAIL.
It works fine when you run the code directly with node. Also here is the API for it:
https://nodejs.org/api/timers.html#timers_timeout_unref
Run Jest again with --debug and provide the full configuration it prints. Please mention your node and npm version and operating system.

node 4.5
npm 3.10.8

Debug has basically the same output as above:
jest version = 16.0.1
test framework = jasmine2
config = {
  "testPathDirs": [
    "/Users/jamuferguson/dev/paypal/p2pnodeweb"
  ],
  "setupFiles": [
    "/Users/jamuferguson/dev/paypal/p2pnodeweb/node_modules/babel-polyfill/lib/index.js",
    "/Users/jamuferguson/dev/paypal/p2pnodeweb/tests/lib/setup-test-env-jest.js"
  ],
  "testRegex": "tests/test2.js",
  "testURL": "http://example.com",
  "testPathIgnorePatterns": [
    "/node_modules/",
    "/vendor",
    "/helpers/",
    "/fixtures/"
  ],
  "moduleNameMapper": [
    [
      "^.+\\.svg$",
      "/Users/jamuferguson/dev/paypal/p2pnodeweb/public/js/__mocks__/custom/file-mock.js"
    ]
  ],
  "coverageThreshold": {
    "global": {
      "branches": 29,
      "functions": 22,
      "lines": 32,
      "statements": 29
    }
  },
  "collectCoverageFrom": [
    "!public/js/{vendor,view,widgets,hooks,collection,model,lib}/**",
    "!public/js/{konami,bootstrap}.js"
  ],
  "coverageDirectory": "/Users/jamuferguson/dev/paypal/p2pnodeweb/coverage/react",
  "rootDir": "/Users/jamuferguson/dev/paypal/p2pnodeweb",
  "name": "-Users-jamuferguson-dev-paypal-p2pnodeweb",
  "testRunner": "/Users/jamuferguson/dev/paypal/p2pnodeweb/node_modules/jest-jasmine2/build/index.js",
  "scriptPreprocessor": "/Users/jamuferguson/dev/paypal/p2pnodeweb/node_modules/babel-jest/build/index.js",
  "usesBabelJest": true,
  "automock": false,
  "bail": false,
  "browser": false,
  "cacheDirectory": "/var/folders/9d/p3qfw0g94yz7qh3z9dr4btwm391xgk/T/jest",
  "clearMocks": false,
  "coveragePathIgnorePatterns": [
    "/node_modules/"
  ],
  "coverageReporters": [
    "json",
    "text",
    "lcov",
    "clover"
  ],
  "globals": {},
  "haste": {
    "providesModuleNodeModules": []
  },
  "mocksPattern": "__mocks__",
  "moduleDirectories": [
    "node_modules"
  ],
  "moduleFileExtensions": [
    "js",
    "json",
    "jsx",
    "node"
  ],
  "modulePathIgnorePatterns": [],
  "noStackTrace": false,
  "notify": false,
  "preset": null,
  "preprocessorIgnorePatterns": [
    "/node_modules/"
  ],
  "resetModules": false,
  "testEnvironment": "jest-environment-jsdom",
  "timers": "real",
  "useStderr": false,
  "verbose": null,
  "watch": false,
  "cache": true,
  "watchman": true,
  "testcheckOptions": {
    "times": 100,
    "maxSize": 200
  }
}
Determining test suites to run...watchman warning:  Recrawled this watch 1 times, most recently because:
/Users/jamuferguson/dev/paypal/p2pnodeweb: kFSEventStreamEventFlagUserDropped
To resolve, please review the information on
https://facebook.github.io/watchman/docs/troubleshooting.html#recrawl
To clear this warning, run:
`watchman watch-del /Users/jamuferguson/dev/paypal/p2pnodeweb ; watchman watch-project /Users/jamuferguson/dev/paypal/p2pnodeweb`

 FAIL  tests/test2.js
  ● thing

    TypeError: timer.unref is not a function

      at Object.<anonymous> (tests/test2.js:3:10)
      at process._tickCallback (node.js:364:9)

  ✕ thing (3ms)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 total
Snapshots:   0 total
Time:        3.936s
Ran all test suites.

So far it seems my issue is related to the fact that the setInterval implementation being used is not the native node one, but instead one provided by JSDom.
console.log(setInterval.toString()) // inside the test
``

matches https://github.com/tmpvar/jsdom/blob/a6acac4e9dec4f859fff22676fb4e9eaa9139787/lib/jsdom/browser/Window.js#L176

The solution is to use "testEnvironment": "node"

I got here by googling this.pruneTimer.unref is not a function. I thought I was not gonna get any results but this issue was at the top. It was not an exact match yet I still decided to read it. The error was coming from connect-pg-simple. I'm using connect-pg-simple for session storage on an express app.
Setting "testEnvironment": "node" as mentioned here works.
Just leaving this here for anyone else that might need it.

I'm also getting the TypeError: this.timerId.unref is not a function when testing React elements interacting with Firebase Firestore.
Changing the testEnvironment to node fixes the above error, but then obviously none of the useful dom implementations are available from jsdom :/
Does anyone have any thoughts when you are trying to test React elements that need the jsdom environment for things like localStorage?

@NathTech I'm getting the same issue, trying to write some integration tests wish Jest + JSDOM + React + Emulators but I'm blocked testing anything that uses firestore

Have the same warning when using Jest to test React + Firestore.

Yes! someone else! Please comment on the issues I raised in other repositories for visibility :)


Command update regression

CommandManager is not set up to handle API v1_19_R1.

Case has been added for v1_19_R1, and a new exception VersionNotHandledException has been added as a default case.


use downloaded font

Use a downloaded font for amp-form. This fix the currently broken sample.
@sebastianbenz PTAL

LGTM


Can you publish the latest stable release for NSwagStudio.msi?

Can you publish the latest stable release for NSwagStudio.msi? The most recent is from 20 days ago.
I need the latest swagger api.

I will release it next week, but you can download the latest MSI here: https://ci.appveyor.com/project/rsuter/nswag-25x6o/build/artifacts

Thx!

I downloaded it, and generated the Client. But it gives me an error:

Client.cs(170,170): Error CS0246: The type or namespace name `JsonExceptionConverter' could not be found. Are you missing an assembly reference? (CS0246)

I fixed this in this commit: https://github.com/NSwag/NSwag/commit/11ee063e1d74eff0cd87c0c3e43b62eaebae1332
Try installing the latest MSI...

Ok, I installed the last version now.
The error about JsconExceptionConverter is gone now.
But there is another error that I get compiling the class:

Client.cs(42,42): Error CS0117: System.Net.Http.HttpMethod' does not contain a definition for Patch' (CS0117)

The line of code with error is this:
 request_.Method = HttpMethod.Patch;
I googled it, and I found this on stackoverflow:
http://stackoverflow.com/questions/23599817/web-api-routing-constraint-patch-missing-in-httpmethod

Ok, so
request_.Method = new HttpMethod("PATCH")

compiles and works?

Yes, I changed the line of code to that and now compiles with 0 errors.

Ok fixed: https://github.com/NSwag/NSwag/commits/master
Just wait for the CI build to finish then download the MSI

Sorry, this commit: Fixed bug with special http methods (CS)

Please report any other bugs so that we can release a good version next week...


Default avatar image missing in chat for users

For users who haven't set an avatar image, we're not seeing the default image in the chat pane:

Either I can't reproduce this, or I'm not understanding the description. I'm seeing the default image normally in the chat pane for users with no avatar.


Fixed docs typo

Just saw this and couldn't keep myself from adding a PR ;)

That's why the PR system is there, isn't it? :)


super slow

when writing data to about 6000 cells, it took about 15min. The cpu usage is very high.

Thank you for your report.
We will look into improving the efficiency of the writing process.
Please wait for our next update.

Thank you for your report. We will look into improving the efficiency of the writing process. Please wait for our next update.

reading is super slow too. An xlsx file about 240kb with about 30 sheets, reading a sheet contains 50 cells took about 10min too.

It seems that this crate will read whole workkbook, rather than the used sheet.
And It will create a whole new workbook when saving rather than save the modified part.
At least, reading part can be changed to read sheets lazily.

First of all, we reviewed the load-bearing processes without making major changes to the configuration.
We will update it sequentially for further speed improvement.

First of all, we reviewed the load-bearing processes without making major changes to the configuration. We will update it sequentially for further speed improvement.

Thanks for your excellent work and speedy response.
With latest version, the read time drops from 35s to 30s.

Thank you for the quick confirmation.
We will aim to implement lazy loading as the next measure to improve processing speed.


Compute the cost of chat completion tasks

Describe the solution you'd like
Reporting cost + the total number of tokens + potentially saving for chat completion tasks.
Currently, OpenAI doesn't report these aspects like the completion API.
Additional context
The main logic is as follows:

Compute the total tokens for each LLM response
1.1. Leverage the attribute usage under the class ChatDocMetaData. Set this attribute response.metadata.usage = self.num_tokens(response.content) + self.chat_num_tokens(hist)
1.2. Add usage attribute under the class LLMMessage. Then, for every LLM response,  set this attribute.
1.3. make sure to set the usage attribute when the response is converted to LLMMessage
Aggregate the total tokens for ALL LLM responses
2.1. iterate over the list message_history and get the value of the usage attribute, and sum them up. Should I exclude cached responses??
2.2. I may need to add an attribute total_tokens to the class ChatAgent to report the total number of tokens
Compute the cost

@pchalasani what do you think about this logic?

I can't add usage to LLMMessage

Token cost is incurred every time we get an LLM response. I think the right place to track & accumulate token cost is inside the ChatAgent. When ChatAgent.llm_response_messages is invoked with a list of messages $m_1, m_2, ... m_k$, and a response $r$ is obtained, total token cost incurred is:

if $r$ is a cached response, then 0
otherwise, it is roughly $T(m_1) + T(m_2) + ... T(m_k) + T(r)$, where $T(.)$ is the num_tokens fn

LLMResponse does have a usage field, but it is not populated in some scenarios, e.g. streaming. I think extracting usage info in streaming mode is not straightforward. Maybe check the OpenAI forums and OpenAI cookbooks on cost/usage computation in various scenarios.

This is what I did, but I had to create a List that will maintain all responses (similar to message_history) to compute total tokens.
But for streaming, don't we have access to the content field in the ChatDocument? so after receiving the response, we can do something like this to count number of tokens in the content field:
with StreamingIfAllowed(self.llm):
            response = self.llm_response_messages(hist, output_len)
            response.metadata.usage = self.num_tokens(
                response.content
            ) + self.chat_num_tokens(hist)

Agreed. Output cost is double the Input cost so it’s not a trivial diff

So I'm working now on changing usage field in the class LLMResponse to a Dict, then we can easily compute the cost

solved on this PR https://github.com/langroid/langroid/pull/227


Add Dex repository to /opensource

Apparently, a lot of people still don't know this is a thing, so this might be a good way of making people know it's actually a thing.

I'm not sure this is the right place to spread it, necessarily, but I suppose it can't hurt.


Feature Request: Center thumbnails even when resizing sidebar

This is a feature request. It looks better if the thumbnails are always at the center when you resize the sidebar.
Tried to do some css work but no luck

This sounds like something with a clearly disproportionate implementation effort in relation to it's usefulness. Especially considering that it'd likely required a bit of JavaScript, as opposed to just CSS, since if the thumbnails were centred they wouldn't just automatically wrap when the sidebar width changes.
All in all, I'd suggest WONTFIX as an appropriate resolution here and also point to http://mozilla.github.io/pdf.js/getting_started/#introduction (emphasis mine):

The viewer is built on the display layer and is the UI for PDF viewer in Firefox and the other browser extensions within the project. It can be a good starting point for building your own viewer. However, we do ask if you plan to embed the viewer in your own site, that it not just be an unmodified version. Please re-skin it or build upon it.

Finally, you need to fill out https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md when opening an issue!


bug and enhancement of -centerline viewer

This PR aims at:

fixing the bug #2168
implement the suggested enhancement: #2167

Syntax bug:
There was a confusion between an old and new naming convention for -centerline manual versus -centerline file. We have now file everywhere, in both sct_deepseg_lesion and sct_deepseg_sc.
For instance:
sct_deepseg_sc -i t2.nii.gz -c t2 -centerline file -file_centerline t2_ctr.nii.gz

New output:
Output the viewer labels when using -centerline viewer. Implemented in both sct_deepseg_lesion and sct_deepseg_sc. The output file has the following suffix: _labels-centerline.nii.gz.
To test:
sct_deepseg_sc -i t2.nii.gz -c t2 -centerline viewer

Fixes #2167, Fixes #2168

Good to re-review!

@charleygros there might be conflicts due to the recent merge of #2189 so i will let you update your branch and then i can review


github corner covers menu-button on mobile view

Hi,
many thanks for providing the theme (see in action: www.templer.se).
I needed to drop the 'github-corner' from source/_layouts/default.html to not cover the menu button that appears instead of the site links in the header when viewing with Safari from iOS. On home it does not matter.
Regards,
Sven

Created pull request #27

Pull request accepted, thanks for your contribution @keeratsingh


fix elasticsearch async check

Description
Small fix for async elasticsearch function
Fixes https://github.com/run-llama/llama_index/issues/10548

This is not a bug, and the patch will not help, you have to use elasticsearch async client to configure llama-index connections:
$ pip install elasticsearch[async]
then in llama-index code:
from elasticsearch import AsyncElasticsearch
client = AsyncElasticsearch(
"https://localhost:9200", )
vector_store = ElasticsearchStore(
index_name="your_index", es_client=client
)
and the index is created, and the rest of async methods in this package are working properly.


/var/log/couchdb/couchdb.log no longer written by default on RHEL 8

Description
On older versions (i.e. 3.3.3) couchdb would write to /var/log/couchdb/couchdb.log by default. As of 3.4.x this no longer appears to be the case. This is specific to RHEL 8 (I can't repro this on other Linux distros).
Steps to Reproduce
[jefferbrecht@couchdb2 ~]$ cat install
sudo yum install -y yum-utils
sudo yum-config-manager --add-repo https://couchdb.apache.org/repo/couchdb.repo
sudo yum install -y couchdb
cat << EOF > local.ini
[couchdb]
[couch_peruser]
[chttpd]
[httpd]
[chttpd_auth]
[ssl]
[vhosts]
[admins]
admin = otelp
EOF
sudo mv local.ini /opt/couchdb/etc/local.ini
sudo chown couchdb:couchdb /opt/couchdb/etc/local.ini
sudo chmod 0755 /opt/couchdb/etc/local.ini
sudo systemctl enable couchdb
sudo systemctl restart couchdb

[jefferbrecht@couchdb2 ~]$ sudo bash install
...<truncated>...
Installed:
  couchdb-3.4.1-1.el8.x86_64

Complete!
Created symlink /etc/systemd/system/multi-user.target.wants/couchdb.service → /usr/lib/systemd/system/couchdb.service.

[jefferbrecht@couchdb2 ~]$ sudo cat /var/log/couchdb/couchdb.log
cat: /var/log/couchdb/couchdb.log: No such file or directory

[jefferbrecht@couchdb2 ~]$ sudo ls /var/log
audit  boot.log  btmp  chrony  cron  dnf.librepo.log  dnf.log  dnf.rpm.log  firewalld  hawkey.log  kdump.log  lastlog  maillog  messages  private  qemu-ga  secure  spooler  sssd  tuned  wtmp

[jefferbrecht@couchdb2 ~]$ sudo cat /var/log/messages | grep "Apache CouchDB"
Sep 27 16:03:05 couchdb2 systemd[1]: Started Apache CouchDB.
Sep 27 16:03:06 couchdb2 couchdb[30145]: [info] 2024-09-27T16:03:06.139728Z<EMAIL_ADDRESS><0.231.0> -------- Apache CouchDB 3.4.1 is starting.
Sep 27 16:03:06 couchdb2 couchdb[30145]: [info] 2024-09-27T16:03:06.229326Z<EMAIL_ADDRESS><0.231.0> -------- Apache CouchDB has started. Time to relax.
Sep 27 16:03:06 couchdb2 couchdb[30145]: [info] 2024-09-27T16:03:06.458724Z<EMAIL_ADDRESS><0.327.0> -------- Apache CouchDB has started. Time to relax.
Sep 27 16:03:06 couchdb2 couchdb[30145]: [info] 2024-09-27T16:03:06.459530Z<EMAIL_ADDRESS><0.327.0> -------- Apache CouchDB has started on http://<IP_ADDRESS>:5984/

Expected Behaviour
couchdb should write to /var/log/couchdb/couchdb.log by default
Your Environment

CouchDB version used: 3.4.1
Browser name and version: N/A
Operating system and version: Rocky Linux 8

[jefferbrecht@couchdb2 ~]$ curl http://<IP_ADDRESS>:5984/
{"couchdb":"Welcome","version":"3.4.1","git_sha":"f504e38a5","uuid":"5281f9398739edc80fc04692299c0db0","features":["access-ready","partitioned","pluggable-storage-engines","reshard","scheduler"],"vendor":{"name":"The Apache Software Foundation"}}
[jefferbrecht@couchdb2 ~]$ cat /etc/os-release
NAME="Rocky Linux"
VERSION="8.10 (Green Obsidian)"
ID="rocky"
ID_LIKE="rhel centos fedora"
VERSION_ID="8.10"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Rocky Linux 8.10 (Green Obsidian)"
ANSI_COLOR="0;32"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:rocky:rocky:8:GA"
HOME_URL="https://rockylinux.org/"
BUG_REPORT_URL="https://bugs.rockylinux.org/"
SUPPORT_END="2029-05-31"
ROCKY_SUPPORT_PRODUCT="Rocky-Linux-8"
ROCKY_SUPPORT_PRODUCT_VERSION="8.10"
REDHAT_SUPPORT_PRODUCT="Rocky Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="8.10"

Additional Context
I couldn't find anything in the 3.4.x release notes about changes to log writer defaults.

thank you, we are looking into this. This should not have changed.

ok, so this was deliberate. https://github.com/apache/couchdb-pkg/commit/366942bb4ca42eafdf69b66471173686ef900631
In addition, since we're updating the RPM package, remove some of
deprecated/redundant settings needed for old CentOS-es:

  - Log to journald instead of a file

we should have called this out in release notes.

--- a/rpm/SOURCES/10-filelog.ini
+++ /dev/null
@@ -1,4 +0,0 @@
-[log]
-writer = file
-file = /var/log/couchdb/couchdb.log
-level = info

Looks WAI then -- thanks for confirming!

I’ve added this to the release blog post:

Heads up RHEL (and compatible) users: our .rpm convenience binaries have changed logging from /var/log/couchdb/couchdb.log to logging to journald directly.


AttributeError: 'NoneType' object has no attribute 'replace' plugin: newznab (with imdb_id)

2019-12-25 23:06:55 CRITICAL task          Usenet_movies_Search BUG: Unhandled error in plugin discover: 'NoneType' object has no attribute 'replace'
Traceback (most recent call last):
  File "/usr/lib/python3.6/threading.py", line 884, in _bootstrap
    self._bootstrap_inner()
    |    -> <function Thread._bootstrap_inner at 0x7f4842a289d8>
    -> <Thread(task_queue, started daemon<PHONE_NUMBER>70688)>
  File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
    |    -> <function Thread.run at 0x7f4842a287b8>
    -> <Thread(task_queue, started daemon<PHONE_NUMBER>70688)>
  File "/usr/lib/python3.6/threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
    |    |        |    |        |    -> {}
    |    |        |    |        -> <Thread(task_queue, started daemon<PHONE_NUMBER>70688)>
    |    |        |    -> ()
    |    |        -> <Thread(task_queue, started daemon<PHONE_NUMBER>70688)>
    |    -> <bound method TaskQueue.run of <flexget.task_queue.TaskQueue object at 0x7f483952e3c8>>
    -> <Thread(task_queue, started daemon<PHONE_NUMBER>70688)>

  File "/usr/local/lib/python3.6/dist-packages/flexget/task_queue.py", line 45, in run
    self.current_task.execute()
    |    |            -> <function Task.execute at 0x7f483ce47c80>
    |    -> <flexget.task.Task object at 0x7f4838ec6da0>
    -> <flexget.task_queue.TaskQueue object at 0x7f483952e3c8>

  File "/usr/local/lib/python3.6/dist-packages/flexget/task.py", line 79, in wrapper
    return func(self, *args, **kw)
           |    |      |       -> {}
           |    |      -> ()
           |    -> <flexget.task.Task object at 0x7f4838ec6da0>
           -> <function Task.execute at 0x7f483ce47bf8>

  File "/usr/local/lib/python3.6/dist-packages/flexget/task.py", line 696, in execute
    self._execute()
    |    -> <function Task._execute at 0x7f483ce47b70>
    -> <flexget.task.Task object at 0x7f4838ec6da0>

  File "/usr/local/lib/python3.6/dist-packages/flexget/task.py", line 662, in _execute
    self.__run_task_phase(phase)
    |                     -> 'input'
    -> <flexget.task.Task object at 0x7f4838ec6da0>

  File "/usr/local/lib/python3.6/dist-packages/flexget/task.py", line 488, in __run_task_phase
    response = self.__run_plugin(plugin, phase, args)
               |                 |       |      -> (<flexget.task.Task object at 0x7f4838ec6da0>, {'what': [{'movie_list': 'Default'}], 'from': [{'newznab': {'website': 'https:...
               |                 |       -> 'input'
               |                 -> <PluginInfo(name=discover)>
               -> <flexget.task.Task object at 0x7f4838ec6da0>

> File "/usr/local/lib/python3.6/dist-packages/flexget/task.py", line 521, in __run_plugin
    result = method(*args, **kwargs)
             |       |       -> {}
             |       -> (<flexget.task.Task object at 0x7f4838ec6da0>, {'what': [{'movie_list': 'Default'}], 'from': [{'newznab': {'website': 'https:...
             -> <Event(name=plugin.discover.input,func=on_task_input,priority=128)>

  File "/usr/local/lib/python3.6/dist-packages/flexget/event.py", line 20, in __call__
    return self.func(*args, **kwargs)
           |    |     |       -> {}
           |    |     -> (<flexget.task.Task object at 0x7f4838ec6da0>, {'what': [{'movie_list': 'Default'}], 'from': [{'newznab': {'website': 'https:...
           |    -> <bound method Discover.on_task_input of <flexget.plugins.input.discover.Discover object at 0x7f483bbdb4a8>>
           -> <Event(name=plugin.discover.input,func=on_task_input,priority=128)>

  File "/usr/local/lib/python3.6/dist-packages/flexget/plugins/input/discover.py", line 290, in on_task_input
    return self.execute_searches(config, entries, task)
           |    |                |       |        -> <flexget.task.Task object at 0x7f4838ec6da0>
           |    |                |       -> [<Entry(title=The Professor (2019),state=undecided)>, <Entry(title=Das hält kein Jahr ..! (2013),state=undecided)>, <Entry(ti...
           |    |                -> {'what': [{'movie_list': 'Default'}], 'from': [{'newznab': {'website': 'https://scenenzb.com', 'apikey': '2cab77fa83b00a8775f...
           |    -> <function Discover.execute_searches at 0x7f483bf7fea0>
           -> <flexget.plugins.input.discover.Discover object at 0x7f483bbdb4a8>

  File "/usr/local/lib/python3.6/dist-packages/flexget/plugins/input/discover.py", line 132, in execute_searches
    search_results = search.search(task=task, entry=entry, config=plugin_config)
                     |      |           |           |             -> {'website': 'https://scenenzb.com', 'apikey': '2cab77fa83b00a8775f429406472f9f4', 'category': 'movie', 'url': 'https://scenen...
                     |      |           |           -> <Entry(title=The Lord of the Rings: The Two Towers (Extended) (2002),state=undecided)>
                     |      |           -> <flexget.task.Task object at 0x7f4838ec6da0>
                     |      -> <function Newznab.search at 0x7f4839f7c840>
                     -> <flexget.components.sites.sites.newznab.Newznab object at 0x7f483960e128>

  File "/usr/local/lib/python3.6/dist-packages/flexget/components/sites/sites/newznab.py", line 87, in search
    return self.do_search_movie(entry, task, config)
           |    |               |      |     -> {'website': 'https://scenenzb.com', 'apikey': '2cab77fa83b00a8775f429406472f9f4', 'category': 'movie', 'url': 'https://scenen...
           |    |               |      -> <flexget.task.Task object at 0x7f4838ec6da0>
           |    |               -> <Entry(title=The Lord of the Rings: The Two Towers (Extended) (2002),state=undecided)>
           |    -> <function Newznab.do_search_movie at 0x7f4839f7c950>
           -> <flexget.components.sites.sites.newznab.Newznab object at 0x7f483960e128>

  File "/usr/local/lib/python3.6/dist-packages/flexget/components/sites/sites/newznab.py", line 122, in do_search_movie
    imdb_id = arg_entry['imdb_id'].replace('tt', '')
              -> <Entry(title=The Lord of the Rings: The Two Towers (Extended) (2002),state=undecided)>

AttributeError: 'NoneType' object has no attribute 'replace'
2019-12-25 23:06:56 CRITICAL manager       Usenet_movies_Search An unexpected crash has occurred. Writing crash report to /home/snikay/.flexget/crash_report.2<IP_ADDRESS>656135181.log. Please verify you are running the latest version of flexget by using "flexget -V" from CLI or by using version_checker plugin at http://flexget.com/wiki/Plugins/version_checker. You are currently using version 3.0.23
2019-12-25 23:06:56 WARNING  task          Usenet_movies_Search Aborting task (plugin: discover)

Issue template was ignored by OP. Please fill it so we can help!

The issue template is here because it helps to ensure you submitted all the necessary information the first time, and allows us to more quickly review issues. Please fill it out correctly and do not ignore it, no matter how irrelevant you think it may be. Thanks in advance for your help with this!

Additional information:

FlexGet version:
Python version:
Installation method:
Using daemon (yes/no):
OS and version:
Link to crash log:

NOTES:
Please verify that the following data is present before submitting your issue:

Link to a paste service or paste above the relevant config (preferably full config, including templates if present). Please make sure the paste does not expire, if possible.
Link to a paste service or paste above debug-level logs of the relevant task/s (use flexget -L debug execute --tasks <Task_name>).
FlexGet version (use flexget -V to get it).
Full Python version, for example 2.7.11 (use python -V to get it).
Installation method (pip, git install, etc).
Whether or not you're running FlexGet as a daemon.
OS and version.
Attach crash log if one was generated, in addition to the debug-level log. It can be found in the directory with your config file.


the images i use become huge while trying to get this to work

short description of the bug / issue, provide more detail below.
when i am using $('.single-item').slick(); the images that im using become insanely big and they are not like that to begin with. it seems to only effect it when im not at the size of an iphone. please help
====================================================================

[  paste your jsfiddle link here  ]
use this jsfiddle to reproduce your bug:
http://jsfiddle.net/simeydotme/fmo50w7n/
we will likely close your issue without it.
====================================================================
Steps to reproduce the problem

...
...

====================================================================
What is the expected behaviour?
...
====================================================================
What is observed behaviour?
...
====================================================================
More Details

Which browsers/versions does it happen on?
Which jQuery/Slick version are you using?
Did this work before?

I've observed the same issue. Also happens it it's just text.


Add support for Strings that are for a specific flavor (Android Gradle projects)

I don't think this is useful. Closing for now.


Wrong flag for Honduras (HN)

https://en.wikipedia.org/wiki/Flag_of_Honduras
Central star is not centered.

Cool stuff man! :) Thanks..


Add error emission over a certain row threshold or query execution time

Warnings often get ignored so emit errors over certain thresholds (these would just be logged errors, not thrown exceptions).
The other thing is to throw an exception over even higher thresholds.

Done.


The tmt.cli.main() method has been moved

Related: https://github.com/teemtee/tmt/pull/3193

Well, yes, until the tmt repo branch is merged. Otherwise we get red jobs here.


Feature: Support for Mac OSX

Hi Arthur,
I came across this great repo and was wondering how this could be used on Mac OS.
Could you please kindly point out whether it's possible, or the way to use it?
Thanks!

well the core code should run on mono but the UI is written in WPF so it needs to be converted to winforms or create an asp.next site from it...
also if you are interested in the one note converter it may not work well as each one note version create a slightly different html.

Okay I got it!
Thanks!


v4 Jamf sample schema 'Soft Days' key is incorrect

v4 Jamf sample schema 'Soft Days' key is incorrect- currently 'DeadlineSoftDays' but should be 'DeadlineDaysSoft'
https://github.com/Macjutsu/super/blob/4.0.0-beta1/Example-MDM/Jamf-Pro-External-Application-Custom-Schema-com.macjutsu.super-v4.0.0.json

https://github.com/Macjutsu/super/releases/tag/v4.0.0-beta2


Connect to signals of the shown window

In the case of OnboardingWindow and MainWindow, the class itself is not shown as a window, but its window field is shown, so when the window is closed, the application does not terminate - the wrong signals is connected to the handler.

I corrected it a little, because it is not always called with all the args. (An error occurred when clicking the "Cancel" button in the wizard.)


[CAS-1002] Bug in X509 when DN attributes contains periods

Any DN attribute that contains a period is truncated in X509CertificateCredentialsToIdentifierPrincipalResolver. Example:
CN="FIRST.SECOND.THIRD"
value="$CN" return "FIRST" only
Version 3.4.7 would return "FIRST.SECOND.THIRD" as expected.
Environment: Linux
Reported by: Alex Cabello, id: mixxen
Created: Thu, 14 Jul 2011 02:03:36 -0700
Updated: Mon, 15 Aug 2011 19:06:08 -0700
JIRA: https://issues.jasig.org/browse/CAS-1002

No longer applicable. Component is removed.


Google Analytics

Have you considered using Google Analytics?
Can help improve the performance and relevance, also really easy to use.
react-native-google-analytics-bridge

Hey, thanks! We're using basic Fireabse Analytics.


boards: rv32m1_vega: Introduce zero-riscy configuration

Introduces a new rv32m1_vega board configuration for the zero-riscy
core. It assumes that the soc has been reconfigured with openocd to boot
to the zero-riscy core instead of the ri5cy core.
Refactors the board-level device tree so the ri5cy and zero-riscy
configurations share common definitions for the led and button nodes.
Tested with:

samples/hello_world
samples/synchronization
samples/basic/blinky
samples/basic/button

checkpatch failure is the same as https://github.com/zephyrproject-rtos/zephyr/pull/11200#issuecomment-437004690, which was overridden

rebased to fix merge conflict

rebased to fix merge conflict


Unable to run make generate

I'm trying to update the heroku.go with the current latest schema but I'm running into the following error:
cd v5 && ../script/generate
+ : 1
+ getopts hnv: opt
+ API_VERSION=3
+ CLIENT_VERSION=v5
+ SCHEMA_FILE=v5/schema.json
+ GENERATED_FILE=v5/heroku.go
+++ dirname ../script/generate
++ cd ../script/..
++ pwd
+ NEW_DIR=/dev_exclusions/heroku-go
+ cd /dev_exclusions/heroku-go
+ '[' 1 -eq 1 ']'
+ curl -so v5/schema.json https://api.heroku.com/schema -H 'Accept: application/vnd.heroku+json; version=3'
+ go get -u github.com/interagent/schematic/cmd/schematic
go: finding github.com/interagent/schematic/cmd/schematic latest
go: finding github.com/interagent/schematic/cmd latest
go: finding github.com/interagent/schematic latest
+ schematic v5/schema.json
Error processing schema element:
    {
      "description": "certificate provided by this endpoint",
      "properties": {
        "ca_signed?": {
          "readOnly": true,
          "type": [
            "boolean"
          ]
        },
        "cert_domains": {
          "readOnly": true,
          "type": [
            "array"
          ]
        },
        "expires_at": {
          "readOnly": true,
          "format": "date-time",
          "type": [
            "string"
          ]
        },
        "id": {
          "description": "unique identifier of this SSL certificate",
          "readOnly": true,
          "example": "01234567-89ab-cdef-0123-456789abcdef",
          "format": "uuid",
          "type": [
            "string"
          ]
        },
        "issuer": {
          "readOnly": true,
          "type": [
            "string"
          ]
        },
        "self_signed?": {
          "readOnly": true,
          "type": [
            "boolean"
          ]
        },
        "starts_at": {
          "readOnly": true,
          "format": "date-time",
          "type": [
            "string"
          ]
        },
        "subject": {
          "readOnly": true,
          "type": [
            "string"
          ]
        }
      }
    }

Failed with: unknown type <nil>
make: *** [generate] Error 1

This has happened to me in the past and if I recall correctly, the API schema needed to be updated.
@bernerdschaefer @mars Could one of you check if the schema is correct?

@davidji99 Thanks for the report, I'll look into this today!

@davidji99 Here's one fix if you've got time to review: https://github.com/interagent/schematic/pull/55. I'm still assembling a PR to the schema itself to fix the other issue.

@davidji99 The corrected schema is also live in production, so this should now be good to go!

Thanks! Will try make generate again.


Change user in base docker image to be non-root

I'd like to see the user in the base docker image to not default to root and use the /root workdir. Can that be changed to create a specific non-root user to avoid potential security risks with having containers running as root?
Update: Created a PR that solves this.

Awesome thanks a lot for your work. Will release this now. Already closing this issue.


Wonder about lrn parameters.

Hi.
I am curious about the 'local response normalization' layer parameters you set.
The paper case is a little bit different from what you set except 'beta'.
What does it mean to be different from the actual setting?

Hello @enterAI,
for the hyperparameters I just followed the working pre-trained implementation here. Did not have the chance to test other parameter setups. If you have the opportunity than let me know the results 👍🏻.


replace GetWorkflowExecutionNextEventID with DescribeWorkflowExecution

remove GetWorkflowExecutionNextEventID from history API
use DescribeWorkflowExecution in place where GetWorkflowExecutionNextEventID was needed.
no behavior changes.

Coverage increased (+0.2%) to 66.122% when pulling 5cf3c08374a42219e328b445f2754dd6ef6d6dab on yiminc:remove_GetNextEventID into 6b3fcff0074dafa1be8128427d577806f9be8385 on uber:master.


Java-MicroProfile: Required "paths" field is missing

when i run microprofile i am noticing the following error...
[Container] [INFO] [ERROR   ] CWWKO1650E: Validation of the OpenAPI document produced the following error(s):
[Container] [INFO]
[Container] [INFO]  - Message: Required "paths" field is missing or is set to an invalid value, Location: #
[Container] [INFO]
[Container] [INFO] [AUDIT   ] CWWKT0016I: Web application available (default_host): http://2f00029ff319:9080/
[Container] [INFO] [AUDIT   ] CWWKZ0001I: Application starter-app started in 3.046 seconds.
[Container] [INFO] [AUDIT   ] CWWKF0012I: The server installed the following features: [appSecurity-2.0, cdi-2.0, concurrent-1.0, distributedMap-1.0, jaxrs-2.1, jaxrsClient-2.1, jndi-1.0, json-1.0, jsonb-1.0, jsonp-1.1, jwt-1.0, microProfile-2.2, mpConfig-1.3, mpFaultTolerance-2.0, mpHealth-1.0, mpJwt-1.1, mpMetrics-1.1, mpOpenAPI-1.1, mpOpenTracing-1.3, mpRestClient-1.2, opentracing-1.3, servlet-4.0, ssl-1.0].
[Container] [INFO] [AUDIT   ] CWWKF0011I: The defaultServer server is ready to run a smarter planet. The defaultServer server started in 14.130 seconds.

I am still able to access the endpoint despite the error so I don't think its a critical error.

@Emily-Jiang Would like to know your thoughts on this issue

this is not really an appsody issue. It is how microprofile deals with OpenAPI where no resources are defined. We do not want to define resources and have users delete it.

Hi @Emily-Jiang Just ran into this myself running the MicroProfile stack.  What about having an "example" resource as part of the template that's clearly labeled as such?

hi @neeraj-laad - can you please clarify which endpoint you are referring to in terms of being accessible?
Endpoints such as /health and /openapi are from Liberty itself, not the app.   This error is saying that the application is not exposing any endpoints, which seems to be true.

Since every app will need at least the 1 resource, then I think it's reasonable to ship a template that has 1 resource, so the user can easily customize it, instead of creating it from scratch.
CC @Emily-Jiang / @seabaylea


Added missing curly braces to example in readme

@tricoder42 can you review and accept the CLA for this repo so we can merge your PR? https://cla.nfl.com/agreements/nfl/react-helmet

Signed :pencil2: :page_facing_up: :white_check_mark:


Bump version to 0.4.2

Update hubot-stackstorm dependency version to 0.9.4+ and bump st2chatops version to 0.4.2

Apologies. This isn't the first time this has bitten me. I'll add a check for that in the Circle configuration so we don't run into this issue again.


Log full error when fails to fetch member

Small update to myst's patch.

Did you try to get an error there?
Only thing which could throw an error there is disabled intents ? Or am I wrong ?


Update openemu appcast checksum

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Updating appcast on the same version of openemu.

Please add this to https://github.com/caskroom/homebrew-cask/pull/40341

@commitay, how do I do this? #40341 has been merged :)

Sorry, I forgot about this when I merged https://github.com/caskroom/homebrew-cask/pull/40341
We have been trying to discourage checkpoint only PRs, the checkpoint doesn't affect the installation of a Cask.

Thanks for clearing this up!
I will update the checkpoint along with the next release of openemu whenever it is available.


Added Parallel Count-Min-Sketch

What is this PR?
Implementation of parrallel Count-min-sketch according to research paper "Parallelizing Count-Min Sketch Algorithm on Multi-core Processors" by Bowen Yu, Yu Zhang, Lubing Li. https://www.atlantis-press.com/proceedings/mmebc-16/25859036.
I will be glad if someone could review my implementation of idea stated in the research paper.

Thanks for the PRs, @ilnarag. I'll get this and #22 looked at hopefully in the next couple of days.


Error while opening setup

Followed instructies as written. When opening setup.bat I get this error.
Keep in mind I am no programming or coding hero.

Disclaimer

The Public Version isn't maintained anymore so you're bound to run into more errors. Getting the Premium Version is highly recommended.

You need to change the contents of requirements.txt to just discord.
This change is necessary since the rewrite version of discord.py is now the official one.


Remove detailed guide categories from database

Detailed guide categories have been removed from the application two weeks ago.
Time to delete the database tables and columns too.
https://github.com/alphagov/whitehall/pull/2283

:+1:


Vertical Height of Pane Restricted by Other Pane

Am I doing something wrong or is this an issue? This behavior seems counter intuitive to me.
The unwanted behavior is the limiting of the vertical size of the lower right side pane.
Rather than try to use words, and risk confusion, here's video:
https://user-images.githubusercontent.com/20944949/116560813-068f1080-a902-11eb-93d2-4fc4f1779e9a.mp4

Origami just creates panes using the ST API and doesn't control the resizing. So I'd suggest reporting this to the ST bug tracker if it's not already reported.


No module named '_ext'

Thank you for your awesome work. I have followed all steps for training your model; however, I encountered the following error which happens for importing your model in line 9 of main.py (import models). Although it seems that models have not been used anywhere in main.py, when I commented import models, another error would occur in line 315 in create_model of main.py.
I would be grateful if you could guide me to deal with this issue.
Traceback (most recent call last):
  File "/home/amir/Documents/PS-ViT/main.py", line 9, in <module>
    import models
  File "/home/amir/Documents/PS-ViT/models/_init_.py", line 1, in <module>
    from .ps_vit import *
  File "/home/amir/Documents/PS-ViT/models/ps_vit.py", line 4, in <module>
    from timm.models.helpers import load_pretrained
  File "/home/amir/anaconda3/envs/ps_vit/lib/python3.7/site-packages/timm/_init_.py", line 2, in <module>
    from .models import create_model, list_models, is_model, list_modules, model_entrypoint, \
  File "/home/amir/anaconda3/envs/ps_vit/lib/python3.7/site-packages/timm/models/_init_.py", line 1, in <module>
    from .ps_vit import *
  File "/home/amir/anaconda3/envs/ps_vit/lib/python3.7/site-packages/timm/models/ps_vit.py", line 8, in <module>
    from layers import ProgressiveSample
  File "/home/amir/Documents/PS-ViT/layers/_init_.py", line 1, in <module>
    from .progressive_sample import ProgressiveSample
  File "/home/amir/Documents/PS-ViT/layers/progressive_sample.py", line 9, in <module>
    'progressive_sampling_backward'])
  File "/home/amir/Documents/PS-ViT/utils/ext_loader.py", line 5, in load_ext
    ext = importlib.import_module(name)
  File "/home/amir/anaconda3/envs/ps_vit/lib/python3.7/importlib/_init_.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_ext'

Process finished with exit code 1

Please run python setup.py build_ext --inplace to compile C++ and CUDA code.


Add Native Code for Android and iOS

##This PR includes:

The native implementations for getting a contact by their id(Both OS)
Updates android add contact function to return the contact for AsyncStorage use
Adds/created a android delete function.

##Notes:

I worked on the changes to these files in out React Native app and sym-linked only the files I changed.

@blenderer @nprz if you want to take a look at some ObjC and Java have at it. Would like to get @MPiccinato to merge if he approves.

@blenderer I did not see this, but I could.


PageFind for users in China

First, this PR has PageFind search available for all users OUTSIDE of China. This is just temporary to facilitate testing and we will revert this before merging the PR. The goal is to show PageFind to all users IN China, as a replacement for Bing, which has been costing CNCF a lot of money recently.
Could someone help integrate this into the Makefile properly. I’m not sure where it should be applied and how to integrate it. You can see what I’ve done to get PageFind to run in the netlify.toml commands but that may not be the right place for them.
Searches with PageFind can be compared against the current site. If they are adequate, then we can get this deployed soon to remove Bing from the site. We should specifically test searches in Chinese but also test English and other languages.
We can enhance the PageFind search functionality in the future with filters, sorting, etc. If the PageFind results become better than those of Google Custom Search, we can have it serve all users, not just those in China.
I pulled ideas from #44655 when creating this PR but started from scratch as I couldn’t get that PR branch to run locally. I also kept the scope narrower and less ambitious, limiting PageFind search to just Chinese users while keeping Google Custom Search for everyone else. This was so that we could address the immediate need of getting off Bing as quickly as possible.
FYI this is the command I use to test this locally:
hugo && npx -y pagefind --site public --serve

Hi @cjyabraham
If I go to https://deploy-preview-46768--kubernetes-io-main-staging.netlify.app/zh-cn/docs/home/ and search for kubectl 命令行工具, I get pages in English. To be fair, I'm using a browser that prefers English but I was looking at the Chinese docs. This may well surprise users.
https://deploy-preview-46768--kubernetes-io-main-staging.netlify.app/search/?q=kubectl 命令行工具

Actually now that I think of it, it's a known issue with doscy which I have already submitted a fix for in this PR. Since that's not merged yet, we can reapply the fix locally within this site...

Ok, I've fixed that issue so now the site should correctly keep the user on the search page of the same language they were on. This is going to be a problem, however, for languages that don't have search pages, like French, so perhaps we should quickly add content files for them?

/cc @windsonsea @tengqm

We shouldn't add “localized” pages where the content is in English. Let's find another way to make that work.
For example, change the search form to post to the localized search page iff there is one; if there isn't post to the English search page.

The only string that needs translating on the new search pages are the words "Search Results" which appears as the <title> for the page. Everything else is in the correct language. Would you be open to us generating a translation for these strings? That way we would be able to immediately provide search results in the languages in question.

The only string that needs translating on the new search pages are the words "Search Results" which appears as the <title> for the page. Everything else is in the correct language. Would you be open to us generating a translation for these strings? That way we would be able to immediately provide search results in the languages in question.

I don't recommend it; you will need to get 14 approvals (base approval plus 13 translation signoffs), and this is not going to be straightforward.
See https://github.com/kubernetes/website/pull/46768#pullrequestreview-2110222558 for the approach I recommend instead. If you're not sure how to code that, it's OK to ask for help with this part - #sig-docs on Slack is another good place to take that conversation.

I don't recommend it; you will need to get 14 approvals (base approval plus 13 translation signoffs), and this is not going to be straightforward.
See #46768 (review) for the approach I recommend instead. If you're not sure how to code that, it's OK to ask for help with this part - #sig-docs on Slack is another good place to take that conversation.

Ok, I've implemented your suggestion.

Please squash out the commits that touched localized files @cjyabraham - we can then take those teams out of the approval process.

Please squash out the commits that touched localized files @cjyabraham - we can then take those teams out of the approval process.

Ok, done.

/language en
/remove-language de
/remove-language es
/remove-language fr
/remove-language hi
/remove-language it
/remove-language ja
/remove-language ko
/remove-language pl
/remove-language pt
/remove-language ru
/remove-language uk
/remove-language vi
/remove-area localization

Before we approve it, I'd also like to see this PR squashed ready for merge.

Before we approve it, I'd also like to see this PR squashed ready for merge.

So do you want me to flip the switch so that PageFind is now only accessible to users in China? Or is there still some testing that needs to happen for approvals? Also, what's the advantage of squashing before merge? Can't that happen during the merge operation?

Squashing for merge will turn the many commit messages into a more readable summary of the change. That really helps people who come back to look at the code in the future, and who want to understand the thinking behind the change(s).
Using /label tide/merge-method-squash doesn't rewrite the commit messages; Tide isn't capable of summarizing text.
You should squash after this is flipped to only serve to Chinese visitors. You could squash before, as well.

The next thing this needs is:

testing by people who can evaluate the Chinese-language search (many users in China will use that localization)
testing by people who can evaluate the English-language search (some users in China will use that localization)

The preview looks good from Shanghai, China:

search zh pages
search en pages

The preview looks good from Shanghai, China:

search zh pages
search en pages

@windsonsea can you confirm that you're getting PageFind search results and not Google Search results? I currently have flipped the logic so that those outside China get PageFind and those inside get Google. But maybe you're on a VPN?

More feedback is welcome

@sftim What does this still need before we can push it live?

What does this still need before we can push it live (apart from me switching the IP check algo back to normal and squashing commits)?

It needs a review (anyone representing SIG Docs) and an approval (SIG leadership). Evidence that this works for users is also helpful.

I also recommend squashing it to fewer commits.

Ok, done.

Commits fd7d531f69b31d2d340bfe5d745bafb7b33c0926 and bede881988db62e829f139c9ed4e8e6590d24915 don't look right.
I can make a branch that has the commits I think you'd want, if that's useful @cjyabraham?

Commits fd7d531 and bede881 don't look right.
I can make a branch that has the commits I think you'd want, if that's useful @cjyabraham?

Yes, I'm not sure how those commits got in this branch. Sure, please go ahead and do whatever is needed. Thanks.

/remove-language en
/remove-language fr
/remove-language id
/remove-language ja
/remove-language pt
/remove-language zh

/remove-area localization blog

See https://github.com/kubernetes/website/pull/46768#discussion_r1650090961 for some feedback. I don't think it blocks a merge, but it could have an effect on end-user experience.

Ready for more feedback from people likely to use this (ie, the far side of China's nation-level content filtering).

/lgtm

See #46768 (comment) for some feedback. I don't think it blocks a merge, but it could have an effect on end-user experience.

LGTM from China.

/lgtm cancel
(need to swap the sense of the change, now it's testing)
But this does actually LGTM
@cjyabraham please rebase out or revert the commit pretend we are in China to test this, and then we may be able to get further testing done by users in China.

@cjyabraham please rebase out or revert the commit pretend we are in China to test this, and then we may be able to get further testing done by users in China.

It was not a clean revert of one commit as a couple changes were needed, so I made a new commit.

(happy for me to squash this down @cjyabraham ?)

Please do. You seem to be particularly good at this. And if you know of a good resource where I can learn more about it then please send it over :)

/hold cancel

@sftim what is the next step here?

what is the next step here?

This PR is ready for review. It needs an LGTM, ideally a bit more evidence that it doesn't break things for users not behind state level censorship, and a website approver must approve the change.
If you use Kubernetes Slack, you can nudge #sig-docs about reviews and approval.

I'm not on Kubernetes slack. Can we tag people here that need to look at this?

Can we tag people here that need to look at this?

Sure. Write a message and tag @kubernetes/sig-docs-leads in your message.

Can we have this PR reviewed please @kubernetes/sig-docs-leads

I've pinged folks in the Kubernetes Slack channels #sig-docs, #sig-docs-localizations, and #kubernetes-docs-zh to see if they can help review.

/lgtm

/hold
This is also serving PageFind to users outside of China

@sftim it seems that your rebase operation removed my last commit. Could you undo that?

/lgtm

I'll update this PR when I get a moment.

/hold cancel

The PageFind functionality is only visible to users in China. However, you can check this out locally and try it by editing the search.js file; see https://github.com/kubernetes/website/compare/c86a1576dedaa56d1fce1ef0cbb7dda149477ba2..a4191d94ea52f8c4935c144f0d89d779b16f5812

/hold
@kubernetes/steering-committee howdy steering! this change has come about out of the CNCF's need to replace Bing for users in China, and our zh-localization community, alongside other folks in SIG Docs, have reviewed this change to ensure it's doing what it describes. we want to check if this needs to be run by you before merging – thanks!

LGTM on my side, I don't think it needs steering sign off :+1:

https://github.com/kubernetes/website/pull/46768#issuecomment-2207446787 Agree.

Feel free to unhold, or wait for a simple majority of steering (routine business rules would apply if a vote were needed for something like this), but I pretty strongly think this does not need to wait for steering and is delegated to SIG Docs.
(re: delegation, maybe reach out to SIG K8s Infra if we have new paid SaaS accounts to manage but AFAICT this is not the case here and still wouldn't involve steering)

sick, thanks @mrbobbytables and @BenTheElder!
/hold cancel
/lgtm

Thanks for this @cjyabraham & @sftim, and thanks to all the reviewers!
/approve

/lgtm

/assign natalisucks

/approve


Push for v0.0.1

I would really like to just get a first version of this library out so rather than try to overhaul the entire underlying parsing/rendering system before v0.0.1 I have decided to, instead, limp on using the existing infrastructure so that I can push out a v0.0.1 and people can start using it WHILE I overhaul the underlying infrastructure... The following list is what I MUST have for v0.0.1:

[ ] CSVelte\Reader - At least a way to loop over a CSV file and read each "cell"
[ ] CSVelte\Writer - Accept an array or array of arrays and write it to an output source (generally a file).
[ ] CSVelte\Taster - A way to automatically detect CSV format (or flavor or dialect... whatever you want to call it).
[ ] CSVelte Convenience Class - The CSVelte class will be used as a convenience adapter to simplify complex functionality.

I'm going to branch off the branch I'm using right now for my v0.0.1 push


Handle install-space state and behavior packages

See team-vigir/flexbe_behavior_engine#56 for further details.

I patched this in our branch just a few days ago, PR now submitted:
https://github.com/FlexBE/flexbe_app/pull/20

Functionality has been merged.


Rename French sources

Remove accents (which were screwing up git/shell), change to underscore instead of dash.

St. Pierre et Miquelon is a skip, not a real failure.


Does anyone know to use Card with BrainTree hosted fields?

hi
first thanks for this lib :)
i try yo use it with braintree hosted fileds (wich load input in iframes) so not from the same domain and so i can t link input to your card library ... debug tell that it can t find inputs...
but i have full events on my inputs thru braintree script...
have an idear on a workaround ?
thanks

I don't have a workaround or known solution for this, but I've tagged it "question," so someone else may come along and help!

+1. Would be great to have some sort of solution for this.

@smolleyes have you solve the problem?


Infinite loop when opening malware sample

Binary Ninja Version: 1.1.1555, 1d59e7dc
Platform: Ubuntu 18.10
Malicious sample attached in archive causes and infinite loop when opening in binaryninja. Password is "infected".
sample.zip

Fixed in build 1.1.1567-dev.


az bot publish fails using enterprise template due to CLI error

Tool
Name: Azure CLI
Version: 2.0.53 (bot extension 0.4.3)
Python (Windows) 3.6.6 (v3.6.6:4cf1f54eb7, Jun 27 2018, 02:47:15) [MSC v.1900 32 bit (Intel)]
OS: Win 10
Describe the bug
Unable to publish bot service using az bot publish
To Reproduce
I have followed the installation instructions for enterprise template bot

Installed latest CLI, NPM, LUISGen etc accoring to this link
https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-enterprise-template-deployment?view=azure-bot-service-4.0

Ran the following with my parameters for YOUR_BOT_NAME, YOUR_AUTHORING_KEY, LOCALE_FOLDER, REGION
msbot clone services --name "YOUR_BOT_NAME" --luisAuthoringKey "YOUR_AUTHORING_KEY" --folder "DeploymentScripts\LOCALE_FOLDER" --location "REGION

It creates everything in azure and updates my local files and then generates this error

Command failed: az bot publish --resource-group MYBOTNAME-n MYBOTNAME --subscription xxxxxxx-xxxxx-xxxxx-xxxxx-xxxxx -v v4 --proj-file "MYBOTNAME.csproj" WARNING: The behavior of this command has been altered by the following ex  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init_.py", l  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init_.py", l  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init_.py", l  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init_.py", l  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init_.py", l  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core_init_.py", line 442, TypeError: init() missing 1 required positional argument: 'logger'bot)py", line 86, in publish_app

I tried to run the publish by itself after using " az bot publish -g MYBOTNAME -n MYBOTNAME --proj-file MYBOTNAME.csproj --version v4" then I got this:

init() missing 1 required positional argument: 'logger'
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\knack\knack\cli.py", line 206, in invoke
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init_.py", line 328, in execute
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init_.py", line 388, in run_jobs_serially
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init.py", line 381, in run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\six\six.py", line 693, in reraise
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init.py", line 357, in run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core\commands_init.py", line 171, in call
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-1r22ouqy\azure-cli-core\azure\cli\core_init_.py", line 442, in default_command_handler
File "C:\Users\jonorlu.azure\cliextensions\botservice\azext_bot\custom.py", line 86, in publish_app
kudu_client = KuduClient(cmd, resource_group_name, resource_name, bot)
TypeError: init() missing 1 required positional argument: 'logger'
Expected behavior
A published bot in Azure
[bug]

@jonasnorlund Thanks for reporting this, we're looking into this now.

FYI: Instead of publishing the bot using CLI I did it directly from within VS and it worked fine.

Closing due to inactivity as this has been addressed with the latest public release of msbot, version 4.3.2. This version of msbot does not use the botservice extension, so you are not required to call az extension add -n botservice.
If you still run into this issue, please @mention me in this issue or open up a new issue.


addPeriodicTimer() blocks the execution of my code

Hi folks!
I have a code like this:

  	if ($message->content == 'startrare') {
  		$loop->run($timer);
  	}

When if = true, loop runs and my code stops executing. Acts like a traditional php while/foreach...
Loop code looks like this:

$loop = React\EventLoop\Factory::create();
$i = 0;
$timer = $loop->addPeriodicTimer(5, function() use (&$myvar0, $loop, &$myvar1, &$myvar2, &$myvar3) {
echo 'OK';
...

I receive 'OK' each 5 seconds, but other code after (or before) loop not working until loop stops :/
http://pastebin.com/5Gq0ujMu

When I run the loop (startloop message on channel), loop runs correctly but DiscordPHP stops listening for any other command!

I'm rewriting the code, will pass a custom Loop with DiscordPHP: 'loop' => $mycustomloop
the problem now is, I can't retrieve any $variable value inside my $mycustomloop
https://github.com/reactphp/event-loop/issues/54#issuecomment-236435544

Are you still having this issue?


[Backport 1.30] Fixed bazel configuration ambiguity for non HTTP3 builds (#34483)

Back port of #34483 to release/v1.30
To avoid ambiguity when building with //bazel:http3=False (on PPC and Windows) this commit adds new config_setting_groups and updates the relevant select() statements.

@phlax how can I make this rerun the checks now that the branch is reopened?

/retest


Silence E501 (Flake8 line length) in tests

In general code in tests does not need to abide by any line length requirements.
This change silences Flake8 warnings about longer lines in test_sesssion.py so other important warnings are not overlooked in the crowd.

Great!


How to report framework security issue?

I send a mail to<EMAIL_ADDRESS>But mail sending Delivery is incomplete due to DNS Error.
I am discover a critical security bug in framework. After founding bug I am directly send a email to this address with bug details but mail is not sending.
Now how I report this issue privatly ?

I'll look into the DNS error. Thanks for bringing that to our attention.
In the meantime you can email me directly at<EMAIL_ADDRESS>


Add Release Notes page for the guest app

Feature Description

Add a release notes page similar to https://github.com/ncosd/food-pantry-app/blob/main/web/admin/src/views/ReleaseNotesPage.vue in the guest app.
Expected solution

Add the page in https://github.com/ncosd/food-pantry-app/blob/main/web/app/src/views/
Add the route to the page
Link the version number in the page footer to the release notes anchor in the release notes page similar to how it is done in the admin app.
Testing

Clicking the version number in the page footer takes you to the release notes page.

@dherbst , can I work on this?

@rakshitriya yes, go for it!

Hi, I have defined the variable in .env file, however, I am getting this:
 VITE v4.3.9  ready in 488 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
  ➜  press h to show help
(!) %VITE_PROJECT_LONG_NAME% is not defined in env variables found in /index.html. Is the variable mistyped?

Double check the .env file is in web/app/.env.
The .env file should look similar to the file https://github.com/ncosd/food-pantry-app/blob/main/web/app/env.demo

I am still not able to set it up, with the same error.
pasting here the .env files

and then
#from  web/app directory
npm run serve:firebase:emulator
npm run dev

Does this setup look fine to you?  I didn't understand firebase use dev command because it said that the project didn't exist, so I used firebase use food-pantry-app-os.
Please let me know if you'd require any more information to debug this.
Thanks,
Riya

The first .env file in the first image, it is in web/app/src/.env.  Move it up one folder to web/app/.env and that should fix the problem.

That is helpful!
I know this is sad but there's another issue: The server doesn't respond anything. It's a blank homepage -

I looked up this https://stackoverflow.com/questions/56158010/firestore-firebase-emulator-not-running, but found that the files - firebase.json, firestore.rules and firestore.indexes.json exist inside  web folder.
Am I missing out on something?

Yes, sorry the instructions said to put the variables in the .env.dev file - but they should be copied into the web/app/.env file and the web/admin/.env files instead.
Try moving the contents of the file in web/functions/.env.dev so they are in the web/app/.env file and copy them from web/admin/.env.dev into web/admin/.env
I've made a change to the instructions to reflect this in this commit: https://github.com/ncosd/food-pantry-app/commit/a9bf655909337c3201687efa699dbc34a3caf98f

And the firebase files are in the correct folder, yes.

@rakshitriya are you still working on this?  If not, I have time to finish it up.

Hey, yes I have been trying to set it up. Give me a day Or two more, else
you can pick if up. The page still shows blank.
On Sun, 7 Jan, 2024, 10:04 pm Darrel Herbst, @.***>
wrote:

@rakshitriya https://github.com/rakshitriya are you still working on
this? If not, I have time to finish it up.
—
Reply to this email directly, view it on GitHub
https://github.com/ncosd/food-pantry-app/issues/246#issuecomment-1880106811,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBO2M5DJGN4BFUHPZ4NC5BTYNLFCXAVCNFSM6AAAAABBA4AQ4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBQGEYDMOBRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@rakshitriya take another look at the https://github.com/ncosd/food-pantry-app?tab=readme-ov-file#setup in the readme - I changed the "Prepare the .env files" section to correct an error since you first looked at it.   Maybe start a fresh clone would help.

The page looks like:

Is this expected?
These are the files which didn't load:

Is this expected?

Yes!  It looks like you have it up and running.

Glad I could make it, please let me know if there's any issue with the PR 😀


Updating Jenkinsfile

Adding limits and requests.  Also fixing the make target.

Can you give me a GitHub review please?

done.
On Fri, Apr 19, 2019 at 8:59 AM Chris Love<EMAIL_ADDRESS>wrote:

Can you give me a GitHub review please?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/gke-managed-certificates-demo/pull/1#issuecomment-484941183,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGBUEEURITGEJRIVRQBF763PRHT55ANCNFSM4HHAEAGQ
.


Crappy fonts

The default font-family in style.css is:
font-family: FontAwesome, Roboto, Helvetica, Arial, sans-serif;
The FontAwesome package is installed.
Last metadata expiration check: 0:07:50 ago on Thu 07 Mar 2024 12:15:01 PM EST. Package fontawesome-fonts-all-1:6.5.1-1.fc39.noarch is already installed. Dependencies resolved. Nothing to do. Complete!
Nevertheless, the font and icons look like crap:

What do you suggest?
Here is my neofetch:

unsure if relevant but I had to specify an additional FontAwesome font in the CSS when I installed FontAwesome 6:
font-family: FontAwesome, Roboto, Helvetica, Arial, sans-serif, "Font Awesome 6 Free";


Additional instructions for SPServiceInstance.Provision Method

Note: You should run this command only from the server on which you would like to Provision the service instance.

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

ID: c916bf49-bbb1-8afe-9769-7e75dc159d5c
Version Independent ID: b9c88795-4fc0-0725-a728-0292046a0647
Content: SPServiceInstance.Provision Method (Microsoft.SharePoint.Administration)
Content Source: sharepoint-server/xml/Microsoft.SharePoint.Administration/SPServiceInstance.xml
Product: sharepoint
GitHub Login: @dotnet-bot
Microsoft Alias: o365devx

Good note, but closing as an issue. Thanks!


Permissions and metadata not copied

It would be great if the function applied the same permissions and metadata to the new image that existed on the original image.

:+1:
Currently I have public images with metadata applied to the original images. The new images are created with no public access and no metadata.  The lack of public access is a killer for me because it means that the newly created image can't be accessed.

Did you try adding the ACL prop to S3.putObject? Like
S3.putObject({
  Body: buffer,
  Bucket: BUCKET,
  ContentType: "image/png",
  Key: key,
  ACL: "public-read"
}).promise()

The demo code should apply a bucket policy to allow all objects in the bucket to be public. If you're not seeing that, @derN3rd's approach would also work nicely. Let me know if I can help!

Is there a way to dynamically set the ContentType to the correct image/type.  For example if I input a .jpg file the ContentType currently is set to image/png, but I'd like it to be image/jpeg.  And if I input a .png file I'd like the ContentType to be image/png, etc...

Today I've simplified to set all images to png for this reference code just to keep it small. You can certainly extend the project to properly maintain the file type through the resize and use the correct mimetype, but that's an exercise left to the reader. Hope that helps!


RSS links generates incorrectly

it generates:
xml:base="//hottake/2024/07/30/Friend_Partner/"

instead of
 xml:base="https://<website>/hottake/2024/07/30/Friend_Partner/"

could be user error?

incase: blog

Do you have the base url in the config file?

Do you have the base url in the config file?

I figured it out a long while ago without closing this issue, sorry.


Add an Akka Stream Sink graph stage for Kinesis.

Since there is now a Source as Kinesis Consumer, a Sink is needed to act as Kinesis Producer.
This PR adds a KinesisSinkGraphStage that uses an underlying KinesisProducerActor to do the heavy lifting. The stream manages back pressure by allowing only a fixed number of outstanding messages. A materialized value is used to indicate, when a stream has been finished (or failed).
A unit as well as integration test is in place.
Open topics:

[x] Can we use throttling conf for the number of outstanding messages or should this be a separate config option?
[x] The current actor based implementation does not limit the number of outstanding messages by default. This PR does not change this, but uses a default (of 1000 messages) if throttling conf is not available, since a defined maximum is a better default IMO. WDYT?
[x] Documentation in README.md is missing. Todo on my side.

🎉 🎉 🎉


Integration Test - Release - 2018/07/13 05:29:17

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest2 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/IntegrationTest/BuildIntegartionTestFiles/ChangeSentToAuthor.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added Orion Health api

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Thanks

My bad. The branch was not up to date

No problem. Thanks


feat(Split): Add Split funtionality

This change is

Closing this PR in favour of n-split implementation done in this PR


No perceptual clear the conversation context

This model's maximum context length is 4096 tokens. However, your messages resulted in 4119 tokens. Please reduce the length of the messages. Sorry, I am not feeling well. Please try again.
After exceeding the limit of tokens, we can only execute /clear to clear conversation context to restore function.
This seems to be the only option, so is it possible that sending the message again after reaching the limit will automatically execute the /clear command?

or automatically delete the earlier conversation?

We will implement this later.

I thick it can also be possible with embeddings. Another advantage can store days worth of conversations in case you need long term memory and brings the cost of conversations down by 5 times depending on usage. I've been reading on ways to implement it from these sources
Article:
https://towardsdatascience.com/generative-question-answering-with-long-term-memory-c280e237b144
Implementation examples:
https://github.com/openai/openai-cookbook/blob/main/examples/Question_answering_using_embeddings.ipynb
There's also a guide on pinecone too. A site one can utilize for the database
https://docs.pinecone.io/docs/openai


Ability to place Mirrors in seperate file

Is your feature request related to a problem? Please describe.
Having hundreds of line of code just to mirror data structs artificially bloats the api file
Describe the solution you'd like
An additional api_mirror file, that holds the mirrors
Describe alternatives you've considered
Creating multiple files, but as this feature is not officially supported (tested) it is not “safe”.
Additional context

Ok, and it also works with multiple C files for iOS Headers? I haven't got
that to work, maybe I did sth wrong
On Tue, 21 Mar 2023, 01:52 fzyzcjy, @.***> wrote:

Creating multiple files, but as this feature is not officially supported
(tested) it is not “safe”.
IIRC this is officially supported and pure_dart_multi tests it. If you
find anything breaking, feel free to file a bug or PR!
—
Reply to this email directly, view it on GitHub
https://github.com/fzyzcjy/flutter_rust_bridge/issues/1135#issuecomment-1477138425,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIBZJBY4N4DKBZ5CEHNHYWDW5D3WVANCNFSM6AAAAAAWBSVV6Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

Possibly related: https://github.com/fzyzcjy/flutter_rust_bridge/issues/1111#issuecomment-1477183483
Feel free to file an issue with more details of the bug you see!


Is the MSBuild task able to generate Fakes assemblies if MSBuild from the Visual Studio folder is used ?

Hello,
my question is concerning the marked section in this screenshot.

Will the MSBuild task be able to generate Fakes assemblies when i tell it to use MSBuild from a VisualStudio path like this one ?
C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin\MSBuild
Regards,
Holger

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

ID: 069d29c9-b269-6e1f-08af-91ac135ebb1e
Version Independent ID: cd993f76-8077-d566-4c48-0ab425c62b83
Content: Isolating Code Under Test with Microsoft Fakes - Visual Studio (Windows)
Content Source: docs/test/isolating-code-under-test-with-microsoft-fakes.md
Product: visual-studio-windows
Technology: vs-ide-test
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@impact79 -- Holger, thank you for your question.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@Mikejo5000 -- Mike, please look into this issue and related issue #7866.

@AbhitejJohn Abhitej, do you have any insight on this question?

Will the MSBuild task be able to generate Fakes assemblies when i tell it to use MSBuild from a VisualStudio path like this one
It would but it depends on what version of VS 2019 is used and what target framework the project targets. If the user is on 16.8 and above, things should work for both .NET Framework and .NET Core based projects. It would be great if we could get more details or a repro project if that isn't working for some reason.

Closing issue as currently we don't a repro