Forgot/reset/confirm reset pages design is broken

After login page redesign forgot/reset and confirm reset pages look strange:

We need to prepare new design and update pages accordingly.

@rahuldhide can you please take a look?

I have updated the screen designs based on the Argo design guidelines.

There is one more screen: https://dev.applatix.net/setup/signup/faketoken
Do we still need it? I created new account and this screen is not used during process of creation. I'm not sure but it looks like death code.

@wokeGit  It is not dead code. This screen is required for group invitation: go to invite new user screen and check 'Mailing group' checkbox. Please apply new design to this screen as well

@wokeGit From the design perspective, we can reuse the 'Welcome- Create Account Password' screen for this requirement. We just need to update the panel title to- Create your Argo account.

There is one more screen not available on the mocukps. I updated styles for that as well.

@rahuldhide Do we need any update for messages for that screen?
All the screens are updated and ready to merge:
https://github.com/argoproj/argo/pull/83


No "thumbnail" on Android, present on IOS

On Android, if you render the Video element with the 'paused' prop set to true, no "thumbnail" is shown. On IOS, a thumbnail is displayed.
I'm guessing that this is due to the difference in how the videos are handled, perhaps IOS is fetching some data on load when Android is not, haven't had a dig for the cause.

As a workaround to this I have found that if in the onLoad event if you seek to zero the thumbnail appears.

@selsamman How did you do this?

<Video source={{uri: uri}}               paused={true}               ref={(ref)=>{this.player=ref}}               onLoad={()=>{this.player.seek(0)}} \>
Still sometimes it doesn't load when working with the debugger. I am working with local files rather than urls.  If it is not reliable as I move to production I will use a screen shot stand in.

iOS can generate thumbnail with uri of video but android doesn't.
You can generate the thumbnail from the uri of video with react-native-thumbnail.
Then place the exported thumbnail on top of the video and then play video once you click play button or any other actions.

@selsamman , I am getting this error this.player.seek is not a function on your solution

<Video source={{uri:${props.source}`}}
paused={paused}
style={{ ...StyleSheet.absoluteFill }}
onLoad={load}
onProgress={progress}
resizeMode="cover"
ref={ref => (video = ref)}
seekColor="#fff"
/>
const load = ({duration}) => {
video.seek(0)
setDuration(duration);
} // now here the duration is update on load video
`

i have a swiper which contains multiple videos and if i use seek method for thumbnail then last video of the swiper seeks to 0 but other's don't so, how to create thumbnail for all swiper elements ?

i have a swiper which contains multiple videos and if i use seek method for thumbnail then last video of the swiper seeks to 0 but other's don't so, how to create thumbnail for all swiper elements ?

I guess because you just bind the last video to the ref. I suggest that you can create a new react component, put  into this component and bind  into this component, then import this component to the screen where you were playing the video

my code started working with following code
const videoRef  = useRef();
onLoad={() => {
videoRef.current.seek(0);
}}


Add set_game_tags

I'm also unsure about asserts. While docs saying that tags string can not be longer then 128 symbols (127 + \0), it is still possible to set string with more then above said length. However tags string returned by PingServer after that will still be truncated to 127 + \0 symbols. So the question is should we check the string length?

Thanks for the PR. Since its unintuitive and error prone to silently have your strings truncated, I would say you should assert the length


Bug: Command-ids should not contain spaces

One of the recent updates introduces the creation of custom quick switchers (which was awesome, btw!).
However, with the new commands, they now by default contain spaces (e.g. "Recent Search"). These are carried over to the command id: obsidian-another-quick-switcher:search-command_Recent search
having spaces in the command id is a problem, since it prevents assigning the command in the vimrc plugin via exmap:
" go = open quick Switcher
exmap quickSwitcher obcommand obsidian-another-quick-switcher:search-command_Recent search
nmap go :quickSwitcher
vmap go :quickSwitcher

Replacing all spaces in the command-id with hyphens would fix this issue.

Thank you for your report, @chrisgrieser :)
I agree with you. But I think it affects only vimrc plugin (because Obsidian accepts the key name that contains whitespace). And it affects users who use AQS.
So, I'll implement it in the next major release (maybe soon), which includes the features optimized for Obsidian v0.16.0 (ex: tabs). Probably, in v7.0.0-beta1 👍

Released in 7.0.0-beta1 🚀

Released in v7.0.0 🚀

minor follow-up issue:
it seems that only the first space is replaced, which in some rare cases like the file name search is noticable:
obsidian-another-quick-switcher:search-command_recent-search
obsidian-another-quick-switcher:search-command_file-name search <-- still a space there
obsidian-another-quick-switcher:search-command_landmark-search
obsidian-another-quick-switcher:search-command_star-search
obsidian-another-quick-switcher:search-command_backlink-search

Thanks. Fix in v7.0.1 👍


The useDirtyRect feature should be added to the API doc and etc

The useDirtyRect feature should be added in echarts.init.

It is worth mentioned that when calling echarts.init with opts, a null should also be passed if no need to specify a theme:

const chart = echarts.init(dom, null, {renderer: 'svg'});

#175


[VA-2129] Update XDM with latest Media Analytics changes

Issue #700

Not sure why the Adobe CLA check still fails, my request to join the Adobe Github org had been accepted before opening this PR.


misleading results for ambiguous axes

Reported by hankin on 7 Jun 2000 23:08 UTC
Changes in the behavior of caching cause requests with ambiguous axes to be
evaluated whereas they used to generate errors. e.g.
=========================
! err520_ambig_axes_not_flagged
! sh 6/00
define axis/x=1:10:1 x1
define axis/x=20:40:1 x20
let a1 = X[a20 = X[gx=x20](gx=x1]
let)
! this fails to generate a real error because the default I=1:10
! limits of a1 are acceptable to a20
list a1-a20
! this generates a real error because the default I=1:20
! limits of a2 are not acceptable to a1
list a20-a1
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/72

Comment by<EMAIL_ADDRESS>on 8 Jun 2000 14:51 UTC
Resolved by change to complete_missing_limits -- set region to its world
coordinate limits rather than its subscript limits.
... possibility of unanticipated side-effects from this seems high ...


Update the addon in VS gallery.

The version in VS Gallery doesn't seem to be working.

thanks looking into it, I'll upgrade it to stable Roslyn 1.0.0 (it was created based on a pre-release)

It's been updated https://visualstudiogallery.msdn.microsoft.com/4151b07d-ed17-4158-960b-0c14eae7c330


factor registry connectors add doesn't handle multi-line input

So if you run factor registry connectors add chef it will ask for multiple SSH keys. The input is a single line input so you can't open the SSH key.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/8076462-factor-registry-connectors-add-doesn-t-handle-multi-line-input?utm_campaign=plugin&utm_content=tracker%2F4030409&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F4030409&utm_medium=issues&utm_source=github).

Cut factor registry so this isn't applicable.


how can i set date format?

I have multiple inline editor on page. but i have issue with date type.
when i set date format as 'y-m-d' then date value is pre-selected   in inline editor, but 'm-d-y ' date format is then value is not selected, and it show "dd-mm-yyyy".
now i want my date format as 'm-d-y' so how can i set date format?
can you help me?
i am attaching this SS pleas have a look.
https://www.screencast.com/t/umh3RR1koL4

@xxxtonixxx can you help?

Hi @archish1989, that feature isn't implemented.
And we're working in release the 1.0.0. We add this feature to release with it :+1:

FYI i was in hurry so I've solve my issue like this.
template file:
 <inline-editor (onCancel)="editCancle()" (onEdit)="savePrevValue($event,'paymentrecived')" empty="Enter Payment Received Date" type="date" class="clsProject" [(ngModel)]="paymentrecived" (onSave)="saveEditable($event,'paymentrecived','date')" name="paymentrecived" ></inline-editor>
ts file

let invDat = new Date(projectForm.projects.paymentrecived);
paymentrecived = dp.transform(invDat, 'MM-dd-yyyy');
setTimeout(function(){
     $( "input[ng-reflect-name='paymentrecived']" ).val(dp.transform(invDat, 'yyyy-MM-dd'));
}, 1000);

setTimeout(function(){
$( "input[ng-reflect-name='paymentrecived']" ).val(dp.transform(invDat, 'yyyy-MM-dd'));
}, 1000);
This code done magic for me.

Thanks for share your workaround while this feature is missing :heart:


Publish the opt-in by default script to S3

On release, we need to publish the opt-in script to S3 as well as the opt-out script.

https://downloads.mesosphere.io/dcos-cli/install-optout.sh

@tamarrow Can we do this as part of the release CI build?

fixed on teamcity


Support HOF (React.Element)

Current Renderer support (0.2.0)
React.TV(React.createElement('h2', null, 'This App is available only at LG WebOS');

Next Renderer support:
React.TV(
   React.createElement(
      'div',
      null,
      React.createElement('h1', null, 'This App is available only at LG WebOS')
   )
);

Closed by 20a423a69e5cdc8c41f5f0a8df6db2b4476cd06c


Preact not working after version 5.3

After version 5.3 preact doesn't work anymore.
That's the error:
Error: Invariant Violation: Failed exporting HTML for URL ...: RouterScroller(...):
Nothing was returned from render. This usually mean  s a return statement is missing.
Or, to render nothing, return null.

#521  #471

Well, this is weird. I'll need to dig into it more.


Remove deprecated version property from docker-compose.yml

Currently, running any of the make commands that make use of docker compose shows the following warning:
WARN[0000] /warehouse/docker-compose.yml: `version` is obsolete

The version property has been deprecated and compose does not use it to validate the docker-compose.yml file.
See this issue for more context.
cc @miketheman @woodruffw

Great!


Vanity import path seems to be broken

Hello! I noticed when the Go module proxy (proxy.golang.org) is turned off, the oras.land/oras-go import stop working:
mike@talos /Users/mike/go_test
> go mod init go-test
go: creating new go.mod: module go-test

mike@talos /Users/mike/go_test
> cat go.mod
module go-test

go 1.19

mike@talos /Users/mike/go_test
> ll
total 4
-rw-r--r-- 1 mike 24 Feb  8 13:04 go.mod

mike@talos /Users/mike/go_test
> GOPROXY=direct go get oras.land/oras-go
go: unrecognized import path "oras.land/oras-go": reading https://oras.land/oras-go?go-get=1: 404 Not Found

mike@talos /Users/mike/go_test
> GOPROXY=direct go get oras.land/oras-go/v2
go: unrecognized import path "oras.land/oras-go/v2": reading https://oras.land/oras-go/v2?go-get=1: 404 Not Found

mike@talos /Users/mike/go_test
> go get oras.land/oras-go
go: added oras.land/oras-go v1.2.2

mike@talos /Users/mike/go_test
> go get oras.land/oras-go/v2
go: added oras.land/oras-go/v2 v2.0.0

It seems like the required <meta> tag is missing from https://oras.land/oras-go : https://pkg.go.dev/cmd/go#hdr-Remote_import_paths

Hi @betamike, thanks for reporting this! This was a regression and has fixed by https://github.com/oras-project/oras-www/pull/96.
Could you please try again?

Fixed by https://github.com/oras-project/oras-www/pull/96


Mesh

hi, i try the network / mesh example between 2 pico boards but not get the mesh working...
i try 1 node with Y > 0 and 2nd with Y > 1.. it will not connect to the mesh master. With 1 node as arduino and c++ code it connect to the mesh mast well. If i try just Network it works well togehter. ANy idea here? May i try it in wrong way using your example?

I try the mesh with 2 pico (circuitpython mesh example) and 1x arduino with c++. As soon the arduino is connect to the master then also the 2nd pico can connect. Thats strange. Can you help me to get the example working with just the 2 circuitpython?

The quick answer is that the circuitpython firmware runs slower than the C++ examples.
I documented the connection process in the topology page of the docs. What you're experiencing is a common point of failure (point 2 in the list).
I'm not sure what the best solution is here. Essentially the non-master node needs to have a slight delay so the child nodes on network level 1 have enough time to switch back to RX mode and complete the pairing process.

Are these nodes not stationary? Is that why RF24Network is insufficient for your application?

It means its not possible to connect 2 picos with circuitpython by Mesh? (1 is master and 2nd is a child)
I want use mesh because i not want setup octal node id´s. it should hold it simple by simple set on a child nodes 1,2,3,4 etc

btw if i not use Mesh (in exmaple 'N' > 0 and 2nd 'N' >1) it works great

RF24Network is much more reliable for nodes that don't physically move around. RF24Mesh is designed for nodes that do physically move around, but the connecting process is flawed because the process doesn't use auto-ack. Transmissions in RF24Mesh will also be slower because the octal address has to fetched from the master node before any the message can be sent to the actual destination.

ok i will try that tomorrow.
If i use network instead, is there a way to automatically give the child nodes id on connect to master or a similar workaround to not need to set node id on each child into the code?
also, how is the best way to send a message to all childs? just by a loop only possible or exist a function already for?
Am 7. Nov. 2022, 17:05 +0100 schrieb Brendan @.***>:

The actual line of code that is failing in the master is here:
https://github.com/nRF24/CircuitPython_nRF24L01/blob/c73978c954d074cebd1337227dbc7ae08fbda51c/circuitpython_nrf24l01/rf24_mesh.py#L373
You could try doubling that call so it transmits twice:
self._write(self.frame_buf.header.to_node, TX_PHYSICAL)
self._write(self.frame_buf.header.to_node, TX_PHYSICAL)
Hopefully that will give the child enough time to receive the the response from master and complete the process).
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

is there a way to automatically give the child nodes id on connect to master or a similar workaround to not need to set node id on each child into the code?

No, you need to assign a unique id to each node in the network. There is no way around that.

how is the best way to send a message to all childs?

This is called "multicasting". You can enable multicast_relay and use multicast(). Beware that auto-ack is disabled for multicasting, so there is no guarantee that the message is received by all nodes.

on multicast how can i send to all nodes without worry to set the correct level of the child nodes? basically to all childs not depend on level they are
Am 7. Nov. 2022, 17:24 +0100 schrieb Brendan @.***>:

is there a way to automatically give the child nodes id on connect to master or a similar workaround to not need to set node id on each child into the code?
No, you need to assign a unique id to each node in the network. There is no way around that.
how is the best way to send a message to all childs?
This is called "multicasting". You can enable multicast_relay and use multicast(). Beware that auto-ack is disabled for multicasting, so there is no guarantee that the message is received by all nodes.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

I suppose there should be an example code snippet in the docs for this...
From master node, you just need to
mesh_node.multicast(payload, 0, level=1)

this broadcasts the payload to all child nodes on level 1. If multicast_relay is set to True (on all child nodes in level 1), then the payload is re-broadcasted to all child nodes on level 2. I think you get the idea from there.

Typically you don’t want to enable multicast relay with RF24Mesh it will interfere with mesh functionality. You would need to manually relay multicast packets

On Nov 7, 2022, at 10:55 AM, Brendan @.***> wrote:

I suppose there should be an example code snippet in the docs for this...
From master node, you just need to
mesh_node.multicast(payload, 0, level=1)
this broadcasts the payload to all child nodes on level 1. If multicast_relay is set to True (on all child nodes in level 1), then the payload is re-broadcasted to all child nodes on level 2. I think you get the idea from there.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

it means if i require multicast (what i need) then its better to go with Network instead Mesh?
Is in Network a function available to get a list of all registered childs at master?
Am 7. Nov. 2022, 18:55 +0100 schrieb TMRh20 @.***>:

Typically you don’t want to enable multicast relay with RF24Mesh it will interfere with mesh functionality. You would need to manually relay multicast packets

On Nov 7, 2022, at 10:55 AM, Brendan @.***> wrote:
I suppose there should be an example code snippet in the docs for this...
From master node, you just need to
mesh_node.multicast(payload, 0, level=1)
this broadcasts the payload to all child nodes on level 1. If multicast_relay is set to True (on all child nodes in level 1), then the payload is re-broadcasted to all child nodes on level 2. I think you get the idea from there.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Yup

On Nov 7, 2022, at 12:07 PM, conaito @.***> wrote:

it means if i require multicast (what i need) then its better to go with Network instead Mesh?
Is in Network a function available to get a list of all registered childs at master?
Am 7. Nov. 2022, 18:55 +0100 schrieb TMRh20 @.***>:

Typically you don’t want to enable multicast relay with RF24Mesh it will interfere with mesh functionality. You would need to manually relay multicast packets

On Nov 7, 2022, at 10:55 AM, Brendan @.***> wrote:
I suppose there should be an example code snippet in the docs for this...
From master node, you just need to
mesh_node.multicast(payload, 0, level=1)
this broadcasts the payload to all child nodes on level 1. If multicast_relay is set to True (on all child nodes in level 1), then the payload is re-broadcasted to all child nodes on level 2. I think you get the idea from there.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Is in Network a function available to get a list of all registered childs at master?

No. RF24Network doesn't need to keep track of node addresses that correspond to node IDs (the RF24Mesh master node's DHCP list). All logical addresses (octal numbers) are static in RF24Network.

Double the line 373 not help :( any other idea?
Am 7. Nov. 2022, 23:21 +0100 schrieb Brendan @.***>:

Is in Network a function available to get a list of all registered childs at master?
No. RF24Network doesn't need to keep track of node addresses that correspond to node IDs (the RF24Mesh master node's DHCP list). All logical addresses (octal numbers) are static in RF24Network.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Just in case I get the time to reproduce this,

What version of CircuitPython are you using?
I assume you're using the latest version of this lib (v2.1.2)?

i use on 1 pico CircuitPython 7.3.3 on the other 8.0.0 beta4
the latest from GIT
at moment i play with the network example (non mesh) and hang on send a string (from emit function).. lets say "b24z" and print it on idle function on other pico. i really not get it working. can you give me small example code for both pico´s? (i know it not really lib depending but it help me a lot

Honestly, your English is hard to read and understand. I will not write code for you - that is what the documentation is supposed to help with.
If by string you mean "some value", then that is an invalid data type for a payload. Payloads in this lib should only use bytes or bytearray objects.
Please also read the python docs about using struct.pack() or struct.unpack(). I will not teach you on this.

i understand to need follow on sender message = struct.pack("4s", b"cool") but no chance to get it correct unpack on receiver :(

for actual strings, you could use str.encode()
msg = "cool"
nrf.send(msg.encode(encoding="utf-8"))

and then use bytearray.decode() on the receiver.
rcv = nrf.read()
msg = rcv.decode(encoding="utf-8")

raw bytes objects are easy enough to construct, but the functions exist to deal with data that isn't a simple ASCII character.
>>> b"cool" == "cool".encode(encoding="utf-8")
True

Have you tried using the more basic RF24 examples? This should've been your first step in understanding how to use the library. Once you can get the RF24 examples working, then you know its not a radio hardware problem, and you can graduate to networking usage.

thx a lot about the string message sending - works very well and i can work with it.
about Mesh: hopefully you can find the time to figure out why it not working. yes i also tried other samples and it work too except the Mesh example

I already know why it isn't working. I documented the problem as a known point of failure. As stated in the docs:

If a master node is able to respond faster than the connecting node can prepare itself to receive, then the process will fail entirely. This failure about faster master nodes often results in some slower RF24Mesh nodes only being able to connect to the network through another non-master node.

This is a complete hack, but I'm curious if using the auto-ack would help resolve this...
Instead of
https://github.com/nRF24/CircuitPython_nRF24L01/blob/c73978c954d074cebd1337227dbc7ae08fbda51c/circuitpython_nrf24l01/rf24_mesh.py#L373
try
                    self._write_to_pipe(self.frame_buf.header.to_node, 1, False)

To be clear, the hack I just posted is meant for the master node, not the child node.

i test it out tomorrow and let you know. thank you
Am 8. Nov. 2022, 14:36 +0100 schrieb Brendan @.***>:

To be clear, the hack I just posted is meant for the master node, not the child node.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

It seems there is no way to reliably fix this within the confines of circuitpython. If we make any changes to this lib, then they should also be added to the RF24Mesh C++ lib because the code for this lib's RF24Mesh and RF24Network is directly ported from the C++ libs.


rambox-pro: 1.2.2 -> 1.3.0

Motivation for this change
Updating a package to its latest version.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc maintainers @cawilliamson

Closing this as it is outdated


Fix flavor param with incorrect type hint in read_html

refs:

https://github.com/pandas-dev/pandas/issues/55059
https://github.com/pandas-dev/pandas/pull/55076

Please add a test or this change.

A test similar to https://github.com/pandas-dev/pandas-stubs/blob/6bb12157a810cc26e034e012bff6a0ec2da191d0/tests/test_io.py#L1240 Pandas-stubs writes test to verify that type checkers pick up the proposed changes (and to prevent regressions).

Done, @twoertwein and @Dr-Irv.
The PR https://github.com/pandas-dev/pandas/pull/55529 adding the HTMLFlavors type has been approved in the pandas repository. As discussed, I've updated it here to align with it.


Issue with saved file and location

I've set the file format to: [%disc%-][%track% ][%title%] and the storage directory to "Same directory as track".
This works great but I found one song that's causing a problem.
It's by Bobbie Gentry from the album Ode to Billie Joe.
The song name is: The Seventh Son (Demo) / Bonus Track
This causes openlyrics to create a subfolder 1TNDI~N and a file Bonus Track.lrc
If I rename the title to The Seventh Son (Demo) then it's ok and the file is save as: 1-11 The Seventh Son (Demo).lrc
This new folder is probably due to the / in the tile.
I noticed that other invalid characters are being replaced but this one isn't.
Expected behavior
Save the file as: 1-11 The Seventh Son (Demo) _ Bonus Track.lrc or anything else but do not create a subfolder
Versions
foobar2000: v2.0 32-bit
openlyrics: v1.6

This is working as intended, see discussion in #244


Publishing Status

Hello. I was curious as to what the status is for this project. The last published version on the NPM registry is 0.13.0; however, there's been several updates since then that I'd be interested in using. I also noticed this package - https://www.npmjs.com/package/@codingame/monaco-languageclient which seems to be publishing the updates and is linked to this repo as well.
Is the preference here to use that package or is it possible to get a new version published here: https://www.npmjs.com/package/monaco-languageclient?
Thanks

Hello!
The original maintainers of this project are not active anymore. I don't have access to their npmjs package and the automated workflows are broken.
I now publish every update to @codingame/monaco-languageclient. I plan to mark this repo as deprecated, switch to a CodinGame github repo and automated everything again, but it takes time. It's gonna happen eventually as soon as I manage to get some time to do it.

Hi @CGNonofr! Firstly, thank you very much for maintaining this code!
I'm sorry for the long silence from TypeFox regarding this repository. Let me try to explain.
The initial motivation for the development of monaco-languageclient was to have direct support for language servers in Theia IDE, which uses Monaco as its text editor. With version 1.4.0 published in July 2020, this language server support was deprecated in favor of the VS Code extension API. This meant that monaco-languageclient was no longer needed for that project, and we stopped contributing to it because there were loads of other pressing topics at the time (we developed Gitpod, which was later moved to a separate company).
We would like to get back to contributing to this project. We see Monaco editor being increasingly used within all sorts of web applications, not only web IDEs like Theia. Initially our contributions will be done mainly by @kaisalmen, but others from our team may join as well.
I think it would be beneficial for this project and the community around it to avoid fragmenting the source in multiple active forks. @CGNonofr would you agree in continuing to maintain monaco-languageclient in this repo together with us? If you're interested, I can give you access to publish the original monaco-languageclient package, but of course you can continue publishing to @codingame/monaco-languageclient if you prefer that.
In any case, we want to publish up-to-date versions of the monaco-languageclient package. @CGNonofr do you see any chance in identifying the git commits from which you published the eight versions from 0.14.0 to 0.17.3? There are no git tags for those versions, and the package.json version number in the source is still at 0.13.0. If there's no way to recover that information, we'll just publish the latest version, but it would be great to have the previous versions as well.

Hello @spoenemann!
It's great news!
I'm fine with publishing to the monaco-languageclient package only, the only reason I was publishing it to @codingame/monaco-languageclient was the permissions.
I think I didn't do a great job tagging differents releases, we can probably identify them by hands, there's isn't a ton of commits.
I'm ok to with getting npmjs permissions but I think the first step is to fix the automated workflows

I'm fine with publishing to the monaco-languageclient package only, the only reason I was publishing it to @codingame/monaco-languageclient was the permissions.

Great! Do I see correctly that your npmjs.com username is nonofr? In order to get publishing access, you need to enable two-factor-authentication.

I think I didn't do a great job tagging differents releases, we can probably identify them by hands, there's isn't a ton of commits.

Great! Do I see correctly that your npmjs.com username is nonofr?

Correct!

In order to get publishing access, you need to enable two-factor-authentication.

Done :heavy_check_mark: I should have done it a long time ago

@CGNonofr I want to take the opportunity and say hello.
Like @spoenemann said I will the be the first from the TypeFox team to contribute and help here again. I started to work on topics regarding monaco-editor quite recently and we started to work on web components around monaco-editor and monaco-languageclient this year (https://github.com/TypeFox/monaco-components not yet released on npm) and we agreed that I should start to contribute here and we stop the being silent.
As you said getting everything going again is needed. I propose to open an issue about this topic (release workflow, changelog,  tags, etc.)

Hello :hand:

As you said getting everything going again is needed. I propose to open an issue about this topic (release workflow, changelog, tags, etc.)

good idea :+1:
Some more stuff for the backlog:

The examples/browser is confusing people since it doesn't use the whole lib but only the converters
Some tools/libraries are now quite old (typescript for instance)
A new monaco-editor version was released, we should check the compatibility

Some more stuff for the backlog:

Yes, reconnecting web socket is also a topic where I was asking myself if it could/should be done differently

A new monaco-editor version was released, we should check the compatibility

Here I also need to "learn" how to analyse changes and adapt here

Yes, reconnecting web socket is also a topic where I was asking myself if it could/should be done differently

You're right, I ended up not using it and writing my own implementation (there is already restart policy in vscode-languageclient, I don't thing we need a websocket reconnect lib)

Here I also need to "learn" how to analyse changes and adapt here

When there is breaking changes, typescript tells it, then I usualy  just run it on the most complete language server I know and test all features

Hi.  I wanted to chime in and add my excitement to this issue.  We're using this (and the codingame fork) in our app, and  and have found it tough.  I was checking the issues today because I was worried this had become abandoned.
I'm thrilled to see that this repo will start getting some love again, and am really grateful to the people who take time to maintain it - in whatever capacity - Thank you @kaisalmen , @CGNonofr and @spoenemann.
I wish I could offer to help (I don't have the time), I'd be glad to sponsor or contribute in a way that isn't coding related.  (Fond wishes?)

Thanks @martypitt! @kaisalmen and I work at https://www.typefox.io offering consulting for open source libraries and frameworks. If there's anything you'd like to see improved in short term, sponsoring that work is definitely an option. You're welcome to contact me<EMAIL_ADDRESS>

Publishing was successfully revived! 🎉 Let's close this.


Support forward without a batch dimension

Feature description
Support forward without a batch dimension
Feature motivation
Linear model import from pytorch cannot be used as it for example.
Imported model work with tensor of rank 1 but Burn Linear doesn't not support tensor of rank 1 at this time.
As discuted with @nathanielsimard on discord.

So nice ! When we could expect it to be release ?
In a 0.13.1 maybe ?


CustomButtonComponent type mistakenly named as CustomButtonProps

Type naming issue
While DefaultPublishButtonProps is correctly named as props, CustomPublishButtonProps is actually a component and shouldn't be named as Props. Maybe it should be named CustomPublishButton or CustomPublishButtonComponent?
All custom buttons are affected and this issue is reflected in the docs:
import * as React from 'react'

import {
  CustomSaveButtonProps,
  CustomSaveDraftButtonProps,
  CustomPublishButtonProps,
  CustomPreviewButtonProps,
} from 'payload/types'

export const CustomSaveButton: CustomSaveButtonProps = ({ DefaultButton, label, save }) => {
  return <DefaultButton label={label} save={save} />
}

export const CustomSaveDraftButton: CustomSaveDraftButtonProps = ({
  DefaultButton,
  disabled,
  label,
  saveDraft,
}) => {
  return <DefaultButton label={label} disabled={disabled} saveDraft={saveDraft} />
}

export const CustomPublishButton: CustomPublishButtonProps = ({
  DefaultButton,
  disabled,
  label,
  publish,
}) => {
  return <DefaultButton label={label} disabled={disabled} publish={publish} />
}

export const CustomPreviewButton: CustomPreviewButtonProps = ({
  DefaultButton,
  disabled,
  label,
  preview,
}) => {
  return <DefaultButton label={label} disabled={disabled} preview={preview} />
}

export const MyCollection: SanitizedCollectionConfig = {
  slug: 'my-collection',
  admin: {
    components: {
      edit: {
        SaveButton: CustomSaveButton,
        SaveDraftButton: CustomSaveDraftButton,
        PublishButton: CustomPublishButton,
        PreviewButton: CustomPreviewButton,
      },
    },
  }
}```

## Additional Details

<!--- Provide any other additional details -->

Hey @franciscolourenco - thanks so much for taking the time to share your thoughts regarding our naming convention.
Regarding the suggestion to rename CustomPublishButtonProps - in our current setup, we use the suffix Props to specifically denote types or interfaces that define the properties a React component expects. This convention helps us distinguish between the components themselves and the types of their props.
So, CustomPublishButtonProps is intended to describe the shape of the props for a custom component, indicating that it is not a component but a type definition for the props that such a component would receive.
Does this make sense?
Really appreciate your feedback and always open to discussing our practices and exploring improvements.

Hi @PatrikKozak that makes sense, and the convention is perfect. The problem is that while CustomPublishButtonProps is intended to describe the shape of the props for a custom component, in this case it is actually being used to describe the component itself.
export type CustomPublishButtonProps = React.ComponentType

@franciscolourenco Ah yup I see what you're saying. CustomPublishButtonProps is actually a type for a component - not just props.
I'm thinking a better name might be CustomPublishButtonType then, what do you think?
I'll relay this to the rest of the team and get back to you on this, thanks for bringing this up!


add wrw pass.

Add wrw pass to master

On hold, too much weird conflicts. Looks like the master branch is still in older version.

finally... It is able to merge..


I sent my STX tokens from Ledger to Binance but i did't receive at my Binance acc.

Describe the bug I sent my STX tokens from Ledger to Binance but i did't receive at my Binance account. The Status of transaction is "Confirmed". https://explorer.hiro.so/txid/0x5e5f92073400995c366958b995c6df8f645e024c1fbdeb95b3a85013fd01abc9?chain=mainnet (this is the link of the transaction)
A clear and concise description of what the bug is.
Transaction ID 0x5e5f92073400995c366958b995c6df8f645e024c1fbdeb95b3a85013fd01abc9
Address SPKMPKDH8N0RX6RQEYPX4Z1QM0KSK0XYTHRAWGTH(Ledger Adress) to SP2TA4FGB43WVAS8MVS6YCWTSN2BZNQ1ASGEAKSDD(Binance Adress)
Block#
Time stamp 02/21/2024 11:52 AM
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here:

Tx is confirmed, I suggest you contact the exchange to track down your tokens.


Remove (enrichment) option

This PR closes #308.
(enrichment) option is not available anymore. The contract of (enrichment_for) and (by) options are refined and made stricter.
Further changes to the enrichment functionality should be done in the following PRs. (e.g., see #310 and #311).

@alexander-yevsyukov PTAL again.


Allow escape sequence in Apache access log

http://httpd.apache.org/docs/current/mod/mod_log_config.html#format-notes

Hmm... 2.4 on Mac OS X test failed.
Maybe, your patch is not related. Please wait.

Thanks!


Segmentation fault with mingw-w64 32-bit release binaries

I get a segmentation error when I use 32-bit binaries in the Release configuration, but everything is fine in the Debug configuration.
For renderarea example from SoQt I get:
Reading symbols from renderarea.exe...
(gdb) run
Starting program: renderarea.exe
[New Thread 7596.0x2484]
[New Thread 7596.0x350]
[New Thread 7596.0x176c]
[New Thread 7596.0x2674]
[New Thread 7596.0x197c]
[New Thread 7596.0x228c]
[New Thread 7596.0x1ec0]
[New Thread 7596.0x2754]
warning: mincore\com\oleaut32\dispatch\ups.cpp(2122)\OLEAUT32.dll!773B245F: (caller: 773B3D7F) ReturnHr(1) tid(2798) 8002801D
[New Thread 7596.0x1b38]
[New Thread 7596.0x1434]
[New Thread 7596.0x1ad8]
[New Thread 7596.0x126c]
[New Thread 7596.0x1f94]

Thread 1 received signal SIGSEGV, Segmentation fault.
0x0061fe63 in ?? ()
(gdb)

I also don't have any problems with 64-bit binaries.
I am asking for help in debugging the incident.

Thanks for notifying us about this issue.
I built coin and soqt under msys2 32bit and could reproduce the issue. As I am not very knowledged with gdb I was first checking the link time dependencies of renderarea.exe, libSoQtd.dll and libCoin-80d.dll by using ntldd as I was supposing a runtime libraries mismatch from your report. renderarea.exe correctly links to the debug versions of Coin and SoQt. When checking libSoQtd.dll I noticed, that it links to the release version of Qt5 although a debug version has been installed. Can you attach the output of ntldd libSoQtd.dll, to make sure, SoQt is properly built in debug mode?

Thanks for notifying us about this issue.
I built coin and soqt under msys2 32bit and could reproduce the issue. As I am not very knowledged with gdb I was first checking the link time dependencies of renderarea.exe, libSoQtd.dll and libCoin-80d.dll by using ntldd as I was supposing a runtime libraries mismatch from your report. renderarea.exe correctly links to the debug versions of Coin and SoQt. When checking libSoQtd.dll I noticed, that it links to the release version of Qt5 although a debug version has been installed. Can you attach the output of ntldd libSoQtd.dll, to make sure, SoQt is properly built in debug mode?

@VolkerEnderlein I have no problems when Coin built in debug mode. Segmentation fault actual only for 32-bit built in release mode. The python-pivy scripts also work when build coin package in debug mode without debug postfix.
You may be surprised when you see the filenames of the qt5-debug package.

@VolkerEnderlein I have no problems when Coin built in debug mode. Segmentation fault actual only for 32-bit built in release mode. The python-pivy scripts also work when build coin package in debug mode without debug postfix.
You may be surprised when you see the filenames of the qt5-debug package.

@podsvirov Thanks. I was getting it wrong that the debug version failed and you're right I was also very much intimidated from the filenames of the qt5-debug package.  :) How do I link properly with the debug version of Qt5? Does CMake handle that or do I need to install some additional stuff? Until now I installed Qt5 and Qt5-debug by calling pacman -S mingw-w64-i686-qt5-debug mingw-w64-i686-qt5 The debug build succeeded, but I could not run any program built so far.

@podsvirov Thanks. I was getting it wrong that the debug version failed and you're right I was also very much intimidated from the filenames of the qt5-debug package.  :) How do I link properly with the debug version of Qt5? Does CMake handle that or do I need to install some additional stuff? Until now I installed Qt5 and Qt5-debug by calling pacman -S mingw-w64-i686-qt5-debug mingw-w64-i686-qt5 The debug build succeeded, but I could not run any program built so far.

First update on the issue: I had crashes in all test programs when building current Coin version using MinGW. CMake supports the build type RelWithDebugInfo out of the box and I built Coin/SoQt and the test programs for 32bit using this. I was able to debug the renderarea.exe and embeddedexaminer.exe with gdb but the results do not make much sense to me as they vary from build to build. Sometimes I get an Illegal instruction error, other times a Segmentation fault error. Most times the programs crash when leaving a function scope or when doing sychronisation in SoDB::readunlock() function while traversing the scene graph. One time I was crashing in strcmp! I also built Coin with Autotools (configure) and SoQt with CMake (as SoQt Autotools scripts do not support Qt5) and the program crashed. Even enabling COIN_THREADSAFE on the Coin library did not succeed. Next I will try to build Coin/SoQt with an older gcc (mingw-8.1 or mingw-7.3) to check whether it depends on the compiler.

First update on the issue: I had crashes in all test programs when building current Coin version using MinGW. CMake supports the build type RelWithDebugInfo out of the box and I built Coin/SoQt and the test programs for 32bit using this. I was able to debug the renderarea.exe and embeddedexaminer.exe with gdb but the results do not make much sense to me as they vary from build to build. Sometimes I get an Illegal instruction error, other times a Segmentation fault error. Most times the programs crash when leaving a function scope or when doing sychronisation in SoDB::readunlock() function while traversing the scene graph. One time I was crashing in strcmp! I also built Coin with Autotools (configure) and SoQt with CMake (as SoQt Autotools scripts do not support Qt5) and the program crashed. Even enabling COIN_THREADSAFE on the Coin library did not succeed. Next I will try to build Coin/SoQt with an older gcc (mingw-8.1 or mingw-7.3) to check whether it depends on the compiler.

@VolkerEnderlein thanks for your efforts in researching this issue.

@VolkerEnderlein thanks for your efforts in researching this issue.

Gdb sometimes isn't able to get useful results on Windows, in that cases you can resort to lldb, windbg or winedbg (on Linux).

Gdb sometimes isn't able to get useful results on Windows, in that cases you can resort to lldb, windbg or winedbg (on Linux).

I checked mingw73 and mingw81: all were crashing in release mode for the CMake and Autotools build. Finally, after trying many other things I got the programs to run in Release mode, too. It seems to be an issue with optimization. When lowering the optimization level from -O2 to -O1 or -O0 I got the program running. Will further need to find out what causes this, as the 32bit versions in Ubuntu 16.04 (using GCC 5), 18.04 (using GCC 7), and Visual Studio run flawlessly and did not crash.

@podsvirov So I examined the optimization issue somewhat further. It could be reduced to the -fipa-icf option of GCC that is added when the -O2 option is set (but it occurred only in case of MinGW). Adding all other options (one by one and and all together) does not lead to the faulty behaviour. See here for a explanation of the differences between -O1 and -O2 and here about the optimization it is doing. So how would one proceed to find the code that causes this segfault?

So how would one proceed to find the code that causes this segfault?

On Linux one would typically use C-Reduce to get minimal reproducer.

@mati865 Thanks for the pointer. I will check if it can be used with MinGW too.

Ok, further update on the issue. It all boils down to the combination of the -fomit-frame-pointer and -fipa-icf optimization options. Adding both of them in addition to -O0 makes the program crash with segfault, adding either one of them makes the program run. Weird.

@VolkerEnderlein, your suggestion to add a compile flag via -DCMAKE_CXX_FLAGS=-fno-ipa-icf is applied in the msys2/MINGW-packages#7908. Rebuilt packages are available in the pacman repository with version 4.0.0-3.


How to do CI with such large dependency graph?

Because of this project's dependency on Kubernetes' API, glide up (which I have abstracted behind make bootstrap) resolves a massive graph of 102 dependencies (most of which are probably not needed).  If running from scratch, this can take 15 minutes.  That's not so bad for a developer.  Even if glide up is re-run periodically, in theory, running from scratch is a one-time thing.
But how can we handle this in CI so that tests don't take forevvvvvvvar?
Note that this question first popped into my head after looking at this PR:
https://github.com/deis/etcd/pull/11
Ping @mboersma @sgoings @technosophos for thoughts.

I believe we solved this in github.com/helm/helm. You can take a look at the travis file there and see if that helps.

@technosophos ah... Travis lets us cache directories.  Cool!  And that was in the other PR I referenced to and I overlooked it.


Gradle dependency definition in docs is incorrect

There should be a colon between the group ID and the artefact ID, i.e.
compile 'com.auth0:java-jwt:2.2.0'

instead of
compile 'com.auth0.java-jwt:2.2.0'

@donalmurtagh - thanks for pointing this out.
PR raised


settings.gradle conflict using firebase_auth: ^0.16.1

Error list:
Running Gradle task 'assembleDebug'...
Plugin project :firebase_core_web not found. Please update settings.gradle.
Plugin project :firebase_auth_web not found. Please update settings.gradle.
Flutter doctor result:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.17.4, on Microsoft Windows [Version 10.0.19041.329], locale en-GB)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[√] Android Studio (version 4.0)
[√] Connected device (2 available)
• No issues found!
Fixes attempted:

Tried changing minSdkVersion from 16 to 21.
In the app level build.gradle added multiDexEnabled true to defaultConfig and added dependency for androidx.multidex:multidex:2.0.1.
Updated settings.gradle to -->

include ':app'

def flutterProjectRoot = rootProject.projectDir.parentFile.toPath()

def plugins = new Properties()
def pluginsFile = new File(flutterProjectRoot.toFile(), '.flutter-plugins')
if (pluginsFile.exists()) {
pluginsFile.withReader('UTF-8') { reader -> plugins.load(reader) }
}

plugins.each { name, path ->
def pluginDirectory = flutterProjectRoot.resolve(path).resolve('android').toFile()
include ":$name"
project(":$name").projectDir = pluginDirectory
}

Included firebase_core: ^0.4.5 in pubspec.yaml and did pub get

The errors still remain the same.
Experts, please advise.

I have the same issue exactly. Thought it was worth noting the Flutter code works fine on iOS.
Steps to reproduce: Install stable release of  Flutter 1.17.4, Go through steps of codelab at [https://codelabs.developers.google.com/codelabs/flutter-firebase/ ] adding firebase to the default app.
Expected Result: App runs identically on iOS and Android
Actual Result: App runs on iOS, App fails to run on Android with error
Plugin project :firebase_core_web not found. Please update settings.gradle. Plugin project :firebase_auth_web not found. Please update settings.gradle.

duplicate of https://github.com/FirebaseExtended/flutterfire/issues/2796

I thought about this and thought it was environmental and was able to get my app working by resolving inconsistencies in my android applicationID and package settings, (I had renamed it) and using the settings in #2796 - all working on 1.17.5 now. sorry for the noise.


Long text breaks out of card - visual bug

Feature
Proposal list item
App

[X] Admin
[ ] Summon
[ ] Hub
[ ] Toolbox

Expected Behavior
text should be hyphened, truncated or some other solution so it doesn't break the card
Actual Behavior
Long text breaks out of card
Steps to Reproduce
No response
Browser
No response
Screenshots and Context

another example

@memosys

Fixed in PR #493


Master lab build and signing

Need to reactivate build, verify localization and signing, especially for .NET Core.
Create new build definition

This is actually for msys work. Lab build etc is #3510

Msys was merged


WIP: Added arm64 jobs in Travis-ci

Updated travis.yml file to add ARM64 jobs in Travis-CI. Added support for python version 3.6, 3,7 and 3.8 for ARM64 jobs.
Resolves #1401.

This is great! I think we can do fewer arm versions, just ZMQ= and ZMQ=bundled for python 3.7, 3.8 (4 total) and skip the tornado matrix items, which have no reason to behave differently on arm.
Since this is new, I think we can limit oldest ARM builds to 3.7, unless there's a really strong case that old 3.6 is very widely used on ARM.
For the wheel builds, I think we want to port over the logic from tools/tasks.py, which builds libzmq+libsodium from source, and we want to make sure the linux wheels built on CI do the same. Also for the wheel builds, we probably want to run the tests after installing from the wheel instead of running tests from the source directory prior to building an untested wheel.

I think we can do fewer arm versions, just ZMQ= and ZMQ=bundled for python 3.7, 3.8 (4 total), and skip the tornado matrix items, which have no reason to behave differently on arm.

@minrk As per your suggestion I have updated .travis.yml to run for python 3.7, 3.8 (4 total), and removed all the other jobs for arm64.

I think we want to port over the logic from tools/tasks.py, which builds libzmq+libsodium from source, and we want to make sure the Linux wheels built on CI do the same.

Also, I have updated tools/tasks.py to create wheel for aarch64, now wheels are getting created successfully.
Please have a look and share your opinion.


Java/C++/C#: Add support for dataflow exploration by partial paths.

This adds support for exploring the dataflow graph starting from a source.  It is intended as a drop-in replacement for the workflow where you'd typically use isSink(Node n) { any() } in order to debug missing flow steps.
Using a sink specification of any() had various problems in that flow is still pruned if a field store cannot reach a corresponding field read, so you wouldn't be able to see all of the partial flow.  With this change you can instead use the hasPartialFlow predicate, which will include all partial flow paths regardless of whether a store can reach a read.
As this can generate lots of node-accesspath-callcontext tuples (aka pathnodes) the search is limited by staying within a configurable distance of the source.

Could you add a test case and release note.

LGTM, but a bit of a shame that we have to replicate (and keep in sync) much of the logic from the proper flow relations.

Can any of this be Pyrameterized?

LGTM, but a bit of a shame that we have to replicate (and keep in sync) much of the logic from the proper flow relations.

Can any of this be Pyrameterized?

It's hard because even though there are several similar predicates, they are not quite equal - in this case the flow exploration is mutually recursive with the accesspath cons cell generation, which isn't the case in the same way in the main implementation.  With parameterized modules I'd say maybe, but as long as pyrameterization is all we got, I'd say no.

Rebased to resolve merge conflicts in change notes.

Rebased to resolve merge conflicts in change notes (again).

No further comments in the past week, so let's merge.


(feat)basic auth view and controller

not yet tested signup auth
-added dependencies to bower.json

Can you separate these into separate requests? We should test the authentication code before pushing to master.


chore(master): release 2.1.13

:robot: I have created a release beep boop
2.1.13 (2024-02-21)
Bug Fixes

deps: update dependency laravel/framework to v10.45.1 (746bab7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v2.1.13 :sunflower:


[Bug] get-info.sh file not working

Describe the bug
We followed the steps to install the workbench using AWS Cloud9 and EC2 instances. Deployment was successful, but the summary info that contains the API endpoints and URL is not obtained.
To Reproduce
Steps to reproduce the behavior:

Deploy the workbench using the AWS Cloud9.
Expected Result doesn't match the actual result. The summary analysis is not obtained

Expected behavior

Actual Behavior

Hi @aswin2025!
This is likely due to a PNPM issue of PNPM version 7 (v7.1.2 as of May 2022). You can run the following command to check your pnpm version: pnpm --version.
Workaround: Uninstall pnpm and reinstall the latest pnpm v5
npm uninstall -g pnpm
npm install -g pnpm@latest-5
pnpm --version

We'll be updating the documentation shortly. Thank you for bringing this to our attention. Let me know if the error persists after this workaround, or feel free to close the issue if this fixes it for you!

Thanks for your response @maghirardelli. The workaround helped in resolving this issue


⚠ Bofrost.fr has degraded performance

In a7bb860, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1571 ms

Resolved: Bofrost.fr performance has improved in b4dfb3d after 6 minutes.


[Rquest feature] Static methods for comparison

The library allow to execute a comparison between 2 Big values using the method of first one, and put the second as argument of method:
var a = Big(100);
var b = Big(5);
var areEquals = a.eq(b);

There are circustances where a and/or b can be nothing. Then a static comparison method versions are very useful.
Then instead of do this:
var areEquals = !a && !b
  ? true
  : !a || !b
    ? false
    : a.eq(b);

...it is possible to do this:
var areEquals = Big.eq(a, b);

Yes, that could be useful.
I would probably go with
Big.eq = (a, b) => { try { return Big(a).eq(b) } catch { return false } };

But I don't want to add any further methods, thanks, static or otherwise.

Perfect, good!
Maybe, can a and b to be "equals" when they are "null" / "nothing" together?


Custom policy clone in private repository does not work when using a GitHub App API token

Description
When using a custom policy located in a private repository with a GitHub App API token, the ghascompliance tool is not able to clone the repository containing the policy.
This is due to loadFromRepo function which is directly using the API token with https://  scheme, to retrieve the content of the repository containing the custom policy.
def loadFromRepo(self):
        instance = urlparse(self.instance).netloc
        if self.token:
            repo = "https://" + self.token + "@" + instance + "/" + self.repository
        else:
            repo = "https://" + instance + "/" + self.repository

However, GitHub App API token needs to set x-access-token as username when using https:// scheme to clone a repository (see https://docs.github.com/en/developers/apps/building-github-apps/authenticating-with-github-apps#http-based-git-access-by-an-installation). Thus the clone is failing, when the custom policy is placed in a private repository (when using GitHub App generated API token)
Propose Solution
Add a command line parameter to indicate that a GitHub App token is used, so that the x-access-tokenusername is added in the clone HTTPS URL. Here is a quick example of a code that could do so:
        def loadFromRepo(self):
        instance = urlparse(self.instance).netloc
        if self.token:
            if not self.isGitHubAppToken:
                repo = "https://" + self.token + "@" + instance + "/" + self.repository
            else:
                repo = "https://" + "x-access-token:" + self.token + "@" + instance + "/" + self.repository
        else:
            repo = "https://" + instance + "/" + self.repository

Merged and released in 1.6. @4bg0P If you can help create any documentation that would be great. I'll create an separate issue for this.

Merged and released in 1.6. @4bg0P If you can help create any documentation that would be great. I'll create an separate issue for this.

Hey @GeekMasher! Thank you for the merge. Sure, I will help you document this added feature in the following days.
Have a great day,
Théo


[feaLib] Support CID representation

Currently, feaLib.Lexer supports CID representation but feaLib.Parser doesn't:
$ cat test.fea
feature test {
    sub \1 by \2;
} test;

$ python
Python 2.7.11 (default, Dec 16 2015, 11:56:37)
[GCC 4.2.1 Compatible Apple LLVM 7.0.2 (clang-700.1.81)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from fontTools.feaLib.parser import Parser
>>> Parser('test.fea').parse()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/mnakamura/repos/fonttools/Lib/fontTools/feaLib/parser.py", line 41, in parse
    statements.append(self.parse_feature_block_())
  File "/Users/mnakamura/repos/fonttools/Lib/fontTools/feaLib/parser.py", line 776, in parse_feature_block_
    self.parse_block_(block, vertical)
  File "/Users/mnakamura/repos/fonttools/Lib/fontTools/feaLib/parser.py", line 828, in parse_block_
    statements.append(self.parse_substitute_())
  File "/Users/mnakamura/repos/fonttools/Lib/fontTools/feaLib/parser.py", line 505, in parse_substitute_
    old_prefix, old, lookups, old_suffix = self.parse_glyph_pattern_()
  File "/Users/mnakamura/repos/fonttools/Lib/fontTools/feaLib/parser.py", line 236, in parse_glyph_pattern_
    gc = self.parse_glyphclass_(accept_glyphname=True)
  File "/Users/mnakamura/repos/fonttools/Lib/fontTools/feaLib/parser.py", line 189, in parse_glyphclass_
    self.expect_symbol_("[")
  File "/Users/mnakamura/repos/fonttools/Lib/fontTools/feaLib/parser.py", line 903, in expect_symbol_
    self.cur_token_location_)
fontTools.feaLib.error.FeatureLibError: test.fea:2:9: Expected '['

Are there any plans to support CID representation? It is essential to develop CJK fonts.
You can find actual examples of feature files in adobe-fonts/source-han-sans, such as Regular/features.JP.

Thanks for the report! CIDs should now be handled; just pushed a fix. It will take a bit until feaLib can process the feature file for Source Han Sans, but I’m working on adding support for the remaining tables.

Thank you!


Consider switching to gopacket/gopacket

google/gopacket has become unmaintained as the original maintainer has left Google.
Consider making the switch to gopacket/gopacket, for details see https://github.com/google/gopacket/issues/1016

Sounds good, however, I'd like to wait until v1 is released. Just to be safe.
PS: you probably meant to say "switch to gopacket/gopacket"

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

v1.0.0
GitHub release

Your semantic-release bot :package::rocket:


Add public key to output

https://issues.redhat.com/browse/HACBS-1280
Signed-off-by: Luiz Carvalho<EMAIL_ADDRESS>

I like the addition of the Policy struct, makes sense to pass the evaluation configuration in one parameter. I don't think we should be outputting the key that we were configured with, I think we should be outputting the key(s) from the signatures. Even with that, I think this is a good first pass and can be improved in subsequent stories/PRs.

Let's get this right then. I don't know what you mean by "the key(s) from the signatures". How do I get that with cosign today?

@zregvart, I'm marking this as ready for review after adding the new commit that disables the public key in the hacbs report.
I'm close to being ready to submit the code changes for adding the attestation signatures (per component) in the yaml/json reports. That change adds to the commits of this PR. If we agree that displaying the public key is generally useful, I'd like to merge this PR soon. This will make reviewing easier. Thoughts?

Thoughts?

Let's merge :)


MutiSelect checkboxes clicking not working properly

Version
2.0.5
Here's what went wrong:
In MutiSelect, When I Click on any of checkbox it is hitting onChange two times. Due to this if I write an API calling in onChange method then it is firing two times. Instead of clicking on checkbox if I click on lable in <li> then it is working fine.

Hi @arun567
Weird bug but I can replicate, I'll take a look.
Thanks for raising!

:tada: This issue has been resolved in version 3.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Hi Andrew Durber,
Bug still exist when you select and deselect "All" option.I mean when includeSelectAll={true}

:tada: This issue has been resolved in version 3.0.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


architecture-on-a-page

Overview
The intent of this document is to provide a high level scope, define a requirements-stack,  and illustrate an architectural vision for the effort.  The core practices we followed were:

Active Stakeholder Participation
Collective Ownership
Display Models Publicly
Model In Small Increments
Prove It With Code
Use The Simplest Tools
Create Simple Content
more ...

Sprint 1  'portal-proxy'
Functionality

Order
Description
Comment

1
Implement project security
Missing feature necessary for workflows and cohort discovery (currently mocked within the UI.  target ohsu ldap for authorization and authentication)

2
Load all of OICR
Currently we've loaded only AML projects

3
release ohsu.dcc
demo: 1)branded ohsu.dcc portal, 2) ohsu.ldap & user roles 3) refreshed data[baml, icgc] pancreatic data is not in scope

 from roadmap
Deployment

The dcc-portal-ui, euler.api and openstack.keystone components are self contained in a docker container, quick start

Dataflows

(0) - the projects' meta data and files are loaded into dcc (aka 'release')
(1) - an admin user adds project structure using openstack command line
(2) - a research user logs in and provides their ldap username and password.
(3) - the euler.api calls keystone to get a token and returns it as a JWT (json web token)
(4) - the user makes any /api call, the euler.api validates token and applies filters

Dependencies

openstack versions

Keystone:  VERSION=10.0.0 https://github.com/openstack/keystone/archive/${VERSION}.tar.gz
Openstack  CLIENT_VERSION=3.2.0 pip install  python-openstackclient==${CLIENT_VERSION}

keystone system account

Ability to create domain & projects

exastack

euler nodes needing certs and dns entries in  CCC domain

dms_development
dms-staging
dms-prod

all communications via standard http, https ports
all hosts need access ingress OHSU-SECURE net and  egress to ohsu.ldap
all hosts need dns (already done- check.)

Sprint 2 'add-a-file'
Functionality

Order
Description
Comment

4
Implement lightweight add a file
Missing feature necessary for workflows (includes ccc_client. limited to repository functionality.  release is out of scope)

5
Implement OHSU data store
While we have demonstrated loading and searching OHSU data, we have not demonstrated access to the BAML or other OHSU data (eg. downloading, workflow, BAM Stats,etc.) (aka object store)

Deployment

The dcc-portal-ui, euler.api and openstack.keystone components are self contained in a docker container, quick start

Dataflows

(0) - the projects' meta data and files are loaded into dcc (aka 'release')
(1,2) - An authenticated user adds a file to the object store, and the euler plugin picks up meta data about the container, object(file) and user account and posts it to euler's api service for processing.
(3) - The euler api server consumes the event and composes a Resource message and publishes it to a kafka topic.   A kafka component picks up the message and uses existing calls to the dcc-server(which depends on es,mongo) to retrieve context data and creates a file centric document .
(4) - An authenticated user, using the either the portal or a command line tool, queries for resources or metadata.   The euler api component filters and redacts those calls.
(5) - The euler api component relies on keystone for identity and authorization decisions.

Dependencies

openstack versions

Swift:  SWIFT_VERSION=2.10.0
Openstack  CLIENT_VERSION=3.2.0

swift system account

Ability to install euler swift listener

clients have access to swift
swift has access to euler.api

Future Sprints
Deployment

We move from the 'proof-of-concept' instances of keystone and swift to the ones already installed in exastack.
The openstack.keystone components are removed from the docker container and injected into container via docker-compose-deploy.yml, and .env quick start
The euler.listener is configured and deployed into exastack-00.ohsu.edu swift instance

Functionality

Order
Description
Comment

6
Common ETL with BMEG
Integrate event handling. Leverage same infrastructure (OHSU Kafka)

7
Load additional OHSU projects
Other projects (from CGD?) should be loaded to complement BAML

Dependencies

openstack versions of swift and keystone and system accounts above available in exastack-00.ohsu.edu
deprecate local instances of swift and keystone  for environments other than development

Backlog

Functionality
The stakeholders and sponsors have identified inter-institutional collaboration as a goal.
These features have not been prioritized and approved, but are captured here to provide roadmap.

Order
Description
Comment

8
Federate dcc api
Implement federation layer above dcc-server

9
Non OHSU sites
Support for separately administered keystone instances

10
Cloud datasets
Capture meta data about cloud datasets and workflows

un-groomed features

keystone shibboleth integration
keystone federation
euler api federation
s3, google bucket webhooks

@mayfielg  @k1643 @kellrott   Please review this draft document  in the context of EUL-36

First architecture diagram: looks very good, except we're missing an explicit definition of the ports each piece is using to communicate with the rest of the setup. In EUL-36, we said we wanted to describe those definitively as well. It's helpful to document what ports need to be available externally on which floating IPs, since ITG needs to handle that for us. What are these diagrams drawn in? I'd be happy to make that addition, if I can edit the documents the diagrams came from.
First data-flow diagram: from everything I remember about the dcc-portal-server, it never queries mongo. It only queries elastic, and mongo is only ever used as a resource during the release pipeline. If I'm correct, then this diagram is a bit misleading. Am I missing some detail of the server that does use mongo directly? I'm also a bit unclear on what 'config' in the openstack, keystone box is referring to.
I'd life to clarify the 'future sprints' section a bit. The separation from one dotted box to two from the first diagram to the second refers to moving from using the proof-of-concept instances of keystone and swift to the ones already installed in exastack and ITG, correct?
This backlog appears to deal solely with federation (of authentication, cloud storage, dcc-server instances). It would be clearer to say that explicitly as the over-arching epic of the ungroomed backlog.
More specifically, Are keystone-with-shibboleth and keystone federation different options for addressing authentication of non-OHSU personnel? Or are they supposed to work in conjunction? Or is that unknown at this time and an issue to address when we get to that stage? Also, I believe "s3, google bucket webhooks," is referring to the issue of federation across multiple physical locations, and thus multiple cloud storage sites. At this time, is swift capable of federation across swift instances already, or no? It might be intelligent to include that as a story first, before we start to deal with inclusion of other softwares.

@mayfielg Thanks for comments.  Please check to ensure that I've addressed them.

it never queries mongo
removed mongo from dataflows

I'd like to clarify the 'future sprints' section a bit.
added clarification

This backlog appears to deal solely with federation
added features section

I think the only remaining question is the bit regarding what's proposed for shibboleth and keystone federation.

Are keystone-with-shibboleth and keystone federation different options for addressing authentication of non-OHSU personnel or are they supposed to work in conjunction? Or is that unknown at this time and an issue to address when we get to that stage?

Agreed.  It is an evolving architecture. In the next sprint planning we can discuss w/ Adam M + Kyle + Paul H.

+1


Psalm updates june 2020

have explored psalm's taint analysis a little.
not 100% confident I'm using it appropriately, so leaving PR as draft for review.

Is there still more left to do on this one?

Is there still more left to do on this one?

possibly? I wasn't not 100% confident I was using it appropriately, so I'm wondering whether it'd be better to have this reviewed by someone more familiar with taint analysis or just avoid the PR altogether rather than risk having the sinks & sources misdefined?

We'll revisit this in the near future. Right now, the code has diverged a bit from the development branch.

Done in #142


How to use 'All Configs'？

I'm sorry, I'm not sure how to utilize the My Config option in my project, which is supposed to enable me to select which rules to apply, rather than being limited to just the recommended ones.

Please reopen the issue when you have still a problem to configure this plugin.
Closing for now.


Vim cell commands

It'd be nice if Vim bindings could while in the notebook cell mode (not editing a cell). i.e., dd deletes a cellandyy` copies a cell.
I keep on running into this bug from Jupyter notebook muscle memory. It'd be nice if installing this package meant that those keybindings were enabled (and yes, I could go into settings and change them all but this would be easier).

Thanks for the suggestion, I like this idea.
How would you feel if they were a single letter? So I would make the following mappings:
D: cut cell
P: paste below
Shift P: paste above
Y: yank cell
My main reason for this is I grew tired of double letter commands with jupyter-vim-binding at times and like the quickness with single letter commands. I think I would be more inclined to make it 'D D' if there are other motions that we could do with D. For example if we had 'D J' delete the current cell and the cell below.

I'd prefer the bindings to be the same as vim's actions for a single line. I don't need to learn another set of keybindings then.
Could we present a choice to the user? I think maybe a switch would be appropriate here.

I think others also preferred Ctrl-O D D to just Ctrl-O D for that reason. Yea, a switch would be nice. I haven't looked into what user configs extensions can create yet. I'll take a look at some point.

I'm really missing cell-level commands too.
I agree with @stsievert it's better them to be consistent with vim row-level bindings (at least by default). We're already using this bindings working with cells content, why use anything else on cell level?
I would mention gg / G and visual mode for selecting and moving multiple cells at once.
Also I would be happy to have double esc and ctrl + { switching editor to cell-mode.

@imbolc Apologies for my delayed response and thanks for your feedback.

I'm working on something to add this feature in as a setting, perhaps defaulting to the double commands. In the meantime, you should be able to add them with the advanced settings editor.

Yes, I agree gg and G would be good to implement. I'll open a separate issue to track this.

Would your issues with selecting and moving multiple cells be solved with improving the cut and paste shortcuts? If not, I'd like to make a new issue for this.

Interesting idea. I'm not sure how easy the double escape would be. I think it may interfere with some of the existing handlers but I would need to investigate more. Is ctrl-{ a vim shortcut or are you referring to ctrl-[? Or are you saying you would like ctrl-shift-[ to go to cell-mode?

I personally would be completely satisfied with yy, dd, y add d with selected cells and p
Sorry, I meant just ctrl-[

I personally would be completely satisfied with yy, dd, y add d with selected cells and p

Same here. Thanks for the great work on this @jwkvam!

Just found this issue after wanting this enough to go and do https://github.com/jwkvam/jupyterlab_vim/pull/28 :)
Since this is explicitly a vim extension I think we should stay as close as possible to vim muscle memory shortcuts and make it easy for people to customize if they want other commands. At least for me it'd majorly mess with my sanity if dd deleted two cells.

Is it possible to "y3y" to yank three cells, and "p" to copy them.
(the same goes with "d3d" where 3 is just some arbitrary number)


feat: remove final sorries in positivity tactic

Per https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Last.20sorries.20in.20mathlib4

Fixes #1362.

bors r+


FAQs Page

For the time being, this page is being hosted on Google Docs to make it easy for community members to comment and contribute. For our MVP I think we can just link to Google Doc instead of coding it into the Gatsby page.
We'll bring it over, but let's keep agility on the more important features and copywriting.
It is located here:
https://docs.google.com/document/d/1U9Iyp2y-3eEW3OyV5O9bQrsj9qXl8S3Cxz-cPRruspM/edit

I have populated some questions for the FAQs
@cmgorton would you do me the favour and provide the copy for the answers.

Sure. I'll work on this tomorrow 😀
On Sun, 5 Jul 2020, 5:01 pm Andrew Brown<EMAIL_ADDRESS>wrote:

I have populated some questions for the FAQs
@cmgorton https://github.com/cmgorton would you do me the favour and
provide the copy for the answers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/100DaysOfCloud/100DaysOfCloudApp/issues/8#issuecomment-653949953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAQV6EUG6I6MMXB4H7QH4TR2EA47ANCNFSM4ORCQNIA
.


Fix duplicated elements in S3 put_object / delete_object response XML structure

I think most tools just look at the response headers for PutObject, but where the response body is consumed, I suspect that the top-level <PutObjectResponse> should not be duplicated.
Same issue for <DeleteObjectResponse> nearby, so I altered that too.

On further inspection, the botocore service definition seems to imply headers-only response for both these operations (no XML body at all) :thinking:

Coverage remained the same at 94.292% when pulling 16db824d8ae293ae99877b8c4ea5a73785c5e1a5 on dholroyd:master into 5ac9bb1885ebc4a7ac6573645e478d914c3e657a on spulec:master.

Hi @dholroyd, thanks for providing a PR for this.
You're right on the response - I just tested it, and S3 seems to respond with a headers-only request (no body at all). Will merge this PR for now, as it is an improvement.


[e2e]: generic workflow_dispatch branch1 default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/3247500975
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: branch1
Date: Fri Oct 14 04:47:28 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3252007825
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Fri Oct 14 18:24:57 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/3252574989
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: branch1
Date: Fri Oct 14 20:12:13 UTC 2022
Tests are passing now. Closing this issue.


Add a Gitter chat badge to README.md

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

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

 Updated on successful CI builds.


add evaluator node for distributed evaluation

Summary of the change

When running tensorflow in distributed mode, sets up the evaluator node.
During shutdown, evaluator node is also shutdown similar to ps

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

@bbshetty thanks for the contribution!   Is there an official example somewhere that uses an evaluator node?  (Most of the examples I've seen just evaluate in the 'master').

Thanks again!

Sorry for late response. I couldn't find the example. But I tried it myself to get it working. Thanks for the help.

No worries, I was able to use this example with: InputMode.TENSORFLOW, master_node='chief', eval_node=True.


Improve tests

What I think should be done with tests:

separate logging of correctness tests (that can go to console) and performance tests (that I think we should use separate logger - best to save logs to a file so later it is simple to analyze the contents of the file).
Improve test coverage - especially to check what is a good batch size for different types of single job size.

Is this issue blocking for registration?

There is little benefit from registering a package (apart from minimally easier installation).
I am not actively maintaining this repository right now, so if you want to take it over I can transfer it to you (and then you can go ahead registering it etc.). Of course I would review PRs if some pushed them and asked for help.
CC @pszufe - maybe also you would want to get involved here more?

I am OK with not having it registered, and I don't mind taking ownership of the package either if you don't want to maintain it anymore. You probably know more about multithreading in general than I do though, so I would appreciate your PR reviews every now and then.

It looks that in order to make a transfer you would have to delete your repository with the same name. Anyway - you have full write privileges to this repository and we can transfer it later. Thank you for your work.


Broke my code

Failed to execute script 'CuldR6Unlocker_V1.py' due to unhandled exception: No module named 'sqlite3'
Traceback (most recent call last):
File "CuldR6Unlocker_V1.py", line 2, in
File "", line 4, in
ModuleNotFoundError: No module named 'sqlite3'
Broke my code, any way to fix this?

pip install sqlite3

if he converted the code to exe without specifying the modules, this error is normal, i have already explained this. Also available version 2.0 is much better


Feature Request: Add support for custom API endpoints

Besides the official OpenAI API, there are also other providers like Azure OpenAI. In some regions where the official API is not accessible, users heavily rely on these third-party APIs. Adding support for custom OpenAI keys and URLs would play a crucial role in such scenarios.
There are only a handful of projects that currently offer this feature, one such example is BetterChatGPT.
I apologize for any confusion caused in the previous issue. I mistakenly referred to Azure's Speech API as its OpenAI API.

Would be great. That or support for one of the OpenAI proxy services such as https://github.com/diemus/azure-openai-proxy if you didn't want to rewrite to support Azure.

Interesting. I would definitely be open to PRs implementing this! <3 Which regions are affected?

We might need to build an adapter to house all the different implementations of these APIs. Not sure how different they would be as far as input to output but worth exploring.

There are only a handful of projects that currently offer this feature, one such example is BetterChatGPT.

As far as I can tell, there is no special handling of alternative ChatGPT APIs required. The BetterChatGPT implementation allows one to pick an arbitrary URL. If it's just about replacing the URL that can be done by extracting them into environment variables without the need for a UI. Do you actually need to change endpoints frequently during usage?

That would be great. More and more projects that support to self-host LLMs now ship with OpenAPI-compatible API.
Examples:

https://github.com/oobabooga/text-generation-webui/tree/main/extensions/openai
https://github.com/lm-sys/FastChat#openai-compatible-restful-apis--sdk

Many APIs act as drop in replacements for OpenAI's api with the same schema and a different URL.


Containers exit immediately after docker-compose up

Hi,
Thanks for the project so far. I am excited to get it up and running. However, when I run docker-compose up, it pulls the containers but they(except for postgis) immediately exit with the following exit codes on each container:
05d80291725d        rsilva4/openmaint         "/docker-entrypoint.s"   28 minutes ago      Exited (137) 3 minutes ago                              dockeropenmaint_openmaint_1 3931f07db595        rsilva4/bimserver         "/docker-entrypoint.s"   28 minutes ago      Exited (56) 3 minutes ago                               dockeropenmaint_bimserver_1 cfde15677988        kpettijohn/postgis:9.4    "/docker-entrypoint.s"   28 minutes ago      Up 3 minutes                   <IP_ADDRESS>:5555->5432/tcp   dockeropenmaint_postgis_1 4d452a6abe6d        kartoza/geoserver:2.6.1   "catalina.sh run"        28 minutes ago      Exited (137) 3 minutes ago                              dockeropenmaint_geoserver_1
I am running the latest version of docker and docker-compose on virtualbox instance of ubuntu 14.04
Any ideas why this is happening?
Thanks!
John

Hello John,
Can you please share the logs for each container? You can obtain them using docker logs <container_name>. Example: docker logs dockeropenmaint_geoserver_1
Regards
Ruben

Piped output for Docker logs for bimserver container can be found here:
https://gist.github.com/bigdoods/e518ceb9cb9439cc1868aca96edec3e8
For openmaint here:
https://gist.github.com/bigdoods/2e139c977db89154952f56b4e1965666
And the log for geoserver is:
https://gist.github.com/bigdoods/15078330d84a626a1535889f17d472c2.
Thanks!

Hi Ruben,
I had a similar problem with other docker contaniers that I was using an there was some correlation between the problem- I was using an older version of ubuntu 14.04 (with kernel version 3.1) and I have found out here that it could be the cause of bugs with docker.
I have just pulled your images again and it works fine with a newer version of ubuntu and docker.
Thanks and sorry for the false alarm.
Regards,
John

Great! Let me know if you need something else.


flutter.googlesource.com gerrit instance should be configured to land reverts

To be in alignment with Flutter team policy, the revert here https://flutter-review.googlesource.com/c/recipes/+/56784 should have landed without needing any additional approvals.
cc @godofredoc

Leave this open to validate the logic works.

@zanderso I have updated internal recipes doc. Is there any framework/engine gardener book that need an update?

The issue was not the lack of owners approval. I think I'm an owner already. The issue was that reverts should not need 2-human approval. The bot should have approved the CL, or something like that.

Seems we are already overriding when the cl is a pure revert. Followed up in b/253055983


Feature: Invite users

I'd like to be able to invite users by email.
Adjust the email that gets sent out inviting them.
And view the status of the invitation

When #24 is implemented, this email invitation could override the disable-new-registrations setting, allowing admins to selectively invite users.


BUDGET: Create class method for determining money pledged in a given week

Our goal here: Take all patients whose appointment date is in a given week and calculate the DCAF pledge amount. Here's the rub: I don't know what the best way to do it is.
Here are the requirements: We need to be able to filter to pledges in a given calendar week, and distinguish those between pledges that have been sent and those that haven't.
Here's a suggested path to this, but I'm amenable to another option:

Create a class method on the Patient object that filters down to anyone with an appointment date in a given week and maps that collection of objects down to a hash like { pledged: 1000, sent: 500 }.

We'll in turn use this to build a helper method that takes a hash as an input and spits out a nice looking budget bar.

I'll take a stab at this :grinning:
My only question is if it would be one function or two:
Are there other use cases that would want all the patients with appointments in the next week?

if so, then we could teach the Patient object a more generic function like patients_with_upcoming_appointment taking a duration parameter, and then maybe move the data extraction we're looking for into a helper.

Alternately if that is over-engineering it, we could always make that split later if we ran into another use case later.

Love the idea of abstracting it like that -- if this is something that
lends itself well to that, I'd say go for it!
On Thu, Nov 3, 2016 at 7:38 PM Kathryn Tipton<EMAIL_ADDRESS>wrote:
I'll take a stab at this 😀
My only question is if it would be one function or two:
Are there other use cases that would want all the patients with
appointments in the next week?

if so, then we could teach the Patient object a more generic function
like patients_with_upcoming_appointment taking a duration parameter, and
then maybe move the data extraction we're looking for into a helper.

Alternately if that is over-engineering it, we could always make that split
later if we ran into another use case later.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/colinxfleming/dcaf_case_management/issues/709#issuecomment-258306246,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADsA9IiyMjo4d-zRORcI6UFf6EzePSxkks5q6nB9gaJpZM4Kn6Fa
.

I'm not going to get this to a PR state tonight. I'll try to get it there this weekend!

@colinxfleming I think I have found where this is sticking in my head. I'm used to a slightly different MVC pattern.
I work mostly in Mojolicious, which has distinct model controllers for dealing with single objects vs collections of objects. E.g. CRUD code dealing with a single Patient would go in ./Patient.pm and CRUD code dealing with multiple Patients would go in ./Patient/Manager.pm.
Where does ruby handle manipulating multiple objects of a type? Does this go in the model alongside single-object manipulation? Into the controller? Or since this particular function is enabling a shared functionality, does it go into a helper?
At this point I actually have the logic psuedo-coded, I just don't know where to put it. When I tried to slot it into the model, I realized I couldn't figure out how to write a test for it there.
Psuedocode:
outstanding_pledges = 0
sent_total = 0
Patient.where(appointment_date.lte => Date.today + 7).each do |patient|
  if patient.pregnancy.pledge_sent
    sent_total += patient.pregnancy.dcaf_soft_pledge
  else
    outstanding_pledges =  patient.pregnancy.dcaf_soft_pledge
  end
end
return { pledged: outstanding_pledges, sent: sent_total

@lomky excellent questions, really appreciate that you're engaging with the rails 'style' here!
the short version is that code that deals with that object or objects of that type goes in the same class definition. You can check out some similar code under Patient#urgent_patients -- it's a similar hit of everyone in the db that just filters down to people who have the urgent flag set to true. If this were more of an API-oriented app instead we'd probably do this on the controller level somehow. Helpers are mostly for view-related code.
Here I think is a helpful-ish blog post that's an overview of similar patterns: http://culttt.com/2015/06/10/understanding-class-methods-verses-instance-methods-in-ruby/
. The short version is that methods defined with def self.method are available when you do Patient.method, but methods defined with a regular old def method are available only to individual objects (e.g. you have to retrieve an individual patient from the database to do that, such as in Patient.first.method).
re: your pseudocode, that pretty much looks right to me! Pretty much all you'll need to do is chuck that in def self.whatever -- the Patient on line three will also be inferred, so you should just be able to start it where.
Let me know if that unsticks you -- this looks solid!

@colinxfleming Brilliant! Instance vs Class completely unstuck my head. I have been too long away from a proper OO language 🐫

💯 💯 💯 💯 💯

I cannot seem to get the tests to pass. :( submitting a PR so others can review the code and hopefully help spot my bug!

Deeply appreciate that you took the time to go the extra mile and write
tests! I'll have some feedback for you by the end of the day if not sooner.
On Sat, Nov 5, 2016, 11:18 AM Kathryn Tipton<EMAIL_ADDRESS>wrote:

I cannot seem to get the tests to pass. :( submitting a PR so others can
review the code and hopefully help spot my bug!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/colinxfleming/dcaf_case_management/issues/709#issuecomment-258618197,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADsA9Cfr1fEP2hksce0RA6resMeLeTBzks5q7J5dgaJpZM4Kn6Fa
.

Fixed by @lomky in #717 !


Add artifact schema and tests

Signed-off-by: Sajay Antony<EMAIL_ADDRESS>

@sajayantony #944 is merged. This should be ready to rebase and clean up the ioutil references. Thanks!

I need to resubmit this with the field name changes now :)

Need to validate that that the tests catch example issues like
https://github.com/opencontainers/image-spec/pull/960

@shizhMSFT thanks for fixing https://github.com/opencontainers/image-spec/pull/960 - artifact examples were not validated in this PR and your PR helped :)

Bumping this issue to get feedback from other maintainers @opencontainers/image-spec-maintainers

Closing this one because of #999.


README.md file not needed

Describe the bug
Another README.md file in frontend folder
To Reproduce
Steps to reproduce the behavior:

Go to root directory
Go to frontend folder
Scroll down to see README.md

Screenshots

nvm


⚠ Bookkeeper to the Smart People has degraded performance

In ad29101, Bookkeeper to the Smart People (https://bksp.com.au) experienced degraded performance:

HTTP code: 200
Response time: 7577 ms

Resolved: Bookkeeper to the Smart People performance has improved in 732b154.


Mercury decoders for union inside input/output mercury data

Is your feature request related to a problem? Please describe.
IS it possible to have mercury input and output decoders for unions inside struct (i.e conditionally decode based on type)
struct A {
uint32_t    union_type;
union {
struct B {
int a, b, c;
}
 struct C {
    float temp1, temp2;
 }

}u_entry;
}
Describe the solution you'd like
static HG_INLINE hg_return_t
hg_proc_A(hg_proc_t proc, void *data) {
hg_return_t ret = HG_SUCCESS;
struct A *struct_data = (   struct A *)data;
ret = hg_proc_uint32_t(proc, &struct_data->union_type);
if (ret != HG_SUCCESS)
return ret;
/* Based on union type conditionally decode*/
if( struct_data->union_type == struct B) {
/** decode B's struct*/
}
else  {
/** decode C strcut */
}
}
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Its a question so will create a issue with blank