Deduping translations between pages

Is your feature request related to a problem? Please describe.
As I understand, this new version (Nextjs10) is creating a new i18next client on the server for every page request, as seen by this "stack trace"
https://github.com/isaachinman/next-i18next/blob/next-v10/examples/simple/pages/index.js#L44
https://github.com/isaachinman/next-i18next/blob/next-v10/src/serverSideTranslations.ts#L25
https://github.com/isaachinman/next-i18next/blob/next-v10/src/createClient/node.ts#L7
https://github.com/i18next/i18next/blob/master/src/i18next.js#L499
This should mean that if I go to page A that requires namespace A and then move to page AB that requires namespaces A and B, I will be loading (from the filesystem in this case) two times, right?
Describe the solution you'd like
Shouldn't we be doing something like storing the instance between requests (if possible of course, I think serverless would make it impossible) and just adding new translations/namespaces to the already created instance?

Translations in next-i18next@beta are passed into pages merely as normal props. The issue you are describing is interesting, and certainly valid, but is more of a general request about deduping props in general. Perhaps you can open an issue or discussion on the NextJs repo?

Translations in next-i18next@beta are passed into pages merely as normal props. The issue you are describing is interesting, and certainly valid, but is more of a general request about deduping props in general. Perhaps you can open an issue or discussion on the NextJs repo?

I understand what you said and while it is true, my issue is that AFAIK we fetch all namespaces each page, regardless of if we've fetched them before or not. The deduping of props in general is in fact another issue and can be handled in the Next repo yeah!

I understand what you said and while it is true, my issue is that AFAIK we fetch all namespaces each page, regardless of if we've fetched them before or not. The deduping of props in general is in fact another issue and can be handled in the Next repo yeah!

My point is that this is not specific to next-i18next at all. Presumably lots of SSG NextJs apps have pages which share large amounts of "static" props which could be deduped. I do not think there would be a good way to solve this problem outside of changes to the actual NextJs core.

My point is that this is not specific to next-i18next at all. Presumably lots of SSG NextJs apps have pages which share large amounts of "static" props which could be deduped. I do not think there would be a good way to solve this problem outside of changes to the actual NextJs core.

I see what you mean. But what happens "beyond" gSP is no longer Next's responsibility, and right now this "double fetching" could be avoided if we shared the same translations on the server, because right now neither the app or Next is able to hook into next-i18next to avoid it, because neither know what's going on inside, right? Now, I investigated a bit further and this actually seems like a problem that i18next themselves could solve/have solved, because the Resource Storage layer might cache resources, even between I18next instances.

I see what you mean. But what happens "beyond" gSP is no longer Next's responsibility, and right now this "double fetching" could be avoided if we shared the same translations on the server, because right now neither the app or Next is able to hook into next-i18next to avoid it, because neither know what's going on inside, right? Now, I investigated a bit further and this actually seems like a problem that i18next themselves could solve/have solved, because the Resource Storage layer might cache resources, even between I18next instances.

I think indeed this is i18next's responsibility. Thank you for your time!

I think indeed this is i18next's responsibility. Thank you for your time!

My point is that this is not specific to next-i18next at all. Presumably lots of SSG NextJs apps have pages which share large amounts of "static" props which could be deduped. I do not think there would be a good way to solve this problem outside of changes to the actual NextJs core.

I want to share some common locale in each page. But it seems I need to add  getStaticProps in each page. And these common locale will be loaded repeatedly when router change.  Similar issue with @ #1215 .
I think the next-i18next is used to add translations owned by a specific page conveniently. I haven't found out how to import the public translations separately.
If appWithTranslation support baseResoure, to provide the public resource to i18n context when app init. The public translations can be import and I don’t need to use getStaticProps in unnecessary page.  Will there be any problem？ Hope to get your answer.


DataGrid: Allow user to specify minimum batch actions to display before overflow dropdown

I will make a fix for this

Making note: We are looking at next 2 weeks to wrap up the project we are working on, so the sooner the better on this item would be helpful for us, thanks! I'll be active on any PR comments I get. Thank you!
fyi @matthewgallo

PR got merged in, thanks!


Java generator: Added "java.lang." to all generated instances of type…

… java.lang.Object to enable the usage of "Object" as struct name in thrift.

Hi @ne0h,
could you just create a JIRA ticket for this?
@Java devs: Comments anyone?

Hi Jens-G,
of course: https://issues.apache.org/jira/browse/THRIFT-3954

@Jens-G
IMO this is a valid use case but it is not necessarily a great idea because in general its not very user friendly to design an API that clashes names in java.lang.* ... it is even more complicated in Thrift because other languages may also have constructs called "Object" (javascript for example).  There is not a Java-specific technical reason why this shouldn't be allowed however.
That said, there is a related JIRA, not exactly the same problem because java.lang classes do not require import statements, but the idea is the same:
https://issues.apache.org/jira/browse/THRIFT-3301

@bgould Generated code should always be explicitly fully pathed or namespaced and never make any assumptions.  Thrift already uses namespaces so I would have no problem with org.king.Object as a thrift structure so long as it derives from java.lang.Object.  If the code generator uses "Object" without qualifications that could be ambiguous, and that is a valid defect .. I like the idea of eliminating that.  With regards to using a structure named Object, well that's really up to the folks using thrift, it isn't up to the thrift project to tell them how to name their structures and we should stay out of their way where we can.

@jeking3
I agree completely.  Didn't mean to give the impression that it shouldn't be supported.


Form freezes or unselects the first textfield

Possible bug
When ever I try to edit a first text field I created in redux form, frontend just freezes, or I have to click again to continue typing. Also if there are some inital values set, they reset after that happens.
I'm using react 16.2 with redux form 6.6.3 and redux-form-material-ui 4.3.3.
The form is created with https://redux-form.com/7.3.0/examples/material-ui/ as a guideline.
Thank you in advance :)

Please provide a sandbox that reproduces your issue.

Fixed it, the prop I was sending to the component that had nothing to do with that particular field was  changing for some reason.


No styles in dist

There are not any style files in dist. Version 1.0.37

Sorry for that one. Should be fixed in 1.0.38.


Active tab in dark mode hard to find

Is your feature request related to a problem? Please describe.
When in a dark theme, the active tab is difficult to identify:

e.g. Query 8 is the active tab but the slightly darker black on the tab is difficult to distinguish from the lighter black background on the inactive tabs.
Describe the solution or feature you'd like
More obvious color change.  Perhaps the icon color or the white text changed to yellow or some other contrasting color.
Describe alternatives you've considered
None.
Additional context
Add any other context or screenshots about the feature request here.

Closing since the Dark Modern theme can be used to have a bright blue line on the top border of the active tab

I know this is closed, but I wanted to add to it because I don't think the solution that closed this is all that great.
When using the options to have tab color set to fill or border, the active tab is still pretty hard to pick out.  I've tried the other dark and light themes suggested and even looked for the blue line in the Dark Modern theme.  None of that is very good or obvious.
It would be better if when using tab colors based on server groups, the inactive tabs only have the color as a border where the active tab is filled in the color.
Or at least if the inactive tabs were shaded more and always had the tab text in a different color than the active tab that would be something.


Request to add food icons, such as pizza or burgers

Hello! Could icons like pizza or burgers be added to MingCute?

We will update in the next version

'pizza' and ‘hamburger' was added in new version 2.92


Default guest root crypto mode to XTS for GCE

Default the guest root volume crypto mode for GCE to XTS. This keeps
it consistent with the AWS encryption process.

LGTM


env()  requires index if no $default value present

Explicit is better than implicit.
I think env() should not confuse index not found (this is an error) with null as default value (this is a specified behaivor).
This modified version throws RuntimeException when env var not found, and $default not specified by user.
(i personally use this modified version as env_require() in my project :grinning: )

This makes no sense.

@GrahamCampbell I'd like some explanation in detail. Is the reason of this PR makes no sense one because one of these?

Breaking old implementation?
Treat no default or null as default equally is enough?
Any manner or routine mistake in this PR?

Thank you.

@shamiao What's the purpose of the default argument ($default = null) then, now that we throw an exception if the variable can't be found?


NTUEEPLUS-155. Add history title

What is this PR for?

Add title in out history
What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]
Feature
Todos

[ ] - no

What is the Jira issue?

https://ntueeplus.atlassian.net/browse/NTUEEPLUS-155
Screenshots (if appropriate)

@featherchen finish!


Inlet function cleanup

Summary
Replaces the use of std::tuple in inlet::FunctionWrapper with a type-erased block of memory to store arbitrary std::functions. This lets us remove a significant amount of the template metaprogramming mechanics in inlet/Function.hpp, and shaves about a second off of the cost from including inlet.hpp.

@samuelpmishLLNL

Sorry to ask a basic question but: what is this code (and the code it is replacing) designed to do?

I tested this against Serac and it passes all our tests.  Did a small timing comparison in Serac between current develop of axom and this branch:
~~Develop~~
~Try 1
582.33user
85.96system
1:39.31elapsed

~Try 2
568.07user
76.68system
1:39.77elapsed

~~This Branch~~
~Try 1
536.66user
79.44system
1:25.73elapsed

~Try 2
526.78user
71.87system
1:25.60elapsed

Sorry to ask a basic question but: what is this code (and the code it is replacing) designed to do?
All improvements to compile time are greatly appreciated!

This bit of code allows users of Inlet to call Lua functions inside their input files with a limited amount of types for parameters and return types. We currently allow up to 2 parameters and the three following types: double, vector, and string and of course a void return type and no parameters as well.
The reason this is such a compile intensive operation is because of the combination of return types and parameters.
@kanye-quest  was able to find a drop in replacement for the previous solution that is much faster in regards to compile time.

Thanks @kanye-quest !
@white238 -- Is there sufficient test coverage for all the required/desired cases?

Yes there was a pretty extensive set of tests in Axom and some extra's in Serac.


gstreamer-src.Stop() cancels sends

The PR tackled the problem of "what happens to gstreamer-src.Pipeline when pipeline.in isn't being consumed?"
This problem would manifest itself with pipeline.Stop() hanging because the appsink element was hanging waiting for pipeline.in <- media.RTCSample{} to run.

Pull Request Test Coverage Report for Build 1959

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 79.222%

Files with Coverage Reduction
New Missed Lines
%

rtcdtlstransport.go
1
78.03%

Totals

Change from base Build 1958:
-0.02%

Covered Lines:
3889

Relevant Lines:
4909

💛  - Coveralls


netty 4.1.95

Motivation:
bump Netty version to latest (4.1.95)
https://netty.io/news/2023/07/20/4-1-95-Final.html
Modification:
changed Netty version number
Result:
Test suite passes.

Please rebase with main so CI can pass.


Gradle build error

I'm seeing the following error while trying to build the current main branch:
FAILURE: Build failed with an exception.

* Where:
Build file '***/paccor/build.gradle' line: 118

* What went wrong:
A problem occurred evaluating root project 'paccor'.
> Could not get unknown property 'all' for object of type com.netflix.gradle.plugins.packaging.ProjectPackagingExtension.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 0s

Modifying this line:
https://github.com/nsacyber/paccor/blob/e05b67688177c809bb3d598b57a05999f9152fbc/build.gradle#L118
to
 arch='all'

appears to fix the issue. I don't really know JAVA so it's entirely possible this is not a proper fix.

I believe this is fixed with !109.


redirecting pages while keep python running

Similar to #426 , I am trying to redirect pages while keeping python running
So i have just ran a loading screen on python and i am trying to switch from index.html to home.html once python is done loading.
I have tried the solution on #426 too
import eel
import time
eel.init('web')

@eel.expose
def load_modules():
    import pandas as pd
    from sklearn.linear_model import LinearRegression
    from sklearn.model_selection import train_test_split
    import yfinance as yf
    from datetime import datetime, timedelta
    print('load done')
    done = "1"
    eel.go_to('home.html')
    return done

eel.start('index.html')

<!DOCTYPE html>

    <meta charset="UTF-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    <title>OMEGA</title>

    <link rel="stylesheet" type="text/css" href="style.css">

OMEGA

Loading...

eel.expose(go_to)
function go_to(url) {window.location.replace(url);};
function callback(done){
    if (done === done) {
        //window.location.href = "home.html";
        //go_to('home.html')
    }
}

function load_modules(){
    eel.load_modules()(callback)
};

so once its done loading, it should be redirected to another page called
home.html

Problem is:
Once everything is done and index.html is redirected to home.html, my python code stops. I think I know why..? (probably because index.html is closed to the program stops) is there a way to keep the python code running despite changing to another page?
Many thanks

Apparently, all your html files need to have this link to eel modules, using this should keep app running after changing the url.
Like this:


feat: Add support for update_policy

Description
Add update_policy to compute_environment
Motivation and Context
Feature is missing

what version was this added within

Added on top of 4f83e6544564429e116ed8417880ec56e19e44ee, right after 2.1.0

no no - what version of the AWS provider was this argument added

looks like this one 5.32.0?

then this is a breaking change https://github.com/terraform-aws-modules/terraform-aws-batch/blob/c478369fdc5a73bbe334b6159fbce7e9a0937198/versions.tf#L7

Ah! Fair point. What's the SOP in this case?

Hello, random question, any update on this?

we'll include it in #38


What is the pricing for E-Invoicing in Saudi Arabia for Hyper market business?

We are in the implementation stage right now, still we don't have any clarity from either partner side or Microsoft side for pricing of E-Invoicing in Saudia arabia, especially in hyper market business.
we are looking for more than 6 months. still we did not get the clarity so far.
Here is what we noted in "Dynamics 365 Licensing Guide"
"Electronic invoicing is the process of creating, presenting, and exchanging structured,
transactional invoice documents between businesses and governments for tax
reporting purposes, or trading partners in an integrated electronic format
Dynamics 365 Commerce, Finance, Project Operations, and Supply Chain Management applications include 100
electronic invoice transactions/tenant/month. This included capacity does not rollover and is 100 transactions
per tenant, regardless of the number of Dynamics 365 licensed applications.
If you need additional transactions, you can buy the Electronic Invoicing additional capacity license for 1K
electronic invoice transactions/tenant/month. The transaction capacity is use-it-or-lose-it on a monthly basis,
and customers will need to purchase for peak capacity."
We are running with 7 stores, each store will have 1500 - 2000 transaction in one day. [7 store * 2000 = 14,000 per day]
Now, How this pricing_(as mentioned in Dynamics 365 Licensing Guide.pdf)_ will be applicable for Hypermarket business?.

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

ID: 84f21366-6fa4-9eb7-3b88-09ae6e9d815d
Version Independent ID: 84f21366-6fa4-9eb7-3b88-09ae6e9d815d
Content: Get started with Electronic invoicing for Saudi Arabia - Phase two - Finance | Dynamics 365
Content Source: articles/finance/localizations/e-invoicing-sa-get-started.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @mrolecki
Microsoft Alias: mrolecki

Hi @liamneesangamer ,
the information you posted from the "Dynamics 365 Licensing Guide" is correct and valid. For details on how this is applied to "hypermarket" business, I suggest you contact with the Microsoft licensing specialist you work with.
Thanks,
Tomás Navarro.


Update files.rst

Fixes a couple typos, the most important of which is the description of the template loaders that mentioned django instead of django_tenants.

@johnfraney thanks


pytest --json-fixtures doesn't work without another argument

The README for your project shows an example command of pytest --json-fixtures but this doesn't work. If I do that, it complains that it's expecting another argument. Doing pytest --json-fixtures foo.json does work. So I think your handling of command line arguments needs some work.

Looking into this more, I can't get it to work right with pytest --json-fixtures foo.json either. So something else is wrong. I will close this issue until I can figure out what my real problem is.

I'm reopening the issue as it's not working for me also without the file name argument. However, with the file name argument it correctly creates the file. Which version of Python and pytest are you using?
If you cannot wait for a fix, you can try to implement the code yourself, as I explained on this answer for a Stackoverflow question. Have a look at the main file here.

Yeah, that Stack Exchange answer is what led me to your project in the first place.
Regarding this bug, I did get this plugin to generate fixtures.json files, both with the default name and with a custom name provided on the command line.
I was using Python 3.11 and pytest-json-fixtures 0.1.2.
However, I found a couple other bugs, which I'll write up separately. And then I got so frustrated I just decided to use --fixtures instead.

Ok. Thank you for the reports. I'll check them when possible.
You're welcome to contribute to the project if you want.

Closed with https://github.com/marcosanchotene/pytest-json-fixtures/pull/5


Slowness when pairing smaller width with long strings

I was hoping you would be able to help us with a performance issue when we use zprint to format our Clojure code. In some files we are experiencing slowness (takes a few seconds to format the file) and I think it's connected to the string lengths because they can't get broken up. It looks hard for zprint to rule out branches and it seems to try a lot of options until it finds a solution.
Here is a prototypical example of how the code looks in those files (I just ran an obfuscation on a real function):
(defn lysnxrdilceshiq
  [{:keys [bpwwdtdrbcbeqhtzpm lbbfwa scrx as]}
   {:keys [suufsw raerqzadu] :as vjzoqll}]
  (let [zcxjsmvvgtjdjhgb (z (:rh gjnz) (:rehsrqsjammnytt fakm))
        pbaaxrgfwlf      (:rgjnpzgkigr opkf)]
    {:yniykpynbytpcl
     (isvmgl
      aezz
      [{:wmujdsnfal "ludrasz"
        :oqxmk (xi "dkualogsvwfdlodqbhpkymsosayibtdgggagmbwukjj")
        :uwdwj
        (zsnkrm
         xxjk
         [(when (vues (:yw xulg))
            (whvkjcahz
             "ctrhhbi"
             (rp "gdndmfcqbgeclpmalnjkgzdaceuoufjemuszxnqhqjusxygeblrkimafbo"
                 {:bs techhugyrzuyjqfh})
             (grtpccxmsjnm (:szpdnoueqmzoa twdk) vflrimegn hoeowpljulp)
             {:clbvjylm
              (qt
               "hwdxyisaezgwplnfljfnvyildhsjbggkbplkwjpszemjvyhawdmrfumitlowu"
               {:edofyanir (:qo udbs)
                :qvwepdygi (r (:jsyqczfxlxrzfcg ciba) (:zy ncbz))})}))])}])}))

and the .zprint.edn
{:style       [:community :justified :map-nl :pair-nl :binding-nl :respect-bl]
 :width       80
 :fn-force-nl #{:arg1-body :binding :fn}
 :map         {:comma?    false
               :sort?     true
               :force-nl? false
               :justify   {:max-variance 40}}
 :pair        {:force-nl? true :justify {:max-variance 40}}
 :binding     {:justify {:max-variance 40} :justify? true}
 :vector      {:wrap? false}
 :fn-map      {;; clojure.core and special forms
               "def"           :arg2
               "do"            :flow-body
               "try"           :flow-body
               "finally"       :flow-body}}

It takes around 1s to format the above. It's a lot faster when I change the :width to 120.
Thanks!

Thank you for taking the time to look into this and explain it so thoroughly! I can verify that setting max-variance to 0 improves performance for the whole file dramatically. We'll play around with this next week and I'm sure we'll find a solution that works for us.

I'll close as we managed to improve the config for our use case (with a width set to 100 now and a few exceptions). I appreciate this tool a lot, thank you for building it!

I have released 1.2.6.  I found several places where the justification code was missing optimizations that were available, and fixed those.  Where above I saw a 7 times slowdown when doing justification using :max-variance 20, now I see a slowdown of <2.  Which is a lot better, but still slower.
I also added a new configuration option: :max-depth 100 to all of the :justify key values.  Part of what was an issue in your example was that there were things to justify that were pretty deeply nested.  If you run into this again, you could try setting :max-depth 4 or maybe 5 or something and see if it helps. I think it will.  It will only justify down to :max-depth and then it won't even try.  But I hope you don't need that.
Thanks for the appreciation.  It means a lot!


docker not running

Hi All
I can make mount and get inside the vm. But i can't see anything running by "docker ps", why?
~/workspace/OpenLane>make mount
cd /Users/peter/workspace/OpenLane && \
		docker run --rm -v /Users/peter/workspace/OpenLane:/openlane -v /Users/peter/workspace/OpenLane/designs:/openlane/install -v /Users/peter/workspace/OpenLane/pdks:/Users/peter/workspace/OpenLane/pdks -e PDK_ROOT=/Users/peter/workspace/OpenLane/pdks --user 501:20 -ti efabless/openlane:2021.12.21_01.44.20
bash-4.2$ exit
exit
~/workspace/OpenLane>docker ps
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES

thanks

When you exit the container it is no longer running so there is nothing to see in ps.

thank you hero @maliberty


Add command for removing tag pool permissions.

Currently we don't have a command to do this.

Ready for review.
I ended up having to rework some bits of the command more than I might have liked, but the original structure was making things difficult.

Just the one comment, otherwise looks good

:+1:

:+1: from me too


Add useTheme composable function

Add a useTheme composable function, usable with the Vue Composition API, that will allow to keep track of the currently used theme, set a theme, or manually get the current theme, for an element.
The theme.config.js would be given to the use function (or detected), or some options would be passable to it.
The current theme on an element would be detected according to the detected strategy, the default theme in the configuration, and the current prefers-color-scheme.
A theme would be changeable, the use function would detected the strategy and update it accordingly.

[ ] Get theme

[ ] Detect ThemeBuilder configuration
[ ] Handle options if no configuration passed
[ ] Detect current theme

[ ] With a default
[ ] With the media query
[ ] With the given strategy (or the default one)

[ ] Set theme

[ ] With the given strategy (or the default one)

See #22


Hide History Button when History is disabled

Not sure if history is disabled by default, but it should imo. And then I don't expect to see the history button when I test timelines.

oh hmm yeah the button's initial visibility state is supposed to be set based on whether the whole History subsystem is enabled or not


feature: config control  for colors in logs

When VERDACCIO_LOGGER_COLORS is provided - it's cast to boolean and cascades any config.
When logs.colors is provided - it's used.
otherwise - default to process.stdout.isTTY
I updated the docs I found, still don't know where to update the site and the reference for the logger config.

Uh... can we please have the rejections of the format check in the CI log?

Ha. Green.
LMK if/what tests you'd like for this 😉

I wish the prettier rejections were observed on the build output...... 😞

did not leave, just got some backpressure from the business backlog.
what's stopping me is that the build does not show the rejections of prettifier...

I just upgraded and lost console colors. It's a breaking change?
Also, please implement https://no-color.org/ spec.

Opsss, that was not intended I will fix that bug in next release

Are you sure does not have something else in your config file? It works for me.

5.6.2? colors was introduced on 5.7.0. It indeed remove colors by default (I'll fix it) but if you help me to reproduce locally that broken state you mean, I'm happy to fix it.
btw, v6 does not have this feature 🤷🏼♂ at all

I mean't after I upgraded to 5.7.0 and edited config.yaml to add colors: true in logs {} it broke verdaccio and now it won't start at all. I removed the colors from my config, still couldn't launch verdaccio, then I tried upgrading and downgrading and reinstalling verdaccio but it won't start anymore. Strange.

I think I tracked my problem down, it was very confusing (mostly because I'm not used to functionnal indentation (python, yaml) but a few spaces on the logs: line were responsible for the error. Now sure when they crept in, maybe when I tried to add colors: true. But anyway, it all works now as described.
Hoping a 5.7.1 will bring back color by default  but it's not a huge deal, just unexpected for a minor version change. Also, < https://no-color.org/> please :-)

On my notes thanks @osher nice catch


(WIP) Combine maps in WAL partition into one map

Experimental change to the WAL to reduce memory usage.

Bunch of failing tests on this. Might be better to just combine the cache into the DatabaseIndex object since it already has a map of the key to the Series object. We can put the cache logic on the series object itself.


Cannot parse named windows with modifiers

sqlparser-rs only supports named windows that are an identifier. This is fine for queries like:
SELECT sum(salary) OVER w, avg(salary) OVER w
  FROM empsalary
  WINDOW w AS (PARTITION BY depname ORDER BY salary DESC);

However, it is insufficient for queries in at least BigQuery and MySQL where modifiers can be added to the named window in the OVER clause.
MySQL named window example
SELECT
  DISTINCT year, country,
  FIRST_VALUE(year) OVER (w ORDER BY year ASC) AS first,
  FIRST_VALUE(year) OVER (w ORDER BY year DESC) AS last
FROM sales
WINDOW w AS (PARTITION BY country)

BigQuery named window example
SELECT item, purchases, category, LAST_VALUE(item)
  OVER (
    item_window
    ROWS BETWEEN 1 PRECEDING AND 1 FOLLOWING
  ) AS most_popular
FROM Produce
WINDOW item_window AS (
  PARTITION BY category
  ORDER BY purchases)

This would build upon prior to work:

https://github.com/sqlparser-rs/sqlparser-rs/issues/867
https://github.com/sqlparser-rs/sqlparser-rs/pull/881

Thanks for the report @jesse-shopify  -- any contributions to add such support are most welcome


captureBreadcrumb - can't see stored data in Sentry issues

OS:

[ ]  Windows
[X ]  MacOS
[ ]  Linux

Platform:

[X ]  iOS
[ ]  Android

Output of node -v && npm -v && npm ls --prod --depth=0
v10.11.0
6.4.1
├──<EMAIL_ADDRESS>
Config:
Sentry.config('myKey').install();

I have following issue:
It doesn't capture the breadcrumbs on iOS, didn't have an opportunity to test it on Android yet. Here's the code I use for capturing the breadcrumb:
Sentry.captureBreadcrumb({
    timestamp: +(new Date()),
    type: 'navigation',
    data: {
      from: 'Profile',
      to: 'Settings',
    },
  });

Does the log say anything?
Try removing the timestamp key since we add it for you, maybe it's formatted the wrong way.

I am closing all old issues, please if this is still a problem feel free to revive it.
Also, consider moving to our new SDK @sentry/react-native if it still happens there please open a new issue.


How do you display seconds in time?

please help me
How do you display seconds in time?
like the bash cmd
histtimeformat="%F %T"

%*
Current time of day in 24-hour format, with seconds.

@eremit
Uncomment the following line if you want to change the command execution time
stamp shown in the history command output.
The optional three formats: "mm/dd/yyyy"|"dd.mm.yyyy"|"yyyy-mm-dd"
HIST_STAMPS="mm/dd/yyyy"
how to set it? please help me , 3q~

@ancon okay, probably I was not really paying attention to your report..
see the following issue as reference:
https://github.com/robbyrussell/oh-my-zsh/issues/3466
In a nutsell:

add one line HIST_STAMPS="yyyy-mm-dd"
add one alias: alias history='fc -il 1'

@eremit thanks~
but it didn't work...
it can display the minute, but it display the seconds~

@ancon use this one:
alias history="history -t'%F %T'

Hiya @ancon
is your issue now resolved by the alias that I provided?

@eremit thanks a lot.
it work, but did not perfect.
I just think that display the second is unnecessary.

I just had no idea why I wasn't seeing timestamps in my history either after setting this.

@tjharman said:

I just had no idea why I wasn't seeing timestamps in my history either after setting this.

If you are interested in this feature you can support this pull request: PR https://github.com/robbyrussell/oh-my-zsh/pull/6770

I've merged #6770 to include this feature. Now setting HIST_STAMPS to a custom format will automatically update the history alias to show the custom timestamp, no need to re-alias history.

I find that it works if it's set to mm/dd/yyyy
But in New Zealand we use dd/mm/yyyy and if I set that, my output looks as follows:
   21  dd/mm/yyyy  vi .zsh
   22  dd/mm/yyyy  vim
   23  dd/mm/yyyy  vim .zshrc
   24  dd/mm/yyyy  history
   25  dd/mm/yyyy  vim .zshrc

That's not a supported format. It has to be either one of these: mm/dd/yyy, dd.mm.yyyy, yyyy-mm-dd. If your desired format is not one of these, you have to specify your custom format using strftime strings. Your specific case would be: %d/%m/%Y.


docs: include instructions in getting-started pages for quickstart

This adds synchronized tabs throughout the getting started pages where a user can choose between "Testnet" or "Standalone" information.
Closes #637

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Can we change the "Standalone" to "Local"? We deprecated the term standalone in quickstart, and quickstart itself uses the option --local now to run a local only network.

Yeah, for sure!

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

@leighmcculloch Really, really good suggestions! Thanks! I've committed them, and made a couple small wording changes here-and-there to reflect them (for instance, removing "Soroban network" in some other places). Ready for another review when you're ready.

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/

I just noticed that the getting started guide shows a dev how to deploy to a local network, but not how to run a local network. Or is that instruction somewehre I'm not seeing it?
I'm looking at this:

Ref: https://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/docs/getting-started/deploy-to-a-test-network

@leighmcculloch We do have that command on the "Setup" page: https://soroban-docs-pr659.previews.kube001.services.stellar-ops.com/docs/getting-started/setup#configuring-the-cli
That's a couple pages before you actually deploy, though. Should we include the command here again?

Ah, thanks for pointing out where it lives.
It's really easy as user to fly through the setup and skip over configure the CLI steps because they feel unimportant.
I think we could add a info/tip box under both the testnet and local tabs on the deploy page, that says something like "this method of deployment requires XYZ to have been done, see this other page for the commands to run to complete the setup: [other page](link directly to the commands in the setup showing the appropriate tab).

I'm gonna close this, and re-open a PR based on the current structure/content in the stellar/stellar-docs repository.


DHCP v6 - VLAN priority not honored

Important notices

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
I am running the OPNsense 23.1_6-amd64 (FreeBSD 13.1-RELEASE-p5, OpenSSL 1.1.1s 1 Nov 2022) release.
My ISP requires all WAN communications on VLAN 832 and the use of VLAN priority/DSCP CS6 for all control comms like DHCP and CS0 for the rest. I have configured the 'Use VLAN priority' field in the DHCP v6 config as 'Internetwork Control (6)'.
Capturing the network traffic on the interface shows that all DHCP v6 related comms from the firewall to the ISP router does not honor  the DSCP CS6 setting, whilst all related comms back from the ISP is tagged as CS6 (as expected).
To Reproduce
Steps to reproduce the behavior:

Create a VLAN tagged 832 on the WAN interface
Set 'VLAN priority' to 'Internetwork Control (6)' in 'DHCPv6 client configuration'
Start packet capture on interface
Renew the lease
Inspect the DHCP related packets and look for DSCP label

Expected behavior
DSCP property of DHVP v6 related packets to be labeled as CS6.
Describe alternatives you considered
I tried this on the most recent set of OPNSense releases, all with the same result.
Screenshots
interface configuration:

Wireshark packets overview:

Wireshark request packet details:

Wireshark reply package details:

Environment
OPNsense 23.1_6-amd64 (FreeBSD 13.1-RELEASE-p5, OpenSSL 1.1.1s 1 Nov 2022)
Intel(R) Celeron(R) N5105 @ 2.00GHz (4 cores, 4 threads)
Network Intel i225-V B3 2.5G

PS: Does IPv6 connectivity work in general? Tapping from WAN interface with bpf might show the state before the outgoing traffic is modified accordingly.

It's set to log so you you should at least see if it matches in firewall live log?
If it matches it would be better to dump traffic in front of the opnsense to see the actual traffic on the wire. If there is no match in rules something else seems to snatch this traffic.
Worst case igc driver could refuse to set the tag for one reason or another. Was this working in 22.7.x or new setup?
Orange and conformity is a joke with what hoops one has to deal with regarding their "standard" connectivity ;)

The packet capture in front of the OPNsense is crucial information now. It looks correct so far :)


refactor Error and increase test coverage

Useless make_unexpected_value_error function is removed.
Unit test is provided for make_missing_parameter function.

Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

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

Project coverage is 88.58%. Comparing base (4ca09c2) to head (4adc314).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #591      +/-   ##
==========================================
+ Coverage   88.54%   88.58%   +0.03%
==========================================
  Files         109      109
  Lines       34905    34924      +19
==========================================
+ Hits        30906    30936      +30
+ Misses       3999     3988      -11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


How to solve this problem?(download_dependencies.sh)(c++ tensorflow)

downloading https://github.com/google/googletest/archive/release-1.8.0.tar.gz
downloading https://mirror.bazel.build/github.com/google/nsync/archive/0559ce013feac8db639ee1bf776aca0325d28777.tar.gz
gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now

@heartraeh
It looks like you haven't used a template to create this issue. Please resubmit your issue using a template from here. We ask users to use the template because it reduces overall time to resolve a new issue by avoiding extra communication to get to the root of the issue.
As per the error please refer to #31799 #35747, we will need the tensor flow version and steps performed before you encountered the error to help you resolve the issue

@heartraeh
please update on the above comment

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


feat: add github action to add new issues to project board

Description of changes
Adds GitHub Action to add all new issues to the project board (beta)

Issue #, if available

n/a
Description of how you validated changes
Checklist

[x] PR description included
[ ] Generate Personal Access Token with org:read/write scope
[ ] add token to repository action secrets as TOKEN

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

@josefaidt are you still working on this?

Hey @danielleadams no, I've gone ahead and closed this 🙂


Backend/future#62 feedback

This PR represents the backend part of rating and feedback functionality.
A new method has been added to the user serializer - get_average_rating, that calculates the average rating for the user.
To view feedbacks and ratings for all users use the following endpoint:
/api/feedback/

the response:

[
  {
    "rated_user": 3,
    "order": 1,
    "given_from": 4,
    "user_rating": 5,
    "comment": "very good"
  },
  {
    "rated_user": 2,
    "order": 2,
    "given_from": 4,
    "user_rating": 5,
    "comment": "not so good"
  },
  {
    "rated_user": 2,
    "order": 3,
    "given_from": 4,
    "user_rating": 1,
    "comment": "bad"
  },
  {
    "rated_user": 2,
    "order": 4,
    "given_from": 4,
    "user_rating": 3,
    "comment": "good"
  },
  {
    "rated_user": 2,
    "order": 6,
    "given_from": 4,
    "user_rating": 1,
    "comment": "so bad"
  },
  {
    "rated_user": 2,
    "order": 7,
    "given_from": 3,
    "user_rating": 3,
    "comment": "so so"
  },
  {
    "rated_user": 4,
    "order": 8,
    "given_from": 2,
    "user_rating": 3,
    "comment": "not so bad"
  }
]

To view the average rating and all comments for specific user use the following endpoint:
/api/feedback/user/{id}/

the response:
[
  {
    "user": 3,
    "average_rating": 4,
    "all_comments_and_ratings": [
      {
        "related_order": 2,
        "given_from": 4,
        "rating": 5,
        "comment": "very good"
      },
      {
        "related_order": 11,
        "given_from": 2,
        "rating": 1,
        "comment": "string"
      },
      {
        "related_order": 12,
        "given_from": 2,
        "rating": 4,
        "comment": "string1"
      },
      {
        "related_order": 13,
        "given_from": 2,
        "rating": 5,
        "comment": "string11"
      },
      {
        "related_order": 17,
        "given_from": 4,
        "rating": 5,
        "comment": "very good seller"
      }
    ]
  }
]

Overall, great job. I just think that the given_to and given_from should be the same as the sender_user and receiver_user, derived from the order. I don't think there should be a choice here, because this opens for mistakes. @milensski what do you think?

Yes, overall looks good.
I was wondering the same for these variables but was not sure if its just me. So I agree with that.

I thought the same thing when I started, but I thought it might get confusing with the variables, so I'll change them. Regarding where the rating and comments are, what do you guys think, isn't it more correct to have it in the account app, since both are about users?

The variable names have been changed according to the comments.

No, guys, maybe I expressed myself wrong, but I meant something completely different.
What I wanted to say was that when a feedback object is created, we should be taking the receiver and sender users from the order itself programmatically. Because what can currently happen is that you can have a completed order, where buyer_1 bought from seller_1. But in the backend, it can happen that buyer_6 can give the feedback to seller_1, even if that buyer has never been part of that order. Currently in the API, I don't see anything stopping me from doing that (correct me if I am wrong).
There are 2 ways I know on approaching the problem:

Fix the issue on the data side - ensure that the feedback will always use the sender_user and receiver_user from the order.
Fix the issue on the controller side - ensure that through the views and serializers you match both users correctly and all use cases are covered. I would personally go for option 1, but in both cases, the controller side needs to be good.

What do you think? Do you get what I mean?

Sorry for misunderstanding. I thing now it is as expected.


Added the final report sections of analysis and results and discussion

Here is our merged Rmd markdown file. Let me know if you any suggestions or want me to modify anytthing!

This looks nice! Once it is merged I am going to add a couple things from the analysis section that might work better in the data section!

I also modified the readme file to align with the analysis we used in this milestone!

Looks good to me, other than the change I reported.


Upgrade to support expo 52

Expo 52 is released today. As I tried to update to it, the expo-datadog is causing issue:
npm error Conflicting peer dependency<EMAIL_ADDRESS>npm error node_modules/expo
npm error   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error   packages/mobile/node_modules/expo-datadog
npm error<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error     packages/mobile
npm error<EMAIL_ADDRESS>npm error       node_modules/@test/mobile
npm error         workspace packages/mobile from the root project"

As it cannot only work with expo 51

Any workarounds for this issue thus far?

I was able to work around this by manually patching the dependencies of expo-datadog with patch-package.

Open up node_modules/expo-datadog/package.json and change the peerDependency to be "expo": "52.x".
Run npx patch-package expo-datadog --exclude 'nothing'. The --exclude param is necessary since you're only patching the package.json file.
Anything else you need to do to enable patch-package
npm install
You may need to manually edit the package-lock.json file for your project to make sure it has the new peer dependency version number under node_modules/expo-datadog.

Hello 👋 we apologize for the long wait!
We are going to release an official new version that supports Expo 52 soon.
In the meantime, the workaround suggested by @blwinters should work.
I will keep you posted here for updates, and notify you as soon as the new release is out and ready! 🙏


Unexpected token ...

This first error is around ...req. At first it looks like a typo, but it's liste as such in the tutorial, README.md and the tar.gz file. But if it was a type, I deleted the ... and retried. But then it complained about the function keyword.

PS D:\PoCs\hackernews-react-apollo\server> yarn dev
yarn run v1.5.1
warning package.json: No license field
$ npm-run-all --parallel start playground
warning package.json: No license field
$ node src/index.js
warning package.json: No license field
$ graphql playground
D:\PoCs\hackernews-react-apollo\server\src\index.js:19
...req,
^^^
SyntaxError: Unexpected token ...
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.runMain (module.js:604:10)
at run (bootstrap_node.js:389:7)
at startup (bootstrap_node.js:149:9)
error An unexpected error occurred: "Command failed.
Exit code: 1
Command: C:\WINDOWS\system32\cmd.exe
Arguments: /d /s /c node src/index.js
Directory: D:\PoCs\hackernews-react-apollo\server
Output:
".
info If you think this is a bug, please open a bug report with the information provided in "D:\PoCs\hackernews-react-apollo\server\yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
ERROR: "start" exited with 1.
error An unexpected error occurred: "Command failed.
Exit code: 1
Command: C:\WINDOWS\system32\cmd.exe
Arguments: /d /s /c npm-run-all --parallel start playground
Directory: D:\PoCs\hackernews-react-apollo\server
Output:
".
info If you think this is a bug, please open a bug report with the information provided in "D:\PoCs\hackernews-react-apollo\server\yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

PS D:\PoCs\hackernews-react-apollo\server> yarn dev
yarn run v1.5.1
warning package.json: No license field
$ npm-run-all --parallel start playground
warning package.json: No license field
$ node src/index.js
warning package.json: No license field
$ graphql playground
D:\PoCs\hackernews-react-apollo\server\src\resolvers\Query.js:1
(function (exports, require, module, __filename, __dirname) { async function feed(parent, args, ctx, info) {
^^^^^^^^
SyntaxError: Unexpected token function
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (D:\PoCs\hackernews-react-apollo\server\src\index.js:3:15)
error An unexpected error occurred: "Command failed.
Exit code: 1
Command: C:\WINDOWS\system32\cmd.exe
Arguments: /d /s /c node src/index.js
Directory: D:\PoCs\hackernews-react-apollo\server
Output:
".
info If you think this is a bug, please open a bug report with the information provided in "D:\PoCs\hackernews-react-apollo\server\yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
ERROR: "start" exited with 1.
error An unexpected error occurred: "Command failed.
Exit code: 1
Command: C:\WINDOWS\system32\cmd.exe
Arguments: /d /s /c npm-run-all --parallel start playground
Directory: D:\PoCs\hackernews-react-apollo\server
Output:
".
info If you think this is a bug, please open a bug report with the information provided in "D:\PoCs\hackernews-react-apollo\server\yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Any ideas on how to get this working? As the recommended tutorial, I'm not sure where to turn next.
yarn-error.log

Hello - I've run into this error and it happened to be because I was running an older version of Node. I can confirm that it is working on Node version 9.11.1.

The documentation should be updated to require a minimum version, which apparently is NodeJS 6.

Run into the same issue and upgraded to latest Node version solved it. I think the issue is node 6 doesn't support spread operator for objects (only for arrays and iterables), and node 8.3 started to support it: https://stackoverflow.com/questions/40066731/node-v6-failing-on-object-spread?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa

Thanks @MisinformedDNA for opening this issue 😄. It should be working now. Closing this, please, feel free to re-open this issue if you cannot make it work!


MORPH2 dataset

hello，
How to get the MORPH dataset ? If I want to use it as academic purpose ,can I apply the dataset by sending an official Email or I HAVE TO pay for it ?
thank you .

I do not own the dataset. Their rule is payment. There is nothing I can do.


Update hide_phone_number behavior

I updated the API documentation to reflect the new behavior of hide_phone_number. Can one of you take a look and make sure it's accurate? If so, please approve and merge. We can update the master branch when we release 2.70.5.

@chrislrobert what do you think of these changes?


pangram: Update pangram test suite

Updates the pangram test suite to match the new json file in x-common.
Resolves #337

This looks good to me. @Dog - leaving it to you to review and merge.

Thanks


IP address for CNAME, if ALIAS not available

Hi,
I've watched the video and followed the instructions and managed to do all, up until forwarding the root domain (3:28 in the video).
The thing is that some major domain providers (GoDaddy, Google) don't provide ALIAS records, and in addition don't allow CNAME record forward with @, and that port to a website, only to an IP address.
Please elaborate further on how to deal with these situations, perhaps by deomnstrating how to obtain the IP address.
Thanks

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

ID: 190b1ed3-0712-2f19-4e8f-a2a8519d136e
Version Independent ID: b02adbf4-d2a8-6c84-e70a-ff59fb3cbb7b
Content: Set up a custom domain in Azure Static Web Apps
Content Source: articles/static-web-apps/custom-domain.md
Service: static-web-apps
GitHub Login: @burkeholland
Microsoft Alias: buhollan

Hi @weitzhandler, thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

I now managed to achieve this behavior by utilizing the provider's Forward option.
Feel free to close the issue if you see fit. Might still be very handy if you could update the content with this info bit, as I've struggled with it several times myself in the past as well, and I reckon I'm not alone.

Please update documentation Warning/Note:    Static Web Apps do not work with GoDaddy Domains

Was able to add a subdomain just fine.
Cannot add a root domain  DNS Records

Documentation Options Tried
Option 1: A Record

Documentation: asks to create an Alias/A Record with a website address...
Problem: Go Daddy forces to use IP Address, cannot add the Azure Static Web App URL.
Note: Tried adding A record with the ip address of the Azure Static Web APP generated URL, without success. Got a warning page warning connection is not private, hackers may be trying to steal info.

Option 2: CName Record

Documentation: asks to create a C Record
**Problem: **  Go Daddy does not allow the '@' in the field needed when creating a CName Record

Option 3:  Fwd with masking

Weitzhandler Suggestion above  utilize GoDaddy's FWD Option to just FWD to the URL
Problem: I get an error msg stating the connection is unsecure and hackers may be trying to steal info in my browser window when hit the URL

Open to other suggestions :)

@craigshoemaker I'm thinking we add a disclaimer to the docs that GoDaddy doesn't support CNAME A records or CNAME flattening. The question is whether we expand that to include instructions on how to do forwarding. That's quite specific to GoDaddy, but the provider is large enough that it makes sense.
What do you think?

Ya knowing dealing with GoDaddy may have issues when it come to the domain.  I do have the subdomain www working so at least https://www.mydomainName.com is working.   but it still is kind of an expected and raises what i call 'brain hiccups' if the root URL does not work, or worse yet... one goes to the root URL    htps://mydomainName.com   by GoDaddy Forwarding,  and the user is greeted with a message "Warning hackers may be staling your info"..   I'd just rather let it not work at all and force the "www." subdomain instead of getting that hacker message.
Not sure if you are also getting that message when trying to FWD + typing in the RootURL  (https://MydomainName.com) - but i was.  If you are not then maybe those directions would be helpful so i can compare what/if i may be doing anything different.

I am going to #please-close this issue as I have added the disclaimer to the article for now and am working on refreshing the article with more detail about how to approach apex domains with registrars like GoDaddy, Google, etc.

Thanks @craigshoemaker


address markdownlint errors

Addresses #3 before moving this repo to mkdocs on ghpages. Now Travis CI finally passes.

@jzaccone @remkohdev this PR is ready for review.

@jzaccone @remkohdev this PR is ready for review.


fix bugs in restful api

Confusion_Matrix has been a list, so delete the list convertation. Otherwise, detection task would raise a error: "no key of Confusion_Matrix"
Add pppicodet train

LGTM


A lot of failure cases when try to run "python scripts/sim.py --vis"

I tried to visualize the grasping results via python scripts/sim.py --vis. However, I found there are lots of failure cases. Could you give me some instructions about how to visual the successful grasping case?? @jianglongye

Our generative model primarily focuses on searching for successful trajectories, so the sampled results before filtering might include some trajectories that look visually plausible but aren't physically accurate.
Regarding SAPIEN visualization, we didn't test this in the initial code release. Currently, we use viser to visualize trajectories only (not SAPIEN simulation), as seen in scripts/visualize_sampling.py. We plan to add SAPIEN visualization later, but we can't guarantee the exact date—possibly after Sept.


add simple check if pools were added

westend 18408

test.log

all seems working
test_all.log

test_all1.log


Add Postgres extensions (if possible)

I have a Postgres extension installed in my official database, pg_trgm, which wasn't found by the CLI's migration generator.  This may not be possible, or it may be a Hasura thing, so I'd be glad to post a request there.
Anyway, adding
CREATE EXTENSION IF NOT EXISTS pg_trgm;

To my migration's up.sql file cleared up the problem, and the project was able to start.
The code that uses the extension, which (as we found) kills the seeding of the auth.roles, etc., is:
CREATE FUNCTION public.search_recipe(search text) RETURNS SETOF public.recipe
    LANGUAGE sql STABLE
    AS $$
SELECT   *
FROM     recipe
WHERE    search <% ( name || ' ' || tags || ' ' || coalesce(notes, '') || ' ' || coalesce(cookbook, '') )
ORDER BY similarity(search, ( name || ' ' || tags || ' ' || coalesce(notes, '') || ' ' || coalesce(cookbook, '') )) DESC;
$$;

The <% is non-standard syntax, and fails.
This may help writing a test.

Relevant:
SELECT * FROM pg_extension


chore(deps): update dependency tslint-plugin-prettier to v2.1.0

This PR contains the following updates:

Package
Type
Update
Change

tslint-plugin-prettier
devDependencies
minor
2.0.1 -> 2.1.0

Release Notes

prettier/tslint-plugin-prettier
v2.1.0
Compare Source
Features

respect .prettierignore (#317) (0420071)

2.0.1 (2018-10-28)
Bug Fixes

report syntax error instead of throw (#272) (8b62595)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Enabled.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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


aws-lambda: from_security_group_id returns object of invalid type

Describe the bug
The returned object from SecurityGroup.from_security_group_id when used in DockerImageFunction fails to pass the typeguard check in the check_type function.
This error wasn't present when using typeguard version 4.0.0. It used to throw warnings but not errors. The warning is pasted below.
<redacted-path>/lib/python3.12/site-packages/aws_cdk/aws_lambda/__init__.py:1526: UserWarning: Typeguard cannot check the ISecurityGroup protocol because it is a non-runtime protocol. If you would like to type check this protocol, please use @typing.runtime_checkable
  typeguard.check_type(value=value, expected_type=expected_type, collection_check_strategy=typeguard.CollectionCheckStrategy.ALL_ITEMS) # type:ignore

The error appears with typeguard version 4.3.0. The error message is pasted below.
Sequence[aws_cdk.aws_ec2.ISecurityGroup]: item 0 is not compatible with the ISecurityGroup protocol because it has no method named '__jsii_proxy_class__'
NoneType: is not an instance of NoneType

Regression Issue

[ ] Select this option if this issue appears to be a regression.

Last Known Working CDK Version
No response
Expected Behavior
The type check should succeed.
Current Behavior
The type check fails and so does cdk deploy or cdk synth.
Reproduction Steps

Install typeguard version 4.3.0 in your environment.
Try to cdk synth the Stack defined below.

import os

from aws_cdk import App, Environment, Stack
from aws_cdk.aws_ec2 import SecurityGroup, Vpc
from aws_cdk.aws_lambda import DockerImageCode, DockerImageFunction
from constructs import Construct

class AppStack(Stack):
    def __init__(self, scope: Construct, id: str, env: Environment) -> None:
        super().__init__(scope=scope, id=id, env=env)

        sec_group = SecurityGroup.from_security_group_id(
            scope=self,
            id="LambdaSecurityGroup",
            security_group_id="sg-123456",
        )

        vpc = Vpc.from_lookup(
            scope=self,
            id="LambdaVPC",
            vpc_name="MyVPC",
        )

        self.lambda_function = DockerImageFunction(
            scope=self,
            id="AppLambda",
            function_name="AppLambda",
            description="Handles requests for the Design Brief API",
            memory_size=1024,
            code=DockerImageCode.from_image_asset(
                directory=".",
                file="./lambda.Dockerfile",
                build_secrets={"GITHUB_TOKEN": "type=env"},
            ),
            security_groups=[sec_group],
            vpc=vpc,
        )

app = App()
AppStack(
    scope=app,
    id="DummyStack",
    env=Environment(account=os.getenv("CDK_DEFAULT_ACCOUNT"), region="eu-west-1"),
)

app.synth()

Possible Solution
No response
Additional Information/Context
No response
CDK CLI Version
2.162.1
Framework Version
No response
Node.js Version
v22.9.0
OS
Mac
Language
Python
Language Version
Python 3.12.4
Other information
No response

typeguard-4.3.0 is not currently supported due to an issue with how protocols are now checked in the latest version. We will are investigating this, but cannot currently provide an ETA.
Our main priority is that users are not unintentionally blocked by. @jcmonteiro From your description ot sounds like you were explicitly declaring a dependency on typeguard 4.3.0 (as opposed to automatically receiving this version as a transitive dependency). Can you confirm this if this was the case?

Downgrade the typeguard version:
If this problem only occurs in typeguard version 4.3.0, you can try downgrading typeguard to 4.0.0 to see if it solves the problem:
pip install typeguard==4.0.0
In addition, you can use Servbay to deploy your development environment and ensure that there are no compatibility conflicts in the environment

I actually got the dependency via transitivity. I mentioned explicitly stating the version to make it easier to reproduce.
In fact, upon checking the changes to my Poetry lock file, I noticed that a particular commit caused "typeguard" to bump from 2.13 to 4.3. That was due to a cascade of upgrades allowed by the fact that aws-cdk-asset-kubectl-v20 declared support for typeguard = ">=2.13.3,<5.0.0"

Thanks @jcmonteiro Looks like there are a few more packages that need their version constraint updated.

But will close this here as a duplicate of https://github.com/aws/jsii/issues/4658 and track it over there.


Macyy crash when I constantly deleted the clipboard contents.

Can you please provide the Maccy version you use? You can get it in the "About" window.
Can you please record a screen video showing the problem? You can do this using QuickTime.app.
Can you please share the output of running defaults read org.p0deje.Maccy in Terminal.app?

Can you please provide the Maccy version you use? You can get it in the "About" window.
Can you please record a screen video showing the problem? You can do this using QuickTime.app.
Can you please share the output of running defaults read org.p0deje.Maccy in Terminal.app?

Addition, this won’t happen on menu bar.
<IP_ADDRESS>
2.view in accessory.
3.{
"KeyboardShortcuts_popup" = "{\"carbonKeyCode\":9,\"carbonModifiers\":2048}";
"LaunchAtLogin__hasMigrated" = 1;
"NSStatusItem Preferred Position Item-0" = 422;
"NSWindow Frame SUStatusFrame" = "520 556 400 134 0 0 1440 875 ";
"NSWindow Frame com.sindresorhus.Preferences.FrameAutosaveName" = "643 530 542 262 0 0 1440 875 ";
SUEnableAutomaticChecks = 1;
SUHasLaunchedBefore = 1;
SULastCheckTime = "2023-10-08 05:22:58 +0000";
SUUpdateGroupIdentifier = 162219098;
avoidTakingFocus = 1;
enabledPasteboardTypes =     (
"public.html",
"public.tiff",
"public.file-url",
"public.rtf",
"public.utf8-plain-text",
"public.png"
);
historySize = 80;
ignoredPasteboardTypes =     (
"com.agilebits.onepassword",
"Pasteboard generator type",
"com.typeit4me.clipping",
"net.antelle.keeweb",
"de.petermaurer.TransientPasteboardType"
);
maxMenuItems = 0;
menuIcon = clipboard;
migrations =     {
"2020-04-25-allow-custom-ignored-types" = 1;
"2020-06-19-use-keyboardshortcuts" = 1;
"2020-09-01-ignore-keeweb" = 1;
"2021-02-20-allow-to-customize-supported-types" = 1;
"2021-06-28-add-title-to-history-item" = 1;
"2021-10-16-remove-dynamic-pasteboard-types" = 1;
"2022-08-01-rename-suppress-clear-alert" = 1;
"2022-11-14-add-html-rtf-to-supported-types" = 1;
"2023-01-22-add-regexp-search-mode" = 1;
};
pasteByDefault = 1;
searchMode = fuzzy;
showRecentCopyInMenuBar = 0;
sortBy = lastCopiedAt;
}


Oops, used class instead of tag

This is a revised PR to fix compatibility with latest Atom API

Merged in e24812c


BUG: Timestamp properties may return np.int

Closes #10050.
Also added daysinmonth and dayofweek properties to NaT and perform tests.

can you also confirm (maybe add tests?) for Timedelta that it does the same

@jreback That's better. Added type check for Timedelta and its .components

looks good
go ahead and merge on green

Thanks for the review. Rebased and merged.


Add Decimal field

Draft
class Decimal(ModelField, typesystem.Decimal):
    def get_column_type(self):
        return sqlalchemy.Numeric(scale=self.scale, precision=self.precision)

    def __init__(self, scale: int, precision: int, **kwargs):
        assert scale, "scale is required"
        assert precision, "precision is required"
        self.precision = precision
        self.scale = scale
        super().__init__(**kwargs)

Usage
price = orm.Decimal(allow_null=True, scale=8, precision=18)

Well, it is ok, can I continue and offer pull request? Maybe you prefer Django's style max_digits and decimal_places?

Closed by #104


Irc colors on Telegram

Hi, is there a way to show irc colors on telegram? I keep receiving something like this "04mynick". I've seen that "04" is red but I can't find any parameter on "message_content". I've only found something about the pushover app and I don't know if this is telegram related: https://github.com/sopel-irc/sopel/issues/462
Thank you

To strip the colour codes, you can add a new line short_message.StripControls(); after the line 239. The you need to rebuild the plugin.


Feature/better string handling

add more usable string collection handling to publicly visible interfaces of model builder

Codecov Report

Merging #3 (4a6c06b) into main (a192767) will increase coverage by 0.26%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
+ Coverage   87.14%   87.41%   +0.26%
==========================================
  Files           9        8       -1
  Lines         428      429       +1
==========================================
+ Hits          373      375       +2
+ Misses         55       54       -1

Impacted Files
Coverage Δ

src/model/builder/modelfunction_builder/mod.rs
85.18% <83.33%> (+0.56%)
:arrow_up:

src/model/builder/mod.rs
84.28% <100.00%> (+1.42%)
:arrow_up:

src/model/builder/modelfunction_builder/test.rs
83.87% <100.00%> (ø)

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 a192767...4a6c06b. Read the comment docs.


Speed up findall_operations_until_blocked

slicing a Circuit actually creates a new Circuit (see circuits.py:214) which we do not need to iterate on the moments.  Instead, use self._moments
Also return early if we run out of active_qubits.

For an example circuit (depth 1270 circuit of 16 qubits, mixed 1q/2q gates), this speeds the merge_rots optimization (which depends on findall_operations_until_blocked) from 140 seconds to 1.5 seconds.

Nice.


Integration with external libraries - discussion

In our code, we integrate with contracts from external repositories such as bitcoin-spv, OpenZeppelin,keep-tecdsa, and others. We also have a proposition of adding even more external dependencies (see https://github.com/keep-network/tbtc/pull/263). We need to discuss and establish a common approach for integrating those contracts with our code.
There are four approaches for integration contracts from an external repository we experimented with so far:

git submodule - we add an external repository as a git submodule and we can interact with the entire codebase
npm dependency - some libraries have package.json configured in a way allowing us to add them as a dependency to node_modules. We can then interact with their entire codebase
ethereum package manager - we experimented with this project but seems it's still in early days, and may not but sufficient for us
defining interface in our codebase containing only the functions we use and casting contract address to an interfaceinterface ExternalSystem {
  function doWork()
}
address externalSystemAddress = ...
ExternalSystem(externalSystemAddress).doWork()

The qualities of our choice we should measure against:

clear separation of what is our code and what is an external code
easiness of setting up a development environment
gas cost of interacting with the code
the cost of maintaining the dependency

Proposition
There are two important differences between contracts from other repositories:

We use contracts as a library in our code (bitcoin-spv, OpenZeppelin)
We interact with an external system, deployed on the chain (keep-tecdsa)

Those two use cases require different approaches.
Library
For the library, it seems that the node_module approach (approach number 2) should be the preferred one. Both OpenZeppelin and bitcoin-spv support it. We can then interact with the entire codebase - use various exported functions, modifiers, inherit from contracts and so forth. This is what we expect from a library.
External system
For the external system, I think we should interact using interface (approach number 4).
Although it seems to be a common approach in solidity world to import entire codebase of the external system, it doesn't feel right.  Taking an example from other ecosystems, we don't import the source code of payment provider to our application, but instead, we use WSDL.
We can have one place (external directory?) where we can define interfaces to external systems as well as hardcode addresses of their contracts. This would look e.g.:
external/Keep.sol

Keep.sol:
address keepVendorAddress = ...

interface KeepVendor {
  openKeep() address
}

interface Keep {
  sign(byte32 digest)
}

and later we use:
  address vendor = KeepVendor(keepVendorAddress);
  Keep keep = Keep(vendor.openKeep())

Ideally, such interfaces should be exposed by all external systems, without a need of embedding their entire code in our repo as git submodule or npm dependency.
Worth noting, this discussion may influence how people will interact with keep-tecdsa or  random beacon.

I believe this is done and implemented now, as described in the tail end of the issue description. Closing.


Incorrect identity for unsigned int in cuddp_util.h

On line 302 of cuddp_util.h, the CUDPP_MIN identity for unsigned ints is
defined as INT_MAX. This is incorrect, as the maximum unsigned integer is
2*INT_MAX, or UINT_MAX.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 4 Aug 2009 at 3:23

Issue 34 has been merged into this issue.

Original comment by<EMAIL_ADDRESS>on 5 Aug 2009 at 11:34

Original comment by<EMAIL_ADDRESS>on 4 Aug 2009 at 9:35

Added labels: Component-Logic, OpSys-All, Milestone-Release1.2

Issue verified by issue 34.  Easy to fix, will do so soon.  Assigning self.

Original comment by<EMAIL_ADDRESS>on 5 Aug 2009 at 11:35

Changed state: Accepted
Added labels: Priority-Critical
Removed labels: Priority-Medium

This issue was closed by revision r9.

Original comment by<EMAIL_ADDRESS>on 6 Aug 2009 at 8:44

Changed state: Fixed

Assigning self.  Fix is easy, will do soon.

Original comment by<EMAIL_ADDRESS>on 5 Aug 2009 at 11:42


use default charset for cli --properties file

jboss-cli can read only ISO 8859-1 properties files via --properties arg
this patch fix it

Can one of the admins verify this patch?

@wildfly/prod Test mention

This is ok to test.
@jfdenise Can you have a look at this?
@andrew--i If Jeff has no problem with the idea, can you file a JIRA in https://issues.jboss.org/browse/WFCORE and reference its number in the commit message? Thanks.

This is ok to test.
@jfdenise Can you have a look at this?
@andrew--i If Jeff has no problem with the idea, can you file a JIRA in https://issues.jboss.org/browse/WFCORE and reference its number in the commit message? Thanks.

done, WFCORE-4505

@andrew--i , good catch. I am wandering if we shouldn't use UTF-8 instead of the default that is bound to the platform. Long term we should decode all files in CLI using UTF-8 as the default and offer a way to set the encoding.

+1 for using StandardCharsets.UTF_8

@andrew--i , good catch. I am wandering if we shouldn't use UTF-8 instead of the default that is bound to the platform. Long term we should decode all files in CLI using UTF-8 as the default and offer a way to set the encoding.

I made changes

@andrew--i , I think that we shouldn't introduce a new property with your change. It requires a proper RFE (doc, help, ...) and should be applied to various places. Could you just fix the properties file? Thank-you.

@andrew--i , I think that we shouldn't introduce a new property with your change. It requires a proper RFE (doc, help, ...) and should be applied to various places. Could you just fix the properties file? Thank-you.

done

@andrew--i thanks


Can't obtain summaries.

Hello:
Thanks for your excellent work!
When i run your code with "python booookscore/summ.py --book_path data/all_books_chunked_2048.pkl --summ_path data/summaries.json --model gpt-4 --api openai --api_key api_key.txt --method hier --chunk_size 2048 --max_context_len 8192",i meet with a problem that:

I checked my connect to openai.com,and there was no problem.Would you please tell me how to deal with this problem?

Would you please tell me how long did you take to obtain summaries for the example_book?


add SPF record support

Currently record appender doesn't support SPF records. They are similar in format to TXT records but using a TXT record for it obviously has the wrong type.
I'll submit a PR later today if I have time.

Nevermind. Looks like the practice of using an SPF record has been deprecated in favor of just using a TXT record.

In July 2005, IANA assigned the Resource Record type 99 to SPF. Later on,[6] the use of SPF records was discontinued, and as of 2016, it is still necessary to use TXT records.[6]


Query engine binary for current platform "rhel-openssl-1.0.x" could not be found

I have been working through a RedwoodJS tutorial, which uses Prisma and deploys to Netlify. On my local machine, everything works great. However, on the deployed version I see the following error:
Error: Invalid `prisma.post.findMany()` invocation in /var/task/src/api/dist/services/posts/posts.js:16:22 Query engine binary for current platform "rhel-openssl-1.0.x" could not be found. This probably happens, because you built Prisma Client on a different platform. (Prisma Client looked in<EMAIL_ADDRESS>Files in /var/task/src/node_modules/@prisma/client/runtime: index.d.ts index.js index.js.map You already added the platform "native" to the "generator" block in the "schema.prisma" file as described in https://pris.ly/d/client-generator, but something went wrong. That's suboptimal. Please create an issue at https://github.com/prisma/prisma-client-js/issues/new

Some info about the environment:

Prisma is connecting to Postgres through Heroku.
DATABASE_URL is defined as an environment variable in Netlify.
Build image is Ubuntu Xenial 16.04

// schema.prisma

datasource DS {
  provider = ["sqlite", "postgresql"]
  url      = env("DATABASE_URL")
}

generator client {
  provider      = "prisma-client-js"
  binaryTargets = "native"
}

The repo: https://github.com/mkhbragg/redwoodjs-blog-tutorial
The app: https://keen-roentgen-d64850.netlify.app/
Is this error familiar to the Prisma team? I am brand new to Prisma and would love the help. Error told me to open an issue so I'm opening an issue 😂 🙏
Update [SOLVED]
Was able to get the app working properly in Netlify by adding "rhel-openssl-1.0.x" to generator client. Keeping this issue open in case this is a genuine issue from Prisma perspective.
generator client {
  provider      = "prisma-client-js"
  binaryTargets = ["native", "rhel-openssl-1.0.x"]
}

Closing as this is a duplicate of https://github.com/prisma/prisma/issues/4011


Kafka 2.x upgrade

Placeholder issue for upgrading cerner_kafka to kafka 2.x, if any changes need to be made
The latest release was 2.2

One fairly trivial problem we found is that the init.d script no longer detects the running kafka broker process, because with the classpath library additions its full command line size has increased to the point where "kafka.Kafka" occurs beyond the Linux PAGE_SIZE (4096) number of characters.
Changing the pgrep command from pgrep -u kafka -f kafka.Kafka to pgrep -u kafka -x java -P 1 should be safe, and fixes the issue.


List all the /v2 api endpoints used and replace them with a /v1 endpoint

/v1 endpoint are more performant while /v2 endpoint is a RPC endpoint which is slower. This will help prevent RPC congestion.
example done here https://github.com/hirosystems/explorer/issues/586 to replace the account /v2 fetch with a /v1 nonce fetch

To be clear, when you say v1, you really mean /extended/v1 endpoints?

Yes I meant extended/v1. I updated the issue for clarity.

There are no other v2 endpoints to refactor beside the one already done here https://github.com/hirosystems/explorer/pull/760
Closing.


ITVR - Investigate change in approved rebate levels

Problem Description
**The rebate program is running short of funds and want to potentially reduce the amount of rebate available to all levels of approved rebate applications.
Solution Needs

The solution must communicate the changed rebate levels to the applicants and provide this new approved rebate level data to NCDA via the API.

Timebox

See points on ticket

Outcome
Details describing the outcome of the research

Dev team to confirm the feasibility (or not) of making this change to approved rebate levels.
Set out the steps required to make this change
Detail any associated risks.

https://app.zenhub.com/files/223241604/4a7f232f-d173-49ad-9357-4b634467f6f3/download
Feasibility: Yes.
Steps required to make this change: Please see attached pdf at the top.
Risks/questions: When an application is "approved" on our end and "not-redeemed" on the NCDA's end, what exactly does that mean? I seem to recall the NCDA saying something about an applicant being "not-redeemed" even when they've bought a car (they have to keep the car for 6 months or something). I'm just a little bit concerned that we will be telling an applicant their rebate level changed when they've already bought a car...
Another question: Just to confirm, in the same release as the one containing the implementation of this proposed change, we are also changing the rebate amounts for all future applicants?
Notes:
(1) It would be prudent for the ncda to set up a test environment so that when testing this (whether locally or otherwise) we don’t have to use their prod environment; it doesn’t have to be anything fancy, just a new List (e.g. TestITVREligibilityList) that we can perform CRUD operations on via the API would be fine…
(2) We can update 100 approved applications per hour to the new rebate amounts (due to supposed NCDA API limitations); so for 5000 approved applications, that's a bit more than 2 days (48 hours).

Another option: easiest thing to do would be to just continue doing things the way they are until the sum of all the max rebate amounts reach a certain number, and then just stop people from making applications from that point forth (or until there's more funds). We can build something so that analysts can see, in the console, the total value of all rebates approved/redeemed; alternatively, this total might be something they can grab directly from the database using CTHUB.

@airinggov I'm not sure; perhaps the NCDA requires that we send them a number? But I think I see your point; if we refactored to store the levels A,B,C instead of the numbers, and this situation comes up again where we need to reduce the amounts of already approved rebates, then we wouldn't need to send updated data to the NCDA - we could just tell them that, for example, A now means 3500, B means 1500, etc.


Add name to breaker to differentiate in stats + added client panic re…

…covery.

Coverage increased (+0.02%) to 95.346% when pulling d2c9175539116fdd29f6dad6773d88e11d34a06b on breaker-client-fix into caa2a5826bd0c82ac7aeb0c2a6f49e0e2c4c6b11 on master.


Multi touch detection In Flutter flame

I want to implement game control for a boxing game.
Four sections in black represent uppercuts and jabs. In order to block, the player should use multi-touch, where one figure should be on the left half of the screen and another figure on the right half. This is clearly overlapping other controls.
What is the best way to handle multi-touch with little conflicts with other controls in the Flame engine (flutter)? (if the user has two fingers for several seconds, the game should keep the player in the blocking mode)
I tried adding MultiTouchTapDetector but this did not give any indication as to user is tapping and keep pressing from two fingers.

@patolax did you try using Tappble mixin on components? Here is a simple example where I am able to detect block as you mentioned.
class MultiTapGame extends FlameGame with HasTappables {
  @override
  Future<void> onLoad() async {
    debugMode = true;
    await add(Controls());
  }
}

class Controls extends Component with HasGameRef {
  bool isLeftDown = false;
  bool isRightDown = false;

  @override
  Future<void> onLoad() async {
    await addAll(
      [
        TappableRegion(
          position: Vector2.all(0),
          size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.75),
          onTapDownCallback: () {
            isLeftDown = true;
            if (isRightDown) {
              print("Block");
            } else {
              print("Left Uppercut");
            }
          },
          onTapUpCallback: () {
            isLeftDown = false;
          },
        ),
        TappableRegion(
          position: Vector2(gameRef.size.x * 0.5, 0),
          size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.75),
          onTapDownCallback: () {
            isRightDown = true;
            if (isLeftDown) {
              print("Block");
            } else {
              print("Right Uppercut");
            }
          },
          onTapUpCallback: () {
            isRightDown = false;
          },
        ),
        TappableRegion(
          position: Vector2(0, gameRef.size.y * 0.75),
          size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.25),
          onTapDownCallback: () {
            isLeftDown = true;
            if (isRightDown) {
              print("Block");
            } else {
              print("Left Jab");
            }
          },
          onTapUpCallback: () {
            isLeftDown = false;
          },
        ),
        TappableRegion(
          position: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.75),
          size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.25),
          onTapDownCallback: () {
            isRightDown = true;
            if (isLeftDown) {
              print("Block");
            } else {
              print("Right Jab");
            }
          },
          onTapUpCallback: () {
            isRightDown = false;
          },
        ),
      ],
    );
  }
}

class TappableRegion extends PositionComponent with Tappable {
  TappableRegion({
    super.position,
    super.size,
    super.scale,
    super.angle,
    super.nativeAngle,
    super.anchor,
    super.children,
    super.priority,
    this.onTapDownCallback,
    this.onTapUpCallback,
  });

  late RectangleComponent _rectangleComponent;

  @override
  FutureOr<void> onLoad() {
    _rectangleComponent = RectangleComponent(
      size: size,
      paint: Paint()..color = Colors.green,
    );
    return super.onLoad();
  }

  void Function()? onTapDownCallback;
  void Function()? onTapUpCallback;

  @override
  bool onTapDown(TapDownInfo info) {
    onTapDownCallback?.call();
    add(_rectangleComponent);
    return super.onTapDown(info);
  }

  @override
  bool onTapUp(TapUpInfo info) {
    onTapUpCallback?.call();
    _rectangleComponent.removeFromParent();
    return super.onTapUp(info);
  }

  @override
  bool onTapCancel() {
    onTapUpCallback?.call();
    _rectangleComponent.removeFromParent();
    return super.onTapCancel();
  }
}

Wow big fan, I started flame yesterday after watching your spaceshooter series. And the code below you will see that I have used the command system from spaceshooter.
Your approach is much more stable than what I ended up with.
I change the above logic to add stop-blocking animation when the user stops multi-tapping. Hope this is correct.
import 'dart:async';
import 'package:chessboxing/logic/game.dart';
import 'package:chessboxing/models/enums.dart';
import 'package:flame/components.dart';
import 'package:flame/events.dart';
import 'package:flutter/material.dart';
import '../logic/command.dart';
import '../logic/player.dart';
class Controls extends Component with HasGameRef {
bool isLeftDown = false;
bool isRightDown = false;
bool isBlocking = false;
@override
Future onLoad() async {
await addAll(
[
TappableRegion(
position: Vector2.all(0),
size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.75),
onTapDownCallback: () {
isLeftDown = true;
if (isRightDown) {
addCommand(JabTypes.blockStart);
print("Block");
} else {
addCommand(JabTypes.leftUppercut);
print("Left Uppercut");
}
},
onTapUpCallback: () {
isLeftDown = false;
stopBlocking();
},
),
TappableRegion(
position: Vector2(gameRef.size.x * 0.5, 0),
size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.75),
onTapDownCallback: () {
isRightDown = true;
if (isLeftDown) {
addCommand(JabTypes.blockStart);
print("Block");
} else {
addCommand(JabTypes.rightUppercut);
print("Right Uppercut");
}
},
onTapUpCallback: () {
isRightDown = false;
stopBlocking();
},
),
TappableRegion(
position: Vector2(0, gameRef.size.y * 0.75),
size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.25),
onTapDownCallback: () {
isLeftDown = true;
if (isRightDown) {
addCommand(JabTypes.blockStart);
} else {
addCommand(JabTypes.leftJab);
}
},
onTapUpCallback: () {
isLeftDown = false;
stopBlocking();
},
),
TappableRegion(
position: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.75),
size: Vector2(gameRef.size.x * 0.5, gameRef.size.y * 0.25),
onTapDownCallback: () {
isRightDown = true;
if (isLeftDown) {
print("Block");
addCommand(JabTypes.blockStart);
} else {
print("Right Jab");
addCommand(JabTypes.rightJab);
}
},
onTapUpCallback: () {
isRightDown = false;
stopBlocking();
},
),
],
);
}
stopBlocking() {
if (isBlocking) {
isBlocking = false;
addCommand(JabTypes.blockStop);
}
}
addCommand(JabTypes jabType) {
if (jabType == JabTypes.blockStart) {
isBlocking = true;
}
final command = Command(action: (player) {
player.action(jabType);
});
gameRef.addCommand(command);
}
}
class TappableRegion extends PositionComponent with Tappable {
TappableRegion({
super.position,
super.size,
super.scale,
super.angle,
super.nativeAngle,
super.anchor,
super.children,
super.priority,
this.onTapDownCallback,
this.onTapUpCallback,
});
late RectangleComponent _rectangleComponent;
@override
FutureOr onLoad() {
_rectangleComponent = RectangleComponent(
size: size,
paint: Paint()..color = Colors.green,
);
return super.onLoad();
}
void Function()? onTapDownCallback;
void Function()? onTapUpCallback;
@override
bool onTapDown(TapDownInfo info) {
onTapDownCallback?.call();
return super.onTapDown(info);
}
@override
bool onTapUp(TapUpInfo info) {
onTapUpCallback?.call();
return super.onTapUp(info);
}
@override
bool onTapCancel() {
onTapUpCallback?.call();
return super.onTapCancel();
}
}

@patolax that looks correct 👍🏼.
Also just a tip for the future, if you ever run into such "how-to-do-xyz-in-flame" style questions, try posting them in Flame's discord server or in the Discussions of this repo. That will make it visible to more people more quickly 😉

will do. Thanks for the tip.
While you are here, couple of other questions on flame.

what's the best way to detect punches on the opponents? (currently, the sprite is a square, I am guessing the player square and opponent square should be overlapping and the player should be in a jabbing state)
what's the best way to handle skins. Say I want the player to have configurable

skin colour - white, black, brown
head guards with and without
glove colors (red, blue, gold, white)
body types (skinny, jacked, normal)
maybe hairstyles (bold, short and normal)
as you can see if there are so many variations, we have to ship the app with so many sprites. any idea how to implement such thing?

thanks a lot.

will do. Thanks for the tip.
While you are here, couple of other questions on flame.

what's the best way to detect punches on the opponents? (currently, the sprite is a square, I am guessing the player square and opponent square should be overlapping and the player should be in a jabbing state)
what's the best way to handle skins. Say I want the player to have configurable

skin colour - white, black, brown
head guards with and without
glove colors (red, blue, gold, white)
body types (skinny, jacked, normal)
as you can see if there are so many variations, we have to ship the app with so many sprites. any idea how to implement such thing?

thanks a lot.

Take those questions on discord or in separate GitHub discussions (just like DevKage said) since they aren't related to the original question in the issue you opened. :)


Projections for nested documents in query

The problem
Current syntax of projections comes from relational world where everything is a flat table. However there is links and nested records in document model. And there is no possibility to project fields of linked or nested documents and keep nesting.
There is a possibility to prevent loading of some fields of linked documents using fetch plan. But it is not allow to include only specific fields.
For example you have a Shop, Merchandise and Producer. And you want to select all shops with merchandise and its producers, but producer have a lot of properties and you need to load only its name and country.
SELECT name, include(merchandise, 'name', 'producer') FROM Shop

The query above loads all properties of producer.
fetchPlan and include/exclude functions
Some use-cases can be implemented by fetchPlan and include/exclude functions, however this is less convenient and provide less flexibility.
Proposed syntax
selectQuery ::= SELECT <projections> FROM ...
projections ::= <projection>(, <projection>)*
projection ::= <fieldName>(':{' <projections> '}')? (as <alias>)?

Examples
SELECT
  name,
  address:{country:{name, capital} as country, city.name as city, street, building}
FROM Customer

Result example:
{
  name: "Customer1"
  address: {
    country: {
      name: "Ukraine",
      capital: "Kiev"
    }
    town: "Dnipropetrovsk"
    street: "..."
    building: 99
  }
}

Graph example:
SELECT
  name,
  out('repositories').out('commits'):{date, message} as commits
FROM Developer

Result example:
{
  name: "Martin",
  commits: [
    {date: "12-08-2013", message: "Fix minor bug"}
    {date: "13-08-2013", message: "Fix major bug"}
  ]
}

I don't like this syntax very much, in particular I don't see the need for the initial "fieldname:".
My proposed syntax is jut
SELECT name,
{
"country": {"name": name, "capital": capital }}

I don't like this syntax very much, in particular I don't see the need for the initial "fieldname:".
My proposed syntax is jut
SELECT name,
{
"country": {"name": name, "capital": capital }}

I don't like this syntax very much, in particular I don't see the need for the initial "fieldname:".
My proposed syntax is jut
SELECT name,
{
"country": {"name": name, "capital": capital }}

+1 for this syntax

+1 This would be awesome.

+1 very nice

+1

Hi guys,
some news on this issue. In v.2.2 this is already supported in MATCH statement.
The following query is valid (GratefulDeadConcerts)
MATCH {class:V, as: v, where: (name like '%en%') }
return {
   "name":v.name,
   "metadata":{
      "type":v.type
   }
}

Thanks
Luigi

Some more news here. In V 3.0 this syntax is also supported in SELECT statement
Thanks
Luigi

The nested projection syntax is implemented in develop branch (3.0 SNAPSHOT). It will be released with 3.0 M2.
For more details https://github.com/orientechnologies/orientdb-docs/blob/3.0.x/sql/SQL-Projections.md

Big win!

@luigidellaquila will we be able to use the RETURN keyword in SELECT like in MATCH? It is rather nice as it allows one to template a response shape (with easy aliases etc).
In addition will RETURN in MATCH inherit the behaviour of the new SELECT nested projections (I assume this new capability handles returning nested list properties)

Hi @jpstrikesback
there is no need for a RETURN keyword in the SELECT, RETURN is just the equivalent of the projections in the SELECT, eg.
MATCH {class:V, as: v, where: (name like '%en%') }
return {
"name":v.name,
"metadata":{
"type":v.type
}
}
is the same as
SELECT
{
   "name": name,
   "metadata": {
      "type": type
   }
}
FROM V WHERE name like '%en%'

Thanks
Luigi

Is there a way for doing this with the projection api?
https://stackoverflow.com/questions/49495351/orientdb-returning-json-with-embedded-vertices-from-a-match-query
I'm trying for hours now and I can't find the right syntax or any examples of this use case.


Selection issue on the MaterialSegmentedControl

When we change the selected item/index from the code behind or from the ViewModel, the UI isn't updated and the old item continues selected in the UI.
Fix it on both versions (V2 and V3).

We are excited to announce that we are working on a new generation of controls based on Material Design V3, which we will be releasing soon.
This ticket has been included/fixed in the upcoming version of this NuGet package, which will include both current and new generations of controls.


Says config.json is missing but its not, I downloaded all files and edited the config.json but it still says its not there (but it is in my folder with all the other downloads)

[ ERROR ] config.json file is missing. Make sure you downloaded all the files and they're all in the same folder
[ ERROR ] An error has occured check logs.txt for more info
Traceback (most recent call last):
File "C:\Users\thoma\OneDrive\Documents\Bloxflip-Smart-Bet-main\Bloxflip-Smart-Bet-main\Martingale+Predictor (Paid).py", line 728, in
main()
File "C:\Users\thoma\OneDrive\Documents\Bloxflip-Smart-Bet-main\Bloxflip-Smart-Bet-main\Martingale+Predictor (Paid).py", line 40, in init
raise e
File "C:\Users\thoma\OneDrive\Documents\Bloxflip-Smart-Bet-main\Bloxflip-Smart-Bet-main\Martingale+Predictor (Paid).py", line 28, in init
self.getConfig()
File "C:\Users\thoma\OneDrive\Documents\Bloxflip-Smart-Bet-main\Bloxflip-Smart-Bet-main\Martingale+Predictor (Paid).py", line 232, in getConfig
with open("config.json", "r+") as data:
FileNotFoundError: [Errno 2] No such file or directory: 'config.json'

did you unzip the folder


update ready for the flat loader package

Requires release of dojo-loader
Issue reference: https://github.com/dojo/meta/issues/73

Current coverage is 95.96% (diff: 100%)

No coverage report found for master at d20d3d8.

Powered by Codecov. Last update d20d3d8...ef7ecc0


NODE_ENV is undefined

Looks like the process.env.NODE_ENV  is being set to undefined.
The cross-env doesn't seem to function properly from the scripts.
I'm trying to set the flag as development, but getting undefined, and an error as below -
only development, staging, test and production are valid NODE_ENV variables but undefined is specified
Kindly suggest solution.

Hey @ashish00076,
thanks for the issue...
Would you mind to share a bit more details?
Node version, if you are using yarn or npm to start the command.  Can you share your package.json? Have you installed all the packages with npm i or yarn?

Hey @aichbauer ,
I found the cause.
I was running the project with Visual Studio Code in debugging mode, hence it couldn't detect environment.
Devised a workaround for the same by creating a config in VS Code launch.json. This solved the issue. Now I can run the project in debugging mode too.
Thank you.
Regards,
Ashish Deshpande


Exam mode: Update collection of  switch exercises action

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

As noticed in https://github.com/ls1intum/Artemis/pull/5358, there collection of switched exercise actions needs some enhancements.
Description

Switching from the handed early page to the exercise is now correctly visible in the dashboard
Save and continue always puts the save timestamp before continue

Steps for Testing

Prerequisites:

1 Instructor
One/Multiple Student
1 Exam

Log in to Artemis
Create an Exam with Exam Live Statistics enabled
Participate with the student
Check that the switches are detected correctly.

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Since the bamboo tests fail:

Screenshots

https://user-images.githubusercontent.com/60005702/178297679-ecfa79eb-2c61-46fa-9c09-bbb3cd4fa598.mp4

Tested on TS1. Works as expected.


Modified README to remove link to toolbelt.h.c

Now that toolbelt.h.c redirects, this modified the README to remove this reference.

👍