Next data request is blocked by long downloads

I've inspected the logs on MARS and our dataflow jobs and noticed a discrepancy. In our MARS activity log, data takes a certain amount of time to serve a request – say, 2 hours. In the Dataflow logs, the same request takes longer than two hours – maybe, an additional hour longer.
After some analysis, I've found the cause for the difference: The ECMWF Web API, which processes the underlying request to the MARS service, takes a long time to actually download the data – especially if the payload is large. When I've inspected EC's code, I've noticed that it transfers 1 MB at a time of data from EC's servers to the Dataflow job. Since we regularly have payloads in the 10s of GiBs, it makes sense that this takes a long time!
This feature request involves enhancing the fetch stage of the downloader such that subsequent requests don't have to wait on current downloads. As soon as data has been fetched from the MARS archive and is available for download, we should initiate a new data request.
Implementation Notes
A clean way to implement this feature would be to restructure the fetch stage to be a Composite Beam transform, similar to
https://github.com/google/weather-tools/blob/5c12bf694467ba59bb9180d0040f9a07b1a839b6/weather_dl/download_pipeline/partition.py#L32
In parallel, we should extend the interface for a Clients to separate the request stage and the download stage of fetching.
https://github.com/google/weather-tools/blob/5c12bf694467ba59bb9180d0040f9a07b1a839b6/weather_dl/download_pipeline/clients.py#L29
The new fetch stage should involve multiple PTransforms that parallelize the work of fetching a request, waiting for it to become available, and then processing the download when ready.
To better understand what specific steps the clients / fetch should have, please review the flow of the ECMWF WebAPI for MARS, specifically:
https://github.com/ecmwf/ecmwf-api-client/blob/master/ecmwfapi/api.py#L518
Notice that it waits in a loop, checking if the data is ready, before actually performing the download.
https://github.com/ecmwf/ecmwf-api-client/blob/dd20383585359cc80249826977ff102c3a81deee/ecmwfapi/api.py#L532
https://github.com/ecmwf/ecmwf-api-client/blob/dd20383585359cc80249826977ff102c3a81deee/ecmwfapi/api.py#L548
With an extended client interface, we can provide affordances to call these steps in the client APIs ourselves, but in a way that better works with Beam + the global queue.
To complete this bug, all clients (so, CDS and MARS) should have the same interface and optimization, if possible.
Advance implementation ideas
From my read of the Beam docs, this new fetcher might be a good candidate for a Splittable DoFn. A possible advantage of using a SDF is that the CPU utilization of the download might be improved, which could improve allocation. Instead of sleeping before checking if data is available, with a SDF, we may be able to use a timer and schedule the next call to the MARS server. While this implementation approach is not necessary, it may save on both compute and energy costs for the downloader. See the "User Initiated checkpoint" section, for example.

As noted in #140, an acceptance criteria for this issue is that each stage of fetching should include retry logic. It should aim to prevent this bug from occurring again.


Increase saucelab connection timeout

Coverage remained the same at 100.0% when pulling 7bc55dd2774b4443a671a15a1ef474542e84c36a on saucelabs_timeouts into 67a95b3cc433d5f0155299613a152a51b35f6ed9 on master.

Did you try to bump karma* versions?
memo: floydpink/lzwCompress.js CI (looks like working correctly in safari)


optimum

URL to APK: https://apkpure.com/optimum/com.optimum.unity.mobile
Banner(If Submitted):


Added data analysis code for Bond Lengths

Added code to parse and extract molecular data and store them as pickle objects.
Also did data analysis on bond length data.
In the next iteration will add ML models to predict bond length.

I don't think the pickle archives are here. I know you were storing them on Dropbox - maybe add a script to download them?

I'd also suggest some effort to tune the -0.25 exponent (through botorch? 😄) by minimizing MAE or RMSE.

Uploaded the pickle file to Github as well as added an external link.
All the work until building baseline ML models has been uploaded.


Tweak to return null instead of throw exception

Currently "Nested" columns that are created using flatten_nested=0 cause an exception - at least for now I've set it to warn and then ignore that table, though we could explore if it's possible to handle that condition better.

@Paultagoras Do you think we can write a tests for it?


added basic validation of recipient nin and orgno

Description

Validation of digits and length when organization number or national identity number is provided for recipient.
Related Issue(s)

#536

Verification

[x] Your code builds clean without any errors or warnings
[x] Manual testing done (required)
[x] Relevant automated test added (if you find this hard, leave it and we'll help out)
[x] All tests run green

Keeping the duplicated lines as is. Moving common logic to new class only saves us a couple of the duplicated lines.


Fix URL in Psequel.app Cask

URL was unversioned and failed to download.
Also reverted version to 1.1.2 as today's 1.2.0 release is crashing for users, as reported on twitter (@psequel).

Thank you for the contribution. It was merged directly as 4667bcac995f58a623d05b8c00b9a18d468bf8ef to keep commit history cleaner. Your contribution is still credited to you.


room without message

When the room has no message, it keeps loading a spinner animation. How do I disable this, and how do I change the default language?

Hey @mateus3690,
loadingMessages controls the room message loader.
Also you can override the UI language by setting a custom textMessages object. You can find the schema for textMessages here https://github.com/advanced-chat/vue-advanced-chat/blob/master/src/locales/index.js


Problem with imported type operators

If I use Monad.importFrom with Monad.importTypeOp, I'm getting that the return value is still rendering with parens.
The following code produces the following error:

The current solution is just to ignore the return value of the import statement completely and just use ":&" directly wherever it would have been used

Normal operator imports are broken as well 😱


Is FormRenderer exported in the npm package?

If I follow the Docs I can't see how to get access to FormeoRenderer - it is not exported by 'formeo'?

You can import the renderer with import { FormeoRenderer } from 'formeo'


Accounts number / list on mainnet explorer is missing most of accounts

See https://explorer.near.org/accounts
There are far more accounts on mainnet right now than listed there. I guess accounts from linkdrop aren't getting indexed?

Yeah, it all stems from the lack of receipts indexing (#218), which causes issues like this and #223 and #358.
Thank you for bringing it up. Indexer for Explorer will resolve a bunch of issues like this.

Resolved in Explorer baked by the Indexer for Explorer (see #236).


Offcanvas menus

These mixins allow you to create an element with offcanvas menus and a content area.  Each offcanvas container can have 1-4 menus and 1 content area.  All parameters have defaults, but are configurable.
Mixins:
offcanvas()  -  creates an area with 1 menu and 1 content area (uses offcanvas_multi behind the scenes)
offcanvas_multi()  -  container with 1-4 menus and 1 content area
Additive Mixin:
offcanvas_menus()  -  creates classes for creating offcanvas menus
There is currently no default styling.  I also added script tags to the test pages to toggle the open classes.

Wow. I love this.
One minor note, I think "thick" is a bit of an awkward parameter name. Maybe "size" is more intuitive?
Will probably need a note in the doc about the JS required for the toggle to work.

thanks @abstracthat.  I agree about "thick".  It was "thickness", but I thought that was too long.  "size" is much more intuitive.  I'll change that and add a note to the docs about toggling the open classes with JS.

Very cool @rhythnic -- this is pushing pretty far into "you need additional css and js that's not included" land which I want to be careful of, but as long as there are clear instructions and an example I can get down with it.
Could you possibly include a screenshot or gif of it working in the PR just so I can quickly see?

@jenius I'll put up a link to a demo today or tomorrow.  So far as I can see, no other part of axis requires toggling a class, so it does make things more complicated.  I do think it adds value, especially in responsive design, where off-canvas side menus have become heavily relied upon.  But I understand that just because something can be done doesn't mean it's right for the project.  Anyway, I'll put up a demo link soon.

@rhythnic definitely, a well-balanced response. We'll figure it out. Either way, really appreciate the good work and contribution and looking forward to seeing the demo!

Here's a demo using an offcanvas menu on each side.  These are only at the master-container level, but there's no reason an offcanvas menu couldn't be nested on the page.  Menus will overlap, so z-index needs to be set according to priority.
http://offcanvas-demo.meteor.com/

What's going on with the left menu?

The left menu is set to show: 80em, so once the browser window is wider than 80em it is always visible and doesn't respond to the open class on the container.

Ok so I've been thinking about this a little bit. While this is something I'd like to use personally, I'm just not sure it fits as a part of axis, which is supposed to be utilities and light ui structure. This is kind of a major UI piece, and is rather complex, requires custom html and js to get it working as well.
So at the moment I'm leaning towards saying that I don't think we should merge this to axis. However, I do think that you should make it into its own stylus library. This way, people can grab it only when they have a need for offcanvas menus, and on top of that you will get all the props you deserve for all the hard work you have done here since the repo will be under your name. Also, I will definitely use it, and would be happy to contribute and help with maintenance if you want as well. If you want to work together getting it into a stylus plugin container and getting it published would be happy to help out with that as well.
What are your thoughts on this @rhythnic?

If it’s not going to get merged I hope to see it published too. I’ll follow and star it for sure. What’s another npm install and @import anyway?

On Feb 25, 2015, at 3:10 PM, Jeff Escalante<EMAIL_ADDRESS>wrote:
Ok so I've been thinking about this a little bit. While this is something I'd like to use personally, I'm just not sure it fits as a part of axis, which is supposed to be utilities and light ui structure. This is kind of a major UI piece, and is rather complex, requires custom html and js to get it working as well.
So at the moment I'm leaning towards saying that I don't think we should merge this to axis. However, I do think that you should make it into its own stylus library. This way, people can grab it only when they have a need for offcanvas menus, and on top of that you will get all the props you deserve for all the hard work you have done here since the repo will be under your name. Also, I will definitely use it, and would be happy to contribute and help with maintenance if you want as well. If you want to work together getting it into a stylus plugin container and getting it published would be happy to help out with that as well.
What are your thoughts on this @rhythnic https://github.com/rhythnic?
—
Reply to this email directly or view it on GitHub https://github.com/jenius/axis/pull/165#issuecomment-76045491.

In comparing stylus plugins to bootstrap, there are several stylus plugins that fill the gap of what bootstrap does, axis being the style oriented one, jeet for grids, rupture for media queries, etc.  By the way @jenius just noticed that you also created rupture.  Great work.  In keeping with that trend, the offcanvas menu doesn't belong in the styling plugin.  I'll most likely make a plugin not specifically for offcanvas, but for layout mixins in general, like tabs, page scrollers, etc.  @jenius @abstracthat thanks for your support.  I'll let you know when I've got something put together, might be just the offcanvas initially.  @jenius thanks for your offer to help.  i'll read up and use axis as an example.  i'll let you know if i have any questions.

@rhythnic that sounds like a great idea. In fact, if that's going well, I wouldn't be opposed to moving some of axis' "ui" section pieces over to it. This would be a challenging project especially with the docs and integration with html and css.
I've been thinking about how you could pair html modules with stylus mixins as well. I feel like every templating language accepts functions as locals that you can execute to inject html. Perhaps offering each mixin paired with a function to be passed to your templating language as a local would be a good way to generate less confusion with the html and css pairing? This would work with any of the templating languages I have worked with personally (I maintain a library of abstractions over them here). You could also have it expose an object of js functions that could be used to control it. Having compact clean packages like this would be a really cool way to do things, like an alternative to bootstrap but that's actually semantic, clean, and customizable.
Definitely keep us up to date. I'm very interested in this concept, and would be happy to contribute and discuss.

That would be a clean way of doing it for sure.  Before I found my way around to stylus mixins, I had it implemented as a template in meteor, so using the offcanvas looked like:
{{#offcanvas}}
{{#offcanvas_menu}} Menu {{/offcanvas_menu}}
{{#offcanvas_content}} Content {{/offcanvas_content}}
{{/offcanvas}}

That neatly wraps the html and js together, but it's specific to Spacebars.  But in the offcanvas example, all functionality is based on adding and removing classes, and you can use any html elements, so I would want to leave open the possibility of using the most appropriate elements.  Also in client-side frameworks that use data-binding, I don't think they would use the provided JS functions.  For now, I'll start things off with just mixins and focus a lot on good documentation.  As I get around to making more layout mixins, if things get more complicated, it might be the best approach to make JS/HTML libraries to accompany the mixins.

I love the idea of a clean set of mixins and I think that is the simplest place to start. Is that Meteor template published @rhythnic?

@abstracthat I was originally trying to package the template, before I had created the mixin, but I concluded that the user can't customize the offcanvas menu if it's in a package, because the package can't access stylus variables outside of the package.  exposing a stylus mixin from a meteor package is the best way to do it, but currently there's no published meteor package that allows you to expose custom mixins (that I know of).  since you can't have multiple stylus packages (that render css), one needs to be created that establishes a protocol for specifying which published packages should be used and how to include custom mixins from other packages.

Just a quick note, I know you are probably using meteor and that's where you care whether it works or not, but when making a general stylus library make sure you are targeting normal non-meteor users since this is the dominant use case!

Yes, @jenius I agree.  For now I'm just going to do mixins.  If it gets more complicated I'll keep it generic.

:+1:

I put something together.  @jenius I copied a lot from how you setup axis.  I'll put credit in the readme going forward, or if you'd like I'll go at it from a different angle, but I wanted to get your and @abstracthat 's opinion on anything you think could be better.  It's just the offcanvas and a toggle switch to start things off.
https://github.com/rhythnic/shift
http://rhythnic.github.io/shift/

@jenius thanks for your feedback.  i changed the name to maneuver, since shift was taken on npm.
https://github.com/rhythnic/maneuver
http://rhythnic.github.io/maneuver-docs/

:+1:

Cool. I'll have a closer look Monday. I have a toggle and checkbox mixin as well.

@abstracthat what repository is that in?  i think i'll eventually rework the one i did.  i'd like to see how you chose to implement it.

:+1:

Hey there, going to close this as it's not being pulled into axis. But big props to @rhythnic for the sweet library he made out of this, and looking forward to using it in some of my projects!


Voting threshold: old inTx might be double processed if validator set change

Closing as it looks no longer relevant.


How can I interact with the objects in the simulation environment?

Hi,
When running the provided demo mouse_interaction.py, I can only move in the simulation environment and can't interact with objects. How can I interact with the objects in the simulation environment?

Hi @chenguanqi, you can press m to change the mode from navigation to interaction and then be able to interact with objects in the scene. A full guide can be found here: http://svl.stanford.edu/igibson/docs/viewer.html

Thank you! My problem is solved.


HTTP Alternative Services

Alternate Services push alternate means to access resources of an origin, particularly on a different host/port or even protocol. This is works via headers in http 1.1 or an extension frame in HTTP 2.
https://datatracker.ietf.org/doc/draft-ietf-httpbis-alt-svc/

https://www.mnot.net/blog/2016/03/09/alt-svc

Won't fix.  last asked for in 2016.

plus no end user asked (i only did from a spec completion pov and thought it was neat)


Alternate option to create the edges in the edge renderer

While trying to improve the rendering of https://forum.babylonjs.com/t/drawing-outside-edges-of-a-mesh-created-from-csg/11926/9, I ended up writing a new function to create the edges of a mesh that happens to be a bit faster than the existing one.
Here are some testings:

Asset
Existing func
New func
Speed up

dude.babylon
0.2s
0.06s
x3

SSAOcat.babylon
0.46s
0.14s
x3

StanfordBunny.obj
0.2s
0.015s
x13

ufo.glb
1.4s
0.1s
x14

skull.babylon
120s
0.25s
x480

The display is exactly the same with the new and the existing algorithm in all those tests.
Using the new algorithm is an option, however, as there may be some cases which would not lead to the same rendering...
I have also added some options (that don't exist with the legacy algorithm) that allow to improve the creation in some cases, notably when using CSG to create meshes, which can produces cases like this:

There are two triangles, ABC and CDE.
AC is not shared by another triangle, so the renderer considers it to be an edge and draws it.
For this to be rendered correctly by the edges renderer, the triangles EBC and ABE should be created and ABC removed.
The applyTessellation option allows to detect such cases and create additional triangles to deal with that. Activating this option is quite CPU intensive, so it is off by default (also, in normal circumstances, you don't need this option).
Using this option will make this PG to render correctly: https://playground.babylonjs.com/#0HA29N
With the existing algorithm:

With the new algorithm + applyTessellation = true + useFastVertexMerger = false (see the doc for this one):

Should we even keep the old one ? what are the downsides ?

I don't know...
As the algorithm is not the same, it may not produce the exact same result in all cases, that's why I didn't want to remove it completely.

LGTM. let's keep the old one but use the new one by default.
If there is no issue in the next couple of months we removed it

Hello,
I'm the poster of the thread mentioned above.
I'm impressed how fast the process from question asking / "bug" reporting to a pull request is.
Is it possible to give a timeline when this features gets released outside of the alpha version?
Big thank you

probably in less than 2hours :D


Add appropriate action for shell execute failures

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Provide appropriate message when application fails with shell execute failure during dynamic scan. This is typically as a result of attempting to install the wrong architecture of a package. This pull request allows the fabric bot to provide feedback to the user to test the application installation on architectures supplied in the manifest.

@Trenly does this look good to you? I think the second rule needs a "Template: a/b/c".

@Trenly does this look good to you? I think the second rule needs a "Template: a/b/c".

Suggested a template


can not read  real  float value  with precision format mode

write a float value in precision format mode

read value is 1.32
not 1.32342

@xuri

Thanks for your issue, this duplicate with #991, the value of the cell should be the same as in the Excel formula bar, and as the documents say: If it is possible to apply a format to the cell value, it will do so, if not then an error will be returned, along with the raw value of the cell.

It works on version 25, thanks


docs: v4.1.0 release notes

After this merges, I will manually backport by copying this file (without the 4.2.0 section) into a PR against 4.1 branch.

Markdown link test is expected to fail (release not yet tagged)


Simpler argument handling for debug flag

supports direct assignment with --debug=False or --debug=True, but will also support --debug to set True or --no-debug to set False. Default value if not set is false.
Issue #, if available: N/A
Description of changes: removes default false value and adds builtin argparse.BooleanOptionalAction to the debug flag action.
From https://docs.python.org/dev/library/argparse.html#action :

"New in version 3.8.
You may also specify an arbitrary action by passing an Action subclass or other object that implements the same interface. The BooleanOptionalAction is now available in argparse and adds support for boolean actions such as --foo and --no-foo"

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Idea: add correctness tests

Between the query that the users send to Thanos and what is actually used in the code some translation happens. For example, some translation happens here https://github.com/thanos-io/thanos/blob/55cb8ca38b3539381dc6a781e637df15c694e50a/pkg/store/matchers.go and https://github.com/thanos-io/thanos/blob/0bb67bcbc3a75fb274c1649dcfdb13a8c2db4024/pkg/query/querier.go#L151-L171. There might be bugs in that code. It would be nice to have a good way to test it out with a live Prometheus to see if everything really works as expected.
This came up from https://github.com/thanos-io/thanos/issues/2245. We could protect ourselves against this by running some kind of correctness tests.
Some ideas below.
We could add another e2e scenario where a block would be generated with known data that then would be added to a Prometheus instance that has Sidecar in front of it + uploaded to some remote storage that is picked up by Thanos Store. Then, we could run some (randomly generated) queries and see if we get the same results. In all of the cases, they should be the same if everything is correct.
I haven't looked into the Prometheus side of this too much but I have found that there is some kind of PromQL test suite: https://github.com/prometheus/prometheus/tree/master/promql/testdata. Perhaps we could adapt it and reuse the same?
Cortex has this A/B testing tool https://github.com/cortexproject/cortex/pull/2203. It could also be extended to include correctness tests.
Perhaps there is some kind of PromQL fuzzer that I am not aware of? :P Downsampled data probably shouldn't be a part of this.
Thoughts? :)
/cc @yeya24

Is it doable in our e2e? We can start one Promethues, one sidecar and a querier. And simply compare the query results of Prometheus and querier.
https://github.com/prometheus/prometheus/tree/master/promql/testdata is great. But is it easy to add the metrics we need into the e2e Prometheus? The testdata is coupled with https://github.com/prometheus/prometheus/blob/84b4d079c8714be8e8ad071a35b0391df270364c/promql/test.go#L65:1 test framework.
Maybe we can extend the thanosbench. Put all the functionality there.

I think this type of thing is typically referred to as model based testing. I'm 100% for this! Thanos receive definitely could benefit from testing like this a lot as well!

I think that in particular correctness of the interaction between compactor and stores would GREATLY benefit from this. There is a lot of subtlety in the interaction between all of the different metas, deletion markers etc

Still valid.

No updates from me :/ still valid IMHO

No new updates from this front.

Actually, https://github.com/thanos-io/thanos/pull/3631 is a start for this :P It's a bit different but still a step towards having automatic tests for PromQL compliance.

Is this still valid? There is promql compliance test

Don't think so, closing!


[Creator] "type" is missing for parameters

When creating the templates "type" is not included:
"parameters": {
    "ApimServiceName": {
      "value": "product-api"
    }
  },

"type", however, is required according to the spec: https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/template-syntax#parameters

There is not enough information to diagnose this problem.  We recently refactored the extractor and are doing the same with the creator, so this may get fixed anyhow.  If this is an issue with the extractor right now, please open an issue and be more specific about the issue.
Thank you.


Fix data fetching by category and search, and success messages

Description

Fixed issues with data fetching for housing listings based on selected categories and search queries.
Corrected the display of success messages related to profile creation and housing listing creation.

Type of Change

[x] Bugfix

Checklist

[x] My code follows the project’s coding style
[x] I have performed a self-review of my code
[x] I have added tests that prove my fix is effective or that my feature works
[x] My changes generate no new warnings or errors
[x] I have checked my code and corrected any misspellings

Related Issues
Closes issue #107
Additional Information
This update ensures that users can now search for listings by category and keywords effectively and also see the proper success messages when profiles or housing listings are created.

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


This line has some issues. It is not working as expected.

Instead of using the System.getProperty function, we use the environment variable.


Sm/record get where boolean w 6236756

What does this PR do?
turns strings that appear to be booleans into booleans
tests with a NUT
What issues does this PR fix or reference?
@W-6236756@

has this always been a bug? I saw the chatter thread was 2 years old... just surprised no one else has noticed this

has this always been a bug? I saw the chatter thread was 2 years old... just surprised no one else has noticed this

yes.  I just don't think there's that many people using the command.


Support for 1.14

Minecraft CraftBukkit 1.14 doesn't have Apache Commons IO. If you include it in the pom.xml file, it will support 1.13 AND 1.14.

Minecraft CraftBukkit 1.14 doesn't have Apache Commons IO. If you include it in the pom.xml file, it will support 1.13 AND 1.14.

Hey, I cannot compil a knew plugin version. I dunno why. Do you think it's possible to send me the project with this correction withe the extension  ".jar" ?
Thank you a lot.


fixes parse of -1e-308

error was in finish() in exp3 state

The first line of a commit message should be short, certainly not with a number having 500 zeroes...
There's no need to quote the number in the commit message, since it is already visible in the test.

This should be squashed down into just one commit

I think we should merge the mant and mantn states as you proposed today


How to change the color of QR code from black

Can you help me to change QR code color to change that, I tried to give the custom class as well but it was not working.

@erBhushanPawar, for now, is not possible.


feat: add resource configuration for prometheus and grafana

Please note:
grafana_additional_dashboards -> grafana_params.additional_dashboards

Amazing, thanks! 🙏


fix docs bugs

解决文档修复第四题：
https://www.paddlepaddle.org.cn/documentation/docs/zh/practices/nlp/n_gram_model.html
文件位置：
docs\practices\nlp\n_gram_model.ipynb
错误类型：内容错误
错误修复：“由于词表的的长尾”->“由于词表中的长尾效应”

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


Netcdf dataloading

Alright, this is a big one - switching all the data loading code over to NetCDF gets rid of the two-step pre-processing pipeline, speeds up dataloading and will allow us to do some cool things in the future :)

Still need to find a good way to re-implement inference. Should we base that directly on the TIFFs or still include the detour via NetCDF (TIFF->NetCDF->Inference)?

Still need to find a good way to re-implement inference. Should we base that directly on the TIFFs or still include the detour via NetCDF (TIFF->NetCDF->Inference)?

Maybe both ways would be helpful, is that possible? That leaves a few more options and would be better backward compatible.
Does it lead to siginifcant speedup? I guess for upscaling speed becomes a more important topic.


Modify development file name to add postfix

It really bothers me that the link to DEVELOPMENT leads to a 'not found' page, so I tried to fix it and am looking forward to making further contributions.

@huacnlee
Alright. Just changed the link in README.md.


Signed-off-by: Theo Mathieu<EMAIL_ADDRESS>Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

/lgtm


Prepare release

Also standard prep for the release. We are just waiting on the response from @LinguList to see whether we close issue #13 or not for the lexibank release.

I adopted the requested changes in CONTRIBUTORS.md, @FredericBlum and @chrzyki.


fm hangs when navigating to folder with vast amount of files

When I open my downloads folder (which contains half the internet of files of all sorts of types) fm simply hangs and won't let me do anything. Even Ctrl+C hangs.
The folder cointains approx 857 files, many of which are images, pdfs and zips.

It looks like it's due to the file size calculation. If I just let it stand for a while fm will eventually become responsive again. Is there a way to ....

disable the size column, either through a shortcut or through configuration
run the size calculation in a separate thread so it won't block the rest of the UI
?

Oh snap, changing the dir and going back results in a full re-calculation of the sizes.
Would it be possible to cache the values?
I'm not quite certain why this takes that much time however. A simple ls -la in that directory completes in 0.03s using exa and also displays the file sizes.

I had noticed this as well actually with a node_modules directory. Will look at adding a config to disable the size calculation. It runs in separate go routines but for some reason hangs when there is a lot of directories and files. The size calculation is recursively going through directories and calculating the size of all the files in there which ls does not do. That being said I plan to look into why this happens

@mrusme just released 0.13.4 which adds a config value for calculated_file_sizes which by default will false. Going to look into why the UI freezes up on large directories with calculated file sizes on

Just upgraded, works perfectly smooth, thanks! File sizes for files show up instantly and no UI hangs to be noticed.


Handle inconsistent vendor info for IBM Semeru runtimes (backport)

Fixes #23053
It doesn't make sense to have both the IBM and IBM_SEMERU vendor values.

@bot-gradle test QFL

@bot-gradle

@bot-gradle test QFL

OK, I've already triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test QFL

OK, I've already triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test QFL

OK, I've already triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


Sample wanted for kunalsheth/units-of-measure

About the library

Description: Type-safe dimensional analysis and unit conversion in Kotlin.
GitHub: https://github.com/kunalsheth/units-of-measure
Official website: http://units.kunalsheth.info/

What the sample could do
Show how to works with different units of lenght, weight, speed,, ...
Wants to contribute the sample?

Say so in a comment here
Follow instructions in CONTRIBUTING.md
Submit your pull request!

Hey @jmfayard, can I work on this after finishing the Dagger one?

@jmfayard Since this one is using a Kotlin compiler plugin and that API is not stabilized yet and performance measurement is not straightforward, I think we should isolate it in a module or a project.
Once we decide how to organize the project around this, we should edit the contributing guide for kapt and third-party Kotlin compiler plugin dependent libraries.

If it requires a Kotlin compiler plugin, not worth the complexity, I'm closing this issue


Game Jam page updates

Couple updates for the page:
[ ] Link to the YouTube video for Sten's stream (I think this video also needs to be uploaded)

[ ] Under Gardening Tips, add a button (or link) to the "Collect the Clovers" tutorial as a garden-theme game tutorial (i.e. "Get started building a garden game with the Collect the Clovers tutorial") https://arcade.makecode.com/#tutorial:/tutorials/collect-the-clovers

[ ] Move the Gardening Tips section between the Duration and Rules sections so it's higher up on the page

@shakao - shall we close this?


certain videos cannot be downloaded from YouTube

I have found a handful of videos on YouTube that youtube-dl (2016.03.06) is not able to download.  As an example, trying https://www.youtube.com/watch?v=rSy0JtBCZMQ will show:
=================================
$ youtube-dl --prefer-ffmpeg https://www.youtube.com/watch?v=rSy0JtBCZMQ
[youtube] rSy0JtBCZMQ: Downloading webpage
[youtube] rSy0JtBCZMQ: Downloading embed webpage
[youtube] rSy0JtBCZMQ: Refetching age-gated info webpage
[youtube] rSy0JtBCZMQ: Extracting video information
[youtube] RTMP download detected
[download] Destination: STAR BLAZERS Ep. 1 'The Battle At Pluto'-rSy0JtBCZMQ.unknown_video
ERROR: unable to download video
Other videos such as https://www.youtube.com/watch?v=Qt7EKwBPxCo (I just tested it now) works fine (it even handles the Kanji in the file name).

Duplicate of #343.


Multisig. Create multisig. Liquidity restrictions

Description
To create multisig operation, signatory should have a deposit balance on his free account part, otherwise the operation will fall down with LiquidityRestrictions.
https://github.com/novasamatech/polkadot-sdk/blob/5cecb67062199def7dce88a202dbcab835a23f84/substrate/frame/balances/src/impl_currency.rs#L289-L303

test account state:
{
  nonce: 504
  consumers: 1
  providers: 1
  sufficients: 0
  data: {
    free: 57,374,754,473,836
    reserved: 3,359,733,303,600
    frozen: 58,000,000,000,000
    flags: 170,141,183,460,469,231,731,687,363,818,706,470,870
  }
}

Steps to Reproduce

Select signatory with free balance less than deposit amount, but with transferable part more than required
Try to creaate multisig

Expected Behavior
Validate that the account has less than required on his free balance
Actual Behavior
Operation will fall down with an error:
https://kusama.subscan.io/extrinsic/25732620-2
Environment
https://github.com/novasamatech/nova-spektr/commit/1b29836ffc7ca408a44df236bae6f0d8b98bf087

Test accounts:
accounts.json
wallets.json

Status verification: ✅


fix: Updated comparisons in test files

Replaced all reflect.DeepEqual comparisons in test files with cmp.Equal.
Fixes: #1851

Can someone help me understand, why checks are failing?

Looks like there is still a reference to the reflect package, but it's not imported any longer:
Error: github/orgs_members_test.go:835:6: undefined: reflect

Looks like there is still a reference to the reflect package, but it's not imported any longer:
Error: github/orgs_members_test.go:835:6: undefined: reflect

https://github.com/google/go-github/pull/1875/checks?check_run_id=2677192197#step:8:9

Thanks for replying, but I checked there is no "reflect" imported for that particular file in the branch I created PR with.

It looks like you need to rebase onto the upstream master branch, as you're missing some commits that apparently are getting picked up by the CI jobs (notably d39b94da583ae6dbe8f3039a95f7334fd47b253f).
I don't entirely understand why it's not simply running the CI jobs against your branch, which should pass, but would then have problems come merge time.  Maybe it's automatically trying to merge your changes with master, resulting in these errors?

It looks like you need to rebase onto the upstream master branch, as you're missing some commits that apparently are getting picked up by the CI jobs (notably d39b94d).
I don't entirely understand why it's not simply running the CI jobs against your branch, which should pass, but would then have problems come merge time. Maybe it's automatically trying to merge your changes with master, resulting in these errors?

Thanks @willnorris , I didn't realize my branch was not up to date.
Pushed again after rebasing, I hope checks pass now.

okay, so now the error about the reflect reference in orgs_members_test.go is correct :)  You'll need to switch that one to use the cmp package as well.

okay, so now the error about the reflect reference in orgs_members_test.go is correct :) You'll need to switch that one to use the cmp package as well.

Thanks for helping @willnorris . I have fixed the issue and seems like checks also passed.

@gmlewis oops, looks like I missed catching your issue rename before merging :)

:joy: No problem, Will!


[9.x] Add return type declarations to migration stubs

PHP 7 added support for return type declarations, but Laravel stubs still don't use them.
Return type declarations specify the type of the value that will be returned from a function. The returned value must be of the correct type, otherwise a TypeError will be thrown.
I think this is necessary change in all other Laravel stubs as well.

Please see many previous attempts


Error: No such command "serve_chat".

X:\New folder\Github repos\seq2seq-chatbot\seq2seq-chatbot\models\cornell_movie_dialog\trained_model_v2\trained_model_v2>call C:\ProgramData\Anaconda3\scripts\activate.bat C:\ProgramData\Anaconda3
The system cannot find the path specified.
I'm getting the following error:
X:\New folder\Github repos\seq2seq-chatbot\seq2seq-chatbot\models\cornell_movie_dialog\trained_model_v2\trained_model_v2>cd ..\..\..

X:\New folder\Github repos\seq2seq-chatbot\seq2seq-chatbot\models>set FLASK_APP=chat_web.py

X:\New folder\Github repos\seq2seq-chatbot\seq2seq-chatbot\models>flask serve_chat "models\cornell_movie_dialog\trained_model_v2\best_weights_training.ckpt" -p 8080
Usage: flask [OPTIONS] COMMAND [ARGS]...

Error: No such command "serve_chat".

I installed the required and exact version of dependencies. Still no use. Same issue persists. Please look into it.

sorry, I did a mistake and copied content in a folder one level deeper. My apologies. Now I'm having isssue with chat_ui.html My command window is opening succesfully with http://<IP_ADDRESS>:8080/ but It gives me 404 error.

http://localhost:8080/chat_ui.html this too gives me error,  adn this one too http://<IP_ADDRESS>:8080/chat_ui.html

Can you provide a screenshot?
Thanks

I have copied the weights to the required folder, am I supposed to copy chat_ui.html in it too?

chat_ui.html should be in the root seq2seq-chatbot folder, alongside chat_web.py, chatbot_model.py, and the other python files.
The weights (and batch files) should be under the models folder: seq2seq-chatbot\models\cornell_movie_dialog\trained_model_v2

Done. Thanks.

Same error here, batch files and the weights are under that directory.

@MasterZitron The previous solution worked for mr.

@sunn-e what did you do exactly? As I said, the files are in the right place. It might be a config or my firewall not letting me use it.

I think its because its running on <IP_ADDRESS>, how do I change it?

@MasterZitron from your screenshot it looks like your model directory is missing a few required files:

hparams.json
shared_vocab.tsv

The chat cannot operate without these two files.

Having the same problem ..!!!


Git: Showing old commit count

Issue Type: Bug
Its showing old commit counts even after git push using the UI. It gets pushed to the server but after some time the count reappears.

The count appears back when switching between projects or branches.

Attached screenshot, the problem with "synchronize Change".
The count appears back when switching between projects or branches.

What does git status return?

So.. git is reporting the same as VS Code:
On branch <dev>
Your branch is ahead of 'origin/dev' by 17 commits.
  (use "git push" to publish your local commits)

when I do the git push. The counter resets back to zero. But after some time, it reset back to some number i.e., it shows n no. of the commit in the status again like
On branch <dev>
Your branch is ahead of 'origin/dev' by 17 commits.
  (use "git push" to publish your local commits)

so, the problem is related to git, not vscode?
Because I am facing this issue in more than one project in vscode.

It might be related to another piece of software which is messing with your repositories. Can you reproduce when running with code --disable-extensions?

Its working fine with extensions disabled, I guess some extension have the issue. Thanks you for looking into the matter.


Initial work to support _bulk_get

This PR adds the necessary internal support for the new _bulk_get endpoint.
We can switch between _bulk_get and "one document at a time" whilst providing the same interface to the replication code. Currently only the "one document at a time" code path works. A later PR will add the functionality for calling _bulk_get and deserialising responses in a streaming/iterable manner.

Rename existing CouchClient bulkXXX methods to bulkPostXXX
Add dummy CouchClient.bulkGet method (implementation to come...)
Add CouchClientWrapper.bulkGetRevisions - returns an object which
can be iterated over to get each revision in turn
Convert GetRevisionTask to use above iterator

reviewer: @mikerhodes
reviewer: @ricellis

Thanks for the feedback @ricellis - hopefully everything has been addressed. I've raised a follow-on for some small items which aren't urgent (FB 56304).

+1


add readline method to StreamingBody

The readline method is for example required when using psycopg's cursor copy_from to quickly import  csv file data into postgres

Any timeframe on releasing this?

I think this can be closed as resolved by this PR.


apply_action should update page.status

Describe the bug
https://github.com/sveltejs/kit/blob/a404e63cafc4cac13aa2b4b8bf51574352e8982e/packages/kit/src/runtime/client/client.js#L1205-L1210
page.status not updated, so $page.data contains irrelevant values.
Example case:

return invalid(400) from action (status 400) => $page.status === 400
return {} from action (status 200) => $page.status === 400

Reproduction
https://github.com/dmkret/sveltekit-page-status-form-action

Press Produce 400. This calls return invalid(400)
$page.status = 400, expected = 400 ✅
Press Produce 200. This calls return {}
$page.status = 400, expected = 200 ❌

Logs
No response
System Info
System:
    OS: macOS 12.5.1
    CPU: (12) x64 Intel(R) Core(TM) i7-8700B CPU @ 3.20GHz
    Memory: 342.25 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.9.0 - /usr/local/bin/node
    Yarn: 1.22.18 - /usr/local/bin/yarn
    npm: 8.19.1 - /usr/local/bin/npm
  Browsers:
    Chrome: 105.0.5195.125
    Firefox: 104.0.2
    Safari: 15.6.1
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.76
    @sveltejs/kit: next => 1.0.0-next.488
    svelte: ^3.44.0 => 3.50.1
    vite: ^3.1.0 => 3.1.3

Severity
annoyance
Additional Information
my current workaround without patching sources is to return invalid in valid case
export const actions = {
    default: () => {
        // return {}
        return invalid(201, {})
    }
}

I can't reproduce this given the reproducible, it correctly updates the status for me

@dummdidumm I've also created a fork of https://node.new/sveltekit:
https://stackblitz.com/edit/sveltejs-kit-template-default-uahxzt?file=src/routes/+page.svelte
The issue is reproducible there.

Not sure why I wan't able to reproduce it locally, from looking at the code it probably should be this:

				if (result.status !== page.status) {
					page.status = result.status; // <- page object needs to be updated else subsequent checks are wrong
					props.page = {
						...page
					};
				}

@dummdidumm Could you reproduce this in stackblitz? I could in the provided link, in Firefox.

Yes. sry. wasn't clear on that, wasn't able to reproduce it locally earlier, was abel to reproduce it in stackblitz, which made me look at the code one more time and having found the bug (probably).


Hotspot appearance change

Hi I've just stumbled on this hub and not sure if it's the correct forum but..
Would it be possible to get a hotspot to change appearance after being clicked?

Do you mean hotspots on the hotspot image page or one of the other pages? The hotspot image page will add a border around the hotspot when clicked but it's not possible to, for example, insert another image into the hotspot when clicked. You could see if the morph images page will do what you're after but it's only possible to have 1 hotspot on it.
The forums on the community site (http://xerte.org.uk/index.php?option=com_kunena&view=category&Itemid=760&lang=en&layout=list) or the xerte mailing list (http://lists.nottingham.ac.uk/mailman/listinfo/xerte) are probably the best places for questions like this. You might get some suggestions on how other people have achieved what you're after if you post there and describe the activity you're trying to create.

Yes, on a hotspot image page.
When you click on a hotspot you get a boarder or a visible boarder gets thicker. But there is no way to get the hotspot to be tagged to say it has been selected already. So if you have more than one  hotspot the learner doesn't not has a visual indicator that they have visited that hotspot.
The function I'm describing is built into some of the menus that change colour after they have been clicked so with my very limited knowledge of programing, it would seem like something that can be achieved.

Ps. I've not had any luck posting on the forum you mentioned.

Ok, if you have a reasonably up to date version of Xerte then you can do this:

Add the optional property 'Script' to your hotspot page and paste this code into it:

$("#hsHolder a.hotspot").click(function() {   $(this).addClass("visited"); });

Then add the 'Styles' optional property and paste this into it (adjusting colour etc. depending on exactly how you want a visited hotspot to look):

#hsHolder a.hotspot.visited:not(.selected) {   border: 1px solid black !important; }
This will show hotspots that have previously been clicked with a thin black border around them. You could also look at including background colour / opacity in the css to achieve a faded out look to the whole hotspot
I hope this helps. Unfortunately i don't have time to add a more user friendly way of doing this through editor options at the moment.

That is fantastic. Thanks for your time
Just one last question.
I'm a user rather than owner of xerte so if I have an older version of the software is this going to be unachievable or would there be a work around?

No problem. The styles & script optional properties are both reasonably new so won't be available to add on older versions. I'm not sure exactly when they were added though so aren't sure exactly how new your install will need to be. If they don't appear in the optional property list for the page then there is another way to get the code in. Let me know if you can't see the two options and we can try something else.

Thanks. I've just had that conversation and we are on V. 3.0. So the additional properties are not available to me.

Photo SreenShort Samsung TabLetขอมีส่วนร่วมด้วยคน

Ok, this isn't the most user friendly way of doing things but if you can't add the optional properties then try this:
In the 'Text' field on your hotspot image page, expand the editor's toolbar by clicking the small arrow in the top right of the text field.
Click the 'Source' button and paste this code at the very top of the text field (above any text you have entered for the page)
`
`
Publish and preview your project

Thanks again for your time. I will try this out later today.
Would you have any objections to me copying the above over to the xerte forums as I have the same question posted there. It might help others

No that's fine


Work with stable

If the only blocker is black_box, then perhaps for functions that return certain types we could have alternative mechanisms? e,g, calling functions with different input and collecting the output with a summation?

A stable-compatible release is definitely planned. Most of the nightly-only code lives in stats.rs, so it isn't just black_box. I think most of it should be easily converted to work on stable. The bencher crate has a workaround for black_box as well.
There's been a lot of requests for a stable-compatible version. I probably won't get to this right away, but keep watching this space over the next few weeks. If someone out there would like to submit a pull request, I'd be happy to help.

Thanks for the quick and positive response!


TravisCI integration

We are almost there. Just need to fix some path issues on TravisCI. Like this : https://travis-ci.org/eslint/eslint/builds/109641054

Add support for Coveralls.

Integrate it with the pull request system. Like this : https://github.com/eslint/eslint/pulls

Finished 1 and 3.

Will add support for Coveralls later.


Baseline training schedule

First of all, thank you for uploading this very comprehensive and capable code. It surpasses the quality of what others upload on Github in the 3D-Semantic Space.
Has there been any additional training schedule for the baseline model?
Models like Cylinder3D used non-published training schedules and only hinted at some methods in github issues. They used additional Test-time-augmentation, Instance augmentation, unreported augmentation schedule, model depth/channel tuning, Lasermix, ensembling etc.... Basically every trick in the book.
Is that the same case in this code? Or has the baseline only been trained with the details present in this code? I want to train your code, but rather not invest resources if I cannot reach the reported 67.4 test result of the baseline.
I've read Issue 13 , but do the answers in that issue also apply to the baseline?

additional comment, in Requirements the packages "pyquaternion" is missing.

As reported in the paper of Cylinder3D, they do not use depth/channel tuning, lasermix, etc. How do you know that they use the mentioned tricks?

First of all, thank you for uploading this very comprehensive and capable code. It surpasses the quality of what others upload on Github in the 3D-Semantic Space.
Has there been any additional training schedule for the baseline model? Models like Cylinder3D used non-published training schedules and only hinted at some methods in github issues. They used additional Test-time-augmentation, Instance augmentation, unreported augmentation schedule, model depth/channel tuning, Lasermix, ensembling etc.... Basically every trick in the book.
Is that the same case in this code? Or has the baseline only been trained with the details present in this code? I want to train your code, but rather not invest resources if I cannot reach the reported 67.4 test result of the baseline.
I've read Issue 13 , but do the answers in that issue also apply to the baseline?

As reported in the paper of Cylinder3D, they do not use depth/channel tuning, lasermix, etc. How do you know that they use the mentioned tricks?

@bobpop1
I should have written that statement differently. Most likely they used those tricks, based on the following information.
In the pre-CVPR paper they mention Test Time Augmentation
In the CVPR Paper they mention in the caption of table 1 "Note
that all results are obtained from the literature, where post-processing, flip & rotation test ensemble, etc. are also applied." There is no more detail for this.
In the pre-CVPR paper they reach 65.2 mIOU on validation, 61.8 mIOU on test (- 3.4 mIOU)
In the CVPR paper the behaviour is flipped. 65.9 mIOU validation, 67.8 mIOU on test (+ 1.9 mIOU)
The public code matches the pre-CVPR paper (no pointwise refinement, no extra post-processing methods). The model starts to overfit around epoch 15, however the authors trained for 40 epochs in the CVPR paper, indicating they changed the training regiment to counter this.
There are common authors with PKVD and since they use point-distillation, they very likely used the (non-public) CVPR code of Cylinder3D. In an issue about overfitting, their author gave suggestions on additional augmentation.

Hi @L-Reichardt , thanks for your encouraging comments and useful feedbacks. This codebase currently releases the baseline training schedule on validation set. For the online test set, one needs to slightly modify the codebase, and instance augmentation is not supported in current codebase (the detailed training schedule is shown in our supplementary). Moreover, the SemanticKITTI benchmark is a very challenging online competition, I think the best results of all previous works are gained by carefully tuning the models. Directly training this codebase in 64 epochs for online test set may cannot achieve the best results on the benchmark, but will in a very reasonable range.

@yanx27 Thank you for your answer. I am training your baseline model currently, using the default configs from this repo. I am aware I will not reach the test results in the paper yet, as I am not tuning on validation set.
The topic augmentation is still somewhat unclear to me.
In the paper Supplementary - A I read:    rotate, scale
In your code I find:    rotate, flip, scale, translate + instance augmentation (Issue 13)
Are all augmentations from the code used? Or just rotate + scale + instance? Are any augmentations missing in this list?
What exactly do you mean with "one needs to slightly modify the codebase"?

@bobpop1 I should have written that statement differently. Most likely they used those tricks, based on the following information.
In the pre-CVPR paper they mention Test Time Augmentation (August 2020) In the CVPR Paper they mention in the caption of table 1 "Note that all results are obtained from the literature, where post-processing, flip & rotation test ensemble, etc. are also applied." There is no more detail for this. Also, augmentation is not explicitly stated anymore (November 2020).
In the pre-CVPR paper they reach 65.2 mIOU on validation, 61.8 mIOU on test (- 3.4 mIOU) In the CVPR paper the behaviour is flipped. 65.9 mIOU validation, 67.8 mIOU on test (+ 1.9 mIOU)
The public code matches the pre-CVPR paper (no pointwise refinement, no extra post-processing methods). The model starts to overfit around epoch 15, however the authors trained for 40 epochs in the CVPR paper, indicating they changed the training regiment to counter this.
There are common authors with PKVD and since they use point-distillation, they very likely used the (non-public) CVPR code of Cylinder3D. In an issue about overfitting, their author gave suggestions on additional augmentation.

Thank you for your reply.

First of all, thank you for uploading this very comprehensive and capable code. It surpasses the quality of what others upload on Github in the 3D-Semantic Space.
Has there been any additional training schedule for the baseline model? Models like Cylinder3D used non-published training schedules and only hinted at some methods in github issues or in their paper. Most likely they used additional Test-time-augmentation, Instance augmentation, unreported augmentation schedule, model depth/channel tuning, Lasermix, ensembling etc.... Potentially every trick in the book.
Is that the same case in this code? Or has the baseline only been trained with the details present in this code? I want to train your code, but rather not invest resources if I cannot reach the reported 67.4 test result of the baseline.
I've read Issue 13 , but do the answers in that issue also apply to the baseline?

Hi, thanks for your information. Did you reach the reported 72.9 test result of the paper. I only achieve nearly 69 miou which is much lower than the reported scores. I think there are also other tricks for this code.

Hello @bobpop1 I've retrained the baseline using the authors script and reached ~64 validation mIOU so something is still missing. Adding extra augmentations gets me to 67mIOU.
I did not test the rest of their model and have not put the baseline on the KITTI test server yet.

Hi @L-Reichardt, could you share with us what kind of extra augmentations you used for 67 mIOU?


Raw BGGR frame Bayer filter sampling feature?

I saw someone is removing the Bayer filter in the "Little Garden Spectrometer" cameras, and was curious if anyone has already looked into the increased bit depth of the RAW BGGR green channel, and physical pixel location on the sensors (taking the staggered horizontal lines gaps and interleaving  into a half vertical frame resolution to improve horizontal resolution display).
Was about to write something myself in opencv, but saw this project was well developed already.
(mentioned in https://budgetlightforum.com/t/little-garden-spectrometer-impressions-opinions-discussion/225545 )
I guess this is a documentation and or feature request for Linux platforms with some USB cameras.
Best regards =)

This is very interesting! Thanks for sharing.
The question is if the camera can even output the raw sensor data (https://www.kernel.org/doc/html/v5.6/media/uapi/v4l/pixfmt-bayer.html).
The i-PhosHD cannot when I remember correctly.
If you can point me to a model that does, I could have a look. As long as it can be bought somewhere to a reasonable prize, of course.

I ordered 2 samples of Lao Kang's version of the "Theremino spectometer" from here:
https://www.aliexpress.com/item/1005007241834839.html
I will be testing the v4l2 camera driver support for "RG12", "YUYV 4:2:2", or "UYVY" formats
v4l2-ctl -d /dev/video0 --list-formats-ext
v4l2-ctl --list-formats
My intent was to look into the bindings for raw frame access in a rust compatible cam wrapper library:
https://github.com/l1npengtul/nokhwa
If the results are reasonable, eventually I want to add a 3D printed narrow-band optical filter-holder wheel stepper-controller for an Arduino sketch to the project (sending a simple serial command to index the wheel, read from  a zero marker, and suppress the spurious diffraction spikes in the spectra etc.).
I will update the thread with the models PID:VID USB device details when it arrives in a few weeks.

Ok, so I manged to pull the camera hardware model from the device:
0c68:6464 Sonix Technology Co., Ltd. USB 2.0 Camera
The driver does not provide raw frame access, but thanks for the tips.  I will still fork the code to add some features in a few weeks.


Line Numbers

Line Numbers In The Editor.

Completed In Release: v1.1
Commit: https://github.com/markydoodled/Note.it/commit/eeec946d7aac322017398c2c5c2e5d97253140a8


Any documents?

thanks for your awsome work, but I don't how to start.

These largely map to the geth management APIs.  https://github.com/ethereum/go-ethereum/wiki/Management-APIs

thx @carryforward


Add Keyword labels to the README

Reference Article: https://shields.io/

Assign this issue to me.

Maybe I can help you if you want @Shashank7001. Can't work on it until Friday though
Best regards
NH

@SID262000 If I don't need to add anything in the index.html then I can make a pull request and then work on another issue

@SID262000 Kindly assign this issue to me.

@SID262000 Kindly tell me in which parts you need keyword labels I will update the README accordingly.

If you refer to the link attached as reference, those shields usually appear at top of the README
cc: @KartikKar19

Any updates on the issue @KartikKar19 . Do you need any help?

@AryanGupta0007 yeah I am currently working on it locally but the issue is I am just adding keywords on things I think are important.I don't know where to add keywords.Kindly guide me.

Cc:@SID262000 please guide @KartikKar19 on what labels are required.

@KartikKar19 one label that u can add on top is code coverage 96%. Else @SID262000 will guide you.

@AryanGupta0007 okay thanks I will add that.

@SID262000 just tell me which ones we need to add I will add it accordingly and finish this by tomorrow.

@KartikKar19 Please add the most appropriate ones from the list below.
https://github.com/alexandresanlim/Badges4-README.md-Profile

@SID262000 @AryanGupta0007 Kindly see my forked repo. I have added some keyword labels and badges which are interactive and some static . Tell me if you want any changes.

Hi @KartikKar19,
Thanks for your work but I think that you used too many labels. I'd recommend you to look at this repo to see how the labels can be used efficiently.
Repo: https://github.com/CodeEditApp/CodeEdit

@N3v1 Thanks, I will make the changes accordingly and let you guys know.

Many Thanks @KartikKar19 @N3v1 for your collaboration. Looking forward to seeing more progress on this front.


Add PHPUnit

Hi!
I've added Phpunit and Orchestra to helps with tests.
hope your enjoy.
have nice tests xD

references #16

Okay, test deleted.
If your have some function do you like to test, i can do it. I really don't know anything about your package, then i no have idea to do the tests

Yes. Somebody else is already attempting the tests.

yea, don't needed.
i put by habit haha.
--
Ok, if in the future had some tests to do, i will be happy helping the package. =)

If you want you can write tests for the function. Haven't heard from the other guy as yet


SvelteKit support

Please.

I am not a fan of the direction or the usability of SvelteKit (locks you into an overly opionated ecosystem) so this will not happen. Sorry.


build(release): switch to daily release

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Added a asDotString function to visualize a graph

This functionAdded turns a DAG of components into DOT format which can be drawn by GraphViz.
Review @jcranky @taojang

👍 but I don't understand why test fails

@taojang I don't understand it either yet. It doesn't seem to be deterministic and works ok on my machine all the time.

:+1:

which test is failing ? I don't see it

@jcranky I triggered the build again and it passed :)


Port to ST

SublimeText just dropped version 3.2 this morning, with nifty git integration.  ST and Atom are close enough that it would be nice to be able to use ST with this package, too.

any plan to do anything with this repo, now that atom's sunsetting is imminent?


Support GCP KMS credentials using decryption secretRef

This pull request adds support for specifying the credentials for authenticating to GCP KMS with the decryption secret.
GCP KMS credentials should be provided under sops.gcp-kms
Fix: #360
Ref: #324

@SomtochiAma just wanted to let you know that this is in my queue to get reviewed. But given it would require a minor bump, I am giving priority to some things that can end up patches first. Will get to this on Thursday, hopefully :-)


Change name / location of .drone.yml

Is there a way to change the location of the .drone.yml file
Looking at the source code ( though I don't know go) it appears the look up for .drone.yml only occurs at the root of the repository and is not searched for recursively correct?

see drone/server.go,  the DRONE_PATH can be specified.
60 ^I^Icli.StringFlag{
61 ^I^I^IEnvVar: "DRONE_YAML",
62 ^I^I^IName:   "yaml",
63 ^I^I^IUsage:  "build configuraton file name",
64 ^I^I^IValue:  ".drone.yml",
65 ^I^I},

@xiaods but that is a 0.5 feature, that wasn't available on 0.4

closing since this is solved in 0.5


Client Integration tests giving wrong pass/fail indication

I am running Jasmine in the leaderboard example. I am changing the client integration test in the default Jasmine code by inserting this extra expect:

it("should be able to play a Song", function() {
expect(true).toEqual(false);

I find that if I change the false to true back and forth several times, the test results become somehow stale and Jasmine reports success when it should report fail and fail when it should report success. This is a big problem because the test results could say pass when there is still a bug.
I haven't completely figured out the pattern by which to clear the stale behavior. I've tried restarting meteor, deleting packages/tests-proxy/, and reloading the page. Sometimes I can get it to display the correct result, but I haven't been able to figure out a reproducible sequence.
Any ideas? I am running under OSX Yosemite, everything up to the latest. Also Meteor <IP_ADDRESS>. Here are the revisions of all the code:

4pi:todos juan$ meteor list
accounts-password       1.0.6  Password support for accounts
insecure                1.0.2  Allow all database writes by default
iron:router             0.9.4* Routing specifically designed for Meteor
less                    1.0.12  The dynamic stylesheet language
meteor-platform         1.2.1  Include a standard set of Meteor packages in your app
sanjo:jasmine           0.12.0* Easily use Jasmine in Meteor
velocity:html-reporter  0.4.1  Reactive Velocity test reports in your app.

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Thanks! -Juan

Hard to reproduce. I think I got it once. What I see is that the client refreshes and you see the old results for a short time until the new run has completed.
The client integration tests run from the iFrame that is embedded in the app (on the end of the body).

OK, here is a hopefully reproducible sequence. Starting from scratch:

meteor create --example todos & cd todos

meteor add sanjo:jasmine (installs 0.12.1)

meteor add velocity:html-reporter

meteor

open a page to localhost:3000
click on all the buttons to create tests
At this point some tests complete, others are loading. No Karma Chrome window.
So break out of meteor and restart.
Now all tests passing
To the client integration test, add the following line:

it("should be able to play a Song", function() {
expect(true).toBe(false);

Change false to true back to false, 6 times. The side panel results reported correctly.
Quit out of meteor and restart (verify that the Karma window closes.)
Change false to true back to false 6 times. Now the side panel does not report correctly. Reloading page does not help.
Restart meteor. Result is reported correctly.
Edit to change false to true. Save. Result reported incorrectly.

Does that reproduce?

I don't know if this is related, but I found the following error messages in the console log:

[Log] [sanjo:jasmine]: Running Jasmine tests (sanjo_jasmine.js, line 4180)
[Log] Error invoking Method 'jasmine/specDoneConsumer': Method not found [404] (meteor.js, line 887, x5)
[Log] Error invoking Method 'jasmine/doneConsumer': Method not found [404] (meteor.js, line 887)

Hmm, unable to reproduce this morning. It appears intermittent. This smells like something is up with that Karma process. Perhaps it gets wedged and stops running at times? That would explain it. I'm not getting wrong results, exactly, I'm getting stale results because the results are not refreshing. Restarting Meteor helps because Karma gets restarted.
Would it be possible to have a more positive indication of whether the test is being rerun by Karma? I'm thinking something like clear out the old results in the sidebar upon starting a test. Then the new results would be redrawn in the window as they are completed. A time and date would also help.
In the interim, is there a way to see the karma results directly in some kind of console? Or is there a way to run Jasmine for Meteor without the sidebar? (The logs button in the sidebar shows nothing.) I really like the way Jasmine works, and I'd like to use it in my project. It's just this reporting thing is a big obstacle to using it.
Thanks!
-Juan

The log doesn't seem to help. See below the entire contents:

4pi:todos juan$ more .meteor/local/log/jasmine-client-integration.log
[velocity-mirror] Mirror is initializing...
LISTENING
[velocity-mirror] Mirror is ready.

Have save errors on page loads:
[sanjo:jasmine]: Running Jasmine tests
meteor.js?81e2f06cff198adaa81b3bc09fc4f3728b7370ec:887 Error invoking Method 'jasmine/specDoneConsumer': Method not found [404]
meteor.js?81e2f06cff198adaa81b3bc09fc4f3728b7370ec:887 Error invoking Method 'jasmine/doneConsumer': Method not found [404]

I've founs this code in ClientIntegrationTestFramework.js on 121 line:
var serverReporter = {
          jasmineStarted: function () {
            window.ddpParentConnection.call("jasmine/startedConsumer",
              function (err, result) {
                currentId = result;
              });
          },
          jasmineDone: function () {
            window.ddpParentConnection.call("jasmine/doneConsumer", currentId)
          },
          specDone: function (result) {
            window.ddpParentConnection.call("jasmine/specDoneConsumer",
              result,
              currentId)
          }
        }

And all three raises errors that called method not found, just first catch this error and last two not so it displayed in console. Now try to find where this methods registering. Seems like we have wrong load ordering here…

Hmm. seems like this methods don't exists at all… @Sanjo, It's not done now or missed some dependency?

This should be now completely fixed since 0.13.0

I'm using 0.15 and still have the same problem in a very large app. (but unable to reproduce in a small app.)


Installing the TestDive via swift module manager

I encountered with predicament while trying to run command to compile swift module:
swift build -c release -Xswiftc -static-stdlib
Cloning git@github.com:johnsundell/xgen.git
HEAD is now at f463b9d Use SSH to clone Files
Resolved version: 1.0.0
Cloning git@github.com:johnsundell/files.git
HEAD is now at 284d85d Merge pull request #11 from yageek/feature/currentFolder
Resolved version: 1.8.0
Cloning git@github.com:johnsundell/shellout.git
HEAD is now at 38cd16d README: Update for new series of commands + path API
Resolved version: 1.1.0
Cloning git@github.com:johnsundell/releases.git
HEAD is now at fe9b9aa Version: Keep original version string intact
Resolved version: 1.0.5
Cloning git@github.com:johnsundell/require.git
HEAD is now at 44d5266 Remove whitespace & improve code formatting
Resolved version: 1.0.2
Compile Swift Module 'Require' (1 sources)
Compile Swift Module 'ShellOut' (1 sources)
Compile Swift Module 'Files' (1 sources)
Compile Swift Module 'Releases' (4 sources)
/Users/k.kusainov/TestDrive/Packages/Releases-1.0.5/Sources/Array+Version.swift:10:38: error: same-type requirement makes generic parameter 'Element' non-generic
public extension Array where Element == Version {
                                     ^
/Users/k.kusainov/TestDrive/Packages/Releases-1.0.5/Sources/Array+Version.swift:12:34: error: reference to generic type 'Array' requires arguments in <...>
    func withoutPreReleases() -> Array {
                                 ^
                                      <Any>
<unknown>:0: note: generic type 'Array' declared here
<unknown>:0: error: build had 1 command failures
error: exit(1): /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swift-build-tool -f /Users/k.kusainov/TestDrive/.build/release.yaml

It seems the module Array+Version has an internal error

@kos9kus Test Drive requires Swift 3.1. Please update Xcode to version 8.3 or later and the problem should go away 🙂

The issue has been gone, Thanks for help!


bus.sendError() disconnects from the bus

I'm trying to use the sendError() function to return some errors to a method call, but when I do that, the services disconnects form the bus without sending the answer. At the other end, the method caller returns with "recipient disconnecting without replying".
This is much infortunate because it seems I can't return some errors to the caller. I can't seem to track the moment it happens.
sendError() calls connection.message(), which just marshals the message. I have put some console.log() in the stream.on('end') event, which does get called, but I cannot find where or what causes the stream to end.
Can someone point me to the right direction?
Thanks in advance.

would you be able to start wireshark and capture data just before disconnect?

Hum maybe I can try. This is a tricky setup that I have, since I have the scripts run in a Docker container. What filter do I sue on wireshark to get those packets? On what interface should I listen?

Hum I just tried on my host (no container), I have started wireshark and sent gdbus introspect calls, but I could not find it on wireshark (on any interface).
Maybe I'm doing something wrong.
Do you have any idea why the script disconnects from DBus?

if there is no crash/errors on script side it's likely dbus daemon disconnected it. Maybe bug in serialisation, maybe something else

It seems to happen on stream.write() here.
I'll be investigating, in the meantime, if you have some ideas...
Thanks for answering so quickly ^^

could you console.log(msg) and message.marshall(msg).toString('hex') just before that line?

Yes, sure, here it is:

{ type: 3,
  replySerial: 11,
  destination: ':1.206',
  errorName: 'Error',
  signature: 's',
  body: [ 'Error thrown from Avatar' ],
  serial: 1 }

And

'6c0300011d000000010000002f00000004017300050000004572726f72000000050175000b00000006017300060000003a312e32303600000801670001730000180000004572726f72207468726f776e2066726f6d2041766174617200'

Does that help?

yes, thanks. I'll try to check

Okay good. For information, I used node debug to try and find the error, I stepped in as deep as I could and I eventually reach the error in the core or the net.js library, at the moment it writes to the buffer.
But I don't get a chance to see why, even with step in the error just "happens", it seems atomic, I cannot decompose it further :/

I can reproduce this, investigating

I have tried another thing,, here it is.
First, I fixed a typo bug here, in sendReply(), the last line, the one actually sending the message is this.connection.message(msg) and I replaced it with this.connection.message(reply).
I confirmed it worked ok: the service does send an answer when I call self.sendReply(). Then I temporarily edited the code of sendReply() to replace type: constants.messageType.methodReturn with type: constants.messageType.error and then it does exactly as before: disconnect from bus.
So apparently, this is inherently linked to the fact that the type field is set to error. But I cannot find a clue (yet) as to why.
P.S.: shall I send a merge request to fix the sendReply() command?

@nschoe try to set errorName to something which looks like valid interface. I tried sending error with "foo" as errorName and the server was disconnected, and when I used 'com.example.MusicPlayer.Error.OutOfMemory' the error was delivered to caller.
https://dbus.freedesktop.org/doc/dbus-specification.html#message-protocol-names-error

Error names
Error names have the same restrictions as interface names.
Error names have the same naming conventions as interface names, and often contain .Error.; for instance, the owner of example.com might define the errors com.example.MusicPlayer.Error.FileNotFound and com.example.MusicPlayer.Error.OutOfMemory. The errors defined by D-Bus itself, such as org.freedesktop.DBus.Error.Failed, follow a similar pattern.

shall I send a merge request to fix the sendReply() command?

yes please! ( though this might actually been fixed as part of #118 , unfortunately still don't have time to finish that )

might be a good idea to add the check for errorName to the client so it'll fail early with clear message how to fix. Need to investigate what exactly is name requirement and why "foo" is not valid name

Ah! Thanks, you were right for the error name, it must contain a dot (.). Setting the error name to foo.bar works.
Great! Might add a little comment as a reminder, though.
Thanks for your help!
I'm submitting the PR for the typo ;-)

Added a comment to remind people of this in sendError().


Setup malt/learner as a lang

A very basic #lang set up.

The change so far to the docs are probably okay as reference documentation.  I'll try writing up something more instruction-like and see where it would fit into the current documentation.


Don't run multiple instances

When i run:
sudo create_ap wlan0 eth0 CassioAP<PHONE_NUMBER>a --no-virt &
sudo create_ap wlan1 eth0 CassioAccessPoint amanhatedigo --hidden --no-virt

run normally, the problem is you can not assign IP clients simultaneously.

You can use different subnet with the -g. For example use -g <IP_ADDRESS> on the second AP.


Website push notifications

Or RSS feed - whatever the simplest, most effective method of pushing update notices when data is changed.

Have you tried Visual Ping?

well, i am not sure how important this feature is, as data should be updated at the same time every day, around 11. It will once the proper update automation will be done.

I would love something like this too.
I found an option for this in the meantime, I use Zapier to parse the JSON on a schedule so I can be notified about the stats I am interested in daily (I have a filter to make sure the data has been updated using the updatedOn


panic: runtime error: index out of range [1] with length 0

Your environment.

Version: v1.9.4 - v1.9.1 ( last working version v1.9.0 )
Client: Chrome -- Version 88.0.4324.190 (Official Build) (64-bit)
Environement: GCP
Are you using a TURN server? No

What did you do?
Connect to sfu causes crash.
What happened?
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: panic: runtime error: index out of range [1] with length 0
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: goroutine 195 [running]:
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: encoding/binary.bigEndian.PutUint16(...)
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /usr/local/go/src/encoding/binary/binary.go:106
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: github.com/pion/ion-sfu/pkg/buffer.(*Bucket).push(...)
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /srv/ion-sfu/pkg/buffer/bucket.go:59
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: github.com/pion/ion-sfu/pkg/buffer.(*Bucket).AddPacket(0xc000274ff0, 0xc00000ad00, 0xf4, 0xf4, 0x104cb, 0x2, 0x203000, 0x203000, 0
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /srv/ion-sfu/pkg/buffer/bucket.go:37 +0x26f
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: github.com/pion/ion-sfu/pkg/buffer.(*Buffer).calc(0xc000200000, 0xc00000ad00, 0xf4, 0xf4, 0x166bfda7f4c9e7c8)
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /srv/ion-sfu/pkg/buffer/buffer.go:313 +0x1e5
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: github.com/pion/ion-sfu/pkg/buffer.(*Buffer).Bind(0xc000200000, 0xc0000ba7e0, 0x3, 0x4, 0xc0000e0000, 0x7, 0x8, 0x5dc)
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /srv/ion-sfu/pkg/buffer/buffer.go:178 +0x287
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: github.com/pion/ion-sfu/pkg/sfu.(*router).AddReceiver(0xc0001aba40, 0xc000466180, 0xc0000c4d80, 0x0, 0x0, 0x0)
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /srv/ion-sfu/pkg/sfu/router.go:179 +0x47a
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: github.com/pion/ion-sfu/pkg/sfu.NewPublisher.func1(0xc0000c4d80, 0xc000466180)
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /srv/ion-sfu/pkg/sfu/publisher.go:55 +0x354
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]: created by github.com/pion/webrtc/v3.(*PeerConnection).onTrack
Mar 13 19:29:49 tmiv-nakama-1 launch-sfu.sh[15959]:         /srv/go/pkg/mod/github.com/pion/webrtc/v3@v3.0.10/peerconnection.go:451 +0x10f
Mar 13 19:29:49 tmiv-nakama-1 systemd[1]: ion-sfu.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Mar 13 19:29:49 tmiv-nakama-1 systemd[1]: ion-sfu.service: Failed with result 'exit-code'.
Mar 13 19:29:50 tmiv-nakama-1 systemd[1]: ion-sfu.service: Service RestartSec=1s expired, scheduling restart.

@tmiv there was a change in the config file, need to update it.
Sorry for the inconvenience, going to  fix to not cause panics the next update

Oh. I see. Thank you. That fixed it.


添加了sinodb dialect

添加了sinodb dialect

You can open a new pull request if you actually have something that compiles, succeeds tests and follows our code style.


Fixes #1858 - Render of source JSX for Storybook components

Adding the storysource addon to correctly render the source code for Storybook components as JSX.  This currently appears to add a bit of extra cruft to the code, but I was unable to configure the addon not to do that.

Fixes #1858

@rahul1 @mattwiller  - I don't think I follow what we're trying to solve here.  The JSX source already renders, yeah?  See comment here: https://github.com/medplum/medplum/issues/1858#issuecomment-1518133388


Add REPL mode

Need to modificate Main class and even invoke the behaviour of REPL into separate class that will be called by Main according to command line arguments

Implemented at branch 'release-0.2.0'


Parser stucked in loop

Hi,
I have an XML input with encoding problems.
I use the default KeyValue element ;
Because of the encoding problem, the function "keyValue" is strucked in the loop ; the reader object never reach the node "Reader::END_ELEMENT".
I've tried both :
    do {

        if ($reader->nodeType === Reader::ELEMENT) {
            if ($namespace !== null && $reader->namespaceURI === $namespace) {
                $values[$reader->localName] = $reader->parseCurrentElement()['value'];
            } else {
                $clark = $reader->getClark();
                $values[$clark] = $reader->parseCurrentElement()['value'];

                if (false === $values[$clark]) {
                    $reader->next();
                }
            }
        } else {
            $reader->read();
        }
    } while ($reader->nodeType !== Reader::END_ELEMENT);

    do {

        if ($reader->nodeType === Reader::ELEMENT) {
            if ($namespace !== null && $reader->namespaceURI === $namespace) {
                $values[$reader->localName] = $reader->parseCurrentElement()['value'];
            } else {
                $clark = $reader->getClark();
                $values[$clark] = $reader->parseCurrentElement()['value'];

                if (false === $values[$clark]) {
                    $reader->read();
                }
            }
        } else {
            $reader->read();
        }
    } while ($reader->nodeType !== Reader::END_ELEMENT);

It doesn't work, reader styled stucked on the same element =/
Any idea/way to skip element and move further ?
Thanks,
Regards

This is not going to work well, because by the time you call read or next, you might actually be deeper inside the xml document. Calling next will also traverse beyond END_ELEMENT which is why you are never reaching it.
Could you share the xml snippet you are trying to parse, and what PHP data structure you wish to get? I might be able to rewrite it in a way where it's a lot clearer. Ideally you should never really have to do manual traversal like this, unless it's a special circumstance.

Could you share the xml snippet you are trying to parse, and what PHP data structure you wish to get?

I will try to share you only the peace of xml that is wrong.

I might be able to rewrite it in a way where it's a lot clearer. Ideally you should never really have to do manual traversal like this, unless it's a special circumstance.

I use your reader like in the documentation. I just go deeper in the code and found this loop ; i tried to "debug it", tyring to "force" the reader to find the next element. I don't use this loop myself ^^
When my element is parsed, it go an error, but you silence it (https://github.com/fruux/sabre-xml/blob/master/lib/Reader.php#L145);
I var_dumped the content :  LibXML Error Input is not proper UTF-8, indicate encoding ! (don't have the complete message right now)
And so, the reader is still looping on the same element, and always go the same error.
Anyway, I will paste you a complete example =)

The error is indeed silenced there, but the error does get stored and we actually take it out here again:
https://github.com/fruux/sabre-xml/blob/master/lib/Reader.php#L151
There might be a bug in the error handling code though. The error you shared definitely seems to indicate so. So it would then be extra awesome to get a snippet of your xml that reproduces this, so we can make the parser more robust =)

Hi @Gounlaf , I received your data via email. Thanks very much for that.
I tried to reproduce the issue with the following script:
<?php

include 'vendor/autoload.php';

$reader = new Sabre\Xml\Reader();
$reader->open('xml_bug_wrong_encoding.xml');

var_dump($reader->parse());

This causes an exception to be triggered immediately:
Sabre\Xml\LibXMLException: Input is not proper UTF-8, indicate encoding !
Bytes: 0x1A 0x29 0x20 0x61
 on line 24, column 25 in /Users/evert/code/sabre/xml/lib/Reader.php on line 155

Call Stack:
    0.0002     228840   1. {main}() /Users/evert/code/sabre/xml/issue87.php:0
    0.0571     587552   2. Sabre\Xml\Reader->parse() /Users/evert/code/sabre/xml/issue87.php:8
    0.0571     588200   3. Sabre\Xml\Reader->parseCurrentElement() /Users/evert/code/sabre/xml/lib/Reader.php:69
    0.0572     589120   4. call_user_func:{/Users/evert/code/sabre/xml/lib/Reader.php:231}() /Users/evert/code/sabre/xml/lib/Reader.php:231
    0.1170     604496   5. Sabre\Xml\Element\Base::xmlDeserialize() /Users/evert/code/sabre/xml/lib/Reader.php:231
    0.1170     604912   6. Sabre\Xml\Reader->parseInnerTree() /Users/evert/code/sabre/xml/lib/Element/Base.php:86
    0.1171     605328   7. Sabre\Xml\Reader->parseCurrentElement() /Users/evert/code/sabre/xml/lib/Reader.php:161
    0.1171     606096   8. call_user_func:{/Users/evert/code/sabre/xml/lib/Reader.php:231}() /Users/evert/code/sabre/xml/lib/Reader.php:231
    0.1171     606128   9. Sabre\Xml\Element\Base::xmlDeserialize() /Users/evert/code/sabre/xml/lib/Reader.php:231
    0.1171     606128  10. Sabre\Xml\Reader->parseInnerTree() /Users/evert/code/sabre/xml/lib/Element/Base.php:86

I would consider this the expected behavior. Are you also seeing this when you run my test script or do you get the loop?

The problem with your file BTW is that ASCII character 26 (1A) appears in your source. This encodes CTRL-Z and should normally never appear in a text file. But still it shouldn't go in a never-ending loop

Hi @evert,
If I use your example, yes the exception is thrown.
But if i use the example given in "http://sabre.io/xml/reading/ => Using the XmlDeserializable interface", there is the loop. I sent you the "looping example" by email.

Would it be possible for you to email me a script that always reproduces the error. It's a bit hard for me to figure out where this is going wrong.
The example you mention on sabre.io should be fine because it just uses parseInnerTree, but I could be wrong.
So ideally if you could send me a single php file that has a minimal sample that reproduces the problem for you, I would be very grateful!

Hi @evert ,
I did yesterday, with the data I already sent to you, and the "copy/past" of the example, adapted for the data. Anyway, you can find it bellow :)
<?php
include 'vendor/autoload.php';

class Offers implements Sabre\Xml\XmlDeserializable
{
    public $data = array();

    static function xmlDeserialize(Sabre\Xml\Reader $reader) {

        $offers = new self();
        $children = $reader->parseInnerTree();
        foreach($children as $child) {
            if ($child['value'] instanceof Offer) {
                $offers->data[] = $child['value'];
            }
        }
        return $offers;

    }
}

class Offer implements Sabre\Xml\XmlDeserializable {
    static function xmlDeserialize(Sabre\Xml\Reader $reader) {

        $offer = new self();

        // Borrowing a parser from the KeyValue class.
        $keyValue = Sabre\Xml\Element\KeyValue::xmlDeserialize($reader);

//        if (isset($keyValue['{http://example.org/books}title'])) {
//            $book->title = $keyValue['{http://example.org/books}title'];
//        }
//        if (isset($keyValue['{http://example.org/books}author'])) {
//            $book->author = $keyValue['{http://example.org/books}author'];
//        }

        return $offer;

    }
}

$reader = new Sabre\Xml\Reader();
$reader->elementMap = [
    '{}offres' => 'Offers',
    '{}offre' => 'Offer',
];

$reader->open('xml_bug_wrong_encoding.xml');

var_dump($reader->parse());

I'm closing this ticket because it's been a while since the last comment.
If this is indeed still an issue, feel free to comment here so we can continue discussing.
This is just a general cleanup. And unfortunately this ticket never got a fully working sample. Might have gotten it via email back in february, but I no longer have it. If you care about this ticket still, please submit the info again (preferably just on github).


Add Microbundle and also update the api methods to handle errors and return Slice/Type types

This has a lot of changes to the the package.
Added

[x] micorbundle for commonjs or ECMA
[ ] error handling on slices
[ ] error handling on types
[ ] custom responses for insert and update
[x] type definitions for Slices
[x] type definitions for Custom Types

Changed
A lot. But, this one changes the response of things like the getOne methods to return a promised version of the requested object, so no need to handle the actual http information.

Will add the other stuff to a todo, and then link back :)


fix: kubernetes api generation (#73)

This pull request fixes and improves generation support for kubernetes APIs. Besides, it updates dependencies to support go 1.23 and golangci-lint 1.60.3 as well as Kubernetes 0.31. Finally, it implements a new feature for caching targets to improve responsiveness for auto completion.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
310

Relevant Lines:
310

💛  - Coveralls


REL: 0.19

I think this repository is in good shape to have a release, with Pythons 3.6-3.9 tested.
This PR does a little last minute cleanup, bumps the version, and uses the README.md as the PyPI description.
I'd appreciate a review and look over recent changes: https://github.com/python-versioneer/python-versioneer/compare/0.18...rel/0.19
Also, I would ask to be made a maintainer on PyPI. My username there is cjmarkie.
cc @bucknerns @yarikoptic @bashtage @warner

Anybody care to review?

LGTM.  Should try it locally just to verify docs are all OK.

Only thing I noticed: Should 0.19 be added to UPGRADING.md (under the same as 0.16,0.17 and 0.18)?

Thanks for the review. Added 0.19 to UPGRADING.

LGTM. Should try it locally just to verify docs are all OK.

Not sure how to operationalize this. What were you thinking?
I have tested with:
pip install .
# go to some project
versioneer install
python setup.py version

Well, going to get this out. Here goes nothing.


feat: bump terraform version for tf-account

We've let the terraform version lag for a while, but the config-driven state management in version 1.5 is useful for tf-account.

Closes https://observe.atlassian.net/browse/OB-20583
Support for interpolation of id will arrive in 1.6, FYI:
https://github.com/hashicorp/terraform/pull/33618


Flag when a new Operator has been added to the database

Describe the task
When a user creates a brand new Operator (i.e., not one that already existed in our database), we want to add a boolean flag to the record in the database indicating that it's new. This "is_new" flag will indicate whether CAS internal staff need to explicitly approve the Operator at the same time that they're approving the request for prime admin access.
(Operators imported from CIIP will have is_new flag set to false, and will automatically have their status set to Approved when the data is imported).
Acceptance Criteria

[ ] when an internal user is reviewing a prime admin access request, if the Operator info that they're viewing at the same time (https://www.figma.com/file/LsDmLDCdnJqI8UahyqvOD8/OBPS-App-Designs?type=design&node-id=413%3A48&mode=design&t=b0MX1lfLnGp4Rch3-1) has the is_new = True flag, the internal user will have to explicitly Approve the Operator before the prime admin request can be approved. If the Operator is not new (i.e., is_new = False, the internal user will not need to change the Operator's status.
[ ] once an Operator's status has been set to Approved, the is_new flag can be set to False
[ ] if the internal user Requests Changes for the new Operator, the is_new flag will remain as True until the Operator's status is changed to Approved

Additional context

Add any other context about the task here.
Or here

@nanyangpro  do you plan to update the wireframes with some sort of marker so that internal users can tell when an Operator is brand new?

Yes, @andrea-williams. I'll add the link once done. Before I proceed, I've got a question for you in Figma, could you please review and confirm there? Thanks!

Hey team! Please add your planning poker estimate with Zenhub @andrea-williams @BCerki @dleard @JoshLarouche @marcellmueller @Sepehr-Sobhani @shon-button

@andrea-williams I think we said if we auto-approve operators that are imported, we don't have all the data we require from them at that time & there will be updates after approval. I think is_new may still be useful information to the business area even if we decide to auto-approve. Should we compare what we get from importing for an operator vs what we ask for to help inform our decision on auto-approval?

@andrea-williams I think we said if we auto-approve operators that are imported, we don't have all the data we require from them at that time & there will be updates after approval. I think is_new may still be useful information to the business area even if we decide to auto-approve. Should we compare what we get from importing for an operator vs what we ask for to help inform our decision on auto-approval?

@andrea-williams / @dleard I was mostly suggesting that because we are currently not using status field of the Operator data model anywhere in the code, I was wondering if maybe we could benefit from that field or remove it if not needed.

the status field on the Operator will now be used; i.e., new Operators will require explicit approval from CAS internal staff

After talking with @nanyangpro , the design and styling of this feature are going to use the multistep component rather than the collapsible sections. The important pieces to retain from the wireframe are:

text to indicate that new operations are new
approve/reject buttons only affect the piece of the form they show up on (e.g., approving the operator only approves the operator, and the user needs to be approved separately on the next page)


merge from arlac77/npm-package-template

package.json

chore(package): add ^1.31.0 remove ^1.30.1 (devDependencies.rollup)

:tada: This PR is included in version 8.12.12 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve wikipedia retrieval mechanism

Replace manual wikipedia wrapper that we have implemented with wikipedia package
https://pypi.org/project/wikipedia/

Closed in #109