Duplicate

Hi, I am trying to tweak this bot to work for a different wordle like game, it uses different colo emojis and has a different name
Is there a way to make that happen somehow? I'm a huge newbie when it comes to coding but I use python
Thank you!

To help you get started with your discovery on how you would accomplish this, I'd like to point out a few interesting parts of the code.
First, we start in themain.go file. The bot decides to respond to a message if the message contains a mention of the bot or it detects some Wordle content:
botMentionToken := fmt.Sprintf("@%s", botName)
wordleScoreDetected, err := mentionlessWordleScoreDetection(tokenizedContent)

if strings.HasPrefix(tokenizedContent, botMentionToken) || wordleScoreDetected {

The "wordle content detection" where I look for the format of Wordle 292 3/6<new line> is done in the mentionlessWordleScoreDetection() function
func mentionlessWordleScoreDetection(input string) (bool, error) {
	var dataExp = regexp.MustCompile(fmt.Sprintf(`Wordle (?P<game_id>\d+)\s(?P<guesses>\d+|%s)/6[\%s]?\n`, noSolutionResult, hardModeIndicator))
	result, err := matchGroupsToStringMap(input, dataExp)
	if err != nil {
		return false, err
	}

	return len(result) > 0, nil
}

Here I use a regular expression to attempt to search for a message in a specific format:
Wordle (?P<game_id>\d+)\s(?P<guesses>\d+|%s)/6[\%s]?\n

I inject a few special characters like X for no solution or * to indicate hard-mode into the parts that say %s. Afer doing this I end up with a regex that matches the first line of a Wordle score block followed by a new line, and ignore the rest of the emojis. You can see this RegEx in action here: https://regex101.com/r/eCVzSa/1

This would be the first part you would need to swap out to detect scores from other games like Quardle, Worldle, and so on to get the bot to start doing other stuff like saving scores to the database.
All of that fun stuff happens in a big long chain of if message contains ______ do ______ statements in routeMessageToAction() function in the main.go file: if message contains "wordle stuff" do "persist score to database"
func routeMessageToAction(ctx context.Context, s *discordgo.Session, m *discordgo.MessageCreate, input string, account wordle.Account, q *wordle.Queries, botMentionToken string) {
	var r response

	if strings.Contains(input, cmdWordle) {
		gameId, guesses, err := extractGameGuesses(input)
		if err != nil {
			log.Error().Str("server_id", m.GuildID).Str("input", input).Str("author", m.Author.ID).Str("command", cmdWordle).Err(err).Msg("Error parsing guess count")
		}
		log.Info().Str("server_id", m.GuildID).Str("input", input).Str("author", m.Author.ID).Str("command", cmdWordle).Int("guesses", guesses).Int("game_id", gameId).Msg("Found a Wordle")
		persistScore(ctx, m, s, account, gameId, guesses)

If you want to tell me what game your trying to reuse this bot for, and show me a RegEx you would use to detect the content then I can start to help you bit by bit.

Wow thank you for taking the time to type out all of that!
I'm actually trying to use that bot for two games one named SUTOM and the other one TUSMO (I guess I'd need to use something like
 if any(x in message.content for x in ["SUTOM", "TUSMO"]):
the RegEx would look like:
SUTOM (?P<game_id>\d+)\s(?P<guesses>\d+|%s)/6[\%s]?\n

Unfortunately for TUSMO it contains a french flag emoji at the beginning, idk if that changes anything


Add capability for custom financial models

Adds capability for custom financial models, along with a simple example model for NPV and associated tests demonstrating its usage with PV + wind hybrid models.

@Matthew-Boyd This looks good! One reservation I have is that there are the DataClasses for Depreciation and Tax Incentives that aren't used at all in the custom financial model. I don't think the financial model should process those parts as simple is better. But I think it would be better to remove those unused data from the financial model, and add checks within the HybridSimulation code to enable financial models that don't have those data. So there are parts of financial model code that expects Singleowner, and that code should be modified to be more flexible.


Package causes SyntaxError: for including a "-" in the module's name

The README states the package is "warts" however, when I run pip list, here is the package:
$ pip list
scamper-pywarts (0.2.1)

So to import this package, the README should state:
import scamper-pywarts
from scamper-pywarts.traceroute import Traceroute

But when running this in my script I receive this SyntaxError,
File "view-compressed-file.py", line 9
  import scamper-pywarts
                ^
SyntaxError: invalid syntax

Currently on Python3 incase that info is needed.

Hi,
The name in pypi is not necessarily the same as the Python module name. Lots of pypi packages have - in their names.
In this case, the README correctly states that you should use import warts.
Feel free to open a new ticket if something doesn't work as expected after that.

When I run the code with"
import warts
from warts.traceroute import Traceroute

I get this output:
Traceback (most recent call last):
  File "view-compressed-file.py", line 9, in <module>
    import warts
ModuleNotFoundError: No module named 'warts'

How did you install the package?
Make sure you installed the package for the version of python you are using (try pip / pip2 / pip3)

You can try the test.py script, it expects a scamper file on stdin.

Thanks! I was using pip instead of pip3, sorry for the hastle!

No worries, happy to help :)


Better handling of unknown release keys in registry

This commit has the .Net Application checker return the releaseKey from
the registry if it's not a known key tied to a .Net version. The website
then uses this information to tell the user that it can't find the exact
version and suggests they log an issue with the given release key.
This is in response to issue #41: Handle unknown releases better.

Based on your description here, does the website ever try to parse the key value directly itself? I can imagine the scenario where a user runs an outdated version of the client, but the website has been updated with new code that has the ability to interpret the client-provided key and try looking up the name itself right then. Does this happen this way or could this be added too?

The website doesn't try to parse the key. I thought the likelihood of someone using an outdated client wasn't worth the extra effort...

@binki I have moved the translation logic to the website (and hopefully improved it a little). @shanselman is this a move you're happy with?

@marcosnz Nice. One small thing I noticed with this change is that, since the new client does not specify realversion, the JavaScript does not flip the text to show “We found .NET!”. See https://github.com/marcosnz/SmallestDotNet/blob/ce846c63bb7841d922be7c5265e8d20b995ec446/SmallestDotNet/Default.aspx#L152 , probably can just swap in 'releaseKey' there.

@binki Nice catch. Fixed.

@shanselman thanks for merging the pull request. Looks like the exe at https://github.com/downloads/shanselman/SmallestDotNet/CheckForDotNet45.exe  hasn't been updated though? Still returning a version rather than a release key...


Update composer.json

Updated composer package name to match repository.

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


Fontconfig error: Cannot load default config file when resizing SVGs that include text elements

Using docker image v3 (v3.5.0) on amd64:
$ docker images --digests
REPOSITORY          TAG       DIGEST                                                                    IMAGE ID       CREATED      SIZE
darthsim/imgproxy   v3        sha256:d447291d7a679e0fbf2d503ba3a19921445f1b7fbe42b4ce8cc85f625c62ae77   808ea1ce7858   3 days ago   145MB

When resizing an SVG file that includes some text elements, the process fails with the following error:
Fontconfig error: Cannot load default config file: No such file: (null)

(process:1): Pango-CRITICAL **: 21:43:01.939: pango_font_describe: assertion 'font != NULL' failed

(process:1): Pango-CRITICAL **: 21:43:01.939: pango_font_description_get_variant: assertion 'desc != NULL' failed

(process:1): Pango-CRITICAL **: 21:43:01.939: pango_font_get_hb_font: assertion 'PANGO_IS_FONT (font)' failed

It seems that fonts are not configured correctly in the docker image to allow rendering SVG text.

Sorry, wrong repo...


Video aspect ratio cannot change when rotation changes.

override func viewDidLayoutSubviews() {
        super.viewDidLayoutSubviews()

        let ori = UIDevice.currentDevice().orientation
        if ori == UIDeviceOrientation.Portrait {
            self.rtStream?.videoSettings = [
                "width": 272,
                "height": 480
            ]
        } else {
            self.rtStream?.videoSettings = [
                "width": 480,
                "height": 272
            ]
        }
}

I'm not sure if this is the right way to do.

@mrbemani "out of main topic", I saw you use this project, are you make it with wowza?
I'm using wownza, works well, but the audio is not perfect, make a bit of noise, I'm using iphone 6s

audio is fine. maybe I didn't notice it. the noise may be the feedback from your mic, it happened to me when I tried it the first time. try separate your sending and receiving devices, see if the problem goes away.

@mrbemani I recorded it, take a look
http://monosnapmauro.s3.amazonaws.com/monosnapmauro/streaming.m4a
I left the computer recording the streaming and I was in another room, the noise still the same, for you the audio is clear and perfect?

No I don't get this notice. Ha....strange...

So if we have the same code, could be or the iphone model or the wowza server.
I will try reinstall the wowza server

currently... no way X)
I will try fix

@mrbemani currently master branch works ok!!

released https://github.com/shogo4405/lf.swift/releases/tag/0.1.8

That was fast! Nice work!


Initial commit

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Proposed Changes

...
...
...

Relevant issues
Closes #...
Closes #...
Closes #...
Related #...

I think this was an oopsie? If not let me know.


opensuse > 12: it's called docker-engine

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:MichaelSp/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@flavio -- does it look right to you?

No, this isn't the case for the official packages built by openSUSE and by SUSE.

Thanks @flavio @MichaelSp -- we'd like to stick with the official packages


dns.get_qname not found

Hi,
I have installed nginx 1.25.1 + njs 0.8.0 and used "nginx-doh-and-dot-to-dns.conf" to setup DoH service.
After installation it doesn't work properly and I get the following errors in /var/log/nginx/error.log .
js function "dns.get_qname" not found while prereading client data, client: xxx.xxx.xxx.xxx, server: <IP_ADDRESS>:53
js exception: TypeError: (intermediae value)["bytesFrom"] is not a function at module (/etc/nginx/njs.d/dns/dns.js:28)

look into #19. This helps me to get is working with nginx 1.25.2 + njs 0.8.0.
Unfortunately there are still a few log messages but DoT and DoH works.


InheritedNotifier skipping consecutive changes made in a Listenable

Steps to Reproduce

Execute flutter run on the code sample
Tap on the Increment Button
See the output in the Console

Expected results:
All the changes that were notified by the Listenable.
Actual results:
Only the last value was returned.

Code sample
Here's a DartPad Link.
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class CounterModel extends InheritedNotifier<ValueNotifier<int>> {
  const CounterModel(
      {Key? key, ValueNotifier<int>? notifier, required Widget child})
      : super(key: key, notifier: notifier, child: child);

  static int of(BuildContext context) {
    return context
        .dependOnInheritedWidgetOfExactType<CounterModel>()!
        .notifier!
        .value;
  }
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  final String title;

  const MyHomePage({
    Key? key,
    required this.title,
  }) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  late ValueNotifier<int> _counter;

  _incrementCounter() {
    _counter.value = 1;
    _counter.value = 2;
    _counter.value = 3;
  }

  @override
  void initState() {
    super.initState();
    _counter = ValueNotifier(0);
  }

  @override
  void dispose() {
    super.dispose();
    _counter.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return CounterModel(
      notifier: _counter,
      child: Scaffold(
        appBar: AppBar(
          title: Text(widget.title),
        ),
        body: Builder(
          builder: (context) {
            final value = CounterModel.of(context).toString();
            print(value); // values are skipped
            return Center(child: Text(value, style: Theme.of(context).textTheme.headline3));
          },
        ),
        floatingActionButton: FloatingActionButton(
          onPressed: _incrementCounter,
          tooltip: 'Increment',
          child: const Icon(Icons.add),
        ),
      ),
    );
  }
}

Hi @icnahom, Thanks for filing the issue. Running your code sample I see that only 3 is printed and intermediate values (1,2) are not being printed. I think this is working as expected because by the time the dependents are built with new value the
_counter.value = 3 is executed.

InheritedNotifier skipping consecutive changes made in a Listenable

those values are not being skipped you may verify this by adding a slight delay.
  _incrementCounter() {
    _counter.value = 1;
    Future.delayed(const Duration(milliseconds: 10), () {
      _counter.value = 2;
    });
  }

and you will see 1,2 printed.


Mark visualizations as vizjson2

Please CR @gfiorav
cc @luisbosque

Made some changes and added some tests. Can you take a second look?


Add sector descends filters to search

Issue number: DH-1585
Description of change
This makes a few changes to search:

Removes sector.name_trigram from the global search fields (as per DH-1585)
Standardises how sectors are stored in Elasticsearch and includes the sector's ancestors in the search document
Adds a new filter to company, contact and investment project search that matches on a sector or any of its descendants

Will amend the base branch once the other PR has been merged.
Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[x] Has the admin site been updated (for new models, fields etc.)?
[x] Has the README been updated (if needed)?

Codecov Report

Merging #865 into feature/DH-1585-sector-hierarchy will decrease coverage by 0.03%.
The diff coverage is 89.47%.

@@                         Coverage Diff                          @@
##           feature/DH-1585-sector-hierarchy     #865      +/-   ##
====================================================================
- Coverage                             96.66%   96.63%   -0.04%
====================================================================
  Files                                   160      160
  Lines                                  4737     4754      +17
  Branches                                457      459       +2
====================================================================
+ Hits                                   4579     4594      +15
- Misses                                   99      100       +1
- Partials                                 59       60       +1

Impacted Files
Coverage Δ

datahub/search/contact/views.py
100% <ø> (ø)
:arrow_up:

datahub/search/investment/views.py
100% <ø> (ø)
:arrow_up:

datahub/search/company/views.py
100% <ø> (ø)
:arrow_up:

datahub/search/company/serializers.py
100% <100%> (ø)
:arrow_up:

datahub/search/dsl_utils.py
100% <100%> (ø)
:arrow_up:

datahub/search/contact/serializers.py
100% <100%> (ø)
:arrow_up:

datahub/search/investment/serializers.py
100% <100%> (ø)
:arrow_up:

datahub/search/dict_utils.py
75.92% <83.33%> (+0.92%)
:arrow_up:

Continue to review full report at Codecov.

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


ProtocolException("Error in HTTP connection: Http2ZombieException('Connection already dead')")

Problem Description
The thumbnails on YouTube don't load after navigating around the page. I'm not sure what exactly is happening, it seems to be related to HTTP 2? See the attached video.  During recording it didn't happen after the first navigation (of course, why would it) and it took a moment to be reproducible. When I opened the image in a new tab it still didn't load. The second time I [alt+enter]ed it into a new tab.
In dev tools the requests appear as (pending). So mitmproxy never tells Chrome that the request failed?

Steps to reproduce the behavior:

Start mitmdump
Open Chromium
Browse to www.youtube.com
Click on a video
Go back to the front page by clicking on the logo
Most thumbnails will not load (you might need to repeat 4+5). I assume it is related to the error in the title. No amount of waiting makes them load.

Now the question is: is YouTube or Chromium doing something funny or is it mitmproxy that has the issue?
System Information
Mitmproxy: 5.0.1
Python:    3.7.5
OpenSSL:   OpenSSL 1.1.0j  20 Nov 2018
Platform:  Linux-5.3.0-26-generic-x86_64-with-Ubuntu-19.10-eoan

Attachments
video: http://www.prinzhorn.me/mitmproxy/screencast.mp4
video of me hovering the thumbnails: http://www.prinzhorn.me/mitmproxy/screencast-hover.mp4
dump: http://www.prinzhorn.me/mitmproxy/youtube

Thanks for the extensive write-up! The obvious workaround is to disable HTTP2 in mitmproxy, but we want to get that fixed anyways of course.
/cc @Kriechi
(somewhat refs #1775)

Nice, thank you @Kriechi


Simplify the schemas used

Rework the schemas to make the logic clearer, remove a lot of cruft and document things a bit more
Depends on #100

It's just my two cents, but I would stick to Nickel naming convention

Ooh, yes, thanks for reminding me of that :) That should be fixed (at least for the stuff changed in that PR, there might be some older inconsistencies here and there).

I would avoid dash such as in nickel-nix (dashes are allowed more like an escape hatch because external tools might enforce them within names, but internally I personally think they should be avoided, as we already have snake_case's _)

Fair enough. I'll wait for us to have a definite name before changing things


Execution error - Dai From Ethereum to Gnosis

Describe the bug
93k bridge got execution error in 1st instance.

Then it changed to "Boost Liquidity not available"

Finally, after the reconcile got completed execution error came back.
Ive checked there is a lot of liquidity and has more xdai than nextdai.

To Reproduce
Specific case.
Additional context
https://connextscan.io/tx/0x1a7471b5668e6e4dfca75bb819d5a8ca1419fd4d744ad51c3de9feb2fa18a9ba
Ticket: https://discord.com/channels/454734546869551114/1106733283737337946

fixed after clearing cache. will be finally fixed here https://github.com/connext/monorepo/issues/4174


Indexes for lists

Hi, I'm quite new to bbolt, bolthold and using key-value stores in general. I'd like to know whether it is possible to create an index for every value in a list. Let me give an example.
type Item struct {
    Id int
    Name string
    Categories []string
}

Given this struct, I'd like to be able to efficiently get all Items for a given category, i.e. I want an index like { "category 1": [15, 27], "category 2": [3, 9, 25] }, where the numbers correspond to Item.Ids.
How could I achieve this?

Efficiently would be make all operations in single traverse with ForEach:
package main

import (
	"fmt"
	"log"

	"github.com/timshannon/bolthold"
)

type Item struct {
	Id         int
	Categories []string
}

func main() {
	store, err := bolthold.Open("test.db", 0666, nil)
	if err != nil {
		log.Fatal(err)
	}
	store.Insert(15, &Item{
		Id:         15,
		Categories: []string{"cat 1", "cat 2"},
	})
	store.Insert(20, &Item{
		Id:         20,
		Categories: []string{"cat 3", "cat 1"},
	})
	store.Insert(25, &Item{
		Id:         25,
		Categories: []string{"cat 2", "cat 4"},
	})
	cat := map[string][]int{}
	store.ForEach(&bolthold.Query{}, func(record *Item) error {
		for _, k := range record.Categories {
			_, ok := cat[k]
			if !ok {
				cat[k] = []int{record.Id}
			} else {
				cat[k] = append(cat[k], record.Id)
			}
		}
		return nil
	})
	fmt.Println(cat)
}

I didn't write handling possible errors in example above for every operations because too lazy :)

Thanks, but that's not exactly what I was looking for. I want the index to be persistent in bbolt and managed by boltholt. What you suggested is more like a temporary, custom-build, in-memory cache.

So you can create an index on a list of items, but it's not going to do what you expect.  Consider the following records:

ID
Name
Categories

1
John
red, green, blue

2
Bill
red, purple

3
Jane
red, orange

4
Brian
red, purple

You may expect your Categories index to look like the following:

Categories
ID

red
1, 2, 3, 4

green
1

blue
1

purple
2, 4

orange
3

But they'll actually look like this:

Categories
ID

red, green, blue
1

red, purple
2, 4

red, orange
3

So if you did a query like this:
bh.Where("Categories").Contains("red").Index("Categories")

It'll work, but you'll be reading more records than you'd expect.  You'd only "save reads" on values where the list of categories exactly match.
The current behavior appears to match what Postgres does with arrays, and I think, if we'd want to support indexing individual items in slices individually we'd want to have a separate index type that is an explicitly used, such as with a separate struct tag.  Either that, or use an optional interface that you could implement to explicitly determine how any type is indexed.

Thanks for the detailed answer, very cool! Essentially, what I'm trying to replicate here is an index for a join table in a many-to-many relation (speaking in RDMS terminology). Looks like right now there is no way to achieve that using bolthold. What I still could do is manually create an index bucket directly on the bolt.Db instance. But I would manually have to keep it consistent all the time, which is going to be pain.
The Readme mentions that

you can implement the Storer interface, to specify your own indexes

Could this potentially help me? And do you have an example on how to use Storer? The API docs alone didn't really help me.
Also, do you plan to implement one of your two suggested solutions some time in the future?

If you're trying to do a join, then I would recommend de-normalizing the data, just like you'd want to do in Postgres or any other RDMS.
It could be I'm missing specifics because the example we're using is so simple, but if you are wanting to join two different sets of records, they should be stored separately.
There are no joins in BoltHold, so you'd need to do the join manually, by looping through both record types, regardless.
I'll leave this issue open until I implement sliced indexed, or define a clearer issue separately.
Thanks,

Actually "join" was the wrong word. In the simplest case I really just need sliced indexes, so +1 for that feature 😊

Regarding the edit - I think this might be the better route; create a type alias for the string slice, then implement the Storer interface, but I'm just now looking at bolthold for the first time today, so the best I can do is offer the anonymous implementation within the library that uses reflect:
https://github.com/timshannon/bolthold/blob/master/store.go#L143
and
https://github.com/timshannon/bolthold/blob/master/store.go#L161
the hardest part seems like maybe looping through your slice and generating an Index func for each slice index; the rest is be implementing the Storer interface, which if I understand correctly, bolthold will just use if defined.
I'm thinking there's probably more effort to this than it looks on the surface, but I'm going to need a similar structure, so I'll find out in the next few days!


hpsjam. Beautify and added credit to Nils.

You should use the full name here. And ":" after hpsjam in the commit message.

full name you mean Nils's? If so, I don't know it, sorry.

Does Nils have a GitHub account?

i think yes, but also dont know it

Let's try to ask him first.

Sent him an e-mail.

he does
;)

Great, Nils! Can we add your real full name though to the credits? If so, what is it pls?

ahhhh, though it was Nils, but it's you Missy, sorry!

haha ja :P
saw loads of messages in my email so had to check it out. I'm sure he'll reply soon enough himself!

@dingodoppelt

Submitted manually. Thanks!


operator stackable-listener-operator (24.7.0)

New operator bundle
Name: stackable-listener-operator
Version: 24.7.0
Certification project: 645b8655f581e1c69cab4eaf
Test result URL: https://catalog.redhat.com/api/containers/v1/projects/certification/test-results/id/66b5c649d18b4b75ca9f315c
Test logs URL: https://catalog.redhat.com/api/containers/v1/projects/certification/artifacts/id/66b5c6468193a788e0d19d11

DO NOT REOPEN!


doc(readme): removed invalid reference to godoc

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (36d926b) to head (ad6cca4).

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          168       168
=========================================
  Hits           168       168

Flag
Coverage Δ

optional
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

coverage: 100.0%. remained the same
when pulling ad6cca4374a95d60f9862a1f47aa8fde0221aaf9 on feature/readme
into 36d926b34fbe5a65549a5237ce77206ff76a14ff on main.


Add a "locally scoped" ref to avoid unpredictable references in recursive Token definitions

Given the following pseudo-java code which has a recursive Token A:
final Token A = seq("A",
    def("b", con(1)),
    token("AorB"),
    def("c", con(1), ge(last(ref("A.b")))
);
final Token B = seq("B", def("a", con(1)));
final Token AorB = cho("AorB", A, B);

It seems impossible to calculate how far back in the ref("A.b") stream we have to look to get the b field that was parsed at the same level of recursion (or is nesting the better term)?
I would like to propose an alternative to ref which is a ref that only searches the local stack and higher, not in the children.
(problem discovered together with @pinoval )

After some more discussion and considering cases with @pinoval, it might be that we need a kind of stackfull reference. So that we can also pass along a value expression to a function, that keeps on referring to this stack, even though it's used at a deeper level

Sounds interesting!
I think my initial approach will be to solve the case as described at the top. I have some fairly simple ideas, but I'm not sure if they will work. It is quite possible that if those solutions solve this case that they are not adequate for what you are doing -- but that would be the time to come up with a new sample case :)
The most basic approach will be to implement a ValueExpression that refers to a value's current scope with regard to referencing other values. So basically this would allow you to create scopes on the fly based on where you are. I will create a branch for the working test case and possible solutions.

I've tried to create a horrible example:
    final Token A = seq("A", def("b", con(1)), ABC, d(last(ref("A.b"))));
    final Token B = seq("B", def("b", con(1)), d(last(ref("B.b"))));
    final Token C = seq("C", def("b", con(1)));
    final Token ABC = cho("ABC", A, B, C);

    final Token d(ValueExpression x) {
        return seq("D_dynamic", ABC, def("c", con(1), eq(x)));
    }

The d function is a way to reduce duplication, we've seen this pattern before in other metal examples, and its quite a useful one. But with the recursive/nested structures, it's impossible to pass it the right ValueExpression that points to the correct field at the correct nesting level.

The most basic approach will be to implement a ValueExpression that refers to a value's current scope with regard to referencing other values. So basically this would allow you to create scopes on the fly based on where you are. I will create a branch for the working test case and possible solutions.

This sounds like it might be good enough, even for the above example.

For completeness sake, I wanted to add another example that @pinoval and I worked on:
    private static final Token Lower200 = def("lowValue", con(1), ltEqNum(con(200)));

    private static final Token CheckedREP = seq("CheckedREP",
        def("magic", con(2), eq(con(42,42))),
        rep("items", Lower200),
        def("countCheck", con(4), eqNum(len(ref("CheckedREP.items"))))
    );

    private static final Token RepTwice = seq("Container",
        CheckedREP,
        CheckedREP // now for this one, how will `countCheck` be scoped to only get the items inside this CheckedREP, and not the previous one as well?
    );

Thanks for more examples, I will turn them all into actual test cases!

There are more complex examples to think of, when you start making a recursive data structure, but I'll leave those to your appetite.


Document how to do an image that is redrawn/updated

Hi,
I've just managed to get something to work, but something that telling me how to do it properly would have been better!
I've got a fractal renderer that renders into a buffer (with OpenCL), and I want to be able to do a callback that causes it to be re-rendered.   I'm using a Gtk::Image to display the image.  In the end I ended up using a Cairo ImageSurface and using get_data() to get the buffer to pass into the ocl buffer read() method;  What took me longer to figure out was that since get_data() fails with non-exclusive as soon as the ImageSurface is associated with the Image, my redraw code has to do  set_from_surface(None) before the get_data() and then set_from_surface again with the real value.
That's something that coming from C is non-obvious and I don't think there's any current doc explaining how to do it.

There's a solution for this in https://github.com/gtk-rs/gtk-rs/issues/75 and that should be documented.


WIP: SwiftPM does not always build dependency executable before invoking build tool plugin

Background & Motivation
Plugins can specify dependencies on executables.  Currently, those executables get built before running build tool commands returned by the plugin if those dependencies were either:

the path of the executable of a build tool command returned by the plugin, or
explicitly included by the plugin in the list of input dependencies to a build command returned by the plugin

This could potentially be more efficient than building all dependencies, since a plugin could depend on two separate executables and then return commands that only used one or the other of them.  This would result in SwiftPM only building the executables that were actually mentioned by build commands.
However:  This led to confusion and multiple bug reports of failures resulting from not building the executables before trying to use them.  One common case is with SwiftProtobuf, which uses protoc as the executable and then a SwiftPM-built executable as one of the parameters to protoc.  If the plugin didn't explicitly include the code generation executable in the list of input dependencies in the build tools command it returns, the build would fail and it wouldn't be clear why.
Changes
Make the list of input dependencies of an llbuild command that runs a plugin-provided build tool include the paths of all the tools on which the plugin has declared a dependency, not just the one that appears as an executable in the build command.
This can cause more command line tools to be built but is less confusing overall since a dependency on an executable by a plugin understandably implies that the executable should be built no matter what.
Note that if the plugin doesn't return any build commands, none of the dependencies need to be built.  That behavior is preserved.
Specific Modifications:

include the paths of the depended-upon tools in the input dependencies to the plugin-provided build tool commands
stop special-casing the executable path of build tool commands (any tool there is now included in the list of input dependencies from tools)
add a unit test

Results
The executables declared by a plugin are built before any build command returned by the plugin (though not if the plugin doesn't return any commands, which is a no-op).
rdar://93679015
Note:  This is WIP because it still needs the unit test.

@swift-ci please smoke test

Looks like there is at least one test to update (because it codified the previous dependency) in addition to adding the new one.

@swift-ci please smoke test

@swift-ci please smoke test


Reverted c83a712

c83a712 breaks the modeladmin, settings, and snippets edit views when using MultiFieldPanel.

Closed due to #4334 may be reverted entirely. Thanks @gasman.


pull request for e2e test at 01/30/2017 04:15:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


spree 1-0 compatibility

Versionfile: added 1.0
checkout.js removed since it copies spree origin checkout.js
models, controllers, views namespacing
Refactored: don't replace original checkout form. Just prepend our choose address subform
CheckoutHeper removed (not needed after refactoring)
Order decorations removed (seems not needed for spree 1.0)

Is there a schedule to make this 1.0 compatible? If so, what is it, if not, bump!
Thomas

Unfortunately, this doesn't appear to work for now, at least for me.  See: https://github.com/spree/spree/issues/1056
Adding this breaks the js that shows the state dropdown versus text field.


📑 Feature Request:  Email Verification via OTP

IMPORTANT TO CHECK BEFORE ISSUE

[x] You check that same issue is not present in the repository
[X] You have read contribution guidelines and rules

Please describe your feature request related to a problem?
I genuinely loved the product and the application, given that using git and github has become sort of a daily activity.
Currently, users are registered based on their email addresses, but there's no verification in place to ensure the validity of the email provided. Implementing a validation process, such as sending an OTP to verify the user's email, would significantly improve both the security and the practicality of the application. This additional layer of verification will ensure that only legitimate users with valid email addresses are registered, boosting user trust and safeguarding the platform from misuse.
Describe your solution to problem
I would like to use the Nodemailer npm package to set up a smtp server to send otp to the email, entered on the Resgister page input by the user, and store the otp in the database. Then would ask the user to enter the opt sent to him/her by the email. If the opt entered by him/her matches the lastest opt stored in the database with his/her email, we would allow him to register else not.
Describe alternatives you have considered
No response
Additional context
For storing the OTP I would like to create a new OTP Schema in the database, which will have 3 feilds

Email : storing the email, where the OPT is to be sent.
Value : the random six digit value of the OTP
isConsumed : a boolean value which would indicate whether the otp is used up or not. If it is created and not used up for verifying, then it would be false and if used up then true.

@rahulsainlll  please assign me this issue
Add Relevant Screenshot (if available)
No response

We don't need this right now


Error: undefined local variable or method `stdin'

Hello,
starting svn2git with
sudo svn2git http://<IP_ADDRESS>:8080/svn/erpsystem/ --username transfer --verbose
results in this error message:
> #<Thread:0x00005593c592b338@/var/lib/gems/2.5.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:431 run> terminated with exception (report_on_exception is true):
> Traceback (most recent call last):
> 	2: from /var/lib/gems/2.5.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:432:in `block (2 levels) in run_command'
> 	1: from /var/lib/gems/2.5.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:432:in `loop'
> /var/lib/gems/2.5.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:438:in `block (3 levels) in run_command': undefined local variable or method `stdin' for #<Svn2Git::Migration:0x00005593c572f7f0> (NameError)
> Did you mean?  String

I am running Ubuntu 18.04
svn checkout http://<IP_ADDRESS>:8080/svn/erpsystem/ --username transfer
works without any problems.
Any ideas?
Best regards
Christoph

Hello,
same probleme for me, ubuntu 18.04
ruby version is 2.5.0.
It worked for me 3-4 month ago on another computer but now anymore with fresh install.
Perhaps ruby version is the cause ?

Dont really know what I have done but it seem to work.
I installed "rvm"   ruby version manager
I installed ruby 2.4.0  with rvm and set it to be the default
Then I changed "sudo svn2git"  by "rvmsudo svn2git"

same here, if I am local to the machine svn2git works because it pops password prompt graphically. When I am remote, ssh, it does not accept password from the command line same as above.

@adrsr it fails for me when running as you suggested:
 yes <password> | svn2git svn+ssh://<svn-url> --username <username> --authors ~/authors.txt
with the below error:
#<Thread:0x00007febba3e4ee8 /Users/serguei/.rvm/gems/ruby-2.7.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:431 run> terminated with exception (report_on_exception is true):
Traceback (most recent call last):
	2: from /Users/serguei/.rvm/gems/ruby-2.7.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:432:in `block (2 levels) in run_command'
	1: from /Users/serguei/.rvm/gems/ruby-2.7.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:432:in `loop'
/Users/serguei/.rvm/gems/ruby-2.7.0/gems/svn2git-2.4.0/lib/svn2git/migration.rb:438:in `block (3 levels) in run_command': undefined local variable or method `stdin' for #<Svn2Git::Migration:0x00007febba3d5dd0> (NameError)
Did you mean?  String
...

hello @belgoros
I'm afraid I can't help you. Maybe you can try with Debian OS. That's what I used for the migration.
Unfortunately, this project is no longer maintained and there are a lot of bugs.

Hmmm, no longer maintained, but referenced in the official gitlab docs?
https://docs.gitlab.com/ee/user/project/import/svn.html#cut-over-migration-with-svn2git

Hmmm, no longer maintained, but referenced in the official gitlab docs?
https://docs.gitlab.com/ee/user/project/import/svn.html#cut-over-migration-with-svn2git

Same error here.
$ lsb_release -a
Description:    Ubuntu 20.04.1 LTS
$ ruby -v
ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-linux-gnu]

Same error here.
$ lsb_release -a
Description:    Ubuntu 20.04.1 LTS
$ ruby -v
ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-linux-gnu]

For me it was a communication problem with svn2git and the git binary....
can you try (INSECURE, please check / remove your .gitconfig afterwards as it will include plain text passwords):
git config credential.helper store
git config --global user.name "your username"
git config --global user.password "your password"

then retry your svn2git command... everytime it fails it will tell you with what command it fails:
command failed:
git svn init --prefix=svn/ --username=**** --no-metadata --trunk=trunk --tags=tags --branches=branches https://*****

retry that command on the commandline to get more information from git itself... once i ironed out all the password requests from git .... svn2git worked without any problems...
hope it helps.

This problem seems to be related to missing optional (perl?) packages for git.
On Archlinux I can see the optional dependencies with this command:
pacman -Si git
Then I just installed all of them:
sudo pacman -S perl-libwww perl-term-readkey perl-mime-tools perl-net-smtp-ssl perl-authen-sasl perl-mediawiki-api perl-datetime-format-iso8601 perl-lwp-protocol-https perl-cgi python subversion org.freedesktop.secrets libsecret tk
After that it worked.

https://github.com/mazong1123/svn2gitnet
in case I forgot, this is a .net core version of the tool that @mazong made which does not suffer from this issue.
switching to this adaptation solved it for me.
it has both fixed the "--password option not working" and the "stdin loop" issues

Some comments for people equally late to the git train who run into this when following the instructions in the main README:

if you are on Debian or Ubuntu, install the package with apt-get instead of the gem [1], other distributions may have similar fixes
if you are on the console, e.g. via ssh, attempting to provide the svn password and either (a) the --password option of svn2git does not work because git does not have a --password option (any more), or (b) entering a password gets you "No input requested." as a reply, do not waste time trying to get subversion to store the password for you. I could not get it to work, it seems that this feature has been removed in modern versions to some extend?! Instead, try to get a something graphical. I used xrdp and Remote Desktop to get a Terminal window, which then let svn pop-up a GNOME keyring window, which finally stored the password and svn2git began the conversion.

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=905832


Full 0.5.2 changelog

Based on @beruic's #593.
The only thing I left out was #670 since it was just fixing the spelling of a name in the AUTHORS document.

@bmihelac This includes all the changes since 0.5.1 as of now.

Coverage remained the same at 90.48% when pulling 4932d6bf69383f4ee74a35bf48f8e15a0442479b on mgrdcm:full-0.5.2-changelog into 40e7d6dfb302ba384a11ecd61429e724425d1d8d on django-import-export:master.

Coverage remained the same at 90.48% when pulling 705b5f3096db89ad2725c94ba262362f2085f47f on mgrdcm:full-0.5.2-changelog into 40e7d6dfb302ba384a11ecd61429e724425d1d8d on django-import-export:master.

Coverage remained the same at 90.48% when pulling 705b5f3096db89ad2725c94ba262362f2085f47f on mgrdcm:full-0.5.2-changelog into 40e7d6dfb302ba384a11ecd61429e724425d1d8d on django-import-export:master.

Coverage remained the same at 90.48% when pulling 705b5f3096db89ad2725c94ba262362f2085f47f on mgrdcm:full-0.5.2-changelog into 40e7d6dfb302ba384a11ecd61429e724425d1d8d on django-import-export:master.

Coverage remained the same at 90.48% when pulling 705b5f3096db89ad2725c94ba262362f2085f47f on mgrdcm:full-0.5.2-changelog into 40e7d6dfb302ba384a11ecd61429e724425d1d8d on django-import-export:master.

Great, thanks @mgrdcm!


Force selection of first namespace

Closes https://github.com/StatCan/aaw/issues/1812

New pr was created


Renamed tf2onnx tutorial

did you sync the latest code? the formal name of package is onnxruntime_customops, not the abbr.

did you sync the latest code? the formal name of package is onnxruntime_customops, not the abbr.

did you sync the latest code? the formal name of package is onnxruntime_customops, not the abbr.

Oops.  Should be fixed now.

did you sync the latest code? the formal name of package is onnxruntime_customops, not the abbr.

Oops.  Should be fixed now.


Unexpected behavior for higher-order code generation

Unexpected results are produced when the generation of LLVM Code from Thorin code containing higher-order code
is attempted.
Small example:
extern "C" {
    fn g(fn(i32)->i32) -> i32;
    fn h(i32) -> i32;
}

fn main() -> i32 {
    g(h)
}

generates the llvm code
declare i8* @malloc(i64)

declare i32 @g({i32} (i32) // Error
declare i32 @h(i32)

define i32 @main() {
main_1038:
    %g_cont_1071.ret = call i32 @g({i32} (i32 @h) // Error
    br label %return_1066

return_1066:
    %_1077 = phi i32 [ %g_cont_1071.ret, %main_1038 ]
    ret i32 %_1077

}

Steps: impala -emit-llvm -Othorin [file.impala]
The behavior occurs also at similar points where function types are used.
For instance at the allocation of a pointer slot for a function.
Observed behavior: Invalid LLVM Code is produced.
Expected behavior: The code generation should probably throw an error if unhandled higher-order code is encountered at this point.

The LLVM code generator cannot cope with higher order functions. Right now, there are two things in the work to cope with this:

The cleanup branch has a new phase in the work which specializes higher-order functions.
The closeure-conv branch has work on the way to closure-convert such things.

Still, the LLVM code generator should probably just throw an exception with a reasonable error message, if sth unexpected occurs.


Feel++

have you considered Feel++ ?

No, but it would be great if you can provide a PR with a profile for Feel++ :) This is the reason why I am trying to provide a good comparison in a single place - there are so many packages, it is really hard just to find them all...

Ok, I've forked and starting working on a profile.
I guess I only have to fill a profiles/feelpp.profile copied from any other ?

https://github.com/kostyfisik/FEA-compare/blob/master/main-keys.txt is used for key sorting, so  you can use it as an empty template. For sure you can use any available profile as a starting point too.

I haven't seen a clone has already be done in feelpp. A PR will be submitted within the next two weeks.

@vhuber Any update? :)


Oculus Quest 2 test

Someone, please test the running of the platform on Oculus Quest 2

Tested, works the same as Oculus Quest 1.

Tested, works the same as Oculus Quest 1.


Make synopsis scrollable with mouse wheel.

Modules with lots of definitions result in a synopsis which is too large to view, but is not scrollable.
For example see https://hackage.haskell.org/package/ghc-8.4.3/docs/CoreUtils.html

And I've already fixed it with the new theme! 🎉

Still broken in haddock 2.22.0: https://www.stackage.org/haddock/nightly-2019-03-07/http-conduit-2.3.5/Network-HTTP-Client-Conduit.html#v:method

The fix has not been released yet (it will be in the upcoming 2.23.0 release bundled with GHC 8.8). Besides, Stackage uses a separate theme. Proof this is fixed:


Button design

Make them prettier while remaining accessible

Quick idea

button.button{
  width:15em;
  border-radius:.4em;
  display:block;
  margin:.5em auto;
  background-image:linear-gradient(hsl(185, 77%, 27%) 20%, hsl(185, 77%, 22%));
  border:.1em solid hsl(185, 77%, 20%);
  text-shadow: -1px -1px hsl(185, 77%, 20%), 1px 1px hsl(185, 77%, 27%);
  color:#fff;
}
button.button:hover,
button.button:focus{
  background-image:linear-gradient(hsl(185, 77%, 22%), hsl(185, 77%, 27%) 80%);
  text-shadow: 1px 1px hsl(185, 77%, 20%), -1px -1px hsl(185, 77%, 27%);
}

button.button.off{
  background-image:linear-gradient(hsl(185, 17%, 27%) 20%, hsl(185, 17%, 22%));
  border:.1em solid hsl(185, 17%, 20%);
}
button.button:hover,
button.button:focus{
  background-image:linear-gradient(hsl(185, 17%, 22%), hsl(185, 17%, 27%) 80%);
}

Looks amazing. Do you want to create a PR to add it to the CSS?
Also, is it more semantic use Enable Reader View / Disable Reader View for the button text, since that describes the action a user will take by activating the button?

Yeah I was wondering about that. On/Off is simpler language Enable/Disable is more descriptive.
I'm swaying towards Enable/Disable at the moment but my mind is changing every few seconds.
Also i think the off/disabled button need to switch to dark text on a light background so it looks more different. Maybe using hsl(236, 47%, 94%).


Use Z3-TurnKey instead of a bespoke Z3 build

Closes #213 (I hope!)
Our bespoke build and local installation of the Z3 API dependency adds a lot of configuration overhead and brittleness
to our build. While investigating #213, I read about Z3-Turnkey on https://stackoverflow.com/a/62015869/1187277. The purpose of this library is to provide a Java artifact that

ships its own native libraries,
can use them without administrative privileges, and
can be obtained using Maven.

I'm hopeful that it can simplify our build overall, and address #213.

I just finished a clean build on macOS with this branch using Zulu. Here's my configuration:
% mvn -v
Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
Maven home: /Users/sf/opt/apache-maven-3.6.3
Java version: 1.8.0_265, vendor: Azul Systems, Inc., runtime: /Users/sf/opt/zulu<IP_ADDRESS>-ca-jdk8.0.265-macosx_x64/zulu-8.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.15.6", arch: "x86_64", family: "mac"

@istoilkovska, at your convenience, perhaps you could try a clean build off this branch, and see if it works for you as well? I recommend checking out the repo afresh, since this PR changes the 3rdparty libs and the current build configuration doesn't clean up after itself there.
Regardless of whether this solves #213, since it reduces the initial build time considerably and simplifies our dependencies and build configuration, and seems to be passing all our tests, I think we'll probably want to keep Z3-TurnKey until #182 is closed as per https://github.com/Z3Prover/z3/issues/182#issuecomment-564985228, by integrating it into Z3 itself.

@shonfeder I just have merged your PR, as @Kukovec needs it to fix his issue.

Really cool stuff! I didn't know about z3-turnkey, sounds like it could be very useful for the Scala wrapper we've been developing and using for Stainless/Inox at @epfl-lara.

The author is trying to integrate it it into z3, but said they'll continue maintaining in tandem with z3 releases in the meantime: https://github.com/Z3Prover/z3/issues/182#issuecomment-564985228


k8s 1.29 support for eks-operator

Add k8s 1.29 support for eks-operator

Verified on v2.9-4ac81b66b0f971548be78f2d1c72ecb906171a0b-head, eks-operator:v1.9.0-rc.5 ✅
Chart install/upgrade and P0 scenarios with k8s v1.29 passed.


TS Server Very Slow Diagnostics, Suggestions, etc

ts server has been very slow whether it is showing diagnostics or autocomplete suggestions and etc.

I tried it by deleting an import statement and the diagnostics error showed 13 seconds after as the picture above. Even autocomplete showed very slowly sometimes auto import doesn't even work at all. I am genuinely not sure why. I tried deleting watchman there's no noticeable change. Tried coding on vscodium and it works fine and fast.

It's difficult to reproduce the slow type issue. What's your TypeScript version? What's the size of your project? I'm using the coc-tsserver to work on the coc.nvim project, I think it's big enough, didn't came across the slowness .

You can take a look at the projects here I'm unsure as to why it happens not sure if it's my neovim config or tsconfig. My updatetime is 50 I'm on nvim 0.5.1

I just checked this and does this have anything to do with it? The "trying to cancel ongoing request with sequence number x"

So I think I found the problem but I don't know how to fix it. I keep trying to find what's wrong by deleting an import and then undoing it. I found it's not all the files that are slow it's only model.ts files. I thought at first it was because of the import of other model.ts files which is partially true. I thought it has something to do with the class or because of extending objection model. Like it was too big or something. I found that it's this part of the code

When importing other model.ts for the modelClass on this part of the code it became really slow. If I remove this part of the code and import the models it works just fine. I don't know why it's like this but on vscodium it works just fine no performance problem. On my older projects too it's just fine there's no problem at all.

Should be problem of your language server, checkout https://github.com/neoclide/coc.nvim/wiki/Debug-language-server#using-output-channel

We've add tsserver.enableTracing support, you can find trace file by checkout the output of tsserver.

The tsserver could be slow after some time, use latest typescript > 4.9 and disable logging of tsserver should helps.
  "tsserver.log": "off",
  "tsserver.trace.server": "off",

See https://devblogs.microsoft.com/typescript/announcing-typescript-4-9/#file-watching-now-uses-file-system-events

big thanks @chemzqm
update Typescript to 4.9.3 solved the diagnostic issue


Unexpected motion occurs when hydraulic power enabled due to old position request

Issue
If the PLC is already powered and and hydraulic motor is off, system will still accept position requests. If the hydraulic motor is powered on then the press will begin moving to that position. This is unexpected motion! System should only move in the instance position is commanded and not at a later date.
This is a potential safety issue.
Suggested Fixes

Add flag to ensure position requests are only allowed if hydraulic motor is enabled.
Transition communication architecture to ROS Service (not topics).

Bug identified by @Bthurston119 and @acbuynak

FYI @hansen95 @TobiasJMahan


Disable project implicit configuration on cross-project access

Use internal feature flag for projects coupling disabling

Related PR https://github.com/gradle/gradle/pull/26812


Joyride is erroring out on templates

$(document).foundation({
    joyride: {
      scroll_speed: 500,
      template: {
        button: '',
        prev_button: ''
      }
    }
  }).foundation('joyride', 'start');

Produces an error of:
Uncaught TypeError: Cannot read property 'top' of undefined in foundation.joyride.js?body=1:775
It appears as though the error is on this return statement:
return [
        el.offset().top < top,
        right < el.offset().left + el.outerWidth(),
        bottom < el.offset().top + el.outerHeight(),
        w.scrollLeft() > el.offset().left
      ];

So maybe el isn't defined?  I'm using Foundation 5.2.2 because I'm using the Rails gem.  Is this an issue that has been fixed since then?

It's still a bug, tried to work out with deep copy but not working yet.

We added this to our "known issues" section in our planning board for the next version. A PR would be valuable to the community, otherwise It will be addressed then. You can see and vote on the proposed updates here: https://trello.com/b/QBPKlQqf/foundation-sites


Update benchmarking for GPUs

Changes to allow GPU benchmarking.
Some changes are temporary, and will be undone (with notes in the README) or recommented out (e.g. the Python changes).
Also includes GPU binaries, to avoid the need to rerun the Python before further benchmark tests.
Note: these changes will currently cause a merge conflict with #26.

I plan to update the README here with notes about running benchmarking on GPUs (although it should be relatively straightforward), but it might be worth (me) adding a note in the main FTorch branch about moving models to GPUs based on the lessons from last commit: tensor attributes that aren't registered as parameters/buffers will not be moved to the GPU via model.to("cuda").
Solutions are to register the tensor e.g. self.x = torch.randn(420) -> self.x = nn.Parameter(torch.randn(420)), or to manually move the tensors, as I've done here, noting that .to differs in operation between modules and tensors (the former modifying in-place, while the latter does not).

(The merge conflict with #26 has been resolved, but this will now conflict with #30, which is likely to be merged in before this)

Re-implemented via #31


Readme

Coverage remained the same at 44.8% when pulling 4f3af7c1de229a815d257c521ad5ffef9210958e on readme into 2f9d67b99b6e47181582fb18c91e42eb5af9d027 on master.


ARXIVNG-275 added test for classic UI workflow

@DavidLFielding @JaimieMurdock Ok, per our discussion yesterday, focusing on verifying that we are updating the classic submission database correctly. Anything that I'm not checking that should be checked in the test? Am I interpreting the classic columns correctly?

@JaimieMurdock Here's a rough mapping of the classic submission UI onto these commands/events.

Thanks!


Hibernate filter not available

Summary
In the application tab, I am not able to sort my applications which are on hibernating state. A filter can be added for applications which are on hibernating state.
Motivation

Proposal
A filter needs to be added in Status dropdown in application tab.

Status filters are removed now


"An attempt was made to use the context while it is being configured. A DbContext instance cannot be used inside OnConfiguring since it is still being configured at this point" and "An unhandled exception was thrown by the application. System.InvalidOperationException: The connection was not closed. The connection's current state is connecting."

Steps to reproduce
Nothing different in my database configuration. I have configured my database as:
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddAuthentication();

            // Add framework services.
            services.AddApplicationInsightsTelemetry(Configuration);

            services.AddMvc(opts =>
            {
                opts.Filters.Add(new CustomJSONExceptionFilter());
            });

            services.AddDbContext<DbContext1>(options => options.UseSqlServer(Configuration.GetConnectionString("DbContext1")));
            services.AddDbContext<DbContext2>(options => options.UseSqlServer(Configuration.GetConnectionString("DbContext2")));
        }

The issue
As soon as I start the application, the log files show some errors referring to the database access.
The exception details are given as below:
fail: Microsoft.AspNetCore.Server.Kestrel[13]
      Connection id "0HKUNESV0QS7N": An unhandled exception was thrown by the application.
System.InvalidOperationException: An attempt was made to use the context while it is being configured. A DbContext instance cannot be used inside OnConfiguring since it is still being configured at this point.
   at Microsoft.EntityFrameworkCore.DbContext.InitializeServices()
   at Microsoft.EntityFrameworkCore.DbContext.get_InternalServiceProvider()
   at Microsoft.EntityFrameworkCore.Infrastructure.AccessorExtensions.GetService[TService](IInfrastructure`1 accessor)
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.<.ctor>b__2_0()
   at Microsoft.EntityFrameworkCore.Internal.LazyRef`1.get_Value()
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.System.Linq.IQueryable.get_Provider()
   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.ExecuteAsync[TSource,TResult](MethodInfo operatorMethodInfo, IQueryable`1 source, Expression expression, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.ExecuteAsync[TSource,TResult](MethodInfo operatorMethodInfo, IQueryable`1 source, LambdaExpression expression, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.SingleOrDefaultAsync[TSource](IQueryable`1 source, Expression`1 predicate, CancellationToken cancellationToken)
   at MyCompany.Services.Core.Services.OneService.<GetAsync>d__2.MoveNext() in C:\Projects\Source\Repos\MyCompany\Code\Projects\MyCompany.Services.Core\Services\OneService.cs:line 18

Further technical details
EF Core version: (1.0.0)
Operating system: Windows Server 2012 R2
Visual Studio version: VS 2015 update 3
This error comes up first few times the database is accessed in the code, but then this doesn't show up. However, another similar error often comes up when I make several requests on server simultaneously, like load test on several-threads. That error details is given below:
Exception Trace:
fail: Microsoft.AspNetCore.Server.Kestrel[13]
      Connection id "0HKUNESV0QSA4": An unhandled exception was thrown by the application.
System.InvalidOperationException: The connection was not closed. The connection's current state is connecting.
   at System.Data.ProviderBase.DbConnectionClosedConnecting.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.OpenAsync(CancellationToken cancellationToken)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.<OpenAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.EntityFrameworkCore.Query.Internal.AsyncQueryingEnumerable.AsyncEnumerator.<MoveNext>d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.EntityFrameworkCore.Query.Internal.AsyncLinqOperatorProvider.<_SingleOrDefault>d__123`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.EntityFrameworkCore.Query.Internal.TaskResultAsyncEnumerable`1.Enumerator.<MoveNext>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.EntityFrameworkCore.Query.Internal.AsyncLinqOperatorProvider.ExceptionInterceptor`1.EnumeratorExceptionInterceptor.<MoveNext>d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at MyCompany.Services.Core.Services.OneService.<GetAsync>d__2.MoveNext() in C:\Projects\Source\Repos\MyCompany\Code\Projects\MyCompany.Services.Core\Services\OneService.cs:line 18

@ravipunjwani This is most likely happening because the same context instance is being used concurrently by multiple threads. DbContext is not thread-safe. If the context instance is not being used concurrently but this is still happening, then please post some code showing how you are obtaining and using instances of the context.

@ajcvickers I'm using the same context in several classes passing it through constructors. But, each of my methods in all these classes (that use the dbcontexts) are async. So ideally each call in the hierarchy would not get called unless the other calls get completed. Otherwise if they are still called simultaneously, definitely they would belong to separate http requests. The load-test I referred in the description were initiated using separate http-requests (and separate httpclients) for each call, from a single unit-test function.

Below is the code from ServiceHelper class and startup.cs, and you can see I called the erring function serviceHelper.OneService.GetAsync in this place.
namespace MyCompany.Services.Core
{
    public class ServiceHelper
    {
        private DbContext1 dbContext1;
        private DbContext2 dbContext2;

        public SiteService SiteService;
        public OneService OneService;
        public NameService NameService;

        public ServiceHelper(DbContext1 dbContext1, DbContext2 dbContext2)
        {
            this.dbContext1 = dbContext1;
            this.dbContext2 = dbContext2;

            this.SiteService = new SiteService(dbContext1);
            this.OneService = new OneService(dbContext2);
            this.NameService = new NameService(dbContext1);
        }
    }
}

Startup.cs:
----------
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddAuthentication();

            // Add framework services.
            services.AddApplicationInsightsTelemetry(Configuration);

            services.AddMvc(opts =>
            {
                opts.Filters.Add(new CustomJSONExceptionFilter());
            });

            services.AddDbContext<DbContext1>(options => options.UseSqlServer(Configuration.GetConnectionString("DbContext1")));
            services.AddDbContext<DbContext2>(options => options.UseSqlServer(Configuration.GetConnectionString("DbContext2")));

            services.AddScoped<ServiceHelper>();
	}

        public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory, IOptions<ConfigSettings> configSettings, ServiceHelper serviceHelper)
	{
            app.UseBasicAuthentication(new BasicAuthenticationOptions
            {
                Realm = "xxxx",
                AutomaticChallenge = false,
                Events = new BasicAuthenticationEvents
                {
                    OnValidateCredentials = async context =>
                    {
                        int siteId;
                        if (!int.TryParse(context.Username, out siteId))
                        {
                            return;
                        }

                        string key = context.Password;

                        vSite site = await serviceHelper.OneService.GetAsync(siteId);

                        if (site.key != key || site.SiteActive != 1)
                        {
                            return;
                        }

                        if (/*SOME-LOGIC-HERE*/)
                        {
                            ClaimsIdentity identity = new ClaimsIdentity(
                                new List<Claim>
                                {
                                                /* ADD-CLAIMS-HERE */
                                }
                            );

                            context.Ticket = new AuthenticationTicket(
                                new ClaimsPrincipal(identity),
                                new AuthenticationProperties(),
                                context.Options.AuthenticationScheme);
                        }

                        return;
                    }
                }
            });

            app.UseMvc();
	}

Got this solved by myself.
I was incorrectly using my serviceHelper object (can be seen in Configure function parameters injected as service).
Removed this parameter and replaced the line:
                       vSite site = await serviceHelper.OneService.GetAsync(siteId);
with
                        ServiceHelper serviceHelper = context.HttpContext.RequestServices.GetService<ServiceHelper>();
                        vSite site = await serviceHelper.OneService.GetAsync(siteId);

This gets the ServiceHelper instance from the current request context, instead of the one passed in Configure function as injected service.

@ajcvickers  Hey I'm getting the same error message.  Mine has something to do with concurrency issues although I'm not sure why... My app is stateless and basically is an api.  I set up my context with
services.AddDbContext<AvocadoContext>(options => options.UseNpgsql(connectionString));
and is accessed in middleware through a service
services.AddTransient<EventMemberService>();
everything is fine as long as I only perform one request at a time.  If I perform multiple requests however I get
An attempt was made to use the context while it is being configured. A DbContext instance cannot be used inside OnConfiguring since it is still being configured at this point
Entity Framework can handle this right? I thought EF was fine as long as a new Instance of the Context was created on each request, which is the default ServiceLifetime (Scoped).  If so any ideas on why I'm getting this error?  Maybe I have some bad configuration... if so though I'm not sure where to look.  Any help would be greatly appreciated. Thanks!

@snaveevans Have you verified that you are getting a new instance each time?

Hi @snaveevans ,
EventMemberService eventMemberService = context.HttpContext.RequestServices.GetService<EventMemberService>()
The above line of code might fix your problem, as it did for me the same way.
Though I added my service via services.AddScoped compared to yours services.AddTransient.
Both should work, however I wanted to avoid creating another instance within same request. Visit here for more on the difference between transient and scoped.

@ajcvickers not sure how to check if it is creating a new instance or not, how can I do that?  (for future reference) It's now fixed though.
@ravipunjwani Thanks! that code worked for me as well.  I switched them all to Scoped instead of Transient and I still get the same issue.  But once I pull the service directly from the HttpContext it runs just fine... Why is that though?  I used the "standard" way of adding services via constructor injection.

@snaveevans Typical ways are using the debugger or tagging each instance with a unique value and logging it.

@snaveevans The same SO issue that I referred to earlier, shows exactly how to achieve this and see differences visually.
Switching transient and scoped would be same for you because you were using constructor injection, which would work great for the Configure method in Startup.cs. The middleware is placed in the request pipeline and would need a reference to the service instance from current request context (here comes the decision of transient and scoped - transient would give you new instance even in current request and scoped would give you a unique instance per http request lifecycle).
I also got confused earlier but figured out sooner than later as the production app started creating a lot of issues.

I am having the same issue as @snaveevans described. My app is also a webapi, and console application calls this api endpoint everytime a file is dropped into a hotfolder. The web api doesnt throw any errors if I call the endpoint in a sequential manner. As soon as start calling it in async method, the api end point throws this error:
An attempt was made to use the context while it is being configured. A DbContext instance cannot be used inside OnConfiguring since it is still being configured at this point
Any help is really appreciated
Thanks

@nshathish The context is not thread safe. From what you say it seems likely that their are multiple calls accessing the same context instance, possibly due to how your async code is written.

@nshathish  In your WebApi project the DbContext injection has to be Transient or PerThread.


AttributeError: module 'kornia' has no attribute 'rgb_to_grayscale'

hi,
how to solve this problem?

hi,
how to solve this problem?

You must do kornia.color.rgb_to_grayscale

hi,
how to solve this problem?

You must do kornia.color.rgb_to_grayscale

Thank you for your help!


SourceMap is missing when UglifyJSPlugin is enabled

I'm submitting a bug report
Webpack version:
2.1.0-beta.4
Please tell us about your environment:
OSX 10.x
Current behavior:
Source map is not generated when new webpack.optimize.UglifyJsPlugin() is used.
If I remove UglifyJSPlugin or use webpack version 1.13, it does produce source map.
Expected/desired behavior:
Source map should be generated.

My config

const path = require('path');
const webpack = require('webpack');
const WebpackNotifierPlugin = require('webpack-notifier');
const styleLintPlugin = require('stylelint-webpack-plugin');

const env = process.env.NODE_ENV;

module.exports = {
    devtool: 'source-map',
    entry: './src',
    output: {
        path: path.join(__dirname, 'dist'),
        filename: 'app.js',
    },
    plugins: [
        new webpack.DefinePlugin({
            'process.env': {
                NODE_ENV: JSON.stringify('production'),
            },
        }),
        new webpack.optimize.DedupePlugin(),
        new webpack.optimize.UglifyJsPlugin()
    ],
    module: {
        loaders: [
            { test: /\.js$/, exclude: /node_modules/, loader: 'babel' },
            { test: /\.js$/, exclude: /node_modules/, loader: 'eslint' },
            { test: /\.css$/, exclude: /node_modules/, loader: 'style!css?modules&importLoaders=1&localIdentName=[local]_[hash:base64:5]' },
        ],
    },
};

Language: [ES6/7]

new webpack.optimize.UglifyJsPlugin({
  sourceMap: true
})

We changed the default, because it's pretty expensive

Wouldn't it be better if the plugin detects if a devtool has been set, and toggles sourceMap accordingly as a default?

It would also be nice to mention this in the Building for Production docs, because currently it tells us to enable source maps, then to enable UglifyJS, and neglects to mention that the latter disables the former.

We changed the default, because it's pretty expensive

Wow. I have been searching for ages as to why it wasn't working.

Wouldn't it be better if the plugin detects if a devtool has been set, and toggles sourceMap accordingly as a default?

This.

I just lost an hour to this issue, speaking of expensive :(

I lost an entire work day due to this.

How did you make it work?

The solution is in this comment: https://github.com/webpack/webpack/issues/2704#issuecomment-228860162

Doesn't seep to work:
"mappings":"AAAA"

Doesn't work still with solution, no sourcemaps are produced

Had the "mappings":"AAAA" issue too. Fixed by changing production source maps to source-maps and enabling sourmap: true in uglify. Before I had sourcemaps set to cheap-module-source-map which didn't work.

can't work with UglifyJsPlugin and cheap-module-source-map either

Same problem here. I have this :
let config = {
	...
	devtool: dev ? 'cheap-module-eval-source-map' : false,
        ...
}

And this :
new UglifyJsPlugin({
	sourceMap: true,
	comments: false
})

And no source maps generated.

This should respect the devtool setting, or at least detect that devtool is set to produce source maps and emit a warning if the equivalent uglify plugin option is not set.

We should at least add a log message if the devtool is configured and this option is missing.

This was happening to me, I had to set the sourceMapFilename
  output: {
    ...
    sourceMapFilename: 'bundle.map'
  },
  plugins: [
    new webpack.optimize.UglifyJsPlugin({
      sourceMap: true
    })
  ]


New header manipulation API

implements https://github.com/istio/api/pull/709
fix https://github.com/istio/istio/issues/9595

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: itaysk
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: vadimeisenbergibm
If they are not already assigned, you can assign the PR to them by writing /assign @vadimeisenbergibm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS
tests/e2e/tests/pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @itaysk. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

High level comment: if the appendFlag was recently introduced (after Istio 1.0, which is most likely), please guard the config generation pieces with a proxy version check, so that for 1.0 proxies, its only append/remove, while for 1.1 proxies, its set/append/remove [if and only if this functionality didn't exist by Istio 1.0 (around June).

@itaysk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTests

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth-mcp.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-mixer-no_auth-mcp

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-mixer-no_auth-mcp

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

prow/e2e-dashboard.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

prow/e2e-dashboard.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-dashboard

prow/istio-pilot-multicluster-e2e.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

prow/istio-pilot-multicluster-e2e.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-dashboard

/test e2e-dashboard

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

prow/istio-pilot-multicluster-e2e.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Can you sync your branch to release 1.1 fully and see if the test failure goes away?
@douglas-Reid any idea why this dashboard test is failing for unrelated code?

New changes are detected. LGTM label has been removed.

Rebased

/approve
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: GregHanson, itaysk
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [GregHanson]
tests/e2e/tests/pilot/OWNERS [GregHanson]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

prow/istio-unit-tests.sh
278c98e7240c3cc348ef2569bbf1816359dd218f
link
/test istio-unit-tests

prow/e2e-dashboard.sh
278c98e7240c3cc348ef2569bbf1816359dd218f
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/test e2e-dashboard

@itaysk: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
43329fd55bf8648a5ceef2b8f0a5b578a7e42b3a
link
/test e2e-simpleTestsMinProfile

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
278c98e7240c3cc348ef2569bbf1816359dd218f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3


add a re-direct to the front-end

Redirect try.dartlang.org dartpad.dartlang.org (301). It's served on the same server; having the different url looks odd.

Fixed by #401.


Getting a mailbox not found error in Outlook on the web

Hi,
I have this message when trying to sign in to email web, Office Business Standard 365.
There is a license assigned to this user.
The email address is a custom domain name jj-trinquet.com. The email address is<EMAIL_ADDRESS>Any help with this will be greatly appreciated.
Yours kindly JJ Trinquet

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

ID: 00421a8e-42cf-c393-f91d-da1b7d7cc7da
Version Independent ID: de00b816-762d-2171-eb7f-1ce6a55ec189
Content: Getting a mailbox not found error in Outlook on the web - Microsoft 365 admin
Content Source: microsoft-365/admin/misc/mailbox-not-found-error.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

I've hit this error with two accounts today. Both have recently been updated, both have licenses, both have had email addresses and mailboxes in 365 for a while now.

@msbemba  Thank you very much for the contribution and sharing this explanation. @jjtrinquet Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


json-schema-validator build failure

Operating system
Windows
Compiler
MSVC /c++:latest
Steps to reproduce the behavior
Breaks after updating to VS 17.6.2.
Updated vcpkg to latest but it didn't help.

Error seems to be in nlohmann, although we use that separately and it's ok (maybe embedded version is old?).

Failure logs
Using source at C:/Temp/MdpCpp/vcpkg/buildtrees/json-schema-validator/src/2.2.0-e87f98a2b7.clean

 C:\Temp\MdpCpp\vcpkg\buildtrees\json-schema-validator\src\2.2.0-e87f98a2b7.clean\src\json-validator.cpp(606): error C7692: 'bool nlohmann::json_abi_v3_11_2::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::json_abi_v3_11_2::adl_serializer,std::vector<uint8_t,std::allocator<uint8_t>>>::operator ==(const nlohmann::json_abi_v3_11_2::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::json_abi_v3_11_2::adl_serializer,std::vector<uint8_t,std::allocator<uint8_t>>> &) noexcept const': rewritten candidate function was excluded from overload resolution because a corresponding operator!= declared in the same scope

Additional context
No response

/c++20 is the same, so it's not just C++23 mode.

It seems that in the new version, operator== and operator!= has been deprecated . Please use teh latest version and try again.
vcpkg has updated the version. :)
https://github.com/nlohmann/json/releases/tag/v3.11.2#:~:text=🔥,. %233684

Same issue for me. It works fine when installing json-schema-validator standalone without vcpkg (still using vcpkg for nlohmann_json)

There's a workaround according to https://github.com/nlohmann/json/issues/3868#issuecomment-1563726354
by adding following to the overlay-triplet:
if(${PORT} MATCHES "json-schema-validator")
    set(VCPKG_CXX_FLAGS ${VCPKG_CXX_FLAGS} "-DJSON_HAS_THREE_WAY_COMPARISON=0")
endif()


Updated settings UI

Updated settings UI, still need to implement toggle notification feature.

Looks good, merge it


Added a slide on span

and fixed details on the string_view one
Closes #50

Added a line on static size, split in 2 slides and thus expanded the examples (including with static size)


Problem viewing Gaussian Splats in VR - UE5.4

It seems that the gaussian splatting in the vr glasses looks weird. Both in the preview and after packing.
It's as if the same image is being projected in each eye, instead of generating one for each eye.
Any idea how to solve this?

I have the same problem. With one eye, the rendering is fine but with the other eye (right), the splat is missing the color.

Same problem here with VR. Can someone solve it?


MM-17838 Wrap reactions when they exceed screen size

Summary
Replaces the ScrollView with a container view with proper flex settings to wrap the reactions and button when they exceed screen size.
Ticket Link
https://mattermost.atlassian.net/browse/MM-17838
Checklist

[x] Has UI changes

Device Information
This PR was tested on: [
iPhone SE iOS 12.2,
iPhone 5s iOS 12.2 Simulator,
Google Nexus 5X Genymotion Simulator API Version 24
]
Screenshots

iOS
Android

Thanks @cardoso I must say that I just added a comment to the ticket as I'm not sure we should do this and there has to be some discussion around it before having this one merged. I must say that there is a chance that we'll close this ticket.

@enahum I made a suggestion on Jira

@cardoso can you show how this will work on a long post?

Not sure what you mean by long post, but:

@enahum I see there's some discussion on Jira and this issue is still tagged frozen. Please let me know here when I can continue with the implementation and what the decision is.

@cardoso a long post has a Show more control and the emojis sit below.

Actually @andrewbrown00 and @cardoso I was talking when you actually tap on show more and the post opens in a modal with the emojis below

Thanks for the clarification @enahum
@cardoso here's a screen shot for context:

@cardoso we've discussed internally and agree with the wrapping solution, assuming we cap the number or emojis that can be added via the UI.
Would be interested in finishing this change with the direction outlined here? https://mattermost.atlassian.net/browse/MM-17838

@esethna yes. I will as soon as I have the time.

@cardoso how's this going? Do you need any help or guidance?

@esethna didn't have time to touch it yet. Had to travel last weekend. But I should be able to finish it this weekend.

Logic seems to be working, I'll have to check why tests are failing later.

@cardoso tests are failing
app/screens/post_options/index.test.js
  ● mapStateToProps › canFlag is false for system messages

    TypeError: Cannot read property 'posts' of undefined

       97 |
       98 |     const getReactionsForPostSelector = makeGetReactionsForPost();
    >  99 |     const reactions = getReactionsForPostSelector(state, post.id);
          |                       ^
      100 |
      101 |     if (Object.values(reactions).length >= 40) {
      102 |         canAddReaction = false;

      at getReactionsForPosts (node_modules/mattermost-redux/selectors/entities/posts.js:87:25)
      at node_modules/reselect/lib/index.js:86:37
      at node_modules/reselect/lib/index.js:36:25
      at getReactionsForPostSelector (app/screens/post_options/index.js:99:23)
      at Object.<anonymous> (app/screens/post_options/index.test.js:39:23)

  ● mapStateToProps › canFlag is true for non-system messages

    TypeError: Cannot read property 'posts' of undefined

       97 |
       98 |     const getReactionsForPostSelector = makeGetReactionsForPost();
    >  99 |     const reactions = getReactionsForPostSelector(state, post.id);
          |                       ^
      100 |
      101 |     if (Object.values(reactions).length >= 40) {
      102 |         canAddReaction = false;

      at getReactionsForPosts (node_modules/mattermost-redux/selectors/entities/posts.js:87:25)
      at node_modules/reselect/lib/index.js:86:37
      at node_modules/reselect/lib/index.js:36:25
      at getReactionsForPostSelector (app/screens/post_options/index.js:99:23)
      at Object.<anonymous> (app/screens/post_options/index.test.js:49:23)

@enahum tests fixed

I’m not sure when I’ll have free weekends again to finish this (certainly not the next), so I’m opening it up for takeover.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@enahum How should we continue with this? Would someone from Apps be open to finish this?

@hanzei @jasonblais is there a hurry to get this in? this is already on our radar so it will get picked up eventually by the team before we ship 1.26

@enahum no rush, I didn't realize it was already taken by the Apps team for v1.26 from reading the conversations here

@cardoso @hanzei @jasonblais PR updated and ready to be reviewed.
pending reviews from @lindalumitchell and @migbot

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@lindalumitchell @migbot pending your reviews

@cardoso
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#3144-upstream-release-1.26-1574125841
Branch 'automated-cherry-pick-of-#3144-upstream-release-1.26-1574125841' set up to track remote branch 'release-1.26' from 'upstream'.
+++ Downloading patch to /tmp/3144.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/3144.patch

Applying: MM-17838 Wrap reactions when they exceed screen size
Using index info to reconstruct a base tree...
M	app/components/reactions/reactions.js
Falling back to patching base and 3-way merge...
Auto-merging app/components/reactions/reactions.js
CONFLICT (content): Merge conflict in app/components/reactions/reactions.js
Patch failed at 0001 MM-17838 Wrap reactions when they exceed screen size
Use 'git am --show-current-patch' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

+++ Conflicts detected:

UU app/components/reactions/reactions.js

+++ Aborting in-progress git am.

+++ Returning you to the master branch and cleaning up.


[BUG] Date of Birth is not the same when rendered

Describe the bug

Product Flavor

[X] Managed (https://rxresu.me)
[ ] Self Hosted

To Reproduce

When I enter my birth date the preview is fine but when downloading the PDF the date is one day before.
I'm in France if it helps
Expected behavior

The date should be the same as when entered in the "Date of Birth" input
Screenshots

Desktop (please complete the following information):

OS:
Browser
Version

Additional context

Is this something that can be worked on?

@AreYouLoco It definitely is in my top priority to fix this, whenever I get the time to update the project. I have made some attempts at fixing this issue in the past, but looks like none of them panned out.
My only alternative solution is to store dates as strings on the server and then it would get printed as-is instead of having to deduce the timezone. It's a bit of a tricky step as I wouldn't want to break any existing resumes, so requires some more thought. But it definitely should be fixed soon. Thank you for bringing this up, and thank you for continuing to use Reactive Resume despite its shortcomings :)

I asked my brother to take a look as well. He is a JS (react) programmer.

Can encounter the same issue here


Unable to select which vid to play in screen saver options

Ls,
After an update to beta 4 of release 1.2 of the aerial screensaver I am unable to  select which vid to play in screen saver options. The List is empty (see screenshot).
Globally installed for all users.
Regards,
Steve

Duplicate of #26

Closing as duplicate of #26


Add support for getting file attributes

Adds get_attr and get_attr_bytes methods on Repository.
Closes #441

This looks good to me, thanks! I think there's some issues with CI though?

Thanks for taking a look!
I've updated the PR to fix the CI errors, please let me know if you see anything that could be done better.

Thanks!


1.20.8 sap.m.Select uses unknown property from sap.m.Popover

sap.m.Select uses in Version 1.30.8 a property of Popover that is not available yet.
This produces the following error:
Assertion failed: ManagedObject.apply: encountered unknown setting 'showArrow' for class 'sap.m.Popover' (value:'false')
Index: src/main/webapp/js/sap/m/Select.js
===================================================================
--- src/main/webapp/js/sap/m/Select.js	(revision 1161)
+++ src/main/webapp/js/sap/m/Select.js	(working copy)
@@ -521,7 +521,6 @@

 			var that = this,
 				oPicker = new Popover({
-				showArrow: false,
 				showHeader: false,
 				placement: sap.m.PlacementType.Vertical,
 				offsetX: 0,

Thanks for your ticket. This issue has been wrongly downported to 1.30. Just to mention that this does not break functionality, it is an assertion.

@arleytriana thanks for adressing this issue so fast. Even if it is just an assertion it shouldn't be that way.