Statement binding & variable scope

Say I have the following loop, where i is obviously a temp variable:
statement stmnt(db);

for(int i = 1; i < 4; i++)
{
    stmnt.exchange(use(i));
}

stmnt.define_and_bind();

When I later call stmnt.define_and_bind() only the last instance of i is used, so I end up preparing a statement where the value 4 (i post-incremented) is bound 3 times.
Given the style of the SOCI API I can kind of understand why this is, but what would be the best practice workaround? I'd love the answer to be "Hey, you can bind by value, just do ...", but I'm guessing not.
Obviously the above example is contrived, it might more likely be that I'm calling a function several times with different inputs and want to use the result each time as part of the statement.
The only workaround that comes to mind is to store the results first in a vector, then traverse over that, e.g:
statement stmnt(db);

std::vector<int> temp;

for(int i = 1; i < 4; i++)
{
    temp.push_back(i);
}

for(std::vector<int>::const_iterator it = temp.begin(); it != temp.end(); it++)
{
    stmnt.exchange(use(*it));
}

stmnt.define_and_bind();

This has the desired effect, but at the cost of additional memory and repeating what could in the real world be a very long loop.
You could try and capture some reference to the values as they're added to the vector in the for loop, thus negating the need for the later iterater loop, but you've generally no guarantee with these types of containers that the address you capture will remain valid as further elements are added.
So how do you tend to approach this with SOCI? I think it's generally a great API but this feels like a bit of a shortcoming. I'm trying to port a few projects to SOCI (well, I already have, but they weren't previously using prepared statements so now I'm trying to change that too) and this feels like an ugly hack to have to add to existing code.
Any help greatly appreciated,
Matt

On 04/05/2016 19:03, mattj0nes2 wrote:

When I later call |stmnt.define_and_bind()| only the last instance of
|i| is used, so I end up preparing a statement where the value 4 (|i|
post-incremented) is bound 3 times.

It is worse than that. Formally, the variable i no longer exists (it is
local to the loop scope) at the point of statement execution, so you get
the undefined behavior when the statement refers to the dangling
reference or pointer.

Given the style of the SOCI API I can kind of understand why this is,
but what would be the best practice workaround?

Workaround to what goal, exactly? You have shown a code example that
tries to solve some problem, but we do not know what that problem is.

The only workaround that comes to mind is to store the results first in
a vector, then traverse over that, e.g:

It need not be a vector. Any other way of retaining objects around is
OK, and for reasons that you have noted, list might be a better choice.

This has the desired effect, but at the cost of additional memory and
repeating what could in the real world be a very long loop.

No, in the real world it will not be a very long loop, unless you have
an SQL query with a very large number of unbound parameters (and in the
real world you do not).

You could try and capture some reference to the values as they're added
to the vector in the |for| loop, thus negating the need for the later
iterater loop, but you've generally no guarantee with these types of
containers that the address you capture will remain valid as further
elements are added.

That's why list might be better.

So how do you tend to approach this with SOCI?

Approach what, exactly? What do you try to achieve?

great API but this feels like a bit of a shortcoming. I'm trying to port
a few projects to SOCI

And what is the SQL query in that particular case? Does it have a
constant or a variable number of parameters? How many are they?
Regards,
--
Maciej Sobczak * www.msobczak.com * www.inspirel.com

There are two problems I've come up against. Not that I wouldn't necessarily have encountered the same or similar issues with another API, it's just I thought it worth asking in case there are options or workarounds I wasn't aware of. I guess it's really more "Can we have a bind by value option, please?"
The first issue, and the reason for the initial post, was that I had a large number of parameters to pass and it felt a shame to have to buffer them all, something the likes of:
std::ostringstream ssQuery;
int i = 0;

ssQuery << "SELECT a FROM b WHERE c IN (0";

while((i = func()) != -1)
{
	ssQuery << ",:x";

	stmnt.exchange(use(i));
}

As we've said you can of course store things locally for use later, it'd just be nice if the option was there of the API handling this centrally so you didn't have to create local storage wherever a similar situation arises.
The second, slightly more awkward problem I've come across where bind by value would be a god send, is where more than one function shares responsibility for building up the query. Something like:
void TopLevelFunc()
{
	std::ostringstream ssQuery;

	statement stmnt(db);

	ssQuery << "SELECT a FROM b ";

	BuildWhereClause(ssQuery, stmnt);

	stmnt.define_and_bind();
}

void BuildWhereClause(std::ostringstream &ssQuery, statement &stmnt)
{
	int i = 0;
	std::string j;

	ssQuery << "WHERE c IN (0, ";

	while((i = intFunc()) != -1)
	{
		ssQuery << ",:x";

		stmnt.exchange(use(i));
	}

	ssQuery << "AND d IN ("string", "

	while((j = stringFunc()) != "")
	{
		ssQuery << ",:x";

		stmnt.exchange(use(j));
	}

	ssQuery << ")";
}

Again, you could allocate storage and have TopLevelFunc pass int and string lists by reference to BuildWhereClause, but that requires TopLevelFunc to know in advance what argument types BuildWhereClause  will bind. With bind by value the statement could be reasily passed to other functions to be executed later.
Don't forget I'm porting both to SOCI/postgres (from Firebird, which I'll never use again) and attempting to convert code that simply builds unchecked strings at present to the relative safety of prepared statements, so anything that would make this process easier is very much at the front of my mind ...
It's not that I expect bind by value to be made available any time soon, if ever, and maybe I shouldn't have bothered posting, but if you don't ask ... I understand it might not be a walk in the park to implement, but it'd be a stand out feature.
Thanks for your time anyway.

Since there have been no more feedback, can we close this as by design?
Unless @msobczak you want to keep the idea on  backburner, do you?

Related issue with binding https://github.com/SOCI/soci/issues/562#issuecomment-317511977

Closing


Potential problems with masking loads/stores but not pc

According to the current spec, Load and store instructions have their addresses masked, but instruction fetch and page table accesses don’t. This seems to create an excessively confusing scenario for software looking to emulate such accesses, such as emulating misaligned accesses or MMIO accesses.
For example, hlvx.{wu,hu} are defined to mask the address (because they’re load instructions), but the instruction fetch they’re designed to emulate doesn’t. If PMLEN > NVBITS, this would need significant effort for software to work around; even if PMLEN <= NVBITS, it seems to still be a glaring inconsistency.
This can create a problem in, for example, the following scenario:

A hypervisor is running a guest OS
The guest OS configures Sv57 and PMLEN = 16
The guest OS pc points to an address that would be changed if it were masked
The guest OS executes a MMIO load/store instruction that faults and traps to the hypervisor
The hypervisor needs to fetch the instruction to emulate the access

To correctly fetch this instruction from memory, with the current spec, the hypervisor would need to check at which privilege level the MMIO instruction happened, temporarily disable pointer masking at that level, use hlvx.{hu,wu} to fetch the address, and restore the pointer masking state.
(A similar situation applies for mstatus.MPRV == 1 && mstatus.MXR == 1 loads in M-mode.)
This doesn't seem like it is an intended effect of the pointer masking extensions. This wouldn’t be a problem if those implicit accesses are simply defined to also mask their addresses.
Another option would be to make hlvx.* and mstatus.MPRV == 1 && mstatus.MXR == 1 loads not mask the address.

Thanks for bringing up these points. This is a topic that has seen a significant amount of discussion within the TG and with the architecture review committee. A previous version of the standard did support instruction pointer masking, but it was removed since there was not deemed to be a sufficient use case and it would have significantly increased the complexity.
The cases mentioned are each a bit different, so I will address them one by one:

Page table accesses: Pointer masking is a concept that needs to be used in conjunction with code running with HWASAN (or another use case of pointer masking) on top of a particular address space – i.e., pointer masking is intended to be equivalent to manually performing an arithmetic unmasking operation for every load or store, but it is not intended to fundamentally change the address translation once the address reaches the MMU. Page tables are accessed outside of this address space and use addresses that are not part of that application code. It is therefore not clear which privilege mode's pointer masking setting should be applied. Furthermore, if pointer masking were to affect page table accesses, it means that enabling or disabling pointer masking could result in different translations for the same address, which would be very difficult to reason about. The pointer masking spec therefore addresses this in a different way: It requires in Section 2.6 that code must manually perform any masking of addresses before they are written into a register (e.g., satp), since that is the point in time when an address turns from application-visible to only being used within the MMU. The same applies for writing page table entries, which need to contain unmasked addresses.

Emulating misaligned accesses: This is also defined in Section 2.6: "pointer masking is applied for hardware writes to a CSR (e.g., when the hardware writes the transformed address to stval when taking an exception)". The intention is that (e.g.,) the M mode handler emulating a misaligned accesses already sees the unmasked address. The idea is that the handler can be exactly the same whether or not pointer masking is enabled, and does not need to be aware of the pointer masking setting of the privilege mode causing the trap.

Emulating instruction fetch: I agree that this case could potentially use more discussion and that it might make sense for accesses via hlvx.* and mstatus.MXR == 1 to not mask the address. We will discuss this in the TG and with the architecture review committee.

Thanks for the response, points 1 and 2 of your response quite reasonably addresses my concerns, and we can await further discussion on point 3.
This also gave me a new idea: I wonder if we can, like page table addresses, require a correctly masked pc at all times. The problem scenario I had requires PMLEN > NVBITS, which would create a "shadow region" where pc can point to but load/store instructions can't touch. If we require a correctly masked pc for instruction fetch then no such shadow region exists, and in the scenario I gave the exception would be an instruction fetch fault, and the hypervisor would not try to emulate anything.
If we do so, "hlvx.* and mstatus.MXR == 1 don't mask address" should probably be changed into faulting on "shadow region" pc for consistency.
(I hope I'm not missing that the spec already requires this, but I don't find occurrences of "pc", "jump" etc in the spec.)

My sense is that creating new constraints on instruction fetch would likely result in additional complexity similar to applying pointer masking – which was deemed too complex during architecture review without a strong use case (the advantage of the current standard is that it does not affect instruction fetch at all, which is a major source of complexity).
It seems to me like changing hlvx etc. would handle the issue with significantly less complexity.

"hlvx.* and mstatus.MXR == 1 don't mask address" seems like a reasonable solution for that problem.

The new version of the spec (0.8.2 and later) now addresses this issue.

That looks good to me as well


[1.11] BACKPORT: fix(packagerepository): remove url validation on add

This removes the regex validation of the url field and lets rather the backend decide if a url is
valid or not.
Close DCOS_OSS-526

<!-- Suplemental codeblock -->

Checklist

[ ] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

Testing instructions from https://github.com/dcos/dcos-ui/pull/3229
To Test this:

go to Settings > Package Repository
Click the +
Add a random Repository Name
Add a url like http://<IP_ADDRESS>:8080
Add a priority 1
Click Add


Items not showing up

I have Read another issue saying were to put the items and I did that but the Items are installed but are not even in the spawn menu

I just installed it wrong

im a idiot


how to generate client information?

where is the artisan command

looking for it too
how to generate client information ?

Do you mean something like this?
    /**
     * Get user by token (send via http-header or POST/GET)
     * @return json
     */
    public function getUser() {
        $id = Authorizer::getResourceOwnerId();
        $user = User::find($id);
        return response()->json($user);
    }

@kroeder  thanks you'r comment,
I mean is How to add Authorizer infomation in table like oauth_clients table,

It is up to the developers using this to implement a solution for this. We currently don't provide this.


Clarify documentation around default mount permissions for Secrets and ConfigMaps

The default behavior for volumes is mount with readOnly as false. This is not the case for Secret and ConfigMap volumes, which mount by default as ReadOnly volumes.
This change adds the documentation to the main english website under each volume category, and notes the exception for these two volumes on the readOnly configuration flag. In addition, there were inconsistencies around using multiple bullet points within a note vs using multiple notes. Given the similarity in the notes for configmaps and secrets, I condensed the secrets notes into a single bulleted note over two separate note blocks for consistency.
https://github.com/kubernetes/website/issues/41103

Please squash the 4 commits, otherwise LGTM

/lgtm
/approve


Issue when trying to compile myFFmpegConversion.ccp

There is an issue when trying to compile myFFmpegConversion.ccp
There is an undefined reference to extractInteger on line 155 which is causing it to not compile - am I missing something?
I have the latest version of the repo and I am using Linux on Windows.
Many thanks for this repo!

It is strange. Contact me on discord

fifone#2742 doesn't work, I can't add it


Cards in Scrollbox not getting click events

I have an app with a bunch of sprites representing cards, in a horizontally scrolling tray representing a hand.  I have implemented the tray using Scrollbox.  In spite of my setting `stopPropagation' to false, the "click" events aren't getting through to the cards, though the Scrollbox is seeing them. If I change from Scrollbox to Viewport (and add the sprites directly to cardScroller rather than to cardScroller.content) they do get through.
I'm at a loss.  This was working, and has just recently stopped working, and there's no obvious change I've made that seems related.  I've updated to the latest Scrollbox with no effect.
Here's my code.  It's long, but I don't know what is relevant.  Definitely looking over my shoulder on this one.
export function createCardScroller(player, resources) {
  const cardScroller = new Scrollbox({
    boxWidth: resources.cardScroller.width,
    boxHeight: resources.cardScroller.height,
    stopPropagation: false,
    dragScroll: true,
    overflowY: 'hidden',
    fade: true,
  });

  function fetchCard(cardId) {
    return Cards.findOne(cardId);
  };
  const hand = R.map(fetchCard, player.combatant.hand);

  function isInUse(card) {
    return (player.playedCard.get() && card._id === player.playedCard.get()._id)
          || (player.enhanceCard.get() && card._id === player.enhanceCard.get()._id);
  }
  var currentHand = R.reject(isInUse, hand);

  function createCS(card) {
    const cardSprite = createCardSprite(player, card, resources);
    return cardSprite;
  }

  const cardSprites = R.map(createCS, currentHand);

  const desiredHeight = resources.cardScroller.height;
  function setScale(sprite) {
    const scale = desiredHeight / sprite.height;
    sprite.scale.x = sprite.scale.y = scale;
    return sprite;
  }
  R.map(setScale, cardSprites);

  const cardStride = cardSprites.length > 0 ? cardSprites[0].width*1.1 : 50;

  function setPositionAndAdd(accum, cardSprite) {
    cardSprite.position.set(accum, 0);
    //cardScroller.addChild(cardSprite);
    cardScroller.content.addChild(cardSprite);
    return accum + cardStride;
  }
  R.reduce(setPositionAndAdd, 0, cardSprites);

  cardScroller.update();
  cardScroller.on('click', function(e) {console.log('clicked cardScroller'); console.log(e)});
  return cardScroller;
}

That is weird. I checked the scrollbox code and stopPropagation does follow the option. Hmm...can you set up a quick jsfiddle to show this issue is broken so I can debug it?

I'm relieved that you think it's weird.  I looked at the scrollbox code
myself and it all seemed good.  And yet it doesn't work.
I will work on setting up the simplest failing example I can, but I
won't be able to get to it for a day or two.
------ Original Message ------
From: "David Figatner" @.>
To: "davidfig/pixi-scrollbox" @.>
Cc: "gischer" @.>; "Author" @.>
Sent: 6/13/2021 2:50:35 PM
Subject: Re: [davidfig/pixi-scrollbox] Cards in Scrollbox not getting
click events (#40)

That is weird. I checked the scrollbox code and stopPropagation does
follow the option. Hmm...can you set up a quick jsfiddle to show this
issue is broken so I can debug it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/davidfig/pixi-scrollbox/issues/40#issuecomment-860274845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM6NXG7S56Z7ZYBISL7L4LTSUR2XANCNFSM46RRMZHA.

Can you tell me, or point to an example, how to import pixi-scrollbox into jsfiddle?  I'm getting "TypeError: PIXI.Scrollbox is not a constructor" errors, even though I've linked the cdn.  I'm a complete noob as regards jsfiddle.  My stuff is normally embedded in Meteor/Node.

Well, I have a jsfiddle going at https://jsfiddle.net/gischer/5cjzksur/88/  It doesn't work because it gives the error "Scrollbox is not a construtor"  I've put the cdn into jsfiddle, but it doesn't seem to help.

having this issue as well. i'll put together a jsfiddle for you tomorrow, as the one provided above did not work for me at all.

https://jsfiddle.net/slyduda/p3Ljc0ht/173/
I also had trouble importing Scrollbox as a CDN so I attached the whole script in there. My code is at the bottom. The demo should demonstrate two things:

When you mouse leaves (mouseout) an object immediately outside of the scrollbox the event is not triggered
When you mouseout of an object while the mouse is still inside the scrollbox the event IS triggered

My issue is actually worse because in this demo it appears that upon reentry, the mouse event will fire, but for some reason mine doesn't. That's okay though because the main issue is the blocking of the event outside of the scrollbox.
I know its not a click event but I thought this thread may be relevant.


[BUG] Agent metric name validation is inconsistent with documentation

Agent Environment

Cluster Agent version 7.42.0
Describe what happened:
I'm setting up an HPA using an external Datadog Metric and seeing errors in the cluster agent.
HPA Config
apiVersion: autoscaling/v2
kind: HorizontalPodAutoscaler
metadata:
  name: dx-test-deployment
  namespace: dev-dx-test-deployment
spec:
  maxReplicas: 10
  metrics:
  - external:
      metric:
        name: datadogmetric@dev-dx-test-deployment:cpu
      target:
        type: Value
        value: "50"
    type: External
  minReplicas: 2
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: dx-test-deployment

Datadog Metric Config
---
apiVersion: datadoghq.com/v1alpha1
kind: DatadogMetric
metadata:
  name: cpu
spec:
  query: avg:kubernetes.cpu.usage.total{kube_app_name:dx-test-deployment}.rollup(30)

Resulting error
2023-06-28 02:51:31 UTC | CLUSTER | ERROR | (pkg/util/kubernetes/autoscalers/autoscalers.go:108 in inspectHPAv2) | cannot build external metric value for HPA dev-dx-test-deployment/dx-test-deployment, skipping: metric name "datadogmetric@dev-dx-test-deployment:cpu" is invalid

Checking the source here, it does seem that the name violates the regexp. But, this is unexpected as its in contradiction with the documentation which uses @, : and -.
Describe what you expected:
I expected the metric to be properly queried and calculated for the HPA to utilize.
Steps to reproduce the issue:

Spin up a fresh cluster (I used EKS)
Setup datadog agent with cluster agent and metrics enabled per the documentation here
Create a datadogmetric in a namespace with a - or that uses a name with a -.
Create a deployment / HPA that utilizes the metric.
Observe the datadog-cluster-agent for errors.

Additional environment details (Operating System, Cloud provider, etc):
Kubernetes 1.24 / EKS

I'll add that utilizing an inlined metric like
  - external:
      metric:
        name: kubernetes.cpu.usage.total
        selector:
          matchLabels:
            kube_app_name: dx-test-deployment
      target:
        type: Value
        value: "50"
    type: External

Works as expected. Presumably because its able to get past the validation.

The fix for this is to make sure the cluster agent has DD_EXTERNAL_METRICS_PROVIDER_USE_DATADOGMETRIC_CRD. This was not set by the helm chart for some reason, but is ultimately not an issue with the agent itself. If there's a bug its with the helm chart, so I'll close.


Create application-test.properties

@Scythd Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Looks like a hard-coded database URL for a host that won't work for anyone else. So probably not a good idea for this project.


[Slack] Add messages to Later with reminder

Extension
https://www.raycast.com/mommertf/slack
Description
Create a command to provide the following:

Send a message "Get bread" (user input) to user "slackbot", or to "me"
Set a default reminder of 1h

Some nice to have features:

The default reminder duration should be customizable.
The reminder could be parsed from the message content, i.e. "Get bread tomorrow" or "Get bread in 1h"

Who will benefit from this feature?
People using Slack as a task manager, with reminders on messages. This method is more flexible and useful than the built-in Reminders feature in Slack.
Anything else?
No response

I could potentially take this on if other people feel like it would be useful? We don't have a way to send messages via Raycast but maybe we could?
I think this could just be a more generic "Send Message" command. Could use the  layout to pull all users/channels for who you want to send it to, the message itself and then a time for which you want to send (now, 1hr, or set specific datetime). Could use the send or schedule message API?

It's been a little bit but now that the holidays are over I've started to work on this if you want to assign this over to me..


Ctrl + Shift _ o does not show all methods in my controller

When I try to see all methods in my controller, I get only a partly list, some methods are not shown eventhough they are there!

VSCode Version: Code 1.19.3 (7c4205b5c6e52a53b81c69d2b2dc8a627abaa0ba, 2018-01-25T10:36:34.867Z)
OS Version: Windows_NT x64 10.0.16299
Extensions:

Extension
Author (truncated)
Version

Bookmarks
ale
0.18.1

Ruby
reb
0.16.0

highlight-matching-tag
vin
0.4.4

simple-ruby-erb
vor
0.2.1

Steps to Reproduce:

Reproduces without extensions: Yes/No

Please file this issue against your ruby extension


Add the Geant4 Monte Carlo radiation transport toolkit and its Python bindings.

I was just working my way through all the motif packages. I can't evaluate this expression. This is due to the missing license "g4sl" (https://geant4.web.cern.ch/geant4/license/LICENSE.html) in the licenses.nix file.


Windows: buttons blank, Drawer + MenuItems borked

Problem Description
Was getting high GA bounce rate for Windows users on Jobpig; investigated and noticed all the buttons were blank (white on white?). Additionally, Drawer + MenuItems are whack. Here's how it looks on Mac, and here on Windows. Seems to be Windows-specific, not browser - tried FF, Chrome, IE. I can investigate versions, but they all 3 look the same. You can play with it yourself at jobpigapp.com - notice the white buttons (Windows), click Login and notice more. All the code here, Auth dialog as an example. It's all pretty standard MUI - I'm not doing anything CSS-wild.
Versions

Material-UI: 0.15.0-beta.1
React: 15.0.1

Seems in fact to be a 0.15.0-beta.1 issue. I reverted back to 0.14.4 and all's well. (actually upgraded hoping for a separate fix to modals which I'd theorized was the Windows problem... peter_griffin_blinds.gif)

@lefnire beta looks fine on my end

@lefnire make sure you are following the up to date usage guidelines:
https://github.com/callemall/material-ui#usage

@lefnire Looks like you are.
Did you have any console errors after upgrading?

And here's chrome, windows, your site:

@nathanmarks sorry should have been clearer on the second post. I reverted back to 0.14.4 on production, so what you're seeing there is "back to good". upgrade commit | post-upgrade code | revert commit. I'll see if I can get the "bad" version (post 0.15-beta.1) up somewhere to bee played with. In the meantime, the screenshots. I didn't see console errors; save the upgrade warnings I used to guide my upgrade process (solved them all). It seemed CSS, but I was too in over my head inspecting bezier transforms and all that jazz.
Yeah, it was weird what with material-ui.com obviously working on Windows; but not my stuff, which was very unadulterated copy-pasta. I'll see if I can come up with more details.

@lefnire does your repo work if i just boot the client not the server?

Yes it does. (It'll make some call to :3002/cron, you can ignore it) npm run client. Thanks for looking into!

Oo, I will say - the homepage works without the server; but to see the LeftNav (now Drawer) you'll need to log in, which requires the server. The worst of it is installing Postgres & createdb jobs. Happy to bang it out on Gitter if desired.

@lefnire just trying to do the least possible 😄

@lefnire can you reproduce this with a simple example?

@lefnire Looks fine to me, on https://github.com/lefnire/jobpig/commit/140dd9c705dc08ec3d898deb0d39d09dd2d158d3

**deleted my last reply, forgot to reinstall dependencies 😄 **

@lefnire Can you check with the beta2 on your end?

@nathanmarks awesome, will do when I find a moment. Thanks!

@lefnire Any chance you were able to reproduce with the latest beta?

sorry, haven't had a chance yet - hopefully tomorrow

@nathanmarks https://github.com/lefnire/jobpig/tree/lefnire/MUI-0.15 ; beta2, same visuals. Note I also have Bootstrap CSS in there (I needed full-screen modals on mobile); even if you remove the bootstrap  you'll see the same artifacts. Let me know if I should try adding/removing anything

@lefnire I'm not sure what to say as I wasn't seeing it on my end. Sounds weird but have you tried another computer?

I feel ya, it's quite a rabbit hole with no easy leads. If beta2 docs are working on your end, it's obviously some conflict on my end (with another module or such). I'll keep exploring and let you know if anything relevant for MUI's purposes crops up. Thanks for bearing with me!

@lefnire How do the beta 2 docs look on your end?

@lefnire It was actually the beta2 version on your repo that was working for me -- i went back and installed at your commit hash https://github.com/lefnire/jobpig/commit/140dd9c705dc08ec3d898deb0d39d09dd2d158d3

@lefnire that's where this screenshot is from: https://cloud.githubusercontent.com/assets/4420103/14741646/c9bbc244-0865-11e6-85a7-0c795cc2b5fd.png

will check when I get a chance

BTW this is fixed for my case with the 0.15.0 release (whether intentional or side-effect). :+1:


PandasTools.LoadSDF: very verbose infinite loop on bad input

Describe the bug
When passing float('nan') to PandasTools.LoadSDF it seems to get stuck in an infinite loop:
[19:47:53] Unexpected error hit on line 1
[19:47:53] ERROR: moving to the beginning of the next molecule
[19:47:53] Unexpected error hit on line 2
[19:47:53] ERROR: moving to the beginning of the next molecule
[19:47:53] Unexpected error hit on line 3
# snip #

This seems to go on forever (I waited about five minutes before interrupting) with increasing line number. The stack trace after interrupting shows that the problem might be in ForwardSDMolSupplier:
[19:52:45] Unexpected error hit on line 16724731
^C
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/Luis/.conda/envs/rdkit/lib/python3.9/site-packages/rdkit/Chem/PandasTools.py", line 473, in LoadSDF
    for i, mol in enumerate(Chem.ForwardSDMolSupplier(f, sanitize=sanitize, removeHs=removeHs,
                                strictParsing=strictParsing))
KeyboardInterrupt

To Reproduce
from rdkit.Chem.PandasTools import LoadSDF
LoadSDF(float('nan'))

Expected behavior
Raising and error informing about an invalid input file.
Configuration (please complete the following information):

RDKit version: Found with 2021.09.1 and 20121.09.2
OS: Fedora 34
Python version (if relevant): 3.9.7
Are you using conda? Yes
If you are using conda, which channel did you install the rdkit from? Tested both rdkit and conda-forge

Additional context
I encountered this bug by looping over a list of SD file names to load. The list was actually a column of a DataFrame with some missing values, hence the nans. Now I see that the error is totally preventable on my end by checking for missing values. Nevertheless this is not the behaviour one expects from a function which loads files.
If this was a Python issue I'd be happy to come up with a fix myself. But I don't know c++ to investigate ForwardSDMolSupplier.

confirmed.

Something similar happened to me as I tried to load the qm8.sdf dataset from MoleculeNet (https://moleculenet.org/datasets-1) with the following code snippet:
from pathlib import Path
from rdkit.Chem.PandasTools import LoadSDF
def load_qm8(dataset_path):
path = dataset_path.parent / Path("gdb8/qm8.sdf")
print(path.exists())
df = LoadSDF(path, smilesName='SMILES')
The loop on line 470-471 in rdkit.Chem.PandasTools.py iterates passed the number of molecules that are supposed to exist in the dataset and all mol variables are None (line 473) so it seems like something is wrong with Chem.ForwardSDMolSupplier


Restore functionality for manually adding users to the database.

@jpevarnek to review as well.

:+1:  Nice!


Add a Gitter chat badge to README.rst

ethereum/pyethapp now has a Chat Room on Gitter
@heikoheiko has just created a chat room. You can visit it here: https://gitter.im/ethereum/pyethapp.
This pull-request adds this badge to your README.rst:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Coverage remained the same at 28.12% when pulling<PHONE_NUMBER>689f9bdcfcc30ea964515dd0b4fad7 on gitter-badger:gitter-badge into 1da9ef0c035d019404a3c709ddc436f7c4434562 on ethereum:master.


Quick Info stopped showing

I just updated to the latest oni and I've noticed that the quick info stopped showing entirely. It used to stop after about 30 min but now it doesn't work even after restarting oni!
Oni Config
const activate = (oni) => {
  oni.input.unbind('<C-p>')
  oni.input.unbind('<Tab>')
  oni.input.unbind('<C-n>')
  oni.input.unbind('<C-tab>')
  oni.input.unbind('<m-e>')
  oni.input.unbind('<down>')
  oni.input.unbind('<up>')

  oni.input.bind('<esc>', 'language.rename.cancel')

  oni.input.bind(['<shift-tab>', '<up>'], 'contextMenu.previous')
  oni.input.bind(['<tab>', '<down>'], 'contextMenu.next')
  oni.input.bind(['<shift-tab>', '<up>'], 'menu.previous')
  oni.input.bind(['<tab>', '<down>'], 'menu.next')
  oni.input.bind('<m-w>', () => document.querySelector('.tab.selected .corner.enable-hover').click())
}

const deactivate = () => {
}

module.exports = {
   activate,
   deactivate,
  "editor.maximizeScreenOnStart": true,
  "oni.useDefaultConfig": false,
  "oni.loadInitVim": true,
  "ui.fontSmoothing": "subpixel-antialiased",
  "ui.colorscheme": "molokai",
  "editor.fontFamily": "Hack",
  "ui.fontFamily": "Hack"
}

Do you have any related errors on the DevTools?

It's possible, currently seeing a few errors in the console.

@MikaAK can you please provide reproduction steps?
For instance, does the following work as expected?
:e a.ts<cr>aMath.floor(

Most of these errors occur on startup. The child prop errors are already listed in #1261
It works at the start but at some point during my typing and after about 900 of these prop errors it stops working.

Didn't try to get to the bottom of that yet, but might https://github.com/onivim/oni/pull/1457 be related? (you can try using the current master & see if it still happens).
I too had similar problem, though I'm not sure what the pattern is.
It mostly works, but sometimes it doesn't...

@MikaAK unfortunately this error is still present on master I've been looking into it and it seems to be that lsp throws an error not sure why which takes down completion. Discussed this in #1513 if you're interested in the discussion so far but still a [WIP]


Write a Python program to convert miles to kilometers

Description
Write a Python program to convert miles to kilometers
1 mile = 1.609 kilometers
Input  : 5
Output : 8.04672

How to contribute

Save the solution in convert-miles-to-kilometers.py file
Add convert-miles-to-kilometers.py file in convert-miles-to-kilometers folder

!assign

#1147  check it


make install depends on setup

Thanks to @afalcioni I just realized that the make install command depends on the setup command. This implies that requirements.txt is installed before installing the package but this should not be needed since installing a package with pip should already install all its dependencies.
On the other hand, this behaviour could conflict with those commands that pre-require setup_ci since this implies that running install after setup_ci overwrites requirements_ci with requirements, removing some necessary package (e.g. pytest, mypy, flake8,...).

yes, good catch.
My proposal would be to make it such that make install does not install dependencies (flag --no-deps on pip). Within make we have other dedicated instructions that take care of dependencies installation, and as you pointed out, this can generate conflicts and inconsistent behaviour. This was install dependency can be seamlessly used both with setup_ci and setup dependecies


Use new Amazon API Gateway to invoke Lambda functions

I set up an Amazaon API Gateway, so now we can do this to render scores:
jQuery.post('https://7icpm9qr6a.execute-api.us-west-2.amazonaws.com/prod/stable', JSON.stringify({
  body: "% LilyBin\n\\score{\n\t{\n\t\tc' c' d' e' f' f' f'2\n\t}\n\n\t\\layout{}\n\t\\midi{}\n}\n",
  key: "abcdefg"
}))

We almost don't need the EC2 instance after this! Then we can use S3 or GitHub pages to host the front end.

This doesn't look safe from DDoS. What a person cooks up thousands of these requests in a few minutes? The cost that would incur is difficult to control. Does AWS offer any kind of protection or throttling against that?

I'm at the AWS Summit in NY and just saw a presentation on this. It is awesome. You can throttle requests with some limited settings. I assume this service will continue to gain more features.

Those are the controls available now. Like Scott, I expect they'll add more features.

Okay, I guess 200/100 is pretty good for now.

I finally found time to work on this again. Hopefully I'll fix this over this weekend.

Just converted prepare_preview to API gateway. The only thing left now is Gist integration.

Looks like this issue could be closed, after this commit? https://github.com/LilyBin/LilyBin/commit/e761102cd4f5b07d7fcfe93d650b37e6730c4f39

Yep :)


[Question] How could get the version of processor?

Hi, guys. I wonder how could I get the processor version such as gfx1030 with HIP runtime api on AMDGPU. I could get the similar info(compute version) on NVIDIA GPU with cuDeviceGetAttribute
and  CU_DEVICE_ATTRIBUTE_COMPUTE_CAPABILITY_MAJOR.
If it is not appropriate to ask questions here, could you point me to the HIP-related forum community?

https://gitlab.com/cscs-ci/ci-testing/ault-amdgpu/-/blob/master/helloworld.cpp


Support bplist16

Adds support to decode bplist16 data. Reuses same decoding mechanism as bplist17.

Thanks @t0rr3sp3dr0


Gulp Serve Fails (again)

There is another issue raised that was marked as closed from a few months ago. On a fresh build + npm install, I get the following error when running gulp:
[20:58:05] gulp-inject 2 files into index.scss.

events.js:141
      throw er; // Unhandled 'error' event
      ^
Error: spawn sass ENOENT
    at exports._errnoException (util.js:837:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:178:32)
    at onErrorNT (internal/child_process.js:344:16)
    at doNTCallback2 (node.js:429:9)
    at process._tickCallback (node.js:343:17)

This is remedied by changing line 41 of /gulp/styles.js to:
.pipe($.rubySass(sassOptions)
      .on('error', function (err) {
        console.error('Error!', err.message);
     })
    )

Thanks for fixing this!

Fixed. Thanks for proposal


Add support for enumerated lists in the ListItem detection logic

The goal of this issue is to update the ListItem detection logic to include enumerates lists with formats such as (a), (1) and (i). Currently we only detect unicode bullet characters.

Are there specific files that would be good to use as test cases?

is this done?

Closing because longer than 180 days; reopen with comment if the issue is still relevant


Avoid a 32 kB file allocation on every bitBucketFilePutter.Put

io.Copy usually allocates a 32kB buffer, and due to the large number of files processed by tar-split, this shows up in Go profiles as a very large alloc_space total.
It doesn't seem to actually be a measurable problem in any way, but we can allocate the buffer only once per tar-split creation, at no additional cost to existing allocations, so let's do so, and remove the distraction.

Ref. https://github.com/containers/image/issues/1349 , especially https://github.com/containers/image/issues/1349#issuecomment-903009519 , and data in https://github.com/containers/image/pull/1352 .

good stuff! nice work @mtrmac
LGTM
I'll likely got ahead and tag a point release with this in it.
cc @theJeztah @tonistiigi for moby/docker visibility too

https://github.com/vbatts/tar-split/releases/tag/v0.11.2


feat(): previews template customization

Hello !
Ok, a new PR for this feature, on the right branch and with the example as an experimental feature "F3".
I have also written 2 unit tests for the previews component; they use intermediary components that build a template (as an easy way to get TemplateRef to give as the appropriate PreviewsComponent input). One uses a totally "different" template. The other one uses the "default" template. Each test checks that the proper HTML elements are present.
A couple questions!

I used example id 902 for this new example. Within openModal(), I didn't add yet another optional parameters to provide a templateRef, but has based it directly on the 902 id. Optional parameter/magic number, what's your choice here?
If you will, I'd like to propose a description on how to use the previews customization, like on a new left item here (https://ks89.github.io/angular-modal-gallery-2021-v9.github.io/features), but where could I do that? It appears this page is not part of this repo?

Also, 2 hmm... almost related things:

what do you think of adding a 'cursor: pointer' css to the preview images (and actually to the whole preview container I set up, which reacts to (click) and (keyup)) ?
about the initial need for preview customization, I've noted this:
Ultimately, I'd like to use picture-specific information which (if I'm not wrong) can not fit anywhere in the ModalImage object, such as the picture date for instance. So to fully be able to do so, an evolution to add a custom Object in ModalImage, which data could be used in template customizations, would be necessary, so that's to be discussed.
What's your thought about that?

Be well !

First of all thank you for the PR and sorry for my delay.
I add some comments quoting your description.

Ok, a new PR for this feature, on the right branch and with the example as an experimental feature "F3".

Good

I have also written 2 unit tests for the previews component; they use intermediary components that build a template (as an easy way to get TemplateRef to give as the appropriate PreviewsComponent input). One uses a totally "different" template. The other one uses the "default" template. Each test checks that the proper HTML elements are present.

Really appreciated 👍

I used example id 902 for this new example. Within openModal(), I didn't add yet another optional parameters to provide a templateRef, but has based it directly on the 902 id. Optional parameter/magic number, what's your choice here?

I suggest to write a custom method only for this example to keep all others easier to understand. For example something like this:
  openModalWithPreviewsTemplate(id: number, imagesArrayToUse: Image[], imageIndex: number, libConfig?: ModalLibConfig): void {
    if(imagesArrayToUse.length === 0) {
      console.error('Cannot open modal-gallery because images array cannot be empty');
      return;
    }
    if(imageIndex > imagesArrayToUse.length - 1) {
      console.error('Cannot open modal-gallery because imageIndex must be valid');
      return;
    }
    const imageToShow: Image = imagesArrayToUse[imageIndex];
    const dialogRef: ModalGalleryRef = this.modalGalleryService.open({
      id,
      images: imagesArrayToUse,
      currentImage: imageToShow,
      libConfig,
      previewsTemplate: this.previewsTemplate,
    } as ModalGalleryConfig) as ModalGalleryRef;
  }

If you will, I'd like to propose a description on how to use the previews customization, like on a new left item here (https://ks89.github.io/angular-modal-gallery-2021-v9.github.io/features), but where could I do that? It appears this page is not part of this repo?

Very good idea. This is the doc website repo https://github.com/Ks89/angular-modal-gallery-2021-v9.github.io
Feel free to create a PR in that repo.

what do you think of adding a 'cursor: pointer' css to the preview images (and actually to the whole preview container I set up, which reacts to (click) and (keyup)) ?

Absolutely yes, this is important for accessibility. Originally I added both of them, so It's important to preserve this behaviour.

Ultimately, I'd like to use picture-specific information which (if I'm not wrong) can not fit anywhere in the ModalImage object, such as the picture date for instance. So to fully be able to do so, an evolution to add a custom Object in ModalImage, which data could be used in template customizations, would be necessary, so that's to be discussed.
What's your thought about that?

Yes you are right, but probably it's better to finish and merge this PR to release 9.1.0 and then work on that feature in a new PR. This will keep this PR smaller and we can release a new version of the library faster.
Also, we should discuss how to add that info to the ModalImage object. We need a trade off between a generic object of type any and something structured but more stricter.

Hi there!

I suggest to write a custom method only for this example to keep all others easier to understand.

Created a new method as you suggested (I'd totally missed the fact several of these methods existed).

Absolutely yes, this is important for accessibility. Originally I added both of them, so It's important to preserve this behaviour.

Behaviour back on track! Sorry for the mistake.
Now there's a new distinct class preview-container, in between preview-containers and preview-image. This new class handles the lateral margin and the cursor look upon the preview container.

Very good idea. This is the doc website repo https://github.com/Ks89/angular-modal-gallery-2021-v9.github.io Feel free to create a PR in that repo.

Perfect. I'll have a look at it.

Yes you are right, but probably it's better to finish and merge this PR to release 9.1.0 and then work on that feature in a new PR. This will keep this PR smaller and we can release a new version of the library faster. Also, we should discuss how to add that info to the ModalImage object. We need a trade off between a generic object of type any and something structured but more stricter.

Sure thing for the release. I was mentioning that point only to progress on the reflection about it.
In the meantime, I believe that as a user it's possible to tweak the passed ModalImages or even to create an interface extending it, to fill extra information in. I'll check that on my side after the 9.1.0 release and let you know!

Hi there! Just a heads-up.
I'll create a new PR for the little things I was mentionning here, because it will involve several changes, and it'd be better to keep things separated. So if you think this PR #254 is fine, feel free to merge it!

Ok Thanks. Give a couple of days to test it.

@locinus merged and released as 9.1.0-beta.1
I also updated official examples in /examples folder and tested with angular universal.
Please, can you try the new bundled version with npm i @ks89/angular-modal-gallery@beta?
When you'll be ready, please create the PR also on https://github.com/Ks89/angular-modal-gallery-2021-v9.github.io with the doc and a live example.
If everything will be ok, we can release it as 9.1.0 (as stable).


[Bug] Official website spark jumps 404

Search before asking

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

Paimon version
最新版
Compute Engine
spark
Minimal reproduce step

从官网的spark进去回到这个地址：https://paimon.apache.org/docs/master/engines/spark3/ 404，应改成https://paimon.apache.org/docs/master/engines/spark/
What doesn't meet your expectations?
地址跳转
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

This Issue has been fixed https://github.com/apache/incubator-paimon-website/pull/2

Maybe you need to clear your browser cache.


Add unit-test for ValidateCPUQuota

Ⅰ. Issue Description
Add unit-test for ValidateCPUQuota method which locate on apis/opts/cpu.go.
#1638
Ⅱ. Describe what happened
Add unit-test for ValidateCPUQuota method which locate on apis/opts/cpu.go
Ⅲ. Describe what you expected to happen
All test case passed.
Ⅳ. How to reproduce it (as minimally and precisely as possible)
Ⅴ. Anything else we need to know?
Baiji 262 group 2.
Ⅵ. Environment:
pouch version (use pouch version):go1.10.3 darwin/amd64
OS (e.g. from /etc/os-release):  CentOS linux 7
Kernel (e.g. uname -a):3.10.0
Install tools: visualbox git go
Others:none

Codecov Report

Merging #1692 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1692      +/-   ##
==========================================
+ Coverage   41.16%   41.21%   +0.04%
==========================================
  Files         274      274
  Lines       18091    18091
==========================================
+ Hits         7447     7456       +9
+ Misses       9729     9723       -6
+ Partials      915      912       -3

Impacted Files
Coverage Δ

daemon/mgr/system.go
75% <0%> (-1.73%)
:arrow_down:

ctrd/image.go
77.47% <0%> (-0.55%)
:arrow_down:

daemon/mgr/container.go
50.23% <0%> (+0.15%)
:arrow_up:

daemon/logger/jsonfile/utils.go
73.17% <0%> (+1.62%)
:arrow_up:

apis/server/utils.go
66.66% <0%> (+4.76%)
:arrow_up:

apis/opts/cpu.go
50% <0%> (+50%)
:arrow_up:

LGTM


Enhancement Request: Option to hide menu rather than navigating back

I noticed that it currently navigates back from a submenu when clicked on an item. What about adding an option to hide the menu altogether rather than navigating back?
Demo:

Item clicks will always dismiss the menu, but the sample app intentionally navigates back instead: https://github.com/saket/cascade/blob/891031ab4d52dc00f41fdb0fb1e094279e92143c/sample/src/main/java/me/saket/cascade/sample/MainActivity.kt#L82-L87


can i use jdk-11.0.14+9 at your website(https://adoptium.net/?variant=openjdk11) commercially!!

hello sir,
i want to ask that can i use jdk-11.0.14+9 at your website(https://adoptium.net/?variant=openjdk11) with android game development commercially or do i have to buy a license for that. and also if it is free
to use commercially then do i have to give credits to the jdk-11.0.14+9 maker or something like that.
thankyou

Yes, it's free to use (no restrictions) and free in cost!


Feature/api crypto get products  [ 1 ]

Closes https://github.com/Real-Dev-Squad/website-crypto/issues/154

This is being deleted as the contracts were updated, and the project is on hold currently


Finished LazyBailOut and added OOPJIT support.

currently blocking this PR from merging:

PDF.js support (octane benchmark currently fails on the pdf.js test)


Update Deploy command medium.ipynb

Update model package to v1.1.1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat(hz): add (handler.go) more template parameters

What type of PR is this?

Check the PR title.

[ ] This PR title match the format: <type>(optional scope): <description>.
[ ] The description of this PR title is user-oriented and clear enough for others to understand.
[ ] Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo.

(Optional) Translate the PR title into Chinese.
hz 生成handler.go 添加参数,ServiceName
(Optional) More detail description for this PR(en: English/zh: Chinese).

en:add (handler.go) more template parameters
zh(optional):
(Optional) Which issue(s) this PR fixes:

(Optional) The PR that updates user documentation:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Repeat pr?  https://github.com/cloudwego/hertz/pull/795


[CIT-524] Instable tests that do local communication are disabled

Proposed changes
Temporarily disabled two instable testcases.
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactorings that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue
https://cumulocity.atlassian.net/browse/CIT-524
Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Codecov Report

Merging #388 (2e463df) into main (00f3f3d) will increase coverage by 1.36%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #388      +/-   ##
==========================================
+ Coverage   67.33%   68.70%   +1.36%
==========================================
  Files          66      113      +47
  Lines        3885     8145    +4260
==========================================
+ Hits         2616     5596    +2980
- Misses       1269     2549    +1280

Impacted Files
Coverage Δ

tedge/src/cli/certificate/error.rs
12.19% <0.00%> (-17.81%)
:arrow_down:

tedge/src/cli/connect/common_mosquitto_config.rs
98.30% <0.00%> (-1.70%)
:arrow_down:

mapper/tedge_mapper/src/az_converter.rs
97.75% <0.00%> (-0.37%)
:arrow_down:

...er/cumulocity/c8y_translator_lib/src/serializer.rs
96.83% <0.00%> (-0.32%)
:arrow_down:

tedge/src/cli/certificate/create.rs
90.59% <0.00%> (-0.20%)
:arrow_down:

tedge/src/cli/connect/bridge_config.rs
97.52% <0.00%> (-0.05%)
:arrow_down:

tedge/src/main.rs
0.00% <0.00%> (ø)

tedge/src/command.rs
0.00% <0.00%> (ø)

tedge/src/cli/connect/cli.rs
0.00% <0.00%> (ø)

tedge/src/cli/mqtt/publish.rs
0.00% <0.00%> (ø)

... and 66 more

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 00f3f3d...2e463df. Read the comment docs.


No assert generated in Spring method

To Reproduce
Consider the project spring-boot-2-rest-service-filtering from https://github.com/in28minutes/spring-boot-examples
Generate tests for method FilteringController.retrieveSomeBean
Expected behavior
Generated test has at least one assert.
Actual behavior
Generated test looks as follows:
@Test
    @DisplayName("retrieveSomeBean: SimpleBeanPropertyFilterFilterOutAllExcept -> return mapping")
    public void testRetrieveSomeBean_MappingJacksonValueSetFilters() {
        org.mockito.MockedConstruction mockedConstruction = null;
        org.mockito.MockedConstruction mockedConstruction1 = null;
        org.mockito.MockedConstruction mockedConstruction2 = null;
        org.mockito.MockedStatic mockedStatic = null;
        try {
            mockedConstruction = mockConstruction(SomeBean.class, (SomeBean someBeanMock, org.mockito.MockedConstruction.Context context) -> {
            });
            mockedConstruction1 = mockConstruction(com.fasterxml.jackson.databind.ser.impl.SimpleFilterProvider.class, (com.fasterxml.jackson.databind.ser.impl.SimpleFilterProvider simpleFilterProviderMock, org.mockito.MockedConstruction.Context context1) -> (when(simpleFilterProviderMock.addFilter(any(String.class), any(SimpleBeanPropertyFilter.class)))).thenReturn(null));
            mockedConstruction2 = mockConstruction(MappingJacksonValue.class, (MappingJacksonValue mappingJacksonValueMock, org.mockito.MockedConstruction.Context context2) -> {
            });
            mockedStatic = mockStatic(SimpleBeanPropertyFilter.class);
            (mockedStatic.when(() -> SimpleBeanPropertyFilter.filterOutAllExcept(any(String[].class)))).thenReturn(((SimpleBeanPropertyFilter) null));
            FilteringController filteringController = new FilteringController();

            MappingJacksonValue actual = filteringController.retrieveSomeBean();

            MappingJacksonValue expectedMock = mock(MappingJacksonValue.class);
            ((doNothing()).when(expectedMock)).setFilters(any());
        } finally {
            mockedConstruction.close();
            mockedConstruction1.close();
            mockedConstruction2.close();
            mockedStatic.close();
        }
    }

Additional context
Creating mock for expected variable is also very strange!

Actually, this is not Spring specific problem. Expected and actual variables are correct. Then they need to be compared with deepEquals. It's current implementation compared models field by field. As far as expectedMock does not have any fields (we just have some info about the behavior on the mocked function), there is nothing to compare. Behavior will be similar on non-Spring project function with the return type out of package under test.


Evaluate Editor UI Frameworks

While the current editor is in a very good state and WinForms is fine, it does have the major drawback of being Windows only - and Mono can't really fix this in a good way.
This is a long term investigation. Neither do I plan to start implementing something anytime soon, nor do I currently have the resources to do so for a change on this scale - but it can't hurt to do some research.
Required Traits

Cross-Platform support for major desktop platforms: Windows, Linux, MacOS
A versatile Docking Control like the DockPanelSuite that is currently used. Really, there is no way around this.
An OpenGL widget that can be used to render the game somewhere in the editor.
High-DPI awareness. (See issue #140)

Desirable Traits

Written primarily in managed code / C# - if it can be avoided, don't introduce natively compiled things into the otherwise managed Duality world.
Being able to deploy the same binaries on all of the supported systems with runtime detection what backend to use - similar to the goal of Duality's cross platform support.
Being able to embed the UI framework itself inside the game, allowing to use the same UI framework both ingame and in the editor - which would make developing editor plugins a lot more straightforward.
Separate style from structure, providing proper skinning support.
Separate model from view, hopefully making the overall UI code cleaner.

There is nothing wrong with WinForms or the current editor, and while there sure is room for improvement, a UI framework change would be a considerable effort that should not be decided easily. This issue is not about starting something, but about looking around what options there are for the long term future.

The Perspex framework looks promising, but it is unclear whether some of the requirements are met. I have created an issue to talk about their future plans.

Maybe Qt?
It's cross platform, can use OpenGL and can do High-DPI.
I'm don't think you can deploy the same binaries to multiple platforms but other desirable traits should be possible.
A big drawback would be having to rely on some 3rd party project for C# bindings.

There's Eto.Forms https://github.com/picoe/Eto
However, I'm not sure if you can use the docking controls with it. Perhaps you could fork the project and adapt it to Eto?

Maybe Qt?

I really like Qt, but this feels like a bit of an overkill - it's essentially a whole framework, complete with everything down to file system access, strings, collections and all sorts of things. I don't really want to introduce a "second .Net framework" here.
Also, I'd like to avoid unmanaged code. (Added a new trait above, forgot that one :) )

There's Eto.Forms https://github.com/picoe/Eto

This one looks neat. Might be something to keep an eye on.

Took a look at Eto and it seems like the overall concept is to wrap native UI in an abstract way. Favoring Perspex right now for its custom drawing / custom control logic approach, as this has a higher potential to mesh well with the whole "same UI for ingame and editor" idea.
One drawback of Perspex at this point seems to be a focus on very high-level graphics, requiring each backend to implement a full vector rendering stack, which would be a bit too much for Duality purposes. It also seems a lot more heavy-weight than what Duality would really need. Then again, it seems to provide a lot of great functionality.
Research continues.

Apparently, this has come up before. Linking to the old issue for historic reasons: #159

Maybe Xwt?
https://github.com/mono/xwt

Mono launched SkiaSharp, cross platform 2D Graphics API based on Google's Skia library.
You gotta see this.
https://github.com/mono/SkiaSharp

The framework that @sappho192 suggested looks great (I had a quick, quick look). Maybe it could be the first step towards natively supporting android?

@RockyTV Maybe yes, but maybe not. SkiaSharp just provides graphics API so we need to construct UI framework.

SkiaSharp just provides graphics API so we need to construct UI framework.

I'm not really looking for a graphics-only framework though - if I was to create a custom UI, I might as well do it on top of Duality itself. Which I don't plan to do for practical reasons.

Maybe it could be the first step towards natively supporting android?

Maybe - but providing additional core backends is a different issue than this one here, and should be discussed elsewhere :) Android is not on my radar right now to be honest.

Hey. Just a thought to maybe mull over. You could go a similar route that Unreal Engine decided to go and just render your UI using the engine itself and develop your own widgets, controls, etc. This would be a lot more work but you would be able to hit all of your required and desirable traits.
The way Unreal Engine did it is by have several layers of UI functionality. There is the Slate api which is your low level UI system and its at this level they implement all of the Unreal Editor UI. Then there is the UMG api which is built on top of the Slate api that implements the UI designer for game use and the game widgets.

Hey. Just a thought to maybe mull over. You could go a similar route that Unreal Engine decided to go and just render your UI using the engine itself and develop your own widgets, controls, etc. This would be a lot more work but you would be able to hit all of your required and desirable traits.

Yep, this is what I would consider the ideal solution. I don't see this happening in the foreseeable future though, simply because it would be so much work.

Spinning the thought a bit more, what would be required for Duality to build a UI system that is sophisticated enough to serve as a basis for the editor UI?

Multiple Windows.
More sophisticated window handling API, including event handling.

OpenTK may or may not be up to the task, there may be some OpenTK contribution involved.
Make sure its SDL2 backend would still work for the usual single-window game application.

An explicit main loop and message pump for launcher applications.
Global (rather than window-local) input handling for mouse and keyboard.
More sophisticated drawing API, presumably based on the existing Canvas.

Better transformation / "virtualization" capabilities.
First: 9-Patches.
Later: Gradients? Paths? Geometric algebra?

A more sophisticated text rendering system than the current FormattedText implementation can provide. It needs to be much faster and - potentially - more sophisticated / flexible.
A lot of consideration for UI drawing performance.

WinForms has a near-zero overhead while nothing is updated explicitly. Ideally, a replacement would match this as closely as possible.
Expect very complex UIs to be drawn and deal with it.

A layout engine.
A setup that allows skinning and customization to the degree that it's usable in-game.
All the basic UI controls in a very polished, usable state.

A thought for window management. Maybe adopt a model similar to Qt where you don't have to care and the simple rule is "widgets without a parent are in their own native window".
Label, Button, CheckBox, RadioButton, ScrollBar, Slider, TextArea, GroupBox, TabArea, Panel, ComboBox, TreeView, ContextMenu, Tool- and Menustrips
I've actually done this before, but this would need to step it up a bit.

More complex UI controls which are required by the editor specifically.

Most importantly, a DockingControl that is as usable and sophisticated as the current one.

A system design that supports encapsulation enough for a stable editor even when the game itself throws exceptions and leaves rendering, input or audio in an inconsistent state.

Also make sure no bit of code can cross the threshold between game and editor inadvertently.

...and then there's also the whole "porting the editor" set of tasks.
Summarizing this, a custom UI would be a really huge workload. Virtually the only way I would be able to pull this off would be if it was my actual job five days a week and even then things would take a noticeable amount of time as long as I don't have a small team helping me along the way as well.
And yes, it's already Open Source and contributors can help, but that only does so much. For a system of this size to succeed, there needs to be a significant amount of technical leadership, which also consumes a lot of time itself, and it's complicated by the fact that Open Source contributors tend to "come and go" - so a lot of the transferred and taught knowledge doesn't even stay in the project, but needs to be taught again when the next contributors arrive.
So.. yeah. This is probably the long version of my above "I don't see this happening in the foreseeable future".
@xgalaxy Please don't read this post as me trying to prove your idea impractical, but as an acknowledgement of its appeal to me and the project, yet I want to put out there what I'm up against if I was to approach something like this.

I don't want to recommend any particular library or offering, but has there been any thought into using an HTML5 renderer? Most of the offerings I can think of are true open source licensed (MIT or BSD), as most are also built upon the Chromium renderer.
It sounds bizarre, but when you really think about it, everything you need comes together. Most of the basic controls are natively supported by the rendering engine; everything is customizable; completely platform agnostic; easy to write in with lots of 3rd party WYSIWYG editors. HTML5 and CSS3 are completely dynamic too, so having a JUICY (do people still use that term?) interface that animates and reacts is fully supported without additional work.
I've personally never used an HTML5 renderer for this purpose, but from what I understand, implementing it for a real-time program breaks down to a handful of fundamental steps all of them share:

Setup your own real-time rendering context using OpenGL/DirectX/etc.
Initialize the HTML5 renderer; as far as I know, all libs setup and handle their own rendering context. Doesn't seem to be an issue though; I know for Ogre3D, someone implemented Chromium Embedded Framework for use inside the engine, and they just let it handle its own rendering context.
Load pages for it to render, local or fetched from the web.
Pass user input from your rendering context into the HTML5 renderer, so the page can be manipulated (handling roll-over animations, clicking controls, etc.)
Render the page to a texture, and import that into your own context, to be rendered however you like. From what everyone says, this is the most inefficient part, because you essentially need to copy and render a screen-sized texture twice. You render the HTML5 output to a texture, copy that texture data from the GPU to RAM, then copy it back onto the GPU so your own rendering context can use it, where it can be used as a traditional texture.

For handling a traditional orthographic overlay UI (such as the editor), you create a full-screen quad and just render the texture directly to it, with the source page being 1:1 with the rendered texture. Nothing stopping you however from treating the source page as a sprite-sheet, and displaying it on dynamic elements for in-game interactive UI. At least Chromium based renderers are entirely capable of rendering with transparency, so there's no additional work involved.

I don't want to recommend any particular library or offering, but has there been any thought into using an HTML5 renderer?

I actually have! Nice writeup of this approach by the way, thanks for adding it.
While I like the idea of separating structure (HTML), style (CSS) and logic (JS) completely, JavaScript is a terrible language that I don't want to drag into the Duality ecosystem. A also don't want any kind of secondary runtime / VM for this, so there would be the need for engineering a solution where the code part is still handled in C#, or differently put: I'd really emphasize the HTML Renderer part of this, as opposed to considering hosting an actual website in there.
This would of course rule out fetching anything from the web, as well as all the JS frameworks out there or even anything that contains a single line of JavaScript, which would probably rule out most pre-made UI components that are available on the web. It would mean to only use a very limited subset of web technology, basically only reducing it to the HTML structure / CSS style part and maybe even removing some more functionality that doesn't make sense outside the web context.
If you're following along until here, then you might notice some conceptual similarity of this very reduced HTML / CSS / C# logic combo with XAML. Which would put Perspex (now renamed to Avalonia) back into the spotlight.

Hate to resurrect this old ticket from the dead but I thought you'd find this library interesting:

Qml.Net - Qt/Qml integration/support for .NET
https://github.com/pauldotknopf/Qml.Net

What about Uno?

It was just announced that there will be a cross-platform UI solution from Microsoft starting with .NET 6, could be worth a look as well:

https://devblogs.microsoft.com/dotnet/introducing-net-multi-platform-app-ui/
https://github.com/dotnet/maui
Roadmap (Wiki)
Status (Wiki)

Iam watching the The Journey to One .NET. which should mentions the new cross platform UI tonight. Hoping to hear more details.


how to trigger the WebAPI

Dear david,
thanks for this nice rebuild from the algorithm in c#.
a short question:
how can i trigger the web api, i have seen theres running a webserver with a api, must i send the data to it via json?
there is nothing in the documentation, when you me we could create a documentation together.
Greets.

Hello!
The easiest way to launch the web app is to open the ContainerPacking solution file in Visual Studio. Then in the solution explorer right click the DemoApp project and click Set as StartUp Project. And then right click the container-packing.html file in the DemoApp project and click Set as Start Page. Then launch the app with F5. That should set everything up so you can use the page to make calls through the WebAPI.
On the page in your web browser, be sure to click the green plus sign next to the algorithm name to add that algorithm to the list to be used. Same if you manually add items or containers.
Then click Pack Em Up and it should give you results.
Does that help at all?

Dear David,
thanks for this exactly desription, i have forgotten to set the html as startup page, now it works!
But when i click "Pack Em Up", the Modal opens, but the Container View has no space ...
If i zoom in or out, the container is not longer here anymore.
Do you have a idea what happens here, or What I do wrong? (Browser Google Chrome, see Screenshot in the Attachment).
Greets from Austria.

Dear david,
its me again - sorry for the question before, to control the 3d container, i must hold the right click button. now everythink works. Thanks you for the help!
I hope i can write you by another questions 👍
Greets from austria.

Dear David,
can you please start the 3d visualization with the following values from the screenshot below?
in my case, the algorithmus calculates the values, but the 3d presentation is empty, wyh is that?

Info: i have copy my input into the attached text file.
3dView_Issue_Input.txt
Browser: google chrome
thanks you!
Greets from austria.

Dear David,
i think i have find the reason, this dimensions are so big, that the 3d view is only visible if you zoom longer in. But the correct presentation Still does not work, look at the scrrenshot:

Greets from austria.

Hmm, yeah it looks like the visualization is struggling with items and containers with such large dimensions. I will need to look through the JS to see what is going on. Thanks for the heads up! In the meantime, you could try scaling everything down by a factor of 100 or so. All the relative dimensions will be the same, so the packing result should be identical. At least you will be able to see things that way.

Dear David,
Yes, you must zoom in very long...
Thanks, i have already used this "trick".
Greets from austria.

Rather than sending requests via WebAPI in order to view the results. Do you have the codes so that it may be developed locally? Thank you

Hi, FoxBatTango. I went ahead and answered your question in the other issue you opened. Take a look over there and let me know if you have any other questions.

Thank you. I would like an example; I replied.
From: davidmchapman<EMAIL_ADDRESS>Sent: Monday, February 26, 2018 9:22 PM
To: davidmchapman/3DContainerPacking<EMAIL_ADDRESS>Cc: FoxBatTango<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [davidmchapman/3DContainerPacking] how to trigger the WebAPI (#1)
Hi, FoxBatTango. I went ahead and answered your question in the other issue you opened. Take a look over there and let me know if you have any other questions.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/davidmchapman/3DContainerPacking/issues/1#issuecomment-368723921 , or mute the thread https://github.com/notifications/unsubscribe-auth/AjINwP30RIptXszSVl4s7z1Pu7gVsFoDks5tY2azgaJpZM4N6AP7 .  https://github.com/notifications/beacon/AjINwBswEl0YH4DmbEa7174gS81CZu7Mks5tY2azgaJpZM4N6AP7.gif

Hi, I cannot locate the container-packing.html file in the repo/ demoapp project folder. Am I missing something ?


added constant file to optimized string values usage

@avinash201199 Updated code as per new code structure and resolved conflicts, Added constant file to optimize string values usage.Please let me know if anything needs to change,
Thanks!

please add the screenshot of the changes

please add the screenshot of the changes

@avinash201199 added working site with message validation


[Bug] Popup for update but none available (Linux)

Describe the Bug

When BBD initialized, a popup is shown hinting at an update being available. However, the download page links to the 0.3.1 release page which was made in October last year.
Since I run Arch Linux, I cannot use the installer and used to fetch the AUR package for this, which points to the stable16 branch. That still works fine.
However, installing either the 0.3.1 branch or current master with betterdiscordctl results in an error message on startup and then discord closing.
Screenshots

Discord Version

stable 0.0.9

This is due to the Linux installer still using the old injection method with stable16 I will push an update for the stable16 this time but no longer in the future. Please bug the maker of the installer to move to the injection BBD is using as of almost a year ago.

Thanks for looking into it. There are issues for this on betterdiscordctl's side, but they are very recent (https://github.com/bb010g/betterdiscordctl/issues/37, https://github.com/bb010g/betterdiscordctl/issues/38).
Will you push a new tag for the new stable version?

I talked to bb010g about a year ago about these changes that they never made.
I won't push a new tag no*--but you can use bdctl to update using the stable16 branch once more and the popup should go away now.
*The reason I'm not pushing a new tag is because most of the installation stuff happens by grabbing directly from the repo and the releases are for the installers. (Note: this was something that was set in place by Jiiks with the original BD which makes it very difficult to suddenly switch how everything works)

All right, makes sense. No reason to use the tags for fetching the latest version, then. Thanks again and hopefully the new method will be supported soon.

bb010g has been MIA for a while. Let me know how the new injection works and I'll try to implement it one of these days. I think you mentioned in the past but I can't recall the specifics.

@ObserverOfTime just put it over on bb010g/betterdiscordctl#37


Linked list

Features & Tasks:

[x] Node

[x] Create a Node class that has properties for the value stored in the Node, and a pointer to the next Node.

[x] Linked List

[x] Create a Linked List class
[x] Within your Linked List class, include a head property.
[x] Upon instantiation, an empty Linked List should be created.
[x] The class should contain the following methods

[x] insert
Arguments: value
Returns: nothing
[x] Adds a new node with that value to the head of the list with an O(1) Time performance.
[x] includes
Arguments: value
Returns: Boolean
[x] Indicates whether that value exists as a Node’s value somewhere within the list.
[x] to string
Arguments: none
Returns: a string representing all the values in the Linked List, formatted as:
"{ a } -> { b } -> { c } -> NULL"

[x] Any exceptions or errors that come from your code should be semantic, capture-able errors. For example, rather than a default error thrown by your language, your code should raise/throw a custom, semantic error that describes what went wrong in calling the methods you wrote for this lab.
[x] Be sure to follow your language/frameworks standard naming conventions


Mouse Motion Event

So I was just playing around with mouse events and noticed something weird.
If I run the window example the mouse motion event doesn't work.
However, if I add a second surface.commit() right after the first one (https://github.com/vberger/wayland-client-rs/blob/master/wayland-client/examples/simple_window.rs#L115) it does work.
I am currently running into this problem with my own program that I need to do the first commit twice for mouse events to work and I am not sure if this is a bug, since it's even present in the example.
I've tested this with Sway if that matters.

It might be a bug with sway, because for the me example works out of the box on weston.
I'll try it on sway too, to compare.

So I tested this with orbment and had the same issue. Apparently it's a problem with wlc?

As WLC is being deprecated, I'll close this.


Added accidents in city by year and severity query to API

@shaniwein  looks very good. Note the comment above, and also see the travis test that failed due to pylint  - make sure tests are passing


Fix FastFixtureTestCase to support testcases without fixtures.

Since Django 1.7, using the FastFixtureTestCase without specifying a list of fixtures to use raises this exception:
Traceback (most recent call last):
  File "nose/suite.py", line 209, in run
    self.setUp()
  File "nose/suite.py", line 292, in setUp
    self.setupContext(ancestor)
  File "nose/suite.py", line 315, in setupContext
    try_run(context, names)
  File "nose/util.py", line 471, in try_run
    return func()
  File "django_nose/testcases.py", line 49, in setUpClass
    cls._fixture_setup()
  File "django_nose/testcases.py", line 72, in _fixture_setup
    'database': db})
TypeError: call_command() argument after * must be a sequence, not NoneType

triggered here:
call_command('loaddata', *cls.fixtures, **{'verbosity': 0,
                                           'commit': False,
                                           'database': db})

Since Django 1.7 sets fixtures to None by default, django-nose should support this possibilty too. :-)

I also think the way forward is to re-sync w/ the current Django code.  I'd like tests for FastFixtureTestCase (issue #220) before I start merging fixes.


ENH: Added Common test driver, including selxStringConverterTest

https://app.codacy.com/app/FBerendsen/SuperElastix/pullRequest?prid=3291030 still suggests:

Function parameter 'str' should be passed by const reference.

That's easily fixed, I'll amend + force-push

git "force-push" is supposed to be risky, but at least I did do --force-with-lease: https://developer.atlassian.com/blog/2015/04/force-with-lease/

THe pull request has passed sucessfully on 4 out of 5 compilers at https://travis-ci.org/SuperElastix/SuperElastix/builds/508456079 Only "gcc Xcode: xcode8.3 C++" failed:

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

I think I'll just merge anyway!


Deserializing multi search response using RequestResponseSerializer throws an exception

NEST/Elasticsearch.Net version: 7.17.5
Elasticsearch version: 8.4.3
.NET runtime version: .NET 7
Operating system version: Linux
Description of the problem including expected versus actual behavior:
When trying to deserialize a low-level multi search response of type StringResponse using the RequestResponseSerializer, an error is thrown: Constructor on type 'Nest.MultiSearchResponseFormatter' not found..
Steps to reproduce:

Build a multisearch body as documented in Elasticsearch docs.
Using the low-level client, send a request to the multisearch API:

var response = await _client.LowLevel.MultiSearchAsync<StringResponse>(indexName, PostData.String(body));

Given the response is succesful, try to deserialize the response body as MultisearchResponse using the RequestResponseSerializer:

var serializedResponse = _client.RequestResponseSerializer.Deserialize<MultiSearchResponse>(new MemoryStream(System.Text.Encoding.UTF8.GetBytes(response.Body)));

An exception of type MissingMethodException will be thrown.

Expected behavior
The response should be serialized without any issues.

@gilmatok We'll need to investigate this scenario. Is there any reason you're not using the NEST client directly rather than low-level and then deserializing, which essentially repeats the work the high-level client does? Have you tested with the high-level client to see if you get the same outcome?

@stevejgordon Thanks for the follow-up. The high-level client works fine when deserializing Multisearch responses. The reason I'm using the low-level is client is because I'm using Elastiknn plugin which requires me to build the request body manually.


Add --aging-period to experiment with expanding deviation in stale ratings

New option --aging-period (a duration measured in days) makes ratings age continuously.  --aging-period=0 (the default) turns it off.
E.g., setting --aging-period=7 mimicks a window of one week, where the deviation for each rating is expanded according to the (fractional) number of weeks that have passed since the rating was established.
Currently only implemented in analysis/analyze_glicko2_one_game_at_a_time.py. I'll also add it to the ratings-grid script when I get that going again...
One thought: with this in place, I wonder if "step 6" should be skipped. I.e.,
    # step 6
    phi_star = sqrt(player.phi ** 2 + new_volatility ** 2)

is both (a) evaluated fully on every game and (b) continuously, averaging once per period (via --aging-period). Maybe the every-game evaluation should be skipped (and phi_star = player.phi instead inside glicko2_update). I'll experiment with that tomorrow/this week.

One thought: with this in place, I wonder if "step 6" should be skipped. I.e.,

In some sense I think it's basically intended, there's this from the glicko2.pdf:
Note that if a player does not compete during the rating period, then only Step 6 applies

However the context is a little since he's talking about updating everyone's deviation right before a tournament, everyone plays a bunch of games, and then updating it again after all those games, where as we're doing it every game, so maybe it's different.
If we do find that doubling up all the time like this is a problem, I would think that we couldn't skip step 6 though, as that's a function of the outcome of the game. If anything, my gut would point in the direction of doing something in the deviation expansion code or making the periods passed a parameter of the glicko2_update function itself so we account for it there, but both of those would take a bit of pondering.
Alternatively, and perhaps this would be a better experience anyways, we could only run the expansion code if more than X time has passed, then we side step the whole question of how to correctly handle the constant application of that expansion code.


set -o nounset causes unbound variable warnings

With source ~/.hishtory/config.sh in .bashrc.
$> set -u
-bash: HISHTORY_AT_PROMPT: unbound variable
-bash: HISHTORY_FIRST_PROMPT: unbound variable

Reason:
https://github.com/ddworken/hishtory/blob/cc123854a02374a7e4ee7fc87a974b19566fa142/client/lib/config.sh#L10
Solutions: https://stackoverflow.com/questions/7832080/test-if-a-variable-is-set-in-bash-when-using-set-o-nounset

This should be fixed! If you run hishtory update you'll get the latest version with the fix. If you're still experiencing this issue (or run into anything else!) please reopen this so I can take another look.

Seems to be working; thanks for the quick fix @ddworken!


Updated appsync apikey expires to be an int: Fixes #1039

Fixes issue https://github.com/cloudtools/troposphere/issues/1039
The expires prop on the appsync ApiKey class needs to be an int otherwise the decimal place causes cloudformation to throw a Can not deserialize value of type java.lang.Long from String error.

Thanks!


Encountering ParseError

Hi there, I am encountering a parse error with the following files attached.
parse-errors.zip
It looks like it is on the $HANDSEED header.

Thanks for filing the issue!  There is whitespace around the $HANDSEED value that I didn't properly account for.  I've fixed that in commit 00a62846e4d42f854e125964b807b3aaf81fec7e.

I appreciate the speedy response!


community: Improve amadeus tool and doc

Description:
This pull request addresses two key improvements to the langchain repository:
Fix for Crash in Flight Search Interface:
Previously, the code would crash when encountering a failure scenario in the flight ticket search interface. This PR resolves this issue by implementing a fix to handle such scenarios gracefully. Now, the code handles failures in the flight search interface without crashing, ensuring smoother operation.
Documentation Update for Amadeus Toolkit:
Prior to this update, examples provided in the documentation for the Amadeus Toolkit were unable to run correctly due to outdated information. This PR includes an update to the documentation, ensuring that all examples can now be executed successfully. With this update, users can effectively utilize the Amadeus Toolkit with accurate and functioning examples.
These changes aim to enhance the reliability and usability of the langchain repository by addressing issues related to error handling and ensuring that documentation remains up-to-date and actionable.

While learning to use the Amadeus toolkit with examples, I encountered two major issues:

The example provided in the documentation, as mentioned in issue #17375 regarding Amadeus flight search not working, utilizes the ReActSingleInputOutputParser, which fails to parse the request of tools.

The examples in the documentation relied on the tool description, which inconsistently produced outputs that matched the required parameter names for the API.

To address these issues, enhancements have been made to the relevant codebase. Specifically, during the execution of all examples, it was noted that the toolkit did not effectively handle scenarios where API calls failed. Consequently, improvements have been implemented to handle API call failures more effectively.


Configuring Application Insights with user secrets

Added information on sourcing the Application Insights instrumentation key from user secrets (a common scenario) or other configuration providers.

@MartinZikmund : Thanks for your contribution! The author(s) have been notified to review your proposed change.

(Note: If the update is approved, correct the typo in "instrumentation" before merging.)

@mrbullwinkle
Could you review the proposed changes to your article?
Thanks!

@ktoliver, this one can merge if you are able to correct the small spelling error, thank you!

@mrbullwinkle
I propose these edits per Microsoft style:
If you want to store the instrumentation key in ASP.NET Core User Secrets or retrieve it from another configuration provider, you can use the overload with a Microsoft.Extensions.Configuration.IConfiguration parameter:
services.AddApplicationInsightsTelemetry(Configuration);
Or I can fix the spelling error only.

@ktoliver, fixes in addition to the spelling fix are appreciated, thank you!

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks, all

@ktoliver, as always thank you so much!


Send User ID to google tag manager (to use in analytics)

To enrich Google Analytics we'd like to send user IDs for logged in users, via google tag manager.
We're currently using this module for GTM: https://www.npmjs.com/package/react-gtm-module
Please add the userId (when available) to the data layer for GTM to ingest as described in the module example.
The end result is that we can retrieve the User ID from the data layer in GTM and send it to GA (or other scripts)

Thanks for logging this issue @lelecanfora, but please add info and the right labels to it -> this might help -> https://github.com/FightPandemics/FightPandemics/wiki/How-to-"Properly"-create-a-ticket-|-new-issue-on-the-FightPandemics-repo

Thanks for logging this issue @lelecanfora, but please add info and the right labels to it -> this might help -> https://github.com/FightPandemics/FightPandemics/wiki/How-to-"Properly"-create-a-ticket-|-new-issue-on-the-FightPandemics-repo

Done, thanks

Hey @indirabhowmick, can we get this implemented for the soft launch?

Hey @lelecanfora this has been completed. Please verify on staging! We set the userId on the dataLayer as follows.


List and checkbox don't blend with CursorLine

Beautiful plugin, congrats!
I was exploring and I noticed the following when using set cursorline (maybe missing a hl_mode = "combine"?):

It also happens with table borders, but that actually looks kind of nice because it looks like it's highlighting each cell.

This will be fixed in the next update.


Bundle Gradle Tooling API into library

Desired Behavior
Since the Gradle Tooling API is only available from Gradle's own separate repository instead of Maven Central, we should just bundle their classes into this JAR.
Context
We don't want consumers to need to add that repository.

Fixed by cb672a190f4cbccdc1282e1ef948053d56744097


EndNet does not actually have an implementation

The readme mentions that code is available for the EndNet technique, but the actual "code" that the authors made available is not an actual implementation, it merely loads pre-calculated results and plots them.
This is misleading and a waste of time for anyone actually searching for available implementations of spectral unmixing techniques.
I'd suggest that the link be removed, or at the very least renamed from "code" to "results".

Thank you for your feedback. To address your concerns, we have updated the link for EndNet in the README from "Code" to "Result".
This should clarify that the provided resources are for loading and plotting pre-calculated results rather than a full implementation of the technique.
We appreciate your contributions to this project and hope this adjustment will better meet the needs of those searching for spectral unmixing techniques.

Now that I'm reading my post again, I realize that I came across as quite harsh. I apologize and thank you for the update. :)


Add additional pre-commit hooks

🚀 Pull Request
Description
This PR adds Python AST checking, and filename conflict checking for case-insensitive filesystems to the pre-commit hooks.

Consult Iris pull request check list

@trexfeathers Yeah, you really could go to town... but I go back to sitting on my hands again :wink:


Not able to create instance for the store

I am not able to add the instance name for the store for react native. It throws an error. Error: No available storage method found. I am trying to keep separate instance so we do not clear the entire instance the entire for that instance db.
Versions below
{
"expo": "~48.0.5",
"react-native": "0.71.3",
"@aveq-research/localforage-asyncstorage-driver": "^3.0.1",
@react-native-async-storage/async-storage": "^1.17.11",
"localforage": "^1.10.0"
}

Documentation is insufficient and fails to mention this critical step. #7


Remove sass-rails coffee-rails and jquery-rails gem dependencies.

Remove sass-rails, coffee-rails, jquery-rails gem dependencies.
Convert scss to css to get rid of sass-rails.
Upgrade Rails from 6.1 to 6.1.3.

Closing this for #68


TileLink Mask

64: 000 -> 1111_1111
32: 000 -> 0000_1111
32: 100 -> 1111_0000
16: 000 -> 0000_0011
16: 010 -> 0000_1100
16: 100 -> 0011_0000
16: 110 -> 1100_0000
 8: 000 -> 0000_0001
 8: 001 -> 0000_0010
 8: 010 -> 0000_0100
 8: 011 -> 0000_1000
 8: 100 -> 0001_0000
 8: 101 -> 0010_0000
 8: 110 -> 0100_0000
 8: 111 -> 1000_0000

[ where | size ]

[ ] Compute the mask function on the examples given above.

https://github.com/sifive/ProcKami/pull/300

[ ] Move mask and shift functions to MemOpsFuncs.v
[ ] Merge tests and refactored mask and shift functions with current master.

[ ] revise shiftAmt in TlUH.v so that it accepts a PAddr rather than a nat.
[ ] revise the tests for shiftAmt so that the upper bits of the PAddr are random while the lower 3 bits cover all of the cases illustrated in the table above.
[ ] convert the compute statements in the tests for shiftAmt and mask to Goal statements.
[ ] add hex equivalents to the table given above in this Issue.

https://github.com/sifive/ProcKami/pull/301

https://github.com/sifive/ProcKami/pull/301


build: build base docker image

I'm going to have to do something to get these tests to work in CI.

(but for now I'm just not going to run them)


Allow group members to access sockdir

We use this to allow apache cgi scripts to communicate with aqd via the knc socket.
Change-Id: I065cd3b4b766f7e1d726e0e2adcfcf3e4ac0c03e
There is an argument for making this configurable.

Cherry picked from #37.

@jrha we will just remove the line that always changing the dir permissions: os.chmod(sockdir, 0o700)  whatsoever. the if you recreate a dir with any custom permission then it will not overwritten. Let's close this when we merge the fix for this.

Ok, I'll update the PR!

PR updated.

will be merged in MS and synced to the upstream.

https://github.com/quattor/aquilon/commit/c51001cc2a51dc00902118039e2fdb725d3b6328


Pool Support

Thank you very muvh for sharing  this great software
i have several questions
if i want to make a front-end for this pool
can you please point me to the right direction ?
and how can we implement different payout methods like PPS, PPLNS for our workers
Is it possible to add DGB-SHA to suppoerted coins ?

BTCPool's slparser is a PPS based reward statistics program. bpool_local_stats_db.stats_users_day have users' daily PPS rewards, you can multiply them by a ratio (such as 96%, keep 4% as your profit) and then send to your users.
Note that when slparser calculates PPS rewards, it does not consider the transaction fee income of the block, which assumes that the block only receives the basic mining reward. You can then share the fee income with your users by changing the distribution ratio.
Currently slparser does not support PPLNS. You will need to write a new reward analyzer to do similar work, and your program may read bpool_local_db.found_blocks to find the time point of your blocks.

And we have no plan to add DGB-SHA as a suppoerted coin. But because DigiCoin's code is forked from DASH, maybe you can refer to another user's DASH implementation:
https://github.com/btccom/btcpool/files/3077084/btc-pool-dash-service.tar.gz

I made a mistake and it seems that the Digibyte is forked from Bitcoin instead of Dash. This way you can try to link it directly to BTCPool (assuming it is BTC) and then fix the error.

sorry for this noob question:
for DGB-SHA (it's sha256d ) i used :
cmake -DJOBS=4 -DCHAIN_TYPE=BTC -DCHAIN_SRC_ROOT=/work/bitcoin-0.16.0 ..
everything is working except:
JobMakerBitcoin.cc:77] invalid pool payout address
should i edit JobMakerBitcoin.cc to fix this problem ? (and i don't know how to fix it)
or should i use digibyte source for cmake  ? like this:
cmake -DJOBS=4 -DCHAIN_TYPE=BTC -DCHAIN_SRC_ROOT=/work/digibyte-7.17.2 ..
(i tried this also and it gives error during compile)
also the pool found 2 blocks but the reward did not come into my address :-(

i removed the code to check the validity of the payout address :))
cmake with digibyte source has lots of errors, (the errors are related to file name , because for example instead of libbitcoin_common.a the file names are like libdigibyte_common.a .... )
i changed them in CMakeLists.txt , but then more errors related to  IsValidDestinationString ...
so i used cmake with bitcoin source:
cmake -DJOBS=4 -DCHAIN_TYPE=BTC -DCHAIN_SRC_ROOT=/work/bitcoin-0.16.0 ..
and compiled again , and the jobmaker did not show any error anymore
everything is working , the block found successfully , but the reward did not come into my wallet address (still the same problem)
here is the block explorer result of the block my pool found:
https://explorer-1.us.digibyteservers.io/block/0000000000000003eb1cd29f12bb7b7412667f45970cf30adfc00fee42416c10
the receiver address is empty
Unparsed address [1] : 0 DGB

DGB is using a different p2pkh address scheme and you have to use DGB source code. We are going to look into this and try to add some minimal DGB support changes.


feat: allow sending arbitrary signals to running service

Fixes #7391
This started as just adding a new signals enum, and allowing it to be attached to stop. However, I realized - not every signal should necessarily cause a process to terminate, so we should also probably have a way of signalling a process, without waiting for that to happen. So I also added a new signal method.
Bit of a jumble of commits, see each one for more details.

WDYT @jedevc, does this feel like patch change or do we want to push it into v0.12.0 ?