Use task scheduler to process optimistic responses and execute error rollback.

Operations in Relay will be flushed synchronously by default which has been observed to cause potential race conditions in the client app. In practice this is very difficult to reproduce and is dependant on various factors that may affect the component render order of the application.
An effective method to mitigate these potential errors is by configuring a task scheduler to batch updates, however, the OperationExecutor doesn't use the scheduler when processing optimistic responses, or during rollback after a network error meaning these events are still susceptible to obscure race condition errors. This PR uses the configured scheduler in these scenarios to help alleviate this class of error.
Example of an invalid render due to such an error:
Warning: Relay: Expected to have been able to read non-null data for fragment `?` declared in `useFragment()`, since fragment reference was non-null. Make sure that that `useFragment()`'s parent isn't holding on to and/or passing a fragment reference for data that has been deleted.

I intend on raising this PR upstream in the main repo but I'm prioritising updates here initially for early feedback and validation.

This fork isn't operating on a pull-request model, it's just a snapshot which allows us to use github-actions to build.
Want to:

Open a PR upstream for this. I'm interesting in the Relay maintainers thinking on this too
Stack your PR with the current changes and force-push to build

Would also be good to rebase on upstream so we eliminate the currently accumulated drift

In terms of this change itself I'm not sure if this is valid. As I understand it, most of the OperationExecutor runs synchronously (and most of relay internally throughput) and it really tends to expected single-threaded and sequential execution.
For example, this.cancel() is intended to update the current state (this._state = ) and deferring that means that iteration could continue or fall into an infinite loop.
The idea of scheduling throughout relay is not to defer any part of computation, but to batch together things which are already going to be handled concurrently anyway: notably DOM updates and fetching

Have you got a description of the problem or a reduced test case example (internal or external)?
I definitely think you know enough to have a valid case here, but it's likely something which could instead be solved by:

Working out the actual event ordering and fixing a bug (which would most likely rely on some other synchronous checks added to this file)
Working out how the different events to relay are eventually coming out the other side and working to change how those are batched up for react

I've only observed this issue in a local app environment and had very little success trying to debug what are the magic series of inputs/outputs that lead to the error. I also attempted to create a minimal reproduction from the relay-examples repo and again no luck.
To briefly summarise we have a connection of rows in a table and we execute a mutation to create a new row. This row is optimistically appended to the connection using @appendNode, and when a network error occurs Relay will attempt the rollback.

  <Row ref="client:root:tableConnection:edges:0" />
  <Row ref="client:root:tableConnection:edges:1" />
  <Row ref="client:root:tableConnection:edges:2" />

Mutation is executed

  <Row ref="client:root:tableConnection:edges:0" />
  <Row ref="client:root:tableConnection:edges:1" />
  <Row ref="client:root:tableConnection:edges:2" />
  <Row ref="client:root:tableConnection:edges:3" />  // Row is appended after optimistic update

Network error and rollback occurs. During the rollback a synchronous update is flushed causing the table to be re-rendered. Interestingly, the table connection still includes 4 edges, but while trying to call useFragment from within the Row component Relay cannot find the record and logs the warning.
Warning: Relay: Expected to have been able to read non-null data for fragment `?` declared in `useFragment()`, since fragment reference was non-null. Make sure that that `useFragment()`'s parent isn't holding on to and/or passing a fragment reference for data that has been deleted.

A very obscure issue to try and debug. The rendering behaviour appeared to change based on which fields I specified within my Row fragment, i.e. with Row.fieldA the error occurs, and without Row.fieldA it behaves as expected.
What I did narrow down however, is that rollback works as expected when a successful GraphQL payload is returned, and when removing the configured task scheduler it fails in the same fashion. This is how I landed on the above outcome. I'm happy to raise a PR upstream for further feedback, we could also consider moving the scheduler down specifically to where the rollback is executed during cancel so the current state update can remain synchronous.


Getting auth error [Error: Closed. Original cause [Error: [AUTHORIZATION] Authorization failed   Generate a new JWT secret on Supabase

While authentication we are getting success but that auth session isn't persist we are getting logged out immediately and getting following error.
[PowerSyncRemote] [Error: Closed. Original cause [Error: [AUTHORIZATION] Authorization failed
Generate a new JWT secret on Supabase. Cause: unrecognized configuration parameter "app.settings.jwt_secret"
(severity "ERROR", file "guc.c", line 5710, routine "find_option")].]
I have checked all the variables which we have declared in the .env file all values and keys are correct, Although we are getting defined values as well wherever we are using them.

Hey @vaibhavbucchaSureworx, the first question I would ask is whether your Supabase project had been paused before?
You can see in the error the recommendation of generating a new JWT secret on Supabase.
If this is an unreleased product resetting should be safe, otherwise see https://discord.com/channels/1138230179878154300/1288256116408586343/1288612779636428820

Hey @vaibhavbucchaSureworx, any luck with this?

Just I wanna mention, I only encounter this error when I connect to my phone's hotspot.

I had the same issue. In my case, I'm using Supabase but with a third-party authentication integration (Cognito). To fix this error, you just need to change the JWT secret in the settings->api->jwt secret panel


⚠ AGEG website (prod) has degraded performance

In 6a2fe4d, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 1669 ms

Resolved: AGEG website (prod) performance has improved in 98d7605.


Append feature_ojdkmh_exclude.txt to EXCLUDE_FILE

Append feature_ojdkmh_exclude.txt to EXCLUDE_FILE if OJDK MHs are enabled.
TODO: eclipse-openj9/openj9#12811
Signed-off-by: Babneet Singh<EMAIL_ADDRESS>

@llxia @smlambert This PR is dependent on https://github.com/eclipse-openj9/openj9/pull/12779. I have tested these changes locally. They work correct with OJ9 and OJDK MHs. I will be on leave starting next Monday. Can we have this PR merged by Friday?

@llxia can this be merged?

This PR has to be merged together with https://github.com/eclipse-openj9/openj9/pull/12779

Thanks @tajila , I will merge this PR.


SPO tenant

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Task list

[ ] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md in the resource folder.
[ ] Resource parameter descriptions added/updated in schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #39 into Dev will increase coverage by 58.97%.
The diff coverage is 0.43%.

@@            Coverage Diff             @@
##             Dev      #39       +/-   ##
==========================================
+ Coverage   1.19%   60.16%   +58.97%
==========================================
  Files          6        7        +1
  Lines        504      733      +229
  Branches       0        1        +1
==========================================
+ Hits           6      441      +435
+ Misses       498      291      -207
- Partials       0        1        +1

Impacted Files
Coverage Δ

...SC/DSCResources/MSFT_SPOTenant/MSFT_SPOTenant.psm1
0.43% <0.43%> (ø)

...s/MSFT_EXOSharedMailbox/MSFT_EXOSharedMailbox.psm1
60.27% <0%> (+58.9%)
:arrow_up:

...365DSC/DSCResources/MSFT_SPOSite/MSFT_SPOSite.psm1
88.46% <0%> (+86.53%)
:arrow_up:

...SCResources/MSFT_EXOMailTips/MSFT_EXOMailTips.psm1
90% <0%> (+88.33%)
:arrow_up:

...SC/DSCResources/MSFT_O365Group/MSFT_O365Group.psm1
90.24% <0%> (+89.43%)
:arrow_up:

.../DSCResources/MSFT_ODSettings/MSFT_ODSettings.psm1
93.45% <0%> (+92.52%)
:arrow_up:

...5DSC/DSCResources/MSFT_O365User/MSFT_O365User.psm1
95.5% <0%> (+94.38%)
:arrow_up:

Continue to review full report at Codecov.

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

Closing this PR and adding a new one... found another thing to fix


[BUG] deepflow-mysqlis unable to connect, prompting "Host 'xx. xx. xx. xx' is blocked because of many connection errors; unblock with 'mysqladmin flush hosts'"

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

DeepFlow Component
Server
What you expected to happen
通过helm安装后，过一个小时发现deepflow-mysql连不上。经排查发现max_connect_errors数量超过限制。通过调整改参数由于数量不短累积一段时间后又连不上无法真正解决问题。
How to reproduce
helm安装6.4版本后就能复现。
DeepFlow version
Name: deepflow-server community edition
Branch: v6.4
DeepFlow agent list
与agent无关
Kubernetes CNI
calico
Operation-System/Kernel version
5.10.0-<IP_ADDRESS>.oe2203sp3.x86_64
Anything else
mysql> select * from host_cache\G;
发现有其他namespace的服务不停连接deepflow-mysql，然后导致错误连接数一直上涨。最重导致无法连接。

Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

通过跳过 mysql 解析解决，并不是 deepflow 内部原因
https://dev.mysql.com/doc/refman/8.4/en/server-system-variables.html#sysvar_skip_name_resolve


Modify connectedToTheDom check to work correctly for elements owned b…

…y documents other than the global document.
This fixes a bug that prevented the use of backspace in form elements (or content-editables) in secondary windows created using window.open.
Detached elements will still have the ownerDocument property, so this change still works for the use case addressed by #4. @fresheneesz would you mind testing this to make sure it still works for you?
Fixes #5.

@fresheneesz by chance can you check if it works with your recent usecase?
I think it works according to the jsfiddle I made previously here: https://jsfiddle.net/663tr51z/12/

Huh, it looks like chrome very recently got rid of the "back button goes back a page" thing. It now tells you to use alt <-

Seems to still work in firefox, where backspace still would go back. So works for me!

published 1.1.2


chore(docs): mention setHeader

I think it would be useful to have the setHeaders and setHeader functions documented somewhere.
Not sure if this is the right place for them but I think auth is the most common use case.

Thanks @stphnnnn!

Most welcome 🙇


Linking to external files

It seems as though if I clone a repo from github using Termux, and open the directory with Acode I can not like to CSS or JAVASCRIPT files that reside in a directory outside of my HTML. Such as having a folder for my html, JavaScript, and css and try to link to my js/css file '../Stylesheets/style.css it will not link properly.

I also encountered the same issue. I cannot open the files in the upper directory. But I found a workaround.
I have this structure:
folder1
--file.js
--file.css
folder2 // with the project
--subfolder1
--subfolder2
--and other folders with HTML files that point to the folder1 files
--index.html

As a workaround, I open the index.html file from folder2. It has iframes that display the preview of the subfolder files. And it works.
Also, there is a button to open a file in subfolder1 and so on.
So, as a result, I open HTML files in the subfolder[1-20] via index.html. If I do that directly, it doesn't work.


self_reported_ethnicity must allow multiple HANCESTRO terms

Design
Note: Also see #single-cell-data-wrangling.
obs (Cell Metadata)
obs is a pandas.DataFrame.
Curators MUST annotate the following columns in the obs dataframe:
self_reported_ethnicity_ontology_term_id

Key
self_reported_ethnicity_ontology_term_id

Annotator
Curator

Value

        categorical with str categories. If
        organism_ontolology_term_id is
        "NCBITaxon:9606" for Homo sapiens,
        str values MUST be one or more comma-separated HANCESTRO
        terms or "unknown" if unavailable.The order of multiple
        terms is not significant. For example:"HANCESTRO:0005, HANCESTRO:0014" and "HANCESTRO:0014, HANCESTRO:0005" are equivalent.The following terms MUST NOT be used:

"HANCESTRO:0002"
            for regions and its children

"HANCESTRO:0003"
            for country

"HANCESTRO:0004"
            for ancestry category

"HANCESTRO:0018"
            for uncategorised population

"HANCESTRO:0290"
            for genetically isolated population

"HANCESTRO:0304"
            for ancestry status and its children

"HANCESTRO:0323"
            for Finnish founder

"HANCESTRO:0324"
            for Dutch founder

"HANCESTRO:0551"
            for genetically homogenous Irish

"HANCESTRO:0554"
            for Silk Road founder

"HANCESTRO:0555"
            for Arab Israeli founder

"HANCESTRO:0557"
            for Costa Rican founder

"HANCESTRO:0558"
            for French Canadian founder

"HANCESTRO:0559"
            for Italian founder

"HANCESTRO:0560"
            for Northern Finnish founder

"HANCESTRO:0561"
            for Romanian founder

"HANCESTRO:0564"
            for Vis founder

"HANCESTRO:0565"
            for Split founder

"HANCESTRO:0566"
            for undefined ancestry population

            The imported GEO term
            "GEO:000000374"
            for continent and its children:

"HANCESTRO:0029"
                for Africa

"HANCESTRO:0030"
                for Asia

"HANCESTRO:0031"
                for Europe

"HANCESTRO:0032"
                for Oceania

"HANCESTRO:0033"
                for Latin America and the Caribbean

"HANCESTRO:0034"
                for Northern America

Otherwise, for all other organisms the
        str value MUST be "na".

When a dataset is uploaded, the CELLxGENE Discover MUST automatically add the matching human-readable name for the corresponding ontology term to the obs dataframe. Curators MUST NOT annotate the following columns.
self_reported_ethnicity

Key
self_reported_ethnicity

Annotator
CELLxGENE Discover

Value
categorical with str categories. This MUST be "na" or "unknown" if set in self_reported_ethnicity_ontology_term_id; otherwise, this MUST be one or more comma-separated human-readable names assigned to the values of self_reported_ethnicity_ontology_term_id. For example, if the value of self_reported_ethnicity_ontology_term_id is "HANCESTRO:0005, HANCESTRO:0014" then the value of self_reported_ethnicity is "European, Hispanic or Latin American".

Note: The end-to-end modeling is currently blocked when there are a multiplicity of values.

schema v4.0.0

obs (Cell metadata)

Updated self_reported_ethnicity_ontology_term_id
Updated self_reported_ethnicity

Context
To support accurate metadata for multiethnic donors (donors who have selected more than one ethnicity),self_reported_ethnicity must allow multiple HANCESTRO terms.

2 Collections use multiethnic currently so those will be replaced by a list of multiple HANCESTRO terms

https://cellxgene.cziscience.com/collections/7d7cabfd-1d1f-40af-96b7-26a0825a306d (15 donors)
https://cellxgene.cziscience.com/collections/1ca90a2d-2943-483d-b678-b809bf464c30 (3 donors)

In Lattice DB, yet to get to CxG, there are 26 donors in another study that would have multiple terms

For multiple terms, are we OK with comma+space separated, like the current examples? And/or comma only?

Are we prepared for a comma within a single label? HANCESTRO:0015 is Greater Middle Eastern (Middle Eastern, North African or Persian). Is there another delimiter we could use to avoid any parsing confusion/errors?

The order of multiple terms is not significant

While I agree that curators & submitters ideally won't need to worry about it, but seems like they should be ordered for users so the Census users won't see HANCESTRO:0005, HANCESTRO:0008 & HANCESTRO:0008, HANCESTRO:0005 and cxg viewers won't see European, Asian & Asian, European, etc.

Couple typos in the URLs:

"HANCESTRO:0002" - current link is for HANCESTRO_0001
"HANCESTRO:0324" - current link is for HANCESTRO_0323

Couple typos in the URLs:

Thanks for catching. Corrected.

For multiple terms, are we OK with comma+space separated, like the current examples? And/or comma only?

I was tempted to write some ABNF earlier.

Are we prepared for a comma within a single label? HANCESTRO:0015 is Greater Middle Eastern (Middle Eastern, North African or Persian). Is there another delimiter we could use to avoid any parsing confusion/errors?

I meant to check the presence of commas in labels. Doh. This is primarily an issue because we're unable to use arrays for the list of self reported ethnicities. Possible options:

Since there's only case where a HANCESTRO label includes a comma, we could request that it be replaced by an "or"? And ask that no further comma(s) be specified?
We replace the comma with a separator that's unlikely to appear in an HANCESTRO label such as "|".

@pablo-gar @bkmartinjr @atarashansky - do you have preferences?

The order of multiple terms is not significant

While I agree that curators & submitters ideally won't need to worry about it, but seems like they should be ordered for users so the Census users won't see HANCESTRO:0005, HANCESTRO:0008 & HANCESTRO:0008, HANCESTRO:0005 and cxg viewers won't see European, Asian & Asian, European, etc.

I had an earlier draft that normalized the labels by ascending lexcographical order would then require a rewrite of their term id(s) to match. Based on my earlier draft, your example above:
HANCESTRO:0005, HANCESTRO:0008 would be normalized to HANCESTRO:0008, HANCESTRO:0005 to match the ordered labels Asian, European.
Another option is to normalize by ascending term id(s) but then the labels are more random:
HANCESTRO:0008, HANCESTRO:0005 becomes HANCESTRO:0005, HANCESTRO:0008
CC: @pablo-gar @bkmartinjr @atarashansky

RE:ordering - both options result in consistent display of the same values so I don't have a strong preference. Maybe slight preference for lexicographical order of labels (users are more likely to see the A-->Z pattern & therefore less likely to think the order means something significant?)

Re: array syntax:

prefer a single-character delimiter (not the space+comma combination used in the first example).
also prefer a delimiter which is illegal in an ontology term, so we do not have to worry about escaping it. That makes the typical query (via string regex) feasible.  OTOMH, I don't know what is legal/illegal in RDF terms (and I hope OBO follows the same?), but I'd look to that grammar to find a character that we can safely use.  Perhaps a simple space is the best choice?
I don't think the Census cares strongly about ordering unless there are semantics associated with the ordering (there are weak arguments for a (any) canonical ordering as that would help compression).  I agree that visualization tools do care, but I think they care in different ways, and can apply their own logic (i.e., it does not need to be codified in the schema).  In Jason's "cxg" (I think he means Explorer) example, it is trivial for the Explorer backend to sort these at CXG creation time.

Perhaps a simple space is the best choice?

I may be missing your point, but there are many cases where the HANCESTRO labels include spaces like "South East Asian" or consider Jason's outlier :
"Greater Middle Eastern (Middle Eastern, North African or Persian separator European"

prefer a delimiter which is illegal in an ontology term

I could see if OWL(s) have different constraints beyond XML.
What about using multiple characters such as "<br>".  There are some crazier ideas out there like scanning the source text and finding a unique character.

I've asked Dani if the one label could be updated to not use a comma. Fingers crossed.
Could we double-quote terms within single-quoted strings?
label = '"Greater Middle Eastern (Middle Eastern, North African, or Persian)", "European"'
list = (re.findall('"([^"]*)"', label))
which produces a helpful list:
['Greater Middle Eastern (Middle Eastern, North African, or Persian)', 'European']

I may be missing your point, but there are many cases where the HANCESTRO labels include spaces like "South East Asian" or consider Jason's outlier :

You were not - I was myopically focused on the term id syntax, and forgot about the labels.  Sigh. We may simply have to escape labels, although that will make any regex-based search interesting

Everything looks good to me. Great job on going over the HANCESTRO ontology and excluding terms.
Only two points:

I propose we use ; as a separator.
We should include the curated spreadsheet at the very least in this ticket and ideally in the schema. For that spreadsheet I suggest naming the tab as "HANCESTRO_version..." to know exactly what version it is.

We should include the curated spreadsheet at the very least in this ticket and ideally in the schema. For that spreadsheet I suggest naming the tab as "HANCESTRO_version..." to know exactly what version it is.

Can you say more about the rationale? The schema pins a HANCESTRO version. The requirements for self_reported_ethnicity_term_id ensure that only terms appropriate for self reporting can be selected by curators. (There are 290 appropriate terms after the filtering) I'm happy to link the spreadsheet to the issue if it's helpful for others to check the math.
For the future, we've asked Dani to consider adding a property that would allow HANCESTRO consumers to easily filter terms that are appropriate for self reporting rather than requiring the manual review and cherry picking of terms.

After reviewing with Dani, I opened Preferred term label for HANCESTRO:0015.

And since I'm knee deep in clarifying requirements for #216 -  any reordering of terms/labels could impact _colors if a submitter was coloring by this field.

Can you say more about the rationale? The schema pins a HANCESTRO version. The requirements for self_reported_ethnicity_term_id ensure that only terms appropriate for self reporting can be selected by curators. (There are 290 appropriate terms after the filtering) I'm happy to link the spreadsheet to the issue if it's helpful for others to check the math.
For the future, we've asked Dani to consider adding a property that would allow HANCESTRO consumers to easily filter terms that are appropriate for self reporting rather than requiring the manual review and cherry picking of terms.

Mainly a historical record of the decisions that were made, it can come in handy for when explaining the decisions and or in the future when we have to revise HANCESTRO.


preventOverlayInteraction doesn't remove all blocking listeners

Hi! It appears that even with preventOverlayInteraction set to false, this package still applies blocking dom listeners that prevent clicks, focusin, etc.. The following workaround works locally, however in my production app these event listeners get wrapped by sentry and I'm unable to intercept them the same:
 const originalAddEventListener = document.addEventListener;
    document.addEventListener = function (type, listener, options) {
      const listenerStr = listener.toString();
      if (
        (type === 'click' && listenerStr.includes('checkClick')) ||
        (type === 'touchstart' && listenerStr.includes('checkPointerDown')) ||
        (type === 'mousedown' && listenerStr.includes('checkPointerDown')) ||
        (type === 'keydown' && listenerStr.includes('checkKey')) ||
        (type === 'focusin' && listenerStr.includes('checkFocusIn'))
      ) {
        return;
      }
      originalAddEventListener.call(this, type, listener, options);
    };

Is there a reason these blocking dom listeners are still being added?
Thank you!

I see now that this is from using focusTrap in the step component. In my opinion, this should be turned off if preventOverlayInteraction is false.

Same issue here, when trying to close or show an element on the GUI for the next step, I try to simulate a click programatically but it fails as the events are prevented on the overlay. This even when setting 'preventOverlayInteraction' to false.

Hi everyone! 👋
This issue has been resolved and included in v2.9.0.
Feel free to reopen if you encounter any issues. Thanks for your patience! 🙌


cargo-size UI

This issue is about improving the UI of the cargo-size subcommand to make it feel more integrated
with Cargo. Possible additions:

cargo size [--target $T] --example $E [--release] -- $ARGS. Builds $E for $T and then calls
llvm-size $ARGS target/$T/{debug,release}/examples/$E.

cargo size [--target $T] --examples [--release] -- $ARGS. Builds all the examples and then
calls llvm-size $ARGS target/$T/{debug,release}/examples/*.

Thoughts? Suggestions?

Print size in radix:
-o                - Print size in octal
-d                - Print size in decimal
-x                - Print size in hexadecimal

Addresses should be always in hexadecimal form.
Right now it looks awkward:
section       size         addr
.text       171656<PHONE_NUMBER>
.rdata       49016<PHONE_NUMBER>
.data<PHONE_NUMBER>400
.pdata       11340<PHONE_NUMBER>
.reloc<PHONE_NUMBER>784
Total       233252

Do not forget about target-dir.
This path can be changed in .cargo/config

v0.1.2 implements --bin --example, --lib, --release and --target flags which can be used to build a binary / example / library and inspect the build artifact -- no need to pass the path to the artifact as an argument. Examples.
if there's any other UI improvement you'd like to see please open a new issue. I'm closing this one as "done".

Do not forget about target-dir.

Yeah, I totally forgot. :sweat_smile:
I have opened a help wanted issue w/ instructions if you fill like fixing this! #24


Dashboard fails if queue does not exist

startup the crawler without events
startup the dashboard
notice that on the crawler machine the stack below shows up every 5 seconds

The fundamental issue here is that QueueSet.getQueue() throws if the queue is not known.  It could return null.  If it did, we'd have to make sure all the callers check.  Alternatively, queues route could try/catch and return a 404.
trackException:
Error: Queue not found: events
    at QueueSet.getQueue (C:\git\ghcrawler\lib\queueSet.js:86:13)
    at CrawlerService.getQueueInfo (C:\git\ghcrawler\lib\crawlerService.js:84:39)
    at c:\git\ospo-ghcrawler\routes\queues.js:20:37
    at next (native)
    at Function.continuer (c:\git\ospo-ghcrawler\node_modules\q\q.js:1278:45)
    at Q.spawn (c:\git\ospo-ghcrawler\node_modules\q\q.js:1305:16)
    at module.exports (c:\git\ospo-ghcrawler\middleware\promiseWrap.js:9:5)
    at Layer.handle [as handle_request] (c:\git\ospo-ghcrawler\node_modules\express\lib\router\layer.js:95:5)
    at next (c:\git\ospo-ghcrawler\node_modules\express\lib\router\route.js:131:13)
    at validate (c:\git\ospo-ghcrawler\middleware\auth.js:16:12)
    at Layer.handle [as handle_request] (c:\git\ospo-ghcrawler\node_modules\express\lib\router\layer.js:95:5)
    at next (c:\git\ospo-ghcrawler\node_modules\express\lib\router\route.js:131:13)
    at Route.dispatch (c:\git\ospo-ghcrawler\node_modules\express\lib\router\route.js:112:3)
    at Layer.handle [as handle_request] (c:\git\ospo-ghcrawler\node_modules\express\lib\router\layer.js:95:5)
    at c:\git\ospo-ghcrawler\node_modules\express\lib\router\index.js:277:22
    at param (c:\git\ospo-ghcrawler\node_modules\express\lib\router\index.js:349:14)

Resolved by https://github.com/Microsoft/ghcrawler/pull/67


Why use onbuild image for python?

onbuild won't pass the docker context.
Pip requires --proxy to be set or env vars when behind a proxy. Using onbuild prevents the dockerfile from being built behind a proxy.

Ah, onbuild is required to execute further statements in the python image? That's a shame, as it doesn't pass env vars...


pisa currently incompatible with python 3.11 and 3.12

One might be inclined to use the above python releases because the installation guide states that "you can choose your preferred version of python >= 3.10". This is followed by the example shell command conda create -n pisa_env python=3.10.
However, as it stands, the installation fails under python 3.12 and python 3.11. This is probably most easily avoided right now by removing the ambiguity in the python version requirements. In any case, below are some currently existing incompatibilities that prevent using each version (there are probably more).
python 3.12

 versioneer.py is too old (see https://docs.python.org/3/whatsnew/3.12.html#configparser). Is there a reason why we can't ship a newer version of versioneer which supports python 3.12 as well?
numpy only supports python 3.12 starting with v1.26.0, but we require < 1.23

python 3.11

pip seems to be trying to build scikit-learn <= 1.1.2 from its source distribution because there are no wheels for python 3.11 (only for scikit-learn >= 1.1.3), and I'm getting some Cython.Compiler.Errors.CompileError apparently when some extensions are being compiled

there is no conda package for python 3.11 either (check https://anaconda.org/conda-forge/scikit-learn/files?version=1.1.2 for example)

numpy only supports python 3.11 starting with v1.23.2, but we require < 1.23

Hey, thanks a lot for the detailed issue! Yes, I also saw that those install instructions were a bit imprecise, they were very recently set up by some newer collaborators.. Making the requirements more specific sounds like a very good idea to stop people from struggling with the installation.
In any case, if you are building in an environment where cvmfs is available, I suggest do use the instructions I've set up and tested for a specific python version there.

Notes on upgrading versioneer: since we're using the "vendored mode", it doesn't look like we'd need to change anything before upgrading from version 0.16 to the latest release, 0.29, which seems to (unofficially) support python 3.12.
I might test upgrading locally and report back.


Raspberry Pi 4 and raspistill breaking with mmal issues

Tested 2.41.0+rev4 Raspberry Pi4 with raspistill, it is not snapping pictures, and does these errors:
24.09.19 11:50:16 (+0100)  camera-new  snapping a picture via raspistill -o /data/test_new_20190924_105016.jpg
24.09.19 11:50:16 (+0100)  camera-new  mmal: mmal_vc_shm_init: could not initialize vc shared memory service
24.09.19 11:50:16 (+0100)  camera-new  mmal: mmal_vc_component_create: failed to initialise shm for 'vc.camera_info' (7:EIO)
24.09.19 11:50:16 (+0100)  camera-new  mmal: mmal_component_create_core: could not create component 'vc.camera_info' (7)
24.09.19 11:50:16 (+0100)  camera-new  mmal: Failed to create camera_info component

Flowdock conversation

Actually, seems like it still takes a picture properly, so this is more of a warning? But unsure where is it coming from, or how to fix it yet.

i had similar problem with balenaOS 2.73.1+rev1 on rpi4 when i execute camera = PiCamera() from python:
mmal: mmal_vc_shm_init: could not initialize vc shared memory service
mmal: mmal_vc_component_create: failed to initialise shm for 'vc.camera_info' (7:EIO)
mmal: mmal_component_create_core: could not create component 'vc.camera_info' (7)
Traceback (most recent call last):
File "src/main.py", line 21, in
camera = PiCamera()
File "/usr/local/lib/python3.6/site-packages/picamera/camera.py", line 367, in init
with mo.MMALCameraInfo() as camera_info:
File "/usr/local/lib/python3.6/site-packages/picamera/mmalobj.py", line 2346, in init
super(MMALCameraInfo, self).init()
File "/usr/local/lib/python3.6/site-packages/picamera/mmalobj.py", line 633, in init
prefix="Failed to create MMAL component %s" % self.component_type)
File "/usr/local/lib/python3.6/site-packages/picamera/exc.py", line 184, in mmal_check
raise PiCameraMMALError(status, prefix)
picamera.exc.PiCameraMMALError: Failed to create MMAL component b'vc.camera_info': I/O error
the rpi4 was working properly for 5 days straight when suddenly it stopped after pushing new code without updating Dockerfile.template.


Docstring cleanup

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated

After merging to master and before closing the branch:

[ ] bumpversion (minor / major / patch) has been called on master branch
[ ] Tags have been pushed

What kind of change does this PR introduce?

Does this PR introduce a breaking change?

Other information:

This PR fixes #xyz

Pull Request Test Coverage Report for Build 1154

74 of 75 (98.67%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.8%) to 89.615%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xclim/utils.py
20
21
95.24%

Files with Coverage Reduction
New Missed Lines
%

xclim/indices/_multivariate.py
1
99.43%

Totals

Change from base Build 1146:
-0.8%

Covered Lines:
1467

Relevant Lines:
1637

💛  - Coveralls


InputPhone does not render submitted value on update

Describe the bug
XHTML:
<h:form id="main" enctype="multipart/form-data">

    <pe:inputPhone id="str1"
                   widgetVar="str1"
                   value="#{test.str1}"
                   nationalMode="false"
                   initialCountry="nl"
                   placeholder=" ">

    </pe:inputPhone>

      <f:selectItem itemLabel="1" itemValue="1"/>
      <f:selectItem itemLabel="2" itemValue="2"/>
      <f:selectItem itemLabel="3" itemValue="3"/>

</h:form>

Bean:
@ViewScoped
@Named("test")
public class TestBean implements Serializable {

  private String str1;
  private String str2;

  // getter setter

}

Fill in phone, then select one of the select items. Payload:
javax.faces.partial.ajax: true
javax.faces.source: main:str2
javax.faces.partial.execute: main:str2
javax.faces.partial.render: main
javax.faces.behavior.event: valueChange
javax.faces.partial.event: change
main: main
main:str1_input: +31612345678
main:str1_iso2: nl
main:str1_hidden: +31612345678
main:str2: 2
javax.faces.ViewState: xxx

Response:
... <input id="main:str1_input" name="main:str1_input" type="tel" value="" class="ui-inputfield ui-inputtext ui-widget ui-state-default ui-corner-all" placeholder=" " onchange="this.value = PF('str1').getNumber();" /><input id="main:str1_iso2" name="main:str1_iso2" type="hidden" autocomplete="off" value="nl" /><input id="main:str1_hidden" name="main:str1_hidden" type="hidden" autocomplete="off" value="" /> ...

Reproducer
No response
Expected behavior
No response
PrimeFaces Extensions version
14.0.6
JSF implementation
Mojarra
JSF version
2.3
Browser(s)
No response

@melloware this must have something to do with order of JS events I guess. When I change the Ajax handler on the one radio to
 it does work. Not so sure now where the issue is. Here, or in the selectOneRadio JS code.

i think its also related to this: https://github.com/primefaces-extensions/primefaces-extensions/issues/1614

I just migrated from 14.0.2 to see if it was fixed in 14.0.6 😆

OK good so that issue did not break it!

It looks like there is no data posted when setting a value and blurring the phone field.
    <pe:inputPhone id="str1"
                   widgetVar="str1"
                   value="#{test.str1}"
                   nationalMode="false"
                   initialCountry="nl"
                   placeholder=" ">

    </pe:inputPhone>

I think you might have been right, as the default event now is
    @Override
    public String getDefaultEventName() {
        return EVENT_COUNTRY_SELECT;
    }

which is weird IMO

yep

Ok, for now I'll use the explicit event="valueChange" as a workaround. The proper fix would be having one hidden field containing both the phone number and the country IMO.


Externalize macro-related targets

Description
The DependenciesMacros target requires the swift-syntax package, and the number of projects using this package is growing fast.
The main issue is that each library has different swift-syntax version constraints and it can become difficult to use libraries together if they aren't maintained at the same speed.
For anyone who doesn't use DependenciesMacros, getting blocked by it at packages resolution can be quite frustrating 😅
For this reason I believe anything related to macros should be opt-in instead of forced:

it could find its place in a separate repository
or gated behind some flags in the Package.swift — if that's possible (but I don't think it is? except with environment variables maybe? but it would be annoying again to setup when working with Xcode)

I would be glad to contribute to the effort.

The fewest dependencies, the better.

Checklist

[ ] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[ ] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
Being able to resolve dependencies without being impacted by targets that are optional and unused.
Actual behavior
Package resolution fails for swift-syntax because of conflicts with other libraries.
Steps to reproduce

Create a new project
Add swift-dependencies
Add swift-syntax version 600 or later (or any library using swift-syntax with incompatible version constraints, i.e. swift-testing 0.8 or later)
Witness package dependencies resolution fail 😈

Dependencies version information
No response
Destination operating system
Any
Xcode version information
No response
Swift Compiler version information
swift-driver version: 1.109.2 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.300 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0

Hi @abidon, our dependence on SwiftSyntax is quite relaxed:
https://github.com/pointfreeco/swift-dependencies/blob/76b98dea40d2b13ace67216cbb100cef673e8267/Package%40swift-5.9.swift#L25
Which version of SwiftSyntax are you targeting? Is it some pre-release version for Swift 6?
It would be quite a breaking change to do this, and it would be a lot more annoying to maintain. It may be more preferable for us to maintain pre-releases branches ourselves that match SwiftSyntax's branches. But before making any decisions it would be good to get more information on your situation.
Since this isn't an actual bug with the library I am going to convert it to a discussion. Let's continue the conversation over there.


Sorry, your request failed. Please report the following request id b8a1bcdb-fb9a-4671-bb11-a233762b19bf and try again

Sorry, your request failed. Please report the following request id b8a1bcdb-fb9a-4671-bb11-a233762b19bf and try again

The editor could not be opened due to an unexpected error: Failed to get model for chat editor. id: afb7461d-88f7-4bea-9af7-4538f2e708a6

Can you include your logs please via CTRL + SHIFT + U -> GitHub Copilot Chat. Or on mac CMD + SHIFT +U -> GitHub Copilot Chat


[FLINK-11388][fs] Add Aliyun OSS recoverable writer

What is the purpose of the change
OSS offers persistence only after uploads or multi-part uploads complete. In order to make streaming uses OSS as sink, we should implement a Recoverable writer. This writer will snapshot and store multi-part upload information and recover from those information when failure occurs.
Brief change log
- move utility classes to flink-fs-common
- add abstract test classes for recoverable writers(s3/oss)
- add OSS recoverable writer
Verifying this change
This implementation adds some tests to test logic of recoverable writer. However, in order to run these tests, someone need to have his own Aliyun access key id and access key secret. Then, set environment variables below：
export ARTIFACTS_OSS_ENDPOINT=<Aliyun OSS endpoint to connect to>
export ARTIFACTS_OSS_BUCKET=<Your bucket>
export ARTIFACTS_OSS_ACCESS_KEY=<Your access key id>
export ARTIFACTS_OSS_SECRET_KEY=<Your access secret>
These tests are skipped by default.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks @wujinhu , I will take a look~

You can create a JIRA for updating document in: https://nightlies.apache.org/flink/flink-docs-release-1.14/docs/connectors/datastream/filesystem/

@flinkbot run azure


Can I use both icon and button text on ActionButton?

I want two icons to be combined (as in twitter tweet button) on ActionButton. How can I do this ? Can anyone help me in this?

you can probably make this happen using the prop icon on ActionButton


How to set API Key and API Secret to clients?

I want to use Apigility to create an API that several clients will consume.  What I need to do is something like Facebook does: people can create apps that connects to my API,  and I can see what apps are connected.
Also I would like to be able to limit services for some apps.  Something like app A can online view statuses,  App B can view and post and edit statuses,  App C can only edit statuses for examples.
Is that possible with apigility? Where can I find documentation for the API Keys and Secret?
In other words I want application based authorization instead of user based authorization that Oauth2 does.

I have the same question

There is a client_credentials grant type in OAuth2.  If you're asking about scopes I think my api-skeletons/zf-oauth2-doctrine supports them best.

This repository has been closed and moved to laminas-api-tools/api-tools; a new issue has been opened at https://github.com/laminas-api-tools/api-tools/issues/26.


Cannot uninstall a deprecated extension from the editor viewlet

Testing #150746

Look for deprecated extensions by typing deprecated in the extensions search menu
Install tyriar.terminal-tabs or glen-84.sass-lint via the extensions viewlet button
Note that there is no Uninstall button afterwards, you have to open the extension page and click Uninstall from there.

Uninstall action is available in the drop down when you click the gear icon


The proposal of reposting closed issues on Alibaba Cloud Developer Community.

Dubbo 的项目组大家好：
Hello, Dubbo team,
我是阿里云开发者社区的运营人员。Dubbo在全球有众多开发者，他们在GitHub上提问并得到了项目成员的回复，这对开发者学习Dubbo无疑是很大的帮助。作为中国的开发者社区，阿里云开发者社区也希望能将这部分问答翻译并搬运到阿里云开发者社区，以便中国的开发者更好地学习并使用Dubbo。以下是我们对此项合作的一些想法：
I am from Alibaba Cloud Developer Community. Dubbo is loved and used by developers around the world, and they ask questions on GitHub and get replies from the team members. This is undoubtedly a great help for developers to learn Dubbo. As a developer community in China, the Alibaba Cloud Developer Community hopes to translate the questions  and replies of Dubbo and repost them on Alibaba Cloud Developer Community, so that Chinese developers can better learn and use Dubbo. Here are the proposal

由我方找专门的人员进行翻译、发布工作，以 closed issues 中带 question label 的问答作为目标内容进行整理。
We will find specialized personnel to translate questions from closed issued tab and repost on them on  Alibaba Cloud Developer Community;

我们会持续关注项目，后续新增的 closed issues 我们也会持续同步沉淀至开发者社区，以便开发者学习到最新的内容。
We will continuously pay attention to the project to keep updated the latest content;

尊重知识产权，整理发布到开发者社区上的提问与回答都会标注原作者的GitHub 昵称。
We will give credit to the original author by showing their GitHub nicknames on the repost threads.

作为技术社区，我们非常希望Dubbo在中国得到广泛的应用，让中国的开发者第一时间获取到Dubbo的知识。分享、惠普、利他，让更多人学习成长，相信这也是项目开源的初衷。如果项目组对我们对本次的合作方案感兴趣或者有其他想法及建议，请给我留言，期待让更多的中国开发者学习Dubbo。
As a technical community, we hope that Dubbo will be widely used in China, and Chinese developers can obtain the knowledge of Dubbo in the first place. Let more people learn and grow in sharing, I believe this is also the spirit of open source. If the team are interested in our proposal or have other ideas and suggestions, please feel free to leave a comment. I am looking forward to spreading Dubbo in China.
Sofia

sounds great😃


Fixed #30335 -- Fixed crash when ordering or aggregating over a deep JSONField key transforms.

Ticket: #30335

hi, @felixxm thanks. changed test and added note.

Have I should added other related tickets (29139, 28762, 30085) into doc as a note?

30085 is a closed duplicate so definitely no.
For 29139 and 28762 I will try later to push tests that will confirm that.


Update SABS with notSABS

I still run SABS from months ago. If I use notSABS to Update, will it update SABS? So, I can use the same package name as with SABS?

If I remember correctly, yes, I did update SABS to notSABS.
Rename the app using apk editor first.


Fix text changes on DR pages

Also fixing  text and some minor bugs

These kind of breaks are not needed IMO, we don't have to exactly follow the UX (line-by-line). We should only control the max/min width of the popover and use text alignment styles, but let things render based on user's screen size, instead of explicitly breaking the line in mid text ! It is okay to break for new paragraph etc, but not between continuous lines/texts.

make sense!

/approve
/lgtm


⚠ Bofrost.hr has degraded performance

In ee902c9, Bofrost.hr (https://www.bofrost.hr) experienced degraded performance:

HTTP code: 200
Response time: 4890 ms

Resolved: Bofrost.hr performance has improved in 67902bc.


Build error on WSL2 (Ubuntu)

Hello,
I've tried to build the tutorials using WSL2 and Ubuntu but I got some errors.
xxx@PC1686:/home/gltut$ sudo mkdir build && cd build
xxx@PC1686:/home/gltut/build$ sudo cmake ..
-- The C compiler identification is GNU 9.4.0
-- The CXX compiler identification is GNU 9.4.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found OpenGL: /usr/lib/x86_64-linux-gnu/libGL.so
-- Configuring done
-- Generating done
-- Build files have been written to: /home/gltut/build
xxx@PC1686:/home/gltut/build$ sudo cmake --build . --config Release
[  0%] Building CXX object glsdk/CMakeFiles/glload.dir/glload/source/gl_load_cpp.cpp.o
[  1%] Building C object glsdk/CMakeFiles/glload.dir/glload/source/gl_load.c.o
[  2%] Linking CXX static library libglload.a
[  2%] Built target glload
[  3%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/DdsLoader.cpp.o
[  3%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/ImageCreator.cpp.o
[  4%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/ImageFormat.cpp.o
[  5%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/ImageSet.cpp.o
[  5%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/ImageSetImpl.cpp.o
[  6%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/StbLoader.cpp.o
[  6%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/TestLoader.cpp.o
[  7%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/TextureGenerator.cpp.o
[  8%] Building CXX object glsdk/CMakeFiles/glimg.dir/glimg/source/Util.cpp.o
[  8%] Building C object glsdk/CMakeFiles/glimg.dir/glimg/source/stb_image.c.o
[  9%] Linking CXX static library libglimg.a
[  9%] Built target glimg
[  9%] Building CXX object glsdk/CMakeFiles/glutil.dir/glutil/source/Debug.cpp.o
[ 10%] Building CXX object glsdk/CMakeFiles/glutil.dir/glutil/source/Font.cpp.o
[ 10%] Building CXX object glsdk/CMakeFiles/glutil.dir/glutil/source/MatrixStack.cpp.o
[ 11%] Building CXX object glsdk/CMakeFiles/glutil.dir/glutil/source/MousePoles.cpp.o
[ 12%] Building CXX object glsdk/CMakeFiles/glutil.dir/glutil/source/Shader.cpp.o
[ 12%] Building CXX object glsdk/CMakeFiles/glutil.dir/glutil/source/WindowPos.cpp.o
[ 13%] Building CXX object glsdk/CMakeFiles/glutil.dir/glutil/source/ftData.cpp.o
[ 14%] Linking CXX static library libglutil.a
[ 14%] Built target glutil
[ 14%] Building C object glsdk/CMakeFiles/freeglut.dir/freeglut/src/freeglut_callbacks.c.o
In file included from /home/gltut/glsdk/freeglut/include/GL/freeglut.h:17,
from /home/gltut/glsdk/freeglut/src/freeglut_callbacks.c:28:
/home/gltut/glsdk/freeglut/include/GL/freeglut_std.h:121:10: fatal error: GL/glu.h: No such file or directory
121 | #include <GL/glu.h>
|          ^~~~~~~~~~
compilation terminated.
make[2]: *** [glsdk/CMakeFiles/freeglut.dir/build.make:76: glsdk/CMakeFiles/freeglut.dir/freeglut/src/freeglut_callbacks.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1786: glsdk/CMakeFiles/freeglut.dir/all] Error 2
make: *** [Makefile:91: all] Error 2
As far as i've understood from the others issues, that could be freeglut which is the problem.
I've done xxx@PC1686:/home/gltut/glsdk/freeglut$ sudo sh ./configure manually.
But still I got the same error.
CMake version is 3.23.2
g++ version is (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
----- Edit 02/08
I've tried with a clean ubuntu version 20.04
It doesn't work neither
The only package that was intalled were sudo, git, cmake, make, g++ and libgl1-mesa-dev.
----- Edit 03/08
I've installed libglfw3-dev libgl1-mesa-dev
Now the error is
/home/gltut/glsdk/freeglut/src/freeglut_internal.h:111:14:
fatal error: X11/extensions/XInput.h: No such file or directory
111 | #    include <X11/extensions/XInput.h>
|              ^~~~~~~~~~~~~~~~~~~~~~~~~
Does someone have an idea ?

I've success using:
sudo install x11proto-input-dev
and
cd /usr/include/X11/extensions && sudo ln -s XI.h XInput.h
source: https://melp.nl/2013/11/building-opengl-sdk-compile-error-x11-extensions-xinput-h/


Fix error when running in nw.js

Hi Petka,
I noticed an issue with running the latest version inside  nw.js (node-webkit). In schedule.js process.versions.node is assumed to exist. In node-webkit, however, it doesn't so the call to process.versions.node.split(".") fails.
Here's a pull request which I believe fixes the issue. In nw.js process.versions["node-webkit"] is used instead. As you can see from the nw.js changelog (https://github.com/nwjs/nw.js/blob/master/CHANGELOG.md), v0.9.0 of nw.js uses Node.js v0.11 so the new nwjs-related check compares against version 0.8 of nw.js.
Cheers,
Simon

Sorry, yes, I don't think that it the best way to check for the existence of the version. I've modified the line to do what I think is a better check?

This is too verbose, perhaps
var version = process.versions.node || process.versions["node-webkit"];
version = version.split(".").map(Number);

I cannot find any reason to believe this is true in nw codebase, in fact https://github.com/nwjs/nw.js/blob/c636f94c0e0cbdfd52fc89d095414585f6a16275/src/resources/pages/nw_version.html
closing

I've just checked again and yes, you're completely right, the fix isn't needed. I've realised what's going on... I had based the app of requirejs-nodewebkit-template-project and, if you look at app.js in that project you can see it deletes process.versions.node.
Sorry!
Simon


Update to Version 1.4.1 (32) has all the language set to cyrillic characters

I've updated today to the newer version but now all the text is set to cyrillic characters instead of English (or latin characters) - I'm on MacOS Sonoma

Hey @rna84 :)
What language do you have set in System Settings? (Open System Settings -> General -> Language & Region)

I think I've fixed it, I had Russian set on it - now by setting another language is everything fine. Thanks!


Question about Rexray/cinder

Hi,
Do you know if it is possible to use the rexray cli to do snapshot of volume  and create volume from snapshot ?
regards

@fabio-barile This was possibly under the 0.3.3 release, but in the 0.4+ release we stopped including this functionality. At that time the focus was to bring more drivers on and ensure stability of features most used. Snapshots is getting more popular so we will be thinking about where to re-include this functionality.


Fix handling of stop message in couch_changes callback

At one time this callback was handling fabric:changes which sends pending
changes. Then code was optimized to use local shard changes but shard changes
callback doesn't get called with pending info.
end_sending_changes(Callback, UserAcc, EndSeq, ResponseType) ->
    Callback({stop, EndSeq}, ResponseType, UserAcc).

Related to:
https://github.com/apache/couchdb-couch-replicator/pull/45

LGTM


Use GitHub API Token in travis-ci builds, closes #9

Codecov Report

Merging #10 (3537197) into main (a0450fa) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #10   +/-   ##
=========================================
  Coverage     95.23%   95.23%
  Complexity      463      463
=========================================
  Files           129      129
  Lines          1007     1007
  Branches         32       32
=========================================
  Hits            959      959
  Misses           42       42
  Partials          6        6

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


URI matching issue

Two apis are created as following:

api1 ==> uris: /
api2 ==> uris: /aa

When "http://host/aaaa/index.html" is browsed, the matched api is api2.

Hi,
This is expected behaviour. Defining uris like so treats them as prefixes, and /aa is a prefix of /aaaa.
For more details, see https://getkong.org/docs/latest/proxy/#request-path

When proxying with path prefixes, the longest paths get evaluated first. This allow you to define two Routes with two paths: /service and /service/resource, and ensure that the former does not "shadow" the latter.

Alternatively, you can use regexes to enforce /aaaa does not match:
$ curl -i -X POST http://localhost:8001/apis \
    --data-urlencode 'uris=^\/aa\/'

By the way, APIs are deprecated since Kong 0.13.0. I am not sure which version of Kong you are using, but you can check the documentation for Services & Routes at: https://getkong.org/docs/0.13.x/admin-api/#service-object

Thank you very much!


Error with Packer (before adding Expansion)

the strange error, come sometimes. I'll just ignore it for now and implement the other stuff for now, but would be cool if we could fix it, so it becomemore stabile. :)

Is this duplicate of #10 ?

no it's something else. But now it's fixed :)

Failed to build issue due: disconnected -> fixed now (change the reboot to shutdown + wait)
Failed to build issue due: sometimes /tmp/util_fn not found -> fixed now (change the folder now to /home/tmp/)

After Merge Packer folder with another branch, the issue should be fixed.

Failed to build issue due: sometimes /tmp/util_fn not found -> fixed now (change the folder now to /home/tmp/)

Just realized that it is an actually an expected behavior. /tmp is cleaned after reboot, so you need to copy the file again after reboot if using /tmp directory


Broken repo link on 'Before running the sample for the first time'

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [x] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
When you follow the steps on the 'Before running the sample for the first time' on:
#2. git clone https://github.com/Azure/Communication.git
That git repo, no longer contains the calling samples as of Oct 22, 2020.
Any log messages given by the failure
No error message, just the calling samples are missing.
Expected/desired behavior
I expect the calling sample to be included when I clone that repo
OS and Version?
not applicable
Versions
not applicable
Mention any other details that might be useful

Thanks! We'll be in touch soon.

Sorry about that. They have been since migrated to this location.
https://github.com/Azure-Samples/communication-services-web-calling-hero
I'll be working to fix up the documentation. Thanks for raising this issue

Should be solved with this 14a0cd372077a50c77ff667f9e41e7b12fb56f86
I believe this should successfully solve this issue?


NWPS-851 - Website Testing Channel Creation - initial commit

hi @Raq-HEE , @adaMiclos, @dawntaroni
Please review this

Hi @sankar-shunmuga, @adaMiclos - I've just pushed a commit to enable authentication, which was one of the requirements we missed. Can you review please.


pending payment error when i create transaction

Issue Description
I create an order with automatic invoice and payment at the door and add a payment from the transactions page, but the status remains as pending payment and does not change.
I got the error in the video, you can check it
https://drive.google.com/file/d/1b8vYqLHFJDQmCyIAZy6kaxoLAkVKCU7M/view?usp=sharing
Preconditions

I create an order as a customer, I choose the payment method, payment at the door
I add process in admin panel
invoice status pending payment

Bagisto version

[ ] master branch
[ [] ] 1.3.2
[ ] 1.3.1
[ ] 1.3.0

local.ERROR: Call to undefined method Webkul\Product\Type\Configurable::evaluatePrice() (View: /home/ardenoff/public_html/resources/themes/velocity/views/products/price.blade.php) {"view":{"view":"/home/ardenoff/public_html/packages/Webkul/Product/src/Type/Configurable.php","data":[]},"exception":"[object] (Facade\Ignition\Exceptions\ViewException(code: 0): Call to undefined method Webkul\Product\Type\Configurable::evaluatePrice() (View: /home/ardenoff/public_html/resources/themes/velocity/views/products/price.blade.php) at /home/ardenoff/public_html/packages/Webkul/Product/src/Type/Configurable.php:580)
[stacktrace]

Hi @Arsivpro,
I just checked in the fresh instance and it is not reproducible. Do you upgrade the project or using the fresh instance.

@devansh-webkul I am using the project you see. I complete the code update with the incoming commit update

@Arsivpro,
The error evaluatePrice() undefined which you are saying, its already present and i am not able to reproduce this,
https://github.com/bagisto/bagisto/blob/7cd34d0a03ef7b4d0794fcec179971f1ae508075/packages/Webkul/Product/src/Type/AbstractType.php#L802

@devansh-webkul Are you saying it's a bug that's being worked on?

No, I am saying it's not reproduced what you are saying. It looks like you have upgraded your version.

@devansh-webkul this is correct i upgraded i am on latest version

Yes, check whether all changes are present or not. Maybe some small issue is there.

After updating this page, there were a few problems, I think the problem is here,
1st error tax 18%, price 1.6949 should appear price 2, 1.99 rounding error seen
The second error is that it shows the pending payment when the transaction is added to the invoice.
https://github.com/bagisto/bagisto/blob/7cd34d0a03ef7b4d0794fcec179971f1ae508075/packages/Webkul/Product/src/Type/AbstractType.php

In the latest version, we have implemented several features like tax enhancement. I can best suggest you take a replica of the fresh latest version also. If a bug is found in the latest version then let us know.

@devansh-webkul  I did the tax updates without any errors, even I thought there would be a problem and ran it by installing a different bagisto project, but there is a problem with rounding.
Example I entered the product with 18% tax, the price is 1.6949, on the front end it should appear: 2, visible: 1.99

Hi, can you just verify here also,
https://bagisto-speed.webkul.com/

@devansh-webkul You can view this product, tax 18% price 1.6949 should appear $2, display $1.99
https://bagisto-speed.webkul.com/sunglasses

The product price is $1.69.
The 18% of $1.69 is  $0.30.
Total: $1.69 + $0.30 = $1.99
And the default it is rounded to 2 decimal place so it can't be $2.

@devansh-webkul open the calculator and do 2/1.18 and see the result, each of my products is moving in this direction, I write the fraction as 5 numbers, actually 18% of 2 is 0.36

@devansh-webkul but instead of doing 2*18 % + = in tax calculations, I do 2/0.18 because the result is rounded

Results are rounded off to two. The calculations are showing right as per the calculations I can't do anything on this.

@devansh-webkul Your opinion is important to me. Any suggestions on how to write 2$ with 18% tax calculated?

I understand your concern I will update you regarding the same.

@devansh-webkul ok i'm waiting on the issue

Well, I just tested everything rounded off reverse can't be possible because of 18 per cent tax and the round of 2 and this is default behaviour either you need to customize something to meet your need.
Even in the calculator, it's going $1.999995. So logically  $1.99 is right.

@devansh-webkul  Bagisto system does not use 2*18%-= when calculating tax, instead it uses 2/1.18 system, so it is a bit unlikely that my calculation is wrong because it only happens on some products, 6 products out of 200 have this error

Alright, I am closing this as I haven't seen anything. For you, you can customize it, if you want. For discussion, you can use the Bagisto forum.


Make FX2 bus arbiter prevent starvation

Use a leaky bucket algorithm to limit per-FIFO bursts to 16 bytes (or a configurable value).

Actually, the FX2 arbiter does not cause starvation. It only looked like it did because of a deadlock in the host code, which was fixed in e59730f.

Though, that being said, the arbiter is not particularly fair, but it seems pretty much sufficient for now.


Feature Request: Python API Extension for Automatic Conversion of Enclosed Surfaces to Volume

Cadwork supports manual conversion of surfaces that form a closed volume into a volume.  To enhance the functionality and efficiency of Cadwork's Python API, I propose the addition of a new function, tentatively named "convert_multiple_surfaces_to_volume", which would programmatically perform this conversion.
convert_multiple_surfaces_to_volume(elements: List[int]) -> int
Parameters:
elements: A list of element IDs representing the surfaces that enclose a volume. This parameter is required and does not have a default value.
Returns:
The function returns a new element ID that represent the generated volume.

Is this function available? I see it's in the documentation but I get an error. I use cadwork30 and there's no update available.
                print(ids)                 roof_id = ec.convert_surfaces_to_volume(ids)

Looks like your build is too small.
Please contact your cadwork support team to get the SP2024 version.


[fix] 1714 rest params in endpoint

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] fixes #1714
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Thank you. The answer is both simpler and more complicated than this — simpler in that we can just allow any rest segment to have a prefix/suffix (as I believe was the case with Sapper), and more complex in that we need to tweak the implementation a bit. Closing this in favour of #3240.


replace pattern and catalog endpoint trim params with the populate params

Notes for Reviewers

This PR fixes #

Signed commits

[ ] Yes, I signed my commits.

@aabidsofi19 for mesherycil i make it as a populate=pattern_file for apply, onboard and view command.

@amitamrutiya this PR is the root cause for #13042 I did a PR can you take a look please


Mohammad moving adaptive X-over-n-mutation

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test mac on ee02ae1 : invalidated by @Jimmy-INL

Job Mingw Test on ee02ae1 : invalidated by @Jimmy-INL

Job Test Ubuntu 18 PIP on b32986b : invalidated by @Jimmy-INL

Job Test Ubuntu 18 PIP on c418b8c : invalidated by @Jimmy-INL

Job Test Ubuntu 18 PIP on c418b8c : invalidated by @wangcj05

Job Mingw Test on 6b41a5b : invalidated by @Jimmy-INL

Job Mingw Test on 6b41a5b : invalidated by @Jimmy-INL

Job Test qsubs sawtooth on 6b41a5b : invalidated by @Jimmy-INL


DebugSymbolsTest.debugSymbolsQuarkus fails with latest GraalVM master

debugSymbolsSmokeGDB fails with (both with Java 17 and Java 20):
Error:  Tests run: 2, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 724.231 s <<< FAILURE! - in org.graalvm.tests.integration.DebugSymbolsTest
Error:  debugSymbolsQuarkus{TestInfo}  Time elapsed: 543.363 s  <<< FAILURE!
org.opentest4j.AssertionFailedError:
There were errors in the GDB session. Note that commands in the session might depend on each other. Errors:
Command 'b ConfigTestController.java:33' did not match the expected pattern '.*Breakpoint 1 at .*: file com/example/quarkus/config/ConfigTestController.java, line 33.*'.
Output was:
(gdb) (gdb) (gdb) GNU gdb (Ubuntu 12.1-0ubuntu1~22.04) 12.1
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<https://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word".
 ==> expected: <true> but was: <false>
	at org.graalvm.tests.integration.DebugSymbolsTest.carryOutGDBSession(DebugSymbolsTest.java:394)
	at org.graalvm.tests.integration.DebugSymbolsTest.debugSymbolsQuarkus(DebugSymbolsTest.java:219)

Error:  debugSymbolsSmokeGDB{TestInfo}  Time elapsed: 180.857 s  <<< FAILURE!
org.opentest4j.AssertionFailedError:
There were errors in the GDB session. Note that commands in the session might depend on each other. Errors:
Command 'break Main.java:71' did not match the expected pattern '.*Breakpoint 2 at .*: file debug_symbols_smoke/Main.java, line 71.*'.
Output was:
(gdb) Breakpoint 2 at 0x55555569bcf0: file debug_symbols_smoke/Main.java, line 76.
,
Command 'c' did not match the expected pattern '.*Breakpoint 3, debug_symbols_smoke.Main::main.*at debug_symbols_smoke/Main.java:76.*String l = sc.nextLine\(\);.*'.
Output was:
(gdb) Continuing.

Thread 1 "debug-symbols-s" hit Breakpoint 2, debug_symbols_smoke.Main::main(java.lang.String[]*) (args=<optimized out>) at debug_symbols_smoke/Main.java:76
76	                String l = sc.nextLine();
,
Command 'c' did not match the expected pattern '.*Breakpoint 2, debug_symbols_smoke.Main::main.*at debug_symbols_smoke/Main.java:71.* if \(myString != null.*'.
Output was:
(gdb) Continuing.
fdc7c50f390c145bc58a0bedbe5e6d2e35177ac73d12e2b23df149ce496a5572
[Thread 0x7ffff7d68740 (LWP 9357) exited]
[Thread 0x7ffff737f640 (LWP 9358) exited]
[New process 9357]
[Inferior 1 (process 9357) exited normally]
,
Command 'c' did not match the expected pattern '.*Breakpoint 2, debug_symbols_smoke.Main::main.*at debug_symbols_smoke/Main.java:71.* if \(myString != null.*'.
Output was:
(gdb) The program is not being run.
,
Command 'c' did not match the expected pattern '.*fdc7c50f390c145bc58a0bedbe5e6d2e35177ac73d12e2b23df149ce496a5572.*exited normally.*'.
Output was:
(gdb) (gdb) The program is not being run.
 ==> expected: <true> but was: <false>
	at org.graalvm.tests.integration.DebugSymbolsTest.carryOutGDBSession(DebugSymbolsTest.java:394)
	at org.graalvm.tests.integration.DebugSymbolsTest.debugSymbolsSmokeGDB(DebugSymbolsTest.java:137)

Latest passing run was https://github.com/graalvm/mandrel/actions/runs/4878221407/jobs/8703969691 (4 days ago).
The github image runner in both cases is   Image: ubuntu-22.04 Version: 20230426.1 indicating this is not due to a change in the github runners (e.g. gdb version)

See:
https://github.com/graalvm/mandrel/actions/runs/4898986169/jobs/8748948579#step:10:19206
https://github.com/graalvm/mandrel/actions/runs/4910500117/jobs/8768028502#step:10:19184

Copying from: https://github.com/Karm/mandrel-integration-tests/issues/102#issuecomment-1538167651
The regression was introduced with https://github.com/oracle/graal/pull/6504 and more specifically by https://github.com/oracle/graal/commit/b70c6402d6b37e74e4f5c9df4f83ecfafc84c82f
It's also worth noting that https://github.com/oracle/graal/commit/00eaa8ea09a001b61346cf6c0418b62dde08d29c makes -H:-OmitInlinedMethodDebugLineInfo the default.
The above changes result in making the breakpoint appear in a different line (even with -H:+OmitInlinedMethodDebugLineInfo). Further investigation is required to see if this is an upstream issue.

The regression is due to defaulting to not using SourceMappings when building the CompilationResultTree. This choice was made in order to improve build times, but as expected it comes at the cost of less accurate generated code to source mapping.
Building the native image with -H:+DebugCodeInfoUseSourceMappings brings back the old behavior.
This doesn't look like an upstream bug, but we probably want:

to add this flag to Quarkus when creating debug builds.
test both with and without the flag

@Karm @jerboaa thoughts?

Building the native image with -H:+DebugCodeInfoUseSourceMappings brings back the old behavior.
This doesn't look like an upstream bug, but we probably want:
1. to add this flag to Quarkus when creating debug builds.

That seems fine. We'll have to add to the debugtest for the non-quarkus app as well, though.

2. test both with and without the flag

Like test it as two variations of the debug info tests? Not sure if it's worth it.

Re-opening this until the quarkus fix is in.


Fix for bug when retrieving homonym taxa names

issue #47

Also, be sure to run gulp so that the compiled javascript files are created for production deployment.

@jczaplew, I have added the braces in both locations.  isPhylo simply is not used anywhere in the entire Navigator source code. I would have left it there anyway but I was working with Michael and he just said chuck it.
I ran grunt to build everything before I pushed the branch to GitHub, do I also have to run gulp? If so, I don't have see a task runner script for that.
Having made these corrections, should I now push the branch again?

@jpjenk My bad - it is definitely grunt and not gulp.
Yep! Go ahead and push to this branch.

@jczaplew I've pushed the new commit on branch homonym_search_fix which just includes the braces. I ran grunt again before pushing.
After you do the merge, who is responsible for pulling the new master to phylum? Also does this need to be merged into the dev branch as well since that is what is running on teststrata? Thanks

@jpjenk - Looks good. Still not seeing changes to build/script.min.js which means grunt hasn't been run.

@jczaplew Hmm.. Just ran it again and tried to commit and push, showing this on my local machine:
=======
➜ navigator git:(homonym_search_fix) ✗ grunt
Running "htmlhint:build" (htmlhint) task

1 file lint free.

Running "uglify:build" (uglify) task
File "build/js/script.min.js" created.
Running "cssmin:combine" (cssmin) task
File build/css/navigator.min.css created.
Done, without errors.
➜  navigator git:(homonym_search_fix) ✗ git commit -a
On branch homonym_search_fix
Untracked files:
package-lock.json
nothing added to commit but untracked files present
➜  navigator git:(homonym_search_fix) ✗ git push origin homonym_search_fix
Everything up-to-date
➜  navigator git:(homonym_search_fix) ✗

@jpjenk if you do git diff after running grunt what do you see?

@jczaplew, I just checked ./assets/js/navMap.js and taxaBrowser.js and the new {braces} are there. Ran grunt once again and did a git diff ... blank page, it is fully up-to-date with GitHub.  I also ran:
git ls-tree -r homonym_search_fix --name-only
to make sure build/js/script.min.js is included in this branch. It is. Do you think the compiler/minimizer script is just taking the braces back out again anyway since it is a one line if/else? Because, you know, those 32 bits of data are really going to choke the internet :)

@jpjenk - odd. I just pulled it down and was able to minify the css and js. Not sure, but we should be good now! Have you checked this branch out on dev and tested it?

@jczaplew I just checked out homonym_search_fix on teststrata but the changes are not there. Now I'm quite confused because git log shows most recent commit is:
commit b7f565793745972bf1a702cf917009cdeb56ddd8
Author: John J Czaplewski<EMAIL_ADDRESS>Date:   Fri Oct 20 10:29:51 2017 -0500
Minified css and js

@jpjenk - could you clarify by what you mean? Are the changes not showing up in the interface?


Fix for timestamps with index

fix https://github.com/ridgepole/ridgepole/issues/427

coverage: 97.059%. remained the same when pulling c9bdbb16962e69bcd8cc8f44547b1c763d6ef274 on fix_timestamps_with_index into 9c4cce8bf5549f4a505529820bc50849236c9ad4 on 1.2.


Clean up route tables and NAT for quay

Targeting the now-very-much-overgrown #28

@henrikhodne r?

@igorwwwwwwwwwwwwwwwwwwww r?

I'm not 100% sure this will work, either, and given that I'm creating a new main route table association, I think this means I need to backfill some more of the default routes such as <IP_ADDRESS>/8 local.
If dropping the quay NAT is an option, I'm all for it.

Oh look! https://www.terraform.io/docs/providers/aws/r/route_table.html

Note that the default route, mapping the VPC's CIDR block to "local", is created implicitly and cannot be specified.


Disallow dots in folder names

So sub folders with fixtures can not have dots to avoid collisions like below:
+ old/
  + searches/
    log.yaml        # old.searches.log
+ old.searches/
  log.yaml          # also old.searches.log?

Instead, an exception should occur when dot in a folder name is detected.

PR: https://github.com/vkorobkov/jfixtures/pull/107

https://github.com/vkorobkov/jfixtures/pull/110 merged to master


Add old website docs and field type documentation

Added the old website's documentation
Added the field types to the UI (their navigation panel needs to be updated to only include current field types, rather than a static list)
Updates to the documentation
Accidental linting (lint on save) into the Keystone lint style.

Because linting, probably good to review file changes using https://github.com/KyleAMathews/keystone/pull/1/files?w=1 which will ignore the whitespace for you.

Excellent work! Glad to see you picked up the Gatsby idioms! Any problems figuring things out?

Sorry it's taken me a while to circle back around to this!
I'm not sure if I dived in to trying to modify the code before looking enough at/for documentation, but I spent some time tracking down what file to make changes in to pull in the new files.
Once I found where, the concepts were really easy to understand to make changes. Biggest thing was noticing that file reading was occurring from an array, implying I could just add more locations that would be parsed by the same functions.

Cool! Fun to see someone using my new code in Gatsby correctly! The hope is always that new abstractions seem fairly intuitive.

Actually, I've been having a bit of struggle with hot reloading. I'm running gatsby develop, however I seem to have to stop and start gatsby to get md changes to appear, although it is watching the JS, the templates and such.
Is this intended, or should the md be causing updates?

data changes are not setup to hot reload just yet unfortunately. That's
my very next todo item though actually so it'll be coming soon but yeah, in
the meantime, when changing md and other data files, you'll have to restart
the dev server to see the changes.
On Mon, Feb 27, 2017 at 8:17 PM Ben Conolly<EMAIL_ADDRESS>wrote:

Actually, I've been having a bit of struggle with hot reloading. I'm
running gatsby develop, however I seem to have to stop and start gatsby
to get md changes to appear, although it is watching the JS, the templates
and such.
Is this intended, or should the md be causing updates?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/KyleAMathews/keystone/pull/1#issuecomment-282937577,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEVhxYjtgYqxyCEfZ2bOJ2KYcZ4IWjiks5rg5_AgaJpZM4MG8OW
.


Allow simple alias assignments in externs for type inference.

The compiler can correctly infer the types of aliases in externs when they are assigned and we should allow this pattern.
This removes the spurious warning that is generated for such simple assignments.
I can remove 400 lines from the jQuery externs with this change.

All the tests passed locally. Not sure what's up with travis.

Yeah, we've been getting a lot of flaky failures recently for testSpeed(com.google.debugging.sourcemap.Base64VLQTest) on travis. We should probably increase the timeout or just remove it.

otherwise lgtm

nice and simple

@MatrixFrog That does simplify things. Done.

@MatrixFrog ping


Upload CharlieCard inventory into Phil

Creates a LiveView to import CharlieCard inventory into Phil.
https://app.asana.com/0/1204871882394498/1205551049136586/f

Putting back in draft mode because we have some changes based on meeting with CCS.

thanks @Adzz. i tagged @napilla so you all can decide about things like using async or using text instead of string. better to have the same policy throughout.


Update redirect page on 403 and 404 error code

Please add sign-off with every PR.

Sure @sagarkrsd


CVE Scanning

Closing this issue. Now that image-scanning is up we will created more target issues as needed. An issue documenting remaining tasks has also been created.


new project modal state set as closed to avoid fluttering in & out

Setting up the new project modal as closed from the beginning. Otherwise, it pops up briefly and then it's auto-closed when the projects are loaded which looks bad.

Good idea!


web ui

Hello...im having an issue with the web UI...by the way...this is amazing! Absolutely love the idea of running a gaming server on unraid. I am trying to access the WebUI: and can't seem to get it to open...i set the port to one I'm not using..it is 8283. I think the default was something like HTTP://[ipaddress]:[port] but I did it like http://ipaddress:port ... was I supposed to leave these in there "[" ? Thanks. Also...sorry I contacted you this way but I couldn't find another way.

Hello...im having an issue with the web UI...by the way...this is amazing!   Absolutely love the idea of running a gaming server on unraid.    I am trying to access the WebUI: and can't seem to get it to open...i set the port to one I'm not using..it is 8283.   I think the default was something like HTTP://[ipaddress]:[port]   but I did it like http://ipaddress:port   ... was I supposed to leave these in there "[" ?   Thanks.   Also...sorry I contacted you this way but I couldn't find another way.

Hi.
Contacting me like this is fine 😁
That port does not seem like a proper port for web sites. Try one of the following ports ...
80,81,8080,8181

P.s HTTP://[ipaddress]:[port] is just so the webui button works to the correct address and portbut does not actually change the port of the game webui. You still need to change the UI port in the game server settings

Where would you change the UI port? I can't find the right config file. I tried setting the container variable but that didn't change and in the UnRAID container log i still see that it's trying to bind to 8080.

Ah, it seems that the port is hardcoded in the nginx config file. Thus the environment variable seems to be useless.

It should over right it. I'm sure I have done it before as most routers requires port 8080 itself for its own settings

When I search for PORTFORWARD_WEB in your repo I can only find it in two files. start_rust.sh, where the port forwarding stuff happens and in the Dockerfile where it's initialised as empty, but to overwrite the nginx conf you would use that variable, right?

Yh but the file that looks at the variable is inside a container that you can't access. It's inside the virtual os that the container is running and not the files that you have access to

Any chance you can screen shot the files with the variables are and the port config settings just so I can work it out better.
I will see what I can do and if it's a mess up on my end I can try to rectify it tonight or tomorrow?

I can bash into the running container, but I'd be glad if you could tell me the exact nginx file that is used for the webserver to avoid confusion.

Let me have a look at it and get back to you. If I fix it first then expect and update notification :)

I think I know how I got it working on the port required.
First the docker must not run as host mode as connection.
Leave it as the default settings it originally had.
You much change both port settings for Web port, Web port forward to the desired one.
The container runs it as 8080 but it's leaves the container as the port you wanted.
Please test and get back in touch.

I changed the container to bridge mode (was just using your unraid template, so it was host before).
Changed both port settings for the web port, but now it seems that my set port is used on both sides, container and to the outside.

Is it running properly though? And I think it lies about the internal ports in bridge mode

Now I thought about it what network mode can you choose from?

I can have host and bridge, and also I could choose custom docker nets. By default it was in host.

Yh that's right as only host mode supports auto port forward


site_prisms has runtime dependency - capybara ~> 2.12, is support for capybara 3.0.2 expected

Issue to raise for SitePrism framework
I'm not sure I have all of the information you would like for this.  I'm mainly just inquiring as to why site_prism has the gemspec locking Capybara versions ~> 2.12
Environment

Windows 10
Site_Prism 2.12 and Capybara 2.18.0 (hoping to use 3.0.2) (Also selenium 3.11.0)
Ruby 2.3.3p222
Cucumber tests mainly (cucumber-core 3.1.0)
Chrome 66 (chromedriver 2.37), Firefox 59.0.2 (geckodriver 0.20.1)

Expected Behavior
Expected when I update capybara to 3.0.2, site_prism either updates or stays at the current version
Actual Behavior
site_prism downgrades to version 1.3 and I begin getting errors with the wrong number of arguments
I have not looked into the argument errors, to be honest.  I started to and realized that the version had dropped significantly and I don't want to go back in time to before new features and support was added.
Steps to reproduce
Bundle Update?
I'm not sure how relevant all of these fields are for an issue of this nature.  I'll do my best to assist on this issue.

Thanks for filing this @crcraig92
I just ran a quick pass through of the tests with capybara 3.0.2.  I didn't encounter anything unexpected so this may end up being pretty quick.

I have a partially completed change for this.  I ran into a bug while working on it though.  That resulted in PR #243 which will need to be dealt with first.

@crcraig92 To answer your question why site_prism is locked to 2.12. Due to the way semantic versioning works (See semver.org), we cannot guarantee support for the 3.x series of capybara.
However, what we can do is look into it (As @tgaff) has done above.
Main concerns / points to note.
Ruby Version / XPath version bump (Extraneous)
first / all methods have been reworked, will need some additional work.
Generic testing around refactors done (<100 commits so not too worried)
Bumps through the 2.x series (2.12 - 2.18 includes several hundred commits)
If we did land this, I would probably want to halve the enforced capybara updates first (To something around 2.15), to ensure people are being staged with their updates.

@tgaff Thanks for taking an initial look into this effort
@luke-hill Thanks for the information!  That makes sense.  I appreciate your efforts to maintain this gem.

On a side note. At the moment you're not too far behind Capybara using 2.18.0 So hopefully being on 2.18.0 isn't that much of a hindrance.
We definitely will get 3.x unlocked, I promise. Just a question of when, not if.

I've popped on a "Needs decision" label @crcraig92 purely for the reason that "we" or "I" need to decide "how" this will land.
My personal opinion is.
a) It cannot land as the next release
b) It should be staged (Even through fake upped dependencies)
c) (Unsure if we can), We should see what the uptake is RE Ruby version / Capybara version (I know Capybara isn't high on the 3.x usage at the moment, although that might just be because of individual gemfile restrictions. From this we can then decide when to support this.
d) If we do decide to include this, we should probably find out from Capybara maintainers, if there are any other decent changes we can make to facilitate it.
e) If this lands, would we want it to be a breaking change version (And correlate a V3.0 site_prism release accordingly), or are we happy having this in site_prism 2.x

Ping @tmertens @twapole @TheMetalCode for additional thoughts / 2cents.

@luke-hill I haven't looked through any recent updates on this project, so this may have already been handled, but I'm pretty sure the behavioral changes in Capybaras first and all would require at least some changes to SitePrism to maintain its current behaviors around waiting/not-waiting

Cheers. In my eyes I want this project to be as conformant to Capybara as possible going forward. It makes sense given the mass usage that this gem should just wrap your behaviour and not change that much to be honest. (Aside from adding stuff).

I'm not sure if you were asking me for my input on this.  But here is my opinions in regards to yours anyway.

a) It cannot land as the next release

It's not really breaking anything, so there's no need to rush it.  Cucumber moved to 3.x recently and made some big changes and shortly after Capybara also went to 3.x and push some changes I've been looking forward to.  .find now returns ElementNotFound instead of nil and .all has some implicit waits now.

b) It should be staged (Even through fake upped dependencies)

This sounds good to me.

c) (Unsure if we can), We should see what the uptake is RE Ruby version / Capybara version (I know Capybara isn't high on the 3.x usage at the moment, although that might just be because of individual gemfile restrictions. From this we can then decide when to support this.

I'm using Ruby 2.3.3, and I've tested 2.4 for my framework and it's fine.  Some other frameworks are stuck at 2.3.3 for 'reasons' and ruby version management isn't the cleanest thing on Windows.  I would have thought a larger portion of Capybara users would also be using Site Prism but it appears that's not the case.  I'm not sure what other gems might have dependency issues, but for me I believe the only gem restricting the bundler from grabbing Capybara 3.x is Site Prism.

d) If we do decide to include this, we should probably find out from Capybara maintainers, if there are any other decent changes we can make to facilitate it.

As @twalpole mentioned, first and all had significant changes, also Capybara.exact_options might affect users depending on how they created their finders.  I can't think of anything else that would need to be handled.

e) If this lands, would we want it to be a breaking change version (And correlate a V3.0 site_prism release accordingly), or are we happy having this in site_prism 2.x

I'd say it should move to 3.x if at the very least just for clarity.  Selenium, Cucumber, and Capybara are in a 3.x series so it seems like it would fit.  Also, if you need to make changes to implicit/explicit waits people need to pay more attention to updating or they may start seeing timing issues.
I hope that answers any questions if you had them, and didn't generate more questions/concerns you didn't have before.
Thanks again for your efforts with this.

I've been looking at the first/all changes.  I don't think they'll be particularly difficult to take care of.  Those are the main ones mentioned in https://github.com/teamcapybara/capybara/blob/master/UPGRADING.md    Capybara 3.0 doesn't look like a huge departure from 2.x.
The only reason I haven't prepped a PR for this already is because of other blocking issues that should be resolved and merged first.
I also don't think there's a big need to lock site_prism to a last capybara 2.x and then have a new 3.x supporting version.  From what I've looked at so far it seems likely that we can support 2.x and 3.x simultaneously without significant effort.  I think it should be up to the user - if they have specific requirements in a page object that don't work in Capybara 2.x then the user will be responsible for choosing when to switch to Capybara 3.x and make the required changes.

This will form the basis of a 2.14 site_prism release.
2.13 will be released shortly (Around 1-2 weeks). This will then (Whenever complete), hit the following cut.

For anyone still waiting for the officially cut gem, you can use a fork with slightly modified dependencies:
gem 'site_prism', github: 'mateusg/site_prism', ref: 'dc00f2b976e0f801541589ff94ded23cb04cee02'

Fixed in fb182c8eec3d4b689bd906200f9432db6368b685 - Will be released in the next 24 hours as v2.15 (Note there are just fixes in here aside from the uncapping dependencies, just to inform readers that it wasn't as simple as just upping the gemspec requirements)


Add nightly vkcts workflow

Adding a workflow to run vkcts nightly in slang mode using the expected list of passing tests.

Is it possible to run CTS in more than one thread? We should use something like 12 threads.
Need to change back to schedule before merging.

There seems to be a limitation in slang right now that is preventing multi-threaded execution of the tests. However, I opened an issue to address that limitation: https://github.com/shader-slang/slang/issues/3739 When the limitation is resolved then the workflow can be updated to run the tests multi-threaded.


Load 2 batches at once

Detailed Description
Would it be worth loading 2 batches at once and taking and randon (seeded) sample of examples from each one.
This would mean the that the ML models dont see the same batch each epoch. This might help stop over fitting.

Yeah! I think shuffling up the examples a bit more would probably be good.

yup, SGTM!


Updates to application gateway template

There is a new API version available that resolves the conflict issue we've been having by forcing the association between policy and gw. Also setting requestBodyCheck=true on both configs to avoid conflict.

Tested in our appgw test environment (hence the "wip" commits"). Deploying successfully now


Hint doesn't work

Describe the bug
The hint parameter is not built.
To Reproduce
Steps to reproduce the behavior:

Create Widget RoundedBackgroundTextField
add hint: "text"
See error

Expected behavior
A lighter-colored placeholder hint that appears on the first line of the text field when the text entry is empty.

Hello
Same error with

Flutter 3.0.5
rounded_background_text: ^0.1.3

thx


Are there any videos on how to download??

Are there any videos on how to download?? I am not realy great at just reading and doing, its way simple if i could what a video :))

Hi STUNKEST,
Unfortunately there is no video on how to download it, make sure you read carefully the instructions and follow them.
Good luck!


[Bug]: Using vllm as the inference engine, there is an incorrect recognition of GPU computing capabilities for different types.

Your current environment
(py311) (base) lianjh@cloud88:/storage/lianjh/dev$ python collect_env.py
Collecting environment information...
 Your NVIDIA A800 80GB PCIe GPU has compute capability 7.0. You can use float16 instead by explicitly setting the`dtype` flag in CLI, for example: --dtype=half.

My rank:0 GPU is a V100, and I have correctly specified NVIDIA A800, but it still recognizes the compute capability of the [first GPU](rank:0 GPU).

This did not occur in 0.5.0post1

cc @youkaichao as this may be related to the current_platform refactor

By quickly comparing the code, it might be related to this pull request: https://github.com/vllm-project/vllm/pull/6080

@T-Atlas can you try #6216 ? I didn't expect heterogeneous GPUs indeed.

@T-Atlas can you try #6216 ? I didn't expect heterogeneous GPUs indeed.

@youkaichao I'm trying this and once I find this works fine I'll let you know in https://github.com/vllm-project/vllm/pull/6216


Projects/Namespaces show only last policyreport

We have 3 policyreports in default namespace
~ k get policyreports.wgpolicyk8s.io -n default
NAME                                   PASS   FAIL   WARN   ERROR   SKIP   AGE
88f7c0a2-418e-42f5-a099-277fe97d3fe0   0      1      0      0       0      41m
fb6c082b-1469-44ab-a492-0e200dbe959c   3      4      0      0       0      41m
2a6602c4-e86e-438b-b68b-dbe6c00dc2b0   3      4      0      0       0      41m

UI shows numbers only from last report, order changes randomly.

We discussed with @viccuad about 2 options:

as immediate fix we could show compliance report for default namespace itself
show aggregate numbers from all the resources in that namespace, which might be costly for big namespaces

I favour "show compliance report for default namespace itself", which is also simple to implement: obtain uid of namespace (or clusterwide resource) under eval, query for clusterpolicyreport with metadata.name == uid of the resource under eval.


MUC3B does not appear in genome nexus

MUC3B does not appear in GRCh38 genome nexus, but MUC3B is accepted as a valid gene in query in cbioportal.
https://grch38.genomenexus.org/ensembl/canonical-transcript/hgnc/MUC3B?isoformOverrideSource=mskcc
Probably related issue: https://github.com/genome-nexus/genome-nexus/issues/719

MUC3B is not in Ensembl


MNT: Skip missing value lines from CelesTrak files

Sometimes we can get missing values in the CelesTrak file which causes errors when loading the files with numpy. We can ignore these lines and not process them.
closes #71

I do somewhat worry that we might end up skipping a line in the middle of the file somewhere and getting "off" in our dates. But, I think that is a minor issue and can be addressed later. This fixes an issue that happens when the source hasn't updated their file immediately and has some empty entries.


Feat/build scripts for circuit

add build script to build circuits inside disclose directory
fix build scripts for dsc circuit
change output path for dsc and prove circuit
add local verifiers to .gitignore not to bother the production

Deleted all dsc 2048 related files


Why we are checking for if not (power):

@seanpearsonuk @ansaboutin
On line 195 in https://github.com/ansys/pyansys-units/blob/main/src/ansys/units/utils.py why we are checking for if not (power): ?

@hpohekar @ansaboutin
It appears that the intention is to filter out the non-contributing terms.