CIO Tools command unauthorized

I am trying to use this command
npx cio-sdk-tools@latest send-push --api-key API_KEY --token DEVICE_TOKEN --platform DEVICE_PLATFORM
I get this response
{"timestamp":"2024-01-24T16:40:32.294Z","level":"error","message":"could not send push","errors":[{"detail":"unauthorized","status":"401"}]}
In the workspace we have track api keys (which contains siteId and api key) and there is a APP Api key section that contains and app api key. Does it make a difference which api key is being used here. Am I supposed to be using one or the other?

Hey there. Sorry to hear you're having an issue using the tool. Let me see if I can help.
Before I answer your questions, may I ask, are you running this exact command on your machine:
 npx cio-sdk-tools@latest send-push --api-key API_KEY --token DEVICE_TOKEN --platform DEVICE_PLATFORM

Or, are you replacing the placeholders with real values such as:
 npx cio-sdk-tools@latest send-push --api-key "your-app-api-key-here" --token "your-device-token-goes-here" --platform "ios"

Does it make a difference which api key is being used here

APP Api keys do have restrictions on them. I suggest double-checking the settings for your key to see what workspace it works on and if IP whitelisting is enabled or disabled.

I am running the command with the real values

Oh, my apologies. Now I understand your question.
Yes, there is a difference between using the track api key and an app api key.
For using the send-push command, an app api key is required. You can visit https://fly.customer.io/settings/api_credentials?keyType=app to get your API key.

I can see how this is not clear in the README.
I'll make a change to the doc to make this more clear.

Thanks! that is working. I have another customer.io related issue with images not working. Is there a way I can reach out to you?

Glad to hear that helped!
I suggest sending a message to<EMAIL_ADDRESS>about your images issue. Even beyond images, you're always welcome to send any question you have there. You will receive the best help there.
I'll close this issue since your having success. Have an awesome day!


Updated Readme

Just creating PR to check if you are okay with linking the Slack - and some other changes - in the Readme @NicklasWallgren.
@nickpoulos suggested it in #142.

On Fri, Oct 7, 2016 at 4:36 PM, NicklasWallgren<EMAIL_ADDRESS>wrote:

Merged #152 https://github.com/NicklasWallgren/PokemonGoAPI-PHP/pull/152
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NicklasWallgren/PokemonGoAPI-PHP/pull/152#event-816764313,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADxq6T4OC_Wul3HXOKTRpIdSm9UeN2x2ks5qxq04gaJpZM4KRPqE
.

--
Nick Poulos
Lead PHP Engineer
Check Out LittleThings.com http://littlethings.com/


Port LineBasedFrameDecoder and DelimiterBasedFrameDecoder to Buffer API

Motivation:
Helps to have a pipeline working only with Buffer API.
Modification:

Port LineBasedFrameDecoder and DelimiterBasedFrameDecoder to Buffer API.
Adapt the junit tests

Result:
LineBasedFrameDecoder and DelimiterBasedFrameDecoder now are using the Buffer API.

Thinking loud: If delimiters become read-only there are some interesting optimization to optimize the change from slice -> copy in this PR to make the copy to not copy the source content, sharing it

Thinking loud: If delimiters become read-only there are some interesting optimization to optimize the change from slice -> copy in this PR saving the actual copy of the source content

Do you mean the new copy method with read-only?
Also I see this to be used a lot
Unpooled.copiedBuffer("TestLine\r\ng\r\n", Charset.defaultCharset());

and I extracted a method for this, may we can turn this into utility
    private static Buffer copiedBuffer(BufferAllocator allocator, String str, Charset charset) {
        return allocator.copyOf(str.getBytes(charset));
    }

@chrisvest Also used similar helper method in the change for Base64 encoder/decoder

@violetagg thanks!


kubernetes-helm/monocular#412 Add Values in API

kubernetes-helm/monocular#412 Add Values in API

Codecov Report

Merging #415 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #415   +/-   ##
=======================================
  Coverage   84.47%   84.47%
=======================================
  Files          26       26
  Lines        1462     1462
=======================================
  Hits         1235     1235
  Misses        176      176
  Partials       51       51

Continue to review full report at Codecov.

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

I have Rewrited code in new repo

@prydonius done

!

It will be awesome!

Muchas Gracias !
:satisfied:

As we start working on Monocular 1.0 (read the proposal: https://github.com/helm/community/blob/master/proposals/monocular-1.0-improvements.md), we will be removing in-cluster features from Monocular to keep the project focused on the public search and discovery site experience (e.g. hub.kubeapps.com and hub.helm.sh in the future). For in-cluster features, I recommend you look into Kubeapps or possible the RedHat Automation Broker.
Thanks a lot for the contribution!


Consider OCDS alignment

Once the project is delivered, and before any future major development, we should consider whether the country models (Tender, GoodsServices, Buyer, Supplier) can more closely conform to the OCDS model.
We can rename the models and fields in the code, and set the db_table class Meta option and db_column Field option to the old names, to avoid having to rename database tables and columns.

Not going to happen.


Fix/change config publish path

Changing config publishing directory to be the standard config directory instead of using vendor sub directories.
I believe in Laravel 5.0 default publishing directory was just standard config directory instead of using vendor directories.

I have intentionally opted for vendor sub-folder at this time, but I will think this over and see if it makes sense for my goals.
On the one hand, I understand that for one-offs this is fine, but I have a collection of my packages and would not like to make a huge list of files in the config folder, but instead have the config files for my plugins in their own directory.
I will think on this one as well, thanks for posting a PR for this, either way. :) I appreciate all input.

I guess it comes down to personal preference. I create a lot of packages for the systems I'm building and just prefer everything in the main consign directory, but if you prefer the vendor approach it certainly doesn't cause any problems.

Thanks ... will mull it over over the next few days. :)


Default extend directions to 0

Allow any extend direction to be omitted (falsey) for a default of 0.

The way I wrote it, every parameter has to be falsey or valid, and then the falsey ones default to 0. So it already achieves independent setting without individual if statements. You may prefer individual if statements stylistically, though.
I changed it, but I may have committed several new style violations in the process.
Also, my new pull request requires parameters to be undefined or valid, and throws more specific errors.
I don't know how to add unit test coverage. This was my first pull request.

Coverage decreased (-1.1%) to 98.857% when pulling 5e9306187ab5a201b8873e7e56543b8a02a95642 on Pulau-Komodo:patch-1 into bc7ab296ef1fb0097dbcafa0d146e8725e339326 on lovell:master.

Have a look in https://github.com/lovell/sharp/blob/master/test/unit/extend.js for existing unit tests that cover the extend operation. When you run npm test locally you'll see a coverage directory appear with full HTML reports on which new code branches require test coverage.


Magnet Card does not work

When putting the Magnet Card in the Wireless Crafting Terminal nothing happens. Items laying around are not attracted and just go into inventory normally. There is also no key to activate it and key combinations are also not activating it. As README.md (should be updated to this mod and not the fabric one) says, it has also no filtering which is expected.

fixed in 0.1


Form Builder

Maybe create a form builder that extends from the form builder from action view and implement a method like inputs_from_json. Example:
<%= form_for @engine, builder: JsonFormFieldsFormBuilder do |f| %>
  <%= f.text_field :name %>
  <%= f.inputs_from_json :prefix, json %>
  <%= f.submit "Create" %>
<% end %>

Do you can show the HTML output?


Redundant polyfills

Every plugin created with cli adds lot's of polyfills, so they repeat in project scope and create a large redundant boilerplate.
We should exclude polyfills and project developers can add manually https://cdn.polyfill.io/v2/polyfill.js?features=Array.prototype.map,modernizr:es5array|always if needed.

I think this isn't a large amount of memory.
In any case, you can import and minify a *.debug.js bundles

I think this isn't a large amount of memory.

But it's ~6kb. 5 plugins and 30 kb, whats is size of some complete library :/. What is 24 of redundant code.

Can you try to minify *.debug.js bundles while building app?

@trsh what build system are you using? (webpack, rollup, etc)

@Ni55aN i basically cloned vue rendering plugin from git! So is use same as there. United style :)))

@Ni55aN please check this https://medium.com/@addyosmani/the-cost-of-javascript-in-2018-7d8950fbb5d4 or google something better on the topic. Size does matter a lot for smooth experience. Nothing comes for free. If its tons of JS lines, js interpreter will need to process/compile on run-time.  If / src takes 5 kb
and uses some resource loaders (sass, vue.js), the package size after minimization may take more than 5 kb (how loaders work).
Solution: chunks + lazy loading.

@Ni55aN yes, but the polyis are redundant :P. Lots of projects even don't support not modern browsers by default, so its lots ok KB for nothing. Going with debug js or npm and rolling own packer is not for everybody. But it's your choice, so im closing this.


Reconnect not working when the connection was not valid in the first place

When Margin Loader is started and cannot connect to the AMQP broker at first try, it seems to not attempt any reconnects but keeps running doing nothing. It should either try to periodically reconnect or to showdown and let Kubernetes handle it.

It seems that the reconnect doesn't work even when the broker is restarted in the middle. I seems that I have to restart the Margin Loader pod to get it connected again.


oneFingerPan option doesn't work.

I do not have margins set, but still I must use 2 fingers to pan on any of my mobile devices (iPhone, iPad & MS Surface).

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Refs:

https://codepen.io/etipaced/pen/VwvLggo

I'm not so sure about the Codepen I posted. I can't get it to not work on my phone. In fact, even when I disable oneFingerPan but setting it to false, one-finger panning still operates just fine.
In my use-case, however, I cannot get one-finger panning to work at all. FYI, I do have a percentage-sized SVG (vs. pixel-defined size) so maybe that's the problem?

I tested it on my phone and it worked just fine. So it might have something to do with your phone since I can only test on android

I figured out how to get the Codepen working properly on my phone. The oneFingerPan option does work as expected. I'm closing this as there is obviously something wrong with my specific implementation, which I'll diagnose separately.


Supporting Cobalt

Is your feature request related to a problem? Please describe.

Cobalt ( https://cobalt.dev ) currently doesn't work with Shaka out of the box, due to a few minor missing APIs that currently throw JavaScript exceptions.
Describe the solution you'd like
I have a small patch that stubs out the usage of video.textTracks and document.createNodeIterator that would disable textTracks and some XSS security guards in Shaka.
Describe alternatives you've considered
We could also try and add stub implementations of those web APIs in Cobalt quite easily, and we likely will. However, that wouldn't let all the deployed versions of Cobalt to run with Shaka.
Additional context
Preview of quick 15-minute changes here ( i would remove the lint suppressions, of course ):
https://github.com/kaidokert/shaka-player/commit/f5167033cccded38dc27e12b863aac7d9e0b993d

I have reviewed the link, I agree with including the changes in lib/util/xml_utils.js, but I think the changes in lib/text/simple_text_displayer.js should be to make a new text displayer that does nothing, what do you think?

Yep, i think there was only one call site for the textDisplayer so that change would be cleaner and would throw less surprises.

I have created https://github.com/shaka-project/shaka-player/pull/5805 to solve the problem of lib/util/xml_utils.js
For the other change I am willing to review your PR if you are interested in sending it, I think the new class should be called StubTextDisplayer (lib/text/stub_text_displayer.js), then the change would only be to add a new if in https://github.com/shaka-project/shaka-player/blob/main/lib/player.js#L5441

Thanks much for such a quick response - i'll put a PR together and send it.


Add reset flag to make signature drop single phase consistent

@nachoiacovino can u give context for this / how to test?
not sure I understand what is going on in this PR :)

Right now, you cannot save an empty claim phase on Signature Drop, you need to reset claim eligibility and then remove, this fixes that.
Also, this fixes a problem in which Signature Drop behavior is not consistent with NFT Drop.
Example on NFT Drop:

I set it so 200 can be claimed.
100 are claimed.
I click the bin button, and I set it so 1000 can be claimed.
Now 1000 can be claimed.

These same steps on Signature Drop would make it after setting to 1000, only 900 can be claimed, since it's the same phase, but this PR makes it so clicking the bin (removing and "setting" a new one) would make it so it actually resets and you can claim 1000 (like people would expect)


[INFRA] Streamline libraries

Documentation preview available at https://docs.seqan.de/preview/seqan/chopper/255

Looks alright to my limited cmake knowledge. I trust you on that. Can you add a comment about why you did these changes? I don't see the immediate benefits.

It just looks more like you would modern cmake to look like (namespaces library aliases, etc.)


Go back to using the default Gtk.IconTheme

Creating another IconTheme will make it sometimes fail to find
a given icon. It is better to use the default one.
To avoid having problems when multiple threads access that same
object, we will make sure that the main one will do that by using
the fallback image as the placeholder (the image that gets loaded
while the thread loads the real one).
This is actually nicer since the placeholder for an image will be
a image mimetype icon instead of a generic text one.
==
@tomanderson2 this is to fix some issues that I noticed when writing https://github.com/viaforensics/viacore/pull/97

:+1: assigned to @bosonogi for code review

:+1:


Add Tests

We could use simple jest tests for common scenarios like ensuring the flags work correctly and a "happy path" through the program. We should also have tests against a wider variety of .yaml examples just to make sure we're always producing valid YAML.

done


add microshift check

the microshift section was added to advisories group and lead to failure like
https://github.com/openshift/ocp-build-data/pull/2417#issuecomment-1376731132

/lgtm


GCP Identity Aware Proxy interoperability.

AFAICT the acme-challenge urls will be blocked if you enable IAP on ingress this software creates a cert for.
Is there any way to make these two play nicely together?

I don't believe this is an issue, Lego creates a backend service which it appends to the ingress resource. This means unless you enable IAP on this addition backend you should be fine.


onRender filtering doesn't gray out times

The example.html gives the following to disallow the user selecting certain dates:
var checkin = $('#dpd1').fdatepicker({
    onRender: function (date) {
    return date.valueOf() < now.valueOf() ? 'disabled' : '';
}
...

If the date picker is created with pickTime: true, then this function doesn't appear to be called when selecting times, and no times are grayed out. I think it would make more sense if times were able to be grayed out also, simply by calling the same function.

The onRender callback should return the current viewMode also. So you will know exactly in which view the picker currently is and you can have different "disable" functions for the same callback.
E.g. if you don't want the user to select any date & time < now, you should not disable 1:00 if current time is between 1:00 and 2:00.


Some problems about Upload large files with streaming

I want to upload one large file(about 4GB) via http and save it on disk, so I read the guid and follow the document.
I tried the code Upload large files with streaming, but I found I still can't upload a single large file. Because the code try to copy section.Body to MemoryStream in the FileHelpers.ProcessStreamedFile  method. Obviously, this approach is not suitable for uploading a single large file.
This code try to upload multiple files to the serve, and there were multiple files in the request body, so it read every section of request body. Files may be large when they together.
So I think the document is not inaccurate, because it has not provided the method to upload large file, at least not a single large file.
The document description should adjust to Upload multiple files with streaming to avoid misunderstandings, and if want to upload a single large file, the stream should copy directly to a FileStream
PS：If the author of the document thinks that a large file is a file that does not exceed the size of the MemoryStream , it may be noted in the document

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

ID: 9a5cfa38-0e6c-4346-dafc-84ea5c513fcf
Version Independent ID: c6787f0f-3e9a-6efb-0e6e-5ccb957d3f4a
Content: Upload files in ASP.NET Core
Content Source: aspnetcore/mvc/models/file-uploads.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @GadHao ... We don't cover every scenario in doc examples, but this is good feedback on this edge case ... the super-really-BIG file upload  😄. It's apparently not too common ... at least not among devs who work out that they should just adopt a different approach and not report feedback to us. The article has been up in this form for a few years now, and you're the first dev to ask about this.
@Rick-Anderson ... 'The stream should copy directly to a FileStream' is a good idea, and there's the possibility of even merely calling it out in the Server and app configuration section because that's where we keep the various limits that come into play when dealing with large files (e.g., body length limit, Kestrel limit, etc.). I can work on this issue if you want, but it will probably take me until after 6.0 GA to reach it due to its low priority.
@GadHao ... The good news here is that even your issue will help other readers because they'll see it attached to this doc (unless it's closed as a won't fix) or we can refer a reader to it who creates an issue about this scenario. Thanks for commenting on it. Rick will decide the best course of action to take.

@guardrex @Rick-Anderson Thank you for considering the issue, I just think that because everyone thinks that Microsoft Docs is best practice, at least close to best practice. Therefore, it is necessary to remind them that when uploading particularly large files, memory consumption should be considered. When they have to do that, they can use a method similar to directly copying to FileStream to avoid possible difficulties.
I'm gald to do something helpful, such as providing relevant code examples.

@GadHao  if you can write a sample and PR it to https://github.com/dotnet/AspNetCore.Docs/tree/main/aspnetcore/mvc/models/file-uploads/samples/5x/largeFiles that would be great. I'll get the development team to review the sample and help you update the doc.

what the point doing streaming (in context of sample code, because you copy the whole thing to memory in the end), why not just use the buffered model (which also support multipart)
its also a bit lazy to ask someone else to write documentation for you, i don't think this is the purpose of open source, its abusing good intent of people

In the first version (years ago) of that sample code, there was no FileHelper
Upload large files with streaming --> Rename --> Upload files with streaming/multi part content
[ValidateAntiForgeryToken] ---> this is not needed in web api (without view) scenario?
When handling real big file, IsValidFileExtensionAndSignature --> this method is not feasible anymore, just write the file first and validate later?


Interpreter constraints honored even with regular PATH

Solves issue #656, where if no PEX_PYTHON_PATH and no PEX_PYHON are configured, we still honour --interpreter-constraint flag when bootstrapping a PEX.
There is an integration test that worked only when applied the last commit, but there is a lot of nuance to running it that I'd like a hand with. In particular, I'd like to know if there's a way of:

Specifying "please ignore system-wide settings for PEX_PYTHON_PATH" in a test or Specifying
"please override PEX_PYTHON_PATH with None". I think the first one should be easy to implement with a command-line flag defined in the same place as --rcfile.
Specifying "This test only runs on Python3.5", because I need two different interpreters to play with (one for the environment that the pex runs in, another one to pass to --interpreter-constraint).

@blorente for point #2 you want this:
https://github.com/pantsbuild/pex/blob/edc636ce6a004fa1bfc762d74f01c9fd3a862429/pex/testing.py#L324-L366
For point #1 you should just be able to run pex with an env=... that has what you want. And part of what you want is maybe:
https://github.com/pantsbuild/pex/blob/edc636ce6a004fa1bfc762d74f01c9fd3a862429/pex/variables.py#L322-L328

Depends on #673

For some reason I can repro failing test (and others) locally in master, even after blowing up ./.tox. I have no idea what's up or why they don't show up in other places. I will keep investigating further, but ideas are welcome.

@jsirois, any idea why I can repro the failing tests in master? Meaning, how they made it past CI in the first place.

@jsirois or @CMLivingston a review of this would be quite helpful. While I'm more familiar with Pex than a few months ago, I'm still not aware of all the expectations and its history.

@jsirois, any idea why I can repro the failing tests in master? Meaning, how they made it past CI in the first place.

In my experience, this happens because certain env conditions align locally that are not covered in CI, and the developer did not bother to run tox locally before pushing the branch.


Split the OwnedHandle's into a separate generator

This will reduce complexity of the other safehandles for future changes to support foreign record types that require manual implementation.
Related to PR #578

Thank you, good work :+1:

good!


Confusing Privacy Levels disabled / enabled

Why do you say that (#1) 'Combine data according to your Privacy Level settings for each source' means that 'Privacy Levels is not enabled' while (#2) 'Ignore the Privacy levels and potentially improve performance' means 'Enabling Privacy Levels' (you do it in a consistent manner across the file).
Intuitively, having privacy levels being considered (as in #1) feels more like 'Privacy Levels is enabled' while not having these considered (as in #2) feels more like 'Privacy Levels is not enabled' - is it possible that any clarification is missing? or that I'm misreading something ..

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

ID: eb7743c4-abb7-9c9d-6089-eca63ba9ac94
Version Independent ID: c5bfefde-15ff-683c-5a79-e7210b8bf984
Content: Understand Power BI Desktop privacy levels - Power BI
Content Source: powerbi-docs/desktop-privacy-levels.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi, @yoyogin -- thanks for asking! I've referred your question to the author, who will reply with more info.
BTW, the numbers you've included, "#1" and "#2", end up being links going to different GitHub PRs. But I think I see now, you're referring to the 2 different sections of the article?
thanks!

I've updated my original comment (added spaces) - didn't mean to reference PRs but to reference terms showing in the first paragraph of my comment from the second paragraph.

Thanks for posting @yoyogin - and sorry if that's not clear. We believe the document is correct.
For more help and clarification on this, your best venue is the community (https://community.powerbi.com) or from support at https://support.powerbi.com. You can also provide feedback using the Give Product Feedback button at the bottom of the article page.


Support Zod 3.20

The new version of Zod brings new schema features which will need corresponding support in zod-fast-check.
https://github.com/colinhacks/zod/releases/tag/v3.20

The newly released version 0.9.0 add support for the new .pipe and .catch methods. A later release will add support for z.symbol(), z.string().datetime() and z.number().finite().

Hi Deniz. Which of the unsupported features are you using? I can prioritise adding support for that.

Hi Deniz. Which of the unsupported features are you using? I can prioritise adding support for that.

Hi! Thanks a lot, that would be awesome! It's primarily pipe that I'm making heavy use of right now.

pipe should be supported by the zod-fast-check 0.9. If it isn't, could you provide examples of one of your schemas?

@DavidTimms Sorry, I missed your other comment about .pipe working now! Sorry for the noise.

This is resolved with the release of zod-fast-check 0.10.0.


numpy.fromstring deprecated

numpy.fromstring is used in NSD(). Unfortunately, the binary mode is deprecated. It also results in a warning in IDLE. See what can be done about this.

This was opened in error.

This was opened in error.


[rayci] add smarter dependency tracking

when a node is rejected/disabled, also disables all its dependency nodes
when a ndoe is included/selected, also includes all its depended nodes

ping; please help review.


Broken on 0.3.15 latest release

Looks like affected by same issue. https://github.com/progrium/dokku/issues/906

Also seeing this and unable to rebuild an app with persistent volumes. Reading through the issue at progrium/dokku#906 I can't tell if this is supposed to be fixed or not... The issue was closed back on Feb 5 and the commit is tagged v0.3.15.
However, I'm currently running v0.3.15 and can't build/rebuild if I have volumes set up on an app.

This fork has fixes https://github.com/alessio/dokku-volume

Confirmed. The version at https://github.com/alessio/dokku-volume does work properly.

Fixed with last merge, thanks to @alessio


Lint JavaScript code

Also be sure not to merge until Kevin (or you? Did you see it?) checkout the comment I left in bindings.js

Yeah I saw the comment.  I don't personally know the answer.  I will let @kklin merge when he's looked at it.

I updated this to rebase on top of my recent change to fix the containers issue, so this should be a ll set


Adding multiple worker pools to cluster

Closing the PR as we can use for_each on template side to create multiple worker pools


[TF2.0rc]TFRecord file size bigger than original csv file consisting text data.

I am trying transform a csv file consisting two columns: short_description, label. Description consists sentences from length 1 to 4 and label is just an integer number. The csv file has a size of 740 MBs however on converting it to TFRecord the size of the new file increased to 1.2GB, which I think is not correct since TFRecord files are supposed to have lesser size.
For short_description (column 1), I am using BytesList and for label (coulmn2) I am using Int64List as was shown here in this tutorial. Just to be clear, the data read from the new TFRecord file is fine i.e. no problem. It is just the size of the file. Now I do not understand what the problem is. Please help. Thank you!

Request you to provide standalone code and sample data to reproduce the issue. Thanks!

Here is the link to the small part of big csv file (428.9KB). Following is the script used for transforming it into TFRecord, giving the tfrecord file of size 710KB.
from __future__ import absolute_import, division, print_function, unicode_literals
import sys
import tensorflow as tf
import numpy as np
import pandas as pd

dataset = tf.data.experimental.make_csv_dataset('random.csv', batch_size = 1,select_columns = ['short_description','Label'], label_name = 'Label', num_epochs = 1 ,shuffle = True)# (many examples, many labels) pairs

# creating tfrecord
def serialize_example(string_input, integer_input):# Serializing every element
	string_input = string_input.numpy()
	string_list = tf.train.BytesList(value=string_input)
	feature_strings = tf.train.Feature(bytes_list=string_list)
	int_list = tf.train.Int64List(value = [integer_input])
	feature_int = tf.train.Feature(int64_list = int_list)

	#make Feature value as DICT
	elements_dict = {
	  'strings': feature_strings,
	  'Ints': feature_int
	}
	features_dict = tf.train.Features(feature=elements_dict)
	example = tf.train.Example(features=features_dict)
	return example.SerializeToString()

def tf_serialize_example(f0, f1):
	with tf.device('CPU:0'):
		tf_string = tf.py_function(serialize_example, inp = [f0['short_description'], tf.reshape(f1, [])], Tout = tf.string)
	return tf_string
serialized_dataset = dataset.map(tf_serialize_example)# to serialize every element in the dataset

filename = 'random1.tfrecord'# name of the tfrecord file
writer = tf.data.experimental.TFRecordWriter(filename)# writing the tfrecord file
writer.write(serialized_dataset)

I used Python 2 and TF2.0rc.

I have tried on Jupyter notebook with TF2.0rc0 and was able to reproduce the issue.Thanks!

@ravikyram, could you try reproducing the behavior with TF 1.X, I suspect that this is not specific to TF 2.0.
@rishabhsahrawat what leads you to believe that an uncompressed TFRecord file should be smaller than the equivalent csv file? Unlike csv, TFRecord format duplicates the feature key in each record, so I would expect the uncompressed TFRecord file to actually be bigger.

Hi, according to the docs I got the understanding that TFRecord usually
takes less space. If that is not true then it must be written in the docs.
So, all the cases when it will take more space and less space.
Also, as you are saying that the resulted TFRecord is uncompressed file
then how does one get the compressed file with smaller size and can that be
read or does it need to be first uncompressed and read? Thank you for your
valuable time.
On Wednesday, September 4, 2019, Jiri Simsa<EMAIL_ADDRESS>wrote:

@ravikyram https://github.com/ravikyram, could you try reproducing the
behavior with TF 1.X, I suspect that this is not specific to TF 2.0.
@rishabhsahrawat https://github.com/rishabhsahrawat what leads you to
believe that an uncompressed TFRecord file should be smaller than the
equivalent csv file? Unlike csv, TFRecord format duplicates the feature key
in each record, so I would expect the uncompressed TFRecord file to
actually be bigger.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/32075?email_source=notifications&email_token=ADZMMPHL4FHVUXBOIYA5WPTQH7SV7A5CNFSM4IR62YAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD54HXVQ#issuecomment-527989718,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADZMMPFXBNI3TLBHFAPNC23QH7SV7ANCNFSM4IR62YAA
.

--
Sent from Gmail iPhone

Could you point me to the documentation that suggests that TFRecord should take less space then a CSV file?
TFRecordWriter supports the same compression types as TFRecordDataset --  no compression, GZIP, or ZLIB -- specified through the compression_type argument.

I read about it here (
https://medium.com/mostly-ai/tensorflow-records-what-they-are-and-how-to-use-them-c46bc4bbb564),
second paragraph. Although I could not find something similar on TF docs.
As you mentioned earlier the size is expected to be bigger than the csv, so
do you think it is still beneficial to use TFRecords data for very large
files?  Thank you!
On Thursday, September 5, 2019, Jiri Simsa<EMAIL_ADDRESS>wrote:

Could you point me to the documentation that suggests that TFRecord should
take less space then a CSV file?
TFRecordWriter supports the same compression types as TFRecordDataset
https://www.tensorflow.org/api_docs/python/tf/data/TFRecordDataset --
no compression, GZIP, or ZLIB -- specified through the compression_type
argument.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/32075?email_source=notifications&email_token=ADZMMPC3ZERQWBJB5EJ6R4DQIAZ6JA5CNFSM4IR62YAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55G5UI#issuecomment-528117457,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADZMMPHKBWWZZO3GBGZFFJDQIAZ6JANCNFSM4IR62YAA
.

--
Sent from Gmail iPhone

If you compress the file, I would expect it to be smaller. Additional benefits of using TFRecord for large dataset is that you can process the data in a streaming fashion (e.g. with tf.data) as opposed to loading the entire dataset into memory which it might not fit.

Okay, I understand. So, if I store the compressed TFRecord file then can I
read it using tf.data.tfrecord as it is or do I have to uncompress it?

Processing the data in streaming fashion
I think that will be similar to when loading a csv file (e.g. with
tf.make_csv_dataset function) and processing it with tf.data, except maybe
in terms of reading speed of the data. Please correct me if I wrong.
Before trying loading data from TFRECORD, I used tf.data to load my csv
file. If I want to use .shuffle function on this then it fills up the
shuffle buffer before every epoch even though I have set
‘shuffle_before_iteration=False’ and this buffering also eats out my
device’s RAM which eventually run out of memory. I have raised an issue
since months but still waiting for the solution. Thank you for your
valuable time and inputs.

On Thursday, September 5, 2019, Jiri Simsa<EMAIL_ADDRESS>wrote:

If you compress the file, I would expect it to be smaller. Additional
benefits of using TFRecord for large dataset is that you can process the
data in a streaming fashion (e.g. with tf.data) as opposed to loading the
entire dataset into memory which it might not fit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/32075?email_source=notifications&email_token=ADZMMPEUWOJ67PYIF5N3WQTQIFVFXA5CNFSM4IR62YAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6AVRFQ#issuecomment-528570518,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADZMMPFTIW4TRJWFTWVQYHLQIFVFXANCNFSM4IR62YAA
.

--
Sent from Gmail iPhone

You simply need to specify the compression_type argument of TFRecordDataset.
CSV format is amenable to streaming processing but other formats might not be.
As for your shuffle_before_iteration issue, I am not sure which API are referring. Neither CsvDataset nor make_csv_dataset have that argument. Did you file an issue for this? If so, can you please share a link to it here? Thanks.

Sorry for the confusion, I meant the ‘tf.data.Dataset.shuffle’  has a
parameter ‘reshuffle_each_iteration’ even after setting it to ‘False’ it
still shuffles the dataset before every epoch and fills up the buffer which
at some point leads to OOM error. This error sometimes also comes during
the first time it fills up the buffer or before the beginning of second
epoch. Here is the issue that I have raised.
https://github.com/tensorflow/tensorflow/issues/30646#event-2516513806
Thank you again!
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/data/Dataset#shuffle
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/data/Dataset#shuffle
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/data/Dataset#shuffle
On Friday, September 6, 2019, Jiri Simsa<EMAIL_ADDRESS>wrote:

You simply need to specify the compression_type argument of
TFRecordDataset.
CSV format is amenable to streaming processing but other formats might not
be.
As for your shuffle_before_iteration issue, I am not sure which API are
referring. Neither CsvDataset
https://www.tensorflow.org/api_docs/python/tf/data/experimental/CsvDataset
nor make_csv_dataset
https://www.tensorflow.org/api_docs/python/tf/data/experimental/make_csv_dataset
have that argument. Did you file an issue for this? If so, can you please
share a link to it here? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/32075?email_source=notifications&email_token=ADZMMPCHT5HDCTUJOIONGF3QIKK7ZA5CNFSM4IR62YAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6DSQOY#issuecomment-528951355,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADZMMPFVNWUKIT5VABCBSEDQIKK7ZANCNFSM4IR62YAA
.

--
Sent from Gmail iPhone

reshuffle_each_iteration does not control whether elements should only be shuffled every epoch (or only the first epoch). What it controls is whether the order in which elements are shuffled is different (or same) every epoch.
The error on #30646 does not seem related to tf.data as not allocating memory on GPU. I suggest you create a new issue for your problem with minimal reproducible example and reference it here.

HI @jsimsa , here is the new issue. Please let me know if you have any comments or suggestions. Thank you very much!
-Rishabh Sahrawat


Update README.md

I added my favorite one your repository.
And I'm grad you'll come here to tell me your favorite yuru-chara!!
Thank you!

I added!
Thank you!


[Enhancement] Normalize EOL symbols in file

Currently (4.10.0) EOL symbols in output are operating system-dependent, which can clash with git checkout settings for example.
It would be nice to have an option to force them to be LF (Unix-style) or CRLF (Windows-style) manually.

What about lineseparator option?
https://robotidy.readthedocs.io/en/stable/quickstart.html#line-endings
By default it uses native system eol but it can be switched to auto detect or overwrite with specific.


⚠ Cioos National CKAN has degraded performance

In 34e9fb0, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 6487 ms

Resolved: Cioos National CKAN performance has improved in 4ab4781 after 7 minutes.


Export missing signature types

#2 introduced some types, but didn't export them.  This PR adds those types, and adds a test.

Thanks for the quick merge.  I've updated my tutorial to use this version of the bindings https://gitlab.com/uhoreg/matrix-tutorial/-/commit/2fddb7f5fef78b47eb76aa76ab6daaedca34f9e5  I'll do more updating to make it work with #6


Added ability to use a keystore for the pact mock server #241

Added ability to use a keystore
Updated JUnit docs and included tests.

That's awesome, thanks for the PR.


[TC-MC-1.2]Keypad input cluster--TH fails to read all the features from DUT using feature-map attribute.

Problem
As per Spec, TH should read all the features from DUT but currently TH fails to read all
the features from DUT using feature-map attributes.

expected behavior:- TH should read
Bit 0 - Set to 1 if the DUT supports Navigation Keys (PICS_NV_S is true)
Bit 1 - Set to 1 if the DUT supports Location Keys (PICS_LK_S is true)
Bit 2 - Set to 1 if the DUT supports Number Keys (PICS_NK_S is true)

actual behavior:- Bit 1 - Set to 1 if the DUT supports Location Keys (PICS_LK_S is true)

steps to reproduce:-sudo ./chip-tool keypadinput read feature-map 1 1

system configuration:-
Chip-Tool: RPI
DUT: RPI
Platform : IP and Wifi
Commit-ID: 3a7f03a0a1c58c4f6cf69ea7e6b808feff3b6636
Dut: TV-app

Test-Plan reference:-
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/media.adoc#tc-mc-1-3-global-attributes-application-launcher-cluster-dut-as-server
Spec Screenshot:-

PFA log below:-
[TC-MC-1.3]Feature map.txt

Facing the same issue in channel cluster test-case[TC-MC-1.6].
expected behavior:- TH should read
Bit 0 - Set to 1 if the DUT supports Channel Lists (PICS_CL_S is true)
Bit 1 - Set to 1 if the DUT supports Lineup Info (PICS_LI_S is true)
actual behavior:- Bit 1 - Set to 1 if the DUT supports Location Keys (PICS_LK_S is true)
Test-Plan reference:-
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/media.adoc#tc-mc-1-6-global-attributes-channel-cluster-dut-as-server
PFA log below:-
TC-MC-1.5.txt

Hi Lazar, can you confirm whether this is a TH issue or a tv-app issue? I know that you were working in the feature map area recently.

Facing the same issue in Content launcher cluster.
expected behavior:- TH should read
Bit 0 - Set to 1 if the DUT supports Content Search (PICS_CONTENTSEARCH is true)
Bit 1 - Set to 1 if the DUT supports URL Playback (PICS_URLPLAYBACK is true)
actual behavior:- Bit 1 - Set to 1 if the DUT supports Location Keys (PICS_LK_S is true)
Step to reproduce:-
sudo ./chip-tool  contentlauncher  read feature-map 1 3
Test-plan reference:-
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/media.adoc#tc-mc-1-11-global-attributes-content-launcher-cluster-dut-as-server
PFA log below:-
TC-MC-1.11.txt


Destructive sorting in _doApplyEdits

have a execution that uses the clear/append sequence as shown below
:bug: you have no output

The problem is that the sorting _doApplyEdits breaks the order. Instead doing a clearOutput and then appendOutput is does appendOutput and then clear. See the capture from the debugging: rawEdits is correct, but edits if wrong

        for (let cell of cells) {

            // shuffle text
            const rndOut = shuffle(cell.document.getText())
            const exec = controller.createNotebookCellExecutionTask(cell)

            exec.executionOrder = _execCount++
            exec.start();
            exec.clearOutput();

            exec.appendOutput(new vscode.NotebookCellOutput([
                new vscode.NotebookCellOutputItem('text/plain', rndOut),
                new vscode.NotebookCellOutputItem('application/foo', rndOut),
            ]));

            exec.end({ success: true });
        }

I have pushed a skipped test that shows this. I am unsure to best fix this. I see how edits are reverse because that makes sense for cell adding/removing but for outputs this is rather bad.

It was fixed now. To test you can write a controller for an existing notebook you have
	let _executionCount = 0;
	const controller = vscode.notebooks.createNotebookController(
		'test-kernel',
		'mimetype-test', // notebookType
		'Kernel TPI',
		(cells: vscode.NotebookCell[], document: vscode.NotebookDocument, controller: vscode.NotebookController) => {
			for (let cell of cells) {
				// shuffle text
				// const rndOut = shuffle(cell.document.getText())
				const exec = controller.createNotebookCellExecution(cell);

				exec.executionOrder = _executionCount++
				exec.start();
				exec.clearOutput();
				exec.appendOutput(new vscode.NotebookCellOutput([
					vscode.NotebookCellOutputItem.text('123')
				]));

				exec.end(true);
			}
		}
	);

Adding verified label, since Joh's skipped test was re-enabled two months ago.


Type definition misconfigured

First of all, thank you for creating the module!
I started using your zod-to-json-schema and json-schema-to-zod. While the code looks to be working, my VSCode displays errors.
"json-schema-to-zod": "^2.0.14",
"typescript": "^5.3.3",
"zod": "^3.22.4",
"zod-to-json-schema": "^3.22.4"

import { jsonSchemaToZod } from "json-schema-to-zod";
import { z } from "zod";
import { zodToJsonSchema } from "zod-to-json-schema";

const testSchema = z.object({
  name: z.string(),
  age: z.number().optional(),
});
const jsonSchema = zodToJsonSchema(testSchema);
const zodSchema = jsonSchemaToZod(jsonSchema);  // <-- 1st type error with "jsonSchema"
console.log({
  jsonSchema,
  zodSchema,
  properties: jsonSchema.properties,  // <-- 2nd type error with "properties"
});

The result of the console.log is below:

The variable jsonSchema has properties that JsonSchemaObject type expects in json-schema-to-zod.

Hi, thanks for opening an issue.
The first problem is valid. jsonSchemaToZod should accept the output types of zodToJsonSchema. I will look into this... at some point. ;)
The second problem (the "missing" properties) is not a bug. The output will not have inference, like the Zod schema does (meaning the return type just says that it's a JSON Schema, not what the schema contains). Since not all JSON Schemas have "properties" you have to check for it manually.


Multiline inputs resize on any input element change, slowing down entire form

Forms containing a multiline element are nearly unusable at this moment (beta 6) as any input on any form field causes the multiline element to rerender, which includes a very expensive
element.style.height = 'auto';
element.style.height = element.scrollHeight + heightOffset + 'px';

inside handleAutoresize().

Any update on this? I'd love to use a multiline input in a few places but am unable to do so due to the above.
Thanks!


Rename feature/experimental/qos/ate_tests/qos_ecn_config_test/qos_ecn…

…_config_test.go to feature/qos/ate_tests/qos_ecn_config_test/qos_ecn_config_test.go
QoS: Move stable test from experimental folder to official folder

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 56.612%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1203

Relevant Lines:
2125

💛  - Coveralls


Remove own exp(x) - 1 implementation

Hello again! :grinning:
This PR removes the in-house implementation of the exponent minus one function and uses the Rust std version instead.
I tested the std implementation first in ba4c859d3089107b15436abf336868503599a58a and it had shown only one very minor numerical regression.

Pros:

less code to maintain
fewer modules (and tests) ⇒ hopefully faster compilation time

Cons:

nothing I'm aware of

Excellent work again! Really appreciate the help 👍


Regarding Test Sets for the GEM datasets

@yjernite Hi, are the test sets for the GEM datasets scheduled to be added soon?
e.g.
from datasets import load_dataset
DATASET_NAME="common_gen"
data = load_dataset("gem", DATASET_NAME)

The test set doesn't have the target or references.
data['test'][0]
{'concept_set_id': 0, 'concepts': ['drill', 'field', 'run', 'team'], 'gem_id': 'common_gen-test-0', 'gem_parent_id': 'common_gen-test-0', 'references': [], 'target': ''}

Hi @vyraun ! The test references for CommonGen are not publicly available: you can reach out to the original dataset authors if you would like to ask for them, but we will not be releasing them as part of GEM (March 31st was the release date for the test set inputs, references are incidentally released for some of the test sets but shouldn't really be used)
cc @sebastiangehrmann

Oh okay, thanks @yjernite !


Submitting the initial version of examples source code

This example project is developed following N-Tier architecture

Updated VisualBasic Project


Not able to scroll

Describe the Issue
current Behaviour is this as attached screenshot there is no scroll present due to which user not able to see full content of the page
Affected Page
https://forum.freecodecamp.org/t/freecodecamp-challenge-guide-serve-static-assets/301518
Steps to Reproduce

Go to 'any practice courses on freecodecamp' e.g = https://www.freecodecamp.org/learn/back-end-development-and-apis/basic-node-and-express/start-a-working-express-server
Click on 'Get a Hint button'
Then https://forum.freecodecamp.org will open and now you are not able to scroll down in this page and also not able to sign in and sign up e,g = https://forum.freecodecamp.org/t/freecodecamp-challenge-guide-serve-static-assets/301518
See error

Expected behavior
It should scroll and user able to see all the content
Screenshots

System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

Hello there,
I am unable to reproduce any issue with the forum UI/navigation.
Do you have any browser extensions that could be affecting the page? Would you mind filling in the System section of your issue description?

@ShaunSHamilton Are you able to scroll in this url = https://forum.freecodecamp.org/t/freecodecamp-challenge-guide-serve-static-assets/301518

@ShaunSHamilton Are you able to scroll in this url = https://forum.freecodecamp.org/t/freecodecamp-challenge-guide-serve-static-assets/301518

Yes.

@ShaunSHamilton I think you can close this it is not working in my pc but working in other pc I just checked and thanks for confirmation
Do you know what is the main reason of this in my pc?

Do you know what is the main reason of this in my pc?

Do you mind sharing your system info? Specifically, your User Agent could help debug.

@ShaunSHamilton

@Akshaybagai52 Thank you, and can you share what you see here: https://www.whatismybrowser.com/detect/what-is-my-user-agent/

Closing this as can't reproduce


Issue #763 DefaultClusteringService unit tests

This commit focuses on implementing unit tests on
DefaultClusteringService.  It also includes corrections for defects
discovered during unit test construction.

DefaultClusteringServiceTest is substantially extended; this was the
primary objective of this commit.
ObservableEhcacheServerEntityService is added to permit making
assertions about the EhcacheActiveEntity during unit tests performed
with the aid of PassthroughServer.  This class is intentionally
added in a package "owned" by the clustered/server module to permit
access to otherwise package-private details during unit tests.
DefaultClusteringService.create was creating and holding a client
entity (logical connection); this commit closes that entity and no
longer returns an EhcacheClientEntity from create.
EhcacheClientEntityFactory.destroy was implemented providing support
for DefaultClusteringService.destroyAll.
The EhcacheEntityBusyException and EhcacheEntityNotFoundException
are introduced.
ClusteringService.connect is removed; connect handling is entirely
internal to DefaultClusteringService.
EhcacheClientEntityFactory.createOrRetrieve was removed along with
the associated tests.
UnitTestConnectionService is amended to omit the "default"
ServerEntityServices and ClientEntityServices when one is provided.
This was done to allow replacement implementations for these
services.
EhcacheActiveEntity.destroy was to return the instance to a state
more closely resembling the initial instance state.

Rebased with review comment updates.


dragOverClass input

Describe the bug
dragOverClass input is not included in the source. I see it in the demo but after I've installed the project, it does not have it.

Minimal reproduction of the problem with instructions

Screenshots

Versions

This is only available in the 4.x beta's
npm i angular-draggable-droppable@beta

Hope that helps! 😄


Add safesql

I would like to add stripe/safesql to gometalinter -- it finds potential SQL injection vulnerabilities in database/sql calls.
However, safesql operates on packages, not files. It imports packages using golang.org/x/tools/go/loader and also takes the vendor folder into account.
Is there a simple way for gometalinter to pass the Go import path to a linter instead of the file/directory path?

Shouldn't be an issue. It looks like safesql accepts directories, so you can just use {path}. It is mind bogglingly slow though, so it will need to be disabled by default.


Correct method to build this using the go way

This is the way to build this project. It is fast and you don't need to do any manual work. If you try the old way that was described in this document you will have to do a lot of manual work to get all the dependencies.

Can one of the admins verify this patch?

OK to test

Hi @GabrielNicolasAvellaneda , thanks for taking the time to improve our documentation.
Next time, might I suggest you open one PR with three commits instead of three PRs with one commit apiece?

Hi @philipnrmn yes, I agree on this. I just used the edit feature of github directly on your repo and wasn't sure how to do that, next time will be better :+1:

Looks like our CI has had a hiccup and didn't run tests on this series of PRs, so I'm going to merge them manually. Thanks again @GabrielNicolasAvellaneda 🙂


Transfer to fortran-lang?

See discussion at https://github.com/fortran-lang/fpm/discussions/748
This project could be moved to @fortran-lang to officially provide a schema for the fpm package manifest. Alternatively, the files could be checked in as part of another project, e.g. the upstream fpm repository. Maintaining them here is of course also an option.
cc @milancurcic @certik @lkedward @gnikit

I think it would make things a bit easier to maintain if it was part of fortran-lang

Thanks both. This seems like a useful package and fine with me to maintain under fortran-lang.
Consider a rename: pyfpm suggests to me a Python binding to fpm. Perhaps a more descriptive name like fpm-package-schema or fpm-package-model?

Maybe fpm-metadata, similar to the former pep621 project which is now named python-pyproject-metadata.

Fine to move. Is it because Python is easier than Fortran? :)

Is it because Python is easier than Fortran?

The original idea was to provide Python projects a possibility to access the fpm package manifest. Candidates would be fprettify or ford should they decide to store their configuration in fpm.toml.
Once we decide on how to implement a schema library in Fortran the export of the schema can be provided directly by fpm (see discussion in https://fortran-lang.discourse.group/t/3963). Now that TOML Fortran has a JSON frontend there is now the possibility to implement a schema library which is immediately usable by fpm.
In fact this would be preferable because the schema produced by the pydantic model is imperfect and does not 100% reflect the currently allowed rules in the package manifest (the biggest issue is the equivalence between underscored and dashed keywords imposed by pydantic):
>>> from fpm.metadata import load_manifest
>>> load_manifest("name = 'hello'\nbuild.auto_executables = true")
Manifest(name='hello', version='0', license=None, maintainer=[], author=[], copyright=None, description=None, categories=[], keywords=[], executable=[], example=[], test=[], dependencies={}, dev_dependencies={}, extra={}, build=Build(auto_tests=True, auto_executables=True, auto_examples=True, link=[], external_modules=[]), install=Install(library=False), library=Library(source_dir='src', include_dir='include'))

FYI I am happy with the schema generator to be in any language we find easiest to maintain, the only important thing for vscode is the final product i.e. the JSON file. I will create a pre-release of the extension using the schema for fpm validation and see how that goes.

No, I like Python, especially with LPython now. :)

Done

If anyone has a PyPI account and want access to the project, let me know. I also made sure (with some trial and error), that tagging a new release will automatically upload release artifacts for GitHub and PyPI.

Do you think we can add fortran-lang's PyPi account as a maintainer?


Some bounding boxes have very large/small positive/negative coordinates

This could be due to extreme perspective distortions. Such boxes could potentially be filtered out.

@cjnolet is your code for synthtext paper public?

@ankush-me
Hi, we are faced with the same issue. Could you please give some more details on it, i.e. how to filter them out?


🐞 [Bug]: Fix profile card spacing

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
The phrase "Scan the QR code using ThreeFold Connect to fund your account" has no space after it.
Steps To Reproduce

Login to the dashboard
Go to profile

which network/s did you face the problem on?
Dev
version
2.6.0
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
-

Verified on Dev,
Version: 4e2fc0b
The profile card spacing has been fixed.

Verified:

Devnet
4e2fc0b


Build error example "RtspClientSharp.sln". No open file "avcodec.lib"

Hi! Download, open example solution and build error.
VS2017, Win10 x64, Net Core 2.2. Build Solution on error:
Code: LNK1104
Description: no open file "avcodec.lib"
Project: libffmpeghelper
File: Link
String: 1.

i have same problem
Project: SimpleRtspPlayer
dotNet : 4.6.1

Fixed. https://github.com/BogdanovKirill/RtspClientSharp/commit/bc1acd9d46ea18a9c898cd2b3c131c59c3a76735


Add "code-output" class to code-output blocks

Now:
julia> s = """
       ```!
       1+1
       ```
       """ |> fd2html |> println

1+1

2

so same as before but there's the extra class code-output that allows separate styling of code output blocks.
closes #856

Nice feature! Feel free to copy the CSS from Books.jl for output blocks :smile: From the docs:

@rikhuijzer would you mind opening an issue on FT for this please?


453 apply maximize direction

@r-soga
Thank you for reflecting my comments.
Some CI checks failed, but the issue was solved by this PR (https://github.com/pfnet/optuna/pull/537).
Please merge this branch with master.


⚠ PrivateHD has degraded performance

In b4419e6, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 6043 ms

Resolved: PrivateHD performance has improved in 0ca06ac.


The docs image has > 900 MB now. Can we get this down somehow?

I briefly checked that, and this size seems due to the size of page built by hugo:
> docker history gsoci.azurecr.io/giantswarm/docs

IMAGE                CREATED        CREATED BY                                                       SIZE         COMMENT
77415a24369b   3 days ago     COPY /public /usr/share/nginx/html # buildkit   907MB     buildkit.dockerfile.v0
...

It seems confirmed inside the container:
> du -sh /usr/share/nginx/html

900.8M	.

The biggest directory out there seems the changes:
> du -sh /usr/share/nginx/html/changes

822.5M	/usr/share/nginx/html/changes

The content of changes/ directory seems to be a subject of automated updates according to this list of apps.
Looking at it seems like it contains some apps that are deprecated for example, so maybe we could reduce the size by removing entries for them.

Thanks for the investigation!
Since changelogs account for 90% of the entire payload, I'll look into

Removing outdated projects
Removing releases beyond a threshold date
Compressing HTML using gzip

I thought we were doing (3) already, but it seems we're not.

https://github.com/giantswarm/docs/pull/2040 brings the size down to ~160 MB


Redirect users to their dashboard when access denied

I thought through this one a bit and as far as I can tell, I don't think we need any other checks to make sure current_team is okay.
I wonder if we had this line in place because of how we handled current_team in the original bullet-train. Either way, I confirmed that current_team stays the same after trying to access a restricted page and being redirected to the dashboard.
This new redirect will take the user to their Team#index page.

Maybe I'm misunderstanding what's meant by "invalid," so I can look into this one again soon if there's something deeper going on here.


[Feature Request]: option to additionally "Commit&Push" at program closure

Hello,
would it be possible to add a feature to push all outstanding commits at program closure? I've just noticed that I was pending 220 committed changes from five days because I never gave the program time to push. At the same time, I don't want to reduce my timers even more because I already start encountering timing related issues with the program freezing sometimes when running through the routines.
Thank you.
Sincerely,
~Gw

There's already an issue for this, see #13

Whoops, I somehow did not find issue. Will close this one then,


[Feature Request] An option to set SHELL for preview command on Windows

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[ ] Linux
[ ] Mac OS X
[x] Windows
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
Most of the interesting and useful tooling around fzf is built around POSIXy shells such as bash and zsh. I can think of three examples (I'm sure there thousands others) on the top of my head:

https://github.com/wfxr/forgit
https://github.com/joehillen/paruz
https://gist.github.com/junegunn/8b572b8d4b5eddd8b85e5f4d40f17236

Bash and zsh shells are readily available on Windows (Git Bash, MSYS2, cygwin), so the shell is not really an issue. The problem arises with preview commands in these tools, which always assume bash-like shells. So the preview in these tools does not work on Windows because fzf unconditionally uses cmd for the preview command, and doesn't respect SHELL (like it does on nix platforms).
Proposed solutions

When using MSYS2/Cygwin, the SHELL environment variable is set by them. Fzf should respect that, or optionally have a configuration to respect that.
Another solution is to implement an environment variable FZF_PREVIEW_SHELL that only applies to Windows platform, and this can be used to inform fzf about the interpreter (or its path). If the interpreter (or its path) is not found, fallback to using cmd.

Would love to hear more thoughts on this. The lack of preview support extremely limits the amount of fzf tools that can be used in MSYS2/Cygwin.

You're right, SHELL is not set/used by Windows outside of Cygwin/Msys2. But solution 1 presents a problem: the SHELL variable would be set to something like /usr/bin/bash or /bin/bash, which are paths that only Cygwin/Msys2 will understand. Go will error out on finding these paths on Windows.
This is why I had to think of solution 2.
(note that there's a builtin utility called cygpath available inside Cygwin/Msys2, which can be used to convert unix/windows paths to and fro. If we really have to go with solution 1, we will have to use this utility to convert the interpreter's path before passing it to Go.)


[parastate] adding ParaState Staking strategy

We are adding a new strategy to support governance with ParaState staking.
It has passed the tests.

Closing this as stale, feel free to reopen if needed


AoT ERROR in Cannot read property 'codeGen' of undefined

OS?
OSX El Capitan, OSX Sierra, Amazon Linux on EC2
Versions.
beta.22 / angular 2.2.3
Repro steps.
I have a branch cut from a few weeks ago on the above versions.
Several days ago, ng build --aot=true --target=production works fine.
Yesterday, same command fails (log below)
Keep in mind that this is via a build server on Jenkins, which checks out a clean copy every time. To replicate, I did a clean checkout of my repo in another directory, switch to the cut branch that hasn't been modified since our last sprint, and the same issue appears.
I was able to build fine on my local workspace repo, but once I deleted my cached node_module directory and re-installed all dependencies, I get the same error below.
The log given by the failure.
Hash: 298b4d5b0927ce483a6d                                                                Time: 18425ms chunk    {0} scripts.2ec7b6c32f2cecb6ef43.bundle.js, scripts.2ec7b6c32f2cecb6ef43.bundle.map (scripts) 112 kB {4} [initial] [rendered] chunk    {1} main.0ffb0f0b246299e375b6.bundle.js, main.0ffb0f0b246299e375b6.bundle.map (main) 2.02 kB {3} [initial] [rendered] chunk    {2} styles.b2328beb0372c051d06d.bundle.js, styles.fc27ae193c6cf276eb76015a2f77056b.bundle.css, styles.b2328beb0372c051d06d.bundle.map, styles.b2328beb0372c051d06d.bundle.map (styles) 69 bytes {4} [initial] [rendered] chunk    {3} vendor.0c6af147dfcf8a75bb37.bundle.js, vendor.0c6af147dfcf8a75bb37.bundle.map (vendor) 855 kB [initial] [rendered] chunk    {4} inline.d41d8cd98f00b204e980.bundle.js, inline.d41d8cd98f00b204e980.bundle.map (inline) 0 bytes [entry] [rendered]
ERROR in Cannot read property 'codeGen' of undefined
ERROR in ./src/main.ts Module not found: Error: Can't resolve './$$_gendir/app/app.module.ngfactory' in '/Users/aaron/repos/my-clean-checkout/src'  @ ./src/main.ts 4:0-74  @ multi main
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve '/Users/aaron/repos/my-clean-checkout/src/$$_gendir' in '/Users/aaron/repos/my-clean-checkout/node_modules/@angular/core/src/linker'  <EMAIL_ADDRESS>46:15-36 58:15-102<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>@ ./src/main.ts  @ multi main
Mention any other details that might be useful.
I suspect some breaking bug might have been published to beta.22, angular 2.2.3, or one of its dependencies. NPM is fetching the same version from the public repo as before, but getting a bad version now.
Upgrading to beta.24 / angular 2.3.0+ may not be a feasible solution since I use a few libraries that hasn't been exported to be AoT compatible for 2.3.0+, but instead still works for 2.2.3. If I were to upgrade, I get the error reported at  #3674
##UPDATE the internal dependency in question appears to be "@ngtools/webpack". Perhaps the solution is to pin beta.22's ngtools/webpack to version 1.1.9, instead of specifying the latest minor/patch version, which resolves to 1.2.1 according to npmjs

looks like<EMAIL_ADDRESS>from this commit is causing the problem.  https://github.com/angular/angular-cli/commit/db4e0242c482be46b47b78d54c9b4c33bbf57591

@bmbrown10 I figured out a workaround from another user in the other issue I referenced above:
Instead of downgrading to beta.21, pin cli's internal dependencies by adding "@ngtools/webpack": "1.1.9" to your package.json. Once I did that, the error went away and build/serves aot fine.
I will update the original issue to suggest such a fix as well.

@aaronleeucla it works for me also. But i installed @ngtools/webpack using npm install<EMAIL_ADDRESS>and it worked

could be related to https://github.com/angular/angular-cli/issues/3714#issuecomment-269067475

I think we didn't pin @ngtools/webpack on those releases and thus newer ones broke it. The solution is to either pin it on your project, or update.


Describing env variables in readme file

This PR describes all env variables that consumer and producer supports

So according to the https://github.com/nodeshift-starters/reactive-example/pull/99 this PR is the doc related right


Not compatible with Julia 1.0

I am happy to start work moving it to Julia 1.0 if noone else is

i can help you @thattommyhall


Feature Request: Support setting global package

One of the main pieces that would help speed up using git-labelmaker is including an option for using a package from a file as the default package to use when no input is given for Add Labels from Package. This way you can just simply use one package for all of your repos.
I'd definitely be willing to submit a PR if the maintainers think this is a good idea.

This is a great idea! Feel free to submit!

@EmmaRamirez so if implemented, where would this global package live? Would there be a new option to create a global package, or would the user just be able to store it somewhere outside the local directory?

Hmm, that's a good question. I'd imagine an option to create a global package, and storing it somewhere (arguably an absolute path based off OS)


fixing css of testimonials section

making text center and comma inside the box will make it look more beautiful
please assign this issue to me under gssoc

Hello
I am participating in GSSOC'24. I am interested in resolving this issue Could you please assign this task to me under GSSOC'24 with relevant level labels?
Thank you

Hello
I am participating in GSSOC'24. I am interested in resolving this issue Could you please assign this task to me?


Lack of Webpack samples in /examples

Someone pointed out that we don't have anything that uses Webpack listed under https://workboxjs.org/examples/#projects-using-workbox
@prateekbh, would you consider https://github.com/prateekbh/Guess to be a representative example that follows the best practices? If so, could you update it to use the latest workbox-webpack-plugin and then we'll link to it?

yes will do 👍
also a new one is on its way too 😄


Handle already uninstalled applications

Description of the new feature / enhancement
When the files no longer exist for an uninstaller, but the ARP entry does exist, running winget uninstall <package> (especially if run during the uninstallPrevious upgrade flow) results in an error 0x800401f5 : Application not found
This is the same error which causes appwiz.cpl to report this message on an attempted uninstall - “An error occurred while trying to uninstall. It may have already been uninstalled. Would you like to remove from the Add or Remove Programs list?”
As a user, I would expect winget to be able to detect this and proceed with the upgrade, remove the errant ARP entry, and/or provide a clearer error/warning.

See also #2170

This could be resolved by implementing an optional --force or -f parameter to the uninstall command to force removal of manually uninstalled or broken applications.

Same situation here
PS C:\Users\hu> winget list VCRedist
Name                                                               Id                           Version       Source
---------------------------------------------------------------------------------------------------------------------
Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.34.31931 Microsoft.VCRedist.2015+.x64 14.34.31931.0 winget

PS C:\Users\hu> winget remove Microsoft.VCRedist.2015+.x64 --force --purge --verbose
Found Microsoft Visual C++ 2015-2022 Redistributable (x64) [Microsoft.VCRedist.2015+.x64]
Starting package uninstall...
An unexpected error occurred while executing the command:
0x800401f5 : Application not found

@Trenly thank you for letting me know that from appwiz.cpl I'm able to correct that

[Policy] Command-Uninstall

This could be resolved by implementing an optional --force or -f parameter to the uninstall command to force removal of manually uninstalled or broken applications.
A possible work-around is to install the application again, followed by a propper uninstall.
Can anyone let me know where the list of installed applications is stored?

That work-around doesn't work

PS C:\Users\user> winget install JetBrains.IntelliJIDEA.Ultimate
Found an existing package already installed. Trying to upgrade the installed package...
Found IntelliJ IDEA Ultimate Edition [JetBrains.IntelliJIDEA.Ultimate] Version 2<IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.jetbrains.com/idea/ideaIU-2<IP_ADDRESS>.exe
  ██████████████████████████████  1.11 GB / 1.11 GB
Successfully verified installer hash
Starting package uninstall...
An unexpected error occurred while executing the command:
0x800401f5 : Application not found
PS C:\Users\user> winget install --force JetBrains.IntelliJIDEA.Ultimate
Found IntelliJ IDEA Ultimate Edition [JetBrains.IntelliJIDEA.Ultimate] Version 2<IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Successfully verified installer hash
Starting package install...
Successfully installed
PS C:\Users\user> winget uninstall JetBrains.IntelliJIDEA.Ultimate
Multiple versions of this package are installed. Either refine the search, pass the `--version` argument to select one, or pass the `--all-versions` flag to uninstall all of them.
PS C:\Users\user> winget uninstall JetBrains.IntelliJIDEA.Ultimate --all-versions --force
(1/2) Found IntelliJ IDEA 2<IP_ADDRESS> [JetBrains.IntelliJIDEA.Ultimate]
Starting package uninstall...
Successfully uninstalled

(2/2) Found IntelliJ IDEA 2024.2.2 [JetBrains.IntelliJIDEA.Ultimate]
Starting package uninstall...
An unexpected error occurred while executing the command:
0x800401f5 : Application not found

PS C:\Users\user>


Memory leak when using TaggableStore with flush method

Laravel Version
9.52.10
PHP Version
8.1.22
Database Driver & Version
No response
Description
When using the tags method on a cache store in order to namespace certain tagged items, a memory leak occurs if the flush() method is called on the taggable store.
This appears to occur because when the TaggableStore flushes, it does so by resetting the ids used in the tag namespace, meaning attempts to retrieve the data will now result in a cache miss. However, the cached data is left in place indefinitely and cannot be accessed as the tag ids have now been replaced.
As the Store interface does not expose a way of seeing all possible cache keys, the only  fix for this I can think of for this is to have the TaggableStore keep track of the keys within it's namespace, and have it forget() them when flush is called.
Steps To Reproduce
The following code snippet will show a steadily increasing amount of memory in use:
$store = cache()->store('array')->tags(['test']);
while (true) {
    $store->flush();
    $key = str_replace('.', '', uniqid());
    $store->set($key, uniqid());
    echo memory_get_usage(true) . PHP_EOL;
}

However I would expect all cache values to be cleared on calling flush() and therefore the memory use remain steady.

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? If so, please open up a new issue and we'll help you out.
Thanks!

Will try to reproduce with latest, thanks

Was able to reproduce with 10.x so opened https://github.com/laravel/framework/issues/48100


Move from Yarn to NPM

Ticket
https://github.com/navapbc/template-application-nextjs/issues/11
Changes

Move from NPM to Yarn

Context for reviewers
The most interesting change is in the storybook main.js file. Magical line is {loader: 'css-loader', "options": {url: false}}, which disables url resolution. We need this b/c with the dependencies in this repo we're installing a higher version and we need to account for a breaking change from css-loader 4.0.0 around url resolution (link to changelog). This approach mirrors what next.js is doing (link) which explains why the main app was building previously but not storybook. Last piece of context, the reason this was working w/ yarn was b/c the css-loader version was for some reason a lower version.
Testing
Yay:

Thank you! Can we add an ADR for this? There wasn't one for Yarn, which made me uncertain of the original reasoning. It'd be good to document our reasoning for NPM in case a future team wants to switch to Yarn 😅

Added! First time writing one of these so any feedback welcome. Wasn't sure if we were supposed to autogenerate the new file or something, but I just did it manually after reading the MADR docs


Injection Tree Inheritance

As indicated at Angular 2 Docs, if one includes a provider on AppModule, every single child module should inherit those services.
Here's a plnkr I've made pretty quick. I have a service that function as an http interceptor. Inside I create a random seed to see if the service is created anew when a lazy module is loaded. If you run this code, changing from HomeComponent to LazyComponent and clicking on Request, on the console I get the same seed value. So it seems to work as advertised.
Now I'm trying to do the same thing with aspnetcore-spa. The first thing I did was add angular2-router-loader to load lazy modules with webpack, but the injector is not inherited. If I make any Http call inside a component of my LazyModule, nothing is intercepted. Whereas I make a request inside an eager loaded module, it works and with the same seed value.
I've tried adding the CoreModule to the LazyModule imports, but doing so creates a new instance of the interceptor service.
Is this something that should be working and it isn't, or am I missing something?

Also to take into consideration.
This example in github use Angular CLI and Webpack. I downloaded the code, added CoreModule and put it as a provider into AppModule.
In both eager and lazy loaded modules in the example, the interceptor is working and showing the same seed on either module.

Interesting question, but I think it would be better if you followed up with the Angular community given that this is a fairly advanced Angular usage question. The Angular template in this repo is just Angular+TypeScript+Webpack (we haven't in any way modified how any of those things work), so whatever normally works with those will work here too. Sorry I can't give you a more useful answer. Anyone else who has more insight is welcome to post here.
I'll mark this as closed as it doesn't appear to be an issue with any of the code in this repo, but if you do track it down and think we have some sort of defect here, please reopen with details. Thanks!


Update Github about text

Change the GIthub page's about text from:
"Easily install and load packages from the tidyverse"
To:
"Easily install and load packages from the tidyomcis ecosystem"
I would do this myself but don't think I am able. Thanks!

Thanks done


Typescript debugger "broken" on 1.47.x and 1.48.0-insider, working up to 1.46.1 (Ubuntu)

As the title says, looks like Typescript debugger is "broken" on 1.47.x and 1.48.0-insider, was working up to 1.46.1

VSCode Version: 1.48.0-insider, 1.47.3-1595520028, 1.47.2-1594837870, 1.47.1-1594686231
OS Version: Ubuntu 19.10

Steps to Reproduce:

I have the following debug configuration, I'm not using the tsc task inside the debug config because I keep tsc watch running on the background:

{
  "type": "node",
  "request": "launch",
  "name": "API",
  "program": "${workspaceFolder}/../api/src/index.ts",
  "cwd": "${workspaceFolder}/../api",
  "outFiles": ["${workspaceFolder}/../api/build/**/*.js"],
  "sourceMaps": true,
  "protocol": "inspector",
  "env": {
    "MONGODB_URI": "x",
    "EMAILS_QUEUE": "x",
    "AWS_ACCESS_KEY_ID": "x",
    "AWS_SECRET_ACCESS_KEY": "x",
    "AWS_REGION": "x"
  }
}

It works fine on VSCode up to 1.46.1-159242889, on 1.47.x and 1.48.0-insider the debugger crashes telling me:

/home/user/workspace/api/src/index.ts:1
import express = require('express')
^^^^^^

SyntaxError: Cannot use import statement outside a module

Like it is reading the actual Typescript file and trying to execute it with node directly
As simple as that, I've been using this config for quite some time now and it always worked before 1.47.x

Does this issue occur when all extensions are disabled?: Yes

I can confirm this issue is present in VS Code 1.47.3 and Ubuntu 18.04.4

  "program": "${workspaceFolder}/../api/src/index.ts",

You can fix this by pointing your program at the compiled file, rather than the source file.
/duplicate https://github.com/microsoft/vscode/issues/102873


Feature Request: Derive Generalized Critical Value Flag for Labs

Feature Idea
@channel To assist in Hy's Law discussion #1041 and to broaden the scope just because flagging instances of " ># times ULN" for laboratory values can arise beyond Hy's Law implementations, for example:

checking if lab values reach a certain threshold for retreatment criteria after an AE that caused dose delay/reduction
checking for inclusion/exclusion criteria

derive_lab_fl() should create columns containing flags as appropriately based on the desired parameters and their respective cutoffs
Relevant Input
Function arguments should include:

data: some sort of SDTM.LB or ADLB dataset
param: lab parameter like "AST" or "BILI"
crit_seq: a sequence of numbers to create categorical ># times ULN
flname: character string to assist with the naming convention of the columns

Relevant Output
Final returned dataset should be input dataset + N columns where N is defined as length(crit_seq) + 1, for example:
derive_lab_fl(data = adlb, param = "AST", crit_seq = c(3,5,10), flname = "CRITAST") should generate
adlb + 4 more columns these 4 columns named: CRITAST_0_3, CRITAST_3_5, CRITAST_5_10, CRITAST_10_Inf
where if AVAL was 40 and the ULN was 10, then this aval belongs in the 3-5xULN category, so the column CRITAST_3_5 would be flagged "Y" and other 3 columns are blank.
Reproducible Example/Pseudo Code
derive_lab_fl <- function(data, param, crit_seq,  flname){
#assert data is labs dataframe
#assert param is character vector
#assert crit_seq is numeric vector
#assert flname is character vector
flag_dataset <- data %>%
filter(PARAMCD %in% param & is.na(DTYPE)) %>%
select(USUBJID, LBSEQ, PARAMCD, AVAL, ANRHI) %>%
mutate(flag = cut(ANRHI/AVAL, breaks = c(0,crit_seq,Inf)),
flagval = ifelse(!is.na(flag), "Y", NA))%>%
pivot_wider(., names_from = flag, values_from = flagval) %>%
rename_with(., ~gsub(",", "", .), starts_with("(")) %>%
rename_with(., ~gsub("\(", flname, .), starts_with("("))%>%
rename_with(., ~gsub("\]", "", .), starts_with(flname))
#left join back with original dataset
}
no implementation of the direction portion yet

@pharmaverse/admiral check this out!!  Looks exciting!!

I think we need to add a by_vars argument for the join.
Do we need more flexibility, e.g., relation to baseline instead of ULN?
The naming convention for the columns is problematic because in most cases it creates non ADaM compliant variable names (the name must be no longer than 8 characters in length and for flags the FL suffix should be used).
I wonder if we need a new function or if we could use existing ones. For example:
call_derivation(
  adlb,
  derivation = derive_var_merged_exist_fl,
  dataset_add = adlb,
  filter_add = PARAMCD == "AST" & is.na(DTYPE),
  by_vars = vars(USUBJID, ADT),
  variable_params = list(
    params(
      new_var = AST01FL,
      condition = AVAL/ANRLO < 3
    ),
    params(
      new_var = AST02FL,
      condition = 3 <= AVAL/ANRLO & AVAL/ANRLO < 5
    ),
    params(
      new_var = AST03FL,
      condition = 5 <= AVAL/ANRLO & AVAL/ANRLO < 10
    ),
    params(
      new_var = AST04FL,
      condition = 10 <= AVAL/ANRLO
    )
  )
)

The call is a bit longer but I think it is clearer (the condition on DTYPE and the handling of border cases (AVAL/ANRLO = 3) is visible) and the approach offers more flexibility.

I agree, I think I like this more actually to create the flags, just hadn't realized this was a way to do it. Perhaps instead of writing/designing new functions, this becomes more about a vignette on how to create these flags


Wait for AEM to start/stop

Hello Guys,
I hope this finds you well.
This is a version of init.erb that waits for AEM to start or stop.
This is helpful when other things should be done automatically when AEM is up or down.
init.erb.txt
Would be helpful to have that in your module.
Best regards,
Stefan

Grepping in a log is not the best way to detect this. I'm also not sure if this is needed for the start, I think it's suitable to just return when the process has been bootstrapped. The logline you are grepping for is not a guarantee that AEM is fully up and running, it could be that a SP is installing and reboots are occuring for example.
For the shutdown I think it's a good idea to check if the service is shut down completely by polling the pid.
Thanks for the suggestion but if you like to contribute it would ofcourse be better to create a PR instead of dropping in text files ;)

I agree with @bdhoine - Log monitor is the wrong way to make sure that the system is online.
I will also go so far as to say - the existence of the pidfile is all that the init system should care about that the AEM is online.  (I believe that init ensures the service is offline when the referenced pid in the pidfile is no longer active, achieving the other point)
If AEM needs to be validated to be online prior to other Puppet activities - we need a mechanism to check for that and it should be:

All bundles active (Dumb and not necessarily valid but simple to implement)
[Configurable] API call to validate bundle state (available OOTB but assumptions must be made)
Custom - user provided bundle/service API call with pre-defined response to indicate AEM is running (Most complex, but most flexible)

I'd be open to any PRs/issues that support these approaches.


Example Server Code in 0.12.0 Documentation uses 'response'; doesn't compile

http://cpp-netlib.org/0.12.0/index.html
The Hello World HTTP server examples listed in the documentation will not compile because the 'server::response' member is now unused (I guess? First time user) in favor of 'server::connection_ptr' (at least that is what the included hello_world_server.cpp has replaced it with).
This ended up being a significant issue for me because I assumed it was simply my fault for trying to be fancy and do it in Windows (VS 2015), so I then spent several hours trying to get it working in Linux as well, which I'm not super great at (so I again assumed it was my fault).
I probably should have tried looking elsewhere for an example or looking at the documentation for 'response' and seeing it changed or tried an older version, etc; just pointing out that there is an issue.

Oh, yes, that's broken. 😦
Do you have time to fix this and send a pull request to the 0.12-release branch?

Fixed in 0.12, and the documentation is updated in 0.13 :)


Can YUV444 format participate in training?

Dear author, I take the liberty to disturb you. I changed the format of the training data from RGB to YUV format, and during testing, I found that there was a color deviation.

Maybe you can：

Check about the number range carefully of every image channel. Input and output of the network require number range (-1,1).
torch.clamp_(-1,1) has worse impact on YUV than on RGB


Latest HTTP release is broken - cannot find superagent

I'm building based on cyclejs-community/one-fits-all. The bug could well be there - eg tree shaking removing superagent.
Using @cycle/HTTP 14.1.0
If I try to send a HTTP request I get this stack trace in chrome F12 tools
index.js:6 ReferenceError: superagent is not defined
    at optionsToSuperagent (http-driver.js:21)
    at Object.startResponseStream [as start] (http-driver.js:78)
    at Object._start (index.js:44)
    at Stream../node_modules/xstream/index.js.Stream._add (index.js:867)
    at Remember../node_modules/xstream/index.js.Remember._start (index.js:662)
    at MemoryStream../node_modules/xstream/index.js.MemoryStream._add (index.js:1717)
    at Debug../node_modules/xstream/index.js.Debug._start (index.js:289)
    at MemoryStream../node_modules/xstream/index.js.MemoryStream._add (index.js:1717)
    at Flatten../node_modules/xstream/index.js.Flatten._n (index.js:518)
    at Stream../node_modules/xstream/index.js.Stream._n (index.js:791)

Works fine if I fix at version 14.0.0
If I inspect the bundle I do not see the symbol superagent defined in either version
_superagent is defined in both cases. Perhaps a native module is involved?
Very strange

Title is a bit misleading here ^^

D'oh. Fixed.

I've also noticed this issue. Version 13.x worked fine, but 14.x seems to be missing superagent.

14.0 should work

Odd, because superagent is still a dependency, not devDependency. It could be the commonJS import is not working in whatever build system people are using. I'll try using ES6 imports.

Issue 666 closed by cycle/http v14.2 :)

Thanks @staltz !!

The devil is in the details, like this issue #666 :smiling_imp:

This issue came straight from hell

hells bells!


@import syntax error error with webpack when testing (with Mocha)

I am using Mocha to write tests for my application. Any time I try to test a component that uses a widget from this library I am presented with the following error message:
<path_to_my_app>/app/node_modules/react-widgets/lib/less/react-widgets.less:1
(function (exports, require, module, __filename, __dirname) {
@import "./variables.less";

^

SyntaxError: Unexpected token ILLEGAL

I have the following loaders in my webpack.config.js:
loaders: [
      { test: /\.js$/, exclude: /(node_modules|bower_components)/, loader: 'babel' },
      { test: /\.json$/, loader: 'json' },
      { test: /\.css$/,  loader: 'style-loader!css-loader' },
      { test: /\.less$/, loader: 'style-loader!css-loader!less-loader' },
      { test: /\.gif$/, loader: 'url-loader?mimetype=image/png' },
      { test: /\.woff(2)?(\?v=[0-9]\.[0-9]\.[0-9])?$/, loader: 'url-loader?limit=10000&minetype=application/font-woff' },
      { test: /\.(ttf|eot|svg)(\?v=[0-9]\.[0-9]\.[0-9])?$/, loader: 'file-loader?name=[name].[ext]' },
      { test: require.resolve('trackjs'), loader: 'exports?trackJs'}
    ]

I get no such errors when actually running my application, simply when trying to test components that eventually reach this import.

I found people getting a similar error here.

In the above case, the users experience the error in all cases, rather than just in testing, as I do. This leads me to guess it's a web pack issue, but one that only arises under certain circumstances due to the continuation of mocha in my app.


Add API docs file for root_model.py (issue DOC-3)

@lig did you already do this in another open PR? If not let's just merge this.

@lig did you already do this in another open PR? If not let's just merge this.

I didn't.


[Bug]: After modifying the package subroutine content, multiple triggering save, error

ODC version
4.2.1
OB version
oracle shcema <IP_ADDRESS>
What happened?
first save

second save

What did you expect to happen?
Multiple save triggers were successful
How can we reproduce it (as minimally and precisely as possible)?
modify subroutine content, save multiple times
Anything else we need to know?
No response
Cloud
No response

test passed


Finishing touches on JdbcIO

Uses AutoValue
Abstracts away DataSourceConfiguration into its own object
Speeds up test from ~20s to ~6s
Assorted cleanups

Thanks, reviewing and merging.

The changes are really good, thanks for that. I'm merging this PR, and I will add the connection pool configuration properties (lost in this PR).


⚠ ClamAV has degraded performance

In 1929700, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 430 ms

Resolved: ClamAV performance has improved in 79463bf after 535 days, 12 hours, 51 minutes.


Copy / Export Data to Sandbox

I can't seem to find a way to copy all my settings & added data (eg products, vendors and customers) to the sandbox so I can test in a much more replicated environment. Is this there a way to export a whole company for back-up purposes or to export it to the sandbox environment.

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

ID: 17d60d56-bfba-902b-9c10-87bbc3ad72d1
Version Independent ID: 9be1af8c-eeca-c96f-330d-6c08ac03436a
Content: How to Copy Data to New Companies - Business Central
Content Source: business-central/admin-how-to-copy-data-to-new-companies.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Hello Mr-Pingu,
Thank you for your feedback!  You can create a sandbox environment with production data using the preview Sandbox Environment wizard. For more information, see https://docs.microsoft.com/en-us/dynamics365/business-central/across-how-create-sandbox-environment.
Alternatively, if you are a Business Central partner, or you have an agreement with a Business Central partner that has access to the administration center for your Business Central tenant, you can create a sandbox environment with production data in the tenant administration center. For more information, see https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/administration/tenant-admin-center
I hope this answers your question. Otherwise, please let us know.

Hello edupont04,
For the first method, this doesn't transfer all my companies to the sandbox. When I create or reset the sandbox I don't get all the companies and the data from these companies.
For the second method. I passed it to my Business Central Partner and will get back to you when I have more information on this specific matter.


Short sentence have 0 bleu

I found that for sentences with less than 4 tokens, then bleu is always 0. Is this a bug or is it just designed to let them have 0 bleu?

this is probably because there are no higher order n-gram matches for those sentences, so yes it has to do with the design of BLEU and its set of default parameters.

Are you using sentence-bleu or the default corpus bleu by the way?

Are you using sentence-bleu or the default corpus bleu by the way?

I am using this:
from sacrebleu import corpus_bleu

Are all your sentences < 4 tokens in both hypotheses and references?

If any of the corpus-level 1-gram, 2-gram, 3-gram, or 4-gram counts are zero, your BLEU score will be zero. If you want to avoid this, use smoothing. However, BLEU is unlikely to be useful as a metric for quality in such situations.


Emmet suggestions for components

First I wanted to thank you for your LS. It's been very useful.
Is there a way in which I can configure the LS is to autocomplete for React or Vue component syntax.
For example, if I type AppCard>div it should be completed to this:
<AppCard>

</AppCard>

Is this possible?

Hey @Tauromachian in tsx/jsx files you should get that completion, take a look:

But for Vue you should use only the vue language server (volar). Look at #25.
Hope that helps!

@olrtg Thanks for taking the time to answer. Closing this issue.


only fetch utxos when necessary to avoid network call latency

Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
I found that even when I set locktime to ensure that Transaction.build() does not make a network call to get latest blockheight, that it was still taking a long time. I dug in and found setInputsAndOutputs() is always fetching utxos from the network.
The diff below is just for the build, and may have a better way, but I think something like it should work.
Here is the diff that solved my problem:
diff --git a/node_modules/cashscript/dist/Transaction.js b/node_modules/cashscript/dist/Transaction.js
index d1ab5d7..963445e 100644
--- a/node_modules/cashscript/dist/Transaction.js
+++ b/node_modules/cashscript/dist/Transaction.js
@@ -179,10 +179,16 @@ export class Transaction {
         if (this.outputs.length === 0) {
             throw Error('Attempted to build a transaction without outputs');
         }
-        const allUtxos = await this.provider.getUtxos(this.address);
+
+        let fetchedUtxos = [];
+        if(this.inputs.length === 0)
+        {
+            fetchedUtxos = await this.provider.getUtxos(this.address);
+        }
         const tokenInputs = this.inputs.length > 0
             ? this.inputs.filter((input) => input.token)
-            : selectAllTokenUtxos(allUtxos, this.outputs);
+            : selectAllTokenUtxos(fetchedUtxos, this.outputs);
         // This throws if the manually selected inputs are not enough to cover the outputs
         if (this.inputs.length > 0) {
             selectAllTokenUtxos(this.inputs, this.outputs);
@@ -300,7 +306,7 @@ export class Transaction {
         }
         else {
             // If inputs are not defined yet, we retrieve the contract's UTXOs and perform selection
-            const bchUtxos = allUtxos.filter((utxo) => !utxo.token);
+            const bchUtxos = fetchedUtxos.filter((utxo) => !utxo.token);
             // We sort the UTXOs mainly so there is consistent behaviour between network providers
             // even if they report UTXOs in a different order
             bchUtxos.sort(utxoComparator).reverse();

This issue body was partially generated by patch-package.

Regarding scale of impact, this made just one test file in the updated (using 0.9.1) anyhedge library test suite go from 120 seconds to 1.2 seconds.

Created #167 as a starting point for this.

tweaked the PR & merged to fix this issue!


Fork, Commit, Merge - Easy Issue (CSS)

Fork, Commit, Merge - Easy Issue (CSS)
Change The Color of The Header Text From Black to Red
Note: You don't have ask permission to start solving the issue or get assigned, since these issues are supposed to be always open for new contributors. The actions-user bot will reset the file back to previous state for the next contributor after your commit is merged. So you can just simply start working with the issue right away!
Open the tasks/css/easy directory from the root of your project.
In this directory, there is index.html file with h1 tag that has class red.
Your task is to implement that class in styles.css file, to change the color of the h1 text to red.
So go ahead and open the styles.css file to start implementing!
Note: You should only need to modify the styles.css file.
If you are using VS Code and have for example Live Server extension, you can simply open index.html with the server and check if the color of the header has turned to red. If it is, you are ready to make a pull request!
If you do not have Live Server extension, you'll find information about installing and how to use it here.

Check out README.md for more instructions and how to make a pull request.
Feel free to ask any questions here if you have some problems!
Also, kindly give this project a star to enhance its visibility for new developers!

Can I give this an attempt and see where I can get?

Can I give this an attempt and see where I can get?

Yeah, of course.