Improve reads searching

Reads search is specified in such a way that it can be computationally infeasible to implement. The current state of the schemas allows searches across ReadGroupSets through the readGroupIds field of a SearchReadsRequest. This is akin to performing a sort across multiple BAM files.
This PR requires that a user specify a readGroupSetId when searching reads. Additionally I have added a comment that states the array of read group IDs must belong to a single Read Group Set.

addressed by  https://github.com/ga4gh/schemas/pull/570

I'm +0 for now, though leaning towards supporting. Multi-ReadGroupSet search is convenient for clients and it could prove useful to push it into the server for performance gains (if any). As of recently, there continues to be support for leaving the functionality in place. In the paginated API, though, it is less likely that encoding this in the API (as opposed to the client) will yield much in the way of performance gains.
I'm leaning towards supporting this because it simplifies the API, makes it more consistent with the rest of the API surface, and reduces complexity for implementers. In the future we may consider adding this functionality back if it can provably show value.
I'll also add that while I agree that this feature is complicated to implement, I don't understand the assertion that it is 'computationally infeasible' for a BAM-backed server (assuming reasonable preconditions on the BAM files...).

The current API requires doing a sort-merge of reads across a
large range and large number of ReadGroupSets (BAMs) is
computationally expensive.
A 5mb region across 10,000 BAMs is a very reasonable request.
40x coverage with 100 base reads is a sort-merge on about 20
billion reads seems very expensive to me.  Given that the ENA
isn't willing to re-compress start and end BAM blocks, it's hard
see how they would agree to doing the large sort-merge required
by the API.
Do we have a use case for sort-merging reads for all samples?
All use cases I know of would then de-multiplex the results into
the individual samples.  For this quantity of data to be feed
into existing applications, it would mostly like be writing
10,000 BAMs.
My experience (being responsible for ~100,000 BAMs) is that
existing data is almost always stored sorted on a
per-readgroupset basis and analyzed assuming this sorting
According to Jonathan, the Google API doesn't implement sorting
across readsgroupssets and Google at least has the compute power
to do this ;-)
Once streaming is implemented, it's very reasonable to have a
multiple readgroupset query provide the sorting requirement
within a readgroupset, not across readgroupsets.  A reasonable
client API could hide the difference between streaming and
paging, so I would not object to having the query take an array
of readgroupset ids now, as long as the sorting requirement is
removed.
CH Albach<EMAIL_ADDRESS>writes:

I'm +0 for now, though leaning towards supporting. Multi-ReadGroupSet search is
convenient for clients and it could prove useful to push it into the server for
performance gains (if any). As of recently, there continues to be support for
leaving the functionality in place. In the paginated API, though, it is less
likely that encoding this in the API (as opposed to the client) will yield much
in the way of performance gains.
I'm leaning towards supporting this because it simplifies the API, makes it
more consistent with the rest of the API surface, and reduces complexity for
implementers. In the future we may consider adding this functionality back if
it can provably show value.
I'll also add that while I agree that this feature is complicated to implement,
I don't understand the assertion that it is 'computationally infeasible' for a
BAM-backed server (assuming reasonable preconditions on the BAM files...).
—
Reply to this email directly or view it on GitHub.*

There was very strong argument that the reads API needs to do the sort merge is a requirement for multi-sample BAM calling.  The is a strong desire to move the merge to the server, not the client.
We will update the documentation to capture this use case.

For the record, this is not correct:
the Google API doesn't implement sorting
across readsgroupssets

Jonathan misspoke (just talked to him). The Google API's SearchReads accepts one or more read group set IDs and returns a  paginated stream of reads which are ordered by position.
Note: We do not support searching by arbitrary readGroupIds from different read group sets (only from the same read group set) as we remain skeptical that this is a useful query. We may add support in the future - this would not be a significant amount of work, but it's not been requested.

thanks @calbach
I am glad this is coming down to me being unclear uses cases rather
than the API being off.
CH Albach<EMAIL_ADDRESS>writes:

For the record, this is not correct:
the Google API doesn't implement sorting
across readsgroupssets
Jonathan misspoke (just talked to him). The Google API's SearchReads accepts
one or more read group set IDs and returns a paginated stream of reads which
are ordered by position.
Note: We do not support searching by arbitrary readGroupIds from different read
group sets (only from the same read group set) as we remain skeptical that this
is a useful query. We may add support in the future - this would not be a
significant amount of work, but it's not been requested.
—
Reply to this email directly or view it on GitHub.*

@calbach GA4GH is not built for the old way of analyzing data, though it can still be used that way.  So we do not want to only query based on subsets of ReadGroupSets - which can slow us down - but we want triggers attached to metadata and specific sequence variants as the data is streaming into the repositories.
This means auto-sorting for on-the-fly pileups to allow for finding similar, rare samples. And then we want to open a genome browser and find which other rare mutations these highly-qc'ed samples are generating across the genome.
Think of this, let's say a variant is automatically detected based on subsets of auto-triggered alignments of some fresh data to subsets of reads in the system.  Has this been seen before, and were we in need of more samples like this?  Do we have a stored graph of this variant region?  If this is a cancer subtype, are there similar regions of stored matching paths that would project a molecular evolution?  Can we map it to their RNAseq activity for this region and how do they compare?  What are other diseases that map to these regions of the genome, and how similar are their variants.  I can go forever in building it up more and more, but I think you get the idea.
In my view GA4GH opens up genomic analysis to orders of magnitude larger than older tools, which requires a different and more fluid way of processing such data.
~p

@diekhans I don't think you are unclear regarding use cases, and I think those should be covered too or we risk reinventing the old tools again with GA4GH.

@diekhans Just a very careful question: What are, besides benchmarking variation callers, use cases for direct/federated access to reads? Sorry for my ignorance!

@mbaudis, we will find out, I am going to take a crack at the
use cases in the documentation.
Michael Baudis<EMAIL_ADDRESS>writes:

@diekhans Just a very careful question: What are, besides benchmarking
variation callers, use cases for direct/federated access to reads? Sorry for my
ignorance!
—
Reply to this email directly or view it on GitHub.*

:+1: I'd rather have this than an uncertainty in provider's behavior. Clients don't have to deal with the specifics of each provider.
PS sort-merge is "almost analysis" imo. If a server can guarantee sort-merge it is only a step away from providing coverage, consensus, variations etc.


Fix example spec: make it JSON compliant

Change in accordance with #45.

Pull Request Test Coverage Report for Build 936e845572a68ed2907f53038338cfe7945c732c-PR-52

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

Totals

Change from base Build 7daf2983e59fd9bf1b337efa0a16f59b6450e5ca:
0.0%

Covered Lines:
143

Relevant Lines:
207

💛  - Coveralls


Cannot get email from permission scope.

Hello, great job on this integration with codeigniter.
Everything is working except i can not seem to get additional user information other than the ID and the Full Name.
I just modified the way you fetched the user data in your example controller:
$user = $this->facebook->user();
if ($user['code'] === 200)
{
unset($user['data']['permissions']);
$data['user'] = $user['data'];
}
When i print $user i get this:
Array ( [code] => 200 [message] => success [data] => Array ( [name] => NAME [id] => ID ) );
I do not get email, which i also need for my application.
This is my facebook permissions in config:
$config['facebook_permissions']         = array('public_profile', 'email');
Am i doing something wrong or is this an issue with the library?
EDIT: When i print permissions aswell i get:
[permission] => email [status] => granted

I have the same issue here, I can only grab the id and the full name. Others are all null like email , first name, gender etc, Thanks

you have to use graph api
$this->facebook->request('get','/me?fields=email',$accessToken);

@Xoshbin @Xoshbin If you are using version 2.0.0 of the library you will have to manually modify the library source file.
Go to application/libraries/Facebook.php and on line 235 after /me add ?fields=email. You can specify more fields like birthdate and such depending on the permission scopes you have. After modifying, it should look like /me?fields=email
If you are using version 3.0.0-beta1, than you need to do like mentioned by @renjinsk


5k similar to death toll

Describe your problem here.
Is there a way to add 5k similar to death toll (a middle button that plays a custom animation when hit) with lua scripts?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No I did not.

There's a script that adds multi-key support in the Psych Engine Discord Server

Can i make it do custom animations

yes if you know what your doing

ok thanks


Prevent .ftpconfig from uploading itself (to protect passwords)

Think it would be sensible to automatically prevent the .ftpconfig file from being uploaded in order to prevent passwords from being uploaded into public access.

+1
Had to remove a whole github repository once, cause I forgot to add it to my .gitignore...

+1

+1

+1

+1

Put this in your .htaccess in the root of your site:
	<files .ftpconfig> 		<IfModule mod_authz_core.c> 			Require all denied 		</IfModule> 		<IfModule !mod_authz_core.c> 			Order allow,deny 			Deny from all 		</IfModule> 	</files>

You can use the .ftpignore (works like .gitignore) file for this.


Added combobox to the main view for displaying unique labels individually

By using a combobox, unique labels can be seen and displayed easily.

Merge it


How to hide "Error: Unsupported notification message" for jdtls

Can you include the complete error message (including the notification message)?

Can you include the complete error message (including the notification message)?

Error: Unsupported request message received from the LSP server (jdtls), message = {'id': '2', 'jsonrpc': '2.0', 'method': 'workspace/executeClientCommand', 'params': {'arguments': [], 'command': '_java.reloadBundles.command'}}

I'm confused as to if this is related to notifications or request replies in general? As the issue titles mentions notifications, while the copied error is related to the later.

FWIW then you can "silence" notifications by :h lsp-cfg-customNotificationHandlers.

This might be somewhat related to https://github.com/yegappan/lsp/issues/125

Should be fixed by #309


A11y_Model Builder_Create New Experiment_Keyboard: Tool tips that are present for each field are not keyboard accessible. Keyboard focus is not going to tool tips in 'Create New experiment' dialog

Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.
GitHubTags:
#A11y_Model Builder; #A11yAuto; #TabStops; #A11yMAS; #A11yTCS; #WCAG2.1.1; #Keyboard; #Win32; #DesktopApp; #A11ySev2; #ModelBuilder-Win32-Jun2021; #Visual Studio Client;
Environment Details:
Application name: Model Builder
OS Details:
Windows Version 21H1 OS Build (19043.1083)
Microsoft Visual Studio
Note:
This issue is observed in all popups. In 'Create new workspace', 'Create new compute' tool tips are not keyboard accessible
Repro Steps:

Enable ML.NET Model builder as a preview feature in tools>options>Environment>preview features.
Create a .NET C# project and tab till project under solution explorer and Add machine learning.
Tab till Image classification under scenario tab and press enter.
Select Azure under environment tab and tab till set up workspace and press enter.
In 'Create New Experiment' dialog, tab till Subscriptions/Machine Learning workspace/compute name fields
Verify whether keyboard focus is landing on tool tips present for respective fields.

Actual Result:
Tool tips that are present for each field are not keyboard accessible. Keyboard focus is not going to tool tips in 'Create New experiment' dialog
Expected Result:
Tool tips that are present for each field should be keyboard accessible. Keyboard focus should go to tool tips in 'Create New experiment' dialog and info inside tool tip should be visible when keyboard focus lands on tool tip
User Impact:
Keyboard users will not know the info that is present in tool tips if they are not receiving keyboard focus

Validate on rel/candidate build <IP_ADDRESS>1903. See issue 1627 for instructions for install.

Issue is fixed. Tool tips present for all fields are keyboard accessible. Hence closing the issue.
https://user-images.githubusercontent.com/83570638/131445150-88a84e90-b376-40c2-8d1d-fd41bf8401cc.mp4

#Closed;


Installer: Add install path to registry key

Description
Because the installer doesn't store the install path in the registry any application that wants to explicitly load in renderdoc.dll has to figure out the path to it from either the keys in HKEY_CLASSES_ROOT, the Vulkan implicit layers or prompting the user for it explicitly.
This PR simply adds the install path as the value for an empty registry key that the installer was already creating.

I don't want to have other keys that people are relying on, especially when it's not used for anything else and may be removed in future. If you need to locate the installed renderdoc.dll then the existing registry key under HKEY_CLASSES_ROOT is the correct way to do it since that's required for compatibility with other things and won't be changing, especially with programs already using it. I don't see any value in adding a second registry key.


Extraneous Output from liquibase-maven-plugin:4.0.0

Environment
Liquibase Version:
4.0.0
Liquibase Integration & Version: <Pick one: CLI, maven, gradle, spring boot, servlet, etc.>
maven
Liquibase Extension(s) & Version:
N/A
Database Vendor & Version:
N/A
Operating System Type & Version:
N/A
Description
Version 4.0.0 of liquibase-maven-plugin displays warnings with Java stack traces. This extra output looks like errors that require investigation. In reality, it's just a warning.
Steps To Reproduce
Follow the steps at https://github.com/mrgrew/liquibase-warnings
Actual Behavior
See the "Output from Liquibase 4.0.0" at https://github.com/mrgrew/liquibase-warnings
Expected/Desired Behavior
See the "Output from Liquibase 3.10.2" at https://github.com/mrgrew/liquibase-warnings
Screenshots (if appropriate)
If applicable, add screenshots to help explain your problem.
Additional Context
Add any other context about the problem here.
┆Issue is synchronized with this Jira Bug by Unito

Hi @mrgrew Thanks for writing up this issue and providing a code repo test case.  We will add this to our list of issues to process.


recombine: add timeout configuration option

Performance-wise this doesn't show any signs of a regression:
go test -v -count 1 -run ^$ -bench Benchmark -benchmem -count 4 ./... # with patch
goos: darwin
goarch: amd64
pkg: github.com/open-telemetry/opentelemetry-log-collection/operator/builtin/transformer/recombine
cpu: Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
BenchmarkRecombine
BenchmarkRecombine-16             419072              2894 ns/op             280 B/op         10 allocs/op
BenchmarkRecombine-16             395395              2849 ns/op             280 B/op         10 allocs/op
BenchmarkRecombine-16             410451              2874 ns/op             280 B/op         10 allocs/op
BenchmarkRecombine-16             436786              2874 ns/op             280 B/op         10 allocs/op
PASS
ok      github.com/open-telemetry/opentelemetry-log-collection/operator/builtin/transformer/recombine   6.057s

go test -v -count 1 -run ^$ -bench Benchmark -benchmem -count 4 ./... # without patch
goos: darwin
goarch: amd64
pkg: github.com/open-telemetry/opentelemetry-log-collection/operator/builtin/transformer/recombine
cpu: Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
BenchmarkRecombine
BenchmarkRecombine-16             423274              2872 ns/op             280 B/op         10 allocs/op
BenchmarkRecombine-16             434774              2834 ns/op             280 B/op         10 allocs/op
BenchmarkRecombine-16             432498              2845 ns/op             280 B/op         10 allocs/op
BenchmarkRecombine-16             437214              2847 ns/op             280 B/op         10 allocs/op
PASS
ok      github.com/open-telemetry/opentelemetry-log-collection/operator/builtin/transformer/recombine   6.223s

@pmalek-sumo This looks good to me except I wonder if we should name this parameter force_flush_period, since it is basically the same as is done in file_input. What do you think?

@pmalek-sumo What do you think about this suggestion?

@pmalek-sumo This looks good to me except I wonder if we should name this parameter force_flush_period, since it is basically the same as is done in file_input. What do you think?

@pmalek-sumo What do you think about this suggestion?

I've missed that somehow. Seems reasonable. Let me rename this

@pmalek-sumo This looks good to me except I wonder if we should name this parameter force_flush_period, since it is basically the same as is done in file_input. What do you think?

@pmalek-sumo What do you think about this suggestion?

Done, PTAL @djaglowski


how to use the jitter_image() for training face recognition

A function, called  [jitter_image()]  is defined in  [dnn_face_recognition_ex.cpp].
and you say jitter_image() was used during training in [dnn_metric_learning_on_images_ex.cpp].
can you figure it out exactly about how to use the function?

no use


minor bug fix

This is a very minor bugfix -- feel free to reject.
minor change to line 385 -- upon unsuccessful exit, program now replies:
Error: Invalid argument.
Usage:

    shyaml (-h|--help)
    shyaml ACTION KEY [DEFAULT]

Previously:
Traceback (most recent call last):
  File "/usr/local/bin/shyaml", line 389, in <module>
    sys.exit(main(sys.argv[1:]))
  File "/usr/local/bin/shyaml", line 385, in main
    die("Invalid argument.\n%s" % usage)
NameError: global name 'usage' is not defined

Thanks. Wow, the fact that so obvious exception remains startles me. I guess I never hit this case despite the dozens of places I use this piece of software.
Many thanks for your report and your interest for shyaml. Even small report count.


Execution of command missing an option required by usage not reported in output

release: openssl-3.0.5
Hello. The execution of a command missing an option required by usage is not reported in output, as it is elsewhere with "Use -help for summary.". Illustration; missing '-name <argument>'. Command indefinitely processed.
$ openssl ecparam -genkey -noout -out private.ec.key

The manual page of openssl-ecparam says:
       -in filename
           This specifies the input filename to read parameters from or
           standard input if this option is not specified.

Your command is just waiting for domain parameters on standard input.

There is a misunderstanding. The present issue lies in the lack of reaction, for instance from OpenSSL, in regard to task processing a command whose instructions are incomplete. Obviously a generic reaction would be the triggering of an informative message accordingly.

processing a command whose instructions are incomplete

The command is not incomplete. All required parameters are present. The command is doing what you asked it to do, and is waiting for the required input from stdin.
Another question is whether its good design to default to stdin and therefore hang in this way if the required data is not supplied on stdin. Probably not. Better design would be to have a specific parameter requesting input from stdin. But unfortunately that is not possible to change since it would break existing scripts.

All right. Closing accordingly; not planned.

Another question is whether its good design to default to stdin and therefore hang in this way if the required data is not supplied on stdin. Probably not.

Note that this is the exact behaviour of cat(1), so openssl-ecparam is in good company here!


only use on certain trigger divs

Great plugin.
Just wondering how I can selectively target multiple containers to have this functionality.
Lets say my app has a UI as follows:
page 1) landing page with a list of products
page 2) detail view of list of products (open when user clicks on list from page 1)
page3) list of colour options for the product (opened from a button within page 2)
Lets say I want the pulltorefresh function on the list in page 1 and in page 3, but it shouldn't work on page 2.
I can't seem to enter multiple selectors in the mainElement option. I also notice multiple css tags are ignored (for example setting mainElement: 'body.page1, body.page2' doesn't work.

Hey!
That seems like a very specific use case that can be solved by adding a class to those pages, rather than using multiple classes. Maybe you can add a body.pull-to-refresh class to those pages?

doesn't appear to work for dynamically added classes

doesn't appear to work for dynamically added classes.

Hi, unfortunately it does not work for dynamic classes but you can use DOM elements instead, or just selecting the entire body with no classes:
const instance = PullToRefresh.init({
  mainElement: document.body, // or document.querySelector('body')
});

I don't understand what is meant by

This can be illustrated as follows:
// calling init() will return an object with a single method: destroy()
const instance = PullToRefresh.init();

// later, you may want to call it
instance.destroy();

Ok, I got you know. I thought you were reloading between pages. As @pateketrueke said, you can destroy it whenever you don't need it. I think it's best to use shouldPullToRefresh, which starts the whole process. You can pass a function that evaluates !window.scrollY && document.body.classList.contains('my-class')

thanks!


read size exceeds bufferSize

I have on a couple occasions received the following error:
https://github.com/eadle/webm-byte-stream/blob/master/lib/webm-byte-stream.js#L220

Error: read size exceeds bufferSize
at WebMByteStream._read (node_modules\webm-byte-stream\lib\webm-byte-stream.js:220:11)
at EbmlDecoder. (node_modules\webm-byte-stream\lib\webm-byte-stream.js:93:34)
at emitOne (events.js:96:13)
at EbmlDecoder.emit (events.js:188:7)
at readableAddChunk (_stream_readable.js:176:18)
at EbmlDecoder.Readable.push (_stream_readable.js:134:10)
at EbmlDecoder.Transform.push (_stream_transform.js:128:32)
at EbmlDecoder.readContent (node_modules\ebml\lib\ebml\decoder.js:166:14)
at EbmlDecoder._transform (node_modules\ebml\lib\ebml\decoder.js:41:52)
at EbmlDecoder.Transform._read (_stream_transform.js:167:10)
at EbmlDecoder.Transform._write (_stream_transform.js:155:12)
at doWrite (_stream_writable.js:333:12)
at writeOrBuffer (_stream_writable.js:319:5)

I don't know the conditions to reproduce it.  Any suggestions for resolving this problem?  Any way to handle stream errors like this and close the stream gracefully?

The default buffer is 10MB. You can set a larger buffer size as one of the options or fork and use dynamic memory. It's literally just a buffer that loops on itself.
// default buffer size is<PHONE_NUMBER> or 10MB
var webmstream = new WebMByteStream({bufferSize:<PHONE_NUMBER>}); // now 20MB

Thank you.  And, is there a way to handle the error?

The error occurs when a the cluster end  tag is encountered and the length of the cluster exceeds the buffer size. I pretty much forget everything about node and webm, sorry.


fix(hiccup-css): allow negative values for px

Swap use of unsigned shift for OR 0
Fixes issue #383
Example use case:
const MARGIN_PX = -10;
const cmp = div({ style: "margin-left": px(MARGIN) }, "Hi");

Congratulations, Arthur, for finding & fixing a bug which has been sitting there for 5 years! Thank you & hope you're well!

:) you're welcome
I wonder what your thoughts are on https://github.com/thi-ng/umbrella/issues/383#issuecomment-1424399054


[Feature] Disable Database Table lookup/Index

Summary
One of the databases we use for AppSmith is huge and has a crazy amount of tables. Currently Appsmith will try and lookup/index the tables, which will always fail and that can cause some issues on appsmith (errors) and also load on the db server. It would be nice to have an option to opt-out of the tables being lookedup/indexed.
Motivation
Not sure what appsmith needs to capture all tables for, but it can cause load spikes and doesn't appear to be needed.
Additional Context
No additional context required.

We use the DB index to show the DB schema in the entity explorer and soon in the SQL autocomplete. See the screenshot below. Good point about this not working for large databases. We'll make it an optional setting.

Another way round this could be to do an option which is manually updating the schema and allows you to set a higher timeout.

Hey @wlatic, one way around this is to create the datasource with credentials that have limited / bare-minimum access. For example, if your database is Postgres, we could create a user with access only to the public schema or only the tables we intend to use in the app. This way, when Appsmith tries to load the schema structure, it will fail. This won't show up as an error for you, but just that the structure information in the sidebar that @areyabhishek mentioned above, wont' be available.


Improve the docs of the filter() arguments

Please confirm that this PR has done the following:

~Tests Added~
[x] Documentation Added
~Name of contributors Added to AUTHORS.rst~
[x] Description in RELEASE_NOTES.md Added

Description of PR
This PR moves the documentation of the filter options/arguments to an own page of the docs rather that having them as part of the slice() docstring.

Thanks @phackstock - agree that we should extend the docs to include examples and I'll start an issue for it to be tackled before release 3.0, but for now, my aim was to get the docs out of core.py.


Add the configuration usgae of postgresql-datasource.

The solution works with:
$ mvn wildfly:run -P postgres

Here is the server output:
16:46:57,591 INFO  [org.jboss.as.connector.subsystems.datasources] (MSC service thread 1-5) WFLYJCA0001: Bound data source [java:jboss/datasources/PostgreSQLDS]

And I can see the relative SQL log output from PostgreSQL:
2024-11-07 16:00:05.664 CST [58270] LOG:  execute <unnamed>: SELECT JOB_EXECUTION.JOBEXECUTIONID FROM JOB_EXECUTION INNER JOIN JOB_INSTANCE ON JOB_EXECUTION.JOBINSTANCEID=JOB_INSTANCE.JOBINSTANCEID WHERE JOB_EXECUTION.BATCHSTATUS IN ('STARTED', 'STARTING') AND JOB_INSTANCE.JOBNAME=$1
2024-11-07 16:00:05.664 CST [58270] DETAIL:  parameters: $1 = 'simpleBatchletJob'

Now it contains a new profile postgres-default-ds, and I split the h2 database configuration into a default profile.
In addition, I updated the README accordingly.

This could be used as an alternative https://github.com/jberet/jberet-examples/pull/16


Update ember-cli-addon-tests to 0.11.1.

Fixes issues with ember-data#master.

CI is failing against ember-source@beta and canary. Seems like some sort of issue with route dynamic properties.

There's two failing jobs in CI. I haven't dug into them but it'd be nice to get this merged so we can subsequently merge #595

we can close this ; )

This dependency has already been updated. Closing.


Unable to pick up items.

You are unable to pick up items when this mod is enabled in the game. You are able to pick up some select items like scrap but any other item dropped by 3d printers or by mobs or by chests are unable to be picked up.

Yes, yesterday patch seems to have changed something with how items are defined, will have to look at it.

Should be fixed now


fix log in bug

@Amrani-Farouk-Hossam-Eddine please make sure that in the future your PRs include at least a basic description of your contributions and changes as described by the included CONTRIBUTING.md file. Other than that all good here thanks for your contributions :-)


Change how maven pulls jars

Sorry, meant to do this against my own fork which has diverged significantly

No worries!


Combine sensors into multi sensor devices to reuse esp32 MCUs

Proposal
AtomS3-lite engine:

RPM via alternator W terminal pulse counter with optocoupler
Oil pressure (with ina219) resistance based sensor
Engine temperature (Dallas temperature sensor)
Exhaust temperature (Dallas temperature sensor)
Fuel level  (resistance based?)
Fuel flow?
Rudder position (Hall effect based or resistance/potentiometer)
Flame detector
Gas sensors

AtomS3-lite environment:

i2c sensors (air pressure/temp/humidity, illuminance, UV level, TVOC, lightning strikes detection)
GPS
Human presence, motion detection
Rain sensor
Dust sensor
Fridge temperature
Carbon Monoxide
Smoke detector

AtomS3-lite or Atom-Lite alarms and electrical:

Battery Voltage
Battery Temperature
H2 gas
Bilge level with JSN-SR04T Distance Measuring Ultrasonic Waterproof Sensor Module, Range Sensor
Alarm via WhatsApp with Ethernet module

Atom-lite or atomS3-lite fluids levels:

Black water (current loop based sensors)
Grey water
Fresh water
Lubricant
Water quality sensor
Fridge temperature (1-wire Dallas)

AtomS3 with built-in IMU:

Heave
Rate of turn

or
AtomS3R with built in IMU and magnetometer:

Compass
Heave
Attitude
GPS
Rudder angle
Rate of turn
Magnetic variation for location and date
Sunset/Sunrise

use m5stack PortABC base for connections

Done in:
https://github.com/bareboat-necessities/bbn_esp32_sensors_hub
https://github.com/bareboat-necessities/bbn_esp32_gpio_sensors_hub


Collapse side menu bar on responsive view

The side menu doesn't work properly using the responsive option from web browser. To solve the problem I added the classes collapsed in line 45 and collapse in line 266 (you can use the file index.html as reference). I've noticed the problem when I reloaded the page using the responsive mode (Firefox: Ctrl + Shift + M).

Line 45

<button type="button" class="collapsed navbar-toggle" data-toggle="collapse" data-target=".navbar-collapse">
    ...
</button>

Line 266

<div class="collapse sidebar-nav navbar-collapse">
    ...
</div>

There is a pull request open to fix this (and a number of duplicate issues), #174. If you're interested in trying it, take a look at the single commit, https://github.com/fabiang/startbootstrap-sb-admin-2/commit/1fdbfd134a2f77c7b345053ca4ef155b51c91a29


feat: avm/res/apim-management/service/identity-provider Add contentLibrary property to the Identity Provider apim module

Description
This pull request includes several updates to the API management service configuration files, focusing on adding a new clientLibrary parameter and simplifying the codebase by using optional chaining and nullish coalescing operators.
Additions:

Added clientLibrary parameter to the identity provider configuration in README.md, main.bicep, and main.json files. This parameter specifies the client library to be used in the developer portal and is applicable to AAD and AAD B2C Identity Providers. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

Code Simplification:

Replaced contains checks with optional chaining and nullish coalescing operators in main.bicep and main.json files to simplify the code and improve readability. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Metadata Updates:

Updated _generator metadata in various main.json files to reflect the new template hash values. [1] [2] [3] [4]

These changes aim to enhance the maintainability and functionality of the API management service configuration.
Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[ x] Azure Verified Module updates:

[ ] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[x ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[x ] Update to documentation

Checklist

[x ] I'm sure there are no other open Pull Requests for the same update/change
[ x] I have run Set-AVMModule locally to generate the supporting module files.
[x ] My corresponding pipelines / checks run clean and green without any errors or warnings

hi @tony-box, would you be able to approve this quick change? I need the contentLibrary so I can implement MSAL. Please let me know if you have any questions. Thanks!

@anotherRedbeard thank you for the updates! It's been on my to-do list to fix all the contains()--you are awesome!
Please see my minor suggestion. once that is resolved I'll approve the PR

@anotherRedbeard thank you for the updates! It's been on my to-do list to fix all the contains()--you are awesome!
Please see my minor suggestion. once that is resolved I'll approve the PR

Hi @tony-box,
I've made all the suggested changes. It looks like the build is failing to remove something in one of the deploys. Is that going to be an issue?

@anotherRedbeard thank you for the updates! It's been on my to-do list to fix all the contains()--you are awesome!
Please see my minor suggestion. once that is resolved I'll approve the PR

Hi @tony-box, I've made all the suggested changes. It looks like the build is failing to remove something in one of the deploys. Is that going to be an issue?

Hi @tony-box,
I've made the suggested changes. For some reason it's failing to remove a resource, but everything else is passing.

@anotherRedbeard thank you for the updates! It's been on my to-do list to fix all the contains()--you are awesome!
Please see my minor suggestion. once that is resolved I'll approve the PR

Hi @tony-box, I've made all the suggested changes. It looks like the build is failing to remove something in one of the deploys. Is that going to be an issue?

Hi @tony-box, I've made the suggested changes. For some reason it's failing to remove a resource, but everything else is passing.

Hey @anotherRedbeard,
that's unfortunately not entirely accurate.
(detective mode on): When looking at the pipeline's 'Attempt # 1' and into the deployment log in the collapsed section named 'Deploy template file', you can see that the pipeline tried to deploy the service 3 times. The first time failed because of an 'Internal Server Error' which is about as helpful as it sounds, and then both other attempts failed for the same reason the removal failed, that is the error 'The API Service ***apismin001 is transitioning at this time. Please try the request again later.'
Because the removal was not able to run, attempt # 2 failed as well as it happend to choose a different location to deploy to than the one the service was deployed into as indicated by the error message 'Invalid resource group location 'eastus'. The Resource group already exists in location 'eastasia''.
In between attempt # 2 & attempt # 3, you presumably deleted the leftover service, but it unfortunately went into a soft-delete state. Because of that, that attempt failed again as ARM is not able to auto-recover soft-deleted resources.
My advice would be to look for that soft-deleted resource, purge it and then run Attempt # 4 for the failed job - with your fingers crossed in that it does not run again into an internal server error. 😄
On a different note, I strongly recommend contributing from a branch other than 'main' next time. 💪  That's simply because each push to main will trigger the workflows.

@anotherRedbeard thank you for the updates! It's been on my to-do list to fix all the contains()--you are awesome!
Please see my minor suggestion. once that is resolved I'll approve the PR

Hi @tony-box, I've made all the suggested changes. It looks like the build is failing to remove something in one of the deploys. Is that going to be an issue?

Hi @tony-box, I've made the suggested changes. For some reason it's failing to remove a resource, but everything else is passing.

Hey @anotherRedbeard, that's unfortunately not entirely accurate.
(detective mode on): When looking at the pipeline's 'Attempt # 1' and into the deployment log in the collapsed section named 'Deploy template file', you can see that the pipeline tried to deploy the service 3 times. The first time failed because of an 'Internal Server Error' which is about as helpful as it sounds, and then both other attempts failed for the same reason the removal failed, that is the error 'The API Service ***apismin001 is transitioning at this time. Please try the request again later.' Because the removal was not able to run, attempt # 2 failed as well as it happend to choose a different location to deploy to than the one the service was deployed into as indicated by the error message 'Invalid resource group location 'eastus'. The Resource group already exists in location 'eastasia''. In between attempt # 2 & attempt # 3, you presumably deleted the leftover service, but it unfortunately went into a soft-delete state. Because of that, that attempt failed again as ARM is not able to auto-recover soft-deleted resources.
My advice would be to look for that soft-deleted resource, purge it and then run Attempt # 4 for the failed job - with your fingers crossed in that it does not run again into an internal server error. 😄
On a different note, I strongly recommend contributing from a branch other than 'main' next time. 💪 That's simply because each push to main will trigger the workflows.

Thanks @AlexanderSehr. That all makes sense. When I ran it again it did finally successfully complete. Thanks for your help!


wrong documentation for eslint.validate

eslint.validate is now obselated as of eslint extension docs.
please mention that warning or remove eslint.validate from vscode integration section

Thanks for the hint 👍, updated!

Sorry I'm a bit confused. The eslint.validate option is obsolete, but it still seems to be necessary in order for this tool to work correctly in vscode? I can lint .graphql files in the CLI just fine, but in vscode it won't lint any .graphql files unless I specify this option. Is there another option that I'm missing?


Include account information in weep list output

Print the account name as part of the weep list output to help users pick the right account. This will require API changes in ConsoleMe.

Closing issue, since this has been done!


Uncaught Error: Module build failed (from ./node_modules/react-scripts/node_modules/babel-loader/lib/index.js):

Is this a bug report?
Yes

Did you try recovering your dependencies?

1.10.1
Which terms did you search for in User Guide?

Uncaught Error: Module build failed (from ./node_modules/react-scripts/node_modules/babel-loader/lib/index.js):
Environment

npx: installed 63 in 11.394s
Environment Info:
(node:1363) UnhandledPromiseRejectionWarning: TypeError: Cannot read property '1' of null
at a.run.then.e (/root/.npm/_npx/1363/lib/node_modules/create-react-app/node_modules/envinfo/dist/envinfo.js:1:97255)
(node:1363) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:1363) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Steps to Reproduce

(Write your steps here:)

run yarn
browse app
check console

Expected Behavior

App was running perfectly before with the same package.json setup and the same exact code, nothing has been changed, only ran a fresh modules installation and then everything broke, re-installing the node_modules was the only change that has been made, before that app was working, after that, app is not working.
Actual Behavior

I get the following error:
Uncaught Error: Module build failed (from ./node_modules/react-scripts/node_modules/babel-loader/lib/index.js):
Uncaught Error: Module build failed (from ./node_modules/react-scripts/node_modules/babel-loader/lib/index.js):
Error: ENOENT: no such file or directory, open '/src/node_modules/debug/src/debug.js'
    at Object../node_modules/debug/src/debug.js (browser.js:157)
    at __webpack_require__ (bootstrap:782)
    at fn (bootstrap:150)
    at Object.<anonymous> (browser.js:6)
    at Object../node_modules/debug/src/browser.js (browser.js:157)
    at __webpack_require__ (bootstrap:782)
    at fn (bootstrap:150)
    at Object../node_modules/sockjs-client/lib/utils/url.js (url.js:8)
    at __webpack_require__ (bootstrap:782)
    at fn (bootstrap:150)

this is my package.json file:
{
  "name": "wms-pickit",
  "version": "0.1.0",
  "private": true,
  "dependencies": {
    "@material-ui/icons": "^3.0.1",
    "classnames": "^2.2.6",
    "husky": "^1.2.0",
    "lint-staged": "^8.1.0",
    "lodash": "^4.17.11",
    "node-sass": "^4.10.0",
    "prettier": "^1.15.2",
    "react": "^16.7.0-alpha.2",
    "react-dom": "^16.7.0-alpha.2",
    "react-router-dom": "^4.3.1",
    "react-scripts": "2.1.1",
    "typeface-roboto": "0.0.54",
    "uuid": "^3.3.2",
    "serve": "^10.1.1"
  },
  "scripts": {
    "start": "PORT=8080 react-scripts start",
    "build": "react-scripts build",
    "serve": "serve -s -p 8080 build",
    "test": "react-scripts test",
    "eject": "react-scripts eject"
  },
  "eslintConfig": {
    "extends": "react-app"
  },
  "browserslist": [
    ">0.2%",
    "not dead",
    "not ie <= 11",
    "not op_mini all"
  ],
  "devDependencies": {
    "@material-ui/core": "^3.5.1"
  },
  "husky": {
    "hooks": {
      "pre-commit": "lint-staged"
    }
  },
  "lint-staged": {
   "src/**/*.{js,jsx,ts,tsx,json,css,scss,md}": [
     "prettier --single-quote --write",
     "git add"
   ]
 }
}

Yarn lockfile
On Thu, Dec 20, 2018, 8:28 PM Jack Zhao<EMAIL_ADDRESS>wrote:

are you using yarn lockfile or package-lock.json?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/create-react-app/issues/6072#issuecomment-449055794,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMGDYk6QgWYLnOLfVuooglRegUabU-C2ks5u67rEgaJpZM4ZcO1_
.

any idea guys? any input would be appreciated please.

yessir, removed node_modules, re-initiated the whole repo even, thing is, i haven't updated any packages since the last time the app was working, only change is a fresh installation of node_modules (which i found about while deploying the app on a test server).

@mrmckeb i'm on it, meanwhile i just wanted to post more of the errors that i keep getting while trying to fix this, basically i remove node_modules, try to reinstall packages, try with yarn or with npm, etc..it just seems like the dependencies are not getting installed properly, here's one more set of errors that i get while trying to compile:
./node_modules/jss/lib/plugins/rules.js
Error: ENOENT: no such file or directory, open '/src/node_modules/jss/lib/plugins/rules.js'
Failed to compile.

./node_modules/history/es/DOMUtils.js
Error: ENOENT: no such file or directory, open '/src/node_modules/history/es/DOMUtils.js'
Failed to compile.

<EMAIL_ADDRESS>Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>Failed to compile.

<EMAIL_ADDRESS>Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>Compiling...
Failed to compile.

<EMAIL_ADDRESS>Module build failed: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>

@mrmckeb I created a test repo here:
https://github.com/therayess/CRA_DEBUG
I'm simulating how my app breaks by using any react function, in the example you will see i'm using a useState just to demonstrate, and that breaks the app. I hope that helps.

@bugzpodder I installed latest react, 16.7, which doesn't have hooks according to this:
https://reactjs.org/blog/2018/12/19/react-v-16-7.html
And i updated my App.js code to be written in classic react, still getting the same errors.

Also wanted to note that i'm running this app inside a docker container, i included the Dockerfile that i'm using in the sample app.

I'm having the same problem, I updated react and react-dom versions to ^16.7.0-alpha.2, then removed node_modules and yarn.lock files and installed again with yarn.
But hooks still doesn't work, if I log the react runtime version with console.log(React.version);, I still get 16.7.0. How to update the react version? I'd prefer to not eject from create-react-app, or is it necessary?

I tried that.. I got the correct version as I can see in the lock file..
Still got the same errors.
On Mon, Dec 24, 2018, 4:21 AM Joe Haddad<EMAIL_ADDRESS>wrote:

@gusthliv https://github.com/gusthliv you need to add 16.7.0-alpha.2,
not ^16.7.0-alpha.2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/create-react-app/issues/6072#issuecomment-449671807,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMGDYkiwnMEbpWUXiUfJBSRW6j8zDb-iks5u8B4jgaJpZM4ZcO1_
.

@Timer it works. Thanks!

@Timer my bad, it did work, had to re-install container and clear cache then all worked. Thanks!
gonna close this.


fix(admin) - change unauthorized page styling

Signed-off-by: Vlad Tomsa<EMAIL_ADDRESS>Added background styling from 404 page
All boxes should be checked before the PR can be accepted. Exceptions can be made, but they need to be argumented and discussed with the reviewer

BUGFIX
Required

[x] Ran Unit Tests and they pass successfuly
[x] Ran the linter with yarn lint
[x] No TypeScript warnings are introduced

Not Needed

[ ] Translated all the user facing text (static & dynamic)
[ ] I have made corresponding changes to the documentation

References

Include any links supporting this change such as a:

GitHub Issue/PR number addressed or fixed
StackOverflow post
Support forum thread
Related pull requests/issues from other repos

If there are no references, simply delete this section.

By submitting a PR to this repository, you agree to the terms within the Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

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

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


New release including the InjectingPreferenceFragment

Could you please make a new release including the changes to support the new InjectingPreferenceFragment? Or are there any problems with it?

Yes please!

It works, I'm using it in one of my apps.  Sorry, I guess I never got around to putting out a release.  I'll work on that; in the mean time, feel free to just build from master and use that.


Error in MikuMikuModel export

Please answer if you know what the problem is, it will help me a lot, because it is very important to me, thank you very much!
I have some problems in the process of using MikuMikuModel: the model can only be exported in dae format, and the texture cannot be exported.
In addition to the model can be exported, other operations are the following errors:

Did you install the vcredist runtime?
Install both of these and try again:
https://aka.ms/vs/16/release/VC_redist.x64.exe
https://aka.ms/vs/16/release/VC_redist.x86.exe

Unable to install Microsoft .NET Framework
Because a later version is already installed

Yes, I have installed everything I need, but I still get an error. This time I can’t even export the model. I was able to export the model before.


fix: Return system theme when application theme not found

Description
This PR returns the default theme if application theme is not available due to data corruption in DB.
Fixes #20169
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual
Jest

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=d168dde9cfe3ca1d78ccd5e5576f3c0dd3bd214f

/ok-to-test sha=04a6eb7398375f673754cc36f23f96eaa54154e6


Problem with lets encrypt certificates

I've followed this instruction to issue certificates via let's encrypt, but I'm getting this error message:
Warning  CertificateInvalid  9m4s   voyager operator  failed to register user<EMAIL_ADDRESS>Reason: acme: Error 403 - urn:acme:error:unauthorized - Account creation on ACMEv1 is disabled. Please upgrade your ACME client to a version that supports ACMEv2 / RFC 8555. See https://community.letsencrypt.org/t/end-of-life-plan-for-acmev1/88430 for details.
It seems that let's encrypt changed something with their issuing-process.
Can anyone help with this problem ?

Found the issue myself. I was using the deprecated helm chart.


Integrating Invoice Request form into AMY

NumFocus has a Google form used to submit an invoice request.
Right now we have a feature in AMY to hit a button & have that Google form open prepopulated as much as possible.
However, it is still a bit error prone because it still means I have to cross check what's been submitted on the Google form with what's in AMY.  I'd like to think about ways we can integrate this.  This would mean giving NumFocus staff access to AMY.
Possible solution:

Invoice button opens a form in AMY, not an external Google form with the same fields.
Admin fills in rest of information & submits.
NumFocus can see submission, enter in date & notes about when the invoice was sent out and when it was paid, all in AMY.  Right now they are doing this on a Google spreadsheet.
Any of us can then go into AMY to see what the status of each workshop's invoice is.
Include option to have actual invoice amount be calculated based on standard fee less Eventbrite ticket revenue (accessible via Eventbrite API?).  This would be an option & not automatic because sometimes because of bureaucracy and such, a site may not want the Eventbrite revenue applied against the workshop fee.

@maneesha I'm working on this issue now. Question to you: does it happen that you submit multiple different invoices for one workshop?
For example: instead of using the 'breakdown if invoice amount' form field, you submit one invoice request for administrative costs and one for instructor travel costs.
(I'm wondering if the relation is one Event ↔ one invoice request, or does it happen that one Event has multiple invoices.)

It is very rare but it could happen that one workshop has multiple invoices.  This could be because of what you describe, or because the host university is paying out of separate grants, or something like that.


Object.keys() types refinement, and Object.entries() types bugfix

Per @mhegazy's suggestion in #12207, this pr applies the logic from that PR to Object.keys.
It also fixes a bug in that PR (namely, supporting Object.entries([/* some array */])) and adds a test case for that. Finally, it requires that the keys in Object.keys and Object.entries entries be strings, even though the keyof operator can return string | number. (That part of the PR is now covered by #12425.)

Hi @ethanresnick, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@mhegazy I've updated this PR to replace keyof T & string with simply keyof T, as the string intersections are no longer necessary now that keyof always returns a string (from #12425). So this should be good to go.
Cc @ahejlsberg: you expressed skepticism about typing Object.keys with keyof in https://github.com/Microsoft/TypeScript/issues/12289#issuecomment-261113380. But if/now that for in is typed with keyof (#12314), it seems like this makes sense, no?

@ethanresnick My reservations about Object.keys returning (keyof T)[] still hold. It makes sense only in the domain of type variables (i.e. T and keyof T). Once you move to the instantiated type world it degenerates because an object can (and often does) have more properties at run-time than are statically known at compile time. For example, imagine a type Base with just a few properties and a family of types derived from that. Calling Object.keys with a Base would return a (keyof Base)[] which is almost certainly wrong because an actual instance would be a derived object with more keys. It completely degenerates for type {} which could be any object but would return never[] for its keys.
BTW, I have the same reservations about Object.entries which I notice is now typed this way.
Note that the changes we made to for-in specifically apply only when objects have a type parameter type. In for (let x in obj) ... where obj is of type T, if T is a type parameter we infer keyof T for x, but otherwise we infer type string. To do the same for Object.keys would require some way to constrain a type argument to only be type parameter, which currently isn't possible.

@ahejlsberg What you're saying makes sense, assuming I'm understanding it correctly, makes sense. I'll close this PR then. Feel free to revert https://github.com/Microsoft/TypeScript/pull/12207 as well.

@ahejlsberg And what about using it on Readonly<T> type?

When an object has an index though, shouldn't it return the index type?
const a: {[key: number]: any} = {};
Object.keys(a) // should be a number

Basically with an index, we are saying that the index HAS to be a certain type, so the above concern about dynamic keys doesn't follow.

@dyst5422 Keys are returner as strings.
As well in for..in you get strings.
var a={1:2};
alert(typeof Object.keys(a)[0])

@ahejlsberg I'm not sure I understand your reasoning. If you're object has type {[key: string]: number} then Object.keys will return string[]. But if you object is a literal type {a: 1, b: 2} then the return type will be Array<"a" | "b">. That makes sense doesnt it?

@ccorcos no, because lots of objects with other extraneous keys can have type { a: 1; b: 2}. For example,
const x: { a: 1; b: 2 } =  { a: 1, b: 2, c: 3}

for (const k in Object.keys(x)) {
  // what should be the type of k?
}

k: 'a' | 'b' is wrong, becuase k will take on the value 'c' as well!

I see. Sounds like that should be a type error then. haha

I understand the reasoning here and it makes sense, but --strict just made this a lot more annoying because now with strictFunctionTypes you can no longer do this:
interface Foo { foo: string; }
declare const foo: Foo

Object.keys(foo).forEach((key: keyof Foo) => { }) // Error

Error: Argument of type '(key: "foo") => void' is not assignable to parameter of type '(value: string, index: number, array: string[]) => void'.

Which is a pain because with noImplicitAny you can't use string as a lookup:
Object.keys(foo).forEach(key => {
  foo[key] // Error: Element implicitly has an 'any' type because type 'Foo' has no index signature.
})

That's pretty annoying. Is there no way we could allow key to be keyof T? (For cases you're sure T only actually has keys of T)? Since AFAIK there's no way to actually check at runtime that key is in keyof T there seems to be no safe way to ensure that's true, so an assertion seems necessary (without bringing in a runtime type system of some kind).

Yeah, the only other workaround also ends up being kind of gross to write. You can write it, but it's gross:
(Object.keys(foo) as (keyof Foo)[]).forEach((key) => { ... })

I just found my way here after writing up an issue around that b/c of the really poor ergonomics associated with it. Perhaps this could work if the exact types proposal landed? 🤔

I'm not sure why this is worth debating here. It's really easy to write your own function which does this "at your own risk" if you want it:
export function keys<O>(o: O) {
  return Object.keys(o) as (keyof O)[]
}

FWIW, my comment wasn't offered as "debate." I was mostly thinking out loud if there were ways to get both what many users will expect and satisfy the point made up-thread last year. 🙂

I believe the updated version for 2.9 is
export const keys = Object.keys as <T>(o: T) => (Extract<keyof T, string>)[];


Add Show Indexes from tablename syntax

I would like to see the following syntax supported...
Show Indexes from tablename;

This is implemented in version 0.16.0
ql: "show indexes from mytable"
AST: [ShowVariable { variable: [Ident { value: "indexes", quote_style: None }, Ident { value: "from", quote_style: None }, Ident { value: "mytable", quote_style: None }] }]


Help with my three questions

Monolog version 3.3.1
$log->pushHandler(new StreamHandler(__DIR__.'/Warning.log', Level::Warning));
$log->pushHandler(new StreamHandler(__DIR__.'/Error.log', Level::Error));
$log->pushHandler(new StreamHandler(__DIR__.'/Debug.log', Level::Debug));
$log->pushHandler(new StreamHandler(__DIR__.'/Info.log', Level::Info));

Hello

I want when $log->error(); I used it, so that the log is only saved in Error.log, what should I do?
And that I want it to be printed in the terminal at the same time
How to clear the log files? Or limit it and delete it automatically

Use a FilterHandler to select a single level, and put the StreamHandler inside it
Add another StreamHandler with php://stdout as stream
The best is to use logrotate, but we do have RotatingFileHandler as well for basic purposes

Hello @Seldaek , what exactly is meant by "use logrotate" in this context? is logrotate another product/tool/library that can be used with monolog? or some part/configuration/setting of monolog? thank you :)

Here is more info about logrotate https://docs.rackspace.com/support/how-to/understanding-logrotate-utility/ - it's a log rotating utility which is present by default on most linux systems (AFAIK).


Not compatible with 64-bit Windows?

Installed by pip install autosub, then enter autosub, only to find not recognized as an internal or external command, operable program or batch file.
Used where autosub to find the location of autosub executable file, and then found that it had no name extension.
Renamed it to autosub.exe and run, got a ''This version of this file is not compatible with the version of Windows you’re running"
I use Windows 10 21H2 (19044.1706), with Python 3.9.7 (tags/v3.9.7:1016ef3, Aug 30 2021, 20:19:38) [MSC v.1929 64 bit (AMD64)] on win32

There is a autosub file not an exe, how can we build this to exe?


convert samples to new binary format and remove JSON altogether

Still needed in webapp as shared programs have JSON in them

From: Tom Ball @.>
Sent: Saturday, October 14, 2023 11:22:08 AM
To: microsoft/microcode @.>
Cc: Subscribed @.***>
Subject: [microsoft/microcode] convert samples to new binary format and remove JSON altogether (Issue #418)
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/microcode/issues/418 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA73QKPUH5WRJ5I5LEHA3RDX7LJ5BBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJLJONZXKZNENZQW2ZNLORUHEZLBMRPXI6LQMWBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTLDTOVRGUZLDORPXI6LQMWSUS43TOVS2M5DPOBUWG44SQKSHI6LQMWVHEZLQN5ZWS5DPOJ42K5TBNR2WLKJTGI4DOMZSG4YDNAVEOR4XAZNFNFZXG5LFUV3GC3DVMWVDCOJUGM2DKNBRGAYKO5DSNFTWOZLSUZRXEZLBORSQ.
You are receiving this email because you are subscribed to this thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

old shared programs? do we really care about those?


Source navigation isn't working for local variables

Ctrl+Click on local variables aren't directing to definition

Fixed in version 0.1.3


Disable auto-focus on search input

Hello. First of all, I appreciate the work you have done so far. Work beautifully. I have a small problem though. When I toggle the list visible, it auto-focuses on the search box and this brings up the phone's keyboard up automatically. Can I disable this auto-focus?

Ok I got this. https://github.com/toystars/react-native-multiple-select/blob/51b97b710aca4565c7da0a8a60111e34492de93a/lib/react-native-multi-select.js#L540
Just commented out this line and it is done.

Actually you can pass props and do something like autoFocus={false}

thank you!

Hi, passing autoFocus={false} to the multi-select does not work for me. Am i missing something or is the only way to solve this by removing autofocus from the file?

@mo-patel  are passing and object on the property textInputProps? Like this:
textInputProp = {
    autoFocus = {false}
}

@AugustoAleGon Ah, my bad, i completely missed the textInputProp option. It works fine now, thank you very much.

@mo-patel No worries.

this worked for me
textInputProps={{autoFocus:false}}

this worked for me
textInputProps={{autoFocus:false}}


FeatureRequest: Working with overlay file system enabled

For production creating, Overlay File System is a good way to protect sd card. But it seems that when the function is created, pishrink is not working well. Maybe the problem is about resizing a readonly file system?

For me, it's stuck in a boot loop

i would recommend that you just create the image from your sd card before enabling this option. at least until custom first boot scripting is available.

Closing for #286


Fix _vertical-rhythm.scss issue #45

This fixes the bespoke issue #45 where $px-fallback is undefined because of faulty variable scope.

Thanks!


Fixing Juno devices using wrong color temp range

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


comments

we'll be adding comments on this PR

Also, as discussed, please make sure to take an instant snapshot of the data when the pipeline is started, then start syncing each 1m {pollingPeriod}

Done

Have created a PR - https://github.com/neha-Gupta1/conduit-connector-bigquery/pull/12 for all the reviews mentioned in this PR.
Note: I did some implementation on this branch and realized it later. I have reverted to the last commit reviewers did so that reviews are intact and it looks clean. Have raised the above-mentioned PR with all the changes.

https://github.com/neha-Gupta1/conduit-connector-bigquery/pull/12 merged. Should we close this PR @maha-hajja


mesh_utils.create_device_mesh does not correctly create 2x2 ring

Description
For maximum communication efficiency, the 4 chips should be arranged in a ring so each chip is connected to its neighbors. Instead it does this:
In [5]: mesh_utils.create_device_mesh((4,))
Out[5]:
array([TpuDevice(id=0, process_index=0, coords=(0,0,0), core_on_chip=0),
       TpuDevice(id=2, process_index=0, coords=(0,1,0), core_on_chip=0),
       TpuDevice(id=1, process_index=0, coords=(1,0,0), core_on_chip=0),
       TpuDevice(id=3, process_index=0, coords=(1,1,0), core_on_chip=0)],
      dtype=object)

System info (python version, jaxlib version, accelerator, etc.)
jax 0.4.25
TPU v4-8

Hey all, wondering is there any update on this issue, thank you!


Support building parquet with thrift 0.9.0

Add one thrift profile to enable building parquet with thrift 0.9.0
Reporter: Ferdinand Xu / @winningsix
Assignee: ferdinand xu
PRs and other links:

Pull Request Link

Note: This issue was originally created as PARQUET-189. Please see the migration documentation for further details.

Ferdinand Xu / @winningsix:
Thanks @rdblue for your review and commit.

Ryan Blue / @rdblue:
Merged. Thanks @winningsix!


Cleaning up templates in experimental category

fixed #117

These changes don't really seem to address #117. The idea in that issue was to make them simple, so they're not a mashup of various bindings. E.g., for these, maybe we just have them get the queue message and that is it.
What do others think? I think we should rely on doc/samples to show the possibilities for various languages, not the template bindings.

The reason i did not remove table binding from these templates is because these are the only 2 templates with table bindings. I feel we should have at least one sample of table input and output binding until we add more.

Ok.

Addressed the PR comments


Initial Setup

Best way to follow this PR is to look at the ReadMe updates in each commit.

I wouldn't mind a group review to explain the new navigation and "missing" functionality. The rest all seems pretty straight-forward with the understanding of how the old converters worked.

If I knew you were tackling major type-converter changes I would have asked if now is a good time to re-evaluate the use of try/catch for flow control. We had discussed this once in the past and you had a few ideas of how to go about it via a sort of piping/call chain style of flow--is this a good time to revisit that idea?

If I knew you were tackling major type-converter changes I would have asked if now is a good time to re-evaluate the use of try/catch for flow control. We had discussed this once in the past and you had a few ideas of how to go about it via a sort of piping/call chain style of flow--is this a good time to revisit that idea?

I'd love to revisit the idea of moving flow control out of try/catch. In fact this introduces the idea of a result type with success / failure. That doesn't extend into the functions you pass into the pipe or default calls because frankly I think needing to return a success / failure there can make the code harder to read and honestly we need to catch errors when we wrap that code anyway.
This wasn't intended to be set in stone so if you have some design ideas let's chat.

I wouldn't mind a group review to explain the new navigation and "missing" functionality. The rest all seems pretty straight-forward with the understanding of how the old converters worked.

I'm ok doing a group code review on this but I'd to get people looking at it first and commenting so I can hopefully resolve some issues early on.

Closing in favor of #2 which is a direct port of our internal type-converters. I think the better path syntax and some other improvements can be moved over to that in a backwards compatible way before doing the full refresh that was wrapped up in this.

Closing in favor of #2 which is a direct port of our internal type-converters. I think the better path syntax and some other improvements can be moved over to that in a backwards compatible way before doing the full refresh that was wrapped up in this.


Investigate infinity recusive bug in dotnet 2.1

More detail https://github.com/aspnet/Mvc/pull/8255/files

Fixed

would be fixed and closed by https://github.com/zcoinofficial/ztm/pull/157


ls

Hi,
I'm trying to install this code on my local machine to give a try, I'm doing so create a lighter file on the root of my project, named lighter copy and paste you install script, modifying the LIGHTER_VERSION="0.15.1"
I give run permissions to the script, but getting
./lighter.sh: 6: ./lighter.sh: Bad substitution
Anyway the process, continues an create a target folder with the lighter-Linux-x86_64 binary.
Next trying to used it I get,
$ ./lighter --help
./lighter: 6: ./lighter: Bad substitution
usage: lighter COMMAND [OPTIONS]...
Marathon deployment tool
positional arguments:
{deploy,verify}       Available commands
deploy              Deploy services to Marathon
verify              Verify and generate Marathon configuration files
optional arguments:
-h, --help            show this help message and exit
-n, --noop            Execute dry-run without modifying Marathon [default:
False]
-v, --verbose         Increase logging verbosity [default: False]
-t TARGETDIR, --targetdir TARGETDIR
Directory to output rendered config files
Can you please tell me what I'm doing wrong. Thanks for the work, keep it up!!

@mativillagra Thanks for the input!
It should be bin/bash since there is a minor difference we seem to use here, or even "#!/usr/bin/env bash" should be a more portable shebang and should've been used
The script still works as you used it, although producing a warning.
I'll make the change to the example.
Regards, Rasmus


Profile picture does not display in logged in user menu

I'm not sure why is that, the src parameter is passed:

Best course of action is to test with the release version of MUI 0.15 and confirm that the problem persists.

Well, even with the final release of 0.15 it's still not happening.

Jack is back: removing or giving a (valid) value to ->  avatar: { borderColor: null } in the mui-themes folder/files (grayBlue.js etc...) files fixes the problem.

@james0123 thank you I will try this right now.


Ark not backing-up Persistent volumes created from glusterfs

What steps did you take and what happened:
created a backup of jenkins deployment/persistentvolume/service
ark backup create jenkins-backup --selector app=jenkins_backup --snapshot-volumes
What did you expect to happen:
Expected the Persistent volume to be part of the backup.
The output of the following commands will help us better understand what's going on:
(Pasting long output into a GitHub gist or other pastebin is fine.)
Name:         jenkins-backup
Namespace:    heptio-ark
Labels:
Annotations:
Phase:  Completed
Namespaces:
Included:  *
Excluded:
Resources:
Included:        *
Excluded:
Cluster-scoped:  auto
Label selector:  app=jenkins_backup
Snapshot PVs:  true
TTL:  720h0m0s
Hooks:
Backup Format Version:  1
Started:    2018-10-24 12:11:21 +0000 UTC
Completed:  2018-10-24 12:11:22 +0000 UTC
Expiration:  2018-11-23 12:11:21 +0000 UTC
Validation errors:
Persistent Volumes:
time="2018-10-24T12:11:21Z" level=info msg="PersistentVolume is not a supported volume type for snapshots, skipping." backup=heptio-ark/jenkins-backup group=v1 groupResource=persistentvolumeclaims logSource="pkg/backup/item_backupper.go:415" name=pvc-66ec48b9-d785-11e8-8f41-06eac1675816 namespace=default
time="2018-10-24T12:11:21Z" level=info msg="Evaluating resource" backup=heptio-ark/jenkins-backup group=v1 groupResource=persistentvolumes logSource="pkg/backup/resource_backupper.go:138"
time="2018-10-24T12:11:21Z" level=info msg="Listing items" backup=heptio-ark/jenkins-backup group=v1 groupResource=persistentvolumes logSource="pkg/backup/resource_backupper.go:253" namespace=
time="2018-10-24T12:11:21Z" level=info msg="Retrieved 0 items" backup=heptio-ark/jenkins-backup group=v1 groupResource=persistentvolumes logSource="pkg/backup/resource_backupper.go:265" namespace=
Anything else you would like to add:
kubernetes cluster is running on aws
glusterfs is running in the k8s cluster as pods.
jenkins pvc is created from glusterfs storageclass
Environment:

Ark version (use ark version):
Version: v0.9.3
Git commit: v0.9.3
Git tree state: clean

Kubernetes version (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.3", GitCommit:"a4529464e4629c21224b3d52edfe0ea91b072862", GitTreeState:"clean", BuildDate:"2018-09-09T18:02:47Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.6", GitCommit:"a21fdbd78dde8f5447f5f6c331f7eb6f80bd684e", GitTreeState:"clean", BuildDate:"2018-07-26T10:04:08Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes installer & version:
kops version
Version 1.10.0 (git-8b52ea6d1)

Cloud provider or hardware configuration:
aws

OS (e.g. from /etc/os-release):
NAME="Ubuntu"
VERSION="16.04.5 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.5 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

@suhas184 we do not currently have native support for Gluster volumes. Please see our support matrix for the list of supported volume snapshot providers.
I do think you should be able to use our restic integration to back up files in your Gluster volumes, as long as the volumes are not set up in your pods as hostPath (we don't support hostPath volumes with restic for security reasons).


adding pre-initialize task

As a suggestion fix for 863
However, this should perhaps be handled in AzOps and not in the AzOps-Accelerator :)
It's a PowerShell script that will simply run before the Initialize step to check that the required Microsoft.Management resource provider is registered, and if not, it will attempt to register it.

This is the result of the pipeline, in case of error.


various mappings

@kashike Could you please mark your PR with a [WIP] next time so I can pull them in in larger batches?

Looks fine


Deploy is not working

I tried make deploy using the deploy button.
Give an error when bluemix try clone the repository,
http://imgur.com/XxD7jSj

Unfortunately, this demo application doesn't work anymore due to the deprecation of the Q&A service.
We've left the code here to show the integration with Twilio etc.
It won't deploy or work I'm afraid.

You could add this information in the readme file.  :smile: thanks.

I've just opened a pull request (#12) to add this, apologies!


Use newer lief again

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Stuck (again) on https://github.com/conda-forge/boa-feedstock/issues/75

We might need updated lower bounds for conda-build & conda so we update conda 23.3 (as bundled in Miniforge right now) to at least 23.7.4, which will also bring a more recent conda-libmamba-solver.
We might also force this via remote_ci_setup?

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

We might need mamba>=1.5.2 there too, I think.

Alright, that's a combination that works! 🥳

@conda-forge-admin, rerender

something seems to try to pull in an exact mamba version that's too old:

Should be solved by https://github.com/conda-forge/conda-forge-ci-setup-feedstock/pull/290; I'll restart

Nope, still getting lost with looking for 1.4.2

That might be a manifestation of an conda-libmamba-solver bug. The miniforge we are using bundles 23.3 and that one was a bit stubborn about the history if things were not explicitly mentioned in the CLI. Maybe using conda_install_tool=mamba helps? I'll add a comment.

@conda-forge-admin, rerender


Add LICENSE

Adds project copyright licence of MIT and Apache 2.0

Added LICENSE-MIT, LICENSE-APACHE and LICENSE
Updated README.md with Contributing and License Information
Added CODEOWNERS file

changes made in this PR have been reverted

changes made in this PR have been reverted


Support custom ssh binary / ssh options?

Hi! I'm looking to supply a custom identityfile to ssh. It looks like the ssh command is hardcoded in attic/remote.py and at the moment it's not possible to add custom commandline to the invocation of the ssh binary (correct me if i'm wrong...)
Would it be possible to add some e.g. --ssh-command option, so that i can write e.g. --ssh-command="/opt/my/custom/built/ssh -i path/to/id_rsa -o foo=bar" ?
I guess it would be currently possible to workaround by hacking $PATH and pointing ssh to a local wrapper script... but surely custom identity files and other SSH options are common enough to warrant having this in the core (e.g. rdiff-backup has --remote-schema)

Unfortunately it is not possible ("You may only use this configuration option from the command line") to put the "NoneSwitch" option into ssh config. This option is available with the HPN (https://www.psc.edu/index.php/hpn-ssh) patched ssh package. Together with "NoneEnabled" it disables encryption during data transfer which helps to speed up backups over ssh in environments where transport encryption is not necessary (e.g. home LAN) but mounting a writable backup destination is not an option (I don't want my backupped data getting accidentally deleted by the client because someone does an "rm -rf /" or something similar). So at least for this use case it would be good to be able to set custom ssh options with attic.


Adding hosted crawlers section

Is this something that fits with this awesome list? I have a bunch others I can dig up and add to this section.

@BruceDone should I reopen in another PR? Or are you looking for this list not to include hosted crawlers.

Hi @iloveitaly  ,really appreciate for your pr, but this hosted crawler is just some kind of api ,or service , in this project ,we should put in the source code and project ,so i closed this pr.


enforce min_num_years after dropping sites with missing env data

Should finally fix #62

Thanks @sdtaylor. Looks good.

Worked great. Notebook updated in 090c26523bda0ed5df1fbc0eeab1ac4fd7f85236.


Use new alpha release of h2

Replaces #1961

Hmm, not sure why Travis isn't happy. The build succeeded over at https://travis-ci.org/hyperium/hyper/builds/592104830.


Broken bazel test: //test/core/bad_ssl:bad_ssl_cert_server

E1130 00:36:49.964151926      15 server_common.cc:42]        assertion failed: addr
Aborted (core dumped)

https://source.cloud.google.com/results/invocation/05f9065e-93e5-4b35-ad1f-ab6bfa8b383a/targets/%2F%2Ftest%2Fcore%2Fbad_ssl:bad_ssl_cert_server?page=log

Hi David, could you please triage and assign this bug to a proper owner? Thanks!

I took a look. The current bazel rule is outdated and does not make sense. It just runs a server. The actual test is bad_ssl_cert_test which does not have a rule, and adding a rule for that does not seem possible without changing the test.
The way this test is written, they start a server as a separate process, and then run the test. For this to work as a test, we would need the ability to add a binary (for the server) as a dependency to the test. But, that is not allowed from what I read in the bazel rules.
So, the options I see are -

Rewrite the test so that it works with bazel
Simply not covering this test with bazel

Adding @nicolasnoble

Thank you for looking! Adding the binary as data dependency should work.

Update: using it as a data dependency worked. I didn't expect data dependencies to allow cc_binary

It's now failing with https://source.cloud.google.com/results/invocations/de9624c7-15ba-4216-8e18-ba8d44247728/targets/%2F%2Ftest%2Fcore%2Fbad_ssl:bad_ssl_cert_test/log


[WIP] Spectroscopy and calibrations integration V1

Summary
This is a WIP PR to show and discuss how spectroscopy and calibrations can be integrated through the BackendCalibrations class. See an alternative here: #79.
Details and comments
The spectroscopy experiment is integrated with the calibrations through the BackendCalibrations class. The user will execute the following code:
calibrations = BackendCalibrations(qubit)
...
calibrations.qubit_spectroscopy(qubit=5, force_update=True, group="my_group", experiment_options={"amp": 0.2})

The pros of this integration are:

The user does not need to write much code.
The analysis class of spectroscopy is not changed.

The cons of this integrations are:

The BackendCalibrations will contain a function for each calibration experiment that we will support and may become large.

If we go on a dedicated function, then there are three options:

qubit_spectroscopy as a method of calibrations (this PR).
Same method but outside of the calibrations class, not belonging to any class, but written in some utils file.
A composite experiment (overkill).

All these three options share the same pros and cons as written above. There is an additional con (also shared by all of them): although the user enjoys the benefit of not writing a lot of code, she will also suffer from the large number of different functions. She will have to understand which functions exist, what exactly they do, how to use them, what are the differences between them. So a large number of functions to maintain can be a headache not only to developers, but also to users.

Closed by #88


After a power failure, cannot launch spyder

Description of your problem
After a power failure, I cannot launch spyder anymore. I already tryed spyder --reset, uninstalling and installing again. Every time I try to open it, this message appears "

"
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
After the power failure, the folder jedi-0.10.2.dist-info was corrupted. I already removed Jedi using pip uninstall jedi and installed it again.
Versions and main components

Spyder Version: 3.2.1
Python Version: 3.5.3
Qt Version:
PyQt Version:
Operating system: Windows 10

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:

Please try to reinstall Spyder. I really don't know what's happening in this case, so that's my only advice, sorry.


Checkpointing fix

Fixes the issue with use of the checkpointing functionality when multiple finalize() calls are made (#101). Core part of the issue was that when finalize() is called, the function output_strata() is used which tries to define new dimensions and variables - NetCDF does not let you just over-write data like this.
To resolve the issue, the old netCDF file is renamed, and then all of its contents are copied into a new netCDF file excluding values that will be created by output_strata(), then the old netCDF file is deleted. In this way, finalize() can be called and the associated variables can be created at that time without any issues. I don't think this method of copying netCDF variables from one file to another loads them into memory, so this isn't expected to cause RAM issues or be overly slow/cumbersome.
Besides that the PR is mostly unit tests to ensure checkpointing works as intended. Some of the unit tests on windows seem to be failing randomly now? These are not tests (nor functionality) that was modified by this PR, and the failures seem to be related to accessibility of netCDF4 output files... so that is a bit of a mystery, maybe every test should call finalize() at the end to close the associated output file?

To do:

[x] add unit test where checkpoint_dt does not align with save_dt or dt
[x] add unit test where model is run, checkpoint, and finalized - resumed, multiple times
[x] add test for parallel functionality + checkpointing
[x] add test for case where yaml has resume_checkpoint: True but no checkpoint exists

Closes #101.

Codecov Report

Merging #132 (59daaad) into develop (7088ae6) will decrease coverage by 1.39%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #132      +/-   ##
===========================================
- Coverage    81.51%   80.12%   -1.40%
===========================================
  Files           11       11
  Lines         2121     1977     -144
===========================================
- Hits          1729     1584     -145
- Misses         392      393       +1

Impacted Files
Coverage Δ

pyDeltaRCM/init_tools.py
99.72% <100.00%> (+0.02%)
:arrow_up:

pyDeltaRCM/model.py
90.10% <100.00%> (-2.64%)
:arrow_down:

pyDeltaRCM/shared_tools.py
44.82% <0.00%> (-5.18%)
:arrow_down:

pyDeltaRCM/water_tools.py
62.10% <0.00%> (-0.91%)
:arrow_down:

pyDeltaRCM/sed_tools.py
35.96% <0.00%> (-0.76%)
:arrow_down:

pyDeltaRCM/preprocessor.py
92.40% <0.00%> (-0.54%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #132 (59daaad) into develop (7088ae6) will decrease coverage by 1.39%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #132      +/-   ##
===========================================
- Coverage    81.51%   80.12%   -1.40%
===========================================
  Files           11       11
  Lines         2121     1977     -144
===========================================
- Hits          1729     1584     -145
- Misses         392      393       +1

Impacted Files
Coverage Δ

pyDeltaRCM/init_tools.py
99.72% <100.00%> (+0.02%)
:arrow_up:

pyDeltaRCM/model.py
90.10% <100.00%> (-2.64%)
:arrow_down:

pyDeltaRCM/shared_tools.py
44.82% <0.00%> (-5.18%)
:arrow_down:

pyDeltaRCM/water_tools.py
62.10% <0.00%> (-0.91%)
:arrow_down:

pyDeltaRCM/sed_tools.py
35.96% <0.00%> (-0.76%)
:arrow_down:

pyDeltaRCM/preprocessor.py
92.40% <0.00%> (-0.54%)
:arrow_down:

Continue to review full report at Codecov.

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

also, one of the tests is issuing a warning "UserWarning: Replacing existing netCDF ", if you can figure out which one that is and wrap that line in a with pytest.warns (https://docs.pytest.org/en/stable/warnings.html#warns) that would clean up the output.
the offending output in log
Don't worry about all the numpy warnings

I think if this passes the CI tests it's pretty much good to go. Adding finalize() to the tests that caused the Windows VMs some trouble seems to have resolved things.


Hotfix for frame observing on iOS 8

Hi!
Maybe its not the exact solution you wanted, but I just wanted to say there is an issue with the (hazardous) view frame observing on iOS 8. This patch is a workaround for this problem.
Cheers and thanks for the great work!

I noticed the same issue with ios8, this change fixes it.


Chart is wrong

closing since this issue contains no information.  I have no idea what the image is supposed to tell me.   A picture of your cat would have been just as useful.

Hi there,
This also happens to me when I increase the number of users to a couple hundreds and more:

No error reports. I can also see the movement in my monitoring:

@oytuntez It's very hard to guess what might be causing this without more information.
When the chart drops, is the system that you're testing still responsive? How about the CPU usage on the Locust nodes?
Do you get any JS-errors on the Locust webpage?

Yeah, I tried to debug this for a bit, no JS or Python logs, file descriptors might be a problem (but I have 7168 descriptor available).
I will keep this thread updated if I encounter more of this.


Create Typescript Declaration file (*.d.ts)

Let's try to figure out how to expose all of our proptypes so that typescript projects can use these components with accurate typing. Same as https://github.com/SelectQuoteLabs/SQHooks/issues/6 in SQHooks

The package react-to-typescript-definitions seems like a good option. I created a script to try it out, this is the output when fed SQFormDropdown:
declare module "SQFormDropdown" {
  import * as React from "react";

  export interface SQFormDropdownChildren {
    label?: string;
    value?: string | number;
    isDisabled?: boolean;
  }

  export type SQFormDropdownSize =
    | "auto"
    | 1
    | 2
    | 3
    | 4
    | 5
    | 6
    | 7
    | 8
    | 9
    | 10
    | 11
    | 12;

  export interface SQFormDropdownProps {
    /**
     * Dropdown options to select from
     */
    children?: SQFormDropdownChildren[];
    /**
     * Whether to display empty option - - in options
     */
    displayEmpty?: boolean;
    /**
     * Disabled property to disable the input if true
     */
    isDisabled?: boolean;
    /**
     * Required property used to highlight input and label if not fulfilled
     */
    isRequired?: boolean;
    /**
     * Label text
     */
    label: string;
    /**
     * Name identifier of the input field
     */
    name: string;
    /**
     * Custom onBlur event callback
     */
    onBlur?: (...args: any[]) => any;
    /**
     * Custom onChange event callback
     */
    onChange?: (...args: any[]) => any;
    /**
     * Size of the input given full-width is 12.
     */
    size?: SQFormDropdownSize;
    /**
     * Any valid prop for material ui select child component - https://material-ui.com/api/select/#props
     */
    muiFieldProps?: Object;
  }

  const SQFormDropdown: React.FC<SQFormDropdownProps>;

  export default SQFormDropdown;
}

Not 100% sure I got the options right, but it looks pretty good


Add scraper for PH

Location name

PH
Source URL

https://www.doh.gov.ph/covid19tracker
Notes/comments

Tablaeu :'(

https://drive.google.com/drive/folders/1w_O-vweBFbqCgzgmCpux2F0HVB4P6ni2
They have a bunch of spreadsheets in here with sweet data, just gotta figure out how to get the crawler to access them.


Add a splash screen to the program

Hey I finished up the splash screen for the program, I have also seen you solved the ISSUE with the flag generation will get around to some local tests

Nice one!


Refactor blob service properties

Refactors setting of BlobServiceProperties due to error ConflictFeatureEnabled in case of Azure Backup enabled

Accidentally contains changes of pull request #6, too. Closing.


Add ability to optionally add icon to a tab

#508  👈
main

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

Join @nickgraz and the rest of your teammates on  Graphite

publish rc


New CI integration request: OpenLab

Hi everyone!
The OpenStack cloud plugin is quite valuable in both the Jenkins and the OpenStack ecosystem because it allows for the use of one-use slaves which has many advantages, the plugin has a fairly reliant userbase as well.
Unfortunately, there does not seem to be any gating against OpenStack for this and testing is mostly done by manual work against clouds deployed by reviewers.  This probably slows down review and makes it much harder to make sure things are running properly.
OpenLab uses donated infrastructure from several public cloud providers (including us!) to test ecosystem applications.  I'd like to ask if the project would be interested in having it tested by OpenLab.
My ideal goal would be to deploy OpenStack via DevStack and run all the tests against it for every pull request, this will significantly facilitate testing and make it easier to do contributors without worrying about breaking anything.
If this is something that interests the team, we would need 2 main things:

A webhook must be installed in order to notify the Zuul CI system to trigger builds
A .zuul.yaml file will be added to the repo which contains all job definitions.

I can personally pick up the work to get the jobs done if need be, but I think this would be a great option that would hopefully increase the stability of the project.
Thank you!

Mohammed, those are great news! I am definitely interested in public end-to-end testing of the plugin. Do you mind doing the effort of prototyping the zuul file and handing me the webhook url to get this started? Is there any example documentation for github integration? Thanks!

Good morning Oliver,
That’s great. I’m very happy to hear this.  The only thing you have to do at your side is add this app:
https://github.com/apps/theopenlab-ci
From there, I will push up a PR and start working on a Zuul file which deploys the latest OpenStack and runs the end to end testing against that cloud.
Thanks!

Application installed for the project.

Hi @mnaser  @olivergondza ,
I am one of the maintainers of the OpenLab, but I am not familiar with this project. I was just told that a request proposed to integrated with the OpenLab CI, as mnaser described, need to install the github app and add the job definitions included in a .zuul.yaml. currently, we have integrated some projects into the OpenLab CI, the jobs definitions included in the  repo. you can also define jobs in-repo (in the .zuul.yaml).
Additionally, Zuul will scan all the branches of the integrated repos and will try to load jobs definitions of every branches, it is recommended to set the developing branch we want to trigger jobs to be protected branch, usually it is the master branch. after setting that, Zuul will only care the protected branch.
I am glad to help to testing and make it work with the OpenLab CI. if you need any help, please let me know :)

Hi @liu-sheng
If Zuul is ready and configured, I'd be ready to push up a PR for an empty noop job and then start working on jobs after once we're sure everything is going well.
FYI, I'm core on openstack-infra/project-config so I'm very familiar with Zuul's inner workings :)
Thanks,
Mohammed

Awesome @mnaser ! the OpenLab is ready to go. please go ahead to test, if it has any problem, please let me know :)

hey @mnaser just following up with you on this to see where things stand


IntIterator.clone() causes NullPointerException sometimes

E. g. in ImmutableRoaringBitmap.ImmutableRoaringReverseIntIterator, if there are no elements, iter field is not initialized (remains null), and clone() fails with NullPointerException at this.iter.clone().

@leventov  Would you kindly review this commit?
https://github.com/RoaringBitmap/RoaringBitmap/commit/3a1bb7ceb86ea48ba62c0ca39960412f325fcc4f
When you approve of it, I will issue a release and close the issue.

@lemire LGTM


(PDB-4606) partition the reports table

Partition the reports table on producer_timestamp.  This follows the same pattern as PDB-4420.
The report hash has been changed to remove resource_events from the
hash calculation. The resource_events table can be garbage collected
separately from the reports table, and if the hash is still including
the resource_events, we end up with duplicate reports during a pdb
sync. The hash is rewritten during the migration.
There is no trigger to direct inserts to the appropriate partition.
This is because the usage of the reports table relies on the behavior
of a RETURNING clause on the INSERT INTO statement. You cannot rewrite
the query and combine it with a RETURNING clause. All INSERTs occur
directly into the appropriate partition.
During garbage collection, an UPDATE statement is ran across the
certnames table to ensure latest_report_id is set to null when the
report is deleted. Previously, this relied on a cascade during the
delete.

CLA signed by all contributors.

Force pushed to rebase