support win32

First of all, thanks so much for making this project available - there is nothing quite like this out there for c language.
Have you thought about supporting windows ?  I would be happy to work on this, as I need it for one of my projects.
Cheers,
Aaron

Hi boxerab,
I didn't think of supporting windows since I have used this only on Linux. Anyhow if you  implement this for Windows I can merge it after review. The main thing I want to avoid is having too many windows specific things mixed in the code. But if you manage to solve it in an elegant way then I would be happy if you implement it :) Oh yeah, and as long as the API remains platform agnostic!

Thanks! I've decided to move my library to C++11, so I can just use the threading baked into C++.
Closing.


Add checking if given attributes are variant attributes in ProductVariantCreate mutation

If attributes in mutation input are not in product_type.variant_attributes -> raise Validation Error. This is fixing the situation where given attribute is not actually variant attribute.
If attributes in mutation input are in product_type.variant_attributes and product_type.has_variants=False-> raise Validation Error. This is fixing the situation where attributes input is not validated and raw data is passed to further logic.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2873 'str' object has no attribute 'input_type'


Future plans for Code Contracts

This issue is meant to provide a single place where we can keep people notified about the status of the Code Contracts project. We (Microsoft) now have some resources and will do the work (in conjunction with the community of course) to update the sources so the tools will work in VS2015. After that, we are looking to transition the repo outside of the Microsoft organization in GitHub and turn ownership over to any community members that are interested.
As far as we are aware, the current work items are:

Updating the rewriter to understand the new Roslyn code-generation patterns for iterators/async.
Updating the Visual Studio integration so that the Code Contracts pane appears in the project properties window.
Fixing some other missing files, etc.

Separately, there is a Roslyn discussion (https://github.com/dotnet/roslyn/issues/119) about adding contracts to a future version of C#.

There were some work done by the community:
Point 1. is partially addressed by #52.
Point 2. is resolved by #36.
Missing repository content for point 3 is listed in #6.

This is great news! The only obstacle for me to start using VS2015(RC) are the code contracts. Currently almost all my projects use it. Can't wait for the release.

... we are looking to transition the repo outside of the Microsoft organization in GitHub and turn ownership over to any community members that are interested.

What's the reasoning for this?

Possibly related to the "next step" for code contracts being stated as "re-architecturing to integrate Z3" in this document (CodeContracts.pptx).


Unable to add filter

I want to add a simple filter. When I add the filter I get the following error:
Exception: Microsoft.Data.OData.ODataException
Message: The type '' is not supported when converting to a URI literal.
My code is:
var premise =
await
client.Container.For()
.Key(contractId)
.NavigateTo()
.Filter(p => p.Company == this.Company)
.FindEntryAsync();
When I remove the filter it is working.
I have looked at the examples and I don't see what I'm doing wrong.

Can you try assigning this.Company to a local variable "companyName" prior this statement, to the filter expression will look like "p => p.Company == companyName". I suspect the error might have something to do with "this" qualifier.

Yes that was the problem! It works now!
I do have one question. Why don't you support the following? Is that incorrect OData?
await client.Container.For<Account>()
                           .Key(accountID)
                           .Filter(p => p.Company == "contoso")
                           .FindEntryAsync();

The following error is thrown:
if (command.HasKey && command.HasFilter)
                throw new InvalidOperationException("OData filter and key may not be combined.");

I will fix handling of "this.", it should be possible to use it in the expression.
Regarding your other question. OData protocol defines different URI for key-based and filter-based queries, and they can not be combined. By definition key based lookup is the one that results in 0 or 1 result, so if you want to run a non-key based request you should rewrite your call to only use Filter.

Ok thanks!

"this" qualifier is fixed. Will be released in a few days.


custom_bootstrap_compilation not documented

I get prompted custom_bootstrap_compilation [n]:  but I cannot find into the options documentation (http://cookiecutter-django.readthedocs.io/en/latest/project-generation-options.html) what features this enables.

@vladcalin thanks for pointing this out: For historical reference on the issue, see #1181. As far as the docs are concerned, you're right, we don't have this feature documented -- I will address this in a new issue.

@bwarren2 thank you, I've left a note in the PR.

I don't see a note :( .  (I have been getting a lot of Github instability this morning, so it might have dropped?)

@bwarren2 a review note :)


Model Server, No Application Software Template

What does this PR do?:

creates a new software template for model server, no application
pulls in the latest vLLM version 0.6.4 to support the model ibm-granite/granite-3.0-8b-instruct
vllm server containerfile is at https://github.com/maysunfaisal/llm-on-openshift/tree/vllm-0.6.4/llm-servers/vllm/gpu
vllm server image hosted at https://quay.io/repository/redhat-ai-dev/vllm-openai-ubi9?tab=tags with tag v0.6.4
provides new template card techdocs since it does not have an application
this template has no

application support, as a result no image registry
llama model server support or existing model server support
RHOAI support

Gitops changes are at https://github.com/redhat-ai-dev/ai-lab-app/pull/39 https://github.com/redhat-ai-dev/ai-lab-app/pull/40

Which issue(s) this PR fixes:

https://issues.redhat.com/browse/RHDHPAI-84
PR acceptance criteria:
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.

[x] Tested and Verified

[x] Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)

How to test changes / Special notes to the reviewer:
use the template.yaml from the new template to test it out

once the gitops changes are merged, the CI will pass over here.

@maysunfaisal could you please also add a screenshot to the PR for an easier review form the UX side?

@MariaLeonova

The new template card with a description

The template card techdocs are a copy-paste of the existing techdocs with minor updates. For example, linking the model page.

The Template name and description is at the top of the screenshot, this template at the moment only supports vLLM, so there is only one value in the dropdown

Git Repo details section hasnt changed

Deployment section doesn't need anything besides the namespace

@maysunfaisal thank you for providing all the screenshots, those look great!
I only have a few comments:

In the new card layout, the template descriptions are partially hidden behind the ... Is there a way to make that field longer, so more text fits?

Could a description for model server, no application explain a bit more about why would someone want to use that template? I would argue that in case of a chatbot, the use is more or less clear; whilst for the model server on its own it might be not as straightforward.
It says in the related Jira issue:

This would allow me to utilize the Model Server instance with my other Software Templates for example.

So maybe this could be specified along with some other use cases?

@MariaLeonova

@maysunfaisal thank you for providing all the screenshots, those look great! I only have a few comments:

In the new card layout, the template descriptions are partially hidden behind the ... Is there a way to make that field longer, so more text fits?

I think we had this discussion and tried it previously, it does seem like it was not possible to expand further on the template card

Could a description for model server, no application explain a bit more about why would someone want to use that template? I would argue that in case of a chatbot, the use is more or less clear; whilst for the model server on its own it might be not as straightforward.
It says in the related Jira issue:

This would allow me to utilize the Model Server instance with my other Software Templates for example.

So maybe this could be specified along with some other use cases?

I have updated the description a bit to be more specific! https://github.com/redhat-ai-dev/ai-lab-template/pull/59/commits/508394d114d32b3a43a92f431bc919f79ce701a4

Perfect, thank you, @maysunfaisal !

CI should pass once https://github.com/redhat-ai-dev/ai-lab-app/pull/40 is merged

@thepetk I have tested ArgoCD for the Backstage Gitops Resources and they look good:
Model Server Gitops resource

Chatbot Gitops resource and Component (main branch)

@thepetk I have tested ArgoCD for the Backstage Gitops Resources and they look good:
Model Server Gitops resource

### Chatbot Gitops resource and Component (main branch)

Looks nice! I can take a closer look too tomorrow morning if you'd like!


fix: User avatar and email does not show when using Submit

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/44295
PROPOSAL: https://github.com/Expensify/App/issues/44295#issuecomment-2195111222
Tests

Global Create > Track Expense
Input the amount, description > Track
This will take you to your DM
Select Submit it to someone in the actionable whispers
Input the new user email
Submit it to them
Verify that: The avatar and email of the user you've submitted an expense to shows in the new transaction comment

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Global Create > Track Expense
Input the amount, description > Track
This will take you to your DM
Select Submit it to someone in the actionable whispers
Input the new user email
Submit it to them
Verify that: The avatar and email of the user you've submitted an expense to shows in the new transaction comment

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/user-attachments/assets/fb5335a7-460c-4b32-a085-ffdd0bff2f77

iOS: Native

iOS: mWeb Safari

https://github.com/user-attachments/assets/984941ae-6290-4b56-a46a-f78b95eb8002

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/09907068-b464-4036-bcf9-42bac2834406

MacOS: Desktop

https://github.com/user-attachments/assets/d20b5b83-f092-4973-900b-0acabd5610d4

@shubham1206agra I'm still working fine, please check again
https://github.com/user-attachments/assets/415558e3-fd13-4066-a922-0660a0bf88d2

@shubham1206agra please add more detailed steps and video so that I can follow your issue

@shubham1206agra maybe the problem happens when in strict mode, you can turn it off here

@shubham1206agra I can reproduce it now, I will investigate and update tomorrow.

@shubham1206agra I missed checking out this branch yesterday then I can reproduce this bug. I tried many times again but can reproduce now. Sorry for the confusion.

@grgia Can you please add a build here? I need to re-test in the real device to confirm this bug still happens or not.

@shubham1206agra Let's wait for a build here and we can re-test again if it still happens.

Triggered a test build @nkdengineer !

@grgia Unnecessary, the build is failing on all platforms.

@nkdengineer it triggered two builds by accident and I cancelled the first, second is still building!

Thanks.

@shubham1206agra Found the RCA when testing in the build web, ConvertTrackedExpenseToRequest API BE returns only the personal details of the current user, and the participant also only has the current user. If we open another report and go back, OpenReport API is called and then the necessary is returned.
I'm not sure why I can't reproduce locally on my side. Maybe on my side OpenReport API is called immediately after ConvertTrackedExpenseToRequest is called.

@shubham1206agra @nkdengineer

API BE returns only the personal details of the current user, and the participant also only has the current user.

Could you remind me the expected behavior here

@grgia the expected behavior here is. For ConvertTrackedExpenseToRequest API, BE should

personalDetailsList, return the personal details of the user that is payerEmail in the param of the API

report (invoice report),  return the correct participants that contain the payerEmail user. Current this API returns participant only has the current user

@nkdengineer @shubham1206agra Let me take a look at the BE this afternoon, please let me know if there are any specific steps I need to consider

I also see the call to OpenReport after in Dev @nkdengineer but not in the adhoc test as well

Either way I can update ConvertTrackedExpenseToRequest to send both participants, I have a PR up here  (internal repo)

@grgia Got it. We should return the data correctly as I mentioned here https://github.com/Expensify/App/pull/45393#issuecomment-2258124000 to prevent loading show in header before OpenReport API is complete or it's not called.

I verified locally that the linked PR will send the necessary detail. It's in review now and I'll keep you updated

@nkdengineer are you unblocked now? The linked Auth PR is on production

@muttmuure Thanks for your update, let me test again.

@shubham1206agra I tested in the ad-hoc build again and the bug is fixed after the BE changes. Can you please double-check with your local again, thanks.

@shubham1206agra Merged main.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/grgia in version: 9.0.18-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 9.0.18-10 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


deps: pin the versions

Pin the dependencies, so it's clear which ones work

@jacquerie @kaplun looking ok?

@kaplun added your suggestion. Feel free to merge if you're happy with it!

Mmh. Travis is not happy due to missing hdf5.h dependencies...

@kaplun Travis is failing anyways, because it is not set up properly. I can do that in another pull request.

@jstypka if you have time for that would be great! In this way we can ensure real continuous integration.


Route all through CLI

When you add vnet to webapps or functions, there is an option to set 'route all', and I don't see a way to do it by the CLI command, just the UI. So adding the vnet integration to the function through CLI, after that you need to go to the portal and manually  set route all if it's needed for that resource.

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

ID: f714ea49-754e-486f-938d-5394399d233a
Version Independent ID: 2b0a24e2-be87-4dee-9991-5e35589a3b07
Content: az functionapp vnet-integration
Content Source: latest/docs-ref-autogen/functionapp/vnet-integration.yml
Service: functions
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@jhonnatanpb We are looking into it and get back to you for any additional information.

@jhonnatanpb We are looking into it and get back to you for any additional information.

Thank you.

route to CXP team

Hello @jhonnatanpb Thanks for the feedback, we are routing this to the Service Team's attention!

@jhonnatanpb - We have the following CLI commands that set this:
az webapp config set --vnet-route-all-enabled
az functionapp config set --vnet-route-all-enabled
Closing this issue as this should cover the ask. Feel free to reach out with any additional questions.


Logbook chronology not ok

Nightly 1001- 0c8fcf0

Open cache via gc-code search, cache is not saved
The newest friend-log is shown in the normal logbook as first entry independently of the log-date
If you save the cche and update it, the problem disappears

Hm, very strange.
I just checked the named cache. Your logs (as my friend) are at the correct position. Have you played with the log settings ?

I played with the friend-log page setting only after finding this problem.
Here is how it looks to me:
[IMG]http://i40.tinypic.com/14ik77d.png[/IMG]

Another example is GC2F2J0.
dennis2512 is a friend and is again shown as first entry in normal logbook, although its old:
http://i43.tinypic.com/2vi5dnk.png

It's fixed by campbeb now. The list of logs is in "reverse" order, instead of prepending friends logs they've been appended :-(
Please reopen if the bug still exists (what it does not).

I will test with the next nightly. Todays nightly doe snot contain this commit (as far as I can see).

Tested and working (after updating the cache if it is in the cache of caches)


Added badges for downloads

Added 3 badges to start with that covers weekly, monthly and total downloads in the readme file.

Added license badge too! :)

Added a couple more badges related to current version and supported python versions


Tactical Helmet Running out of Battery but the Light Still on (Still happening in AC v1.1.1)

Describe the bug
As the title suggests, when a tactical helmet has run out of battery, if the light is on before running out of battery, the light will be still on even after running out of battery.  But if you drop it down, it will be off.
To Reproduce

Start the game real quick (maybe change the start credit to buy stuff)
Go to Company Building and buy a tactical helmet
Toggle the light on and just wait until it runs out of battery
Light's still on even if running out of battery

Expected behavior
Light should be off after running of battery
Screenshots

Additional context
I don't know why but the log output suggests something's wrong with walkie-talkie function.
PS: don't mind my name being 口口口 in game. I just didn't install Chinese Localization in my testing.
Logfiles
LogOutput.log
Add a list of all of your mods
it's manually installed.
BepInEx-BepInExPack-5.4.2100
Rune580-LethalCompany_InputUtils-0.6.3
PotatoePet-AdvancedCompany-1.0.149

Is this error still happening in the latest versions? I close this ticket as I tested it fixed but maybe its broken again?

ohhh, thanks for the heads up. haven't played LC for some time since I'm getting busy. will definitely try it when I get some time. thanks for the hard work of fixing it!


Added batch operations

Hi, thanks for contributions to the open source. I've implemented batch operations. [http://developer.mailchimp.com/documentation/mailchimp/guides/how-to-use-batch-operations/](Mailchimp Batch Operations)
Hope it's helpful.

Thanks for your contribution ! It is really appreciated !


i18n: Translate theme text variable

thanks for all these translation updates - I think none of them will change behavior to anyone not using translation, so just merging away 👍


Consider enabling strict-casts in core or recommended

Helps in catching unintended runtime errors related to dynamic.
See also https://github.com/dart-lang/lints/issues/44, which is the other part of this puzzle.

The "strict-casts" flag is not a lint, just an analyzer flag (https://dart.dev/guides/language/analysis-options#enabling-additional-type-checks).
It makes it an error to have an implicit downcast from dynamic. That's one of the affordances of dynamic, but if someone wants to disable it, they're free to do so.
I don't think it's something we should enabled by default in the recommended lints, not unless we would also be willing to work towards removing it from the language entirely. I don't think we're there today.

We need more investigation to take action here, and will likely evaluate strict-inference and strict_raw_types as well.

From a discussion with @leafpetersen, he thought this was likely worth adding, if we can handle the special case for common patterns in web code.

If we change the language to give another easy to get a general downcast, out an explicit dynamic section of code where the mode won't apply, or specify specific workaround patterns, then it's OK to discourage uses that don't use those patterns. We have not done that yet.
I still don't think we're ready to effectively remove implicit downcasts from dynamic from the language.
We need an opt-in syntax, and preferably one which isn't too cumbersome, and which works correctly for something like for (int x in listOfDouble as dynamic).


Apptainer build + notebook improvements

Earth-2 MIP Pull Request
Description

Adding Apptainer build for those that cannot run docker.
Checklist

[ ] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.

Dependencies

@modons Please see if this works for you! :)

/blossom-ci


Add get_depth() to KinematicCollision2D/3D

Fixes https://github.com/godotengine/godot-proposals/issues/3066
Add get_depth() to KinematicCollision3D and KinematicCollision2D.
https://user-images.githubusercontent.com/61938263/183789290-2ef5d0a7-cefd-4cff-92c4-0e50c66decba.mp4
physics_test.zip
With this feature, for example, when a player is caught and press between objects, we can calculate damage according to the depth value.
If the penetration is too large, the collision is completely filled and the normals are inverted, making the value of depth not desirable. I think another PR is needed in the future for solving that problem.

Thanks!


chore(main): release sinkchart 1.8.0

:robot: I have created a release beep boop
1.8.0 (2023-01-13)
Features

allow providing custom dep graphs (e3b4a77)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sinkchart/releases/tag/sinkchart-v1.8.0 :sunflower:


Purobot Ultra integration

Hi All
I just got an Purobot Ultra - the kickstarter backed AI litter box, and would love to get it working with the integration. Please let me know what I need to do to make it happen. Are there any guidelines on how to build an integration?
It's WiFi powered, if that helps. Happy to help out with what I can, including implementation, but I'd probably need some guidance
Thanks!

Hi ! i'm working on it with @Cyrelion my fork had partial support for T6 (Purobot ULTRA) if you want to try or to help us :-) https://github.com/Jezza34000/home-assistant-petkit

@Jezza34000 has also build the support for the EverSweet MAX 😍 best man

I'd love to give it a try.  Can you give a hint on how to get it installed?  Adding that url as a custom repo in HACS works, but as soon as you click download it throws this error.  I removed the original integration before adding this repo.
[32m2024-11-24 14:22:05.767 INFO (MainThread) [custom_components.hacs] Starting download, fd00772
2024-11-24 14:22:05.767 INFO (MainThread) [custom_components.hacs] <Integration Jezza34000/home-assistant-petkit> Running pre installation steps
2024-11-24 14:22:05.767 INFO (MainThread) [custom_components.hacs] <Integration Jezza34000/home-assistant-petkit> Pre installation steps completed
2024-11-24 14:22:05.767 INFO (MainThread) [custom_components.hacs] <Integration Jezza34000/home-assistant-petkit> Running installation steps
2024-11-24 14:22:08.217 ERROR (MainThread) [custom_components.hacs] Download failed - Got status code 404 when trying to download https://github.com/Jezza34000/home-assistant-petkit/releases/download/fd00772/petkit.zip
Traceback (most recent call last):
File "/config/custom_components/hacs/base.py", line 709, in async_download_file
raise HacsException(
custom_components.hacs.exceptions.HacsException: Got status code 404 when trying to download https://github.com/Jezza34000/home-assistant-petkit/releases/download/fd00772/petkit.zip
2024-11-24 14:22:08.218 ERROR (MainThread) [custom_components.hacs] <Integration Jezza34000/home-assistant-petkit> Failed to download https://github.com/Jezza34000/home-assistant-petkit/releases/download/tags/fd00772/petkit.zip
2024-11-24 14:22:08.218 ERROR (MainThread) [custom_components.hacs] <Integration Jezza34000/home-assistant-petkit> Downloading Jezza34000/home-assistant-petkit with version fd00772 failed with (Could not download, see log for details)

Hi ! i'm working on it with @Cyrelion my fork had partial support for T6 (Purobot ULTRA) if you want to try or to help us :-) https://github.com/Jezza34000/home-assistant-petkit

At the moment there is no released version yet. Just download it via GitHub and exchange the files in the petkit directory/custom_components directory via file browser/visual studio.
So yet not an in-place replacement. But he’s working on that too

Ok gotcha.  Reinstalled and configured the original integration, overwrote the existing files with the extracted zip, restarted HA, and got the following.  Is there an update or mod for the petkitaio package needed?
Setup failed for custom integration 'petkit': Unable to import component: cannot import name 'FountainCommand' from 'petkitaio.constants' (/usr/local/lib/python3.12/site-packages/petkitaio/constants.py)
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1011, in async_get_component
    comp = await self.hass.async_add_import_executor_job(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1071, in _get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/util/loop.py", line 200, in protected_loop_func
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/config/custom_components/petkit/__init__.py", line 9, in <module>
    from .const import DOMAIN, LOGGER, PETKIT_COORDINATOR, PLATFORMS, POLLING_INTERVAL, REGION, TIMEZONE, UPDATE_LISTENER
  File "/config/custom_components/petkit/const.py", line 8, in <module>
    from petkitaio.constants import FountainCommand, PurifierCommand
ImportError: cannot import name 'FountainCommand' from 'petkitaio.constants' (/usr/local/lib/python3.12/site-packages/petkitaio/constants.py)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 334, in _async_setup_component
    component = await integration.async_get_component()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1031, in async_get_component
    self._component_future.result()
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1023, in async_get_component
    comp = self._get_component()
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1071, in _get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/util/loop.py", line 200, in protected_loop_func
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/config/custom_components/petkit/__init__.py", line 9, in <module>
    from .const import DOMAIN, LOGGER, PETKIT_COORDINATOR, PLATFORMS, POLLING_INTERVAL, REGION, TIMEZONE, UPDATE_LISTENER
  File "/config/custom_components/petkit/const.py", line 8, in <module>
    from petkitaio.constants import FountainCommand, PurifierCommand
ImportError: cannot import name 'FountainCommand' from 'petkitaio.constants' (/usr/local/lib/python3.12/site-packages/petkitaio/constants.py)

At the moment there is no released version yet. Just download it via GitHub and exchange the files in the petkit directory/custom_components directory via file browser/visual studio.
So yet not an in-place replacement. But he’s working on that too

Hi @JrothSKV open an issue on my repo, to avoid being off-topic on this thread. I will explain you how to fix it


Can't define CURL Option (timeout)

Issue Summary
I'm using the lasted SendGrid-php lib "6.0.0" and saw an issue related to curl options.
The method sign of SendGrid/Client constructor Class (file: sendgrid/php-http-client/lib/Client.php) have just 4 input attributes:
   public function __construct($host, $headers = [], $version = '/v3', $path = [])
but the SendGrid constructor Class (file: sendgrid/sendgrid/lib/SendGrid.php) call a SendGrid/Client object sending five attributes, ie, the last attribute is lost:
        $curlOptions = isset($options['curl']) ? $options['curl'] : null;         $this->client = new \SendGrid\Client($host, $headers, '/v3', null, $curlOptions);
In this way, we can't define the CURL options property.
I hope had been clear on my report...

Thanks @jonaswebdev!
Your report is clear and now available for community contribution. I've also added it to our internal backlog for a fix.
With Best Regards,
Elmer

Elmer,
Great, how can I fix it and send a PR?
https://github.com/sendgrid/sendgrid-php/blob/master/CONTRIBUTING.md right?

Yes, @jonaswebdev, thanks! Feel free to add comments here if you have further questions.


Implement admob plugin.

Has been competed at the https://github.com/IgorMats/cocos2d-apple-library/commit/0f0114d53571b78a4e2c53898566d09962e23191.


有无不需要定期更新Secret的cookie的方案

cookie一般七天会过期，能否在Secret里保存Overleaf的账号密码，自动获取cookie，或者不使用cookie走账号密码

That's strange. Last time I checked I found the cookie to be active for more than 2 months. Copying a small fragment of README (below), please have a look at it. If you're 100% sure that in 7 days your workflow isn't working, let me know, I'll spin the same test again to verify the results.

IMPORTANT: I would recommend that after you copy overleaf_session2 cookie and put it in the GitHub actions secret, please delete the cookie from the browser. And then start a new browser session (by logging in again) if required. Please do not logout, it will make overleaf revoke the credentials. If you follow the above point, then the cookie should remain active for atleast 2 months, and the workflow should work without any issues. We've tested it in early 2023 by working a sample workflow for couple of months, and you can find the insights https://github.com/subhamX/overleaf_sync_with_git/pull/4#issuecomment-1355116634.

Finally to answer your question on programatically getting the token from email & password. It's tricky, cuz overleaf is protected by reCaptcha (from google). But will need to check if the API is protected by the login API or the site is using it just  for bots trying to scrape content, etc.

Thank you for answering. I will try again and report issues when any arise.


separate rejection tests

Consider separation of negative (rejection) tests from integration tests.
There might be common fixtures for those tests if it would make sense.
After that, we need to create a full suite of rejection tests --- we want to make sure that validation in ledgerjs and ledger app is consistent (the only exception is BIP44 paths which ledgerjs just passes on to the ledger app).

Done in #148.


The Mnemonics field does not accept more than one character at a time

Description
The Mnemonics field does not accept more than one character at a time. and to enter a full word you have to click inside the field after every character.
Get Mastodon (2).webm

This is a known issue as all users paste their mnemonics directly in the field.


add the rest of the questions along with their associated files

I got a warning from github for day 14. I guess the compressed file is 50.85 MB, and their "recommended maximum" is 50 MB. It looks like it is just a warning so I think we are good, but something to be aware of.
Another thing is that for links in the questions I have been liking the look of ***[]()*** so they are bold and italicized. Maybe worth making a generic style. I did it to probably the last half of the questions, check it out and lmk what you think

For the link styles yeah we could add a style to the question component, and we could make it so it only applies to relative links a[href^=.] I believe should do it for the selector. But will need to figure out how to write that with tailwind


Please merge this, it only modifies the Readme

This PR (commit) can be reverted.
This PR is to verify if the Jenkins Job that publishes the optaplanner-website in a Docker container is triggered after this PR was merged.

@mbiarnes you don't need new commits to test that the the job is triggered. If will automatically execute new build if there are no build yet.

@psiroky I don't understand completely. The job https://kie-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/Empty/job/optaplanner-website-automatic-publishing/ "looks" if there is a new PR. I want to assure that only if this PR was merged that the pulishing job is triggered.

@mbiarnes sorry, I am getting a bit confused here. I am not sure we should be monitoring PR jobs. What happens if someone pushes changes directly? We still want the publish to happen. Also the job configuration seems to be watching the job results, no the actual status. We should not run the publish job until there are changes (e.g. merged PR). Or is there something I am missing?

@psiroky I hope it looks better now ...


Unable to set shield options when using SSL

Hey guys, hope everyone is well.
@tylerjl was kind enough to implement initial Shield support, but sadly I'm encountering one rather important problem.
In our setup, we use LDAP authentication, so I set the following config in my instance:
elasticsearch::instance { 'client-node':
  config => {
      ...
    # Shield configuration
    'shield' => {
      'authc.realms' => {
        # File-based Realm
        'file1' => {
          'type'  => 'file',
          'order' => '0',
        },
        # LDAP Realm
        'ldap1' => {
          'type'         => 'ldap',
          'order'        => '1',
          'url'          => $ldap_uris,
          'user_search'  => {
            'base_dn'   => 'ou=foo,dc=bar,dc=example,dc=com',
            # Disable the LDAP health check (constantly communicates with LDAP
            # and requires a Bind DN which we don't have)
            'pool'      => {
              'health_check' => {
                'enabled' => false
              },
            },
          },
          'group_search' => {
            'base_dn' => 'ou=foo,dc=bar,dc=example,dc=com',
          },
          # There appears to be a bug in Shield whereby the default LDAP
          # search time of 5s is not correctly implemented and thus
          # even very short LDAP lookups timeout with the following error:
          # cause: com.unboundid.ldap.sdk.LDAPSearchException: time limit exceeded
          #
          # Setting this to 120s appears to be sufficient to workaround the
          # problem.
          'timeout'      => {
            'ldap_search' => '120s',
          },
        },
      },
    },
    ...

However, as soon as I enable SSL like this:
elasticsearch::instance { 'client-node':
  # SSL and certificates for Shield
  ssl                  => true,
  ca_certificate       => '/etc/openldap/cacerts/myca.pem',
  certificate          => '/var/tmp/cert.pem',
  private_key          => '/var/tmp/key.pem',
  keystore_password    => 'keystorepassword123',
  ...

My Shield settings are overwritten and I no longer have the ability to setup my LDAP authentication.
Perhaps I've missed something obvious?  Otherwise, would you please perform a recursive merge of SSL settings for Shield with any settings we provide in the config parameter?
Thanks so much!
Fotis

Ah yep, this is a problem I overlooked, thanks for finding it @fgimian . For reference, I define the SSL/TLS configuration hash here, then include it in the instance's config in a merge function.
Solutions to try:

Manually handle keys in the hash the same way its done in other parts of the manifest, it's explicit but messy.
Try to rely on the deep hash merging in puppet-datacat, though it hasn't been tested in this module and there may be a reason we aren't using it here in favor of the manual approach in other places.

@tylerjl Thanks mate, you might also be able to use the deep_merge function in puppetlabs-stdlib.
I think the manual approach is good too, and perhaps you could throw a fail if these keys have already been set by the user, so it's clear that you only do one or the other 😄
I'll be here to test whichever solution you decide to go with buddy, just let me know when ready.

@fgimian I think I've worked around this temporarily in 8f425f9, which I've added to an existing PR (#630) since that fixes some other shield config problems as well.
Fully-qualifying the tls config in the manifest isn't perfect, but I'm hoping to get some time to fix the root cause of these types of problems by controlling the configuration hash much more carefully so the behavior isn't as ambiguous and the manifests don't have to have so much has_key sort of detection everywhere.

Great job @tylerjl, just checked out the PR and that should do the trick for now I think. I'm on leave this week so I won't get to really play with it until Monday mate, but feel free to merge if you're comfortable with it. 😄
Thanks so much for your help and support
Fotis

Closing with the merge of #630, the root cause of this (messy hash handling) will be tracked in #633


test: Add testing to validate do-not-disrupt and do-not-evict

Fixes #N/A
Description
This PR adds testing for both v1alpha5 and v1beta1 components to validate that both APIs respect the do-not-consolidate, do-not-disrupt, and do-not-evict annotation keys across all deprovisioning mechanisms.
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 81.791%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/scheduling/topology.go
2
86.49%

pkg/controllers/provisioning/scheduling/topologygroup.go
2
96.75%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
8903

Relevant Lines:
10885

💛  - Coveralls


Log status code and response text

Description
Log status code and response text

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.90%. Comparing base (3141272) to head (cf586d7).
Report is 6 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1714   +/-   ##
=======================================
  Coverage   94.90%   94.90%
=======================================
  Files          60       60
  Lines        2689     2689
=======================================
  Hits         2552     2552
  Misses        137      137

see 1 file with indirect coverage changes

/assign @tisnik

ping @tisnik

/lgtm

/approve

/retest

/retest

/retest
/lgtm
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

/retest

/retest

/retest

/retest


Check added for binary and octal number

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #268
Added check for binary and octal numbers in the number system conversion calculator so that it gives results only when a valid binary and octal number are entered and shows an error if they are not valid.

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GSSOC'22 contributor

@mansi-garg-12 I have not added this calculator but improvised it. Previously it was giving results even when invalid binary or octal numbers were entered by the user. So, I have added a check so that a result is shown only when a valid number is entered otherwise it shows an error.


investigate user capacity of the collab feature

Investigate the number of simultaneous rooms and maximum number of users per room before the sprig collab feature breaks. This should help us put a cap on the max number of users we allow per collab room.
Things we want to investigate

max. number of collab rooms
max. number of users per collab room.
max. capacity of signaling server

Updates on this issue. I ran a few tests some time and below are the results I got from the test run.
The test is as follows
It starts with two rooms and two clients per room

A random client in each room sends data (a string)
The saving server receives a copy of the data sent as a peer in the room
It saves the data to firebase and sends an acknowledgement to all other peers in the room that the data has been saved
*We measure the time elapsed between when the new update was received via webRTC and when the data was saved to firebase. We later compute the average time here.
On the sender's side, we measure the time elapsed between the last update and when the newest update has been received. We later compute the average time elapsed between the last update and newest update.

We obtain the following information
(The first screenshot is data collected from the saving server)

We see that as we get to about 10 rooms and 10 clients per room, it takes ~7.5 seconds to save data to firebase. Getting to 11 rooms and 11 clients per room, it takes 18 seconds to save data to firebase. I honestly have no clue as to why it takes so long to save data at the moment. Perhaps it's an architectural problem, still investigating.
On the other hand, from the perspective of clients and rooms

At 13 rooms and 14 clients per room, we are in the likes of half a second time elapsed between an update being sent and other peers receiving that update which I think is good.
There's a data point where at 14 rooms and 14 clients per room we're at 2.7 seconds but I think that's just an outlier.
It consistently takes under a second to save data to firebase.


Dragged tags shows incorrect text

I'm submitting a ...  (check one with "x")
[X] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request/question

Current behavior
When dragging tags, the dragged tag is being shown as the incorrect one. Eg. if you drag "Foo" the dragged tag will show "Bar", but when you drop it, it actually moved "Foo". It seems that the dragged one always has the text of the tag to the right of the selected one, except when dragging the last tag in a list in which case it shows the correct tag.
Expected behavior
The dragged tag should always show the correct text.
Minimal reproduction of the problem with instructions (if applicable)
The issue is reproducible by dragging the "C#" tag under "Drag and drop" on the demo site: http://www.buompris.co/ng2-tag-input. The dragged tag will show as "Java" and not "C#" as expected.
What do you use to build your app: angular-cli 1.1.1 (webpack)
Angular version:  4.1.3
ng2-tag-input version: 1.3.3
Browser: Tested in Chrome and Firefox on Linux

Just cloned and played a bit with it. Can reproduce. See following examples:
Case 1
Initial

Have the tags: a, b, c, d, e, f
Repro:
Start dragging f
Drop f in first position
Expected:
f disappears
f is being dragged
f appears in first position
Actual:
f disappears :heavy_check_mark:
f is being dragged :heavy_check_mark:
f appears in first position :heavy_check_mark:

Case 2
Initial

have the tags: a, b, c, d, e, f
Repro:
Start dragging e
Drop e in first position
Expected:
e disappears
e is being dragged
e appears in first position:
Actual:
e disappears :heavy_check_mark:
f is being dragged :heavy_multiplication_x:
e appears in first position :heavy_check_mark:
(This happens with all the letters but the last one, case 1)

A recent PR caused this :( The fix is not super-straightforward so I'll have to find a decent way

WiIl be fixed by https://github.com/Gbuomprisco/ng2-tag-input/pull/462

Wonderful! Thank you very much!

v.1.3.8 released - dragging should work better!

Works beautifully! Another improvement would be "live" updating the positions of each tag, but that's for another day/issue :) Thank you!


new tile for sneaky tricks

there is definitely room to improve it, but i have a new tile that aggregates all sneak sources into one resource. this is not done yet -- at time of writing i neither own cinch nor has it been officially pushed into mafia. this also does not include recommendations on where to use said sneaks, or a supernag warning the user that they have an NC forcer up. still wanted to push this so that we had a base to work off of.

have addressed all prior comments. have also added:
a cincho tile

some logic cleanup on the tile, and a cool mouseover...

that generates a synth-style table sorted by combat rate

think we're about ready? can push this tomorrow i think, or maybe today, idk.


:robot: Triggering CI on branch 'release-next' after synching to openshift/master

/retest

/test e2e

/retest
as today the tests failed due to pod eviction
reason BuildPodEvicted: Pod The node had condition: [DiskPressure].
Note:
yesterday the e2e tests failed on 4.3 (ci/prow/e2e)
today it passed, but jobs on 4.4 failed (ci/prow/variant-x-images)
eventhough the update-to-head script tried to fetch the nightly midstream releaes.yaml again yesterday night, what it got is midstream release 0.12.1. This is same as yesterday. So there is no difference in payload.

OCF Webhook is merging this PR


Eb 1848: Release Preview Compare utility

Changing the class name could avoid any conflict in style and also could help us later supporting color customization.


decimal balance

The system give an error when trying to add a decimal number like 3.5

The feature will be included in the upcoming milestone release.


(#1164) Add types to listing/mixins/redditor.py

#1164
-> ./praw/models/listing/mixins/redditor.py

It's a coverall error please re-run

It's a coverall error please re-run

done.


C++ OSC receiver

Hi,
Thank you for the ES6 update!
I plan to use it with my C++ Cinder project.
I tryed osc.js it the past, unfortunately it was sending a \0 character, which was interpreted as a "end of string" if my c++ app.
So I switched to websockets nodejs ws npm module...
I will try osc.js again and let you know if this is fixed.

Hi @brucelane! Thank you for this! Im curious to hear about this one. Can you maybe give some input on your implementation (with code samples?). Makes sense to write a test for this case

if you're curious, my videodromm project is here: https://github.com/videodromm/Cinder-Videodromm
nodejs-router project is the old one I did with osc.js a year ago, not updated since...
netdromm projectis the websocket server
videodromm-livecoding project is the C++ Cinder app
videodromm-controller-web is the new web project I'm building with React, ES6, netdromm ws client( socket.io is not compatible, so I use ws)
the osc and websocket files are here: https://github.com/videodromm/Cinder-Videodromm/blob/master/src/VDRouter.cpp#L46

ah I forgot, I was planning to use osc.js with LiveOsc for Ableton Live, but since Ableton Live 9.6 LiveOSC doesn't work anymore...

I have written my own Max for Live device for these purposes which I add to the channels I need to control, its quite easy to do since Max for Live gives full control on all Ableton Live controls, then its just about getting the udp data and routing it to the right place

With Max for Live, can you send you info like master meter, tempo, current measure outside ableton using osc or websockets?

Yes, look here: https://docs.cycling74.com/max5/refpages/m4l-ref/m4l_live_object_model.html, for example tempo or signature_denominator. The docs are from Version 5, could be a little bit different for newer versions but Ableton comes along with many examples

I checked back the current osc-js implementation with the OSCListener sample of Cinder and it worked, can you confirm @brucelane?

it's Colin Clark osc.js repo I was using, I will try yours in some weeks...

@brucelane alright, I checked the Cinder OSC library against the osc-js es6 version at I cant find any issues, there must have been a problem with the older implementation, it also based on different string encoding before. Thank you for posting that issue anyways! Please post another one with an example if you find any more problems :+1:


[Windows] Unsuccessful TensorSliceReader constructor

Describe the bug
While running a Director-based workflow on Windows, there is an error while unpickling the Keras model.
This issue is known: https://github.com/keras-team/tf-keras/issues/87
Steps To Reproduce
I ran Director-based workflow from the windows-ci branch, including:

tests/github/interactive_api_director/experiments/tensorflow_mnist/run.py

from pathlib import Path

from tests.github.interactive_api_director import utils

if __name__ == '__main__':
    director = utils.start_director(Path(__file__).parent / 'director')
    envoy = utils.start_envoy(Path(__file__).parent / 'envoy')
    from tests.github.interactive_api_director.experiments.pytorch_kvasir_unet import experiment
    experiment.run()

tests/github/interactive_api_director/utils.py

import subprocess
import time
import sys

def start_director(cwd):
    director = subprocess.Popen(
        'fx director start '
        '--disable-tls '
        '-c director_config.yaml',
        cwd=cwd, shell=True
    )
    time.sleep(3)
    if director.poll() is not None:
        print('Error: failed to create director')
        sys.exit(1)
    return director

def start_envoy(cwd):
    subprocess.check_call(
        [sys.executable, '-m', 'pip', 'install', '-r', 'sd_requirements.txt'], cwd=cwd
    )
    envoy = subprocess.Popen(
        'fx envoy start '
        '-n env_one '
        '--disable-tls '
        '--envoy-config-path envoy_config.yaml '
        '-dh localhost '
        '-dp 50051',
        cwd=cwd, shell=True
    )
    time.sleep(10)
    if envoy.poll() is not None:
        print('Error: failed to create envoy')
        sys.exit(1)
    return envoy

The issue can be also detected with the following code snippet:
import tensorflow as tf
import cloudpickle

model = tf.Sequential()
model.add(tf.keras.layers.Dense(8, input_shape=(16,)))
model.compile(optimizer='sgd', loss='mse')

with open('file.pkl', 'wb') as f:
    cloudpickle.dump(model, f)

with open('file.pkl', 'rb') as f:
    cloudpickle.load(f)

Desktop:

OS: Windows 11
Python 3.10, Python 3.8
TensorFlow 2.8

The error itself:
File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "D:\a\openfl\openfl\tests\github\interactive_api_director\experiments\tensorflow_mnist\run.py", line 49, in <module>
    main()
  File "D:\a\openfl\openfl\tests\github\interactive_api_director\experiments\tensorflow_mnist\run.py", line 41, in main
    experiment.run()
  File "D:\a\openfl\openfl\tests\github\interactive_api_director\experiments\tensorflow_mnist\experiment.py", line 125, in run
    best_model = fl_experiment.get_best_model()
  File "D:\a\openfl\openfl\openfl\interface\interactive_api\experiment.py", line 107, in get_best_model
    return self._rebuild_model(tensor_dict, upcoming_model_status=ModelStatus.BEST)
  File "D:\a\openfl\openfl\openfl\interface\interactive_api\experiment.py", line 137, in _rebuild_model
    return deepcopy(self.task_runner_stub.model)
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\copy.py", line 153, in deepcopy
    y = copier(memo)
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\site-packages\keras\engine\training.py", line 328, in __deepcopy__
    new = pickle_utils.deserialize_model_from_bytecode(
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\site-packages\keras\saving\pickle_utils.py", line 48, in deserialize_model_from_bytecode
    model = save_module.load_model(temp_dir)
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\site-packages\keras\utils\traceback_utils.py", line 67, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "C:\hostedtoolcache\windows\Python\3.8.10\x64\lib\site-packages\tensorflow\python\saved_model\load.py", line 977, in load_internal
    raise FileNotFoundError(
FileNotFoundError: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for ram://d6aec716-7bc3-4fe2-ad0d-6183a87fbe75/variables/variables
 You may be trying to load on a different device from the computational device. Consider setting the `experimental_io_device` option in `tf.saved_model.LoadOptions` to the io_device such as '/job:localhost'.

dill also does not work?

dill also does not work?

@igor-davidyuk No, it does not. Raises the same error.

Closing this issue, as the Interactive API is being deprecated in favor of Workflow API.


Dependency Test Coverage Differences on Windows

Chefspec 4.2.0 in Windows seems to treat coverage of dependencies differently than it does on other platforms. For the same cookbook on Windows and OS X and Linux:
Windows: https://gist.github.com/glasschef/dd0b7c2293a2bf413568
ChefSpec Coverage report generated...

  Total Resources:   19
  Touched Resources: 3
  Touch Coverage:    15.79%

Untouched Resources:

  chef_gem[di-ruby-lvm]              C:0
  yum_package[lvm2]                  C:0
  yum_package[s3cmd]                 C:0
  template[s3cfg]                    C:0
  yum_package[logrotate]             C:0
  directory[/etc/logrotate.d]        C:0
  template[/etc/logrotate.d/mongodb]   C:0
  yum_package[iptables]              C:0
  execute[rebuild-iptables]          C:0
  directory[/etc/iptables.d]         C:0
  template[/usr/sbin/rebuild-iptables]   C:0
  template[/etc/iptables.d/all_established]   C:0
  template[/etc/iptables.d/all_icmp]   C:0
  template[/etc/iptables.d/prefix]   C:0
  template[/etc/iptables.d/postfix]   C:0
  template[/etc/iptables.d/mongodb]   C:0

OS X: https://gist.github.com/glasschef/5e7e6c7ed464b98ee481
ChefSpec Coverage report generated...

   Have a fantastic day

Existing issue in ChefSpec:  https://github.com/sethvargo/chefspec/issues/594

Appears to be an upstream issue, will keep this ticket here to track getting the fix in once it becomes available.

chefspec 4.3.0 was released on July 16th 2015 and I'm getting same result like Mac 100% and Windows 32%. Is there anyone getting same result?

Hey @joelhandwell @glasschef does this bug still occur (sorry it's 18 months later). If it does, can you provide a replication case in the form of the actual cookbook that shows the problem?
Thanks!


aliased inputs

Fix python types for BamlValidationError
Aliased fields should render with the alias when used as function inputs

[!IMPORTANT]
This PR updates BAML to handle aliased inputs by modifying ArgCoercer to use aliases and extending BamlValidationError with additional attributes, along with adding tests for aliased input handling.

Behavior:

Update ArgCoercer in to_baml_arg.rs to use field aliases when rendering function inputs.
Extend BamlValidationError in baml_py.pyi with prompt, raw_output, and message attributes.

Tests:

Add aliased-inputs.baml to test aliased input handling.
Add tests in integ-tests.test.ts for aliased input serialization in classes, enums, and lists.

Misc:

Update runtime_interface.rs to pass environment values for alias resolution.
Add OpenAPI paths for aliased input functions in openapi.yaml.

This description was created by  for 816e6e7772b678bd1b7c228b61de0383d895abcc. It will automatically update as commits are pushed.

before merging this breaks somethings!

e.g.:
enum Foo {
  Bar @alias('bar')
}

template_string Example(foo: Foo) #"
   {% if foo == Foo.Bar %}
   ...
   {% endif %}
"#

I shoudln't have to change my jinja code just cause i added aliases

Good catch, will add an integ test


add custom CSS styles to reports

Adds a "Default report template style" field to the dojo "System Settings" to allow custom CSS styles in reports

Closed as stale


i18n(ko-KR): update configuration-reference.mdx

Description (required)

update configuration-reference.mdx
#8535

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
reference/configuration-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


SSH error in VS Code: Permission denied, please try again.

VSCode Version: 1.43.0
OS Version: Windows 10

I want to access a remote VM with Remote - SSH extension from a Windows 10 machine. But I don't seem to succeed as VSCode is keeping prompting me to enter the password and deny the access each time. The VM runs on a Ubuntu 18.04 LTS OS.
I don't understand why I am being prompted to enter a password in the first place. Normally I should be able to access the VM without a password just like I do from the command line. I generated a ssh key for authentication and that's how I am able to acces the VM from the command line.
Steps to Reproduce:

I access normally the remote VM from MobaXterm.
I generate a ssh key in my windows 10 computer.
I Install the public key in the remote server.
I make sure that I can access the VM from the command line and it works fine.
I install the Remote - SSH extension on VSCode.
Press F1 > search and click Remote-SSH: connect to a host > execute command ssh username@ipaddress
VSCode launch a new instance and prompt me for the password of username@ipaddress.
I enter the password, and it prompts for the same reason with a message: Permission denied, please try again.

This issue occur even when all extensions are disabled.

I have this error as well after the update.  Mine in linux machine.
Version: 1.43.0
Commit: 78a4c91400152c0f27ba4d363eb56d2835f9903a
Date: 2020-03-09T19:44:52.965Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 4.15.0-88-generic

Here is how I managed to connect to the remote VM:

Access normally the remote VM from MobaXterm or other ssh client to verify it's working.
Generate a ssh key:
ssh-keygen -t rsa
Install the public key in the remote server:
ssh-copy-id -i $HOME/.ssh/id_rsa.pub <username>@<ipaddress>
Make sure that you can access the VM from the command line using ssh.
Install the Remote - SSH extension on VSCode.
Press F1 > search and click Remote-SSH: connect to a host > click Add New SSH Host
Write ssh <username>@<ipaddress> and press Enter.
Choose the config file generated with the ssh key.
You can now connect to the remote server.

That should do it, so I am going to go ahead and close the thread.


feat(thumbnail)!: use new vars & create new class names

Description

How and where has this been tested?

How this was tested:
Browser(s) and OS(s) this was tested with:

Screenshots

To-do list

[ ] If my change impacts other components, I have tested to make sure they don't break.
[ ] If my change impacts documentation, I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.

[ ] This pull request is ready to merge.

The checkerboard background is slightly darker in Express, due to the differences in the color vars used for the themes. Is this a problem? Is the checkerboard actually used in production-facing things, or is it simply for demonstration purposes?
I think the solution would be to hard-code a consistent color between the two themes, but I need some guidance/answers on those questions above.

I don't think it's necessary to fix the checkerboard to be consistent for this pr, but you can double check with design in qotd if you'd like.


Enhance pet cards

Hello,
The card that has information regarding pets is not much good as per alignment.

you can see the content dog,male and brown they are probably at center but not looking so good actually if they are leftward then card will look much better and the icons which are before this content  in only 2 fields you have icon which is location and one another. Either i'll add icon to all of them or remove these 2 as well. As with only 2icons the card is looking inconsistent.
So, if you find this issue relevant then please assign this to me under GSSOC'24. Thank you awaiting for your response.

Please assign this to me under GSSOC'24

Please assign this to me under GSSOC'24

Please mam assign this to me under GSSOC'24

Can this issue be assigned to me for gssoc '24


Revisit diagnostic/debug event design

Currently, we have three types of host events -

Contract events - metered, hashed into the ledger, and included in tx meta.
Structured debug events - not metered and not hashed into the ledger, but included in tx meta.
Debug events - metered, not hashed into the ledger, and not included in tx meta.

Contract and Structured debug events contain xdr::ContractEvents, while Debug events are just strings used for local development.
Some questions we need to answer -

[ ] Should Debug events not be metered?
[ ] If Debug events are not metered, do we combine with Structured debug events and just emit the string in the data field as a byte vector for the existing Debug events.
[ ] Instead of having events that are not metered, should we treat Structured debug events like Contract events by hashing them into the ledger and therefore including them into the protocol?
[ ] If we keep non-metered events, should they be put in a separate list from metered events for safety?

Done in https://github.com/stellar/rs-soroban-env/pull/806


Add compare runs list

JIRA: RHOAIENG-4658
Description

Added an expandable list to show the runs that are being compared.

Finish the filter as well.

How Has This Been Tested?

Go to the experiment page
Go to experiment details
Check some runs and click on Compare runs button
You will see this page
Test the sort and filter functions

Test Impact

Add some check to the cypress e2e test to make sure it can correctly render the list.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

works well thanks
/lgtm
/approve


Cargo whines about not finding README.md

I got the following message from cargo recently:
error: failed to parse manifest at /build/hyper-0.13.8.crate/Cargo.toml
Caused by:
  readme file with name 'README.md' was not found
It seems cargo enforces that a readme file exists if it is included in Cargo.toml.
Obviously, hyper has a README.md file.
It is just not included in the crate downloaded from crates.io.
Weirdly, of the crates I have checked, this only seems to apply the hyper crate.
Any idea ?

Which rust toolchain are you using? A similar issue was opened last year, but it was a regression in a cargo nightly https://github.com/hyperium/hyper/issues/2231

Indeed, that might certainly be important for the discussion...
For that build, I used a 2020-06-12 nightly.
To reiterate, the flaw is not in hyper(since the crate has the file that is pointed out by package.readme in its Cargo.toml. Which, of course, makes the issue opened in the wrong place. But, perhaps, this place is a good starting point...
And, it is also NOT in cargo.
The code in cargo that bails out on a lacking readme file is in commit b0a3cc6b7a42a15937cc63d6b26aa4d25af6ce99.
It might be that that commit was later reverted, if it caused too much collateral damage...
But, really...
That code checks that a file exists, if it is pointed out in Cargo.toml.
Is that asking too much ?
The question is rather:
Why isn't README.md included in the crate source archive when served from "crates.io" ???
wget -O - 'https://crates.io/api/v1/crates/hyper/0.14.7/download' 2>/dev/null | tar tvz | grep README.md returns nothing.
Is it because of some misplaced optimization effort ?
Nope, because, for example, for the serde crate version 1.0.117 we have:
wget -O - 'https://crates.io/api/v1/crates/serde/1.0.117/download' 2>/dev/null|tar xzO serde-1.0.117/Cargo.toml|grep readme
returns readme = "crates-io.md".
And that file exists in the archive:
wget -O - 'https://crates.io/api/v1/crates/serde/1.0.117/download' 2>/dev/null|tar tvz|grep crates-io.md
returns:
-rw-r--r-- 1000/1000      2332 2020-07-13 04:03 serde-1.0.117/crates-io.md
Ideas ?

It might be that that commit was later reverted, if it caused too much collateral damage...

Yea I think the regression was fixed a few days later with https://github.com/rust-lang/cargo/pull/8353
Either way, to help avoid issues like that, an option would be to remove the line that specifies README.md since by default (since almost a year ago https://github.com/rust-lang/cargo/pull/8277) cargo checks if a README.md exists if one hasn't been specified, but doesn't error when it doesn't.
https://github.com/hyperium/hyper/blob/960a69a5878ede82c56f50ac1444a9e75e885a8f/Cargo.toml#L5
Alternatively/ in addition, the readme file could be added to the include list
https://github.com/hyperium/hyper/blob/960a69a5878ede82c56f50ac1444a9e75e885a8f/Cargo.toml#L15-L20

Alternatively/ in addition, the readme file could be added to the include list

Are the default include/exclude lists documented anywhere?


v4 Alpha 6 Docs for Modal

Modal Examples #Live Demo has an example where the data-target attribute is not referencing the same id that is in the div.modal element.
<!-- Button trigger modal -->
<button type="button" class="btn btn-primary" data-toggle="modal" data-target="#exampleModal">
  Launch demo modal
</button>

Proposed change target="#exampleModal" to target="#myModal"
<!-- Button trigger modal -->
<button type="button" class="btn btn-primary" data-toggle="modal" data-target="#exampleModal">
  Launch demo modal
</button>

Conversely, it's reasonable to be more instructional with the naming of this hook. Why not use #nameYourOwnID? Unfortunately, as it is now it will break if you don't change the id.

already fixed by https://github.com/twbs/bootstrap/pull/21671


Hot reload/restart not working due to null pointer exception

Steps to Reproduce
on master channel
flutter upgrade.
then change or update a file for reload you get

Error running hot-reload: NoSuchMethodError:
The method 'isAfter' was called on null.
Receiver: null
Tried calling: isAfter(Instance of 'DateTime')

Logs
flutter doctor -v
[√] Flutter (Channel master, v1.4.2-pre.3, on Microsoft Windows [Version 10.0.17763.379], locale en-GH)
• Flutter version 1.4.2-pre.3 at C:\flutter
• Framework revision bafe7cbbb4 (4 hours ago), 2019-03-26 21:40:53 -0700
• Engine revision 0d83a2ecd1
• Dart version 2.2.1 (build 2.2.1-dev.2.0 None)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at C:\Users\Chinedu\AppData\Local\Android\Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• ANDROID_HOME = C:\Users\Chinedu\AppData\Local\Android\Sdk
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
• All Android licenses accepted.
[!] Android Studio (version 3.3)
• Android Studio at C:\Program Files\Android\Android Studio
X Flutter plugin not installed; this adds Flutter specific functionality.
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[√] VS Code (version 1.32.3)
• VS Code at C:\Users\Chinedu\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 2.24.0
[√] Connected device (1 available)
• Nokia 6 1 Plus • DRGGAM5891209280 • android-arm64 • Android 9 (API 28)
! Doctor found issues in 1 category.

Hi, could you please confirm if this problem still exists with the latest versions on stable or master?
Also does this happen on Android and iOS?
Thanks a lot

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.

Sorry,
Github stopped sending me emails about issues I am interested not sure why. But this was fixed a long time ago.

Thanks @codmajik , I'll close the issue now


ban appeal

it says my ban for 5 days on the account 'doopio' was made permanent for ban evading, but I never ban evaded. my friend using the account '_Tem4ik' pretended to be me, but I never did anything myself. this is made evident by the fact that we both have different unique inventories and pokemon and were on the server at the same time doing different quests. you may also recall the time when we both went to the trivia night and answered different questions, which would have been impossible if '_Tem4ik', my friend, was an alt of 'doopio', myself. I'd like to request that we be unbanned from the server as we have both been banned for over a week and are first time offenders.

Just because you are first time offenders, doesn't mean you can get off scot-free. Your actions, as well as your "friend's" leading up to the ban was pretty awful. You had spammed these two big gross paragraph of text at least 5+ times:

Then you said this to which you were promptly banned:

Then no less than 4 minutes later your 'friend' logged on and immediately said:
 - and spammed that last msg in the screenshot few more times.
Don't know why your 'friend' wanted to immediately pretend they were you and continue on the onslaught of spam, but if your actions said anything is that this server would be better off without you and your friend in it. Appeal has been declined.


lightbox html issue

Detailed description
Title and description is not injecting to the right div tag instead it's adding it to the parent div tag.

Describe in detail the issue you're having.
ex:

<div class="bx--modal is-visible bx--modal--expressive bx--modal--expressive--fullwidth" data-autoid="dds--expressive-modal" description="{{ content.field_description }}"
open="" role="presentation"
tabindex="-1" title="{{ node.field_imagefile_s.title }}">

Is this a feature request (new component, new icon), a bug, or a general
issue? Yes

Is this issue related to a specific component?LightboxMediaViewer

What did you expect to happen? What happened instead? What would you like to
see changed? Title and description to added to right div tag

What browser are you working in? Chrome

What version of the IBM.com Library are you using?1.6.0

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

Steps to reproduce the issue

Step one
Step two
Step three
etc.

Additional information

Screenshots or code
Notes

Hi @sridharmula could you provide us with more details?
We currently are not injecting the title and description into any html tags for the lightbox. Are you perhaps looking for the title and description classnames?
bx--lightbox-media-viewer__content__title and bx--lightbox-media-viewer__content__desc

@annawen1 in the story book we have title and desc displayed  on the right side  assuming which was generated from media object
media={
src: select('Image', images, images['1280 x 720 (16:9)']),
alt: 'Image alt text',
title: text(
'title (required)',
'Curabitur malesuada varius mi eu posuere'
),
description: text(
'description (required)',
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Aenean et ultricies est.Mauris iaculis eget dolor nec hendrerit. Phasellus at elit sollicitudin, sodales nulla quis, consequat libero. Here are some common categories:
),
type: 'image',
}
but when we try to generate the html for Drupal those two fields are injecting to parent div tag.
https://ibmdotcom-react-experimental.mybluemix.net/?path=/story/components-lightboxmediaviewer--default

Closing this as I believe the issue has been resolved


Doubt about tickers

Hi ! Just started learning pyton and got to know about this project, really excited to follow and learn !
I just had one question/observation - the tickers.txt file - does the script refer to only the tickers mentioned in this document while scanning the market ?
Or does it by default scan all the cryptos listed?

Hi
The whole idea of Custom Ticker List is to have a customized list. However, I also have an issue with that where if I enable Customer LIst = True, the bot stops working. KeyError BNBUSD and can't seem to get around it. I have turned if off {False}, and put my tickers in the signalsample.txt. However, It seems that it would be much better to have the capability of using tickers.txt for specifics

...KeyError BNBUSD and can't seem to get around it.

Did you remove the BNB token from your tickers.txt?
Line 139 in the moons.py code:

if historical_prices[hsp_head]['BNB' + PAIR_WITH]['time'] > datetime.now() ....etc

...seems to suggest that the BNBUSDT price has to have been pulled from binance previously.
If you've removed BNB from your tickers.txt you won't have a history for it to call, and the program will bork.
Solution - in moons.py replace the BNB on line 139 with one of your tokens specified in tickers, or add BNB back into tickers.


CITE-seq-Count is working without HTO (antibody barcode)

Hi,
I am using CITE-seq-Count on our CITE-seq fastq files. I am not sure if this is an issue or not: I checked our HTO_R2.fastq file, there are no HTO, or antibody barcode, at the beginning of the reads in HTO_R2.fastq file, but CITE-seq-Count is totally working fine...
There are two fastq files for each individual sample in our study. One is named as HTO_R1/2.fastq, the other one is cDNA_R1/2.fastq. Weirdly, I found these HTOs at the beginning of the reads in both cDNA_R1.fastq and cDNA_R2.fastq files.
Is that normal to have CITE-seq-Count running fine but without HTO (antibody barcode) in the HTO_R2.fastq file?
thanks,
Shan

Hello @shannjiang,
you might have some reads here and there that are in both libraries if the protocol is not super clean but I would only worry about it if you have similar numbers in each libraries that are linked to real barcodes.
Normally your cDNA and HTO libraries are split by index which is then demultiplexed after sequencing to split them.

@Hoohm So do you think it's totally fine in my case? As long we have similar numbers in each libraries that are linked to real barcodes?
If it is so, I am wondering how this command is working totally fine even without HTOs in the TAGS_R2.fastq.gz:
CITE-seq-Count -R1 TAGS_R1.fastq.gz -R2 TAGS_R2.fastq.gz -t TAG_LIST.csv -cbf X1 -cbl X2 -umif Y1 -umil Y2 -cells EXPECTED_CELLS -o OUTFOLDER

Do you get high numbers of HTO? I'm guessing mostly you should get unmapped high results.
What does the report look like?

Any updates regarding this issue?

Hi Patrick Roelli,
Thanks for the comment. I am asking someone in my lab to check if most of these reads were unmapped because I am not the person to run the pipeline directly.
I found an error info in the log file:
[WARNING] Read1 length is 28bp but you are using 26bp for Cell and UMI barcodes combined.This might lead to wrong cell attribution and skewed umi counts.
From my understanding, if that means the sequences for the genome are only 2/28 because these barcodes took a large proportion of the read1 (26/28)?
So probably our experiments were failed?
thanks,
Shann
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Patrick<EMAIL_ADDRESS>Sent: 2020年9月20日 12:01
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Hoohm/CITE-seq-Count] CITE-seq-Count is working without HTO (antibody barcode) (#129)
Any updates regarding this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Hoohm/CITE-seq-Count/issues/129#issuecomment-695803509, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHTEBSC3MNKAMMJYKSCUMQLSGYRL3ANCNFSM4PRDHNGA.

Oh no, not at all.
It just means that R1 is 28bp of length but you are using only 26bp from R1.
I've added this warning because sometimes people make a mistake int he cell barcode or umi attribution.
This probably just means that you sequenced 28 instead of 26 for R1. Nothing to worry about.

Thanks @Hoohm , yes you are right, most of these reads were unmapped. I would close the thread.


How to adjust the view state for a view on a page? (Drill down dashboard)

Hi,
I have a question regarding generating a view using data from a 1:n table relationship for a drill down dashboard:
I have a page with a view state account=1; There are several views embedded on the page and one view requires a different view state called 'number' which can be found by a table 'Relation' with the fields 'account' and 'no'.
My question is, how do I pass to the view a state 'number=no'?
I guess it can be done in the View settings as an Extra state Formula, however I cannot figure out the syntax.
Any help appreciated.
Thank you!

I have also been struggling with 1:n relationships using the Extra State Formula - it feels like there is so much power in this field although it can be tricky how to work out how to harness it. Although yours sounds a bit different to mine, I have also wondered about the syntax, and examples. (Mine is how do i use the extra state formula to filter results based on who a user is 'following' {stored in another joined table}).
Whoever has any suggestions for this question, it would be good if as a community we could to start documenting some common examples that we could provide the wiki or other FAQs.

you're right, @rbrasier we need to start documenting some examples to solve common issues. There's so much potential in Saltcorn but also lots of trial and error to get things to work. Hope someone has the time to document their journey through saltcorn to improve the wiki a bit.

I'm struggling too. For shared views in the same page, it replaces a user while wanting to change another field related to the LoggedIn user (and not having the right to change user BTW). I simply don't fully get it and I miss examples, simple ones and complex ones.

@j2l what you you trying to do? Maybe we could start a little cookbook/FAQ on this topic

Well, for example including views from different tables in the same page. Let's say we have two tables (A and B) and a join table to make N:N relations with both (that makes 3 tables in total). How would you display a "show" view for item 1 and 2 in table A which relates to items 10 11 and 12 in table B (with their respectives show views).
I think you made a video on the topic in youtube @glutamate but left the N:N part out of the explanation.

@Txikivasco in your example can you clarify:
Table A has rows with id 1 and 2 (call these A1 and A2)
Table B has rows with id 10 11 and 12. (call these B10, B11 and B12)
there is some N:N relation (say table C)- say A1 related to B10 and B11 and A2 related to B11 and B12. Is this correct?
Now you want to show A1 together with B10 and B11. And this is not just badges, you want to show B10 and B11 in full together with A1 in full - is this correct?

Yes, that's correct. Forgive my bad explanation.

@Txikivasco this solution to your problem involves no formulae. I will sketch it here and then as we discussed I want to write these things on the wiki in full.
I will just make it a bit less abstract, otherwise my head will explode. Say we have tables Person (A above; fields name, age etc) and Group (B above; fields name, purpose, budget etc) and PersonParticipatesInGroup (C above; fields person and group, both Keys to their respective tables)
You want to show a page focusing on a single person which shows their name and age, and also a bunch of cards, on for each Group they participate in together with the budget, name and purpose of that group.
you need these views basic views

ListPerson - not really part of the solution but just a list of persons with name and a link to the ShowPerson view
ShowPerson - to begin with just put the normal fields, name and age. At the end we will embed a view here
ShowGroup - just put the fields: name, purpose, budget.

Now here is the real solution - create these views:

ShowPersonParticipatesInGroup - Show on PersonParticipatesInGroup. Delete the default fields and embed a view (i.e. in builder, drop in "View" component), view to embed ShowGroup with the relation called something like group.groups.
FeedPersonParticipatesInGroup - A Feed on PersonParticipatesInGroup. The single item view is ShowPersonParticipatesInGroup. Leave all other settings, except "Each in card?" If you really wanted each of the groups to be in its own card.
Now go to ShowPerson and embed the FeedPersonParticipatesInGroup view, selecting as relation the one called "PersonParticipatesInGroup.person"

I thought you were replying to Txikivasco but now realize you were replying to me :smile:
Thanks @glutamate


version 2.4 for a new project?

Hi All
My concern is: I see some issues in the list which haven't been addressed for a long time.
Commits are also rare, I'm not sure that there will be update releases anytime soon..
Would it be safe to use AdminLTE 2.4 version for a new project?
Thank you

I would say you can, I use it on a internal project for work with no issues, but for me, I wanted just the ui, and none of the other things so it works for my situation.


Parent child/parent relations on org teams

Mirrors the parent/child relationship for teams on a target organization, based on the relations in the (teams) source org.
It doesn't create any teams, it just mirrors the parent of a team.

Nice one! 🎉


No flexible enough

Why hard coded that values have to be in " "? Most have not.

Can you please give some sample input and output?

I did change in CSVExport.swift
open func readFromPath(filePath: String) -> NSMutableDictionary
//let valuesArray = self.splitUsingDelimiter(row, separatedBy: "\"\(div)") as NSArray;

let valuesArray = self.splitUsingDelimiter(row, separatedBy: div) as NSArray;
func splitUsingDelimiter(_ string: String, separatedBy: String) -> [String]
        return string.components(separatedBy: separatedBy) as [String];
        //CORRECTION: FS 16.07.2018

//            let t1 = string.components(separatedBy: separatedBy) as [String];
//            return t1.filter{ !$0.isEmpty }
Currently I using the lib only for reading and the cvs file I am reading does not need to have the values in " ".
This is in the most cases not necessary and the most CVS files don’t have.

Am 17.07.2018 um 13:09 schrieb Vignesh<EMAIL_ADDRESS><EMAIL_ADDRESS>Can you please give some sample input and output?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/vigneshuvi/SwiftCSVExport/issues/13#issuecomment-405545445, or mute the thread https://github.com/notifications/unsubscribe-auth/AhwELgqAd3gkQnx2RLiVRwO2g33-HSMlks5uHcXTgaJpZM4VQuT1.


remove import/no-cycle

It slows things down a lot on large projects.
On the app, its the difference between a 75s lint and a 35s lint.
On RainbowKit its the difference between a 3 second lint and a 35s lint.
We can disable it an selectively turn it on in projects, or visaversa. Cycles just aren't a huge issue most of the time.

Fixed in #79 🤗
We've got a ticket to conditionally enable in in CI too 🔜


Filter Google fonts font-display property

Issue description:
Hi,
I would like to be able to change the font-display property in the generated Google fonts css. However the Google fonts url is hardcoded here :
https://github.com/kirki-framework/kirki/blob/a9c4ee53d2fc88ba9bed6b7b508443083a3c2de5/modules/webfonts/class-kirki-modules-webfonts-embed.php#L146
Which makes it seemingly impossible to change the display=swap query arg in that url.
I found that there used to be a filter for this called kirki_googlefonts_font_display in a past version of Kirki but it has disappeared since. Would it be possible to re-implement that filter ?
Version used:
This is the same on both the release version (3.1.5) and the master branch

Hello and thank you for reporting this issue.
We are closing most open GitHub issues with the release of Kirki 4 to start fresh with this major release.
If you are still running into issues after you have tested your theme with Kirki 4, please let us know so we can reopen the issue.
Please note that Kirki 3 is no longer actively supported. Moving forward, we will only provide support the very latest version of Kirki.
Thank you for your understanding.
Best,
David


Chaincode should be re-deployed after restarting peer node

Description
Chaincode runs as a seperate docker container. After it has been successfully deployed, it could not connect to peer anymore after peer had been restarted, unless the chaincode is re-deployed.
Describe How to Reproduce

Start the peer node using docker on baremetal

docker run --rm -it --name peer1 -e CORE_VM_ENDPOINT=http://<IP_ADDRESS>:25 -e CORE_PEER_ID=vp0 -e CORE_PEER_ADDRESSAUTODETECT=true hyperledger-peer peer node start

Deploy a chaincode

{
  "jsonrpc": "2.0",
  "method": "deploy",
  "params": {
    "type": 1,
    "chaincodeID":{
        "path":"github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02"
    },
    "ctorMsg": {
        "function":"init",
        "args":["a", "1000", "b", "2000"]
    }
  },
  "id": 1
}

Restart the peer node
Query the chaincode

{
  "jsonrpc": "2.0",
  "method": "query",
  "params": {
      "type": 1,
      "chaincodeID":{
          "name":"7b07c59e9b9405c1aef33493b63b9a766d9bb836989ded1730052de650aa8ce5654274d148ceff96a4e5bd43bca26aba099f55c400e4befdc8b2ee4c0a94e30b",
          "path":"github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02"
      },
      "ctorMsg": {
         "function":"query",
         "args":["a"]
      }
  },
  "id": 5
}

When using docker inspect to find the status differences, I've found that {{ .NetworkSettings.Networks.host.EndpointID }} has been set, while after restarting peer node, that field is empty. It's reasonable to enhance the connection management between chaincode and peer, such that the chaincode is not needed to re-deploy whenever peer had been restarted.

@muralisrini


Screenshot of a deferred javascript generated iframe outputs a white image.

I'm trying to generate an image of a widget generated by a thirdparty javascript, the full code is as follows:
from html2image import Html2Image

hti = Html2Image(
    custom_flags=['--virtual-time-budget=10000'],
)

html = '''

'''

hti.screenshot(html_str=html, save_as='test.png')

I added the virtual time budget flag to give time for the script to fun and the page to render but the program doesn't seem to slow down and the output image is just blank.
How can i render the image before taking the screenshot?

replace with local script


Get rid of attributesHtml

See comments in #15 and #14
This is a straight forward issue.

Done in #23


Add E2E Localhost Transfer Test

Description

ref: #3033
Commit Message / Changelog Entry
N/A
see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Superseded by https://github.com/cosmos/ibc-go/pull/3119.


Add Global Mosaic of Imagery

https://registry.opendata.aws/noaa-gmgsi/
Hourly, global spanning image of the world from EUMETSAT, GOES, and Himwari satellites. Could be an interesting input into GraphCast or similar global forecasting model. For local area models, the individual satellites give more spectral channels and information.

This is done now.


增加一个一键导出对话记录的问题

new bing 有对话轮数限制，超过限制后就要强制开启新话题，原来的内容就都被清空了。希望能增加一个一键导出对话记录的功能，会方便不少，多谢了。

BingGPT supports exporting full conversation as image from v0.1.7.
PDF and markdown will be supported in the next release.

BingGPT 0.3.0 has been released, which supports markdown, png and pdf export.