fix: escapeObject parameter order for Postgres dialect.

Found this bug when I added support for writing moment instances (which in turn I did to fix timezone issues I had when writing DATE types).
const moment = require('moment')

moment.fn.toPostgres = function () {
  return this.format()
}

thanks, this needs a test case too.

Can you suggest a suitable location where I can add a test case?

I found a location for a test @elhigu.  Please let me know if you'd like me to squash the commits.

Thanks! Fix looks ok, it hides worst bugs from that part of the code.
I'm pretty sure that actually timezone + seen parameters should be completely removed and replaced with ctx object and then some more changed should be probably done to support seen and timezone parameters as they were intented...
I'm merging this one for now and add an issue about that function...

@elhigu is this toPostgres functionality documented anywhere?

@hulbert no and it should not be. It is not part of public API.


[5.4] Scout - get() not filtering columns

Laravel Version: 5.4.10
PHP Version: 7.1
Database Driver & Version: Mongodb (jenssegers/laravel-mongodb)

Description:
When I use the scout search function and perform a get() command, it is getting all the columns. Even if I add a array of columns I wish my collection to have and add to the get(array) it doesn't filter the columns.
Steps To Reproduce:
My model is called Item, and has the following attributes:
id
name
price
img_url
date_added
$collection = App\Item::search('pencil')->get(['id', 'name']);
I want a collection of items with only id and name attributes.

As I checked the laravel\scout\builder class, the get() method doesn't has the column filtering implemented...

@c0h1b4 , u can use pluck method after fetching results from Scout. $collection = App\Item::search('pencil')->get()->pluck('id', 'name');

@c0h1b4 , use only() method, to get what you want.
$collection = App\Item::search('pencil')->get()->only(['id', 'name']);

@bozhkos, I tried but I'm getting an empty collection.

Bug is with that mongo package.


Autodiff

Overall looks good. Anything in particular I should focus on?
Is there a reason you aren't using ChainRulesTestUtils for the tests?


Update dependencies

The ci seams to fail for no reason. Would you mind restarting the job?

Thank you for letting me know @Hanzei!
After restarting job, it becomes success.


Link IFC results in null in revit 2023

Dynamo Revit <IP_ADDRESS>91
Genius Loci 2023.7.13
DynamoIronPython2.7 2.1.0
In revit 2022 works perfect only in revit 2023 not. What's wrong?

Hi,
It is working for me with Revit 2023. Not sure where your issue is.

HI all,
I've got the same issue too :

Not sure what is happening
My packages seem ok too :

Link to discussion added: https://forum.dynamobim.com/t/importing-ifc-files-in-bulk-with-dynamo-into-revit/102060/4


Does Ollama have methods similar to Docker's load and save?

What is the issue?
I want to deploy Ollama offline recently, and plan to save the model to a local tar file through save, and then load it into Ollama.
What did you expect to see?
No response
Steps to reproduce
No response
Are there any recent changes that introduced the issue?
No response
OS
No response
Architecture
No response
Platform
No response
Ollama version
No response
GPU
No response
GPU info
No response
CPU
No response
Other software
No response

Dup of #335


Enable use of session based threadpool.

Enlightening the design of ORT to control parallelism and concurrency.  This first set of changes enables a common threadpool to be configured and used per session.  This threadpool is used by operators for scheduling parallel units of work.  The parallel executor currently shares the same group of threads, but this will be changed in the next set of changes.

Thanks for your contribution. In general, it's excellent. Just please fix the build errors.
Moreover, I have two questions:

Do think would it better to change std::mutex to OrtMutex ? So that we can leverage nsync on Linux
Do you have any perf number about this change? would it change the perf?

Thanks

I did not change the std::mutex in the threadpool as that code comes from Eigen and will be replaced in my next PR to update the version of Eigen used by ORT.  I agree that in general OrtMutex is preferable.  I will fix the mutex in the performance runner code.

std::thread::hardware_concurrency() returns an unsigned int ,you need to cast it to int before passing it into onnxruntime::concurrency::ThreadPool

The deadlock issue is an artifact of how the parallel executor interacts with other parallel tasks causing too many threads in wait states.  This is a non-issue in the final implementation.  This code fix updates the parallel executor to implement a separate thread pool, but it comes at a cost.

Did not intend to close the PR.

Hi @shschaefer , any updates?

/AzurePipelines run

I fixed the bad merge of Tracy's MLAS changes and added the temporary inter-op threadpool for the parallel executor as Yufeng asked.

/AzurePipelines run

/AzurePipelines run

Hi, when I ran 'git-clang-format'
This is what it changed:
https://github.com/Microsoft/onnxruntime/commit/a70cb094beb85bd08e0d927d45e4c80c4a9fe689

I'm doing a perf measurement. I'll post some numbers out. The result doesn't seem good.
So, please hold on a moment.

The new OnnxRuntimeTestSession class that you added @snnn is not reentrant.  Its Run() method prevents parallel invocation.  Should I assume that I need to fix this?

Hi @shschaefer
Why the function need be thread-safe?
The program is for testing latency, and it assumes requests are arriving one-by-one. It won't start to process the next unless the current one is finished.
If we run multiple InferenceSession::Run() in parallel, there is no meaning to measure the wall time cost of each  InferenceSession::Run function. Because they interfere each other.

Hi @shschaefer
I observed a big performance drop on inception_v4 model.
Before the your change:
$ ./onnxruntime_perf_test.orig -x 2 -t 30 -b ort /home/chasun/src/py1/opset_7/inception_v4/model.onnx 1.json
Setting thread pool size to 2
Total time cost:30.1579
Total iterations:201
Average time cost:150.039 ms
After your change:
$ ./onnxruntime_perf_test -x 2 -t 30 -b ort /home/chasun/src/py1/opset_7/inception_v4/model.onnx 1.json
Setting thread pool size to 2
Total time cost:30.1142
Total iterations:120
Average time cost:250.952 ms

I got it. You are adding a new option: 'concurrent_session_runs'  into the perf test runner, that's why you expect the Session::Run() function be thread-safe.

@snnn That is correct.  The test program measures average latency, but had a simple measure of throughput under the assumptions you listed before.  To maximize throughput, parallel or pipelined invocations can make better use of the available resources.  This is a very common scenario that we have seen.  This test harness and the runtime need provide the options for developers to control the tradeoffs between latency and throughput.

Absolutely we care about throughput, but we should measure it more seriously. I suggest we temporarily disable this feature first, until we get a better implementation, measure it in a good way.

Hi, would you close this PR and recreate it in a different way? That will make the CI build triggers happier.

This PR is being redone as a direct checkin against the repository.


sorbet - attempts to open a file named -

Problem
I'm trying to have sorbet read from stdin.
Proposed solution
It's a common idiom in command-line programs to treat the filename - as meaning "read from standard input". Sorbet does not special-case the filename - and treats it like a regular filename.
This feature is very niche and we already have -e to pass in a string to check at the command line. But, I thought I'd make this issue anyway just so that if we decide this is something we don't want to support, we can point other people at this issue to show that it's been considered before. Of course, with some amount of effort we could support this.

I'd like to not have "one more way to do thing".
Do you feel strongly about this? Would you prefer to deprecate -e in favor of this?

If I had to choose between either - and -e I would choose -e.

Ruby does both and I think we should mirror it:
$ echo 'puts "hi"' | ruby -
hi

$ ruby -e 'puts "hi"'
hi

The thing I care about more is making Sorbet not crash on things like this:
cat foo.rb | sorbet /dev/stdin
cat foo.rb | sorbet /dev/fd/0
sorbet <(cat foo.rb)

right now the only workaround is
sorbet -e "$(cat sorbet.rb)"


Updated github.com/kr/pty for OpenBSD support.

This corrects "unsupported" on OpenBSD.  Hasn't been tested for regression, although the diff between vendor hashes includes only a fix for leaky file descriptors, and OpenBSD support:
https://github.com/kr/pty/pull/57
https://github.com/kr/pty/pull/58

Thanks @derekmarcotte!

We will get this in a release asap.

Hmmm, seeing test failures for openbsd/386 with this:
# github.com/buildkite/agent/vendor/github.com/kr/pty
--
  | vendor/github.com/kr/pty/pty_openbsd.go:24:10: undefined: ptmget
  | vendor/github.com/kr/pty/pty_openbsd.go:25:34: undefined: ioctl_PTMGET

Any ideas @derekmarcotte?

Interesting.  I'll get a 386 builder going next. ;)

It's a cross-build, will that no longer be possible?

Not without changes to the upstream vendor, is my gut on that.  You'll need the OpenBSD header files that define these things at least.  Perhaps there's a "nice" way to avoid CGo upstream.

It's weird that the amd64 variant builds fine.

Thoughts on how to proceed @derekmarcotte? My BSD experience is minimal, we'd prefer at this stage not to have to have native BSD builders for the variants.

Offline till 4 Est.  Ill see about removing cgo upstream.

No stress, consider any Github things from us asynchronous 🙇🏼♂

Okay - good news: the explanation is that upstream has this file:
https://github.com/kr/pty/blob/master/ztypes_openbsd_amd64.go
but not the equivalent 386 file.  I created a symlink, and it built fine.  I'll spin up a openbsd 386 builder and run the go tool cgo -godefs types_openbsd.go, and that should create the equivalent file.  I'll put a pull request upstream, but don't know how you want to manage it until it gets merged?

Fix submitted upstream.  I'll keep my repo around too, if that's helpful?

Thanks very much for adding OpenBSD support to the agent @derekmarcotte -- we're using this to support OCaml package builds on OpenBSD now via Buildkite!

@avsm no problemo.  Very pleased it will help the project more broadly.
You might be interested in this shameless self-promotion: https://github.com/derekmarcotte/kitefactory/tree/dm-openbsd <- soon to be integrated, if you plan to scale horizontally.  Perhaps there's common ground?


Events to EventHub

This image is slightly ambiguous

Event Producers are sending in HTTP(S). Events to event hub are posted in HTTPS so slight conflict?
eg : POST https://your-namespace.servicebus.windows.net/your-event-hub/messages?timeout=60&api-version=2014-01 HTTP/1.1

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

ID: 71adf8e4-a51b-d20d-39e1-fd7551bc94b9
Version Independent ID: 3d26fbc0-b6ae-e0ca-089f-e4966c19ef22
Content: Overview of features - Azure Event Hubs
Content Source: articles/event-hubs/event-hubs-features.md
Service: event-hubs
GitHub Login: @ShubhaVijayasarathy
Microsoft Alias: shvija

@tedmanlee Thanks for the feedback! We are currently investigating and will update you shortly.

@tedmanlee Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.
@ShubhaVijayasarathy There is slight conflict in the document regarding "HTTP" & "HTTPS", including the image pointed by the customer. Could you please take a look at this issue.

#in-progress

#in-progress

Fixed the image now. Apologize for the delay. I have been going through the old issues and resolving them. Thanks.
#please-close

Fixed the image now. Apologize for the delay. I have been going through the old issues and resolving them. Thanks.
#please-close


Enable TLS for UNIX Sockets via Env Var

This patch provides a way to reenable TLS for UNIX sockets by setting the environment variable LIBSTORAGE_TLS_SOCKITTOME to a truthy value.

Codecov Report

Merging #546 into master will decrease coverage by 0.08%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #546      +/-   ##
==========================================
- Coverage    28.4%   28.32%   -0.09%
==========================================
  Files          34       34
  Lines        2003     2009       +6
==========================================
  Hits          569      569
- Misses       1373     1379       +6
  Partials       61       61

Impacted Files
Coverage Δ

api/utils/utils_tls.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Link to code sample please.

Link to code sample

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

ID: c6606cce-8a4e-587b-ded2-bd775a4cd7c7
Version Independent ID: 6f637bca-8cc0-a526-4cbd-9c5f9f94ca2b
Content: Create advance conversation flow using branches and loops - Bot Service
Content Source: articles/v4sdk/bot-builder-dialog-manage-complex-conversation-flow.md
Service: bot-service
Product: bot-framework
GitHub Login: @v-ducvo
Microsoft Alias: v-ducvo

This one uses the Echo bot created in the quickstart. See "Create the dialogs for the hotel bot" section for links to the quickstart.

@kaiqb can you please add the link for   Create the dialogs for the hotel bot because i can't find it


feat: add Key Metrics table

Summary of changes

Build the basic layout for the whole site
Build the Key Metric Table component with no number data (have tested using test data)

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/stats-dapp
[x] apps/hubble-stats
[ ] apps/webbsite
[ ] apps/faucet
[ ] apps/tangle-website
[ ] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes #1302

Screen Recording
Default

After add test data:

Code Checklist
Please be sure to add .stories documentation if any additions are made to libs/webb-ui-components.

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Looks good.
Are we implementing the dark theme components in parallel or separately? If in parallel can we get a preview of the dark theme components as well?

Looks good.
Are we implementing the dark theme components in parallel or separately? If in parallel can we get a preview of the dark theme components as well?

The themes for these components are inherently built within the components themselves, thereby enabling the support for multiple themes per component.
I will shortly establish the preview link for this application. In the meantime, @vutuanlinh2k2, could you please provide an image of this component in dark mode?

Looks good.
Are we implementing the dark theme components in parallel or separately? If in parallel can we get a preview of the dark theme components as well?

The themes for these components are inherently built within the components themselves, thereby enabling the support for multiple themes per component.
I will shortly establish the preview link for this application. In the meantime, @vutuanlinh2k2, could you please provide an image of this component in dark mode?

@monaiuu here is the UI for dark theme at the moment


Clarify usage of Cross-References

Hi, thank you for creating this project!
I have a question regarding cross-references. When you generate the document into multiple pages, it no longer seems to be possible to refer to IDs in different files. Consider the following:
chapter_a.adoc
[#section-a]
== Section A
...

chapter_b.adoc
Please refer to section <<section-a>>.

When generating just one document ($ asciidoctor main.adoc), pressing on the hyperlink correctly directs you to the defined section. When you generate that document into multiple pages with your plugin, it no longer works. Ideally it should direct you to the other document.
Do note that I define the main.adoc like this:
include::chapter_a.doc[]
include::chapter_b.adoc[]

Is this feature supported? Thanks.

My mistake, <<section-a>> actually does work. pandoc auto-generated link::#section-a[] for me, kind of missed that.

Hi @lamafab, I'm glad you got it working, and thanks for the update.


Testing: empty filter field is not persisted correctly

VS Code Version: 1.68.0-insider (Universal) Commit: 17c75e1180
OS Version: macOS 12.4

Steps to Reproduce:

Type "abc" into the filter field of the test browser
Reload the window -> "abc" comes back, nice
Clear the filter field
Reload the window again -> "abc" comes back again (expected an empty filter field instead)

Cc @connor4312

Duplicates https://github.com/microsoft/vscode/issues/150120


 PS: If you get the DBX 286s you probably don't need the cloudlifter anymore, since you have multiple gain stages (pre-amp, output make up gain) that make up for low output of the SM7B.

I actually have a very nice pre-amp (SPL Channel One) and it still benefits very much from having the cloudlifter
https://www.youtube.com/watch?v=RYhxtLFL27Y for reference.
Thanks for the fixes!


Set up production frontend

Our current "production" steps that run services via docker involve running the frontend in dev mode. We could make the necessary changes to run the frontend for production (except deployment), such as:

Set up env files for production and update docs such as instructions
Add root level script to build & serve (preview) production frontend, e.g. pnpm run -F frontend build && pnpm run -F frontend serve

Preview seems to default to port 4173, and dev to 5173

Not in requirements


when i let users sign in with either email address or username, and they go to edit their email address, the form displays their username

I've followed the guidelines in the Wiki for allowing my users to log in with either username or email address. It works great, but I notice that for some users, when they go to my custom form for editing their email address (not devise/registrations/edit.html.erb, but another view I've created) their username is displayed in the email field for the form.
I'm using simple_form, so the form helper code looks like this:
<%= f.input :email, required: true, autofocus: true %>
Any ideas what might be causing this behavior or suggestions for how to fix it?
Thanks,
--Dean

Please use StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only.
Thank you!


Delay in loading custom application attribute in application edit view

Duplicate of https://github.com/wso2/product-apim/issues/6675 hence closing the issue


Can't start server with --log-format none

Starting with v0.5.20, it's not possible to disable console logging.
$ ws --log-format none

/Users/adam/.nvm/versions/node/v0.12.4/lib/node_modules/local-web-server/node_modules/connect/index.js:87
  if (typeof handle.handle === 'function') {
                   ^
TypeError: Cannot read property 'handle' of undefined
    at Function.use (/Users/adam/.nvm/versions/node/v0.12.4/lib/node_modules/local-web-server/node_modules/connect/index.js:87:20)
    at launchServer (/Users/adam/.nvm/versions/node/v0.12.4/lib/node_modules/local-web-server/bin/ws.js:113:9)
    at Object.<anonymous> (/Users/adam/.nvm/versions/node/v0.12.4/lib/node_modules/local-web-server/bin/ws.js:88:5)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3

All other logging modes seem to work fine.

Fixed and released, thanks for the report

That was fast! :beers:

it would have been done quicker if i had a reliable wifi connection atm! tbh it was a simple fix.. i have a couple of new features to add when i get time - proxying and user-defined routes..


Check in Manifest for experiments, and avoid using LOAD_PATH in experiments

Describe the bug
To ensure that the experiments can be reproduced if packages are updated, it is helpful to check in experiments/Manifest.toml. That way there is a known working set of packages that can be used to run the experiments.
If you do this, then you should remove the LOAD_PATH changes in the experiments files (e.g. https://github.com/CliMA/ClimaLSM.jl/blob/51d561608c85edab1430047b218ca5185d33c8b1/experiments/LSM/ozark_test.jl#L14-L16), and instead ]dev .. to make sure that the Manifest is using the correct vertsion of ClimaLSM.jl

This also appeared to be the cause of the issues @gagnelandmanna was having.

@simonbyrne  Is using ]dev ..  and making the ClimaLSM package part of the experiments/test/docs project environments the recommended approach? vs the LOAD_PATH one, where it is not part of the experiments/test/docs environments?


Facilitate replacing AuthToken model

See this for context: https://github.com/James1345/django-rest-knox/issues/251

For completeness: the CI failure is not related to the changes introduced in this MR.
I looked into it, and it's due to a version of cryptography package requiring a rust compiler, which is missing on CI.
See these for context:

https://github.com/James1345/django-rest-knox/issues/244
https://github.com/pyca/cryptography/issues/5771

This has been implemented in #275


FANBOX failure with 'Unsupported post type'

Prerequisites

[O] Did you read FAQ section?
[O] Did you test with the latest releases or commit ?

Description
[Description of the bug or feature]
Steps to Reproduce

[First Step]
Run PixivUtil2.exe and input 'f1' ( Download from supported artists (FANBOX))

Expected behavior: [What you expected to happen]
Download all the fanbox posts from artist who I support.
Actual behavior: [What actually happened]
Stuck at certain post with error 'Unsupported post type'
Versions
PixivDownloader2 version 20190919 (latest at 2019/10/23)

can you upload the log file with the error message? (not the screen shot).
I will recommend to delete the old log and then rerun with the problem post/id.

pixivutil.log
I backed up old log and reproduced error with fresh log file.
I think line 740~ have information about the error.

2019-10-23 17:57:27,572 - PixivUtil20190919 - DEBUG - Sanitized Filename: Dump for Unsupported post type = entry for post = 540.html

Can you upload the 540.html file? it should be generated from https://fanbox.pixiv.net/api/post.listCreator?userId=867590&maxPublishedDatetime=2018-01-23 17%3A00%3A00&maxId=540&limit=10

Dump for Unsupported post type = entry for post = 540.zip
I found 'Dump for Unsupported post type = entry for post = 540.html' file in folder.
Is it the file you asked for?
I zipped the file because html files are not supported.


Is there a specific Julia version we should use?

Let's stick to Julia v1.2 if I don't hear objections.


Unable to use FileStreamResult class in blazor project targetted to .netstandard 2.0

Bug Description:
I am working on blazor project, to create an application which is majorly focused on file operations. To write the file response, i use fileStreamResult class in my project. To use this class, i have tried to install the NuGet package Microsoft.AspNetCore.Mvc.NewtonsoftJson version=3.0.0-preview6.19307.2 in my project. On searching in web, i found this class available within this package, only.
But, this package is not getting installed. Anyhow, this package is installing successfully with blazor project targetted .netstandard 3.0.
Could anyone, help me to find the solution to use the fileStreamResult class within project targetted to .netstandard 2.0 ? or let me know any other possible solution available ( any other way to use the fileStreamResult class through other NuGet package in blazor project).
Hope, i described my issue, precisely.!
Waiting, for the solution from experts ..!
Thanks in Advance.

Why are you using FileStreamRest in a Blazor application? Can you post the code?

Why are you using FileStreamRest in a Blazor application? Can you post the code?

I am building a application which simply do file operations like read, create, delete, download, update. For this, i am using blazor's shared application which comprise of both client-side and server-side project. I build a client-side UI which access files and folders from server-side using ajax calls(i.e, i call server side method to perform file operation like downloading a specific folder). For achieving this case, i need to return a filestream from physical path to the client-side.
This is the reason why i use FileStreamResult class in my project.
public FileStreamResult GettingImageResult(string path, string id, bool allowCompress, ImageSize size, params FileManagerContent[] data)
{
try
{
AccessPermission PathPermission = GetFilePermission(path);
if (PathPermission != null && !PathPermission.Read)
return null;
String fullPath = (contentRootPath + path);
FileStream fileStreamInput = new FileStream(fullPath, FileMode.Open, FileAccess.Read);
FileStreamResult fileStreamResult = new FileStreamResult(fileStreamInput, "APPLICATION/octet-stream");
return fileStreamResult;
}
catch (Exception)
{
return null;
}
}

Hi @davidfowl ,
I am waiting for your update. mistakenly closed the thread last time.

Hi @Shameer-Ali. The FileStreamResult type is specifically for returning a file stream from an MVC action. It is not intended for client-side use in Blazor apps. Starting with .NET Core 3.0, you can only use FileStreamResult from an ASP.NET Core project targeting netcoreapp3.0. In your server project you can use FileStreamResult to return a file to the Blazor client app in response to an HTTP request, but you can't use FileStreamResult directly from the Blazor client app.


Hardness value of waystone editable in config

I wanted a way to make waystones unbreakable for my mod pack. This adds a new config option waystone_hardness_value. Setting this value high makes the waystone take longer to break and setting this value to -1 makes the waystones unbreakable.

There's a new version in the works, so I won't be pushing this to CurseForge until all of that is done.

There's a new version in the works, so I won't be pushing this to CurseForge until all of that is done.


feat(Pagination): add new component

This PR adds the fully featured pagination to SUIR using existing Menu component.

@levithomason I'll be glad to hear your opinion there.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2352   +/-   ##
=========================================
  Coverage          ?   99.73%
=========================================
  Files             ?      154
  Lines             ?     2680
  Branches          ?        0
=========================================
  Hits              ?     2673
  Misses            ?        7
  Partials          ?        0

Impacted Files
Coverage Δ

src/addons/Pagination/PaginationItem.js
100% <100%> (ø)

src/addons/Pagination/Pagination.js
100% <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 f9348a2...c91dedb. Read the comment docs.

This is so cool and much needed!  Here are my initial thoughts when trying the component.
showFirstAndLast
I would expect this prop to show the first and last page number in the list of pages.  However, that is already shown.  This prop seems to add first and last navigation buttons but those are duplicating the first and last page number buttons:

💡  Suggestion:
Perhaps split this into showFirstAndLastNav and showFirstAndLastPage?
showPreviousAndNext
Since the item prop is prevItem I think we should use showPrevAndNext.  Also, if we split the first/last nav vs page props above, then this would be showPrevAndNextNav for consistency.
Item shorthand
I can't seem to turn off the first/last page number.  Passing null/false still renders them.  I think we should allow turning these off.
<Pagination firstItem={null} lastItem={false} />

Oh, right.  You'd pass showPreviousAndNext={false} 😊.  Do we even need the show* props?  The *Item props could work as <Dropdown icon={null} /> and <Modal dimmer={false} /> which removes the shorthand items.  Thoughts?

@levithomason I've applied proposed changes and removed all boolean show* props 👍


Error: listen EACCES: permission denied /run/user

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:1.63.2
OS Version: Windows 11 and RHEL on Remote

I'm able to SSH into the remote host, but when I run code  I get Unable to connect to VS Code server. I looked at the log file and it looks like its trying to access /run/user/104025, which I have no idea who that folder is and why it would be trying to access it.

Closing issue. I realized I didn't have that folder /run/user/104025 and I had to manually create it using sudo. Once I created the folder and changed the ownership to my username, it worked.


Does not work with CPU: Grouped Convolution

Hey,  first of all, thanks for your work - pretty fast :)
I just wanted to test your repository and noticed that the code fails for inference on CPU due to the grouped convolution.
Code:
model = NFNet(num_classes=1000, variant=variant)
model.build((None, 320, 320, 3))
model.load_weights(f"{variant}_NFNet/{variant}_NFNet")

test_image = tf.zeros(
    shape=(1, 320, 320, 3), dtype=tf.float32
)
model(test_image)

Error:
UnimplementedError: The Conv2D op currently does not support grouped convolutions on the CPU. A grouped convolution was attempted to be run because the input depth of 256 does not match the filter input depth of 128 [Op:Conv2D] (https://github.com/tensorflow/tensorflow/blob/669993ebe8534eac877eec61225925cff737eac2/tensorflow/core/kernels/conv_ops.cc#L160)
I already started debugging and as far as I see, the error occurs in the Second Block in conv1 when calling WSConv2D(). Here, the inputs are of shape (1, 68, 120, 256), while the weights are (3, 3, 128, 256).
I am not that familiar with grouped convolutions and NFNets in general. So I thought, you maybe already know how to solve the issue, if possible?
Edit:
Is it possible that the filters are already divided by the number of groups (in this case 2) and the inputs are not? See here

hi there, can I just ask were you testing on your own data, or on ImageNet?

Hey were you able to find a workaround to run it on CPU?

Hey, sorry for late response @Sicily-F.
I run everything on custom data. Moreover, I could make it run on the CPU via @tf.function(experimental_compile=True) (Issue 29005), but it was not applicable wrt. perfomance/time.


Added Catch_Stars Game

PR Description 📜
"Catch Star" is a fun and engaging game where players must catch stars that appear randomly on the screen. The game tests your speed and reflexes as you try to catch as many stars as possible within a time limit. As you catch more stars, the game becomes more challenging with more stars appearing at a faster rate.
Fixes #3824

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @manishh12 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @manishh12 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @manishh12,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


Added other useful functions:

logEventWithNameAndData // logEventWithNameAndData('event_name', { msg: 'hi' });
updateUserWithAttributes     // updateUserWithAttributes({ fname: 'John', lname: 'Doe' });
setDeviceToken                     // setDeviceToken('token', Ti.App.Android.R.drawable.appicon);

Hey - Thanks for the PR.  Would you mind updating the Readme, bumping the version number and providing a new dist with these changes.

Hey - I think there's something wrong with my env - the module works well on android 5, but got this error on android 6:
lib/arm/libti.intercom.android.so: has text relocations
I also tried to create a module, then package and run with a sample project.. still got .so has text relocations
This is my env:

android-ndk-r9d
Android SDK 5.0.x (API 21), 6.0.x (API 23)
Android SDK Tools Rev 23
Android SDK Platform-tools/Build-tools Rev 21
Appcelerator Studio, build: <IP_ADDRESS>602170821
Titanium CLI: 5.0.6
Java Development Kit: 1.7.0_80
Mac OS X 10.11.2

Your build 1.0.1 works well on android 6. Could you please share your env settings?

I won't know my exact environment until I get back to that machine later this week.  I'll let you know then - Jerod

ndk
android-ndk-r8d
build.proprties
titanium.platform=/Users/jerodfritz/Library/Application Support/Titanium/mobilesdk/osx/5.1.1.GA/android
android.platform=/Applications/Android/platforms/android-23
google.apis=/Applications/Android/add-ons/addon-google_apis-google-23
ti
Appcelerator Studio, build: <IP_ADDRESS>602170831
Titanium Command-Line Interface, CLI version 5.0.6, Titanium SDK version 5.2.2.GA
os
OSX El Capitan 10.11.4

Thank you for your information. I tried to install android-ndk-r8d, also setup the same build.proprties, but still got .so: has text relocations error. I'll report this issue to APPC soon. In the meantime, it'd be great if you could help to make a build on your env.

Finally it works after I downgraded the build tools from 21.1.0 to 21.0.0 :)

Excellent.  Will you replace the ti.intercom.android-android-1.0.1.zip with your ti.intercom.android-android-1.0.2.zip and update the Readme to include the new functions

done!

Looks good.  I'll merge.  Thank you.
There has got to be a way to retrieve the appicon from within the module rather than having to pass it along as a parameter with setDeviceToken

Yes that's possible to retrieve the appicon from within the module. But I think user might want to make this configurable.


Teaser component - 404 documentation link

Please see this page, https://github.com/Adobe-Marketing-Cloud/aem-core-wcm-components/tree/master/content/src/content/jcr_root/apps/core/wcm/components/teaser/v1/teaser.
The Documentation link at the bottom of the page is an invalid/404 GO url, https://www.adobe.com/go/aem_cmp_teaser_v1

@sschnoor Thanks for reporting this issue. The GO url should redirect back to the teaser.


PEM loss=NaN

Thanks a lot for this code!
I simply run the script to train BMN model. However I got PEM loss=NaN , and I find that label confidence is all 0. What 's wrong there?

Thanks a lot for this code!
I simply run the script to train BMN model. However I got PEM loss=NaN , and I find that label confidence is all 0. What 's wrong there?

I met the same problem when I debuged the model, but I have sloved this problem before I release the code. I forgot what's the problem and how to slove it. I will check the code again.

Thanks a lot.
I think I have found the problem..
In dataset.py line114 and 115,when using python 2, because of division difference, all results are 0.
So I fixed it by change line 114&115 like below:
tstart = s / float(self.temporal_scale) + self.temporal_gap / 2
tend = e / float(self.temporal_scale) + self.temporal_gap / 2

After test, it worked.

that's the problem, I met it when I change the code from the BSN in python2 to my code in python3.
the hyperparameters may also not the optimal choices, I dont save these parameters and the model, you may need to do more expriments to get a good result. In my memery, the best result of my expriments is near to the BSN but could't the get results as the paper.


Update "cross-env" version

I'm not sure that cross-env should be a dependency of Mix in the first place, so my hope is to remove it with the next major release.


Unable to remove (missing) repositories

I am currently in the process of automating the creation of Git repositories as part of a larger project which creates a project folder, assigns NTFS permissions, creates an email distrubution for said project and finally, a Git repository. During my testing I have created a number of bare repositories via Git and then scanned them in from Bonobo so they appear in the portal and I can assign permissions. I have since removed the repositories but they are still displaying within Bonobo as (missing). The delete button is no longer present and selecting the hyperlink produces an error 'An error occurred when processing your request. Please try again or contact the system administrator.'
I am currently using the AD integration feature and I have attempted to remove the respective .json files under the \App_Data\ADBackend\Repos directory and they are still appearing. There are also not present within the Bonobo.Git.Server.db SQLite DB, although I believe this is because I am using AD auth as no reps or AD users exist in the table. Can someone confirm if where else these repositories would be referenced within Bonobo?
Thanks in advance for yor help,
Adam

They're nowhere else, but they're only read from the json files when the app starts, which might be why you're not seeing them missing.  Do they go away if you delete the json files AND do an iisreset?

I had removed the json files at the time of writing but they were still appearing. Having just done an IIS restart has now removed the deleted repositories which were listed as (missing). If I wanted to generate my own repositories using Git and not have to do a rescan of new repositories from within Bonobo, could I generate my own json file without it having any adverse affects? How are the ID names of json files which relate to the repositories generated by Bonobo?

Had same issue, just removed corresponding records in Bonobo.Git.Server.db database.

Serf


Deprecate passing query, mutation and subscription types to Schema constructor directly

After this change following code will trigger deprecation notice:
$schema = new GraphQL\Schema($objectType, $mutationType);

It should be rewritten with
$schema = new GraphQL\Schema([
    'query' => $objectType,
    'mutation' => $mutationType,
    // Other possible options:
    // 'directives' => $directives
    // 'subscription' => $subscription,
    // 'types' => $types
])

Why not make use of this change to be a little more OOP:
$schema = new GraphQL\Schema();
$schema->setQueryType( $objectType)
    ->setMutationType($mutationType)
/* Other possible setters:
   ->setDirectives($directives)
   ->setTypes($types)
*/

Doing so it will be easier for php developers to get the possible options...

Mostly because there is additional validation after schema construction which requires all options to be set before validation is performed. In your example this validation would have to be called explicitly at some point of time.
Another reason is that am personally not a fan of partially initialized objects and try to avoid them when possible %)
But I do agree that it might be convenient, so maybe it make sense to add Schema builder class which will have all options expressed as chainable methods.

Or use named constructors:
public static function byTypeMutationType($queryType, $mutationType) {
  return new static(['query' => $queryType], 'mutation' => $mutationType]);
}
public static function byTypeMutationTypeDirectives($type, $mutationType, $directives) {
  return new static(['query' => $queryType], 'mutation' => $mutationType, 'directives' => $directives]);
}

I think that schema construct does too much validation... It is not possible to instantiate schema before instancing all other types. The array options force to add custom validators. Using setters add TypeHint and help to produce more comprehensive code.

@mcg-web On schema validation - that's pretty much how it works in reference implementation. But I've been thinking about changing this. Because in PHP world we don't have a luxury of having one schema instance for all requests, instead every request has it's own instance and it doesn't make sense to validate schema every time.
In our case schema validation should be a part of development workflow and probably build / deployment step. But it shouldn't be executed on every request on production. Still it should be enabled by default for backwards compatibility (but with ability to disable it for production).
Also based on your feedback I will definitely add some chainable syntax for schema creation - either via separate Builder class or to schema class directly.
@jayS-de It kinda works, but I think fluid interface (chainable methods) is more expressive and convinient way to do this.

One thing that is important (in my opinion) is that the schema must be serializable (and so cacheable). If resolve* properties are no longer closure but something that is understandable by a service (e.g. an expression language string, a serializable callback, etc) then the schema may be easily cached to avoid build or validation costs.

@ooflorent I've been thinking about this.
But I suspect that deserialization might be even slower than creating new schema object (with disabled schema validation).
It's a long time when I wanted to build some profiling tests for parsing, validation, execution and schema building steps in order to make rational decisions on various performance improvements. Never found time though %)
If deserialization proves to be faster - we can move in this direction. Another thing to consider is that application may define own callbacks for type definitions (e.g. similar to how validation rule for query complexity expects some callbacks in type config).
So deserialization must be extendable in app-space.

Another alternative to schema serialization is constants expressions (new feature of PHP5.6+). Nice blog post on the subject - http://blog.blackfire.io/speeding-up-autoloading-on-php-5-6-7-0-for-everyone.html
So the idea is that schema could be dumped into pure constant array shared by all requests. Then you can avoid deserialization step completely and reduce memory consumption by re-using this immutable schema instance in all requests.
The downside is that it requires major code rewrite and 5.6 as minimal PHP version.

Ah, the dreaded core problem of PHP's shared nothing architecture (and why such things as constant expressions are created). This is why I am working with appserver.io and putting my graphql service in a servlet.:smile:
I'd say any major change would need some sort of POC/ bench marking. And still, I'd bet everyone in this discussion a beer that the constant expressions would win over any caching solution.
I personally don't see a PHP5.6 minimum as an issue either. :smile:
Scott

@mcg-web Just wanted to note that I remember about discussion about chainable syntax, but decided to release what was ready at the moment. Will get back to this in future versions. In the meantime it is quite possible to implement schema/types builder as a separate project with rather simple set of classes.

Closing, as this is already done, but created separate issue #58 to track schema builder question


Improve the async handling of the select

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


Does not work well with the templater plugin

I had some trouble using this plugin in combination with Templater. I tried using it with "trigger templater on new file creation". Templater wants to only work on empty files, and this seems to interfere with "rollover daily todos".
As a workaround, I recreated some of the functionality of "rollover daily todos" by copying a bit of code and making it available as a templater "user script", pasting the gist to make it possible for others to find a workaround in case they stumble upon this :-)
https://gist.github.com/lenalebt/8e58aa21392c388d8c7d65bad3206e35

Thank you @lenalebt! I had trouble getting this plugin to work, and noticed I also had Templater enabled. Tried your workaround and it worked perfectly.

Oh, that's interesting - I don't have that problem. I noticed that your tasks end up in a code box (three back ticks). I think that's where the non-clickable boxes come from.
If you have further questions, feel free to directly comment on the gist - then we can leave this ticket related to things about the plugin :slightly_smiling_face: . It might help if you post your templater source to the gist, then I'll help you work this out over there!

Thank you for your input and contribution @lenalebt.
If more people are affected by this, please feel free to submit a PR to the README file in the Bugs/Issues section, where we can redirect people to your solution.

I've been having the same issue. It seems to be an incompatibility with templates's "Trigger Templater on new file creation" setting.
If I turn off that setting, everything seems to work fine. It seems both plugins are simultaneously trying to write to the file and Templater ends up erasing the Todos

I too found this issue after having trouble with this: I have templater create my daily notes (in a couple of stages, i.e., templates inside templates) and it overwrites the rolled over todos.
I know next to nothing about how these plugins work, but would it be possible/easy to build in a very slight delay (0.5 seconds) before the rollover happens?

I am throwing my experience/2 cents in. I've been using the Periodic Notes community plugin before adding Rollover Todos. I enabled "Trigger Templater on new file creation" and set the template up in the Periodic notes plugin; removing it from the Templater "Folder Templates" section. This way, the periodic notes plugin creates the file with the Templater template in place, Templater is then working with an existing file with its tags already inline and the Rollover Todos plugin adds its lines to the heading it needs to. I've only been using this setup for a few days, so I don't have enough experience to say this will work every time, but so-far-so-good. I am adding screenshots of Templater and Periodic Notes settings as an example.

I too found this issue after having trouble with this: I have templater create my daily notes (in a couple of stages, i.e., templates inside templates) and it overwrites the rolled over todos.

I am having the same issue.

Hi there,
I've just pushed a new release that introduces the option to disable the automatic rollover when creating the daily file.
For those utilizing the Templater plugin, ensure you turn off the automatic rollover feature within the Daily Rollover Settings tab. You can then manually roll over the to-dos via the provided command.
Currently, this approach ensures optimal compatibility and avoids potential conflicts with the Templater plugin.

For reference, I was able to fix this using templater by adding this at the end of the daily note template:
<%* app.commands.executeCommandById('obsidian-rollover-daily-todos:obsidian-rollover-daily-todos-rollover') %>


fix: broken link in docs

CI/CD is failing because one of your links is broken in the documentation

Codecov Report

Merging #264 (483d60c) into main (01738a2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #264   +/-   ##
=======================================
  Coverage   64.21%   64.21%
=======================================
  Files          17       17
  Lines        2540     2540
  Branches      402      447   +45
=======================================
  Hits         1631     1631
+ Misses        767      765    -2
- Partials      142      144    +2

see 2 files with indirect coverage changes

Hi @RobPasMue , thanks.
I had this change on a Pull request, but it is taking a bit of time to get merged

No problem - let's try to solve it asap and you can update your PR afterwards! You shouldn't see any conflicts on your side if the change is the same :)

I'm not sure we need this PR.  I removed that text/reference from the RST file for now.   The concern was with how the file got there in the first place and if the file should move to the API wheel?   I'm not sure about the latter as currently we do not use it for the API, just docs, but in the future I could see it being part of the consumed API.    Given the changes pending in that interface (e.g. server-side-event support), that might be a good time to make the move.

Okay got it. I can close the PR then. Thanks @randallfrank!


Image are always flagged

![image info](./pictures/image.png) is flagged with
Link definition for 'image info' not found.
If this is not a link reference, consider bracket escaping '\\[' or using code block."

Sorry for quite a delay. I can not reproduce this problem. If it is still important for you, feel free to provide more information and reopen the issue.

I'm not using the extension anymore as the features I was using have been integrated into vscode.
Thanks anyway as it was a really useful extension.


How to get trial index in task?

When set num_trials>1, how to get trial index inside a task?

Solved by threading.get_ident()


[UI] Navigation Menu Icon Not Scaling

Description
The lefthand navigation menu icons do not scale on browser zoom.
Expected Behavior
The icons should enlarge/shrink as the user zooms their view.
Screenshots

Related issue - https://github.com/layer5io/meshery/issues/1146

Assign this to me.

Thanks for picking this up, @codetheorem. There are a few other icons that don't scale here that are not encircled in the screenshot.

I can work on this issue

Thanks for picking this up, @arpitkarnatak

Hey @arpitkarnatak how is this coming up?

@leecalcote I would like to work on this.

Hi @Ayush-Khandelwal-007, that would be great. 👍


Ignore mypy error when overwriting label_dict with new type

See code

@rsmith49 I think this is a better solution to deal with the mypy error on overwriting the type of invariant variable


Usage with binary data ?

This is useful for me. In the roadmap, have you considered  binary data ( images etc ) and tensors ?
I do a fair bit of ETL and fbp style work in golang, and so might be able to contribute

Hello, right now I'm working on storing arbitrary types inside the columns (A column can only have elements of the same type). This would allow to store lists or other data structures as cells inside the DataFrame matrix as long as they comply with the necessary interfaces.
How exactly do you want to use images or binary data with gota?
If it is for matrix computations I recommend you to take a look to https://github.com/gonum, for it is was conceived with that in mind.
Right now I'm focusing on data wrangling with this package, including things like subsetting, sorting, merging or function application but I don't discard adding more complex operations in the future.

The reason I asked is because I need to stream binary data through
different computation nodes.
Use cases are real time video transformations, real time convolution &
analysis.
Gonum and its packages I presume can do the computation, but dataframes and
other buffered data structures are still needed to move the data between
the compute nodes.
Curious what you think.
On Sat, 13 Feb 2016, 14:47 Alex Sánchez<EMAIL_ADDRESS>wrote:

Hello, right now I'm working on storing arbitrary types inside the columns
(A column can only have elements of the same type). This would allow to
store lists or other data structures as cells inside the DataFrame matrix
as long as they comply with the necessary interfaces.
How exactly do you want to use images or binary data with gota?
If it is for matrix computations I recommend you to take a look to
https://github.com/gonum, for it is was conceived with that in mind.
Right now I'm focusing on data wrangling with this package, including
things like subsetting, sorting, merging or function application but I
don't discard adding more complex operations in the future.
—
Reply to this email directly or view it on GitHub
https://github.com/kniren/gota/issues/1#issuecomment-183667765.

I suppose that you could implement your own type to store in the DataFrame cells, but keeping in mind your use cases I would say it makes more sense to use a less convoluted data structure: slice arrays, multidimensional arrays or hash maps should be able to do the job as long as you implement the real time algorithms correctly.

Your right.. Will try out different way
On Sun, 14 Feb 2016, 01:39 Alex Sánchez<EMAIL_ADDRESS>wrote:

I suppose that you could implement your own type to store in the DataFrame
cells, but keeping in mind your use cases I would say it makes more sense
to use a less convoluted data structure: slice arrays, multidimensional
arrays or hash maps should be able to do the job as long as you implement
the real time algorithms correctly.
—
Reply to this email directly or view it on GitHub
https://github.com/kniren/gota/issues/1#issuecomment-183780875.


add bufBinWriter to emit functions in order to catch errors

Problem
Multiple errchecks #297
...
Solution
this patchset
...
closes #297 and #457

Open for suggestions. I'm not quite sure about having
// WriteBytes writes a variable byte into the underlying io.Writer without prefix.
func (w *BinWriter) WriteBytesNoPrefix(b []byte) {
	w.WriteLE(b)
}

Open for suggestions.

I'm thinking about making s/WriteBytes/WriteVarBytes/ and then adding WriteBytes that would do WriteLE inside. It will make us more consistent with C# implementation (where everybody knows what VarBytes means both for read and write). But it would be nice then to also fix some WriteLE users (basically, everywhere []byte is being passed into WriteLE) to use WriteBytes, so it's quite time-consuming.

Open for suggestions.

I'm thinking about making s/WriteBytes/WriteVarBytes/ and then adding WriteBytes that would do WriteLE inside. It will make us more consistent with C# implementation (where everybody knows what VarBytes means both for read and write). But it would be nice then to also fix some WriteLE users (basically, everywhere []byte is being passed into WriteLE) to use WriteBytes, so it's quite time-consuming.

no problem! Will do

Open for suggestions.

I'm thinking about making s/WriteBytes/WriteVarBytes/ and then adding WriteBytes that would do WriteLE inside. It will make us more consistent with C# implementation (where everybody knows what VarBytes means both for read and write). But it would be nice then to also fix some WriteLE users (basically, everywhere []byte is being passed into WriteLE) to use WriteBytes, so it's quite time-consuming.

@roman-khimov take a look at
41156eda6c20bf3915ee0e56c044770e34a5687e . Hope that is what you mean

	w.WriteVarBytes(b) fixed


version 3.1.1

On https://pub.dartlang.org/packages/flutter_secure_storage the latest version is 3.1.1, however on github the latest version is 3.1.0. Could you push your latest branch please?

Hi Jeroen,
Done!


Database - further constraints on foreign keys

Is it possible with this:
array(
     'key' => 'signout_instructor_id',
     'reference' => array(
          'table' => 'peoples',
          'column' => 'id',
     ),
),

to actually check if the people/id is also flagged as an instructor?  Unlikely, as this inferred by them being in the instructor table, so likely going to require the controller business logic to be the goalkeeper here.

Possible solution is a trigger in the database - "BEFORE INSERT/UPDATE"???


Export single modules of sdk

Currently, we are exporting a whole sdk as one single export and then use it globally in the web app. This leads to a need of passing around the sdk context between different parts which haven't got the access to it. A perfect example of this is the store. This is a bit tedious and it would be great if we could import the required modules directly when needed.

Another advantage we could get with this would be tree shaking. It would probably require a different approach to storing some of the values in helpers. Maybe in a window object? Thoughts @kulmann @fschade ?


Header doesn't slide over other elements anymore

Fixes

Fixes #207 by @FZehra1512
Description

This PR makes the Header not slide over other elements, just like the one in creativecommons.org website.
Technical details

Added CSS in style.css to change the position of #global-header from absolute to relative. Also, made changes in to ensure smooth transition. Changed body padding to remove extra padding introduced at the top after change in header position.
Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Hi! thanks for your contribution, but this PR includes changes to Gemlock.file, and this issue was resolved in #209. Closing.


[FEA] Persist workspace data on PVC for premerge

Signed-off-by: Peixin Li<EMAIL_ADDRESS>Job on blossom is running in pod, by default the workspace data will be disappear when the container is terminated.
This PR is to persist the workspace data on PVC storage for per-merge pipeline. Because the PVC provided by blossom is backed by NFS, we run the job as root(uid:0) to save us a headache (https://github.com/kubernetes/examples/issues/260)
Every build will have it own workspace folder on PVC, and I have setup a hourly-cleaner to clean out-dated workspaces

build


add style guide

Looks good @realdezzy! I made a minor change. Let me know if you agree and then we can merge this.

Looks good @realdezzy! I made a minor change. Let me know if you agree and then we can merge this.

Approved 🚀


fix: Update Design Tokens

Before (the change)
Design tokens package was 1.1.1
After (the change)
Design tokens package should be 1.2.0
Make sure that you've checked the boxes below before you submit the PR

[x] I have read and followed the setup steps
[x] I have created necessary unit tests
[x] I have tested the functionality in both React and Angular.

Steps needed to test
Nothing needed to test

:tada: This PR is included in version 1.17.0-alpha.70 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0-alpha.4 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.22.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.21.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.17.0-alpha.32 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


LUA is too slow for Pi4

Hi there, @icculus
I was amazed to see this core today, so I went and built it for testing on the Pi4 on RetroArch.
Thing is, video decompression doesn't take much CPU (TOP reports ~30% among all CPU cores), but sections where LUA code kicks in take up as much as 80% of the CPU, causing massive sound drop-outs.
So, since this is now a very young project, would it be any possibility of using lua-jit or other technology that doesn't waste as much CPU as LUA normally does, so it's usable on mid-range aarch64 machines like the Pi4?
Thanks!

Hmm, this worked on a Pi4 last time I tried it, a month or so ago. I'll have to check again, in case I broke something.
In the meantime: are you sure it built with optimizations? It can make a huge difference on the Pi.
Also, I'm reworking the audio code right now, which will buffer a little better, which might solve the problem too. That work is almost done!

Just realized I tried the standalone SDL app on the Pi4 and not the libretro core.
I do think my audio changes will fix the problem, but I'll verify either way.

I will be testing the new audio code as soon as it's available, too!
But from what I see here, CPU usage spikes come from LUA code, not from audio/video. As long as I leave the attract mode running, there are no CPU usage spikes, it's only during gameplay due lo LUA code being run.
LUA is very slow on ARM in general: the LUA machine is veeeery slow.

I have not tried this on a Pi4 yet, but if you're bored and want to see if it has improved, go ahead and pull the latest.
Don't forget to replace the contents of the RetroArch/system/DirkSimple directory in your install with the latest in revision control, as the Lua code has changed and .wav files were added so Dragon's Lair can make beeps and buzzes.

I have been trying this on the Pi4 this morning.
I still get those audio drop-outs when LUA code kicks-in: during active gameplay, or when Dirk "rises from the death" when he gets killed, etc.
Same as with the previous code, I can see on TOP how CPU usage goes from 30% during attract sequence to 80% when LUA code/game logic runs.

Ok, I'll try it over here. Are you using stock RetroArch, or is this a RetroPie install?

I use latest RetroArch code, built against latest Wayland/wlroots using the labwc compositor (also Weston), against latest stable Mesa.
Also RetroArch on KMS/DRM.
Always the ALSA audio backend.
And everything on Aarch6.

(Sorry, this is a giant pile of text to read.)
Okay, just as a sanity check, I ran this on a Pi4 running RetroPie (because it's what I had available). I simply quit EmulationStation and ran the core from the command line:
/opt/retropie/emulators/retroarch/bin/retroarch -L ./dirksimple_libretro.so ./lair.ogv

...and it played at full speed without audio dropouts. So something else is going on, but I don't know what yet.
I think the assertion that it's Lua causing the problem is a red herring. The Pi4 should be powerful enough to handle Lua in general, and we don't run very much Lua code per-frame (maybe 50 lines of code?)...and notably, the attract sequence runs the exact same Lua code as the rest of the game (attract mode just looks like a level where the only correct input is the start button). It wouldn't surprise me if Lua is the majority of our CPU time per-frame, because we don't do a whole lot per-frame on the main thread.
There is, in the libretro core, other places that would suffer from high CPU usage, though.
Notably, it has to convert the YUV video data from the .ogv file into RGB format, which is very CPU intensive. The SDL app gets around this by using an OpenGL shader to do the work at render time behind the scenes, but right now the libretro core is just being fed a software-rendered buffer of pixels. A debug build might not be able to do it in real time. The conversion happens in a background thread, so it won't show up in a profile, but audio can't be sent to the core for processing until that thread gets through the video frames that are in the way of more audio frames being decoded...which might explain the audio issues.
It might be worth trying the SDL app on this hardware, and seeing if it works better than the libretro core here, just as a data point. Improving this in the libretro core, to optionally use OpenGL so I can move this to the GPU, is on my wishlist.
Also, I'm working from a 480p resolution Dragon's Lair video; there are HD (even 4K!) upscales people have produced on the internet (and I think Digital Leisure shipped a 1080p Blu-Ray at some point?)...if you happen to be using these, they take a lot more CPU to decode the video, and compounds the YUV/RGB conversion on the CPU that the libretro core is doing. If you want to make sure we're working from the same video file just to rule this out, drop me an email and I'll get you what you need to test.

Hi again, Ryan! Don't worry about the wall of text, that was a pleasure to read, thanks for taking the time to test this, really :)
Some "special" configurations I use in RetroArch that may explain the differences between what you see there and what I see here with regards to CPU usage and audio dropouts:
-I use 2 buffers in Settings->Video. RetroArch uses 3 buffers by default, which adds another frame of input lag. Tried 3 buffers but I don't see much difference in the dropouts.
-I use a basic shader with Dirksimple. Any simple shader (crt-pi, zfast...) will do, but I use Fakelottes. After all, that's one of the motivation for a libretro core, isn't it? :)
Tried with no shaders, but well, no such a difference either.
I also tried the SDL2 version, and there are no dropouts there, you are right.
Everything LUA I have tried in RetroArch has massive CPU usage spikes, audio dropouts, etc. But it's not only LUA's fault, but the proverbial Pi memory bandwidth limitations kicking in when LUA is combined with RetroArch. In SDL2, it seems that hardware video conversion does a good job and alleviates part of the problem, hence the dropouts do not appear.
I am using a 480p video source, so that can be ruled out; I am sure we are already using a similar source.

Next thing to try, just as a guess:
Let's buffer more audio per-frame and see if it helps:
diff --git a/dirksimple_libretro.c b/dirksimple_libretro.c
index d0e8547..8e62c2c 100644
--- a/dirksimple_libretro.c
+++ b/dirksimple_libretro.c
@@ -622,7 +622,7 @@ static void feed_audio(void)
     // mix 3K samples of audio and attempt to feed it to the system.
     // 3K == 1.5 stereo frames per frame, which at 30fps is slightly more
     // than you need to not starve at 44.1KHz.
-    static float fl32buf[1024 * 3];   // make this static in case the system has a pathologically small stack. This function isn't ever reentered afaik.
+    static float fl32buf[1024 * 6];   // make this static in case the system has a pathologically small stack. This function isn't ever reentered afaik.
     int space_left = sizeof (fl32buf);
     int space_used = 0;
     for (DirkSimpleDiscAudioQueue *i = disc_audio_queue_head.next; space_left && (i != NULL); i = i->next) {

After trying this, I'm going to dig up the NEON version of the YUV->RGB conversion code I wrote before, which will definitely speed up that bottleneck on a Pi.
If all else fails, I'll try to get an OpenGL renderer in there (probably not Vulkan, because I just need the thing to draw some basic things on the GPU and don't want to add a thousand lines of code to do that).
One of these things is bound to fix the problem!  :)

I built a version with that modification and doesn't seem to make any difference, sadly :(

I don't have progress to show yet, but the gameplan for dealing with this is twofold:
First, I'm writing some NEON code for the YUV->RGB converter, which will speed up the biggest bottleneck in the video decoding on a Pi. That's going on in #23.
Second, I'm going to have the libretro core use OpenGL if possible (and fall back to the existing software code if the system doesn't have a GPU). That's in #24.

Sounds great!
May I ask that you do OpenGL/Software an option, please?
I for one use Vulkan as my daily driver for RetroArch, and it provides precise buffer control on Wayland for low video lag (while GL does not), so using OpenGL would cause more lag on Wayland (triple buffer by default on OpenGL on Wayland, no way to make it 2 buffers).

Moving back over here, since the NEON work didn't pan out.
It was suggested that maybe the dropping of audio when seeking is causing issues, and that's worth exploring:

Hmm...we don't explicitly disable the audio system, but we don't feed it anything until there's audio to feed it...maybe we should feed it silence in these cases. I'll try that.


Need guidance on using and filling the line [boxy_depth = ...]

I'm having trouble understanding how to use and fill the line [boxy_depth = ...] within the context of the code. Could someone please provide some guidance or examples on its proper usage?

I think you can just simply place in a depth graph in RGB in a notebook. Here is my example: https://github.com/rfan-debug/LooseControl/blob/test-branch/test_note.ipynb


Multiple Resends

I'm having an issue where I send one notification but am receiving 5 of them on my device.

There is some bug in this API. The default retry strategy retries sending notification for 5 times in case of any errors. And I don't know exactly what but they are catching some exception that isn't really a valid exception and retrying on that. To overcome this, I'd manually set retry to 1 so that I only get one notification. geeknam@ please work on fixing this please.

@scmurray1 @ishansharma29 Will look into this. Thanks for reporting!

@scmurray1

Can you provide code details and input data on how exactly are you calling the API, I cannot reproduce the problem even after explicitly rasing GCMUnavailableException and returning error from make_request a certain number of attempts.
Can you please enable logging by adding gcm = GCM(API_KEY, debug=True) to the object and try to send a notification that raises the GCMUnavailableException and attach the logs here?
Are you using json_request() or plaintext_request() ?

@ishansharma29 What do you mean by an exception that is not really a valid exception?
An exception that is not coming from the GCM server but from the code itself? i.e. a situation where an HTTP POST request is actually made and a success response is returned, but an error is raised by some other part of the code and it retries to send the notification again.
For example, in the case of json_request() GCMUnavailableException is only raised in two places: #L314
       elif response.status_code == 503:
            raise GCMUnavailableException("GCM service is unavailable")
        else:
            error = "GCM service error: %d" % response.status_code
            raise GCMUnavailableException(error)

Maybe GCM server actually sends the notification and returns an HTTP response status code other than 200 for success?

I'm using plaintext from the example provided. I'm using python 2.9. I'll enable debugging and look at the output when I can. Probably a little later today. Thanks for the reply.
from gcm import GCM
gcm = GCM(API_KEY)
data = {'param1': 'value1', 'param2': 'value2'}
Plaintext request
reg_id = '12'
gcm.plaintext_request(registration_id=reg_id, data=data)

Ok here is the ouput. I had changed the retries from 5 to 1 but changed them back to 5.
gcmdebug.txt

@scmurray1 @ishansharma29 The problem has been fixed in develop branch https://github.com/geeknam/python-gcm/commit/6ef1313455f049f318c1381cb5c37238ed24a1aa#diff-464232c9a5921970b623c217c57caae7R443 , after the major refactoring that took place last year seems like an error slipped through.
Sorry for the incovenience, but the recommended way to use GCM is with the JSON request payload, that's why the plaintext_request was a bit neglected.
Anyway, I've updated the README to reflect this and also added an examples directory https://github.com/geeknam/python-gcm/tree/develop/examples
To solve this problem try the fix introduced in the 6ef1313455f049f318c1381cb5c37238ed24a1aa commit on the develop branch

Works perfect. Thank you for the quick fix.


[tests] variable fee structure tests

User Story

As an <audience/user>:  dev
I want to <action/task>: want to test that we have correct fees flexibility in contracts
so that <outcome/goal/benefit>: we can deploy them and know we have all needed states.

Type

[x] Enhancement
[ ] Maintenance
[ ] Refactor

Definition of Done

[ ] deploy a contract that only has fees in MKT and ensure that

Fees are calculated correctly
Fees are not able to be paid in Collateral Token
Fees are not able to be bypassed
Fees are able to be withdrawn by owner
Fees are not able to by withdrawn by non owner

[ ] deploy a contract that only has fees in Collateral Token and ensure that

Fees are calculated correctly
Fees are not able to be paid in MKT token
Fees are not able to be bypassed
Fees are able to be withdrawn by owner
Fees are not able to by withdrawn by non owner

[ ] deploy a contract that has fees in Either Collateral Token or MKT

Fees are calculated correctly
Fees are not able to be bypassed
Fees can be paid in MKT or Collateral Token
Fees are able to be withdrawn by owner (both MKT and Collateral Token)
Fees are not able to by withdrawn by non owner

[ ] deploy a contract that has no fees

Fees are calculated as 0
Minting works correctly with no fees
Even is user calls the mint function with isAttempToPayFees in MKT, everything still works.

This should help in understanding fee structure states.
COLLATERAL_TOKEN_FEE_PER_UNIT;
MKT_TOKEN_FEE_PER_UNIT;
IF both zero === no fees charged
IF both non zero === user decides which to pay
IF one zero (other not zero) === fees must be paid in non zero token.


Change the unzip of Linux binary from local to remote

Vincent needs this change for a problem he is seeing in certain install/builds.


typescriptify phenyl-standards

typescriptify phenyl-standards

The implementation of StandardUserDefinition should be discussed

the original removePasswordFromResponseData function could remove the nested property with the feature of  power-assign,  but after using sp2/updater, the new removePasswordFromResponseData cannot remove nested property.
should we support the feature of removing the nested property, in the new removePasswordFromResponseData?
ref: https://github.com/phenyl-js/phenyl/pull/178/files#diff-92409f7e80e7941d5fb2b4894451a330R36


[Event Request] codeunit 10861 "Payment-Apply" - OnAfterValidate

Hi, would it be possible to get an event OnAfterValidate codeunit 10861 "Payment-Apply", like this :
        "Amount (LCY)" := GenJnlLine."Amount (LCY)";
        Validate(Amount);
        Validate("Amount (LCY)");
        if ("Direct Debit Mandate ID" = '') and (GenJnlLine."Direct Debit Mandate ID" <> '') then
            Validate("Direct Debit Mandate ID", GenJnlLine."Direct Debit Mandate ID");
            OnAfterValidate(GenJournalLine);

    [IntegrationEvent(false, false)]
    local procedure OnAfterValidate(GenJournalLine: Record "Gen. Journal Line")
    begin
    end;

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Custom Login UI: API detect user need to update his password (changeRequired)

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions
[X] I have joined the ZITADEL chat

Describe your problem
Forward from https://discord.com/channels/927474939156643850/1153246325551415356
Describe your ideal solution

As an admin manager, I create a user with API: v2beta/users/human
{ password: { password: '..........', changeRequired: true } }

In Custom login UI: I wanna an API that I can detect user needs to update his password (get data object have field changeRequired)

Version
No response
Environment
ZITADEL Cloud
Additional Context
No response

Hei @quochuydev
Thanks for adding the issue.
We do not yet have all the functionalities in our new API.
We discussed about where this attribute should go and came to the conslusion, that you should get that when requesting the user.
We do have an issue that is already estimated #6434 to extend the user api.
I added a note and liked this issue in there.


Chained Dependency doesn't work

Say, that the user is installing a package with Dependency A, and if that dependency has another dependency, Dependency B, the tool doesn't install Dependency B.

A temporary workaround is to press update on the main dependency so that its dependencies are downloaded. Looking for a fix in the meanwhile.


Question about the mmcv version

Dear Author：
I have a problem，when I run this command:
pip install mmcv-full==latest+torch1.5.0+cu101 -f https://openmmlab.oss-accelerate.aliyuncs.com/mmcv/dist/index.html
mmcv-full's version will be 1.2.0.
then,the program will report an error:
 f'MMCV=={mmcv.__version__} is used but incompatible. ' \ AssertionError: MMCV==1.2.0 is used but incompatible. Please install mmcv>=1.0.5, <=1.0.5.
I have no idea about it. Can you give me an answer or some advice？
Best wishes

You can use pip install mmcv-full==1.0.5+torch1.5.0+cu101 -f https://openmmlab.oss-accelerate.aliyuncs.com/mmcv/dist/index.html to install the right version for mmcv.


consider removing copy of theories or making deepcopy?

In this code (which is a simplified example to debug something that happened elsewhere), if I comment in the cosmoother lines (which, note, naively don't look like they should do anything), the final likelihood call dies horribly. If I use deepcopy it is fine. I understand copy vs. deepcopy in standard python, where sometimes there might be a reason for the distinction, but... does one ever need/want to make a non-deep copy here? (if yes, maybe the help could explain what it does...)(I admit, a lot of my problems probably stem from not being an instinctive python user)
from desilike import Fisher, setup_logging
from desilike.likelihoods.cmb import BasePlanck2018GaussianLikelihood, TTHighlPlanck2018PlikLiteLikelihood, TTTEEEHighlPlanck2018PlikLiteLikelihood, TTLowlPlanck2018ClikLikelihood,
EELowlPlanck2018ClikLikelihood, LensingPlanck2018ClikLikelihood
from desilike.likelihoods import SumLikelihood
from desilike.theories.primordial_cosmology import Cosmoprimo
setup_logging()
Planckavg={"h":0.6736,"omega_cdm":0.1200,"omega_b":0.02237,"logA":3.044,"n_s":0.9649,
"tau_reio":0.0544}
Planckbest={"h":0.6732,"omega_cdm":0.12011,"omega_b":0.022383,"logA":3.0448,"n_s":0.96605,
"tau_reio":0.0543}
cosmodefault = Cosmoprimo()
cosmo=cosmodefault.copy()
#cosmoother=cosmodefault.copy()
cosmo(**Planckbest)
#cosmoother(**Planckavg)
likelihoods = [Likelihood(cosmo=cosmo) for Likelihood in [TTTEEEHighlPlanck2018PlikLiteLikelihood, TTLowlPlanck2018ClikLikelihood,
EELowlPlanck2018ClikLikelihood, LensingPlanck2018ClikLikelihood]]
likelihood_clik = SumLikelihood(likelihoods=likelihoods)
likelihood_clik()

Thanks for this actual bug report! it should be fixed now.
Usually, indeed, you'll want to use deepcopy().
copy() is only useful in case one wants to duplicate a calculator, without copying its dependencies.
If, e.g., we had a Theory giving the power spectrum at redshift z, given some cosmology cosmo:
theory1 = Theory(cosmo=cosmo, z=0.)

Then, if:
theory2 = theory.copy()
theory2.init.update(z=1.)

theory2 would use the same cosmo instance, but output the power spectrum at redshift z=1.

I didn't check the fix, but I guess this does make sense.


bug: User update is broken

Edit: This seems to affect any changes made to a user. When enabling 2FA in the user profile, for example, the same error occurs.
Login via OIDC currently seems to be broken using the latest Docker image. After being redirected from the IdP, PowerDNS-Admin fails to update the corresponding user object:
[2022-12-13 22:55:44,704] [app.py:1891] ERROR - Exception on /login [GET]
Traceback (most recent call last):
 File "/usr/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
   response = self.full_dispatch_request()
 File "/usr/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
 File "/usr/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
 File "/usr/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
 File "/usr/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
 File "/usr/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
 File "/app/powerdnsadmin/routes/index.py", line 388, in login
    result = user.update_local_user()
  File "/app/powerdnsadmin/models/user.py", line 463, in update_local_user
   user.password = self.get_hashed_password(
AttributeError: 'NoneType' object has no attribute 'decode'

I tracked this issue down to commit 4fd1b1001847eaef73b3da9a222c011068a3b2b7. The new code uses hasattr() to check if plain_text_password exists:
https://github.com/PowerDNS-Admin/PowerDNS-Admin/blob/4fd1b1001847eaef73b3da9a222c011068a3b2b7/powerdnsadmin/models/user.py#L461-L464
But plain_text_password always exists. It's None if not set otherwise. This leads to hasattr() always returning True, which in turn runs the code inside the if statement with invalid input.
Calling get_hashed_password() returns plain_text_password if it's None. So effectively the function returns None in this case:
https://github.com/PowerDNS-Admin/PowerDNS-Admin/blob/4fd1b1001847eaef73b3da9a222c011068a3b2b7/powerdnsadmin/models/user.py#L99-L106
This leads to update_local_user() effectively trying to call None.decode().
https://github.com/PowerDNS-Admin/PowerDNS-Admin/blob/4fd1b1001847eaef73b3da9a222c011068a3b2b7/powerdnsadmin/models/user.py#L463-L464

Just popping in here to say I've seen the same. Thanks for providing the commit that broke it - I can fight with that 👀


Permission Rename

Please describe the changes this PR makes and why it should be merged:
the meaning of this permission bit has changed slightly and now also affects voice channels and any future channel types that may come.
Semantic versioning classification:

[ ] This PR changes the library's interface (methods or parameters added)

[x] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

Duplicate of #1788


Bulk Local Injestion: How to initiate from Dockerfile? & "--watch" is not working

I'm running PGPT in docker and following the documentation here to create an "injestion" folder: https://docs.privategpt.dev/manual/document-management/ingestion
After I run the container I able to get the following command to work (i.e., PGPT checks the folder and embeds the documents)
make ingest /home/worker/app/injest -- --watch --log-file /home/worker/app/injest/.PrivateGPT.log
Ideally I'd just like this to be set up before the container runs. I can't seem to figure out how to modify the Dockerfile to have this run when the container initially starts.
Also, I've noticed the --watch flag doesn't seem to work. I have to re-run the entire command to have PGPT re-scan the folder
However, despite it not actually embedding the new files based on the log it looks like the error log is constantly detecting changes?
[2024-05-06 11:44:33.883] [INFO] Detected change in at path=/home/worker/app/injest/.PrivateGPT.log, ingesting
[2024-05-06 11:44:33.884] [INFO] Started ingesting file=/home/worker/app/injest/.PrivateGPT.log
[2024-05-06 11:44:34.359] [INFO] Completed ingesting file=/home/worker/app/injest/.PrivateGPT.log
[2024-05-06 11:44:34.359] [INFO] Detected change in at path=/home/worker/app/injest/.PrivateGPT.log, ingesting
[2024-05-06 11:44:34.359] [INFO] Started ingesting file=/home/worker/app/injest/.PrivateGPT.log
[2024-05-06 11:44:34.911] [INFO] Completed ingesting file=/home/worker/app/injest/.PrivateGPT.log
[2024-05-06 11:44:34.911] [INFO] Detected change in at path=/home/worker/app/injest/.PrivateGPT.log, ingesting
[2024-05-06 11:44:34.911] [INFO] Started ingesting file=/home/worker/app/injest/.PrivateGPT.log
[2024-05-06 11:44:35.382] [INFO] Completed ingesting file=/home/worker/app/injest/.PrivateGPT.log
[2024-05-06 11:44:35.383] [INFO] Detected change in at path=/home/worker/app/injest/.PrivateGPT.log, ingesting
[2024-05-06 11:44:35.383] [INFO] Started ingesting file=/home/worker/app/injest/.PrivateGPT.log

FYI, I'm only able to get the command make ingest to work when I run it as root & after I copy both Makefile and pyproject.toml into the /home/worker/app directory. I can't seem to figure out how to run this command "detached"

Have you figured it out? I tried adding the command into the Dockerfile before the entry point (like you said, after copying Makefile and .toml, and it seems that script runs successfully but the files aren't present in the DB once the container is up.


file to import not found or unreadable: bootstrap

Hello,
I am having trouble using your add-on. I get the following errors when trying to build:
File: /Users/me/projects/blah/tmp/tree_merger-tmp_dest_dir-c9AFWRJW.tmp/app/styles/app.scss (1)
file to import not found or unreadable: bootstrap
Current dir: /Users/me/projects/blah/tmp/tree_merger-tmp_dest_dir-c9AFWRJW.tmp/app/styles/

app/styles/app.scss:
@import "bootstrap";

package.json:
...
"devDependencies": {
    "broccoli-asset-rev": "^2.0.0",
    "ember-cli": "0.2.0-beta.1",
    "ember-cli-app-version": "0.3.1",
    "ember-cli-babel": "^4.0.0",
    "ember-cli-bootstrap-sassy": "0.0.12",
    "ember-cli-dependency-checker": "0.0.7",
    "ember-cli-htmlbars": "0.7.4",
    "ember-cli-ic-ajax": "0.1.1",
    "ember-cli-inject-live-reload": "^1.3.0",
    "ember-cli-qunit": "0.3.8",
    "ember-cli-sass": "^3.1.0",
    "ember-cli-uglify": "1.0.1",
    "ember-data": "1.0.0-beta.15",
    "ember-export-application-global": "^1.0.2",
    "express": "^4.8.5",
    "glob": "^4.0.5"
  }

ember-cli v. 0.2.0-beta.1
ember v. 1.10.0

Resolved issue by adding this to config/environment.js:
  ENV.sassOptions = {
    includePaths: [
      'bower_components/bootstrap-sass-official/assets/stylesheets'
    ]
  }

Yes I'm afraid this is due to a breaking change in the way ember-cli v0.2.0 handles cascading dependencies (aka, bootstrap sass depends on sass). It's on my list of things to fix but I'd happily take a PR if someone has the time for that.

@ksnyde Would mind give us an overview on how to fix it?I'm willing to fix it, as I'm upgrading an ember-cli project from 0.1.3 to 0.2.0, but I'm still missing what bits have changed ever since.

On the Ember CLI site they make reference to the fixes needed but when I was glancing at this the other day it occurred what I think happened may have been that the underlying ember-cli-sass addon changed and now is using ENV based configuration rather than modifying the Brocfile. Not sure about this though as it was just a passing glance.
The Ember CLI reference I was referring to is:

In reference to setupPreprocessorRegistry:  0.2.0 breaking changes

Historically all that was needed was non-destructively adding the path to Boostrap's path into ember-cli-sass's configuration:
https://github.com/lifegadget/ember-cli-bootstrap-sassy/blob/master/index.js#L22

would love it if you could give me a PR ... been slammed recently so haven't had any cycles but will get it merged in promptly if you have some bandwidth

@ksnyde Thanks, sounds good. I'll hit you up when I've got more questions!

I did not have success with @sbsurf 's suggestion, but "solved" this by using the full path in my app.scss:
@import "../../bower_components/bootstrap-sass-official/assets/stylesheets/bootstrap";

@ballPointPenguin what version of CLI are you using? I have used @sbsurf approach without issue on 0.2.0.

@vicentereig have you made any progress on that PR?

@ksnyde it looks like the sassIncludes need to be in the environment config, and not in the app options.  I'll be able to test in the next two days. Working on cli version 0.2.1.

@tgandee79 yup, that's a change from earlier. It happened at the same time as the API breaking change in 0.2.0 so I wasn't sure the source of the problem at first.

@ksnyde I have a working branch with a number of updates.  I can send a PR if you give me permissions.  The branch updates the node_modules to use the latest, removes hbs support in favor of htmlbars, moves the sassOptions to treeForStyles.  I updated the dummy app to have working bootstrap classes.

@tgandee79 sounds awesome. Hope to see that soon.
@ksnyde I was using a very fresh ember-cli 0.2.1

I think with all the changes that have been made this is closable. If anyone disagrees please reopen.


Demo on load clustering improvement

Now the demo allows the user to "Apply Markov clustering on load" and bring the graph already simplified through possibly multiple levels of clustering. However, when we expand these collapsed clusters, we get poor initial positions for expanded content. So we should try applying partial layout on each cluster content before collapsing them. That way collapsed content will have some decent layout which would be used upon expand to produce a stable layout.

Now, when we expand collapsed clusters we get better initial positions for expanded content.
However, when we create a node and move a cluster as the child of this node, position and size values of the node are not updated immediately and the collapse operation applied subsequently cannot get correct position and size values of the node. I temporarily solved the position problem but size problem still exits and affects the fish-eye operation. I suspect this can be a cytoscape.js bug, so it should be further explored and an issue should be opened in cytoscape.js if necessary.

Closed with keyword 'resolve'.


Enable bracketed paste mode when sending lines to the repl

The problem
Recently, after returning to some Python projects, I noticed a problem with iron.nvim's core.send_file() function. While the function still pushes all lines to the REPL buffer, each line's indentation increases by one. This issue arose in my environment where I'm using ptpython on macOS.
The fix
Upon investigating, it appears the issue stems from lowlevel.send_to_repl(). The function wasn't initiating bracket paste mode, which seems to be the root of the problem. I implemented a patch that triggers bracketed paste mode before sending code to the REPL, which resolved the issue in my setup. While I'm not certain if this issue is widespread or what other setups it might impact, this solution has worked well for me so far.
I hope this fix can be helpful to others experiencing similar issues!

Also note that when configuring the plugin, I’ve had to disable the default format function when using ptpython due to an issue with the bracketed_paste_python function. The function was adding unnecessary carriage returns which caused discrepancies between the content sent to the REPL and the actual buffer content. As a result, errors reported by the REPL did not correspond accurately to the line numbers in the buffer, complicating debugging. Here’s how I configured the plugin:
    require("iron.core").setup({
      config = {
        repl_definition = {
          python = {
            command = "ptpython",
            -- Disabled `format` function to avoid carriage return issues
          },
        },
      },
    }


ARC-1448 adding api gateway

What's in this PR?
Changing service descriptor to add API gateway in front of our app so we can shove github and jira webhooks directly into an SQS queue to prevent losing information if the app goes down for any reason.
Why
Because we would sometimes lose webhooks because of a bad deployment or code issue.
Added feature flags
None
Affected issues
ARC-1448
How has this been tested?
Manual
Whats Next?
Relax

What's our migration strategy of the enterprise customers? When this will happen? I believe our GHE server app needs to have Organisation hooks perms, how can we make sure they have it for sure? https://docs.github.com/en/rest/overview/permissions-required-for-github-apps?apiVersion=2022-11-28#organization-hooks

@bgvozdev for GHE customers, we can update the app webhook url via an admin API.  I haven't implemented it yet because the old webhook are still going to work, so nothing broken.  Wanted to get this out into prod and start moving the production app webhook first, then add that admin API to change the GHE apps.

@mboudreau But does the app has enough permissions to update it in GHE?

Close it for now as it is too long. Will discuss later. Should be tracked in this ticket https://softwareteams.atlassian.net/browse/ARC-1448


Modified title to show AWS instead of GCP

Description

The document title said to register inside GCP while we are in the AWS guide.
Motivation and Context

The document is about AWS and GCP is mentioned at its place.
It solves incoherence between title and real context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ X ] Have not tested

Test not really necessary. It is a small change in the documentation. All tests should pass as before.
Types of changes

[ X ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ X ] My change requires a change to the documentation.
[ X ] I have updated the documentation accordingly.

@juangascon Could you add a signed off by line for DCO using
git commit -s --amend --no-edit;git push -f

How I do that? Should I clone this repo and use the commands you show me?


Support for blocking raw IP addresses

Hi,
From this issue https://github.com/mholt/caddy/issues/335, I would like to make a feature request.
At the moment you can only block countries but I would love the ability to block particular IP address(es). Also, the database requirement may become optional and only be required if you need to block a country.
This is a really useful feature.

working on it

done


The HBC version (76) is not supported.

Add support for latest hermes

@ErbaZZ is working on it. Cooming soon !!!!

@bongtrop  Please publish it. Latest version is 0.1.3

@OrkhanAlikhanov We are testing with a real app. We will release the new version and publish it soon.

Cool. I disassembled boltfood app. Version 1.9.0 for android. Didn't reassemble it back though, I only needed to see something there and I were able to thanks to you guys' effort. I really appreciate 👍🏽👍🏽👍🏽


Error During Testing My Own RL Experiment with Code from Simple Run

Hi,
I used the code in simple run - single.py for the stock management game example to test it on my own RL experiment. But the following bug popped up.
AnyLogic Version: University 8.8.1
Alpyne Version: 0.2.0
Java Version: 19.0.1
Python Version: 3.8.8
I have attached my full log here.
Could anyone help me on this? Thank you.
Exception during discrete event execution:
Cannot invoke "com.anylogic.engine.EventOriginator.execute()" because "this.dg" is null
java.lang.NullPointerException: Cannot invoke "com.anylogic.engine.EventOriginator.execute()" because "this.dg" is null
	at com.anylogic.engine.Engine.f(Unknown Source)
	at com.anylogic.engine.Engine.runFast(Unknown Source)
	at com.anylogic.engine.Engine.runFast(Unknown Source)
	at com.anylogic.rl.connector.bonsai.BonsaiBrainLearningPlatform.run(BonsaiBrainLearningPlatform.java:271)
	at com.anylogic.alpyne.sim.api.ModelRunPlatform.run(ModelRunPlatform.java:224)

alpyne.log

Closing as this is likely resolved in v1.0.0. Feel free to create a new issue if it remains.