Seed Settings

Hi, how do you take the values of seed during training and seed+seed_offset during testing? When I take seed=[0,1,2,3,4], seed_offset=100, the result I get is very different from the result presented in the paper.
Thanks!

I have the same issue with reproducting results.

Hi, really sorry for the late reply
We apologize for the bug in our code from the previous version. We have updated the code.
Note that there is some mismatch between our provided code and the code that we used to produce the paper results. The copyright of our original code belongs to the previous company I worked for. Unfortunately, we didn't have permission to open-source it and the current github version is our re-implementation. Although we try to reimplement it as similar to the original version as possible, there is still one dataset, "hopper-medium-v2" datasest, where our re-implementation version achieves a 78.6 score, not able to reproduce 98.2 in the white paper. Other scores almost match the results in our paper (may have some minor difference due to random seeds)
Note that some configurations and experimental details are different from the original code (e.g., the value of $\alpha$, learning rate, and so on). We have updated our paper in the arxiv to reflect that and the mismatch results in the hopper-medium-v2 dataset (lower it from 98.2 to 78.6). If you want to compare POR with your work, you can refer to this paper version.
Let me know if you have any further questions!


Dependency Loop Detected

Trying to install DSInternals on Powershell 7 on Linux / Ubuntu is getting a dependency loop error
PS /home/ubuntu> Install-Package dsinternals.common
Install-Package: Dependency loop detected for package 'DSInternals.Common'.

Hi @mubix , unfortunately, DSInternals is not supported on Linux at all, as it heavily utilizes Win32 API for almost everything it does (database layer, replication and LSA over RPC, hashing algorithms).


domvm 2.x-dev (part 3)

...continuation of #101
The view layer is now in final form.
Hopefully this discussion will be the last needed prior to v2.0.0 release and focused mostly on ironing out the router.
Some internal refinements have been postponed till after 2.0.0:

Exposed ref cleanup from ancestors when unmounting subviews. The fact that they can also be namespaced makes this somewhat tricky and possibly expensive.

Features that will be considered for exploration post-2.0.0:

fragment views
dom recycling of unmounted views
internally using addEventListener instead of on* props. This would allow things like ontransitionend which cannot be bound using props. Tracking a registry of attached events and patching them in all forms (including delegation and parameterization) makes this tricky/possibly expensive.

Tracking a registry of attached events and patching them in all forms....makes this tricky/possibly expensive.
Indeed, but if you have plans to suports that class of events, you can start by switching the strategy with those events which cannot be bound using props, seudo below:

var trackEvents = { ontransitionend: 0, onxevt: 0}
if (trackEvents.hasOwnProperty(evtName)) {
    detachEvt(elem, evtName)
    attachEvt(elem, evtName, callback)
} else {
    elem[evtName] = callback
}

@yosbelms when I tried to do this initially during the v2 rewrite, it complicated and slowed down the diffing/patching code significantly to the point that it was not worth it so I rolled back the impl.
If you want to take a stab at it, feel free, but all the following forms must be properly supported with minimal or no perf loss and must ensure that there is no detach/rewrap/re-attach on every redraw:
{ontransitionend: fn}
{ontransitionend: [fn, arg1, arg2]}
{ontransitionend: {".moo": fn}}
{ontransitionend: {".moo": [fn, arg1, arg2]}}

The relevant code is in [1] https://github.com/leeoniya/domvm/blob/2.x-dev/src/view/patchEvent.js

@leeoniya
Check this[1] out, see how Maquette solves whith the rule number 1 some things related to event patching, that simple.
However, I think management of that special events can be made in app-space by using lifecycle hooks.

http://maquettejs.org/docs/rules.html

domvm cannot adhere to rule #1 because it supports ad-hoc definitions of parameters and delegated selectors which must be wrapped before being attached. Currently they are shallow-diffed to avoid the rewrap/re-attach on each redraw call. if the supported signatures were only the first variation (basic) then addEventListener support would not be an issue in the first place.

docs?

should be a fairly straightforward port of 1.x docs with some tweaks: hyperscriptify, views can return {render:..., hooks:..., etc...}, some vnode props have turned into lazy getter functions. surprisingly little has changed in the public API besides the hyperscript transition.
are you interested in helping port them or just wondering for own use? if it's the latter, you can look at the ported demos for the time being. if it's the former, we can discuss more of the details.

defineFragment looks like it may land with 2.0, see https://github.com/leeoniya/domvm/issues/106

After a bit of discussion [1] and a bit of benching [2], i think it's worthwhile to change domvm's null-removal behavior to instead be false-removal. This means that any undefined or null which appear in templates will be now get stringified to text nodes, while false will be spliced out. This is an inversion from the current behavior and BC with respect to 1.x but easy to port and ultimately for the best.
There's no measurable effect on perf.
[1] https://github.com/lhorie/mithril.js/issues/1014#issuecomment-264217972
[2] https://github.com/leeoniya/domvm/tree/preProc-false-null

My (cautious) comment, assuming I am understanding the proposal correctly, is that it seems weird and counter-intuitive.
Omitting the node for null/undefined is to me very intuitive -- this thing is "nothing". Using false do the same thing is not -- this thing is "false"?
That said, Barney did make good points, that seem to be valid given the right context.
But my brain is futzed by two 11 hour days working on a problem for a customer that (a) ended up being their gateway acting up, and (b) literally disappeared as I was writing up the final report for senior management. I think I'd need to see a simple example of what would change.

i'm somewhat on the fence about this.
false does have a lot going for it

more elegant template construction (as shown in the linked thread)
false is more likely to be an explicit indication of absence than an accidental omission or an uninitialized variable (as would happen for undefined and null)
stringifying null and undefined may help to visually spot obvious errors rather than having them be silently swallowed and squashed by the lib.

The latter 2 points relate to debugging and helping avoid carelessness, but the first point really interests me the most. The best solution may be a hybrid. By default, squash all three: false, null, undefined. This keeps pretty intuitive behavior and has the benefit of working with the inline conditional node declarations. If the user prefers, a dev option can be exposed that will stringify null and undefined to help catch errors during dev. This flag can be added later, if the need arises.

By "more elegant template construction" I assume mean a template like:
  (
  vnode.state.writePermissions
  && (
       vnode.state.editing
     ? m(EditButton)
     : [
         m(CancelButton),
         m(SaveButton)
       ]
     )
  || null
  )

(BTW: I think the original was actually broken; I've tried to correct it, and the fact it was not obviously wrong actually lends support to the idea that it's not good).
But, frankly, and only IMHO, that's an awful template and not at all clear what it's going to do without puzzling over it. That grok-effort will reduce with familiarity, but it seems so much like a side effect of Javascript's condition evaluation that it's borderline detestable to me.
Why is it any more desirable than:
vnode.state.writePermissions
?   vnode.state.editing
    ?   m(EditButton)
    :   [ m(CancelButton), m(SaveButton) ]
:   null

which at least has the benefit of not being obtuse and not exploiting a by-chance-not-design feature of the language.

With that said, I do think that arguments from the accidental null/undefined being silently discarded carry much more weight that abominations like the cited template.

Finally, I suppose this might have some merit:
vnode.state.writePermissions
&&  vnode.state.editing
    ?   m(EditButton)
    :   [ m(CancelButton), m(SaveButton) ]

Really finally, I tend to agree with Isaiah Medows and Barney Carrol that falsey nodes are expensive and I think they'll lead to unexpected corner cases. Perhaps strictly false will work, but then things like the above will do weird things if the (e.g.) writePermissions was say, 1/0 instead of true/false.
In the end, I think null/undefined is more logical, more efficient and has less wrinkles.
All just my 50c worth.

By "more elegant template construction" I assume mean a template like:

without the trailing || null

which at least has the benefit of not being obtuse and not exploiting a by-chance-not-design feature of the language.

i could be wrong but i think the following return semantics are not specifically javascriptisms:
(false && "moo") === false
(true  && "moo") === "moo"

Really finally, I tend to agree with Isaiah Medows and Barney Carrol that falsey nodes are expensive and I think they'll lead to unexpected corner cases.

I don't intend to squash all falsey values, only 3: null, undefined, false. The test for them is *not expensive. I have profiled it there's no overhead for adding a strict false test.
val == null || val === false

Hey @leeoniya I've found this Virtual DOM benchmarks, I've made one for domvm https://yosbelms.github.io/domvm_uibench/ If you want to include in the repo I can make a PR.
src: https://github.com/yosbelms/domvm_uibench

@yosbelms : Interesting. Play around a bit. Compared Inferno and DOMVM and was pleased to see the DOMVM consistently comes out ahead when running the two side by side.

@yosbelms thanks for this!
a couple notes from quick look (will look more carefully soon):
other implementations barf out a json report at the end, this one doesnt.
function TableCell (text) {
	return el('td', {class: 'TableCell', onclick: onClick.bind(text)}, [tx(text)])
}

calling .bind creates a new re-scoped function every time, so it will get re-bound every time. this could be avoided by using onclick: [onClick, text], where text becomes the first argument to onclick. try this out and see if it's any faster. this is assuming delegation cannot be used, otherwise it would be cheaper to just store the text as _data: text and access it from the delegated handler.
td.TableCell will get cached, so might be faster.
el('table', {class: 'Table'}, [el('tbody', {}, children)]);

maybe table.Table, also avoid the extra {} allocation. el(tag, children) signature works fine.
if there are no lifecycle hooks to fire, the domvm.FAST_REMOVE optimization could be appropriate. as well ass domvm.FIXED_BODY for elements whose children do not need reordering/removal/insertion.

@lawrence-dol
Inferno is faster than domvm, if it comes out slower in any benches it's either an old version or a shitty bench implemenation.

@leeoniya Inferno benchmarks are all tricky. A real life big production app shuoldn't be developed using must of the hat-tricks Inferno comes with. For the sake of mantainability. Here is where domvm beats Inferno. It is very fast, event without hacks.
PD: I'll try your tips later, wait for feedback.

I've added a {_list: true} optimization that enables a binary search for vtree reconciliation. It expects all children to be homogeneous and uniquely keyed. Enabling it yields pretty significant improvements in huge keyed lists. For instance, it brings the replace-all-rows metric [1] from ~260ms down to ~190ms (on my machine) and the delta is higher when replacing 10k rows by 1k new rows.
[1] https://github.com/krausest/js-framework-benchmark/pull/95

enables a binary search

Does that not also imply that the keys are ordered in a known (presumably ascending) sequence?

they're sorted internally to a detached list prior to each reconciliation, otherwise it'd be pretty useless to the user :)

Due to the rather limited resources of a sole developer, I've decided to split the router into its own repo and let it live or die there without dragging domvm down with it. It has some neat features, but it lacks some things that I consider important. Mainly, it...

has no tests whatsoever
has no isomorphic routing (while domvm's view supports it)
has no ability to add/remove routes on demand, or mount sub-routers
has a rather odd initialization API that was mimicked after domvm v1, but feels wrong with v2.

I have wanted to version it separately from the rest of the repo for a long time anyhow, since it'd fully decoupled.
https://github.com/baseprime/grapnel seems like a decent small isomorphic replacement, but I'm not sure how complete its server-side powers are relative to more mature and server-centric options like Express.
I gave some consideration to forking grapnel and adding some domvm router features to it, such as href generation and parameterized route creation, optional replaceState/hash rather than pushState. I don't know how good an idea this is. One thing I know for sure is that adding isomorphism to domvm's router is not a trivial feature, and having a view layer that supports SSR and a router that doesnt is too weird to live in the same repo.
This will let me focus on getting domvm v2 finished.

well, after trying some of the routers (to port the ThreaditJS demo) i got sad again.
and so I put some effort into untangling the internal mess that was the router, which allowed a lot of api simplification [1] and the implementation of a unified prefix: in place of useHist & root.
hash routing seems to work as intended, still need to test other ported/rewritten stuff.
var r = createRouter({
  prefix: "#/",
  routes: [
    {
      name: "root",
      path: "",
      onenter: function() {
        console.log("root");
      }
    },
    {
      name: "moo",
      path: "moo/:id/:date",
      onenter: function() {
        console.log("moo");
      }
    }
  ]
});

r.boot(true);

//  console.log(r.href('moo', {id: 123, date: "2016-12-19"}, {foo: "a", bar: 456}));

[1] https://github.com/leeoniya/domvm-router/blob/master/src/router.js#L316

I wondered if you might, in the end.
If you are interested at some point, I grabbed and enhanced/rewrote the V1 fetch module as part of preparing for V2. I found it was better than the other micro-libraries I tried, and smaller, and has a small, intuitive API surface.

@leeoniya

I put some effort into untangling the internal mess that was the router

Thank you so much. Your router was the first one I found that didn't confuse me - it does exactly what's needed and nothing more.
For my server-side-rendering use case, the router works fine, even though you say there's no support for isomorphism. :) Okay, the interface on the server is a little odd (passing location.href through a global variable), but since your code is so clear I could give fixing that up a try. At least so far, this minor oddness is bothering me far less than trying to deal with what people out there seem to think routers should be doing.
@lawrence-dol
Please do share if you can. That sounds interesting.
I tried to use fetch but didn't quite manage to be happy doing so. Ultimately, what drove me away was the lack of timeouts (https://github.com/whatwg/fetch/issues/20) and the need to use polyfills. Though, I think that could be worked around. If ServiceWorkers come to all relevant browsers, and if they really won't support XMLHttpRequest, I think the case for using fetch is getting much stronger than it is today.

@grumpi
i'm afraid the isomorphic story is a bit worse in the refactor, since i changed the route parser to rely on a helper <a> element that's created using the DOM. this makes things a bit less fragile than implementing/maintaining own regex parsing.
a uniform router API would need to add some indirection for accessing route params so that the handler signatures can be the same eg. (req, resp). in this case resp is kinda useless on the client, though we can encode side-effects into some returned object from the handler. req.headers, req.cookies, req.files/formdata, req.body would be absent on the client. again, there's just a huge surface area of differences that would need to be abstracted and make client-side behavior clunkier for the sake of a deceivingly uniform skin-deep syntax. i personally don't see much value in doing this.
i'm fairly certain about not wanting to make domvm's router isomorphic. in my opinion, SPAs should work plenty fast without needing SSR to begin with, and they don't need search engine indexing because their use case is functionality (lots of dynamic things that are user-specific and pre-session-unique), not indexable content. Mostly static sites should use traditional html with a sprinkling of jquery/bala [1]/vanilla js.
@lawrence-dol definitely share it, though i gotta say i'm quite happy with xr.
[1] https://github.com/finom/bala

i changed the route parser to rely on a helper  element that's created using the DOM. this makes things a bit less fragile than implementing/maintaining own regex parsing.

Interesting. Yes, I can see how that saves some space and should be more maintainable. I do like that, even though it means that route-parsing on the server needs to be done in a different way.

i'm afraid the isomorphic story is a bit worse in the refactor

I don't think so, actually. Admittedly, it will be necessary to override parseUrl to get it to run at all, but the ability to expose a clean interface with locFromUrl removes the need to place the url into a global variable in nodejs. :) I see this as a step in the right direction. Overriding parseUrl on the server is no trouble at all if one wants to do server-side-rendering - while everyone benefits from the reduction in complexity on the client-side.

a uniform router API would need to add some indirection for accessing route params so that the handler signatures can be the same eg. (req, resp). in this case resp is kinda useless on the client, though we can encode side-effects into some returned object from the handler. req.headers, req.cookies, req.files/formdata, req.body would be absent on the client.

Ah, I suppose where we're on different pages is in the expectation that an isomorphic domvm-router should deal with resp. I explicitly don't think it should do that. All it should do is parse routes of the SPA in the same manner as on the client so that fallback/preview-while-app-is-loading HTML can be delivered without thinking. :) It is a SPA-router, after all.
On the server, there should be something else that deals with the response-related stuff. I don't see value in a uniform interface for the routes of the SSR-related part of a server and the part that communicates with the SPA (if those two parts are even running on the same server(!)). Having a clear break between those two parts seems quite natural to me.

SPAs should work plenty fast without needing SSR to begin with, and they don't need search engine indexing because their use case is almost 100% functionality (lots of dynamic things that are user-specific and per-session unique), not indexable content - to make a blog using domvm rather than Express & backend html rendering would be foolish

I agree with you that the majority of apps won't benefit enough from server-side-rendering to incur the cost of maintaining it. It would be foolish to make a router that, by default, assumes that server-side-rendering is what should be done. I think the way you're going with the router is fine - keep it lean for client-side use.

so here's how the current router API works:
https://github.com/leeoniya/domvm/blob/2.x-dev/demos/ThreaditJS/app.js#L250-L291

alright gentlemen...
the v2 api is stable. all existing tests/demos/benches are passing.
more tests will be needed, and docs!
but enough with polishing this turd. v2.0.0 is tagged and bagged 🎉

v2.0.0 is tagged and bagged

Ummm... that's what one does to a corpse in the morgue.

lol yeah, but it's more a combo of tagged & [1] in this case :D
[1] http://idioms.thefreedictionary.com/in+the+bag


Modifying TimeoutTest to check for connection usability.

Codecov Report

Merging #278 into dev will increase coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #278      +/-   ##
============================================
+ Coverage     34.53%   34.61%   +0.07%
- Complexity     1561     1566       +5
============================================
  Files           102      102
  Lines         23660    23660
  Branches       3873     3873
============================================
+ Hits           8172     8189      +17
+ Misses        13874    13854      -20
- Partials       1614     1617       +3

Flag
Coverage Δ
Complexity Δ

#JDBC41
34.52% <ø> (+0.12%)
1561 <ø> (+4)
:arrow_up:

#JDBC42
34.48% <ø> (+0.04%)
1559 <ø> (+4)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/microsoft/sql/DateTimeOffset.java
37.14% <0%> (-2.86%)
8% <0%> (-2%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
46.24% <0%> (-0.08%)
194% <0%> (ø)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
39.01% <0%> (+0.13%)
0% <0%> (ø)
:arrow_down:

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
30.13% <0%> (+0.44%)
56% <0%> (+1%)
:arrow_up:

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
26.33% <0%> (+0.45%)
185% <0%> (+4%)
:arrow_up:

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
44.94% <0%> (+3.37%)
16% <0%> (+2%)
:arrow_up:

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 b26d102...abd1bf1. Read the comment docs.


Doesn't see a playbook when running with docker. With podman works well.

ISSUE TYPE

Bug Report

SUMMARY
Doesn't see a playbook when running with docker. With podman works well.
It throws the error ERROR! the playbook: /root/test_localhost.yml could not be found
ANSIBLE-NAVIGATOR VERSION

3.3.1

CONFIGURATION
Fresh installation, no custom settings
LOG FILE
The only entry is:
2023-05-09T12:19:58.027513+00:00 CRITICAL 'ansible_navigator.actions.run_playbook._handle_message' Unhandled message from runner queue, discarded: {'event': 'verbose', 'uuid': '99d812d0-3517-4c66-a0d4-060d5e6d2bcd', 'counter': 1, 'stdout': '\x1b[0;31mERROR! the playbook: /root/test_localhost.yml could not be found\x1b[0m', 'start_line': 0, 'end_line': 1, 'runner_ident': '0ddf5377-903e-4681-9c1b-c70a8d060547', 'created': '2023-05-09T12:19:58.026932'}

STEPS TO REPRODUCE

Install docker
build the image with ansible-builder
create a playbook in your home dir
run the following from your home dir

[root@fedora03 ~]# ansible-navigator run /root/test_localhost.yml --execution-environment-image postgresql_ee --mode stdout --pull-policy missing
ERROR! the playbook: /root/test_localhost.yml could not be found
Please review the log for errors.
[root@fedora03 ~]#
[root@fedora03 ~]#
[root@fedora03 ~]# cat test_localhost.yml
---
- hosts: localhost
  become: yes
  gather_facts: yes
  tasks:
  - name: Print facts
    ansible.builtin.debug:
      msg: '{{ ansible_facts }}'

If I uninstall docker, install podman, rebuild the image with the same build command and definition, and run the same command, everything works as expected
[root@fedora03 ~]# ansible-navigator run /root/test_localhost.yml --execution-environment-image postgresql_ee --mode stdout --pull-policy missing
[WARNING]: No inventory was parsed, only implicit localhost is available
[WARNING]: provided hosts list is empty, only localhost is available. Note that
the implicit localhost does not match 'all'

PLAY [localhost] ***************************************************************

TASK [Gathering Facts] *********************************************************
ok: [localhost]

TASK [Print facts] *************************************************************
ok: [localhost] => {
    "msg": {
        "ansible_local": {},
        "apparmor": {
            "status": "disabled"
        },
        "architecture": "x86_64",
...

EXPECTED RESULTS
No errors. Works well as with podman.
ACTUAL RESULTS
ERROR! the playbook: /root/test_localhost.yml could not be found

I suspect this is beacuse the user in the container when using docker is 1000, and does not have access to the /root directory where the playbook is being mounted.
We can check this using the exec command:
(venv) m910x ➜  ansible-navigator git:(main) ✗ ansible-navigator exec --ce docker --eei test-ee --pp never

dr-xr-x---   1 root root   8 Jun  8 13:30 root

Does this work in any directory other than /root ?
If you have to mount the playbook into /root, you might have to be the root user in the container to access it:
(venv) m910x ➜  ansible-navigator git:(main) ✗ ansible-navigator exec --ce docker --eei test-ee --pp never --co="--user=root"
[root@8226ac2388a9 ansible-navigator]# touch /root/foo.txt
[root@8226ac2388a9 ansible-navigator]#

Keep us posted on you findings!

@cidrblock i'll try the solution next week (probably on Tuesday), thanks a lot

Yep, it works in another directory.
Our getting-started guide doesn't suggest using root and, moreover, we've recently changed it to use a project dir, i.e. not to use a home dir, so should be fine then
I don't think we should add any clarifications about this (i'd say) corner case,
closing, thanks!


[WIP] Redesigned Landing-Page

This is a work in-progress. I am using the landing page to get the basics of how we're going to organize our css figured out. Currently only has the "Getting Started" section decently put together. Very Bare-bones right now.

Nice, will double check this tonight after work.

@brotzky Ahh, this wasn't quite ready for merge yet. I was running into bugs/issues and mostly published this so @mxstbr and @oliverturner could play around with the code.

Sorry! Fairly new to this process. Will learn...

@brotzky As am I 😊. I don't think it's any harm done as we aren't publishing anytime soon.
It probably doesn't help that we've been having the majority of our conversations on gitter. Probably been leaving you out of the loop. I'll try to use github issues/prs a bit more in the future so discussions are a bit easier to keep track of. Also I think a bit do not merge will help next time as well (see that pretty commonly in other repos).
@mxstbr should we just leave this PR as is or revert it? I can always start another PR.

It's fine, I need @brotzky to give me access to the domain anyway, until then I can't point it towards the server

Added to domain manager on namecheap @mxstbr. Point it where you like.


Fix attachments in comments

Fixes #562 and #510
The problem was that ADO does not support adding filePath to the Relation attributes.
Long ago, we decided not to pass the full file path as the fileName any longer, and in the same PR we added the filePath into the comment attribute. See this picture:

This PR will make it so that the importer identifies the attachment by its filePath taken from the comment attribute.
Tested locally and everything works.
Before (JIRA)

After (ADO)

Integration tests passing: https://dev.azure.com/solidify/Internal/_build/results?buildId=9367&view=results

@madkoo, I changed it so that the IdentifyAttachment() function looks at the OriginId instead of the filePath. The OriginId is now passed as part of the comment attribute

@madkoo, I changed it so that the IdentifyAttachment() function looks at the OriginId instead of the filePath. The OriginId is now passed as part of the comment attribute

Perfect ! Thats the way it should be

@madkoo, what do you think? Is it good enough to identify an attachment based on its full path (from the comment attribute)? It's not very pretty to have it in the comment attribute, but can't think of a better solution atm.
We could simply compare the attachment file names, but will that open up a risk of conflicting file names? I don't know if an ADO work item can have multiple attachments with the same name?
@moerketh @omustapasa @C-F-Palmer, any input is appreciated :)

Hi @Alexander-Hjelm,
I planned another migration trial to fine tune some things on my side, I can check it as a part of the process.
I will keep you posted.

Agree, logging and comparing the ID is how it should be. Is comparing the original Id to the attachedFiles of a workitem enough? Is the "IsAttachmentMigrated" still adding value here?

Hi @Alexander-Hjelm, I planned another migration trial to fine tune some things on my side late this week, I can check it as a part of the process. I will keep you posted.

Perfect! Let us know how it goes. Either by emails or through the issue board as usual :)

Agree, logging and comparing the ID is how it should be. Is comparing the original Id to the attachedFiles of a workitem enough? Is the "IsAttachmentMigrated" still adding value here?
To clarify, I think there should be one of two approaches: Via AttachmentJournal, this needs the attachment original id and the devops attachment url Via comparison to the workitem attachments (using the attachment original id)
Or perhaps I am misunderstanding the purpose of the Journal

The Id should be sufficiently unique to use here, as it is globally unique per work item and attachment :)
Regarding the migration journal, I think this is a separate issue altogether. I'm not quite sure of the purpose tbh. @madkoo, would you like to coment?

Agree, logging and comparing the ID is how it should be. Is comparing the original Id to the attachedFiles of a workitem enough? Is the "IsAttachmentMigrated" still adding value here?
To clarify, I think there should be one of two approaches: Via AttachmentJournal, this needs the attachment original id and the devops attachment url Via comparison to the workitem attachments (using the attachment original id)
Or perhaps I am misunderstanding the purpose of the Journal

The Id should be sufficiently unique to use here, as it is globally unique per work item and attachment :)
Regarding the migration journal, I think this is a separate issue altogether. I'm not quite sure of the purpose tbh. @madkoo, would you like to comment?

The purpose of the Journal was to track the progress of the migration so you would be able to resume if needed.


Text changes

Description
supports tickets:

https://apps.nrs.gov.bc.ca/int/jira/browse/SHOWCASE-1897
Rename 'Licensees' field to 'Organizations'.

https://apps.nrs.gov.bc.ca/int/jira/browse/SHOWCASE-1903
only implements the changes highlighted in word document

and temporary fix to prevent dev crashing from keycloak admin refresh token issue. possibly due to RH sso upgrade.
(see second commit)

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
.. with the db migration file, i am adding in three extra fields to the mines_os_submission_attestation table.
it seems with knex that (unless youre using MySQL) you have to add them to the end of the table (after the timestamp fields) which is a bit ugly.

PR Deployed at: https://comfort-dev.apps.silver.devops.gov.bc.ca/pr-161

just to summarise:
the changes from Brittany for the mines form only include the 3 extra checkboxes and an additional supporting document on the landing page. I just noticed the supporting documents are listed on the submission confirmation page.. so i'll add the extra one now.

PR Deployed at: https://comfort-dev.apps.silver.devops.gov.bc.ca/pr-161

PR Deployed at: https://comfort-dev.apps.silver.devops.gov.bc.ca/pr-161


cu

In case Cuba is not part of the world code CU, why the discrimination with the island

https://www.casasparticulares.net/codigos-postales-cuba.php


How to predict images using trained mobilenet model?

Hi ,I somehow managed to train my mobile net model but i want to know with the model trained, how can i  use it to make predictions about images?
Kindly Suggest it will be a great help!

@harshita-pal Can you please try going through this Link. It contains step-by-step procedure and instructions to manage the model.
Let us know if you still need help 😄

@harshita-pal In future, Please post this kind of support questions in stackoverflow. We would like to keep GitHub less cluttered for open source developers. Close the issue as soon as possible. Thanks!

I am implementing MobileNet Modal but it is not giveing me a relevent result see : https://prnt.sc/mwuz3r
this is a Iphone and it is saying Remote.
can I trained my own modal ?

This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!


tinyusb issue

TinyUSB Host CDC MSC HID Example
assertion "ep->active" failed: file "C:\pico-sdk\lib\tinyusb\src\portable\raspberrypi\rp2040\hcd_rp2040.c", line 164, function: hw_trans_complete
I updated the SDK this morning and started getting this issue. Happens with any USB device I plug in, whether it be a mouse, keyboard or gamepad.

the bug is in TinyUSB... i think you can workaround it for now by switching to a release build (or removing the offending assert)

As far as I understand I am using a release build.

well assert is a no-op in release builds (which define preprocessor variable NDEBUG)

If you aren't using an IDE, then make sure you pass -DCMAKE_BUILD_TYPE=Release to cmake


I have a test data as csv file and properties file which i load using properties.load(is); and my tests are still running in sequential mode. if i need to use this plugin how should i handle the test data, properties file, webdriver code thread safe?

Please discuss such general usage questions to the Gitter channel, or more general Cucumber/Selenium mailing lists.


Support of DX12 RenderPass

Description
Render Passes have been introduced in DX12 recently, so an application can now use BeginRenderPass and EndRenderPass instead of OMSetRenderTargets and ClearRenderTargets: https://docs.microsoft.com/en-us/windows/win32/api/d3d12/nf-d3d12-id3d12graphicscommandlist4-beginrenderpass However, when the application uses render passes, RenderDoc is unable to attach to the application and instead it closes it when (probably) encountering the first BeginRenderPass command.
It is a very new feature and Nsight also doesn't support it yet, but it would be nice to have in the future.
Steps to reproduce
Use BeginRenderPass and EndRenderPass and RenderDoc will close the application.
Environment

RenderDoc version: 1.5
Operating System: Windows 10, 1903
API: DirectX 12

There was code in there to capture and replay but it wasn't unwrapping the handles on capture (leading to the crash) and the replay was a bit buggy too.
This has still had only minimal testing because I don't have any example use cases to see the ins and outs, so there might still be bugs in there if you're using it. If you run into anything else please report it.

Thanks for the fix. Now capturing works, but RenderDoc will crash while trying to load a capture which contains render passes. I have some debug layer errors in there regarding resource states which I will fix before saying for certain.


CreateTable Not Working

struct defined here:
type Business struct {
	gorm.Model
	name string
	status string
	public_name string
}

and db setup:
func setupDB() {
	var err error
	db, err = gorm.Open("postgres", "host=localhost user=gorm dbname=gorm sslmode=disable password=mypassword")
	if err != nil {
		log.Fatal("DB error:", err)
	}
	db.LogMode(true)
	db.CreateTable(&Business{})
}

Here is the generated query:
CREATE TABLE "buinesses" ("id" serial,"created_at" timestamp with time zone,"updated_at" timestamp with time zone,"deleted_at" timestamp with time zone , PRIMARY KEY ("id"))
Why is it completely ignoring name, status and public_name?

@brandonecraig  trying to help here with a spare hour... and not familaiar with postgres...
What is the problem exactly, is it potgres specific etc.. ?
and triaging to help jinzhu, who up go.mountains of stuff..

My issue is that when creating a table off of a struct none of my fields are being added to the table. Just id, created_at, updated_at, deleted_at.
My configuration appears to be correct according to the docs yet create table is not including any of my fields.

...its because my structs are lower case. Womp, womp.
Nothing to see here.

he he .. need that gotcha ;-)


--exclude support

Describe the changes here.
Contributor checklist

[ ] Provided the tests for the changes
[ ] Added the changes to CHANGELOG.md
[ ] Requested (or received) a review from another contributor

Closed due to a lack of response.


[BUG] Error after installing: Cannot find module 'ajv/dist/core'

Describe the bug
I get the following output when I try to run the generate operation:
 Error ---------------------------------------------------

  Error: Cannot find module 'ajv/dist/core'
  Require stack: <blah>

So I can't run the plugin at all.
Command: npm run openapi
NPM Script: openapi: "serverless openapi generate --output public/openapi_raw.yaml --format yaml --indent 2 --openApiVersion 3.1.0"
To Reproduce
Steps to reproduce the behavior:
1. run `npm install` running node 14.21.3
2. run `npm run openapi` with the above script configured in your 'scripts' section of your package.json file.

Expected behavior
The plugin to start and at least check my serverless.yml config
Desktop (please complete the following information):

Serverless version: 2.73.3
serverless-openapi-documenter version: 0.0.72

Additional context
Able to get around this by installing the ajv dependency directly with npm install --save-dev  ajv

I'll see if I can have a look tomorrow, but I wonder if it's related to
trying to use OpenAPI 3.1.0... 3.1 isn't quite supported yet.  It's
something I need to look at desperately now that Swagger supports 3.1
On Mon, 18 Sep 2023, 22:25 Matthew Jourard, @.***>
wrote:

Assigned #139
https://github.com/JaredCE/serverless-openapi-documenter/issues/139 to
@JaredCE https://github.com/JaredCE.
—
Reply to this email directly, view it on GitHub
https://github.com/JaredCE/serverless-openapi-documenter/issues/139#event-10403113935,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF4CYDAKA44BAVUN3WO2XDX3C333ANCNFSM6AAAAAA45KRXU4
.
You are receiving this because you were assigned.Message ID:
<JaredCE/serverless-openapi-documenter/issue/139/issue_event/10403113935@
github.com>

🤞
For what it's worth, after tinkering and getting it to generate something, I saw that it required a 3.0.x openapi spec and had to drop it to 3.0.3.
Just tried to remove the ajv dependency and got this:
→ npm run openapi

> <package_name>@<version> openapi ~/<repo>
> serverless openapi generate --output public/openapi_raw.yaml --format yaml --indent 2 --openApiVersion 3.0.3

...

 Error ---------------------------------------------------

  Error: Cannot find module 'ajv/dist/core'
  Require stack:
  - ~/<repo>/node_modules/ajv-draft-04/dist/index.js

Also, ty for the quick responses!

So I had a small play this morning, but i can't replicate the issue with ajv/dist/core...  I notice you're suggesting you're using a version of the frame work i can't install: Serverless version: 2.73.3 i've tried npm i -D<EMAIL_ADDRESS>and receive
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>

 I've got 2.72.3 in both my package.json and package-lock.json file.

I'm not able to replicate this bug, I've installed node 14 and serverless 2.72.3 and can't seem to replicate it.
my package.json looks like:
{
  "name": "serverless-2",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "generate": "npx serverless openapi generate --output public/openapi_raw.yaml --format yaml --indent 2 --openApiVersion 3.0."
  },
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "serverless": "^2.72.3"
  }
}

and you can find an instance of a serverless 2 file in the repo's directory: test/serverless-tests/serverless 2

I was wondering, are there any dependencies you have installed that might be overriding mine, somehow?  Do you have a list of all your dependencies?

Sorry to go dark, work got hectic. To answer your question: can't show you my dependencies list but I can tell you that ajv is used a lot. A couple popular packages that I'm using that ajv was found in:

webpack
eslint
AWS sdk libs

So ya, good call that something else was overriding it.

FYI this issue can likely be closed


@order.proceed_to_confirm

In page "Checkout" after fill order form I click on Continue.
Exec method:
def checkout
    @order = Shoppe::Order.find(current_order.id)
    if request.patch?
      if @order.proceed_to_confirm(params[:order].permit(:first_name, :last_name,  :billing_country_id, :billing_postcode, :email_address, :phone_number))
        redirect_to checkout_payment_path
      end
    end
  end

@order.proceed_to_confirm return false, ActiveRecord rollback transaction on self.update.
What fields are required?

I believe all the address details are required

But I don't need require this info from customers because I'll sell online goods. How to I fill required fields?

My form
.row
.col-md-9
= render 'items', :order => @order
.col-md-3
= form_for @order, :url => checkout_path do |f|
.form-group
= f.label :first_name, 'First name'
= f.text_field :first_name, class: 'form-control', placeholder: 'First name'
.form-group
= f.label :first_name, 'Last name'
= f.text_field :last_name, class: 'form-control', placeholder: 'Last name'
.form-group
= f.label :billing_postcode, 'Post code'
= f.text_field :billing_postcode, class: 'form-control'
.form-group
= f.label :billing_country_id
= f.collection_select :billing_country_id, Shoppe::Country.ordered, :id, :name, { prompt: 'Select country' }, { class: 'form-control' }
.form-group
= f.label :email_address
= f.text_field :email_address, class: 'form-control', placeholder: 'your e-mail'
.form-group
= f.label :phone_number
= f.phone_field :phone_number, class: 'form-control', placeholder: '8-123-456789'
.form-group
= f.hidden_field :billing_address1, value: '-'
= f.hidden_field :billing_address2, value: '-'
= f.hidden_field :billing_address3, value: '-'
= f.hidden_field :billing_address4, value: '-'
= f.submit 'Continue', class: 'btn btn-primary'
I fill all fields, but order don't proceed.

It's hard to say without seeing the rest of the code. Have you followed the
tutorial on the Shoppe site?
On Thursday, May 7, 2015, Aleksey Svechkar<EMAIL_ADDRESS>wrote:

My form
.row
.col-md-9
= render 'items', :order => @order https://github.com/order
.col-md-3
= form_for @order https://github.com/order, :url => checkout_path do |f|
.form-group
= f.label :first_name, 'First name'
= f.text_field :first_name, class: 'form-control', placeholder: 'First
name'
.form-group
= f.label :first_name, 'Last name'
= f.text_field :last_name, class: 'form-control', placeholder: 'Last name'
.form-group
= f.label :billing_postcode, 'Post code'
= f.text_field :billing_postcode, class: 'form-control'
.form-group
= f.label :billing_country_id
= f.collection_select :billing_country_id, Shoppe::Country.ordered, :id,
:name, { prompt: 'Select country' }, { class: 'form-control' }
.form-group
= f.label :email_address
= f.text_field :email_address, class: 'form-control', placeholder: 'your
e-mail'
.form-group
= f.label :phone_number
= f.phone_field :phone_number, class: 'form-control', placeholder:
'8-123-456789'
.form-group
= f.hidden_field :billing_address1, value: '-'
= f.hidden_field :billing_address2, value: '-'
= f.hidden_field :billing_address3, value: '-'
= f.hidden_field :billing_address4, value: '-'
= f.submit 'Continue', class: 'btn btn-primary'
I fill all fields, but order don't proceed.
—
Reply to this email directly or view it on GitHub
https://github.com/tryshoppe/shoppe/issues/183#issuecomment-99884490.

--
Sent from Gmail Mobile

And what's the issue you're having? I assume the product has stock when
purchasing?
On Thursday, May 7, 2015, Aleksey Svechkar<EMAIL_ADDRESS>wrote:

My form
.row
.col-md-9
= render 'items', :order => @order https://github.com/order
.col-md-3
= form_for @order https://github.com/order, :url => checkout_path do |f|
.form-group
= f.label :first_name, 'First name'
= f.text_field :first_name, class: 'form-control', placeholder: 'First
name'
.form-group
= f.label :first_name, 'Last name'
= f.text_field :last_name, class: 'form-control', placeholder: 'Last name'
.form-group
= f.label :billing_postcode, 'Post code'
= f.text_field :billing_postcode, class: 'form-control'
.form-group
= f.label :billing_country_id
= f.collection_select :billing_country_id, Shoppe::Country.ordered, :id,
:name, { prompt: 'Select country' }, { class: 'form-control' }
.form-group
= f.label :email_address
= f.text_field :email_address, class: 'form-control', placeholder: 'your
e-mail'
.form-group
= f.label :phone_number
= f.phone_field :phone_number, class: 'form-control', placeholder:
'8-123-456789'
.form-group
= f.hidden_field :billing_address1, value: '-'
= f.hidden_field :billing_address2, value: '-'
= f.hidden_field :billing_address3, value: '-'
= f.hidden_field :billing_address4, value: '-'
= f.submit 'Continue', class: 'btn btn-primary'
I fill all fields, but order don't proceed.
—
Reply to this email directly or view it on GitHub
https://github.com/tryshoppe/shoppe/issues/183#issuecomment-99884490.

--
Sent from Gmail Mobile

Yes, I follow tut.

fix it

In case anyone else stumbles on this page, this drove me nuts for the longest time until I finally realized I needed to increase the stock levels of the product. I thought I followed the tutorial, but I probably missed something that went along these lines.

.... and delivery service needs to be defined.


Add extensibility point for IQueryOptimizer

This work supports #431 by providing extensibility points for query rewriting expression visitors.

@roji this is ready for review.

@austindrenski the general approach and goals look good. However, the two services you're replacing (IExpressionFragmentTranslator, IQueryOptimizer) are from an EF Core internal namespace, which is something we should be wary of doing.
How about submitting an issue/PR to EF Core itself, for moving these services out of Internal and providing the extensibility point? If you explain the Npgsql needs clearly I don't think they'll object to it. Many services were defined as Internal because they didn't want to commit to a stable interface for them early on, but it seems that things have matured enough for these to be nailed down.
Let me know what you think, and please cc me from any issue/PR.

However, before working on an EF Core issue/PR, it seems like we need to decide whether the null element discrepancy (described here) prevents us from translating constructs such as array1.All(x => array2.Contains(x)) altogether, in which case this PR would have no real use.

@austindrenski is this still actually useful for anything?

@roji Ehh.. It's on the 3.0 milestone for EF Core, but that's a long ways off still. I'll close this, and reopen in the future if the upstream changes are made.

@austindrenski to be clear, I'm actually not 100% against us overriding internal EF Core types - I've done this several times in the past (not sure if there are any currently). The only consequence is having to be reactive to EF Core minor version releases, since breakage may be introduced to these types. But as a general rule we've always released in lock-step with EF Core anyway, so it's not a big deal.
The question is whether there's any value in this, in light of the null behavior discrepancy of array1.All(x => array2.Contains(x)). If there's real value I'm fine with providing our own implementation of IQueryOptimizer, otherwise we can forget about it...

Makes sense, thanks!


typo in email recovery "you" in yellow box should be "your"

This is a (Bug Report / Feature Proposal)
:tophat: Description
For bug reports:

What went wrong?
What did you expect should have happened?
What was the config you used?
What stacktrace or error message from your provider did you see?

For feature proposals:

What is the use case that should be solved. The more detail you describe this in the easier it is to understand for us.
If there is additional config how would it look

:pushpin: Related issues

#010101

:clipboard: Additional Data

Sovereign deployment where you found the issue:
Browser & version:
Screenshot:
Error messages:
URL to reproduce the error:

fixed.


refactor to use fxml

Reactorings for https://github.com/Esri/arcgis-runtime-samples-java/pull/352. The diff looks like I rewrote everything, but mostly all I did was refactor to use fxml. I think it helped separate out some of the view logic and cleaned up the initialize method.

Thanks @tschie. I see quite a few changes in there actually. You took out the enums and replaced with private Feature selectedFeature.
Looks like the enums and decision making when clicking were the original cause of the mess, so I'm happy with your suggestions.


feat: create version metadata config map

during install we should create a config map in the embedded cluster operator namespace containing the version metadata (add-on versions, values, images, etc). this is specially useful for airgap environments.

I still have to adjust a test to check the cm is created correctly so moving this to a Draft.

OK, this is worse than I thought, apparently version is empty in our CI.


Create netsdk1022.md

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Contributes to #21029


Document logging setup

I think it should be toplevel rst file with instructions for logging setup and description for available format options

Fixed somehow


[DO NOT MERGE] Auto PR to update dependencies of mixer

This PR will be merged automatically once checks are successful.
none

This change is

Codecov Report

Merging #1453 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1453      +/-   ##
==========================================
- Coverage   84.66%   84.64%   -0.03%
==========================================
  Files         122      122
  Lines       11921    11921
==========================================
- Hits        10093    10090       -3
- Misses       1632     1635       +3
  Partials      196      196

Impacted Files
Coverage Δ

adapter/prometheus/server.go
89.83% <0%> (-5.09%)
:arrow_down:

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 2105dd3...969a7d4. Read the comment docs.


Make copy & move return list of tasks

What changes are proposed in this pull request?
Make copy & move return list of tasks
Why are the changes needed?
make worker busy
Does this PR introduce any user facing changes?
na

alluxio-bot, merge this please

@Xenorith needs to cherry-pick but not sure which branch we are working on

alluxio-bot, cherry-pick this to branch-3-0.0.x please


SparqlProxy - queryForLabel not defined reasonably for generic SparqlProxy

Currently, @JanVana just copied queryForLabel from dbpediaProxy to generic sparqlProxy - but this is not correct, in most cases there is no sense to look e.g. for some yago resource etc. - so it is necessary to provide more suitable generic version of queryForLabel, which is valid for a generic SPARQL Endpoint
See:
https://github.com/odalic/sti/blob/develop/sti-kbproxy/src/main/java/uk/ac/shef/dcs/kbproxy/sparql/SPARQLProxy.java#L228-L276

I have changed the query for label method. The yago check is now in the DBpediaProxy. Everything else seems to be general enough to be in the SPARQLProxy.
I thought that the Yago is a general ontology used in many knowledge bases, but what I didn't realize is, that the code fixes only DPpedia links pointing to Yago concepts.

Should be fixed in the #333 pull request


Downloaded translations don't match the current project configuration

Describe the bug
Some languages are missing from our build seemingly at random.
To Reproduce
Repository: https://github.com/vuetifyjs/vuetify/actions/workflows/ci.yml
This build worked: https://github.com/vuetifyjs/vuetify/runs/1959052568?check_suite_focus=true
The next one failed on eo and ja: https://github.com/vuetifyjs/vuetify/runs/1959098539?check_suite_focus=true
Expected behavior
publish-docs workflow should download eo-UY, ja, and zh-Hans from crowdin
Screenshots

Additional context
zh-Hans seems to always work, ja and eo randomly fail though. Seems to be most commonly just ja failing.

Hi! Already checking everything

I'm afraid we didn't reproduce this issue. We are still investigating this situation, it seems that the files were not in the branch at the moment of downloading the language eo and ja. In case this happens again - please let us know

Yeah we only store en in git, the other languages are downloaded by github actions before each build.

Hi!
Sorry for the delay! It seems we've missed the message in the flow.
Could you please share if the behavior is still observed?

Yes, this one did it today: https://github.com/vuetifyjs/vuetify/runs/2020840737

Kael, thanks for checking it! Already looking deeper into the case!

Sorry I missed this
@VBeytok Unexpected input(s) 'debug_mode', valid inputs are ['entryPoint', 'args', ...

@VBeytok @og-fox has there been any progress on this?

@KaelWD It may be caused by default API limitation (projects can be built once per 30 minutes, if you trigger builds more frequently - there is a chance that the next build will be skipped). It is now disabled for your project, please let us know if the problem is gone.
Commonly, this limitation is not even noticeable but it may occur if project builds are quite frequent.
You can test it this way:

Add a translation;
Run crowdin download or crowdin download -b branch_name
Add another translation after download is finished;
Run crowdin download again. The project build should not be skipped this time.

Thanks,

That makes sense, looking back it does seem to happen when there's a few CI runs close together. Having a rate limit is always a good idea, but I think some better messaging would be good because currently if you hit the limit it just says "Downloaded translations don't match the current project configuration" with no reason given. If the limit is only for building the project, having it cache builds for 30min would also be acceptable.

Was this intentional? I now get a runtime exception when there's too many CI runs close together:

Hi @KaelWD,
does this message contains the Code: 409, Message: Another build is currently in progress. Please wait until it's finished. response?
If so, then this is expected behavior because parallel downloads are not allowed by Crowdin.

Ah yep. I think we'll need to switch to a daily cached crowdin build then.

Hi @KaelWD,
Sure, you're welcome to try it out 😉

Would be nice if it was built-in though, no need to build the project again if nothing has changed since last time.

Hi @KaelWD,
Sorry for the delayed answer, however, we're wondering if the inconvenience with the build was fixed on your side?

Any update on this, I had the same issue with my project?

Hi there! Previously, you have mentioned that you will switch to a daily cached crowdin build, could you please let us know if you did that?

https://github.com/crowdin/github-action/wiki/Handling-parallel-runs


Added MySQL relay script and documentation

This is a trivial variation on your Docker for Windows script, but I thought someone else out there might be looking for a MySQL relay for WSL, so maybe it's worth including?

Thanks for the contribution!


SVG Download on Safari

I can not properly download SVG file on safari. Was able to properly download SVG of exact same chip design on google chrome.

Tested working on Safari 10.1.1


Add GitHub link to readme - Jakob

Markdown to add a link:
[@yourusername](https://github.com/yourusername)

Markdown to add a link:
[@yourusername](https://github.com/yourusername)


Examples don't woek:

[Enter feedback here]

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

ID: 836d726d-1f95-51e7-c5c3-a7200adc5772
Version Independent ID: 3e9e6e89-7017-7a7c-faea-4c0e76cef442
Content: Get started with F# with command-line tools
Content Source: docs/fsharp/get-started/get-started-command-line.md
Product: dotnet-fsharp
GitHub Login: @cartermp
Microsoft Alias: phcart

closing as dupe of https://github.com/dotnet/docs/issues/22170


[Store] Unnecessary horizontal scrollbar appear in overview and docs pages

In the developer portal, an unwanted horizontal scrollbar appears in the overview page and in the docs page. This is not the case for the other pages. Shown in the below screenshot is one such page.

Also as shown in the below screenshot, there appear to be 2 background images for the "comments" and "documents" sections under the overview page.

But once a comment is added to that particular API, both the backgrounds disappear as shown below (Here since the documents section is still empty, the image should not dissapear).

Consider a scenario where the admin is logged in and an API is selected. As shown below, under the API Credentials section, there is a subheading called "View Credentials". Here "1 Subscriptions" is a link, but it just looks like a text. Therefore suggestion is to improve the UI so that it is clearly visible that it is a link.

Similarly when just a document is added the background should disappear, instead it remains as shown in the screenshot below. Also there should be a "SHOW MORE>>" option as indicated with "1", so that the user is redirected to the "DOCS" page.

Fixed with https://github.com/wso2/carbon-apimgt/pull/7568

This issue is fixed in the latest source


Drone hawk security

DONE. DECEMBER
Launched a red mark detection node on my quadcopter with Raspberry. Further, based on this code, instead of a HSV filter, i will use YOLO v4 will be used to detect different drone models in motion instead of a red mark and will be published to the Following node.
Detection node
Allows you to detect the red mark attached to the drone and published the coordinates to the topic.

Get the image from the camera.
Convert the video stream from the camera into hsv.
Fllter by the desired range hue, saturation, value, using cv2.inRange.
Invert the image to be able to filter everything except the mark.
Get the filter and find the label on the filter image using the contour selection function cv2.findContours.
Go through all the found contours, select those that are more than 10x10 points, and select the largest of them - this will be the label.
Based on the image from the camera, the program calculates the distance to the mark and the yaw and pitch angles by which the mark deviates from the center of the camera image.
Publish the coordinates of the label in the corresponding ROS topics / label_detector / twist

Based on the code of user copterspace from github adapted for python3 noetic and bug fixes.
Demonstration of the Detection node
Following node
Receives the coordinates of the red mark and moves the drone.

The drone's velocity vector is controlled by information about the position of the red label, obtained from the topic / baloon_detector / twist
Send a message about controlling the speed of the drone to the topic / mavros / setpoint_velocity / cmd_vel_unstamped
For the speed of movement forward (relative to the drone), track the current position (pose) of the drone

Link to all code
TO DO. JANUARY 2022

[ ] 1. Test the Following node in real
[ ] 2. Replace the hsv filter with YOLO v4 algorithm

DONE. JANUARY

YOLO4 neural network is configured.
A trained dataset was found to detect different models of drones.
Tested the detection of drones on photos and videos in different conditions and different models of drones. High accuracy obtained (See example in Result).

Result
Process video
Video 1 result
Video 2 result
You can try the model in my  Google Colab
Important! Need to load weights from Github
TO DO. FEBRUARY

Run YOLO4 on Raspberry.
Connect Following node and YOLO4 neural network.
Test all the code on the drone in real.

FEBRUARY
-Developing a YOLOv4 package for ROS Noetic;
-Trying to run https://github.com/slavaroot/darknet_ros on Raspberry;
PLANS. MARCH

Modify and run the YOLOv4 package;
Connect Following node with YOLOv4 package.

Dear R.S. team,
We are closing pull request for your project as you didn’t get to the final round of the competition.
You are welcome to submit your article to the clover documentation as an clover platform-based project. In order to do this please create a new pull request or make a comment to the existing one.
We wish you success in implementing your project and invite you to the next events.


feat(avm): cmp gadget

Please read contributing guidelines and remove this line.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8165  👈
#8164
#7967
master

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

Join @IlyasRidhuan and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

avm_simulation_time_ms (Token:mint_public): 69.4 (+53%)
avm_simulation_time_ms (Token:assert_minter_and_mint): 41.9 (-37%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,750
1,553 (-1%)
708 (-1%)
779 (+6%)
769 (-1%)

proof_construction_time_sha256_30_ms
11,453 (-1%)
3,102
1,373 (-1%)
1,428
1,458

proof_construction_time_sha256_100_ms
43,990 (-1%)
11,796
5,429
5,624
5,745 (-1%)

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
57.0 (-2%)
88.0

proof_construction_time_poseidon_hash_30_ms
1,527
421
201
228
271 (+1%)

proof_construction_time_poseidon_hash_100_ms
5,635
1,514
671
735 (+3%)
745 (-1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
4,356
7,876
14,884

l1_rollup_calldata_gas
50,208
92,902
178,192

l1_rollup_execution_gas
1,396,459
2,130,049
3,915,395

l2_block_processing_time_in_ms
244 (-5%)
443
817 (-1%)

l2_block_building_time_in_ms
8,970
17,363 (-1%)
34,925

l2_block_rollup_simulation_time_in_ms
8,970
17,363 (-1%)
34,925

l2_block_public_tx_process_time_in_ms
7,577
15,869 (-1%)
33,383

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
2,871 (-1%)
3,852 (+4%)

node_database_size_in_bytes
12,611,664
16,736,336

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

FeeJuice:_increase_public_balance
55.7 (+2%)
7,739

FeeJuice:set_portal
11.2 (+22%)
2,354

Token:constructor
82.3 (+2%)
26,051

FPC:constructor
53.5
18,001

FeeJuice:mint_public
40.0 (+1%)
5,877

Token:mint_public
:warning: 69.4 (+53%)
10,917

Token:assert_minter_and_mint
:warning: 41.9 (-37%)
7,512

AuthRegistry:set_authorized
38.0 (+4%)
4,391

FPC:prepare_fee
226 (-3%)
7,043

Token:transfer_public
25.5 (-11%)
39,426

FPC:pay_refund
53.8 (+4%)
10,234

Benchmarking:increment_balance
930 (-1%)
6,563

Token:_increase_public_balance
46.8 (+15%)
8,433

FPC:pay_refund_with_shielded_rebate
68.8 (+9%)
10,783

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.157 (-5%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
2.20
3.85 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.114 (+1%)
0.109 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
11.5 (+2%)
17.4 (-1%)
30.5 (-1%)
58.6 (-1%)
113

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.111 (+2%)
0.101 (-1%)
0.0995 (-1%)
0.101
0.101

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
14.5 (+1%)
25.5 (-2%)
43.4 (-1%)
82.3 (-2%)
161

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.110
0.102 (-2%)
0.105 (-1%)
0.101 (-2%)
0.103

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
16.5
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
132
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.106
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
64,779
668,997

Transaction size based on fee payment method
| Metric |  |
| - |  |

Merge activity

Aug 29, 1:34 PM EDT: @IlyasRidhuan started a stack merge that includes this pull request via Graphite.


helm: Allow templating of RBD striping parameters

Describe what this PR does
PR https://github.com/ceph/ceph-csi/pull/3132 added support for specifying stripeUnit, stripeCount and objectSize as storageClass parameters. However, these parameters were not added to storageClass templated by the Helm Chart at the time. A user who uses the Helm Chart to deploy ceph-csi-rbd could therefore only set these parameters if they managed the storageClass separately.
This PR therefore adds optional support for specifying stripeUnit, stripeCount and objectSize as parameters to the storageClass templated by the ceph-csi-rbd Helm Chart.
The descriptions for each option are taken from the "Available volume parameters" section of the RBD deployment docs, and extended slightly to reference the upstream Ceph documentation.
Is there anything that requires special attention
I've tried to keep the ordering of the new Helm Values similar to the ordering in the RBD deployment docs, however there are already a few inconsistencies. I'm happy to adjust the ordering if needed.
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow
guidelines in the developer guide.
[x] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next major release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)

@Mergifyio queue

/retest ci/centos/k8s-e2e-external-storage/1.27

/retest ci/centos/k8s-e2e-external-storage/1.28

/retest ci/centos/mini-e2e-helm/k8s-1.26

/retest ci/centos/mini-e2e-helm/k8s-1.27

/retest ci/centos/mini-e2e-helm/k8s-1.28

/retest ci/centos/mini-e2e/k8s-1.27

/retest ci/centos/upgrade-tests-cephfs

@mergifyio rebase

@mergifyio queue

@mergifyio requeue

@mergifyio rebase

@mergifyio queue

/retest ci/centos/mini-e2e/k8s-1.26

/retest ci/centos/mini-e2e/k8s-1.26

unrelated failure:
  [FAILED] failed to validate rbd images in pool replicapool trash: failed to list images in trash 2023-11-09T08:10:35.922+0000 7fbabb115700 -1 monclient: handle_auth_reply_more no authorizer?

logs

@mergifyio requeue


Add leaderboard versions

Outcomes

Customers should be able to create multiple versions of a leaderboard.
The public leaderboard view should allow users to view any published version of a leaderboard.
Customers should be able to push data to any version of a leaderboard.

What it should do

[x] User can create draft versions,
[x] publish draft versions as public,
[x] and unpublish published leaderboards back into draft state.
[ ] take snapshots at any time point

Snapshots at any point in time is ambiguous. Versions are snapshots.


[Bug] Can't close depending on application position

Describe the bug
When closing Notepads with an unsaved file open Notepads prompts the user if they would like to save, discard, or cancel. If Notepads is dragged halfway under the taskbar/bottom of the screen and the user attempts to close the application with an unsaved file open it the dialog choices are unselectable, and the user cannot close the application without using task manager or a similar method (You cannot drag the application back into view while the unsaved changes dialog box is open).
To Reproduce
Drag Notepads halfway off the bottom of the screen with an unsaved file. Click close button.
Expected behavior
Allow user to move application while unsaved changes dialog box is open.
Screenshots

Desktop:

OS: Windows 10 Pro 2004 19041.508
Notepads Version <IP_ADDRESS>

It's possible to drag and move the window if you click on a small area next to the left of the minimize button. This is because, by default UWP apps can't have a fully customized title bar, see a related issue here On the other hand, I understand that this is not user friendly at all, but this is not a Notepads bug, and instead this happens because a design on the Content Dialog control. This control is show modally and currently there isn't a modeless version of it. Again, there is a relevant issue here.


Exclude test libraries from being marked as test directories in IDEA.

Fix for issue #95 - leave test libraries as [production] source directories in IDEA, so that tests can depend on them.

@tkrullmann Could you please consider this PR? This bug is effectively preventing us from using test libraries, as it breaks for everyone using IntelliJ.

Thank you!


Fixes to the install scripts for bullseye & 64bit Raspberry Pi OS installs.

Fixes the following issues:
openaps-setup trying to install python-pip, which does not exist under bullseye
openaps-packages trying to install python, python-pip, and python-numpy expecting Python2, when Bullseye is python3-based and does not include the python-pip package (just python3-pip).
oref0-setup not having an 'elif' for aarch64/not installing go because of this.  Note that armv61l go will not work on an aarch64 OS - you need the arm64 version.
closes #1422

Do we also need python3 installed?

Yes.
Python3 and python3-pip are installed during oref0-setup if not already
present (even on buster and earlier).
The issue is that on Buster and earlier, python2 was the 'default python',
with python3 accessed via the python3 executable not the python one.
On bullseye the 'default' is 3 unless you install  the python-is-python2
and python-dev-is-python2 meta-packages.
Eventually the distros will drop python2 compltely (or at least the python
project wants them to). But for now the above works.
On Mon, Feb 28, 2022 at 10:07 AM Jon Cluck @.***> wrote:

Do we also need python3 installed?
—
Reply to this email directly, view it on GitHub
https://github.com/openaps/oref0/pull/1428#issuecomment-1054524360, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAS6ZR3Y342A5MA2BUQZ33U5O2WJANCNFSM5PLPIYGQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

@dcacklam is this repeat of #1428 or something separate?

It merges the same changes into master that 1429 does into dev.
The reasoning I had for doing it that way is that when following the
documented install process, the install scripts I was fixing always pull
from master to start even if they are going to be used to install dev.
On Sat, Jun 18, 2022, 7:33 PM Dana Lewis @.***> wrote:

@dcacklam https://github.com/dcacklam is this repeat of #1428
https://github.com/openaps/oref0/pull/1428 or something separate?
—
Reply to this email directly, view it on GitHub
https://github.com/openaps/oref0/pull/1428#issuecomment-1159603641, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAS6ZXVUNAESHFF3NBSENDVP2BIPANCNFSM5PLPIYGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok, thanks for confirming! I merged the other one to dev after we released 0.7.1; we can include this in the next release once someone else tests. Closing this since #1429 achieves the same thing. Thanks again!

It will take slightly different steps to test without 1428, as the install
routine will fail on Bullseye or with 64bit OS before it reaches the code
in 1429....
On Sat, Jun 18, 2022, 7:43 PM Dana Lewis @.***> wrote:

Ok, thanks for confirming! I merged the other one to dev after we released
0.7.1; we can include this in the next release once someone else tests.
Closing this since #1429 https://github.com/openaps/oref0/pull/1429
achieves the same thing. Thanks again!
—
Reply to this email directly, view it on GitHub
https://github.com/openaps/oref0/pull/1428#issuecomment-1159604510, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAS6ZXL5RCGJUF43UG5PHDVP2CNDANCNFSM5PLPIYGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


ui in server should show formatted code

this view is not really friendly
the ui should show indented code

i would like to suggest some modification, on the UI, but the js code is compressed https://github.com/cpacker/MemGPT/tree/main/memgpt/server/static_files/assets
Is there a repo for the frontend where we could contribute ?


Some Ubiquiti Devices Don't import

I've just run the script today to import Ubiquiti devices to my existing docker instance and noticed that some (two?) devices weren't imported. Seemingly with no errors displayed - but perhaps I'm not reading the log correctly.
The output says 118 device-types found, but only 116 imported.
I noticed that the UniFi Dream Machine Pro wasn't imported (But the UDM & UDM Pro SE were imported successfully), and another unknown device-type also wasn't imported.
I've temporarily solved this my manually importing the UDMP, but it would be good to see this fixed please.
Here's the script I ran:
docker run -e "NETBOX_URL=http://:/" -e "NETBOX_TOKEN=" -e "VENDORS=Ubiquiti" ghcr.io/minitriga/netbox-device-type-library-import > devimport.log
If it's helpful, here's the output:
devtypeimport.log

@savaloi Can be closed?
Using the latest version of the device importer and netbox 3.7.4. They are imported.

@savaloi the docker image is not updated, we currently do not have a way and do not plan on figuring out a way to update it at the moment. One day we will get around to it, but it is best to use this locally


[Feature] Obfuscate strings with number.toString

My idea to use number.toString(base) to obfuscate parst of strings.
For example, 57005..toString(16) equals dead, 29234652..toString(36) equals hello.
Base (aka radix) can be any between 2 and 36, but it can encode not all letters. With 2 we can encode only 0 and 1, with 16 - 0-9a-f combinations, 36 - 0-9a-z.
https://gist.github.com/da411d/51a19300c85bb875ce5f4b93ea209bb0

Interesting

Of course, not every matching part of string should be replaced. It may select randomly only some of them


CLI docs update and markdown linting.

Update CLI documentation per #3426 to acurately reflect the process for local CLI build.
Description
Update to ansible/README.md to reflect changes in the CLI build process and to clarify the need
to run mode=clean as part of attempting an in-place install.  This change alone does not close #3426 as there are still changes needed to the incubator-openwhisk-cli documentation.
For @dubeejw @lornajane
Related issue and scope

[X] I opened an issue to propose and discuss this change (#3426)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[X] Documentation

Types of changes

[X] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[X] I signed an Apache CLA.
[X] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[X] I updated the documentation where necessary.

Thanks!  These instructions enabled me to get much further with this setup.  It would be great if after the instructions to install the CLI, we also then gave instructions on how to run/check it?  The commands completed succesfully but it's not clear what I should do next (I will go and rummage around the docs, but we could make this easier I think)

@lornajane A fair point.  I think that’s covered elsewhere in the ‘docs’ folder, but if you find the right place to go next, feel free to add a pointer to this file.


Code cleanup

Tidy up bits that haven't been touched yet:

[x] Better dev error logging
[ ] Add more tests
[x] Remove debug code
[ ] Investigate upgrading dependencies

Two outstanding A/Cs, moving back to ready

Investigate upgrading dependencies

I think this has since been captured here? https://github.com/silverstripe/silverstripe-webauthn-authenticator/issues/11#issuecomment-501055966

Sure 👍


Add a language selector dropdown

This drop down does not require JavaScript and is computed at build time.
This PR is a JS-free alternative to:

https://github.com/artichoke/www.artichokeruby.org/pull/452

Screenshots

mobile screenshot


VB interpolated strings topic

VB interpolated strings topic
Fixes #3547
Suggested Reviewers
@BillWagner @AnthonyDGreen

Sorry, @BillWagner, I've updated the metadata in this PR, #3546, and not #3549, as I'd commented.


How to move selected objects using the keyboard?

Is it possible to move selected objects using the cursor keys on the keyboard? If appears that the keyboard focus on the canvas is the zoom slider.

We can't do that right now. I'll add that functionality.


Check Rails::VERSION instead of just Rails

Sometimes gems will introduce the module 'Rails' just to get around internal works (sucks but its true);
Since this is trying to access something that could or could not be defined; why not just check the entire tree instead for safety reasons.
Anyone that actually has Rails should have the VERSION field.
FIXES: this stops error collecting if Rails module is defined; but Rails::VERSION is not.

@AlexRiedler thank you for the contribution! There are a few more places this constant is checked, was there something specific about this one that was causing you problems vs. the others? Are you only concerned with it not collecting errors? Also, can you give us an example or two of a gem that defines Rails outside of rails?

@kenichi The reason why is cause error collection fails; but error counts are still collected (very confusing experience). I am not sure why the others do not seem to be affected by this but this is the only one that affected me.
Hmmm I am using rails-html-sanitizer; but I just read now that you should not use it outside Rails and just use Loofah. Maybe I can avoid this in my codebase directly (seems to be a dependency issue on my end).
Maybe this change is not needed then; although I feel like if rails framework is off this shouldn't happen? hmmm, should that be addressed instead? (otherwise I think closing should be fine!)

@AlexRiedler I think the change is not needed, so I'm closing this. Please let us know if you need help with anything else :)

Hi!
We just encountered the same problem. As a lot of gems, not only the rails-html-sanitizer define the rails module (for whatever reason), it would be nice if that check works properly.
The check itself only needs the Rails::VERSION, so it would be proper to only check for Rails::VERSION.
But @danascheider I would also vote for adapting it at all the places where it is used.
Thanks in advance.

Thanks for the contribution @AlexRiedler. Your changes are part of Ruby Agent version <IP_ADDRESS>9 which was released on 12/19.


SPEI : can't figure it out

Discussed in https://github.com/Ouranosinc/xclim/discussions/1589
See comment : https://github.com/Ouranosinc/xclim/discussions/1589#discussioncomment-8155294 for a description of the bug.
Issue comes from this line : https://github.com/Ouranosinc/xclim/blob/bd6a6f87f9cf5897de198108b369efe5356bed87/xclim/indices/stats.py#L765
Having freq='D' (or freq=None with a daily dataset) results in the step previous to the above line grouping the data with time.dayofyear. Thus at the line above, the "temporal" dimension is named dayofyear and not day, which is why this fails.
Code of Conduct

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

I thought I had a test for daily-SPI, apparently not.
I've made the changes, and wanted to add tests to compare with monocongo, but the results don't match. It's a bit hard to find a 3rd party for a tie-breaker between xclim and monocongo since the daily SPI is not that common. I wanted to try with this one: https://github.com/e-baumer/standard_precip, but it only partially obtain the correct values I've established for monthly SPI where monocongo and xclim do match.

I think #1607 fixed that issue? Can you confirm @coxipi ?


Add travis CI

It's nice to have CI for PRs and just in general. This should work, they are hard to test so...

👍

Thanks, Russel!
I didn't have experience with Travis so far but after reading the documentation everything looks fine.


KBFS not available

My K:\ drive appears in explorer but attempting to access it says it's unavailable.
[Window Title]
Location is not available

[Content]
K:\ is unavailable. If the location is on this PC, make sure the device or drive is connected or the disc is inserted, and then try again. If the location is on a network, make sure you’re connected to the network or Internet, and then try again. If the location still can’t be found, it might have been moved or deleted.

[OK]

It used to work, but suddenly has stopped.
I uninstalled the windows client and reinstalled the newest version.  Starting the keybase desktop app the window opens an error message
KBFS client wasn't found
Error: KBFS client wasn't found at t.Error (native) at new t
(file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:11:5254) at a
(file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:11:4885) at o
(file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:11:4810) at
file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:5:23565 at
file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:53:23817 at e.t.Deferrals.e._call
(file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:35:1893) at
file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:35:2044 at c
(file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:35:1671) at e.t.Deferrals.e._fulfill
(file:///C:/Users/.../AppData/Roaming/Keybase/Gui/resources/app/desktop/dist/index.bundle.js:35:2003)

my log id: 1a072efb9d1b82002405bd1c

Hi, I see a failed installation in your logs at one point due to vcredist_2015_x86, which is odd because that shouldn't be necessary on your system. Also, there is a c:\bin\keybase.exe running, of an older version not matching the other .exe files, which is not a target directory we use for that file. I would recommend removing this executable and rebooting, and if the right service still isn't running (C:\\Users\\Lea\\AppData\\Roaming\\Keybase\\keybase.exe), try uninstalling and then installing again.

Thanks that did it.  I don't know why that exe was in c:/bin but I bet I felt real slick when I put it there.


Missing CORS headers

inventory URLs (such as https://ipt.gbif.org/inventory/dataset) cannot be used from webbrowsers due to missing CORS headers
blocks https://github.com/gbif/management-tools/issues/2

@mdoering can I ask you to review this code ?


Now Loading time (future work)

display some image during rendering scenes.

TODO:

Add loading flag in SceneTransition component.


Seaborn pairplot not compatible

Describe the problem
Modin dataframe cannot be passed into sns pairplot function.

Source code / logs
#scatterplot
sns.set()
cols = ['SalePrice', 'OverallQual', 'GrLivArea', 'GarageCars', 'TotalBsmtSF', 'FullBath', 'YearBuilt']
sns.pairplot(df_train[cols], size = 2.5)
plt.show();

@labanyamukhopadhyay thank you for reporting this! This is a duplicate of #847. In both cases, seaborn expects a regular pandas DataFrame, but it gets a Modin dataframe.

duplicate of https://github.com/modin-project/modin/issues/847


feat: add a new class IsZeroOrProbabilityMeasure

The need for this class has shown up in PFR: when conditioning on a set, one gets a measure which is either zero or a probability measure, and many results that hold for probability measures also hold trivially for the zero measure.

Looks good!
bors r+


[ISSUE #3257] Ignore irrelevant commit log files on startup

Prevent irrelevant files under $storePathCommitLog failing startup
Make RocketMQ easy to operate and bring less surprise to average administrators

Make sure set the target branch to develop
What is the purpose of the change
Make RocketMQ more tolerant to minor administrator mistake and reduce service outage in case less experienced admin
create irrelevant files under $storePathCommitLog directory.
Brief changelog
Add regex (20-digit nubmer) filter irrelevant MappedFile names. Ignore files don't match the regex.
Verifying this change
Separate unit test is created to simulate irrelevant files present under commitlog and consumequeue directory.
Unit test is passed.
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.09%) to 54.01% when pulling c43e99c3a6318a474431fc88707f7fa393b5546f on schnell18:master into a20f31bb3de242756542c552d7212a7a40000ba4 on apache:master.

Codecov Report

Merging #3256 (a061fae) into master (a20f31b) will decrease coverage by 3.26%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master    #3256      +/-   ##
============================================
- Coverage     47.72%   44.46%   -3.27%
+ Complexity     4544     3455    -1089
============================================
  Files           552      470      -82
  Lines         36628    28869    -7759
  Branches       4844     3740    -1104
============================================
- Hits          17482    12837    -4645
+ Misses        16917    14327    -2590
+ Partials       2229     1705     -524

Impacted Files
Coverage Δ

.../main/java/org/apache/rocketmq/common/UtilAll.java
35.59% <50.00%> (-2.07%)
:arrow_down:

...tmq/namesrv/processor/DefaultRequestProcessor.java
28.13% <0.00%> (-7.80%)
:arrow_down:

...org/apache/rocketmq/common/stats/StatsItemSet.java
43.28% <0.00%> (-7.47%)
:arrow_down:

...apache/rocketmq/remoting/netty/ResponseFuture.java
85.00% <0.00%> (-5.00%)
:arrow_down:

...ache/rocketmq/broker/topic/TopicConfigManager.java
54.44% <0.00%> (-4.64%)
:arrow_down:

...sumer/rebalance/AllocateMessageQueueAveragely.java
56.52% <0.00%> (-4.35%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
58.11% <0.00%> (-4.28%)
:arrow_down:

...rocketmq/remoting/netty/NettyRemotingAbstract.java
46.86% <0.00%> (-4.06%)
:arrow_down:

...e/rocketmq/remoting/netty/NettyRemotingClient.java
44.05% <0.00%> (-3.19%)
:arrow_down:

... and 94 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 a20f31b...a061fae. Read the comment docs.

IMO, MappedFileQueue and MappedFile are basic components which may be used in other storages.
Checking their names will decrease the flexibility of these components.


Problems with latest version

Hello before Arcana Soul was running without any problem after an automatic update I can't get it running anymore, also Luigi's mansion 2d same story.

Post logs on the related compatibility list entries for those games.

Luigi's mansion 2d crashes
Arcana soul sends back to livearea
arcane soul yyl.log
luigi mansion 2d yyl.log

When you have some spare time can you please solve these problems


ods-buildah build completes successfully without Aqua being installed correctly

In a new ods-pipeline installation I did for a project the pipeline runs were failing in the package-image step with the following error:
aqua scan: start cmd: fork/exec /usr/local/bin/aquasec: exec format error

As there was an issue with aqua somehow I checked the logs of the successful ods-buildah build an noticed the following:
< HTTP/2 200
< date: Fri, 02 Dec 2022 07:56:48 GMT
< content-type: application/json; charset=utf-8
< server: openresty/<IP_ADDRESS>
< strict-transport-security: max-age=31536000; includeSubDomains
< content-security-policy: img-src *;
< x-frame-options: SAMEORIGIN
< x-content-type-options: nosniff
< referrer-policy: no-referrer-when-downgrade
< permissions-policy: midi=()
< docker-distribution-api-version: registry/2.0
< www-authenticate: Basic realm="Aqua Download"
< x-xss-protection: 1; mode=block
<
{ [0 bytes data]
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
* Connection #0 to host download.aquasec.com left intact
Aqua scanner version:
Aqua scanner installation completed!

compared to a successful installation of aqua in a different project:
< HTTP/2 200
< date: Fri, 02 Dec 2022 11:13:46 GMT
< content-type: binary/octet-stream
< content-length: 86966209
< server: openresty/<IP_ADDRESS>
< strict-transport-security: max-age=31536000; includeSubDomains
< content-security-policy: img-src *;
< x-frame-options: SAMEORIGIN
< x-content-type-options: nosniff
< referrer-policy: no-referrer-when-downgrade
< permissions-policy: midi=()
< docker-distribution-api-version: registry/2.0
< www-authenticate: Basic realm="Aqua Download"
< x-xss-protection: 1; mode=block
< x-amz-id-2: 4oEy/bQm6T2zQ340wvv1LSUpGFO62wVA2+eINWZUn9hJTth2oWTsbkbghbr9xqxk6w5NGcEwF1k=
< x-amz-request-id: THTJPZ41JJT8VXX5
< last-modified: Mon, 28 Feb 2022 16:53:49 GMT
< etag: "3b35ac02dae988dd031bd712a64e6398-11"
< accept-ranges: bytes
<
{ [15612 bytes data]
0 82.9M 0 495k 0 0 558k 0 0:02:32 --:--:-- 0:02:32 557k 25 82.9M 25 21.3M 0 0 11.4M 0 0:00:07 0:00:01 0:00:06 11.3M 52 82.9M 52 43.7M 0 0 15.2M 0 0:00:05 0:00:02 0:00:03 15.2M 80 82.9M 80 66.3M 0 0 17.2M 0 0:00:04 0:00:03 0:00:01 17.2M100 82.9M 100 82.9M 0 0 18.3M 0 0:00:04 0:00:04 --:--:-- 18.3M
* Connection #0 to host download.aquasec.com left intact
Aqua scanner version:
scannercli version 5.3.22055, compiled Feb 24 2022 16:53:20
Aqua scanner installation completed!

It is obvious that something is incorrectly setup leading to the aqua scanner not being downloaded, but no matter the reason why the aqua installation failed, the build should not complete successfully if the aqua scanner wasn't download and installed correctly.

Interesting, this happened some time ago as well, I think there is something wrong with how the Aqua scanner is provided.
There is an aqua-gate param (see https://github.com/opendevstack/ods-pipeline/blob/master/docs/tasks/ods-package-image.adoc). Unless this is set to true, the build won't fail. In your specific case it should distinguish though which kind of error it is and fail if aqua cannot be executed at all. BTW, which version are you using? 0.7?
Finally, just for context: the download of the Aqua scanner will likely be moved to happen in the task instead of being downloaded in the wrapper image, see #621.

Sorry for not clarifying, yes, 0.7.0 was used.
good to know wrt. #621. Then I will remove Aqua in this project for now.

@henninggross Simply rebuilding the image should fix your issue and you can use Aqua. #621 is not removing Aqua, it is just changing how it is installed.

I tried rebuilding three times but that did not fix the issue. Actually today I tried a fourth time and Aqua was installed successfully.

I am going to reopen this because the issue that the tasks succeeds without the scanner being involved is still there.

Closing now that #621 is merged.


checkout is not working

when paywithflutterwave is click nothing shows

Hi @ONUOHA92, apologies you're experiencing this, please can you provide a code snippet of your implementation? Please remember to remove your public keys when sharing.
Thanks.

i am trying to implement this library in my react native project but it show error that one or more required parameter missing @ONUOHA92 can you help me in this?
thanks in advance.

Hi @pallavbansal, can you share your implementation of PayWithFlutterwave, I can help you out?


HODL, Server,  API to subscribe to HODL

get Message to sign for HODL;
Input:  public key hash
Output: message to sign.
Subscribe to HODL:
Input: message, public key,  signature

done by Chris


Duplicated spans for cassandra

Describe the bug
By default cassandra instrumentation attaches itself to cqlsesion. Because reactivesession wraps around sqlsession and it's being instrumented as well, we're getting spans for the same "requests" twice. Right now to fix that you can simply exclude TraceCassandraAutoConfiguration.class in @SpringBootApplication from autoloading, but it would be nice if those would detect which one should be loaded and when.

Can you provide a complete, minimal, verifiable sample that reproduces the problem ? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.
Can you also provide the Sleuth version?

Depending on setup it sometimes happens to inserts, sometimes to selects. In our services it happens to both. With this example it mostly occurs on inserts.

https://github.com/maciej-gromul/problems-with-cassandra-autoconfiguration

@mp911de do you recall these problems?

When i do breakpoints it registers both CqlSession and ReactiveSession tracing so the problem is most likely somewhere in that area.

Thanks for having a look. There's indeed a risk that the instrumented CqlSession serves as underlying object for ReactiveSession and that is being used later in ObservableReactiveSession. In Spring Data Cassandra, we introduced a ObservableCqlSessionFactoryBean accepting CqlSessionBuilder to mitigate that risk.

It seems to be closed with issue 1366

Really this one https://github.com/spring-cloud/spring-cloud-sleuth/issues/1366 ? It doesn't seem to have anything to do with cassandra

Sorry issue from spring-data-cassandra, my bad https://github.com/spring-projects/spring-data-cassandra/issues/1366

Please upgrade to Micrometer Tracing. Spring Cloud Sleuth is feature complete and out of OSS support.


[WIP] Make  "extensible" for other plugins.

Great plugin! I realized that this plugin clobbers any prior mapping to <CR> in insert-mode. Specifically, I noticed that vim-endwise doesn't work because it maps <CR>, which this plugin clobbers in any buffer where it is activated.
When I used COC, I ran into the same problem. This comment is my solution to that problem: map a plugin mapping to the added functionality and chain that to the desired mapping.

Right now, I have most of the cases worked out. The only case I can't quite figure out is when the PUM is open and nothing is selected. With the current code, I have to hit <CR> twice to engage endwise. I assume that's because of the order of the mappings, but if I change them around, I get a recursive mapping (norecurse isn't viable here, plug maps need the regular map commands).

Duplicate of #154