Spring Transactions in GraphQLResolver methods not working

Hi,
I am starting using GraphQL with Spring, but having an issue in which transactions started on the  GraphQLResolver do not seem to be joined by the transactional calls of injected beans (using JPA with Hibernate).
For example, assume a Post entity with a lazily fetched list of Comment. A  GraphQLResolver  for the comments could be (in Kotlin):
@Transactional
 fun getComments(post: PostEntity): List<CommentEntity> {

       println("------- TRANSACTION: " + TransactionSynchronizationManager.isActualTransactionActive())
       val reloaded = postRepository.findById(post.id).get()
       return reloaded.comments

        //return commentRepository.findByParentPostId(post.id!!)
}

when that method is called, a transaction is started (the print method is true), but then ending in a  java.util.concurrent.CompletionException: org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: org.tsdes.advanced.graphql.database.entity.PostEntity.comments, could not initialize proxy - no Session when the execution does reach reloaded.comments.
Somehow, the findById does not see to join the transaction.
This seems the same kind of problem reported here:  https://stackoverflow.com/questions/48037601/lazyinitializationexception-with-graphql-spring#
The workaround is to do not rely on GraphQLResolver being transactional, and put all transactional code in a separated injected bean (e.g, the commented out last line in the code snippet).
But I am wondering, is anything special on on how those GraphQLResolver beans are created? Any idea or solution on why this is not working? Anyway, could be good to mention this issue (if it is indeed an issue and not something silly I misunderstood) in the documentation (I guess I am not the first one trying to use GraphQL with JPA).
Btw, full code example (with running tests) can be found here

@apottere Do you have an idea why this is off the top of the head or does this need to be investigated?

I haven't run into this issue myself, and I don't use transactions at all, so I'm not sure what's causing this off the top of my head.  It might have something to do with the way we call reflective methods on the object - AFAIK the transactional annotation works by creating a proxy to the real bean that starts the transaction.  It could be we're somehow bypassing the proxy when we invoke the method?  I would be surprised if this were the case but I can't think of anything else that would be causing it.

hi,
the transaction seems created (isActualTransactionActive() returns true), but somehow it is still not working. Maybe the injected beans do work on a different DB session? but that seems like very low-level details of Spring JPA which is way beyond my knowledge :(

Ah, sorry, I missed the part about the transaction being created.  In that case I'm really not sure what would be causing it to fail if the resolver is transactional but succeed if the service is transactional.

I am the creator of the mentioned stackoverflow question and i might be able to shed some light (but be aware that i myself am no expert on this matter).
I assume that you are querying for some properties of the comments, right? So what happens is that your getComments returns a proxy hibernate object (because it is lazily fetched) and your transaction is getting closed after the return. Then as soon as graphlql tries to resolve the properties defined in the query it tries to access a property of the proxy and thus triggers it to be resolved. This throws the exception, because it is no longer inside the initial transaction.
Could you try to do a comments.size() before returning the list? If that works, then it kind of confirms my suspicion.

Hi,
you are right! that worked. So this is something related to Hibernate, and not really related to GraphQL in the end... I am sorry. Many thanks for your time

@arcuri82 @KammererTob @apottere A few more thoughts and information on that.
I think in a mutation resolver it could be necessary to set a transaction on a method, because on the one hand the state is changed and on the other hand several database calls can take place which have to be packed in a transaction bracket. So I tried @Transaction again and it works!!. The database is rolled back.
@Component
public class RootMutationResolver implements GraphQLMutationResolver {

    @Transactional(rollbackFor = DataIntegrityViolationException.class)
    public Route createRoute(String flightNumber) {
    	Route route = new Route(flightNumber);
        routeRepository.save(route);
       throw new DataIntegrityViolationException("Test");
    }
}


SWDEV-372006 - Defined HSA_PATH in hipvars.pm

HSA_PATH was removed from hipvars and hipcc.pl. But this is causing build failure in pytorch as reported in swdev ticket
Adding HSA_PATH back to unblock staging testing
Depends-On: Ia461cb4da2c0e0967703033f5c2c79b67732f5b5
Change-Id: I8f18b27f40f4080b68a936450ba6e092f997ec47

Closing in favor of #3120.


Errors during docker building

Hi, after I run:
docker build . \
    -t imagesweserv \
    --build-arg NGINX_VERSION=1.17.10

Then the following error shows:
Adding repo from: https://libjpeg-turbo.org/pmwiki/uploads/Downloads/libjpeg-turbo.repo
libjpeg-turbo official RPMs                      11  B/s | 276  B     00:25
Errors during downloading metadata for repository 'libjpeg-turbo':
  - Curl error (47): Number of redirects hit maximum amount for https://kumisystems.dl.sourceforge.net/project/libjpeg-turbo/repodata/6e70518d5966a892606a62893fc4e4763ae41089-primary.xml.gz [Maximum (6) redirects followed]
  - Curl error (47): Number of redirects hit maximum amount for https://kumisystems.dl.sourceforge.net/project/libjpeg-turbo/repodata/3d3dfab4b2b690768b053775d41fbd3859a7eb2c-filelists.xml.gz [Maximum (6) redirects followed]
Error: Failed to download metadata for repo 'libjpeg-turbo': Yum repo downloading error: Downloading error(s): repodata/6e70518d5966a892606a62893fc4e4763ae41089-primary.xml.gz - Cannot download, all mirrors were already tried without success; repodata/3d3dfab4b2b690768b053775d41fbd3859a7eb2c-filelists.xml.gz - Cannot download, all mirrors were already tried without success

Then I try curl -i https://kumisystems.dl.sourceforge.net/project/libjpeg-turbo/repodata/6e70518d5966a892606a62893fc4e4763ae41089-primary.xml.gz
HTTP/1.1 302 Moved Temporarily
Server: nginx
Date: Fri, 04 Dec 2020 03:59:39 GMT
Content-Type: text/html
Content-Length: 138
Connection: keep-alive
Location: https://downloads.sourceforge.net/project/libjpeg-turbo/repodata/6e70518d5966a892606a62893fc4e4763ae41089-primary.xml.gz?download&failedmirror=kumisystems.dl.sourceforge.net

<title>302 Found</title>

302 Found

nginx

Anyway ideas to fix it?

Never mind, just fix it by using a mirror repo: https://npm.taobao.org/mirrors/libjpeg-turbo/


When setting Dataset filesystem_location, create class method that ensures the location is relative to MEDIA_ROOT

Added to engineering debt list.


Bugfix/ar connection too soon

I'll be testing this more, but wanted to get some early feedback.
I know the gem isn't super popular, but I'm surprised no one else has run into this yet. It'd happen for starting a new project or when running rails db:drop (which works without this fix) and rails db:create (which doesn't work without this fix) because it's trying to connect to the database before one exists while also trying to create it.

Well, I guess this isn't going to work :D
[1] pry(main)> User.organizers
NoMethodError: undefined method `organizers' for User (call 'User.connection' to establish a connection):Class
from /Users/jon/.rvm/gems/ruby-2.5.1@headway_rails_template/gems/activerecord-5.2.1/lib/active_record/dynamic_matchers.rb:22:in `method_missing'


Terraform 0.14 update

This PR was made using the tf14.rb script.

Thanks for the review! Merging now.

https://github.com/gruntwork-io/terraform-kubernetes-namespace/releases/tag/v0.2.0


Unable to create Browser when HTTP_PROXY environment variable is set

Ruby version: 2.1.2
watir-webdriver version: 0.6.11
selenium-webdriver version 2.44.0
Replication steps:
>irb
2.1.2 :001> require 'watir-webdriver'
2.1.2 :002> ENV['HTTP_PROXY'] =''
2.1.2 :003> b = Watir::Browser.new
firefox launches, it is running for some time, then it is automatically closed. After that, exception is logged in console
Additional info:
I was able to replicate issue using ENV['http_proxy']
I was not able to replicate issue using ENV['ftp_proxy'] and ENV['https_proxy']
Here is exception stack:
URI::InvalidURIError: bad URI(absolute but no path): http://
	from /Users/karlosmid/.rvm/rubies/ruby-2.1.2/lib/ruby/2.1.0/uri/common.rb:146:in `split'
	from /Users/karlosmid/.rvm/rubies/ruby-2.1.2/lib/ruby/2.1.0/uri/common.rb:211:in `parse'
	from /Users/karlosmid/.rvm/rubies/ruby-2.1.2/lib/ruby/2.1.0/uri/common.rb:747:in `parse'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/http/default.rb:92:in `new_http_client'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/http/default.rb:17:in `http'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/http/default.rb:83:in `response_for'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/http/default.rb:39:in `request'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/http/common.rb:40:in `call'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/bridge.rb:640:in `raw_execute'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/bridge.rb:101:in `create_session'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/remote/bridge.rb:68:in `initialize'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/firefox/bridge.rb:36:in `initialize'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/common/driver.rb:31:in `new'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver/common/driver.rb:31:in `for'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/selenium-webdriver-2.44.0/lib/selenium/webdriver.rb:67:in `for'
	from /Users/karlosmid/.rvm/gems/ruby-2.1.2@lawfirmmatrix/gems/watir-webdriver-0.6.11/lib/watir-webdriver/browser.rb:46:in `initialize'
	from (irb):7:in `new'

This is a feature, not a bug.
ENV['HTTP_PROXY'] is what Selenium uses to send the processes through a running proxy that you've set up.
It is expecting the value to be nil or to have an url in it. An empty string should cause it to fail, as should any other string containing an invalid url.
Regardless, this is something for the selenium project, not the watir project.

From watir-webdriver documentation:
http://watirwebdriver.com/browser-proxies/
So http_proxy env variable is preceding setting proxy using browser profiles?

I'm not all that familiar with the proxy code since I've never used it. I'm not sure how the precedence works, just that it can't be an empty string for what you were running.

The point is that HTTP_PROXY is used as a proxy for communicating via wire protocol (i.e. from selenium client to selenium server). That's probably not what you want to achieve, so just get rid to env variable and configure browser proxy instead.

Thanks @p0deje for clarification!
I also noted possible issue for different values of http_proxy, I will post that to selenium project.
Regards, Karlo.


Unable to scan POST requests

Using extended.py with Scope options with request method option throws following error.
Actual behavior
Version 3.0.6 - Copyright 2017 Tijme Gommers<EMAIL_ADDRESS>
Traceback (most recent call last):
  File "extended.py", line 168, in <module>
    main()
  File "extended.py", line 69, in main
    Request.METHOD_GET,
NameError: global name 'Request' is not defined

Steps to reproduce the behavior
Added following scope options to the request.py main() method
options.scope.request_methods = [
    Request.METHOD_GET,
    Request.METHOD_POST
    ]

Running default scan runs only on GET requests.
Also - Since I could not find the answer in README, asking here: Will it be possible to scan POST request and inject in specific parameter?

Hey @yogendra1911 ,
Could you try to add from nyawc.http.Request import Request? I think it will resolve the issue.
The FormDataAction.py class will be used and automatically inserts the payload in <form>'s that are found on the website.


[Feature Request] Creation of InStreams directly from TextBuilder

The new TextBuilder is indeed very practical, and also very fast in terms of performance. It would be very helpful if you could generate an InStream directly from this and download it with the function "DownloadFromStream". You could then avoid the laborious detour to create this InStream first.

Hi! Thank you for your suggestion. We are working on focusing the activity in this repository on addressing issues related to the AL development tools.
As this issue appears to require changes across multiple areas of our product, we recommend that you post this on our Ideas forum at aka.ms/BCIdeas, or vote up the idea if its already there.
We are constantly monitoring top ideas and will consider them for a future release.


[FEATURE] 启动时发生错误

python3 startup.py -a
Traceback (most recent call last):
File "/Users/hbz/git/Langchain-Chatchat/startup.py", line 35, in
from server.utils import (fschat_controller_address, fschat_model_worker_address,
File "/Users/hbz/git/Langchain-Chatchat/server/utils.py", line 4, in
from fastapi import FastAPI
ModuleNotFoundError: No module named 'fastapi'
is:issue is:open  No module named 'fastapi'

/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/langchain/chat_models/init.py:31: LangChainDeprecationWarning: Importing chat models from langchain is deprecated. Importing from langchain will no longer be supported as of langchain==0.2.0. Please import from langchain-community instead:
from langchain_community.chat_models import ChatOpenAI.
To install langchain-community run pip install -U langchain-community.
warnings.warn(
/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/langchain/llms/init.py:548: LangChainDeprecationWarning: Importing LLMs from langchain is deprecated. Importing from langchain will no longer be supported as of langchain==0.2.0. Please import from langchain-community instead:
from langchain_community.llms import OpenAI.
To install langchain-community run pip install -U langchain-community.
warnings.warn(
/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/pydantic/_internal/_config.py:322: UserWarning: Valid config keys have changed in V2:

'schema_extra' has been renamed to 'json_schema_extra'
warnings.warn(message, UserWarning)
/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/langchain/document_loaders/init.py:36: LangChainDeprecationWarning: Importing document loaders from langchain is deprecated. Importing from langchain will no longer be supported as of langchain==0.2.0. Please import from langchain-community instead:

from langchain_community.document_loaders import JSONLoader.
To install langchain-community run pip install -U langchain-community.
warnings.warn(
ImportError: dlopen(/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/_cffi_backend.cpython-312-darwin.so, 0x0002): tried: '/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/_cffi_backend.cpython-312-darwin.so' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/System/Volumes/Preboot/Cryptexes/OS/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/_cffi_backend.cpython-312-darwin.so' (no such file), '/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/_cffi_backend.cpython-312-darwin.so' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64'))
thread '' panicked at /Users/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/pyo3-0.20.3/src/err/mod.rs:788:5:
Python API call failed
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Traceback (most recent call last):
File "/Users/hbz/git/Langchain-Chatchat/startup.py", line 38, in
from server.knowledge_base.migrate import create_tables
File "/Users/hbz/git/Langchain-Chatchat/server/knowledge_base/migrate.py", line 11, in
from server.knowledge_base.kb_service.base import KBServiceFactory
File "/Users/hbz/git/Langchain-Chatchat/server/knowledge_base/kb_service/base.py", line 29, in
from server.embeddings_api import embed_texts, aembed_texts, embed_documents
File "/Users/hbz/git/Langchain-Chatchat/server/embeddings_api.py", line 3, in
from server.model_workers.base import ApiEmbeddingsParams
File "/Users/hbz/git/Langchain-Chatchat/server/model_workers/init.py", line 2, in
from .zhipu import ChatGLMWorker
File "/Users/hbz/git/Langchain-Chatchat/server/model_workers/zhipu.py", line 11, in
import jwt
File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/jwt/init.py", line 17, in
from .jwa import std_hash_by_alg
File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/jwt/jwa.py", line 26, in
from cryptography.hazmat.primitives.asymmetric import padding
File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/cryptography/hazmat/primitives/asymmetric/padding.py", line 9, in
from cryptography.hazmat.primitives import hashes
File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/cryptography/hazmat/primitives/hashes.py", line 9, in
from cryptography.hazmat.bindings._rust import openssl as rust_openssl
pyo3_runtime.PanicException: Python API call failed  还有如上错误

@imClumsyPanda 麻烦看下哦

目前3.12还没有进行测试，建议按照项目readme和wiki中的版本要求及依赖包安装完成后再启动项目


customizable date format

users using YMD date format may have difficulty reading 'Human' date format.
I'm using patch file to work around the issue, but It'd be great to be able to customize date format.
func FormatDateTimeHuman(date time.Time) string {
	return date.Format("2006-01-02 15:04:05 Mon")
}

func FormatDateHuman(date time.Time) string {
	return date.Format("2006-01-02 Mon")
}

https://github.com/muety/wakapi/blob/7f2544f003415eef26eda3c5e0778a8e51814486/helpers/date.go#L35-L42

Hi @scarf005, thanks for this feature request! I see how customizable dates would add to better user experience.
However, this touches on the whole big topic of internationalization / i18n, which we decided to be out of scope for Wakapi. Currently, all of Wakapi is in en_US locale (speaking in nerd's terms) as sort of the lowest common denominator. I would love to add in multi-language support, customizable dates, etc. to improve the experience for international users. But, unfortunately, I think it would be way to big of a effort for a project of Wakapi's size and I'd rather prefer to spend our very limited capacity on feature development instead. Wakapi's philosophy is to be minimalist and keep things simple in many different regards.
Sorry to close this issue! Hope you'll still enjoy using Wakapi and keep contributing :-)

I don't think this issue is related to internalization at all. What i meant was options to choose such as:

YYYY-mm-dd
mm/dd/yyyy
dd/mm/yyyy
dd Month year

Yes, I got that point. But I think it add comparatively low benefit.


iOS Safari only the visible part of the calendar is rendered (container isn't scrollable)

react-dates version
<EMAIL_ADDRESS>Describe the bug
Only the visible part of the calendar renders on iOS.
Source code (including props configuration)
Steps to reproduce the behavior:
Visit this codesandbox using a browser on iOS.
<DayPickerRangeController
  startDate={selectedDates.start}
  endDate={selectedDates.end}
  onDatesChange={(start, end) => setSelectedDates(start, end)}
  focusedInput={focusedInput}
  onFocusChange={(focusedInput) => {
    setFocusedInput(focusedInput ?? "startDate");
  }}
  orientation="verticalScrollable"
  numberOfMonths={12}
  hideKeyboardShortcutsPanel={true}
  isOutsideRange={() => false}
  firstDayOfWeek={1}
/>

Screenshots/Gifs

Smartphone (please complete the following information):

Device: iPod touch
OS: iOS 13.6
Browser: Safari
Version: 13

Additional context
Probably related to https://github.com/airbnb/react-dates/issues/1829
Issue doesn't occur on MacOS safari.
Turning the phone's display off and on causes the rest of the calendar to render.

It seems like you can fix this issue by adding transitionDuration={0}  to your react-date component. It's not optimal, but for now I think I'll do. Let me know if it works for you.

We have the same issue on iOS tablets.
You can always swipe two times either to the left or right side.
transitionDuration={0} fixes the issue for us, but would be great if there could be a general fix for this issue.
It is also easily reproducible by storybook implementation.
http://airbnb.io/react-dates/?path=/story/daterangepicker-drp--default


Fix: Use tracefile size/count to prevent storing the trace

The --no-output flag is ignored in live mode, the traces are actually
stored by the relay in ~/lttng-traces/<hostname>/<session-name>. In
live mode we need some storage because the client asks the relay for
data and the relay cannot store the trace in memory for an arbitrary
amount of time.
A way to prevent the trace from always increasing in size is to use the
--tracefile-size and --tracefile-count options to limit the size of the
tracefiles on disk, the relay will automatically overwrite older files a
keep a ring-buffer of tracefiles on disk. If the client is too slow to
fetch the data before it is overwritten, we will see lost packets, so we
need to increase the amount we keep.
In this patch, we keep at most 5 files of 1M of trace for each CPU, if
it is not enough, it will be easy to change.
Signed-off-by: Julien Desfossez<EMAIL_ADDRESS>

Thanks for this!


Installed Packages Not Properly Handled

I just updated my fork of ChocolateyGUI after ~43 commits to upstream, and built and run the program. When I view the details of a locally installed package, the "Update" (if applicable) and "Remove" buttons are not shown. Instead, I see an "Install" button that does nothing when clicked. (The window dims as if the PowerShell overlay would appear, but it never does and the dimming disappears less than a second later.)
A few weeks ago I installed Chocolatey 0.9.10-beta1-329-g3b447fc3. I suspect a version incompatibility is the cause of this problem.

Could you pull again and let me know if you're still seeing issues? If so, could you provide screenshots or steps to reproduce? Thanks!

After I did a pull and merge (my HEAD is now e89a9dcc), the ChocolateyGUI behavior described above does not change. Furthermore, when I actually click the erroneously displayed Install button, the PowerShell overlay appears, displays “Installing Package” and then hangs indefinitely. Screenshots forthcoming.

Here are the screenshots:

You don't see it, but it's actually throw an exception. Still bad behavior either way :|. Will look into it.

AutoHotKey is installed already is what you're also saying, yes?

Yes, AHK is already installed. It is simply the package I chose as an example.

Could you screenshot the results of choco list -lo AutoHotkey?

NVM, I can reproduce. Odd

Yes, Update and Reinstall do appear in the right-click menu, but when I click Update, I get the same “Updating package”-and-hang behavior I saw before.

Try the latest develop and let me know if you still have issues. Progress is still a little derpy so it won't do much visibly until it completes the choco operations.

Yep, that did it! Works like a charm. Thank you so much!

Awesome! Let me know if you run into any other issues. I want to make sure that 0.14 is a high quality release.


Rethink building

The usage of CMake and bundled dependencies through submodules (#34) wasn't a bad idea. However, it did make nene much harder to use, much more storage-heavy and way less flexible for other languages (Nelua usage got much harder compared to before).
This process should be improved, and should consider the usage by other languages (#35) while still being cross-platform.

After getting some feedback, I decided to follow how it's done on SDL


Update readme

/assign @mdelder

I removed the origin arch diagram and grc part, since according to the feedback. We should put open source part only  at first, and update this when more components are open sourced.
FYI @juliana-hsu @mikeshng

Some rewording recommendations for the main readme:
open-cluster-management uses a hub - agents model. A hub centralizes control.  Agents (which we call klusterlet) reside in each cluster to manage registration to the hub and execute hub instructions.
API and controller for cluster registration and manifests delivery to clusters.  Simple core functions to connect clusters (klusterlet) to the hub.  Other components run upon this base. The following repositories describe the API and controllers for registration.
There are demonstrations available here:  https://github.com/open-cluster-management/demo-subscription-gitops .

About the api section:
The api repository defines relevant concepts and types for problem domains related to managing 0..* Kubernetes clusters.

Is the 0..* a typo? Or should we say various?

Contributing section:
Now, you can follow the getting started guide to work with the xxx.

What is xxx?

development.md section (some typos):
Generating a CRD for the first time
Make sure you have controller-gen locally installed in the _output/tools/bin directory.
For example, addon APIs are in the ./addon/v1alpha1 directory.
and you can simply remove these two lines:
in calls to the Makefile targets.

operators-subscription section:
Find the nginx pods deployed to current namespace, and the number of backend pods is overwritten to 3

Apparently looking at the dictionary definition, overwritten is to replace completely and overridden is to replace for some cases (learn something new every day).  Which is applicable here?

Troubleshooting
Please refer to Troubleshooting documentation for further info.
If you have any further questions, please refer to help documentation for further information.

the help documentation link seems to be missing?

operators-deployable readme is missing?

operators-placementrule readme is missing?

operators-channel section:
Channel controller syncs and promotes resources from the target source to a channel namespace on your hub cluster or to a host(such as object bucket). Then your subscription can consume these resources from the channel directly.
Examples can be found under the examples folder.
For a quick start, we are going to create a namespace channel, meaning you can add the resource to a namespace, which is watched by our channel operator, then the operator will promote this resource to the namespace channel, which should be the namespace where the channel operator sits.
The following example is tested on a minikube, so that you can play with this operator without worrying about things such as security.
Setting up a channel to sync resources between your hub cluster and an object bucket
Create a channel and deploy the payload (the resource you want the channel operator to help you move/promote)
The result, a config map wrapped by deployable in the default namespace.  It will also deploy a channel resource in the ch-ns namespace.
The resource is moved to its destination, and subscription can take over from here, deploying this resource (it will be the ConfigMap deployed) to your managed cluster.

singular resource?

Troubleshooting
Normally we check the reconcile trace. For each flow, it should at least have a valid reconcile log for the deployable.
Please refer to Troubleshooting documentation for further info.

operators-subscription section:
Subscribes resources from channels and applies them to kubernetes
Find the nginx pods deployed to current namespace, and the number of backend pods overwritten to 3

see previous comment about same sentence

Troubleshooting
Please refer to Troubleshooting documentation for further info.

@juliana-hsu you could directly add comments on each line of code, that would make it easier to understand :)

Where is the Contributor License Agreement mentioned in the Contributing a Patch section?  Should I reach out to legal to get one?

Some comments on the policy collection community.  The DCO needs to be added to the CONTRIBUTING Doc.  Small typo (italicized):  "Resources: View the following resources for more information on the components and mechanisms that are implemented in the product governance framework."
Need to put on the website a rough timeline for the availability of GRC.

@cadawson thanks would  you like to add comment by lines?

@cadawson would you like to review  this again?


missing libavcodec.so.59/Update to libavcodec.so.60

Discussed in https://github.com/clangen/musikcube/discussions/587

Originally posted by Argonlifesee March 12, 2023
With the update to the libvacodec.so.60 Musikcube does not find libavcodec.so.59, which makes perfect sense as it has been replaced with the newer version, so my question is there away to get musikcube to use libavcodec.so.60. I downloaded it from the Aur and am on version 0.99.5

Yeah, this is annoying. You'll either need to (1) reinstall musikcube from AUR to force a re-link against the new library names, or (2) install musikcube-bin that will use pre-compiled/pre-bundled versions of the libraries.


Document uninstalling flutter sdk for macOS

I want to remove flutter, but there is no document on the internet.

just rm -rf the sdk floder,

A request for a flutter uninstall is being tracked with https://github.com/flutter/flutter/issues/15202, but it's not split out by platform (maybe it should be).
We can call this a request for documentation for uninstalling on macOS, though.  I don't see anything about uninstalling in any official documentation.  This is the closest thing I see:
https://github.com/flutter/flutter/wiki/Workarounds-for-common-issues#flutter-installation-corrupted

Windows request here: https://github.com/flutter/flutter/issues/29278

@jmagman should this issue go in the wiki or the flutter site?

The wiki is generally documentation for Flutter contributors (like running Flutter's tests or compiling the engine) and flutter.dev is for developers who use Flutter to make apps.  So this should go on the website https://github.com/flutter/website

@jmagman Thanks, I'll reopen an issue there about this. Also, I think we can provide a simple script for windows and Unix/Linux within the flutter SDK that will clean all folders, I'll open a separate issue on it and tag you.

@xietx1995 you can track this progress on this PR


Re-enable integration tests for clients.

I broke these when I added the local worker.

I've restored all the tests that were deleted. I restored them not as integration tests though. In addition I did add some integration tests of the client library.
I have yet to add any integration tests of cargo-maelstrom though. I think there should be a different issue for that

okay I added some cargo-maelstrom integration tests also actually


[type: performance] Implement zero-copy in TCP plugin

Make sure that:

[x] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Before

After

Codecov Report

Merging #5066 (9fe4615) into master (2088df1) will decrease coverage by 0.31%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #5066      +/-   ##
============================================
- Coverage     63.31%   63.00%   -0.31%
+ Complexity     8549     8500      -49
============================================
  Files          1221     1221
  Lines         36367    36353      -14
  Branches       3328     3324       -4
============================================
- Hits          23025    22905     -120
- Misses        11454    11561     +107
+ Partials       1888     1887       -1

see 58 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

NB


form.rst: fix reST code samples

Per https://docs.python.org/3.0/documenting/rest.html, any inline formatting such as bold or code must be bounded by non-word characters on each side.  As a result, part of the sentence was getting formatted as code. This fixes that using the suggested workaround of putting an escaped space after the code sample so pop()\ ping looks correct.
It also adds in the extra consonant so the English flows better (popping vs. poping).

Thanks!


⚠ GitHub has degraded performance

In b23c285, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 384 ms

Resolved: GitHub performance has improved in 6d2e8de.


Detect click on notification

I'd like a way to detect a click on the notification to differentiate it from a typical resume. I don't need data because the UI is running so I have all the data in the notification, just a way to know the notification was clicked. Am I missing something easy? Thanks

@dhjw You will get notified when the app moves to foreground. Should be the right event for you or?

I made do with that. Thanks


Add more Go benchmarks

[x] Cat files (local) go0 - Go
[ ] Cat files (webrtc, mplex, secio) go01234->go5 - - Go

Added a few go benchmarks but had to remove them due to https://github.com/ipfs/benchmarks/issues/241

The tests written but removed from the runner ( dashboard) due to the issue above

go


Action log

Action log implemented. Please review and merge.
@mkasztelnik, @tbartynski

And please remove all Hound comments after commiting the revised code :). It makes it a lot easier to see what happens here.

@bwilk today you mentioned that all issues reported during code review were fixed. I don't see any changes - did you push latest version into github?

Tested on devel - no problems. For sure we will need to add more detailed information into action log for initial optimization and scaling, but introduced mechanism works for me -> :+1:

Looks okey.

To me everything seems fixed. Please rise your hands (1 more needed) to let mi finish this job @paoolo @nowakowski @tbartynski

@paoolo, @nowakowski, @tbartynski can you review these changes and can we merge it and close it today?

For me looks good, but I guess that something is hidding if so many conversation was made in this PR which has only 0.5k lines changed and most in test. So I would like to say :+1: if this PR will only add action log support and has no impact to other things.

:+1:


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2023-12-14T02:59:53+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot_istio_release-1.20
7b0162bad096b8d41ce8fb9f18e6c24efc1efdd6
link
true
/test integ-pilot

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot


Upgrade wildfly Chart to 1.4.0

This add supports for WildFly 24.0 release
Signed-off-by: Jeff Mesnil<EMAIL_ADDRESS>

mmhm afaict, the failure in the GH action is not related to the PR commit but to the setup of the action...

@pedjak Hi, is there anything holding up that upgrade? As far as I can tell, the GitHub actions is not related to the chart upgrade.

thanks!


Fixing broken installation hyperlink

Thanks for contributing to exp! exp has been replaced by Expo CLI. We welcome contributions in the new Expo CLI repository.
Contributing to Expo CLI


Clarify what Astro is in the README

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description

What does this PR change? Give us a brief description.

Changes the description of what Astro is in the README, because I don't think "site builder" is accurate (makes me think of wix, squarespace, weebly, etc). I swapped it out with the description from the main repo, but I'm not certain that's the best option. Happy to hear other suggestions too :) And if you think "site builder" is fine I'm happy to close the PR as well.

Love it, @MarcusOtter! As a maintainer, you have the power to merge your own PRs. In Docs, our policy is to let maintainers merge their own PRs after another set of eyes (typically another maintainer unless it's just for proofreading purposes) has reviewed and explicitly said "LGTM."
So, LGTM!  🥳  Merge at will!


cannot create instance

I am new top PWK. I logged in and opened my first session. I clicked a button to create new instance. There was a message "creating instance" for few seconds. After short time, message was hidden but I cannot see any new instance, no error or other message. I can press button to create new instance and repeat a try to start new instance...
What is the problem? Is your lab cluster busy and out of free space to launch another instance? Or is there other problem?  Please, report to user why instance was not created.

try again, it should work better now.

It is not better. I  press "CREATE NEW INSTANCE", text changes to "CREATING..." but after two seconds it is replaced with "CREATE NEW INSTANCE". I cannot see any instance. I don't see any error.

try closing your session and creating a new one.

I tried. I use Chromium browser running at Ubuntu 18.04. I tried with Firefox at Ubuntu. No difference. Well, there is a difference, now I see that "creating..." state is shorter, just about 1 second. It was 2 seconds before and, maybe, 5 seconds when I tried my first try.
Just to be sure, this is my procedure. I go to https://labs.play-with-k8s.com/, I login with my GITHUB account, than I start with START button. Than new session with timeout 4 hour is started, I see countdown timer counting down and I can press button "CREATE NEW INSTANCE".

sorry about that. It should work now.

I had the same issue. In my first session, I tried "Add New Instance" multiple times. It never worked. I closed the session and started a new session. In the second session, "Add New Instance" worked on the first attempt.


Can't use async generators in javascript on mac Os 10.13.6

Describe the bug
Running on MacOs 10.13.6, the javascript engine within Tauri does not support async generators.
Safari, on the same machine does.
To Reproduce
Steps to reproduce the behavior:

On an updated MacOs 10.13.6, create a blank Tauri app
Go to the console
Type var f = async function* () {}
See error `Unexpected token '*'. Expected an opening '(' before a async function's parameter list.

Platform and Versions (please complete the following information):

[tauri]: running info
Operating System - Darwin(17.7.0) - darwin/x64
Node.js environment
Node.js - 15.2.0
tauri.js - 0.13.0
Rust environment
rustc - 1.47.0
cargo - 1.47.0
tauri-bundler - 0.9.4
Global packages
NPM - 7.0.8
yarn - 1.22.10
App directory structure
/.git
/.idea
/build
/node_modules
/public
/src
/src-tauri
App
tauri.rs - 0.10.0
mode - no-server
build-type - bundle
CSP - default-src blob: data: filesystem: ws: http: https: 'unsafe-eval' 'unsafe-inline'
distDir - ../build
devPath - http://localhost:3000

I think that this will be resolved with the next release. If it doesn't solve this problem for you, then another approach you can use is to actually transpile your source to CJS or something along those lines.


[docs] OffCanvas autoFocus

In docs it says that autoFocus / data-trap-focus is set to false by default but it appears it is actually set to true - I had to manually disable it when instantiating the class to make sure it doesn't focus on the first item in the container:
new window.Foundation.OffCanvas(element, {
    autoFocus: false
});

I've checked foundation.offcanvas.js and on line 519 found:
/**
   * Force focus to the offcanvas on open. If true, will focus the opening trigger on close.
   * @option
   * @type {boolean}
   * @default true
   */
  autoFocus: true,

Hey @sebastiansulinski
Had a look at the docs and code base of 6.4.1 this morning and 'data-auto-focus' and 'data-trap-focus' are not the same.
[data-auto-focus] merely forces the focus to the off-canvas on open.
[data-trap-focus] triggers optional focus trapping when opening an off-canvas, automatically setting tab index of [data-off-canvas-content] to -1 for accessibility purposes.
The docs & code base in this respect are aligned with one another.

Oh I see - you're right. Thanks @codetheorist.

No problem @sebastiansulinski, happy to help.


[IMPROVE]List of all the available names that can be used in a custom ratio

What's the feature that should be improved?
All the valid names that can be used in a custom ratio.
Describe how you would like the feature improved
Include a list of all the valid names that can be used in a custom ratio.
Possibly describe the ideal way to improve this
Using pandas should be easy to get the column names.
Additional information
At this point in the code you already have all the data available.
https://github.com/JerBouma/FinanceToolkit/blob/82550930d55bd0f63611e7750201735edaaab692/financetoolkit/ratios/ratios_controller.py#L299

Are you feeding the custom ratios into the Toolkit initialization or directly in the custom ratios function? I can indeed quite easily add it in for the latter!

Will do a release early this year with this functionality, already have it available on the develop branch.

That's exactly what I was excepting! Thank you!

Fixed in v1.7.3 👍
https://github.com/JerBouma/FinanceToolkit/releases/tag/v1.7.3


multi deposit address

I'm trying to generate deposit address for each account of my exchange website users by using COINEX API I've faced a real problem imagine there is 2 users in my website and USER1 wants to deposit BTC so when he clicks on generate deposit address code runs : PUT /balance/deposit/address/BTC and address displays to him. but he hasn't used this address yet suddenly USER2 wants to deposit BTC too and then code runs again: PUT /balance/deposit/address/BTC but it says { code: 318, data: { }, message: "Deposit address not used." } should USER2 wait untill USER1 completes his deposit?! it can't be like that.. any idea for solving this? how everyone make an exchange with coinex api, it doesn't make a trouble for them?

Hello, I have never used coinex multiple deposit addresses. I'm not very clear about this, as you said there shouldn't be that kind of situation. I suggest you ask an official technician


Scripts for running Atlas in different AirTag experiments

The paper has used Atlas for running multiple experiments to set up comparisons with AirTag. Specifically, Atlas was used for:

Running UDatasets
Running Depimpact datasets
Measuring ATLAS’ performance based on different starting points for attack investigation

However, there are parts of the Atlas pipeline where manual intervention is required:

After running atlas.py, redundant attack entities need to be removed and related attack entities need to be added.
A “clean” list of attack entities needs to be added to the eval_** json file

Could you please provide us with the scripts you used for running Atlas that would outline any parts where such user input is required so we can match the paper experiment? Additionally, could you also provide us with the ground truth files compatible with ATLAS’ detection granularity for UDatasets and Depimpact datasets?

These manual parts are used for graph-level evaluation to get the entities, we evaluate on the log level by use the malicious entities to match the logs. Ground truths are provided in the repo.

I could not find the ground truth logs for the UDatasets and DEPIMPACT datasets in the repository, could you guide as to where these are?
They are not present in the ground_truth tarball shared in the Google Drive.


chore: Upgrade to Go 1.17

This PR

upgrades all Go based services to Go 1.17
upgrades all pipelines to use Go 1.17

Integration test run: https://github.com/keptn/keptn/actions/runs/1956596889
(this run already includes the go 1.17 versions of go-utils and kubernetes-utils to test the full picture)

looks good! can you please resolve the conflicts and let the unit tests run to see if everything works as expected ?


[BUG] Resources page broken after last update

Hi,
first of all thank you for all the effort and work going into this. Having said that, it seems like the last update broke the resources page. Since it’s also happening on your own league page it’s an issue in general.

There was a security issue with the previous xml parser. I had to update it in the last update. I think that may have unintentionally broken the news API. I'll try to look into it this week. Thanks for reporting!


fix(manager-core): add missing peerDeps ngOvhSwimmingPoll

Add missing peerDeps
Since we upgraded to<EMAIL_ADDRESS>we have to add "@ovh-ux/ng-ovh-swimming-poll" as a peerDependency

The commit message type has been set to a fix in order to be able to target a new patch for the given module @ovh-ux/manager-core.
Let's assume that you want to start the @ovh-ux/manager-sms.
Actually you get this following error from the console:
Error: [$injector:modulerr] Failed to instantiate module smsApp due to:
[$injector:modulerr] Failed to instantiate module ovhManagerSms due to:
[$injector:modulerr] Failed to instantiate module oc.lazyLoad due to:
[$injector:nomod] Module 'ngOvhSwimmingPoll' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.


Lock/Unlock door sound

Changed FMOD dependency from CSC8503 to CSC8503CoreClasses, also move SoundObject from CSC8503 to CSC8503CoreClasses. GameObject contains SoundObject so SoundObject should be better in CoreClasses, and SoundObject needs FMOD, so I moved FMOD dependency.
Added lock and unlock door sound

edited sounds, cut all the gaps of sounds so the sound played immediately, also increased the volume of footstep sound


Improve multi-cluster unit test coverage for multicluster/controllers

This is a sub task of https://github.com/antrea-io/antrea/issues/4345.
Our final goal of Antrea unit test coverage is 70%. We need improve multicluster unit test since there are still a gap.
Multi-cluster's unit test coverage can be found here https://app.codecov.io/gh/antrea-io/antrea/tree/codecov-ut/multicluster
We can improve the unit test from below two major parts.

This is resolved by #4377


Allow users to upload profile image

I think you need to map a volume for the image on disk storage.


Allow a user to put a single task into "maintenance mode"

We've run into several problem scenarios:

We want to run a Java heap dump on a specific instance of a task - doing so will cause the health check to become unhealthy. We want marathon to keep the task running even though it is unhealthy.
A single instance of an application has a problem and an operator wants to keep it alive for a developer to review

Both of these scenarios could benefit from a feature that allows an operator to put a single task or group of tasks into "maintenance" - here's how I imagine it would function:

When a task is put into maintenance, a new task is spawned to replace it.
New healthchecks from this task are ignored
The task stays alive until the task is either killed by a user or the task dies.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-1765. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


optimize mem use for cpu-sharemem and gpu.

include chol_packed as share arrays
optimize gpu memory usage, tested locally to find 15+GB mem reduced for a 5GB chol file

That test failure isn't your problem, so this is safe to merge.


No module named 'modules.textual_inversion.textual_inversion'

I don't know what the problem is, I seem to have reinstalled everything, the server is not being created.
a possible reason is that I share a folder from my main account to another one, where I have more space and the neural network itself is located
But then I didn't fully understand how to share a Google drive
I also clicked on installing several add-ons at some point, and it looks like they somehow conflicted
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 31, in
from modules import extra_networks, ui_extra_networks_checkpoints
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/ui_extra_networks_checkpoints.py", line 5, in
from modules import shared, ui_extra_networks, sd_models
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/ui_extra_networks.py", line 8, in
from modules.images import read_info_from_image
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/images.py", line 21, in
from modules import sd_samplers, shared, script_callbacks, errors
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/sd_samplers.py", line 1, in
from modules import sd_samplers_compvis, sd_samplers_kdiffusion, shared
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/sd_samplers_compvis.py", line 9, in
from modules import sd_samplers_common, prompt_parser, shared
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/sd_samplers_common.py", line 5, in
from modules import devices, processing, images, sd_vae_approx
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/processing.py", line 15, in
import modules.sd_hijack
File "/content/gdrive/.shortcut-targets-by-id/15sLV1KrOBvYzBm3Hi5zZf5M8I60szdJ-/sdапв/stable-diffusion-webui/modules/sd_hijack.py", line 5, in
import modules.textual_inversion.textual_inversion
ModuleNotFoundError: No module named 'modules.textual_inversion.textual_inversion'

I also clicked on installing several add-ons at some point, and it looks like they somehow conflicted

I am also interested in the possibility of using Google drive as a place to store a neural network, and using the local colab server to use its power for calculations.Since I caught the limits of colab.But when I connect from a local server, notepad does not see google drive.I would be very grateful if you could tell me how to solve this problem.Will the Internet connection greatly affect the generation time?

you can't use a shared folder, use the default folder /content/gdrive/MyDrive/sd/stable-diffusion-webui

same problem


Shapes (64,) and (1,) are incompatible

Dear author, the problem occurs when I run evaluate.py, the problem occurs in
model.load_weights(f'{weightsPath}')
throw exception display
Shapes (64,) and (1,) are incompatible
The weight information of the model has been obtained in the previous training, but the problem was found during the evaluation, so I would like to ask you for advice, thank you very much.

the error doesn't show when i run evaluate.py on the lab


Parse model before validating on model.set(pojo, {parse:true})

If I have a Backbone model that expects a wrapped object from the API e.g.
{a: {a: 3}}
But I want to work with it as
{a:3}
It is nice to use parse like
parse: function(data) {
  return data.a;
}

And validate like
validate: function(attrs) {
  if(!_.has(attrs, 'a') || !_.isNumber(attrs.a)) {
    return 'incorrect signature';
  }
}

And when I mock out the model in a test I do:
model.set({a: {a:3}}, {parse: true})

But this fails because model.set calls validate before parsing my object. This doesn't make a whole lot of sense to me. If I say {parse: true} it should parse before validating. My workaround is
var o = {a: {a:3}};
model.set(model.parse(o));

But I feel like it would be less confusing to have {parse: true} actually work as expected when you have a validate function.

Model#set doesn't have a parse option. I think you're thinking of Colleciton#set which (sadly) does, but that'll be gone in v2.


Little confused…

So uh I tried installing it and everything but the game won’t start and it’s just a black screen. I downloaded the apk, created a folder in my storage called omori and put it in there with the GitHub files and the www folder from my omori game on steam. I also tried installing one loader with it, but it didn’t detect it, so I tried putting the one loader files from the www folder it had, into the game directory instead of merging it with the www folder from my steam game. When I tried this it said something about a catastrophic error… pls help I’m so confused and I feel stupid

Please send your debug log ("Save debug log" in main menu) and show the content of your selected installation directory

omori.log

I can't open this log file for some reaaon, maybe it's too large for github. Try to clear your log, then launch the game once and send your log again

omori.log

Still can't open...🤔 Upload it to pastebin, I think it would work

Idk how to...  Try again with disabled oneloader in menu

Just a black screen with the buttons on screen… I guess I just have to accept that it won’t work on my phone :(

Clear logs, start the game and send logs again, it should be different now

Closed due to inactivity


CCM - Update the SMP doc

Preview build: https://651169650c4ca406562d6cd7--harness-developer.netlify.app/docs/cloud-cost-management/get-started/ccm-smp/aws-smp#air-gapped-environment

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Prevent non-negative time on pedestrian simulation

Scales emmision rates with simulation time scale

Should solve #94

Should solve #94

@coolmule0 Why have the emission rates scaled. We need to have a chat about this before it is merged.


Consolidate all outputs into one out folder

Problem
Currently artifacts compiled with Nargo are "scattered" into multiple folders (i.e. contract, target), which might be slightly confusing to some.
Happy Case
Consolidate all artifacts into one e.g. out folder instead. Naming suggestions welcome.
Alternatives Considered
No response
Additional Context
Suggestion was collected from Noir User Testing Session 1.
Would you like to submit a PR for this Issue?
No
Support Needs
No response

@TomAFrench thoughts?

I think this is bad because you often want to check in some artifacts and not others. For example, Dark Forest checked in the smart contracts but not the PK or VK.

I think this is bad because you often want to check in some artifacts and not others. For example, Dark Forest checked in the smart contracts but not the PK or VK.

Closing as "not planned" then


feat: export roomInfo and users

export room's Info and users(include phone number)

I'll be happy to finish it


Add Signal messening to Ferdi. A Web application of Signal is already available.

Hello, I submitted a request to ask you to add Signal messenging to Ferdi.
I got a reply, that Signal has no web application. So, the request is closed.
But, I'd like you to reconsider your position. Indeed, Signal messenging has already developped a web application which works perfectly, on Windows, Mac and Linux. I am using it currently on my PC.
You can check the following URL, wich give the possibility to download the corresponding web application regarding the OS used:
https://signal.org/fr/download/windows/
Hope that, your  sympathic team will add Signal to Ferdi.
Thank you

Duplicate of #1155.
Please don't open any more new issues on this. If you want to discuss further, do so under #1155, but keep in mind that the we won't be able to distribute any Signal client unless the Signal Foundation changes its policy on third-party clients.


Redo chat parsing and validation

Components are rendered to DOM elements (eliminates hand-rolled escaping)
Structures are validated up-front instead of throughout the code (including depth checks)
I also fixed some incorrect types from chat.mjs where I thought we stored parsed messages

@creesch the structure from prismarine-chat is out-of-date. The value field is deprecated. Perhaps only used by old Minecraft versions?

@creesch the structure from prismarine-chat is out-of-date. The value field is deprecated. Perhaps only used by old Minecraft versions?

That is entirely possible as they support multiple version of minecraft, if we are sure it is no longer used by newer versions of minecraft we can leave that field out I am not planning on figuring out support for older versions of minecraft.
I'll have a proper look a bit later, on the face of it it seems fairly solid.  Some things that do jump out:

Are we sure that for titles it is enough to rely on the browser to sanitize what we put in there? I feel like I have seen some stuff about attribute injection in the past.
One potential bug in initial json handling.
Error handling feels like it might be too strict in some cases but I want to test it first before I comment on that.

Little bit of work todo

{"translate":"chat.type.text","with":[{"text":"creesch","hoverEvent":{"contents":{"type":"minecraft:player","id":[2097578677,-1517796232,-2140994661,-1953866359],"name":"creesch"},"action":"show_entity"},"insertion":"creesch","clickEvent":{"action":"suggest_command","value":"/tell creesch "}},"test"]}
chat.mjs:79 Invalid component: Component is not an object at .with.1
displayMessage @ chat.mjs:79
chat.mjs:68 {"translate":"chat.type.text","with":[{"text":"creesch","hoverEvent":{"contents":{"type":"minecraft:player","id":[2097578677,-1517796232,-2140994661,-1953866359],"name":"creesch"},"action":"show_entity"},"insertion":"creesch","clickEvent":{"action":"suggest_command","value":"/tell creesch "}},"cool"]}
chat.mjs:79 Invalid component: Component is not an object at .with.1
displayMessage @ chat.mjs:79
chat.mjs:68 {"translate":"chat.type.text","with":[{"text":"creesch","hoverEvent":{"contents":{"type":"minecraft:player","id":[2097578677,-1517796232,-2140994661,-1953866359],"name":"creesch"},"action":"show_entity"},"insertion":"creesch","clickEvent":{"action":"suggest_command","value":"/tell creesch "}},"works"]}
chat.mjs:79 Invalid component: Component is not an object at .with.1
displayMessage @ chat.mjs:79
chat.mjs:68 {"translate":"chat.type.text","with":[{"text":"creesch","hoverEvent":{"contents":{"type":"minecraft:player","id":[2097578677,-1517796232,-2140994661,-1953866359],"name":"creesch"},"action":"show_entity"},"insertion":"creesch","clickEvent":{"action":"suggest_command","value":"/tell creesch "}},"like it should"]}
chat.mjs:79 Invalid component: Component is not an object at .with.1
displayMessage @ chat.mjs:79
chat.mjs:68 {"translate":"chat.type.text","with":[{"text":"creesch","hoverEvent":{"contents":{"type":"minecraft:player","id":[2097578677,-1517796232,-2140994661,-1953866359],"name":"creesch"},"action":"show_entity"},"insertion":"creesch","clickEvent":{"action":"suggest_command","value":"/tell creesch "}},"awesome! "]}
chat.mjs:79 Invalid component: Component is not an object at .with.1
displayMessage @ chat.mjs:79
chat.mjs:107 Connected to server
chat.mjs:68 {"translate":"chat.type.text","with":[{"text":"creesch","hoverEvent":{"contents":{"type":"minecraft:player","id":[2097578677,-1517796232,-2140994661,-1953866359],"name":"creesch"},"action":"show_entity"},"insertion":"creesch","clickEvent":{"action":"suggest_command","value":"/tell creesch "}},"does this come through?"]}
chat.mjs:79 Invalid component: Component is not an object at .with.1

Also going to be a bit real, I like code to be actually tested before a PR is made. Or have the PR marked as a draft if you haven't been able to test it.

That's fair. I did test the PR. I did not see any errors when I tested. I do not have a good understanding of what cases to run through.

That's fair. I did test the PR. I did not see any errors when I tested

Interesting, I just realized that this only happens in a game I am running locally.  I did open it up to LAN so it acts as a server and the same still happens there.

I do not have a good understanding of what cases to run through.

At the very least the messages from above ;)
I have also been using https://mcstacker.net/1.21.php  to create /tellraw commands to test various formatting.

So it looks like that in Minecraft's chat format, the elements in the with array can actually be a string.  Same as you handle extra

Are we sure that for titles it is enough to rely on the browser to sanitize what we put in there? I feel like I have seen some stuff about attribute injection in the past.

I think that's an issue if:

We are doing string interpolation - at which point an attacker could terminate the attribute's string and inject a <script> tag.
We put raw HTML into the title attribute and read it back later. We might do this at some point for more rich hover information, but that's currently not the case.

Alright, did some testing and seems to work as expected now.


Carthage build not working

carthage build
*** xcodebuild output can be found in /var/folders/6h/nc4rf_dj3bdd9_0yz7vgtwb00000gn/T/carthage-xcodebuild.hAIujh.log
*** Building scheme "Dronecode-SDK-Swift" in Dronecode-SDK-Swift.xcodeproj
Build Failed
        Task failed with exit code 65:
        /usr/bin/xcrun xcodebuild -project /Users/julianoes/src/DroneCore-Swift/SampleCode/SwiftSampleCode/Carthage/Checkouts/DroneCore-Swift/Source/Dronecode-SDK-Swift/Dronecode-SDK-Swift.xcodeproj -scheme Dronecode-SDK-Swift -configuration Release -derivedDataPath /Users/julianoes/Library/Caches/org.carthage.CarthageKit/DerivedData/9.3_9E145/DroneCore-Swift/ed3cd194a2c71ad62a4147010eb572c3823e311a -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive -archivePath /var/folders/6h/nc4rf_dj3bdd9_0yz7vgtwb00000gn/T/DroneCore-Swift SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /Users/julianoes/src/DroneCore-Swift/SampleCode/SwiftSampleCode/Carthage/Checkouts/DroneCore-Swift)

The log ends with:
/Users/julianoes/src/DroneCore-Swift/SampleCode/SwiftSampleCode/Carthage/Checkouts/DroneCore-Swift/Source/Dronecode-SDK-Swift/Dronecode-SDK-Swift/Generated/telemetry.grpc.swift:25:8: error: no such module 'gRPC'
import gRPC
^

** ARCHIVE FAILED **

The following build commands failed:
        CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
        CompileSwift normal arm64
(2 failures)

Closing in favor of #47.


travis: run ci on all branches

Was playing with a few items for rust-embedded-bot support
but this item is not necessary and prevents travis from
running tests on branches other than 'auto'.
CC @nastevens

@rust-embedded-bot r+

@rust-embedded-bot r+


Archive outdated assets

Pull Request
Related issue
Partially fixes #503
What does this PR do?
Archives outdated assets (list of mentions, love displays from users, companies & orgs that used Meilisearch and old logos)
PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

I never used these assets so I trust you on this one :p

Haha, we still had the old M&Ms logos 😂


Restructure the contents of the readme

In order to make the useful parts of the readme (important information and code examples) easier to find I've restructured the document. The content is the same, but the order is different. Also changed the heading levels for some of the sections.

This change is

@kbrabrand Thanks for the offer to help. For now, I'm going to keep our contact info at the top, as ShakaCode depends on community support to keep investing in these projects.
We very much appreciate your involvement and welcome your contributions, such as other useful enhancements or help with other docs and PR reviews.


question: How can I use other models in llamaedge?

Summary
I see that the models are downloaded from "https://huggingface.co/second-state". Is it possible for me to modify run-llm.sh script and run another model?
PS: I am trying to run 'starcoder2-7b-Q5_K_M.gguf' As it is recommended in the list.
Appendix
No response

Yes. If the model is in the gguf format, it should be supported. If not, please raise an issue.

I was trying the StarCoder2 series of models. I see in the description that there is no "Prompt Type" for them.

Also, trying to run a listed models in the bash script, I am getting this output
repos/llamaedge took 2m57s
❯ bash run-llm.sh --model llama-3-8b-instruct

[+] Using cached model Meta-Llama-3-8B-Instruct-Q5_K_M.gguf

repos/llamaedge
❯ ls -alh | grep Meta-Llama-3-8B-Instruct-Q5_K_M.gguf
-rwxrwxrwx 1 User User 5.4G May 23 16:14 Meta-Llama-3-8B-Instruct-Q5_K_M.gguf

You can use the --interactive flag to confirm all steps.
https://llamaedge.com/docs/user-guide/quick-start-command#interactively-choose-and-confirm-all-steps
But, the run-llm script is designed to get started quickly. In order to make full use of the tools, I would suggest to go through the entire process by hand. It only takes 10 minutes most:
https://llamaedge.com/docs/user-guide/get-started-with-llamaedge

Hi @aeroaks,
Just want to follow up if your question is solved.


common/gpu/nvidia/default.nix: the PR was merged for modesettings

The NVIDIA modesetting is now enabled by default for driver versions 535 and above this option is no longer needed.
Description of changes
Removes hardware.nvidia.modesetting.enable = lib.mkDefault true; as the PR was merged and it is no longer needed to be set by nixos-hardware.
Things done

[ ] Tested the changes in your own NixOS Configuration
[ ] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

is that default backported?

Oh I completely forgot to check no it is not, ignore this sorry.


refining html redirect configuration

Here's what DILA will currently want to do (maybe that will change in the future):

/resource/A000001 redirect to http://authority.dila.edu.tw/person/?fromInner=A000001
/resource/PL000000060269 redirect to http://authority.dila.edu.tw/place/?fromInner=PL000000060269

this is currently not possible with our system for two reasons:

the redirection doesn't contain the prefixed version (dila:A000001)
the redirection depends on the first letter of the local name of the resource (A -> person, P -> place)

Here are some ideas to fix that:
First we should differentiate the redirect from /resource/{res} and /admindata/{res} so that we don't need to attach the prefix, we would for instance have for our config:
endpoint.resource.htmlredirect=https://library.bdrc.io/show/bdr:
endpoint.admindata.htmlredirect=https://library.bdrc.io/show/bda:

that fixes the first issue. The second problem is a bit more difficult to tackle, we need to specify in the configuration a default html and some exceptions based on either regexp or just the first letter of the local name of the resource. So DILA could use something like:
endpoint.resource.htmlredirect=http://authority.dila.edu.tw/person/?fromInner=
endpoint.resource.htmlredirect.exceptions=exceptionP
endpoint.resource.htmlredirect.exceptions.exceptionP.localNameFormat=P.*
endpoint.resource.htmlredirect.exceptions.exceptionP.redirect=http://authority.dila.edu.tw/place/?fromInner=

this requires a little bit of code to understand the config file but it's not too conceptually difficult... This is quite high priority

for the second issue, why don't we use instead a type parameter in the query string ?
So for /resource/{resId}?type=type_value,
if type not present-> default url
else
URL = http://authority.dila.edu.tw/ +type_value + ?fromInner={resId}
That would avoid obscure configuration, especially if the number of format increase with time (P, A, W, X T, etc...)
WDYT?

Diving into this a little bit deeper, I don't think Ldspdi should implement specifics of any organization using it. I think an organization should adapt to BUDA generic features and not the opposite. If we don't follow that pattern, then we will end up updating our software on a regular basis according to the variation of many organizations use cases.
In other words, in that very case, the logic that parse an organization resourceID and decides which endpoint the request should be redirected to shouldn't be part of ldspdi implementation.
DILA should have a  http://authority.dila.edu.tw/redirect/{resourceId} endpoint (for instance) that takes the resourceId and redirect the request to the right endpoint.
Ldspdi should therefore offer the generic possibility of having a redirect URL (as it is for us) pointing to a service implemented by the organization (for instance, since DILA implements http://authority.dila.edu.tw/person/?fromInner=A000001 the DILA can implements the logic mentioned above in a "redirect endpoint").
As a matter of fact, we could even think that requiring the resource type at a /resource endpoint is somewhat a data model design flaw (i.e we should be able to access any resource with its unique IRI without mentioning its type)
I am going to implement your proposal since this feature is needed rapidly, but it's worth thinking about generic vs. specific features implementation in BUDA in general.

Sigh... fine, I'll do it when I'll come back

Well, it's not really straightforward, since we also have to modify the QueryProcessor that takes a prefixed resource...

I don't understand, what do you mean?

In order to do a sparql request, you must have the full URI or at least the prefixed URI. So what do we do when /resource/A000001 is called with Accept:text/turtle, what do we do ?
Not mentionning that this QueryProcessor.getCoreResourceGraph method is used elsewhere in other endpoints.

Will the prefix (for ANY resource) be always the same ? (if so, then it is already set in properties we just created:
endpoints.resource.shortprefix = bdr:
endpoints.resource.fullprefix = http://purl.bdrc.io/resource/

In that case there's no need to use:
endpoint.resource.htmlredirect=https://library.bdrc.io/show/bdr:
endpoint.admindata.htmlredirect=https://library.bdrc.io/show/bda:

as we already set this recently in properties:
endpoints.admindata.shortprefix = bda:
endpoints.admindata.fullprefix = http://purl.bdrc.io/admindata/

so at that point, without changing anything we can differentiate the redirect from /resource/{res} and /admindata/{res} and generate showUrl+prefix in these endpoints implementation.
At the same time, prefixed resources are still valid for the query processor in case we don't have a redirection (i.e when Accept is not text/html).
Now, we still have to create the "typed" redirect url :
3 ways at first sight:

via properties (as you described it)
via a type param in the query string (along with optional and existing "graph" and "describe" parameters (as I suggested it)
via temporary code inside PublicDataResource (as below)

private String getDilaResourceType(String res) {
        String type = "";
        if (res.startsWith("A")) {
            return "person";
        }
        if (res.startsWith("P")) {
            return "place";
        }
        return type;
    }

Conclusion: if the prefix for the /resource endpoint is always the same (as it should be defined in  endpoints.resource.shortprefix property), then the implementation (let's call it a customization) is possible without changing anything to the properties file, using one of the 3 ways of handling the resource type.

I don't really understand what's going on... what I need is a system where an ID which local name is starting with A is redirected in html to
http://authority.dila.edu.tw/person/?fromInner=AXXX
And an ID which local name is starting with starting with P is redirected in html to
http://authority.dila.edu.tw/place/?fromInner=PXXX
There's no need to change the title, there's no notion of prefixes, etc.

This has been done (see commit 930094d), but:

it's not like you think it is because the redirection is a tiny little part of what the endpoint actually does.
this endpoint (and the whole system) must work for both DILA and BDRC requirements using a single endpoint and config structure. I am sure that it is fully functional for BDRC settings and dataset while I doubt seriously it is with DILA settings and dataset.

I think it will be more efficient to actually talk a about this.

Done. Should we close this ?


Fixing dockerfile

FROM alpine:latest breaks the build. Based on current dependencies alpine 3.4 is used.
Also added some prerequisites for when alpine:latest will be used. There are still some c dependencies failing on alpine:latest but until then this should do trick.
Thanks.
@constantincristian @cristianconstantin please find the fix attached I also tested that this works on building

@atrifan why do we need autoconf and automake to remain on the docker image ?

@atrifan thanks ! 👍


Give ability for users to use their own models

Instead of using the OpenAPI, for example, a user could use the Stanford Alpaca Models training weights, if they were to obtain these.

Will support this eventually. Keeping my eye on this, but not ready for it yet (someone else could potentially work on it sooner if interested).

This is how i got it working: https://github.com/mkellerman/gpt4all-ui


remove restriction of nim version < 0.15

IMHO better to break and then be able to fix than not to be able to install at all in latest nim

(Sorry for the delay. I am back from holidays)
I had to think about this. I tend to agree at this point. If you use the code from master, then yes encountering breakage with upgrades is par for the course. Fixes can be made quickly anyway so it is fine. The trickier part is for the releases.
Releases should be more stable; rubber-stamping them as valid for a given language version is a plus for the user I think. At the same time though, it is a minus if that means you can't use them or they keep you from upgrading to the latest language version. Given that Nim is still pre 1.0, at the end of the day I don't want to promote the stableness of any prior version. You should be upgrading language version.
So :+1:


Move CONFIG_SERVICE_PATH into ConfigConsts

Just a simple variable move. It makes more sense in ConfigConsts than ConfigUtils.

merged into develop


use of assembly aliases in auto-generated jbrowse2 configs is problematic for some purposes

I just ran into some issues with text searching that seem to be caused by inadequate understanding by some parts of the jbrowse ecosystem of the use of reference genome aliasing (ie things specified as --alias to add-assembly).
In my case it was both:

the jbrowse text-index command did not recognize tracks were associated with reference genomes when they use the alias (ie the indexing of tracks would simply be skipped, unless you specified the alias explicitly as jbrowse text-index -a arahy.Tifrunner.gnm1
the jbrowse client would completely ignore indexes generated in this way

I think @ctcncgr has also seen some other contexts in which similar problems arise if you try to use the alias for a genome rather than the given "name". So, this may well be a jbrowse2 bug but for now I think it would be better for the config generation to use the forms like "arahy.Tifrunner.gnm1" as the name and use --displayName to set the "human readable" version.
This looks like a simple change in way the jbrowse cli is called, I can PR it but this provides the context for why I'm proposing the change.

Closing. Accepted PR #15
I do believe this to be a bug in jbrowse2, but maybe it isn't. Either way this is a good solution. Thanks @adf-ncgr


Packed Sequence Vision Transformer (aka NaViT)

A big WIP, pushing early to resolve masking stability issues with F.sdpa

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

is this still in progress ?

Hello @rwightman !
Any updates regarding this PR? Is there anything I can help with?

@Adenialzz @b5y sorry for delay, been trying to get some other things out the door.
So, where I got with this, I verified the modelling aspect works. The masking / handling of the packed patches seems fine. If you've looked I implemented a very rudimentary packing that is currently just injected in the forward of the model (it takes standard uniform batches of images, splits and then repacks). This is obviously not the point, but was a quick hack to allow me to test.
For this to work efficiently the packing needs to be integreated into the datapipeline with extra buffering and a better thoughtout packing algorithm (essentially online bin packing). The data augmentations need to be tuned wrt to the dataset image size range such that you end up with a distribution of image sizes and patch lengths that's optimally packable.
I hope to get back to this. The feature is definitely more data pipeline & packing working than modelling...
Right now I'm working on a data loading library oriented towards large document (pdf) and image + text datasets and associated augmentations/preprocessing. I was thinking of moving the packing/pipeline code there once I get the initial version of that public & released...

Hello @rwightman I was wondering if there are any updates for this PR? thank you


Improve CMake II

This is a rebased version of #82. I'd just like to see if there are still problems.
Locally, everything looks fine.

Codecov Report

Merging #164 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #164   +/-   ##
=======================================
  Coverage   89.51%   89.51%
=======================================
  Files          53       53
  Lines        2622     2622
=======================================
  Hits         2347     2347
  Misses        275      275

Impacted Files
Coverage Δ

tests/test_hierarchy_helpers.hpp
65.33% <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 3402a36...8392f2d. Read the comment docs.

This is a rebase of WIP PR. I made it work on my machine and I've hacked docker so that it will work but that's not the real problem.


localhost debugging?

Hi there, I want to debug the vuejs on my localhost (http://<IP_ADDRESS>:8000). It works fine when I open a file:/// path but not a local host. Is there a way to make this work?

You have to check an option in the extensions chrome page (chrome://extensions)


DateTimeParseException

I'm getting the below error message when trying to read DICOM files from the TCIA gs bucket.
java.time.format.DateTimeParseException: Text '122734.625000' could not be parsed, unparsed text found at index 10

Code to reproduce (pyspark):
APOLLO_FILE_PATH = "gs://gcs-public-data--healthcare-tcia-apollo/dicom/"
apollo_df = spark.read.format("dicomFile").option("recursiveFileLookup", "true").load(APOLLO_FILE_PATH)
apollo_df.collect()

we could also catch parse exceptions and report on those in an "parse error" column instead?
I've seen this work well with big data on not so well defined data sources (due to DICOM "semi" standard).


Improve bus write read transactions

Change extends bus driver operations to allow faster transfer.
New driver function write_read allow for faster transaction implementation.
Existing function bus_node_write_read_transact() was using write that read.
It turns out that for short transfers (1 or 2 bytes written) several bytes read it is worth
to implement custom function that writes and read.
This code also adds first implementation of such function for spi_hal driver that can
benefit around 50% time reduction on single transfer.

I do like the idea and I think this is needed for sure. As per above dummy reads/writes are handled by the underlying api which is great.


Getting incompatible image

Hi,
It seems that the generic image for 1903 doesn't work.
When I try to install the image BCsandbox:dk, I get the message "The container operating system does not match the host operating system".
I use the command:
New-BCContainer -containerName 'VER15' -accept_eula -licenseFile 'C:\mylicense.flf' -alwaysPull -imageName 'bcinsider.azurecr.io/bcsandbox:dk' -memoryLimit '4G' -updateHosts -useBestContainerOS -isolation 'process' -includeTestToolkit -includeTestLibrariesOnly
the output I get is:
NavContainerHelper is version <IP_ADDRESS>
Host is Microsoft Windows 10 Enterprise - Unknown/Insider build
Docker Client Version is 19.03.2
Docker Server Version is 19.03.2
Pulling image bcinsider.azurecr.io/bcsandbox:dk-ltsc2019
dk-ltsc2019: Pulling from bcsandbox
... but the after the image has been downloaded I get:
A better Generic Container OS exists for your host (1903)
Creating temp container from bcinsider.azurecr.io/bcsandbox:dk-ltsc2019 and extract necessary files
... and ...
Removing temp container
Using generic image mcr.microsoft.com/dynamicsnav:generic-1903
Pulling image mcr.microsoft.com/dynamicsnav:generic-1903
... and finally:
DockerDo : docker.exe: Error response from daemon: hcsshim::CreateComputeSystem c936868a1c25e0d46dfee00a520a371d7b32d193cb77c58f1e5243ead6f838ae: The container operating system does not match the host operating system.
I have tried to remove the -UseBestContainerOS switch, but I get the same result.
My version of Windows is:
Windows 10 Enterprise
Version 1903
Build 18999.1
Will it be possible to use process isolation with this build?

If you are trying to get the lastest public version you should use
"mcr.microsoft.com/businesscentral/sandbox:dk"
and not the insider-link.

Remove -usebestcontaineros - you are running an unknown windows build - you cannot use process isolation
Hent Outlook til iOShttps://aka.ms/o0ukef

Fra: Palle Arentoft<EMAIL_ADDRESS>Sendt: Thursday, October 10, 2019 11:21:07 AM
Til: microsoft/navcontainerhelper<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Emne: Re: [microsoft/navcontainerhelper] Getting incompatible image (#659)
If you are trying to get the lastest public version you should use
"mcr.microsoft.com/businesscentral/sandbox:dk"
and not the insider-link.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fnavcontainerhelper%2Fissues%2F659%3Femail_source%3Dnotifications%26email_token%3DACSGUA7NTMTQFJXUG7OPHUTQN3XYHA5CNFSM4I7KBXU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA3RI7Q%23issuecomment-540480638&data=02|01|freddy.kristiansen%40microsoft.com|d9bcd30e391e4c27a6df08d74d632ec4|72f988bf86f141af91ab2d7cd011db47|1|0|637062960696936554&sdata=n56RuSrQgpl4Uv7elG2%2FXifjxIKcqIHb723D3%2Bkl9TI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACSGUA6BEYS67BAHWM7JFULQN3XYHANCNFSM4I7KBXUQ&data=02|01|freddy.kristiansen%40microsoft.com|d9bcd30e391e4c27a6df08d74d632ec4|72f988bf86f141af91ab2d7cd011db47|1|0|637062960696946550&sdata=vjCYzLzo2qqd1g3tgYU3mEBwp07T9stQclwh8GgkHnY%3D&reserved=0.

I will fix new-bccontainer so that -usebestcontainer won't select an invalid image.

Actually NavSpecialist is 100% correct - when you specify --isolation Process, then I will not determine the isolation - and you will get what you ask for.
-usebestcontaineros will select the image closest - but will default to hyperv because the OS doesn't match, but then you specify --isolation process and then you get that.
NavSpecialist is also correct as to where you find the images.


Add tests for production builds.

I'd specifically like to add a smoke test using ember s --prod and puppeteer (aka not ember test) that confirms things "work" when an addon / app depends on ember-test-waiters that it does not error.

We still need to add the scenario that uses a prod build + puppeteer (or whatever). Will work on that.


RFC: Output for New Task

Output for New Task

Feature name
Output for New Task

Author(s)
Nicholas Albion<EMAIL_ADDRESS>

RFC PR:
Leave blank

Updated
2024-02-11

Summary
POST /ap/v1/agent/task should allow an output
Motivation
I struggle to understand how people would be using POST /ap/v1/agent/task, which accepts an optional (!!) input but allows for no output. Can you imagine calling somebody on the phone and the person at the other end picks up, but doesn't say anything? You generally want to hear "Hi, thanks for calling XYZ, if you're calling about ..." - or atleast "Hello..."
An agent could:

Ignore (or just log) the task.input and provide a generic welcome/instructional prompt.
Use an LLM to process task.input and select a specialist agent to handle the task/steps, and provide a welcome prompt from the specialist agent
Provide a rejection message explaining why subsequent requests will not be accepted
The client might use the task.output as a title for the task?

Agent Builders Benefit
Agents would be able to provide a welcome message, guiding the user on what to say first.
This allows for greater flexibility & more options:

task.inputtask.ouputcomments

Write the word 'Washington' to a .txt file"
undefined
AutoGPT logs `task.input`.User initiates conversation in `POST /agent/task/id/step`

Write the word 'Washington' to a .txt file"
"Sorry, I'm unable to write to files"
Agent responds to `task.input`maybe sets `"is_last": true`?

undefined
"Hi, welcome to XYZ bank, can we start with your account number?"
Agent initiates with welcome prompt

undefined
undefined
User initiates conversation in `POST /agent/task/id/step`

Design Proposal
Alternatives Considered

Remove task.input if the Agent can not respond with task.output.

Compatibility
AutoGPT seems to be sending the same message to createTask() and sendChatMessage() in chat_view.dart#L126:
                  if (widget.viewModel.currentTaskId != null) {
...
                  } else {
                    String newTaskId = await taskViewModel.createTask(message);
                    widget.viewModel.setCurrentTaskId(newTaskId);
                    widget.viewModel.sendChatMessage(
                        (message == "") ? null : message,

In ForgeAgent they suggest that in create_task you might just log the task.input.
   async def create_task(self, task_request: TaskRequestBody) -> Task:
        """
        The agent protocol, which is the core of the Forge, works by creating a task and then
        executing steps for that task. This method is called when the agent is asked to create
        a task.

        We are hooking into function to add a custom log message. Though you can do anything you
        want here.
        """
        task = await super().create_task(task_request)
        LOG.info(
            f"📦 Task created: {task.task_id} input: {task.input[:40]}{'...' if len(task.input) > 40 else ''}"
        )
        return task

Questions and Discussion Topics
Add open questions or options you require feedback on from the community.

as raised in #63


avoid running TestSplitAxis tests on NumPy 1.10

Fix regression introduced in #4153.

Thanks!
LGTM!


Set hhea caretSlopeRise and caretSlopeRun based on italicAngle value

It appears that Glyphs (or makeotf?) calculates caretSlopeRise and caretSlopeRun from the italicAngle, but glyphsLib doesn't do this anywhere. ufo2ft checks for openTypeHheaCaretSlopeRise and openTypeHheaCaretSlopeRun so all we need to do for the default is to add these to the ufo.
I tested on a few files and this gives me the same hhea values that are generated when exported from Glyphs.
@schriftgestalt does this look correct to you?

@moyogo if this looks correct to you, shall we put something like that in ufo2ft fontInfoData as a fallback when the hhea carte values are not explicitly defined?

In Glyphs, those values are still calculated by makeOTF.

@schriftgestalt Ok, that's what it looked like. Here's the makeotf code from makeotf hhea.c which is basically the same thing.
    if (g->font.ItalicAngle == 0) {
        h->tbl.caretSlopeRise       = 1;
        h->tbl.caretSlopeRun        = 0;
    }
    else {
        h->tbl.caretSlopeRise       = 1000;
        h->tbl.caretSlopeRun        =
            (short)RND(tan(FIX2DBL(-g->font.ItalicAngle) / RAD_DEG) * 1000);

Zachary, could you please open a similar pull request for ufo2ft? I prefer to have this in there instead.
You’d would define a new special fallback function inside ufo2ft.fontInfoData module.
Thanks

+1 for having this in ufo2ft.

Yes, ufo2ft makes way more sense. I was thinking it was done by Glyphs, but now I see it's all makeotf so I'll close this and open a new pull request for ufo2ft.


Integration Test - Release - 2019/02/14 07:52:16

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

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


Invalid logical input for file "DTU_10MW_NAUTILUS_GoM.fst".  The error occurred while trying to read Echo.

Bug description
when i set the fst file like this:
"------- FAST v8.16* INPUT FILE ------------------------------------------------
LIFES50+ NAUTILUS Semi-Sub Gulf of Maine Public Design with DTU 10MW Reference Wind Turbine, Michael Borg (borg@dtu.dk), Antonio Pegalajar-Jurado<EMAIL_ADDRESS>and Freddy Madsen (fjma@dtu.dk), DTU Wind Energy, January 2018
---------------------- SIMULATION CONTROL --------------------------------------
True          Echo            - Echo input data to .ech
"FATAL"       AbortLevel      - Error level when simulation should abort (string) {"WARNING", "SEVERE", "FATAL"}
4000   TMax            - Total run time (s)
0.005   DT              - Recommended module time step (s)
2   InterpOrder     - Interpolation order for input/output time history (-) {1=linear, 2=quadratic}
1   NumCrctn        - Number of correction iterations (-) {0=explicit calculation, i.e., no corrections}
99999   DT_UJac         - Time between calls to get Jacobians (s)
2.36E7   UJacSclFact     - Scaling factor used in Jacobians (-)"
it still produce the wrong about
"  Using the FAST input file: DTU_10MW_NAUTILUS_GoM.fst
Running FAST (v7.02.00d-bjj, 20-Feb-2013).
Invalid logical input for file "DTU_10MW_NAUTILUS_GoM.fst".
The error occurred while trying to read Echo.
Aborting FAST.".
i will  be thankful if anyone can answer my question！！！！！！

Dear @pushandpush,
It looks like you are using an input file formatted for FAST v8.16 while running a FAST v7.02 executable.  The input file formats are quite different between these version of FAST.
Regardless, unless you have a strong reason to use older versions, I would recommend upgrading to OpenFAST (please note, though, that the input file formats will change again when switching to OpenFAST).
Best regards,


Clarify hostile fork purposes (LGPL)

Making forks with different licenses isn't nice.

That seems extremely petty for a developer of another youtube-dl fork.

Maybe someone has invested some time into theirs project, and doesn't want somebody to take it and incorporate into closed source projects, or change the license?

Sorry


Filter for installed Plugin Hub Plugins

Is your feature request related to a problem? Please describe.
It's impossible to filter the plugin list to only show plugins installed via the Plugin Hub, which would be useful for organisation.
Describe the solution you'd like
Add a filter to the search box that shows only plugins installed via the Plugin Hub
Describe alternatives you've considered
Another option would be having a button to enable/disable plugins from within the plugin hub interface, but this could potentially add confusion and ambiguity with regards to the difference between enabling/disabling and installing/uninstalling.
Additional context

You can configure, and therefore enable/disable, plugin hub plugins from the plugin hub panel, but enforcing a tag onto them is probably a good idea

The current issue with that approach is more a visual one - It's not obvious which plugins are enabled/disabled from this view.
Adding that to this list would probably introduce a bit too much ambiguity between the difference of installing and enabling, so I do think it's probably better giving them a tag.

Closed by ab8fb76cac590d79415e668c41c48687d9ceaf4d


Assert function can be used to change types, not narrowing down types.

Bug Report

🔎 Search Terms

Assertion
Assert function
Narrowing
Type guard

🕗 Version & Regression Information

I don't know when this started.
My current typescript version is 4.8.3.

⏯ Playground Link

Playground link with relevant code
💻 Code

// Define Container for Type(Not for value).
// ALWAYS SATISFIED: SetValue<A, V> is assignable to V

interface Box {
    value: (this[_GetLastKey<this>] extends never ? unknown : this[_GetLastKey<this>]);
};

type CreateBox<V> = SetValue<Box, V>;

type _GetLastKey<T, _Key extends string = ''> =
    `${_Key}0` extends keyof T ?
        _GetLastKey<T, `${_Key}0`>
    :
        _Key extends keyof T ?
            _Key
        :
            never;

type GetValue<T extends Box> = T['value'];
type SetValue<T, V> =
    T
    & { [K in Exclude<keyof T, 'value'>]: never }
    & { [K in (_GetLastKey<T> extends never ? '0' : `${_GetLastKey<T>}0`)]: V };

// EXAMPLE to use Box

// Container to contain string or number

class StringOrNumber<TypeBox extends Box> {
    private value: string | number;

    constructor(value: GetValue<TypeBox>) {
        this.value = value as any;
    }

    setValue(str: string): asserts this is StringOrNumber<SetValue<TypeBox, string>>;
    setValue(num: number): asserts this is StringOrNumber<SetValue<TypeBox, number>>;
    setValue(value: string | number) {
        this.value = value;
    }

    getValue(): GetValue<TypeBox> {
        return this.value as any;
    }
}

function main() {
    const container: StringOrNumber<CreateBox<number>> = new StringOrNumber<CreateBox<number>>(1);

    const num1 = container.getValue(); // getValue returns number

    container.setValue('ABC');         // This changes container type to string
    const str1 = container.getValue(); // getValue returns string (not number)

    container.setValue(234);           // This changes container type to number
    const num2 = container.getValue(); // getValue returns number (not string)

    container.setValue('DEF');         // ... (SAME) ...
    const str2 = container.getValue();

    container.setValue(567);
    const num3 = container.getValue();
}

🙁 Actual behavior
Return type of container.getValue changes depending on each container.setValue calls. The type is not narrowed down, it's changed.

🙂 Expected behavior
container.setValue(123);
container.setValue('ABC');
container.getValue(); // Should return never

Assertion should only narrow down types, should not be able to change types freely. So after two different types of setValue function called, return type of getValue should be never.

But the value is changing in the implementation. The actual behavior is modeling your code, whereas your expected behavior would introduce errors on this perfectly working implementation. 😄 I think you could come up with an example that demonstrates why your expected behavior could be desirable, but this one seems to support the actual behavior.

Note that the behavior you expect does happen when the thing you start with is a union:
declare function f<T>(x: any): asserts x is T
declare const x: string | number;

f<string>(x);
x; // string
f<number>(x);
x; // never

Non-unions and type predicates / assertion functions are a little different. The only thing we could possibly do is take the intersection of every type you’ve asserted container is, which would probably render it unusable like you expect, but it would also be super expensive to compute those intersections. And, the current behavior correctly models what’s happening here. I think this is working as intended.


Multiline favourite row

Longer station names don't fit into the 1 line cell. Thanks @BalestraPatrick.

I think this was on the phone app


Allow data pre-processing via secure sandbox

Check out ses

Alright, this is going to require some more thought. I tried a number of options tonight:

ses wouldn't let me call functions
jailed wouldn't let me pass functions
AdderScript wouldn't let me assign global variables

Several other options I looked at (including scheme and lua variants) included DOM manipulation and js interop, which is obviously not what I want.
An iframe jail is probably good enough for what I want to do, but will require some architectural thought because message passing is global.