Using json-schema-faker with produces/consumes check

Use json schema faker if produces and consumes contain application/json

This PR also release 2.1.0


Added dependencies

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Type of change.

[x] FIX - a PR of this type patches a bug.
[ ] FEATURE - a PR of this type introduces a new feature.
[ ] BUILD - a PR of this type introduces build changes.
[ ] CI - a PR of this type introduces CI changes.
[ ] DOCS - a PR of this type introduces DOCS improvement.
[ ] STYLE - a PR of this type introduces style changes.
[ ] REFACTOR - a PR of this type introduces refactoring.
[ ] PERFORMANCE - a PR of this type introduces performance changes.
[ ] TEST - a PR of this type adds more tests.
[ ] CHORE - a PR introduces other changes than the specified above.

We don't need to merge dev with main in this PR just so we can update dependencies in main, we will do it during the release process.
Can you close this PR and merge the other dependabot PRs related to WalletConnect deps?


ccm release notes

May release notes

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


After upgrade to 1.12.2, extension not working any more

Do you read the FAQ?

[x] Yes.
[ ] I am going to read now.

Describe the bug
After update VSCode to version 1.33.1 disappeared icon of the extension, and SFTP:Config command leads to error sftp.config command not found.
Desktop (please complete the following information):

OS: [Win]
VSCode Version [e.g. 1.33.1]
Extension Version [e.g. 1.12.2]

Extension Logs from Startup - required
Logs are empty.

Extension Logs from Startup - required

Open User Settings.

On Windows/Linux - File > Preferences > Settings
On macOS - Code > Preferences > Settings

Set sftp.debug to true and reload vscode.

Reproduce the problem, get the logs from View > Output > sftp.

Extension Logs from Startup - required

Open User Settings.

On Windows/Linux - File > Preferences > Settings
On macOS - Code > Preferences > Settings

Set sftp.debug to true and reload vscode.

Reproduce the problem, get the logs from View > Output > sftp.

I've reinstalled VSCode and removed folder .code from my user folder and plugin began to work.


Request for Docker images in "amd64" architecture

Hi,
I see that the new docker images are built only for arm64 architecture. However, the old oscelot images supported amd64 architecture. If possible, can you please add amd64 architecture images to the new docker repo? I understand that I can build it myself and I will be doing that for now. However, having both architecture images can help a wider audience. I can issue a PR if you can direct me to the build pipeline.
Thanks,
Chinmaya

Closing this issue as the new 1.2.9 images are amd64 now.


Update update-product-form.tsx

fixing auto fill from browser to prevent error validiation

Can you fix the type error?

@sadmann7  done

Thanks


Renames env script to avoid possible POSIX env shadowing

Got an issue with other shell scripts, turns out that when one prepends $PATH by activemq/bin, env is shadowed by ActiveMQ's env.
That means that script that want to run commands in customized environment will simple don't be executed like :
env A=aaa B=bbb mv ...

Worse such lines won't fail, because this will return the 0 exit status. The workaround is simple, i.e. append in %PATH instead of prepend. But a proper fix is needed. This PR proposes to rename the shell script from env to activemq-env.

There's other executable that may be renamed, like diag, I can extends this PR if this makes sense.

Another matter why this PR should be accepted, env is actually part of the POSIX specification : http://pubs.opengroup.org/onlinepubs/9699919799/utilities/env.html

While I agree that env should not be shadowed, you probably don't need it. Your example works fine without env:
A=aaa B=bbb mv ...

The env command is more useful when you need :

to print your shell script/prompt environment (env)
to run a command with a clean environment (env -i A=aaa B=bbb mv ...)
an absolute path for a command, like in scripts shebangs (#!/usr/bin/env python)

@Dridi The problem is it's a POSIX command. And any script ha a right rely on that command. It's even about scripts I own, it actually broke scripts written by others. And they shouldn't have to rewrote them because of this.

@bric3 As I said, I agree with you. I highlighted the fact that you don't need env in the example you give, hoping that would help you until it's fixed in activemq.

@Dridi Yeah sure, but still

Does this happen because the file "env" has exec permissions and the activemq bin folder is part of the PATH variable?

@scoopex Yes, just like any developer add path/to/maven/bin to $PATH

If "activemmq/bin/env" is not an executable, you should not have the problem.
("env" is just a configuration for the initscript - so it might be better to move it to the "conf" folder)

If "activemmq/bin/env" is not an executable, you should not have the problem.

It is executable otherwise I wouldn't have encountered the issue, agreed that is another solution, but with that solution the problem can come back in the future, any developer of activemq may want to chmod +x the bin folder.

Can one of the admins verify this patch?

Superseeded by #1074 1074


chore: Update actions to latest available versions

What?
The current GitHub Actions workflow  in this repository is still using actions/checkout@v2, which is still running on Node v12 runtime. Due to this we are seeing deprecation warning in every action run.
GitHub Actions: All Actions will run on Node16 instead of Node12 by default
Solutions
Update actions/checkout@v2 to the latest version, which uses Node v16 as their runtime. This update is necessary to avoid actions failure in the future when GitHub Actions begins enforcing the use of Node16, replacing Node12.

Thanks!


Implement multi docs

[x] Add workflow to generate doc
[x] Add new dropdown widget in navbar to select doc version

Why is a new workflow needed?  We already have build.yml and release.yml.  BTW release.yml should have been modified to deploy.

There are at least 2 ways to build versioned documentation:

Static: available versions are hardcoded when building docs, which means that an older documentation can provide links to master and previous releases, but not future ones
Dynamic: all available versions are retrieved by clients.  This comes at a cost, Github limits the number of API requests, so the dropdown menu may be unavailable when displaying lots of pages.

Your commit message for 5ab99ab is way too terse. You should describe the different options, and why you implemented it that way.

Will improve commit description.

Why is a new workflow needed? We already have build.yml and release.yml. BTW release.yml should have been modified to not deploy.

docs.yml has been removed. Docs is generated through build.ymlwhen on master or through release.yml workflows.


Empty links list

Hello,
sometimes method getDownloadLinks() returns empty list, for example for this video:
https://www.youtube.com/watch?v=dQw4w9WgXcQ
Regards.

Thats because YouTube changes for the most Videos the Signature Cipher.
find in YouTubeDownload.php line 138
$cipher = isset($item['cipher']) ? $item['cipher'] : '';
change it to
if(isset($item['cipher']) || isset($item['signatureCipher']))
{
    $cipher = ($item['signatureCipher']) ? $item['signatureCipher'] : $item['cipher'];
}

Fixed in this release:
https://github.com/Athlon1600/youtube-downloader/releases/tag/v2.0.7
Thanks to rusty nail


feat: implement multiplication crusade trends

Description
Implements screens so that users can view their multiplication trends after filling their forms

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
This was tested on my local machine

Screenshots/Videos (if appropriate):
Please pardon my bad internet😞

https://www.loom.com/share/0eee9409dbc7496985cdda5e8d543650

@ahadziii Left comments on your loom.


Use mode argument in make-new-file

An attempt to address #98.

I'm not sure, mode is means the file open flags, mode normally refers to the permission bits, as far as I recall

I'm just going by the code that was mostly there already (I think it got put there in https://github.com/janet-lang/spork/commit/ae9db3d5c953d54a0cca74f9352db160e749fce9#diff-addba8464816d2a707000ee1f01e77510dc484a709013f5b62979b3834ef8f11R76) as well as the docs:
    (file/open path &opt mode)

    Open a file. `path` is an absolute or relative path, and `mode` is
    a set of flags indicating the mode to open the file in. `mode` is a
    keyword where each character represents a flag.

Or may be I'm missing something?

Oh, I stand corrected, at least in the context of Janet file handling, mode seems to mean exactly what you suggest.


Rewrite/document standard folders for ALB and variables names

At this moment we only use as custom directory that needs documentation the /var/log/application_load_balancer/ for logs. But short name "alb" seems not have issues with other software (at least not for Debian distros) and also is shorter.
OpenResty store all data on /usr/local/openresty/, even configuration files from original NGinx it uses /usr/local/openresty/nginx/ to avoid conflict. I guess that we should do the same.
For now, we're need at least places for logs. But we will really need soon some places to put at least some debuggin files (or hello world pages). And also to allow some shorter configuration options (and avoiding users specifying everything) we would need in addition so logging files, also default place to, for example, put files for static content based on app_uid.
Also, this structure have to somewhat works fine with the ap-application-load-balancer-extras

We will also change variable names for shorter ones.
Also "alb" or at least "alb-" is nearly not used for any software at all. That's makes things easier, even if this would no be some registered trademark

ALB have (without the apps) already lots of places to find about. So even without looking for manuals, this new approach should help someone find where things are stored by just cd /opt/alb.
Current state:

Still need some changes from old approach. The use of symlinks, even if a bit redundant, is intentional if have to debug production very likely to be short on time. This approach also could reduce need to have some graphical or web gui to check all logs

Most of the work from this issue was done at release v0.4.5-beta.


Automatically use query parameters from FormRequest rule

We're using laravels Request classes to set up the query parameter.
A while ago the generator extracted the query parameter automatically from injected Request rules.
Since this isn't working anymore, has it been removed?
I also wasn't able to find anything according to this in the documentation. Do I have to set all query parameter automatically now? If so why has the automatic extraction been removed? Will it come back?
Eg.:
/**
     * Report an image
     *
     * @response { response }
     *
     * @param Image $image
     * @return \Illuminate\Http\JsonResponse
     */
    public function image(Image $image, ReportRequest $request) <- extract from ReportRequest rules?
    {
        $report = $this->reportInterface->store($image, $request);

        return $this->sendResponse(__('report.image_reported'));
    }

Do you mean the validation rules? If so, yes, that has been removed since version 3.0. See the docs.

Is there any way to use query parameters from FormRequest rules?

FormRequest support is back in Scribe.😅


mkp224o: fix typo in amd64-64-24k version

Description of changes
I noticed there was a typo in the mkp224o executable names. This fixes that typo.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 187199 run on x86_64-linux 1

1 package built:

mkp224o


num_workers=0 in ImageNet DataLoader

This is more of a question than an issue.
The number of workers for torch DataLoader is set to zero for ImageNet during evaluation (on train, valid and test splits). However, this makes evaluation extremely slow in my setting.
Maybe I am missing something, hence I would like to ask:

Why is the number of workers set to zero during evaluation?
Are we allowed to change the number of workers for evaluation to be 4 when self-reporting results?
Under the rules, model evaluations should be untimed and not contribute to the submissions score, so this shouldn't influence the final score. Is this correct?

Here is the relevant code:
https://github.com/mlcommons/algorithmic-efficiency/blob/11b11ec8160b0ca5b1503d85125420247fc58d16/algorithmic_efficiency/workloads/imagenet_resnet/imagenet_pytorch/workload.py#L129
Notice that for other datasets like librispeech, the number of workers is always 4:
https://github.com/mlcommons/algorithmic-efficiency/blob/11b11ec8160b0ca5b1503d85125420247fc58d16/algorithmic_efficiency/workloads/librispeech_conformer/librispeech_pytorch/workload.py#L176
Why this discrepancy?
Thanks in advance!

@pomonam do you have any context around this?

Sorry for the late reply! Yes, changing num_workers should not affect the final results because the evaluation time is not measured. Please feel free to change these numbers when running your experiments.

@runame added a flag to control the num_workers for some datasets during eval.
Hope that helps!

Thank you, @runame!


Variable Value Query on Hover Broken

Variable query on hover has been broken since November 2021 upon a refactor that added hover query to linear view. The result was the ability to query wrong values from linear view, and graph view for all SSA forms. Value query was also enabled for HLIL which also returned bad values.

Fixed in 3.0.3216-dev.
Note: Query in HLIL has not been added yet.


Support 'defaults' with [action] syntax for attribute routes

We built a lovely feature to do token replacement on attribute route templates. This will end up generating routes like the following:
[Route("[controller]/[action]/{id?}")]
public class HomeController
{
    // Equivalent to [Route("Home/Index/{id?}")]
    public IActionResult Index()
    {
    }

    // Equivalent to [Route("Home/About/{id?}")]
    public IActionResult About()
    {
    }
}

The suggestion is that we allow 'defaults' in these tokens like this:
[Route("[controller]/[action=Index]/{id?}")]
public class HomeController
{
    // Equivalent to [Route("Home")]
    // Equivalent to [Route("Home/Index/{id?}")]
    public IActionResult Index()
    {
    }

    // Equivalent to [Route("Home/About/{id?}")]
    public IActionResult About()
    {
    }
}

Very nice. So after that no need to register for default routes in startup.cs I guess!!

@NeelBhatt - these are alternatives, and many people prefer one or the other for their own reasons :+1:
Using attribute routes gives you more precise controller over the URL space, but is often more verbose and usually means putting attributes in your controller classes which some people don't like for code-organization reasons.
Using conventional routes (in startup code) is terse and broad and is the way most old school MVC users are used to doing it.
This change is a suggestion to bring some of the terseness of the 'conventional route' way for a pretty common usage pattern.

Okay I got your point and you explained it very nicely :+1:  I have used both previously and I prefer attribute routing as per my experience. But as you said, it totally depends on people's preference.

This doesn't look quite right to me. An entire segment is never omitted from the route. I think that if anything, the attribute route behavior for defaults should be:
[Route("[controller]/[action=Index]/{id?}")]
public class HomeController
{
    // Equivalent to [Route("Home")] - if the "action" segment is omitted, all subsequent segments must be omitted
    // Equivalent to [Route("Home/Index/{id?}")]
    public IActionResult Index()
    {
    }

    // Equivalent to [Route("Home/About/{id?}")]
    public IActionResult About()
    {
    }
}

However, I think that though this behavior would make sense, it's perhaps not super useful because you can achieve 99% of that by declaring this attribute route:
[Route("[controller]/{action=Index}/{id?}")] // 'action' is a regular parameter
public class HomeController
{
    ...
}

The only thing that's different about this compared to my previous example is that it would allow any action value, so you'd have to constrain it to Index|About|Contact|....
But, having said that, I'm not sure how useful this feature is...

it's perhaps not super useful because you can achieve 99% of that by declaring this attribute route

This is explicitly not allowed. You can't use any action-selection tokens ({contoller}, {action}) as parameters in an attribute route.

Ah, well, then nix that :smile:
But we still have different ideas about what the behavior should be regarding omitted sections. Let's chat tomorrow if it's not clear.

But we still have different ideas about what the behavior should be regarding omitted sections. Let's chat tomorrow if it's not clear.

That was a mistake, I corrected it in my sample.

Oh, great, because @danroth27 and I were freaking out about that :smile: Well, @danroth27 was being rather patient and I was freaking out! :laughing: Not only that, but all the rest of my diatribe no longer makes any sense :trollface:
So, yeah, now that we all agree about what the feature even is, it's certainly an interesting one.

You should visit Arizona. It's relaxing. :cactus: :cactus: :cactus: :cactus: :cactus: :cactus: :cactus: :cactus:

That sounds like a thorny situation.

This is funny! :) The only real mistake is the one from which we learn nothing. ;)

Moving to backlog.

Just a reminder - I still like this idea.

It's been about 11 months since I last bumped this thread. This item is elegant and cool and I think we should do it.

I still like this idea :)

+1

@rynowak, will this be a good fit for 2.2?

IMO this is a p3 for dispatcher. If we get an chance to do it, then that would be awesome - but I can't imagine us cutting other work to make this fit.

@JamesNK, seems like another issue to be handled as part of the dispatcher work. Will you create a new issue in the routing repo, or just move this one?


ATK-Feature request: provide "simple" format routine for whole disks

suggestion: implementation of a simple format routine to verify disk-writes and detect broken floppy-heads.
open points:

lowlevel format MFM format? or sync'ed 4489 ?
write of "all zero", "all 1", patterns?, real AmigaDOS-format structure

The existing write test writes a whole track to each head. Is that insufficient?

i know, but it will only write to the last track.. so when you want to "test" the seeking of heads and track-steps, then you'll need a formated disk to check whether the sector-ids match the addressed track or not.
of course, if the drive is misaligned, then this write would also be misaligned... so the format procedure should happen to a "known" good drive.. then the disk will be swapped and the read-test can confirm your drive-under-test is working correctly.
with this, you wont need to step out of ATK to format the disk.
its just the procedure everyone does, when testing disk-drives: you use a formated disk, read it back, format the disk again with the drive and read it back on the "known-good"... currently i'm using Xcopy for this...

to support you, i can maybe enhance your write-test procedure on my own and commit a pull-request. ?

I don't know whether I care to support it. You just need any known good Amiga disk eg professionally mastered, and do read test on that? If you want to make your own from a known good drive you can do it in workbench. It's not a test.

but how can you test a drive, without knowing that the data on the media is correctly written in the way the ATK read-test expects it? e.g. using a PC formated disk will lead to lots of errors even on a 100% perfectly working drive, because ATK expects a certain (=AmigaDOS) format.
sorry.. but this argument is a bit weak; for testing memory you are also implemented "write" routines... not only read-tests... same situation; you can state that writing to a memory-block can be done with some non-ATK tool and inside ATK you would just read back whether the memory is containing the right data. but what if the outside-tool writes 1s instead of zeros, as you expect it?
okay.. lets put it that way; writing and reading sector-by-sector is actually "testing the media" inside a floppy drive.. so this would cover the "test"-requirement... and why not write a correctly sync'ed structure to the disk which would then be able to pass the head-alignment test as well? could you support this, maybe?

ATK test expects an AmigaDOS disk. If you run ATK from disk you already have a suitable disk (the ATK disk). If you run ATK via HDD and Workbench, you can format a disk in Workbench. Most users of ATK will in fact have a pile of AmigaDOS disks.


Provide foundation packages via PluginSDK

⚠ This PR depends on changes that will get introduced with #1362

Adjust the plugin loader and module configuration to provide the respective packages via the PluginSDK interface. This a temporary solution which will be replaced once we have a proper process to build and integrate packages.

mesosphere-ci retest this please

This PR needs to only integrate the mount package, as we're not going to ship the auth package.

@nLight We can close this PR once https://github.com/dcos/dcos-ui/pull/1413 is merged. Correct?


Add more v1 API examples

:+1: I feel like we may need a TOC at some point.

Agreed re:TOC.  I like what some repos have done with a bulleted list of
features, too.  And there's also linking to alternatives that have
different strengths.  All TODO.
On Wed, Jun 1, 2016, 23:01 Jesse Szwedko<EMAIL_ADDRESS>wrote:

👍 I feel like we may need a TOC at some point.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/urfave/cli/pull/444#issuecomment-223183408, or mute
the thread
https://github.com/notifications/unsubscribe/AACwV7XUUz13pk4GEhQ_FOpdUxVxf-wlks5qHkd8gaJpZM4IryfC
.


split out WebKit and ch.boye stuff into standalone libs

The new model for the structure of NetCipher is to have sub-libraries for each chunk of code that relies on an external library.  These already exist:

netcipher-okhttp3
netcipher-volley
netcipher-httpclient

This pull request splits out these from the old netcipher 1.x library:

netcipher-webkit (needs useLibrary 'org.apache.http.legacy')
netcipher-chboye (needs ch.boye jar and useLibrary 'org.apache.http.legacy')

merging despite the build failure, since it is caused by the emulator crashing :-(


[otl libaaplus forge] Fix build issues

Fix build issue:

otl/libaaplus: Update hash
forge: fix cmake targets path on Linux.

otl and forge have no features, libaaplus feature 'tools' test pass on x86-windows x64-windows and linux.
Note: ogre failed due to missing ‘libxaw7-dev installation on test machine, it passed after install it locally.

/azp run

/azp run

/azp run

/azp run

/azp run


When running the dbt-meshify connect command, I get an error if I haven't already run dbt deps in all of my projects

09:35:10 | INFO | Executing dbt parse...
09:35:11 | INFO | Generating catalog with dbt docs generate...
09:35:24 | INFO | Executing dbt parse...
16:35:24  Encountered an error:
Compilation Error
  dbt found 2 package(s) specified in packages.yml, but only 0 package(s) installed in dbt_packages. Run "dbt deps" to install package dependencies.
Traceback (most recent call last):
  File "/Users/gracegoheen/.venv/meshify-current/bin/dbt-meshify", line 6, in <module>
    sys.exit(cli())
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/gracegoheen/dev/dbt-meshify/dbt_meshify/main.py", line 142, in connect
    all_dbt_projects = [
  File "/Users/gracegoheen/dev/dbt-meshify/dbt_meshify/main.py", line 143, in <listcomp>
    DbtProject.from_directory(project_path, read_catalog)
  File "/Users/gracegoheen/dev/dbt-meshify/dbt_meshify/dbt_projects.py", line 284, in from_directory
    manifest=dbt.parse(directory),
  File "/Users/gracegoheen/dev/dbt-meshify/dbt_meshify/dbt.py", line 30, in parse
    return self.invoke(directory, ["--quiet", "parse"])
  File "/Users/gracegoheen/dev/dbt-meshify/dbt_meshify/dbt.py", line 25, in invoke
    raise result.exception
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/cli/requires.py", line 87, in wrapper
    result, success = func(*args, **kwargs)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/cli/requires.py", line 72, in wrapper
    return func(*args, **kwargs)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/cli/requires.py", line 143, in wrapper
    return func(*args, **kwargs)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/cli/requires.py", line 172, in wrapper
    return func(*args, **kwargs)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/cli/requires.py", line 219, in wrapper
    return func(*args, **kwargs)
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/cli/requires.py", line 246, in wrapper
    manifest = ManifestLoader.get_full_manifest(
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/parser/manifest.py", line 307, in get_full_manifest
    projects = config.load_dependencies()
  File "/Users/gracegoheen/.venv/meshify-current/lib/python3.10/site-packages/dbt/config/runtime.py", line 366, in load_dependencies
    raise UninstalledPackagesFoundError(
dbt.exceptions.UninstalledPackagesFoundError: Compilation Error
  dbt found 2 package(s) specified in packages.yml, but only 0 package(s) installed in dbt_packages. Run "dbt deps" to install package dependencies.

I think this should be a matter of either detecting missing deps or catching the exception, running deps, and then moving on with the original command. Assigning to myself.

@nicholasyager I am bored waiting for a delayed flight! i will take this lil guy!

Do it!
On Sat, Sep 16, 2023, 11:30 dave-connors-3 @.***> wrote:

@nicholasyager https://github.com/nicholasyager I am bored waiting for
a delayed flight! i will take this lil guy!
—
Reply to this email directly, view it on GitHub
https://github.com/dbt-labs/dbt-meshify/issues/156#issuecomment-1722254168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAY6GSUQHZOQLIV3VQLM6DDX2XA27ANCNFSM6AAAAAA4DHYYNU
.
You are receiving this because you were mentioned.Message ID:
@.***>


GtfsFrequency - StartTime and  EndTime regex is wrong

The regex
 ^([0][0-9]|1[0-9]|4[0-7]):[0-5][0-9]:[0-5][0-9]$

in startTime and endTime doesn't match strings like: 20:00:00
A good Regex could be:
^([0-3][0-9]|4[0-7]):[0-5][0-9]:[0-5][0-9]

matching times larger than 24:00:00 (as expected by the GTFS Specification).
At the same time this avoids frequencies larger than 48 hours.

@emastrosimone an easy fix, could you provide a contribution?

fixed


[BUG] Can not add block disk on longhorn node

Describe the bug
Can not add block disk to node on longhorn master-head, this symptom did not observe in v1.6.x and v1.5.x branches
instance-manager log :
[2024-04-09 03:58:03.390455] bdev.c:8020:bdev_open_ext: *NOTICE*: Currently unable to find bdev with name: disk-1
[longhorn-instance-manager] time="2024-04-09T03:58:03Z" level=error msg="Failed to get disk info" func=spdk.svcDiskGet.func1 file="disk.go:130" diskDriver= diskName=disk-1 diskPath=/dev/loop3 error="rpc error: code = NotFound desc = cannot find disk bdev with name disk-1"
[longhorn-instance-manager] time="2024-04-09T03:58:03Z" level=info msg="Disk Server: Creating disk" func="disk.(*Server).DiskCreate" file="disk.go:121" blockSize=4096 diskDriver= diskName=disk-1 diskPath=/dev/loop3 diskType=block
[longhorn-instance-manager] time="2024-04-09T03:58:03Z" level=info msg="Creating disk" func=spdk.svcDiskCreate file="disk.go:45" blockSize=4096 diskDriver= diskName=disk-1 diskPath=/dev/loop3 diskUUID=
[longhorn-instance-manager] time="2024-04-09T03:58:03Z" level=error msg="Failed to create disk" func=spdk.svcDiskCreate.func1 file="disk.go:48" blockSize=4096 diskDriver= diskName=disk-1 diskPath=/dev/loop3 diskUUID= error="failed to get disk driver for disk disk-1: unsupported disk driver  for disk path /dev/loop3"

Not sure if this is due to some reviewing PRs, create this ticket for tracking.
To Reproduce
Add block disk to Longhorn master, the disk stuck at unschedulable
Expected behavior
Add block disk to Longhorn master success
Support bundle for troubleshooting
supportbundle_7dd7a80c-0211-4fa9-8bd2-870722164cec_2024-04-09T04-01-20Z.zip
Environment

Longhorn version: master

Additional context
N/A

The PR in longhorn-manager is still under review.
https://github.com/longhorn/longhorn-manager/pull/2642

The PR in longhorn-manager is still under review. longhorn/longhorn-manager#2642

The PR has been merged.

Verified pass on longhorn master(longhorn-manager 09633d), block disk add success and create workload with v2 volume success.


[WIP] Use React Router NavLink in menu

Fixes #1487

We don't have any documentation yet for the MenuItemLink. Moreover, we need to document the CSS API for our components which uses classes.

The use of secondary color is ugly IMHO. Material-ui uses an inverse background, I think it's better


CSS not transpiled

Ionic version:  (check one with "x")
[X] 2.2.11
I'm submitting a ...  (check one with "x")
[X] bug report
Current behavior:
Every time I run ionic serve, the interface do not display correctly:

-Title align to left.

button padding not correct.

but If I refresh the page, it will appear correctly. I suspect the CSS is not correctly transpiled.
Expected behavior:
The correct layout is like this:

title in the center
button padding correctly.

Steps to reproduce:
run ionic serve for the first time, it will display wrongly. Then try press r in CLI a few times, it will turn out correctly.
Other information:
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.5.0
Gulp version:  CLI version 1.2.2
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-rc.5
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.0.0-beta.20
ios-deploy version: 1.9.1
ios-sim version: Not installed
OS: Mac OS X El Capitan
Node Version: v7.4.0
Xcode version: Xcode 8.2.1 Build version 8C1002

I suspect you are first viewing android, then, after every refresh you see iOS. (because title on left is android, title centered is iOS)
Please try to Ionic serve --lab to confirm or refute my suspicion.

@sooonism Yeah, as AmitMY stated, the first screenshot is Material Design and the second is iOS. By default the browser will use Material Design. So when you first launch the app if you turn on device mode in the browser it will still show the Material Design UI until you refresh, at which point it will use the device's mode.
I'm going to close this since I suspect that is what's happening, but please let me know if this is not the case. Thanks!


abstracting out serializer

Hi, first of all - I would like to thank you - I really love the API, it's kind of brilliant.
That noted - I think there is an important functionality lacking - abstraction over serialization protocol.
I see that RPC module has json hardcoded as serialization mechanism.
That presents several problems:

it's hard to replace json with, for example ujson which is much faster. ( maybe even messagepack / cap'n'proto, etc )
This disallows for serializing types like Decimal, which json does support through the use of custom encoder/decoder (ujson unfortunately doesn't).

Are there plan for abstracting it ? It doesn't seem to be much work, but I only glanced over rpc implementation, so I could be mistaken.

Thanks! Glad you like it :)
Abstracting the serialization is certainly something we'd like to do. Kombu supports pluggable serializers (see http://kombu.readthedocs.org/en/latest/userguide/serialization.html) but we lazily hardcoded to JSON for the serialization checks in the RPC implementation.
Removing the hardcoding should be straightforward and let you fall back to using kombu's serialization, which is a step in the right direction.

Agree, that definitely is a step in the right direction - though kombu's API for registering custom serializers is not pretty - it requires a module installed with a registered entry point, but at least it's doable.
There still would need to be a configuration setting in nameko' - since serializers are registered at a consumer level, which nameko  abstracts over.

You can also register them programmatically - http://kombu.readthedocs.org/en/latest/reference/kombu.serialization.html#kombu.serialization.register
Quite right about the config setting.

#244

Done and landed! (finally) Thanks @ayoshi


fixing greeting action

fixing the indentation in the Greetings.yml GitHub action

@demilolu merge conflict fixed

thanks @fawazsiddiqi and @upkarlidder !


Is GLNet suitable for small size images?

@chenwydj
I have test it on GlobeLandCover dataset. The image size is 2448*2448, but my data image is 256*256.
I wonder if GLNet also work on small size image?
Or if I want to make it work better on my own dataset, what changes can I do?
Do you have any suggestions?

Hi @dagongji10! Thank you for your interest in our work!
We did not test our work on small images like 256*256. You may still try with our patch-based method, by using a crop size smaller than 256.

Hi @dagongji10! Thank you for your interest in our work!
We did not test our work on small images like 256*256. You may still try with our patch-based method, by using a crop size smaller than 256.

Thanks for reply!
I have trained fpn_deepglobe_global.pth use size_g=size_p=112, it runs normally. But when I train fpn_deepglobe_global2local.pth, I got error :
RuntimeError: invalid argument 0: Sizes of tensors must match except in dimension 0. Got 16 and 8 in dimension 2 at ../aten/src/TH/generic/THTensor.cpp: 711
I guess the reason is my input image is (256, 256) not (2448, 2448), the feature_map size come to 0 after downsampling in the middle.
And what do you mean patch-based method? Must I train all the 3 model: fpn_deep_globe_global, fpn_deep_globe_global2local, fpn_deep_globe_local2global ? Or only train one of them for small size image is enough?


Get-PnPList Error in Azure Function App, Error: No list found with id, title or url

Notice
Many bugs reported are actually related to the PnP Framework which is used behind the scenes. Consider carefully where to report an issue:
Reporting an Issue or Missing Feature
When I try to run the following code on my local machine I have no issue the list is returned but as soon as I put this code into a Function App I get the following error:
Code runs without issue on local client:
   $webUrl = "https://xxx.sharepoint.com/sites/Test2023"
   $templateSiteUrl = "https://xxx.sharepoint.com/sites/SPOSiteProvisioning"
   $siteDirList = "Site Directory"

   $webConnect = Connect-PnPOnline -Url $webUrl -Interactive
   $tplSiteConnect = Connect-PnPOnline -Url $templateSiteUrl -Interactive

    write-host -f Cyan "Logging Site information in Site Directory for" $webUrl
    $Web = Get-PnPWeb -Connection $webConnect
    write-output $web
    $list = Get-PnPList -identity $siteDirList -Connection $tplSiteConnect -ThrowExceptionIfListNotFound -ErrorAction Stop
    write-output $list
    if ($webUrl -ne $null) {
      $item = @{}
      $item = @{
          "Title" = $Web.Title;
          "URL" = $Web.URL;
          "SiteDescription" = $Web.Description;
      }
      Add-PnPListItem -List $list -Values $item -Connection $tplSiteConnect
  }

Code fails in functionApp PowerShell Core 7.0
    $siteDirList = "Site Directory"
    write-host -f Cyan "Logging Site information in Site Directory for" $webUrl
    $Web = Get-PnPWeb -Connection $webConnect
    write-host "Web Details:" $web.Title
    $list = Get-PnPList -identity $siteDirList -Connection $tplSiteConnect -ThrowExceptionIfListNotFound -ErrorAction Stop
    write-host "List details" $list
if ($webUrl -ne $null) {
    $item = @{}
    $item = @{
        "Title" = $Web.Title;
        "URL" = $Web.URL;
        "SiteDescription" = $Web.Description;
    }
    $addItem = Add-PnPListItem -List $list -Values $item -Connection $tplSiteConnect
}

Expected behavior
I would expect the Get-PnPList to return the list...
Actual behavior
[Information]   INFORMATION: Error: No list found with id, title or url 'Site Directory'
Steps to reproduce behavior
Function App Config

Runtime version ~4
What is the version of the Cmdlet module you are running?
PnP.PowerShell 1.12.0
Which operating system/environment are you running PnP PowerShell on?

[ ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[x] Azure Functions
[ ] Other : please specify

Since the last two versions we've made a change to the code where Connect-PnPOnline no longer returns an instance of the connection unless you explicitly add -ReturnConnection to your statement. I notice this line in your script:
   $webConnect = Connect-PnPOnline -Url $webUrl -Interactive

Here $webConnect should be NULL. Can you change it to the following and see if that resolves it for you:
   $webConnect = Connect-PnPOnline -Url $webUrl -Interactive -ReturnConnection

Perhaps I am wrong, but can you even do an -Interactive connection from an Azure function?
I thought you would have to provide the credentials non-interactively, i.e. via an App Registration. or user details held in a KeyVault, etc.

Very good point @BaronSparky, didn't even think about that. There is no interactivity possible in Azure Functions indeed. Try using a non interactive method instead.

Perhaps I am wrong, but can you even do an -Interactive connection from an Azure function?
I thought you would have to provide the credentials non-interactively, i.e. via an App Registration. or user details held in a KeyVault, etc.

Thanks for the reply! Yes I am aware that “-interactive” only works on my local client as stated above. That was the working code that succeeded on the client!
My issue was that I hadn’t included the “-returnconnection“
Thanks again!


Simple Browser not working when running from source

Steps to reproduce:

Build VS Code from source using Insiders
Launch Code - OSS Dev, either from Debug or from .\scripts\code
Open Command Palette, run Simple Browser: Show and enter a URL such as http://localhost

Toolbar icons are missing and page doesn't load.
Resolve the problem by copying codicon.css, codicon.ttf and index.js into the extensions\simple-browser\media folder from C:\Users\XXX\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\extensions\simple-browser\media
Presumably the build task is missing a step.
/assign @mjbvz

Work for me on MacOS. How are you compiling VS Code? I tested with yarn watch

Seems to be working for me now. Sorry for the noise. Closing.


[Gekidou MM-39728] - Websocket Events - Channels

Summary
This PR handles the following Channels websocket events:

[ ] where does pinned_post_count, member_count come from? not included with channel updated WS event
[ ] When you receive a channel created or direct added (not sure, maybe both) event, make sure that you populate the displayName field for DM's and GM's (don't you miss that very very important detail)
[ ] fetchMyChannel returns {channels: [channels], memberships: [memberships]} but could return the singular {channel, membership}
[ ] no user preferences are stored in WDB until they make a change through the webapp and receive the Websocket event. Ex. (Settings > Display > Teammate Name Display)
[ ] GM websocket is group_added and not yet handled or defined for V2
[ ] channelInfo header/purpose getting updated, but not guest_count, pinned_post_count, or member_count

TBD - Channel created

should we only save channels we belong to?
msg doesn't contain enough data.  have to get from the server

DONE - Channel deleted - done by Daniel
DONE - Channel unarchived

unarchive via System > Channels
unarchive via channel view and you are sysadmin
initally believed that I could handle with channelUpdateEvent, but that is only called when unarchive through System Console.

TBD - Channel updated

Header
Purpose
Rename Channel Displayname

DONE - NOT NEEDED - Channel converted

Private <=> Public
Covered by channel updated event
msg doesn't contain the Type value. Would have to:

flip Type in DB to from O<>P or P<>O

DONE - Channel viewed
TBD Channel member updated

can be done through adding groups to a channel (sys console)

DONE - NOT NEEDED - Channel scheme updated (see https://community.mattermost.com/core/pl/i15h9yjiwjbzxksgwwiir7imky)
DONE Direct Added

Questions
ChannelMembership stores id as <channel_id>-<user_id>with fieldschannel_idanduser_id`. If only checking for existance of user_id in channel, only need to check for existance of the id
Ticket Link
https://mattermost.atlassian.net/browse/MM-39727
Device Information
This PR was tested on:
Screenshots
Release Note
Notes
handleChannelUpdatedEvent
{ "broadcast": {
      "channel_id": "ursurixqpig198pf7zohmu3qfy",
      "omit_users": null,
      "team_id": "",
      "user_id": ""
    }
  "data": {
    "channel": {
        "create_at":<PHONE_NUMBER>504,
          "creator_id": "wkostedqntnofxoih86bkdri3r",
          "delete_at": 0,
          "display_name": "membership testing",
          "extra_update_at": 0,
          "group_constrained": false,
          "header": "Header",
          "id": "ursurixqpig198pf7zohmu3qfy",
          "last_post_at":<PHONE_NUMBER>430,
          "last_root_post_at":<PHONE_NUMBER>430,
          "name": "membership-testing",
          "policy_id": null,
          "props": null,
          "purpose": "",
          "scheme_id": "",
          "shared": null,
          "team_id": "5nchh6mqubrh5edjkom55bxozo",
          "total_msg_count": 7,
          "total_msg_count_root": 7,
          "type": "O",
          "update_at":<PHONE_NUMBER>256
        }
    }
}

/release-note-none

Most of the feedback addressed, waiting for @shazm comments.

@enahum (and @larkox) - Channels can't be created directly in a category; the work I did in v1 to make it look like so actually makes two calls - a channel create, and then an update to move the channel to the category the user wants to create in. This sends a sidebar_category_updated event; which will have the new list of channel_id's in the updated category 👍
I don't think you need to get categories here; as the category websocket handlers will cover that - if you have a live websocket connection I think it's safe to assume that the category websockets are also working, and they have their own try/catch's to fetch via the API if the WS fails.
Probably my only concerns are race conditions (category update being worked before channel create, which I don't think can happen but worth double checking), and WS connection failure half way (so channel was created in the category, app got the channel create WS but dropped out before getting the category update WS).

@shazm Just to make sure I understood. If a channel is created in a category, we will receive two different events:

Channel created
Channel added to category

But what happens if the channel is created in the default category? (Or it is a DM, created in the default dm category) We won't be receiving the second websocket event, right? So we have to either play it safe and fetch always the category, or set the category to default, and hope the websocket event comes later to update that. Am I right?

@shazm Just to make sure I understood. If a channel is created in a category, we will receive two different events:

Channel created
Channel added to category

But what happens if the channel is created in the default category? (Or it is a DM, created in the default dm category) We won't be receiving the second websocket event, right? So we have to either play it safe and fetch always the category, or set the category to default, and hope the websocket event comes later to update that. Am I right?

Yes exactly.
I would make sure the channel goes to the DM or channels category by default when creating via the WS event. If it was created in a category, then the category update event will take care of moving it to the correct place.

@shazm Just to make sure I understood. If a channel is created in a category, we will receive two different events:

Channel created

Channel added to category

But what happens if the channel is created in the default category? (Or it is a DM, created in the default dm category) We won't be receiving the second websocket event, right? So we have to either play it safe and fetch always the category, or set the category to default, and hope the websocket event comes later to update that. Am I right?

Yes exactly.

I would make sure the channel goes to the DM or channels category by default when creating via the WS event. If it was created in a category, then the category update event will take care of moving it to the correct place.

Ok let's be clear about the expected.
When a new channel is created we receive two WS one for the created channel and another one for the category the channel belongs to?
if the answer to that question is YES, then we need to remove anything category related in the handler, if the answer is no, then what do we do?

assume the category is the default and add it to the db directly?

fetch all categories from the server?

@shazm can you be extra clear on this please?

Yup of course:

If a channel is created, there will always be a channel create event

If a channel is created inside a category (currently only possible in mobile v1), there will be a category update event as well.

There is no category update event if the channel is just created in the default channels category.
We have to add the channel to the default category in that instance; either by fetching categories or just doing it locally (my recommendation).

Thanks @shazm that is super clear...
@larkox I believe what we should do then is for channels add the channel locally directly to the Channels category on the team the channel was created and for DM/GMs let's add them locally to the direct_messages category of every team.


Fix mouse input events being read multiple times

@alice-i-cecile I saw you were busy moving at the moment (congrats c:), but this should fix the issue with Bevy 0.12.1 that people mentioned in the discord.
Fixes #285
Cleanest fix I could come up with was to replace the Events<T> in InputStreams with Vec<T>, and then collecting the events into that with an EventReader in update_action_state.

I think this change broke the egui feature. I'm getting this compile error on 0.11.2
   --> /home/keis/.cargo/registry/src/index.crates.io-6f17d22bba15001f/leafwing-input-manager-0.11.2/src/systems.rs:130:9
    |
127 |       let (mouse_buttons, mouse_wheel) = if ctx.is_pointer_over_area() || ctx.wants_pointer_input() {
    |  ________________________________________-
128 | |         (None, None)
    | |         ------------ expected because of this
129 | |     } else {
130 | |         (mouse_buttons, mouse_wheel)
    | |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `(Option<_>, Option<_>)`, found `(Option<&Input<MouseButton>>, Vec<MouseWheel>)`
131 | |     };
    | |_____- `if` and `else` have incompatible types
    |
    = note: expected tuple `(std::option::Option<_>, std::option::Option<_>)`
               found tuple `(std::option::Option<&Input<MouseButton>>, std::vec::Vec<bevy::input::mouse::MouseWheel>)`

Sorry about that. I'll have another PR up in a bit.

Got a fix for that here: https://github.com/Leafwing-Studios/leafwing-input-manager/pull/421.
In the mean time, it does compile if you keep the default features (ui and block_ui_interactions) enabled.


Added Django Cheat Sheet links.

What does this PR do?
This PR is a collection of some useful links of Django Cheat Sheets.
For resources
Description
This PR provides some useful Django Cheat Sheet links which can be opened with one click and doesn't ask for any verification or signup related issues.
Why is this valuable (or not)?
This is a valuable PR because it contains some beginner friendly Django Cheat Sheets links which are easy to understand and apply in code.
How do we know it's really free?
The sites doesn't asks for any email/mobile no. verification and signup related issues. All the links are followed by https:// which makes it more secure.
For book lists, is it a book? For course lists, is it a course? etc.
This PR contains some free Django Cheat Sheet links.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Link of the issue - https://github.com/EbookFoundation/free-programming-books/issues/7106

I think these are supposed to go on the chet sheet list
Please read CONTRIBUTING.md guidelines!

I think these are supposed to go on the chet sheet list Please read CONTRIBUTING.md guidelines!

Alright, I will check it.


Configuration to include avro schema

Would it be possible to change some configuration to include the avro schema instead of just payload?
On documentation we have, can schema be enabled?

Value/Key schema will not be presented in output file, even if value.converter.schemas.enable property is true. But, it is still important to set this property correctly, so that connector could read records correctly.

Hi,
Thank you for using the connector!
Outputting the schema is not implemented at the moment: the whole schema functionality relies on Schema Registry.
We could implement this at some point. How do you see the target file layout?

Most flexible would be setting the path to save the schema in configuration otherwise I think it would make sense to have in the headers.
This would be specially useful because removes the dependency of the schema registry for off-line analyses of messages.


Remove duplicate URL

There was a duplicate https://login.microsoftonline.com in CommonDomainURLs.

@chcomley
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

@juliakm: I'm sorry - only the author of this article, @chcomley, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Clean up addr2line so it is a real library

Sorry for not breaking this down into smaller commits, but there wasn't a good way of doing that without the intermittent commits being unintuitive, broken, or both. This moves all the logic into the library, provides a documented API (with #[deny(missing_docs)]), and extends the binary such that its interface matches that of addr2line. It builds cleanly on both stable and nightly.
Next steps would probably be to add support for extracting containing function, not just file + line. I think this shouldn't be too painful, but you never know. I would also like to try and find a good way to test this, but I'm not entirely sure how to go about that. Suggestions welcome!
The darkest corners of this PR is the nested use of owning_ref::OwningHandle. This is necessary so that we can construct a struct which contains a memmap, an object::File that borrows the map, and gimli objects that borrow the object::File. Luckily, it all works out somewhat nicely. The biggest bummer at the moment is that errors in the construction of the object::File and gimli objects cause panics, which is due to the lack of being able to forward errors through owning_ref::OwningHandle. I've filed a bug about this, so hopefully it will be resolved in not too long.
I've left the Unit stuff mostly intact, including the comment saying

TODO: most of this should be moved to the main library.

Coverage decreased (-1.7%) to 0.99% when pulling cdc9973708bee223537b7127c63fb639a1fe0294 on jonhoo:master into 03318459cfc6fb0ad916c474fe143ff0bc860229 on gimli-rs:master.

Coverage decreased (-1.7%) to 0.99% when pulling cdc9973708bee223537b7127c63fb639a1fe0294 on jonhoo:master into 03318459cfc6fb0ad916c474fe143ff0bc860229 on gimli-rs:master.

Thanks for the pull request! Digging in now...

Sorry for not breaking this down into smaller commits, but there wasn't a good way of doing that without the intermittent commits being unintuitive, broken, or both.

No worries.

This moves all the logic into the library, provides a documented API (with #[deny(missing_docs)]), and extends the binary such that its interface matches that of addr2line. It builds cleanly on both stable and nightly.

Thank you!!

I would also like to try and find a good way to test this, but I'm not entirely sure how to go about that. Suggestions welcome!

It should be easy enough to write integration tests by using nm on our addr2line itself to find addresses and function names, and then pass those addresses into our addr2line and make sure that we get something.
We could also try comparing our results with the canonical addr2line's results.
As far as unit tests go, its more about semi-tediously writing test data and breaking down each function as far as possible.
These things can, of course, be done in follow up PRs if you are so inclined :)

It should be easy enough to write integration tests by using nm on our addr2line itself to find addresses and function names, and then pass those addresses into our addr2line and make sure that we get something.

Do we have a way of referring to the path to build artifacts from inside an integration test?
Also, how were you thinking of using nm here? It wouldn't actually give us the mapping from source line to address (though we could cook something up by using placeholder functions).

We could also try comparing our results with the canonical addr2line's results.

This was more in line with what I was thinking, though the questions above still apply with regards to getting access to our own build artifact in the tests. Also, can we rely on the test running (Travis) to have addr2line?

As far as unit tests go, its more about semi-tediously writing test data and breaking down each function as far as possible.
These things can, of course, be done in follow up PRs if you are so inclined :)

I think I'd prefer for them to be follow-up PRs :)

Just pushed the Cargo.toml version change. Do you also want the commits squashed?

Coverage decreased (-1.7%) to 0.99% when pulling 59bec17c4d95b0257d67fdd4abb21157d29e431f on jonhoo:master into 03318459cfc6fb0ad916c474fe143ff0bc860229 on gimli-rs:master.

Coverage decreased (-1.7%) to 0.99% when pulling 59bec17c4d95b0257d67fdd4abb21157d29e431f on jonhoo:master into 03318459cfc6fb0ad916c474fe143ff0bc860229 on gimli-rs:master.

Do we have a way of referring to the path to build artifacts from inside an integration test?

From http://doc.crates.io/environment-variables.html:

CARGO_TARGET_DIR - Location of where to place all generated artifacts, relative to the current working directory.

So we could do something like this:
use std::env;

let mut addr2line = PathBuf::new(env::var("CARGO_TARGET_DIR").unwrap());
addr2line.push("addr2line");

Also, how were you thinking of using nm here? It wouldn't actually give us the mapping from source line to address (though we could cook something up by using placeholder functions).

Right -- we would just want to make sure that we didn't get ?:?, but I'm not sure how reliable that would actually be... Maybe if we filtered for only known functions within gimli or addr2line...

Also, can we rely on the test running (Travis) to have addr2line?

I think so. If not, we can install it with apt from whatever package it might be in from inside .travis.yml.

I think I'd prefer for them to be follow-up PRs :)

Of course :)

Do you also want the commits squashed?

Nah, its fine.
Thanks again! Very appreciated!

Unfortunately CARGO_TARGET_DIR is a configuration option, not something that's set by cargo. We may be able to use OUT_DIR, but I don't know that there's any guarantee that the binary has been built by the time the tests run.. :/

One alternative is to just stuff some C code in there and use native code compilation.

What would the C code do that we can't do in rust?
You could look at how others have solved it, eg https://github.com/BurntSushi/ripgrep/blob/master/tests/workdir.rs, doesn't look like there is a simple solution though.

It's just easier to compile reliably from running commands is all. See https://github.com/gimli-rs/addr2line/pull/4. std::env::current_exe is a neat trick that might be applicable, though I think getting up-and-running on analyzing a trivial C program first is a good first step.


Tail of g doesn't show on certain window sizes.

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?

As in the image, the tail of the "g" of "config" doesn't show in this window size.

But when I slightly change the window size, the tail of g is visible again.
I've also tried advanced shaping, but nothing changed.
Code:
                container(column![text("Failed to read the config file.").size(30), text(err).shaping(text::Shaping::Advanced)]).width(Length::Fill).height(Length::Fill).center_x().center_y().into()

What is the expected behavior?
Expecting that the tail would be visible when the text itself is also visible.
This issue seems to happen because the tail of "g" overflows the bounding box.
Version
master
Operating System
Windows
Do you have any log output?
No response

This is a known issue blocked by https://github.com/pop-os/cosmic-text/issues/70.


Implement OAuth 2.0

For:

Google
Facebook
Office365
GitLab
LDAP
SAML
Am I going crazy?

We should just do Google for now
Yes


unable to run az storage list

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az storage account list
Errors:
The command failed with an unexpected error. Here is the traceback:
function() argument 'code' must be code, not str
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 561, in execute
    self.commands_loader.load_arguments(command)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/core/__init__.py", line 516, in load_arguments
    loader.load_arguments(command)  # this adds entries to the argument registries
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/command_modules/storage/__init__.py", line 31, in load_arguments
    load_arguments(self, command)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/command_modules/storage/_params.py", line 41, in load_arguments
    t_share_service = self.get_sdk('_share_service_client#ShareServiceClient',
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/core/__init__.py", line 756, in get_sdk
    return get_sdk(self.cli_ctx, kwargs.pop('resource_type', self._get_resource_type()),
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/core/profiles/__init__.py", line 91, in get_sdk
    return _sdk_get_versioned_sdk(cli_ctx.cloud.profile, resource_type, *attr_args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/core/profiles/_shared.py", line 661, in get_versioned_sdk
    loaded_obj = _get_attr(sdk_path, mod_attr_path, checked)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/cli/core/profiles/_shared.py", line 614, in _get_attr
    op = import_module(full_mod_path)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/multiapi/storagev2/fileshare/v2021_06_08/__init__.py", line 8, in <module>
    from ._file_client import ShareFileClient
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/multiapi/storagev2/fileshare/v2021_06_08/_file_client.py", line 30, in <module>
    from ._shared.base_client import StorageAccountHostsMixin, parse_connection_str, parse_query
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/multiapi/storagev2/fileshare/v2021_06_08/_shared/base_client.py", line 40, in <module>
    from .authentication import SharedKeyCredentialPolicy
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/multiapi/storagev2/fileshare/v2021_06_08/_shared/authentication.py", line 22, in <module>
    from azure.core.pipeline.transport import AioHttpTransport
  File "<frozen importlib._bootstrap>", line 1075, in _handle_fromlist
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/core/pipeline/transport/__init__.py", line 68, in __getattr__
    from ._aiohttp import AioHttpTransport
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/azure/core/pipeline/transport/_aiohttp.py", line 35, in <module>
    import aiohttp
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/aiohttp/__init__.py", line 6, in <module>
    from .client import (
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/aiohttp/client.py", line 35, in <module>
    from . import hdrs, http, payload
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/aiohttp/http.py", line 7, in <module>
    from .http_parser import (
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/aiohttp/http_parser.py", line 15, in <module>
    from .helpers import NO_EXTENSIONS, BaseTimerContext
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/aiohttp/helpers.py", line 667, in <module>
    class CeilTimeout(async_timeout.timeout):
TypeError: function() argument 'code' must be code, not str

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az storage account list

Expected Behavior
Environment Summary
macOS-12.4-x86_64-i386-64bit
Python 3.10.4
Installer: PIP

azure-cli 2.38.0

Additional Context

storage

Might be related to https://github.com/aio-libs/aiohttp/issues/6794, but I can't reproduce.
Could you please do a clean install and try again?


IETF Draft calls it Sensor Measurement Lists not Sensor Markup Language

Not sure if this change is intentional but it would be great to get a clarification.

My fault. When we started, it was called "Markup Language", and the name stuck...
I'll fix it.

Thanks

No worries. Thanks for the quick clarification.


Added video tutorial link.

:tada: This PR is included in version 6.5.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update login state to rely on cookie instead of local storage

The backend (as of 0cd008c7d536e804b6a997e7d887e9b2ac82fecf) now sends back a Javascript accessible couchers-user-id cookie. The web frontend should be modified to use this to track login state, instead of local storage.
The issue with the old system is that sometimes local storage is cleared for no good reason (seems to occur on certain updates). This means people get logged out for no good reason and it's quite annoying.
This is takes over #4693 which was a non-cookie based hacky solution to fix this.

This was merged Sep 5th, in #4726. Let's wait a few more months so all active users will have the cookie set before considering switching to that.


Update simple_setup.sh

Just some minor safety things: double quotes, local declarations in function bodies, and set -eo pipefail. Ping me if you have any questions.

Did you happen to test whether it successfully sets everything up, and did it happen to work?

Not yet. I'm working on a docker-compose and Dockerfile for a Debian install and I'll let you know how it goes.


aws-lambda: FunctionProps's Environment is map of any

The aws-lambda module's FuncitonProp's Environment property is defined as a map of any. I think this is incorrect type for a property that is used for users to specify the environment variables.
I think FunctionProp.Environment property should be a map of string. Nearly all OS environment variable values are always string type, and map of any is confusing for users of language bindings generated from the CDK's source.
I suggest changing the type of this any similar parameters to be map of string.

I think this is a relic of when we supported both string and cdk.Token. Since cdk.Token is passed around stringified, it should be safe to adjust this to {[key: string]: string}. It also more accurately models environment variables which are always consumed as strings.

This has been long fixed. The environment property type is now { [key: string]: string }


Setup codecov.io

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f5103cf). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage          ?   98.53%
=========================================
  Files             ?       59
  Lines             ?      890
  Branches          ?        0
=========================================
  Hits              ?      877
  Misses            ?       13
  Partials          ?        0

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 f5103cf...df16d64. Read the comment docs.


Add user controller

User controllers has been update:

added post request to add 1 user in the database
added get all users request

In the model:
Only username, email and password are now required and can't be null inlike the others values.
Routes:
Get all users : GET /users
Add a user : POST /user

Added one more request:

put request that remove a user from the Users model based on ID


The documentation is missing information that CRD's need to be installed prior helm chart install

The documentation page https://docs.dapr.io/getting-started/install-dapr/#install-with-helm-advanced  is missing information that the custom crds need to be installed prior attempting to install the helm chart.
I resolved this issue by running this commands
kubectl apply -f https://raw.githubusercontent.com/dapr/dapr/master/charts/dapr/crds/subscription.yaml
kubectl apply -f https://raw.githubusercontent.com/dapr/dapr/master/charts/dapr/crds/configuration.yaml
kubectl apply -f https://raw.githubusercontent.com/dapr/dapr/master/charts/dapr/crds/components.yaml

The CRDs are installed with the Helm Chart.. which version of Helm are you using?

I used the lates version of the helm chart, just followed the documentation and it failed the first time I tried indicating that the configuration crd is missing.

Sorry I wasn't clear: which version of Helm itself (the binary, not the chart) are you using?

Ah sorry I also did not read good :) it is Helm 3.3.4


Bulk API

Does crate provide an API for bulk inserts similar to the one elaticsearch provides? If not, is it acceptable to enable the elasticsearch API?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29507113-bulk-api?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github).

@afrozl crate does not support auto-increment. I would rather consider to remove the id column and use the auto-generated document-id column _id instead which is a UUID and therefore unique for each row.


Queue models to train

I can't find information on docs.
Can I queue eg. 10 models with different hyperparameters, and Trains will run them one by one?

TRAINS does not including a job scheduling solution, though it will seamlessly track any job you schedule :)

Closing, due to lack of activity


Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps …

…to suppress warning

Could you please fix the conflicts? I will than take a look at the PR, test it and possibly merge it to remove warnings :)

When may we expect a new version that contains this fix? 😀
Thank you for your work!

Hi!
i am currently working on building an unstable release version so that I can test the package in our production product, I will let you know if the unstable version is ready and will create a release when it is tested.

I just released a ReleaseCandidate, if you can test it with me that would  be amazing! I have not seen any errors on the UNSAFE part yet! https://www.npmjs.com/package/react-swipeable-views

🤔 it seems the fix did not make it into release 0.13.9, did it?


Adding shape inference for RNNs

Note that there was a strange behavior that I fixed, @tungld .
RNNs with layout=1 were changed by a Rewrite pattern, where inputs and outputs were transposed, and fed in the SAME RNN op. That op was then sent to recompute shape. But it had erroneous output shapes (from before the transformation, as the same op was used, not a new one). Thus the regular shape inference, which picks the "best" out of the shape inference and the output types, it was picking the output type. That problem did not exist before as it was not using the shape inference scheme, and thus did not use the code that merge old type and new interested type shapes.
So my fix was to reset the output types of reused op before being sent to shape inference to have -1 only shapes. That was much cleaner than starting to distinguish in shape inference which ops have valid output shapes vs which ones do not.
As mentioned in the code, it might be cleaner to create a new RNN op instead of reusing the previous one... but it felt more intrusive a solution at this time. I let you assess if that would be a simpler fix @tungld .

Jenkins Linux amd64 Build #9186 [push] Adding shape inference f... started at 10:44

Jenkins Linux s390x Build #9203 [push] Adding shape inference f... started at 11:44

Jenkins Linux ppc64le Build #8250 [push] Adding shape inference f... started at 11:49

Jenkins Linux amd64 Build #9186 [push] Adding shape inference f... passed after 1 hr 11 min

Jenkins Linux s390x Build #9203 [push] Adding shape inference f... passed after 1 hr 56 min

Jenkins Linux ppc64le Build #8250 [push] Adding shape inference f... passed after 2 hr 5 min


M3-5220 & 5213: Firewall Details table header and Longview Plan chip

Description
This PR defines the table header as 46px so it'll be consistent throughout the app and also adjusts the Longview Plan chip style to match the one used on the Kubernetes Landing table (I also tightened up the spacing while I was in here)
How to test
M3-5220:
Go to any Firewall Detail page and on the Rules tab, check the height for the table header and row. The table header should be 46px and the row should be 40px.
M3-5213:
Make sure the account you're logged into is not managed  and go to /longview/plan-details
Before:

After:

@HanaXu good point, I removed the Paper when it's only the table that's visible. I also hid the table for restricted users so lmk what you think!

@tiffwong Good catch on the restricted users, changes LGTM! Another thought is to toast the success/error messages similar to longview/clients, but that can be handled in another story since it's a bit out of scope.

@HanaXu turns out that users with read or read-write access can see the current plan but users with no access don't so I opted to only hide the table for users with no access 😅

@tiffwong If a user has no access, should we be displaying Unauthorized (similar to /account/billing etc) instead of You don't have permissions to change the Longview plan...?

@HanaXu hmm that's a good point. I just left it as it currently is in production but this might be a good thing to bring up to Matthew. In the meantime, I might merge it and address that in a future PR if everything else looks good


IndexDB: handle missing object store more gracefully

While fixing my storage service in web that is using IndexedDB  I noticed that the user data provider leveraging IndexDB is checking whether the object store exists or not:
https://github.com/microsoft/vscode/blob/be2f951fde6f53672630d78fb81e3b091e7ad13c/src/vs/platform/files/browser/indexedDBFileSystemProvider.ts#L58-L62
I wonder if you should adopt a similar strategy that I do by recreating the IndexDB and store so that the user is not ending up with an in-memory solution forever.
Here is my fix: https://github.com/microsoft/vscode/commit/6c302fbb07d515e4b9d04c65f35a4f3641433789
https://github.com/microsoft/vscode/blob/6c302fbb07d515e4b9d04c65f35a4f3641433789/src/vs/platform/storage/browser/storageService.ts#L252-L276

Not fixed.


GraphQL - Project Attachment Resolver + AttachFilesToProject Mutation

What does this PR do and why?
Describe in detail what your merge request does and why.
Adds attachments as a field to GraphQL projects query
adds attach_files_to_project GraphQL mutation
refactored duplicate functions from AttachFilesToSample into BaseMutation
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other pull requests.
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
Test getting, filtering and ordering of attachments

Sign in with user1
go to GraphiQL
test filtering and ordering

query get_project_attachments{
  projects(first:1){
    nodes{
      name
      attachments(
        filter:{filename_end:".fastq.gz"}
        orderBy:{field:created_at, direction:asc}
      ){
        totalCount
        nodes{
          filename
          createdAt
        }
      }
    }
  }
}

Test attaching files to project

First wipe the db and seed without files

bin/rails info db:drop db:create db:migrate
SEED_MAXIMUM_TOTAL_SAMPLE_ATTACHMENTS=0 bin/rails info db:seed

sign into admin and go to GraphiQL
Get a project you have ownership of

query get_proj{
  projects(first:1){
    nodes{
      name
      fullName
      puid
    }
  }
}

We will use the puid

Create a blob for your file
File: https://github.com/phac-nml/irida-next/files/14129150/myfile.json

mutation makeblob {
  createDirectUpload(
    input: {
      filename: "myfile.json", # file name
      contentType: "application/json", # file content type
      checksum: "Y33CgI35hFoI6p+WBXYl+A==", # md5
      byteSize: 151 # size in bytes
    }
  ) {
    directUpload {
      url,
      headers,
      blobId,
      signedBlobId
    }
  }
}

We will use url, headers and signedBlobId

Upload your file

Using Postman, make a PUT request, using the returned url from above

Attach the file as a binary

Verify your content type matches the headers from above. Postman should auto-generate this based on the file you attached.
When Send is clicked, you should receive a 204 No Content

Use the new mutation to attach the file to the project

mutation attachfilestoproject {
  attachFilesToProject (
    input: {
      files: ["YOUR signedBlobId FROM ABOVE"],
      projectPuid: "YOUR puid FROM ABOVE"
    }
  ) {
    errors{
       path
       message
    },
    project{
      name
      puid
    }
    status
  }
}

Expected response

{
  "data": {
    "attachFilesToProject": {
      "errors": [],
      "project": {
        "name": "Outbreak 2022",
        "puid": "YOUR puid FROM ABOVE"
      },
      "status": {
        "YOUR signedBlobId FROM ABOVE": "success"
      }
    }
  }
}

Check that the file was attached

query get_project_attachments{
  projects(first:1){
    nodes{
      fullName
      name
      puid
      attachments{
        totalCount
        nodes{
          filename
          byteSize
          createdAt
        }
      }
    }
  }
}

Expected response

{
  "data": {
    "projects": {
      "nodes": [
        {
          "fullName": "Bacillus / Bacillus anthracis / Outbreak 2022",
          "name": "Outbreak 2022",
          "puid": "YOUR puid FROM ABOVE",
          "attachments": {
            "totalCount": 1,
            "nodes": [
              {
                "filename": "myfile.json",
                "byteSize": 151,
                "createdAt": "2024-09-12T20:56:48Z"
              }
            ]
          }
        }
      ]
    }
  }
}

PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

[x] I have evaluated the PR acceptance checklist for this PR.

This PR is working as intended, but there is some kind of problem with bot accounts not having permission to attach files to projects even when they have permission and a valid token. The test case that is failing is from this issue, I'm not sure if it's the result of this PR or one of the other namespace/project changes made this week.

This PR is working as intended, but there is some kind of problem with bot accounts not having permission to attach files to projects even when they have permission and a valid token. The test case that is failing is from this issue, I'm not sure if it's the result of this PR or one of the other namespace/project changes made this week.

The test case is failing because can_view_attachments? and can_create_attachments? only allow Maintainer and Owners to perform the action. To make this work with Uploader you could refactor those two methods to be more like can_modify_sample?, the methods can be found in app/models/member.rb.

This PR is working as intended, but there is some kind of problem with bot accounts not having permission to attach files to projects even when they have permission and a valid token. The test case that is failing is from this issue, I'm not sure if it's the result of this PR or one of the other namespace/project changes made this week.

Hey Jeff, sorry for overlooking this. The fix has been merged to main and will fix the testing issues you were having


gCNV genotyping tool

... to replace and/or build upon HMMPostProcessor.
Milestones:

[ ] read/write forward-backward results

Issue moved to broadinstitute/gatk #2994 via ZenHub


Feature: Cross compilation of sqlite extension for ios and android for react native apps

Describe what you are looking for
I am keen to use sqlite extension with react native application
To make it happen i could use op-sql library that load extensions but for this i need a extension compiled to next platforms
For ios and mac we need Xcframework like structure  .
Sample could be found https://github.com/vlcn-io/cr-sqlite/blob/main/core/all-ios-loadable.sh
ios x64
ios 64 simulator
android arm64-v8a
android armeabi-v7a
android x86
android x86_64
Can you contribute to the implementation?

[ ] I can contribute

Is your feature request specific to a certain interface?
Other
Contact Details
<EMAIL_ADDRESS>Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

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

This task is very niche and would significantly complicate what is already one of trickiest GitHub CI pipelines.

@ashvardanian  sqlite for mobile is one of the biggest usecases . it make sqlite most popular db in world. it sad that mobile development in general ignored by team


fix(#2573): Fix broken resequenzing of requests

Description
Fixes broken resequenzing of Requests. The seq property is in the Top-Level of the JSON instead of inside meta.

Merged!
Thanks for the quick fix @Its-treason !


Stop territory labels obscuring the territory ownership in animations

The output of displays are not rendered in the produced animations. As such the current labels are rendered as an opaque disk over the territory floor so the owner's colour is not visible.
This reduces the display to the area used by the labels so that the territory floor is still visible in the animations. If we want to have the labels visible in the animation outputs we will have to generate the labels as textures.

Alternatively we can directly use textures for the labels, as in d2954764bacb4d0b5a907ced35eaacbde943d1f9. This has the added benefit of keeping the labels visible in the animations and only increases the texture size by ~50kB.

50kb is "basically free", especially with #315. Personally i'd vote for #314.

srobo/competition-simulator#314 is a better approach


mesos-hdfs keeps giving me error when I start

I keep getting the following error when I attempt to start mesos-hdfs..
Error: Could not find or load main class org.apache.mesos.hdfs.scheduler.Main
I am guessing its the java app not being able to find the right library, but aren't all the libraries packaged in the tarball?

hm, yes it should be built with the package. Could you try with the latest version of the code? Are you running on master? Were you able to build successfully?

So I went ahead and downloaded your code again. I am now getting this error.. I was running it last week with the nocomplie option since gradle was giving me fits. I just ran the build and I am getting this. Am I missing any prerequisite software? I am new to mesos, so I am definitely thinking I am making a mistake.. this is on a VM by the way.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':hdfs-commons:compileJava'.

Could not find tools.jar

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Okay so I am making progress here.. I removed and reinstalled java to my mesos-master. CentOS had installed OpenJDK 7 by default. I downloaded JDK 8 from Oracle and installed that. Once I did that, I was able to do the build with gradle and then I could launch. Now I just need to tweak my site files so that it work with my configuration that I am setting up. Thanks for getting back to me. As I figured it was a problem on my end, but if I can provide something here to help others, then I guess I at least accomplished that. Guess I just needed a sounding board to bounce the idea around..

Sounds great! Let us know if there are other issues or feel free to close the issue.


Bizarre channel state problem

I have two channels in a really bizarre state:
The show up in Zap, and appear to be "pending"
On the blockchain, they are confirmed.
In LND: They don't show up as active, inactive, nor pending.
In zap, there is no 'disconnect' button.
Below is the link that opens when you click the box on the channel.
https://www.smartbit.com.au/tx/505637e0685355b0fdf37d1c81f00601dcafc07dac89ed0e32a7c4162218e149

@ZapUser77 where are these channels when you use lncli if they aren't pending?

Ok.  I see the issue.
I was using lncli getinfo, which shows 0 pending channels.
Lncli pendingchannels shows the channel in "pending force close" state.
A legend of what the colored dots mean may be beneficial to users.
You can close the ticket.


No Issues found with large Function Groups

Unfortunately we have a large Function Group (14 Function Modules) from a external Consulting Company (so refactoring everything would be no option right now). But we would like to check every new development with abaplint.
But we get zero issues but would expect over a hundred.
2023-04-21 12:51:14, listRules
2023-04-21 12:53:52, check_file, FUGR, ZINBOX_0, 0 issues, 2023834 bytes, 12035 ms
2023-04-21 12:53:52, ping

Log seems like everything is send correctly.
Version abaplint: 2.97.12

moved to abaplint-sci-client


Add implication concise form

Hi!
This PR adds the concise form for the math implication.

Not sure I understand.
This code is saying "If A is true, then B must also be true."
if (A === true) {
  console.assert(B === true)
}

However this has a lot of different implications in JavaScript:
! A || B

If A does not not coerce to a truthy value OR if B coerces to a truthy value.

A === false || B === true coule fix the coerce issue?
Accoridng me,
if (A === true) {
  console.assert(B === true)
}

is equivalent to:
console.assert(A === false || B === true)

They are similar but not exactly the same. Also I think simplicity is best here, rather than overwhelming the reader with alternatives. :smile:
See the Wikipedia note here.

A ⇒ B means if A is true then B is also true; if A is false then nothing is said about B.

The goal of my original code was to convey that nothing is said about B when A is false.

Ok. This is convincing :)


Issue importing the exported DB

Hi!
Thank you for the caring support.
I am using version 2.1.0-4 of the plugin and have some problem importing the DB that I have already exported. Here is the steps that I have taken ( the scenario is based on the andular-sqlite-app-starter):

create the tables using the following statement (with no data inserts)

      let result:any = await this._SQLiteService.openDB("test-sqlite");
      if(result.result) {

        result = await this._SQLiteService.createSyncTable();

        // create tables
        const sqlcmd: string = `
        BEGIN TRANSACTION;
        CREATE TABLE IF NOT EXISTS users (
            id INTEGER PRIMARY KEY NOT NULL,
            email TEXT UNIQUE NOT NULL,
            name TEXT,
            company TEXT,
            size FLOAT,
            age INTEGER
        );
        CREATE TABLE IF NOT EXISTS messages (
            id INTEGER PRIMARY KEY NOT NULL,
            userid INTEGER,
            title TEXT NOT NULL,
            body TEXT NOT NULL,
            FOREIGN KEY (userid) REFERENCES users(id) ON DELETE SET DEFAULT
          );
        CREATE TABLE IF NOT EXISTS images (
          id INTEGER PRIMARY KEY NOT NULL,
          name TEXT UNIQUE NOT NULL,
          type TEXT NOT NULL,
          size INTEGER,
          img BLOB
        );
       CREATE INDEX users_index_name ON users (name);
        PRAGMA user_version = 1;
        COMMIT TRANSACTION;
        `;
         result = await this._SQLiteService.execute(sqlcmd);

export the DB in "full" mode:
and then import the exported json string

      console.log('****** export ******');
      let result:any = await this._SQLiteService.openDB("test-sqlite");
      if(result.result) {
        let ret:boolean = true;
        let result:any = await this._SQLiteService.exportToJson("full");
        console.log('---> result fullexportToJson ',result);
        if (Object.keys(result.export).length === 0)  ret = false;
        const jsObj: string = JSON.stringify(result.export);
        result = await this._SQLiteService.isJsonValid(jsObj);
        if(!result.result) ret = false;

        console.log('****** import ******');
        let impresult:any = await this._SQLiteService.importFromJson(jsObj);
        if(impresult.changes.changes === -1 ){ ret = false;}

the exported json:
jsonObject  {"database":"test-sqlite","encrypted":false,"mode":"full","tables":[{"name":"users","schema":[{"column":"id","value":"INTEGER PRIMARY KEY NOT NULL"},{"column":"email","value":"TEXT UNIQUE NOT NULL"},{"column":"name","value":"TEXT"},{"column":"company","value":"TEXT"},{"column":"size","value":"FLOAT"},{"column":"age","value":"INTEGER"}],"indexes":[{"name":"users","column":"users_index_name"}]},{"name":"messages","schema":[{"column":"id","value":"INTEGER PRIMARY KEY NOT NULL"},{"column":"userid","value":"INTEGER"},{"column":"title","value":"TEXT NOT NULL"},{"column":"body","value":"TEXT NOT NULL"},{"value":"REFERENCES users(id) ON DELETE SET DEFAULT","foreignkey":"userid"}]},{"name":"images","schema":[{"column":"id","value":"INTEGER PRIMARY KEY NOT NULL"},{"column":"name","value":"TEXT UNIQUE NOT NULL"},{"column":"type","value":"TEXT NOT NULL"},{"column":"size","value":"INTEGER"},{"column":"img","value":"BLOB"}]}]}
and here is the output of importFromJson() method:
changes: {changes: -1}
message: "importFromJson command failed: import JsonObject not successful"

As a side note, I first thought, the definition of indexes in the json object, returned by the exportToJson() might not be correct and this is causing the issue.
which is:
"indexes":[{"name":"users","column":"users_index_name"}]
In my opinion it should be (could you please check this also) like this:
"indexes":[{"name":"users_index_name","column":"name"}]
Nevertheless, I also tried removing the CREATE INDEX definition from the sql statement, but the issue was still there and I had no success importing what I had just exported. Could you please check this?
I truly appreciate all your support!

@samirane, thanks for the issue, i got the fix but it is more than one hour i tried to publish it without success time out issue so hopefully it will come soon

@samirame the new release<EMAIL_ADDRESS>is fixing the issue

Thanks for the new release!
However, it seems to me there is still some problem. In the generated json, for the indexes, the "column" value is always set to the string "name".
this is what I actually get:
"indexes":[{"name":"table1_index_wrd_fk","column":"name"},  {"name":"table1_index_cat_fk","column":"name"},{"name":"table1_index_typ_fk","column":"name"}]}
this is what I expect to get:
"indexes":[{"name":"table1_index_wrd_fk","column":"wrd"},  {"name":"table1_index_cat_fk","column":"cat"},{"name":"table1_index_typ_fk","column":"typ"}]}
seems that "name" is the hardcoded value for the column now ;)

@samirame Thanks for your contribution to the debug process. This is hopefully solved in release 2.1.0-6 of the plugin. Do not hesitate to come back if any think else is wrong.

@jepiqueau Hi.
I'm getting the same error trying to import the export JSON as per the original issue using version 2.1.0-7, however, I don't have any table indexes and the JSON string is passing validation using the isJsonValid method.
The JSON string is 18423 characters long. Is there a limit on the import?
Additionally, the table and column names have capitalisation. Would this be likely to cause the error?
Thanks!

@mpat8121 Hi, thanks for using the plugin, the JSON string has for limit the limit for a string which is different on IOS and Android. The problem is coming from capitalization in column names see my answer to issue #25 Android sqlite recognize capitalization letters not ios sqlite. so you should not use capitalization letters in your column names. I may be dealing with that later, but it will slow down the query process of IOS. Is Capitalization letter matter for you?

@jepiqueau Thanks for getting back to me so quickly & for the plugin.
This issue is occurring on an Android emulator. I have been able to import JSON with capitalisation in both table and column names. It only seems to be after exporting then trying to re-import that it fails.
I'll re-write it to be lowercase.

@mpat8121 Going lowercase is better, i will have a look later as i am busy working on the capacitor-video-player

@jepiqueau Thanks. I have changed over the SQL schema to be all lowercase but I am still getting the same error.
The JSON string is 19553 characters long so I don't believe it should be exceeding the maximum allowed.
What other information can I provide you to make this easier to identify?

@mpat8121 Can you provide the schema of the database as well as the number of data in each table.
Is it a full export? Can you provide the error and on which line of the code it is happening. Did you had any chance to test it on IOS to see if you get the same error. Please provide me any data that allows me to reproduce. I thought it was due to the Capitalization but it seems not so i need to dig out. Many thanks  for participating to the debugging process, i am sorry for this but testing is not an easy task

@jepiqueau I've attached the object that is being imported via a service during app initialisation (Angular 8+ & Ionic 5). This same data is exported and errors when imported.
It is a full import initially and a full export.
I am having trouble getting the app to hit breakpoints within the vendor.js to identify which line(s) it is failing on within this plugin. The response object is:

changes: {changes: -1}
message: "importFromJson command failed: import JsonObject not successful"

I will spin up the iOS version and let you know if it is any different.
schema.txt

@mpat8121 First in the schema.txt you have to modify
       { foreignkey: "siteid", value: "REFERENCES Sites(id)" },
as
       { foreignkey: "siteid", value: "REFERENCES sites(id)" },
you have to do the same for "REFERENCES Defects(id)"as "REFERENCES defects(id)"
Now the issue is that for an import full as of to day you must have for each table the definition of the schema and data in the jsonSQLite Object so if you do this :
so you have to split in to import but not like you do by splitting table schemas and table data but
the first import with mode "full" having the table areas, elements, issues with their schema and data an do a "partial" import with all the other table schema definition.
I understand that the way you did it can be interesting but i will have to revisit the code to take this methodology into account. This will be look at later when i will have completed the work on the capacitor-video-player. Hope that for the time being you can cope with the present solution for a while

@mpat8121 i tested it for both ios and android and it works

@jepiqueau Thanks for the hints in the right direction. I think I've figured this out that even though the import of the tables (the second partial import) always returns changes: {changes: -1} message: "importFromJson command failed: import JsonObject not successful" the tables themselves are actually created.
It looks to me like the importFromJson method in SQLiteDatabaseHelper.java (specifically from line 648) always attempts to create data if the table was successfully created but then returns changes: -1 if there are no value arrays to import. This was leading me to think it hadn't worked when it in fact had.

@mpat8121 You right, i will go through the complete process again soon, with a more exhaustive testing including your original case. the success = false was added at a certain stage invalidating the originals tests where you could have schemas without data. Give me a bit of time

@jepiqueau There is no hurry. I have it now working with this understanding. Thanks again for the assistance.

@mpat8121 This is now fixed in release 2.1.0. Capitalization characters in field name are also fixed enjoy...

@jepiqueau Thanks! Everything seems to be working perfectly. I was not expecting you to re-work this so quickly 😮

@mpat8121 thank you for the feedback you are welcome


Add time range to observations

Useful to indicate a temporal window within the video. See also #219.

This will be solved with the new data model proposal #314 with adding columns eventStart and eventEnd. Can we close this one?

See specifically commit 59cc20a496b40c4cdd6cd4c7f09547717dc0d082 to see how eventStart/eventEnd relate to Audivisual Core