Case insensitive Name and Category

Changes

[x] Change Name and Category fields of Product to be case insensitive
[ ] Test model Name and Category
[ ] Test FindCommand

Related Issues/ PRs

Fixes #200
Fixes #206

Codecov Report

Merging #221 (8d051fb) into develop (d15d2eb) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop     #221      +/-   ##
=============================================
+ Coverage      58.50%   58.52%   +0.01%
  Complexity       635      635
=============================================
  Files            128      128
  Lines           2492     2493       +1
  Branches         248      248
=============================================
+ Hits            1458     1459       +1
  Misses           926      926
  Partials         108      108

Impacted Files
Coverage Δ

.../main/java/seedu/ibook/model/product/Category.java
83.33% <100.00%> (+1.51%)
:arrow_up:

src/main/java/seedu/ibook/model/product/Name.java
81.81% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Reduce the size of glyph fonts

Clean-up the code by removing unused helpers-glyphs.
Remove the use of Font-awesome and instead create a customised font set using glyphter.

Removed redundant glyph references. But due to FA's very successful 2016 kickstarter, have decided to keep font-awesome hosted on a CDN instead of glyphter.


Websocket.frame how to create constructor or any property?

my data payload morethan 20kb and neary oversize .Can i use Websocket.frame for split my payload data How do i?

There is documentation for that. Please read the doc.


Publish timeout when a nats streaming server is restarted

I'm experiencing timeout errors from a nats streaming client when a nats streaming server is restarted. I'm not sure if it's a matter of misconfiguration (a server is started in a docker container) or an error with a library itself. What can be done to avoid this problem?
For a reproduction scenario:
Start a nats streaming server using docker container
docker run -n natss -d -p 4222:4222 -p 8222:8222 nats-streaming

Run a sample publisher
var stan = require('node-nats-streaming').connect('test-cluster', 'test');
stan.on('reconnecting', () => console.log('reconnecting'));
stan.on('reconnect', () => console.log('reconnect'));
stan.on('disconnect', () => console.log('disconnect'));

setInterval(() => {
  stan.publish('foo', 'Hello node-nats-streaming!', function(err, guid){
    if(err) {
      console.log('publish failed: ' + err);
    } else {
      console.log('published message with guid: ' + guid);
    }
  });
}, 3000);

Restart a nats streaming server container:
docker restart natss

Output:
published message with guid: S0C2ORNCLYBL7059JCLPET
published message with guid: S0C2ORNCLYBL7059JCLPKI
disconnect
reconnecting
reconnect
publish failed: Error: stan: publish ack timeout
publish failed: Error: stan: publish ack timeout

It looks like I've tested with an older version - after updating my dependencies to the latest version I'm not experiencing this behavior anymore. Sorry for the confusion!

Regardless of the version, you are running a server without a persistent store, so no state will be recovered after the server restart. Actually with latest server and node client you should get a notification that the connection is no longer valid.

@kozlovic It's very helpful - I didn't expect persistent store options to influence this. Thank you.


composerModel API: return multiple outputs from forward

I am training a bert-like model in which the output of the model returns two vectors a class vector and a sequence vector.  The class vector is put through a separate linear head and then I want to calculate a total loss summing up the masked loss from Bert and the loss for the linear head.  Looking at the documentation I am unclear how I would do such a thing using the ComposerModel API.  It seems the output of forward is a class called output, but what do I do if I have two outputs?  Also I would like to log separate  metrics for each output e.g. accuracy and AUC for each of the tasks.  Is this sort of thing possible in Composer?  I have all of the logic written as a PyTorch lightning module if it would help to debug further.  I didn't see any multi-task learning examples so maybe this isn't possible with Composer?

Hi @cmlakhan, when defining a ComposerModel, you control your forward function and loss function (see the docs for more detail https://docs.mosaicml.com/en/latest/composer_model.html). The output of forward is just passed to your loss function, so you should be able to define how it gets interpreted to compute the loss.

What would be the analogue to outputs in the loss function if I return 2 tensors in forward?
class ResNet18(ComposerModel):
def __init__(self):
    super().__init__()
    self.model = torchvision.models.resnet18()

def forward(self, batch): # batch is the output of the dataloader
    # specify how batches are passed through the model
    inputs, _ = batch
    return self.model(inputs)

def loss(self, outputs, batch):
    # pass batches and `forward` outputs to the loss
    _, targets = batch
    return F.cross_entropy(outputs, targets)

Sorry, I don't quite understand. outputs in the loss function is just exactly what you return from forward.

sorry if I am being dense, but if I have a model like this
def __init__(self):
    super().__init__()
    self.model = model()

def forward(self, batch): # batch is the output of the dataloader
    # specify how batches are passed through the model
    inputs, _ = batch
    vec1, vec2 = self.model(inputs)
    return vec1, vec2

how would I access vec1 and vec2 in my loss function would it be output[0] and output[1] or something else?

So is the correct way to do things?  sorry just trying to make sure I understand the function inputs for loss

class model(ComposerModel):

def __init__(self):
    super().__init__()
    self.model = model()

def forward(self, batch): # batch is the output of the dataloader
    # specify how batches are passed through the model
    inputs,_ ,_ = batch
    vec1, vec2 = self.model(inputs)
    return vec1, vec2

def loss(self, vec1, vec2, batch):
    # pass batches and `forward` outputs to the loss
    _, targets1, targets2 = batch
loss1 = F.cross_entropy(vec1, targets1)
loss2 = F.cross_entropy(vec2, targets2)
return loss1+loss2

I think you want something like, let me know if that works for you!
class Model(ComposerModel):
    def __init__(self):
        super().__init__()
        self.model = model()

    def forward(self, batch): # batch is the output of the dataloader
        # specify how batches are passed through the model
        inputs,_ ,_ = batch
        vec1, vec2 = self.model(inputs)
        return vec1, vec2

    def loss(self, outputs, batch):
        # pass batches and `forward` outputs to the loss
        _, targets1, targets2 = batch
        loss1 = F.cross_entropy(outputs[0], targets1)
        loss2 = F.cross_entropy(outputs[1], targets2)
        return loss1+loss2

Thanks, will try this out and let you know

@cmlakhan wanted to check to see if you were able to resolve this?

You can close it and I will reopen when if I have issues.

Got it. Please feel free to reach out if you come back to this!


Gift Wrapping Algorithm

Description
Gift Wrapping Algorithm is used to find the convex hull (smallest convex polygon that covers set of points) of a given set of points. In this algorithm, the leftmost point is taken and then the next point to be connected to the hull is chosen if all the points are to the right of the line formed by joining the previous and the chosen point. This point is found in O(n) time complexity. This algorithm has a complexity of O(nm), where n is the number of given points and m is the number of points on the convex hull.
@Lakhankumawat I'm a GSSoC 2022 participant. Please assign this issue to me.
Domain
Algorithm
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

@Lakhankumawat Can assign this issue to me. Thank You.

please have look at this @iamgunjan

assign to me

hi @Lakhankumawat please assign it to me


Fixing the scoring system

it seems that the score and the collisions keeps increasing when the rock or the flower is in contact with the main character
i can make it so that they only increase by one

@dhruv6-6 Assigned

Pull Request merged successfully 🎉


No item is getting pasted from Maccy

Hello,
I have iMac M1 and got the Maccy App. Maccy seems unable to paste a text that I copied earlier. Command + Shift + C shows a list of item but when selecting one, nothing is happening and no text is being pasted. I tried that with different applications.
I am using Maccy 0.20.1 running on iMac M1. Let me know how I can help.
Thanks.

You have to enable "Paste automatically" in preferences first.


Add precompilation for Pluto notebooks

Codecov Report

Merging #4 (f5d22cf) into dev (9a2f852) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff          @@
##             dev      #4   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          1       1
  Lines          2       8    +6
=====================================
- Misses         2       8    +6

Impacted Files
Coverage Δ

src/HelloJulia.jl
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


[BUG] Installation Error 137 with MIB Toolbox 4.1

On Year 2019 and 2020 i can't install the MIB2Toolbox on my MIB2HIGH. It has firmware 0687. It always comes an METAINFO error.
I tried the manual installation but i have no root password.
Now i tried the new MIB2Toolbox 4.1 with the new installation method.
This installationpoints are successful:
If you've installed a previous version (before V4.0) of the toolbox: clean your SD-card before trying to install.
Download all files from the repository. This can be either as a git clone or "Download zip" from github then extract the zip.
Put all files and folders on an empty SD-card, preferable >1GB.
Put the SD-card in one of the slots of your MIB2-unit.
Make sure there's only 1 SD-card in your unit, otherwise the scripts don't know where to look.
Hold the MENU button on your MIB2 until the service screen appears.
Select the "Software updates/versions" menu, then hit the "Update" button in top right corner.
Select the SD-card and select MQB Coding MIB2 Toolbox.
But when my MIB2HIGH want to install the Toolbox i only get an error message:
Error: Gerät
Error: 137
Detail: 0
Gerät: MQB Coding MIB2 Toolbox: 0
I hoped after 2 years i can now install the MIB2Toolbox. But now i think i can never install the MIB2Toolbox on my MIB2.

Error 137 means there is a metainfo checksum error... maybe we've an failure in the curretn files... will check.

I found a checksum mismatch for the finalscript folder. Please replace metainfo2.txt and try again. You maybe need custom SWDL now to update the same module again. See commit: https://github.com/jilleb/mib2-toolbox/commit/3bb538c7f8a7334cb7a00c50f5b9cee91c0b4c2b

I tried it again. But there is the errorcode 137 again. I think i can't install any updates, because the unit is a retrofitted unit.

should also work on it.
Can you please post you Train and MU version.
Do you get the 137 on Toolbox or Finalscript installation?

I get the 137 on Toolbox.
SW Train: MHI2_ER_VWG11_P3250
SW MU: 0687
TN: 3G0035021D
Software-Version: H28.28.62_1_HIGH2_EU

This is odd... could be a problem with line endings in the code... you somehow edited the files yourself?

No i have never edited the files myself.  Because i don't have the root password. I searched many times in the Internet to find the correct password. But i never found it. Thats the reason because i can't install the toolbox manually. I think i can never install your toolbox on my mib2 unit without flashing a complete New firmware.

Maybe try with Ca2MewGI or UaIIm8IY maybe they work

Thank you very much for your help. I will try it tomorrow or friday i give you a Feedback.

It seems (nor surprisingly) that branch merging can often throw out the checksums, we might want to look at changing the merge behavior for the repo.
I've just fixed the checksums and pushed directly to master so if you download a fresh copy now it'll hopefully be fixed.

Andrew you didn't changed much. I already edited it before. You now just made the checksum in uppercase. So I don't think this will resolve this issue.

Oh yeah, good point - I hadn't noticed that all the tool changed. Perhaps the issue is how old the firmware is.

I tried it again. But there is the errorcode 137 again. I think i can't install any updates, because the unit is a retrofitted unit.

This being a retrofit the unit must be patched for activations and/or component protection. There is a good chance the root password was changed and/or the public keys were altered and therefore the Metainfo signature check fails.
You can try to run the original 2021 map update to see if that gives no errors. Don't go through with the install in case you don't have the activation for them!

So I just released v 4.1 and also uninstalled everything from my unit before. On a stock unit, non patched, this is working all fine.
So this problem seems to be something related to your retrofitted unit which we can't account for.

does this still relay on POIUpdate to install, or could I finally install it on my MIB2 standard without Navigation?

No you can't. This isn't compatible with standard units.
There is currently a toolbox for standard in the making but that one also is only possible an units with nav.

How did you solve the error code : 137?


[es6] support PropertyName : BindingElement syntax in ObjectBindingPattern

Brought up by ato in #jshint on freenode. This might be a duplicate but it should be easy to implement

is that just this?
let { a: b } = { a: 1 }; // b = 1

If so, that passes in 2.8.0.
If not @nicolo-ribaudo has made alot of enhancements in master.

PropertyName can only be an IdentifierName, a StringLiteral, a NumericLiteral or a ComputedPropertyName. The first one is already supported, the other ones are implemented by #2495.
So this issue can be closed.

That's merged, so closing. will be in release after 2.8.0


Fixes in Adjust plugin

fix: type and null check custom event tokens
fix: Handle failed segment settings parsing for Adjust Plugin

@QuirijnGB thank you for the PR!


Cannot create account at poshtestgallery.com

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest version
[X] Search the existing issues.

I cannot create an account at poshtestgallery.com.
Steps to reproduce
Create account at poshtestgallery.com with email or username which are already exists at powershellgallerry.com
Expected behavior
I could login to my account from the real powershellgallerry or create a new at poshtestgallery.
Actual behavior
Throws error when I try to login to my Microsoft account.
Error details
Oops!

Error 500 - Internal Server Error

An error occurred while processing your request. We really messed up this time...

We have logged the error and will look into it so that it doesn't happen again; however, if you keep seeing this page, please file a bug!

Get me out of here! Go home

Wondering if PowerShell Gallery is down? Check our status

Looking for a package? Try searching

Package you're looking for doesn't exist? Make one

Pretty sure we messed up? File a bug

@rinrab poshtestgallery is not a public repository, it is just a test environment for PsGallery updates-- only used by our team

Ou, sorry. I thought that this is a powershell gallery to test my module before publish. I think it would be great to have the same gallery for example for testing publish scripts and having nighty-built modules.


Pin down the type of HttpHeaders.ContentMd5

std::vector<uint8_t> or ContentHash.

The name should be ContentHash, the type is also struct ContentHash


fix: s3 endpoint url field while importing json

Description
Before the merge of this PR, the entire datasourceConfiguration object in the exported app json was set as null.
Example JSON for s3 plugin earlier :
{
  "datasourceList": [
    {
      "name": "S3",
      "pluginId": "amazons3-plugin",
      "messages": [],
      "isAutoGenerated": false,
      "deleted": false,
      "gitSyncId": ""
    }
  ]
}

Post the merge, the datasourceConfiguration.properties field was removed and the datasourceConfiguration.endpoints field was kept populated.
JSON for s3 plugin post the change :
{
  "datasourceList": [
    {
      "datasourceConfiguration": {
        "endpoints": [
          {
            "host": ""
          }
        ]
      },
      "name": "Untitled datasource 1",
      "pluginId": "amazons3-plugin",
      "messages": [],
      "isAutoGenerated": false,
      "deleted": false,
      "gitSyncId": ""
    }
  ]
}

Due to the above change the S3 plugin form.json condition here started displaying the field.
This was also due to a combination of a pre-existing error in the form.json, which on importing the app json showed the first dropdown value from the S3 service provider, but did not initialised the value.
On changing the dropdown the value was automatically assigned.
https://github.com/appsmithorg/appsmith/assets/25542733/78d5f693-11fc-4bd6-8cd7-90ee300b2b58

This PR updates the form JSON to initialise the default value which leads to the amazon-s3 plugin getting auto-initialised with some default values.

PR fixes following issue(s)
Fixes #29408
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Bug fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

Refactor

Improved the handling of sensitive information in data source configurations to enhance security.

Tests

Updated tests to align with the new logic for data source configuration properties.

/ok-to-test

/ok-to-test

/build-deploy-preview skip-tests=true recreate=true


wg_engine: integrate with thorvg player

[x] Research and prototype the Emscripten build with WebGPU.
[x] Implement the necessary integration logic in the tvgWasm part.
[ ] Integrate with the ThorVG player.
[ ] Support Multi-Canvases. (Multiple ThorVG Players)

see also:

https://eliemichel.github.io/LearnWebGPU/getting-started/hello-webgpu.html
https://github.com/gfx-rs/wgpu-native/tree/trunk/examples/capture

    {
        WGPUSurfaceDescriptorFromCanvasHTMLSelector fromCanvasHTMLSelector;
        fromCanvasHTMLSelector.chain.next = NULL;
        fromCanvasHTMLSelector.chain.sType = WGPUSType_SurfaceDescriptorFromCanvasHTMLSelector;
        fromCanvasHTMLSelector.selector = "canvas";

        WGPUSurfaceDescriptor surfaceDescriptor;
        surfaceDescriptor.nextInChain = &fromCanvasHTMLSelector.chain;
        surfaceDescriptor.label = NULL;

        return wgpuInstanceCreateSurface(instance, &surfaceDescriptor);
    }

WGPU / DAWN – which backend is currently used in the development of the wg_engine?

WGPU / DAWN – which backend is currently used in the development of the wg_engine?

Currently we are using wgpu_native pre-builds
Please, see pre-buils
https://github.com/gfx-rs/wgpu-native/releases

Hello, @SergeyLebedkin.
I'd like to discuss interface for WebGPU initialization part. Since current methods don't support browser to have multiple canvases.
According to wg_engine, WGPUAdapter and WGPUDevice are generated when calling WGContext::initialize. Which means those Adapter and Device could be generated in duplicate.
It's going to happen when we have multiple animations or try to resize more than once.

This function is called, following the steps:

wasm::resize -> canvas::target -> renderer::target -> context::initialize -> then the WGContext potentially tries to call adapter and device by multiple.

And this is basic WebGPU usage in multiple canvas, it's written in JavaScript, but you can understand the logic.
https://webgpufundamentals.org/webgpu/lessons/webgpu-multiple-canvases.html

As you can see, the adapter and device are initialized in the head of code and reused. Even if they have more than one canvases, the code guarantee it's called only once.
Seeming like browser returns out of memory error, specifically when we send requestAdapter/Device calls again.
Hence in my opinion, we should guarantee that the one page has only adapter and device.

WGPUInstance, WGPUAdapter, WGPUDevice should be singleton instance and shared by multiple players.
We will take reusable WGPUInstance, WGPUAdapter, WGPUDevice approach.
Request adapter, request device should be called while instance and surface are being ready from out of WGContext.

To do this, these methods will be affected, or something more else.
Result target(void* instance, void* surface, **void* adapter, void* device**, uint32_t w, uint32_t h) noexcept;

void WgContext::initialize(WGPUInstance instance, WGPUSurface surface, **WGPUAdapter adapter, WGPUDevice device**)

Let's disuss :)
Just for sharing, I've succeed to draw multiple animations by changing the interfaces I mentioned. The only thing I need is to have reasonable interface for the change.

Please, take a look on https://github.com/thorvg/thorvg/pull/2695

@tinyjin Could you please confirm whether overlaying other elements on the WGPU canvas is working properly?

@SergeyLebedkin Please double-check that the current WGPU canvas doesn't work properly with a transparent background.

Yes, @hermet. It's doable.
HTML element overlay including the text goes with WebGPU well.

@hermet

Please double-check that the current WGPU canvas doesn't work properly with a transparent background.
i.e) background should be transparent, (not black)

Do you mean in system apps or in browser? In any case by default alpha channel in the target buffer are 0

@SergeyLebedkin Yes but, it doesn't look like it on web.  @tinyjin the masking sample the above, the background color should be white, doesn't it? Could you please confirm this?

Please double-check that the current WGPU canvas doesn't work properly with a transparent background.
i.e) background should be transparent, (not black)

For a proper fix transparent bg emsdk should fix wasm workflow
See https://github.com/emscripten-core/emsdk/issues/1443

@tinyjin Hello, the fix looks went into the emscripten core main, could you please check & try with this change in emscripten locally?  https://github.com/emscripten-core/emscripten/pull/22494

@hermet @SergeyLebedkin
Confirmed the update(https://github.com/emscripten-core/emscripten/commit/c82a307c612d430028a3d06ddf8b3fdd2852b2fb) enables WebGPU with premultiplied alpha mode in the browser.

Tossed the issue to the ThorVG web project. => https://github.com/thorvg/thorvg.web/pull/49


Add a compulsory header to every JSON file

https://groups.google.com/forum/#!topic/bids-discussion/TjxOKEB1DD4
add a 'header' block to every JSON file that specifies the type of parameter file that it is and a spec version. This makes it a bit easier to verify that the file is what you think it is. This is particularly useful due to the very few mandatory keys in the files.

Original authors: @ccraddock

Hey @tsalo if you're not part of the group you unfortunately can't see the discussion.

I don't think that's a fixable issue, unfortunately. But the BIDS google group should be open-join. I know it's not optimal to require people join a google group to access the original discussion, but it's best route available, I think.

add a 'header' block to every JSON file that specifies the type of parameter file that it is

How would this be different than using the information that is present in the JSON file name and especially the suffix, i.e. T1w or eeg, just before the extension.

... and a spec version

Would this open the possibility for having filees within one dataset complying to different versions of the specification?

Hey @tsalo if you're not part of the group you unfortunately can't see the discussion.

I just tried in a private browser (not logged in or anything) and I could see the discussion :thinking: the settings for the bids-discussion google group are "open", so I don't know what went wrong
Screenshots:

Hm, today I don't get the same error. Weird. Thanks for checking on it.

I think this issue is at large addressed by introducing the schema to BIDS. Although indeed every file could potentially point to the schema and its version, ATM we rely on BIDSVersion to be specified dataset wide and IMHO it is sufficient.  Since there were not much interest in this issue, and because I think it is principally addressed, I will just close. Please reopen if disagree


ci: Fix ffmpeg install failure for Linux+Firefox

CI for Firefox on Linux is failing because of (I suspect) an outdated
package index in Ubuntu.  This addresses it by updating the package
list before installing ffmpeg.

This appears to be working.  In other recent test runs, Linux+Firefox failed right away on apt install ffmpeg.


update document for windows build with vs2019

This PR tends to update window build document for the support of vs2019.

vs2019 directory copy is no longer needed, and document is updated by https://github.com/sony/nnabla/pull/1179 .
So, close this PR.


Problem red hat virtio scsi disk device 6.3.9600.18758

Hello,
We have a cloud environment that consists of hosts backed by KVM hypervisors. About half of the virtual machines are running Windows Server operating systems (2008R2, 2012R2, 2016, 2019), there are hundreds of such instances and almost all of them use VirtIO drivers (mostly 0.1.160).
Sometimes (it occurred about 3-4 times) we encountered the following glitch: a guest operating decides that its primary disk storage has more size than it actually is. For example, an instance had a virtual drive of 200 GB, it had been worked fine for years, but at some moment (no one knows at which exactly one) primary partition (we mean "drive C:", which usually is the 2nd one, as the 1st one is being used by the operating system) became the size of 210 GB just out of the blue. After that, the system event log started growing with the following error messages: The driver detected a controller error \ Device \ Harddisk0 \ DR. Obviously, it happens when the operating system tries to write pieces of data to the sectors that don't exist.
Once we expand this virtual drive to 210 GB, the error messages don't appear anymore. Still, after that we find some part of the data corrupted (maybe some fragments of files are being stored to the non-existent sectors), so it seems to be a real problem for us when it happens.
Alas, we didn't find a way to reproduce that. As we stated before, it happened only 3-4 times, though each time the outcomes are quite unpleasant.
Should we provide with more data regarding this issue? Should we consider upgrading the driver? Perhaps, we just don’t know that it’s a bug that had been fixed after the 0.1.160's release? Just curious, did anyone send a similar bug-report before? We tried to find them, though with no luck.
Thanks in advance for your feedback.

Please see my comments at
https://bugzilla.redhat.com/show_bug.cgi?id=1879396#c1
Cheers,
Vadim.

Closing based on the comment https://bugzilla.redhat.com/show_bug.cgi?id=1879396#c16
Best regards,
Yan.

Closing based on the comment https://bugzilla.redhat.com/show_bug.cgi?id=1879396#c16
Best regards,
Yan.


Cannot parse OTP Release version string: 17 on windows

Hi!
On my Windows 7 system function call erlang erlang:system_info(otp_release). show "17", but regexp on
https://github.com/rebar/rebar/blob/master/src/rebar_require_vsn.erl#L126 not support this format and i can not compile app. This error occurs only if configure erlang {require_min_otp_vsn, "R14B03"}

We've updated rebar_utils:otp_release/0, but it looks like we forgot to do the same for rebar_require_vsn.erl. Would you like to submit a patch? If you do, you'd have to update the tests in https://github.com/rebar/rebar/blob/master/test/rebar_require_vsn_tests.erl as well.

see #587

Closing since #587 as since been merged


.gitignore not interpreted correctly?

Version
$ ag --version ag version 0.31.0
Files and contents in minimal example
$ cat .gitignore
*
!yes_search_this

$ cat yes_search_this
somethingthing

$ cat no_dont_search_this
something

Behaviour
$ ag -U something
yes_search_this
1:somethingthing

no_dont_search_this
1:something

$ ag something

(i.e. no output from the last command)
Expected behaviour
$ ag something
yes_search_this
1:somethingthing

+1 for this. It seems that negated git ignore patterns are not interpreted correctly.
Would it make sense to re-use some code from git to ensure this is done correctly? See for example https://github.com/git/git/blob/master/Documentation/technical/api-directory-listing.txt

+1

+1
I have noticed that adding a ./ before the file name fixes it for the silver searcher, but this breaks the gitignore from actually working :confused:

:+1:

This fits the subject: .gitignore not interpreted correctly, but does look like it isn't exactly the same issue. Let me know and I'll open a new issue instead:
# ag finds something inside debian/install when run from here
git/dist> ag BLOCK:
debian/install/file/path/RunMySQL.pm
35:BLOCK: More code to remove

# but doesn't when run from here
git/dist> cd debian/
git/dist/debian> ag BLOCK:

# .gitignore is setup in a way that git understands
git/dist/debian> grep '^\/install$' .gitignore
/install

So it seems this is another aspect of .gitignore files not being interpreted correctly. Or at least not the same way that git interprets them, which I'm assuming is the point.

Another (presumably faster) way to "resolve" this could be to document exactly what subset of .gitignore syntax is supported and which not. And (optionally?) issue warnings about unsupported .gitignore syntax.

Another example of it not quite matching the gitignore spec:

/node_modules/ is a valid gitignore pattern which will ignore a directory called node_modules only if it is at the root of the project and not if it is one or more levels deep, it is ignored by ag
node_modules/ is a valid gitignore pattern which will ignore a directory called node_modules wherever it exists in the directory structure, it is picked up correctly by ag

+1
Here is another example where all files in /src are ignored:
/src/*
!/src/app

Just to add my 2 cents on the matter:
I usually make a gitignore like this:
*
!*.c
!*.h

Because I usually end up with more trash files than source files during development.
For individual, non-source files like the README, I don't bother putting them in the .gitignore and directly git add -f it. This is not a problem as files that are versioned bypass the .gitignore pattern.
If this does not slow down the initialization too much, I would suggest adding to the ! pattern list individual files from git ls-files.
Correctness against the behaviour of git status should be enforced if a .gitignore is to be interpreted.

This is a duplicate of #168.


Splash screen patch not working on linux

Reported by @juddey (https://github.com/infinitered/ignite-ir-boilerplate-bowser/pull/48#issuecomment-396789803)

Hello,
excuse me jumping on this PR and all that.... I've cloned bowser locally, checked out this branch and run ignite new.
When I name the projects with only lowercase in the name, I can build the project, however when I CamelCase the project name it fails. Heres a picture:

It could be a red herring due to my setup but thought I'd let you know anyway. I'm on ubuntu 16.04 if that makes any difference. ☮

I have a similar issue.

@wayson I believed it's fixed in the latest version. Could you test it?
@juddey Are you still having this issue?

@leonskim Yeah, it's working now. Thanks mate.

@juddey Let's do that. Feel free to open a new issue and tag me please. Thanks!

@leonskim I'm getting this as well. I'm on Ubuntu 16.04 and did ignite new PizzaApp and chose Bowser.


ttest_from_stats warning dependent on size of passed array

This is a fairly specific bug. If you pass scipy.stats.ttest_ind_from_stats a sample with equal mean and zero s.d. in an array of length > 1, it will throw warnings. If you pass a sample with equal mean and zero s.d. by itself, you won't get a warning. The warning comes from operations in stats.distributions.t.sf.
Reproducing code example:
from scipy import stats
import numpy as np

# This does not throw an warning
stats.ttest_ind_from_stats(
    mean1=np.ones(1), std1=np.zeros(1), nobs1=10,
    mean2=np.ones(1), std2=np.zeros(1), nobs2=10,
)

# This throws a warning
scores, pvals = stats.ttest_ind_from_stats(
    mean1=np.ones(2), std1=np.zeros(2), nobs1=10,
    mean2=np.ones(2), std2=np.zeros(2), nobs2=10,
)

Warning
/usr/local/lib/python3.7/site-packages/scipy/stats/_distn_infrastructure.py:901: RuntimeWarning: invalid value encountered in greater
  return (a < x) & (x < b)
/usr/local/lib/python3.7/site-packages/scipy/stats/_distn_infrastructure.py:901: RuntimeWarning: invalid value encountered in less
  return (a < x) & (x < b)
/usr/local/lib/python3.7/site-packages/scipy/stats/_distn_infrastructure.py:1892: RuntimeWarning: invalid value encountered in less_equal
  cond2 = cond0 & (x <= _a)

What I think should happen
I think that this behaviour should be consistent, an not depend on the size of the passed array. I also think this probably shouldn't throw a warning. There's already handling done to not warn the user if equal s.d.s are passed in scipy.stats.stats._equal_var_ttest_denom and  scipy.stats.stats._unequal_var_ttest_denom.
Scipy/Numpy/Python version information:
This also occurs in the v1.2.1 releases of scipy.
scipy==1.3.0 numpy==1.16.4
sys.version_info(major=3, minor=7, micro=3, releaselevel='final', serial=0)

Neither of these raise a warning for me.
@swallan can you test this?
If it's OK for you, I think we can close.

They seem to run just fine for me as well.
I'll close as a result.


Feature Request: Retain Audio File

Would it be possible to add a toggle to keep the audio file, maybe linked in the document?

Yes it would we really useful as a back-up in case of many problem there is no track of the recording

Adding to this, it might be nice to be able to transcribe an existing recording as well - which would enable this workflow.

Yes, I agree with all of this. If it saved the recorded audio and had the option to transcribe existing audio files would make this perfect!

Yes I have been looking for that feature as well! It would be great to have the audio file saved to the vault and linked to the transcription. I could then listen to my own voice reading it out loud. Currently this is just lost, although I put in the effort.

Yes! Retaining recordings and transcribing existing recordings is what this plugin misses the most. (Especially that the other Scribe-based plugin apparently charges by several orders of magnitude more money than OpenAI itself does) It would also be an excellent failsafe for when Whisper returns an error instead of the transcription.

I hear you, folks! Will get this done in the next few days!

I test the plugin and stop using it after a day because the recording where lost. Really a big improvment if done ! thanks

Add this functionality in the latest release
https://github.com/nikdanilov/whisper-obsidian-plugin/releases/tag/1.5.0


Pick a topic

Come up with a topic for your project. Discuss the topic with your TA or lab instructor and proceed only after your topic has been approved by one of them.

Here is my first idea:
Package: mdslabutils
A package to help MDS students manage their labs in iPython notebooks and R markdown.

Function 1: Parse an MDS lab (.ipynd or .Rmd) and determine how many optional and required points there are. Return a table of totals.
Function 2: Perform common checks:

Check that you have included a Github repo link
Check that you have pushed the latest version
Check that you have at least three commits
Check that there is an HTML version. Maybe check it matches the latest committed version (probably difficult).
(Maybe) Check that there is a version on canvas
Function 3: List which questions have been left unanswered.

Creating folder structure for R and Python projects
function 1 would be creating folders for the Python project
function2  would be creating folders for the R project.
We can add templates for a Team contract, contribution and code of conduct files as well in these functions

Making our own NN package:

Loss Function
Forward Propagation
backward propagation

Depending on the complexity of package, we can add more function to improve the package. The problem with this is it would be challenging to implement it in R.
The second suggestion can be a package for wrangling data. This helps to have the same functionality between R and Python.

Cleaning header name
dropping NA
Imputing data with several options

We decided to go ahead with the package: mds-lab-utils-py and mds-lab-utils-r.


feat: update mongo query to show active repos

Fixes Issue #8475

Changes proposed

Update mongo query in pages/api/repos.js to filter repos where repos.dates.pushedAt > the last 30 days.

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Looks good 👍
This is my first opensource contribution.
Thanks @eddiejaoude

Awesome! Keep going 👍


Fetch shop data after user is authenticated

I want to merge this change because it:

Gets rid off the unused channelCurrencies field
Delays the query after the user is authenticated, which allows to include staff-only fields (like version)

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://master.staging.saleor.cloud/graphql/

Task linked: SALEOR-4442 Permissions list is not showed on non full access group


Delegation fails with error code 1300

I am using GAM 4.11 on Linux.  We have a user who used to have delegated access to another account, but no longer seems to have it.  This user does not appear in the list of delegates for the target account.  But when I try to add him as a delegate, I get an error saying his account already exists.  In the examples below, target is the account to which delegated access is being granted, user1 is a user who has delegated access, and user2 is the user who used to have delegated access but does not anymore.
$ gam version
GAM 4.11 - http://git.io/gam
Jay Lee<EMAIL_ADDRESS>Python 2.7.12 64-bit final
google-api-python-client 1.6.1
Linux-3.16.0-4-amd64-x86_64-with-debian-8.7 x86_64
Path: /home/dparker/bin/gam

$ gam user target show delegates
Getting delegates for<EMAIL_ADDRESS>Delegator<EMAIL_ADDRESS> Delegate: Some User
 Status: ACCEPTED
 Delegate Email<EMAIL_ADDRESS> Delegate ID<EMAIL_ADDRESS>
$ gam user target delegate to<EMAIL_ADDRESS>Giving<EMAIL_ADDRESS>delegate access to<EMAIL_ADDRESS>(1/1)
ERROR: Could not create delegation - 400 - <?xml version="1.0" encoding="UTF-8"?>
<AppsForYourDomainErrors>
  <error errorCode="1300"<EMAIL_ADDRESS>reason="EntityExists" />
</AppsForYourDomainErrors>

Full steps to reproduce the issue:

Attempt to grant delegated access for this specific user to this specific target account (I don't know what caused the delegated access to disappear in the first place, so I can't give any more steps)

Expected outcome (what are you trying to do?):
The user should gain delegated access to the target account.
Actual outcome (what errors or bad behavior do you see instead?):
I get an error which indicates that Google thinks the user already has delegated access to the target account.
This is the first time I have ever seen this problem.  Any help is greatly appreciated.  Thanks!

I have also opened a support case with Google regarding this.  I will update this issue if I get a solution from them.

I heard back from Google.  They said that this error is known to happen intermittently under the current Email Settings API, and often the delegation works if you wait a day or so and try again.  They have been working to solve it, but the Email Settings API is being deprecated and most efforts have been redirected to the upcoming Gmail API, which does not have this problem.  So, it sounds like this is a bug which they never really tracked down, and have now given up on fixing because it's not present in the newer API.
Definitely not a GAM issue.

hi guys, find reason and solution :)
it happens when you want to give access to account, which was never opened - you have to open this account in browser and accept fiorst page...
then run GAM...

@jay0lee
This issue is preventing us to create delegations since, at least the 11/3/17
Any suggestion?
gam user<EMAIL_ADDRESS>delegate to<EMAIL_ADDRESS>Giving<EMAIL_ADDRESS>delegate access to<EMAIL_ADDRESS>(1/1)
ERROR: Could not create delegation - 400 -

The accounts are both active and they are not admin accounts.
Thank you!


fix wercker examples

refs #1994
Still need to think about how to improve the base class, but lets fix this one example as a first step.

Warnings

:warning:

This PR modified service code for wercker but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS


fix for rq-worker running template_parser

[Current coverage][cc-pull] is 90.06% (diff: 100%)

Merging [#115][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master       #115   diff @@
==========================================
  Files            22         22
  Lines          2526       2527     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2275       2276     +1
  Misses          251        251
  Partials          0          0

Powered by Codecov. Last update [6e82923...84c1690][cc-compare]
[cc-base-branch]: https://codecov.io/gh/CyberReboot/vent/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/CyberReboot/vent/compare/6e82923438082e690e4a0bb207a70d9420f062f0...84c1690932907142eed2ac5a2819843728cf1363?src=pr
[cc-pull]: https://codecov.io/gh/CyberReboot/vent/pull/115?src=pr


let hf trainer handle torch compile

It seems like there may be small benefits to leaving it to default on. Is there any case having it off is better?

Is there any case having it off is better?

it cannot default on because it doesn't work. if you try to train it errors out.


Move api_keys file to root directory

The readme file says to put the api keys file in the root directory but when you start up the server, it looks for the api keys file in /lib.   So we should change either where the server code is looking or fix the install instructions

It used to be in the root, but it was moved a while back. Putting it in /lib has the advantage that you can make the url "package:coUserver/..." instead of having to do a relative URL to wherever it is being loaded from. This seems better to me. I think the instructions should be changed.


fix Issue 21796 - Revert "std.file docs: Tweak dirEntries examples"

Reverts dlang/phobos#7906

Unit-tests should never invoke the compiler (think ARM, or PPC where dmd doesn't exist).
PR caused a regression https://issues.dlang.org/show_bug.cgi?id=21796

What happened here?
Why are we running code in version (StdDdoc)? Aren't many StdDdoc declaration stubs and are not meant to be compiled?

@berni44 @ibuclaw  Are you sure this is what caused the regression?

Maybe the test_extractor ignores the version (StdDoc)? (since it caused failures for the publictests target)

Ahh, that does sound plausible.

I'll check and probably raise a PR for the test_extractor


VesselLandedStateFix possible issue

Every once in a while I get an odd bounce when landing and I'm wondering if the code in this fix is causing the issue.  There's a check to to see if vessel.AltitudeFromTerrain is above 50.  When testing on the runway, this value returns 282.  During a takeoff roll, this code is triggered many times.  Should the code check AltitudeFromSurface instead?  What is the difference between AltitudeFromTerrain and AltitudeFromSurface?

The code should not be affecting any physical properties of the craft at all, only the "theoretical" state that the game considers the vessel to be in, so I don't think this could be causing any sort of bouncing. But to make sure, could you please try to test this with and without the fix enabled?
However, you are correct in that I should be using AltitudeFromSurface for better accuracy, thanks for letting me know, I'll switch to that in the next update.

After checking this again, I cannot verify any sort of correlation between any physical occurrences such as landing gear bounces and this fix. The AltitudeFromSurface change will be implemented in the next release.


a way to get container width with not renderable children ?

hi , existe a method or a arg to pass, for get a width of a pixi container, also if all children have renderable false.
i look for method: getBounds or calculateBounds, but i not found way.!
i need width for allow me to set the x and y with % on a container pivot.
My idea looks like this at the moment, I'd like to do more tests, but I'm blocking, because I do not have access to width, unless the whole child is visible.
animationControler_Pictures.prototype.setMotionsProps = function(setupObjProp) { //set for all motions global prop from parent animationControler_Pictures
    //prop possibility:  x, y, scale:{x:?,y:?} ,skew:{x:?,y:?}, pivot:{x:?,y:?}, alpha , blendMode , buttonMode, _mask
    for(propName in setupObjProp){ //if it scale ,skew or pivot, preCheck if as the x,y or set one of this valur by default
        if( propName === 'pivot' ){ // if assign only one, get default the one not assign
            var prop = setupObjProp[propName];
            // allow 0.? for say it need calcule %
            if( isFinite(prop.x) ){ prop.x = (prop.x % 1 !== 0) && this.width*prop.x || prop.x } else { prop.x = this[propName].x };
            if( isFinite(prop.y) ){ prop.y = (prop.y % 1 !== 0) && this.height*prop.y || prop.y } else { prop.y = this[propName].y };
            this[propName].set(prop.x,prop.y);
        }else{
            this[propName] = setupObjProp[propName];
        };
    };
    return this
};

thanks

rewrite calculateBounds of that particularContainer.
container.calculateBounds = function()
    {
        this._bounds.clear();

        this._calculateBounds();

        for (let i = 0; i < this.children.length; i++)
        {
            const child = this.children[i];

            if (!child.visible /*|| !child.renderable*/)
            {
                continue;
            }

            child.calculateBounds();

            if (child._mask)
            {
                child._mask.calculateBounds();
                this._bounds.addBoundsMask(child._bounds, child._mask._bounds);
            }
            else if (child.filterArea)
            {
                this._bounds.addBoundsArea(child._bounds, child.filterArea);
            }
            else
            {
                this._bounds.addBounds(child._bounds);
            }
        }

        this._lastBoundsID = this._boundsID;
    }

thank i will take a look, ideally I do not wish to alter the normal behavior of the container.

Container is really simple thing, and it has only a few uses. If users would find every way they want in it, it would be 10.000 lines of code, not 600. OpenFL/flash has even more lines of code in their  DisplayObjectContainer.

indeed I will try to see, I may have an idea to try.
I will see if i can get width or height, with child rendered, but container not rendered.

I think I have found my solution!
I created a method called forceCalculateBounds
I do not know if this is likely to cause me problems, but it seems to work.
I just have a bizard behavior with rotations. It does not seem to follow the mathematical logic of angle.
rotation 90 degrees from a central pivot gives me an angle of about 120 degrees.
animationControler_Pictures.prototype.forceCalculateBounds = function() { //FORCE getbound*1 also if childrens visible and renderables FALSE
    if(this._bounds.isEmpty()){
        //this._bounds.clear();
        for (var i = 0; i < this.children.length; i++) {
            var child = this.children[i];
            child.calculateBounds();
            //this._lastBoundsID = this._boundsID;
            this._bounds.addBounds(child._bounds);
        };
        this._lastBoundsID = this._boundsID;
        return [this._bounds.maxX.valueOf(), this._bounds.maxY.valueOf()];
    };
};

// pixiSSA.createAniPicture(0,'bob',5).setMotionsProps({ pivot:{x:0.5,y:0.5 } }).play('idl1')
animationControler_Pictures.prototype.setMotionsProps = function(setupObjProp) { //set for all motions global prop from parent animationControler_Pictures
    var wH = this.forceCalculateBounds(); // _________ NEED TO FORCE calculateBounds childrend for get width on non renderable child __________
    //prop possibility:  x, y, scale:{x:?,y:?} ,skew:{x:?,y:?}, pivot:{x:?,y:?}, alpha , blendMode , buttonMode, _mask
    for(propName in setupObjProp){ //if it scale ,skew or pivot, preCheck if as the x,y or set one of this valur by default
        if( propName === 'pivot' ){ // if assign only one, get default the one not assign
            var prop = setupObjProp[propName];
            // allow 0.? for say it need calcule %
            if( isFinite(prop.x) ){ prop.x = (prop.x % 1 !== 0) && wH[0]*prop.x || prop.x } else { prop.x = this[propName].x };

            if( isFinite(prop.y) ){ prop.y = (prop.y % 1 !== 0) && wH[1]*prop.y || prop.y } else { prop.y = this[propName].y };
            this[propName].set(prop.x,prop.y);
        }else{
            this[propName] = setupObjProp[propName];
        };
    };
    return this
};


How to connect memgraph replication with library neo4j

How to connect memgraph replication with library neo4j. Thanks

Hi @mjnk9xw! Could you be a bit more specific about what "library neo4j" means in your use case?


Exception: Can't handle non-square pixels (0.293388, 0.293389)

Hi,
I am trying to run ashlar as part of the mcmicro pipeline with nextflow.
However, I get a Command error related to non-square pixels.
Command error:
  WARNING: Stage coordinates' measurement unit is undefined; assuming μm.
  Traceback (most recent call last):
    File "/usr/local/bin/ashlar", line 8, in <module>
      sys.exit(main())
    File "/usr/local/lib/python3.7/dist-packages/ashlar/scripts/ashlar.py", line 184, in main
      args.quiet
    File "/usr/local/lib/python3.7/dist-packages/ashlar/scripts/ashlar.py", line 213, in process_single
      process_axis_flip(reader, flip_x, flip_y)
    File "/usr/local/lib/python3.7/dist-packages/ashlar/scripts/ashlar.py", line 302, in process_axis_flip
      _ = metadata.positions
    File "/usr/local/lib/python3.7/dist-packages/ashlar/reg.py", line 177, in positions
      return Metadata.positions.fget(self)[self.active_series]
    File "/usr/local/lib/python3.7/dist-packages/ashlar/reg.py", line 94, in positions
      self.tile_position(i) for i in range(self._num_images)
    File "/usr/local/lib/python3.7/dist-packages/ashlar/reg.py", line 94, in <listcomp>
      self.tile_position(i) for i in range(self._num_images)
    File "/usr/local/lib/python3.7/dist-packages/ashlar/reg.py", line 375, in tile_position
      position_pixels = position_microns / self.pixel_size
    File "/usr/local/lib/python3.7/dist-packages/ashlar/reg.py", line 292, in pixel_size
      % tuple(values))
  Exception: Can't handle non-square pixels (0.293388, 0.293389)

Reading the Parameters with ImageJ, we have the following values:
Width:  3705.8078 µm (12631)
Height:  4720.6434 µm (16090)
Size:  194MB
Resolution:  3.4084 pixels per µm
Pixel size: 0.2934x0.2934 µm^2
I think it would be fair to assume that this is a precision problem and the pixels are somewhat square. The display precision in ImageJ is sub-nanometer, and I think this would be a good cutoff. I would therefore propose to round to 4 decimals here.
Happy to make a small pull request with a proposed solution in case.

Thanks for reporting this, I completely agree that the values should be rounded when comparing them for equality. I'll fix this and tag a new release right now -- watch for v1.15.3.


Exercice 3. Lock funds forever in the plutus script

The order in which you specify the --tx-out flags when building the transaction matters a lot.
cardano-cli transaction build-raw --alonzo-era \
--tx-in aa1d18824746b58ce4ed242211f4e785d3b97f9fe6b255de27d5c3bb1056a55b#0 \
--tx-out $changeAddress+value \
--tx-out $plutusScriptAddress+value \
--tx-out-datum-hash $hash \
--invalid-hereafter $slotHeight \
--fee value \
--out-file tx.raw

will succeed. However, the same transaction as written below will end up locked forever without datum associated with the newly created script utxo.
cardano-cli transaction build-raw --alonzo-era \
--tx-out-datum-hash $hash \
--tx-in aa1d18824746b58ce4ed242211f4e785d3b97f9fe6b255de27d5c3bb1056a55b#0 \
--tx-out $changeAddress+value \
--tx-out $plutusScriptAddress+value \
--invalid-hereafter $slotHeight \
--fee value \
--out-file tx.raw

My understanding from this is that the datum will go to the closest --tx-out when building the transaction. Is this correct ?

Answer (thanks @mdebruijne):
Order matters a lot because every UTxO can have different predicates. That's why you need to group the predicates that apply to that specific UTxO together.
If you provide a tx-in, all subsequent tx-in-... arguments apply to the UTxO referenced by that tx-in until the next tx-in or tx-out.
If you provide a tx-out, all subsequent tx-out-... arguments apply to the UTxO referenced by that tx-out until the next tx-out or tx-in.

Perhaps it should be an error to supply a --tx-out-datum-hash without a --tx-out before it?

Indeed it does. Here's the relevant output from the cardano-cli transaction build-raw --help:
[--tx-out ADDRESS VALUE [--tx-out-datum-hash HASH]]

So yes, you can have any number of --tx-out and after each one an optional --tx-out-datum-hash.
(Sadly the [] notation does not distinguish between zero-or-one and zero-or-more.)
And if one looks at the full --help output, you'll see the same applies to the other script-related flags. This is pretty much unavoidable for the CLI given the complicated structure of transaction inputs etc, at least if we stick to the style of being able to build the whole tx body in one go.

We should perhaps look at whether we can disallow reordering since for these ones it could cause confusion. I don't know for sure if optparse-applicative would let us ban reordering on some but not all flags.

@dcoutts comment above solves the question.


Remove convert_data() deprecation warning

Motivation
A long time ago, we renamed convert_data to run_conversion to match the syntax between DataInterfaces and the NWBConverter. We provided a deprecation warning for this change for at least six months, and with the new major version so much else is changing we might as well get around to actually removing this.
Also added some missing docstrings while I was working in that class.
Checklist

[X] Have you checked our Contributing document?
[X] Have you ensured the PR description clearly describes the problem and solutions?
[X] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?

@alejoe91 If you could take a look since Ben is busy - very simple change


Cloudbees JDK causes VerifyError

Seen in #721, #718, #723

I was able to temporarily revert the Cloudbees change, but it got re-applied this morning.
Looking into how I can get my config change to stick...

I found a place to explicitly set the JDK to 7u60.  That allowed 2 PRs to pass on rebuild, so that's a good change.  I'm not 100% confident that it won't get changed back out from under me, so I'll leave this issue open for now until I have more confidence in that.


chore(release): v2.3.0 🎉

:robot: I have created a release beep boop
2.3.0 (2023-08-07)
Features

cache: expose getSimpleLambdaClientCacheKey method (7b09bf2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/simple-lambda-client/releases/tag/v2.3.0 :sunflower:


求助

This is not an issue of OpenLauncher. Check your Internet connection, Android SDK installation, and be sure you have accepted the licenses.
Maybe the echos of https://gitlab.com/fdroid/fdroidserver/merge_requests/225/diffs#55bf5e6391d4e4297af5d0a50c839d60b05f8fa9  + write access to android sdk resolves your problem.
I made a fresh clone, and did gradle assemble without any issues.


Added responsiveness for desktop and tablets

Issue #79 This is the first draft which will work only for devices with more than 500px widths. For mobile devices (less than 500px width) I need to change the layout to something like this

@IAmTamal Here is the PR please have a look

Added mobile elements as well
Outcome
https://user-images.githubusercontent.com/92074003/194582099-aa9b986f-dbb6-456a-ad46-b3a926c25076.mp4
but I got one conflict message for the App.js file. I did resolve it (Preetier formatted some code but I have kept the new changes in it) Still please check whether it is ok or not  @Jai-Solania-29 @Kanika637

Hey, @ShubhamPatil0723 thanks for contributing, Great work 🚀
Merging your PR 🎉
Don't forget to give a ⭐ to the repo 😃

Hey @ShubhamPatil0723, I had to revert the PR as you worked on the old design of the header. If you open it in your local environment then you will see that the header's design has been changed the search bar is smaller in size and other components has some spacing too. So you have to do the responsiveness on that header.


Followed steps exactly, but still points to jitsi.net

Hello,
I've explicitly followed all the steps on customizing my chrome plugin.
After I clear the cache and try to use my customized plugin, it still points to jitsi.net.
Am I missing something or is the current release not working?
Thanks!

Same here

The instructions are rather incomplete - you also need to update BASE_DOMAIN in /calendar/meet-calendar.js to get the URLs to generate correctly.

Just FYI once/if #54 merges I have the code ready to give the plugin settings so you can configure this.


Enable TLS integration test

We already have a TLS integration test here
https://github.com/influxdata/rskafka/blob/3af1939fd47f8680d40074dc3fd2e2a4a0da6b8c/tests/client.rs#L73-L75
but it is disabled because our CI doesn't set up any form of TLS. Would be nice if we could get this to work. Ideally the test would be gated by some environment variable like TLS_ENDPOINT so that developers can still run a rather simple cargo test locally.

Would it be possible to switch ci to github actions?

@toondaey what would be the advantage of switching?


fix Can't open k8s session cluster webUI #2248

What changes were proposed in this pull request
Issue Number: close #2248

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

hi dyccode, there is a checkstyle issue, please fixed, you can run command:
mvn spotless:apply


ZenjectBehaviour

Hello, guys. You know we have PostInject method where we can start using injected fields. I've tried to fix this injecting in Awake method. For that I need DiContainer which is provided by BehaviourInjector. DiContainer is private there and it's accessible only by ZenjectBehaviour. If DiContainer is not initialized yet, it is initialized with CompositionRoot's Awake method. And finally if Awake on CompositionRoot is called one more time by Unity, there is check for that DiContainer is already initialized. Everything else is test files to prove the concept.
Main benefit is that we don't have to add PostInject method and can simply override Awake with base.Awake. We can put game objects anywhere and we don't have to add factory to every MonoBehaviour and register them in installer. And we can simply use prefabs as before instantiating them using Unity Instantiate method (I guess it's prototype pattern).
Anyway please tell me is it going to work since I haven't look deeper and is it going to be included in Zenject.

I can see how this would make create prefabs much simpler.  Being able to avoid having factories everywhere would probably be nice.  Before doing something like this though I would want to work out the issues that I bring up in #38 , since that could have implications for this kind of thing (since it would mean there might be multiple containers in a scene).  If there's just one container though this solution is nice.  So I'll leave this open for now.

I took another look at this approach.  I don't think this can work because there might be multiple DiContainer's, especially when using Zenject 3 (which has better support for nested containers).  This is why I don't like having a static global way of using DiContainer.  Also, defining a common MonoBehaviour base class seems too intrusive to me (though I know other DI unity frameworks do this and it works fine) even if it does make things a bit easier in that you can avoid needing to create factories everywhere and can just instantiate prefabs directly


Post-merge review(s) of PR #347

This is to be closed once all comments mentioned below are resolved.

[ ] https://github.com/ROCmSoftwarePlatform/MIOpen/pull/347#pullrequestreview-469461853

/cc reviewers of #347 -- @daniellowell @asroy @zjing14 @carlushuang @TejashShah

@atamazov, should we close this issue as #347 has been merged?

@aserio Just the opposite.

@atamazov, Ah! Skimmed through to fast!

@jane-zxy What is the status of these post-merge reviews?

@jane-zxy What is the status of these post-merge reviews?

After double checking the comments of, I think there are not issues with original PR
https://github.com/ROCmSoftwarePlatform/MIOpen/pull/347#issuecomment-713729502
@atamazov @jane-zxy

I just resolved all review comments but one, -- NextFlag<false, false> is not good.


Which skin?

@olli991
a request if you don’t mind.
I would like to change the png’s for the menu (navigation, media, traffic, full link,…).
But the png’s are found in skin1,2,3,4&5 (in skin 4&5 the resolution is slightly bigger).
How do i know which skin to modifie? Or do you have to modifie all png’s in all the skins?
Thank you.

You can check on what skin you're running with VCP, VCDS, OBD11,...
Just look into Coding of ECU 5F into Byte 17: Skinning: Skin_4 your current Skin is listed in there.


feat(Data/Setoid): another way to define piecewise functions

Use IndexedPartition to define piecewise functions. This is natural for piecewise functions that are split into many pieces.

I made some suggestions to clean up / shorten the proofs. @j-loreaux did you have more comments in mind?

I don't remember why I self-requested a review here.

bors r+


Support obsidian wiki links

Use https://github.com/flowershow/remark-wiki-link-plus
Acceptance

[x] Obsidian wiki links e.g. [[...]] are working ✅ 2022-07-08 /demo/wiki-links
[x] Bonus: also document in developer section how we add remark / rehype plugins ✅ 2022-07-11 /meta/developers

Tasks

[x] Add a "test" (e.g. add to a demo page and then add a proper test)
[x] Add the plugin
[x] Document in e.g. /meta/developers

@khalilcodes the demo should have a working example not just the code for doing it. Could we add that?

FIXED. Actually all working. Note will move docs/demo to https://flowershow.app/docs/wiki-links


Brat export of documents without entities

Hi,
I used the Brat export function of a protected corpus of a given BioC-XML-file, but I have an error
AttributeError: 'BioCDocument' object has no attribute 'entities'
Is it possible, to create BioC files without the definition of 'entities'?
I created the entities by my self:
`    for passage in doc.passages:
i = i + 1
annotations = []
    for ann in passage.annotations:
        off = ann.locations
        key = len(annotations)
        start = off[0].offset
        end = off[0].offset + off[0].length
        ann = 'T' + str(key) + '\t' + ann.infons['type'] + ' ' + str(start) + ' ' + str(end) + '\t' + passage.text[off[0].offset:(off[0].offset + off[0].length)]
        annotations.append(ann)

`
Do you have an idea?
Best regards, Christina

@chlor I am also running into this issue.
I opened #22 about bioc2brat not having a reverse conversion. Afterwards in the encoding a brat object section of the documentation I noticed that you can serialize a "doc" object to a brat string. I was wondering what kind of doc object it was referring to, and if I could use that to do the reverse conversion even if it wasn't included in the brat2bioc module.
I pulled a BioCDocument instance out of my collection, and ran the following code:
from bioc import brat
brat.dumps_ann(doc)

However, this gave me the following error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/mnt/home/lotrecks/.local/lib/python3.7/site-packages/bioc/brat/encoder.py", line 71, in dumps_ann
    dump_ann(doc, output)
  File "/mnt/home/lotrecks/.local/lib/python3.7/site-packages/bioc/brat/encoder.py", line 55, in dump_ann
    for ent in doc.entities:
AttributeError: 'BioCDocument' object has no attribute 'entities'

Searching the repo code it looks like there are two types of doc objects: BioCDocument and BratDocument.
From what I can tell, these two document classes are not interchangeable, and I would assume the documentation for encoding a brat document refers to a BratDocument, and therefore dumping a BioCDocument to a brat file isn't possible.
@chlor Were you able to get this working by manually adding entities?

Since it is a bioc package, we only provide functions to convert to the bioc format. We don't provide a reverse conversion.


h264dec: tidy code

As for rearranging the logic. the two lines are extra.
the code at line 127 has same effect.

merged as 6456e2383dac3a58f1983b7da8ac885b639451a7


Filter on basedir

If -p is given the files linked to the output dir are limited to source
files that have the path given with -p in the name.
This is usefull if you have a couple of mail accounts, and want to
search only one account.
Example:
I have three email accounts I sync with offlineimap and index them all
together with one notmuch index.
To search globally, I run:
$ mutt-notmuch -g ~/mail/temp/search
To search only my work email, I run:
$ mutt-notmuch -g -p 'mail/work' ~/mail/temp/search
I know I could have done this with multiple notmuch databases but that
was too much hassle with environment variables, especially with the
combination of offlineimap, mutt, notmuch etc

I like this! :sparkles:


Improved multiple file upload handling

This function solves the problem (这个功能解决的问题)
When dropping multiple files into the upload area, we can obtain the full path of files within sub-directories. This is useful if the user drops one or more folders, instead of individual files. It is made possible by using non-standard APIs that (I believe) are supported by major browsers (https://caniuse.com/?search=webkitGetAsEntry). As a result, you get a full path like my-folder/images/foo.jpeg
Here is the code to support this behaviour.
async function drop(e) {
  e.preventDefault()
  e.stopPropagation()
  let transfer = e.dataTransfer
  let entries = await getAllFileEntries(transfer.items)

  entries.forEach(async entry => {
    entry.file(async file => {
      console.log(entry.fullPath, file)
    })
  })
}

async function getAllFileEntries(dataTransferItemList) {
  let fileEntries = []
  let queue = []
  for (let i = 0; i < dataTransferItemList.length; i  ) {
    queue.push(dataTransferItemList[i].webkitGetAsEntry())
  }

  while (queue.length > 0) {
    let entry = queue.shift()
    if (entry.isFile) {
      fileEntries.push(entry)
    } else if (entry.isDirectory) {
      let reader = entry.createReader()
      queue.push(...await readAllDirectoryEntries(reader))
    }
  }
  return fileEntries
}

async function readAllDirectoryEntries(directoryReader) {
  let entries = []
  let readEntries = await new Promise(directoryReader.readEntries)
  while (readEntries.length > 0) {
    entries.push(...readEntries)
    readEntries = await new Promise(directoryReader.readEntries)
  }
  return entries
}

Expected API (期望的 API)
The API can remain unchanged, but UploadFileInfo should have fullPath or similar.

Is it possible to also support directory?
https://developer.mozilla.org/en-US/docs/Web/API/HTMLInputElement/webkitdirectory

Good question, maybe! I would be interested to see what is returned from the input change event. It may include e.items or something which could be the DataTransferItem type.


Microsoft Defender for DNS pricing

On this page you have this link:

Pricing: 	Microsoft Defender for DNS is billed as shown on the pricing page

The linked page has no mention of DNS or og Microsoft Defender for DNS.
Where can we find pricing information on Microsoft Defender for DNS?
Regards
Geir Helland
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 608b5811-b62d-d7b0-8b70-99253c49a0ee
Version Independent ID: 56cedb4d-d286-c9f7-fe9b-7678f2967f74
Content: Microsoft Defender for DNS - the benefits and features - Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/defender-for-dns-introduction.md
Service: defender-for-cloud
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@gh-skatt
Thanks for your feedback! We will investigate and update as appropriate.

@gh-skatt
Thanks for bringing this to our attention.
Our team is already working on this, they have already created a backlog item for it. You can stay updated on DNS pricing.
here https://github.com/MicrosoftDocs/azure-docs/issues/114870
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Disallow guava jre

Codecov Report

Merging #194 (8172c61) into main (a85b8cc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #194   +/-   ##
=======================================
  Coverage   73.82%   73.82%
=======================================
  Files          62       62
  Lines        2800     2800
  Branches      453      453
=======================================
  Hits         2067     2067
  Misses        488      488
  Partials      245      245

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 9d52017...8172c61. Read the comment docs.


Update JSON LD for tiles

print-json-ld.js generates the JSONLD
This needs to have the appropriate keywords, variables, etc. Suspect it's a grab bag captured from a jinja2 python project.

title: appropriate title (datasets for hydrography org tile hxxvxx, lat long)
description (work with hydorgraphy.org ("this is a tile containing like to datasets for (location), this includes (basic names like elevation, discharge, etc)
org correct
contact correct
date published, etc
add keywords, if appropriate
update variableMeasures see if we cannot capture appropriate variables to be for each dataset in tile. maybe do this as a csv so we can use them when generating the layer jsonLD
add appropriate name to distributions.

ADD SPATIAL
each tile has a spatial component. Figure out what it is, and use it to generate the spatial.
drop temporal, unless it's needed

close as it's done


fixonly enable colocate scan for one phase global parttion topn in some condition

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall

run buildall


RetrieveCatalogObject returns results out of order for related object images

Describe the bug
retrieveCatalogObject returns results out of order for related object images
Expected behavior
Return the same order as in the GUI.
To Reproduce
Steps to reproduce the bug:

Call batch retrieve API through sdk
Check result images order with GUI

Square SDK version
square": "^37.1.1",

hi @kyeshmz - can you provide a little more detail about your request and response?
I don't think order responses are going to be guaranteed on any of these batch API calls, but I just want to make sure I'm understanding your question.

@zenmasterjobo I am utilizing the
 await squareClient.catalogApi.retrieveCatalogObject(
				id,
				true
			)

API call. It returns a category and images that are related to the certain object.
The order of the images differs from the one that you can setup at square.online, where you can configure the order of images that would appear, for example in a carousel.

Ah yes, I see. Unfortunately Square Online operates on different APIs that that of the APIs exposed on our public platform. And again in this case, order isn't guaranteed. I Will forward this feedback to the relevant team as I did with your previous post.
Also, for issues like this that concern the API and not the SDK, please use our developer forums - Thank you!

It would be great to put it in the documentation somewhere!

Is there a possiblity of making this the same as the Square Online? Currently, I get rate limited because related object images returns the wrong order, but querying by image Ids returns in order @zenmasterjobo

Hi @kyeshmz
I don't think we can make the related objects list have the same sort order because the related objects can contain way more than just the images. However, because the image_ids array is respecting the correct order, we can use this array to sort the related object images so that way we don't need to query the API again.
After you have queried the object you want with retrieveCatalogObject(id, true) you can do something like this in javascript
let sorted = related_objects.sort((a, b) => image_ids.indexOf(a.id) - image_ids.indexOf(b.id));
By using javascript's sort method, we can write our own compare function that will sort the related objects to match the order of the image_ids array. This does assume that your related objects for a given catalog item only has image objects in it. If it does not, you would first need to go through the related objects and make a new array of only the objects that have type: 'IMAGE'
Let me know if you have any questions about this!

@zenmasterjobo This worked for my usecase, thanks!


Make docs reproducibility more robust

Commits:

Replace faketime with SOURCE_DATE_EPOCH for docs

On Debian some architectures don't support using the faketime wrapper and
it will simply bail out. Turns out pdflatex now natively supports
SOURCE_DATE_EPOCH though as long as you also set FORCE_SOURCE_DATE,
otherwise it won't take effect. Graphviz (dot) also supports the former.

Introduce GIT_EPOCH for getting SOURCE_DATE_EPOCH date in tarball

Problem that was needed solving here is that dot and pdflatex did not generate PDF files in docs/images/011 directory only based by content but also by current time. That produced issues of CI updating https://github.com/YosysHQ-Docs/yosys-cmd-ref/tree/main/images since then all these PDFs are always found to be different.
For creating documentation Debian packages, it would be perfectly fine to get PDF files with any timestamp in them since anyway you would rebuild it for each version, so I see no need to enforce any specific timestamp.
In this case I would rather define  FAKETIME variable empty and use that on these two places and then override it in CI, that way your builds will not be affected and it will work same as always for us.
Does this solution looks fine to you, and does it solve issues you experiencing ?

Does this solution looks fine to you, and does it solve issues you experiencing ?

Reproducibility isn't just a convince feature, it's a table stakes distribution security mechanism see https://reproducible-builds.org/
The idea is that anyone can check a Debian package corresponds to it's source code by recompiling and comparing the resulting .debs bit by bit. This helps detect compromised build infrastructure.
So we really do need to get rid of faketime here.
Currently this patch is still broken due to CreationDate etc. see https://wiki.debian.org/ReproducibleBuilds/TimestampsInPDFGeneratedByLaTeX I'll have to do slightly more invasive surgery.
--Daniel

Ok, my previous analysis was wrong. Graphviz dot doesn't seem to support SOURCE_DATE_EPOCH and that's the root problem I have in the Debian packaging.
The problem with faketime is this:
faketime: sem_open: Permission denied
The faketime wrapper only works on platforms that support the sem_open()
system call. However, you may LD_PRELOAD libfaketime without using this wrapper.

So instead of removing the use of faketime I'll switch to setting the LD_PRELOAD and FAKETIME envvars directly.
--Daniel

As per your own links;

Another option for Debian would be to remove the field completely with dh_installdocs or similar, or at least set its content consistently (package build date, installation directory, and so on).

This is what faketime is doing for us.  While the reason was to prevent git from detecting a change in pdf when there was no change in content, this accomplishes the goal of a reproducible build that (should) produce the same result from the same source, rather than the output changing due to differences in build system.
Unless there is something else affecting the reproducibility (the PDF ID in your link for example), then this is just about building on Debian and the name of the pull request should be updated to reflect that.

Turns out the faketime problem (faketime: sem_open: Permission denied) was related to a build chroot misconfiguration not an architecture specific limitation of faketime.
So in fact this entire PR is ill-conceived :)

Unless there is something else affecting the reproducibility

I'm still seeing PDF CreationDate differences (debci), I think those are due to $TZ. I'll send an update once I confirm overriding $TZ gets rid of the non-reproducibility.


chore: update pr template to improve doc control

Brief Information
This pull request is in the type of:

[ ] bug fixing
[ ] new feature
[x] others

What does this PR do?

Improve the PR template so that doc can always under control for each issue.
Fixed issues

Details
Before: What was the problem?
The release manager has to manually look through all pull requests in a certain release to check whether there should be document changes and whether they have been made into the echarts-doc repo. Sometimes it's hard for the release manager to know the detail of all PRs so it's not always an easy job to do.
After: How is it fixed in this PR?
So I decide to improve the workflow so that:

The author of a PR is mainly responsible to decide whether a PR requires document changes and whether a PR in echarts-doc repo has been made. This is done by editing the description of the PR.
The reviews of the PR should check this info before merging.

The Apache ECharts bot is recently under debugging for this feature. It should add one of PR: doc unchanged, PR: doc awaiting, PR: doc ready to the PR based on the description. It will also give a warning if none of the three checks are made when a PR is opened and when a reviewer approves it.
I will also look into if we can made checking labels to be a prerequisite before merging.
Misc

[ ] The API has been changed (apache/echarts-doc#xxx).
[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

Is it necessary to label PR: doc unchanged if one PR is unrelated to the documentation?


Husky pre-commit hooks don't work with Docker

The Husky pre-commit hooks rely on locally installed node modules. So if you're deploying locally via Docker, the pre-commit hooks fail. Feels "wrong" to be forced to run a local install of dependencies when I'm already running Docker.
Possible solutions:

Make these hooks work when deploying via Docker
Remove them entirely. These hooks are pretty heavy-handed and users (outside of HAUS) might not want these as part of their workflow.

Open to other suggestions.

I need more context into this issue. Husky should only run when making commits so its a dev dependancy. It is running during deployment? If so that shouldn't be happening. It should only run against staged files when making a commit.

Husky should only run when making commits so its a dev dependancy.

Correct. It's doing that now, but it relies on locally installed node_modules. One of the benefits of Docker is that your dependencies are not installed locally, but they reside inside your container. After a quick Google, there appear to be some solutions in place. I don't mind looking more into this when I have some time.

It is running during deployment?

"I'm deploying Docker locally" is synonymous with saying "I'm running Docker locally". Sorry for the confusion.

As of now the a host machine must also have node installed to use husky and commit via Tower or any other tool that does not have access to bash exports. This seems to be the one place where we're breaking the encapsulation of a docker running node.
I'd ❤ to get this working across the board. The pre commit hooks are awesome.

Check out the PR #30. Runs (exec) inside the web container

Note: This pre commit requires you have to already built the web container at one point


Expose SQLiteError and Problem

When using FluentSQLite, the raw SQLiteError ends up getting propagated all the way up to the caller, which is usually a route. If you need to know what kind of error occurred (eg if it was specifically a unique constraint violation in order to implement concurrency-safe model creation), the only way currently is to do string matching on the error's description, which is shoddy, error-prone, and fragile.
With the error type public, you can do
if let error = error as? SQLiteError, case .constraint = error {
  /* handle UNIQUE constraint violation */
}

This is much cleaner. (This PR also changes the properties to lets because they don't need to be var in the first place, which is just good style)
There is a FluentSQLiteError type, but not only does it not preserve any of these specifics, it never actually gets thrown!

This is possible now using SQLiteError: https://github.com/vapor/sqlite-nio/blob/master/Sources/SQLiteNIO/SQLiteError.swift


Integrate Jenkins with explore microservice

Visit the following link to check configurations:
Job: http://<IP_ADDRESS>:8080/job/explore/
Node: http://<IP_ADDRESS>:8080/computer/exploreNode/


Triggering IncrementConfig, DecrementConfig and ToggleConfig while in a mode causes keybinding issues

Version (which release did you use?)
Development branch e87df58
Describe the bug
Because IncrementConfig, DecrementConfig and ToggleConfig all modify the in-memory config object, triggering them while a mode is active causes nog to become unresponsive in some ways. It won't respond to key commands but you can still switch workspaces by clicking the workspace number and the timer component keeps ticking.
To Reproduce
Steps to reproduce the behavior:

Create a mode and add a keybinding for either IncrementConfig, DecrementConfig or ToggleConfig
Run nog and activate the mode, then press the keybinding configured in step 1

Expected behavior
The keybinding action should occur successfully and nog should keep running
Actual behavior
All keybindings stop working
Notes
I haven't dug too deeply into this but it happened to me today after I added a IncrementConfig/DecrementConfig keybinding for padding/margin to my resize mode. I suspect it's because those three functions call the src/hot_reload.rs update_config function and it might just get weird with the registering/unregistering keybindings while still in a mode?

I think it would be best to look at this after the cross-platform stuff is merged, since it tries to minimize the lock usage and overall global state.

This should be fixed in cross-platform.


load_gdiplus fails

Executing load_fontsmooth rgb
Setting font smoothing to rgb
Executing wine C:\windows\syswow64\regedit.exe C:\windows\Temp\fontsmooth.reg
Executing wine64 C:\windows\regedit.exe C:\windows\Temp\fontsmooth.reg
Executing w_do_call gdiplus
warning: You are using a 64-bit WINEPREFIX. Note that many verbs only install 32-bit versions of packages. If you encounter problems, please retest in a clean 32-bit WINEPREFIX before reporting a bug.
Executing load_gdiplus
grep: warning: stray \ before /
Executing cabextract -q -d /home/ghosty/cc21//Adobe-Photoshop/dosdevices/c:/windows/temp -L -F x86_microsoft.windows.gdiplus_6595b64144ccf1df_1.1.7601.17514_none_72d18a4386696c80/gdiplus.dll /home/ghosty/.cache/winetricks/win7sp1/windows6.1-KB976932-X86.exe
/home/ghosty/cc21//Adobe-Photoshop/dosdevices/c:/windows/temp/x86_microsoft.windows.gdiplus_6595b64144ccf1df_1.1.7601.17514_none_72d18a4386696c80/gdiplus.dll: No such file or directory
warning: Note: command cabextract -q -d /home/ghosty/cc21//Adobe-Photoshop/dosdevices/c:/windows/temp -L -F x86_microsoft.windows.gdiplus_6595b64144ccf1df_1.1.7601.17514_none_72d18a4386696c80/gdiplus.dll /home/ghosty/.cache/winetricks/win7sp1/windows6.1-KB976932-X86.exe returned status 1. Aborting.

I'm closing it due to lack of activity and not knowing what the issue cause...


The textbox control does not display when used as an Adorner

The textbox control does not display when used as an Adorner。Replacing the textbox with a textblock or button control will display it normally。
Avalonia version：11.1.3
operating system：windows10
test code:
var text = new TextBox(); //TextBlock
text.Text = "asdfasdfsdf";
text.Background = Brushes.Red;
text.Width = 10;
text.Height = 10;
AdornerLayer.SetAdorner(this, text);
AdornerLayer.SetIsClipEnabled(text, false);
Sorry, I'm not very good at operating GitHub

I can only repro when trying to adorn a Window (which may or may not be a supported use case). The adorner shows for any other case I've tried: a UserControl that creates + attaches the adorner to itself and creating+attaching the adorner to a child StackPanel.
Can you create a minimal repro and post the project?

thanks for your answer
i create a project that Similar to my project，it Reproduce the fault
wpf project is normal
thanks
test.zip

I'm sorry
Because my custom panel is incorrect, the textbox is not displaying. I will study again. Thank you

A new problem has been discovered.
The TestText Box control inherits from the Text Box and does not display when using it as a decorator. I have created a test project, thank you
test.zip

Thank you
I uploaded the project, but I don't know why it hasn't.
I used StyleKeyOverride，I'll stick out the code。
test code:
public partial class MainWindow : Window
{
public MainWindow()
{
InitializeComponent();
    var testPanel = new TestPanel();
    testPanel.Width = 600;
    testPanel.Height = 500;

    this.Content = testPanel;
}

}
public class TestPanel : Panel
{
public TestPanel()
{
Background = Brushes.Red;
    var testControl = new TestControl();
    testControl.Width = 200;
    testControl.Height = 200;

    this.Children.Add(testControl);
}

}
public class TestControl : Control
{
public TestControl()
{
this.PointerEntered += TestControl_PointerEntered;
}
public override void Render(DrawingContext context)
{
context.DrawRectangle(Brushes.Transparent, new Pen(Brushes.Black, 1), new Rect(0, 0, Width, Height));
}
private void TestControl_PointerEntered(object? sender, Avalonia.Input.PointerEventArgs e)
{
var text = new TextBox();//TextBox
text.Text = "asdfasdfsdf";
text.PointerExited += Text_PointerExited;
    AdornerLayer.SetAdorner(this, text);
    AdornerLayer.SetIsClipEnabled(text, false);
}
private void Text_PointerExited(object? sender, Avalonia.Input.PointerEventArgs e)
{
    AdornerLayer.SetAdorner(this, null);
}

}
public class TestTextBox : TextBox
{
protected override Type StyleKeyOverride => base.StyleKeyOverride;
public TestTextBox() { }
}
thank you very much


Add CRD installation hints in Helm chart

When installing the Operator using Helm, it is necessary to install the CRDs first.
Helm provides mechanisms towards this. In Helm 2.0, there is 'crd-install' hook annotation. In Helm 3.0, there is the 'crd' directory. Check following for an example of 'crd-install' hook.
https://github.com/cloud-ark/kubeplus/blob/master/Guidelines.md#include-crd-installation-hints-in-helm-chart
Given that current chart is being targeted for Helm v2 (https://github.com/datastax/cass-operator/blob/master/charts/cass-operator-chart/Chart.yaml#L1), it will be good to include the crd-install annotation on the CRD YAML.

apiVersion: 2 is for helm 3.x I believe.
I saw in some docs that the crd-install doesn't support templating, which we are using to support multiple versions of k8s.

@jimdickinson You are correct. v2 is for Helm 3.x
In that case you don't need crd-install hook. But then I would suggest putting the crd in its own dedicated directory named 'crd's.
https://helm.sh/docs/topics/v2_v3_migration/


upload/list logic

while you hit UPLOAD button in menu, you are redirected to LIST OF UPLOADS, which is not user friendly.
1, add LIST OF UPLOADS button into the menu, which heads you to LIST OF UPLOADS (todays UPLOAD)
2, change UPLOAD to be more user friendly (call for action is UPLOAD, so it have to be the most visible option); UPLOAD button will:
2, 1, it will leads to LIST OF UPLOADS and automatically open selection (camera/gallery) dialogue
2, 2, it leads to new page with some info about uploads + UPLOAD buttons
2, 3, it leads to LIST OF UPLOADS and instead of first picture there is CAMERA/FILE (see the attachment, sorry for shitty graphics :)

mPOV: start with 2.3 and have 2.2 as nice-to-have bcs we will need a designer for that

I totally agree with all of that.
The camera/gallery buttons are much too small right now.

Now UX looks okay based on this scenario.


calyx-py correctness testing: icarus-verilog disagreeing with verilog

calyx-py/test/correctness not only tests the builder's ability to generate Calyx code, but also the correctness of that Calyx code. The .expect files have veri-style outputs, not Calyx code.
I am seeing a disagreement between verilog and icarus-verilog outputs for the file arbiter_6. Verilog is working as expected, while icarus-verilog is failing with lots of complaints in the vein of
/tmp/tmpkl5k4brj:4307: error: together3_mem5_done Unable to assign to unresolved wires.

Originally posted by @anshumanmohan in https://github.com/cucapra/calyx/pull/1514#discussion_r1204218034

@anshumanmohan the following two commands work fine for me:
name=$(basename calyx-py/test/correctness/arbiter_6.py .py) &&
dir=$(dirname calyx-py/test/correctness/arbiter_6.py) &&
python3 calyx-py/test/correctness/arbiter_6.py |fud e --from calyx --to jq --through verilog --through dat -s verilog.data "$dir/$name.data" -s jq.expr ".memories"

(extracted from runt -i arbiter_6 -n)
and the modified version that runs using icarus-verilog:
name=$(basename calyx-py/test/correctness/arbiter_6.py .py) &&
dir=$(dirname calyx-py/test/correctness/arbiter_6.py) &&
python3 calyx-py/test/correctness/arbiter_6.py |fud e --from calyx --to jq --through icarus-verilog --through dat -s verilog.data "$dir/$name.data" -s jq.expr ".memories"

If you can reproduce the bug again, feel free to reopen. My guess is that there was some bad interaction with code in initial blocks that was fixed when we disabled the generation of initial blocks.


Facebook authenticator fails if "Not now" is clicked without providing credentials

Steps to reproduce:

Setup an SP for SSO
Configure a multi option step where FB is one of the options
At the option selection page select FB
At FB login page click "Not now"

Then following error occurs:
[2018-02-07 15:28:40,176] ERROR {org.wso2.carbon.identity.application.authentication.framework.handler.request.impl.DefaultRequestCoordinator} -  Exception in Authentication Framework
org.wso2.carbon.identity.application.authentication.framework.exception.FrameworkException: No authenticator can handle the request in step :  1

Fixed with https://github.com/wso2-extensions/identity-outbound-auth-facebook/pull/33


Full Duplex Operation

Description of the new feature
In https://github.com/microsoft/terminal/issues/18265 @carlos-zamora informed me that while operating the terminal as full duplex (reading from stdin and writing to stdout at the same time) does work in the stable version of Windows Terminal, Microsoft did not recommend doing so, so a bug with full duplex operation was not actually a bug.
Given that the last half-duplex terminal to exist was decommissioned sometime in the 1990s, however, I suggest that Microsoft undergo a project to add full duplex operation to Windows Terminal. And bring it into line with similar products. While Carlos did not believe there were any known use cases for a full duplex terminal, I'm inclined to disagree. While I can't speak for anyone else, the Go programming language does not have the capability to cancel reading from stdin, meaning that no application written in Go is capable of half-duplex operations.  I believe that adding Full Duplex capability to Windows Terminal would be a major boon for Go developers, especially since we all thought Windows Terminal already had it.
Proposed technical implementation details
I'm not clear on the technical specifics, but probably do whatever the current version of Windows Terminal does, since Full Duplex works in that version.

I'd rather reopen #18265 for discussion than entertain "full duplex operation" as a di novo feature request, if you wouldn't mind. There are a lot of assumptions and messages flying by without response, so please hold up.

In general, turning a legitimate concern (which you do have!) about how your bug report was handled into a somewhat facetious feature request is not conducive to discussion.

Unfortunately, I don't have a twitter account, so there wasn't really another option.