Additional Accuracy Undeniable Stack Overflow

Hi,
When using the juggernaut node undeniable, and the accuracy gem, mousing over the modifiers to attack speed in the calculation tab causes a stack overflow error.
Probably not a major issue, but could help you find other problems.
Love your work,
Theshadedone

I can't manage to reproduce this exact case, but I presume this is the same overflow error that's been appearing since the last update; I've already fixed it for the next update, which will be out shortly. Anyway, the error message should stop appearing if you close and reopen the build, assuming it's the same error.


Change GwentAPI.map second argument to default to .one()

Since GwentAPI.map will be removed, there's no point tweaking the current implementation.


Rogue Trader Ranged Weapons->Misfortunes section overlapping Insanity/Corruption Section.

When you add Ranged Weapons, Gear, and/or Acquisitions everything begins to shift downward but does not move the next section of the sheet (Wounds/Fate Points/ Insanity/Corruption) down as well. You can add a Melee Weapon and things will shift down just fine.
This issue was also posted on the forms here: https://app.roll20.net/forum/post/2229833/slug}

I've already posted on the board.
I think I've managed a solution.
The sheet builder is around here? :+1:


Globally define custom icon by type

Here is how it was handle in vue-toastification:
import { TYPE } from 'vue-toastification';

import CheckIcon from '@/assets/svg/success.svg?inline';
import ErrorIcon from '@/assets/svg/error.svg?inline';
import InfoIcon from '@/assets/svg/info.svg?inline';
import WarningIcon from '@/assets/svg/warning.svg?inline';

const update = {
    toastDefaults: {
        [TYPE.DEFAULT]: {
            icon: InfoIcon,
        },
        [TYPE.INFO]: {
            icon: InfoIcon,
        },
        [TYPE.SUCCESS]: {
            icon: CheckIcon,
        },
        [TYPE.WARNING]: {
            icon: WarningIcon,
        },
        [TYPE.ERROR]: {
            icon: ErrorIcon,
        },
    },
};

export default function ({ app, beforeNuxtRender, nuxtState }, inject) {
        app.$toast.updateDefaults(update);
}

I will improve the documentation, you can  use like this.

main.ts

const CustomIcon = (props: IconProps) => {
  switch (props.type) {
    case 'info':
      return '🎈';
    case 'success':
      return '✌';
    case 'error':
      return '🤣';
    case 'warning':
      return '😢';
    default:
      return undefined;
  }
};

app.use(
  Vue3Toasity,
  {
    icon: CustomIcon,
   // ...
  } as ToastContainerOptions,
);

I'm by no mean a TypeScript exert, so maybe I'm missunderstandning, but I think there is a type error in the code or your exemple is incorrect. I get this error:

Type '(props: IconProps) => "👌" | "..." | "🎈" | "✌" | "🤣" | "😢" | undefined' is not comparable to type 'IconType | undefined'.
But IconType is this:
export declare type IconType = boolean | string | number | VNode | ((props: IconProps) => VNode) | DefineComponent<IconProps, {}, {}>;
I believe the part (props: IconProps) => VNode is incorrect because "👌" is not a VNode, but a string

fixed from v0.2.7 @mrleblanc101
https://vue3-toastify.js-bridge.com/usage/icons.html#custom-icons-globally

Awesome man 🙌

@jerrywu001 You can't override one specific icon tho !
You need to override them all, I wanted to override all of them except the spinner.
I'm trying to import the default spinner from the files, but it's not exported.

@jerrywu001 You can't override one specific icon tho ! You need to override them all, I wanted to override all of them except the spinner. I'm trying to import the default spinner from the files, but it's not exported.

???? Does it mean this?
import { VNodeIcon } from './icons.tsx';

createApp(App).use(
  Vue3Toasity,
  {
    icon: VNodeIcon, // use unified icon
  },
).mount('#app');

toast.info

toast.loading

@jerrywu001 Not exacly, I want to override default/info, success, warning, error with different icons, but now I need to specify a loading icon too. I can't omit it, if I omit it there is no icon instead of falling back to the default.

Found a way in the meantime:
import { Icon } from '#components';
import Vue3Toastify, { toast, type ToastContainerOptions, type IconProps, Icons } from 'vue3-toastify';
import 'vue3-toastify/dist/index.css';

const CustomIcon = (props: IconProps) => {
    switch (props.type) {
        case 'default':
            return h(Icon, { name: 'material-symbols:info-rounded', size: '32px' });
        case 'info':
            return h(Icon, { name: 'material-symbols:info-rounded', size: '32px' });
        case 'success':
            return h(Icon, { name: 'material-symbols:check-circle-rounded', size: '32px' });
        case 'warning':
            return h(Icon, { name: 'material-symbols:warning-rounded', size: '32px' });
        case 'error':
            return h(Icon, { name: 'material-symbols:report-rounded', size: '32px' });
        case 'loading':
            return h('div', { class: 'Toastify__spinner' });
        default:
            return undefined;
    }
};


Store.Chat.find is not working

This method Store.Chat.find("5511912345678") is not working anymore to send messages to contacts not present in chat history.
Anyone solved this issue?
Thanks

https://github.com/mukulhase/WebWhatsapp-Wrapper/issues/382

@bobaoapae that solution only works if you already have a chat open with that id. To open a new chat if it doesn't exist, as Store.Chat.find used to do, you should use the following snippet:
var chat = Store.Chat.find("IDTOFIND") ;
if(chat === undefined)
{
chat = new Store.Chat._model({id:new Store.UserConstructor("IDTOFIND")});
Store.Chat.add(chat);
}
return chat;

check this https://github.com/mukulhase/WebWhatsapp-Wrapper/issues/383

@bobaoapae i was getting some weird error today doing exactly that lol

@bobaoapae  doesnot work  i get this : sentinel: "INIT DERIVED PROP"

http://prntscr.com/mxtvta


[stable/prometheus] adding custom scrape_configs seems convoluted

Is this a request for help?: No

Is this a BUG REPORT or FEATURE REQUEST? (choose one): FEATURE REQUEST

Version of Helm and Kubernetes:
Helm v2.8.2
Kubernetes v1.9.6
Which chart:
stable/prometheus
What happened:
I couldn't find a simple way to add scrape configs. The only functional way I found to do that was to copy the whole contents of serverFiles.prometheus.yml.scrape_configs default over to my values.yaml and then append my scrape configs to that
What you expected to happen:
The contents of the prometheus.yml file could be broken in such a way to allow concise addition of content to each of its sections.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:
I implemented changes in my fork. I'll follow up with a PR

same problem here :-/

Similar problem. I tried both prometheus.yml value and extraScrapeConfigs.

Hey @spiffytech !
Could you provide an example?
I can successfully add extra configs directly from the values.yml file of the chart, but when I try to pass it as argument with --set nothing happens.
This works:
# adds additional scrape configs to prometheus.yml
# must be a string so you have to add a | after extraScrapeConfigs:
# example adds prometheus-blackbox-exporter scrape config
extraScrapeConfigs: |
  - job_name: 'sample-job'
    scrape_interval: 1s
    metrics_path: /
    kubernetes_sd_configs:
      - role: endpoints

this does not:
sudo helm upgrade --install prometheus \
--set rbac.create=true \
--set server.persistentVolume.enabled=false \
--set alertmanager.persistentVolume.enabled=false \
--set alertmanager.enabled=false \
--set kubeStateMetrics.enabled=false \
--set nodeExporter.enabled=false \
--set pushgateway.enabled=false \
--set extraScrapeConfigs="|
  - job_name: 'sample-pods'
    scrape_interval: 1s
    metrics_path: /
    kubernetes_sd_configs:
      - role: endpoints
" \
stable/prometheus

I don't even see the documentation for extraScrapeConfigs on master anymore.  Does anyone understand what's going on with this repo?

Have you tried adding further scarpe configs via:
kubectl edit configmap prometheus-server


iOS: Mark stories read on scroll

Like the web feature. Super useful, in fact, more useful on mobile than on the web.

I don't know if you want to include this in todays release, or leave for the next one.  It's via a pref, off by default, so should be safe.

Sure, it can go in today's release. More the merrier! It's a whopping v5.


Cache generated documentation

Figure out a way to cache and update the documentation registry, and when to or not to regenerate individual pages.
This seems not so hard. We can just take a hash(even simple will be fine, faster is better) for every file, gather it in something like hashes.txt, then while building we check every page for a change of hash and re-generate/pass. We must, of course, update hashes.txt if some page was re-generated.
This issue is a part of TODO.

I thought about it for some time, so if my description above is good enough, I'm ready to assign myself for this issue.

Is there a way to do it cleanly, though? For example, I see /todo.html page is generated during build, but it's not a separate file in the source.
So, if we add the cache and I go and modify /testing POD. Will it now be smart enough to know to remove /todo.html?

I suppose - no.
If we have dynamic pages(and we have them, obviously), then we should keep something like dependency graph, which is not so simple.
But plain hash system(without dynamic pages) will cover many cases when you just want to change, for example, Operators.pod and re-generate only Operators.html, not all in the world types/exceptions/etc.
If someone willing to help with dependency graph, it's great, of course.

Sounds like a good start.

Basic implementation can be seen and tested here - https://github.com/perl6/doc/pull/725

#725 was rejected. Is this going to to proceed in any way?

Not sure about this one. The main idea is that you do not want to re-generate all pages if you fixed a typo in a single one.
With dynamic site / documentation database, this ticket seems redundant to me(but we do not have one yet).

We do have #1823. This kind of thing would have to go through that.

Re-open if needed.

See also #2542

@JJ the Pod is already cached with help of Documentable++, I am somehow sure this can be closed now. Is there anything preventing closing?

Let me check and find a suitable thing to do so that it can be closed from a commit.


Small cosmetic changes

This branch comes with the following changes:

Modifies a couple of error messages in Finder module
Remove ~== as it was unnecessary

Current coverage is 92.69%

Merging #31 into master will decrease coverage by -0.05% as of 672bb8c

@@            master     #31   diff @@
======================================
  Files           61      60     -1
  Stmts         1612    1602    -10
  Branches         0       0
  Methods          0       0
======================================
- Hit           1495    1485    -10
  Partial          0       0
  Missed         117     117

Review entire Coverage Diff as of 672bb8c

Powered by Codecov. Updated on successful CI builds.

Can you please squash these or open 2 separate PRs?

Looks awesome 😁 Just make sure to open an issue regarding ErrorPrinter functions wrapped in an enum. Thanks! :beers:


Using wrong time window when aggregating by day

Apache StreamPipes version
dev (current development state)
Affected StreamPipes components
Processing Elements
What happened?
When aggregating a field over 1 day in the Data Explorer, it uses the wrong time window to aggregate this field. When using data within the CET time zone and you try to aggregate them, it uses the time window from 1:00 a.m. on his day to 1:00 am on the next day, which is probably the the time window of this day in UTC.
How to reproduce?
Load this data into StreamPipes.
When looking at the raw data you will see that the values are just the current number of hours of this day for every half hour (This may depend on your current timezone):

But when you aggregate the values for every day, you can see that the first value of the day is 1 and the last value is 0.5. And a day appears for which normally no data exist:

Expected behavior
The problem is probably that the UTC time is used to aggregate the data. While the raw data is displayed depending on your current time zone.
Additional technical information
No response
Are you willing to submit a PR?
None

Let me take a look at this issue :)


AssertionError

I got errors like these:

my environment like this:
pytorch==1.7.1    python==3.8    cuda==11.0
and i run this:
conda install pytorch==1.7.1 torchvision==0.8.2 torchaudio==0.7.2 cudatoolkit=11.0 -c pytorch

Please check if you can run pytorch on your "cuda" device or not:
torch.cuda.is_available()

I checked justnow, the pytorch is on my 'cuda' device.

A typical reason causing this issue is the incompatibility  of (1) GPU device (2) CUDA version and (3) Nvidia-driver version. Have you been able to run other pytorch code on cuda on the same machine?

I tried others pytorch code on the same environment , and failed.
So i changed another environment to run your code, and got en error like this

File "tools/demo_track.py", line 77, in inference
img, ratio = preproc(img, self.test_size, self.rgb_means, self.std)
TypeError: preproc() takes from 2 to 3 positional arguments but 4 were given
What should i do?

Now i can successfully run the the tracker on a provided demo video from Youtube: python3 tools/demo_track.py --demo_type video -f exps/example/mot/yolox_dancetrack_test.py -c pretrained/bytetrack_dance_model.pth.tar --path videos/dance_demo.mp4 --fp16 --fuse --save_result --out_path demo_out.mp4
But i failed to run the evaluation code with mot17 or mot20.
I got an error like this:
OSError: [Errno 28] No space left on device

I want to know that how big about the device should i use.

You have no disk space. You do not need much, 50MB might be just enough if you don't save any visualization.


allow selection of public and active when creating watchlists

Currently these options are only available when editing a watchlist:

This is in pull-request #133


Players show on wrong teams

Example demo:
https://www.hltv.org/matches/2311884/g2-vs-cloud9-pgl-major-krakow-2017-main-qualifier
Shox shows under Cloud9 and n0thing shows under G2.

A new major version has been released, closing the issue 🧹


Test Ruby release candidates on CircleCI

It's not the latest nightly build, and no RC image is available for JRuby, but perhaps the MRI Release Candidate offers enough coverage that we can drop ruby-head and jruby-head from .travis.yml
The RC builds should be “allowed failures” (if we enable “Require status checks to pass before merging” on master branch.)

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

You'll have to rebase on top of master.

MRI Release Candidate offers enough coverage that we can drop ruby-head and jruby-head from .travis.yml

Agreed.

The RC builds should be “allowed failures” (if we enable “Require status checks to pass before merging” on master branch.)

Yep, totally.

MRI Release Candidate offers enough coverage that we can drop ruby-head and jruby-head from .travis.yml

Agreed.

🎉
Expect one more PR from me for cleaning up .travis.yml etc. 😄

I'm sure @koic will disagree with this :smile:. I'm kind of with him here, but I can see how keeping a single CI system & config might be worth removing ruby-head builds.

Oh, you are absolutely right. I am sorry Iforgot about that comment @koic.
It would be very, very convenient if CircleCI would just maintain a ruby:trunk Docker image :-)

I also think about this problem when I have time 😃 Thanks for the great work!


Various pattern additions.

Grok pattern additions related to following log lines:
Sep 07 07:14:19 mail1 postfix/smtpd[40744]: disconnect from unknown[unknown] commands=0/0
Sep 07 07:17:39 mail1 postfix/anvil[40746]: statistics: max connection rate 1/60s for (smtp:unknown) at Sep  7 07:14:19
Aug 31 02:08:14 mail1 postfix/smtpd[34815]: lost connection after BDAT from bid47lw.transactional.hubspotemail.net[<IP_ADDRESS>]
Aug 31 02:08:14 mail1 postfix/smtpd[34815]: disconnect from bid47lw.transactional.hubspotemail.net[<IP_ADDRESS>] ehlo=2 starttls=1 mail=1 rcpt=0/1 bdat=0/1 commands=4/6
Aug 27 18:48:55 mail1 postfix/smtp[31995]: connect to mx1.FreeBSD.org[2610:1c1:1:606c::19:1]:25: Address not available
Aug 30 13:56:20 mail1 postfix/smtp[34352]: connect to mx.impressivevalue.com[<IP_ADDRESS>]:25: Operation timed out
Sep 25 15:33:04 mail1 postfix/smtp[9922]: 745137AA: lost connection with hotmail-com.olc.protection.outlook.com[<IP_ADDRESS>] while sending RCPT TO
Oct 21 05:35:02 mail1 postfix/smtpd[36291]: disconnect from unknown[<IP_ADDRESS>] ehlo=2 starttls=1 mail=1 rcpt=0/1 rset=1 noop=1 quit=1 commands=7/8
Nov 03 09:08:48 mail1 postfix/smtp[9862]: 840197AA: host mg2.egov.bg[<IP_ADDRESS>] refused to talk to me: 450 4.3.2 try again later
Aug 27 21:57:11 mail1 postfix/verify[32135]: cache lmdb:/var/lib/postfix/verify_cache full cleanup: retained=724 dropped=6 entries
Sep 26 03:41:14 mail1 postfix/verify[10346]: cache ??????????????????????????????????? partial cleanup: retained=1 dropped=0 entries

Thanks for all the work @dpslavov, I cleaned it up a bit and committed, see #184.


chore(deps): tweak definition of libpfm dependency

Followup to #12568.
Remove the str() wrapper around Label(), which confuses the deps cacher script bazel/deps-cache.py.
Also remove the /download suffix from the URL, that similarly confused the deps cacher, and which appears to be superfluous.  This also removes the need for the type specifier.
Finally, reorder the maybe() arguments to match the other invocations. Provide an alternative URL to

This change is

I wonder if we should tweak this to define an empty project instead. We don't really use it, and it is optional in google/benchmark

Understanding the benefits of "perfcounters" is another whole thing, so I'll leave this as-is for now.  Thanks.


Add EVM requests

[x] Implement NativeOracleRequest EVM contract
[x] Stream from accountstate table with Hyperion and parse data passed for request function
[x] On request found in accountstate notify oracles
[x] Implement oracle EVM request parsing
[x] Implement oracle EVM response

Script was moved to the Telos Oracle Scripts repo


Problem with Dead Link?

Challenge Make Dead Links using the Hash Symbol has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/601.4.4 (KHTML, like Gecko) Version/9.0.3 Safari/601.4.4.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Please be specific in you issue next time.
Have you confirmed others are getting the same error in the Chat Room?
Read the Help I have found a bug before posting a issue.
Closing as not meeting guidelines.
Thanks and Happy Coding.

The same problem happened to me too, there is certainly a bug there.

I refreshed the page and the link in the anchor element was empty when the page loaded.
I then inserted # character to make the link dead, like this
Ctrl+Enter and it worked.

Yes, bug exists. Refreshing the page fixes it.

Issue is still there. Kindly reopen the bug.

Hi @makcakaya @AT1012 @chennakesava thanks for reporting this, but to resolve any issue we need very specific steps to reproduce and screenshots to support the analysis.
I request you guys to please provide the same, and we will be more than happy to understand the problem and Re-open the issue, and try to fix it.
I have tried to check this again but with the little information as it has I have been unable to reproduce the same.
If you guys need help in describing the steps please reach us in the help chat, we will definitely help in resolving this.
Thanks and Happy coding!

There is still a bug.

Mine has worked thanx @Kayelo

I have fixed my issue.


On-Demand Image reading

It seems that currently images are being read into memory all at once and then processed.
This has caused more than one experiment of mine to fill the memory, for example for more than 6000 1080p images.
Wouldn't it be better to read images on-demand, when they need to be processed?
This could even be achieved by creating a class (could even derive of cv2.Mat) that works with images and reads them only when their contents need to be accessed.

Yes, you are right. I think this is easily do-able. I will look into it after my on-going submission, but I presume you would have already solved it on your end?

I have not. I'm trying to get to a solution that won't involve rewriting every compute_map_features function, but I'm not so sure that's possible because of the list-exclusive functions called (like len)

For the future, I believe it's more pythonic to do for i,e in enumerate(list) rather than for i in range(len(list)), plus that would work with generators as well as lists. I might simply change that in every compute_map_features and replace ref_images_list with a generator.


Tests are leaving empty temp dirs around on Linux

After the unit tests run (even if they pass successfully, empty temp dirs are being left around:
$ ls ~/.armi/
0wg1-2024071536823796283756293752/
0wg1-2024071536823796283326236423/
0wg2-2024071536823796226263623623/
0wg2-2024071536823796283756623623/

I imagine this is happening due to some logic not working on Linux in TemporaryDirectoryChanger.
NOTE: I can reproduce this bug on various Linux distros, but not on Windows.

Those kinds of files can be left around when a test fails and the cleanup steps don't work. There is a big platform-dependent difference between how cleanup gets triggered after ctrl-c aborts based on signals which may be causing this:
https://github.com/terrapower/armi/blob/d0092e8b022c593e341c09cecb4f597687222417/armi/__init__.py#L350-L352
Contextual sidenote: the ~/.armi dir is made in $HOME on linux (aka ~), but it should be identically made in Windows, generally in %USERPROFILE%, which I think is more like C:\users\[username]\AppData\armi or something like that.


Feature Request: change behavior of Adapter when it detects a scan already in progress

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[X] iOS
[X] Android
[] tvOS
[] macOS
[] .NET Standard
UWP IS NOT SUPPORTED DURING THE BETA PERIOD - ONLY PRs ARE WELCOME FOR NOW

Version of OS(s) listed above with issue
ios 10+, post-Lollipop android, but really any version
Version of Library
based off of 5.2.0, but applies to any
Expected Behaviour
FEATURE REQUEST:
For one of our applications we needed to achieve a specific plugin behavior: When the plugin started a scan for BLE devices and the adapter determined that there already was a scan ongoing it throws an exception: throw new ArgumentException("There is already an active scan"); This happens in the Scan function for each Adapter.
Android :
public override IObservable Scan(ScanConfig config)
{
if (this.IsScanning)
throw new ArgumentException("There is already an active scan");
...
iOS:
public override IObservable Scan(ScanConfig config)
{
config = config ?? new ScanConfig();
        if (this.Status != AdapterStatus.PoweredOn)
            throw new ArgumentException("Your adapter status is " + this.Status);

        if (this.IsScanning)
            throw new ArgumentException("There is already an existing scan");

We modified this behavior to automatically stop the detected scan and then continue to start a new scan. It would be great if this could at least be an optional behavior (or the standard behavior).
What we are doing is this:
Android:
public override IObservable Scan(ScanConfig config)
{
if (this.IsScanning)
{
this.context.StopScan();
this.scanStatusChanged.OnNext(false);
}
iOS:
public override IObservable Scan(ScanConfig config)
{
config = config ?? new ScanConfig();
        if (this.Status != AdapterStatus.PoweredOn)
            throw new ArgumentException("Your adapter status is " + this.Status);

        if (this.IsScanning)
        {
            this.context.Manager.StopScan();
            this.ToggleScanStatus(false);
        }

Actual Behavior
Please see the description above.
Thank you very much!
Code of Conduct

No Library Update Requests
No Timeline Requests
Please read through the documentation and FAQs
Please be respectful of my time. I put a lot of effort into my libraries.  I try to support the issue requests within a day.  I'm not psychic, so please take the time to fill out the above form as complete as possible

@aritchie Where is reported this

For iOS, are you using any of the built in plugin iOS backgrounding stuff on the adapter (BleAdapterConfiguration)?

@biapar I have no idea what you are asking me here?

I read comments and you wrote about BleAdapterConfiguration. So I try to find some info about BleAdapterConfiguration.

@biapar This only applies to iOS, where the plugin provides additional configuration options for accessing the BLE adapter. That BLEAdapterConfiguration class passed to the Init function can be viewed here: https://github.com/aritchie/bluetoothle/blob/787fd343f35e25bba0c004110c9f0655f76f6e55/Plugin.BluetoothLE.Apple.Shared/BleAdapterConfiguration.cs
It's easy to find with the github repo search tool at the top of the page.
Hope this helps!


encode the fields __typename as part of the inserted object

This ticket is about addressing the current dependency of the resolver functions on the object ID for accessing the object type name. Inline fragments (whether used for interfaces or unions) depend on the method __resolveType to get the underlying object type. For example, we currently do:
    SomeInterface: {
        __resolveType: (parent, args, context, info) =>
            return parent._id.split('/')[0]
    }

However, if we instead assume that the __typename field is actually stored as part of the document then we would return this field immediately, regardless of the ID format:
    SomeInterface: {
        __resolveType: (parent, args, context, info) =>
            return parent.__typename
    }

This also means that we need to update create and createEdge to include this new field.

I knew I had seen __typename somewhere! https://graphql.org/learn/queries/#meta-fields
Is this not reachable at the resolver level? And we hence don't have to actually store it ourselves in the db.
Or do we only have access to later, and then can't use it in the resolver as we require?


faq smooth transition and animations

Issue #660 : Solved FAQ js
Before version can be seen in issue
After:
Changes made:

change color on hover
button rotate 180 deg with smooth transition - no jerk now
smooth transition while opening and closing faq.
https://github.com/user-attachments/assets/4b42f0fa-0b1e-4c16-9694-73a2db5c5626

Additional changes: Also enhanced theme switch button on top right corner.

@mansiruhil13 kindly review this PR.

@mansiruhil13 kindly check if the issues are resolved and can be merged now?


No one responding to Terra Bridge issues

What’s going on? Why hasn’t anyone addressed all these incomplete transfers? I’m having an issue completing a transfer from Terra to Harmony and no one with Harmony or Terra seems to care.

experiencing the same thing right now

Can you post txid?? I can check the relay result.


Checking if composer exists

The composer step should check if composer exists globally, if not then composer.phar then check for both in the deploy path and finally full back to downloading it into the deploy path

Resolved in f1ca4f10328a253bd323a8a198d617a63c11450f


[EventHubs/ServiceBus/SchemaRegistry] add pytyped line to setup.py

adding pytyped to setup.py as per typing doc

@swathipil please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

@microsoft-github-policy-service agree company="Microsoft"

API change check
API changes are not detected in this pull request.

@microsoft-github-policy-service agree company="Microsoft"


Better README needed

What should we have in the README file?

We should also have a better README on the dockerhub page


fix: update to aegir v31

This is just a draft to get some metrics from aegir v31 in this repo, theres some types checking failing, i can only bundle with yarn and not npm, browser cov hangs.
ref:
https://github.com/ipfs/aegir/pull/737

As you mentioned before, one of the things that needs to be changed here is the types.ts file.

Yes the migration guide for v31 will cover all of that, i just mentioned in the OP issues i found specific to this repo.


HAWQ-156. Fix hd_work_mgr unit tests.

The tests were using an the wrong cast to access segment info. The old struct CdbComponentDatabaseInfo should have been replaced by Segment.

+1. good catch


Mark unused but desired private constructors as such

Description
The flutter codebase has "unused" but intentional private constructors, in place to prevent instantiation and extension. The analyzer will begin to report unused private constructors. These constructors need to be marked with // ignore: unused_element.
Related Issues
https://github.com/dart-lang/sdk/issues/35108
Tests
I added no tests.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

CC @goderbauer


Theme color - individual setting for different wallet

Can we have an individual setting for different wallet?
I always prefer black theme but my other wallet have no luna staked, when I switch to it, it reset to basic theme
and i need to change it back everytime its quite troublesome

I understood. I will improve it.

6bbefdc solved this.
Actually it does not work exactly like what you said, but at least you will keep the dark mode.


Tooltip on avatar not working

<vs-tooltip text="Add account"  position="bottom" >
	<vs-avatar size="45px" text="+"/>
</vs-tooltip>

Hi,
I have the same issue but in my case the tooltips never open.

I have the same issue. Even the same code from the official site did not work.

@bigperson @SinestroWhite You could try this tricks https://github.com/lusaxweb/vuesax/issues/490#issuecomment-567346453.


build(ci): add greenkeeper-lockfile

resolves #19

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


touch: implement -

Closes #3157
Issues:

Still doesn't seem to pass the GNU empty-file test nevermind, it passes, I just forgot to rebuild after fixing it.
Solution isn't portable (just opens /dev/fd/1)
On Windows, involves a bunch of low-level unsafe code

This is failing on windows:

---- test_touch::test_touch_changes_time_of_file_in_stdout stdout ----
current_directory_resolved:
touch: C:\Users\RUNNER~1\AppData\Local\Temp\.tmpDT63Tf\test_touch_changes_time_of_file_in_stdout
run: D:\a\coreutils\coreutils\target\x86_64-pc-windows-msvc\debug\coreutils.exe touch -
thread 'test_touch::test_touch_changes_time_of_file_in_stdout' panicked at 'Command was expected to succeed.
stdout =
 stderr = touch: cannot touch '/dev/fd/1': No such file or directory
', tests\common\util.rs:174:9

This is failing on windows:

---- test_touch::test_touch_changes_time_of_file_in_stdout stdout ----
current_directory_resolved:
touch: C:\Users\RUNNER~1\AppData\Local\Temp\.tmpDT63Tf\test_touch_changes_time_of_file_in_stdout
run: D:\a\coreutils\coreutils\target\x86_64-pc-windows-msvc\debug\coreutils.exe touch -
thread 'test_touch::test_touch_changes_time_of_file_in_stdout' panicked at 'Command was expected to succeed.
stdout =
 stderr = touch: cannot touch '/dev/fd/1': No such file or directory
', tests\common\util.rs:174:9

Do you know of there's a good cross-platform way to get a Path to stdout? The GNU touch just uses the file descriptor number (1) - wasn't sure how to do that in Rust idiomatically

dunno, maybe look at other tests ? :)
S

Seems like this will be tricky, the only option I've found so far to support this is to do std::io::StdOut, convert it to a raw file handle, and then call this unsafe Windows API: https://microsoft.github.io/windows-docs-rs/doc/windows/Win32/Storage/FileSystem/fn.GetFinalPathNameByHandleA.html
Will set up a windows VM soon and try it out.

Almost got it, there's some gross <?> in the output probably because I am not navigating the string conversions correctly.

Ok, I think this should be okay for now. This is my first time doing unsafe Rust and using Win32 APIs, so there may be improvements to be made.

Not really sure what is causing this CI failure: https://github.com/uutils/coreutils/runs/5267054489?check_suite_focus=true#step:12:318

I've never seen that before, but it might indeed be unrelated. We'll see if it oersists on this run

I did do some UTF-16 string stuff here, maybe it's not impossible I've inadvertently caused some spooky action at a distance.

This function has a comment about the same function I'm using : https://github.com/uutils/coreutils/blob/8e79b7fcb88916805e2a9b005900488217e83eea/tests/common/util.rs#L770-L776
maybe I've done something not thread-safe. Investigating further

Wasn't able to reproduce using this setup on Windows 11:
$ gcc -v
Using built-in specs.
COLLECT_GCC=C:\Users\devin\Documents\compilers\mingw32\bin\gcc.exe
COLLECT_LTO_WRAPPER=c:/users/devin/documents/compilers/mingw32/bin/../libexec/gcc/i686-w64-mingw32/10.3.1/lto-wrapper.exe
Target: i686-w64-mingw32
Configured with: ../configure --prefix=/R/winlibs32_stage/inst_gcc-10-20211112/share/gcc --build=i686-w64-mingw32 --host=i686-w64-mingw32 --with-pkgversion='MinGW-W64 i686-posix-dwarf, built by Brecht Sanders' --with-tune=generic --enable-checking=release --enable-threads=posix --with-dwarf2 --disable-sjlj-exceptions --disable-libunwind-exceptions --disable-serial-configure --disable-bootstrap --enable-host-shared --enable-plugin --disable-default-ssp --disable-rpath --enable-libstdcxx-pch --enable-libstdcxx-time=yes --disable-libstdcxx-debug --disable-version-specific-runtime-libs --with-stabs --disable-symvers --enable-languages=c,c++,fortran,lto,objc,obj-c++ --disable-gold --disable-nls --disable-stage1-checking --disable-win32-registry --disable-multilib --enable-ld --enable-libquadmath --enable-libada --enable-libssp --enable-libstdcxx --enable-lto --enable-fully-dynamic-string --enable-libgomp --enable-graphite --enable-mingw-wildcard --with-mpc=/d/Prog/winlibs32_stage/custombuilt --with-mpfr=/d/Prog/winlibs32_stage/custombuilt --with-gmp=/d/Prog/winlibs32_stage/custombuilt --with-isl=/d/Prog/winlibs32_stage/custombuilt --enable-install-libiberty --enable-__cxa_atexit --without-included-gettext --with-diagnostics-color=auto --enable-clocale=generic --with-libiconv --with-system-zlib --with-build-sysroot=/R/winlibs32_stage/gcc-10-20211112/build_mingw/mingw-w64 --enable-large-address-aware CFLAGS=-I/d/Prog/winlibs32_stage/custombuilt/include/libdl-win32
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 10.3.1 20211112 (MinGW-W64 i686-posix-dwarf, built by Brecht Sanders)

$ rustup show
Default host: x86_64-pc-windows-gnu
rustup home:  C:\Users\devin\.rustup

installed toolchains
--------------------

stable-i686-pc-windows-gnu (default)
stable-x86_64-pc-windows-gnu
nightly-x86_64-pc-windows-gnu

active toolchain
----------------

stable-i686-pc-windows-gnu (default)
rustc 1.58.1 (db9d1b20b 2022-01-20)

Very weird. I'll see if I can get closer to the CI environment when I have time.

oh wait! I repro'd it locally (turns out rustup toolchain default didn't recompile the tests that way), investigating

Ok, I figured it out. It was from mixing the windows crate with the winapi crate. I don't know the exact reason why but using winapi like the rest of the codebase does fixed the issue.

@tertsdiepraam I'm unable to reproduce the Documentation/warnings failure locally and I'm not sure it has anything to do with my code - please advise

I think they're due to another PR (https://github.com/uutils/coreutils/pull/3182/checks), so you don't have to fix them :)

Wahou
Warning: Changes from main: PASS +3 / FAIL -3 / ERROR +0 / SKIP +0
bravo

@sylvestre @tertsdiepraam - should I rebase onto main or merge main back into my branch?

Preferably rebase, but I don't think it's necessary. GitHub executes the actions on a temporary commit that merges your branch into main. So I've restarted the CI, which should be enough.

cool:
Warning: Congrats! The gnu test tests/touch/empty-file is now passing!
Warning: Congrats! The gnu test tests/touch/read-only is now passing!


fix(nuxt): remove trailing slash before checking if prerendered

🔗 Linked issue
resolves https://github.com/nuxt/nuxt/issues/24456
resolves https://github.com/nuxt/nuxt/issues/24392
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
As rightly pointed out by @Adamkillander96 in https://github.com/nuxt/nuxt/issues/24456#issuecomment-1827500433, we strip trailing slashes when creating the app manifest, and it makes sense to remove these when checking if a route has been prerendered (for fetching payloads, etc.)
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

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

:exclamation: No coverage uploaded for pull request base (main@09161d0). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #24516   +/-   ##
=======================================
  Coverage        ?   58.65%
=======================================
  Files           ?        5
  Lines           ?      861
  Branches        ?       46
=======================================
  Hits            ?      505
  Misses          ?      356
  Partials        ?        0

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

Wow, that was fast! 😎


rprompt on same line as lprompt, not one second line

Greetings,
when I set
POWERLEVEL9K_PROMPT_ON_NEWLINE=true

the left prompt appears on the same line as the rprompt, and not the line below (as seen in the image associated with the variable).

My config is as follows. My terminal is "terminator", though this also happens on "gnome-terminal"
POWERLEVEL9K_MODE='awesome-fontconfig'
POWERLEVEL9K_LEFT_SEGMENT_SEPARATOR='▶'
POWERLEVEL9K_RIGHT_SEGMENT_SEPARATOR='◀'
POWERLEVEL9K_PROMPT_ON_NEWLINE=true
POWERLEVEL9K_MULTILINE_FIRST_PROMPT_PREFIX=""
POWERLEVEL9K_MULTILINE_SECOND_PROMPT_PREFIX="\e[44m\e[1m\e[33m $ \e[0m\e[34m▶\e[0m "

POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(context time dir)
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(status vcs)

POWERLEVEL9K_SHORTEN_DIR_LENGTH=3
POWERLEVEL9K_SHORTEN_STRATEGY="truncate_middle"

POWERLEVEL9K_STATUS_VERBOSE=false
POWERLEVEL9K_TIME_FORMAT="%D{%T}"

@Tritlo just to be clear: You want the RPROMPT on the second line, right? I think there is no flexpoint for that now. We put some effort in showing the RPROMPT on the first line...
Here are the the relevant lines.

Yes, on the second line. Thanks, I'll make a fork with the relevant changes, which I can use for me.
I just want to be able to put more info, yet still be able to read the commands.

@Tritlo - We would be interested in merging this feature into mainline. Once you have developed it, could you submit it as a PR so that we can pull it in?

This adds a new variable POWERLEVEL9K_RPROMPT_ON_NEWLINE, which, if set to true, makes the rprompt appear on the newline, instead of the same line as the lprompt. If accepted, I'll add this variable to the wiki documentation as well.


Old throttled package location has changed

Throttled package handlers have changed their package links to:
gopkg.in/throttled/throttled.v2
gopkg.in/throttled/throttled.v2/store

Can you remove the packages changes from mdrollette to darknetj so this PR only includes the throttled package renaming? Thanks!

Of course! haha.


🚀 [Feature Request] - Improve Footer Section

Is your feature request related to a problem ? Please describe. ✍
Currently the footer is divided into two sections, one is for the social media icons and another for the contribution.
I believe, as an user it does not look good and appealing.
Describe briefly how you would implement this enhancement.
We can make it into one component for a better look and good User experience.

1. We can contain both the div into one footer component
2. Using flex property of CSS we can divide the both into one half each.
3. Also, we can add some styles to make it look simpler and modern.
4. Also, we can use modern looking icons for the social media icons, ex- react-icons, mui-icons, etc.

Additional context or screenshots
Current version:

Proposed changes (idea):

P.S. - I am willing to contribute to this wonderful project under GSSoC '23. Thank You !
Are you contributing for a program?
GSSoC2023
Code of Conduct

[X] I have read and agree to the Contributing Guidelines of this project.

sorry! we not need improvement in footer please


Add Rider to list of available IDEs

PR Details
JetBrains Rider installed on machine would be detected and can be used as External Editor side by side with Visual Studio.
Description

Motivation and Context
Helps to select Rider as External Editor.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

 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.

Would love to see this merged in... looking to swap to Rider.

@herocrab You can already use rider, set default ide to default in settings, associate *.sln with riders and you're good to go.
This one just enforces opening with rider instead of leaving that up to windows.
I definitely do not agree with merging this in its current state but you can fix this pr if you want as it doesn't look like @van800 has the time to clean it up.

Sorry for delay with this. I plan to return to this pull request, when I finish my primary tasks on Rider 2020.3 release.
However if anyone wants to take it over, it would also be fine.

I would still need to use VS for the shader .cs file generation, correct? Is there anything else that relies on the VSX plugin?

@herocrab and shader highlighting but I believe that's it. Anyway this is not the right place for those kinds of conversations, if you have more questions we'll continue in a github issue or on discord

I need help https://github.com/stride3d/stride/pull/818/files#diff-53f7df9d6d9e8f444e0d8cd13a43386c455d1c9237dce6b21bc0653831b10e41R32
with proper logging.


Request Table type Missing

Issue in help/c-implementing-target/c-implementing-target-for-client-side-web/adobe-target-getoffers-atjs-2.md
The Request table does not specify the accepted types for the fields. The limitations of order > purchasedProductIds makes it seem that purchasedProductIds should be passed as a string value. Sending a string value results in a 400 error - invalid JSON (itself not a very useful error). Only after viewing the documentation for sendNotification was it apparent that purchasedProductIds should be an array of strings.
The request table on this page should also have a column for type.

Thanks for your suggestion. Added link to API doc that contains that information. You can drill down to each field to see its allowable type.
https://docs.adobe.com/content/help/en/target/using/implement-target/client-side/functions-overview/adobe-target-getoffers-atjs-2.html


Make @ character removal optional when generating scoped libraries

🚀 Feature request
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[x] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
While running ng generate library with a scoped package, let's say @ui/mylib, the generated folder is ui/mylib.
An explanation for this behavior was given here : #646
Unfortunately this breaks the behavior of much monorepo tools like lerna or oao. In a big project, you'll have several libraries that have dependencies on other ones in the same monorepo. To rebuild them all in order, you'll want to automatize launching ng build task on each library, by respecting the dependency graph (starting by the leaves).
Tools like lerna can do this, but will fail when looking at a folder that has the same name as the dependency in the package.json => @ui/mylib
Therefore these tools become unusable.
Describe the solution you'd like
Make optional the removal of "@" character in the folder name, using a setting in angular.json for example ?
Describe alternatives you've considered
No workaround found yet, unless I would consider redevelop a graph building system like lerna, just because of this folder glitch.

Hi @djflex68,
Lerna
Not having @ shouldn't be a blocker from using Lerna at all. Adding the below should be sufficient. The directory name is meaningless  as it uses the name field from the package.json manifest.
{
  "packages": [
    "projects/ui/*"
  ],
  "version": "0.0.0"
}

$ yarn lerna ls --toposort --long
@ui/mylib  v0.0.1 projects/ui/mylib
@ui/mylib2 v0.0.1 projects/ui/mylib2
@ui/mylib1 v0.0.1 projects/ui/random-folder-name

oao
SInce oao uses Yarn workspaces not having @ isn't a problem at all, as again the directory name doesn't matter.
Adding the below to the root level package.json should be sufficient.
  "workspaces": [
    "projects/ui/*"
  ],

@alan-agius4  I finally get this to work, following your suggestions. My problem was a bad usage of the lerna --scope flag in combination with a wrong configuration file.
Many thanks for pointing me in the right direction !

Glad to hear that it worked for you 😀


Run tests on python 3.13

@sisp I run poetry lock because tests breaks with coverage==7.0.5. After lock only one matrix fail with python 3.13 build (windows-latest, 3.13).
You can see more details here:

Windows fail https://github.com/jairhenrique/copier/actions/runs/11149852649
Coverage fail https://github.com/jairhenrique/copier/actions/runs/11149733141

I see. Would you mind only updating the locked version for coverage instead of refreshing all locks?
poetry update coverage

It seems some other dependencies need to be updated as well.

It seems some other dependencies need to be updated as well.

I found it, is a nodeenv dependency. Updated!
Now to pass all tests we need a new version of pywin32. The project already supports python 3.13, but a new version has not been generated. https://github.com/mhammond/pywin32/pull/2260

seems like it's available already.
https://github.com/mhammond/pywin32/blob/842fbef58770d8c3a634b2e6033b95f80d19cbd3/CHANGES.txt#L32
could you update that dependency too please?

It seems GitHub hasn't added Python 3.13 to their CI runners. :see_no_evil: We'll have to wait then ...

It seems GitHub hasn't added Python 3.13 to their CI runners. 🙈 We'll have to wait then – or revert to prereleases: true .

I revert the commit

I think CI on Windows with Python 3.13 is failing now because the locked regex version has no prebuilt wheel for Python 3.13 yet. Would you mind trying to update the lock file with poetry update regex?

@yajo Could you please configure the CI jobs for Python 3.13 to be required? It's something we can't configure via PR.

Done, thanks


Improve security constraints of the envoy image

It currently needs to run as root and needs write access to the file system. Can we improve that?
See https://github.com/knative-sandbox/net-kourier/pull/272 for reference.

/close
This was done by https://github.com/knative-sandbox/net-kourier/pull/934 on upstream.


plugins/lsp: Fix enable conditions

There were two bugs in the lsp server modules:

They unconditionally evaluated extraConfig
They did not take into account if the global lsp plugin was enabled or not

Fixes #2543

I've not checked that, I'll check beforce queueing

I had a very quick scan through, all I could find was one server that has a now-redundant mkIf: https://github.com/nix-community/nixvim/blob/main/plugins%2Flsp%2Flanguage-servers%2Fdefault.nix#L29

I saw another place where there was a redundant mkIf, but I saw no other problem

@mergify queue


Exclude hidden fields from validation error

Documentation said that to exclude hidden fields from validation I should add this tag to form
data-parsley-excluded="input[type=button], input[type=submit], input[type=reset], input[type=hidden], [disabled], :hidden"
And if works perfectly... but... validation messages disappearing after that (on visible elements) . The :hidden argument cause that.
Need assistans here. Regards

Please provide a working example.

**Working**


NESTED: using a nested GET api/serializer writes only headers, fields are empty - fix in PR #36

there is already a fix, a PR (#36) opened by @paveloder
I have applied the fix locally and I can confirm it works.
My API was returning:
HTTP 200 OK
Allow: GET
Content-Type: application/json
Vary: Accept

[
    {
        "id": 1,
        "rfid_tag": {
            "id": 1,
            "number": "1234567890",
            "product": {
                "id": 1,
                "name": "Prosop mare",
                "ean": "11111",
                "description": "",
                "category": 1,
                "info": []
            },
            "status": "I",
            "info": []
        },
        "security_location": 1,
        "aknowledged": false
    },
    {
        "id": 2,
        "rfid_tag": {
            "id": 2,
            "number": "1234567891",
            "product": {
                "id": 1,
                "name": "Prosop mare",
                "ean": "11111",
                "description": "",
                "category": 1,
                "info": []
            },
            "status": "I",
            "info": []
        },
        "security_location": 2,
        "aknowledged": false
    }
]

viewset

class SecurityEventsGetXLSXViewset(XLSXFileMixin, ReadOnlyModelViewSet):
    """ Generate XLSX File """
    queryset = models.SecurityEvents.objects.all()
    serializer_class = serializers.SecurityEventsGetSerializer
    renderer_classes = (XLSXRenderer,)
    filename = 'SecurityEventsGet_export.xlsx'

serializer

class SecurityEventsGetSerializer(serializers.ModelSerializer):
    rfid_tag = RfidTagNestedSerializer()
    class Meta:
        model = models.SecurityEvents
        fields = ['id', 'rfid_tag', 'security_location', 'aknowledged']
        read_only_fields = ['created_at']

Before PR #36 patch

After PR #36 patch

@ionescu77 I've merged in pull requests #35 and #36, which created a conflict. I think I've sorted out both properly, but could you please run a test against the latest on the main branch before I push out a release and confirm that it works?

Hello, I will give it a try and report back. Thank you!

It's been a while. Thanks for your patience.
I tried local with the renderers.py from main branch. It does not seem to work as expected.
Using the example from 14 days ago I get something like

Let me try to look at #35  #36  see if I spot the trouble.

Excuse all, I deleted previous comment (it was my problem, I made some changes to serializers.py in my project and it impacted XLSX generation, too).
Test results
So:

[ ] I have copied renderers.py from main branch into my project venv lib/python3.9/srf_renderer_xlsx.
[ ] I used another nested serializer in order to test (actually multiple level nesting: RfidTag -> Product -> Category)
[ ] It seems I get my XLSX file as expected (nested product info is added as columns next to RFID Tag epc number):
[ ] congrats for the PR merge and thank you! 🏆

# api_views.py
class SecurityEventsGetXLSXViewset(XLSXFileMixin, ReadOnlyModelViewSet):
    """ Generate XLSX File """
    queryset = models.RfidTag.objects.all()
    serializer_class = serializers.RfidTagNestedSerializer
    renderer_classes = (XLSXRenderer,)
    filename = 'RfidTagNested_export.xlsx'

# serializers.py
class RfidTagNestedSerializer(serializers.ModelSerializer):
    product = ProductNestedSerializer()
    class Meta:
        model = models.RfidTag
        fields = ['id', 'number', 'product', 'state', 'info']

[
    {
        "id": 1,
        "number": "111abc",
        "product": {
            "id": 1,
            "name": "default_product",
            "ean": "",
            "description": "default product for foreign/unknown RFID tags",
            "category": {
                "id": 1,
                "name": "default_category",
                "description": "default category for foreign/unknown RFID tags"
            },
            "info": []
        },
        "state": "",
        "info": []
    },
    {
        "id": 2,
        "number": "111ABC",
        "product": {
            "id": 1,
            "name": "default_product",
            "ean": "",
            "description": "default product for foreign/unknown RFID tags",
            "category": {
                "id": 1,
                "name": "default_category",
                "description": "default category for foreign/unknown RFID tags"
            },
            "info": []
        },
        "state": "",
        "info": []
    },
    {
        "id": 3,
        "number": "111ABCD",
        "product": {
            "id": 1,
            "name": "default_product",
            "ean": "",
            "description": "default product for foreign/unknown RFID tags",
            "category": {
                "id": 1,
                "name": "default_category",
                "description": "default category for foreign/unknown RFID tags"
            },
            "info": []
        },
        "state": "",
        "info": []
    },
    {
        "id": 4,
        "number": "111ABCDE",
        "product": {
            "id": 3,
            "name": "Prosop mic",
            "ean": "222",
            "description": "",
            "category": {
                "id": 2,
                "name": "Textile Camera",
                "description": ""
            },
            "info": []
        },
        "state": "I",
        "info": []
    },
    {
        "id": 5,
        "number": "111ABCDEF",
        "product": {
            "id": 2,
            "name": "Prosop mare",
            "ean": "111",
            "description": "",
            "category": {
                "id": 2,
                "name": "Textile Camera",
                "description": ""
            },
            "info": []
        },
        "state": "I",
        "info": []
    },
    {
        "id": 6,
        "number": "string",
        "product": {
            "id": 1,
            "name": "default_product",
            "ean": "",
            "description": "default product for foreign/unknown RFID tags",
            "category": {
                "id": 1,
                "name": "default_category",
                "description": "default category for foreign/unknown RFID tags"
            },
            "info": []
        },
        "state": "",
        "info": []
    },
    {
        "id": 7,
        "number": "abc",
        "product": {
            "id": 1,
            "name": "default_product",
            "ean": "",
            "description": "default product for foreign/unknown RFID tags",
            "category": {
                "id": 1,
                "name": "default_category",
                "description": "default category for foreign/unknown RFID tags"
            },
            "info": []
        },
        "state": "",
        "info": []
    },
    {
        "id": 8,
        "number": "3034802c801c4ad6a5a4372e",
        "product": {
            "id": 1,
            "name": "default_product",
            "ean": "",
            "description": "default product for foreign/unknown RFID tags",
            "category": {
                "id": 1,
                "name": "default_category",
                "description": "default category for foreign/unknown RFID tags"
            },
            "info": []
        },
        "state": "",
        "info": []
    }
]

And the XLSX:

The fix has been released in version 0.4.1: https://pypi.org/project/drf-renderer-xlsx/0.4.1/


Add Recommendations and Contact Me buttons to navbar

Currently, there are no buttons to "recommendation" and "contact me" sections like the other sections.
It would both make the user aware of those sections and have a direct link to them.

@robzwolf done with this, already did a PR

issue #78 continues this issue after one navbar merge


cannot decode json payload into protobuf object

I had a custom resource definition named ServiceMonitor, which was created by prometheus-operator (https://github.com/coreos/prometheus-operator). And I created struct definitions as follows:
type ServiceEndpoint struct {
	Port *string `json:"port"`
}

type ServiceNamespaceSelector struct {
	MatchNames []string `json:"matchNames"`
}

type ServiceSelector struct {
	MatchLabels map[string]string `json:"matchLabels"`
}

type ServiceMonitorSpec struct {
	Endpoints         []*ServiceEndpoint        `json:"endpoints"`
	NamespaceSelector *ServiceNamespaceSelector `json:"namespaceSelector"`
	Selector          *ServiceSelector          `json:"selector"`
}

type ServiceMonitor struct {
	Metadata *k8s_io_apimachinery_pkg_apis_meta_v1.ObjectMeta `json:"metadata"`
	Spec     *ServiceMonitorSpec                              `json:"spec,omitempty"`
}

Then I called the client Create() function, it returned an error:
decode error status 404: cannot decode json payload into protobuf object
How can I solve this issue?

Does ServiceMonitor implement protobug encoding? If it does this package assumes we can't naively decode into a proto object unless it implements json.Unmarshaler

I found the root cause. The name I registered for serivice monitor was wrong. It should be "servicemonitors", previously I set it "ServiceMonitor"


Accessors or Getters

I'm trying to make function that operates with these values but it's impossible to access them once set (if values are not some kind of global constants).
Is it possible to add these getters for min, max and stepValue?

@TheKvikk Actually, min, max and stepValue are made readonly intentionally. Can you elaborate your use case? Let us discuss if we should make them public.


Prettier

Hey, I'd like to propose switching from clang-format to Prettier. I was recently going through the code of LitElement and the output of clang-format seems less clean and readable, at least to me. Other Google projects are happily using Prettier, like firebase-js-sdk.
What do you guys thinks? I'd be more than happy to make a PR.

Thanks for the feedback. Not saying you're wrong, but we're going to stick with clang-format for now since it's got some configuration options we need.


docs: update element template samples

Related to https://github.com/bpmn-io/internal-docs/issues/499

Shall we target this to master since we already merged develop into master (to build the RC)?

Good Catch :+1: Updated it.


[Q] I hit the Filestream rate limit exceeded

Hi Team,
I'm trying to fine tune starcoder from hugging face and I'm getting these errors:
"wandb: 429 encountered (Filestream rate limit exceeded, retrying in 4.4 seconds.), retrying request"
Can anyone help solve this?
!python finetune/finetune.py \   --model_path="bigcode/starcoderbase"\   --dataset_name="dataset"\   --split="train"\   --size_valid_set 10000\   --streaming\   --seq_length 2048\   --max_steps 1000\   --batch_size 1\   --input_column_name="instruction"\   --output_column_name="output"\   --gradient_accumulation_steps 16\   --learning_rate 1e-4\   --lr_scheduler_type="cosine"\   --num_warmup_steps 100\   --weight_decay 0.05\   --output_dir="./checkpoints"

Hey @ashutoshrana171, we recently rolled back a change made to rate limiting - are you still encountering these issues?

Hey @umakrishnaswamy
I settled for a lighter version of LLM model and did not see any issue.
I'll highlight if I have the same issue with heavier models.
Thanks

Uma Krishnaswamy commented:
Thanks for letting me know, and please let me know if you encounter this in the future!

WandB Internal User commented:
umakrishnaswamy commented:
Hey @ashutoshrana171, we recently rolled back a change made to rate limiting - are you still encountering these issues?

WandB Internal User commented:
ashutoshrana171 commented:
Hey @umakrishnaswamy
I settled for a lighter version of LLM model and did not see any issue.
I'll highlight if I have the same issue with heavier models.
Thanks