Don't include , use std::make_shared

I've been evaluating this library for usage in an embedded software that runs on an MCU with on-chip flash.  Memory on these devices is a premium.  Just including the <iostream> file, makes the ROM image grow by about 200 KByte, and I've got only about 768 KByte in total.  This is a known problem of <iostream>.  One workaround is not to include it in (library) header files, but only in source files where it is actually needed.  Header files should include <iosfwd> instead.
While skimming over the code, I've also spotted some opportunities to use std::make_shared.
Tested with 'make' and 'make test' on GCC 5.4.0 x86_64-linux-gnu.
Also briefly checked that the library can be used without including <iostream> on GCC 6.4.1 rx-elf.

Coverage remained the same at 100.0% when pulling e3bb156f826edaa25dd4186cd094440274b9b8b1 on olegendo:develop into f1c543cb7b229d8ff26a4d45ba92f8be55d2a47a on nlohmann:develop.

Thanks a lot!
Can you comment on the memory usage of the library? Is it usable in your usecase?

Ugh, sorry,  I didn't notice the examples.
I can't comment on the memory footprint of the library yet.  The <iostream> was a no-go for my case.  But now I can try using it for some data exchange between the embedded http server and the browser.
I'm wondering why you haven't used a base class and virtual functions for json_value instead of the open coded type-switch-cases?  At the first glance, it looks like it could chop off quite a bit of code, but I might be missing something here.  What's the reasoning behind the choice of data layout for class basic_json?
Notice that the statement in the "Memory efficiency" section of README.md is not complete.  sizeof (basic_json) is greater than sizeof (void*) + sizeof (uint8_t), because of struct alignment rules.

Good question. I think such a base class would be handy when it comes to using different types, because instead of assuming string types to have the basic_string interface, we could make this explicit (this may be relevant to https://github.com/nlohmann/json/issues/474#issuecomment-312450848). I'm not sure about the memory requirements though.
In any case, the library just "evolved" into this state. Using a union was perfectly reasonable when I started.

Great patch!

What I meant is something like that:
`
struct value_base
{
virtual ~value_base (void) { }
virtual bool empty (void) const noexcept = 0;
virtual size_type size (void) const noexcept  = 0;
virtual void clear (void) noexcept = 0;
...
};
class string_value : public value_base
{
public:
virtual bool empty (void) const noexcept override { return m_value.empty (); }
virtual size_type size (void) const noexcept  override { return 1; }
virtual clear (void) noexcept override { m_value.clear (); }
...
private:
std::string m_value;
};
class boolean_value : public value_base
{
public:
virtual bool empty (void) const noexcept override { return false; }
virtual size_type size (void) const noexcept  override { return 1; }
virtual clear (void) noexcept override { m_value = false; }
...
private:
bool m_value;
};
...
`
But I just noticed that you store bool and number types by value in the union.  This brings up the idea of storing all the various value types by-value in basic_json (via std::aligned_storage).  On a 32 bit target and libstdc++, sizeof (json) = 12 bytes, sizeof (std:.vector) = 12 bytes, sizeof (std::string) 24 bytes, sizeof (std::map) = 24 bytes.  So sizeof (json) would become 28 bytes, which means json numbers would take double the space than before.  On the other hand, it would eliminate one indirection when accessing json objects, arrays and strings.  Also, for each json object/array/string the heap management overhead per allocation should be taken into account (~8 bytes), which would be eliminated.
Is it worth going down that rabbit hole?  Hmm ... If you have access to some benchmarks, I could give it a try, just out of curiosity.

This sounds interesting! Related: #456 #474

I thought about this idea. It would mean a complete refactoring of the library. For instance, we would need to move the serialization implementation to the individual classes, and also a lot of the internal conversions which currently depend on the mere types (e.g., any kind of string representation can be somehow converted to string_t - we would now require these constructors in the string_value class).
It is a very interesting approach, and I shall open a feature branch. But I am not sure how long it would take to be able to really use it.
(Or maybe I am just missing something and things are much easier ;-))

I opened a feature branch: c1f4b249732400d64383824168d33ea953af5b1e. There are a lot of open questions (at least to me), but it's late and I'm not in a rush.

@olegendo Completely off-topic to this thread, is your project, where you are trying to use this library, open-source?

@pboettch Sorry, no.  Unfortunately it's not open source.

@nlohmann I have also created an experimental branch here https://github.com/olegendo/json/tree/virtual_funcs
Unfortunately, I can't get it to fully compile yet.  Somehow the to_json machinery is broken ... but I hope that it shows my intention.
I haven't done a lot of replacements that could be done with the json_value_* classes in place and left the type-switches.  Even if this might not make it into mainline as-is, I think it could be a good idea to wrap the direct member accesses (m_type, m_value.number_float and so on) in functions.  It will make future refactoring work easier.


Triggering a deploy

This commit is just to trigger deploy pipeline

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/webex-orb/webex-orb?version=dev:6f3527a804e20e4beb6697413daeb82fbdb77ca8

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/webex-orb/webex-orb?version=dev:6f3527a804e20e4beb6697413daeb82fbdb77ca8

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/webex-orb/webex-orb?version=dev:16c3b1a7d8342e18458ea49a192b193c3679e543


Removing KubeCon EU

Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you added your SVG to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Ooh fun! It needed the 'ads' so passing to @jordinl83 for 'why is broke'

Have Alex C come up with the new ad for NA :)
On Wed, Jul 7, 2021 at 6:31 PM Amye Scavarda Perrin <
@.***> wrote:

Ooh fun! It needed the 'ads' so passing to @jordinl83
https://github.com/jordinl83 for 'why is broke'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/pull/2201#issuecomment-876001701, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPSIKYA37MYD4DTTI5RVDTWTPUPANCNFSM477TAQAA
.

--
Cheers,
Chris Aniszczyk
https://aniszczyk.org

I've also removed the need for having ads

can we add kubecon na ad?
I'd rather not remove by default
On Thu, Jul 8, 2021 at 9:53 AM Jordi Noguera @.***>
wrote:

I've also removed the need for having ads
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/pull/2201#issuecomment-876507191, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPSIJM6B54QPNBOGRPRC3TWW3W7ANCNFSM477TAQAA
.

--
Cheers,
Chris Aniszczyk
https://aniszczyk.org


Upgrade duckdb dependency to 1.1.3

Upgrade duckdb dependency to 1.1.3
https://mvnrepository.com/artifact/org.duckdb/duckdb_jdbc

Test Duck

QuestDB / PostgreSQL

Will be released 4.04.


[launcher] Result view items overflow

We can either scale the list view height with the number of elements or set a maximum number of items in the list view based on screen size in the settings.

@crutkas PT Run setting has an option to set this manually. So, this should not be an issue anymore. I think this could be closed.


Full draft  of user example complete

This is the first draft of the package examples for the documentation! Please take a look.
@zackt113 I was going to put an example of how you can save your plots to your computer but I realized your function doesn't output the plot object, it only renders it! I am not sure if this is something we can easily change (ultimately it would be great if you could easily save your plots).

Updates have been made! I will merge!


add thread_id and message_id to basic message response and callbacks

Brian -- can you add DCO signoff (using the "-s" option on the commits).  Click on the "Details" link related to DCO to see what is needed at minimum.  All commits must be done with a "commit -s" option.
DCO - Developer Certificate of Origin - https://github.com/apps/dco

It appears this PR may contain changes from other branches that are not ready to be merged into master.  @brianorwhatever could you review the commit history on this PR and ensure it contains the changes you expect to be submitting and that the PR is targeted to the correct branch.

I'll close this as it's no longer important. Sorry!


[FLINK-9805][rest] Catch JsonProcessingException in RestClient

What is the purpose of the change
With this PR RestClient#readRawResponse catches all JSON-related exceptions. The exact circumstances where a non-JsonParseException is thrown isn't clearly defined, but one case when it can occur is when the input is empty.

+1


Release/0.5.0

@alexanderdean I think this PR is ready for review. Travis should be green as soon as schema-ddl will be published.

Can we get a blog post for this release please @chuwy ? It can be a short one (new features/bugs, plus how to upgrade).

This PR is top priority Tues.


The "effective security rules" link lands on a wrong page

The effective security rules link does not land on a page explaining the Effective Security Rules.
https://docs.microsoft.com/en-us/azure/virtual-network/virtual-network-manage-nsg-arm-portal

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

ID: 5e905802-0b08-4b3b-25ec-0f88c023bc32
Version Independent ID: f31325e5-49ea-b28c-336e-10f2a3fc12b4
Content: Azure network security overview
Content Source: articles/virtual-network/security-overview.md
Service: virtual-network
GitHub Login: @jimdial
Microsoft Alias: jdial

@SamirFarhat Thanks for the feedback! We are currently investigating and will update you shortly.

@jimdial can you take a look at this? The link does seem to be incorrect. However I am not sure of which link is supposed to be there. If you can point me to the right link I am happy to fix it. Else I will let you make the change :)
@SamirFarhat thanks for the find!

@SamirFarhat Thanks for catching.  I've fixed the link. It will publish later today.

@SamirFarhat  Closing this thread as resolved. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Question: I try use set_high() method but not found

I try to compile mi first blink for nrf52840 uC and review your examples for learning and see use directly set_low() after creating Pin instance  with into_push_pull_output
follow this example: https://github.com/nrf-rs/nrf-hal/blob/master/examples/spi-demo/src/main.rs#L31
this is my code
#![no_std]
#![no_main]

extern crate panic_halt;
use cortex_m_rt::entry;

extern crate nrf52840_hal;
use nrf52840_hal::gpio;
use nrf52840_hal::gpio::*;
use nrf52840_hal::gpio::p0::*;

use nrf52840_hal::Timer;

#[entry]
fn main() -> ! {
    let p = nrf52840_hal::pac::Peripherals::take().unwrap();
    let p0 = p0::Parts::new(p.P0);
    let mut led1: P0_24<gpio::Output<PushPull>> = p0.p0_24.into_push_pull_output(Level::High);

    let mut timer = Timer::new(p.TIMER0);

    loop {
        led1.set_low();
        timer.delay(2_000_000);
        led1.set_high();
        timer.delay(2_000_000);
    }
}

And this is an error message, both equal for set_low and set_high
error[E0599]: no method named `set_low` found for struct `nrf52840_hal::gpio::p0::P0_24<nrf52840_hal::gpio::Output<nrf52840_hal::gpio::PushPull>>` in the current scope
  --> src/main.rs:26:14
   |
26 |         led1.set_low();
   |              ^^^^^^^ method not found in `nrf52840_hal::gpio::p0::P0_24<nrf52840_hal::gpio::Output<nrf52840_hal::gpio::PushPull>>`

Use this debs
[dependencies]
nrf52840-hal = "0.11.0"
cortex-m-rt = "0.6.12"
panic-halt = "0.2.0"

set_low and set_high are from traits defined in embededd-hal, the traits must be in scope for them to work.
In the example you posted it was in this line
use embedded_hal::digital::v2::OutputPin;

but I think you will be better doing:
use nrf52840_hal::prelude::*;

Thanks for your quick response, my goal is to be able to add a new plate that my friends created.
I will continue learning from your catch and your experience, thank you


feat(shield): upsert service data API

Add upsert service data api for user and group entity.

@FemiNoviaLina please remove the unnecessary import. The rest lgtm


Updated user profile

Muutoksia profiilidatan responseen, halutaan että jotkin palaavat arvot voivat olla null -arvoja. Meneeköhän tällä muutoksella?

Muutoksia profiilidatan responseen, halutaan että jotkin palaavat arvot voivat olla null -arvoja. Meneeköhän tällä muutoksella?

Almost like that, but not quite right. When you want to switch the field to be Optional you also need to change the default value in the Field from ... (ellipsis, a special value, that happens to be used by Pydantic to indicate a field is required) to None (Python's equivalent to null) to indicate it's optional.
Thus for example this:
class Address(CamelCaseModel):
    street_address: str = Field(
        ...,
        title="Street address",
        description="Street address",
        example="Mannerheimintie 42",
    )

should become:
class Address(CamelCaseModel):
    street_address: Optional[str] = Field(
        None,
        title="Street address",
        description="Street address",
        example="Mannerheimintie 42",
    )

If you do this change to all those fields, you'll also see the OpenAPI spec gets changes to not mark the field any longer as required when the pre-commit hooks run.
P.S. I hope it was OK to answer in English; we've got people in the team that do not speak Finnish, so I prefer to keep discussions in English.

@lassipatanen, would you mind confirming it's OK that @LauriGofore wants to modify your definition? I guess there's many clues on you being colleagues and working together on this, but I just want to confirm this to be on the safe side. If this is not OK, then he should submit it it as a copy under his own name instead (so in src/test/LauriGofore/... instead).

Yes it's fine. We were working together on updating the model.


add link to helm repo to readme

closes #6

The release failed for this? Did we need it to succeed? Can we make sure that only real chart changes trigger the release action perhaps? (Run only if changes in the helm dir and not md files?)


gMSA \ LdapRelayScan - verify the location

Hi !
How i can fix this trouble?
`
[*] gMSA Dump
[-] Please verify the location of gMSADumper.py
[*] LdapRelayScan checks
[-] Please verify the location of LdapRelayScan.py
`

Hello !
Did you try executing the linWinPwn.sh script from the linWinPwn folder ?
If the error still occurs, you can also modify line 46 scripts_dir="./Scripts" to the full location of the scripts (for example /opt/linWinPwn/Scripts  or /root/linWinPwn/Scripts, depending on where you cloned the repo)

It helped me to fix the error by changing the path on line 46 to full. Thanks


fail to serve RPC(upload-pack): exit status 128

Gogs version (or commit ref): <IP_ADDRESS>1
Git version: 1.9.1
Operating system: Ubuntu 14.04.5 LTS
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[x] Yes (provide example URL) read description (server DOS will occur)
[ ] No
[ ] Not relevant

Logs:

2016/11/29 10:39:56 [...routers/repo/http.go:384 serviceRPC()] [E] fail to serve RPC(upload-pack): exit status 128
Description
When cloning the (big) repo https://github.com/firehol/blocklist-ipsets and pushing to my local gogs instance (8GB RAM 4 CPU), I got a DOS when trying to render the repo (CPU are all spinning for some minutes) with final web timeout.
Watching the processes taking up all the CPU (they are continuously switching files):
git      19521  9.0  0.6<PHONE_NUMBER>4 ?        R    10:54   0:00 git log -1 --pretty=format:%H 8abe9916a386b27434c5e8a72897dfb10bb5ca34 -- bambenek_cryptowall_ipdeny_country.json
git      19522  8.0  0.5<PHONE_NUMBER>0 ?        R    10:54   0:00 git log -1 --pretty=format:%H 8abe9916a386b27434c5e8a72897dfb10bb5ca34 -- bambenek_cryptowall_retention.json
git      19523  0.0  0.3<PHONE_NUMBER>8 ?        R    10:54   0:00 git log -1 --pretty=format:%H 8abe9916a386b27434c5e8a72897dfb10bb5ca34 -- bambenek_dircrypt.json

I did the same on https://try.gogs.io/gelim/blocklist-ipsets and it looks the same. WARNING if you access this URL your server/VM will have a hard time :-|
NB: The new repo will properly appear in my main list: https://try.gogs.io/gelim/
$ git remote add test https://try.gogs.io/gelim/blocklist-ipsets.git
$ git push test
Username for 'https://try.gogs.io': gelim
Password for<EMAIL_ADDRESS>
Counting objects: 314375, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (4287/4287), done.
Writing objects: 100% (314375/314375), 363.89 MiB | 6.06 MiB/s, done.
Total 314375 (delta 310921), reused 313466 (delta 310012)
To https://try.gogs.io/gelim/blocklist-ipsets.git
 * [new branch]      gh-pages -> gh-pages
$

If you try to access directly one of the files, for instance README.md, it should be here:
https://try.gogs.io/gelim/blocklist-ipsets/src/master/README.md
you'll get a 404. Files haven't been unpacked in the repo.
Is it an upload limit? (NB: I push via SSH on my local gogs instance)

Please upgrade to latest release 0.9.113 and redo the test, you'll got more specific error message other than just 128.


vectorstores: add Milvus

In quest to run langchaingo locally without network dependencies. I've added the milvus vector store and huggingface's text-embeddings-inference api client.
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

fixes #59

@pattonjp why close this?

created in this repo accidentally. will reopen as draft while testing.

@pattonjp @tmc  How about this vectorstores Milvus？we need this 😄

@pattonjp can the text-embeddings-router go into the docker-compose config?

@pattonjp why not use ollama for embeddings?

Specifically, the tei provider has some additional options - but could these be passed to the underlying client directly, instead of to the embedder? The embedder then just wraps any client

sure will have pr today


Fix for issue 3797.

Docker's logic for resolving credentials from .dockercfg accepts two kinds of matches:

an exact match between the dockercfg entry and the image prefix
a hostname match between the dockercfg entry and the image prefix

This change implements the latter, which permits the docker client to take .dockercfg entries of the form:
https://quay.io/v1/
and use them for images of the form:
quay.io/foo/bar
even though they are not a prefix-match.

LGTM.  Thanks for the quick fix!


First example to actualy use smesh

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

Comparison is base (c457554) 100.00% compared to head (df096ac) 87.50%.

Additional details and impacted files
@@             Coverage Diff              @@
##              main       #3       +/-   ##
============================================
- Coverage   100.00%   87.50%   -12.50%
============================================
  Files            1        1
  Lines            1        8        +7
============================================
+ Hits             1        7        +6
- Misses           0        1        +1

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

@bennibolm this is ready for review now and can be merged once you're happy

Thanks a lot @sloede. Nice explanation on how to actually use it. Here and for smesh 💪

Thanks! feel free to squash merge if you're happy!


ExportSession Error Domain=AVFoundationErrorDomain Code=-11800 "The operation could not be completed"

Hi,
I'm trying to export my video but when I try export 3rd or more video thats giving me that error.
Error Domain=AVFoundationErrorDomain Code=-11800 "The operation could not be completed" UserInfo={NSUnderlyingError=*********{Error Domain=NSOSStatusErrorDomain Code=-12983 "(null)"}, NSLocalizedFailureReason=An unknown error occurred (-12983), NSLocalizedDescription=The operation could not be completed})
Also this is my code.
let assetExportSession: SCAssetExportSession = SCAssetExportSession(asset: self.avAsset)
            assetExportSession.outputUrl = self.recordSession.outputUrl
            assetExportSession.outputFileType = AVFileTypeMPEG4
            assetExportSession.videoConfiguration.preset  = SCPresetLowQuality
            //assetExportSession.videoConfiguration.preset = AVAssetExportPresetLowQuality
            assetExportSession.exportAsynchronouslyWithCompletionHandler({
                if assetExportSession.error == nil {
                    objUploadVideoPath = "\(assetExportSession.outputUrl!)"
                    print(objUploadVideoPath)
                    uploadVideoFeed()
                    self.performSegueWithIdentifier("showVideopostSegue", sender: self.imgCapturedImage)
                } else {
                    print(assetExportSession.error)
                }
            })

Problem is solved with                    self.recordSession.deinitialize()

@mkaya93 Glad you figured it out. AVFoundation is very particular about how many instances of certain classes can exist in memory.

@mkaya93 where to use self.recordSession.deinitialize()???

@guodong000 when you closing the view you have to use this.

@guodong000 He's most likely calling it in viewWillDisappear


Add introduction to opam release process

This is kind of like #27, but I don' think people who are just starting out should worry about how to set up a package to go into opam.
What I have in mind is a guide that tells you how to release a package in simple cases, but notes likely potential gotchas, and has links to further info for more complex cases and for understaning the process under the hood.
This is something I'm trying to figure out how to do now.  My impression is that Dune allows you to avoid many of the steps in the opam docs (in which case they should be left out of the initial presentation in the Ocamlverse guide), but I'm not sure, and in any event after reading relevant pages in the Dune docs, I have questions that neither set of docs answers.
They guide could be linked to from both Learning and Ecosystem.

I don't know what you mean by "to set up a package to go into opam" — you just create an opam file (which says a few things like what command builds the thing and what versions it's good for) and that's most of the work. The docs are reasonably clear (which is unusual in OCamlland.) Dune doesn't change anything here, you just end up invoking it instead of (say) make.

OK: I know what "build" means, and I know thaat I get packages installed, along with dependencies when I type opam install.  And that allows me to build my projects that use those packages.  I even know a bit about what's under .opam .  But I don't know what you mean by "build with opam".
I have foo.opam files--jbuilder requires them--but their content doesn't matter for anything I've done in the past, afaics.
I also don't know how packages  get into the database, and I am trying to learn how to put a package into the database.  That's somethimg others will want to understand, and it's the point of the suggested guide.
(That's not request for help with opam in an ocamlverse issue, of course.  Just an illustration of how this stuff can be confusing if you don't already kind of understand it.)

On publishing to opam, I'm thinking that the Packaging guide (and the one for opam2) is quite detailed. It involves creating an opam files (the guide mentioned which fields are important for publishing), test to install it locally with opam install ., and publishing (create a PR) automatically via the opam-publish tool.
Topkg (topkg-care) will also help with that (and more), with the addition of automatically testing your lib, creating release notes, tagging, creating and publishing your artifacts and docs, substituting version strings, and creating the PR.
These steps (which are intended to be a common flow of publishing packages) are documented in the man page, so to find it you must opam install topkg-care and run topkg help release. Daniel has written a lengthy explanation on each steps of the flow, but unfortunately it's not really straightforward to get to the manpage. To use topkg with dune, there's the topkg-jbuilder package.


When you add to home screen you see "React App" as title

This is just a simple change to the index.html title
I'll do it in a sec, just reminding myself. Anyone could jump on this too.

Seems that it has already been done in https://github.com/austintgriffith/burner-wallet/commit/9f2103a1e74324ef9e797b2d0d0991dd3795ce91#diff-528c3923d718a8860f5d8c05c3931c55


Full project using the A9G - mail notifier (not an issue, but could help with project design)

This isn't an issue, but it could help people with their firmware and hardware designs for the A9/A9G. I've made a mail notifier using the A9G that sends JSON data to a web server when triggered. It also has a BME280 which communicates with the module via I2C, uses UART to communicate with an ATtiny402 microcontroller and can last for around 4-5 years on a single charge of a 10440 lithium battery (~320mAh). It also has a GPS tracker mode, and has hardware for controlling power to an active GPS antenna.
https://github.com/zkemble/MailboxNotifier
https://blog.zakkemble.net/remote-mail-notifier-and-gps-tracker/

Got I2C working thanks to @zkemble but it took me to long to learn how to use the API's.
Loaded the AT firmware and hooked up a ATMega328 (TQFP body).
Now after a few hours programming I have low power mode (0,22uA), triggers, SMS, RTC and store settings into EEPROM. I'm happy now :)

Sorry for sending the code @Pasboe been busy busy week.
So are you using an atmega to control the rtc which then wakes the pudding?

@zkemble Do you have a simple code about connect and get temperature data from the Adafruit_MLX90614 temperature sensor with the A9G development board
Can you help me please?


Examples in Readme.md are deprecated compared to the version ^4.21.0

Confirm this is a Node library issue and not an underlying OpenAI API issue

[X] This is an issue with the Node library

Describe the bug
The examples provided in the Readme.MD are deprecated in ^4.21.0, it's really that hard to provide a link to the docs or just update the read.me togheter with the code before releases a new version?

To Reproduce

Install  "openai": "^4.21.0",
Try to use client.beta.chat.completions.runFunctions
Function already deprecated, what's the new function example?

Code snippets
OpenAI from 'openai';

const client = new OpenAI({
  apiKey: OPENAI_API_KEY, // This is the default and can be omitted
});

async function main() {
  const runner = client.beta.chat.completions
    .runFunctions({
      model: 'gpt-3.5-turbo',
      messages: [{ role: 'user', content: 'How is the weather this week?' }],
      functions: [
        {
          function: getCurrentLocation,
          parameters: { type: 'object', properties: {} },
        },
        {
          function: getWeather,
          parse: JSON.parse, // or use a validation library like zod for typesafe parsing.
          parameters: {
            type: 'object',
            properties: {
              location: { type: 'string' },
            },
          },
        },
      ],
    })
    .on('message', (message) => console.log(message));

  const finalContent = await runner.finalContent();
  console.log();
  console.log('Final content:', finalContent);
}

async function getCurrentLocation() {
  return 'Boston'; // Simulate lookup
}

async function getWeather(args: { location: string }) {
  const { location } = args;
  // … do lookup …
  return { temperature: 40, precipitation: 30 };
}

main();

// {role: "user",      content: "How's the weather this week?"}
// {role: "assistant", function_call: "getCurrentLocation", arguments: "{}"}
// {role: "function",  name: "getCurrentLocation", content: "Boston"}
// {role: "assistant", function_call: "getWeather", arguments: '{"location": "Boston"}'}
// {role: "function",  name: "getWeather", content: '{"temperature": "50degF", "preciptation": "high"}'}
// {role: "assistant", content: "It's looking cold and rainy - you might want to wear a jacket!"}
//
// Final content: "It's looking cold and rainy - you might want to wear a jacket!"

OS
macOS
Node version
v16.14.0
Library version
openai v4.21.0

Changed the branch to beta shows the real code snipped.

Thanks for reporting, we'll fix this shortly.


Handle non-UTF-8 OsStr on Windows

I noticed this issue, so I thought I would help move it forward and see how my crate handles a larger project.
Closes #339

Checks not running for PRs from forks is fixed in GH-574.

Thanks. I was wondering what was blocking them.

The crate referred to in the PR description is https://crates.io/crates/os_str_bytes:

Traits for converting between byte sequences and platform-native strings

Thanks for the PR @dylni. I'll try to review it tonight (PDT, UTC-7).

Great. Thanks for the interest.

Ok @lopopolo. bstr does still require valid UTF-8 on Windows, but I understand if you want to use a crate that has existed longer for this.


BUG:Negative pricing

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The Book Exchange Hub has a pricing part where the number decrement  to negative values which is not realistic. So it can be limited to a certain minimum value.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned @ananya-dhiman
Make sure to star the repo before start working on!


Add image buffer upload

I see by a comment on another issue that this seems to be no longer maintained. Given this, feel free to close this PR


CFR days start when request changed to CFR state

As an analyst and ministry coordinator
I want to a 10 day clock to start when a request is changed to CFR
so that I can track how towards the CFR deadline

Assumptions & Scope
What are the assumptions for this story?
What is in and not in scope?
Changing out of the CFR state
Extensions
Notifications that the CFR due date is near
Acceptance Criteria
Scenario 1: Start 10 day clock

GIVEN a request is in an open state
WHEN the IAO/Flex team analyst changes the state to Call for Records
THEN a 10-day count down will start before the CFR is due
AND the number of left days will appear at the bottom of the left nav bar above the legislated due date
AND a Call For Records due date field will appear on the request view (not editable)

Scenario 2: Update clock

GIVEN a request is in a Call For Records state
WHEN a 24 hour time period expires
THEN the number of days remaining displayed in the left navigation menu will decrease by one day

Scenario 3: xxxxxx
...
Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

@arielleandrews, @JHarrietha-AOT, please review the above scenarios and advise on edits/omissions.

Thanks Loren. For Scenario 2, the clock should update at/after midnight, rather than a 24-hour period.
Should we include a Scenario 3 where after the due date has passed it turns red and counts down into the negative? Is there an action we should include in the design that will stop the day counter?

@arielleandrews
I'll update the scenario for a midnight change. Just confirming if we send a CFR to a ministry at 5pm on a business day, that when the clock strikes midnight, they are down to 9 days?
Yes, to a scenario 3.

@arielleandrews I agree, I currently have the Item in the queue in red if it is past the IAO Records Due Date. Let me know if you think this is an effective way to show that in the queue. Was thinking about having it turn to orange/yellow when it is day before due date as well, as there were a few Ministries who colour coded there trackers manually and found it visually helpful. I also agree and have "RECORDS DUE' go into the negative once the due date has passed should be included

@lmullane confirming the day the CFR is sent to the Ministry is not counted in the 10 days they have to respond. Example: CFR is sent on Sep 9, response is due Sep 23.
@JHarrietha-AOT that makes good sense to me. Visual reminders are always helpful.

Thanks, @arielleandrews. Is it the next business or the next calendar day?

@lmullane next business day

@arielleandrews, @JHarrietha-AOT, please review updated scenarios above reflecting your comments.
@JHarrietha-AOT, instead of negative days when overdue suggest we just state the number of days overdue.

@lmullane Scenario 2 would follow the same principle and the clock/number of business days would update at midnight. We track always according to the number of days to the legislated due date and don't get into partial days

@lmullane @arielleandrews Agree with Arielle that scenario 2 shouldn't be about the 24 hours passing but about it being past midnight and the next buisness day. Ans agree that days should show at 'X days past due' not in the negative.
Otherwise looks good.

Displaying the CFR due date on the Request View page is not in scope for this story; it will be covered in a different story.


[move-prover] collect types in write-back chain for mono analysis

In mono analysis, an implicit assumption is that all types used in the function body will appear either in parameters, locals, return tyupes, or spec exprs.
While this is true and in a rigourous implemention, should hold, it might be hit by a cornor case where an imcomplete (and potentially) inconsistent modeling of borrow natives are in place.
An example is shown in the dynamic_field::borrow_mut model:
module sui::dynamic_field {
    struct Field<Name: copy + drop + store, Value: store> has key {
        id: UID,
        name: Name.
        value: Value,
    }

    public fun borrow_mut<Name: copy + drop + store, Value: store>(
        object: &mut UID,
        name: Name,
    ): &mut Value {
        let object_addr = object::uid_to_address(object);
        let hash = hash_type_and_key(object_addr, name);
        let field = borrow_child_object_mut<Field<Name, Value>>(object, hash);
        &mut field.value
    }
    spec borrow_mut {
        pragma opaque;
    }

    public(friend) native fun borrow_child_object_mut<Child: key>(object: &mut UID, id: address): &mut Child;
}

With a declaration of in the prover CLI:
        options
            .prover
            .borrow_natives
            .push("dynamic_field::borrow_child_object_mut".to_string());
        options.backend.borrow_aggregates.push(
            move_prover_boogie_backend::options::BorrowAggregate::new(
                "dynamic_field::borrow_child_object_mut".to_string(),
                "GetDynField".to_string(),
                "UpdateDynField".to_string(),
            ),
        );

Now, the opaque pragma in borrow_mut mark that this funciton is abstract and when this borrow_mut is used, it actually causes an write-back to the internal Field struct which does not show-up in the function locals (or anything that is collected by mono analysis that has a type).
Of course in an ideal situation, the borrow_mut should be modeled completely such that the Field struct is either eliminated from the borrow chain or the Field should be exposed. However, achiving this complete modeling takes time and this half-backed solution might need to stay a bit longer. It is also possible that we do need this facility (of using Field but not exposing Field in the Prover). In any case, it merits a PR as a block.

Motivation
Unblock partial modeling
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
It is very hard to create a simplified test case for this, unfortunately.

I think this fix probably worth a wider audience as it highlights a very corner (although highly unlikely) case in the mono process. @rahxephon89 might also comment on whether this is needed on Aptos side.


Add --build=outdated to conan lock commands

After a series of tests I came to the conclusion that this flag is
needed when creating the full lock-file. Without it conan won't include
build requirements into the lockfile for dependencies that need to be
built locally. And when these are not included the next step (conan install) will fail because it is missing a locked dependency from the
graph.

@pierricgimmig This should hopefully also resolve the  "cmake_installer not in lockfile"-problem. Sorry that it took so long.


Chatbot History Reset.

Chatbot History in Branch main+ChatBot_merge doesn't work.

This issue is fixed

Need to work on Bringing Landing Page question into chatbot page

Another issues open pertaining to this, will close now due to completion.


gam.summary() - return String rather than "printing" to standard out

Or add another method that returns as String to retain backwards compatibility.
This would enable easier saving of statistics to the file system, which can be more useful if incorporating pyGAM into a pipeline.
Currently I'm having to redirect stdout, which works, but it's a bit inelegant.
Thanks!

@marknormanread thats interesting!
most of the statistics printed by the summary() method are stored in gam.statistics_ although they might require a little post-processing
which statistics are you interested in!
-dani

@marknormanread i am going to close this issue with your permission.
please re-open it if you see fit!


bootstrap: fix use of deprecated -Xmanifest flag

We should use -Xbuild-tools-swiftc instead, as the deprecation warning recommends.

@swift-ci smoke test

macOS failure:
--- bootstrap: note: Building SwiftPM (with a freshly built swift-bootstrap)
Error: Missing value for '-c <configuration>'
Help:  -c <configuration>  Build with configuration
Usage: swift-bootstrap [--package-path <package-path>] [--scratch-path <scratch-path>] [--configuration <configuration>] [-Xcc <Xcc> ...] [-Xswiftc <Xswiftc> ...] [-Xlinker <Xlinker> ...] [-Xcxx <Xcxx> ...] [--verbose] [--very-verbose] [--use-integrated-swift-driver]
  See 'swift-bootstrap --help' for more information.

I wonder if this could be caused by a bug in ArgumentParser?

IIRC, swift-bootstrap has its own set of arguments that's more restrictive than regular swift build, so we may need to add the new argument there. It doesn't need deprecation since it is supposed to be build from source when used, so maybe we can just rename it.

@swift-ci smoke test

@swift-ci test windows


Loot tracker does not keep track of all the kills

So basically i was killing Skeletal Wyverns and i closed the client yesterday on ~1300 kills, i reopened the client the next day and only had the loot showing from 1050 kills.
Please have a look into the loot tracker plugin :(

The loot tracker only loads the newest 1000 drop stacks when the client opens. This is by design

Check https://runelite.net for full history

A bit more is explained in #7485 as well. But as deathbeam said, you can log in to runelite.net to see the full loot history.

The loot tracker only loads the newest 1000 drop stacks when the client opens. This is by design

a friend of mine was killing zulrah and had 50 kills one day the next day he had 31 on loot tracker, how is that possible?


Update the title of the Dusky deployment tile

Azure Data Studio Version:

The title of the Dusky deployment tile in ADS is incorrect:

It should be:
PostgreSQL Hyperscale server groups - Azure Arc (Preview)

Being addressed in https://github.com/microsoft/azuredatastudio/pull/12187


Custom components aren't being assigned a key

Markup is being applied to the page, but React is complaining it doesn't have a key,

This error only occurs when the custom component is on the page

This is fixed in new version about to arrive :)

Next version is out :)

@christianalfoni awesome, I look forward to trying it out soon!


Fixed Extreme Weather Warning Bug

Fixed the issue #93 where after selecting the city with extreme weather, the warning stays even if you select some other city with normal weather
Video Demonstrates the fixed change
Warning Fix.webm

Bug fixed and website functions normally. Will merge this PR now.


Got cairoimpl.nim(646, 22) Error: undeclared identifier: 'Rectangle' when compile sample

.nimble/pkgs/gintro-0.6.1/gintro/cairoimpl.nim(646, 22) Error: undeclared identifier: 'Rectangle'

Seems you are from China, can you access Nim forum?
Because it seems that your error is
https://forum.nim-lang.org/t/5420#33920
Basically you use an too old GTK, the issue is with 3.22 while 3.24 is fine.
If you can not read forum I will post some more details here soon...

Or maybe you can access
https://github.com/StefanSalewski/gintro/blob/master/README.adoc
There we tell
"Starting with current version v.0.6.0 we support gstreamer (gst). At the same time we have split cairo module into an gobject-introspection basic part and an manually created part. Unfortunately the gobject-introspection is not available for very old GTK/cairo libraries, so installation may fail for you. Use v0.5.5 in this case. Also we support gBoxed types now, this is assumed to work well but is not well tested yet. Command to install older releases should be something like nimble uninstall gintro followed by nimble install<EMAIL_ADDRESS>In the free world nearly every serious user should have GTK 3.24 already. In China that may be different. If there are many people with serious desire using GTK3 with Nim then we would have to provide a fix. Maybe simplest fix for you is to remove "include cairoimpl" statement in
$ grep include ~/.nimble/pkgs/gintro-0.6.1/gintro/cairo.nim
include cairoimpl

That will restrict you when you want to use cairo drawing, but other GTK should work. Or just use old version 0.5.5.

Hi, thanks for reply, I can see problem now by old gtk version. And I'm not Chinese, I'm Vietnamese , I'm free what's problem with my country ? I'm using ElementaryOS with GTK 3.22 as below
$dpkg-query -W libgtk-3-bin
libgtk-3-bin	3.22.30-1ubuntu4

Great that you live in a free country. I hope that you can get a never version of GTK then. For testing gintro 0.5.5 is not that bad, it should  work with GTK 3.22. Maybe you will not like GTK so much at all...


get-next-version can't handle a commit tagged with multiple version granularity tags

What happened incorrectly?
A git commit tagged with, for example, v4, v4.5, and v4.5.14 results in  get-next-version complaining that the commit is tagged with multiple semver versions. This tagging format is common in some workflows to always point to the most recent v4 release or v4.5 minor release sequence in addition to the specific release commit.
What should have happened instead?
get-next-version should use only the actual semver formatted tag (the one with all version elements) to calculate the next version.
What steps are needed to reproduce the bug?

Tag a commit with tags in the format v<major>, v<major>.<minor> and v<major>.<minor>.<patch>
run get-next-commit and see the error.

Thanks for bringing up this issue.
Since we do not use these shortened version numbers in our processes, we are not planning to implement this.
However, if you come up with a PR, we would be happy to merge it.


Is this relevant to version 2.0?

I note that this page calls out "x5t" but in other areas of the documentation such as the access token reference (https://docs.microsoft.com/en-us/azure/active-directory/develop/access-tokens#validating-tokens) it says 'x5t' "Functions the same (in use and value) as kid. x5t is a legacy claim emitted only in v1.0 access tokens for compatibility purposes".

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

ID: 7e44b783-3ca5-6b6a-1f33-d7a107b8f8e0
Version Independent ID: aa2bb5fd-bd0d-830c-84c7-0525657b6d4f
Content: Certificate credentials in Azure AD
Content Source: articles/active-directory/develop/active-directory-certificate-credentials.md
Service: active-directory
Sub-service: develop
GitHub Login: @CelesteDG
Microsoft Alias: celested

@icebrg, Thanks for the feedback! We are taking a look into this and will get back to you soon.

Hi @icebrg ,
"x5t" is X.509 Certificate Thumbprint which is used to identify specific certificates. This claim value emitted only in v1.0 jwt access tokens. As per the document, The v1.0 endpoint returns both the x5t and kid claims, while the v2.0 endpoint responds with only the kid claim. Going forward, we recommend using the kid claim to validate your token and it is not relevant to version 2.0.
I hope the above information is helpful. We will now close this issue. Should you have any further queries, please tag me to your reply and we can gladly continue the conversation.
Thank you.


seth: init at 0.5.0

Motivation for this change
Seth is a command-line utility for working with the Ethereum blockchain. It is a dependency for some other Ethereum-related programs as well as being useful in its own right.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Looks good to me. Could you squash your change into a single commit?

@pSub Done, thanks a lot.


BinOp questions

I am trying to understand the signature of the binOp methods in G2 kernels. For example, I see:
void CartFieldBinOpMultiply1xSer_P2(binOpData_t* data, const double *A, const double *B, const short int Ncomp, const short int eqNcomp, double *out)

What do args Ncomp eqNcomp accomplish? Is this to allow multiplying say a sub-set of components? Is this facility used anywhere and should we retain it (in some form or the other)?
Also, I see the arg `binOpData_t* data'. I see it used for example:
data->AEM_S = Eigen::MatrixXd::Zero(3,3);

It seems to me this defeats the purpose of having data, right? I mean, this call will create a new matrix of 3x3 and copy it to the data->AEM_S. Hence, the original contents will be deleted, new copy made and copied into the variable. So it is even more expensive than creating the matrix in-line! So why even have a data if again and again in each cell in each step we alloc/dealloc matrices etc?

eqNcomp is used to differentiate between vector-vector and scalar-vector
multiplication. The former was used for a diagnostic that computed kinetic
energy direction-wise, but is no longer used I think. Kolter is dropping it.
nComp is used to tell the kernel how many vector components there are
(e.g. 2 for a 2v M1i), to perform operations component-wise.
Eigen::MatrixXd:Zero(3,3): I looked at that a couple of days ago and was
under the impression that that is simply calling the zero-out function in
the Eigen::MatrixXd class, not actually creating a new function.
But you may be right, and it's creating a new matrix. I just did a quick
search and found that static matrices like the ones we are using have
a setZero()
method
https://eigen.tuxfamily.org/dox/classEigen_1_1DenseBase.html#af230a143de50695d2d1fae93db7e4dcb.
So this perhaps should be replaced by data->AEM_S.setZero()
On Sun, 18 Jul 2021 at 09:53, Ammar @.***> wrote:

I am trying to understand the signature of the binOp methods in G2
kernels. For example, I see:
void CartFieldBinOpMultiply1xSer_P2(binOpData_t* data, const double *A, const double *B, const short int Ncomp, const short int eqNcomp, double *out)
What do args Ncomp eqNcomp accomplish? Is this to allow multiplying say a
sub-set of components? Is this facility used anywhere and should we retain
it (in some form or the other)?
Also, I see the arg `binOpData_t* data'. I see it used for example:
data->AEM_S = Eigen::MatrixXd::Zero(3,3);
It seems to me this defeats the purpose of having data, right? I mean,
this call will create a new matrix of 3x3 and copy it to the data->AEM_S.
Hence, the original contents will be deleted, new copy made and copied into
the variable. So it is even more expensive than creating the matrix
in-line! So why even have a data if again and again in each cell in each
step we alloc/dealloc matrices etc?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ammarhakim/gkylzero/issues/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ6DKDFZALMHD6DIF7AYHDTYMBGZANCNFSM5ASJX4NQ
.

When you say "perform operations component-wise" do you mean multiplying two DG vector fields together, for example?
Before any new kernels are generated, please talk to me. As I said last week, I will do the multiply kernels myself (done already). At this point I believe the more important thing is to design the multi-species collisions properly and not worry about the binops per-se. I will set the code structure over the next few days for this and then divisions can be implemented. More importantly for now:
Please list the precise set of bin-ops needed in collisions. So, for example, scalar-scalar multiply, vec-vec multiply etc etc. That will allow us to make sure these ops are ready.

I mean multiply uu* where u is a vector, so that in 2v the result is a
vector field with [u_x^2, u_y^2].
Conf-space operations needed are

scalar * scalar
scalar * vector
scalar / scalar
vector / scalar
vector . vector
Phase space operations needed:
conf scalar * phase scalar

vector*vector component wise is not needed (for now).
On Sun, 18 Jul 2021 at 11:36, Ammar @.***> wrote:

When you say "perform operations component-wise" do you mean multiplying
two DG vector fields together, for example?
Before any new kernels are generated, please talk to me. As I said last
week, I will do the multiply kernels myself (done already). At this point I
believe the more important thing is to design the multi-species collisions
properly and not worry about the binops per-se. I will set the code
structure over the next few days for this and then divisions can be
implemented. More importantly for now:
Please list the precise set of bin-ops needed in collisions. So, for
example, scalar-scalar multiply, vec-vec multiply etc etc. That will allow
us to make sure these ops are ready.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ammarhakim/gkylzero/issues/7#issuecomment-882099736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ6DKFEVNOJX5CVIP7WU23TYMNJRANCNFSM5ASJX4NQ
.

I am proposing to reorg the bin_op kernels in the following way. The kernel signature will be changed to (for example)
void binop_mul_1d_ser_p0(const double *f, int fstart, const double *g, int gstart, double *fg, int fgstart )

The fstart gstart and fgstart will indicate the starting point in the respective arrays. So scalar-scalar multiply to copy into a scalar file would look like:
binop_mul_1d_ser_p0(f, 0, g, 0, fg, 0);

Scalar-vector multiply would need a loop:
for (int d=0; d<NDIM; ++d)
  binop_mul_1d_ser_p0(f, 0, g, d, fg, d);

where f is a scalar field and g and fg are vector fields. This would basically scale every vector component by f.
This is considerably more flexible than at present as it allows essentially arbitrary combinations of ops by calling things in a loop with the appropriate start index.
Let me know if this is an issue or if there are other suggestions.

Actually, now that I think of this: there is really no need for any such extra start indices. The code that calls the kernels can increment the pointer appropriately as it wants. The kernel should assume that only a nbasis length array is passed to it, leaving to the caller to ensure proper pointer arithmetic has been done to access the appropriate vector/scalar as needed, and also add the loops as needed.

Presumably pointer arithmetic doesn't cost anything. So this seems fine.
On Sun, 18 Jul 2021 at 18:20, Ammar @.***> wrote:

Actually, now that I think of this: there is really no need for any such
extra start indices. The code that calls the kernels can increment the
pointer appropriately as it wants. The kernel should assume that only a
nbasis length array is passed to it, leaving to the caller to ensure
proper pointer arithmetic has been done to access the appropriate
vector/scalar as needed, and also add the loops as needed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ammarhakim/gkylzero/issues/7#issuecomment-882156503,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ6DKAHUZ6IXAXLTZC2ISLTYN4WPANCNFSM5ASJX4NQ
.

This is now resolved. I fixed the bin_op signatures to be like array_ops without the need for an updater. This allows using the ops in a more flexible manner. All combinations of scalarscalar, scalarvector etc are supported.


Wooden trapdoor crucibles to iron

Should be iron trapdoor.
(only seen in JEI, not in-game yet)

Thanks! Fix in next release


Can't resolve 'dns','net','tls'

Can't resolve 'dns','net','tls'  in  xxx\node_modules\ioredis\built\connectors'

How to solve?


Directories and files naming convention

Description
The files and directories naming is not consistent and does not follow the flutter style guide found in the link below.
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
Domain
Android (Flutter)
Code of Conduct

[X] I follow Code of Conduct of this project.

#15

Closing this issue


UofT-DSI | SQL - Assignment

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
I designed and updated the schema
What did you learn from the changes you have made?
I learned how to designed an ERD and also the privacy and safety of data
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ 1] I can confirm that my changes are working as intended

Hello, I realized that the differences I highlighted for Q4 may not be uploaded, please see below
LMu4W-2.pdf


Release/1.0.3 ahjumawi

#584 - Capitalize README
#688 - Design tokens add a reference page
#694 - Update PR to allow capitalization in github branches
#698 - Accordion: Better multiline summaries, fixes to open indicator icon
⚡ Markup change since 2.0.0: accordion markup will need to be updated (ds-accordion, 2.0.1)
CHANGELOG
#703 - Step List: Adjust padding on ds-step-list

Looks good! Reviewed major templates and accordion updates and new design tokens page in Chrome, Firefox, Safari on mac.
New design tokens page looks excellent @kkoryaka!


lable:question Is this module a performance boost?

I would like to see fs do things a lot faster! I need to fs.writeFile 112 times (writing separate png files) once every 13 seconds. I want to do this in a for loop with fs writing each whole file regardless of weather it has finished writing the previous.
Normally fs would start to write the file and have to complete the job before it can begin to write the next. Just so I absolutly understand what you have made...does your module let fs multitask when writing in a for loop?
Thanks

If you use the async methods of fs instead of the sync your code will perform multiple concurrent file system operations. This may make your code faster.
this module is used to copy files to remote servers.
because this issue is not related to multifs itself im going to close it.


Fix missing root dir prefix when loading dynamic config

The cadence server would fail to load the dynamic config file
when the working dir wasn't the parent dir of dynamic config
file path.

What changed?
Fix missing root dir prefix when loading dynamic config

Why?
The cadence server would fail to load the dynamic config file
when the working dir wasn't the parent dir of dynamic config
file path.

How did you test it?
local test

Potential risks
no

@longquanzheng Can you have a look at this PR?

@longquanzheng Can you have a look at this PR?

Thanks for finding and fixing it.

Actually, this will break the case that using absolute path.
Maybe we should check if a path is absolute before transformation

I didn't check  the path  since the comment says that should be relative to the root directory
// Filepath should be relative to the root directory
DynamicConfigClient dynamicconfig.FileBasedClientConfig `yaml:"dynamicConfigClient"`

Actually, this will break the case that using absolute path.
Maybe we should check if a path is absolute before transformation

I didn't check the path since the comment says that should be relative to the root directory
// Filepath should be relative to the root directory
DynamicConfigClient dynamicconfig.FileBasedClientConfig `yaml:"dynamicConfigClient"`

I see. That is misleading, sorry for that. We should remove the comment.
maybe that’s the initial intent but the code doesn’t restrict it. As a result, people are using it with absolute paths now. See https://uber-cadence.slack.com/archives/CNC31V3K7/p1616084908003400?thread_ts=1616084757.003300&channel=CNC31V3K7&message_ts=1616084908.003400
so I think we have to support it anyway. Otherwise I don’t think those people have a workaround

so I think we have to support it anyway. Otherwise I don’t think those people have a workaround

Okay, I'll fix it and remove the comment later 👍

so I think we have to support it anyway. Otherwise I don’t think those people have a workaround

Okay, I'll fix it and remove the comment later 👍

Cool. Can you do the same for loading config? I think it’s okay if config is outside root directory. This will probably make it easier/simpler  to use homebrew. And also add description to the root CLI parameter: it will not apply to absolute path.

so I think we have to support it anyway. Otherwise I don’t think those people have a workaround

Okay, I'll fix it and remove the comment later 👍

Cool. Can you do the same for loading config? I think it’s okay if config is outside root directory. This will probably make it easier/simpler to use homebrew. And also add description to the root CLI parameter: it will not apply to absolute path.

Done, you can have a look again


PWN-8206 - added at sign to username

Jira Ticket
https://p2pvalidator.atlassian.net/browse/PWN-8206
Description of Work

added @ sign to username

We have special feature toggle for username domain. I think we need to connect it somehow with your method. How about a class UsernameFormatter or smth like that?

@gslevinkov check please again, I've added UsernameFormatter


add two swipe screen operation

Codecov Report

Merging #70 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   98.81%   98.81%
=======================================
  Files          21       21
  Lines         927      927
  Branches      153      153
=======================================
  Hits          916      916
  Misses         11       11

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 20a0650...26b3846. Read the comment docs.


Fix xtensa build error with choice LIBSUPCXX by default.

Summary
Due to our default modification of the C++ low-level library to LIBSUPCXX, some configurations that require LIBCXXABI have encountered errors https://github.com/apache/nuttx/pull/14471#issuecomment-2433322936 . Some configurations may not have been added to the CI, and similar issues may still arise. If you encounter the same problem, we hope you can manually select LIBCXXABI to give it a try.
Impact
The configuration that needs to use LIBCXXABI as the exception handling library.
Testing
ci test

@tmedicci please try the fix.

Thanks @cuiziwei1 , I'll test it internally and return when I get the results...


issue facing

Error running app. If this keeps happening, please contact support.

@saqlain51214 This was due to resource constraints on Streamlit Cloud. I have rebooted the app to temporarily resolve the issue. It’s working now, kindly check.


Add ImageMagic service to handle raw and bmp

I want to use Magic.NET for ARW, CR2 and BMP images
Jpegs and Pngs should be handled by Imageflow since it's fast

Resizing raw files on the fly - like I do with jpegs - doesn't work.
Who could've think about it rly.
It works, it's just really slow, too slow.
The only solution I can thing of is to create thumbnails on the fly when scanning the folder. I.e. we encounter a raw file - we transform it to 1920x1080 jpeg and put this jpeg to the application folder. Maybe we can even use smaller image, 800x600. These images should be used for previews, in the folder view, and for image view we can ask for a full raw -> jpeg conversion. Which is also slow, meh.
The obvious downside is that this folder will grow very big very fast. It's basically the copy of the gallery.

Current plan:

if folder contains jpg and raw with the same file name, then return jpg version; add icon/overlay on the preview that marks the image as "has raw version"
do not show folder in the folder explorer until its scan is complete
when scanning the folder, transform all raw image to 1024x768 jpegs and save them in the app folder
when the folder is opened, return pre-generated jpegs instead of raw files
add icon/overlay to mark the image as raw
when the image is opened in the popup there should be a way to open the full raw->jpeg version, without resize
on admin sidebar show how much space does the folder with pre-generaged jpegs take


Logging: Error trying to Create ClusterFlow or Flow

Installed Logging into a cluster and go to the Logging product.
Click on either ClusterFlows or Flows. Click the Create button - you get an error 'filtersYaml.substring(...).replaceAll is not a function'.
In 2.5.5 you got this message: "This resource contains a match configuration that the form editor does not support. Please use YAML edit." - which is slightly confusing, as there is no way to edit the YAML.
Click 'Create' here:

Error seen:

@nwmac are you still able to reproduce this? It seems to be working for me in the latest.

Thanks @codyrancher - turns out my Chrome on my linux box is version 81 (April 2020).
The code uses String.replaceAll, which was not available in Chrome until version 85 (Aug 2020)  - https://caniuse.com/mdn-javascript_builtins_string_replaceall
I've added a fix for this in this PR: #2336

Tested on Rancher v2.5.6-rc5 and Chrome 88, Brave 1.18.78, Safari 13.1.3
Steps:

Provision cluster
Navigate to Cluster Explorer
Deploy Logging App
Navigate to Logging > ClusterFlows, Click Create (observe no issue)
Navigate to Logging > Flow, Click Create (observe no issue)

This looks good from my tests
@nwmac can you confirm this works in your earlier version of Chrome?

@nwmac Seems like this can only be tested if you have an older version of Chrome. Do you still have that older version? Otherwise, we should close this.

@denise @ryansann I have verified that this is fixed for older Chrome (80).


Update Post “found-flying-termites-on-the-gold-coast”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
0b1c6c53be07e12fba76b7a39259870fe5c9abb0

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/65517424f119c200084e0633


missing module in pyproject.toml

Possibly module is missing in the pyproject.toml
Following the installation document for building the package and try to run the dlr simulation run -p localization  -l play_rate:=0.5 then I encountered some issue like ModuleNotFoundError: No module named 'pyquaternion' with python.
Also, to run the test for package, I needed nuscenes-devkit to pass the test with colcon test --event-handlers console_cohesion+ summary+ --packages-select driving_log_replayer.
So, I think we need to add nuscenes-devkit for package depedency.
If it was my environment issue, I'm sorry.

Maybe you didn't do rosdep install
If you have installed driving_log_replayer described in the following page, nuscenes-devkit is installed automatically.
https://github.com/tier4/driving_log_replayer/blob/develop/docs/quick_start/installation.en.md
The mechanism is that driving_log_replayer depends on perception_eval and perception_eval depends on nuscenes-devkit.
https://github.com/tier4/driving_log_replayer/blob/develop/driving_log_replayer/package.xml#L38
https://github.com/tier4/autoware_perception_evaluation/blob/develop/perception_eval/package.xml#L16

The packages listed in pyproject.tom are those needed to run the cli, and those needed to run the ros node are listed in exec_depend in package.xml


Why this behaviors /problem with responsive design

I don't have requested desktop site
Can someone turn off zoom
Use this snippet to stop zoom
 <meta name="viewport" content="height=device-height, width=device-width, initial-scale=1.0, minimum-scale=1.0, maximum-scale=1.0, user-scalable=no, target-densitydpi=device-dpi" />

And please someone work on responsive design

same as #135

@MrMalloc ok then please turn off zoom using that above code

The Bug will be solved soon. :)

Ok It will be solved soon. :)


[Product Page] Metadata Servlet

[ ] Write Servlet to call Coveo daily and create a metadata-products file in the Franklin folder
[ ] File would be in JSON format and contain the metadata information like what is pasted below

Microfuge 16 Microcentrifuges: Efficient, Fast, Safe, Compact | Danaher Life Sciences

@mhaack  - created this task for @davidzochowski  as we discussed.

@davidzochowski here is a sample metadata JSON with the format we need: https://main--danaher-ls-aem--hlxsites.hlx.live/metadata-products.json also available on Stage Author in https://author-p93411-e849602.adobeaemcloud.com/ui#/aem/assets.html/content/dam/danaher/franklin

@mhaack This is just to make sure I understand the process being proposed.
We want to create a json file within the DAM that contains all of the products currently within Coveo. This would act as a 'cache' and would remove the need to query Coveo for each Product page load like we currently do.
To do this, a job needs to be created within AEM that will fetch all of the products within Coveo and save them as a json file in the format you mentioned above. This job would run every 24 hrs.


Project is not open source without a license file

This repo is missing a license. Without a license, all code is copyright the author and may not be used by anyone else.
Please use something like http://choosealicense.com/ to decide what license to use. I recommend MIT or GPL.

Hey! Thanks for your interest in this project, I will update all repositories with a LICENSE file soon!

Done!


Fixing future compiler error "unaligned_references" (82523)

Fixing a future compile error, that is currently a warning. More info: https://github.com/rust-lang/rust/issues/82523
The  fix with curly-syntax is quite simple. It creates a new copy of the values, iff the are copy. Easy solution.

Thanks!


feat: multi entity time series heatmap

Auxiliar function that showcases the distribution of each entity's data across the time axis.
profile = ProfileReport()
profile.timeseries_heatmap(dataframe, "entity_column", "time_column")
# if the time column is not available the index will be used as the timestamp

Codecov Report

Merging #1030 (4293b7f) into develop (22d91f2) will decrease coverage by 0.05%.
The diff coverage is 85.96%.

@@             Coverage Diff             @@
##           develop    #1030      +/-   ##
===========================================
- Coverage    90.96%   90.90%   -0.06%
===========================================
  Files          173      173
  Lines         4868     4924      +56
===========================================
+ Hits          4428     4476      +48
- Misses         440      448       +8

Flag
Coverage Δ

py3.8-ubuntu-latest-pandas
90.90% <85.96%> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/pandas_profiling/profile_report.py
80.00% <74.19%> (-0.91%)
:arrow_down:

src/pandas_profiling/visualisation/plot.py
88.83% <100.00%> (+0.66%)
:arrow_up:

tests/unit/test_time_series.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Please specify the last version of Visual C++ Redistributable that works on Windows Vista

Under “Notes” in this page, it is specified the last version of Visual C++ Redistributable that works on Windows XP.
Recently, a friend of mine tried to install the latest version of Visual C++ Redistributable on his Windows Vista computer, that displays this error message -

It's not easy to find the version available for Windows Vista.

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

ID: 75e6432b-c6ec-e208-5f1c-bbc4fde9929b
Version Independent ID: 9cbb940e-cddd-f82f-e8bb-ee62b17af794
Content: Latest supported Visual C++ Redistributable downloads
Content Source: docs/windows/latest-supported-vc-redist.md
Product: visual-cpp
Technology: cpp-windows
GitHub Login: @MahmoudGSaleh
Microsoft Alias: msaleh

Officially, Visual Studio 2022 toolset (MSVC >= 14.30) does not support targeting Windows Vista
Technically, 14.31.30919 is the last VC_redist installer that is permitted on NT 6.0 (with SHA-2 support)
Practically, 14.32.31332 is the last version that works on NT 6.0 (with SHA-2 support)
for reference:
https://github.com/microsoft/STL/pull/2317

Closing as part of backlog grooming. We have accrued more issues than our resources can manage.
Criteria for closing include severity of the issue, how broadly applicable it is, whether the issue enough information is provide to act on the issue, etc.
If you think this issue should not have been closed, reopen it with a comment explaining why.


Feature/sample push notifications

Adds bindings for calling registerDeviceToken()
Adds methods for handling push notifications to the iOS app delegate.
Enabled remote notifications in plist.

@egorpushkin PTAL

LGTM


Fixes #493 : Add partial regex match assertion

Will review it shortly, thanks.

Thanks Pierre, merged into 2.x and will be available in master soon.


Add standard for js lint

We want to implement standard package for linting action of code

Add new script for package for lint, similar to "lint": "standard"
Add dev command to run app, like "dev": "gulp set && gulp"

Reference: https://www.npmjs.com/package/standard

Also, solve lint issues


Hang when checking compatility

Using 1.40.1
I can enable and disable mods fine but when I click "Check Compat" and click anywhere in that Mod Compatibility dialog WH3MM hangs ("not responding" in Windows). I can close the Mod Compatibility dialog but any other action causes a hang. Checking log files there are some errors thrown by fastlist, e.g.
[2024-01-09 20:42:42.871] [info]  Error: spawn K:\projects\whmm\node_modules\ps-list\vendor\fastlist-0.3.0-x64.exe ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:283:19)
    at onErrorNT (node:internal/child_process:478:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)

Some files that maybe relevant:
wh3mm.zip

So just gave it 20 minutes and the dialog populated. 12700KF, 32GB.
Maybe some kind of spinner for this dialog then?


All Live Share commands return errors

Product and Version [VSCode]: Version 1.25.1
OS Version [macOS]: 10.14.6 (Mojave)
Live Share Extension Version: 1.0.905
Target Platform or Language: Any language
Steps to Reproduce / Scenario:

Install live share extension

Reload vs code

No live share button appears

Check logs at vs code start up and see this error:

Try to run Live Share: Launcher Setup and get this error message "command 'liveshare.launcherSetup' not found" plus this error log

What I've tried:
Have uninstalled/reinstalled multiple times
Including removing live share from extensions directory the reinstalling
Updated computer/restarted computer
Any command I try to run responds with an error message.

In the end I fixed the issue by completely uninstalling vs code and reinstalling using the following instructions:
rm -rf $HOME/Library/Application\ Support/Code
rm -rf $HOME/.vscode
Then reinstall


updated po instances and created prod ids

@nassibnassar Would you mind looking at the conflict? I tried to do this with Natalya but we didn't get the option to resolve the conflict.

closes #346

Do you have a sense of what was changed in po_instance.sql?  Otherwise it will require a word by word comparison.

yes, we added wf_status, po_line_number, and changed created_by from json to user_users table.

From: Nassib Nassar @.>
Sent: Wednesday, March 24, 2021 2:49 PM
To: folio-org/folio-analytics @.>
Cc: Natalya Pikulik @.>; Author @.>
Subject: Re: [folio-org/folio-analytics] updated po instances and created prod ids (#390)
Do you have a sense of what was changed in po_instance.sql? Otherwise it will require a word by word comparison.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/folio-org/folio-analytics/pull/390#issuecomment-806071408, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO5MI7WL4KVTLILG3VNWQ5DTFIX2LANCNFSM4ZX2KC4A.

Thank you.  I also see pol_instance_hrid added.
I'll fix the typo in
CREATE INDEX ON folio_reporting.po_instance (pol_intance_id);
Has this been tested?

Also added po_lines.selector.

@nassibnassar I can test in sandbox and Natalya also tested it at Cornell and Christie at Chicago.

@natalyapik @jenmawe I'm unsure about this:
json_extract_path_text(po_purchase_orders.data, 'metadata', 'createdByUserId') AS created_by,

vs.
user_users.username AS created_by,

Which one would you like to keep?  Or both?


star-citizen: Can't download game - launcher version 2.0.0 (beta)

When trying to install the game with the latest launcher, I get the following error:

21:00:08.440 > "[Installer] - Installing Star Citizen LIVE 3.23.1-live.9179031 at C:\\Program Files\\Roberts Space Industries\\StarCitizen"
21:00:08.447 > "[Installer] - Starting download of base pack (SC LIVE 3.23.1-live.9179031) in C:\\Program Files\\Roberts Space Industries\\StarCitizen"
21:00:08.449 > "[Initial-Download] - Initial download start"
01ac:fixme:explorerframe:taskbar_list_SetProgressState iface<PHONE_NUMBER>642E30, hwnd<PHONE_NUMBER>01008E, flags 2 stub!
01ac:fixme:explorerframe:taskbar_list_SetProgressValue iface<PHONE_NUMBER>642E30, hwnd<PHONE_NUMBER>01008E, ullCompleted 1, ullTotal 64 stub!
01ac:fixme:winsock:setsockopt Unknown IPPROTO_TCP optname 0x00000003
21:00:08.476 > "[InitialDownload Error] - verification download files failed: UnknownRemoteResourceError: Unknown error"
21:00:08.479 > "[Initial-Download] - Verification header not found for initial download"
21:00:08.480 > [
 "[Initial-download] - createInitialDownload error",
 {
  "error": {
   "name": "VerificationHeaderError"
  }
 }
]
21:00:08.482 > "[Installer] - Error installing at C:\\Program Files\\Roberts Space Industries\\StarCitizen\\LIVE: VerificationHeaderError: [Initial-Download] - Verification heade
r not found for initial download"
21:00:08.484 > [
 "[Installer] - Error: ",
 {
  "error": {
   "name": "VerificationHeaderError"
  }
 }
]

I was getting a similar error with the 2.0 launcher when trying to install it in proton earlier, but mine included an issue about being unable to create the installation folders because the user hadn't given permission. I assumed that was due to a broken UAC query or something. In this case I don't see it, but I wanted to throw my info here in case it was helpful. The legacy launcher seems to be doing well...

@LovingMelody

2.0 Launcher requires wine 9.5 or newer, wine-ge is a much older and does not have the patches back ported., If  you wish to use it anyways you will need to override the wine package and edit your host file to block the installation of EAC or use a patched version of wine that has the EOS_USE_ANTICHEATCLIENTNULL patch included
1.X launcher isnt going away any time soon and currently the 2.X launcher (which is in beta) only causes performance issues and wont really serve a purpose till SQ42 comes out..

Thanks for the response!
After your response I went back to the 1.X launcher and everything works.
I mistakingly thought that the 2.X launcher replaced the 1.X launcher.
In the future there'll probably be an issue on enabling the 2.X launcher,
but I'm closing this issue, since I see no reason to maintain the 2.X launcher for now.


Add node name to StatisticsUpdated node events

Is your feature request related to a problem? Please describe.
The way I collect RX/TX metrics (liked dropped commands, etc) is to subscribe to the _EVENTS/<mqtt_name>/node/statistics_updated MQTT topic. I can then parse the JSON body and insert that data into a time-series database. However, each message to that topic only includes the node ID. This is pretty hard to quickly parse when viewing the metrics in something like Grafana (i.e. seeing node numbers instead of names).
Describe the solution you'd like
I'd like the Include Node Info setting to also insert a field or something into the payload of those messages that includes the node name (in addition to the ID). Maybe just another item in the array? or a field in the data object?
Additional context
I'm referring to this API: https://zwave-js.github.io/node-zwave-js/#/api/node?id=quotstatistics-updatedquot

I would submit a pull request, but I'm particularly strong with JavaScript, and I don't have the spare time to fully parse the moving parts of all the different classes and code structure.
If this is something easily possible, I'd love to see it implemented.

Awesome thank you! That was quick! I'll test it out once it hits the next release and report back.  Be aware that the PR that fixed this also aligned all node events to have the complete zwaveNode object as first element in mqtt payload data array

Understood. I'll adjust my metrics collection accordingly. Thanks!

Release is out now


Evaluation Form: Score Mapping

User Story
As a challenge manager, in order to accurately calculate score the evaluators will provide, I would like to provide score value/points mapping for non-numerical scores (for example likert scale scores: likely, neutral, not likely)
Acceptance criteria:

[ ] A challenge manager can enter numerical points to assign value to non-numerical score field options (dropdown, radio buttons, checkboxes)
[ ] Score values are required
[ ] The system validates if a numerical value was provided by a challenge manager
[ ] The system uses score values to calculate the total score, #71

Definition of Done
Doing (dev team)

[ ] Code complete
[ ] Code is organized appropriately
[ ] Any known trade offs are documented in the associated GH issue
[ ] Code is documented, modules, shared functions, etc.
[ ] Automated testing has been added or updated in response to changes in this PR
[ ] The feature is smoke tested to confirm it meets requirements
[ ] Database changes have been peer reviewed for index changes and performance bottlenecks
[ ] PR that changes or adds UI

[ ] include a screenshot of the WAVE report for the altered pages
[ ] Confirm changes were validated for mobile responsiveness

[ ] PR approved / Peer reviewed
[ ] Move card to testing column in the board

Testing (dev team)

[ ] Security scans passed
[ ] Automate accessibility tests passed
[ ] Build process and deployment is automated and repeatable
[ ] Feature toggles if appropriate
[ ] Deploy to staging

Staging

[ ] Accessibility tested (Marni)

[ ]  Keyboard navigation
[ ]  Focus confirmed
[ ] Color contrast compliance
[ ] Screen reader testing

[ ] Usability testing: mobile and desktop (Tracy or Marni)
[ ] Cross browser testing (tool to be determined) (Tracy or Marni)

[ ]  UI rendering is performant

[ ] AC review (Renata)
[ ] Deploy to production (production-like environment for eval capability) (dev team)
[ ] Move to production column in the board

Production

[ ] User and security documentation has been reviewed for necessary updates (Renata and Michelle)
[ ] PO / PM approved (Jarah or Renata)
[ ] AC is met and it works as expected (Jarah or Renata)
[ ] Move to done column in the board (Jarah or Renata)

Figma link (see CM Evaluation): https://www.figma.com/design/VfdBFG8ShVnOStdSNYy9fR/Challenge.Gov-Component-Library?node-id=909-27788&t=bN3FSfpATePzF43T-0

Score values are required
The system validates if a numerical value was provided by a challenge manager
aren't these the same thing?

aren't these the same thing?

@stepchud I reworded the second one, saying that the system will need to validate if the input is numerical. So they are not the same, one will require an input, the second one will validate that the input is correct type.

The user story is not longer needed and will be archived.


Translation interface description box size

The size of the description box is absurd. It should be bigger in both directions, while also not being hideous.

Maybe you could have a pop-out.

So, I wanted to make it resizable, but since it is contained in a table, that's impossible. I feel like simply increasing the width (TilesheetsForm::createFormRow('tile-translator', 'description', $description); -> TilesheetsForm::createFormRow('tile-translator', 'description', $description, 'text', 'style="width: 400px;\"');) is sort of a shitty solution, because it still sucks to use it when you have really long descriptions, like &9Mod: None (Vanilla Material)/&7Contains 0.111 Units of Redstone/Melt: 500 K - Boil: 1500 K - Weight: 80.290 kg. @retep998


Hash of zrok v0.4.23 release tarball has changed

👋 Hey!
When doing a build with

https://github.com/Homebrew/homebrew-core/pull/157782

it failed with error:
  ==> Downloading from https://codeload.github.com/openziti/zrok/tar.gz/refs/tags/v0.4.23 Downloaded to: /Users/brew/Library/Caches/Homebrew/downloads/1d5a0aa58eb7f9a54fad745f57d0f557d70cdc1004c6e6c67b0ee3e1c987cb15--zrok-0.4.23.tar.gz SHA256: 4ca9c52b8ad56498fe3cc73a65e0f6381f1d1afa2f9be2c569dc6f2e3fb810f9 Warning: Formula reports different sha256: 020a1875fa7a62bccbeff6c21fc9292a02c97283796dfe7b4820be0b6c6c7946
Meaning that the hash of v0.4.23 release tarball has changed since

https://github.com/Homebrew/homebrew-core/pull/160632

I opened the fix PR:

https://github.com/Homebrew/homebrew-core/pull/162351

But we need your confirmation that there was some intentional release tag manipulation (eg moving it) for some legitimate reason going on, and not a supply chain security incident.
Thanks for taking the time to look into this and possibly ensuring it does not happen in the future

Not sure what could have caused the issue that you're seeing. We have not manipulated any tags or changed the release process in any way for v0.4.23.

The homebrew release action is running against releases, not against tags. It's a separate stage of our build pipeline. Most of our release build machinery runs against a tag being created.
Nothing has changed with that process since at least a few releases before v0.4.23. Not sure what's going on here or how to fix it.
@camotts do you want to chime in here and take a look?

It doesn't get triggered manually... nobody is manually executing the homebrew release action. It just runs when the release is published.

@michaelquigley so far, the only thing I observed weird is the line encoding with some files, so I filed #564 to hopefully resolve this checksum issue.

Weird. So what exactly is going on with the homebrew stuff that is unhappy? This is the homebrew build process, yes? I would assume it's just downloading an archive from Github? How are we ending up in a situation where the checksum is mismatched?
I'll approve and merge the PR in a minute... seems very straightforward. But how exactly would changing the line endings on documentation fix this issue? Just trying to understand what's going on.

I'll approve and merge the PR in a minute... seems very straightforward. But how exactly would changing the line endings on documentation fix this issue? Just trying to understand what's going on.

line encoding affects the github source tarball files, that is why we have seen the source tarball file checksum changes, but we might need to rerun the pipeline to validate the line encoding changes as well.

github support case #2600624

What's going on? :point_up:

@chenrui333 @stefanb Getting ready to drop release v0.4.25... is there anything that I can do before this release drops to help with this issue?

@dehanj another idea, can we upload the source tarball as github asset into each release (in that way, it wont be changed at all)

@dehanj another idea, can we upload the source tarball as github asset into each release (in that way, it wont be changed at all)

For now, I think the issue is pretty much with the metadata and caused the tarball stability.

per github investigation, it is the same root cause as cog, which is this, https://github.com/python-versioneer/python-versioneer/issues/217

Performing the same steps:
brew install gzip
git clone git@github.com:openziti/zrok.git
git -c tar.tgz.command="$(brew --prefix gzip)/bin/gzip -cn" archive v0.4.23 --prefix=zrok-0.4.23/ -o ~/Desktop/test1.tgz
git reset --hard v0.4.23
git -c tar.tgz.command="$(brew --prefix gzip)/bin/gzip -cn" archive v0.4.23 --prefix=zrok-0.4.23/ -o ~/Desktop/test2.tgz
cd ~/desktop
╭─x@x-mbp ~/desktop
╰─➤  shasum -a 256 test1.tgz
4ca9c52b8ad56498fe3cc73a65e0f6381f1d1afa2f9be2c569dc6f2e3fb810f9  test1.tgz
╭─x@x ~/desktop
╰─➤  shasum -a 256 test2.tgz
020a1875fa7a62bccbeff6c21fc9292a02c97283796dfe7b4820be0b6c6c7946  test2.tgz

These checksums match those shared in the issue linked:
it failed with error:
==> Downloading from https://codeload.github.com/openziti/zrok/tar.gz/refs/tags/v0.4.23 Downloaded to: /Users/brew/Library/Caches/Homebrew/downloads/1d5a0aa58eb7f9a54fad745f57d0f557d70cdc1004c6e6c67b0ee3e1c987cb15--zrok-0.4.23.tar.gz SHA256: 4ca9c52b8ad56498fe3cc73a65e0f6381f1d1afa2f9be2c569dc6f2e3fb810f9 Warning: Formula reports different sha256: 020a1875fa7a62bccbeff6c21fc9292a02c97283796dfe7b4820be0b6c6c7946
After unpacking the tarballs:
╭─x@x-mbp ~/desktop
╰─➤  diff -rq ./zrok-0.4.23 ./zrok-0.4.23\ 2/
Files ./zrok-0.4.23/sdk/python/sdk/zrok/zrok/_version.py and ./zrok-0.4.23 2/sdk/python/sdk/zrok/zrok/_version.py differ
╭─x@x-mbp ~/desktop
╰─➤  diff ./zrok-0.4.23/sdk/python/sdk/zrok/zrok/_version.py ./zrok-0.4.23\ 2/sdk/python/sdk/zrok/zrok/_version.py
29c29
<     git_refnames = " (tag: v0.4.23)"

git_refnames = " (HEAD -> main, tag: v0.4.23)"

Gonna close this issue for now.

Thank you for the help with this! :+1:


Uploading Illumina data set

Dears,
Probably I am writing about something that has been discussed in the past. I have checked all of the relevant topics but could not find any useful information that might help me solving the issue I am facing at the moment.
I am trying to analyse data obtain from Illumina microarray beatchip. The accessions are annotated with ILMN_ prefix for each gene with additional numeric code (proprietary annotation) which unfortunately are not recognized by PathVisio when I am trying to import the file. I am getting a message that all of lines are unrecognized (ecceptions - while reading the data).
The gene list is Hs_Derby_Ensemble downloaded from the PathVisio homepage. I am using System Code Il - relevent to the Illumina chip. The rest of the columns are A, B and Fold change.
Does anyone have any idea, why I cannot import my data successfully?
Thank you for your help in advance.

Identifiers are only recognized when they are in the identifier mapping database. Can you provide a couple of example identifiers?

Dear Tina,
Thank you for your swift reply.
The Illumina`s microarray annotation could be found on the following link:
https://support.illumina.com/array/array_kits/humanht-12_v4_expression_beadchip_kit/downloads.html
I am also uploading a sample file from my data.
I hope you find this information useful for you.
Kind regards,
Petar

Dear Tina,
For some reason I was not able to upload my example data in Github so I am
sending it via e-mail.
Regards,
Petar
2017-12-05 16:20 GMT+02:00 Martina Summer-Kutmon<EMAIL_ADDRESS>
Identifiers are only recognized when they are in the identifier mapping
database. Can you provide a couple of example identifiers?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PathVisio/pathvisio/issues/60#issuecomment-349317832,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agrdy1loc5HFeVVjLjKqzSRe1tJIH6_Aks5s9VFKgaJpZM4Q160G
.

Dear Tina,
I have found out that after saving the file in text (tab delimeted) file I still have to upload it using the comma as separator. In tutorial 2 the example file should be uploaded using 'tab" separator. In this regard I think the culprit might be in some hidden characters within the excel file. Have you any idea how can I make a clean text (tab delimited) file for PathVisio.
Thank you in advance.
Kind regards,
Petar

I haven't received the example data file. If you could send me the file then I can check how to import it. You can import comma-separated files - just select comma as the separator in the import dialog. Also check in the preview if the data looks like you would expect!

Dear Tina,
Thank you for your e-mail.
I am importing the document using the "comma" separator and the data does
really looks as it is in the xls, cvs, text (tab delimeted) file.
Unfortunately whne I click 'Next' I see that all of the entries are
parcelled as 'ecceptions' and at the end I have all of the lines as
unrecognized. I assume the culprit might be some hidden characters within
the data file (such as commas) which are blocking the software from
obtaining the right identifier.
I am attaching the example sheet one more time. Please, let me know if you
can open it.
Thank you for you support.
Kind regards,
Petar
2017-12-07 10:31 GMT+02:00 Martina Summer-Kutmon<EMAIL_ADDRESS>
I haven't received the example data file. If you could send me the file
then I can check how to import it. You can import comma-separated files -
just select comma as the separator in the import dialog. Also check in the
preview if the data looks like you would expect!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PathVisio/pathvisio/issues/60#issuecomment-349897976,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agrdy-xMCqshhiMv6_hssQXMb6pOYyY8ks5s96JsgaJpZM4Q160G
.

I checked the annotation file now and verified that as an example for gene "ST8SIA3" (Entrez Gene identifier: 51046), there are the correct Illumina identifiers in the mapping database (ILMN_1855910) which means they should be recognized. The file attachment on github doesn't seem to work. Can you send me the example file to my email directly?

See solution: https://groups.google.com/forum/#!topic/wikipathways-discuss/HZo62y10SUw


feat: logdialogeditor

Separate LogDialog editing from the LogDialog page.   This allows log editing to take place on other pages (in prep from Review page).  Also, with some additional work will help us unify editing with the TrainDialog page.

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


NTR: brassica vegetable food product

NTR: brassica vegetable food product
Parent term: vegetable food product (FOODON_00002142)
Child terms:

Brussels sprout vegetable food product (FOODON_00001670)
cabbage vegetable food product (FOODON_00001677)
broccoli vegetable food product (FOODON_00001666)
cauliflower vegetable food product (FOODON_00001703)
rapeseed vegetable food product (FOODON_00002187) (note: rapeseed meal (FOODON_03310043) is child under this term, but maybe move rapeseed oil (FOODON_03310482) as also a child under rapeseed vegetable food product.)

There may be additional plants that fall under this category as well.

We have included this now in our foodon-edit.owl draft.

loaded up to live!


Remove block_on in poll_x function

From my understanding, calling block_on inside a poll_x function is a bad practice. However, I'm not sure how else to resolve a call to an async function within a poll_x function.
https://github.com/sachanganesh/eventador-rs/blob/57be6ef5cb65a832bfab65a4b2785283316d650c/src/futures/publisher.rs#L53-L54

Resolved by 70e50c5561f18354408a9f54f8ae28ab9c869f4b, through the use of the async-streams crate as recommended by this post.


Update Github Actions workflow to properly parse dotnet version

Fixes #207 macOS CI Integration tests running .NET 5 instead of 3.1
Additionally, renamed workflow file/name to more conventionally represent its purpose vs. the repo it is in. If we ever add more actions, ie publish, it'll make more sense in terms of naming.

[x] Related GitHub issue(s) linked in PR description
[x] Destination branch merged, built and tested with your changes
[x] Code formatted and follows best practices and patterns
[x] Code builds cleanly (no additional warnings or errors)
[x] Manually tested

https://github.com/brandongregoryscott/AndcultureCode.Cli/runs/2965839307

[x] Automated tests are passing
[x] No decreases in automated test coverage
[x] Documentation updated (readme, docs, comments, etc.)
[-] Localization: No hard-coded error messages in code files (minimally in string constants)

@all-contributors add @dylanjustice for review

@all-contributors add @jstevenson for review