fix: fix filtering for output-dependent compare slices

Description
For slices dependent on the output of a model in the compare tab, have to selectively filter for a selected model.
Fixes ZEN-103 : Filtering doesn't work for slices with predicates dependent on the model in comparison tab

This is a bit more complicated than I thought, worth rethinking or refactoring how we save slices.... the name of the column is tied to a model currently. Either have to store without a model reference or column name, or update them when doing calculations. A bit messy

Alright looks like @Sparkier had already sort of fixed this in the backend when creating filter_sql, just had to be updated for the table fetching with the current model.
The only open issue is that filtering for a specific model in the comparison tab is very complicated and wasn't working (have to somehow specify in the selection store which model its for, would be a major refactor). I removed it for now, it only filters using Model A. Added a linear issue:
ZEN-104 : comparison view only filters slices using first model

This is a bit more complicated than I thought, worth rethinking or refactoring how we save slices.... the name of the column is tied to a model currently. Either have to store without a model reference or column name, or update them when doing calculations. A bit messy

Haha that's what I thought when first stumbling over this.


Fix issue when uploading built frameworks to ITC.

Because the order in which the frameworks were copied (simulator vs
device) could vary, sometimes we ended up with the Info.plist used by
the simulator, which upsets iTunesConnect with the following message:
[12:54:45]: ERROR ITMS-90542: "Invalid CFBundleSupportedPlatforms value. The key 'CFBundleSupportedPlatforms' in the Info.plist file in bundle 'Payload/Skyscanner.app/Frameworks/OAStackView.framework/OAStackView.bundle' contains an invalid value '[iPhoneSimulator]'. Consider removing the CFBundleSupportedPlatforms key from the Info.plist. If this bundle is part of a third-party framework, consider contacting the developer of the framework for an update to address this issue."
Fix is to copy the simulator frameworks first, so the device Info.plist
overwrites it.

@neonichu Not sure if this is the right fix or not (what should actually be in the Info.plist for a fat framework?), but it resolves the immediate issue for me at least!

LGTM

Cheers @neonichu !


considering using this for serious production use... any production case studies / actual production use case now?

considering using this for serious production use...

any production case studies / actual production use case now?
who is using it?
recommended for production use?
what kind of limitations / gotchas / caveats need to look out for? can u pls help specify these onto the front page / readme?

thx

=> I'm using it in production for several project but all his private sorry :/
Me, and other people but dont have references
Depends of your project off course, all MDN is not yet implemented. If you need some function like webrtc, you can PR me and integrated on hogosuru , or just use javascript. hogosuru can be easly mixed with JS.
Limitations caveats?: No problem found. Like all projects sometime bug can happens but i'l corrected it.

close for no activity. If you have question please reopen


sgxtuner_core_1 exits with code 101

Every time I tried to run it, I got this error. Would you please help me to fix this?

see c1a759d7c6f7df5d351b2f9390092e728ef476e9 for a potential fix


Support long running background uploads/downloads

Is your feature request related to a problem? Please describe.
The main scope is to support uploads mainly while the application is in the background. Currently this is not supported by Amplify SDK. You can read more about it in #638 .
Secondly, there is no support for setting up a AWSServiceConfiguration with AWSStaticCredentialsProvider  in Amplify.
Describe the solution you'd like
A possible solution would be something like AWSTransferUtility that aws-ios-sdk has and supports background uploads. Also, AWSTransferUtility can be setup with a AWSStaticCredentialsProvider. However, the main issue here is the support of long running background uploads.
Describe alternatives you've considered
No alternative with Amplify SDK. I still use the aws-ios-sdk.
Is the feature request related to any of the existing Amplify categories?
No response
Additional context
No response

See Discussion :

Storage: Long Running Multipart Uploads

Hi, @nick3389, the AWSS3StoragePlugin in Amplify version 2 uses a background URLSession so that downloads can continue even when the application is in the background:
https://github.com/aws-amplify/amplify-swift/blob/main/AmplifyPlugins/Storage/Sources/AWSS3StoragePlugin/Service/Storage/AWSS3StorageService.swift#L73
Would you please let us know if this was what you were referring to?

Hi @jcjimenez,
Does this support uploads as well?

Hi, @nick3389,
Yes, this works for uploads also. However, there's one caveat: if the size of the file being uploading is larger than 5MB, then the upload will be split into separate [part] upload tasks. So, any tasks that are in progress when the app goes to the background will proceed, and if connectivity remains stable, will complete successfully. However, any subsequent [part] tasks will depend on the app coming back into the foreground so they can be started.
In other words, imagine the file being uploaded is divided into 10 parts and the first 3 start immediately after the call to Amplify.Storage.uploadFile. These 3 tasks will continue even after your app goes into the background. However, the other remaining 7 tasks will not be created until a one or more of the first 3 complete successfully.


Make relative reporterOutput paths optional

As discussed in #202, making the reporter output filenames causes problems in some cases - in my specific usecase, I'm using the junit reporter, and putting the reporter output into a folder for use in CircleCI reporting.
This change adds the reportOutputRelative option that can be set to false to disable the altering of paths to be relative.

I just ran into this issue, could this be merged? Thank you.
/cc @XhmikosR @vladikoff


[Technical] Refactored to use a parameter object for the ProcessRunner

No changes in behaviour. All existing tests pass.
The ProcessRunner already has six arguments and three "Execute" overloads. Not logging all arguments to the command line (SONARMSBRU-126) will add a seventh parameter.
This code review of mainly for information and to clean up the code before fixing SONARMSBRU-126.

LGTM


Mark scheduler pod resource metrics as beta

They will be part of 1.21 as beta stability as of kubernetes/enhancements#1748

/assign @mvortizr
/sig instrumentation
/milestone 1.21

@smarterclayton looks like your branch might need a rebase

/remove-language id it ko zh
/lgtm

/approve


Create experimental GET APIs for CF custom resources

Background
We want to get a better understanding of what building a CF v3 API that is backed by custom resources actually looks like.
Using the CRDs we created in #1, let's spike out a couple of custom resource-backed API endpoints that are used by the cf push workflow.
The GET /v3/apps (listing apps) and GET /v3/apps/:guid (fetch a single app) endpoints should be a good starting place as they are neither too complicated nor too trivial.
Questions we'd like to answer:

How to implement filters (e.g. the supported query params on the API)
How to list apps across CF spaces (assume the apps are in different Kubernetes namespaces)
Don't need to solve scalability issues or permissions yet for the MVP spike, but keep these thoughts in the back of your mind.

A goal of this work is to be able to share our findings with the CF community.
Deliverable

You should be able to curl the /v3/apps and /v3/apps/:guid endpoints and see minimal differences from what you'd see on CF for VMs/cf-for-k8s
Keep a journal of what was difficult, what was easy, etc.

Scenario: Apps Across Namespaces
Given a kubernetes cluster
And there are multiple app CRs in different namespaces
When I deploy the component that is serving the /v3/apps API endpoint
And I run curl /v3/apps and curl /v3/apps/:guid
Then I get back a list that represents all app CRs in my cluster.
Scenario: Filtering
Given a kubernetes cluster
And there are multiple app CRs
When I deploy the component that is serving the /v3/apps API endpoint
And I run curl /v3/apps with a name filter
Then I get back a list that only includes app CRs that match the filter.

Dev Notes

Refer to this Miro board for inspiration.
v3 API docs: https://v3-apidocs.cloudfoundry.org/version/3.100.0/index.html#list-apps
If there is a filter parameter that relies on a relationship with another resource, bring up with team.

Dev Provided Acceptance Steps:

Get cluster & clone repo https://github.com/cloudfoundry/cf-crd-explorations
make install to install CRDs
kubectl apply -f config/samples/ --recursive
make run to run the shim locally
curl http://localhost:81/v3/apps?lifecycle_type=kpack to find the app created above and grab 'guid'.
curl http://localhost:81/v3/apps/<guid>
modify config/samples/apps_v1alpha1_app.yaml with a different metadata.name (k8s uniqueness) and spec.name (cf api queryable)
curl http://localhost:8082/v3/apps?names=my-app-name
curl http://localhost:8082/v3/apps?names=my-app-name,<new spec.name>

Note: non-existent filter fields will not restrict results. In the case of a bogus filter, all results will be returned. We should discuss what our intended behavior is in the future.

We validated this spike code out of band and it's no longer in flight. Closing it out.
If folks are interested the code for these GET endpoints can be found here:
https://github.com/cloudfoundry/cf-crd-explorations/blob/main/cfshim/handlers/app_handler.go


ENH: Displaying status_code of the email status would be good for better error handling

I tried to find this feature but I didn't find it. so I am trying to get status_code of the email's status like "If the email was successfully sent or failrd" and "If failed then How email failed", I mean what reason effected the email failure. If the console is showing error like -

smtplib.SMTPRecipientsRefused: {'user': (553, b'5.1.3 The recipient address  is not a valid RFC-5321 address. Learn\n5.1.3 more at\n5.1.3  https://support.google.com/mail/answer/6596 ik12-20020a170902ab0cm2094481plb.177 - gsmtp')}

or

socket.gaierror: [Errno 11001] getaddrinfo failed

so, A user will want to show or pass the appropriate message to the user in frontend on "What went wrong" or "What causes the email failure" ? but in this case It will only be seen as "something went wrong" (No status code or a clean message) to pass forward.
For Example -
email = EmailSender(
    host="smtp.gmail.com", port=587,
    username="xyz123", password="xyz123"
)

email.send(
<EMAIL_ADDRESS><EMAIL_ADDRESS>   subject="An example email",
)

and If I try to print the EmailSender class calling variable print(email). Then it prints Class object Binary.
so It would be better to show more information about email just sent.

Red Mail should not do any silencing on the error: whatever the smtplib raises will be passed through Red Mail to the user of the library. And the user can process the exception however needed.
Do you mean you would like to have better errors than what smtplib produces? Perhaps a valid point but it should be done with extreme care so that no information is lost. Error silencing for the sake of a status code is something I'm very against.
Furthermore, your example produces quite an intuitive exception though (you are missing a quote):
smtplib.SMTPAuthenticationError: (535, b'5.7.8 Username and Password not accepted. Learn more at\n5.7.8  https://support.google.com/mail/?p=BadCredentials q3-20020a0565123a8300b00498fc3d4d15sm1004920lfu.190 - gsmtp')

It's not just about 'status_code', it can be done by passing status_code with appropriate message. But that's all upto you.


[native filters] Applying selected options in parent filter also apply available options in child filter

Is your feature request related to a problem? Please describe.
Using hierarchical filters, when I select options in parent filter, I see that options are updated in the child filter. But when I click on the apply button, only the parent filter apply. It means that there remains values displayed in charts that do not respect updated values in the child filter. For more explanation, see the gif below.
Describe the solution you'd like
I would like that even if not options are selected in the child filter, applying the parent filter would take into account available options in the child filter.
Describe alternatives you've considered
An alternative would be to force the dashboard user to select values in the child filter. But in some case the user would like to select all available options in the child filter with regard to the parent filter and with a high cardinality.
Another alternative would be to merge different dataset into one, indeed the child filter options would be taken into account indirectly. However I see several drawbacks to it :

In some case we can't merge all datasets together because of the resulting cardinality
When merging datasets, you need to choose an order and types of join (left, right, outer, inner) with complexity

Additional context
Here is an exemple.
I have two table charts with two associated datasets :

Disponibilités : the associated dataset contains two columns, one is a login and the other is called 'dispo' and take values among ('dispo','sous l'eau'). It represents the work availability of coworkers
Taux horaire : the associated dataset contains two columns, one is a login and the other is called 'prix' and take values among ('cher', 'normal','good'). It represent the cost of coworkers.

In the following gif, I created two hierarchical filters on the 'Disponibilités' dataset :

The parent filter is on the column 'dispo' and has a scope only on the 'Disponibilités' chart
The child filter is on the column 'login' and has a scope on both charts.

When I select options in the parent filter, the options in the child filters are updated.
When I click on apply, I see that only the first filter is applied, I would like logins to be filtered too in the chart 'Taux horaire'

I do not see this feature in the following SIP :
I found an issue maybe related to my feature :

Steve Marek (TRAC VC) helped me find a workaround with Jinja tempates :

I build a saved query from my second dataset and I display this saved query in a table :
SELECT prix, login from taux_horaire where 1=1 {{ "AND login IN (SELECT login FROM dispos WHERE dispo IN ( '" + "','".join(filter_values('dispo', remove_filter=True)) + "') ) " if filter_values('dispo') is defined and  filter_values('dispo') else "" }}
Here is our discussion : https://app.slack.com/client/T7GMC60JX/CCKHMGRRB/thread/CCKHMGRRB-1634639904.159900
I will build a dashboard with real datasets and verify there is no performance issues.


add skeletons/placeholders for settings page for loading period

When visiting /settings, there's a brief moment in which half of the settings is shown as a call is still being made fetching the marking details. Instead, show a skeleton/placeholder while the call is not yet finalized and then show the complete settings page at once.

Solved!
Skeleton is added!


Feature Request: Pick which mouse button is held down during the drag

When using certain CAD software, it is convenient to click and drag the MMB for certain commands. It appears this app will do exactly what I would like, but it only emulates the long-click of the left mouse button. If it could instead emulate the MMB, that would be very helpful.

Good idea, this is a really simple feature to add. I will probably add it in the next release.

I added this option in the app and it will be available in the next release in a few days. Thank you for the idea!


Admin progress bar does not automatically updated

Updating only dot in the upper right corner and status near the dot, when hovering mouse.
But status not updating. Only at start some update and stops.
Photos are rescaning without issues.
https://imgur.com/a/91VNiVW

This was added in the migration to mantine! :)


Join function update

Replaced Unix with UnixNano in Join function.
Changed condition to remove old items from right stream buffer.

Pull Request Test Coverage Report for Build 753

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build 752:
0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

Created another PR https://github.com/ReactiveX/RxGo/pull/232


Choice of currency pairs

instead BTC only and FIAT only
(like LTC/DOGE or anything available)

Part of the problem is that there are hardcoded constants throughout the code which assume that the price of the currency pair you are trading is about $100-$1000. Making it configurable is on my list of things to do.

Where are some of these constants? I'd love to help out with this feature, but could use a few pointers.

@holoiii the problems is these constants are spread pretty evenly throughout the codebase, and it requires a substantial refactoring of how gateways are constructed and currency pairs are validated.  I put up a PR #98 to address this. Please take a look at it.


Add an option to exit non-zero on linter output

The current default behavior, where gometalinter exits 0 even when there are linting errors (for example with gofmt), makes this harder to use in a commit hook than it should be.
It'd be nice to have an option that caused it to always exit non-zero if any linter finds an issue.

gometalinter exits 0 even when there are linting errors

That is not the behaviour:
https://github.com/alecthomas/gometalinter/blob/b25b44d18fb62e4454847292388fbf3f9c503d68/main.go#L233-L246

Doh, sorry. I was misinterpreting the issue with my git hook.


Add "Search in Transcript" buttons to right of search box on transcript search-results snippets page and transcript text-display pages

Add "Search in Transcript" buttons to right of search box on transcript search-results snippets page and transcript text display pages

Done, but not quite deployed yet. Will be a few minutes.


Align the two configure programs (configure.sh and configure.c)

Summary
With #1614 when using the tool with -E and no configuration was already in place, the general "NuttX has not been configured" from the toplevel Makefile is printed before configuring.
Also apply the same changes to configure.c
Impact
Testing

@ghn-certi just FYI.

@ghn-certi just FYI.

My only concern was about whether the existence of the .config file is a requirement executing the [-E] command.
Again, if the -E is an enforcement, it should always execute regardless of any condition.

My only concern is about whether the existence of the .config file is really a requirement for executing the [-E] command.
IMO, if the -E is an enforcement, it should always execute the distclean regardless of any condition.

If .config doesn't exist it means that distclean is not necessary, so -E won't have any effect (no other make target will have an effect if .config is not present).  This should work exactly as the last PR with only removing the unnecessary output when the project is not configured.


Check flutter dependencies and cleanup unused dependencies

@ewanas @bryanchriswhite  I used this dependency to resize mobile UI to fit desktop. We don't need it now because we have totally separate UI for desktop.
Originally posted by @shareef-dweikat in https://github.com/LeastAuthority/flutter_wormhole_gui/pull/2#discussion_r764276637

Done https://github.com/LeastAuthority/flutter_wormhole_gui/pull/56


What is the relationship between the output label of pretrained model in model zoo and wordnet synset id?

we can easily access pytorch pre-trained model like VGG, AlexNet and SqueezeNet by
import torchvision
torchvision.models.vgg16(pretrained=True)

can anyone point out what's the relationship between the output label(index of maximum output value) and the actual category?
i downloaded ILSVRC2012_devkit_t12 and got the imagenet id and other metainfo provided by meta.mat, however it seems pre-trained model have some different id. because when i evaluate the network with ILSVRC2012 validation set, it reports 100% error.

You can find an example here.
You will need two files:

imagenet_classes.txt
imagenet_synsets.txt

Thank you for your help, they are exactly what i need.

I think we should provide somewhere the mapping between the class names and the indices. Not sure if it should be in pytorch/examples or in torchvision. What do you think?

That's great! i think it will be torchvision to provide those mapping, and pytorch/examples to have some example on it. :grinning: am i too greedy?

I think this totally make sense to provide this mapping.
I'll think about a good way of doing it, maybe making a base class TorchvisionModel, that contains the basic pre-processing parameters that are used for all the models, as well as the image synset.

+1 would appreciate a model.class_ method or model.labels_ kind of like sklearn. Developers can easily see the mapping between output labels and index. May be this feature already exists? Would love to know how to do it. Thanks!

Thank you for your help, they are exactly what i need, as torchvision modules itself didn't provide any documentations on this.
How did you manage to make it work, even with these two text files there still a discrepancy between the output predicted labels and the imagenet validation dataset.
I am using Squeezenet to predict the validation classes on ILSVRC2012. The pretrained model seems working well, however, the accuracy is 0!

I think that class names (i.e.  a map idx -> class name) are an integral part of any pre-trained classifier.  Would a PR with class names be welcome?


Reuse

moving the dep5 to recommended folder .reuse
adding LICENSES folder with corresponding Apache-2.0.txt

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Vlad338769:x: alldantasYou have signed the CLA already but the status is still pending? Let us recheck it.


Rename updateEnrolledHeight to addValidator

As we are splitting the EnrollmentManager into EnrollmentPool and ValidatorSet, this function had to be renamed to fit its new purpose.
Before, enrollments that were not in the chain would have an enrolled height of 0, and when they appeared in the chain, the actual enrolled height would be set through this function.
Soon the storage will be separate, and instead, this function will just add new enrollments and clear up the pool.
Additionally, this fixes a bug, where catchup didn't work when an Enrollment seen in a block was not found in the Enrollment pool.

I applied @TrustHenry 's comments to the code. Thank you.


Re-enable S107 once Sonar issue is fixed

See https://github.com/SonarSource/sonar-dotnet/issues/3727.

It's now fixed, waiting for release.


deal with pymultinest with Cray MPI at NERSC

See https://github.com/LSSTDESC/desc-help/issues/66#issuecomment-1092008402
The td_env conda environment includes mpi4py and the Cray mpi libraries.  Until pymultinest offers a way to set an env variable to skip the attempted import of mpi4py when running on a Cori login node (which includes jupyterLab), we need another way to avoid the mpi4py import.
Contacted the mpi4py developer who suggested:
import sys
sys.modules["mpi4py"] = None

Could use PYTHONSTARTUP in the setup script when $SLURM_JOB_ID  is unset (meaning we're on a login node rather than compute node)  Perhaps not the best option for all cases:

starting a batch job from a jupyter notebook, I don't think the python environment would be automatically re-initialized on the compute node, rather the current environment without the mpi4py library available would be inherited.
setting up the environment on a login node before submitting a batch job which inherits the current environment.  This could be avoided if the env is set up directly from the batch job or when attached to the "interactive" queue.  Actually, this is fixed by checking if SLURM_JOB_ID is set in the PYTHONSTARTUP script itself
PYTHONSTARTUP is only utilized for an interactive session, so it won't fix the case where a python script is run on a login node
So not a perfect solution - but I think this might be ok for now where most users are likely not so concerned about MPI.

Ended up using sitecustomize.py instead and that seems to cover all the use cases properly.

seems fixed


scanVI

I have run scanvi with an internal unlabelled dataset and a publicly available, labelled dataset as inputs following your annotations notebook. Once we trained the model and looked at the UMAP, we found that the two datasets were very separated.  The entropy of batch mixing of the two dataset was also 0. We don’t have access to the raw public dataset, so we normalized our internal one accordingly. Could you provide more insight on what can be done to produce better annotations? Is there certain preprocessing we should follow for optimal results?

Hi @gandladm,
Can you get the datasets to mix with scVI only? It is a problem the reference data is normalized, indeed. You might be able to get it to work by putting it on a reasonable scale. But if the datasets are separated with scVI, they will be as well in scANVI. Also, if one data has UMIs and the other one does not, it can be tricky.
It will be hard for me to help without further details about the experimental setting.
Best,
Romain

Closing due to inactivity


Rename LICENCE to LICENSE?

@srstevenson thus, may I also ask you to not install that files into the site-packages directory as well? :)

Good catch, thanks for flagging this! Addressed in #183.


Resharding

@enisoc @dumbunny This is a start.

Should we also add the new test to test/config.json?

Done, will merge and import then.

LGTM


Can't use webfonts with webpack

Describe the bug
I am trying to use vuetify with the mdi iconfont. When I import css which contains webfont, application cannot resolve font file and prints the warning
unsupported route: /materialdesignicons-webfont.woff2?d1bdfb3838e2f78edf1ede85f56eabc9; error: {\"code\":404}
infinitely until OOM. It works sometimes but most of the times it fails.
I re-checked vue-starter's doc and vuetify's. Didn't miss anything.
To Reproduce

yarn add @mdi/font
Go to App.vue
paste in style block
  @import<EMAIL_ADDRESS>restart the app

EDIT: Also sometimes I got the infinite warn log and this raises up even without webfonts :
warn: unsupported route: /client/785c8cdd333e6e8a782e.hot-update.json; error: {"code":404}

@ayZagen I finally had some time to tackle this issue. In the latest version of the vue-starter it works in the way you describe it. I was able to do exactly what you describe without getting an error.

@devCrossNet I will look into it when I got spare time. Closing the issue for now. Thanks for keeping your support up!


ROCm: HIP does not provide math_constant.h

In a number of places in the codebase, we use constants like CUDART_NAN:
https://github.com/cupy/cupy/blob/fd11f0d917426cea9c4f3658e5d8b8cf7c9d56f2/cupyx/scipy/ndimage/_filters_core.py#L261-L266
But HIP does not provide math_constant.h, so any code involving math constants will fail. Looks like we need to provide a stub file on our own?

In I prior PR, I had originally tried to define NAN and INF when I didn't know about the CUDART_NAN and CUDART_INF defines. Perhaps those can be used to define CUDART_NAN and CUDART_INF for HIP? see: https://github.com/cupy/cupy/pull/2861/commits/c06acf6d6632975c8e6c3f92df3c9bf59bd1d843#

Cool, yeah I think this is exactly what math_constants.h in CUDA does too:
/* single precision constants */
#define CUDART_INF_F            __int_as_float(0x7f800000)
#define CUDART_NAN_F            __int_as_float(0x7fffffff)
#define CUDART_MIN_DENORM_F     __int_as_float(0x00000001)
#define CUDART_MAX_NORMAL_F     __int_as_float(0x7f7fffff)
#define CUDART_NEG_ZERO_F       __int_as_float(0x80000000)
#define CUDART_ZERO_F           0.0f
#define CUDART_ONE_F            1.0f
...

So a stub file can be generated as easy as copying all contents from there.


docs(README): add @datepicker-react/styled to the "Built with Styled System" section

Add @datepicker-react/styled to the "Built with Styled System" section.

Awesome work! Thanks!

Thank you. Styled System is one of the best libraries I have ever used.

Thanks! Feel free to PR that as a testimonial on the landing page 😉

Will do 👍


I can't add buttons

After configuring the redactor using one of the provided json files, I cannot see the buttons on the toolbar.
From #8 it appears that the plugin is not compatible with redactor II.
Is that the case?

Just to confirm my hunch, I downgraded to Redactor I using P&T plugin but the issue still remains.
Am I supposed to see the additional buttons in the toolbar? Perhaps, my understanding is incorrect

nevermind, I forgot a closing parenthesis
It works now. I am going to stay with Redactor I which is much better


Add parameters to trade service api

Add the following parameters to the trade service api that lists all active bots

[x] Chain ID (depending on the chain that the vyper smart contract is deployed on - we will launch with ethereum and bnb) chain: "1", "56" (Please return chain ID)
[ ] type (currently only momentum, we will add another bot hopefully next week) bot: "Momentum Bot", "DCA Bot", "Limit Order Bot", "Stop Loss Bot"
[x] deposit id
[x] token name and token image for token1 (from coingecko api)
[ ] DEX name (this can be inferred from the vyper smart contract since we have one contract per DEX) dex: "Uniswap V2", "PancakeSwap", "ApeSwap"

https://service.robots.palomachain.com/robots?depositor=0x83334ef0C6f6396413C508A7762741e9FD8B20E9

the front end should be able to break down the information.. do we need this ticket?

@rootedbox I'm not sure what you mean by that. @jamie088 do you have everything that you need from this API or do we need more changes?

@rootedbox No, we can't get the chainId and bot type with current information (deposit ID, token address) on frontend. I think the backend uses the contract address and chain when receiving the deposit event from the smart contract, so it knows the chainId and the bot type. Right? cc: @verabehr

@rootedbox added one additional item to this ticket per https://discord.com/channels/987730379182051408/1014207212954066975/1113906188954898432

@rootedbox moving this back into your todo. LMK if you have any questions

bot: "Momentum Bot", "DCA Bot", "Limit Order Bot", "Stop Loss Bot"
chain: "1", "56" (Please return chain ID)
dex: "Uniswap V2", "PancakeSwap", "ApeSwap"

@rootedbox why was this closed? I still see 2 out of the 3 tasks open. Also I don't see the latest code PR 50 reflected in the api yet


Simplify protocol hand shake state management

No issue to close
related to #377
Description
simplification in PG protocol state machine transformation between stages
Is it a feature that change user experience?
no
Client output
no changes to client output

Pull Request Test Coverage Report for Build 312421307

131 of 135 (97.04%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 89.051%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/protocol/src/hand_shake/mod.rs
76
77
98.7%

src/protocol/src/lib.rs
15
18
83.33%

Totals

Change from base Build 310774910:
0.06%

Covered Lines:
7922

Relevant Lines:
8896

💛  - Coveralls


#709 Add ASL license header into the files

I dont think we need double # in the yaml files, a single # is okay ?

@davsclaus single # should be fine. I will updated the code for it.

I updated the yaml header and rebase the change of master, I also added rat-check.sh for verification.


Prevent errors on PirateBay plugin when scanning ads

Motivation for changes:
Currently the Piratebay plugin gives errors when scanning their site, because of advertisements that are also marked as class="detLink". This change detects if the ULed user was piratebay, which indicates that it is an advertisement.
DEBUG    utils.requests Series-Trakt    GETing URL ... with args () and kwargs {'allow_redirects': True, 'timeout': 30}
DEBUG    plugin        Series-Trakt    decorator caught ValueError. handled traceback:
Traceback (most recent call last):
  File "flexget/plugin.py", line 119, in wrapped_func
    return func(*args, **kwargs)
  File "flexget/components/sites/sites/piratebay.py", line 164, in search
    entry['torrent_seeds'] = int(tds[-2].contents[0])
ERROR    discover      Series-Trakt    Error searching with piratebay: invalid literal for int() with base 10: '\n\t\t\t\t'

@paranoidi is this ready for you to merge?

Sure, thanks for contributions!


Config to use PostgreSQL instead of mysql

I'm setting up openeats and I use PostgreSQL for other projects in my servers already and would prefer to use it instead of MySQL. I configured it instead of MySQL and it seems to work so far on my dev env. Would you be interested in being able to choose between the two?

More options are always better. I don't think OpenEats will ever need to use an special DB settings. Please open a pull request If you have something. Otherwise we can discuss more here.
Thanks,
-Ryan

I did a quick change to just get it working, here: https://github.com/renatolond/openeats-api/commit/22d1cb932843f68fb02b395d331ba55968e5fc48
But ideally I think it would be better to have something more like
if USE_POSTGRES:
    DATABASES['default']['ENGINE'] = 'django.db.backends.postgresql'

And then maybe for the existing ENV variables, to avoid breaking existing apps something like
os.environ.get('DATABASE_NAME', os.environ.get('MYSQL_DATABASE', 'openeats'))

The OPTIONS part I don't know what's the best way to deal with :thinking:

I'm less worried about the settings file than I am about the requirements file. Your commit reminded me that we need different dependencies for postgres and mysql.
The setting's file can just contain a separate block for the DATABASES.
if postgres:
    DATABASES = [...postgres]
else:
    DATABASES = [...mysql]

The tricky part is making it work without having the overhead from the extra deps. I'll think on this.

So I don't think that I'm going to include this. Having multiple supported DBs in the docker image doesn't seem very clean. you are free to keep suing what you have.
Going to close this for now. If I get a better idea I will reopen.


CV: 2020 - 2021 Worked for a known fraudster to create his pump and dump scheme.

https://tinyurl.com/3xaej452

Spam or truth? There is only one way to find out. Look into the sources yourself.

tari-project/tari#3035

tari-project/tari#2997

tari-project/tari#2904

tari-project/tari#2893

tari-project/tari#3039

tari-project/tari#3038

tari-project/tari#3037

tari-project/tari#3036


Update the docs to add InheritDefaultEvictionValues feature gate

This is a Feature Request

This issue is created regarding the k/k PR #127577
What would you like to be added

Update the docs to add the InheritDefaultEvictionValues feature gate that will cause change in behaviour of the default values of the eviction hard parameters. Existing behaviour can also be executed by disabling the feature gate.
Why is this needed
Currently if the default values of any one of the parameters of eviction hard is changed, then the other parameters are set to 0. After the introduction of the new feature gate, if default value of any of the parameter of eviction hard is changed then the other parameters will inherit their default values.
Comments

/language en
/sig node

/assign
I'll work on this and open the PR once the code PR is merged


Linearize power

Linearizing power.
only POW(b,n) == IV supported, with n being an integer, post as bb...*b (n times) == IV

Yes, you should be able to just enable the testcase for Exact/Gurobi in tests/test_constraints.py using the EXCLUDE_OPERATORS  dict

I don't think it's that easy
First of all, this outputs non-flat constraints for n > 2
secondly gurobi supports power for n = 2, but not for n > 2. Right now we just say gurobi supports pow but then throw an exception when n > 2. So we want to linearize it when n > 2. Not sure how to do this with our supported argument (unless we add a quadratic thing).

I think it's ok to only support powers of 2 for now as it is the most common case probably. (Still need to check for it of course).
In the future, we can make intermediate result vars:
IV0 == b * b
IV1 == IV0 * b
...
But that might be tricky as these aux vars have to be assigned toplevel, but linearize has no knowing of toplevel/nestedness...

Hmm actually indeed it might be fine as the multiplication is functional - let's just make sure to write some extra tests for it ;)
So then the question just remains if we support arbitrary powers, or only ^2.

Updated this, should be ready for review @Wout4

I would still add somewhere that we only support integers as the exponent, maybe the solver interface or the power constructor? right now ortools will interpret any expression as being an exponent of 2 since we check exp == 2 (which will make a truthy expression if exp is an expression)


Update Python/Client README.md to explain our release process for python client

With #1072 we are now able to manually run a release github action.
https://github.com/G-Research/armada/blob/master/client/python/README.md#releasing should be updated to reflect our new release process.

Create a PR that updates the pyproject.toml version number for the python client.
Merge that PR into main.
Select the Release PyPI action and deploy a new wheel file to PYPI.

This is how you select the release action for deployment.

Going to knock this out + make a general pass over client/python/README.md for accuracy/layout


Prevent NullReferenceException in UseIdentityServerBearerTokenAuthentication constructor

Issue occurs when IdentityServer is set up before setting up IdentityServer Bearer Token Authentication.
I ran into this issue myself after a refactoring that happened to move lines about in a way which didn't seem as if it should break things. The issue is also discussed here: http://stackoverflow.com/questions/31977522/owin-middlware-with-identityserver3-accesstokenvalidation-in-asp-net-5-returns#31989005 with the workaround being to move usage of UseIdentityServerBearerTokenAuthentication above usage of UseIdentityServer.
I'm not advocating this as the only possible way to solve the issue. This is just one way and it has the following benefits:

Decouples IdentityServerBearerTokenValidationMiddleware from (imo) bad OWIN behaviour
Would not throw the deeply-concealed NullReferenceException that is (eventually) discovered after trawling through TargetInvocationExceptions

I'm not sure if there's a good reason why UseIdentityServerBearerTokenAuthentication must be called before UseIdentityServer when embedding IdentityServer. I'm open to correction on this PR.

Hi @fluffynuts, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@fluffynuts, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

Isn't there a default logger/factory in Katana itself. Why not simple use that when the logger has not been setup by the host?

As noted in the PR: this isn't necessarily the /best/ way to solve the problem. It's just one way. But considering that the problem comes from bad OWIN behaviour,  it might be a valid choice to eschew OWIN here. Alternatively, I'd welcome solid, working OWIN-based behaviour which circumvents the erroneous outcome, so whatever works. I confess to not knowing enough about OWIN to whip up an alternative PR aligned with your suggestion though.
tl;dr I'd just really like to see the issue fixed and I'm not too concerned about how, though at the juncture that got us here, tbh, I don't trust OWIN. It already misbehaved once, imo.

I don't have the time to look into that right now.
But if you do - check the OWIN logging source. IIRC there is a default implementation and you could set that when needed.
This PR is against master - we would need a new one against dev.

see here
https://github.com/IdentityServer/IdentityServer3.AccessTokenValidation/blob/master/source/AccessTokenValidation.Tests/Util/PipelineFactory.cs#L16


[Bug] The sync server returns 403 Forbidden for some block requests

🐛 Bug Report
Sync improved dramatically after #1896; however, snarkOS main thread again panics and the process dies while syncing.  Happens consistently.  Restart and it will sync several more blocks, if it's lucky, but will eventually hit this issue.
2022-09-27T13:04:30.857069Z TRACE Requesting block 20123 of 25922
2022-09-27T13:04:31.585474Z  INFO Synced up to block 20024 of 25922 - 77% complete (est. 17 minutes remaining)
2022-09-27T13:04:31.585510Z TRACE Requesting block 20124 of 25922
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Block parse error', /jihad/snarkos/snarkos/ledger/mod.rs:328:39

Steps to Reproduce

Step 1.   Run snarkos.
Step 2.  Watch it sync.
Step 3.  It panics and dies.

Expected Behavior
Sync completes.
Your Environment

testnet3
Rust 1.64.0
Ubuntu 20.04.5 LTS

It's due to the server not being able to handle that many requests. I don't have access to its configuration, but I've reported this further.

Now that rate limiting is turned off, it seems that some blocks might be missing; I encountered it with 25976, but others have been reported before.
We need to find out the reason why this happens; for reference, this is the output of curl -I https://vm.aleo.org/testnet3/block/testnet3/25976.block I'm currently getting:
HTTP/1.1 403 Forbidden
Server: Cowboy
Connection: keep-alive
X-Powered-By: Express
Access-Control-Allow-Origin: *
Surrogate-Control: no-store
Cache-Control: max-age=60
Pragma: no-cache
Expires: 0
Date: Fri, 30 Sep 2022 09:06:35 GMT
Content-Length: 231
Content-Type: application/xml
Accept-Ranges: bytes
X-Amz-Request-Id: tx0000000000000ccc49635-006336b18c-3f19a6f7-nyc3c
Strict-Transport-Security: max-age=15552000; includeSubDomains; preload
Vary: Access-Control-Request-Headers,Access-Control-Request-Method,Origin, Accept-Encoding
X-Hw:<PHONE_NUMBER>.dop089.dc2.t,1664528795.cds063.dc2.shn,1664528795.dop089.dc2.t,1664528795.cds078.dc2.c
Via: 1.1 vegur

Frequently get a similar but different error:
0|cargo  | thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: error sending request for url (https://vm.aleo.org/testnet3/block/testnet3/26580.block): error trying to connect: Connection reset by peer (os error 104)
0|cargo  | Caused by:
0|cargo  |     0: error trying to connect: Connection reset by peer (os error 104)
0|cargo  |     1: Connection reset by peer (os error 104)
0|cargo  |     2: Connection reset by peer (os error 104)', /root/snarkOS/snarkos/ledger/mod.rs:328:39
0|cargo  | note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
PM2      | App [cargo:0] exited with code [101] via signal [SIGINT]

It's always an HTTP 104 in my case and I need to restart the process constantly to attempt a sync. Same issue across multiple instances.

This seems like a transient sync server issue; how long did it take for the sync process to resume without issues?

It happened constantly, a few times an hour at least. Syncing to completion wasn't even possible until I wrapped the process in PM2 with autorestart. Since the sync server seems to just be proxying an S3 bucket the failure rate seems really high and something worth looking into. Agree that the linked issue #1973 looks like it may have fixed it at least client side. Thanks.

Hi ,@emkman
I met the same issue with error 104, can you provide resolutions step by step?
thanks

@erickingxu I would git pull and make sure you are up to date as this may be fixed. Regardless, if you want to ensure automatic restarts, pm2 makes this very easy. Install nvm if you don't have node on your system.
# curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.2/install.sh | bash
npm install pm2 -g
pm2 start "cargo run --release"

This will ensure the node automatically restarts on any failure

I don't think we've seen it happen in a while now, so I'm assuming it's fixed now. Please feel free to reopen if it ever does.


Slow Network Transfer Speeds (?)

I tested with iperf and I don't see any problem. This must come from the hardware (switch, cable, USB key, etc.)


Add status to deployment chain model

What this PR does / why we need it:
Which issue(s) this PR fixes:
Part of #2964
Does this PR introduce a user-facing change?:

NONE

Nice.
/lgtm

There you go!
/approve


User Authentication

A user should be able to log into an account. The user should not be able to access the listings page until they are logged in. The user should be able to log in with Google, Facebook (Easier), or a Custom Account (Harder). The user's account should have a username (unique), a display name, a location, a profile picture (Hard?), a custom text section (Hard?). The user should be able to log out, and they will be taken back to the landing page. The user should also have an area where they can view their post-interaction history (past viewed, shortlisted, applied).

Work started with commits 59dece7d55c6501d4534a4814ac0a4379e53ef1a and 11eaf504f53600b60fa15520ecaf30e50af4aa55 by Janice.

User auth almost complete, only profile integration left.


build: Use setup-envtest to download later version of envtest

controller-runtime provides a convenient tool to download various versions of k8s. This commit switches to using that to ensure we use a single version for testing in unit tests and in e2e tests, namely v1.21.x right now so to keep the version stable while just changing the installation mechanism.

/lgtm


bomber-go: int at 0.3.4

Description of changes
Tool to scans Software Bill of Materials (SBOMs) for vulnerabilities
https://github.com/devops-kung-fu/bomber
Related to https://github.com/NixOS/nixpkgs/issues/81418

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 198779 run on x86_64-linux 1

1 package built:

bomber-go


Mobile > QR-Code: Scan of an unknown endpoint in org should not be allowed

Issue is in the backend: QA QR-Code is generated using the SCP endpoint instead the QA.
New issue opened.


Favor sdkv2

This PR does two things. The first is that in AwsShim, it changes the behavior to favor AWS SDK v2 if it's in the load path over v1. Presumably if you have both in your load path you want the most recent. The other thing that this PR does is allow you to manually specify which version of the SDK to use through the config object. That would look like:
Humidifier.configure { |config| config.sdk_version = 1 }

Changes Unknown when pulling f8cc73b8830376e0ee8ff7af888f0aed45c7d2dc on favor-sdkv2 into ** on master**.

Changes Unknown when pulling f8cc73b8830376e0ee8ff7af888f0aed45c7d2dc on favor-sdkv2 into ** on master**.

Changes Unknown when pulling f8cc73b8830376e0ee8ff7af888f0aed45c7d2dc on favor-sdkv2 into ** on master**.


Portal does not handle non-text files in templates properly

If we have a non text file added to the template. The portal does not fetch the file correctly.
Resulting

Either the file to be ignored because of which it is not copied over
or
Portal shows error trying to read a binary file as a text file

PR for a template with non text file in it
https://github.com/Azure/azure-webjobs-sdk-templates/pull/86

I recommend we put this in the Current milestone

@davidebbo, does kudu support binary data in files collection when we PUT new function? base64 string?

@alrod yes kudu supports binary data with vfs PUT. you can test it in the kudu console by dragging and dropping any binary file there.

Oops I should have read your whole comment:). No that doesn't work today will need to get kudu to support that.

I don't think that's worth supporting as that would be tricky. Note that the only reason we need this is to work around a temporary issue with restoring NuGet packages. Once addressed, I don't think any of our templates should have binaries.

I agree with you David. This is a bunch of work that is throw-away once proper support for project.json in templates is implemented. Closing.


Site down ?

website not accessible .

If anyone could deploy on heroku or any other service please share the link

Moved to http://issuehub.pro


Connection.upsert for Inserting New Records Using Id as the External ID

I encountered an error METHOD_NOT_ALLOWED: HTTP Method 'PATCH' not allowed. Allowed are GET,HEAD,POST when connection.sobject('Account').upsert({Name: 'foo'}, 'Id').
I found description below in developer guide.
https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/dome_upsert.htm

Inserting New Records Using Id as the External ID
This example uses the POST method as a special case to insert a record where the Id field is treated as the external ID. Because the value of Id is null, it’s omitted from the request. This pattern is useful when you’re writing code to upsert multiple records by different external IDs and you don’t want to request a separate resource. POST using Id is available in API version 37.0 and later.

So if external ID field is Id and Id is not present, HTTP method for upsert should be POST.
These lines:
https://github.com/jsforce/jsforce/blob/a99a38b55346388f53a47d8b2e5afcace43c232b/lib/connection.js#L754
      var url = [ self._baseUrl(), "sobjects", sobjectType, extIdField, extId ].join('/');
      return self.request({
        method : 'PATCH',
        url : url,

should be like:
      var url = _.compact([ self._baseUrl(), "sobjects", sobjectType, extIdField, extId ]).join('/');
      return self.request({
        method : (extIdField.toUpperCase() === 'ID' && extId == null) ? 'POST' : 'PATCH',
        url : url,

I came across this problem, too.
Does someone have any solution?

The upsert is not supporting the ID based upsert. How about using create() if you surely want to insert the new records ? (Personally I'm not sure why these special upsert is existing)

I don't have the code anymore, but I remember monkey patching the internals of jsforce to add this functionality wasn't very hard.

The bulk API supports this use-case.

@stomita Hi, this is the current use-case I am looking to use upsert for:
In my api, I am making a request to update a master object in SalesForce, when this request is made, there may or may not exist a related detail object to this master object. I determine this by making a query to check for that detail object where the Master-Detail field is equal to the master object Id, e.g. SELECT Id FROM DetailObject__c WHERE MasterDetailField__c = 'MasterObjectId'; If no records are returned I am creating that detail object, if records are returned than I know I can update the detail object.
So instead of conditionally checking if there are records returned myself and either making a create or update call, I wanted to use upsert to do it for me, using @tarot 's suggested change, I was able to get this to work:
const record = {
    Id: detailObjectRecordExists ? detailObjectId : null,
    Field1__c: "foo",
    Field2__c: "bar"
};

if (detailObjectRecordExists) {
    record.MasterDetailField__c = MasterObjectId;
}

conn.sobject("DetailObject__c").upsert(record, "Id", function(err, ret) {
    // ...
});

@bsansone OK with your use case which totally may happen.
However still I cannot understand why the API is existing, it still requires switching to call proper http method internally in theupsert() method. The special interface for Id as external ID upsert is not so much needed.)
Anyway, at least JSforce library user would get benefit without thinking the switch, it is worth considering implementation.

Is it possible to simply change the HTTP method from PATCH to POST in this scenario?
const result: RecordResult = await salesforce.sobject('FieldPermissions').upsert(fieldPermissionsToSaveValues, 'Id', {method:POST});

Your payload is missing the external field as such the request URL is breaking - I submitted PR#931
In the mean time try adding a slash to the extIdField field so that it reads:
connection.sobject('Account').upsert({Name: 'foo'}, 'Id/')
or adding Id to your record (doesn't work if Id is empty thought).
connection.sobject('Account').upsert({Name: 'foo', Id:123}, 'Id/')


#641 feat(templates): add swarm attachable networks

Closes #641

@WTFKr0 small update to do

Cheers @WTFKr0 :+1:


Create TranslationBaseName from filename only

I'm  in the last leg of some improvements in Hugo which requires some simplification in the content path logic . This is also looking a little to the future where content will not just be files, and the task of finding the cannonical path of a piece of content should be a simple task.
Given this content:
content
├── foo.en.md
├── foo.jp.md
└── foo.no.md

Currently, the value of .Page.File.TranslationBaseName, depends on the language configuration. If only en is configured, then TranslationBaseName for foo.jp.md will be foo.jp.
In the new setup, every part of the base filename after the first "." separator will be considered identifiers:
 foo.no.md =>  foo + IDENTIFIER_LANGUAGE + IDENTIFIER_EXTENSION

Note that the Page backed by a foo.unknownlanguage.md will still be assigned the defaultContentLanguage.
I'm sure there will be some subtle site breakage in the above, but I'm also sure that we need to do this.
Shout if you disagree.

Currently, with hugo new content/foo.bar.baz.de.md, .File.TranslationBaseName is foo.bar.baz. Will this change? I have seen several sites where the "." separator is used... abundantly.

Will this change? I have seen several sites where the "." separator is used... abundantly.

No, your example will still be foo.bar.baz (if that is how it is today). But, we will treat de as a language identifier even if it's not defined as a language in config.toml.

That said, I would not recommend using the "." ... abundantly.

That said, I don't see us ever adding more identifiers to the filename itself. I have at least one more idea for a role based partition (RBAC), but that fits much better in a mount configuration.

No, this is not happening.


[BEAM-4016] Invoke Setup and TearDown on SplitRestrictionFn and PairWithRestrictionFn

R: @jkff

LGTM, thanks!


3.8.0 - Daily Quest Fail (JSON Client-Error)

General information:
Daily Quests failed with Peacekeeper Daily throwing an error related to JSON.
Cannot deserialize the current JSON array (e.g. [1,2,3]) into type 'RawQuestClass' because the type requires a JSON object (e.g. {"name":"value"}) to deserialize correctly.

To fix this error either change the JSON to a JSON object (e.g. {"name":"value"}) or change the deserialized type to an array or a type that implements a collection interface (e.g. ICollection, IList) like List<T> that can be deserialized from a JSON array. JsonArrayAttribute can also be added to the type to force it to deserialize from a JSON array.

There's also an error regarding to moving a weapon to the Primary Slot. Should be a VSK-94. I believe I was moving the VSK-94 at almost the exact time the quests failed.

Attachments

65988eb30001fbdb6f6af6c3.zip (39 KiB)
server-2024-01-08.log (527 KiB)
2024.01.08_16-44-14_<IP_ADDRESS>.28375 traces.log (296 KiB)
LogOutput.log (109 KiB)

Originally written by Jester_Stacks

Peacekeeper also sent a blank message like the previous error https://dev.sp-tarkov.com/SPT-AKI/Issues/issues/261

Originally written by Jester_Stacks


It seems that dist and source versions are different. There are no mentions of console in source and it is definitely in dist.

Sorry for the long delay, i've rebuilt into dist :)


Completely broken in 1.20.1

Would you mind sending a link to the modpack, or try removing as many mods as you can while still replicating this error?

Unsure how to diagnose this mod without any info on what version, what mods you're playing with, or what mod loader you're using. Closing this issue.


Apache 403 Forbidden under Windows 10

Under Windows, VirtualBox IP address by default is <IP_ADDRESS>.
So, we'll get a 403 Forbidden because it's not appears in .devilbox/etc/apache-2.4/01-vhost-default.conf.
Just include it there as a alias and all stuff will be Ok.
<VirtualHost _default_:80>
	ServerName  localhost
	ServerAlias <IP_ADDRESS>
	ServerAlias 172.16.238.*
        ServerAlias <IP_ADDRESS>
	ServerAlias httpd

Always nice when ticket resolved itself. ;)
That's it!
Note1: devilbox branch master
Note2: I included also devilbox.loc as alias because I put that in C:\Windows\System32\drivers\etc\hosts file

@dany-eudes Thanks for the info.
Isn't it also possible to just add:
<IP_ADDRESS> httpd

in your C:\Windows\System32\drivers\etc\hosts file instead of changing the above vhost settings?

@cytopia, sure it's possible insert httpd alias into hosts file. But I prefer devilbox.loc because it's a so good name! If I were you I'd put that name in vhost conf file and I'd update the docs to make a direction to Windows guys. BTW, great job! Congrats!

@dany-eudes I keep that in mind.
It will not be before end of summer until i will have my own windows machine. So until then I rely on you guys :-) Keep up ticketing for any problems on windows.

@dany-eudes I will have adjust all 4 webserver docker container in order to simply add an IP address to .env as an injectable value

Btw, are you using Docker Toolbox or Docker for Windows

@dany-eudes the whole problematic will be resolved in the upcoming release: https://github.com/cytopia/devilbox/pull/130
The intranet itself is setup to be served from a default virtual host container and should be reachable from any domain (except project domains) pointing to the webserver.
If you want to check it out before the actual release will be merged, follow the instructions described in the above linked pull request.

The above mentioned PR #130 has been merged and should fix this issue.
I have added some information regarding Docker Toolbox: http://devilbox.readthedocs.io/en/documentation/installation/docker-toolbox.html


django 2.0.x support and drop versions reaching EOL

Do you have any news on this ? It would be nice to have an ETA so we can plan our own migration to Django 2.0 :)

If you check the master commits I have fixed many django 1.11/2.0 errors, now you could try the master branch localy and report which errors you are facing

c6daba02cafad522c12bdbe2828d1e0cf8df78bd 1b25e992e68009cf39e4ebcf16976bbd2300e012

Seems to be working fine for me locally. 👍

some works needed to update the tests though :D to make the travis green again! if anyone could check the errors and tell me the fix i will be happily able to implement the code changes :+1:

fixed on master


feat: password visibility toggle button

Description
I will add the password visibility toggle button.
Screenshots
No response
Additional information
Can you please assign this issue to me under gssoc? @subhashis2204

Hello @SINJINI-G nice to have you here. But sorry to say that this issue has already been resolved in one of the PRs waiting to be merged. You can join the #project-annapurna channel on discord where we can find about your skills and find an appropriate issue for you..


Add flutter example

Adds a Flutter example!
Notes

Functionality should be comparable to the vanilla Typescript example

For a comparable "other API company that provides a flutter example" you can look to supabase.

Testing

Tested most extensively on web, tested quite successfully on iOS, and on Android with caveats.

No comments! This look great and I was able to run everything locally without issue 🙌


Add ProjectDTO details API

Adding API to read the details required for ProjectDTO
Partially fixes #1453

@swarajsaaj : sorry, can you solve the conflict plz?

@swarajsaaj : sorry, can you solve the conflict plz?

Done @pascalgrimaud

Thanks for your work here @swarajsaaj :-)))


php extension not working

I am creating a project using php. I use the php intellisense extension to validate stuff but now the extension itself is not working properly. It used to work but now it dosen't work.

Looks like no one wants to respond...


Completed Task 483 : Created New Grid Component

Description:

Add Drag Drop functionality to grid column
Add Resize functionality to grid column
Add Filter functionality to column (for all column or column wise)
Add Search functionality to column (for all column or column wise)
Add Pagination for grid
UI changes for Filter popup for dark and light theme
Soring functionality for column

Screenshots:
Default

With All Search

With All Filter

With Sort

With Custom Filter

With Custom Search

With Pagination

With Wrap Text

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Hello @hemalatagaikwad  please refer RdsCompDataTable component we want structure like wise in this newly created component as data-table is same as grid so once go through it that Component as we are passing props through on interface only.

@Ketan-Khalate I am changing the code and creating a common interface; can you please look into it?

i) Replace Inline css with css classes.
ii) Added custom css.
iii) Added export to interface.
iv) Change the code place accordingly our code.


INBOs name appears on top of map

When you browse to the bird tracking explorer en scroll down, the text "Research Institute Nature and Forest" appears above the map instead of in the header bar.

Will be closed in release 2


Education Page: Eliminate Additional Space

Remove the additional space as observed in smaller screen sizes for the education page

@smaranjitghose can you assign this to me?


Instructions on setting up dynamic volume provisioner

Resolves kubeflow/kubeflow#3774
The current instructions for kfctl_k8s_istio.yaml deployment of Kubeflow lack volume provisioner instructions before Kubeflow's deployment which lead to all of Kubeflow's PVCs to Pending state in Kubernetes clusters not having a dynamic volume provisioner.
This PR updates the instructions for the setup of a dynamic volume provisioner or alternative methods of handling the same issue.
/assign @krishnadurai
/cc @johnugeorge @jlewi

This change is

/lgtm
/cc @sarahmaddox

/unassign @krishnadurai
/assign

Preview: https://deploy-preview-1040--competent-brattain-de2d6d.netlify.com/docs/started/k8s/kfctl-k8s-istio/

Thanks!
/lgtm
/approve


Pip pinned package version dependency issues

PR #514 added many packages to the riscv-isac and riscv-ctg requirements.txt files and pinned them to specific versions. Many of these versions are fairly old, and other python packages depend on newer versions of these dependencies. Even within the riscv environment, riscv-config depends on PyYAML >= 5.2 while riscv-isac requires PyYAML == 5.1.1.
If these tools are both installed simultaneously using pip, then pip generates the following error message:
ERROR: Cannot install riscv-config and riscv-isac because these package versions have conflicting dependencies.

The conflict is caused by:
    riscv-isac 0.18.0 depends on pyyaml
    riscv-isac 0.18.0 depends on PyYAML==5.1.1
    riscv-config 3.18.3 depends on pyyaml==5.2

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip to attempt to solve the dependency conflict

Even in the CI logs for this repo, the following error is generated (though the CI still passes):
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
riscv-config 3.18.3 requires pyyaml==5.2, but you have pyyaml 5.1.1 which is incompatible.
pipx 1.7.1 requires packaging>=20, but you have packaging 19.0 which is incompatible.
httplib2 0.20.2 requires pyparsing!=3.0.0,!=3.0.1,!=3.0.2,!=3.0.3,<4,>=2.4.2, but you have pyparsing 2.4.0 which is incompatible.

The simplest fix is to switch the dependencies to be minimum required versions instead of exact versions, which I did in PR #519.

Nice work @jordancarlin .
Appreciated.

Approved your Pull request. Will close this issue once your PR is merged (after passing the CI)

Perfect. Thank you for the prompt response.

@jordancarlin there is one more thing which can be done i.e, to split rv32 and rv64 tests in 2 separate runs to achieve some parallelization. Are you available for running a quick update and making another PR?

Sure. I'll update the workflow matrix to run them as two separate jobs.

Great. Thanks


fix test failure.

fix spec/features/cms/workflow/search_approvers_spec.rb failure.

Coverage remained the same at 81.58% when pulling cc13685e3ed8b669a40296deffce147fa514ede8 on sunny4381:fix-test-error into 4558770f70a49f078ecef4cab3a326ac5291ceb2 on shirasagi:master.


new-log-viewer: Implement log-level filtering.

References
new-log-viewer series: #45 #46 #48 #51 #52 #53 #54 #55 #56 #59 #60 #61 #62
Description

Implemented buildIdx() to do deserialization in JSONL decoder. Previously, deserialization was done inside constructor. In addition, now we are parsing log level inside in buildIdx() and not in decode().
Implement log level filtering with JSON logs.
TODO: CLP IRv1 log-level filtering is to be added once the equivalent is implemented in clp-ffi-js. The implementation in clp-ffi-js will start once we agree with the decoder handling in this PR.

Validation performed

Referred to #46 , launched debug server.
Referred to #54, tested page switching with Monaco editor with filters off.
Referred to #54, tested page switching with single (TRACE) log level filter.
Referred to #54, tested page switching with multiple (TRACE, DEBUG) log level filter.

@davemarco I haven't been able to take a careful look into the code yet, but it seems there's one issue whenever I was switching the log level filter from ["TRACE"] to ["DEBUG"] - there are exactly 200 [Renderer -> MainWorker] code=loadPage requests sent to the service worker, and occasionally the logEventNum does not update on editor cursor updates. Could you take a look before we proceed to the review?
Here's an example file I used -
example.zip
pageSize was set to 10 and I started from http://localhost:3010/?filePath=http://localhost:3010/test/example.jsonl#logEventNum=10

Thanks for the additional comments, @junhaoliao. Since this PR ended up being quite large and we are reworking the design, @davemarco is going to submit smaller PRs rather than trying to refactor this one completely.

@kirkrodrigues @davemarco i think we can close this now, right?


add in_max_unit

Allows to get a gross estimation of the time, as you would in a
conversation. For example: 1 year, 3 months, etc.
Signed-off-by: Dimid Duchovny<EMAIL_ADDRESS>

@dimidd  Does this solve the problem you are looking for ? https://github.com/tmlee/time_difference/pull/15

No, sorry. I'm looking for a very gross estimation, e.g. "2 years", instead of "2 years 3 months, 5 days".

In my view, this library should return the results "as-is" as much as possible with as little surprise. The answers obtained from the method should cross match how a machine calculates time difference (like Wolfram Alpha). Further manipulation can be done at the next level of abstraction.


Handle out of bounds and restricted NCRs.

This is in response to unbescape/unbescape-tests#1 and fixes those failing tests.
Fixes unbescape/unbescape#7.

Looks great Brandon, thanks so much.
In order to merge, I need you to please read the Guidelines for Contributing at https://github.com/unbescape/unbescape/blob/master/CONTRIBUTING.markdown and explicitly state (a comment here will suffice) that you agree with those terms.
Also, if you want to be added to http://www.unbescape.org/team.html as a contributor (see some examples here http://www.attoparser.org/team.html and here http://www.thymeleaf.org/team.html) I would need from you a location (country -- U.S.A I suppose) and (optionally) a non-corporate email.

Yes, I've read and agree to those terms. My email address is already in the commit messages and my location is the United States as you have said.

Merged, and there you are: http://www.unbescape.org/team.html
This will be included in Unbescape 1.1.1.RELEASE

FYI, version 1.1.1.RELEASE has just been published and pushed to the maven central repository.


8378 Handle uploading malformed tar package

Fixes #8378
Catch EOFError when uploading malformed tar package. Unit test uses the contents of the test tar.gz provided in the original ticket.

Looks like a false negative on the CI failure, probably just needs re-ran.

Thanks @nicholasserra!


Fix typo

github -> GitHub

@changeworld : Thanks for your contribution! The author, @wesmc7777, has been notified to review your proposed change.

Editorial fix verified.


"More..." link when search is multiple-page scrolls-to-top on-click

"More..." link when search is multiple-page scrolls-to-top on-click.  It should maintain the current page scroll.

Closed by 24bd3cd.


ocamlPackages.plotkicadsch: init at 0.9

Description of changes
Also added ocamlPackages.git to the propagatedBuildInputs of ocamlPackages.git-unix to allow plotkicadsch to build properly.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 189045 run on x86_64-linux 1

3 packages marked as broken and skipped:

ocamlPackages.irmin-containers
ocamlPackages.irmin-unix
ocamlPackages.wodan-irmin

2 packages failed to build:

ocamlPackages.kicadsch
ocamlPackages.plotkicadsch

4 packages built:

ocamlPackages.git-unix
ocamlPackages.irmin-git
ocamlPackages.irmin-http
ocamlPackages.irmin-mirage-git

Result of nixpkgs-review pr 189045 run on x86_64-linux 1

3 packages marked as broken and skipped:

ocamlPackages.irmin-containers
ocamlPackages.irmin-unix
ocamlPackages.wodan-irmin

6 packages built:

ocamlPackages.git-unix
ocamlPackages.irmin-git
ocamlPackages.irmin-http
ocamlPackages.irmin-mirage-git
ocamlPackages.kicadsch
ocamlPackages.plotkicadsch

Result of nixpkgs-review pr 189045 run on x86_64-linux 1

3 packages marked as broken and skipped:

ocamlPackages.irmin-containers
ocamlPackages.irmin-unix
ocamlPackages.wodan-irmin

6 packages built:

ocamlPackages.git-unix
ocamlPackages.irmin-git
ocamlPackages.irmin-http
ocamlPackages.irmin-mirage-git
ocamlPackages.kicadsch
ocamlPackages.plotkicadsch

Result of nixpkgs-review pr 189045 run on x86_64-linux 1

3 packages marked as broken and skipped:

ocamlPackages.irmin-containers
ocamlPackages.irmin-unix
ocamlPackages.wodan-irmin

6 packages built:

ocamlPackages.git-unix
ocamlPackages.irmin-git
ocamlPackages.irmin-http
ocamlPackages.irmin-mirage-git
ocamlPackages.kicadsch
ocamlPackages.plotkicadsch

Result of nixpkgs-review pr 189045 run on x86_64-linux 1

3 packages marked as broken and skipped:

ocamlPackages.irmin-containers
ocamlPackages.irmin-unix
ocamlPackages.wodan-irmin

6 packages built:

ocamlPackages.git-unix
ocamlPackages.irmin-git
ocamlPackages.irmin-http
ocamlPackages.irmin-mirage-git
ocamlPackages.kicadsch
ocamlPackages.plotkicadsch


lightgbm still can not do multiclass?

look! I got 3 classes,but when the native lightgbm builting booster,it always throw a error to me!

this method throw the error ,because the numclass it calculateing is always 1?
is this a lightgbm's bug?or a mmlspark's?
why did still no one try to fix it?
is there some way to work around to let me build a normal booster for multiclass task?

@puyvqi sorry about the trouble you are having, indeed, there is already an issue for this, it is an enhancement I am planning to add in the future: https://github.com/Azure/mmlspark/issues/342
There was a recent change that allowed you to set the object which wasn't exposed before, and I think it's partly due to this that the issue has become worse, since you are allowed to explicitly set multiclass objective for the classifier.
However, since LightGBMClassifier follows spark ML's API, you could use OneVsRest model to make the binary classifier a multiclass classifier:
https://spark.apache.org/docs/latest/api/scala/index.html#org.apache.spark.ml.classification.OneVsRest
But this won't be as optimal as using the native multiclass classifier from lightgbm.


map of AuthProperties pointers causes all parsed Workflows to have the same AuthProperties

What happened:
When parsing multiple Workflows with Bearer Auth, the tokens all end up the same.
What you expected to happen:
Each Workflow would have the bearer token specified in the Workflow YAML/JSON.
How to reproduce it:
Parse multiple Workflow files into Workflow models and print the pointer address of workflow.Auth.Properties and they will all be the same.
Anything else we need to know?:
Environment:

Specification version used:
2.1.1 and 2.2.2
Go version:
1.16

Hi @calvinmclean, thanks for reporting this, I'll be taking a look on this next week.

Hi @spolti, thanks for your quick reply. I have a PR open now with a simple fix.


Changing a package description?

Is there a way to modify an existing package description (to improve its wording) without building a new release?

Nope. You have to release a new version.
On Wed, 13 Dec 2023 at 14:48, Davin Church @.***> wrote:

Is there a way to modify an existing package description (to improve its wording) without building a new release?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks. It would be nice to be able to make minor adjustments like that, but I guess that's too difficult.

It's not difficult, but it's difficult to draw the line. What should be allowed, what shouldn't?
Also, when I download a version and compare the ZIP with the very same version downloaded yesterday, I expect them to be equal, and allowing something like this would make that impossible.


This is empty-commit to retrigger-build

Motivation
[Describe why this change is needed]
Modifications
[Describe the code changes]
Result
[Describe how the changes affects existing behavior and how to test it]
Related Issues
[Resolves #123]

/test images

/lgtm

/approve


Race condition when running two builds with the same registry, image name and tag

Running two same builds results in
2015-04-01 04:13:30,841 - dock.build - INFO - push built image to registry
2015-04-01 04:13:30,841 - dock.core - INFO - tag and push image
2015-04-01 04:13:30,841 - dock.core - DEBUG - image = 'user/image', target_image_name = 'user/image', reg_uri = 'registry.example.org', tag = ''
2015-04-01 04:13:30,841 - dock.core - INFO - tag image
2015-04-01 04:13:30,841 - dock.core - DEBUG - image = 'user/image', target_image_name = 'user/image', reg_uri = 'registry.example.org', tag = ''
Traceback (most recent call last):
  File "/usr/bin/dock", line 9, in <module>
    load_entry_point('dock==1.1.1', 'console_scripts', 'dock')()
  File "/usr/lib/python2.7/site-packages/dock/cli/main.py", line 187, in run
    cli.run()
  File "/usr/lib/python2.7/site-packages/dock/cli/main.py", line 170, in run
    args.func(args)
  File "/usr/lib/python2.7/site-packages/dock/cli/main.py", line 66, in cli_inside_build
    build_inside(input=args.input, input_args=args.input_arg, substitutions=args.substitute)
  File "/usr/lib/python2.7/site-packages/dock/inner.py", line 215, in build_inside
    image = dbw.build_docker_image()
  File "/usr/lib/python2.7/site-packages/dock/inner.py", line 152, in build_docker_image
    self.builder.push_built_image(target_registry, insecure=self.target_registries_insecure)
  File "/usr/lib/python2.7/site-packages/dock/build.py", line 159, in push_built_image
    response = self.tasker.tag_and_push_image(self.image, local_registry, registry, tag=self.tag, insecure=insecure)
  File "/usr/lib/python2.7/site-packages/dock/core.py", line 394, in tag_and_push_image
    final_tag = self.tag_image(image, target_image_name, reg_uri=reg_uri, tag=tag)
  File "/usr/lib/python2.7/site-packages/dock/core.py", line 355, in tag_image
    response = self.d.tag(image, repository, tag=tag, force=force)  # returns True/False
  File "/usr/lib/python2.7/site-packages/docker/client.py", line 897, in tag
    self._raise_for_status(res)
  File "/usr/lib/python2.7/site-packages/docker/client.py", line 95, in _raise_for_status
    raise errors.APIError(e, response, explanation=explanation)
docker.errors.APIError: 409 Client Error: Conflict ("Conflict: Tag latest is already set to image 38b331ae434a3490e76ab6d10979a53ffb8ef003991d39fb253c2dbcb1bd2f59, if you want to replace it, please use -f option")

I suppose what happened is that build1 tagged the image and build2 tried to tag the image with the same tag before build1 deleted the tag.
I'm not sure what the expected behavior should be - if we force-tag in the second build then one of the builds will fail because it will try to delete nonexistent tag. We'd have to make the tag & push sequence atomic somehow ...
Found by @goern.

maybe prefix the image name with the build id? so that concurrent builds dont collide and let have each build job upload the image to the registry... so that the faster wins and makes the result a little more unpredictable but working aka producing images in a registry

You can explicitely ask for latest tag, then it won't fail (but one of your images will be lost, pretty much).
With prod strategy we are constructing image names that we put timestamps inside:
timestamp = datetime.datetime.now().strftime('%Y%m%d%H%M%S')
image = "%s/%s:%s-%s" % (self.user, self.component, self.koji_target, timestamp)

Maybe we could solve this by applying unique tag, e.g. ":ttomecek-20150402-133759357" (and then tag it according to user request -- and if that fails, we will inform user and offer our tag).

this is already solved in whole osbs combo


ManagedCluster is not sync to hub of hubs

I deployed the hub of hubs, with the hub of hubs cluster and 1 leaf hub, which has 5 managed clusters.
In the beginning, I can deploy the policies from the hub of hubs and the status was able to sync back
to the hub of hubs.
After I imported the 6th managed cluster and redeploy a policy, the new imported cluster's status
can't be synced back.
After some looking, I can see the following on hub-of-hubs-status-transport-bridge from hub of hubs
{"level":"info","ts":1631807423.040543,"logger":"policies-transport-to-db-syncer","msg":"start handling 'ClustersPerPolicy' bundle","Leaf Hub":"hub1","Generation":10}
{"level":"error","ts":1631807423.0506718,"logger":"policies-transport-to-db-syncer","msg":"failed to handle bundle","error":"failed handling clusters per policy bundle - rescheduling bundle 'ClustersPerPolicy' from leaf hub 'hub1', generation 10 - cluster 'ec2-micorshift' \n\t\t\t\t\tdoesn't exist","stacktrace":"github.com/go-logr/zapr.(*zapLogger).Error\n\t/go/pkg/mod/github.com/go-logr/zapr@v0.2.0/zapr.go:132\ngithub.com/open-cluster-management/hub-of-hubs-status-transport-bridge/pkg/controller/dbsyncer.(*PoliciesTransportToDBSyncer).syncBundles.func1\n\t/workspace/hub-of-hubs-status-transport-bridge/pkg/controller/dbsyncer/policies_transport_to_db_syncer.go:126"}

On the leaf hub, I can see the following, you can see the 6th cluster ec2-microshift managed cluster
is sent to hub via transport laywer... Can you please let me know how to actually debug this?
2021/09/16 18:20:16 ***************
2021/09/16 18:20:16
{"objects":[{"kind":"ManagedCluster","apiVersion":"cluster.open-cluster-manageme
nt.io/v1","metadata":{"name":"cluster3","uid":"e1dcbaba-1349-4d54-83a1-45e2a3ed4
1b6","resourceVersion":"321872","creationTimestamp":"2021-09-15T17:49:05Z","labe
ls":{"name":"cluster3","vendor":"Kind"},"annotations":{"open-cluster-management/
created-via":"other"}},"spec":{"hubAcceptsClient":true,"leaseDurationSeconds":60
},"status":{"conditions":[{"type":"HubAcceptedManagedCluster","status":"True","l
astTransitionTime":"2021-09-15T17:49:05Z","reason":"HubClusterAdminAccepted","me
ssage":"Accepted by hub cluster
admin"},{"type":"ManagedClusterConditionAvailable","status":"True","lastTransiti
onTime":"2021-09-15T17:49:36Z","reason":"ManagedClusterAvailable","message":"Man
aged cluster is
available"},{"type":"ManagedClusterJoined","status":"True","lastTransitionTime":
"2021-09-15T17:49:36Z","reason":"ManagedClusterJoined","message":"Managed
cluster
joined"}],"capacity":{"core_worker":"0","cpu":"8","memory":"32115Mi","socket_wor
ker":"0"},"allocatable":{"cpu":"8","memory":"32115Mi"},"version":{"kubernetes":"
v1.19.1"},"clusterClaims":[{"name":"id.k8s.io","value":"cluster3"},{"name":"kube
version.open-cluster-management.io","value":"v1.19.1"},{"name":"platform.open-cl
uster-management.io","value":"Other"},{"name":"product.open-cluster-management.i
o","value":"Other"}]}},{"kind":"ManagedCluster","apiVersion":"cluster.open-clust
er-management.io/v1","metadata":{"name":"cluster4","uid":"9ef93cb5-bda7-46ab-8c5
3-3868289eaf99","resourceVersion":"320858","creationTimestamp":"2021-09-15T17:49
:18Z","labels":{"name":"cluster4","vendor":"Kind"},"annotations":{"open-cluster-
management/created-via":"other"}},"spec":{"hubAcceptsClient":true,"leaseDuration
Seconds":60},"status":{"conditions":[{"type":"HubAcceptedManagedCluster","status
":"True","lastTransitionTime":"2021-09-15T17:49:18Z","reason":"HubClusterAdminAc
cepted","message":"Accepted by hub cluster
admin"},{"type":"ManagedClusterJoined","status":"True","lastTransitionTime":"202
1-09-15T17:49:47Z","reason":"ManagedClusterJoined","message":"Managed cluster
joined"},{"type":"ManagedClusterConditionAvailable","status":"True","lastTransit
ionTime":"2021-09-15T17:49:48Z","reason":"ManagedClusterAvailable","message":"Ma
naged cluster is
available"}],"capacity":{"core_worker":"0","cpu":"8","memory":"32115Mi","socket_
worker":"0"},"allocatable":{"cpu":"8","memory":"32115Mi"},"version":{"kubernetes
":"v1.19.1"},"clusterClaims":[{"name":"id.k8s.io","value":"cluster4"},{"name":"k
ubeversion.open-cluster-management.io","value":"v1.19.1"},{"name":"platform.open
-cluster-management.io","value":"Other"},{"name":"product.open-cluster-managemen
t.io","value":"Other"}]}},{"kind":"ManagedCluster","apiVersion":"cluster.open-cl
uster-management.io/v1","metadata":{"name":"cluster0","uid":"70e569d9-32b7-45d0-
bd04-50332657b1fb","resourceVersion":"322423","creationTimestamp":"2021-09-15T17
:48:23Z","labels":{"name":"cluster0","vendor":"Kind"},"annotations":{"open-clust
er-management/created-via":"other"}},"spec":{"hubAcceptsClient":true,"leaseDurat
ionSeconds":60},"status":{"conditions":[{"type":"HubAcceptedManagedCluster","sta
tus":"True","lastTransitionTime":"2021-09-15T17:48:24Z","reason":"HubClusterAdmi
nAccepted","message":"Accepted by hub cluster
admin"},{"type":"ManagedClusterJoined","status":"True","lastTransitionTime":"202
1-09-15T17:48:50Z","reason":"ManagedClusterJoined","message":"Managed cluster
joined"},{"type":"ManagedClusterConditionAvailable","status":"True","lastTransit
ionTime":"2021-09-15T17:48:50Z","reason":"ManagedClusterAvailable","message":"Ma
naged cluster is
available"}],"capacity":{"core_worker":"0","cpu":"8","memory":"32115Mi","socket_
worker":"0"},"allocatable":{"cpu":"8","memory":"32115Mi"},"version":{"kubernetes
":"v1.19.1"},"clusterClaims":[{"name":"id.k8s.io","value":"cluster0"},{"name":"k
ubeversion.open-cluster-management.io","value":"v1.19.1"},{"name":"platform.open
-cluster-management.io","value":"Other"},{"name":"product.open-cluster-managemen
t.io","value":"Other"}]}},{"kind":"ManagedCluster","apiVersion":"cluster.open-cl
uster-management.io/v1","metadata":{"name":"cluster1","uid":"20a2f29a-fcc2-43fb-
bf94-110ac169785c","resourceVersion":"321230","creationTimestamp":"2021-09-15T17
:48:36Z","labels":{"name":"cluster1","vendor":"Kind"},"annotations":{"open-clust
er-management/created-via":"other"}},"spec":{"hubAcceptsClient":true,"leaseDurat
ionSeconds":60},"status":{"conditions":[{"type":"HubAcceptedManagedCluster","sta
tus":"True","lastTransitionTime":"2021-09-15T17:48:37Z","reason":"HubClusterAdmi
nAccepted","message":"Accepted by hub cluster
admin"},{"type":"ManagedClusterConditionAvailable","status":"True","lastTransiti
onTime":"2021-09-15T17:49:02Z","reason":"ManagedClusterAvailable","message":"Man
aged cluster is
available"},{"type":"ManagedClusterJoined","status":"True","lastTransitionTime":
"2021-09-15T17:49:02Z","reason":"ManagedClusterJoined","message":"Managed
cluster
joined"}],"capacity":{"core_worker":"0","cpu":"8","memory":"32115Mi","socket_wor
ker":"0"},"allocatable":{"cpu":"8","memory":"32115Mi"},"version":{"kubernetes":"
v1.19.1"},"clusterClaims":[{"name":"id.k8s.io","value":"cluster1"},{"name":"kube
version.open-cluster-management.io","value":"v1.19.1"},{"name":"platform.open-cl
uster-management.io","value":"Other"},{"name":"product.open-cluster-management.i
o","value":"Other"}]}},{"kind":"ManagedCluster","apiVersion":"cluster.open-clust
er-management.io/v1","metadata":{"name":"cluster2","uid":"998aaf43-99db-4139-a99
7-94bf287c0013","resourceVersion":"319675","creationTimestamp":"2021-09-15T17:48
:51Z","labels":{"name":"cluster2","vendor":"Kind"},"annotations":{"open-cluster-
management/created-via":"other"}},"spec":{"hubAcceptsClient":true,"leaseDuration
Seconds":60},"status":{"conditions":[{"type":"HubAcceptedManagedCluster","status
":"True","lastTransitionTime":"2021-09-15T17:48:52Z","reason":"HubClusterAdminAc
cepted","message":"Accepted by hub cluster
admin"},{"type":"ManagedClusterJoined","status":"True","lastTransitionTime":"202
1-09-15T17:49:16Z","reason":"ManagedClusterJoined","message":"Managed cluster
joined"},{"type":"ManagedClusterConditionAvailable","status":"True","lastTransit
ionTime":"2021-09-15T17:49:16Z","reason":"ManagedClusterAvailable","message":"Ma
naged cluster is
available"}],"capacity":{"core_worker":"0","cpu":"8","memory":"32115Mi","socket_
worker":"0"},"allocatable":{"cpu":"8","memory":"32115Mi"},"version":{"kubernetes
":"v1.19.1"},"clusterClaims":[{"name":"id.k8s.io","value":"cluster2"},{"name":"k
ubeversion.open-cluster-management.io","value":"v1.19.1"},{"name":"platform.open
-cluster-management.io","value":"Other"},{"name":"product.open-cluster-managemen
t.io","value":"Other"}]}},{"kind":"ManagedCluster","apiVersion":"cluster.open-cl
uster-management.io/v1","metadata":{"name":"ec2-microshift","uid":"eef218d7-8c97
-469a-adb2-947da1db57c7","resourceVersion":"1009486","creationTimestamp":"2021-0
9-16T18:18:41Z","labels":{"cloud":"auto-detect","name":"ec2-microshift","vendor"
:"auto-detect"},"annotations":{"open-cluster-management/created-via":"other"}},"
spec":{"hubAcceptsClient":true,"leaseDurationSeconds":60},"status":{"conditions"
:[{"type":"HubAcceptedManagedCluster","status":"True","lastTransitionTime":"2021
-09-16T18:18:42Z","reason":"HubClusterAdminAccepted","message":"Accepted by hub
cluster
admin"},{"type":"ManagedClusterJoined","status":"True","lastTransitionTime":"202
1-09-16T18:20:13Z","reason":"ManagedClusterJoined","message":"Managed cluster
joined"},{"type":"ManagedClusterConditionAvailable","status":"True","lastTransit
ionTime":"2021-09-16T18:20:13Z","reason":"ManagedClusterAvailable","message":"Ma
naged cluster is
available"}],"capacity":{"core_worker":"0","cpu":"4","memory":"15830Mi","socket_
worker":"0"},"allocatable":{"cpu":"4","memory":"15730Mi"},"version":{"kubernetes
":"v1.20.1"}}}],"leafHubName":"hub1","generation":21}
2021/09/16 18:20:16 ***************

2021/09/16 18:26:51
{"objects":[{"policyId":"bba3170e-3c5c-43ca-a17e-501df55d8e3e","clusters":["clus
ter0","cluster1","cluster2","cluster3","cluster4","ec2-microshift"],"remediation
Action":"enforce","resourceVersion":"1017857"}],"leafHubName":"hub1","generation
":14}
2021/09/16 18:26:51 ***************
2021/09/16 18:26:51 ***************
2021/09/16 18:26:51
{"objects":[{"policyId":"bba3170e-3c5c-43ca-a17e-501df55d8e3e","nonCompliantClus
ters":[],"unknownComplianceClusters":["cluster0","cluster1","cluster2","cluster3
","cluster4","ec2-microshift"],"resourceVersion":"1017857"}],"leafHubName":"hub1
","baseBundleGeneration":14,"generation":24}

hi @ianzhang366
was it able to recover from this issue eventually?
the straight forward way to debug it would be to check in the DB if the new cluster was inserted.. if not then clusters per policy should fail by design (since one of the clusters doesn't exist).
and if it doesn't exist in DB, need to continue the investigation and check why it doesn't exist.. it seems from LH logs that it was sent to transport bridge so the question is why transport bridge didn't process it properly.
let me explain what you were seeing:

in Leaf-hub-status-sync, all bundles are sent periodically from LH to HoH.
more specifically different controllers (MCs and Policies) send their bundles independently, so the order is not guaranteed.
(and anyway in sync service order is not guaranteed, but that doesn't matter for this discussion).
in Status-Transport-bridge side, we tried to avoid using foreign keys in the DB due to performance issues, and therefore we check in our code if the clusters exist when trying to process the clusters per policy bundle. if one (or more) of the clusters still doesn't exist, we retry the bundle processing after waiting for 1 second (assuming in the meantime MC bundle will arrive and get processed).
this was the most naive and simple initial implementation.
this is causing in some cases for clusters per policy bundle to be handled before (or in parallel) to MCs bundle which may cause the errors you are seeing in the logs.

I finished coding the new implementation of status-transport-bridge (new design) and all is left is to test things before being able to run this version with more confidence.
the new implementation was designed with this issue in mind.
it still can happen in cases where bundles arrive in the wrong order but the new implementation minimizes the number of times it will happen. if we know in advance it's going to fail due to dependencies chain, we do not start processing the bundle until it's dependencies bundles are processed.
in addition the new impl guarantees we won't process in parallel two bundles from the same (LH + type) which also reduces num of failures.
I will be happy if instead of debugging the old code (which was changed significantly), you will be able to wait few more days until I finish testing new code and then try to repeat the above scenario.

@ianzhang366  and of course I didn't mention in previous message, in case it still happens in new version I'd be happy if you can debug and solve :)

@nirrozenbaum thanks for your response, I think we can wait for the new code and debug from there.
For this one, I checked the DB, there's no entry for the 6th cluster... meaning LH sends a message to CSS, but somehow the HH bridge didn't process the message from LH correctly... I will try to see if I can find which part of the bridge is failing while you preparing the new code base as an exercise.

FYI, after my cluster rebooted from hibernation... the status was synced back to HH correctly...

hi @ianzhang366
I finished implementing the new design including a basic sanity test of the new version.
we're going to start again running the simulations in order to test it in large scale scenarios.
in case you want to start using it as well, you can use the following image:
nirrozenbaumibm/hub-of-hubs-status-transport-bridge:latest-dev

of course all code was updated in our git repo as usual.

hi @ianzhang366,
we found another minor bug in LH that you may have ran into in your scenario.
I've fixed that bug both in LH-status-sync and in the new status-transport-bridge.
please restart your LH-status-sync to grab latest bug fix and use the new version of status-transport-bridge which contains the bug fix in that HoH side as well..
we've tested the new version in our simulation env and were able to verify correctness and performance.

can't reproduce this on the latest build, will close for now


Layout misplaced.

To reproduce:

Launch YT app

Watch any Shorts, either Shorts in Home feeds or Shorts in Shorts tab is fine.

Leave the Shorts, back to Home feeds.

Click and watch any video in Home feeds.

The buttons of YouMute and YouQuality are misplaced.

Tested on YouTube 18.34.5 and 18.35.4.
Environment:
iPhone 11 Pro Max
iOS 14.8
unc0ver 8.0.2
Images:

Fixed in 1.1.1

Thanks ❤


Send Input to Stereo Matrix not triggering

We're running Avantis firmware v1.25 and Companion 3, but can't get this module to adjust the send level on an input routed to a Stereo Matrix.
We've got it configured as per the screen shot below but there's no change on the console. We've checked both IP address and MIDI channel configuration to ensure they match.

Hi, @jameskirsop , i am busy upgrading this to support v3. so will have a look at that.


Enrich ctkSearchBox API

@finetjul Hi J2, I am looking into using the ctkSearchBox and would like to be able to change the icons used for the search and clear actions. Could you review my feature branch adding the necessary API and comment on it? Thanks!

The feature branch can be found here
https://github.com/saschazelzer/CTK/tree/133-enrich-ctksearchbox-api

If you do not have any objections, I would merge these changes into master.

Good with me.


wolfssl: backport patch to fix tests

This fixes the following build failure I get locally:

ERROR - tests/api.c line 5684 failed with:
expected: test_ssl_memio_do_handshake(&test_ctx, 10, ((void *)0)) == (1)
result:   0 != 1
passed (  0.00945)
845: test_dtls13_bad_epoch_ch                            : skipped
846: test_wolfSSL_dtls13_null_cipher                     : skipped
847: test_dtls_msg_from_other_peer                       :Client message: hello wolfssl!
Server response: I hear you fa shizzle!
passed (  0.00186)
848: test_dtls_ipv6_check                                : passed (  0.00008)
849: test_stubs_are_stubs                                : passed (  0.00000)
FAILURES:
24: test_wolfSSL_CertManagerCheckOCSPResponse

This fix is not included in any released version of wolfssl, so this change is orthogonal to #264440.

Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

23.05 is also affected: https://hydra.nixos.org/build/240244035
but this upstream patch won't apply


Question about extending a message with an auxiliary field

I have two message definitions:
syntax = "proto3";
message Inner {
  uint32 f = 1;
}
message Outer {
  Inner inner = 1;
}

And I want to extend Inner with an auxiliary field refcount existing only at C++ code level, that is without changing on-wire format. Using PB_BIND
#include "generated/test.pb.h"
struct Wrapped : Inner {
  int refcount;
};
PB_BIND(Inner, Wrapped, AUTO)

makes Wrapped with the field I want, but Outer still uses the original Inner.
Is there a way to make the type of the Outer.innerfield also Wrapped?

Hmm, good question. It would certainly make sense to have a way to do this.
But currently I'm not sure if there is one, short of rewriting whole struct Outer definition manually and binding that.
It would even make sense to have a more straightforward way to add custom fields to a generated struct.