How to remove immutable js

I really see no need in my project for usage of immutablejs and learning curve it introduces, in my opinion using something like redux-freeze and spread operators is a much more elegant solution. I have tried to attempt and remove immutable but it quickly became very overwhelming as it is used in so many different places. Could anyone point to sections / areas where I need to replace immutable js to vanilla js?
It seems like there are several people who are likeminded, If successful I could submit a new entry in remove docs for this.

I'm maintaining a copy of this repo without Immutable.js: https://github.com/Dattaya/react-boilerplate-object . What do you think about it?
Relevant discussion: #441

If successful I could submit a new entry in remove docs for this.

Would definitely appreciate a PR!

@Dattaya looks really good :) I will move my project over to this starter tomorrow. Would you mind outlining the steps you took to remove immutable so I can check them agains the code and make a pr for remove docs?

This commit sums it up: https://github.com/Dattaya/react-boilerplate-object/commit/712b0726acca8f0da4a2aa4eb19f9321b35ad6e9
Except two things:

Adding React Immutability Helper (react-addons-update ) is unnecessary, but I wanted to show how to use it. It's used in only one file.
Changing react-router-redux path in reducer from router to routing is unnecessary. The reason it was done is that it allowed me to remove custom made routeReducer and simplify config in app.js (because it's the default path react-router-redux expecting) thus the project is easier to understand and use.

@Dattaya works very well! Was also easily able to remove reselect using this template. I believe you don't have any middleware checking for mutability implemented though. Have you considered adding something like https://github.com/buunguyen/redux-freeze into the project? I attempted adding it within store.js, but wasn't able to. Have you got any suggestions for this?

Have you considered adding something like https://github.com/buunguyen/redux-freeze into the project

Done in this commit https://github.com/Dattaya/react-boilerplate-object/commit/8536eab32423a690762bc03173bbfb30378f7186 Please, try it out but before you do, remove service workers cache.

Was also easily able to remove reselect using this template

I wouldn't recommend doing that, memoization is very important. I usually use reselect, another memoization function based on reselect's defaultMemoize, lru cache and in some situations time based cache.

Thanks a lot for that repo @Dattaya!
I've just started a new react-boilerlate project and the first thing I did was to remove ImmutableJS. This library was created in 2014 and at that time it was a really helpful general-purpose syntax-sugar tool. Keeping things immutable has become much more convenient in ES6 with spread operators and other language improvements, to so the main purpose of ImmutableJS shifted to just avoiding accidental mutations.
In redux apps the mutations of the state only happen in reduces, unless you do something completely wrong. Guarding yourself against mutating a state in a reducer seems very easy with redux-freeze. Now we can also use react-addons-update if we want some deep changes in the data trees - that's a nice lightweight replacement to chains of deleteIn, setIn, etc.
Given the above, I don't see what ImmutableJS really adds. In the boilerplate project I did this summer it actually created quite a headache: I remember spending a couple of days trying to persist certain bits of my state and to use customized redux devtools (not all of them worked with a top-level Immutable object). Besides, some of my selectors ended up becoming a mess: for certain parts of the data I had selectors that returned an Immutable Map, then an Immutable array, then an normal id-indexed map and a normal array :laughing: Getting stuff from Immutable trees was a bit tricky and caused null pointer exceptions - it felt less stable than a good old _.get(obj, ['attr', 'subattr', 'subsubattr'], 'VALUE NOT FOUND, MAN!').
I might be somewhat biased in my conclusions because there is a chance that have simply missed a good use case for ImmutableJS in a today's redux app. But before I get this additional evidence I vote for removing ImmutableJS from this boilerplate and just sticking with the solution that @Dattaya has in his fork :–) It worked like magic!

another memoization function based on reselect's defaultMemoize, lru cache and in some situations time based cache.

@Dattaya Any chance you can share these memoize functions, lru and time based caches? I need to build something similar, and would help to have some examples! Thank you!

I'm on the same page with @kachkaev, at least for now, thanks for sharing your experience btw. Oftentimes I see people have difficulties integrating a library w/o Immutable.js support into their project, another thing is that console.log or inspecting object in a debugger wasn't very useful compared to regular objects and arrays, don't know if things somehow got better or not. I hope sebmarkbage's imm. data structures proposal or some parts of it become stage 0. I'm ready to embrace them from day 1 when they're part of the language, until then, I'd rather stick with redux-freeze, immutability-helper and the like.
@oyeanuj, I just used a library, initially started with erikras/lru-memoize noticed that this issue was left unanswered: https://github.com/erikras/lru-memoize/issues/73 and switched to memoizerific. Other cache libs:

memoizee ttl, lru
memory-cache simple ttl cache
lru lru + ttl
cache-lru
node-lru-cache

My favorite is reselect, because it's very cheap: one if call, I use it whenever I can, also created one modification that judges whether to retrieve an item from cache or not only by the first length arguments, other arguments are passed down to the original function. I use lru-cache only for heavy/heavy-ish computations I know take more time than the storage/retrieval from the cache.

@Dattaya thank you for the detailed answer, appreciate it! i'm using reselect atm but am looking to enhance it with custom memoization function with a slightly larger cache to have the selector functions cache per component!
memoizee and memoizerific look promising for that - thank you! (also, if you ran into the usecase mentioned, do share your learnings/solutions!)

Thanks @Dattaya  I'm going to move everything over and try that out. Immutable just seemed like too many headaches considering as has been mentioned the ease of immutability with es6 and redux

Closing due to inactivity and it seems like its solved!


Hybrid mode selector missing

Hello, in the new hybrid gas-pump ariston \ chaffoteaux devices is very important to switch between "only boiler" "only HP" or "auto".
Please is it possible to add it?
This selector was not present in the original Ariston-v2 integration but i managed to add it with the old code, i was supposed to share it on the other project but would be awesome if you can include also in this with direct API.
Please find "hybrid mode" version of old ariston integration
ariston_Hybrid_Mode.zip

Hi, I was also looking for that function in V2 and I've never been able to find it.
I also needed another selector, high temperature, low temperature. is point 4.2.0
how can i add them?

@fustom please let me know if you need support testing new code of v3 implementing this. I would switch to v3 but those hybrid features i embedded in v2 are needed

Hi @samuele2723 I don't know any endpoint to do that. If you trust me please send me your login name and a temporary password. I will find a way to add it.

https://github.com/fustom/ariston-remotethermo-home-assistant-v3/releases/tag/v0.16.3
I recommend to delete and re-add the device to your home assistant.


feat: allow extensions to prevent results from being fetched

Currently we have to qs._fetch_all() because graphql-core will try to iterate
over the QuerySet in a sync way, thus causing issues when running ASGI.
This adds a way for extensions to prevent that from happening, as discussed in https://github.com/strawberry-graphql/strawberry/issues/3364
When grapghql-core 3.3.0 is released we can probably stop fetching results at all,
as it will give preference to use __aiter__ if the object provides one (and the QuerySet does)

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

Comparison is base (94830a1) 87.90% compared to head (47f20e8) 87.85%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #481      +/-   ##
==========================================
- Coverage   87.90%   87.85%   -0.06%
==========================================
  Files          37       37
  Lines        3167     3170       +3
==========================================
+ Hits         2784     2785       +1
- Misses        383      385       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


FIX-#6297: fix experimental numpy.argmax/argmin with Nans in data

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #6297
[ ] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@RehanSD please review

@RehanSD thanks for the review! ready to merge


QuartzDeadlineManager does not recover from failures

Basic information

Axon Framework version: 4.6.1
JDK version:  N/A

Steps to reproduce
Shut down Java application while executing a deadline message.
Expected behaviour
Message is retried automatically later, by another node
Actual behaviour
Message is lost
Additional details
The QuartzDeadlineManager does not set the requestsRecovery flag on a job. This means that if a message fails while being executed, it's not picked up later.

Closing this issue as it's resolved in pull request #2472.


Module 1 exercises [Abigail Amankwah]

Please "drag & drop" exercise 1 file below.
WS2_M1E1_Submission - Abigail.docx

Please UPDATE THE T

If you haven't done so, please go back to Module 1 and make sure to take
the post-training survey.

Thank you for your successful submission to the practice exercise! Please be sure to fill out the post-Module survey.


Can You add Starter template to deploy this in cloudflare wokers

hey i love your work
and i wanna make an music app using your api but i wanna use cloudflare because cloudflare workers is fast and more cheaper then vercel i try to deploy this on cloudflare but i cant
please can you add cloudflare .toml file to easy deployment

it should work out of the box without needing any configuration, you can try this command to deploy to cloudflare -
npx wrangler deploy src/server.ts --name jiosaavn-api --compatibility-date 2024-09-28


DB Tx pool replay

Pull Request Test Coverage Report for Build 587264152

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

Totals

Change from base Build 587063394:
0.0%

Covered Lines:
1558

Relevant Lines:
5602

💛  - Coveralls


feat(link): migrate link component

Migrate link component
Fixes: AB#2399

Please leave only the "Default" story in this one. It is a very simple component. Thanks for fixing the disabled state too.

Removed the other stories.
Also added a demo class with a font size so the text is not tiny.


Width of columns

Transmission: 2.94 (0), RPC: 15
Tr Web Control version:  WEB Control: 1.6.0 beta(20180906)
Browser name & version： Any
Problem description: Width of columns

Is it possible to save the width of columns?
For example, as in the screenshot.

This may depend on the language chosen.
Too many default columns are active, in my opinion the defaults should be only basic and the remaining user could activate itself.


Add 'Money' struct and 'Currency' interface

public interface ICurrency : IEquatable<ICurrency>
{
  string Code { get; }
  string Symbol { get; }
  string Name { get; }
}

public interface IMoneyFormatter
{
  string Format(Money money);
}

public struct Money
{
  decimal Amount { get; }
  ICurrency Currency { get; }

  // Ctor taking amount and currency

  // Overload operators for:
  // Multiplying and dividing by primitive numbers

  // Addition and subtraction of other money amounts
  // But only when the currencies are the same

  // Equality (currency and amount same)
}

Also add an interface for an exchange rate. Defines 'From' and 'To' currencies as well as a decimal rate.
Then on top of that an interface for a money exchanger, which converts money amounts between currencies (given an exchange rate).

Add implementation of currency for single-currency apps: PoundsSterling this is hard-coded with the GBP currency, suitable for apps with no need to worry about multiple currencies.

Money also needs an optional constructor parameter defining how we perform rounding (standard rounding or banker's rounding)

All arithmetic operators should perform no rounding. Rounding should only occur when explicitly requested.

In fact, the money class should not have any rounding functionality at all.
A separate service should be created which performs rounding.


Scene

No we only need:

Constructor for component from parent component
Dynarray of child compoonents
Destructor that unregisters itself from its parent children.
Unit tests

Implementation of SetParent will be done after matrix decomposition implementation and is not required in this PR.

Matrix methods required by your change are now live on dev branch. When you rebase you can finish what's left to implement and then we merge.


Downloading an ipa from a url fails to install on a real device

When downloading an ipa from a url, the install fails when using a real device with the following error:
[INST STDERR] Instruments Usage Error: Specified target process is invalid: [redacted]
This is caused by the fix at https://github.com/appium/appium-ios-driver/blob/master/lib/driver.js#L209. This helped .ipa files to be installed correctly, but the BaseDriver unzips the .ipa and opts.app is now pointing at an .app file instead. This causes a mismatch of file paths between opts.ipa and opts.app and the ipa url is attempted to be installed (which doesn't fail).
[debug] [iOS] App is an ipa
[BaseDriver] Using downloadable app 'http://[redacted]/real_device_tests.ipa'
[debug] [BaseDriver] http://[redacted]/real_device_tests.ipa downloaded to /var/folders/n_/d_bw41_d1fjd_nn22sdw_4d80000gn/T/201642-5150-148yoxr.65h9cc8fr/appium-app.zip
[BaseDriver] Downloaded app to '/var/folders/n_/d_bw41_d1fjd_nn22sdw_4d80000gn/T/201642-5150-148yoxr.65h9cc8fr/appium-app.zip'
[debug] [BaseDriver] Unzipping /var/folders/n_/d_bw41_d1fjd_nn22sdw_4d80000gn/T/201642-5150-148yoxr.65h9cc8fr/appium-app.zip
[debug] [BaseDriver] Testing zip archive: /var/folders/n_/d_bw41_d1fjd_nn22sdw_4d80000gn/T/201642-5150-148yoxr.65h9cc8fr/appium-app.zip
[BaseDriver] Unzipped local app to '/var/folders/n_/d_bw41_d1fjd_nn22sdw_4d80000gn/T/201642-5150-148yoxr.65h9cc8fr/Payload/[redacted].app'
[debug] [iOS] Removing any remaining instruments sockets
...
...
...
[debug] [iOS] App is not installed. Will try to install the app.
[debug] [iOS] app: /var/folders/n_/d_bw41_d1fjd_nn22sdw_4d80000gn/T/201642-5150-148yoxr.65h9cc8fr/Payload/[redacted].app
[debug] [iOS] ipa: http://[redacted]/real_device_tests.ipa
[debug] [iOS] Installing ipa found at http://[redacted]/real_device_tests.ipa
[debug] [iOS] Nothing found on device, going ahead and installing.

Note: [debug] [iOS] App is an ipa, app: [path], and ipa: [path] were logs added by me for testing

Related https://github.com/appium/appium/issues/6456

Also https://github.com/appium/appium/issues/6399


Refactoring to split container config from container runtime

Introduce Container, AbstractContainer and GenericContainer for container properties and configuration, and attach this to a Runtime (or ContainerService) for execution and lifecycle management. This architecture will allow for properly typed specific containers like MySqlContainer, RedisContainer, HazelcastContainer, ElasticSearchContainer etc.

Implemented


Latest NW isn't latest in versions list

The latest version from NW is v0.12.0. But in the versions list that version is on the fifth place. RC & Alpha versions are lower then the latest released version v0.12.0.

Yes, this has been bugging me as well. I use a library to filter and sort versions that doesn't really support rc1 and alpha extensions. I'll try to come up with my own that supports all types.

Python right? https://github.com/keleshev/version

Nice. That should work nicely :) Thanks for the search. I'll put it in the next build!

Build is up!


Change Serbian translation from Latin to Cyrillic

In Serbian, we use both Latin and Cyrillic lettering, and most of Google services (for example) use Cyrillic. I think users woud feel more welcoming if they had the text in Cyrillic, it feels more personalized, and kinda looks more professional.
There is not a big difference, but I think it would affect Serbian users, as they usually feel that they're left behind in the tech world, so making this change would welcome them, I think.
Attached images show the difference:

I am willing to rewrite all the strings if we decide that this is a change that we should go with.

In my opinion, it is easy to decide.
If elementary itself is translated into Latin, then the website should be in Latin too. If it is translated into Cyrillic, the website should be in Cyrillic.
So, we should probably check that out and act accordingly.

It's not that simple, in elementary OS there are sr_RS@latin that uses the Latin script, whereas sr_RS uses the Cyrillic script to write Serbian.
So here as the @latin represents a variant (here for more info: https://www.gnu.org/software/gettext/manual/html_node/Locale-Names.html ) so the default should be Cyrillic indeed.

Although I do not agree with some of the arguments expressed in this thread, I do agree that Serbian translation should be done in Cyrillic. I have already completed the transliteration (plus a few minor changes to the existing translation) and made a pull request. Hoping to see this issue closed soon.

This one should be closed because it is already resolved in https://github.com/elementary/mvp/pull/410


RnnSeqEncoder doesn't work with num_layers>1

Getting this error when increasing the number of RNN layers from 1 to 3:
File ~/.cache/pypoetry/virtualenvs/ar-resp-vArrGwjy-py3.10/lib/python3.10/site-packages/ptls/nn/seq_encoder/rnn_encoder.py:134, in RnnEncoder.forward(self, x, h_0)
    132     out, _ = self.rnn(x.payload)
    133 elif self.rnn_type == 'gru':
--> 134     out, _ = self.rnn(x.payload, h_0)
    135 else:
    136     raise Exception(f'wrong rnn type "{self.rnn_type}"')

File ~/.cache/pypoetry/virtualenvs/ar-resp-vArrGwjy-py3.10/lib/python3.10/site-packages/torch/nn/modules/module.py:1194, in Module._call_impl(self, *input, **kwargs)
   1190 # If we don't have any hooks, we want to skip the rest of the logic in
   1191 # this function, and just call forward.
   1192 if not (self._backward_hooks or self._forward_hooks or self._forward_pre_hooks or _global_backward_hooks
   1193         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1194     return forward_call(*input, **kwargs)
   1195 # Do not call functions when jit is used
   1196 full_backward_hooks, non_full_backward_hooks = [], []

File ~/.cache/pypoetry/virtualenvs/ar-resp-vArrGwjy-py3.10/lib/python3.10/site-packages/torch/nn/modules/rnn.py:953, in GRU.forward(self, input, hx)
    948 else:
    949     # Each batch of the hidden state should match the input sequence that
    950     # the user believes he/she is passing in.
    951     hx = self.permute_hidden(hx, sorted_indices)
--> 953 self.check_forward_args(input, hx, batch_sizes)
    954 if batch_sizes is None:
    955     result = _VF.gru(input, hx, self._flat_weights, self.bias, self.num_layers,
    956                      self.dropout, self.training, self.bidirectional, self.batch_first)

File ~/.cache/pypoetry/virtualenvs/ar-resp-vArrGwjy-py3.10/lib/python3.10/site-packages/torch/nn/modules/rnn.py:237, in RNNBase.check_forward_args(self, input, hidden, batch_sizes)
    234 self.check_input(input, batch_sizes)
    235 expected_hidden_size = self.get_expected_hidden_size(input, batch_sizes)
--> 237 self.check_hidden_size(hidden, expected_hidden_size)

File ~/.cache/pypoetry/virtualenvs/ar-resp-vArrGwjy-py3.10/lib/python3.10/site-packages/torch/nn/modules/rnn.py:231, in RNNBase.check_hidden_size(self, hx, expected_hidden_size, msg)
    228 def check_hidden_size(self, hx: Tensor, expected_hidden_size: Tuple[int, int, int],
    229                       msg: str = 'Expected hidden size {}, got {}') -> None:
    230     if hx.size() != expected_hidden_size:
--> 231         raise RuntimeError(msg.format(expected_hidden_size, list(hx.size())))

RuntimeError: Expected hidden size (3, 128, 32), got [1, 128, 32]

Fixed in #109


Add some niceties to help new people quickly run code

Add a stub README file to help people run code
New module, check_environ for loading env variables cleanly/immediately find out what you don't have set
Check in template .env file that new people can fill in to set env variables that way
Add requirements.txt file to help people install various Python dependencies
Add venv to .gitignore so people following instructions won't have their virtual environment dragged in to git

NOTE
I generated the requirements.txt file using pipreqs and I'm not sure everything is totally needed there. Sanity check before we merge this would be a good idea

@Jemoka tell me if this looks good, especially the requirements.txt file which I autogen'd/have limited context on

Looks awesome. I will test the requirements.txt and merge after that.

Actually, still missing is a note about provisioning Elastic. I will get that set up shortly.

Looks awesome. I will test the requirements.txt and merge after that.

Yeah I've done some local testing and had to add in openai and tiktoken manually but it seems ok now.

Actually, still missing is a note about provisioning Elastic. I will get that set up shortly.

I've been testing with the Elastic Cloud Trial and so far only "gotcha" was being pushed into using cloud_id instead of the URL form. Might tweak check_environ to gracefully handle either a URL or CLOUD_ID later, but don't want to clutter this PR

Yes, but search will break silent if you don't seed the index: elastic will index your embeddings as a numerical array and not a dense vector, which means KNNs will return nothing.

Yes, but search will break silent if you don't seed the index: elastic will index your embeddings as a numerical array and not a dense vector, which means KNNs will return nothing.

Yeah I also had to manually do the _seed_schema thing, was thinking might pull some of that into a little helper script or something so it's not buried in playground.py

Ah ok. Wonderful. I will add a note about _seed_schema(es), but yes eventually a setup script is nice.
I usually orchestrate this using something like make, so users can just call make init and have everything happen. I'd love to know what the standard tooling looks like.

If you have a makefile already written, might make sense to just check it in?
For other projects in the past I've written little bash scripts to help people install a virtualenv + dependencies all in one go, but that was just me trying to make it easy for teammates who were less comfortable with command line. Whatever works, works 🤷

Let's not worry about adding too much more right now, I think this should be a decent enough boost to newbies for now. If other people stumble they can add the things they had trouble with. Human-driven discovery!

(Leaving this for you to merge if you're happy with it; can always add more in follow-ups)

One more thing: I don't really like how an example .env is committed here. Perhaps we can make an .env.example?

gotta run, but will work more on this later. good first draft for now.


fix(ssr): skip updateCjsSsrExternals if legacy flag disabled

Description
The updateCjsSsrExternals function updates server._ssrExternals that's only used here:
https://github.com/vitejs/vite/blob/3224a51702b97fe9d51b07b38c03ec188db750c2/packages/vite/src/node/plugins/importAnalysis.ts#L488-L493
Since that code only runs on config.legacy?.buildSsrCjsExternalHeuristics: true, we can also add the same guard for updateCjsSsrExternals
Additional context
cc @delucis who found this. It was logging the ... doesn't appear to be written in CJS ... warning unnecessarily.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Description
The updateCjsSsrExternals function updates server._ssrExternals that's only used here:
https://github.com/vitejs/vite/blob/3224a51702b97fe9d51b07b38c03ec188db750c2/packages/vite/src/node/plugins/importAnalysis.ts#L488-L493
Since that code only runs on config.legacy?.buildSsrCjsExternalHeuristics: true, we can also add the same guard for updateCjsSsrExternals
Additional context
cc @delucis who found this. It was logging the ... doesn't appear to be written in CJS ... warning unnecessarily.

What is the purpose of this pull request?

[x] Bug fix

[ ] New Feature

[ ] Documentation update

[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.

[x] Read the Pull Request Guidelines and follow the PR Title Convention.

[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.

[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).

[ ] Ideally, include relevant tests that fail without this PR but pass with it.


peoplepicker: allow to navigate on search result lists using arrows (…

Q
A

Bug fix?
no

New feature?
yes

New sample?
no

Related issues?

What's in this Pull Request?
NEW feature:
Now in the search results popup user must to click desired element by mouse. This update allow user to select element using up/down keys and press enter to select it.

Hi @konradsikorski, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

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

TTYL, MSBOT;

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

Nice addition @konradsikorski . We'll test this and get it merged. Thx for your contribution

Hi konradsikorski, just an update that we have not forgot you. We'll get this merged soon. One ask though: If you'd like to get credited with your actual name, would suggest to update profile in the GitHub, so that we can give you credits on the contribution.

Great news, I have one more update to this control. I will send it soon :)
BTW I have updated my profile.

Hi Konrad,
I've merged your change, apologies for the delay but summer holidays delayed things.
Question: I'm not able to get the select style to show in Chrome/IE...it's correctly applied and works (pressing enter selects the right user) but somehow the visual elements (background / border) are now showing up. Would be good if you could take a look at this...I'm not the CSS expert :-)


"Waiting Label" for Form: FR/IT/DE/PL

Waiting labels are needed for the other language contact forms:
Italian: Attendi
French: Veuillez patienter
German: Bitte warten
Polish: Proszę czekać

We are updating but you can follow same steps mentioned in https://github.com/hlxsites/mammotome/issues/476#issuecomment-1635291394 for future updates


Provide auto-updates for linux

Users on linux still have to go to website to download updates.

I agree :smiley: duplicate of https://github.com/Microsoft/vscode/issues/2973 and https://github.com/Microsoft/vscode/issues/229 (coming soon)


Add extended color palette

Feedback from 05/19 backlog grooming: need a technical spike on this. Currently, color definitions exist in different files in the Stencil library as CSS variables, and in SaaS variables in Angular.  Further investigation needed for switching HSL to HEX.
Problem
Our current color palette has a limited set of greyscales and status colors. It's limited in several ways:

When designing for hovering/active state style for an element using existing colors, the lack of shades variations make it impossible to do so. We tried solving this problem by using HSL and create the shades mathematically, but the conversion between HSL and HEX can cause inconsistent HEX values when converting back. Besides, some of the shades calculated with HSL could make the color contrast of text and background become not accessible.

For the upcoming work for the mini donut chart in table rows, the existing status colors don't perform well enough. All status colors are in similar shades, and their positions are very close when put on a color wheel, making the chart difficult to read when the size is small.

A similar situation appears when trying to make the badge component accessible in color contrast and consistent in styles.

For the EAS project, we need a status color for an icon to represent warning status for services and service groups. The current chef-major and chef-minor are too similar to chef-critical. Previously, we tried to avoid using the conventional yellow or orange colors for warning, because of the difficulty to make the color contrast accessible with white text, but that problem can be solved with careful selection of the shades and style design (see example in badge component).

Furthermore, if we take a look at the current greyscales used in Automate, there a quite a few places where we use customized greyscales outside the existing palette, further proves the need for more greyscale colors.

Therefore, we propose adding an extended color palette to the existing one. The only change is to remove chef-grey or replace it with another shade in the fog or grey color shades, depending on how we use it today (needs further investigation).
Definition of Done

Add a new set of color variables to the existing color palette.

Remove any HSL colors, and use HEX values for all colors.

Share the final color variable names with the design team, so they can update the Sketch master file to match the color variable names in the code. In the future, designers and engineers can refer to colors with the same name to avoid mismatch of colors.

updating chef-grey will be a separate card

Color Palette Spec
All color names start with chef are the existing colors found in the UI library. The rest color names are auto-generated and can be updated as we see fit.
See palette here: https://chef.invisionapp.com/share/AWRWGIMVMDC#/362080628_Extended_Color_Palette
Note: after the UI team decided on the names for the colors, @apriofrost can export all colors with names and HEX values to a .css file.

this work is deprioritized, so closing.


Please restore original two column design with more "Recent" items

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

Version: 1.77.1 (system setup)
Commit: b7886d7461186a5eac768481578c1d7ca80e2d21
Date: 2023-04-04T23:21:11.906Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045
Sandboxed: No
More information please refer to https://stackoverflow.com/questions/75936254/is-there-a-way-to-put-the-recent-list-in-the-welcome-page-in-its-own-column
Original page design , before v1.77, is more suitable:

/duplicate of https://github.com/microsoft/vscode/issues/178588

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

@gjsjohnmurray Are you sure they are the same problem?
I think this issue is caused by #177415.

@gtianw are you able to test with Insiders?

@bhavyaus now that 1.77.3 has shipped with #178831 I am still seeing the single-column Welcome screen. In #179157 @miguelsolorio showed screenshot evidence that this only happened at 1.77.
As pointed out above by @gtianw this seems to have been a deliberate change by you in #177415.
Please review and comment.


Event for country code changed

Which event get called when change Country code?

I think there is no event called when the country code is changed some one need to fix that ....

I think there is no event called when the country code is changed some one need to fix that ....

Same as #23


Cannot set isolation level in ibm_db_sa versions >= 0.3.5

The issue is as fixed in this PR https://github.com/ibmdb/python-ibmdbsa/pull/79
stack trace from source to fail (engine is a sqlalchemy Engine)
    with engine.connect() as connection:
 /venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 2272, in connect
    return self._connection_cls(self, **kwargs)
  File "/venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 126, in __init__
    self.dispatch.engine_connect(self, self.__branch)
  File "/venv/lib/python3.7/site-packages/sqlalchemy/event/attr.py", line 322, in __call__
    fn(*args, **kw)
  File "/venv/lib/python3.7/site-packages/sqlalchemy/engine/default.py", line 505, in set_isolation
    self._set_connection_isolation(connection, isolation_level)
  File "/venv/lib/python3.7/site-packages/sqlalchemy/engine/default.py", line 531, in _set_connection_isolation
    self.set_isolation_level(connection.connection, level)
  File "/venv/lib/python3.7/site-packages/ibm_db_sa/ibm_db.py", line 143, in set_isolation_level
    attrib = {SQL_ATTR_TXN_ISOLATION:_get_cli_isolation_levels(self,level)}
NameError: name '_get_cli_isolation_levels' is not defined

Hi,
The PR is currently not merged currently that is why you might be seeing the issue.
I will have a look at this and update at the earliest.
Meanwhile can you provide a sample code that you are running to fetch the isolation level in iseries.
Are you using the I-series ODBC driver or are you using the direct connection to I-series.
Thanks.

Here's the sample code I'm using to set the isolation level, it's passed through sqlalchmey's create_engine method and the isolation is fetch when a connection is made using the sqlalchemy engine (so, not a direct connection)
sqlalchemy_connection_string = 'db2+ibm_db://username:password@host:port/database;'
sqlalchemy_connection_kwargs = {'execution_options': {'isolation_level': "UR"}}
engine = create_engine(sqlalchemy_connection_string, **sqlalchemy_connection_kwargs))

hi @singhshraddha
Thank you for raising this to us.
I have tested the fix #79
The same code has been merged to the git.
You can clone the same and install for you till we make another release.
Thanks.

Please test it once after installing the code from the master branch and then let me know.
I have tested the same from my end.
Thanks.

Hi @amukherjee28, I've tested the code off of the master branch and it works as expected. Let me know when this fix goes in a release

New version is released and this issue had been addressed in the new release 0.3.7.
Please check and let me know if this can be closed.

good to close on my end


[WIP] scan config, scan params and handle info are shared_ptrs

The current implement was quite ugly in the way it handled the getting and setting device configuration, which could be done from more than one class. The PR introduces them as shared_ptr which cleans up a lot of unnecessary code.

The mutex introduced in https://github.com/PepperlFuchs/pf_lidar_ros_driver/pull/75/commits/c303f088f7eda7f3c342765500ccd243ed33edd1 blocks changes to device configuration until the current scan cycle is completed. For R2000 this means all the packets for the current scan are received. For R2300 it means that all the active layers are received.

Not merging this yet since R2300 is having a deadlock very occasionally while dynamically reconfiguring layers_on

Merging this as the intended functionality is achieved. Will add fix to https://github.com/PepperlFuchs/pf_lidar_ros_driver/pull/75#issuecomment-1122216816 in https://github.com/PepperlFuchs/pf_lidar_ros_driver/pull/77


range limit slider on frequency widget

Would be nice if frequency widget had a slider with min / max knobs and only words in that frequency range were colored. The slider could also act as the color legend.

One thing that has to be considered here is how this will work if the heat map display is broken out from the frequency annotation.
I guess it's still the case that the range limit slider is applicable for anything you use the heat map for, not just frequency.


The rowHeader was unable to align with cells when slided quickly

The rowHeader was unable to align with cells when slided quickly

Hi @yyy000112
Row header width is a constant value which is determined by you. So, you need to set the proper value.

@evrencoskun I have set the width, this issue like #153

This bug has been already fixed. However, the latest version has no this fix yet. So, you need to wait for the next release or use the sources as a library.

Hi @yyy000112
Row header width is a constant value which is determined by you. So, you need to set the proper value.

@evrencoskun how to set the header width? I set it in the view xml but no effect

It is also good to have a look at the number of columns. The number of columns should match the number of cells in each row. It can be easily tested with this piece of code:
 var columnCount=0;
        for(i in 0 until 10){
            mRowHeaderList.add(RowHeader("$i"))
            val tempLiss:ArrayList<Cell> = ArrayList()
            for(j in 0 until 15){
                if(i==0){
                    //add it once for a row alone
                    columnCount++
                    mColumnHeaderList.add(ColumnHeader("Column $columnCount"))
                }
                tempLiss.add(Cell("[$i][$j]"))
            }
            mCellList.add(tempLiss)
        }
        myTableViewAdapter.setAllItems(mColumnHeaderList, mRowHeaderList, mCellList);

If this piece of code works well, with scrolling, then the problem is mismatch in size of columns and cells.


Error: unsupported operand types

Using docker implementation, I get the following error upon starting the container:
Traceback (most recent call last):
  File "itchclaim.py", line 3, in <module>
    from ItchClaim.__main__ import main
  File "/app/ItchClaim/__main__.py", line 27, in <module>
    from .web import generate_web
  File "/app/ItchClaim/web.py", line 30, in <module>
    from .ItchGame import ItchGame
  File "/app/ItchClaim/ItchGame.py", line 32, in <module>
    class ItchGame:
  File "/app/ItchClaim/ItchGame.py", line 178, in ItchGame
    def claimable(self) -> bool|None:
TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

For reference, I am using docker compose:
services:
  itchclaim:
    container_name: itch
    image: ghcr.io/smart123s/itchclaim
    volumes:
      - /root/docker/appdata/itch:/data
    command: [
      "--login", "${USERNAME}",
      "--password", "${PASSWORD}",
      "claim"
    ]

Fixed in https://github.com/Smart123s/ItchClaim/commit/706fd967a6b86837f981fc9b7abd88365811f5ed


Idea: Schema information with Java Annotations

In relational ORMs (like Doctrine) schema information of columns can be directly defined with annotations to the fields in a class.
RDFS and OWL information could be added to properties the same way in Anno4j.
For example:
@Iri(EX.MARRIED_TO)
@Symetric @Cardinality(min=0, max=1)
private Set<Person> marriedTo;

Given this information, Anno4j could perform checks on each call to the setter method, whether the new set has at most one element and automatically set the inverse triple (because of the symmetry of the property).

A search for these annotations could be performed when an Anno4j instance is created.
At this point the annotations could be translated to RDFS/OWL statements and inserted into the triplestore.
As I discussed with @berndl, we could introduce a ValidatedTransaction, which checks at commit time whether the resulted state conforms to the schema.
Given the schema information as RDF the validation could be done by SPARQL queries, which will be (hopefully) faster than parsing the annotations using reflection.

Here is a preview of the schema annotations that will be available:

Annotation
OWL equivalent
Description
Parameters

@AllValuesFrom
owl:allValuesFrom
All values mapped by the annotated property must be of the given classes (intersection).
value: Array of classes that all values must be from

@Functional
owl:FunctionalProperty
There can be only one distinct value for each individual.

@InverseFunctional
owl:InverseFunctionalProperty
There can be only one distinct individual for each value.

@Bijective

Shorthand for @Functional and @InverseFunctional

@MinCardinality
owl:minCardinality
Minimum number of values a property must have for each individual.
value: The minimum cardinality onClass: Class restriction for the minimal count

@MaxCardinality
owl:maxCardinality
Maximum number of values a property must have for each individual.
value: The maximum cardinality onClass: Class restriction for the maximal count

@Cardinality
owl:cardinality
Shorthand for @MinCardinality and @MaxCardinality with same value.
value: The cardinality onClass: Class restriction for the maximal count

@InverseOf
owl:inverseOf
The property is the inverse of another.
value: Array of the IRIs of the inverse properties.

@SomeValuesFrom
owl:someValuesFrom
At least one value mapped by the annotated property must be of the given classes (intersection).
value: Array of classes that at least one value must have.

@SubPropertyOf
rdfs:subPropertyOf
Denotes that the property is the subproperty of another, i.e. all superproperties have at least the values of this property.
value: Array of the IRIs of the superproperties

@Symmetric
owl:SymmetricProperty
If X is mapped by the property to Y, then also Y is mapped to X by it.

@Transitive
owl:TransitiveProperty
If X is mapped by the property to Y and Y to Z, then also X is mapped to Z by it.

It will be possible to ensure schema consistence using these annotations and the new ValidatedTransaction class.
@Iri(EX.ARTICLE)
public interface Article extends ResourceObject {

    @Cardinality(1)
    @Iri(EX.ID)
    public Set<Integer> getIDs();

   // ...
}
// ...

Transaction transaction = anno4j.createValidatedTransaction();
transaction.begin();

Article article = transaction.createObject(Article.class);
article.setIDs( Sets.newHashSet(1, 2, 3) );

transaction.commit(); // ↯↯↯ Throws ValidationFailedException

The annotations will be scanned via reflection when a ValidatedTransaction is created the first time.

It could also be possible to automatically update the graph by subclassing a OWLSchemaSupport class in your support classes.
@Iri(EX.PERSON)
public interface Person extends ResourceObject {

    @Symmetric @Cardinality(1)
    @Iri(EX.ID)
    public Set<Person> getPartners;

   // ...
}

@Partial
public abstract class PersonSupport extends OWLSchemaSupport implements Person {

    @Override
    public void setPartners(Set<Person> partners) {
         validateSchema(); // Check schema still valid. Throws IllegalArgumentException if not
         updateSchema(); // Sets this individual as the the partner of the only element in partners (because of @Symmetric)
   }

}

The schema validation at this point has the major disadvantage that the values are already commited at the point in time when the support setter is called.
This problem can be circumvented only by @Iri annotating fields in the support class.

Regarding the Support-class providing functionality for automatic updates to resources property values:
The current implementation of SchemaSanitizingSupport provides automatic (sanitizeSchema() must be called) completion of statements for properties in terms of:

Superproperties automatically receive the values set for their subproperties
The inverse relationship is set for symmetric properties
The transitive hull is completed for transitive properties
The inverse inverse properties of property values are set accordingly

Due to performance considerations the updates are only performed locally at the resource which invokes the sanitizing. This is inevitable (see issue #138) but imposes an order for updates to be performed by the user in order to ensure correct completion of transitive properties:
// r1, r2, r3 some resource objects implementing Schema
r1.setFoo();

After discussion we decided to implement the second possibility from above, because its seems the less unexpected way for Java programmers.
I added one small improvement that will make updates to the value set of superproperties more convenient and straightforward.
Now the values of subproperties are only removed if they (or equivalent instances by owl:sameAs) do not also occur in the value set of the last modified superproperty.
This way updates (through e.g. add* and remove* methods) will lead to the behaviour that one would expect in Java.
// Let bar be a subproperty of foo and let them have the following values for a resource r:
r.getFoo() == [1, 2]
r.getBar() == [2]
// Now a value is added to the superproperty (e.g. through a add-method):
r.setFoo(Sets.newHashSet(1, 2, 3));

// The '2' is still present in the subproperty:
r.getFoo() == [1, 2, 3]
r.getBar() == [2]

In order to prevent sanitizeSchema() from undoing the removal of values the method removeValue() must be used.
This method removes statements from the repository using SPARQL update queries.
Unfortunately this can lead to the situation that old values are still returned by @Iri-annotated getter-method/fields, because the CachedPropertySet used under the hood isn't refreshed.
Therefore a call to Refreshable.refresh() is necessary by the user after a call to removeValue().
This can't be done by removeValue() because the cached set is not accessible from its scope.
Example:
removeValue("http://example.org/foo", myValue);
((PropertySet) getFoo()).refresh(); // Otherwise: getFoo().contains(myValue) == true

This refreshing is now done in generated code (see c40f6c65fa035bd87614e03e7a578471a972749e), but must be done manually by users using SchemaSanitizingObjectSupport.


How to add custom models?

Sorry if this is a silly question, but how do I install custom models? I can't find a way of doing so in the UI, and trying to install an extension like SD Hub doesn't seem to work

Since the Flux.1 models go into the .../models/Stable-diffusion folder, I would assume that any custom models' safetensor files also go into that folder and should then appear in the dropdown in the webUI.


chromium: missing font package

Description of changes
Adding the libpangocairo package. Without this, I received font-related errors and then crashes when using chomium headless.
Unfortunately, I have no idea how to build, test, or contribute to this package. I'm hoping by raising this PR I can get some guidance of what I should do next to get this package upstreamed.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This sounds to me like #176081.
Can you check and report back please?
Thanks :)


advisor-result.yml fails yamllint

yamllint reports indentation and line-length errors on the generated advisor-result.yml file:
  32:5      error    wrong indentation: expected 6 but found 4  (indentation)
  48:7      error    wrong indentation: expected 8 but found 6  (indentation)
  50:5      error    wrong indentation: expected 6 but found 4  (indentation)
  53:7      error    wrong indentation: expected 8 but found 6  (indentation)
  63:81     error    line too long (233 > 80 characters)  (line-length)
  65:81     error    line too long (83 > 80 characters)  (line-length)
  68:81     error    line too long (142 > 80 characters)  (line-length)
  70:81     error    line too long (83 > 80 characters)  (line-length)
  85:7      error    wrong indentation: expected 8 but found 6  (indentation)
  95:81     error    line too long (154 > 80 characters)  (line-length)
  97:81     error    line too long (83 > 80 characters)  (line-length)
  100:81    error    line too long (144 > 80 characters)  (line-length)
  102:81    error    line too long (83 > 80 characters)  (line-length)
  117:7     error    wrong indentation: expected 8 but found 6  (indentation)
  119:7     error    wrong indentation: expected 8 but found 6  (indentation)
  129:81    error    line too long (158 > 80 characters)  (line-length)
  131:81    error    line too long (83 > 80 characters)  (line-length)
  134:81    error    line too long (148 > 80 characters)  (line-length)
  136:81    error    line too long (83 > 80 characters)  (line-length)
  151:7     error    wrong indentation: expected 8 but found 6  (indentation)
  153:7     error    wrong indentation: expected 8 but found 6  (indentation)
  161:81    error    line too long (151 > 80 characters)  (line-length)
  163:81    error    line too long (83 > 80 characters)  (line-length)
  166:81    error    line too long (141 > 80 characters)  (line-length)
  168:81    error    line too long (83 > 80 characters)  (line-length)
  183:7     error    wrong indentation: expected 8 but found 6  (indentation)
  185:7     error    wrong indentation: expected 8 but found 6  (indentation)
  194:81    error    line too long (155 > 80 characters)  (line-length)
  196:81    error    line too long (83 > 80 characters)  (line-length)
  199:81    error    line too long (145 > 80 characters)  (line-length)
  201:81    error    line too long (83 > 80 characters)  (line-length)

This happens with any generated advisor-result.yml file, and seems to break tools like yq.

The line too long "errors" are completely optional; line length may only matter for human readability, but not for machine progressing.
And the wrong indentation "errors" are reported for list entries like
projects:
- id: "Maven:com.vdurmont:semver4j:3.1.0"

where yamllint wants
projects:
  - id: "Maven:com.vdurmont:semver4j:3.1.0"

instead. But actually, both are valid according to the YAML specs. We simply rely on Jackson's default here, which is to omit unneeded indentation to reduce file size.

and seems to break tools like yq.

Actually, I know several parties are successfully parsing ORT results files with yq and have not reported any issues so far. If there was an issue, I'm rather tempted to say that yq is overly strict and does not support all syntax that is valid according to the YAML specs.

@sschuberth there are two reasons of this issue:

one is that we have yamllint in our CI pipelines, which configuration now we'll have to modify
the most important one, I'll get an intern and thought, as I mentioned to @tsteenbe , that this could be a good harmless first issue to get acquainted with the code. Shall you have another proposal, pls let me know. I don't have your email address but Thomas does.

we have yamllint in our CI pipelines, which configuration now we'll have to modify

IMO it's really over the top that yamllint reports these as "errors". These should be "hints" at most.
On the other hand, yamllint itself says that is checks for "cosmetic problems such as lines length, trailing spaces, indentation, etc." (emphasis mine). I'm not sure how valuable it is to check YAML files that are (mostly) meant for machine processing (in this case) for cosmetic problems...

that this could be a good harmless first issue to get acquainted with the code.

You mean the issue of addressing "errors" reported by yamllint? I'd disagree that that's a good issue to address to get started: First of all, nothing is really broken, and secondly, we have some tests in the code base that strictly check actual results to match expected results incl. exact formatting, and these would break without an update of the expected results.

we have yamllint in our CI pipelines, which configuration now we'll have to modify

IMO it's really over the top that yamllint reports these as "errors". These should be "hints" at most.
On the other hand, yamllint itself says that is checks for "cosmetic problems such as lines length, trailing spaces, indentation, etc." (emphasis mine). I'm not sure how valuable it is to check YAML files that are (mostly) meant for machine processing (in this case) for cosmetic problems...

that this could be a good harmless first issue to get acquainted with the code.

You mean the issue of addressing "errors" reported by yamllint? I'd disagree that that's a good issue to address to get started: First of all, nothing is really broken, and secondly, we have some tests in the code base that strictly check actual results to match expected results incl. exact formatting, and these would break without an update of the expected results.

Shall you have another proposal, pls let me know.

That largely depends on the skills of the intern...

I don't have your email address but Thomas has mine.

You can retrieve my email address easily from the Git history of this project.

Ok. I'll file an issue in yamllint proposing to lower the severity of those 2 findings.


chore(main): release 1.1.3

:robot: I have created a release beep boop
1.1.3 (2024-02-02)
Bug Fixes

Adds all the versions to peerdeps to silence warns (#31) (408fca0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mine-scripters/minecraft-script-dialogue/releases/tag/v1.1.3 :sunflower:


Allow dashboard modules to wrap onto next line

Fixes #707

Dashboard modules will wrap onto next line if there are more than 5 on desktop, more than 3 on tablet
May require design input regarding borders, this is the current output:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Create context that satisfies the traits required by ibc-rs ics07-tendermint implementation

Completed with the help of Farhad and Rano.


Fix Issue #4050

Fix Issue #4050
修复了\后接字母为非转义字符时，JSON.parseObject 报错但是validator返回true的问题。
增加了validator对转义符 \t \n \r \b \f \v ' 的检测。

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Huaiwu Li seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Warning  The part below the calendar in the 2nd screenshot is not provided.

Any plans to add the events to the view?

Sorry about this post.

https://github.com/jonathantribouharet/JTCalendar#warning
Sorry it's to specific to your implementation and design so it can't be done.


Update the documentation to specify a given container name for docker run/exec commands

Proposed changes
Update the documentation to specify a given container name for docker run/exec commands
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Pull Request Test Coverage Report for Build 6362

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

Files with Coverage Reduction
New Missed Lines
%

core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java
1
81.08%

server-base/src/main/java/org/apache/kylin/rest/util/QueryRequestLimits.java
1
47.62%

Totals

Change from base Build 6349:
-0.002%

Covered Lines:
26248

Relevant Lines:
93537

💛  - Coveralls


SLAM & Nav2 Availability

Hi, I saw that both Slam and Navigation features are currently being worked on, will they be available in the next few weeks?

Hello @RiccardoZuppetti
Yes, you could say that it is finished, just needing to give it a few more tests. Everything is in #20 , which I encourage you to try and give us your opinion on whether it works correctly.
It says foxy but it can be tested in humble and jazzy versions, as long as it is launched from your computer, or launched inside the robot in its foxy version.
Greetings.

Hi @Juancams
Thanks for the prompt feedback, I'll definetly give a try to the content of the PR and I'll give to you my feedback usage as well. By the way, I've noticed that still there is not a config for the slam feature, it will come after these final tests?

For slam you would have to use the same launcher configurations that were made for browsing, and regarding the params file, you would only have to indicate the topics and frames.
We will try to have everything uploaded at the same time.

Hi, could you please tell me how to implement SLAM and nav2? Thanks in advance.
Also,could you tell me when you the SLAM and Nav2 part will available?


'fix: app crash on going back from similar fragment'

Fixes: #1474
Changes:

On selecting a similar event from the EventDetailsFragment, a newEventDetailsFragment is created for the selected event.
However using Koin DI, the same adapter object was being used.
This resulted in app crashes on going back from the new event.
This commit fixes the issue by instantiating the SimilarEventsListAdapter using its constructor and not KOIN DI

Additional Changes

Removed shuffling of list of Similar events so that the fragment returns to the original order on pressing back
Corrected a typo in the word 'similar' in SimilarEventsViewModel

Screenshots for the change:

Nothing is breaking and I think this should be good for this release


Tabular: Fixed refit_full models without bagging

Issue #, if available:
Description of changes:
Fixed refit_full models without bagging being in subdirectory of their original model.
This previously caused 'optimize_for_deployment' preset to crash when auto_stack=False, refit_full=True, and set_best_to_refit_full=True. This now works correctly.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-472-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-472/1/index.html


[Event Request] Table 38 "Purchase Header" - OnBeforeValidateBuyFromVendorName

Hello Microsoft,
can we get the following event subscriber in field 79 "Buy-From Vendor Name" similar to the following request for Vendors in "Purchase Header" instead of Customers in "Sales Header":
https://github.com/microsoft/ALAppExtensions/issues/9248
field(79; "Buy-from Vendor Name"; Text[100])
{
    Caption = 'Buy-from Vendor Name';
    TableRelation = Vendor.Name;
    ValidateTableRelation = false;

    trigger OnLookup()
    var
        VendorName: Text;
    begin
        VendorName := "Buy-from Vendor Name";
        LookupBuyFromVendorName(VendorName);
        "Buy-from Vendor Name" := CopyStr(VendorName, 1, MaxStrLen("Buy-from Vendor Name"));
    end;

    trigger OnValidate()
    var
        Vendor: Record Vendor;
        LookupStateManager: Codeunit "Lookup State Manager";
        StandardCodesMgt: Codeunit "Standard Codes Mgt.";
	// >>>>>>>>>>>>>>
	IsHandled: Boolean;
	// <<<<<<<<<<<<<<
    begin
        // >>>>>>>>>>>>>>
        IsHandled := false;
        OnBeforeValidateBuyFromVendorName(Rec, Vendor, IsHandled);
        if IsHandled then begin
            if LookupStateManager.IsRecordSaved() then
                LookupStateManager.ClearSavedRecord();
            exit;
        end;
        // <<<<<<<<<<<<<<

        if LookupStateManager.IsRecordSaved() then begin
            Vendor := LookupStateManager.GetSavedRecord();
            if Vendor."No." <> '' then begin
                LookupStateManager.ClearSavedRecord();
                Validate("Buy-from Vendor No.", Vendor."No.");
                if "No." <> '' then
                    StandardCodesMgt.CheckCreatePurchRecurringLines(Rec);
                OnLookupBuyfromVendorNameOnAfterSuccessfulLookup(Rec);
                exit;
            end;
        end;

        if ShouldSearchForVendorByName("Buy-from Vendor No.") then
            Validate("Buy-from Vendor No.", Vendor.GetVendorNo("Buy-from Vendor Name"));
    end;
}

The integration event is:
[IntegrationEvent(false, false)]
local procedure OnBeforeValidateBuyFromVendorName(var PurchaseHeader: Record "Purchase Header"; var Vendor: Record Vendor; var IsHandled: Boolean)
begin
end;

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


JSON Schema new branch

Description
Removing _extra from JSON Schemas
Type of change

[X] Bug fix

Nice!


Documentation doesn't represent current published version

I've been attempting to use wrangler publish --env … but found was hitting errors. Then I discovered that the feature hadn't been released yet and I was reading future API docs.
This is quite confusing I caused be to lose quite some time. All Wrangler documentation links I've stumbled across. point to master/README.md. I should have been reading v1.2/README.md as that is the current @latest.
Not sure what the best way of solving this is … ? Maybe keeping master as current and working on new versions in a separate branch. zeit/next.js have a similar "docs in README.mdapproach and however they manage that seems to be working OK. I think cus they dev on acanarybranch and periodically merge that intomaster`.
I dunno! Just thought you'd like the feedback to help other users in future :D

hi @wilsonpage ! thanks so much for filing and sorry you are running into this issue. we're super excited to release the environments feature and we're amped that you want to use it!
it's true that much of our documentation currently exists in our readme and, in some cases, confusingly reflects unreleased features. this is a relatively classic issue on github that many projects that i've worked on suffer from.
while we could put in efforts to change our development process such that our README reflects what is currently released and not what is on master, we're opting instead to move the entirety of our documentation to our primary docs page (developers.cloudflare.com/workers) and not have it tracked on the README. we'll publish those docs alongside a release to ensure that the docs accurately reflect the currently released build. we're aiming to have this docs challenge solved by early Q4!
in the meantime- if you build from master (cargo install --git https://github.com/cloudflare/wrangler) you can use those new features today! we'd love feedback :)

Great thanks for the clarification! I'm very excited about this project and CF Workers in the general!
While you're here can you point me to any known limitation of CF-Workers. For example is it possible to connect to a database other than KV? Would be interesting to know what applications CF-Workers aren't good for :D

I dunno! Just thought you'd like the feedback to help other users in future :D

Thanks @wilsonpage, that saved me some time!
@ashleygwilliams having up-to-date would be very helpful. Maybe some info that GitHub docs can be out-of-date and we should use the docs on the website?

while we could put in efforts to change our development process such that our README reflects what is currently released and not what is on master, we're opting instead to move the entirety of our documentation to our primary docs page (developers.cloudflare.com/workers) and not have it tracked on the README. we'll publish those docs alongside a release to ensure that the docs accurately reflect the currently released build. we're aiming to have this docs challenge solved by early Q4!

@pasierb we're working on it! Thanks so much for the interest in the project :) The release today will sync the README with the behavior of the software. Appreciate the patience!


python test_env.py [Warning] Check game status error, stop it directly. HTTPConnectionPool(host='<IP_ADDRESS>', port=23432): Max retries exceeded with url: /v2/stopGame (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fddd0e6add8>: Failed to establish a new connection: [Errno 110] Connection timed out',))

    I am really thankful for your reply.And I solved some problems with your help.But unfortunately，I meet some new problems now.

First,I got the nameserver and connect it with my command telnet.

Then I delete the inbound rule as what you tell me to do.But when I start python test_env.py ,there is still some fault there.

I am looking forward to your reply.Thank you! @@hongyangqin @Caiderkoujiao @tencent-ailab @HIT-jixiyang
Originally posted by @Logan-lxw in https://github.com/tencent-ailab/hok_env/issues/11#issuecomment-1362536285

If you can telnet to the target server successfully, no need to reset the inbound rule of firewall again.
So retry to telnet the server to check if the network is accesable.

Thanks for your reply again! Now I solved the problem.


Is axios required?

Instead of requiring axios we could pass in a HttpClientAdaptor interface that would abstract the calls to the underlying software. Could use axios, fetch, node-fetch.
My only issue with this starting out is that it clouds the usability of the package since this package would ALWAYS be relying on another package to use.

Split axios into a separate package in #13


How to work with smaller batch size & Replicate results

Hello Loic,
I am trying to obtain the same results as you. I am using a 22GB GPU. I am trying to figure out a way to obtain the same results as you while using a 22GB GPU.
Lowering batch size allows me to start training but this does not allow me to get the same mAP scores as you.
Should I lower batch size while increasing number of epochs?
Can you please share how I can get close to your results with the current hardware that I have?
Regards,

I come with the same question!!! My training result is quite bed.


HA Install Fails on Windows Server 2016 v1709 (Core)

Trying to install WAC on 2 x Windows Server 2016 v1709 Core Servers
Failover Cluster all built with CSV for storage (15GB) and Witness etc. Usual stuff.
When I run the installer this is what I'm executing
.\Install-WindowsAdminCenterHA.ps1 -clusterStorage C:\ClusterStorage\Volume1 -c
lientAccessPoint wac -msiPath .\WindowsAdminCenter1809.msi -certPath .\wac.pfx -certPassword $certpasswor
d -staticAddress <IP_ADDRESS>
I get this, on either node I run it on:
Processing data for a remote command failed with the following error message: The I/O operation has been aborted
because of either a thread exit or an application request. For more information, see the about_Remote_Troubleshooting
Help topic.
At C:\InstallFiles\Windows Admin Centre\Install-WindowsAdminCenterHA.ps1:115 char:9

    Invoke-Command -ComputerName $computer -ScriptBlock $command  ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (WAC01:String) [], PSRemotingTransportException
FullyQualifiedErrorId : JobFailure
PSComputerName        : WAC01

I've set executiion policy on both nodes for Local Machine and Process to Unrestricted, no change.
Both servers have 16GB Ram etc.
Winrm quickconfig done
Any ideas?
Thanks

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

ID: 32deb7a0-4ea0-caf1-0777-580539173e44
Version Independent ID: a6ee1f9d-b316-c69d-fa9d-2a1556454868
Content: Deploy Windows Admin Center with High Availability
Content Source: WindowsServerDocs/manage/windows-admin-center/deploy/high-availability.md
Product: windows-server-threshold
GitHub Login: @Haley-Rowland
Microsoft Alias: harowl

I am having the same problem with Windows Server 2016 1607.  I had Windows Admin Center 1804.25 installed without any problems.  I tried to upgrade that installation to 1809 and it failed.  I then discovered I was able to install the 1808 preview version of Windows Admin Center without any problem, but again if I try and upgrade the 1808 version to 1809 it fails.  When I try a brand new install of 1809 on my Windows 2016 1607 HA it fails with the same error you are seeing on Windows Server 2016 1709.  I am using the GUI version and not the Core install of Windows Server 2016.

Thanks, if I do the install on a Single Windows 2016 v1709 Core server as a standalone install it works fine, only if I try to run the HA cluster scripts does it fail. I'm going to try the cluster on the long term channel Windows 2016 (as a Core install) and see if the same problem occurs.

Hi, @andyhud and @hinz888 - just wanted to let you know that we're aware of this problem and are working on a fix. Sorry you're having problems. :(

Thanks for the update.  I am looking forward to the fix.

Thanks @lizap - Is this issue specific to v1709 Server OS?

I don't know why this issue was closed. I've been watching it because I've had the same issue.
Today, I downloaded the HA scripts again and they're exactly the same as I had last month - all with a date of 4/10/18
After attending Ignite I was really excited to deploy WAC since it was talked up so much. This I/O operation error is a total show-stopper for us.

I agree.  I am not aware the problem has been fixed yet.

Yep, I saw the Closed too and though "Oh wow, they fixed it".. oh wait, they just closed it with no reason.. Wicked.

Hi @brian0975, @hinz888 and @andyhud,
This Issue was closed because it has been answered. We provide Github Issues for reporting inaccuracies in the Windows Server documentation set. This Issue pertains to software functionality and has been reported to Engineering, as Liza answered. When an issue is closed, it is not necessarily because it has been fixed or resolved, but because we have taken note and provided all the information we have at the time. We do not use Github Issues to track resolution of software requests or bugs, but we do report them to the Engineering group and let you know when we've done so. We apologize if this has caused confusion and we will soon provide a better explanation of how we use Github Issues for addressing documentation inaccuracies in the repo readme and Issues template.
Please note that this issue, although closed, is not locked and you are still able to comment in it, and all participants are still notified when you do. Before I closed the issue, I added Jeff Woolslayer, the Engineering Program Manager, so he can track any further conversation.
Thanks,
Corey

Hi @coreyp-at-msft
Thanks for the clarity, that does now make more sense and knowing the reason why you closed this here for me at least, is clearer.
Where is the best place to ensure we are kept aware of progress of this issue?
Thanks
Andy

Any update on this?

After studying the HA scripts for 1804, and much trial and error, I have been able to successfully install 1809 in HA mode. I have written this powershell script.
Please open the file in your editor of choice, read the comments, understand it, and run each section manually, as appropriate. DO NOT simply run the script.
Install_WAC_1809.zip

I was also able to manually install using the scripts provided by @brian0975 . Thank you.


Functionality with Tensorflow Serving

Do you have any intuition as to how this model could be served using Tensorflow Serving? The part I'm struggling with relates to converting the incoming serialized data (that was encoded as a sentence, say "That movie was the best one I have ever seen.") back into a regular string to be passed into getSentenceMatrix(), to then be passed through the network.
All examples of Serving in action are using text images, the file I am (sort of) replicating is: https://github.com/tensorflow/serving/blob/master/tensorflow_serving/example/inception_saved_model.py.
In particular this part:
# Input transformation.
serialized_tf_example = tf.placeholder(tf.string, name='tf_example')
feature_configs = {'image/encoded': tf.FixedLenFeature(shape=[], dtype=tf.string),
}

tf_example = tf.parse_example(serialized_tf_example, feature_configs)
jpegs = tf_example['image/encoded']
images = tf.map_fn(preprocess_image, jpegs, dtype=tf.float32)
# ^ Essentially this step but relating to strings instead of an image (.jpeg)

Any help is much appreciated! Thank You!!

Hey man, I've definitely meaning to learn more about serving pretrained models but unfortunately, I don't have as much experience with it and so I'm not sure about the answer to your question.
I'll keep this issue open in case anyone else knows and I'll let you know if I find anything once I learn more about it LOL.
(Btw, I do have some code where I show how to load a pretrained model into a Flask server if you'd like to see that https://github.com/adeshpande3/Chatbot-Flask-Server. Doesn't use Tensorflow Serving per se, but it could shed light on some other approaches you could take)

hi i would like to know more if you success in deploy this model with tensorflow serving


[Feature Request]: IMPLEMENTING PURE TAILWIND CSS (REMOVING CSS FILES)

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
As we are using tailwind css, there is no need of external css file. so i suggest removing external css file and implementing those classes usinf tailwind css
Use Case
Tailwind helps us to define minimal css file, and gives us only necessary css, removing unwanted or repeaded styles
Benefits
improves network overhead, ultimately improving performance and load time of website
Add ScreenShots

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Sourabh782 go ahead, assigned to you
if you could fix it completely, its a level 3

@Sourabh782 go ahead, assigned to you if you could fix it completely, its a level 3

Sure will implement it 👍👍👍

@Sourabh782 make sure to do it for each and every page and components

@Sourabh782 make sure to do it for each and every page and components

Yupp understood👍👍

hello @shaurya35 i have implemented it, and also checked the pages after commenting css import statements. its working as expected. So in PR should i remove css files or make pr with css files not deleted ?


Votes should be hashed according to EIP-712

Transition objects will be hashed accordingly. It will be checked in the block store to deem a vote valid. The vote object should be constructed the same way.
See MetaBlock in contracts/core.
See also EIP 712 and the example solidity implementation.
Contracts:

All contracts that work with vote hashes

This issue is only about vote objects.


Unicode characters breaks the force_text method

To override it, we overrided the CACHALOT_QUERY_KEYGEN method with the following code:
(we also catched all errors in non-debug mode to avoid breaking production queries if something fails)
import logging
from hashlib import sha1

from cachalot.utils import UncachableQuery, check_parameter_types
from django.conf import settings
from django.utils.encoding import force_text
from django.utils.six import text_type

logger = logging.getLogger(__name__)

FAIL_SILENTLY = not settings.DEBUG

def get_query_cache_key(compiler):
    """
    Generates a cache key from a SQLCompiler.
    This cache key is specific to the SQL query and its context
    (which database is used).  The same query in the same context
    (= the same database) must generate the same cache key.
    :arg compiler: A SQLCompiler that will generate the SQL query
    :type compiler: django.db.models.sql.compiler.SQLCompiler
    :return: A cache key
    :rtype: int

    NB: modified version of 'cachalot.utils.get_query_cache_key' to use
    force_text instead of text_type + silent fail if FAIL_SILENTLY is True
    as we never want the caching to break any SQL request in production
    """
    try:
        sql, params = compiler.as_sql()
        check_parameter_types(params)
        cache_key = text_type('%s:%s:%s') % (
            compiler.using, sql, [force_text(p) for p in params]
        )
        return sha1(cache_key.encode('utf-8')).hexdigest()
    except:
        if FAIL_SILENTLY:
            # declare as UncachableQuery (caught by cachalot)
            logger.exception("in get_query_cache_key: ")
            raise UncachableQuery
        else:
            raise

Fixed in https://github.com/BertrandBordage/django-cachalot/pull/93

Thank you for the report! Could you share a minimal example leading to this error?

And what error is this?

I can't remember of a specific example, but I guess if you make a query using a non-unicode string containing accents (like "é"), it will break with a UnicodeDecodeError

Closing for now as it requires a minimal test to reproduce the issue.


Remove docs from the packages folder

Why?
Just because packages folder should be destinated to additional packages related to the core
How?
Just move the folder to a projects new one at the root

i think i can handle this 😤😅


Race condition when input file is updated while compiling

I'm new to Meson+Ninja (ninja 1.8.2 on Ubuntu 18.10). As far as I understand, meson corresponds roughly to autotools and ninja to make. Please tell me if I'm wrong and this bugreport should be filed elsewhere.
This report is the same as the make issue at http://savannah.gnu.org/bugs/?46242 (see http://lists.gnu.org/archive/html/bug-make/2015-10/msg00016.html for followup discussions, developer responses).

There's an inherent race condition in the way ninja handles timestamps. If you save a newer version of a file while ninja is running and is just compiling that input file, the change won't be picked up by subsequent runs of ninja, and you'll be left with an out-of-date binary.
Steps to reproduce:

Have some project.
Open one of the large source files that takes noticeable time to compile.
Edit this file in some valid way (changing the functionality) but do not save it yet.
In a terminal, run ninja to compile your project.
While ninja is running, go back to your editor, but keep your eyes on ninja's output.
When ninja launches that particular gcc (or whatever) that compiles this large file, save the new version.
Go back to the terminal. ninja might have completed by now. If not, you might either – if it has finished processing that file – interrupt it, or wait for it to complete.
Execute ninja again to catch up with the edits you performed. Wait for it to complete.
Launch your app, notice that it does not include your recent changes.

The behavior is obvious from the way ninja checks the timestamps. The source of the file has an earlier timestamp than the compiled object file, since you saved the file before gcc wrote out its output. But these are not corresponding versions.

Please see over there in the make bugreport and mailing list discussion for further thoughts on this issue.

#1165 fixes this, but I'm not sure we want to merge it.

Ah, dupe of #1162.

Oh, I missed that one, sorry and thanks!


Fixed Search bugs and avoid re-renders

Ticket Link
https://mattermost.atlassian.net/browse/RN-431
https://mattermost.atlassian.net/browse/RN-438

@jarredwitt changes made
Also this PR includes handling the {data, error} returned by the mattermost-redux actions https://github.com/mattermost/mattermost-redux/pull/265

@jarredwitt please re-review
@hmhealey can you review this one please

@hmhealey changes made


The package problems.

Have u met the problem that u can not install the torch_scatter and pointnet2_batch_cuda package. Could u give me some suggestions? Thank u.

Thank u for your interest in our paper.
From our knowledge, there is an issue in installing pointnet2_batch_cuda package when using CUDA 12.X version. We think that it is a compatibility issue, which is difficult to solve from our level...
Thank you:)

Thank u for your interest in our paper.
From our knowledge, there is an issue in installing pointnet2_batch_cuda package when using CUDA 12.X version. We think that it is a compatibility issue, which is difficult to solve from our level...
Thank you:)

Thank you for your reply! It works.


Clarify behavior of shared executor memory sharing when using Bind

In the docs for MXExecutorBindEX in https://mxnet.incubator.apache.org/doxygen/c__api_8h.html, the parameter shared_exec is somewhat cryptically described as "input executor handle for memory sharing".
I'm using shared_exec to do automated bucketing in Mathematica's deep learning framework, but what I'd like to know is what happens when I bind an executor that requires more memory than the memory used by the shared_exec that I passed in.
The reason I need to know that is the following scenario: when more than one sequence is involved in a net, it is impossible to predict which bucket will need the maximum temporary memory (ahead of time). Therefore it is impossible to automatically pick a 'master executor' which will require more scratch mem than all other executors.
So here we really need to know what the semantics of shared_exec is! Can the shared_exec have less scratch than what is needed by the executor being bound? What happens in that case? Or do I need to pick a master executor and always pass it as the shared_exec? If that is the case, how can I know in advance whether my master executor needs to be retired in favor of another executor, if the only way to measure the scratch taken by an executor is after it is already created (and even then one has to parse a string to find the answer in megabytes, IIRC).
@sbodenstein

If shared_exec doesn't have enough scratches new ones will be created and attach to it.
@reminisce shall we add more description here?

Seems like that shared_exec requires having the same param array size as the exec being bound if the param has the same name in two executors.
https://github.com/apache/incubator-mxnet/blob/master/src/executor/graph_executor.cc#L776

@eric-haibin-lin if that is true then that is ideal -- i don't have to pick a 'master' executor, instead we have a kind of memory-sharing democracy among all executors.
@reminisce i don't think that is directly related to my question. the param array size should be the same: you have the same number of weights, you just unrolled e.g. LSTMs by different amounts. that's not the question. the question is  about the memory needed by the executor to store intermediate tensor results ('scratch'). this memory isn't seen or accessible by the user of mxnet at all, it is something mxnet budgets when an executor is created based on all kinds of things like whether certain ops can be done in place, whether you need grads, etc.


Disabling line wrap for long lines

Is there any way to disable line wrapping within the editor and have horizontal scrolling instead? This would be important for editing YAML containing long strings.

@itssimon This will complicate the editor. If you need to use more complex functions, you can choose @uiw/react-codemirror, @uiw/react-monacoeditor.


Watchdogs, Keep-alive, recovery, and fail-safe framework.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Keep-alive fixed, a new PR created for watchdog framework


Need accordion declare it in the controller

I need to declare a scope variable in the controller for this accordion. Can I do that?

Not sure if this is what you're asking for, but take a look at issue #13

Anyway, I'll close the issue, but feel free to re-open it if that is not the case.


OSError: [Errno 121] Remote I/O error when using screen

Steps to reproduce

set up cattmate and start with a screen attached and enabled in config.py (use_display = True)
Adjust the volume a bunch

expected: volume is adjusted and shown on screen
actual: volume stops being adjust and screen stops working  because of an error:
send vol update: 9
Traceback (most recent call last):
  File "cattmate.py", line 112, in <module>
    main()
  File "cattmate.py", line 91, in main
    screen.display(current_volume)
  File "/home/pi/cattmate/Ssd1306.py", line 18, in display
    self.screen.begin()
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_SSD1306-1.6.2-py3.7.egg/Adafruit_SSD1306/SSD1306.py", line 148, in begin
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_SSD1306-1.6.2-py3.7.egg/Adafruit_SSD1306/SSD1306.py", line 247, in _initialize
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_SSD1306-1.6.2-py3.7.egg/Adafruit_SSD1306/SSD1306.py", line 129, in command
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_GPIO-1.0.4-py3.7.egg/Adafruit_GPIO/I2C.py", line 116, in write8
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_PureIO-0.2.3-py3.7.egg/Adafruit_PureIO/smbus.py", line 256, in write_byte_data
OSError: [Errno 121] Remote I/O error

I wonder if we're updating the screen too often? Right now it's getting updated every cattmate_config.refresh_wait which ships at 0.01 seconds ( 10ms) which maybe is too fast?

Huh - changing cattmate_config.refresh_wait to 1.0 (1000ms) doesn't seem to fix it.... Maybe we could put the calls to ssd1306.display in a try/except and do a retry some how?  See this (simple) and that(complex) for a howto.

i just refactored the screen code in #2, so updated the stack trace above.  I put another defensive time.sleep() in there which didn't seem to help.  Oh...maybe if I increase it now to 100ms?  I'll try now ;)

interesting! at 500ms i'm still getting the error, so likely the timing theory as root cause isn't right.  Also, I saw it error on the first call to screen.display() (currently line 75), which would be the very first update.  Maybe the screen gets wedged in a bad state some how? Oh...I'll try a totally different screen in case this specific piece of hardware is bad.  hadn't thought of that!

trying the same hardware still, but I noticed that it's erroring out in different spots in the class, just in case it's relevant:
Traceback (most recent call last):
  File "cattmate.py", line 112, in <module>
    main()
  File "cattmate.py", line 103, in main
    screen.display(current_volume + ' ;)')
  File "/home/pi/cattmate/Ssd1306.py", line 51, in display
    self.screen.display()
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_SSD1306-1.6.2-py3.7.egg/Adafruit_SSD1306/SSD1306.py", line 167, in display
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_SSD1306-1.6.2-py3.7.egg/Adafruit_SSD1306/SSD1306.py", line 129, in command
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_GPIO-1.0.4-py3.7.egg/Adafruit_GPIO/I2C.py", line 116, in write8
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_PureIO-0.2.3-py3.7.egg/Adafruit_PureIO/smbus.py", line 256, in write_byte_data
OSError: [Errno 5] Input/output error

different screen and different jumper cables didn't fix it :(

refactored the class a 2nd time and still getting the error :(
Traceback (most recent call last):
  File "cattmate.py", line 113, in <module>
    main()
  File "cattmate.py", line 104, in main
    screen.display(current_volume + ' ;)')
  File "/home/pi/cattmate/Oled.py", line 72, in display
    self.disp.display()
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_SSD1306-1.6.2-py3.7.egg/Adafruit_SSD1306/SSD1306.py", line 167, in display
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_SSD1306-1.6.2-py3.7.egg/Adafruit_SSD1306/SSD1306.py", line 129, in command
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_GPIO-1.0.4-py3.7.egg/Adafruit_GPIO/I2C.py", line 116, in write8
  File "/usr/local/lib/python3.7/dist-packages/Adafruit_PureIO-0.2.3-py3.7.egg/Adafruit_PureIO/smbus.py", line 256, in write_byte_data
OSError: [Errno 5] Input/output error

latest PR #5 seems to have fixed it?  Let's close and hope it has.


Some elbow tracking info

TODO:

Component lists page
Body proportion image (with purple person)

Can merge this PR now btw.
Current component lists page seems to only work with 5 full trackers, then it just adds more IMUs.
I know some Javascript but I'd prefer someone else with more experience to do it :p
As for the body proportion image, I'd like @calliePepper to help make pictures :3


how to run all test caes from a line number

hi,folks
if i want to run all test cases from a line number like 99, how can i achieve it. thx.
now i know follow cmds format:

cucumber ${file_path}
cucumber ${file_path}:${num}
cucumber ${file_path}:${num}:${num}:${num}

if i have lots of test cases , using the third cmd is very Uncomfortable.

Hello @cucushift - for questions, please try one of the support channels. You can find them here: https://cucumber.io/support. GitHub issues are for bugs or feature requests.

That said, maybe using tags can help you? See the docs: https://docs.cucumber.io/cucumber/api/#tags

@mlvandijk thx, i know tags, it can't offer much help.


Fixed #20456: easier unit-testing for CBV

Added a class method to return an instance of a CBV.
This allows to unit test part of a CBV directly, without using
the client.
Note: as the ticket didn't have much discussion going on, I did a POC (only modified a single test to show how it would work), feedback on it needed before going further

Closing in absence of follow-up, feel free to send a new one if you want to continue working on this, thanks!


Add CDNJS version badge in readme

This will add a badge that shows the lib version on CDNJS and link to its page on CDNJS!
Thanks!

I think I should fix your end...

Fixed.

Cool!

Good to merge now?


Study a way of managing size changes

What to do when the view size changes? The signature should be cropped or shrinked?

IMHO, just scaling it should be the propper behaviour. I will never crop it.

Scaling to fit would be my preference too, or otherwise allow programmer to set own preference.
Working right now on a way of restoring the signature on activity restart and on orientation change (in my project, not the library) - in my case the size of the SignaturePad would change depending on orientation.
Have been trying to use getSignatureSvg() to get the signature as SVG and then later parsing and painting it as a bitmap using BigBadaboom/androidsvg library. It seems to work fine - let me know if you want I can share the code with you - aside from a couple of issues, on to set a viewbox and new document size on the SVG to be able to scale it and second where subsequent getSignatureSvg() after the setSignatureBitmap() would return an empty SVG root with just the size and no contents.

library should manage this automatically, scale image would be the easier way

Scaling to fit would be my preference too, or otherwise allow programmer to set own preference.
Working right now on a way of restoring the signature on activity restart and on orientation change (in my project, not the library). In my case the size of the SignaturePad would change depending on orientation.
Have been trying to use getSignatureSvg() to get the signature as SVG and then later parsing and painting it as a bitmap using BigBadaboom/androidsvg library. It seems to work fine (let me know if you want I can share the code with you) aside from a couple of issues: one where it needs to set a viewbox and new document size on the SVG to be able to scale it and second where subsequent getSignatureSvg() after the setSignatureBitmap() would return an empty SVG root with just the size and no contents.

My signature bitmap getting changed on orientation change. Struggling to resolve the issue from few days. Can you please share the code of this.


Update local-windows-event-log-source.md

Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

Thank you so much, @jd-sumo !


Testing in compose-destinations

Hello,
I to write a test case for my project with compose-destinations but can't get the call back method can you please explain with example how can we achieve UI testing using compose-destination

Hi @bikrammhz1
I need more context to be able to help you. Can you show me an example of a test you're trying to write?

Close since we have another issue to improve documentation on this topic


Multi-Arch Issues with AWS CLI

Feature Request

What problem are you facing?

When running function-shell on ARM-based nodes, calls to the AWS CLI installed in the function fail, because the Dockerfile for this function only installs the x86 version of the AWS CLI.

How could this Function help solve your problem?

Update the Dockerfile and build pipelines to install the appropriate aws-cli based on the arch

closing due to PR #25


Dask dataframe cannot do any group_by sum

Hi All,
I have a dask dataframe with 5 columns
Categorical: Priority_Of_Inc
Categorical: Type_Of_Inc
Categorical: YEAR
Categorical: MONTH
Integer: r c(constant 1)

for col in ['Priority_Of_Inc', 'Type_Of_Inc', 'YEAR', 'MONTH']:
df[col] = df[col].astype('category')
print(df.groupby(['Priority_Of_Inc'])["r"].sum())

And NOPE, doesn't work. And yes I have tried

df.compute()

, but it keep hitting errors. So I had to convert back to pandas, so that I can to groupby-sum.
How should I do groupby-sum in dask dataframe?
FYI, crosstab also fales with dask dataframe
Thanks!
All the best,
Kathy

Hi Kathy,
Thanks for the issue.  If possible, would you be able to produce a minimal reproducible example with the error that you've run into?  It's somewhat difficult to help otherwise.
Groupby-sums do generally work, here is one such example
In [1]: import dask.dataframe as dd, pandas as pd

In [2]: df = pd.DataFrame({'x': [1, 2, 1, 2], 'y': [1, 2, 3, 4]})

In [3]: ddf = dd.from_pandas(df, npartitions=2)

In [4]: ddf.groupby('x').y.sum().compute()
Out[4]:
x
1    4
2    6
Name: y, dtype: int64

If you have the time to construct a similar example that fails that would be very helpful.
See also: https://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports

Hi @KathyGCY , any further information here?

Closing as additional information is needed to reproduce the original issue. @KathyGCY please feel free to re-open this issue if you would like to continue the discussion.