Update Composer dependencies (2020-08-31-00-13)

Visual regression test passed!
View the visual regression test report


Can You Make A Youtube Tutorial On How To Install It Step By Step Please?

I just tried to install everything from a different (Windows) device. Some problems emerged so I updated the README.md tutorial section accordingly

Also, a video just showing the software at work would be cool!

A video should come out next week. It will include installation and Demo

Video out! https://youtu.be/AcCG7DOwhG8
Hope it helps.

thank you very much


bug with get_filtered_records function

I am not able to download data using the python library. It could be an issue on my end, but I don't think so
When I follow instructions here, I get this error
Traceback (most recent call last):
  File "test_openeew.py", line 18, in <module>
    eew_data = eew_aws_client.get_filtered_records(start_time, end_time)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/openeew/data/aws.py", line 350, in get_filtered_records
    key_records = loop.run_until_complete(
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/home/ubuntu/.local/lib/python3.8/site-packages/openeew/data/aws.py", line 305, in _download_keys
    key_records = await asyncio.gather(*coros)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/openeew/data/aws.py", line 276, in _get_records_from_key
    await async_s3_client.download_fileobj(
AttributeError: 'ClientCreatorContext' object has no attribute 'download_fileobj'
sys:1: RuntimeWarning: coroutine 'AioSession._create_client' was never awaited

I'm having the exact same issue.
Trying to execute code as written in https://openeew.com/docs/analyze-historic gives
RuntimeError                              Traceback (most recent call last)
<ipython-input-10-5ac17f81d3ef> in <module>
----> 1 records = data_client.get_filtered_records(start_date_utc, end_date_utc)

D:\Applications\Anaconda3\lib\site-packages\openeew\data\aws.py in get_filtered_records(self, start_date_utc, end_date_utc, device_ids)
    348         loop = asyncio.get_event_loop()
    349
--> 350         key_records = loop.run_until_complete(
    351                 self._download_keys(keys_to_download)
    352                 )

D:\Applications\Anaconda3\lib\asyncio\base_events.py in run_until_complete(self, future)
    590         """
    591         self._check_closed()
--> 592         self._check_running()
    593
    594         new_task = not futures.isfuture(future)

D:\Applications\Anaconda3\lib\asyncio\base_events.py in _check_running(self)
    550     def _check_running(self):
    551         if self.is_running():
--> 552             raise RuntimeError('This event loop is already running')
    553         if events._get_running_loop() is not None:
    554             raise RuntimeError(

RuntimeError: This event loop is already running```

Then, executing the jupyter sheet available in this respository with the nested asynchio fix

Allow nested asyncio event loop
See https://github.com/erdewit/nest_asyncio
import nest_asyncio
nest_asyncio.apply()
produces the same error as @mgrobaker .

Thoughts on this @andygrillo?

Thanks for reporting this error. It looks like it's caused by a breaking change in aiobotocore to do with how clients are created: https://github.com/aio-libs/aiobotocore/blob/master/CHANGES.rst#100-2020-03-31.
As mentioned in https://github.com/terrycain/aioboto3#async-aws-sdk-for-python, clients must now be created as async context managers. As it happens, this change has already been made as part of the unreleased https://github.com/openeew/openeew-python/pull/10. Until a new version of the library has been released, using the current master branch should work.


grammar.gedstruct      has infinite loops

in     https://github.com/FamilySearch/GEDCOM/blob/main/extracted-files/grammar.gedstruct
we have types, that reference    types  going round in a loop
, for example, look at the pattern   we see a loop starting at line 569
hence, the schema has become an infinite loop, and we can't build from it
GEDCOM 5.5.1      carefully avoided this type of error.
`
line=35       "        +3 <<ADDRESS_STRUCTURE>>           {0:1}"             in HEADER
line=52       "  +1 <<NOTE_STRUCTURE>>                    {0:1}"             in HEADER
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
line=614       "     +2 <<DATE_VALUE>>                     {0:1}"             in SOURCE_CITATION
line=623       "  +1 <<MULTIMEDIA_LINK>>                   {0:M}"             in SOURCE_CITATION
line=624       "  +1 <<NOTE_STRUCTURE>>                    {0:M}"             in SOURCE_CITATION
line=569       "  +1 <<SOURCE_CITATION>>                   {0:M}"             in NOTE_STRUCTURE
java.lang.RuntimeException: Recursive
`

GEDCOM 5.5.1 carefully avoided this type of error.

I disagree, v5.5.1 has this issue as well.

Actually,  now you have jogged my memory,
When I created the machine readable ged.5.1.1.txt,
I think I had to remove  something to prevent infinite loops
At least we agree, that this is still an issue.,       even if not a regression.

The problem is that the term “source” is used incorrectly!  The design and people say, the provider of a fact, object or note/statement is the “source” of the information, we should be using a citation referencing the provider. Therefore, an object, note, fact are cited, but a citation would never have an object, note or fact associated with it (no provider), so no circular reference! GEDCOM needs a citation record and deprecate the “Source_Citation” and the Source_Record!

The spec already identifies this case explicitly

A NOTE_STRUCTURE can contain a SOURCE_CITATION, which in turn can contain a NOTE_STRUCTURE, allowing potentially unbounded nesting of structures. Because each dataset is finite, this nesting is also guaranteed to be finite.

I do not see an issue here. This is as designed. Can you clarify what is wrong with it as it stands?

To fix this you would need a new   "NOTE_STRUCTURE_FOR_SOURCES"
"A NOTE_STRUCTURE  can contain a SOURCE_CITATION, which in turn can contain a NOTE_STRUCTURE_FOR_SOURCES"
NOTE_STRUCTURE_FOR_SOURCES  doesn't contain any SOURCE_CITATION's  (preventing looping)
It would be very helpful if this was part of the spec.

The option of deep nesting is a feature that has been supported since NOTE.SOUR was added in version 5.5 (1996) and has valid semantic meaning. To remove it would be a breaking change, and thus could not happen before version 8.0 at the earliest.
That said, while I've never gotten to arbitrary depth I have used the SOUR.NOTE.SOUR.NOTE several times to express things that really benefit from this structure, and I'd be sad to see this removed from the spec. I'm also not seeing why we need to remove it, unless it is to support the specific operation of enumerating all possible paths.
A cleaned up example of how it can be used to convey valid semantic information is
1 BIRT
2 SOUR @S1@
3 NOTE The applicability of that source to this event has been challenged
4 SOUR @S2@
5 NOTE Several blogs have weighted in on the validity of this source's arguments in this case
6 SOUR @S3@
6 SOUR @S4@

...where S1 is the source record, S2 is an article questioning it, S3 and S4 are blog posts about if S2 applies to S1 or not. And there's no need for this to stop at any given depth: I might have notes about if S3 actually applies to my note on S2, and sources for that note, and so on.

First let me say that I'm not advocating the removal of these relationship per se!
I have actually seen this in a GEDCOM I received!
An image O1 was added and some one wanted to give credit to the provider of the image, so they created a "Source of the Image" record S2.  (An aside: This is why I indicated that images, notes, and facts should not have "sources" but should have a citation record that has no lower links).
S2 receives a NOTE N3 describing some additional information about the source.
At a later date a person looking at N3 decides that the NOTE needs a link to the image O1 because the NOTE refers to something in the image, and when displaying the NOTE N3 they want to see the image O1 along side the note.
And thus we have a loop.
Again I'm not advocating changing the current relationships, an object should have notes and notes should have objects, but something in the program implementing these relationship needs to stop O1 from having a link to N3 when N3 already has a link to O1!  This is a simple check, but not so simple in the first example!

thanks for the analysis, of this issue
it seems like GEDCOM can have an infinite number of paths,
I will think it over, and how this could be implemented


sql/analyzer: do not use plan.Inspect to find available tables

Friday night this possible bug came to my mind, but PR was merged because I got a chance to fix it tomorrow 😅
Because of how plan.Inspect works, whenever a table is found in a branch the other branches are not inspected anymore. Instead, doing recursive calls all branches are visited.

Shall we merge this, @ajnavarro?


Add link to metadata endpoint docs

LGTM!

Thanks for reviewing! I forgot to add the skip ci tag and the failure is unrelated to my change so I am going to merge.


Disable autoconverting HTML entity references

As the title says, is there a way to disable autoconverting HTML entity references, e.g. α to α? I am saving the converted HTML in a DB which is Windows-1252 encoded (can't be changed right now) so having some problems.

What if the input contains α already? Just having an option to disable converting HTML entity references wouldn't be enough.
We could add an option to the HTML renderer to encode all non-ASCII characters using numeric character references, but I'd rather not encourage using legacy encodings.
Can you do post-processing on the resulting text instead as described above? You'd have to be careful to handle surrogate pairs correctly, but I can help you with that.

I used post-processing as you suggested.

Cool. Would you mind sharing that piece of code here? Surrogate pairs can be tricky to get right, maybe other people would also be interested in this.

I didn't deal with surrogate pairs, I just put something together that worked for our use-case.
That could be useful for others so I'll leave it here but it is basic and hacky and probably won't be enough for most.
private static final Pattern sf_htmlEntityPattern = Pattern.compile("&(#[xX][a-fA-F0-9]{1,8}|#[0-9]{1,8}|[a-zA-Z][a-zA-Z0-9]{1,31});");
private static final Pattern sf_escapedHtmlEntityPattern = Pattern.compile("&(#[xX][a-fA-F0-9]{1,8}|#[0-9]{1,8}|[a-zA-Z][a-zA-Z0-9]{1,31});");

/**
   * Escape HTML entities (e.g. α) as the markdown parser auto-converts to unicode characters.
   * &xxx; escapes to &xxx; then find and replace these using unescapeHtmlEntities.
   *
   * Call escapeHtmlEntities on pre-parsed markdown and unescapeHtmlEntities on renderer html.
   */
private static String escapeHtmlEntities(String text) {
  return sf_htmlEntityPattern.matcher(text).replaceAll("\\\\&$1;");
}

private static String unescapeHtmlEntities(String text) {
  return sf_escapedHtmlEntityPattern.matcher(text).replaceAll("&$1;");
}

Ah, I see, thanks for sharing. When you're doing it like this, you don't have to handle surrogate pairs, so that's fine.
But if there's already non-ASCII in your input, that won't be escaped. But I guess in your use case you ensure that the input is already ASCII only?

@robinst, I would like to voice my opinion that there really should be an option to disable this sort of auto-conversion. I build my own workaround, but think this should be a feature (as is evident by the multiple closed-without-a-fix issues requesting it, like this one). I couldn't use the escaping method mentioned by @md384 (though it did borrow from it) because it was messing up my anchor links, which do need to strip out special characters. Here's what I did:
	private static final Pattern htmlEntityPattern = Pattern.compile(
			"&(#[xX][a-fA-F0-9]{1,8}|#[0-9]{1,8}|[a-zA-Z][a-zA-Z0-9]{1,31});");

	/**
	 * Converts the given Markdown to HTML and writes to a file
	 * @param markdown
	 * @param outputFile
	 * @throws IOException
	 */
	public static void writeHTML(String markdown, File outputFile) throws IOException {
		List<Extension> extensions = Arrays.asList(TablesExtension.create(), HeadingAnchorExtension.create());
		Parser parser = Parser.builder().extensions(extensions).build();
		Map<String, String> htmlSymbols = new HashMap<>();
		// track any html entities that were explicitly included, so that we can re-escape them after commonmark un-escapes them
		Matcher symbolMatcher = htmlEntityPattern.matcher(markdown);
		while (symbolMatcher.find()) {
			String symbol = symbolMatcher.group();
			if (!htmlSymbols.containsKey(symbol)) {
				String concrete = StringEscapeUtils.unescapeHtml4(symbol);
//				System.out.println(symbol+" => "+concrete);
				htmlSymbols.put(symbol, concrete);
			}
		}
		Node document = parser.parse(markdown);
		HtmlRenderer renderer = HtmlRenderer.builder().extensions(extensions).build();

		FileWriter fw = new FileWriter(outputFile);
		String html = renderer.render(document);
		for (String symbol : htmlSymbols.keySet())
			html = html.replaceAll(htmlSymbols.get(symbol), symbol);
		fw.write(html);
		fw.close();
	}

@kevinmilner What is your reason/use case for that? Would encoding non-ASCII characters as character references work?

@kevinmilner What is your reason/use case for that? Would encoding non-ASCII characters as character references work?

@robinst I've noticed portability problems. Specifically, I generate Markdown programatically with HTML4 symbols, then use commonmark-java to convert it to HTML. A user of my code recently did this on a Mac, then uploaded to the resultant HTML to a linux server and all of the symbols were garbled. Here's the generated HTML that has garbled greater-than-or-equal-to symbols (see Mâ‰¥5.0 Hazard Change Over Time in the table of contents), and here is the markdown used to generate it, which includes text like M≥5.0 Hazard Change Over Time.
Thanks for the reply and the great package!


Split hub-depositing UI into (a) depositing and (b) waiting for mining

I think this would require the hub sending the transactionId of the deposit transaction to the xstate-wallet.

I think this would require the hub sending the transactionId of the deposit transaction to the xstate-wallet.

The wallet could also monitor the chain and identify a transaction based on the channelId

I think this would require the hub sending the transactionId of the deposit transaction to the xstate-wallet.

The wallet could also monitor the chain and identify a transaction based on the channelId

Currently since we're not waiting for a transaction to be x blocks deep, waiting for the transaction to be published to the chain is the same as waiting for the event to occur, I believe?
If our strategy around chain re-orgs is to just wait until a certain amount of blocks, then this probably makes sense once we're doing that.

I think we should add transaction to the wire protocol like @tomclose suggested


No picker on modal

I have this code inside modal(made by jquery, not angular-bootstrap lib)
<div class='input-group date datetimepicker'>                     <input type='text' class="form-control" datetime-picker ng-model="promo.time_from" date-format="yyyy-MM-dd HH:mm:ss"/>                 </div>
When i click on input there's no reaction. I'm looking at the end of page code, but there's no new elements, so this module is not reacting at all. ;/

My bad, modal was outside ng-app. Sorry


Ways to access all billing data in the SEA

Describe the Issue

Create a way to access all billing data across all project sets within the AWS SEA
Additional Context

Acceptance Criteria
3 possible options

Fix the billing role for the management account
fix rls lambda to check master account billing role access and give access to all billing data
create a Keycloak group that only has QuickSight reader role and make rls lambda to give access to all data

No longer relevant - SEA outdated - will need to reconsider for LZA


Multinode on MPS

What does this PR do?

Fixes #15713
Distributed Training is not supported on MPS devices -> catches this for Lite and PL Multinode components, nothing we can do on raw pytorch though.
Also adds unittests for lite and pl multinode executors beyond the changes on this PR.
Does your PR introduce any breaking changes? If yes, please list them.
None
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @borda

@Borda Yes it is and this is why it is skipped for non-mps.
The other tests are missing tests added to ensure I break nothing

@justusschock, I have pretty much just set to run the test for lightning only not for Lite/App/Pytorch... I hope it is fine, mind checking it :otter:


Differentiating e (Euler's constant) and scientific notation e?

Is this the expected behaviour?
When I enter "5e" the e is interpreted as Euler's constant (2.71xxx).
When I enter "5e0" its it evaluated as 5.
I would expect 5e to be interpreted as 5e0 (or an error).
We already have e() to get Euler's constant. Why does a lonely 'e' get interpreted as (e)?

The 5e0 is scientific notation for 5 x 10⁰. However, scientific notation only works if there's an exponent number (the 0). If it's not there, then the parser will only parse the number as 5, since 5e is not a number. Once 5 has been parsed as a number, it parses the next token as e. e is interpreted as a function expression, under the rules of argument-less functions. And since there's a function defined as e() (to return 2.718....), 5e gets parsed as 5 * e().

Thanks! That was quick :-)
As you know I have an iOS app that looks like a regular calculator (but supports long formulas) so it feels a bit weird while people type '5e' and get ready to type the exponent and it appears as (2*2.71...).
Where'd you'd think it would be the best place to overload this behaviour without messing stuff up too much.? I am thinking about disabling argument-less functions altogether but maybe there's a simple way. I wish I could just rename e() to euler() or something like that.

I am thinking about calling     [[DDMathEvaluator defaultMathEvaluator] unregisterFunctionWithName:@"e"];
And then implementing and registering my own euler() (basically copying the existing one). Is that crazy?
I am afraid since unregisterFunction is deprecated ...

Well I found a frictionless way to do it, subclassing DDMathParser and overloading the evaluateFunctionExpression method.
It's a bit dirty but it's better than maintaining a fork just for that.`

(NSNumber *)_evaluateFunctionExpression:(_DDFunctionExpression *)e withSubstitutions:(NSDictionary *)substitutions error:(NSError **)error {
NSLog(@"evaluating function express:%@",[e function]);
if ([[e function] isEqualToString:@"e"])
{
*error = [NSError errorWithDomain:DDMathParserErrorDomain
code:DDErrorCodeUnresolvedFunction
userInfo:@{NSLocalizedDescriptionKey: [NSString stringWithFormat:@"unable to resolve function: %@", [e function]],
DDUnknownFunctionKey: [e function]}];
return nil;
}
(...)
`
Closing this issue. Thanks for your comments!

I think an easier approach might be to delete the implementation of the e function: https://github.com/davedelong/DDMathParser/blob/master/DDMathParser/_DDFunctionEvaluator.m#L931

Sure. That was the obvious one. I wanted as much as possible to avoid maintaining a fork just for that. Thanks for your kind help!


Unexpected panic when using datetime formatter

While experimenting with datetime formatting, I accidentally entered a bad format and caused a panic. Here it is with standard backtrace:
>>> format_datetime("%Y-%m-%dT%H%:M", now())
thread 'main' panicked at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/alloc/src/string.rs:2611:14:
a Display implementation returned an error unexpectedly: Error
stack backtrace:
   0: rust_begin_unwind
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/std/src/panicking.rs:645:5
   1: core::panicking::panic_fmt
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/panicking.rs:72:14
   2: core::result::unwrap_failed
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/result.rs:1649:5
   3: numbat::ffi::format_datetime
   4: numbat::vm::Vm::run_without_cleanup
   5: numbat::vm::Vm::run
   6: <numbat::bytecode_interpreter::BytecodeInterpreter as numbat::interpreter::Interpreter>::interpret_statements
   7: numbat::Context::interpret_with_settings
   8: numbat::Cli::parse_and_evaluate
   9: numbat::Cli::run
  10: numbat::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

This should probably not leak (and certainly shouldn't crash the program). I also saw some unrelated panics in WASM, which is more annoying to debug because you can't easily see the backtrace. 🙂 Unfortunately, I didn't note down what caused them.

This should probably not leak (and certainly shouldn't crash the program)

Definitely not, no. I'm trying hard to properly turn every possible internal error into a corresponding Numbat error, but for this method (DateTime::format), it's not even documented that it can panic(?).

This is now fixed: https://numbat.dev/?q=format_datetime("%25Y-%25m-%25dT%25H%25:M"%2C+now())%E2%8F%8E


tcp forwarder net.Dial failures should be trace

Changed net.Dial() failures for the tcp forwarder from log.Error to log.Trace.
See issue #74

Thanks!
/lgtm


feat: Rewrite the EJS language definition

This rewrites the EJS language definition to fix many issues.
Inspired largely by what I did with Nunjucks.

Fixes #3
Fixes #7
Fixes #20

This has been published to VSCode Marketplace


Easter egg in chapter 7 challenge

This is a good place to put an Easter egg for someone who submits a good answer.
Maybe it's a few sats but I'd prefer a call with me which gives them the opportunity to raise their hand as the first step in the signal machine.
Can anyone think of a better idea?

Is this something where you'd like it to be limited to the "best" answer that chaincode still has hidden?
I was thinking we could create and env variable such that even in the source code we don't give it away like so
if (answer === PROCESS_ENV.best_chaincode_secret_answer) {
  return [
    true,
    'Wow! That's amazing, you really know how to optimize a mempool! Reach out to `jonas @ chaincode` as I'm sure he might like to have a chat!
  ]
}

Best answer only.


2024-12-13_01-36-04_master-next_python3-botocore

Automatically created.

Successfully created backport PR for kirkstone-next:

#10775

Successfully created backport PR for scarthgap-next:

#10776

Successfully created backport PR for styhead-next:

#10777


Support collectionFormat documentation for parameters.

@dimitertodorov … thanks for the work,
but I'm missing specs for complex arrays, such as one could find here: https://github.com/ruby-grape/grape-swagger/blob/master/spec/swagger_v2/api_swagger_v2_param_type_body_nested_spec.rb,
how would you handle this?
and yeap, add entry to the changelog

Thanks @LeFnord I will take a look this week at those specs and resubmit.

@dimitertodorov @LeFnord why was this PR closed?

@jdurand … think that @dimitertodorov did some re-work on it

OK, but documentation: { collection_format: 'multi' } is not yet handled right?
@dimitertodorov this is definitely something I could use, are you still working on this?


Only some mp3 files can be played?

Rodio version: 0.17.0
Rust version: 1.69.0
Decoding function:
pub fn read_file_from_beginning(file: String) -> Result<Decoder<BufReader<File>>, rodio::decoder::DecoderError> {
    let reader = BufReader::new(File::open(file).unwrap());
    let decoder = Decoder::new(reader);
    decoder
}

Which will get added to a sink and played as normal, exactly as the docs have it.
The problem I have, it only about half of the songs I have will be played as expected. The other half will not play, it will throw a thread '<unnamed>' panicked at 'called Result::unwrap()on anErr value: IoError("end of stream") On this line:
self.sink.append(read_file_from_beginning(self.list[self.count as usize].savelocationmp3.clone()).unwrap());

(Which is just adding the song's path to the sink ( as a decoder ) ).

Moving and renaming the files seems to have no effect on whether or not they fail
Every song is able to be interpreted just fine by windows media player, other media programs etc...
I tried compiling with the mp3, minimp3 and symphonia_mp3 flags, which did not make a difference.

No clue exactly what causes this, but the video were download from pytube. If you turn those mp3s -> wav -> mp3s using ffmpeg, it works perfectly fine and can be read. I should also note that other crates such as mp3-duration are unable to read the mp3s either. So i dont think it is exactly rodio's fault

I think it is was a permission problem. Maybe there should be a test to see if the files can be opened


GUI elements don't take into account parent's scale

Bug repro: https://www.babylonjs-playground.com/#XCPP9Y#648
Expected result: The entire button can be clicked
Current result: The button as if it's not scaled can be clicked

Awesome, it's fixed :+1:


Add support for optional params to venues endpoint

Hello Mike,
I suggest to add support for optional params to venues endpoint. This tiny change would allow passing additional params to this endpoint just as to other endpoints, eg. locale parameter for i18n.  Thanks.
Best regards,
Dmitry

Looks great. Thanks @dmytro-lebedev

foursquare-1!2015.5.26 includes this fix.


Make it a bookmarklet for a colletion of endpoints

Should make bookmarklets for every interesting endpoint:

[ ] Sanity.io
[ ] JSONbin.io
[ ] Zapier.com

Each bookmarklet will need to be set up with your ID's, secret-keys etc.
The script extracting url, title and content (body) will stay the same. Easy to add new endpoints.

[ ] A vue component setting reading indexedDB for values set for different endpoints
[ ] A bookmarklet URL for each endpoint to be dragged to the bookmark toolbar. These needs to feed the pure JavaScript which endpoint to use.
[ ] The script reading the indexedDB for one spesific endpoint, extracting info from the page and sending to the endpoint

The last script needs to be a webpack'ed script with a dexie.js import.

Won't work. The embedded script will use indexedDB on the domain you embed it to.

Each bookmarklet will generate a script to embed in a URL that can be dragged to the bookmark toolbar. Then it is just a static link (script)

Creating separate issues for JSONbin.io and Sanity.io


Fix for 23w16a

Fixes the issues for 23w16a

while this does "fix" 1.20 support, it does this by essentially removing code. I'll pin this so i remember to work on it when i have the time.

i'll make a branch for this actually, and see if that will work better for actions (you made quite a few tasks that keep running)


store VCF file in db

Do not have separate vcf file but instead store the variant - indivdual relationship in db.

This is now partly done by @dvarrazzo import scripts.


Back/forward browser commands don't work with this plugin enabled (Brave/Chrome, macOS)

Steps:

Navigate to a Bandcamp album page from somewhere else (so there is browser history when clicking the browser back button).
Start playing a song
Use CMD+LEFT ARROW keyboard shortcut to navigate back to previous page

Result:
Nothing happens. Back/forward commands in Chrome/Brave on macOS no longer work (CMD+LEFT/RIGHT ARROW).
Expected:
Back/forward commands still work
Notes:
Maybe a different key combo could be used? Or perhaps there is a way to get the normal left/right arrows without CMD to work as is, and allow CMD+LEFT/RIGHT ARROW to also work as intended with the browser back/forward?

Duplicate of https://github.com/sabjorn/BandcampEnhancementSuite/issues/56


Configure reply urls using OAuth2 auth flows

Hi all,
is there a way to configure Redirect Url when using OAuth2? I'm getting this error while trying to set up implicit flow authentication with swagger:
The reply URL specified in the request does not match the reply URLs configured for the application
If I check the authorize endpoint I see that it's automatically assigning this as the reply url for the OAuth flow:
https://localhost:5001//oauth2-redirect.html
thanks in advance.

Sorry guys, now I understand that url is needed by swagger-ui to retrieve auth result from IDP... I'm closing this question....


Loss function for sparse goal scoring

Hi. Thanks for the great work!
It seems that the implementation scores the sparse goals and chooses the top k (say k = 150) sparse goals to generate dense goals around them. May I ask why there is no loss function for the sparse goal scoring part? I can see that the lane scoring loss and dense goal scoring loss are computed.

Top k goals and dense goals are both goals, and the loss of goals is calculated in goals_2D_per_example_calc_loss. So we don't need to calculate loss of top k goals seperately.

Top k goals and dense goals are both goals, and the loss of goals is calculated in goals_2D_per_example_calc_loss. So we don't need to calculate loss of top k goals seperately.

I see, thanks for the response!


Check Cargo.lock for Bin Crates

I've got a two year old repo that shows as up to date:

Crate
Required
Latest
Status

gtk
^0
0.3.0
up to date

hyper
^0
0.11.19
up to date

This is because the deps are specified like:
[dependencies]
gtk = "^0"
hyper = "^0"

But the lockfile shows the deps are actually locked to much older versions:
dependencies = [
 "gtk 0.0.7 (registry+https://github.com/rust-lang/crates.io-index)",
 "hyper 0.9.7 (registry+https://github.com/rust-lang/crates.io-index)",
]

Thanks for raising this!
Lockfile support is something I‘m working on. I‘ll make sure to cover this use-case as well!

This would indeed be nice to support both for binary program crates as well as for git repositories!

We had a specific problem in https://github.com/EmbarkStudios/rust-gpu/pull/132 where our tar dependency, and as such the entire repo in the badge, got flagged as "insecure", even though we through Cargo.lock was using a newer secure patch version of it.
That was easy to workaround to also specify that patch version in Cargo.toml, but if deps.rs would have supported Cargo.lock it wouldn't have been flagged incorrectly to start with.
No big deal, but thought I should mention that specific use case here also


Update app.py

edited loading json file

Thanks! That is good.
Can you please also fix the other pylint errors?
https://github.com/PedestrianDynamics/Motivation/actions/runs/5832192346/job/15817115165


Contribute Page

We need a page about how people can contribute.  Some topics for this page:

How to contribute to the website
How to sponsor meetups or contribute resources
How to reach out with data or other information

I think we can add a CONTRIBUTING.md file in the vein of https://github.com/jden/CONTRIBUTING.md/blob/master/CONTRIBUTING.md and make it a page on the site as well.

:+1:


Releasing code

Hello,
I'd like to thank you for your work.
I would like to know when you could make the code available?
Best regards,
Sylvain

Will try to do it by this weekend!
Sincerely,
Chufan Gao
On Wed, Jun 12, 2024 at 08:17 Sylvain Verdy @.***>
wrote:

Hello,
I'd like to thank you for your work.
I would like to know when you could make the code available?
Best regards,
Sylvain
—
Reply to this email directly, view it on GitHub
https://github.com/chufangao/TTM-RE/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ3C6K23F6P5OZPY5I2TI3ZHBC7JAVCNFSM6AAAAABJGLGUTKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2DQNZVGY3TOMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Released, please let me know if there are any questions! Note that the performance varies slightly from paper due to inherent randomness.


Obsidian 1.0 Changes & More

Since Obsidian v1.0 released I made changes here and there. Maybe you want to check them out before merging into main. An updated changelog is in the most recent commit.
Before:

After:

Also I'm wondering how we should handle future changes to this theme. Should I just push all the ideas and fixes I might come up with to main or should I await your feedback first? This is your theme after all and I don't want to stray too far from your original vision.

For changes going forward, feel free to add whatever you like straight to main, no need to go through me. I've been super busy with work recently, and haven't had the time to give this theme any love, and I'm super grateful for your help with it.

Alright. I'll try to keep it somewhat down to earth :)

Let me know if you are missing any permissions on the repository that you need

I think I have sufficient permissions.


🛢 Add consistent URL ("/latest/") for most recent product version.

Steps to test/reproduce
Please explain how to best reproduce the issue and/or test the changes locally (including the pages/URLs/views/states to review).

Navigate around site; see /latest/ in URL

 I want the URL for the most recent version of a plan to be a permalink.

I couldn't find a clear way of getting a link to the specific version in case I don't want to point to the latest one. Not sure if that's a valid use case for MetaDeploy 🤷♂

That's a good point. I generally think we're trying to avoid that -- @davisagli what do you think?

Yeah that's right, we want to discourage that because generally using an old installer at some point in the future is only a valid use case for internal folks trying to reproduce some issue -- we push upgrade customers to the latest version and want to make sure they are starting out there as well.

@davisagli I think this is ready to go. Would you prefer to review it here, or should I open up a PR upstream into https://github.com/SFDO-Tooling/MetaDeploy?


How to install brotli library on ubuntu xenial?

I integrated jbrotli within my apache httpclient but can't use it because    BrotliLibraryLoader.loadBrotli(); fails. I already installed brotli via apt-get install brotli and also compiled brotli by myself and created the symlink to the brotli.so file but it's also not working.
Couldn't load native library 'brotli'. [LoaderResult: os.name="Linux", os.arch="amd64", os.version="4.4.0-51-generic", java.vm.name="OpenJDK 64-Bit Server VM", java.vm.version="25.111-b14", java.vm.vendor="Oracle Corporation", alreadyLoaded="null", loadedFromSystemLibraryPath="false", nativeLibName="libbrotli.so", temporaryLibFile="/tmp/brotli5285008184952161942/libbrotli.so", libNameWithinClasspath="/lib/linux-x86-amd64/libbrotli.so", usedThisClassloader="false", usedSystemClassloader="false", java.library.path="/usr/java/packages/lib/amd64:/usr/lib/x86_64-linux-gnu/jni:/lib/x86_64-linux-gnu:/usr/lib/x86_64-linux-gnu:/usr/lib/jni:/lib:/usr/lib"];
What's the correct way to install brotli on ubuntu xenial that jbrotli can make use of it?

UPDATE: I think i got it working. I was missing a dependency:
   compile('org.meteogroup.jbrotli:jbrotli-native-linux-x86-amd64:0.5.0')
I didn't see this in the documentation, got it from the sourcecode and pom.xml out of the subprojects in this repository. This should be described within the documentation or README.

Hi Patrick,
thank you for your comprehensive feedback.
jbrotli brings it's own binaries via pom.xml dependencies.
If you add "org.meteogroup.jbrotli::jbrotli" to your maven or gradle build dependencies,
these build tools should automatically fetch the dependencies.
It seems to me, you're using gradle.
May I ask you to give some information about your build config?
I would like to make sure, my original intended automated dependency fetching works ;-)

What gradle version?
global 'gradle' or project-local 'gradlew'?
what dependencies, when it crashed?
what dependencies, when it worked?

Thanks in advance.


Feature/akolade compute salary test v1

Hello @emmakolade, closing this PR as part of cleanup work in preparation for our project. Just FYI.


discuss ways to deploy infra and common content changes across many spec repos

I did find this GitHub Action the other day to update the submodule: https://github.com/marketplace/actions/github-action-submodule-updates
Perhaps we can use this, or something like it to automatically pull submodule changes.


Adding Shapefile

This pull request includes:

Adding a Geofile folder under the current data folder, including the shapefiles of NUT1 boundaries of the UK.
Adding a paragraph in the readme.md from line 100-102 to introduce the geographical dataset

This pull request is linked to issue #86 for adding the shapefiles.

Looks good to me. Just made a minor suggestion for the README.md text, mainly a link to the Open Government Licence. Only other thought is whether the shape of London mentioned in #68 needs any further checks.

The London shapefile actually is a clip from the NUT1 data, so it is from exactly the same source.

@griff-rees  I revised the README.md, and also added a sentence to clarify the three.cities folder.


travis: add windows

Landing on red CI because this shows the problem, the windows build is broken


why the request is requestTimeout，but it didn't throw exception

i had set connectTimeout and requestTimeout in DriftNettyClientConfig.
driftNettyClientConfig.setConnectTimeout(new Duration(connectTimeout, TimeUnit.MILLISECONDS));
driftNettyClientConfig.setRequestTimeout(new Duration(requestTimeout, TimeUnit.MILLISECONDS));

but it  didn't work;
i use 1000 threads to pressure test for 1 minutes,it always like the below:

and the result is like below:

My guess is your application had a long GC which can not be interrupted.  So it is likely the timer fired but the successful result was available, so the system returned that result.  Also, make sure you are using connection pooling, and I would recommend async APIs for anything high volume.


Updating and uninstalling jenkins plugins with the jenkins_plugin resource does not work

I have this in my chef cookbook:
# Uninstall core plugins
jenkins_plugins = %w(
  cvs
  javadoc
  maven-plugin
  subversion
)
jenkins_plugins.each do |plugin|
  jenkins_plugin plugin do
    action :uninstall
    notifies :execute, 'jenkins_command[reload-configuration]', :delayed
    notifies :execute, 'jenkins_command[safe-restart]', :delayed
  end
end

# Update core plugins
jenkins_plugins = %w(
  credentials
  git-client
  mailer
  matrix-auth
  junit
  matrix-project
  antisamy-markup-formatter
  pam-auth
  script-security
  ssh-credentials
  translation
  windows-slaves
)
jenkins_plugins.each do |plugin|
  jenkins_plugin plugin do
    notifies :execute, 'jenkins_command[reload-configuration]', :delayed
    notifies :execute, 'jenkins_command[safe-restart]', :delayed
  end
end

The output is as expected, e.g.
Recipe: jenkins_digipolis::jenkins
  * jenkins_plugin[cvs] action uninstall
  Recipe: <Dynamically Defined Resource>
    * file[/var/lib/jenkins/plugins/cvs.jpi] action delete
      - delete file /var/lib/jenkins/plugins/cvs.jpi
    * directory[/var/lib/jenkins/plugins/cvs] action delete
      - delete existing directory /var/lib/jenkins/plugins/cvs
    - Uninstall jenkins_plugin[cvs]

However, the plugins aren't updated and uninstalled according to the UI.

The same is happening to me with Jenkins 1.609. While investigating it I noticed that it actually seems like an issue with jenkins-cli.jar. It works good when you install a plugin using just a name (getting the latest) but it fails when using an url or file path (that what the jenkins_plugin is doing under the hood).

Maybe related: https://issues.jenkins-ci.org/browse/JENKINS-29065

I just ran into something similar. Its very likely that this is because you are trying to update core plugins, ie plugins that are bundled with the core jenkins install. See https://wiki.jenkins-ci.org/display/JENKINS/Pinned+Plugins for details. To workaround;
  file "#{node['jenkins']['master'].home}/plugins/#{plugin}.jpi.pinned" do
    content ''
    mode '0644'
    owner node['jenkins']['master']['user']
    group node['jenkins']['master']['group']
  end

inside your jenkins_plugins.each block

That indeed works, thanks.
I won't do this anymore, because I want to review updates again, but thanks! :)

This is necessary even if you use jenkins_plugin to install a specific version, which is what I do, and how I review and select updates.


Install external repos

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
Add option to install external repos inside src ws
Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings
Please replace this line with instructions on how to test your changes.
[optional] Are there any post deployment tasks we need to perform?

@Cyano0 can you please merge this once finished the checks, this now will be able to install the external repos inside the src

@marc-hanheide is this the correct way to add external repos to the package?

@Cyano0 please merge if you are happy with the changes


Custom themes

Hello! What about to allow to users create custom themes? I think it can be very useful.

+1

@lamartire an excellent feature thank's

I found a related feature #142
we will solve this in the other issue
Thaks


End of life 3.6

This version of Python is now end of life.
We might remove support for it, if and when it starts to cause us problems.
So far we saw one problem - detection of the line numbers where functions are declared has changed.

yet again the wrong upstream, sorry!


readme doc add option description

i think readme doc can add more description of cli option. like this:
-r,--read [path]    read [path] file

@zhoushaotian Cool, I will add it. Thanks!
And we should add more Get Started tutorial for different platform. :)

All done, so close this issue


Specifying the interface to run the command Set-DataResidencyLocation -Europe:$true or $false

Regarding the KI "Geographical region not provided"
I really appreciate for writing down the known issues and the mitigation steps. This was quick to identify what the alert is for and what should be done further. Thanks for your efforts.
Just one feedback, I thought may be specifying where to run this command Set-DataResidencyLocation -Europe:$true or $false will really help customers to follow the procedure easily. Whether it should be run connecting to Admin ARM or by connecting to PEP and if it is PEP, it is required to unlock the session.
I tried running this command connecting to locked PEP session and it worked.

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

ID: 0094b74f-e85b-6983-5f58-ce24f825b63c
Version Independent ID: a90feafe-0401-3da0-bb95-4dbc8868ea0f
Content: Azure Stack Hub known issues - Azure Stack Hub
Content Source: azure-stack/operator/known-issues.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@NikhilBhoga Thank you for the feedback! We will update this information ASAP. #please-close

Hi @sethmanheim, thank you for acknowledging. May I know how many days you usually take to update a document.


adjusting block light color

I want to set block light in the dev version to be more white/closer to vanilla instead of yellow. Are there any variables I can adjust?

shaders/include/light.glsl
line 26: vec4(1.2, 0.80, 0.45, 1.0)
Change this to whatever color you want.

Is that rgba or hsva? The x is greater than 1 so how does that work? ^-^

rgba. As it is right now there is some clipping since MC only supports RGBA8 output. I will probably change this later when I introduce HDR using some hacks.

Thank you!


DragAdornerTemplate for Files

Hello,
is it possible to change the DragAdornerTemplate for dropping files into the view?
I have used the DataTemplate "DragAdorner" from the example. It works great for items in a Listbox but not for files. Am I doing something wrong, or is it not possible?
Thanks and best regards

-> todo https://github.com/punker76/gong-wpf-dragdrop/issues/229


Name error on https://github.com/bokkypoobah/TokenTrader/wiki/LOVE-‐-I-Love-You

https://github.com/bokkypoobah/TokenTrader/wiki/LOVE-‐-I-Love-You
Token Contract Information have an error, name say: Ethereum Movie Venture
Name is: I Love You
Regards

Updated. Please check.


Network tag

After importing the .ovpn profile to the VPN GUI, there was an error connecting to the server because the openvpn instance is not accessible to the public. Had to manually add a "public" network tag to the openvpn istance.
Kindly see screenshot attached.

@tomisinlalude-deimos  The problem is; Openvpn is configured to listening on port 1194 (as default on installation script)
After added UDP with port 1194 to network firewall(https://github.com/DeimosCloud/terraform-google-openvpn/blob/master/main.tf#L21-L28), I can connect to Openvpn.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter epson_stylus_px830 for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.epson_stylus_px830
   -
ERRORS:

[ ] :heavy_exclamation_mark: [E254] Versions "0.1.4, 0.1.0" listed at common.news at io-package.json do not exist at NPM. Please remove from news section.

WARNINGS:

[ ] :eyes: [W113] Adapter should support compact mode
[ ] :eyes: [W127] Missing suggested translation into pl,uk,zh-cn of "common.titleLang" in io-package.json.
[ ] :eyes: [W134] Missing suggested translation into uk,zh-cn of "common.desc" in io-package.json.
[ ] :eyes: [W154] Missing suggested translation into uk,zh-cn of some "common.news" in io-package.json.
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W953] .npmignore found - consider using package.json object "files" instead.

SUGGESTIONS:

[ ] :pushpin: [S522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Fri, 13 Sep 2024 10:35:38 GMT based on commit 4c3bfa5ea962d50e7e2f8ce02ab0d915b3b0fdcd
ioBroker.repochecker 3.0.5

This issue has been updated by ioBroker Check and Service Bot
The following issues have been fixed
[E955] iob_npm.done not found in .npmignore
:thumbsup:Thanks for fixing the issues.


Ingress node firewall configuration resource

configuration controller
bindata daemonset manifest
render and apply packages are based off sriov operator implementation
Unit-test

 k get ds -n ingress-node-firewall-system
NAME                           DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR                     AGE
ingress-node-firewall-daemon   0         0

@msherif1234 can you remove the BPF byte objects from this commit as well?

/lgtm
API renaming will be discussed later.

/retest


Implement LFC prewarm

Problem
We periodically need to restart computes.
As far as performance of Neon critically depends on local caches (shared buffers or LFC), it is important to provide fast rewarm of restarted node.
Standard Postgres pg_prewarm extension provides autoprewarm mode in which is saves state of shared buffers and loads it after restart. But current autoscaling policy assumes small shared_buffers and large LFC. So we need to prewire LFC rather than shared_buffers.
Summary of changes
This PR implements the following:

Add hook to checkpoint allowing to perform custom action during checkpoint.
Save state of LFC as shutdown checkoint (metadata of neon.file_cache_prewarm_limit most frequently accessed chunks) in AUX file. This AUX file is automatically included in basebackup and loaded on startup
Start background worker which loads pages of this chunks.

Possible concerns:

AUX file size - limited by neon.file_cache_prewarm_limit
Extra storage IO - LFC state is saved only once at shutdown
Race conditions with LFC accessed by other backends - addressed using conservative policy  using prewarm_requersted and prewarm_started flags to skip updates of LFC in case of possible risk of contention.

TODO:

Use vector protocol for retrieving pages from PS once it will be available.
Start multiple background workers to prewarm LFC in parallel

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Replaced by https://github.com/neondatabase/neon/pull/9587


Trying to include static files using pkger.Dir("../frontend/build") results in "exit status 1: no Go files in /Users/me/app/frontend/build"

Basic example:
package main

import (
	"github.com/gofiber/fiber/v2"
	"github.com/gofiber/fiber/v2/middleware/filesystem"
	"github.com/markbates/pkger"
)

func main() {
	app := fiber.New()

	//serve react app
	app.Use("/", filesystem.New(filesystem.Config{
		Root: pkger.Dir("../frontend/build"),
	}))

	app.Listen(":3000")
}

Is there some other way to specify that the static files are up a directory?
This appears to work fine if the build directory is in the same directory as the go code itself (i.e. Root: pkger.Dir("/build") ).

Digging into this a little more - looks like maybe I'm misunderstanding how this works and this is by design per "There are no relative paths. All paths are absolute to the modules root."


Fix 'global' and 'window' scope by using the 'global' library

Now 'global' and 'window' scope should work correctly.
Before it was removing XMLHttpRequest from the window scope when created with xhr-mock.
This uses the global library, which is also used by xhr-mock.
Fix #9.

@henri-hulski I hope you are better know what you're doing, because, to be honest, I haven't quite grock what's going on here :-). Thanks.

I have tested it on my code base and everything works fine now.
If there comes up any issues I can take a look.
Will you make a release?

@henri-hulski Sure, already did it: v1.0.5.  I only meant that I have to trust people, who want to contribute to this project, because I'm not keeping in pace with all things going on in JS dev. Your changes seems OK, though I don't completely understand what under the hood.

Thanks! Before global.windowwas set to empty object which removed XMLHttpRequest created by xhr-mock.
Now we use the global lib which is also used by xhr-mock to create the global variables.
This seems to work in a more conform way.
But I also don't know the implementation details of global.
So if there will be any issues please ping me and I will try to fix it.


Update description of webhook config

The old text seems very different from our current configuration.

CR :shipit:

CR :guitar:

QA 📚


SphericalUtil.interpolate bug

Summary:
SphericalUtil.interpolate(from, to, fraction) does not work when from and to are close together (about 2 meters)
Steps to reproduce:
select any two locations that are close together and perform interpolation
Expected behavior:
interpolate between two locations in an indoor map
Observed behavior:
convert to XY locations to do interpolation
Device and Android version:
nexus 6
Screenshots:
no screenshot

@hptran could you give us some actual numbers for the input you're using, the expected output, and the actual output? This will help us debug and write unit tests.

@barbeau I've encountered the same issue with the following figures and code.
Log.d(TAG, "ShortestLine: " + shortestLine.first + ", " + shortestLine.second);
Log.d(TAG, "SecondShortestLine: " + secondShortestLine.first + ", " + secondShortestLine.second);

LatLng pointOne = SphericalUtil.interpolate(shortestLine.first, shortestLine.second, 0.5);
LatLng pointTwo = SphericalUtil.interpolate(secondShortestLine.first, secondShortestLine.second, 0.5);

Log.d(TAG, "PointOne: " + pointOne);
Log.d(TAG, "PointTwo: " + pointTwo);

Logs:

ShortestLine: lat/lng: (-37.756891,175.325262), lat/lng: (-37.756853,175.325242)
SecondShortestLine: lat/lng: (-37.756831,175.326874), lat/lng: (-37.756764,175.326839)
PointOne: lat/lng: (-37.756891,175.325262)
PointTwo: lat/lng: (-37.7567975000013,175.32685649999212)

The interpolation on the second shortest line (length ~8.5m) is working correctly and outputting PointTwo with a believable value. This is not the case however with the shortest line (length ~3.9m) and the output of PointOne appears to be clamped to one of the two input points regardless of the 0.5 fraction passed in.
In this case I would expect a value closer to say (-37.756872, 175.325250) for PointOne.
Google Play services version: 42
Android device: Nexus 5X
Android version: 7.1.2

@jcfarsight Thanks for the numbers!  From a quick look, it appears that the clamping is intentional - if the sine of the angle between the two points is less than 1E-6 then it returns the end point:
From https://github.com/googlemaps/android-maps-utils/blob/master/library/src/com/google/maps/android/SphericalUtil.java#L136:
if (sinAngle < 1E-6) {
    return from;
}

From a quick scan I'm guessing this is a limitation of Slerp algorithm which is used to do the spherical interpolation.  I think in the case where Slerp can't be used, the library should fall back to plain linear interpolation.
@broady @stephenmcd Does this sound right to you?

https://github.com/googlemaps/android-maps-utils/commit/0824b9259b27628f9df52cc6eb5a8533469548f3


viewportPaintRect needs to be explained

Currently it just says "pre-paint window". I don't know what that means, what units it's in, what it's relative to, etc.

Sounds like there's relative consensus that this isn't appropriate for a V1, and perhaps not at all. Will remove.
Thanks for the feedback!


Can't find instructions on how to configure when the integration page says "proceed to the Configuration section below"

Documentation has been updated.


Add copies of base project licenses to patches directories

Each of our repositories should have an Apache license for the repo and if we have patches then the patches directory should have the license associated with the associated repo. So for example our gitport patches directory should have a license file that is the same as the associated git repo. If we have missed one or it’s the wrong license in a patches directory then that is a bug
Originally posted by @MikeFultonDev in https://github.com/ZOSOpenTools/meta/pull/102#discussion_r1058526025

Repos needing license files in their patches directories are:

coreutilsport
cmakeport
gitport
vimport
makeport
m4port
libtoolport
xxhashport
opensslport
gettextport
autoconfport
automakeport
zlibport
bzip2port
rsyncport
lz4port
lessport
gawkport
diffutilsport
perlport
zstdport
bisonport
jqport
tclport
expatport

@IgorTodorovskiIBM fyi...

Hey I can I help with this. I am working at IBM and I heard about this project, attended the guild meeting and would like to contribute. @IgorTodorovskiIBM @ijmitch

Hi @NFuture, I believe this is already implemented.
Have a look at any recent port :)
This issue can probably be closed.

Hi @NFuture, thank you! As a first step, we should update this list (https://github.com/ZOSOpenTools/meta/issues/103#issuecomment-1366960380) with the ports that remain without a license.

Repos needing license files in their patches directories are:
coreutilsport - added
cmakeport - added
gitport - added
vimport  - added
makeport  - added
m4port  - added
libtoolport - added
xxhashport - added
opensslport
gettextport - PR open
autoconfport
automakeport
zlibport - added
bzip2port - added
rsyncport
lz4port - PR open
lessport
gawkport - added
diffutilsport
perlport
zstdport
bisonport
jqport
tclport
expatport

What should I do as next step @IgorTodorovskiIBM . I went through the PRs that has been merged as part of this issue and the licenses added in different repos are different. How can I find which license should be added to a particular repo.

Thanks @NFuture ! You can typically find the license by browsing the upstream community's github page or by locating the source code and searching for a COPYING or LICENSE file. For example, for Bison, it's available under https://github.com/akimd/bison/blob/master/COPYING

@IgorTodorovskiIBM and @MikeFultonDev  I have opened a PR in opensslport. I would like to confirm this is what you are expecting so I can move forward with the same way in the other repos.


ConfigParser.NoSectionError: No section: 'general'

After removing python2 from my system, I migrated from the ubuntu (19.04) debian package to the offlineimap snap package.
After this, offlineimap fails with:
 ~ $ /snap/bin/offlineimap -c /home/varac/.offlineimap/.offlineimaprc
Traceback (most recent call last):
  File "/snap/offlineimap/188/bin/offlineimap", line 22, in <module>
    oi.run()
  File "/snap/offlineimap/188/lib/python2.7/site-packages/offlineimap/init.py", line 89, in run
    options, args = self.__parse_cmd_options()
  File "/snap/offlineimap/188/lib/python2.7/site-packages/offlineimap/init.py", line 265, in __parse_cmd_options
    config.set_if_not_exists('general', 'dry-run', 'False')
  File "/snap/offlineimap/188/lib/python2.7/site-packages/offlineimap/CustomConfig.py", line 135, in set_if_not_exists
    self.set(section, option, value)
  File "/snap/offlineimap/188/usr/lib/python2.7/ConfigParser.py", line 753, in set
    ConfigParser.set(self, section, option, value)
  File "/snap/offlineimap/188/usr/lib/python2.7/ConfigParser.py", line 396, in set
    raise NoSectionError(section)
ConfigParser.NoSectionError: No section: 'general'

 ~ $ head /home/varac/.offlineimap/.offlineimaprc
[general]
accounts   = varac
fsync      = false
…

 ~ $ snap list offlineimap
Name         Version  Rev  Aufzeichnung  Herausgeber   Hinweise
offlineimap  v7.2.3   188  stable        snapcrafters  -

The (reinstalled) ubuntu debian package (7.2.2+dfsg1-1) works fine with the exact same config file:
  ~ $ /usr/bin/offlineimap -c ~/.offlineimap/.offlineimaprc
OfflineIMAP 7.2.2
  Licensed under the GNU GPL v2 or any later version (with an OpenSSL exception)
imaplib2 v2.57 (system), Python v2.7.16, OpenSSL 1.1.1b  26 Feb 2019
Account sync varac.varac.net:
 *** Processing account varac.varac.net
 Establishing connection to imap.immerda.ch:993<EMAIL_ADDRESS>Folder Drafts [acc: varac.varac.net]:
 Syncing Drafts: IMAP -> Maildir

Same issue with OfflineIMAP 7.3.3 installed as snap on ubuntu 20.04

Issue solved by moving ~/.offlineimap/ to ~/snap/offlineimap/current/ and ~/.offlineimaprc to ~/snap/offlineimap/current/.

The resolution is a workaround, not a fix.  The user should not expect to have to change the behavior of their scripts depending on what installation method was used.  For example, I have a handful of scripts which use the -c option, and these are broken with the snap installation.


Register request make form fields display with the html style

the form fields should look exactly like the currently implemented html

suggestions: also include password/confirmpassword as form fields (accessed the same way in views for post/authentication), then delete html input fields (except submit button). this would make it easier i think to make the whole form display a certain way without worrying about other intercalating fields (and also inherit current html style). Daniel (since you're working authentication), would this be a good idea? it might make things easier!

..also, this way we would have automatic confirmation that user has inputed both. (and perhaps also that they match).

...ok, so, i am able to display all fields (the password as well as a form field - but this should not affect how you access it for authentication Daniel (no verification for password confirm though, i think now this is left for authentication)). need to inherit style though...anyway, we will see exactly at the meeting, and then i can push the update.

..no longer use forms


 Don't start with the first item selected.

Version: v0.5.9-nightly 16e8ccbb
OS: 11.2.3 (20D91)
Current behavior: GoTo symbol quick open starts without select the first item.
Suggestion: It would be nice to have it starting with the first options selected to keep the same behavior of the GoTo file.

Fixed with #3766


chore: update travis to support automatic release

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


pyDecoder doesn't free memory after the object is killed until the process is killed

Thanks for supporting vpf library.
i notice that the when we create py decoder object it allocates some amount of gpu space. this gpu space is not freed until the processed is killed. so when we want to read multiple file, after a point i get out of memory issue in gpu. i am using gradio to host my service, within each request i download video and read the video using pydecoder. After 10 request or so i keep getting oom error.

Hi @sumuk
Please share more details about your setup. What’s the OS, driver version and HW ? Does it happen with any given input or only with some specific videos ?
I’ll try to repro on my machine.

Hi @sumuk
I do acknowledge the repro, vRAM consumption does increase between multiple PyDecoder classes creation. Looks like there's a bug somewhere in TaskDecodeFrame.cpp.
I'll come back as I find something.

@sumuk
Please check v.3.2.4, it fixes the memory leak issue.
If the problem persists, please feel free to reopen.

thank you for fixing this quickly, will test it.


Tab 4 (Show taxes) does not always close when another tab is opened

When opening another tab in the accordion, tab 4 (taxes) stays open.

 tags were misaligned. Solved.


fix run_constraints

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

pin_subpackage for some reason doesn't work here and ignores the hash leading to a broken run_constraint.
For eg: the previous build had,
  "constrains": [
    "libgfortran 5.0.0.dev0 6"
  ],

which is wrong since it should be,
  "constrains": [
    "libgfortran 5.0.0.dev0 h19a3b67_6"
  ],

ouch - conda build needs a bug report

thanks @isuruf !


Display whitespace

Closes #877

@baktiaditya can you rebase it on master so that only your changes are present in this PR? Sorry about all the hassle! I don't have time to do it myself at the moment

@baktiaditya what if instead of using cm-show-invisibles, we just replaced the whitespace characters in this.props.children?
Something like:
this.props.children
  .replace(/(\r\n|\r|\n)/g, '¬\n')
  .replace(/ /g, '·')

Thoughts?

@all-contributors add @baktiaditya for code


Fix rendering of ui.date_range embedded example

📝 Summary

The embedded interactive example of marimo.ui.date_range is not rendering correctly as pointed out in #2621.
Currently, it looks like this:

It seems that the error is due to double lines of
[``](https://docs.marimo.io/api/inputs/dates.html#id1)[`](https://docs.marimo.io/api/inputs/dates.html#id3){eval-rst} .. marimo-embed:

🔍 Description of Changes

📋 Checklist

[x] I have read the contributor guidelines.
[ ] I have run the code and verified that it works as expected.

📜 Reviewers

@akshayka OR @mscolnick

I have read the CLA Document and I hereby sign the CLA

The documentation now renders correctly:

Looks weird when expanded:


Document tree-shaking, or provide precise imports

The docs say that this library weighs anywhere between 5 and 15kb, depending on if you use tree-shaking. It doesn't specify how to get tree-shaking to work, so I guess it assumes you have a tool that does this automatically.
Is there a way to import only the hooks that you need without automatic tree-shaking?
Like this, no surprises, configuration on guess-work:
import useTable from 'react-table/useTable';
import useSortBy from 'react-table/useSortBy';

Maybe there already are import paths like these, but it doesn't look like it.

I think that this could be very useful.
I'm using rollup as bundler and it seems that the whole library is imported even if only few hooks are used.

This is working well for me in a preact typescript project:
import { useTable } from "react-table/src/hooks/useTable";
import { useSortBy } from "react-table/src/plugin-hooks/useSortBy";

@dpacmittal Not sure that's what you want in production though, I don't think that code is minified (maybe I'm wrong, or maybe your setup minifies it)?

Separate imports will be the default and preferred way in v8. For now, it is not v7's responsibility to implement tree shaking in your applicaitons, nor would it be possible to cover all of the scenarios and edge case around. Stay tuned for v8!


data available for P1

Hi team,  Can this be added or not? please let me know

@rdiazel : Thanks for your contribution! The author, @MarkusVi, has been notified to review your proposed change.

The necessary changes have been applied to the article.
#please-close


Check if it is a temporary problem with your computer program.

Sorry for your data loss jordan2907. Could you give some more information about your project structure and operating system?


Correcting statement about Service ports

Correcting details about service ports and protocol. Removed statement saying Service supports single port definition only and added details about multiport support.
This also fixes #13034

@sftim i remember you done some work on https://github.com/kubernetes/website/pull/12669 , are you able to comment from tech side ?
I haven't dived into the code ...

We already document this: https://kubernetes.io/docs/concepts/services-networking/service/#multi-port-services

Thanks for this PR @rajeshdeshpande02

I'm happy to see the misleading documentation fixed. I think we could tweak the formatting though.

This looks OK. The rest of the document isn't 100% following the style guide; however, fixing that ought to come via a different PR.
/lgtm

/lgtm

@DanyC97 @sftim Thank you for your review. As it looks good to you, can you please approve this PR

/approve


add npm run build (noop with a helpful message)

The guide notes that npm run build is only necessary for TypeScript, but it's easier to skip over that when you're scanning. It's mentioned again under "Rename Your Component" without a reminder that this is only for TypeScript.
This changed adds a build script that does nothing but remind the user they can skip that step.

Tweaked comment. Docs are updated to show the correct warning in Rename your component as well: https://lit-element.polymer-project.org/guide/start#rename-your-component


RedundantStringChars auto-correct

str = ''

# Existing code:
str.chars.last(2)
  # => []

# Rubocop suggested and auto-corrected to:
str[-2..-1].chars
  # => NoMethodError (undefined method `chars' for nil:NilClass)
# Not equivalent to `str.chars.last(2)` because below returns `nil`
str[-2..-1]
  # => nil

Expected behavior
Do not set this cop as auto-correctable as it can cause breakage
Actual behavior
Auto-correct transforms into potentially broken code
Steps to reproduce the problem
NA
RuboCop version
$ [bundle exec] rubocop -V
1.14.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.5.0, running on ruby 2.5.7 x86_64-linux)
  - rubocop-performance 1.6.1
  - rubocop-rails 2.5.2

I experienced string.chars[0] → string[0].chars
rubocop-performance 1.11.5 (held back by standardrb)

@jjb Thank you for the feedback. I've opened #273.


Fix incremental compilation for classes with $ in name

Fixes #19257

@bot-gradle test this

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

@bot-gradle test this

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

@oehme  If you might find time to review this I would appreciate it

@bot-gradle test this

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

PullRequestFeedback build

@bot-gradle test and merge

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


changing camera

how can change camera?

Hi
I solved the problem with
adding videoConstrains prob to your code
But i suggest you to add videoConstrains to your code and also turn on
flashlight
On Thu, Dec 8, 2022, 6:43 PM ysuzuki19 @.***> wrote:

Thank you for asking question.
Now, its not supported.
By my search, we might to solve it with videoConstraints prop of .
Its documented here https://github.com/mozmorris/react-webcam#props.
Do you fix it yourself?
If not, I will try when I have time. :)
—
Reply to this email directly, view it on GitHub
https://github.com/ysuzuki19/react-qrcode-reader/issues/1#issuecomment-1342883662,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALB63GALRG7GQHWPVLSUD23WMH3KZANCNFSM6AAAAAASVMTJZA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@kazem3d
Thank you for the idea!
Result of my consideration, I adopt deviceId prop for the component as optional (since v1.1.0).
But I didn't implemented flashlight (or torch) because I cannot test it.

Thanks a lot
On Mon, Dec 12, 2022, 10:51 PM ysuzuki19 @.***> wrote:

@kazem3d https://github.com/kazem3d
Thank you for the idea!
Result of my consideration, I adopt deviceId prop for the component as
optional (since v1.1.0).
But I didn't implemented flashlight (or torch) because I cannot test it.
—
Reply to this email directly, view it on GitHub
https://github.com/ysuzuki19/react-qrcode-reader/issues/1#issuecomment-1347151225,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALB63GCXHXIA56KYQMRVWYTWM53KJANCNFSM6AAAAAASVMTJZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

The interface of this feature is changed by #6 .
For migration, please read the doc


Send a message back to the channel on modal submit

Description
Hello.
I would like just to ask an advice for bot development.
I assume having a pretty common usage flow:

call some /command;
inside command listener open some modal;
on modal submit validate inputs and if everything is fine do some calculation.
As a result return information back to the user who has initially called the command.

And the issue is on the final step. Inside the modal submission handler I couldn't find a proper way to respond back to the exact chat where the command has been triggered.
As for now I don't really like an idea to add channel_selector into modal (user waits a response in the same channel)
And I couldn't find the way to use incoming webhooks since it may be any channel where the user initially triggered the command
My initial idea was to pass response_url received in command to the modal through private fields, but I for now it didn't work for me. Maybe I did something wrong.
Would appreciate any advice.
Thanks in avance.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hi @cooltolia, thanks for asking the question!
You can use response_url in your app.view listeners! but how to do so is a bit different. Please refer to the following replies for more details:

https://github.com/slackapi/bolt-python/issues/560#issuecomment-1004543914
https://github.com/slackapi/node-slack-sdk/issues/970#issuecomment-604315752

If you don't like adding a channels_select block to your modal, as mentioned in my reply at the first link, you can use private_metadata to transfer the channel's ID.
I hope this helps.

Hey @seratch
Thanks a lot for a such quick response.
Great to hear that the approach with transfering channel's ID to the modal is a valid.
The problem is that I'm not sure what to do with it later in modal_submission
In the all provided links TS uses app.action. But I subscribe on app.view('modal_callback_id). And as I see It doesn't have say function.
I was trying to use client.chat.postMessage instead. But again without success.

if initial command has been called from the channel and I try to post message back to it, getting the error:
UnhandledPromiseRejectionWarning: Error: An API error occurred: not_in_channel
if command has been called in DM, getting another error:
UnhandledPromiseRejectionWarning: Error: An API error occurred: channel_not_found

The second one makes sense for me, but I can not guarantee, where exactly would be called slack-bot command.

@cooltolia The "not_in_channel" error indicates your app's bot user is not a member of the conversation. That's why your chat.postMessage API call does tno work. Alternatively, you can start a new channel/DM instead. Otherwise. you may want to use bolt's respond method (under the hood, it's a response_url call). When your end user does some interactions with your app in a private channel, response_url can be a magical way to send a reply without being a member of the convo. My explanation here may not be clear enough but I hope this helps. I wil be offline for a while. If you have further questions on this, my colleages may be able to help you out.

Unfortunately, I still didn't get. But I appreciate your help.
For now I will just try to describe what am I doing a bit more in details.

user post a command in test-bot channel
application handles this request: app.command('/command', handler(app));
and inside handler opens a modal passing test-bot channel id to private_metada
application listens to app.view('modal_id_callback'), get's the channelId and tried to send message back to test-bot channel (error happens)

say and respond methods are not part of the arguments for app.view handler. So I can't use them in a clean way.
test-bot channel was registered as a default channel when adding bot to the workspace. So it's not clear why on step 4

app's bot user is not a member of the conversation

@cooltolia You can use client method for performing a chat.postMessage API call with the transferred channel_id in the view submission listener code. The code can be something like const response = await client.chat.postMessage({ ... });. Refer to https://api.slack.com/methods/chat.postMessage for details of available arguments.

@cooltolia

test-bot channel was registered as a default channel when adding bot to the workspace. So it's not clear why on step 4

I cannot guess your situation with given information. Also, we are unable to support you with your app / workspace information in this public GitHub issues. So, if you need help on the situation with more investigation, please contact our customer support agents via either /feedback in your Slack workspace. They can help you as long as questions are not SDK specifc.

@seratch No problem.
Anyway thanks for the help.
Closing the question


Add Isolation x 2.2 Predictions by NightHawk22

Song Name
Isolation x 2.2 Predictions
Artist Name
NightHawk22
Source
Youtube
Youtube Link
KmCOLqXlQL0
Direct File Link
No response
Song ID
76743
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 86084399

accept


made the heading links only show when hovering over them

Resolves #299
Screenshots after the change:

Thanks!


Compiling On Windows X64 VS2015

Anyone got feather-master built on Windows X64 using VS2015's compiler? I copied the ./cpp/src  directory to ./python. :question:
I'm running into these errors:
python3 setup.py install > out.txt
warning: no files found matching '..\LICENSE.txt'
warning: no previously-included files matching '*.so' found anywhere in distribution
warning: no previously-included files matching '*.pyc' found anywhere in distribution
warning: no previously-included files matching '*~' found anywhere in distribution
warning: no previously-included files matching '#*' found anywhere in distribution
warning: no previously-included files matching '.git*' found anywhere in distribution
warning: no previously-included files matching '.DS_Store' found anywhere in distribution
cl : Command line warning D9002 : ignoring unknown option '-std=c++11'
cl : Command line warning D9002 : ignoring unknown option '-O3'
error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\BIN\\amd64\\cl.exe' failed with exit status 2

out.txt:
running install
running bdist_egg
running egg_info
writing feather_format.egg-info\PKG-INFO
writing dependency_links to feather_format.egg-info\dependency_links.txt
writing top-level names to feather_format.egg-info\top_level.txt
writing requirements to feather_format.egg-info\requires.txt
reading manifest file 'feather_format.egg-info\SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'feather_format.egg-info\SOURCES.txt'
installing library code to build\bdist.win-amd64\egg
running install_lib
running build_py
copying feather\version.py -> build\lib.win-amd64-3.5\feather
running build_ext
building 'feather.ext' extension
C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\amd64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -Ifeather -IC:\Users\ddrmanac\AppData\Local\Programs\Python\Python35\lib\site-packages\numpy\core\include -IC:\feather-master\python\src -IC:\Users\ddrmanac\AppData\Local\Programs\Python\Python35\include -IC:\Users\ddrmanac\AppData\Local\Programs\Python\Python35\include "-IC:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\INCLUDE" "-IC:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\ATLMFC\INCLUDE" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.10240.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um" "-IC:\Program Files (x86)\Windows Kits\8.1\include\\shared" "-IC:\Program Files (x86)\Windows Kits\8.1\include\\um" "-IC:\Program Files (x86)\Windows Kits\8.1\include\\winrt" /EHsc /Tpfeather/ext.cpp /Fobuild\temp.win-amd64-3.5\Release\feather/ext.obj -std=c++11 -O3
ext.cpp
C:\feather-master\python\src\feather/common.h(35): warning C4800: 'int': forcing value to bool 'true' or 'false' (performance warning)
C:\feather-master\python\src\feather/metadata.h(39): error C2528: '__timezone': pointer to reference is illegal
C:\feather-master\python\src\feather/metadata.h(126): error C2269: cannot create a pointer or reference to a qualified function type (requires pointer-to-member)
C:\feather-master\python\src\feather/reader.h(101): error C2269: cannot create a pointer or reference to a qualified function type (requires pointer-to-member)
C:\feather-master\python\src\feather/reader.h(94): error C2059: syntax error: '('
C:\feather-master\python\src\feather/reader.h(98): error C2039: 'timestamp_meta_': is not a member of 'feather::metadata::TimestampColumn'
C:\feather-master\python\src\feather/metadata.h(121): note: see declaration of 'feather::metadata::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(102): error C2440: 'return': cannot convert from 'std::string' to 'const std::string *(__cdecl *)(void)'
C:\feather-master\python\src\feather/reader.h(102): note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
c:\users\ddrmanac\appdata\local\programs\python\python35\lib\site-packages\numpy\core\include\numpy\npy_1_7_deprecated_api.h(12) : Warning Msg: Using deprecated NumPy API, disable it by #defining NPY_NO_DEPRECATED_API NPY_1_7_API_VERSION
c:\feather-master\python\feather\interop.h(65): error C2766: explicit specialization; 'feather::py::npy_traits<9>' has already been defined
c:\feather-master\python\feather\interop.h(64): note: see previous definition of 'npy_traits<9>'
c:\feather-master\python\feather\interop.h(71): error C2766: explicit specialization; 'feather::py::npy_traits<10>' has already been defined
c:\feather-master\python\feather\interop.h(70): note: see previous definition of 'npy_traits<10>'
c:\feather-master\python\feather\interop.h(367): warning C4244: 'initializing': conversion from 'int64_t' to 'int', possible loss of data
c:\feather-master\python\feather\interop.h(420): error C2196: case value '9' already used
c:\feather-master\python\feather\interop.h(425): error C2196: case value '10' already used
feather/ext.cpp(2310): warning C4800: 'int': forcing value to bool 'true' or 'false' (performance warning)
feather/ext.cpp(2608): error C2059: syntax error: '('
feather/ext.cpp(2628): error C2039: '__pyx_t_2': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2628): error C2039: '__Pyx_GetModuleGlobalName': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2628): error C2039: '__pyx_n_s_tobytes': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2628): error C2039: '__pyx_filename': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2628): error C2039: '__pyx_f': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2628): error C2039: '__pyx_lineno': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2628): error C2039: '__pyx_clineno': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__pyx_t_4': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__Pyx_PyObject_GetAttrStr': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__pyx_v_tz': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__pyx_n_s_zone': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__pyx_filename': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__pyx_f': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__pyx_lineno': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2630): error C2039: '__pyx_clineno': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2632): error C2039: '__pyx_t_3': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2633): error C2039: '__pyx_t_2': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2633): error C2039: 'ob_type': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2660): error C2059: syntax error: '('
feather/ext.cpp(2671): error C2039: '__pyx_t_1': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2671): error C2039: '__pyx_f_7feather_3ext_check_status': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2671): error C2039: '__pyx_v_self': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2671): error C2039: 'writer': is not a member of 'feather::TimestampMetadata'
C:\feather-master\python\src\feather/types.h(157): note: see declaration of 'feather::TimestampMetadata'
feather/ext.cpp(2671): error C2228: left of '.get' must have class/struct/union
feather/ext.cpp(2671): error C2227: left of '->AppendTimestamp' must point to class/struct/union/generic type
feather/ext.cpp(3679): error C2059: syntax error: '('
feather/ext.cpp(3679): error C2039: '__pyx_t_3': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'
feather/ext.cpp(3679): error C2039: '__pyx_filename': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'
feather/ext.cpp(3679): error C2039: '__pyx_f': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'
feather/ext.cpp(3679): error C2039: '__pyx_lineno': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'
feather/ext.cpp(3679): error C2039: '__pyx_clineno': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'
feather/ext.cpp(3681): error C2039: '__pyx_t_4': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'
feather/ext.cpp(3682): error C2039: '__pyx_t_2': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'
feather/ext.cpp(3682): error C2039: 'ob_type': is not a member of 'feather::TimestampColumn'
C:\feather-master\python\src\feather/reader.h(88): note: see declaration of 'feather::TimestampColumn'

Looks like a patch will be required for Windows -- it would be great if someone could set up Appveyor CI for this project, too.

I installed VS 2015 on a windows box and spent about a half hour tinkering with this and I have absolutely no idea. I hope that someone with Windows knowledge can help.

Any chance someone at Continuum Analytics might have builds going? 😃

Thanks for putting in the fix!
I got it to compile but am now running into this on master: ImportError: No module named 'feather.ext'
ImportError                               Traceback (most recent call last)
<ipython-input-1-588f42580122> in <module>()
----> 1 import feather

C:\Users\ddrmanac\Desktop\feather-master\python\feather\__init__.py in <module>()
     15 # flake8: noqa
     16
---> 17 from feather.api import read_dataframe, write_dataframe
     18 from feather.ext import FeatherError, FeatherReader, FeatherWriter

C:\Users\ddrmanac\Desktop\feather-master\python\feather\api.py in <module>()
     19     raise ImportError("feather requires pandas >= 0.17.0")
     20
---> 21 import feather.ext as ext
     22
     23

ImportError: No module named 'feather.ext'

The module folder looks ok:

Are you trying to import from the source directory?

@wesm Yes, how silly of me! :stuck_out_tongue_winking_eye: The result of quickly launching IPython to test!
All is workign well now. Thanks again for getting the build working on windows! :+1:

@dragoljub I'm encountering the same error. I have an anaconda installation on win64. Trying to import feather but receiving the same error message.
How did you manage to fix it?

@engin-g  I copied the ./cpp/src directory to ./python then ran python3 setup.py install from the ./python directory within the feather master folder.


Provide config to rename index.html in dist folder

This is a feature request.

TL;DR; As a full-stack developer, I want to be able to rename the file extension of the generated HTML file in dist folder, so I can easily integrate it into my server-side project.

Right now (beta-15), I can rename the source HTML file in angular-cli.json using the index property, but the output file in dist is still called index.html.
I don't care whether the implementation chooses to carry the name forward from the index property, or introduce another property for it.
Use Case:
I need to use this inside an ASP .NET MVC application (.NET 4.5 not .NET Core).
My plan is to:

Include the folder as sub-folder in the project, called say client.
Set redirects in the root server-side project to go to client/dist when it's not matching files / folders or APIs
Run ng build --watch for dev, while IIS is looking at my parent server-side project folder

But I may well end up needing a legacy layout file, and some other server-side code to run before returning the HTML file, so my plan is to have a root server-side rout + MVC controller that returns the HTML file, using the Razor template engine (a .cshtml file, like Views/Home/Index.cshtml), which will still have the layout.
But I still want Webpack to generate the script tag etc. So, I want to:

strip everything from the CLI project index.html file
Include the generated file, which only has the generated script tag etc, in the server-side Index.cshtml file.

But to include it in my server template, it's way easier for me if its extension was cshtml not html. It's doable without that, but it's too much config, and part of this is showing the guys here how super easy it is to include Angular 2 seamlessly.
P.S.
Sorry if this sounds like a very complex scenario, I believe it really isn't, and it's awesome to have something like this to integrate Angular 2 with server-side frameworks; and the change -I hope- isn't a big one really to support this.
Thanks a lot for creating such an awesome starter that makes developing Angular 2 much easier, and making it available for everyone for free, while you definitely do not have to, and owe nothing to any of us.

I think the existing config can be used for the output as well. This isn't a big priority, I want to be honest with you. There's just a lot of bigger fish to fry right now.
But I can point you in the right direction in case you wanna do a PR.

We use this plugin, it seem to allow you to chose the destination filename: https://github.com/ampedandwired/html-webpack-plugin#configuration
This is where you should change it https://github.com/angular/angular-cli/blob/master/packages/angular-cli/models/webpack-build-common.ts#L128
appConfig.index is what's in angular-cli.json, specifically apps[0].index
you'll need tests, see https://github.com/angular/angular-cli/blob/master/tests/e2e/tests/build/output-dir.ts for a build test and https://github.com/angular/angular-cli/blob/master/tests/e2e/tests/third-party/bootstrap.ts on how to update the angular-cli.json file
do node tests/e2e_runner.js to run the e2e test suite

Definitly need it too, and already have done something similar with my wepback config in an other project.
I will push a PR this week-end if I can

PR created for publicPath support
#3285


Have a read-only mode for workflow editor

The following PR addresses this issue: https://github.com/Kitware/tangelohub/pull/87