Dcwc 114 error pages

DCWC-1144

Very short PR to add the 500.js and 404.js page. Both are empty.
Added rendering tests in our unit tests.

Notes

I couldn't figure out how to render the 500.js in dev mode. i think its built on purpose this way to help developers see the error message when running the app locally.

your branch does not have the latest code from dev, why?


Add deploy scripts dir to PATH if not already there

Background
Fix: https://dexairobotics.atlassian.net/browse/DEX-6496
What's new

✅ [New feature description. Only a single new major feature is allowed per PR.]
❌ New feature is accompanied by new test: [name and description of new test].

Tests

✅ Tested on simulated robot: [Describe the tests/commands used.]
❌ Tested on physical robot: [Describe the tests/commands used.]

Quality

✅ New code is written in pure C++17 to the best of my knowledge.
✅ New code follows established C++17 best practices (C++ Core Guidelines).
✅ New code passed clang-format and cpplint at least, if not all of our static code analysers.
✅ I have included Doxygen-style documentation in the new C++ code.

Note to reviewers
Please verify that all sections are accurately filled out and that all 6 numbered entries above are present and ticked/checked off, with their requirements met, if applicable.

https://github.com/DexaiRobotics/fullstack/pull/2910


Added Variants DropDownBox for Shell Sort

I added a 'Variant' Box next to 'Algorithm' selector box with possible variants of each algorithm. Here I have only connected the button actions to Shell Sort's different variations which were already implemented. But I aim to add variants of other implemented algorithms too. @LucasPilla  tell me what you think.

This is in response to this issue : https://github.com/LucasPilla/Sorting-Algorithms-Visualizer/issues/35


bring back hasteb.in

Is your feature request related to a problem? Please describe.
my problem is that you got rid of hastebin
Describe the solution you'd like
bring back hasteb.in
Describe alternatives you've considered
give me the domain so i can bring back hasteb.in
Additional context
bring back hasteb.in

I do not own the hasteb.in domain.


https://github.com/lautr3k/lw.svg-parser/issues/4

Added silly symbol support  (only visualization, treated as a def.)

Cool!
Before i merge your commit, could you change
let element = this.parser.defs[target]
if (! element)
    element = this.parser.symbols[target];

by
let element = this.parser.defs[target] || this.parser.symbols[target];
Thanks (btw i always use brackets, even when there is only one instruction)

Done!

Hmm with https://github.com/lautr3k/lw.svg-parser/pull/5/commits/bf11bcab26427baf86cc4115eb8982349fc9c45f the symbol is never rendered on <use>. The no rendering is correct, but then the symbol children should be appended as use children.
I've added array support for use but is surely wrong as replaces the use tag with the instance. Propose a non-standard element <instance> or <g> to wrap the symbol childnodes.

I worked on this all the day and I think there's a big problem in the heart of the parser... I will resume the investigations quietly tomorrow...

I have found a way to have full CSS support ;) and a better way to handle attributes, I am refactoring the entire library...

Cheer Up!


Peer dependencies out of date

We've been using React v15 with canon-react on apollo and haven't run into any problems (knock on wood)... I was just wondering if you had looked into updating the peer dependencies in canon-react to support the more up-to-date version of React.
https://github.com/rackerlabs/canon-react/blob/master/package.json#L45-L46

I believe that Canon-React is committed to supporting the current and previous versions of React, which means we have to ship with 0.14.8 until v16 (or whatever they decide to call it) is released.

@wpwood Thank you for the insight.  I will close this issue.


Proposal to add import/export to region demand timeseries

Hi all,
I have a proposal and I hope I am not too late with it. Why haven't we simply added the timeseries for import and export to neighbouring countries to the demand timeseries of the region connected to the country? wouldn't that have been easier? I would be happy to have such an option as it is a bit difficult for me to integrate that into the model. I think it is not that much effort to still make that option possibel, right?
For Base 2016: I think we have to add export here because otherwise a large share of demand is missing as the exports were quite high in recent years. Also, for the policy paper we need to be as close as possible to the actuall generation in 2016 because otherwise the paper is a joke or we really have to explain ourselves. :)
Let me know what you think!
Best
Jonas

Hi
I guess my questions to the new 'trade' data fits here - First, thanks for clarifying the interpretation of ‘trade’ yesterday.
However, just now I realize that I might still not be completely sure what the idea is, and how the data fits the idea.
From our discussion yesterday I understood that
We assume that the trade.volume timeseries is fixed hourly trade and that this hourly trade happens at the hourly price in the timeseries trade.price
Then questions:

Why do we then have fixed trade for 2016, 2030 and 2050 but only prices for 2030 and 2050?
Why do we have an ‘installed capacity’ for ‘trade’? I mean, if we can just add or subtract the trade.volume from the demand, would the installed capacity then have any meaning?  (For the model only having fixed trade it would be redundant to have installed capacity as this then only relates to a question of feasibility in input data)

Good question, we also should have prices for 2016 actually ...
I think some models can model the capacities with fixed timeseries. But I just don't use those capacities. So depends on how you want to model it I would say :)

Thanks for a fast answer!

With prices added for 2016, it makes sense to me. Also, @chrwm could it be that the trade.volume has a mistake since with the new ID32 data I all of a sudden get a 'NS-transmission-trade-trade volume' profile and we have not earlier considered a NS region?
I will ignore the capacities for now as well then. To this I can add, that yesterday in the mathematical comparison group we had a discussion whether we should make an analysis where the frameworks that can make additional trade for the price within the capacities are allowed to do so (to show differences in the framework). However, this morning I found out that the trade-addon in Balmorel is based on an estimated stepwise price function and not an hourly timeseries. I could implement it with the timeseries instead but I am also very(!) fine with just having the fixed trades and paying for these (and ignoring capacities) - but others opinion on this is highly welcome

NS also(!) part of the trade.price timeseries

Good question, what is NS? :)

I have ignored that as well...

I would vote here, that each framework models the imports/exports in the way they can (i.e. exogenously by fixed volumes, or endogenously by constant/hourly prices/capacities). I think it would be an interesting when we then showcase the differences in results. What do the others think?

Hi all,

For 2016 federal state-specific prices couldn't be found and in mid-March it was decided to leave them out.
Yes NS was a typo and should have been NI -> This is already corrected and uploaded in ID33.
Exactly as Jonas said the capacities could be necessary for modelling trade other than adding or subtracting it from the demand.
I agree with Soner the difference in results could be very interesting using different modelling approaches.
However, I would argue at least for one model run in the base scenario with the same approach across all frameworks (I guess adding/substracting to demand can be implemented in all) for comparability reasons.

Hi all,
one thing now that we found in ID33 is that in SH, for some of the hours the imports exceed the demand. If we go with the proposed solution of just adding net-export to the demand, this results in a negative demand for these hours which our Framework is not really suited to handle, I expect the same situation might occur for others.
The "clean" way of implementing this in our case would be to define an import and an export technology, which then have "fixed" production for the respective hours. However, especially the export technology would need some finetuning: do we assign costs (negative?) to it and allow overproduction to export electricity? Is it "just" a dummy and export is not allowed apart from the given values?
How would the case be for the other developers? Do you have an issue with negative demands?

Hi all,
I saw that a new data ID was uploaded today (ID 34) and in the comments it says '* Updated trade volume data in oed_timeseries in 2016'. Maybe this has the purpose of 'solving' the negative demands? I have not checked yet, but in Balmorel I implemented the import and export seperately from the demand and therefore negative demand does not seem to be a problem for me.

Okay, will check, thanks Stefanie.
One question with respect to the export technology: do you allow the model to export electricity (at which price? Same as the import price?) Or is it just a placeholder and all trade volumes are fixed and not subject to the optimization?

I have decided (for now) just to work with fixed import/export corresponding to the traded volumes. So the model trades what the trade timeseries suggest and the trade happens at the prices in the price timeseries.  Therefore, the whole thing is currently just a 'constant' to the model and therefore not affecting the optimization. I understand that, the idea of having it is that it changes the demand pattern and enables us to compare the results to historical values.
Balmorel is also able to have additional trade which I have been talking to Soner and Christoph about. The idea would then be to let the model optimize the trading when using the price timeseries as driver. However, Balmorel is not using an hourly price timeseries for this, but a stepwise approximation - and therefore I would have to implement it seperately with the prices we have. I therefore prefer not to do so :)

@khainsch

If we go with the proposed solution of just adding net-export to the demand, this results in a negative demand for these hours which our Framework is not really suited to handle,

The trade volumes are aggregated so that "negative values are net import (imports are higher than exports)". This is stated also in the "comment" column. This would mean that if you would add "net-export" it should be a positive trade volume added to a federal state demand.
I recognise the fact that the demand+trade volume in a federal state may become negative. I just wanted to say that to avoid confusion.
@StefanieBuchholz

I saw that a new data ID was uploaded today (ID 34) and in the comments it says '* Updated trade volume data in oed_timeseries in 2016'. Maybe this has the purpose of 'solving' the negative demands?

The updated trade volume time series ensures that the total trade volume across all federal states in 2016 corresponds to a net export of 49TWh according to historical data. It does not address the problem of possible negative demand in individual federal states per time step.

Okay, so negative damand values are a problem. I wouldn't have thought that something like this could even happen. I think we have to fix this somehow :(
So first of all, does someone have an overview how many negative values do we get? Is this just for very few time steps or does this occur regularly?

@chrwm

The trade volumes are aggregated so that "negative values are net import (imports are higher than exports)". This is stated also in the "comment" column. This would mean that if you would add "net-export" it should be a positive trade volume added to a federal state demand.

Sorry for the misunderstanding, what I meant by "net-export" is that positive values are exports, negative imports which is why I called it that way. :)
@jonasVano

So first of all, does someone have an overview how many negative values do we get? Is this just for very few time steps or does this occur regularly?

For 2016, there are 120 values for SH which are negative, for 2030 we are talking about 1414 hours and for 2050 200. So I think the option from Stefanie would be the cleanest way to include two technologies (one for export and one for import) with the respective time series. Would love to hear how other frameworks would ideally implement this.

So we found the solution to the Problem:
Exports: We add them as positive values to the demand to increase the demand
Imports: We model an inflexible power plant with a pre-defined timeseries as output that corresponds to the imported energy
The changes will be added in the final update of the input data in the coming days.

I just need some details on how you decided to model it, for implementing it in the input data. The "trade volume" timeseries is currently given as the sum of imports and exports.
Do you want to split the data as in the table below?

timestep
trade volume
import
export

 1
280.65
0
280.65

2
55.55
0
55.55

3
-20.5
-20.5
0

For 2016, I can disaggregate the data specifically to import and export, as the data is available.

timestep
trade volume
import
export

 1
280.65
-10
290.65

2
55.55
-244.45
300

3
-20.5
-500
479.5

For 2030 and 2050, however, I cannot, as just the sum of imports and exports is available to me. If you intended the implementation in the 2nd table, then @sonercandas can you provide the disaggregated imports and exports from the precalculations?
What did you decide with regard to the trade prices in 2030 & 2050?

Hi Soner, I implemented it as you said.
I split the 'trade volume' as shown in the first table above.

The positive values are in the timeseries "trade export" and should be added to the demand
The negative values are normalised between [0,1] and correspond to a dummy power plant with 'technology_type' = 'trade import'. It has specific regional 'installed capacity', due to the normalisation.
Hence the technology should behave like a renewable technology with capacity factor.


Postgresql container healthcheck probably needs work

I've only seen this on windows, but in windows test I see
psql: error: connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: FATAL:  the database system is starting up
--
  | ddevapp_test.go:1584:

That's right after a snapshot restore, so it's possible that startup is not really complete after start/snapshot restore. This is the case on MariaDB/MySQL containers, so it may well be here as well. On those, we actually wait until the restore is complete.

Looks like pg_isready may be the thing, https://www.postgresql.org/docs/9.3/app-pg-isready.html -
that can be added to the healthcheck and may do the job, or perhaps the code that does a wait after snapshot restore on MariaDB can be adapted.

Hmm, it already has pg_isready:
https://github.com/drud/ddev/blob/391494d56f0b05c7e105720ed1ae08e4f6dc4217/pkg/ddevapp/app_compose_template.yaml#L71-L79

I haven't found anything yet in postgres docs that helps me here, and I haven't seen a problem except the one time, in a test, on Windows, that could have been a race condition or something.
Happy to reopen this if a postgres guru stops by.

Turned out the healthcheck wasn't right at all, reopening, will fix.


Checklist generation for the Replication standard

Please, provide checklist generation functionality for https://github.com/acmsigsoft/EmpiricalStandards/blob/master/docs/Replication.md.

@david-istvan the replication standard looks ready. Do you want to add a selection for it in the tool generator and see if it works?

All done.


[Feature Request] More to Python > 3.6 later this year (2021)

Describe the solution you'd like
Python 3.6 is going to EOL later this year (2021) https://endoflife.date/python This latest CI build shows we are currently supporting our site with 3.6 officially https://github.com/pycontw/pycon.tw/runs/2876830688 .
We may want to upgrade python after the main conference event.
Describe alternatives you've considered
We may also consider to upgrade python after talk proposals are announced.

 upgrade python


Invalid markup not throwing an error

Nested math tags should throw an error as they are not valid.

I tried to reproduce it several times and every time there was an error, hence all works as expected.

Argh, I can't reproduce this in an isolated minimal example and of course I fixed the sample where I ran into this and don't recall which of the expressions caused the issue.
Sigh.
Sorry for the noise. I'll re-open if I run into this again.

Got it. The problem is inside epub-switch elements.
Something like
<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE html>

  <title>nested math tags should be invalid</title>

nested math tags should be invalid even inside epub switch

 <epub:switch>
   <epub:case required-namespace="http://www.w3.org/1998/Math/MathML">
    <math alttext="placeholder" display="block" overflow="scroll" xmlns="http://www.w3.org/1998/Math/MathML">
      <math alttext="a"><mn>11</mn>
       <mo>,</mo>
       <mo>−</mo>
       <mn>3</mn>
       <mo>,</mo>
       <mn>5</mn>
      </math>
    </math>
   </epub:case>
  <epub:default>
 </epub:default>
</epub:switch>

With 4.0.2, this gets validated without errors.
Sample nested-math-tags.zip  (change extension to epub).

I didn't test master so this may be fixed already.

Tested against epubcheck 4.0.3 built from master.  No errors.


Alternative NVM driver interface for using with existing database

Is your feature request related to a problem? Please describe.
Currently, the stacks implements it's own "database" module in order to store objects to NVM. NVM abstraction is possible thanks to NVM port driver. Let's say for example a "database" module already exists with his features, capabilities, etc: it would be possible to implement NVM port driver to read/write from/to this "database"? Right now I think it's not, since NVM driver interface is really basic: read/write starting from an offset for a specific size. It's not possible to redirect these calls to existing database since they are raw data: no interpretation of these data is possible.
Describe the solution you'd like
I'd like to have an alternative NVM driver interface, where user can decide to redirect read/write calls to its own database.
Describe alternatives you've considered
The solution I thought is to change CO_IF_NVM_READ_FUNC and CO_IF_NVM_WRITE_FUNC to something like:
typedef uint32_t (*CO_IF_NVM_READ_FUNC )(uint32_t objectId, uint32_t subIndex, uint8_t* ptrBuffer, uint32_t size);
typedef uint32_t (*CO_IF_NVM_WRITE_FUNC)(uint32_t objectId, uint32_t subIndex, uint8_t* ptrBuffer, uint32_t size);
Maybe the NVM driver interface (and implementation of internal code) may be choose at compile time with a specific compilation define in co_cfg.h (e.g. #define CO_USE_INTERNAL_DATABASE_IMPLEMENTATION -> use the current implementation, #define CO_USE_EXTERNAL_DATABASE_IMPLEMENTATION -> use different NVM driver interface.
Additional context
/

Hi, thank you for your enhancement idea!
Can you explain to me more, what data you intend to store in the external database? I have the impression, you want to replace the object dictionary with an external database. This is really difficult, because the read and write activities triggers at some object entries actions (these are managed with the CO_TYPE functions).
The NVM drivers are responsible for saving the current value of a collection of object entries from the dynamic runtime area (RAM) into a persistent area (FLASH, EEPROM, or anything other). These values are used as start values after a restart of the node. If you consider an external database for this kind of persistent storage we could use the argument start in the NVM interface as a selector for the given collection of object entries:
Let's try to brainstorm this kind of use-case:
Assuming you have two parameter groups. One with data for the NMT heartbeat configuration and another for the ADC calibration values:
struct NMT_PARA_T {
  uint16_t Obj_1017_0_Heartbeat;
} NmtParaMem;

struct ADC_PARA_T {
  uint8_t Obj_2100_0_AdcFactor;
  uint8_t Obj_2100_1_AdcOffset;
} AdcParaMem;

Then we prepare the configuration for save, restore and reset behavior, factory settings, and the offset (typical placement in non-volatile memory - for us we could use this as the identifier of the collection queries.
const CO_PARA {
  0L,   /* query-id for this parameter group */
  sizeof(NmtParaMem),
  &NmtParaMem,
   :
} NmtParaObj;

const CO_PARA {
  1L,   /* query-id for this parameter group */
  sizeof(AdcParaMem),
  &AdcParaMem,
   :
} AdcParaObj;

Note: these parameter group configurations are placed in the object entries 1010h and 1011h.
Now, instead of placing the NvmParaMem and AdcParaMem with the NVM drivers somewhere in NVM memory, we could store the contained object entries in an external database. First, we need to manage the queries for each entry in the parameter groups:
/* structures to support the query management: */

typedef struct PARA_QUERY_T {   /* data for a single query */
  uint16_t Index;               /* all data we need to access database */
  uint8_t  Subindex;
  uint8_t  Size;
} PARA_QUERY;

typedef struct PG_QUERY_T {   /* structure for a collection of queries */
  PARA_QUERY *Query;          /* start of para query array */
  uint8_t Length;             /* number of para queries */
} PG_QUERY;

/* definition of queries (I would place them near the parameter group definitions): */

PARA_QUERY NmtQueries[1] = {
  { 0x1017, 0x00, 0x2 } /* heartbeat (word) */
};
PARA_QUERY AdcQueries[2] = {
  { 0x2100, 0x00, 0x1 }, /* adc factor (byte) */
  { 0x2100, 0x01, 0x1 }  /* adc offset (byte) */
};

PG_QUERY PgDb[2] = {
  { NmtQueries, 1 },
  { AdcQueries, 2 }
};

With the definitions in place, we can prepare the NVM driver for the external database (here only write as an idea):
uint32_t DrvNvmWrite (uint32_t start, uint8_t *buffer, uint32_t size)
{
  uint8_t  idx;
  uint8_t *value = buffer;
  uint8_t  bytes = 0;

  /* loop through addressed query collection and perform database action */
  for (idx = 0; idx < PgDb[start].Len; idx++) {
    pq = PgDb[start].Queries[idx];

    /* place your database query here, e.g.:  */
    ExtDatabaseQuery (pq->Index, pq->Subindex, value, pq->Length);

    /* navigate to next value in parameter memory */
    value += pq->Length;

    /* count all processed bytes */
    bytes += pq->Length;
  }
  return (bytes);
}

!! Note: this is only typed in this issue - never compiled nor tested. I never did this in a real project. It is just the try to investigate, how we can achieve your use-case !!

Hi @michael-hillmann,

I have the impression, you want to replace the object dictionary with an external database

Exactly!
First of all, thanks for your answer: I've just started to use the stack, so I'm still in "learning" phase. I will answer to your in detail next week!

Closed due to inactivity, the question is answered.


High level methods?

Hi, thanks for releasing this.
I wondered if you had any plans for high-level methods that would make it easy to just pull out the final list of commands to run along with dependency information?
At the moment I'm stuck right at the start wondering how best to deal with cwl files that don't need an inputs json, or that do or do not have $graph, or that might specify just a workflow or just commandlinetools etc.

seems good. just let you know

cwl.go is just a parser for CWL files

constructing command line should be out of this project
because the final commandline depends on the engine and its context by which this CWL is handled

cwl.go project is started for yacle, yet another CWL engine

See here for how yacle generate commandline from cwl.go struct

I'm not sure it could be an answer for your question. I'm sorry I'm late, anyway.


vhost mode append domain!

after checking in my environment during enumeration for vhost figured out that if the --append-domain not use the vhost mode does not work correctly!
version 3.6.0

it works as described


Removed redundant tests on CI

Codecov Report

Merging #2 (cd9ad87) into main (72825eb) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   89.38%   89.38%
=======================================
  Files           4        4
  Lines         179      179
=======================================
  Hits          160      160
  Misses         19       19

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


Compare with a new revision

Previously we've used the version from 2021, but currently the book version is from August 7, 2022.

#12


auto translator suggestion

it would be nice to understand russians in textchat, aswell as swearing at them in their language.
i imagine this feature kind of like flowhooks translator feature.
it would be really nice if you added this.

the google translate API rate limit is way too low to be used. i dont know about microsoft translate or any other alternatives


Key error when running filtercache on non Search diags

sperf search filtercache doesn't handle non search tarballs correctly
sperf -v search filtercache -d ~/code/sperf/tests/testdata/diag/DSE_CLUSTER
sperf filtercache: 0.6.0

from directory '/Users/rsvihla/code/sperf/tests/testdata/diag/DSE_CLUSTER':
Traceback (most recent call last):
  File "sperf", line 13, in <module>
  File "pysper/commands/sperf.py", line 41, in run
  File "pysper/commands/sperf.py", line 38, in run
  File "pysper/commands/search/filtercache.py", line 37, in run
  File "pysper/search/filtercache.py", line 124, in parse
  File "pysper/search/filtercache.py", line 124, in <listcomp>
KeyError: 'date'
[47050] Failed to execute script sperf

Issue here is the same output but different scenario https://github.com/DataStax-Toolkit/sperf/issues/10


Don't coerce rational splits to decimal

This was blowing up in production with "Splits add to more than 100%":
rate = Rational(-5, 1201)
Money.new(1).allocate([rate, 1-rate])

This PR ensures that we don't needlessly coerce rationals to decimal when checking if "splits add to more than 100%".
I'm currently running a Shopify CI build against this branch to ensure no test failures.
@Shopify/merchant-data

lol at the branch name.
Good catch. :ship:

:+1:


Use super

We should use the super method when available instead of rewriting code. also changed update button for admin section to follow the same pattern as the rest of the application
cc @HospitalRun/core-maintainers

Looks good.  I'll merge.


ENH/BUG: Implement a NOT operator for sqlite impala and postgres

Closes #947 #952

Fixing merge conflicts now.

@wesm Merge in a few.


Earlgrey: bitstream doesn't work after parsing prim_fifo_sync

Parsing prim_fifo_sync.sv using Surelog/UHDM fails in vivado with errors:
ERROR: [Place 30-640] Place Check : This design requires more Slice LUTs cells than are available in the target device. This design requires 308330 of such cell types but only 133800 compatible sites are available in the target device. Please analyze your synthesis results and constraints to ensure the design is mapped to Xilinx primitives as expected. If so, please consider targeting a larger device. Please set tcl parameter "drc.disableLUTOverUtilError" to 1 to change this error to warning.
ERROR: [Place 30-640] Place Check : This design requires more LUT as Logic cells than are available in the target device. This design requires 307874 of such cell types but only 133800 compatible sites are available in the target device. Please analyze your synthesis results and constraints to ensure the design is mapped to Xilinx primitives as expected. If so, please consider targeting a larger device. Please set tcl parameter "drc.disableLUTOverUtilError" to 1 to change this error to warning.
ERROR: [Place 30-640] Place Check : This design requires more DSPs cells than are available in the target device. This design requires 817 of such cell types but only 740 compatible sites are available in the target device. Please analyze your synthesis results and constraints to ensure the design is mapped to Xilinx primitives as expected. If so, please consider targeting a larger device.
ERROR: [Place 30-640] Place Check : This design requires more DSP48E1 cells than are available in the target device. This design requires 817 of such cell types but only 740 compatible sites are available in the target device. Please analyze your synthesis results and constraints to ensure the design is mapped to Xilinx primitives as expected. If so, please consider targeting a larger device.

Blocked by: https://github.com/alainmarcel/Surelog/issues/1100, UHDM lacks of information about array in array access.

After Surelog fix, bitstream is now generated by vivado, but it is not working on HW.


docs: add allmytoes.yazi

Thank you for creating this awesome plugin!
BTW some users have asked me before if they can generate thumbnails using Allmytoes. I'm glad to see it implemented as a plugin!


Set up a documentation page showing other people's uses of Tasks, for ideas

Description
Seeing this page made me think it would be useful for Tasks to have something similar:
https://charts.phibr0.de/Meta/Charts/Other+Resources
Required Steps
Candidate template vaults:

https://github.com/mulfok/periodic-note-templates

Videos on using the Tasks plugin:

(find some!)

How to Test
No response

@aubreyz showed me a really nice rich system where custom statuses are used to indicate the type of a piece of information, in a body of research.


Add Ć, Felix, Flix, Gocaml, Mint, Muon and Peregrine

Adding a language

[x] The code displays "Hello World"
[x] I have updated the readme to include the new language
[x] I have incremented the language count in the readme
[x] I have no association with the language
[x] There are no copyright issues with this code
[x] The language has not been added prior to this pull request

Link to programming languages:
Peregrine: https://github.com/peregrine-lang/Peregrine
Gocaml: https://github.com/rhysd/gocaml
Felix: https://github.com/felix-lang/felix
Flix: https://flix.dev/
Mint: https://www.mint-lang.com/
Ć: https://github.com/pfusik/cito
Muon: https://github.com/nickmqb/muon

Check again, and then twice over

my bad

any issue with mergin it?


Proposal: Add Makefile for common tasks

This PR is optional. Using the Makefile was helpful for me lately and maybe it could be useful for others, so please give feedback.
One reason to not add this: It might confuse people not familiar with the project structure. Instead of following the documentation, using a target of the Makfile without a set-up will fail.
But the intention of this file is not to create an automatic build flow, just a way to run some tasks for the experienced user.
Instead of copying the commands from the documentation or using the
shell history to find the commands use a Makefile as a convenience to
run some common tasks.
A shell's completion system can make use of the targets and provide tab
completion.

Thanks for the review Philipp. I did some clean up. Should there be any other targets included? I just used the ones I am familiar with. But I guess if there is a need by someone, more can be added later.


Fails while loading yaml with numeric keys

Hi!
I currently work on a project with translations in yml.
We manage errors by assigning numeric codes to them which we use as keys. E.g
"000": System error
But I get the error:
Octal literals are not allowed in strict mode.
After searching hard, I realised that JS was having issues with having numeric keys with leading zeros in an object while using strict mode.
Searching further, I also confirmed it because of the use of the package tosource.
Duplicating the code and replacing the line where tosource was used solved my problem.
Hence I propose that we should also allow for use of raw option if it works in some cases.
With the right permissions, I can create a PR of a solution.

Thanks for contributing this issue! We'd gladly welcome a PR adding this option 👍
You should be able to fork this repo and create a PR from it


fix hibernate error in book sub-folder

Description
fix hibernate error in book sub-folder
Related Issue :
How Has This Been Tested?

create sub-folder in workspace
click folder
see folder without error

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

run build
deploy to 5


Disable Arc PG Manage action and clean up commands

The manage* commands weren't being used so removed them
Updated activation events with correct set of commands
Disabled PG manage action until the dashboard can be fixed

Coverage remained the same at 40.332% when pulling ed1234eb7ccf386b77f077e50aec0fd199ba306f on chgagnon/disablePg into 384553c2312b4bbcbe0bd641cf3fc9901cda41d3 on main.


graphql peer dependency should be updated to include ^0.11.0 as well

https://github.com/graphql/graphiql/blob/476aa2d8380e3d3d83f63ae7fda4347fb8b52e1a/package.json#L58
should be updated to also include ^0.11.0, right?

Yes.


📝 Readme.md: Add explanatory comments to usage

Based on: https://github.com/blakeembrey/pluralize/pull/91#issuecomment-379467416

Coverage remained the same at 100.0% when pulling dc7d87d35305b775adf138562f05094efcd11577 on pzrq:patch-1 into 0a9407718092a44abb8e67400ba2371c56576434 on blakeembrey:master.

Thanks @pzrq! Made some changes to the language used, but LGTM 😄


add forceRedraw option and ability to update payload without updating progress

New forceRedraw option forces render update to redraw the bar even if string has not changed. This is useful in situations when another piece of code is writing to the terminal (e.g. outputting logs while progress bar is running). Without this option progress bar disappears until the next progress update. When a lot of things get logged it makes progress bar virtually invisible.
It also adds ability to update payload information without touching the progress value itself. Can be useful when progress bar gets updated in multiple different scopes at the same time.

hi @dolsem
thanks for your contribution.
i'll check the changes in the next days/next weekend

look fine - it will be added to the next release


How do I export a unbuilded widget?

var frame = ExportFrame(
  frameId: 'someFrameId',
  exportDelegate: exportDelegate,
  child: RawImage(
    image: uiImage,
  ), // the widget you want to export
);

frame.build(context);

await exportDelegate.exportToPdfPage('someFrameId');

I tried the approach above, but the frame.build(context); part seems odd.
How can I export the frame without building it?

To export an unbuilt widget, one would have to call _exportWidget directly.
This is the case since the 'ExportFrame's are registered as soon as they are built.
Since the widget is not built yet no frame will be found. Calling _exportWidget directly would bypass all the logic of registering and getting the corresponding frame.
To be able to call the _exportWidget method (https://github.com/joseph-grabinger/flutter_to_pdf/blob/main/lib/export_delegate.dart#L94) it must not be private, so it could be called from outside the ExportDelegate.
This is how you then could call exportWidget :
Widget widget = SomeWidget();
final pw.Widget pdfWidget = await exportDelegate.exportWidget(widget, null);

// Add the pdfWidget to a document.
final pw.Document pdfDoc = pw.Document();
pdfDoc.addPage(pw.Page(
      build: (pw.Context context) => pdfWidget,
));

If this is what you're looking for, feel free to open a PR that implements this functionality so unbuilt widgets could be exported. Within the PR you could then test the functionality to ensure everything works as expected, as this explanation is only theoretical and not tested.
Note: Looking back it,  would also be possible to just implement a non-private wrapper function for the _exportWidget  method. Maybe something like:
Future<pw.Widget> exportUnbuiltWidget(Widget widget) async => await _exportWidget(widget, null);


The "Explain" block attempts a POST request that returns a 405 (method not allowed) status when attempting to explain code

When using the "Explain" block, which is supposed to explain code using ML, it returns a blank response. The reason for this is that it's trying to make  POST request that returns a 405 response (method not allowed).
Some reasons why this might be happening might be:

The server does not accept POST requests to that URL or there’s a redirect involved converting the request to a GET.
Your browser is making a preflight request, which uses the OPTIONS HTTP method. This is to check whether the server will allow the POST request (the 405 status code is sent in the response to the OPTIONS request, not the POST request).
The POST method might not be allowed on which you are sending the request and the request is simply denying you to make any POST request by sending 405 error response

See the screenshots below for more details. The endpoint it is trying to reach is "https://blocks-sandbox.githubnext.com/api/explain", so it could also be that you have to remove the 'sandbox' part from it.
Screenshot of the POST request that's not being allowed

Example of 'blank' explanation returned by the "Explain" block

The "Explain" block

Relocated to the right repo (githubnext/block-examples)


Debugging JS based Control AddIns

Hello,
today I successfully tested the first JavaScript based control add-in with AL on Dynamics 365.
The implementation of my existing add-ins works without any problems - GOOD WORK!! :)
But is there a possibility to debug the control add-in directly in VS Code?
If I try to set a breakpoint inside my js files there appears a warning message "file is not part of the current workspace".
It is possible to debug the control add-in js files in VS Code?
At the moment, I only can debug in browsers.
Thank you!

This is already possible using VSCode chrome debugger.

Install Chrome debugger extension
Make sure Chrome has remote debugging enabled.
In launch.json, add a new configuration for attaching to Chrome:

{
  "type": "chrome",
  "request": "attach",
  "name": "Attach to Chrome",
  "port": 9222,
  "webRoot": "${workspaceRoot}"
}

Click F5 to start AL debugger.
In VSCode Debug panel, choose "Attach to chrome" and run it as well. At this point we are running two debuggers - AL debugger connected to the server and Chrome debugger connected to the browser.

Hit some JavaScript code. I think it could be a bit problematic to debug StartupScript, since it is embedded directly in HTML code. I couldn't do it from the first try, but maybe Chrome debugger can be configured to debug JS inside HTML as well. Other scripts work fine though.

See attached sample project.
ControlAddIn.zip
JS breakpoint when clicking a button in the control add-in:


support for hip-profiles has been removed in 10.1.5 and 10.2.x, but panos still tries to put the hip-profiles in security rules

Describe the bug
When using ansible or the panos python sdk to create security rules, the panos python sdk will add a default element of hip-profiles with the value of Any into the request xml.  Such requests will fail on 10.1.5 and 10.2.x because support for the hip-profiles element in security policies has been removed.  This breaks all ansible playbooks that manage security policies on newer versions of panos.  Yuck!
Expected behavior
Security policies creation or updates should succeed without failure.
Current behavior
security policy creation and commits fail because of extraneous hip-profiles elements in the request xml.
Possible solution
Modify versioning so that on version 10.1.5 and 10.2.x, hip-profiles elements are not submitted.
Steps to reproduce

Start with a firewall running panos 10.1.5
Try to create any security policy using the panos python sdk
Cry as you realize that you can no longer do so, and find a dark quiet corner in which to be alone with your thoughts.
Grab kleenex to wipe away the tears as you realize that your automation stacks no longer work.

Your Environment
AWS vm-series firewall running PanOS 10.1.5 (we were afraid to upgrade to 10.2.0 because of the whole "you might lose your ip addresses from time-to-time" issue.

Version used: 10.1.5
Environment name and version (e.g. Chrome 59, node.js 5.4, python 3.7.3): python3.8.10 panos==1.17.0
Operating System and version (desktop or mobile): wsl under windows 10 on desktop, focal 20.04.3

PAN-OS 10.1.5-h1
pan-os-python 1.7.1
Expected:
 or
Observed:

Msg:
hip-profiles unexpected here

This is fixed in 1.7.2

Patch in SDK 1.7.2 fixes the issue for PAN-OS 10.1.5+
Same issue is also affecting PAN-OS 10.0.9, can you please also apply same patch to that release ?

Patch in SDK 1.7.2 fixes the issue for PAN-OS 10.1.5+ Same issue is also affecting PAN-OS 10.0.9, can you please also apply same patch to that release ?

How can I do a patch in SDK 1.7.2? do I have to replace the SDK version?


short-url https://git.io/kube-dashboard no longer supported by kubernetes

The k8s PWK welcome screen states:
 3. (Optional) Initialize kube-dashboard:

 curl -L -s https://git.io/kube-dashboard  | sed 's/targetPort: 9090/targetPort: 9090\n  type: LoadBalancer/' |\
    kubectl apply -f -

According to kubernetes/dashboard#2477, https://git.io/kube-dashboard is no longer supported, and should probably be https://raw.githubusercontent.com/kubernetes/dashboard/master/src/deploy/recommended/kubernetes-dashboard.yaml

Yes it should point to full link as git.io shortened links can not be updated, we prefer not to use them anymore.

Instructions have been updated. Seems like kube dashboard now uses https and requires authentication. @floreks is there a way to disable both?. As we'd like the PWK experience to be as smooth as possible.

Just found the alternate deployment method. Now figuring out a way to deal with the Authorization header.


updated test case to make it build successfully

was getting message as below
Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.874 s <<< FAILURE! -- in org.springframework.ai.reader.pdf.PagePdfDocumentReaderTests
Thank you for taking time to contribute this pull request!
You might have already read the [contributor guide][1], but as a reminder, please make sure to:

Sign the contributor license agreement
Rebase your changes on the latest main branch and squash your commits
Add/Update unit tests as needed
Run a build and make sure all tests pass prior to submission

Not sure how you got this error?
Locally and on GH's build it works expected:

[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.389 s -- in org.springframework.ai.reader.pdf.PagePdfDocumentReaderTests
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

Not sure. I am attaching surefire-reports here.
Which I executed today 1-Nov-23 at 9.45pm CST>
org.springframework.ai.reader.pdf.PagePdfDocumentReaderTests.zip


Can't start a stack if catalog version > 9 since v1.0.0

Rancher Version: 1.0.0
Docker Version: 1.10.3
OS: Ubuntu 14.04 LTS
Steps to Reproduce: Create a catalog with version > 9, (template/foo/10) then try to start a stack from this template
Results: Stack does not start and there is no message in rancher server log
Expected: Stack starts

It used to work in v0.63.1

This is really blocking when using catalogs as a mean of deployment workflow.

Reproduced bug:
https://github.com/LLParse/community-catalog/tree/issue-4195-test/templates/issue-4195
Empty stack appears but nothing is scheduled

@raphink Agreed. I haven't been able to deploy for 2 weeks due to url bugs, haha!

@willseward if you told us about it 2 weeks ago it wouldn't have been in 1.0.0 :smile:
Anyway, this will be in 1.0.1.

No, sorry, a different issue was affecting the deployment. You all have been great about everything!
Congrats on 1.0! I'm very proud to be introducing this technology to Texas Christian University

Awsome!   I was able to compile the fix from your PR branch and install the updated rancher-catalog-service and I now have deployments working again.   This should take care of us until v1.0.1.

Tested with Rancher server Version: 1.0.1-rc1.
Created a catalog with version > 9, (template/foo/10, template/foo/100) .
Able to start the catalog by choosing a version > 9 successfully.

Great, looking forward to it!


Does LLaVA-NeXT support 336x336 image inputs, like LLaVA-1.5?

Hi there! 😊
First of all, thank you so much for your amazing work on LLaVA-NeXT!
I was reading about the performance improvements and how it maintains the minimalist design and data efficiency of LLaVA-1.5, which is fantastic! 🎉
I was wondering, does LLaVA-NeXT also support 336x336 image inputs, similar to LLaVA-1.5?
Thanks again for all your hard work—looking forward to hearing from you! 😄

"image_aspect_ratio": "anyres_max_9",   "image_crop_resolution": null,   "image_grid_pinpoints": [     [       384,       384     ],     [       384,       768     ],     [       384,       1152     ],     [       384,       1536     ],     [       384,       1920     ],     [       384,       2304     ],     [       768,       384     ],     [       768,       768     ],     [       768,       1152     ],     [       768,       1536     ],     [       768,       1920     ],     [       768,       2304     ],     [       1152,       384     ],     [       1152,       768     ],     [       1152,       1152     ],     [       1152,       1536     ],     [       1152,       1920     ],     [       1152,       2304     ],     [       1536,       384     ],     [       1536,       768     ],     [       1536,       1152     ],     [       1536,       1536     ],     [       1536,       1920     ],     [       1536,       2304     ],     [       1920,       384     ],     [       1920,       768     ],     [       1920,       1152     ],     [       1920,       1536     ],     [       1920,       1920     ],     [       1920,       2304     ],     [       2304,       384     ],     [       2304,       768     ],     [       2304,       1152     ],     [       2304,       1536     ],     [       2304,       1920     ],     [       2304,       2304     ]   ],   "image_split_resolution": null,


Datasets Update

@jph00
In this PR I added functionality to enable 2 high level functions (examples in index.ipynb).  These 2 high level functions allow to either pass a list of libraries or a requirements.txt to maintain and update kaggle datasets.
One this is approved and merged I plan to use fastkaggle to generate lots of top used ML datasets and schedule them to auto push updates to Kaggle datasets for anyone to use - so that for commonly used libraries people can use the ones I maintain and not have to manage their own.

Somehow my notebooks are being seen as invalid json after cleaning.  I am closing this PR out until I debug and figure out what's going on with it.


PIM ARM Role Management Policy (List) returns a different "rules" object if you have made edits to it via the API

When I use the Role Management Policies List API on any object I haven't also edited via the Update API, it returns a response like the below***** :
scope                 : /subscriptions/5715b9e3-5625-447f-89ea-bed0a29e57a6/resourceGroups/MadeUp-RG
isOrganizationDefault : True
lastModifiedBy        :
rules                 : {@{isExpirationRequired=True; maximumDuration=P90D; id=Expiration_Admin_Eligibility; ruleType=RoleManagementPolicyExpirationRule; target=}, @{enabledRules=System.Object[]; id=Enablement_Admin_Eligibility;
                        ruleType=RoleManagementPolicyEnablementRule; target=}, @{notificationType=Email; recipientType=Admin; isDefaultRecipientsEnabled=True; notificationLevel=All; id=Notification_Admin_Admin_Eligibility;
                        ruleType=RoleManagementPolicyNotificationRule; target=}, @{notificationType=Email; recipientType=Requestor; isDefaultRecipientsEnabled=True; notificationLevel=All; id=Notification_Requestor_Admin_Eligibility;
                        ruleType=RoleManagementPolicyNotificationRule; target=}...}
effectiveRules        : {@{isExpirationRequired=True; maximumDuration=P90D; id=Expiration_Admin_Eligibility; ruleType=RoleManagementPolicyExpirationRule; target=}, @{enabledRules=System.Object[]; id=Enablement_Admin_Eligibility;
                        ruleType=RoleManagementPolicyEnablementRule; target=}, @{notificationType=Email; recipientType=Admin; isDefaultRecipientsEnabled=True; notificationLevel=All; id=Notification_Admin_Admin_Eligibility;
                        ruleType=RoleManagementPolicyNotificationRule; target=}, @{notificationType=Email; recipientType=Requestor; isDefaultRecipientsEnabled=True; notificationLevel=All; id=Notification_Requestor_Admin_Eligibility;
                        ruleType=RoleManagementPolicyNotificationRule; target=}...}
policyProperties      : @{scope=}

Note the isOrganizationDefault setting is True. This role has not been touched in anyway. In this case, this is the Log Analytics Reader role policy above. If I then expand the rules property specifically, this is what it looks like, consistent with what the response documentation says you should get back :

(screenshot trimmed for brevity)
However, after editing the policy via the Update API, I now get a rule attribute formatted completely differently. Exact same rule after being edited:

(comprehensive screenshot this time)
I've only edited the isExpirationRequired in my specific testing. I am using the APIs via the Invoke-RestMethod cmdlet in PowerShell. The different formatting returns the same even if it has been edited in the portal. The only difference I can trigger is if the API has made an update.
Quite frankly the second response is easier to read and handle I believe.  That being said, is this change in response structuring intended?
*****I'm filtering for just a single policy related to a specific role, as is shown here.

Adding Service team to look into this.


Added String functions in Java

#171 Issue Add string functions -Added inbuilt string functions in Java

This PR is from an invalid issue.


New Option: Use SwiftPM for building when possible

Currently this projects uses Carthage for building frameworks. While this especiallly is a great solution for frameworks specifically written for iOS etc. (#18 being an improvement which drops the Carthage requirement), some libraries were written for SwiftPM usage only and might not integrate well with the current approach in Accio.
For example, when I tried adding NIOTransportServices to an iOS application, it build all just fine with Accio but then when running the app, I got linker errors and the app crashed.
Maybe the approach described in this article or used by this tool might work better in such cases.
Therefore investigating other approaches of building and integrating frameworks could bring Accio another step forward. This doesn't mean we should remove the current behavior though, I more feel like this would be an option one might choose for specific libraries or Accio could even figure out itself, which libraries this approach should be applied to.

Closing this, as Accio is now deprecated.


How can i know websocket connecting status?

client.isConnecting()  is also return false
client.isOpen is also return true;
but if phone out of network
client.isOpen will return true
how can i get this websocket status?

Hello @maybea,
the best way to detect a lost connection is to ping the client and check if it succeeds.
E.g.
public void sendAllPing() {
    FramedataImpl1 frame = new FramedataImpl1(Opcode.PING);
    frame.setFin(true);
    Collection<WebSocket> con = connections();
    synchronized ( con ) {
        for( WebSocket c : con ) {
            c.sendFrame( frame );
        }
    }
}

Even tough, yes I see the need to improve lost connection detection.
Greetings
marci4


Year added to return by date for FWMT

Motivation and Context
FWMT tool requires the return by date sent to them to include the year.
What has changed
SOCIALICF added to clause where it decides which date format to use. A unittest has been added to test that the correct date is used, and the existing social and business unittests have had an action type added to ensure they have the correct date formats.
How to test?
This can be tested by creating two collection exercises in the rasrm-ops tool locally, one for SOCIALICF and one for SOCIALNOT. Once these are ran, you should see that the messages on the Field RabbitMQ queue have the return by date in the format dd/mm/yyyy, and the messages on the Printer queue should have the return by date in the format dd/mm.
Links
https://trello.com/c/kvBHikuB

 BUG302: Send 'return by' year to FWMT


NuxtJs - Navigator is not defined

Hi everybody,
We are using handsontable-pro version through the library @handsontable-pro/vue.
I defined my plugin and load it at ssr false but i got this error :
navigator is not defined in the function setBrowserMeta() of node_modules/handsontable/commonjs/helpers/browser.js
My plugin file looks like :
import Vue from 'vue'
import HotTable from '@handsontable-pro/vue';

export default HotTable;

Thanks a lot for your help.
Best regards,

Hi @Geminii
do you not import the template?
In my case it's
import SampleApp from './sampleApp.vue';

Sorry for misunderstanding.
Can you try to change
let HotTable = null;

  if (process.browser) {
   HotTable = require('@handsontable-pro/vue')
  }

in the app.vue
instead of import HotTable from '~/plugins/handsontable';
and then go with
build: {
    vendor: [
      'handsontable'
    ],
  }

in the nuxt.config.js file

OK i finally did it among to you :)
If someone is interresting to include handsontable into NuxtJs :
nuxt.config.js file :
build: {
  vendor: ['handsontable'],
},
plugins: [
   '~/plugins/vue-handsontable'
]

vue-handsontable.js file :
let HotTable = null;

if (process.client) {
  HotTable = require('@handsontable-pro/vue')
}

export default HotTable

app.vue file :
<script>
  import HotTable from '~/plugins/vue-handsontable'

  export default {
    data: function() {
      return {
        settings: {
          data: [
            ["", "Ford", "Volvo", "Toyota", "Honda"],
            ["2016", 10, 11, 12, 13],
            ["2017", 20, 11, 14, 13],
            ["2018", 30, 15, 12, 13]
          ],
          colHeaders: true,
          rowHeaders: true,
        }
      };
    },
    components: {
      HotTable
    }
  }
</script>

And job is done :)
Thanks a lot again @AMBudnik for your quickly help !
Have a good day.
Regards,

Great! Thanks for an update :)

Hey @KWeaver87
sorry for keeping you waiting.
I just got a reply from our CTO that

User mentions that HoT works when navigating from other subpage, that's when page is rendered by the browser, not server side. Another reason to believe the problem is with SSR. But have to be confirmed.

but if you could share the whole code it will be much easier to debug it.

Thanks for the reply! I just pushed the code out to a new repo: https://github.com/KWeaver87/MappingFrontEnd
Apologies if things are messy or hard to follow, I'm still fairly new at development. This is just a side project for an internal tool at work, so I don't have much direct support from my co-workers.

Hey @KWeaver87
I got an error after running the server

can you write me at<EMAIL_ADDRESS>

This is what worked for me:
// @ts-ignore
const HotTable = !process.client ? null : require('@handsontable/vue').HotTable
export default HotTable

Atry it this solution but hi have this error
ailed to compile with 1 errors                                                                                friendly-errors 14:20:48
This dependency was not found:                                                                                friendly-errors 14:20:48
friendly-errors 14:20:48

nuxt_plugin_vuehandsontable_755616e5 in ./.nuxt/index.js                                                                 friendly-errors 14:20:48
friendly-errors 14:20:48
To install it, you can run: npm install --save nuxt_plugin_vuehandsontable_755616e5

@venkatperi Hi,  I follow your code.  But when I refresh the page, It's still has error info : navigator is not defined. Hope to get your help! thanks

I am implementing handsontable in nuxt js and facing same error. Please help

Hey Guys,
please share the code if you're still facing this issue. I'd build it on my device and check what's wrong.

Missing  navigator is a configuration issue with SSR. Please follow this article: https://medium.com/@codebeast_/why-your-third-party-plugin-dont-work-in-nuxt-and-how-to-fix-it-d1a8caadf422 to configure SSR for Handsontable


feat: update bundle-loader endpoint to v3

Changes:

Update bundle-loader to v3
Change display name for app id parameter

@TomyCesaille Should this change have backward compatibility and support loading the bundle still if the applicationId is a number?

@TomyCesaille Should this change have backward compatibility and support loading the bundle still if the applicationId is a number?

Ah! Excellent question. If someone updates the tags in GTM without looking at the Gorgias page... This person might upgrade its applicationId number to v3.... that's bad because this endpoints expects the appKey 🙈
In that case, I think the error returned in the browser is clear enough for support lvl1 to kick-in and solve the problem with the customer. Looks like the best option to me because 1. we don't do backward compatibility 2. we help merchants to migrate in v3 for real.

I think there is another description file to update (version at least)

Also, make sure that this endpoint works :D (with the GTMContainerId parameter, since this part of the code was dormant for a year or so)

@TomyCesaille The endpoints supports the GTMContainerId query param

I believe the metadata.yaml file will have to be updated after this PR is merged? Since we will get a new commit sha?

@gorgias-botond-dombi yes, we need to use the sha commit from this PR, and could you please also update the doc link in there for this one?
Thanks

I've pushed another commit to update the metadata.yaml @juancarlosqr included the doc update too


US Census Geocoder addition

Email from Micheal Slade:
I have just started doing geocoding in Python, primarily trying to use
http://www.census.gov/geo/maps-data/data/geocoder.html
for batch geocoding but haven't been able to retrieve any data.
Any possibility of you adding this one to your set of geocoders?
I think that all the other US geocoders eventually rely on maps, etc from
the Census such as the TIGER maps so it would be a great primary source!

Unfortunately I don't have much time to concentrate on the Python Geocoder, however Pull Requests/contributions are welcomed.
There are a lot of examples in the Python Geocoder source code that will show you how to add a new provider.
Mapzen
https://github.com/DenisCarriere/geocoder/blob/master/geocoder/mapzen.py
Bing
https://github.com/DenisCarriere/geocoder/blob/master/geocoder/bing.py

USCensus is currently supported (refactored, and tested)
I guess we can close this ticket :-)
If I have missed something, just open it again with a comment !


Properly set the Cache-Control to no-cache for index.html

This prevents caching index.html across deploys so users get the latest
index.html and fingerprinted assets.
index.html was being cached so that after a deploy you would receive a cached file. It would require a full page refresh to get the latest code. I've been running this change in production for a while, and it solves the problem. expires -1 correctly sets Cache-Control: no-cache.

Thanks for the heads up! Looks good.


Web - location API

Create API version for web location to gather all informations on current HTML version but in JSON (xml, geojson etc).

coordinates (WGS 84, MGRS...)
various codes (OLC, Geohash, Ge0Code, ...)
websites (Google, Waze, Mapy.cz, ...)
address

In future:

timezone area (eg Europe/Prague)
current time

Link could be something like https://better-location.palider.cz/50.087451,14.420671?output=json

Already done, just little different URL: https://better-location.palider.cz/50.087451,14.420671?format=JSON


Validator node hang after upgrade from v0.3.1 to v0.4.0

Validator node
Since the 0.4.0 version update, there have been about 10 reported issues, including mine. When nodes are stopped and restarted, some nodes get stuck and fail to restart. The problematic nodes repeatedly fetch only a specific part of a block's TX and do not receive the next block information, causing the block height to not increase.
The above issue seems to have been experienced by some users. This issue occurred when updating to 0.3.1 and 0.4.0 governance.
Steps to reproduce:
Upgrade manually from v0.3.1 to v0.4.0, in my case it happened to 2 public RPC nodes, while the validator node (not serving as public RPC) was not stuck

If it’s stuck on “executed block,” it may be because the background process is currently pruning. Try setting pruning to “nothing,” then restart to see if it still gets stuck there.

@roycclu could you ping the bug reporter here for the follow up?


The example title says its for .NET but all the code examples are in Azure CLI

I clicked on the link for "Set and retrieve a secret -.NET".  I expected the examples to be in C#.  However, all the examples are in Azure CLI.  Do you have examples of how to use this in C#?

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

ID: bc43d849-5b06-7185-0c9e-aaf240363aca
Version Independent ID: 385f4dfe-326a-9e63-80ce-3c71903edf3c
Content: Quickstart: Set and retrieve a secret from Azure Key Vault by using a node web app - Azure Key Vault
Content Source: articles/key-vault/quick-create-net.md
Service: key-vault
GitHub Login: @prashanthyv
Microsoft Alias: barclayn

@clisc Thank you for your feedback . We will investigate and update the thread further.

There is a good blog post on this that you can reference.
As for Microsoft official documentation, this might be a better guide to use than the one here. https://docs.microsoft.com/en-us/azure/key-vault/tutorial-net-create-vault-azure-web-app
There is a Nuget package that allows you to connect to Azure Keyvault without specifying the Azure Active Directory Client Id.
Microsoft.Azure.Services.AppAuthentication
You also need to install the Microsoft.Azure.KeyVault nuget package.
using Microsoft.Azure.KeyVault;
using Microsoft.Azure.Services.AppAuthentication;

...

var azureServiceTokenProvider = new AzureServiceTokenProvider();
var keyVaultClient = new KeyVaultClient(
     new KeyVaultClient.AuthenticationCallback(azureServiceTokenProvider.KeyVaultTokenCallback));
var secret = await keyVaultClient.GetSecretAsync(
    "https://{{my-vault-name}}.vault.azure.net/", "{{my-secret}}");

If you would like more documentation I would recommend leaving this request in User Voice where the product team can directly address it. https://feedback.azure.com/forums/169401-azure-active-directory

We will now proceed to close this thread. If you have further questions feel free to tag me in the comments or reach out on MSDN and I will gladly continue the discussion. https://social.msdn.microsoft.com/Forums/en-US/home?forum=WindowsAzureAD%2CAzureKeyVault&filter=alltypes&sort=lastpostdesc

It doesn't look like the issue with the documentation was fixed:

Can you see how I clearly clicked on "Set and retrieve a secret - .NET" but the example on the right shows "Azure CLI"?  This is not correct.  Azure CLI is the first item in the list of Quickstarts in the menu on the left.

How do I reopen this issue since it was not resolved?

Hi @clisc ,
The title and the document are correct and intentional. We want to demonstrate how to retrieve a secret in .NET. But before that all other commands are usually 1 time and can be done via Command Line. Is there anything specific you want to see done in .NET?

I guess I don't understand.  I only see "Azure CLI" code examples in the quickstart and I don't see any .NET code at all.  Where are you demonstrating how to retrieve a secret in .NET?

If you go through the entire tutorial starting this section - https://docs.microsoft.com/en-us/azure/key-vault/quick-create-net#clone-the-repo which asks you to clone a repo is written in c#. And it shows how to fetch a secret from Key Vault in C#

Thanks - I was looking for the code inline with the documentation like other examples and did not see that the example had to be downloaded.

@clisc - Glad you found it. In the other example also we ask for users to git clone the repo - https://docs.microsoft.com/en-us/azure/key-vault/quick-create-node


az aks create not working with ServicePrincipal secret stored in Keyvault

Describe the bug
I want to create an AKS cluster with the command shown below. A ServicePrincipal, which has the acrpull permission should be used to pull images from an attached ACR. This ServicePrincipal was created using az ad sp create-for-rbac with the cert being stored in a Keyvault.
It is unclear to me how I can correctly specify the ServicePrincipal's secret (the Keyvault cert/secret) in the --client-secret command line argument. I outlined steps I tried in the last section of this post.
Command Name
az aks create
Errors:
Operation failed with status: 'Bad Request'. Details: The credentials in ServicePrincipalProfile were invalid. Please see https://aka.ms/aks-sp-help for more details. (Details: adal: Refresh request failed. Status Code = '401'. Response body: {"error":"invalid_client","error_description":"AADSTS7000215: Invalid client secret is provided.\r\nTrace ID: 4435d409-074a-4c4d-9429-efe8925e1500\r\nCorrelation ID: 51dac364-9a0c-496e-8b2d-ac0911506c1f\r\nTimestamp: 2020-04-15 10:40:05Z","error_codes":[7000215],"timestamp":"2020-04-15 10:40:05Z","trace_id":"4435d409-074a-4c4d-9429-efe8925e1500","correlation_id":"51dac364-9a0c-496e-8b2d-ac0911506c1f","error_uri":"https://login.microsoftonline.com/error?code=7000215"})

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Environment Summary
macOS-10.15.3-x86_64-i386-64bit
Python 3.8.2

azure-cli 2.2.0 *

Additional Context

The command is working when the SP is authenticated with a password.
Tried both /certificates and /secrets as segments in the fully qualified ID of $(CERT_ID).
Tried passing specific certificate versions as suffix to the fully qualified ID of $(CERT_ID).
Gave all possible Keyvault permissions through the Keyvault policies to both SP1 and the ServicePrincipal that is running the az commands.
Enabled and disabled Azure Virtual Machines for deployment setting in the Keyvault.
This has been reported in 2018 already but was unanswered: https://github.com/Azure/azure-cli/issues/8133 .

Thanks in advance.

thanks for reporting, routing to appropriate team

The --client-secret argument does not accept a keyvault certificates.  You have to use a service principal with password based authentication.

Closing this issue for now as the answer as been addressed above. Feel free to reopen if this doesn't answer your question!


Namespace support seems to be incomplete

I can't get it to work right. I'm assuming that it's not fully implemented in the version 9.5.0
In this example

2012!

let rmfNS = "http://www.frankston.com/public";
hp.setAttributeNS(rmfNS, "test", "value");

2012!

When I try to get the attribute using the NS name I get nothing. The value of hp.attributes[0] is {} (according to JSON.stringify). When I set the attribute the namespace is not set.

So, we probably do have bugs here. But, some troubleshooting tips:
JSON.stringify will not give useful values in jsdom, just like in the browser. I'd suggest logging hp.attributes[0].namespaceURI, localName, and prefix.
It would be ideal if you had a test that worked in https://jsbin.com, but not in jsdom, i.e. using a


Consolidate init recipes

Hello! There are a few small changes here:

A new attribute: node["lita"]["daemon_group"]. Using a 'nobody' group is technically incorrect, even though you can get away with it sometimes. 'nogroup'' is the correct group to use with the 'nobody' user.

I threw a guard on including the apt cookbook to only include it for relevant platform families.

Lastly and the primary reason for this PR: I consolidated the SysV and Runit init recipes into a single recipe. I'm interested in adding CentOS 7 support to the cookbook, which will likely means a systemd service file for managing Lita. The return strategy to optionally include those recipes will get messy fast.

I wanted to get your take on consolidating this before I get too deep into the CentOS stuff.

I'm +1 on this. Thanks for your work!

I'm definitely +1 on this. A took a PR using the return method because the submitter said it was becoming the pattern of choice and I didn't have any info to dispute that. But my preference has been (and was in the original) the way you are doing it now.
One thing, since I DID accept that PR, this is technically a breaking change (in the case that someone was calling the runit_service recipe directly). So I think (based on my limited understanding of semver) we'd have to bump the version to 2.0.0. But it's a relatively small change to do that for.
So I'd like to go two ways: A) we leave a version of runit_service in there that just calls init_service and as a log message with a deprecation notice ... or B) you can roll this up with any changes to support Centos and THAT would feel like a 2.0.0 change. Again, it's possible I'm being too nerdy with the semver. I don't feel that strongly about it, but I thought I'd get your thoughts before I merged/bumped/released.
And thanks for this. I appreciate your work a great deal.

Agreed on the SemVer point actually. It shouldn't block on merging this, but we should block on releasing to the supermarket until we have CentOS support.

I'm mostly done with CentOS support, if it's easier we can just do this as one PR.

Either way is fine. One side note, I'm on vacation this week so might be slow to respond.
Again, thanks for all of this work.

@notnmeyer any update on the CentOS support?  I would love to be able to use this cookbook on a CentOS host

@shortdudey123 My attention at work got pulled somewhere else and I haven't gotten back to working on CentOS support. It's still something I'm planning on getting back to but I don't have much in the way of an update unfortunately.
@harlanbarnes @jimmycuadra Based on your earlier feedback:

A) we leave a version of runit_service in there that just calls init_service and as a log message with a deprecation notice

I went ahead and added this. Shouldn't be a breaking change now. How are you feeling about getting this merged?

LGTM

Pulled in the commits, bumped version, added changelog, squashed into a single commit, tagged and released to SuperMarket. Thanks!


XML Doc comments referenced via include tag are not pulled into Swagger UI

I want to add a bunch of JSON examples to my controllers, but I don't want to swell the controller source file with a bunch of XML.
Fortunately, XML Doc Comments support the use of an include tag to reference a separate file and XPath.  So instead of having:
/// Summary goes here
/// <remarks>Remarks go here</remarks>

I can do the following:
/// <include file='./MyController.xml' path='/MyController/MyEndpoint'/>

Where the XML file would look like the following
<?xml version="1.0" encoding="utf-8" ?>
<MyController>
  <MyEndpoint>
    Summary goes here
    <remarks>Remarks go here</remarks>
  </MyEndpoint>
</MyController>

Obviously that is a very contrived example, but when you start adding examples and parameters and other things to the XML file, it can get pretty big (and if you do multiple examples in the remarks section like I do, it's a lot better off in a separate file).
So that all sounds great, except that Swashbuckle doesn't render the external comments in the UI.
I have created a quick ASP.NET Core WebApi project in .NET 7 that reproduces the issue and have attached it here.
In short, I created a PingController that has 2 methods - one with the docs embedded, and one with the docs referenced.  The first one renders; the second one does not:

I verified that the references do work by newing up the controller in a code snippet and verifying that when I hover over the call to the embedded controller the docs appear in IntelliSense:

And I verified that the docs are pulled into the generated .xml file on build:
<member name="M:XmlDocExamples.Controllers.PingController.ReferencedPing">
  <ReferencedPing>
    Does a ping.
    <returns>Returns a pong.</returns>
    <remarks>XML doc comments are included from a separate file.</remarks>
  </ReferencedPing>
</member>

I did notice that the generated doc has an extra tag layer around the referenced comments, and maybe that is causing a problem?  But I don't know how to not generate that, and if Visual Studio supports it, it seems to be working as designed.
mjl-XmlDocExample.zip

This is supported in DotSwashbuckle 3.0.4


Adjustments on the SignInButton Component as per requirement

Description
This is to address the issue #100 . I renamed the file names as per requirement. changed css class name to "button-sign-with-slack" and changed onclick to onClick.
Fixes #100
How to test?
It can be tested with storybook.
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

rework.. started afresh


Invalid eye transform

The plugin doesn't seem to properly set the eye transforms as the rendered 3D content is not stereoscopic.

I can observer the same issue when running the demo application on my quest. In the attached screenshot with only the table it looks like no stereo separation at all.

Hi,
i tried your changes that you mentioned previously from your branch at https://github.com/m4gr3d/godot_oculus_mobile/tree/fix_invalid_eye_transform.
I also tried to rewrite the the OvrMobileSession::get_transform_for_eye function to use directly the data provided in head_tracker.Eye[eye].ViewMatrix from the ovr SDK. This is how the VrCubeWorld_NativeActivity sample renders. This is also how godot_open_vr extracts the eye transform.
You can see my test implementation in https://github.com/NeoSpark314/godot_oculus_mobile/commit/ae2ad9c98931ba6c19f14c13f1c79ca12b76c46a
I then compared the resulting eye transform using godot_transform_as_string from your code and from using my code (ViewMatrix directly) rotation is identical; but the eye position is slightly different:
here is the Log output (first transform is my code; second transform line is your code; your code seems to only move the eye along x-axis independent of head rotation...)
08-04 17:04:08.408 14144 14160 V GodotOVRMobile: Transform Comparison for godot_eye 1
08-04 17:04:08.408 14144 14160 V GodotOVRMobile: 0.715116, -0.121121, 0.688432, 0.216813, 0.974734, -0.053725, -0.664531, 0.18768, 0.723308 - 0.163868, 1.011757, -0.242753
08-04 17:04:08.408 14144 14160 V GodotOVRMobile: 0.715116, -0.121121, 0.688432, 0.216813, 0.974734, -0.053725, -0.664531, 0.18768, 0.723308 - 0.154765, 1.018685, -0.263985
08-04 17:04:08.408 14144 14160 V GodotOVRMobile: Transform Comparison for godot_eye 2
08-04 17:04:08.409 14144 14160 V GodotOVRMobile: 0.715116, -0.121121, 0.688432, 0.216813, 0.974734, -0.053725, -0.664531, 0.18768, 0.723308 - 0.209564, 1.025612, -0.285218
08-04 17:04:08.409 14144 14160 V GodotOVRMobile: 0.715116, -0.121121, 0.688432, 0.216813, 0.974734, -0.053725, -0.664531, 0.18768, 0.723308 - 0.218667, 1.018685, -0.263985
From testing using the ViewMatrix directly feels quite good; but hard to tell if it is correct.
Regarding the other issue you mentioned of objects disappearing; it is still there also with my code; maybe a clipping issue?

@NeoSpark314 Good catch! Using the ViewMatrix is definitely the way to go.
I'm making headway with the clipping issue. The visual_server_scene has some strange logic to calculate the camera frustum when in VR, and disabling that logic fixes the clipping issue (here's the relevant commit.
@BastiaanOlij any idea why that logic would be an issue for the oculus_mobile plugin, and not for the other VR plugins?

@BastiaanOlij In addition, that change seems to drastically improve performance for the oculus mobile plugin. To a point where the requirement to enable extra latency mode (added by PR #24) is no longer necessary.

The code in the visual server attempts to create a view frustum that
encompasses the frustums of both eyes so we only need to cull once and do
stuff like updating shadows once. It may be that it fails if the frustums
are identical or not set up right.
On Mon, 5 Aug 2019 at 2:06 pm, Fredia Huya-Kouadio<EMAIL_ADDRESS>wrote:

@BastiaanOlij https://github.com/BastiaanOlij In addition, that change
seems to drastically improve performance for the oculus mobile plugin. To a
point where the requirement to enable extra latency mode (added by PR #24
https://github.com/GodotVR/godot_oculus_mobile/pull/24) is no longer
necessary.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GodotVR/godot_oculus_mobile/issues/32?email_source=notifications&email_token=AAO262NJ5MKHRDCG7S3ZLDTQC6RK3A5CNFSM4IHO6IN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3QT4NI#issuecomment-518078005,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO262PTLUWS6PCFTQNDRULQC6RK3ANCNFSM4IHO6INQ
.

--
Kindest regards,
Bastiaan Olij
https://www.facebook.com/bastiaan.olij
https://twitter.com/mux213
https://www.youtube.com/BastiaanOlij
https://www.youtube.com/channel/UCrbLJYzJjDf2p-vJC011lYw
https://github.com/BastiaanOlij

Btw, the change you did in the visual server turns stereo rendering off,
now you’re just rendering a normal camera. It makes sense it ‘fixes’ the
culling if it could merge the frustums together (which it only uses for
culling)
On Mon, 5 Aug 2019 at 2:25 pm, Bastiaan Olij<EMAIL_ADDRESS>wrote:

The code in the visual server attempts to create a view frustum that
encompasses the frustums of both eyes so we only need to cull once and do
stuff like updating shadows once. It may be that it fails if the frustums
are identical or not set up right.
On Mon, 5 Aug 2019 at 2:06 pm, Fredia Huya-Kouadio <
<EMAIL_ADDRESS>wrote:

@BastiaanOlij https://github.com/BastiaanOlij In addition, that change
seems to drastically improve performance for the oculus mobile plugin. To a
point where the requirement to enable extra latency mode (added by PR #24
https://github.com/GodotVR/godot_oculus_mobile/pull/24) is no longer
necessary.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GodotVR/godot_oculus_mobile/issues/32?email_source=notifications&email_token=AAO262NJ5MKHRDCG7S3ZLDTQC6RK3A5CNFSM4IHO6IN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3QT4NI#issuecomment-518078005,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO262PTLUWS6PCFTQNDRULQC6RK3ANCNFSM4IHO6INQ
.

--
Kindest regards,
Bastiaan Olij
https://www.facebook.com/bastiaan.olij
https://twitter.com/mux213
https://www.youtube.com/BastiaanOlij
https://www.youtube.com/channel/UCrbLJYzJjDf2p-vJC011lYw
https://github.com/BastiaanOlij

--
Kindest regards,
Bastiaan Olij
https://www.facebook.com/bastiaan.olij
https://twitter.com/mux213
https://www.youtube.com/BastiaanOlij
https://www.youtube.com/channel/UCrbLJYzJjDf2p-vJC011lYw
https://github.com/BastiaanOlij

I have the same culling problem when using two asymmetrical projection
matrices for stereo rendering on the Powerwall. Could the code for
combining the frustums have a problem with of center projections?
On Mon, Aug 5, 2019, 06:28 Bastiaan Olij<EMAIL_ADDRESS>wrote:

Btw, the change you did in the visual server turns stereo rendering off,
now you’re just rendering a normal camera. It makes sense it ‘fixes’ the
culling if it could merge the frustums together (which it only uses for
culling)
On Mon, 5 Aug 2019 at 2:25 pm, Bastiaan Olij<EMAIL_ADDRESS>wrote:

The code in the visual server attempts to create a view frustum that
encompasses the frustums of both eyes so we only need to cull once and do
stuff like updating shadows once. It may be that it fails if the frustums
are identical or not set up right.
On Mon, 5 Aug 2019 at 2:06 pm, Fredia Huya-Kouadio <
<EMAIL_ADDRESS>wrote:

@BastiaanOlij https://github.com/BastiaanOlij In addition, that
change
seems to drastically improve performance for the oculus mobile plugin.
To a
point where the requirement to enable extra latency mode (added by PR
#24
https://github.com/GodotVR/godot_oculus_mobile/pull/24) is no longer
necessary.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/GodotVR/godot_oculus_mobile/issues/32?email_source=notifications&email_token=AAO262NJ5MKHRDCG7S3ZLDTQC6RK3A5CNFSM4IHO6IN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3QT4NI#issuecomment-518078005
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAO262PTLUWS6PCFTQNDRULQC6RK3ANCNFSM4IHO6INQ

.

--
Kindest regards,
Bastiaan Olij
https://www.facebook.com/bastiaan.olij
https://twitter.com/mux213
https://www.youtube.com/BastiaanOlij
https://www.youtube.com/channel/UCrbLJYzJjDf2p-vJC011lYw
https://github.com/BastiaanOlij

--
Kindest regards,
Bastiaan Olij
https://www.facebook.com/bastiaan.olij
https://twitter.com/mux213
https://www.youtube.com/BastiaanOlij
https://www.youtube.com/channel/UCrbLJYzJjDf2p-vJC011lYw
https://github.com/BastiaanOlij
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GodotVR/godot_oculus_mobile/issues/32?email_source=notifications&email_token=AAAXWZFJ26NTIZZNVWOJGSDQC6T7VA5CNFSM4IHO6IN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3QUVXA#issuecomment-518081244,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAXWZGXGBU6NAHBHYBB5XTQC6T7VANCNFSM4IHO6INQ
.

Can you clarify how stereo rendering is turned off?
Before the change, the logic would call _prepare_scene once with the combined frustum, and call _render_scene twice, once for each eye.
With the disabling change, the logic is calling both _prepare_scene and _render_scene twice, once for each eye.
When testing on the device, all 3d objects (controllers, table) are rendered at the proper depth with the correct stereo separation.

@Schroedi looking at the comments I would assume so.

Can you clarify how stereo rendering is turned off?
Before the change, the logic would call _prepare_scene once with the combined frustum, and call _render_scene twice, once for each eye.
With the disabling change, the logic is calling both _prepare_scene and _render_scene twice, once for each eye.
When testing on the device, all 3d objects (controllers, table) are rendered at the proper depth with the correct stereo separation.

Thats what I get for reading it on a mobile phone :)
Yes you are right, it will do prepare and render twice, however it doesn't make sense that it would render differently when you prepare once with the combined frustum and then have problems calling render_scene. It is render_scene that will define the projection matrix and render out the scene. all prepare scene does is the culling and shadow updates and stuff like that.
So while the culling may fail and not cull the right objects, render_scene should still render with the same stereoscopic parameters.  There must be something else going on here.
Keep in mind, this is working fine for every other headset so if it fails, it fails because we're not returning the correct matrices

One thing that may throw things off is that it is expecting the camera left/right offset for each eye to be returned in get_transform_for_eye not mixed in with the projection matrix returned by get_projection_for_eye

Hi,
I think I found the issue with the clipping:
In OvrMobileSession::fill_projection_for_eye the oculus provided projection matrix is directly used and the given function parameters z_near and z_far are ignored.
This is fine in theory because the function VisualServerScene::render_camera in godot takes the projection matrix and computes near and far values from it for the merged frustum computation (the code is in camera_matrix.cpp CameraMatrix::get_z_far())
The problem now is that oculus uses an +infinite far plane for their projection (the reason is documented here: https://developer.oculus.com/reference/mobile/1.24/vr_api_helpers_8h/ in the documentation of the function static ovrMatrix4f ovrMatrix4f_CreateProjection: "...except for things right up against the near plane, it always provides better precision...")
But in this case the godot z far computation code in CameraMatrix::get_z_far() returns 0.0 instead of +inf.
One solution to this problem would be to add to visual_server_scene.cpp in the function VisualServerScene::render_camera
directly after the near and far computation (line 1799) a check and set it to +infinity like this for example
if (z_far <= z_near) z_far = 1.0f/0.0f;
this then produces correct results in the following computations.
An alternative is to respect the given near and far values in vrMobileSession::fill_projection_for_eye
and overwrite them in the projection matrix returned by doing
matrix.M[2][2] = -(z_far + z_near) / (z_far - z_near);
matrix.M[2][3] = -(2.0f * z_far * z_near) / (z_far - z_near);

at line 198 after getting the ovrMatrix4f matrix from the tracking state.

Indeed, having an invalid far plane value would screw up the entire calculation.
Interesting choice from Oculus because the reason you don't have an infinite far plane is because the near<->far range directly impacts the precision of your Z-buffer. With an infinite far plane the depth buffer would fail to work properly.
So they're obviously doing more here.

(btw, if we check for z_far == 0 in the code that merges the frustums and change it to something arbitrary like 4096.0 (max value for the far plane property in Godot) in theory the logic will work.

The logic seems work even when setting z_far to +infinity if it was 0 but I did not check the result in detail.
Oculus references this paper to explain the choice https://www.cs.cornell.edu/~paulu/tightening.pdf and writes in there documentation
https://developer.oculus.com/reference/mobile/1.24/vr_api_helpers_8h/
"""
The projection matrix transforms -Z=forward, +Y=up, +X=right to the appropriate clip space for the graphics API. The far plane is placed at infinity if farZ <= nearZ. An infinite projection matrix is preferred for rasterization because, except for things right up against the near plane, it always provides better precision: "Tightening the Precision of Perspective Rendering" Paul Upchurch, Mathieu Desbrun Journal of Graphics Tools, Volume 16, Issue 1, 2012
"""

Fixed by PR #36


Nested arguments not parsing correctly

I've got a query with a nested argument but the object keys are getting parsed as strings.
{
  query: {
    allAccounts: {
      __args: {
        or: [
          {accountid: {equalTo: "somestring1"}},
          {accountid: {equalTo: "somestring2"}},
          {accountid: {equalTo: "somestring3"}}
        ]
      },
      nodes: {
        accountid: true,
        balance: true
      }
    }
  }
}

query {
    allAccounts (or: [{"accountid":{"equalTo":"somestring1"}},{"accountid":{"equalTo":"somestring2"}},{"accountid":{"equalTo":"somestring3"}}]) {
        nodes {
            accountid
            balance
        }
    }
}

For this to be a valid query accountid and equalTo would need to not be returned as strings

Ah yeah, looks like it's not handling arrays as you'd expect. Will have a look tonight :)

Should be fixed in version 1.1.0 just published :)


How to listen to events

Hello,
We have a node based socketio server, which is broadcasting an event, the channel name is 'pathfinder.49ee9c7a-d8cc-4374-a4ca-931b1275d621' and the even name is 'App\Events\Pathfinder\PathfinderGreet'.  How would I then listen to this using your client?
Thanks.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/38003761-how-to-listen-to-events?utm_campaign=plugin&utm_content=tracker%2F388415&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F388415&utm_medium=issues&utm_source=github).

Hello,
I would like to recover the data for you to use but after I had a problem

You are welcome to schedule a call if you still need support on this issue: https://calendly.com/crosscompute/30min


Sorry need your help

I found your github profile by accident. I need your help. if you have time is it possible to write a readable powershell script where i can monitor windows update soap traffic

You can use something like this https://gist.github.com/jborean93/6c1f1b3130f2675f1618da56633eb1fa to generate the SSLKEYLOGFILE that Wireshark can use to decrypt TLS traffic to capture what you need. Anything else is dependent on you knowing what API to hook and implementing it.

Thank you for help

I try but didn't work. Wireshark very complicated. Is there a way to do it with fiddler? actually i can do this with fiddler but the process fails because windows update gives ssl certificate error. I can't see all the traffic just until one point

Ok thank you I posted it here because I thought you might be able to help.


[Foundation] Update UndoManager registerUndo to adopt new naming rules

What's in this pull request?

This updates the overlay naming of UndoManager registerUndo to adopt the new naming rules for consistent naming
Resolved bug number: (SR-)

rdar://problem/26500390 registerUndoWithTarget in overlay not updated for new API names

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

Addresses rdar://problem/26500390 registerUndoWithTarget in overlay not updated for new API names

@swift-ci Please test

@swift-ci please test OS X


console log strategy

Log modes:

User log
Verbose log: include (User log)
Dev log: include (User log, Verbose log)

ok


Rename html smuggling "location" file name rule to be consistent with other html smuggling rules

If the user doesn't have this rule installed, it will show up as 1 new rule added to the feed
If the user has this rule installed, it will show up as 1 rule deleted and 1 added

This is the URI that I see in the platform:

https://platform/feeds/0214465b-004a-4952-8469-4bd705c52f8e/rules/f541a262-97b6-5b87-aa65-c3210b459833

If you add id: 'f541a262-97b6-5b87-aa65-c3210b459833' to the YAML, it'll recognize it as the same rule before and after the change.


[Bug] Heatmap visualmap filtering not working - CoordSys is undefined

Version
5.3.1
Link to Minimal Reproduction
No response
Steps to Reproduce
I'm following the Cartesian Heatmap chart example (https://echarts.apache.org/examples/en/editor.html?c=heatmap-cartesian). But the filtering with the visualMap doesn't work properly. Once I move the cursors to select the desired range, it throws this error.
Uncaught TypeError: Cannot read properties of undefined (reading 'type')
    at HeatmapView.render (HeatmapView.js?d7f5:137)
    at Task.progress (Chart.js?fc49:233)
    at Task._doProgress (task.js?d463:187)
    at Task.perform (task.js?d463:153)
    at eval (echarts.js?f2a6:1821)
    at GlobalModel.eval (Global.js?05a0:654)
    at Array.forEach (<anonymous>)
    at each (util.js?d5c9:205)
    at GlobalModel.eachSeries (Global.js?05a0:651)
    at renderSeries (echarts.js?f2a6:1809)

The source code line that breaks is the 137 in HeatmapVIew.js file.
    this._progressiveEls = null;
    this.group.removeAll();
    var coordSys = seriesModel.coordinateSystem;

   // The following check breaks because coordSys is undefined
    if (coordSys.type === 'cartesian2d' || coordSys.type === 'calendar') {
      this._renderOnCartesianAndCalendar(seriesModel, api, 0, seriesModel.getData().count());
    } else if (isGeoCoordSys(coordSys)) {
      this._renderOnGeo(coordSys, seriesModel, visualMapOfThisSeries, api);
    }

I've debugged the code, and when the component renders(I'm using Vue@3) on mounted, coordSys.type is cartesian2d. But once I move the visualmap cursor, that variable becomes undefined.

Current Behavior
Everything works as expected, until I use the visualmap to filter values. The chart breaks and doesn't display values anymore.
Expected Behavior
It should apply a filter and display the values that falls within the selected range from the visualmap to the heatmap chart.
Environment
- OS: Linux
- Framework: Vue@3

Any additional comments?
No response

Check if you are using ref or reactive to make echarts instance reactive. If so, try to use shallowRef or shallowReactive or a common variable instead.

我把const chart =ref(null)
改为
let chart = null就行了欸

I'm using a common variable. In vue data() I have a heatmap_chart variable initialized to null, and on component mounted() event  I create the chart and assign it to the variable. It doesn't work :woman_shrugging:
mounted() {
   let heatmapDom = document.getElementById('chart');
   this.heatmap_chart = echarts.init(heatmapDom);

   this.heatmap.setOption(data);
}

In vue data() I have a heatmap_chart variable initialized to null,

@JiahuiChen99 This is not expected. Just remove the heatmap_chart variable from vue data block.

In vue data() I have a heatmap_chart variable initialized to null,

@JiahuiChen99 This is not expected. Just remove the heatmap_chart variable from vue data block.

Declaring the chart in the mounted scope, like this, seems to work.
mounted(){
    let heatmap_chart = echarts.init();
}

But, how am I supposed to reference the chart later in the beforeUnmount block, so that I can dispose it? And what's the reason why, what I did before the doesn't work?

Don't worry. You may still use this.heatmap_chart = echarts.init() to mount the echarts instance to this. Just please don't put it in data block to avoid making echarts instance reactive.
data() {
    return {
        // DON'T declare here!
        // heatmap_chart: null
    }
},
mounted() {
    // Mount this echarts instance on current vue instance using `this`
    this.heatmap_chart = echarts.init();
},
beforeUnmount() {
    // destroy echarts instance
    this.heatmap_chart && this.heatmap_chart.dispose();
}

Thank you @plainheart for the explanation! It does work now :100:

BTW, please refer to the Vue documentation for the reason.

Some values simply should not be made reactive, for example a complex 3rd party class instance, or a Vue component object.


Show instructions on all tabs.

Fixed by https://github.com/damellis/sensors/commit/2f3bb496983ea99fe1a2d1227e3b128213a6041e and https://github.com/damellis/sensors/commit/877a35b7484f3fbb76444b12a286d0dfec79336e


Add Shortnames Option

Enhancement for issue #4

I have some questions about the test, but otherwise this looks :+1:. The addition of the template looks great.


Feature/modify end point

Change api endpoint because EAN team said api.ean.com already released and api.eancdn.com will close soon.

The tests are failing for this PR. @nakamaru Can you please fix those So we can close this...


Fix go mod download error

When I run go mod download:
verifying<EMAIL_ADDRESS>checksum mismatch
        downloaded: h1:oKBSVatmEathy76UXTbTI32uXG5PYOjE8vG54lzsVgA=
        go.sum:     h1:235kOdqFQ6Io+hsI/SvHfoYDHkSNEmb7cu6TG24O93Y=

SECURITY ERROR
This download does NOT match an earlier download recorded in go.sum.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

go version go1.15.2 linux/amd64
Signed-off-by: zouyu<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

@Hellcatlk thanks for the PR. We use vendoring mode. I don't think that you have to extra download the dependencies. If you want to update this library, you will also have to run make deps-udpate to update the vendor dependencies.