Received length loss

This question comes from last branch main.
    fn handle_read(&mut self, ip: &IpAddr) -> bool {
        let intf_sock = match self.intf_socks.get_mut(ip) {
            Some(if_sock) => if_sock,
            None => return false,
        };
        let mut buf = vec![0u8; MAX_MSG_ABSOLUTE];
        let sz = match intf_sock.sock.read(&mut buf) {
            Ok(sz) => sz,
            Err(e) => {
                if e.kind() != std::io::ErrorKind::WouldBlock {
                    error!("listening socket read failed: {}", e);
                }
                return false;
            }
        };

        debug!("received {} bytes", sz);

        // If sz is 0, it means sock reached End-of-File.
        if sz == 0 {
            error!("socket {:?} was likely shutdown", intf_sock);
            if let Err(e) = self.poller.delete(&intf_sock.sock) {
                error!("failed to remove sock {:?} from poller: {}", intf_sock, &e);
            }

            // Replace the closed socket with a new one.
            match new_socket_bind(&intf_sock.intf) {
                Ok(sock) => {
                    let intf = intf_sock.intf.clone();
                    self.intf_socks.insert(*ip, IntfSock { intf, sock });
                    debug!("reset socket for IP {}", ip);
                }
                Err(e) => error!("re-bind a socket to {}: {}", ip, e),
            }
            return false;
        }

        // ******************** buf.len() == MAX_MSG_ABSOLUTE != sz
        match DnsIncoming::new(buf) {
            Ok(msg) => {
                if msg.is_query() {
                    self.handle_query(msg, ip);
                } else if msg.is_response() {
                    self.handle_response(msg);
                } else {
                    error!("Invalid message: not query and not response");
                }
            }
            Err(e) => error!("Invalid incoming DNS message: {}", e),
        }

        true
    }

buf.set_len(sz) is unsafe
buf.truncate(sz) is ok

Two panic!
        for _ in 0..n {
            let name = self.read_name()?;
            let slice = &self.data[self.offset..];
            // ***************** slice.len() < 10
            let ty = u16_from_be_slice(&slice[..2]);
            let class = u16_from_be_slice(&slice[2..4]);
            let ttl = u32_from_be_slice(&slice[4..8]);
            let length = u16_from_be_slice(&slice[8..10]) as usize;
            self.offset += 10;
            let next_offset = self.offset + length;

            // Check the first 2 bits for possible "Message compression".
            match length & 0xC0 {
                0x00 => {
                    // regular utf8 string with length
                    offset += 1;
                    // **************** (offset + length as usize) > data.len()
                    name += str::from_utf8(&data[offset..(offset + length as usize)])
                        .map_err(|e| Error::Msg(format!("read_name: from_utf8: {}", e)))?;
                    name += ".";
                    offset += length as usize;
                }

In my environment, MDNS has a relatively large amount of data, and the problem of data loss is often caused by the read cache being full. So it also causes the above problems.

// ***************** slice.len() < 10

I think it's a good idea to add a check for the slice length and return Err if fails.

// **************** (offset + length as usize) > data.len()

Same here, I agree it's good to check the length here.

In my environment, MDNS has a relatively large amount of data, and the problem of data loss is often caused by the read cache being full. So it also causes the above problems.

 Did you mean the mDNS message in your env. is larger than MAX_MSG_ABSOLUTE i.e. 8966 bytes?   The RFC says "a Multicast DNS packet, including IP and UDP headers, MUST NOT exceed 9000 bytes" (here).

For the sanity checks, I've opened a PR to add them. Let me know if you have any questions or comments.

PR merged. Let me know if you have any questions.

In my environment, MDNS has a relatively large amount of data, and the problem of data loss is often caused by the read cache being full. So it also causes the above problems.

 Did you mean the mDNS message in your env. is larger than MAX_MSG_ABSOLUTE i.e. 8966 bytes? The RFC says a Multicast DNS packet, including IP and UDP headers, MUST NOT exceed 9000 bytes (here).

I did receive an oversized package, if it's illegal, then ignore it.
mdns-big.zip

PR merged. Let me know if you have any questions.

Good!

In my environment, MDNS has a relatively large amount of data, and the problem of data loss is often caused by the read cache being full. So it also causes the above problems.

 Did you mean the mDNS message in your env. is larger than MAX_MSG_ABSOLUTE i.e. 8966 bytes? The RFC says a Multicast DNS packet, including IP and UDP headers, MUST NOT exceed 9000 bytes (here).

I did receive an oversized package, if it's illegal, then ignore it.
mdns-big.zip

According to WireShark, the packet size is 1030 bytes, far from the max size (8966 bytes). I didn't see obvious problems here. Let me know if you encountered any particular issues.

All 11 frames form one package.

Oh I missed that, sorry.  It is a fragmented IP packet. We don't support fragmented IP packets yet. This would be a new feature and it will take some time to implement if I would do it. Would you be interested in creating a PR to add this new feature to support IP fragments?

Basically, an mdns datagram can not be larger than 9000 even when
fragmented into multiple UDP packets.  And also can not contain more
than one resource record.
From the mdns rfc  rfc6761:

Multicast DNS Message Size

The 1987 DNS specification [RFC1035] restricts DNS messages carried
by UDP to no more than 512 bytes (not counting the IP or UDP
headers).  For UDP packets carried over the wide-area Internet in
1987, this was appropriate.  For link-local multicast packets on
today's networks, there is no reason to retain this restriction.
Given that the packets are by definition link-local, there are no
Path MTU issues to consider.
Multicast DNS messages carried by UDP may be up to the IP MTU of the
physical interface, less the space required for the IP header (20
bytes for IPv4; 40 bytes for IPv6) and the UDP header (8 bytes).
In the case of a single Multicast DNS resource record that is too
large to fit in a single MTU-sized multicast response packet, a
Multicast DNS responder SHOULD send the resource record alone, in a
single IP datagram, using multiple IP fragments.  Resource records
this large SHOULD be avoided, except in the very rare cases where
they really are the appropriate solution to the problem at hand.
Implementers should be aware that many simple devices do not
reassemble fragmented IP datagrams, so large resource records SHOULD
NOT be used except in specialized cases where the implementer knows
that all receivers implement reassembly, or where the large resource
record contains optional data which is not essential for correct
operation of the client.
A Multicast DNS packet larger than the interface MTU, which is sent
using fragments, MUST NOT contain more than one resource record.
Even when fragmentation is used, a Multicast DNS packet, including IP
and UDP headers, MUST NOT exceed 9000 bytes.
Note that 9000 bytes is also the maximum payload size of an Ethernet
"Jumbo" packet [Jumbo].  However, in practice Ethernet "Jumbo"
packets are not widely used, so it is advantageous to keep packets
under 1500 bytes whenever possible.  Even on hosts that normally
handle Ethernet "Jumbo" packets and IP fragment reassembly, it is
becoming more common for these hosts to implement power-saving modes
where the main CPU goes to sleep and hands off packet reception tasks
to a more limited processor in the network interface hardware, which
may not support Ethernet "Jumbo" packets or IP fragment reassembly.
-Dale

Regarding IP fragmentation, it seems that UDP packet received by the socket should already be reassembled, if no dropped packets.  From ChatGPT:
In summary, IP handles fragmentation and reassembly, and this applies to both TCP and UDP traffic. UDP itself, being a simple and connectionless protocol, doesn't have built-in mechanisms for handling fragmentation or reassembly.

That means we need not (and should not) have special logic to handle IP fragments.
And as what Dale mentioned :

Basically, an mdns datagram can not be larger than 9000 even when
fragmented into multiple UDP packets.  And also can not contain more
than one resource record.

We should be good with the current max size (around 9000). For any large RR (resource record), it should be sent in a separate mDNS datagram.
Looking back at your capture, it seems that all 377 RRs are sent in a single datagram of 15794 bytes, which conflicts the RFC recommendation.

I'm curious what is generating that huge datagram!  Is that Bonjour or Avahi?
Some other library?  Something homebrewed?
What kind of device is sending that?
Thanks!
-Dale
On 2/5/24, keepsimple1 @.***> wrote:

Regarding IP fragmentation, it seems that UDP packet received by the socket
should already be reassembled, if no dropped packets.  From ChatGPT:
In summary, IP handles fragmentation and reassembly, and this applies to
both TCP and UDP traffic. UDP itself, being a simple and connectionless
protocol, doesn't have built-in mechanisms for handling fragmentation or
reassembly.

That means we need not (and should not) have special logic to handle IP
fragments.
And as what Dale mentioned :

Basically, an mdns datagram can not be larger than 9000 even when
fragmented into multiple UDP packets.  And also can not contain more
than one resource record.

We should be good with the current max size (around 9000). For any large RR
(resource record), it should be sent in a separate mDNS datagram.
Looking back at your capture, it seems that all 377 RRs are sent in a single
datagram of 15794 bytes, which conflicts the RFC recommendation.

--
Reply to this email directly or view it on GitHub:
https://github.com/keepsimple1/mdns-sd/issues/168#issuecomment-1928457607
You are receiving this because you commented.
Message ID: @.***>

I'm curious what is generating that huge datagram! Is that Bonjour or Avahi? Some other library? Something homebrewed? What kind of device is sending that? Thanks! -Dale

I guess it comes from the jmdns package used by my colleague, but I haven't personally verified it yet.

I don't think we need to support such datagram that is clearly outside of the RFC. On the other hand, I will think of how we can handle such cases more gracefully.

I had to re-learn some details about UDP socket API.  Here is a related info about the recv/recv_from API on Linux:
If a message is too long to fit in the supplied buffer, excess bytes may be discarded depending on the type of socket the message is received from.
and in the classic book "TCP/IP Illustrated, volume 1", chapter 11,  section 11.10 Maximum UDP Datagram Size, it says this about "Datagram Truncation":
What happens if the received datagram exceeds the size the application is prepared to deal with?  Unfortunately the answer depends on the programming interface and the implementation.
It then says the sockets API under SVR4 Unix does not truncate the datagram.
Based on the above, I think there is no much more we can do (or should do) besides the sanity checks in the current code.
If a mDNS datagram happens to exceed the max buffer size, it will not be decoded correctly / fully.  And if there is no truncation, the follow-up datagram will have invalid headers, etc, and will fail the decode process with an Err return. As long as there is no crash, I think we are good.

I've merged in the minor change of MAX_MSG_ABSOLUTE and added comments. Will close this issue. If you have any additional questions, please re-open this issue or open a new issue. Cheers!


Re-Review Account Soft Credits Based on New Improved Doc

https://powerofus.force.com/s/article/NPSP-Create-Soft-Credits#topic-7810

@Billflorio  do you have a doc started for this, or is something we can do in the sprint? IT was in Drafts in progress, so just wanted to make sure there wasn't a draft in progress :)

I do not. I think I just added when someone mentioned the update

Dianna and Amy are working on this

We are asking Cori to help us track down Dianna and Amy from the quip registration doc from the February sprint. We likely will start over on this script.

Uncovered that this is not about finding a script, it's about the review of the existing video and the NPSP article, as the NPSP article has had a change of intro...
I have reviewed the article and the video. I would suggest that we still keep this video (https://www.youtube.com/watch?v=6HwFzj1MXZk&list=PLU8xqF8ZASbXWAHIF0SB1P9_3G4vvRMXI&index=10) but instead of it being attributed to the Manually Create Soft Credits documentation, I would move it to the Soft Credits Overview (https://powerofus.force.com/s/article/NPSP-Soft-Credits-Overview) - the video summaries what a soft credit is, roles and rollups, with a creation demo included.
I would then suggest we have a NEW video for full soft credits vs partial soft credits showing the soft credit attribution related list, and then how to assign a full soft credit - here's the article: https://powerofus.force.com/s/article/NPSP-Create-Soft-Credits

@Salesforce-Em gotcha. So perhaps we can change the card title to something like soft credits type comparison and forget the account soft credits intro need


fix(web): don't set Accept-Encoding for requests without compression

Closes #49
Proposed Changes
Don't set Accept-Encoding header for request which doesn't use gzip - web.
Checklist

[ ] CHANGELOG.md updated
[x] Rebased/mergeable
[x] A test has been added if appropriate
[x] pub run test completes successfully
[x] Commit messages are in semantic format
[x] Sign CLA (if not already signed)

Codecov Report

Merging #50 (d228d84) into main (46fb189) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage   20.86%   20.86%
=======================================
  Files         117      117
  Lines        7542     7542
=======================================
  Hits         1574     1574
  Misses       5968     5968

Impacted Files
Coverage Δ

lib/client/influxdb_client.dart
76.30% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46fb189...d228d84. Read the comment docs.


Add support for various webpack loaders to typings

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request

It would be good to support for seamless importing with url-loader or file-loader in webpack. I really wanted html-loader to be enabled again, but sadly it doesn't seem to happen in near future.
This is what I currently have in typings.d.ts:
// url-loader
declare module "*.jpg" { const url: string; export default url; }
declare module "*.png" { const url: string; export default url; }
declare module "*.gif" { const url: string; export default url; }

I'm looking forward to adding this to the default template, or even better embedded in webpack config itself.

Turns out this is not possible with current TS spec, closing.


Play again feature

Is your feature request related to a problem?
its a pretty good feature after the match ends it should give a play again paper like mmc
Describe the solution you'd like.
add it if possible :
Describe alternatives you've considered.
nothing
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I have ensured the feature I'm requesting isn't already in the latest Frost.

Other
No response

Feature being added for Frost 9.0

Feature added.


On v2, Text component is a div

It should be a paragraph, ain't it?

You can set the HTML element with the is prop where appropriate, e.g. <Text is='p' />

yes I know we can set it with is prop.
But isn't it better to set Text as p element by default? Like what we have on v1?

@jxnblk What are the motivations behind making blocks of text a div by default? I think that using p for text is more frequent. Using <Text is="div" /> would still be possible.

Not all text on a page is a paragraph, so overusing the
 tag can also not be great. The Text component can also be used as a wrapping component around other semantic elements, where you could have something like
<Text fontSize={3}>

...

...

...

</Text>


Update Composer dependencies (2022-02-06-00-07)

Visual regression test passed!
View the visual regression test report


Duplicati

Name the integration
Duplicati - one of the most popular backup app.
https://www.duplicati.com/
You can make integration to display information:

Name of the current task (if running);
transfer rate;
amount of data.

The current task is displayed in the Duplicati as follows:
16999 files (114.23 GB) to go at 4.14 MB/s
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You're not just putting an idea out there and actually give usefull information about how to implement your module idea

I use duplicati too, I wonder if we could make a small progress bar or something similar. It could be integrated easily if the UI is good

Nevermind, they don't have an API https://github.com/duplicati/duplicati/wiki/Developer-documentation#rest-api 🗿


City status report bugged

City status report screen is bugged, showing only "$Housing$" and such placeholders for everything except city name, population and growth rate status.
I'm on a windows 10 machine, using version <IP_ADDRESS>2 with Rise and Fall and most DLC.

Could you use the bug report format provided for you when you open an issue? This will make our work a lot easier. Thanks.


address useful hl7 feedback from AZ PHD

Comments from AZ in email, 1/20/2021: regarding issues found in hl7 data:
FHS/BHS
Header information doesn't make sense. Looks like it's missing a File/Batch Sending Facility value.
MSH4, ORC21
It would be nice if this value matched what you were sending in the flat file.
HL7-Sierra Vista Canyons
Flat File-Via Elegante, Sierra Vista Canyons
OBR3
This is where we primarily look for the Accession number. It looks like you are populating OBR2 with that ID, could you also populate OBR3 with that same ID?
OBX17
It would be nice to have a human readable value in this field instead of the ID. Below is an example of how others are populating this field. We could make the ID work if necessary.
What is currently being populated:<PHONE_NUMBER>1290_DIT^^99ELR
What others are populating in this field: BinaxNOW COVID-19 Ag Card_Abbott Diagnostics Scarborough, Inc._EUA_Abbot_EUA^^99ELR

This was done

To be clear, I have not yet done any work on this.  Did @mauricereeves ?

@jimduff-usds - I have addressed issues with the MSH, FHS, and BHS segments.
For OBR-3 we won't be sending the Accession number, and we are currently going to send the Message ID instead of any other value. I don't know if that's going to be acceptable to AZ or not. I can discuss with them if that would help clarify.
OBX-17 is sending through the value that they want now.

Sorry I closed this by accident, but coincidentally was revisiting this just now!
@MauriceReeves-usds I'll add you to the thread. I like the idea of proposing that we send the Message ID

@mauricereeves , Dunno if this is possible, but it would be a nice win if we passed the message_id to all downstream hl7 schemas (and we'd also need to make sure its mentioned as the "trackingElement" for each schema)
This would be a good thing because the item lineage code attempts to grab trackingElement actual values and store them in the audit/lineage tables, since they are a value the external users would be able to refer to.  However, this hasn't worked well for hl7 and redox.  I think at the moment, few or none of these schemas have any trackingElement defined.
Strangely, the AZ HL7 schema has message_id as the trackingElement, but then never contains message_id!    I suspect this is a mistake on my part from when I set it up.  But it should throw an error in our Schema.kt code.
Jim

@jimduff-usds it's definitely possible. Right now we send it as the specimen ID and OBR-3. We could add it to the NTE section as well if you want some custom place to put it that we could refer back to, but for now I think having it in SPM-2 should be sufficient.


[CH] wrong negative decimal32 values after columnar to row

Backend
CH (ClickHouse)
Bug description
create table my_decimal(id decimal(5,2)) using parquet;
insert into my_decimal values (7),(-7);
select * from my_decimal;
[Expected result]
7.00
-7.00
and
[actual result]
7.00
42949665.96
Spark version
Spark-3.3.x
Spark configurations
No response
System information
No response
Relevant logs
No response

@liuneng1994 Did you have the same problem?

The root cause is that when convert CH Column to Spark Row,decimal32 is treated as 4 bytes length and copyed to Spark Row, while Spark expected 8 length for all decimal types.

@loneylee has met the same issue before.

#2767
It will be fixed soon.


How to simply clone an entire registry?

With clone() removed earlier this year, it's unclear what is the intended way to simply, completely clone a registry, all entities and components. Based on the example in the wiki, use of visit() is part of the long term plan, but currently to do anything with the id types it provides you need to in some way declare what component types you're using. I'm looking for a solution that doesn't require knowing the component types.

I'm working to reintroduce clone as well as stamp but in a way such that users won't pay for them if they aren't interested in using these feature.
In the meantime, even though cloning a whole registry is somewhat discouraged, it offers all the functionalities to do that already.
As for the last request, you can use a reflection system to do that if you don't want to list all types.

Alright. What syntax would you suggest for doing that? Do I need to make any special consideration to preserve entity ids and destroyed entities?

What syntax would you suggest for doing that?

Doing that... what? Cloning the registry and the pools that matter? You can use the range functionalities for that.
There is all you need to clone entities (assign) and components (emplace) from a bunch of iterators (those returned by views, groups or registry::data/registry::raw.
Let me know if something is unclear. I'm a little in a rush today but I can provide an example tomorrow if needed.

Okay, I think I have entities worked out with assign() and data().
By that I meant how do I use reflection to copy components without specifying their types?
For example, assuming registry::raw<Component>() is what I want to use, I don't know how to invoke that while iterating component types with visit().
So how would I do that? Is that the best way to go about it in the current version? Do I have to register the types beforehand like the example in the wiki, or can I avoid that?

You should register meta functions for that, something like:
entt::meta<T>().func<&clone<T>>("clone");

Where clone is defined as:
template<typename T>
void clone(entt::registry &src, entt::registry &dst) {
    auto view = src.view<T>();
    dst.emplace(view.data(), view.data() + view.size(), view.raw(), view.raw() + view.size());
}

Then:
src.visit([&dst](auto component) { entt::resolve_id(component).func("clone").invoke({}, std::ref(src), std::ref(dst)); }

Everything out of my mind, so it may contain errors.
For a future release, the idea is to make this transparent and easier to wire up, use, whatever.

In terms of errors, clone would be something like
void clone(entt::registry& src, entt::registry& dst)
{
	auto view = src.view<T>();
	if constexpr (ENTT_IS_EMPTY(T)) {
		dst.insert<T>(view.data(), view.data() + view.size());
	}
	else {
		dst.insert<T>(view.data(), view.data() + view.size(), view.raw(), view.raw() + view.size());
	}
}

Though I can't get the rest of the example working because I can't get resolve_id nor resolve_type to work.
However, I'm looking for a solution that doesn't involve ever explicitly registering or listing the specific types I'm using for components. I'd like to clone a registry regardless of what's in it.
For myself I'm going to try reverting b5e411d251e334a7b050a504bb47a10680f56987 on my fork and just use the old clone functionality, but it'd be useful to have a convenient alternative in the current version until the future release where it can be easily automated with meta.

Out of curiosity, what's the use case for cloning a registry as a whole?
I do this but only for some pools, so as to offline computations. I have never had the necessity of cloning everything so far.

As I believe some other users have attempted based on the wording of their issues, I want to use an ECS game state with speculative rollback netcode.
The first core requirement of rollback is that the game state can be serialized or cloned, so that whenever the input prediction on speculative frames is wrong the last correct state can be restored, and that simulation can continue from there with the corrected inputs.

Ok, in this case I wouldn't serialize everything though. For example, components that contain handles to resources.
Put aside the serialization process, the less you serialize, the better. Serializing a state for this purpose is better achieved by manipulating the components and transforming them in a form that is less demanding.

More than likely, but I also don't want to optimize prematurely. And it'll be difficult to make those decisions or design specialized serialization when I'm still familiarizing myself with how to use ECS, and haven't even decided what kind of game I'm making.

Alright, think I got it reverted onto the current master without issue (639b1ec if anyone is interested; had to make adjustments to account for renamed and modified functions).
Still not sure why resolve_id() and resolve_type() weren't working for me, but for now I don't need them. Will be looking forward to the future updates that make clone truly obsolete.

I'm closing this since the question has been asnwered and there is already a plan to get around what you did.
Thanks for discussing your issue and providing more details for future users.


[UNTHEMED]: Server boost status bar thingy in stickers and "server boost status" tab is not themed

Client Mod
Vencord (online theme)
Please provide screenshots of the unthemed areas.

Before Submitting

[X] I checked for similar reports of this issue.
[X] I am on the latest stable Discord version.
[X] I am on the latest version of the theme.
[X] I have reinstalled the theme.

I know they look similar but the 1st screenshot is the stickers tab, while the second is the server boost status tab


How To Trace the output in Composer Generated Adaptive Dialogs

Hi, I successfully integrated composer into my V4 echo bot. And it is working pretty fine. My problem is after reading and loading the root dialog into Dialog manager, we are calling method:
DialogManagerResult dialogresult = await this.dialogManager.OnTurnAsync(turnContext, cancellationToken: cancellationToken);.
After Executing this method we can able to see the bot response in our chat window. but i want to log that response from composer.
But the Method mentioned above just returning Result which contains "dialogresult.TurnResult", which conatins just the status of the dialog.
But, how to Trace the output for logging??

@rajeshyalamarthi - I would like to help, but I have a few questions.

What do you mean when you say you've integrated Composer into your bot? Composer is a tool for building bots. Are you saying you had a preexisting bot that you were able to load into Composer?
What do you mean when you say you want to log the response and trace the output? Do you mean you want to save telemetry from your users or do you want to view some kind of output while you're talking to the bot or do you want to make the output available to the bot itself or what?
Can you provide screenshots that show what you're experiencing now and what your desired behavior is?

thanks @v-kydela. Yeah what iam trying to say is with the help of composer tool we can able to generate dialogs which includes .lu, .lg, .dialog files. I loaded all those dialogs into my v4 echo bot. And i set up my echo bot in order to read all the dialogs by installing all the nudget packages required.
What i want is when i ask question to the bot, it gives response based on dilaogs and common.lg file.
i want to log the output which bot is sending to the user.

=> In the above image after executing the dialogManager.OnTurnAsync() method we can able to see the output in chat window.
=>But  dialogManager.OnTurnAsync() method returning dialog result which contains TurnResult.status,TurnResult.Result==null,etc..

in the above chat window, i asked bot by typing weather. And It replied with the below responses.
How can i log those responses in code level ??

@rajeshyalamarthi - You've explained more about how you've set up your bot, but you still haven't answered the question of what you mean by "logging" responses.
I can sort of see that you want to access the activities that your dialog sends to the user. This can be done by providing a SendActivitiesHandler delegate to your turn context's OnSendActivities method. This is normally done by middleware.
If you want to log the responses to a telemetry service like Application Insights, you can use TelemetryLoggerMiddleware.
Does this resolve your issue?

@v-kydela. Not exactly. what i am trying to say is in the above chat window you can see the responses from the bot like Let's check weather and what is your zipcode?.
what i am asking how can i get these responses while debugging the code after exceuting the method DialogManagerResult dialogresult = await this.dialogManager.OnTurnAsync(turnContext, cancellationToken: cancellationToken)
=>By Logging what i mean is how to log those responses from bot in some database

@rajeshyalamarthi

By Logging what i mean is how to log those responses from bot in some database

Application Insights uses a Kusto database. Can you explain why the TelemetryLoggerMiddleware solution isn't acceptable to you?

@v-kydela  , what i meant by db is on order to log my chat in sql db. But thanks @v-kydela , as you mentioned in above comment with the help of send activities handler to onsendactivities method with the help of this implementation i can able to see the activities that bot sending that to the user. Thanks a lot.

@rajeshyalamarthi - I'm glad we could get this resolved


fix: ignore extra settings fields

Some settings fields can be removed over time, but the values in the DB still remain.
If extra fields are present then an exception is thrown:
2023-01-23 14:36:46.93 | ERROR | lnbits.app:lnbits_startup:155 | 2 validation errors for SuperSettings
lnbits_ext_github_token
  extra fields not permitted (type=value_error.extra)
lnbits_extensions_manifests
  extra fields not permitted (type=value_error.extra)

This PR ignores extra fields.

Codecov Report

Merging #1398 (fef96f3) into main (e106ec3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head fef96f3 differs from pull request most recent head f67bd6b. Consider uploading reports for the commit f67bd6b to get more accurate results

@@           Coverage Diff           @@
##             main    #1398   +/-   ##
=======================================
  Coverage   48.65%   48.66%
=======================================
  Files         294      294
  Lines       16575    16577    +2
=======================================
+ Hits         8065     8067    +2
  Misses       8510     8510

Impacted Files
Coverage Δ

lnbits/settings.py
85.71% <100.00%> (+0.15%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


_processForeignKeys | Identifier name is too long

Describe the bug
using a table name and a fairly long column name exceeds the limit of 64 characters
CodeIgniter 4 version
"codeigniter4/framework": "^4"
vendor\codeigniter4\framework\system\Database\Forge.php
protected function _processForeignKeys
Line: 1298 ->  $nameIndex = $table . '_' . $field . '_foreign';
Possible Fix:
$lenght_nameIndex = strlen($nameIndex);

if ($lenght_nameIndex > 64) {
	$character_to_remove = $lenght_nameIndex-64;
	$nameIndex  = substr($nameIndex, $character_to_remove, 64);
}

In the first place, why would you name your table and/or fields too long?

In the first place, why would you name your table and/or fields too long?

cause I can do it?

I'm not trying to be rude or something. I'm asking a genuine question. If there exists a limitation of 64 characters then developers should be wary of that and try to limit their wordings of their used DB identifiers.

Why 64 characters?

Why more than 64 characters?

Mysql Limit: https://dev.mysql.com/doc/refman/8.0/en/identifier-length.html

I'm not trying to be rude or something. I'm asking a genuine question. If there exists a limitation of 64 characters then developers should be wary of that and try to limit their wordings of their used DB identifiers.

how the name of an index is generated by a method of a framework should check if the generated name is legal and possibly modify it to make it legal.
there are several reasons why the limit can be reached:

use of table prefixes chosen by the user.
talkative developer

the truth is that the naming of an index is arbitrary regardless of how the table and the field are named
in many frameworks these names are not dynamically generated like on Forge but the developer can choose it.
Maybe that's the right way to go

@samsonasik
We support:

MySQL (5.1+) via the MySQLi driver
PostgreSQL via the Postgre driver
SQLite3 via the SQLite3 driver
MSSQL via the SQLSRV driver (version 2005 and above only)

And want to support Oracle #2487, too.

Might be related to #5075

Oracle 12.1 is only 30 characters.
Therefore, if you want to put in a limit of 64 characters or not, I think it is preferable to do it per driver.

Oracle 12.1 is only 30 bytes.

This is not a bug.
It is possible to specify an identifier that always exceeds the character limit. Also, in implementations such as how many characters are used from the beginning, there is a possibility that the keys will be duplicated.
#5075 would resolve the issue.


Added job_role and industry fields to user

We need this two fields to integrate them in the explore api.
Please @Kartones check this two so you have check the other related PRs. Thank you

+1


💄 Autofill with URL 🪄

#220  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RTLS and the rest of your teammates on  Graphite


Timeshift sign inverted

I'm submitting a ...

[ X ] Bug report
[ ] Feature requestdd
[ ] Question / Support request

If you don't send bug report, remove sections below
For bug report please include this information:
Software Versions

Grafana
Zabbix
Grafana-Zabbix Plugin

v5.0.0
2.4.8
3.9.0

What did you do?
Use timeshift function
What was the expected result?
The function display correct values.
What happened instead?
The signal is inverted: If a minus sign ( - ) is used the shift metric forward in time (timeshift_-24.png).

If a plus sign ( + ) is given, the metric back in time (timeshift_+24.png

It's not a bug. Behaviour is described in docs http://docs.grafana-zabbix.org/reference/functions/#time


Manage access autocomplete fails for geographic raster resources

This is a quirky bug.  On www I have created a geographic raster resource.  I want to add access for another user and click the manage access button.  However when I try enter a name, it does not autocomplete, and as a result I am not able to add access for anyone else.  The manage access works fine for other resource types I have worked with (generic and geographic raster).  To provide access to the resource that exhibits this I had to make it public, as I can not change its access.  To reproduce you will need to download the contents from https://www.hydroshare.org/resource/78f573e7ffb34e568bdf3a8702efcea4/ then create a new resource yourself to try.  This bug also occurs on beta.

@dtarb The problematic resource you have provided seems to be of type geographic feature.

@dtarb hmmm...it is weird...the auto-complete works in EDIT mode, not VIEW mode for Feature res.

Pabitra, thanks for noticing the typo in the title and Drew thanks for fixing.

@dtarb Can you verify and close if it works now?

Tested on www.  Closing.


Add automated CI job to push protos to BSR

It appears https://github.com/mercari/grpc-federation/commit/baca78bf2421322c97e6977a06931fed29e4058a updated the proto extension numbers, however, this is not yet reflected in https://buf.build/mercari/grpc-federation
Suggestion, add a push action or script to update this anytime proto changes land on main.

Thank you for your reports ! We'll try it !


Circuit Visualization

For disscussion about circuit visualization.

Maybe you discussed this already, but Plots.jl might be worth a look. Especially the concept of recipies and the independence from the actual graphics framework are very interesting IMO.

Some more engines and libraries from the Gitter discussion :

Latex package for circuits : http://physics.unm.edu/CQuIC/Qcircuit/
@i2000s might know about this.

If you just want circuit visualization, then a function to write a latex file which visualizaes the circuit based on this package can be developed. This is the same approach used by QuTip here : https://github.com/qutip/qutip/blob/master/qutip/qip/circuit.py#L927. The latex code generated by this function based on the circuit and gates is then compiled by https://github.com/qutip/qutip/blob/master/qutip/qip/circuit_latex.py#L45
It can be directly used since Julia has a Latex package with image handling.

Using compose.jl : https://github.com/dcjones/Compose.jl
It can create shapes and polygons easily as you can see in the example.

Using the Cairo plotting engine directly. This gives more flexibility and is more like a pixel by pixel drawing of shapes.

OpenGl based package GLVisualize. (http://www.glvisualize.com/)
This gives much more flexibility and will especially be helpful for 3D plotting such as states in a Bloch Sphere or to animate rotation of state vectors etc etc

@acroy yes, I just move gitter discussion to this issue, in case we lost some idea... I'll try Plots.jl separately to see if it could acheive the goal of interactivity.
@sahmed95 you are right I'm thinking about interactive visualization for circuit indeed. Quirk is what I imagined for the visualization. JS engine always have a good interactive engine in a browser. I'll try the latex implementation.
I guess a parser for generating js file for the plots which is similar to qutip's implementation will be a better solution.


Assessment 8/27/2024

We have deleted our original nighthawk directory and everything in it. We remade the directory and cloned in the portfolio_2025, student_2025, and rohan_2025. We installed all the necessary bundles, requirements, and made sure to do all of this in a venv, or virtual environment. My parter (Nikhil Maturi and I, Rohan Bojja), collaborated with Yash Parish and Neil Chandra. Please find the following screenshots:
Rohan's Copies:

Nikhil's Copies:

Yash's Copies:

Neil's Copies:

Selfie of Group:

Great work! ! Mr M.


fix: Validate 'expires_in' as a convertible number

Summary
This PR aims to enhance the flexibility of the processGenericAccessTokenResponse function by allowing the expires_in field to be a string that can be converted to a positive number.
Background
In most cases, the OAuth 2.0 specification suggests that the expires_in field should be a numeric value. However, some OAuth providers deviate from this and send expires_in as a string. This creates compatibility issues when integrating with such services.
Changes
The proposed change is a minimal alteration to the existing conditional check for expires_in. It now uses Number and isNaN to validate that expires_in can be converted to a positive number, whether it comes as a string or a number.
Benefits
Enhances compatibility with OAuth providers that send expires_in as a string.
Maintains the original check for expires_in being a positive number.
Provides a more flexible and forgiving user experience.
Testing
The change is minimal and does not introduce new dependencies or require new tests. Existing tests should suffice to validate this change.
Thank you

There's already been PRs/issues open for this. expires_in in JSON responses is defined as a number, please reach out to your provider to get a fix on their end instead, or workaround the provider issue by instantiating a Response object with the original Response properties and the body type fixed.

Thank you for the quick response and clarification. I'll look into working around the issue as suggested.


Serverless: 'Too many requests' received, sleeping 5 seconds

Not much of feedback to give you but noticed this message whenever i make use of your plugin.
Serverless: 'Too many requests' received, sleeping 5 seconds
Serverless: 'Too many requests' received, sleeping 5 seconds

If i remove the plugin i do not get the message any more.

This is expected behavior. AWS doesn't allow to perform multiple requests at one time. After setting up an API Gateway, plugin needs to send another request to set up binaryData.


to introduce a new function timerfd_settime() and timerfd_gettime()

Is it possible to introduce a new function timerfd_settime () and timerfd_gettime ()?

I would prefer not.  These are non-standard GLIBC functions.  NuttX follows the standards defined at OpenGroup.org which does not include these interfaces.
There are exceptions to this in the code base now, but I don't think we should proliferate non-standard interfaces.

The solution would also required timerfd_create().  https://linux.die.net/man/2/timerfd_settime
While I don't advocate including non-standard interfaces in the OS, I would not oppose the inclusion of these GLIBC/Linux interfaces if they were properly implemented in the nuttx/fs directory (with new interfaces to sched/timer, were configurable, and had proper system calls implemented in syscall/.

yes, of course, the timerfd_create () function is also required. I think it would be very useful for this project to have such a set of functions:
timerfd_create ();
thimerfd_settime ();
thimerfd_gettime ();
Thank you in advance if they appear !

We can add eventfd and signalfd to the list too, these three API set is the good addition to POSIX since they strengthen the core UNIX design principle: Everything is a file.

I agree !

eventfd APIs introduced with PR #1479.   PR #1479 does not address signalfd and timerfd interfaces, however.

Hello!
I'm interested in timerfd_create() as well. Does anyone have an implementation ready? If not, I might try to implement it myself.

@eerimoq as far as I know nobody start the work, it will be great to add timerfd support! Actually, you can reference the recent addition of eventfd PR #1479 and #1479.

Trying to address this issue in scope of https://github.com/apache/incubator-nuttx/pull/5014

Nice!

Ok, we support the final one(signalfd), great!


Azure Custom RBAC with wildcard in assinableScopes - is it possible?

I am trying to create Azure Custom RBAC and it accepts wildcard in action/noaction but it does not work when I try wildcard in assinableScopes.
I need to restrict permissions for certain resource group but I don't know the exact name of the resource group. However, I do know the naming convention and I would like to be able to use wildcard in the assinableScopes.
Example of what I would like to do but Azure does not allow:
{
	"properties": {
		"roleName": "MySampleCustomRole",
		"description": "My Sample Custom Role",
		"assignableScopes": [
			"/subscriptions/*/resourceGroups/ABCDXYZ-*"
		],
		"permissions": [{
				"actions": [],
				"notActions": [
					"Microsoft.Compute/snapshots/delete",
					"Microsoft.Compute/snapshots/write",
					"Microsoft.Compute/snapshots/beginGetAccess/action",
					"Microsoft.Compute/snapshots/endGetAccess/action",
					"Microsoft.Compute/disks/beginGetAccess/action"
				],
				"dataActions": [],
				"notDataActions": []
			}
		]
	}
}

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

ID: e0091902-7cb0-8dc3-2a7a-dc644f39a961
Version Independent ID: d2757c88-1bad-8d27-b548-9e04a8ec7fe1
Content: Understand Azure role definitions - Azure RBAC
Content Source: articles/role-based-access-control/role-definitions.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@aspnet4you
Thanks for your feedback! We will investigate and update as appropriate.

@aspnet4you
Thank you for your feedback!
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an Azure support request. Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
Teams Q&A forum for technical questions about the configuration and administration of Microsoft Teams on Windows.
Microsoft Teams Community forum
Thank you for your time and patience throughout this issue.


Add configuration option to disable modules at runtime

It should be configurable which modules are used to sample data on the device to send location data to the cloud at runtime so the performance of this data can be compared.
See https://github.com/NordicSemiconductor/asset-tracker-cloud-aws-js/issues/460

Introduce a generic way to disable sampling of data from specific modules at runtime:
{
  "cfg": {
    "nod": [
      "gnss",
      "ncell"
    ]
  }
}

This disables sampling of data ("no data") of the GNSS module.

Added a separate entry for neighbor cell data.

We should start off with these two


GEBT Refactor Design

Initial design of data structures for refactoring:

Initial concept in branch on Github
Waiting for feedback from David

Derek has David's comments and moving forward

In progress and going fine.   Next step getting the solver and constraints to work together with refactor.

Main beam part is largely done -- assembly into the global iteration matrix.  Now working on the solver.

Continuing to work on the solver. Would like to refactor the Vector, Quaternion, and RotationMatrix classes for use in the functors.

Still working on solver, implementing tests.

Completed here.


[FancyZones tests] virtualize file system

Some tests use the same files as the installed application and they will fail if those files are present on disk
FancyZonesActiveZoneSets.json, FancyZonesAppliedZoneSets.json, FancyZonesDeletedCustomZoneSets.json

Close this issue since it does not affect release?


feat: Create a backend to transform PubMed XML files to DoclingDocument

feat: Create a backend to transform PubMed XML files to DoclingDocument

Introduce PubMedDocumentBackend to (1) parse elements from PubMed XML files and to (2) convert them to a DoclingDocument. Convert to DoclingDocument the authors, the abstract, the main-body text, the tables, the tables captions, the figures captions and the references. The hierarchy of the main-body text is preserved but limited to 2 levels (i.e. we know which paragraphs belong to which sections).
Add end-to-end tests for PubMed XML conversion (./tests/test_backend_xml.py).

Known limitations:

Compared to the original PDF documents, some sections are missing: page footers, page headers, figures, sections outside the main-body (for instance: Authors’ Contribution, Funding, Conflict of Interest, Acknowledgment, or Keywords).
Column headers for tables with multi-column headers are duplicated.
Some PubMed XML tables (stored as HTML) are occasionally not properly converted to Docling tables and then ignored.

Issue resolved by this Pull Request:
Resolves #446 (Partially)
Checklist:

[x] Documentation has been updated, if necessary.
[x] Examples have been added, if necessary.
[x] Tests have been added, if necessary.

Some additional comments:

Rename the input format objects from PUBMED to XML_PUBMED for verity clarity (first the type, then the collection)
rename test files from nxml to xml (since the .xml is the only extension for PubMed, if InputFormat object is not provided)
rebase to latest main
style suggestions (but it's up to you):

no need to import Dict, List, .... from typing library, you can use the built-in types dict, list, ... respectively
you can shorten expressions like if len(reference_node.xpath("article-title")) == 0 by using falsy and truthy values: if not reference_node.xpath("article-title"), and instead of if reference["author_names"] != "" you can use if reference["author_names"]

I think we should switch all function typing to Pydantic BaseModel.

If we input a .xml file which do not match the JATS schema, docling raises an error:

docling.exceptions.ConversionError: File format not allowed: /home/lum/Documents/public/docling/tests/data/pubmed/test.xml

If we rename one of the test documents from .xml to .nxml, the conversion remains the same.

If we input a .xml file which do not match the JATS schema, docling raises an error:

docling.exceptions.ConversionError: File format not allowed: /home/lum/Documents/public/docling/tests/data/pubmed/test.xml

2. If we rename one of the test documents from `.xml` to `.nxml`, the conversion remains the same.

This is good. My last question on this topic, do we have already a way for a different XML format?

This is good. My last question on this topic, do we have already a way for a different XML format?

@dol This is addressed in #606 , which should be rebased (and therefore slightly refactored) once this PR #557 is merged.
@lucas-morin as discussed, let's give a more readable and type-safe schema to the parsed content instead of generic dictionaries (dict[str, Any]) by leveraging pydantic models. Also, we should rebase to pick-up the commits from last week.


PCWG Share 1 report fails if inner range A fails

If there isn't enough data in the inner range type A, then calculations are not finished and report not produced.

See also:
https://github.com/peterdougstuart/PCWG/commit/1f42d01aeef4cfc921e5ac80eec4faf15ad18b52
https://github.com/peterdougstuart/PCWG/commit/dc8ad06c55ef09dfaa253bfc94568176eef15944


Logging that works after unplugging

I'm testing an app that involves moving with the app open to test a feature. Beyond using an emulator, what I've been doing is carrying both my laptop and my phone, with the phone plugged in, to do final testing on the feature.
A more convenient way to do this would be to be able to unplug the phone, the app under development just saves print calls until it gets plugged in again, at which point it dumps all of the prints that happened while unplugged into the console.
Alternatively I've seen that Android supports debugging over WiFi, didn't look into iOS, but this seems less than ideal because of all the issues that networking usually introduces into things that should be easy.

Closing this as you can already do this with platform tools as mentioned in the previous comment.


Certificate Authority

Been having some trouble getting the certificate authority working:

If I start it in secure mode, the client lacks the SSLContext to connect to it.
If I start it in insecure mode, it cannot communicate with the other core systems (which is still in secure mode), so it cannot get the public key from the authorisation system.

Kinda got it all to work in secure mode, but had to bypass a few security measurements to do so:

Had to call setNeedClientAuth(false) in the CA, so it doesn't require a certificate from the client, which it cannot provide at this stage.
Changed CertAuthorityACF to not filter anything - again as the client cannot provide any certificates to filter on.
I manually transferred the cloud trust store to the client and loaded it in the CertificateBootstrapper class, so that the client would accept the certificate from the CA system. Was playing around with getting it through an insecure version of the CA, started simultaneously with the secure version - didn't get that to work though.

Anyhow, know that this is probably not the correct/secure solution to take, just thought I'd report on my progress playing with the CA. Haven't pushed any of this in, so it won't get mixed up in the release code (if I do I will keep it on a separate branch in my own fork).

Well, my changes would make it run with just the secure versions...
Coming to think about it, the CA should probably just accept any self-signed certificate from the client, and use the client's public key to encrypt the response - that way only the client is able to read it.

I think that's a good idea.


TECH-5070: add initial declare_schema_build.yml workflow

Jira Ticket
Description

Set up GitHub Workflow for CI/CD build, remove Travis

Summary of Changes

Create .github/workflows/declare_schema_build.yml
Remove .travis.yml

BTW, re: all those commits, before you send out the first PR, it's best to squash junk commits together with git rebase -i and to overwrite failed experiments with git commit --amend. Once you have a PR out, that's it's best practice not to do those things since the git push --force may cause reviewers to have to look at changes they've already reviewed.


Fix bug for bringing to settings page if trying to add song w/o login…

… to Spotify
#191
This was implemented a while ago, but I misunderstood what SpotifyIsAvailable was doing... my bad.
It doesn't show a popup, but it pushes the SettingsVC on top, to imply that they should log in. Made it so clicking back button returns them to their previous VC easily

looks good


4.18.1 cannot access snapshot in GUI

 After updating from 4.18.0 to 4.18.1, All admin accounts are unable to access and list snapshots in the gui.
ISSUE TYPE

Bug Report

COMPONENT NAME
UI - snapshot

CLOUDSTACK VERSION
4.18.1

CONFIGURATION
advance networking
OS / ENVIRONMENT
management server/secondary/mysql on ubuntu 22.04
SUMMARY

when trying to access snapshot in the gui, nothing gets display other than a red X at top of page.
STEPS TO REPRODUCE
UI, click storage -> snapshots

EXPECTED RESULTS

list snapshots

ACTUAL RESULTS

noting gets listed but an error symbol.
From the mgm log:
2023-09-24 02:31:41,015 ERROR [c.c.a.ApiServer] (qtp9190301-17:ctx-929f6bb9 ctx-afcecb42) (logid:f3a11c73) unhandled exception executing api command: [Ljava.lang.String;@6a2520b5
java.lang.NullPointerException

I can reproduce your issue @Chih80 , I'll have a look for a fix.

@Chih80 , can you look in the database if the snapshots still exist? In my case they dissapeared. I created a new snapshot and this show as expected.

yes the snapshots still exist. In my case for what ever reason, their were two accounts with the same name when I checked mysql db. One being user and one being admin, that user account was never made with the admin role so how that came up in the db is a mystery. The UI only show the correct user account. I updated the user account with the admin role to user role and a min later, everything started working. Admins and users with admin roles can not access the snapshots in the UI. The only thing is, now I still have 2 user accounts with the same name.

So both accounts are in the same domain @Chih80 ? if so, a workaround would be renaming one.
You can use for instance cloudmonkey for that.  see https://cloudstack.apache.org/api/apidocs-4.18/apis/updateAccount.html

@Chih80 is this still an issue for you, or can we close?

Hi @Chih80 it appears you're hitting some bug, could you check and share the management server logs. For example, you can set the page size to 1 in the URL and try to go through the page -> you'll hit some API for some snapshot where the NPE is happening - chances are something in the DB is wrong for the specific snapshot.

This issue was resolve. After checking the db, I saw that a single user was also assign as admin, basically one account set as a user and also an admin. I have no idea how that happened. After manually deleting the admin assignment to the account, I was able to access snapshots.

This issue was resolve. After checking the db, I saw that a single user was also assign as admin, basically one account set as a user and also an admin. I have no idea how that happened. After manually deleting the admin assignment to the account, I was able to access snapshots.


keybase: on Linux, symlink kbfsfuse, git-remote-keybase into main package

Motivation for this change
This fixes some innocuous errors when keybase tries to detect the version kbfs, which it does by default now to warn of upgrade compatibility.  Furthermore including git-remote-keybase seems rather sensible, considering I (at least) expected the 'keybase' application to simply include these things
This does have the downside of bloating the closure of keybase by about ~100mb on x86_64 Linux.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build keybase

@thoughtpolice I don't think we should be doing this, see https://github.com/NixOS/nixpkgs/issues/50999#issuecomment-461852250.

@worldofpeace I see, so we shouldn't even need this patch, since it should just be done for us -- and the build of keybase will contain everything we need? Is that my understanding? (FWIW, the kbfs version check here is actually non-fatal, but nice to fix, so it can be left for now)
If that's the case then I agree we shouldn't do this -- we should instead update keybase and drop the kbfs package instead, and ensure keybase includes working git-remote-keybase, etc. (In particular I didn't see any direct mention as to whether they will ship binaries with the same name or e.g. if they might fold kbfsfuse into a keybase fuse command or anything. So we'll at least need to look at the changes more closely to be sure.

@worldofpeace I see, so we shouldn't even need this patch, since it should just be done for us -- and the build of keybase will contain everything we need? Is that my understanding? (FWIW, the kbfs version check here is actually non-fatal, but nice to fix, so it can be left for now)

Yes this is true, they have everything there, we might even eventually be able to build the gui from source.

If that's the case then I agree we shouldn't do this -- we should instead update keybase and drop the kbfs package instead, and ensure keybase includes working git-remote-keybase, etc. (In particular I didn't see any direct mention as to whether they will ship binaries with the same name or e.g. if they might fold kbfsfuse into a keybase fuse command or anything. So we'll at least need to look at the changes more closely to be sure.

Looks like arch linux splits the package, I tend to like that approach.

@worldofpeace @thoughtpolice is this still relevant? Seems like keybase packages have changed a lot. See #64248 #64377

I'm not sure if we want everything in one main package.

This is no longer relevant.


updated to point to running dgs pr

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Multiple order issue
Change description
Fix for multiple order issue
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ X] No

this change already merged closing


Issue 29

Type of Change

[ ] Bug: [Title]
[x] Feat: Converted static footer to dynamic links
[ ] Doc: [Title]
[ ] Test: [Title]
[ ] CI: [Title]

Description

Converted the static lists in footer to link components. Also set contact us link to contact-us route. While others are being directed towards home page as requested.
Linked Issue

https://github.com/iZooGooD/stay-booker-hotel-booking-react-frontend/issues/29
Screenshots

Tests

Clicking on the link to see it redirects to the correct route. At the moment contact us goes to contact-us. while  the rest goes to home page.

One of the commit is already merged in issue #31.

One of the commit is already merged in issue #31.

Sure let me remove that.

One of the commit is already merged in issue #31.

I have updated this branch. Now it contains only the footer changes. Please check

Reference

Great!


Labels: severity.Low type.DocumentationBug
original: yiiilonggg/ped#7

Duplicate


Added RequestAccessEnabled, AvatarURL and WebURL to Group struct (GetGroup)

The Group struct misses three values returned by the GetGroup details API, namely RequestAccessEnabled, AvatarURL and WebURL:
$> curl --header "PRIVATE-TOKEN: <my token>" http://mygitlab/api/v3/groups/57
returns
{   "id":57,   "name":"first-group",   "path":"first-group",   "description":"",   "visibility_level":10,   "lfs_enabled":true,   "avatar_url":"http://mygitlab/uploads/group/avatar/57/home-icon.png",   "web_url":"http://mygitlab/groups/first-group",   "request_access_enabled":true,   "full_name":"first-group",   "full_path":"first-group",   "parent_id":null,   "projects":[],   "shared_projects":[] }

Thanks @dihedron


Add descending + helpers

Hey let me know if this pr is acceptable. (#1 ) I added the following methods:

Descending, Ascending (alias for New), DescendingFrom, AscendingFrom

Added a basic test for descending, let me know if there's anything else you'd like me to change/add.

Sorry for being so terribly late.
That being said, I don't see the purpose of adding descending.


Investigate OoM errors in Mosaic Tile Server

There are a few cases where we are seeing some OoM errors in the mosaic tile server.
It's unclear what the cause is -- these particular scenes are really high resolution compared to the Landsat 8 and Planet data we've worked with previously.

This could be related to when metadata doesn't exist (at least that seems to occur near the OoM error).

This seems to be a straggler, closing


Tables are empty for both Recursive and Forwarding examples.

Tables are empty for both Root Hints and Forwarding examples.

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

ID: 9d4646a7-2ad0-f5d2-fa69-d7e197942870
Version Independent ID: d0822252-30f8-8060-e9d6-85dfaa07e6fb
Content: Appendix A - DNS Inventory
Content Source: WindowsServerDocs/identity/ad-ds/plan/Appendix-A--DNS-Inventory.md
Product: windows-server
Technology: ad-ds
GitHub Login: @iainfoulds
Microsoft Alias: daveba

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Migrating to a monorepo

In the next couple of months, we will migrate the following projects to a monorepo:

enketo-express (which will become the monorepo)
enketo-core
enketo-transformer
openrosa-xpath-evaluator

When complete, the enketo-express repository will be renamed to enketo, with each project included in the same repo as sub-packages.
Why?
Migrating to a monorepo will improve:

tracking issues affecting multiple sub-packages, or manifesting further up the stack
ability to navigate between related code across packages
identification of dependencies which are less obvious with separate repositories today
safety, productivity, success delivering features and fixes
flexibility to reassess boundaries and responsibilities between projects, and to generally take a more holistic approach to changes
reduced friction for posting and tracking feedback/issues/PRs
ability to increase consistency across all packages

When?
We intend to make this change within the next couple of months, as it will unblock stalled and pending changes which are high priority but difficult to move forward otherwise.
Goals
To the extent possible, we consider these goals essential:

Preserve/migrate git history, and any other repository-specific content such as issues, for all projects
Continue to publish enketo-core/enketo-transformer/openrosa-xpath-evalutor as standalone packages to NPM
Update documentation as appropriate to ensure it remains correct and accessed easily
Make the most minimal, least disruptive change possible without introducing unnecessary hindrances

Expected impact

For users installing the existing NPM packages, there should be no immediate impact.
For users providing feedback, reporting will be simpler as there will be one place to file all issues. Otherwise, some URLs will obviously change, but we'll ensure there redirects where possible and provide links otherwise.
For contributors:

The enketo-core/enketo-transformer/openrosa-xpath-evaluator projects will be moved to the monorepo, so forks and clones of those repositories will need to be updated/replaced
The directory structures will change
We may make some related tooling changes (such as package manager)

The monorepo is now up at https://github.com/enketo/enketo! We will work with it for a week or two as we prepare the next releases to make sure everything is structured in a usable way. We expect to move issues and archive this repo in mid-October. 🚀


Add Ledger Wallet connection

Hello @earrietadev if it’s a Ledger related issue i will suggest you contact a Ledger support like i did and got help on their general chain link Ledger Live Support Agent
So i will suggest you do the same. You can use the  ticket ID DR3486 as well for a faster response.
Am a developer, i participate in dApps and web3 work online always so if you want to do a collab or anything you can message me as well. Thanks

@earrietadev  can i work on this issue?

Hi @earrietadev My name is Lindsay Morales. I'm a Software Engineer based in Costa Rica and this is my third OD hack!
I'd love to have the opportunity to tackle this issue, and be able to learn more about this project. I've never integrated a wallet before, but I'm characterized by being a quick learner and I love challenges. So first thing I'll do is check the code to check the other wallets implementation, I already check the project docs and I saw the interface I should implement for new modules or wallets so I'll research about this specific wallet integration then.

Hi @earrietadev My name is Lindsay Morales. I'm a Software Engineer based in Costa Rica and this is my third OD hack!
I'd love to have the opportunity to tackle this issue, and be able to learn more about this project. I've never integrated a wallet before, but I'm characterized by being a quick learner and I love challenges. So first thing I'll do is check the code to check the other wallets implementation, I already check the project docs and I saw the interface I should implement for new modules or wallets so I'll research about this specific wallet integration then.

Hi @earrietadev
I'm Nitin Verma, a typescript fullstack developer. This is my first ODhack and would like to work on this issue. I have integrated a metamask wallet before using javascript in one of my project . I'm sure will be able to connect Ledger Wallet.
I will be able to complete it in 3-5 days.
Here is my github https://github.com/vnitin08.

Thanks! @earrietadev will let you know if I have any doubts.

Thanks! @earrietadev will let you know if I have any doubts.

Hi! @earrietadev I’ve been working on the wallet integration and I created a draft PR, in case you want to check something while I finish the implementation.
i have a question… i was wondering, because I’ve been implementing the module based on the ledger dev documentation and other resources, but how am I supposed to test it? If we are talking about a hw wallet which I don’t have? 😅

Hi @earrietadev is this issue stil open?, if yes can you pls assign it to me?, i woulld love to work on this.

Can I jump on this task?

Hi @earrietadev, I would like to request assignment to this issue. Is it possible to get it assigned?
As a Stellar ecosystem contributor, I have extensive experience in the ecosystem and would love to contribute to this project.

Closing this as this is already included in the library


how can we add images to using this library?

func qb_imagePickerController(imagePickerController: QBImagePickerController, didFinishPickingAssets assets: [AnyObject]) {
}
uisng this Function how can i add multiple images ?.. uisng Swift Please help me..

anyone plz tell me plz


[perf_tool] Add bigquery storage format definitions

Summary: Add definitions for how both results and specs will be stored in bigquery. Specs and results will be stored in separate tables. The spec table is only written to on experiment success, so joining the spec and result tables on the ExperimentID will remove all result rows from failed experiments.
Type of change: /kind test-infra
Test Plan: Tested that bigquery tables get created correctly based on the implicit schemas of these structs.

@pixie-io-buildbot test this please


Write Initial Rebasing Token Educational Challenge & Tests

Description
Create Rebasing Token Challenge as per issue #5.
TODO

[x] Create initial RebasingERC20.sol
[x] Debug tests and contract to ensure challenge works
[x] Create initial RebasingERC20.t.sol tests (pseudo-code)
[x] Add any more relevant tests, I think I'm missing just some on events and whatnot.
[x] Write README sections
[ ] Resolve PR Review Change Requests - notably: rounding error tests, and README / natspec updates.

Related Issues
Closes #5
Your ENS/address: steve0xp.eth

Update Since Last Submission

Created most of the tests for the challenge
Researched, Iterated (upon feedback and research) on challenge scope, Redesigned challenge code
Updated README
Now submitting this for first PR Review to get more detailed feedback on direction, implementation, and thoughts on remaining tests (specifically how to go about the rounding error test(s)).

Remaining Work

[ ] Resolve PR CRs once @escottalexander has a chance to provide a PR review.

[ ] Finish off remaining tests (rounding error test, and transferAllFrom after rebase test)
[ ] Update README to align with code challenge nat spec for rounding errors

@escottalexander thanks for your reviews! I've responded to your comments and made changes to the code accordingly. It should be ready to review again and hopefully we can merge it to the repo soon :)

@escottalexander changes have been made. Thanks again for all of your help and these reviews!


[Bug] When set ‘lower_case_table_names=1’  , the table alias cannot be capitalized

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
Version : 0.15.1-rc09
What's Wrong?
Database changed
mysql> select * from  TB1 a  where A.apply_id='001'  limit 10 \G;
Empty set (0.01 sec)
ERROR:
No query specified
mysql> select * from  TB1 A  where A.apply_id='001'  limit 10 \G;
ERROR 1054 (42S22): errCode = 2, detailMessage = Unknown column 'apply_id' in 'a'
ERROR:
No query specified
mysql> show  variables like  '%low%' ;
+---------------------------------+---------+
| Variable_name                   | Value   |
+---------------------------------+---------+
| allow_partition_column_nullable | true    |
| lower_case_table_names          | 1       |
| max_allowed_packet              | 1048576 |
+---------------------------------+---------+
3 rows in set (0.01 sec)
mysql>
What You Expected?
When using SQL queries, I want table aliases to be capitalized
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

it must be configured at cluster initialization.

it happened in 0.15.1-rc09 and has been fixed in 1.0.0-preview and following version.

under version 0.15.9-rc09, it looks like:
mysql> select * from part A where A.p_name = "rose moccasin";
ERROR 1054 (42S22): errCode = 2, detailMessage = Unknown column 'p_partkey' in 'a'
mysql> select * from part A where a.p_name = "rose moccasin";
ERROR 1054 (42S22): errCode = 2, detailMessage = Unknown column 'p_partkey' in 'a'
mysql> select * from part a where a.p_name = "rose moccasin";
+-----------+---------------+--------+------------+-----------+------------+--------------------------+--------+-------------+
| p_partkey | p_name        | p_mfgr | p_category | p_brand   | p_color    | p_type                   | p_size | p_container |
+-----------+---------------+--------+------------+-----------+------------+--------------------------+--------+-------------+
|     94251 | rose moccasin | MFGR#1 | MFGR#15    | MFGR#1510 | forest     | LARGE POLISHED COPPER    |     22 | MED CAN     |
|    132258 | rose moccasin | MFGR#4 | MFGR#41    | MFGR#4123 | bisque     | SMALL BURNISHED NICKEL   |     46 | WRAP BAG    |

mysql> select * from part a where A.p_name = "rose moccasin";
+-----------+---------------+--------+------------+-----------+------------+--------------------------+--------+-------------+
| p_partkey | p_name        | p_mfgr | p_category | p_brand   | p_color    | p_type                   | p_size | p_container |
+-----------+---------------+--------+------------+-----------+------------+--------------------------+--------+-------------+
|      3187 | rose moccasin | MFGR#5 | MFGR#55    | MFGR#5529 | honeydew   | SMALL POLISHED NICKEL    |      1 | SM DRUM     |
|     10974 | rose moccasin | MFGR#1 | MFGR#12    | MFGR#1211 | steel      | LARGE POLISHED BRASS     |     29 | WRAP DRUM   |
|     55272 | rose moccasin | MFGR#3 | MFGR#33    | MFGR#3325 | seashell   | MEDIUM PLATED COPPER     |     32 | WRAP PKG    |

And in 1.0.0-preview
mysql> select * from part C where c.p_name = "turquoise sandy";
+-----------+-----------------+--------+------------+-----------+------------+---------------------------+--------+-------------+
| p_partkey | p_name          | p_mfgr | p_category | p_brand   | p_color    | p_type                    | p_size | p_container |
+-----------+-----------------+--------+------------+-----------+------------+---------------------------+--------+-------------+
|    233418 | turquoise sandy | MFGR#1 | MFGR#14    | MFGR#1418 | lace       | LARGE ANODIZED COPPER     |     29 | LG PACK     |


Cannot open the app after minimization to tray icon from start menu or pinned taskbar entry

My Setup

Operating System: Windows 10 Pro
App Version: 2.15.3
Installation type:  exe

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description
After clicking X to minimize rocket chat to the tray, you can only reopen the window through the tray icon, and not through the start menu or from a pinned taskbar entry.
Current Behavior
After clicking X to minimize RocketChat to the tray icon, you cannot relaunch/restore the app through a pinned taskbar icon or the start menu. The only way to restore the app is through the tray icon. Reproducible on two different machines.
Expected Behavior
After clicking X to minimize RocketChat to the tray icon, you should be able to relaunch/restore the app through a pinned taskbar icon or by launching it through the start menu.

Same on Windows 7

When booting the local machine or closing the app and opening the app it will not actually open and stay within the system tray. This can be replicated on many workstations on our domain. It dose not just occur when closing the app by clicking the X or file>quit rocket chat.

Same on Ubuntu 19.04

Same on Windows 10 Enterprise LTSC (build 1809) with Rocket.Chat App Version 2.15.3

Some on Windows 7, 8.1 and 10. We have the same "problem" here on the office. People say that rocket is now working, because they have rocket pinned on task bar or try to launch it again by clicking on desktop icon.

I also encountered the same problem. How to fix ?

I also encountered the same problem. How to fix ?

I tried ! But I want the app to reopen right away, not to render


@gregorgorjanc To access the version from the terminal, we can use pip show AlphaPeel:

          @gregorgorjanc To access the version from the terminal, we can use `pip show AlphaPeel`:

(base) evie@Xingers-MacBook-Air AlphaPeel % pip show AlphaPeel
Name: AlphaPeel
Version: 1.1.1
Summary: Multilocus and hybrid peeling
Home-page:
Author: Andrew Whalen
Author-email<EMAIL_ADDRESS>License: MIT license
Location: /Users/evie/mambaforge/lib/python3.10/site-packages
Requires: numba, numpy
Required-by:

We can also add an option -version if necessary so that we can access the version of AlphaPeel via AlphaPeel -version. But for the commit hash, I am unsure if we would include the information in the software when we upload it to the PyPI.
Originally posted by @XingerTang in https://github.com/AlphaGenes/AlphaPeel/issues/61#issuecomment-1694649335

For example every time I open R I get the version e.g. I am on 'Beagle Scouts'.  Likewise ASReml prints the version as it runs e.g. 'AlphaPeel version 1.1.3 ...'. In Rstudio with the interface there is a help which gives the version and build.  The latest 'brick in the wall' is I am no longer automatically an administrator due to a change in UoE policy so I then have to remember to 'Make Me An Adminstrator', but that is just another hurdle to climb over and which may appear at any time.

I will close this issue since we track this one at https://github.com/AlphaGenes/AlphaPeel/issues/61, but will post the above comment from JW in there.


DISCUSSION: How to proceed with versioning

Since this project tries to simply re-export PouchDB, it would be good to co-version releases with the parent code. However, I foresee the need to implement bugfixed in the work-arounds and polyfills. A few ideas:

Break out WebSQL and IndexedDB polyfills into their own repo for independent versioning
Add a semver tag with own version, e.g. 7.2.2-DENOv1.0.0
Invert the tag relationship, like 1.0.0-POUCHDBv7.2.2
Version completely independent of PouchDB, and notify users of Pouch version changes via README

Would happily take feedback and any other ideas out there.

Upon further thought, WebSQL and IndexedDB will get pulled into their own repository for separate versioning.

I'm leaning towards semantic versioning with a build tag to indicate the bootstrapped PouchDB version.
The syntax would be the following subset of semver:
<major> "." <minor> "." <patch> "-" <pre-release> "+" <build>

This would result in a version like v1.0.0-PouchDB+7.2.2.
It's a little bit of an abuse of semver, but it would parse as valid and should allow meaningful readability for both machines and humans.

Decided on the most recent comment. Closing; will re-open if further discussion is required.


Added tests for azblob/service

[ ] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[x] MIT license headers are included in each file.

/azp run go - azblob

/azp run go - azblob

/azp run go - azblob

/azp run go - azblob

You'll want to rebase on top of main so that you can use test assets and remove the checked in recordings.


Clone to PR18229 - AzureFunctionApp@2: Better alias for Service Connection

Task name: AzureFunctionApp@2
Description: "Azure Subscription" is a terrible variable name for something that is not an Azure Subscription.
Documentation changes required: Y
Added unit tests: N
Attached related issue: https://github.com/MicrosoftDocs/azure-devops-docs/issues/13281
Checklist:

[X ] Task version was bumped - please check instruction how to do it
[ X] Checked that applied changes work as expected

/azp run

/azp run

/azp run

/azp run

This was the original PR - https://github.com/microsoft/azure-pipelines-tasks/pull/18229 created for the fix and this PR is based on that.

/azp run

/azp run


enable streaming for ts list keys

RTS-697, RTS-718.
This PR uses the new {o, ...} sext-encoded keys for TS, and enables streaming list_keys.
Complementary PRs: https://github.com/basho/riak-erlang-client/pull/253 and https://github.com/basho/riak_test/pull/953.

:+1: 87d4a5d

@borshop merge


invalid literal for int()

Hi I've just connected a dishwasher for fun an giggles, here's the stack trace I'm getting.
I'm guessing one of the sensors is expecting an int when the server is returning a float
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 383, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/hon/__init__.py", line 32, in async_setup_entry
    await hon.setup()
  File "/usr/local/lib/python3.10/site-packages/pyhon/api.py", line 61, in setup
    await asyncio.gather(*[
  File "/usr/local/lib/python3.10/site-packages/pyhon/device.py", line 115, in load_commands
    cmd = HonCommand(command, attr2, self._connector, self, multi=multi, program=program)
  File "/usr/local/lib/python3.10/site-packages/pyhon/commands.py", line 13, in __init__
    self._ancillary_parameters = self._create_parameters(attributes.get("ancillaryParameters", {}))
  File "/usr/local/lib/python3.10/site-packages/pyhon/commands.py", line 23, in _create_parameters
    result[parameter] = HonParameterRange(parameter, attributes)
  File "/usr/local/lib/python3.10/site-packages/pyhon/parameter.py", line 51, in __init__
    self._min = int(attributes["minimumValue"])
ValueError: invalid literal for int() with base 10: '0,5'

Whoops I messed up, please try version v0.5.0-beta.3 :slightly_smiling_face:

Something that worries me a little and that I noticed later was how the value is using a coma instead of a dot to separate digits. ValueError: invalid literal for int() with base 10: '0,5'
I'm in France and I'm wondering if the server is formatting that to whatever locale the user is in (we would format it 0,5 in France as opposed to 0.5 in the US). Yesterday I hacked around and got it to work by casting to floats as you did, but I also had to replace , with ., which felt very hack-ish because if it were indeed formatting to a specific locale then the parsing might be more complicated :/

Hey, I take care of it and also used a replace. I saw this and found it also a little strange why they localize the api values. But here in Germany we also use comma as separator and I know these formatting problems well.
I think this isn't going to be a big problem when using the replace, if it's already a dot than just nothing gets replaced. For my washing machine there is a delay parameter with values that can go above 1000 and they doesn't use a thousand separator there, so I think this also wouldn't be a problem.

Hey thanks, I can confirm the fix works. I'll look into supporting DW devices though I can't promise anything.


BlockStore should have a getter for the transition hash

User Story
As a validator
I want to get the transition hash from a smart contract
Because I don't want to know how it is generated.
Scenario
Given that I want to vote and need the transition hash
When I use the getter on the block store
And specify the block hash for which I want the transition hash
Then I get the transition hash of the given block hash.
Info
Later we may generate the transition hash in the validator node. But for now there should be a getter on the block store.
The block store can create the transition hash from the available data and should not need to cache all transition objects for all block hashes.
Contracts:

BlockStore

Like #382, but the hash of the object.


where is GraphQL Playground

in docs>Getting started with Strawberry>step6 say:
'Strawberry comes with a tool called GraphQL Playground.'
but But i got GraphiQL....
(maby i need to wait...)

@LiGuoV oh, we removed GraphQL Playground because it didn't support subscriptions well and didn't have the explorer. I'll update the docs as soon as I can :)


WIP[build]: identify app types and select respective RC template

according to the publish release will identify if app is using dockerfile or not and select specific image and RC template.
requires https://github.com/deis/builder/pull/12
https://github.com/deis/builder/pull/9
issue with deis:config set which will not deal with changing images rather updates RC templates to update environment variables

closing this PR in favor of https://github.com/deis/workflow/pull/95


Toggling the preview window many times crashes fzf

Category

[x] fzf binary

OS

[x] Linux

Shell

[x] zsh

For instance if I run fzf --preview='sleep 2; echo "foo"' --bind='?:toggle-preview' and I hold ? it eventually crashes and prints junk to the terminal, sometimes immediately, sometimes after several seconds...
I think this happens when the preview function doesn't finish? But that's just a guess.
I'm not sure how helpful it will be but here's a strace: http://sprunge.us/YHQX

Nice find. Thanks for the report. I believe the above commit should fix the problem. I'll release the version shortly.


Compute: Enable Favorites Widget from Home Page

What is this, and do we still need it?

Idea was to provide fast access to most used apps (models). I think that's not a prio. Will close.


add required middleware to set actor_id in auditlog entries

I thought we already did this one, but looks this middleware is not included in settings.dist.py
It's needed to populate the author field for auditlog entries. Currently they're all empty (None).

Shows no files changed?

haha, that show it was in fact done at some point!


Bugfix for POST Ops not setting Content Length

This patch sets the content length of the request or else a chunked transfer is declared, and this corrupts outgoing payloads. the content's length must be set prior to the body being written per the spec at https://golang.org/pkg/net/http
 If Body is present, Content-Length is <= 0 and
 TransferEncoding hasn't been set to "identity", Write
 adds "Transfer-Encoding: chunked" to the header. Body
 is closed after it is sent.

To that end a temporary buffer, b, is created to produce the payload body, and then its size is used to set the request's content length.

lgtm


STORM-469

-1. I don't think the change is necessary. Storm ships with a logviewer daemon. If users want more details on some exception in ui, they have the ability to do so by ensuring that they have the logviewer daemons started which should allow them to navigate and view the entire log file through ui.

Parth-Brahmbhatt: As you like it, Parth :-)
IMO, if a truncated errormsg is shown , a link on it to the logs is a good ui design., and more user friendly.
Agree with Jason Kania on it.It might be even cooler to show last few bytes of log link, or complete log link…as in hadoop jobtracker design.

jmlogan: you are right about the extra files.few more changes.will update commit.thanks!

@rekhajoshm  Thanks for the patch. Few observations here

you are changing interfaces to pass a codeDir param. I can't find it where codeDir used to fixed this issue. Whenever you are changing apis like this you should introduce a new method definition not change the existing ones as these are storm-core apis.  Please also explain whats the necessity of codeDir.
you added dependency-reduced-pom.xml and any reason to remove org.clojars.runa from pom.xml. Also maven-eclipse.xml shouldn't be part of this commit.
I am -1 on the current approach, changing apis to solve this is not a good idea. This pretty much breaks current topologies.
I suggest you to look into ui/core.clj , one approach is to have a search functionality in logviewer  and provide a link next to error message with param to logviewer as the errormessage. When user clicks on that it goes to logviewer and highlight the error in the log file.

Thanks Harshch for having a look.Somehow you seem to have reviewed both storm-469 and storm-528 commits together?

commit 2931487 is for storm-469, tested and works on my storm cluster and gives me link to truncated text the log link, which works.
commit ef19e71 is for storm-528 and as informed to jmlogan, i will update the files in a while.Thanks!

@rekhajoshm  Ok. Open up a different pull request for each of those JIRAs. Once the PR approved by the committers we will merge pr directly into master branch. combining two JIRAs into one PR is not recommended.

created separate pulls.


Fix CoP-out-of-Bounds

I accidentally the whole Ley Lines.
 While fixing the way CoP windows work relative to Ley Lines, I ended up the whole module.
This looks a lot crazier in the diff than it really is, probably the easiest way to review is to just look at the whole file. The main thing here is that methods are more generic, commented, less prone to errors with weird ACT/FFLogs behavior, and much more DRY.
This follows a pattern that other jobs (DRG/MNK and probably anything else Toast did) use which I think also helps make maintenance easier if someone else needs to look at a job.

Big thanks to @rexatlanti and @AkairyuGestalter on Discord for their help in review and suggestions.


Get date "Last Modified" paquet files

Hi,
I need to read the paquet files from the S3 bucket.
I can do this from wr. read_parquet.
I mean, is it possible other than reading files from a certain modified date/time, to actually get the "Last Modified" to commit to its data. Like when using boto3
parquet = self._parquet_client.list_objects(Bucket=self._bucket)
for parquet_inforamtion in parquet["Contents"]:
LastModified = parquet_inforamtion["LastModified"]

You are looking for this method: https://aws-data-wrangler.readthedocs.io/en/stable/stubs/awswrangler.s3.describe_objects.html#awswrangler.s3.describe_objects which describes S3 objects under a given prefix, including their last modified date


chore(main): release never-bot 2.0.0

:robot: I have created a release beep boop
2.0.0 (2024-04-10)
⚠ BREAKING CHANGES

Removed GPT imagine_v3 command

Features

Add GitHub Actions deployment workflow (73d52a2)
Enhance GitHub workflows and Discord bot commands (d7eac4e)
Image Generation DALL-E v3 (7d3afb1)
Update Codacy workflow, gitignore, and package dependencies (a1d10ab)
update dependencies and enhance chatbot functionality (f9bfdd2)
Update dependencies and remove unused image (172a1fb)
Update dependencies in package.json (9855058)
Update dependencies in package.json (eaf3820)
Update dependencies in package.json (b4c8da4)
Update dependencies in package.json (097cd18)
Update dependencies in package.json (d734c40)
Update dependencies in package.json (a3da0cc)
update package dependencies (eab17f4)
Update package dependencies (9d62404)
Update package dependencies (37b7b7a), closes #123
Update package dependencies and chatbot response (554cf2c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/neverased/NeverBot/releases/tag/never-bot-v2.0.0 :sunflower:


chore(build): add short summaries to targets

Welcome to Cryostat! 👋
Before contributing, make sure you have:

[x] Read the contributing guidelines
[x] Linked a relevant issue which this PR resolves
[x] Linked any other relevant issues, PR's, or documentation, if any
[x] Resolved all conflicts, if any
[x] Rebased your branch PR on top of the latest upstream main branch
[x] Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
[x] Signed all commits: git commit -S -m "YOUR_COMMIT_MESSAGE"

Fixes: #634
Description of the change:
This change will make it easier for people to see Makefile targets clearly, with topic handles and short summaries using make help. Let me know if it needs readjusting in any way.
Motivation for the change:
To contribute to an open-source project.
How to manually test:

Execute make help
???
Profit

That's very cool.

I guess that should be everything? Thanks for the reviews ^^

I guess that should be everything? Thanks for the reviews ^^

Guess not lol

Uhh... what. I guess that's a failure in my rebase attempt.

@pektezol just force-push from your local main on your computer back to your remote main branch

Or at worst, the Makefile patch in the force-push comparison is still there and contains basically the reverse patch needed:
https://github.com/cryostatio/cryostat-operator/compare/c16c7f0e74b6cf127df635285fb8068689d5d1ab..5c3769f5a2823847214528fb655b297d688dac14#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52

Ahh that must something troublesome to do with pushing to fork main? We can move work to #637 then. Contributions are still recorded for @pektezol anyway^^


Explore Crowdfunding

How can we implement crowdfunding functionality?
e.g. https://github.com/liberapay/liberapay.com

Hi @abrichr , Here is how I think we can implement crowdfunding :

Create web page
It provides a detailed explanation of how the funding will be utilized, the goals of the project, and a roadmap outlining the planned activities and milestones.

Introduce new feature to marketplace web app #91 that allow users to contribute to the crowdfunding campaign. For example, integrating payment processing functionality with the liberapay crowdfunding platform.

Provide a range of contribution options.
Display various ways individuals can contribute to OpenAdapt on the webs app and README, with monetary donations being just one of the possibilities.

Spread the word.
To inspire action, we can promote OpenAdapt through multiple channels (create twitter, tiktok, instagram account) ensuring that people receive exposure to the information through various sources.

Let me know if anyone has feedbacks or suggestions! Thank you! 🙏

Look into https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/displaying-a-sponsor-button-in-your-repository

While trying to create the github sponsor account, I came across this tutorial (https://docs.github.com/en/sponsors/receiving-sponsorships-through-github-sponsors/setting-up-github-sponsors-for-your-organization) which suggests  @abrichr as the organization owner should set up the git hub sponsor account. But I'll be happy to help setting up the account. Let me know what you think!

Done! Pending Github approval. Thank you @jesicasusanto !


support for quarkus in native mode

Quarkus support would really shine in native mode in combination of Knative auto-scaling.
The process of building native images is still very heavy and it may not be the best fit with Camel K incremental build mechanism, still Time-To-First-Bytes is deemed critical for serverless applications.
One approach would be to schedule a native build in parallel of the JVM mode build, and perform a rollout deployment once the native build has completed.

It'll great having this feature, resource consumption reduce significantly in native mode, not only in terms of execution time or  throughput, but also in memory consumption making the same infrastructure able to increase the number of microservices deployed, which is in my case one of the biggest problems.

It'll great having this feature, resource consumption reduce significantly in native mode, not only in terms of execution time or  throughput, but also in memory consumption making the same infrastructure able to increase the number of microservices deployed, which is in my case one of the biggest problems.

@astefanutti @jamesnetherton @ppalaga anyone interested in implementing this issue ?

@lburgazzoli happy to work on it, or help if @jamesnetherton @ppalaga you want to do it.
It seems building Quarkus applications in native mode on OpenShift has been smoothed: https://www.openshift.com/blog/introducing-the-quarkus-helm-chart.

@astefanutti feel free to go ahead!


Update User stories and Non-Functional Requirements in the Developer Guide for v1.1

Update metadata in related files in /docs
Update User stories and Non-Functional Requirements sections in the Developer Guide for v1.1
Close #4

Looks Good! I think it can be merged now.


test runs much slower from 1.3.3 to 2.0.0

The issues forum is NOT for support requests. It is for bugs and feature requests only.
Please read https://github.com/angular-ui/bootstrap/blob/master/CONTRIBUTING.md and search
existing issues (both open and closed) prior to opening any new issue and ensure you follow the instructions therein.
Bug description:
We use selenium to run integration test. Whole test takes about 24 minute as use version 1.1.0. After I upgrade angular-bootstrap to latest 2.4.0, whole test takes about 37 minutes to finish. I notice this and try several version to figure out which version did this. I figured out 1.3.3 is 24 minutes and 2.0.0 is 37 minutes. There are a lot changes in 2.0.0 change log. I couldn't figure out which one could be the cause of this. Is there any one having the same problem as me?
Link to minimally-working plunker that reproduces the issue:
I cannot reproduce this issue.
Version of Angular, UIBS, and Bootstrap
Angular: 1.5.3
UIBS: 2.4.0
Bootstrap: 3.3.6

Per the issue template, the issues forum is not for support requests.  Please read the link contained therein for how to ask for, and receive, assistance.  Thanks.


Refresh required

See Issue https://github.com/pffy/java-swing-hanzitopinyin/issues/10

Use the new tone specification:

https://github.com/pffy/pffy/blob/master/obj/lang/Tones.md

LTGM
--QA